tsvector column on model combined with association searches?

Hi folks. I have a tsvector column on my users table with an upsert trigger that combines several searchable elements (name, email, description). In my application, a user belongs to an organization, so I'd also like to search users by organization name. Has anyone successfully handled association searching through a tsvector column, or managed to use an existing tsvector column along with associated searches? Cheers.

Would it be possible to use something like this with gem globalize?


[TIR] Fix compute inline for dynamic loop and nontrivial store indices

Hi, the change wants to fix https://github.com/apache/tvm/issues/14364 where our compute-inline validity check failed.

Do not depend on undefined var counting, since dynamic loop extents may also be counted. The fast path just check the store indices should be the same with the block iter vars, which would cover the most cases.
Add fallback implementation for store indices which are bijective wrt block iter vars, which is already utilized in reverse-compute-inline implementation.
The change do not check the inlined producer's region cover the consumer. Refer to unittest test_compute_inline_out_of_bound_consumer, the previous behavior is not changed in this MR.

Hi @wrongtest-intellif , this PR looks good to me, we can still use GetNumUndefinedNonpointerVars to check if we can avoid visiting buffer shape/strides in the UndefinedVars function (I have created another PR #https://github.com/apache/tvm/pull/14502 for that), what do you think?

Hi @wrongtest-intellif , this PR looks good to me, we can still use GetNumUndefinedNonpointerVars to check if we can avoid visiting buffer shape/strides in the UndefinedVars function (I have created another PR ##14502 for that), what do you think?

Thank you for the remind! It seems great to rely on #14502 to solve the dynamic buffer issue and then I could go on for the
patch when the write indices are not in the ordered form.

Hi @wrongtest-intellif , any updates on this?

Hi @wrongtest-intellif , any updates on this?

Sorry to be late, I would turn back to it soon.


Enhancement: Allow to update the clientSideComponentId of an application customizer that's deployed tenant-wide

Allow to update the clientSideComponentId of an application customizer that's deployed tenant-wide.
With spo tenant applicationcustomizer set we've introduced a command that can currently update the title and the properties of an application customizer. However: we forgot an option for updating the clientSideComponentId
Original issue: https://github.com/pnp/cli-microsoft365/issues/4266
Docs: https://pnp.github.io/cli-microsoft365/cmd/spo/tenant/tenant-applicationcustomizer-set
Options

Option
Description

--newClientSideComponentId [newClientSideComponentId]
The new Client Side Component Id (GUID) of the Application Customizer

Can I work on this?

Sure @nanddeepn, thanks again!


check-in remote shuffle based on DAOS Object API

reorganized the existing remote-shuffle project to include two type of remote shuffle plugins as submodules. Changes are,

make remote-shuffle as parent POM project.
move the existing remote-shuffle project to submodule, shuffle-hadoop, which bases on Hadoop Filesystem.
add a new remote shuffle plugin in another submodule, shuffle-daos, which bases on DAOS Object API and takes DAOS as remote shuffle data store.

@carsonwang I merged the PR first. You can submit issue later if you find anything wrong. I'll submit more PR here.


testnames with special characters are skipped

testnames with special characters like + are skipped
i think we need to escape those because they are interpreted as regex

fixed in 0.4.3


Global CSS Stylesheet Solution

We should define a global CSS stylesheet where components can import from.

Use slot properly to solve this issue. I think it's what slot is exactly for.

Clear up current code for this.


Chore/masthead v2 dev manual merge

Trying to get this branch fully up-to-date so the automerge will work.

recheck


Gitleaks alibaba-access-key-id rule matches on word boundaries

The rule, as it was written, matched substrings within longer strings.
This caused a false-positive match against a package hash in a go.sum file.
This PR tightens the rule to only match on word boundaries.
This will alert on:
var x = "LTAI563KZ5YKYRhT3MFKZMbjx"

But will not match:
github.com/go-logr/stdr v1.2.2 h1:hSWxHoqTgW2S2qGc0LTAI563KZ5YKYRhT3MFKZMbjag=

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 8.981%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
222

Relevant Lines:
2472

💛  - Coveralls


Address dirname suggestion in internal codegen script

Summary

Changes import.meta.dir to
import path from "node:path";
import { fileURLToPath } from "node:url";

path.dirname(fileURLToPath(import.meta.url))

In case we want to switch in the future from Bun to something like tsx or ts-node (which are Node.js based and do not support Bun APIs) to run the internal codegen script for tailwind class sorting.

Test Plan

Run just gen-tw.

Thank you @DaniGuardiola! 🙏


Check for feature parity with Python netifaces library

Check if we expose functionality equivalent to https://github.com/al45tair/netifaces

Other similar libraries:
https://github.com/ifaddr/ifaddr
https://github.com/EstebanBorai/network-interface


Added test for when limit_time itself changes.

Codecov Report

Merging #14 (f16cf5c) into main (06a04ff) will increase coverage by 0.46%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #14      +/-   ##
==========================================
+ Coverage   95.67%   96.14%   +0.46%
==========================================
  Files          17       17
  Lines        1064     1064
  Branches      161      161
==========================================
+ Hits         1018     1023       +5
+ Misses         37       34       -3
+ Partials        9        7       -2

Impacted Files
Coverage Δ

parent_control/my_router/views.py
98.94% <0.00%> (+1.05%)
:arrow_up:

Continue to review full report at Codecov.

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


perf(linters): Greatly improve the npm run lint task speed

The npm run lint task now takes about ~3s on my machine (was
taking XXX).

For scss, I also cleaned up a bit the list of ignore patterns (as we
now use Jekyll 3, some files do not have the front-matter anymore and
so we do not need to exclude them).

@vvo You were right that it wasn't checking at the root. I updated it

Thx :)


PowerMock is of no use for code coverage

From<EMAIL_ADDRESS>on March 21, 2014 08:12:16
What steps will reproduce the problem? 1.Write a succesful testcase using PowerMock
2.Run maven command for cobertura report (or Sonar even)
3.Open cobertura report to see if code is covered What is the expected output? What do you see instead? Expected is - the code in the report must be shown as covered. But it is observed that the code remains uncovered. What version of the product are you using? On what operating system? easymock 3.0
powermock-module-junit4 1.4.12
powermock-api-easymock 1.4.12
Windows 7 for Cobertura and Linux for Sonar Please provide any additional information below. When the code coverage remains unchanged it is of no use using powermock. Would be delighted to see the resolution.
Original issue: http://code.google.com/p/powermock/issues/detail?id=489

From<EMAIL_ADDRESS>on March 26, 2014 04:08:51
What version of cobertura are you using?

From<EMAIL_ADDRESS>on August 25, 2014 21:40:21
I have the same problem.
Do we know which versions does get the report if powermock is used ?


NPI Lookup Tool

Problem Statement
As the ReportStream team, we should create a tool that allows us to access the NPI facility data maintained by CMS so that we can lookup facility information when senders' data is incomplete.
Notes

According to Mike, there is an API for this NPI data. https://npiregistry.cms.hhs.gov/api/demo?version=2.1
Refer to the following document from Rick Hood for background info: https://docs.google.com/document/d/12t5Dsld7ecTE-MMCDeJ33af9Zcub_XL0-wmIYSD8XSM/edit

Placeholder.  On hold for now.


64Bit Download

#650

Thanks, this is not needed at the moment, as the x86 architecture is still available on their servers. We are waiting for any news on this matter, and as soon as something changes, I will take action immediately.


Release AzConfig Track 2 Library Preview 5 for Python

API Review

[x] Proposal ready
[ ] Design approved

Features

[ ] Integrate authentication from azure-identity
[ ] React to swagger changes from the service team
[ ] React to arch board feedback

Docs

[ ] Samples
[ ] Readme update
[ ] Change Log

Preview 5 is released


MFA Support

This is a 🙋♂ feature or enhancement.
This is a 🧐 documentation change.

[ ] I've added tests (if it's a bug, feature or enhancement)
[x] I've adjusted the documentation (if it's a feature or enhancement)
[ ] The test suite passes (run bundle exec rspec to verify this)

Summary
Add MFA support for standard IAM users.
Context

https://github.com/tongueroo/jets/issues/213
https://github.com/tongueroo/jets/pull/388

Version Changes
Patch

Release in v2.3.5


The last moment for moment

Mandatory moment of silence for moment.js before reviewing this PR: https://www.youtube.com/watch?v=Fw3SHWroQdw
I chose to replace moment.js with day.js since the set of API we use in the code base has the same format in dayjs.
A comparison for the functionality is found here: https://github.com/you-dont-need/You-Dont-Need-Momentjs#feature-parity
we can't simply change the import from import moment from 'moment' to import dayjs from 'dayjs'. Some APIs require extra plugins since day.js opted for keeping the package as lightweight as possible. The list of functions that require plugin is found in the comparison document above and in the day.js docs https://day.js.org/docs/en/plugin/plugin

You might wonder why we still have moment in `yarn.lock. Here's why:
yarn why v1.22.10
[1/4] 🤔  Why do we have the module "moment"...?
[2/4] 🚚  Initialising dependency graph...
[3/4] 🔍  Finding dependency...
[4/4] 🚡  Calculating file sizes...
=> Found<EMAIL_ADDRESS>info Reasons this module exists
   - "gatsby" depends on it
   - Hoisted from "gatsby#moment"
info Disk size without dependencies: "5.14MB"
info Disk size with unique dependencies: "5.14MB"
info Disk size with transitive dependencies: "5.14MB"
info Number of shared dependencies: 0
✨  Done in 1.65s.

You might wonder why we still have moment in `yarn.lock. Here's why:
yarn why v1.22.10
[1/4] 🤔  Why do we have the module "moment"...?
[2/4] 🚚  Initialising dependency graph...
[3/4] 🔍  Finding dependency...
[4/4] 🚡  Calculating file sizes...
=> Found<EMAIL_ADDRESS>info Reasons this module exists
   - "gatsby" depends on it
   - Hoisted from "gatsby#moment"
info Disk size without dependencies: "5.14MB"
info Disk size with unique dependencies: "5.14MB"
info Disk size with transitive dependencies: "5.14MB"
info Number of shared dependencies: 0
✨  Done in 1.65s.


now interpreted as UTC

This also affects creating "today" events. If the date is tomorrow in UTC, the event will be created for tomorrow 😕

Fixed in 0868099bdfbbf6ea0c21e6ab9da328eaca7db50d

Fixed in e3ac0f15481487b4976674db503a1aa4bd0344d9


Add contribution guide and virtualenv set up in documentation

[X] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[X] I've read and agree to the Code of Conduct.
[X] I've been mindful about doing atomic commits, adding documentation to my changes, not refactoring too much.
[X] I've a descriptive title and added any useful information for the reviewer. Where appropriate, I've attached a screenshot and/or screencast (gif preferably).
[ ] I've written tests to cover the new code and functionality included in this PR.
[X] I've read, agree to, and signed the Contributor License Agreement (CLA).

PR Summary

Included instructions on installing virtualenv, creating a new env and starting the env before building the docs to be more explicit.
Added a link to the slack contribution guide in the contributions section of the documentation.

Related Issues

None

Test strategy

No tests needed

Coverage remained the same at 61.569% when pulling da84c25e3028856904d351360c25caf10e5c0152 on karishay:master into 875839bb314a9da2091d36e3476480e0f10dfa67 on slackhq:master.

Coverage remained the same at 61.569% when pulling 918d82c267c9eb45bf0335dab37f3126a4c2907a on karishay:master into 875839bb314a9da2091d36e3476480e0f10dfa67 on slackhq:master.

Coverage remained the same at 61.569% when pulling 918d82c267c9eb45bf0335dab37f3126a4c2907a on karishay:master into 875839bb314a9da2091d36e3476480e0f10dfa67 on slackhq:master.


Copy Paste not working when editing XML files

Description
When copy pasting from an xml file, it pastes only the first letter of the copy operation.
For example, if you select copy text "Hello World" then it will paste only "H".
Steps to Reproduce

Open the attached example xml file (See ZIP download).
Copy a piece of text.
Paste back into xml file or notepad window.
Example - Copy.zip

Expected behavior:
The full text should be copied.
Actual behavior:
Only the first letter is copied.
Versions
Atom 1.7.4

Cannot reproduce with Atom 1.7.4 on OSX 10.11. @Kungfoowiz: Do you observe the same behaviour when running atom in safe mode? atom --safe

Thank you for your issue!
We haven't gotten a response to our questions in our comment https://github.com/atom/atom/issues/11881#issuecomment-223233767. With only the information that is currently in the issue, we don't have enough information to take action. I'm going to close this but don't hesitate to reach out if you have or find the answers we need, we'll be happy to reopen the issue.

Sorry, I got busy and did not use Atom for some time, lol. Now that I am back using it again, I tried testing this in Atom --safe mode and noticed something else. It appears to happen only in Mixed mode (see screenshot below). Mixed mode regarding CR / LF characters, you can see this on the file details on the bottom right in the screenshot.
You can also see when I copied the text from the Mixed mode XML file that it only copied the first character. I used the exact same example XML file as I originally attached to this issue.
And then I used a sample XML file using CRLF mode (Download from here: https://www.w3schools.com/xml/simple.xml). I was able to copy the text fine with this example, but not with the above example.
I wonder also, the encoding type set in the above XML is UTF-16, yet Atom reports it as UTF-8. Maybe it is having an issue reading the characters because of this?
It could also be the Mixed mode?

I wonder also, the encoding type set in the above XML is UTF-16, yet Atom reports it as UTF-8.

Does everything work properly if you set the encoding to match the encoding of the file?

Nope, I tried setting it in N++ (USC 2 little endian and big endian which I think is UTF-16 roughly) and Atom still reports it as UTF-8.
When copying from the opened file in Atom, it still only copies a single letter, rather than what was selected.
As a test, and also to get familiar with packages in Atom, I quickly wrote something to check the file for NULLs. I think what may be happening is when Atom hits a NULL it stops copying from that point and ignores the other characters.

Nope, I tried setting it in N++ (USC 2 little endian and big endian which I think is UTF-16 roughly) and Atom still reports it as UTF-8.

Atom doesn't autodetect encoding well, this is a known issue that is being tracked elsewhere. What I was asking for you to do is if you know that the encoding is UTF-16LE, do things work properly when you tell Atom that the file is UTF-16LE?

The xml file was originally exported from Windows Task Scheduler, so I suspect that the encoding is not quite correct anyway.
I set the file to UTF-16LE in N++ but Atom still reports it as Mixed UTF-8 XML.

However, reading the file back into N++ it reports it as UTF-16LE, yet Atom does not.

Yes, like mentioned previously, Atom doesn't do well with detecting the encoding.  You need to set it manually using Ctrl+Shift+U.  After doing that, does copy/paste start working?

Sure, I understand the encoding detection is not properly working at the moment.
As you suggested, I set the encoding manually in Atom and copy/paste still only copies a single letter.

I thought to try with another set of programs I am using, so it is tested with something completely different.
It saves the file as utf-16 encoding and when I open in atom it only copies the first character from the selection, as if the null characters are interrupting it. The file is identified by atom as Mixed UTF-8 XML.

Please lets limit reproduction steps to Atom doing things incorrectly when the encoding is set correctly in Atom. Atom doing weird things when the encoding is incorrect is almost guaranteed to be a wontfix.


What does this actually do?

As far as I can tell, setting a Pin's type to these different values makes no difference in the Pin's behavior. Is that wrong?

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

ID: e444e0b5-20ee-68a4-5f6f-6035bfae4c5f
Version Independent ID: 01afee4c-0f03-1ace-ab5d-3fdf49f7b12e
Content: PinType Enum (Xamarin.Forms.Maps)
Content Source: docs/Xamarin.Forms.Maps/PinType.xml
Product: xamarin
Technology: xamarin-forms
GitHub Login: @dotnet-bot
Microsoft Alias: dotnetcontent

Looking at the code, it doesn't actually seem to do anything... Not sure why since it has been there since the beginning and has no committer/comments.
I'll try poke around some more and figure it out.

Looking around, it never did anything inside forms. It could be that it was useful for 3rd party libraries with custom renderers?

Microsoft support for Xamarin will end on May 1, 2024 for all Xamarin SDKs including Xamarin.Forms. In preparation for this, all issues and PRs in this repository are being closed.
.NET MAUI is the replacement for Xamarin.Forms, and its API docs are built from /// in the dotnet/maui repository. Therefore, any issues with the .NET MAUI API docs should be opened in that repository.


plan9/client: Add ReadByte method to implement ByteReader interface

plumb.Recv expects an io.ByteReader, but the client.Fid type returned
from plumb.Open doesn't implement the ReadByte method, making it impossible
to receive plumber messages using 9fans.net/go/plumb.
This adds a small wrapper around Read() to read a single byte and return it,
thereby fulfilling the required interface.

It does fulfill the interface but it may not work very well, because it results in 1-byte read RPCs.
The usual way to convert a Reader r to a ByteReader is to use bufio.NewReader(r).


TestStmtPanic

Codecov Report

Merging #73 (d2b41e1) into main (3968146) will increase coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #73      +/-   ##
==========================================
+ Coverage   93.48%   93.54%   +0.05%
==========================================
  Files          13       13
  Lines        5390     5390
==========================================
+ Hits         5039     5042       +3
+ Misses        269      267       -2
+ Partials       82       81       -1

Impacted Files
Coverage Δ

stmt.go
94.44% <0.00%> (+0.66%)
:arrow_up:

Continue to review full report at Codecov.

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


enable babel-loader cache

This speeds up repeated webpack builds by around 20%. It will use the default cache directory node_modules/.cache/babel-loader.

Rewrote cache invalidation to use cacheIdentifier option and not depend on make any more.


ScrollTop input to grid fails if the scrollHeight of the Div to be virtualised exceeds 1.5 million. Happens on all browsers except chrome

Reporting a Bug?
Please include either a failing unit test or a simple repro. You can start by forking one of these:

Code Sandbox:https://codesandbox.io/s/rl5jv04k3q

#1076
I was unable to repro on safari/firefox/chrome. Manually setting scrollTop to 15 million causes no errors

In sandbox that i created , we should scroll to the top of the div . Instead we scroll beyond that. Please try again

https://codesandbox.io/s/p56k0l9x90 I've reworked the sandbox to focus on the issue.
If set rowHeight to 1000 and rowCount to 1600, scrolling using number input will cause scrolling pass beyond head of div in firefox, but in chrome it will scroll to exactly head of div.
Chrome also has the same issue but can tolerance to much greater number. For me, it's rowHeight 1999, rowCount 8393 when it starts acting weirdly
Browser

Chromium Version 69.0.3497.81.
Firefox 62.0.3

OS: Kubuntu 18.04

I believe the issue is because of the offset scaling issue. From here https://codesandbox.io/s/rl5jv04k3q you can see, the 90th Div start offset is not accurate any more. But if you try the 10th Div, the scrolling will be right.
Grid has 2 methods, scrollToPosition &  scrollToCell. The scrollToPosition will directly use the provided scrollTop offset to adjust the scroll bar position. But the scrollToCell has an extra step to calculate the row offset.
The issue is caused by this extra calculation. Because Grid is using the "ScalingCellSizeAndPositionManager" to calculate the offset. When the scroll height is greater than the 1.5M pixels, it will scale the actual pixel offset as the percentage of the current Grid height (like the pagination).
In another words, client can't directly use the actual element pixels as the Grid scrollTop parameter.

To addo, ScalingCellSizeAndPositionManager has methods to convert the pixel offset to the "safe" offset and vice verse, _safeOffsetToOffset & _offsetToSafeOffset.
These are should be exported for client side.


[OCaml] Pressing RET inside a comment adds **

[X] I have searched the FAQ and the documentation for solutions.
[X] I have searched the issue tracker for similar issues (the closed issues too).
[X] I have searched the Discourse for any errors and solutions.
[X] I can reproduce my issue on the latest commit of Doom Emacs.
[X] I have read How to Debug Issues and found no solution.

What did you expect to happen?
The doc convention in OCaml seems to be "Start the first line with (* (for comments) or (** (for documentation), next lines are only indented, the last line ends with *)".
However when in this situation (| represents my cursor in Insert mode):
(** some text
    and another line|
    and the last one *)
val a : unit

pressing RET should give this:
(** some text
    and another line
    |
    and the last one *)
val a : unit

What actually happened?
It actually gives this:
(** some text
    and another line
 ** |
    and the last one *)
val a : unit

Describe your attempts to resolve the issue
I found https://github.com/hlissner/doom-emacs/issues/2081 which looks similar (but for Coq), however the fix mentioned does not work:
;; config.el
(setq-hook! 'tuareg-mode-hook comment-line-break-function nil)

I notice that even after a restart, the value of comment-line-break-function remains +ocaml/comment-indent-new-line. I guess this is "expected" as https://github.com/hlissner/doom-emacs/blob/develop/modules/lang/ocaml/config.el#L24 sets the value (and it probably runs after my conf).
I also searched for any conf flag related, but found none.
I did search a bit in Doom Emacs OCaml conf history, and I believe there was a past attempt at providing such a conf flag +ocaml-prefix-comments-with-asterisk but this has apparently been rolled back since.
Steps to reproduce

Open an ml or mli file
Create a comment
Inside the comment, press RET

System Information
https://pastebin.com/UH3Vy6s7

Hi,
Sorry to bump this issue: would you have at least an idea on how to circumvent the problem? I'm ok with a hack/ugly solution for now. This is driving me a bit crazy, while this is otherwise a pleasant Emacs distribution :sweat_smile:

Since my latest update, I don't have the problem anymore.

At one point I downgraded to stop using the native compiler and this issue was fixed, but later upgraded to the native compiler again and it returned.
I'm not sure what that means or how to fix it, just wanted to +1 the OP and suggest that it might not be as simple as updating.

This should work around it, until I can cook up a better solution:
(after! tuareg
  (setq-hook! 'tuareg-mode-hook comment-line-break-function nil))


Added BRLTTY package

@bramd Did you test your service?

@bramd Could you also update our release notes for the next stable version? I think the right category would be Following new services were added since the last release:. You can see some examples in older release files

@Geerds: Yes, the service works. I might add more configuration options in the future, for now it uses the default BRLTTY configuration which does autodetection and will work for most devices connected via USB.
The service has been added to the release notes.

I can't test BRLTTY myself because I don't own such a device but it builds successfully on my machine. So let's merge it. Thank you very much for your work!


MAINT: Add coordinate/satellite inheritance on 0.3.x

The current structure of the Coordinate and Satellite classes are clumsy to instantiate with the need to initialize the Time and/or Coordinate classes beforehand. Appealing to inheritance, the input arguments were modified to include simpler data types and reduce difficulty in Time access from the Coordinate class.

Better inheritance; looks good on a high level. Didn't have a close look for bugs.


feat(component): create navigation component

BRIDGE-145
General summary

Create a navigation component.
This component can include buttons, links and subsections (opening on click)
It can be displayed vertically and horizontally.
When displayed horizontally, the subsections are displayed as popovers.
Each item can display an icon and text
Screenshots

We'll reconsider the Navigation.Link/Button and create a Navigation.Item instead with href required by default. Then customizable with as (ex: as=button or as=CustomLink) which should copy props from this component

Rebasing & fixing merge conflicts (on design tokens and CHANGELOG)

Test

Navigation vertical

Navigation item link ok ✔ (DOM/a11y semantic)
Navigation item as=button ok ✔ (DOM/a11y semantic)
Navigation section (disclosure button) ok ✔
Navigation item label truncated on reduced space & tooltip activated ✔
Dark theme ✔

Navigation horizontal

Navigation item link ok ✔ (identical as in vertical mode)
Navigation item as=button ok ✔(identical as in vertical mode)
Navigation section opening as dropdown and sub sections as disclosure panel ✔
Navigation item label truncated on reduced space & tooltip activated ✔
Dark theme (+light theme forced inside dropdown sub navigation) ✔


test: adds playwright e2e tests

Hi @vault-developer
Initial e2e playwright setup is ready for review. Random changes are due to prettier command.
Additionally I can also add github action for CI testing.

Hey @malcodeman  thank you for the PR 👍
I believe it's a good starting point for our e2e tests, we can add more / extend existing ones in the future.


Component for CosmosDB change feed

The case of "tailing" the database, subscribing to it's "change feed" has a lot of possibilities.
Combining a state store and tailing would open up many scenarios.
Since the building blocks for supporting this scenario might be found in bindings or pubsub, it's a matter of implementation.
There is a lot of conceptual similarities with consuming Event Hub partitions.
I don't know what kind of support different go libraries and SDK has working with the CosmosDB change feed.
But I think this kind of components have a lot of potential if possible.

This is something that would be great to have and probably easily created as an input binding. Having this would enable the out box pattern, from there you can do a lot of interesting things

@perokvist this can be done as an input binding for CosmosDB, which recently saw a new Go-SDK. Some work needs to be done to investigate if the SDK support change feed.

@yaron2  Great news! Hope its feasible with the new SDK.
Since the change feed probably not going to support PostgreSql, maybe an input binding for LISTEN/NOTIFY could be another suggested "issue" ?
(there is only an output binding listed - https://docs.dapr.io/reference/components-reference/supported-bindings/)


Add query and chart on Dashboard

Summary of Changes
Added:

Query model
Chart model
ModeDashboardChartsExtractor that extracts chart_id and chart_url. Other metadata is on hold for Mode API response schema is not clear and we are in talk with Mode.
ModeDashboardQueriesExtractor that extracts query_id, query_name, and query_url

Tests
Unit test has been added. Integration test has been done.
Documentation
docstring
CheckList
Make sure you have checked all steps below to ensure a timely review.

[x] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[x] PR includes a summary of changes.
[x] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[x] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain docstrings that explain what it does

[x] PR passes make test

👀 @feng-tao


Delete direct item - Wrong permissions

Form
Requirements

[x] I've searched the Issues
[x] I've read the basic concepts
[x] I'm using the latest version
[ ] I've debugged my code using the DEBUG variable.

Platform

[x] I'm using Node.js version 14.17.0
[ ] I'm using electron
[ ] I'm using the browser
[ ] I'm using some other environment

Description
I'm trying to delete a direct item, but it gives me api error.
Please help me about it.
Code
ig.entity.directThread(threadID).deleteItem(itemID).then(() =>{
      return resolve(true);
})

Error and Output
IgResponseError: POST /api/v1/direct_v2/threads/{thread_id}/items/{item_id}/delete/ - 400 Bad Request; Wrong permissions

at Request.handleResponseError ({path}/node_modules/instagram-private-api/dist/core/request.js:125:16)
at Request.send ({path}/node_modules/instagram-private-api/dist/core/request.js:53:28)
at async DirectThreadRepository.deleteItem ({path}/node_modules/instagram-private-api/dist/repositories/direct-thread.repository.js:185:26)

Is this in a group chat?

Is this in a group chat?

No it's not

FIXED
Wrong permission is because it can't delete user messages, it just can delete my mesages
For deleting my messages
ig.entity.directThread(threadID).deleteItem(itemID);

ig.directThread.deleteItem(threadID, itemID);

For delete entire direct
ig.directThread.hide(threadID);


adding all_rows_overlap to fugue

Ref: https://github.com/capitalone/datacompy/issues/214
Support for all_rows_overlap with Fugue

@goodwanghan FYI. Had some time, so adding in some more functionality to the Fugue side.

 Seems like the last release of fugue is breaking some of the tests :(

Oh what? I will take a look
On Fri, Nov 10, 2023 at 09:08 Faisal @.***> wrote:

Wow nice
Well this sucks. Seems like the last release of fugue is breaking some of
the tests :(
—
Reply to this email directly, view it on GitHub
https://github.com/capitalone/datacompy/pull/244#issuecomment-1806099972,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFA5Q7ZDLOIP3LORSNNNFYTYDZNQRAVCNFSM6AAAAAA7GLTQDSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMBWGA4TSOJXGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

I can reproduce this issue, will take a look today

So after some initial investigation, it is related with pandas. Fugue 0.8.7 has a big update that is it relies more on pandas extension d types. It is an interesting case why the current datacompy logic breaks with this change. I will investigate more.

So after some initial investigation, it is related with pandas. Fugue 0.8.7 has a big update that is it relies more on pandas extension d types. It is an interesting case why the current datacompy logic breaks with this change. I will investigate more.

Ahh interesting. Thanks for helping out here and taking a peek, really appreciate it.

no problem, actually think this is a very good case, i have found an example
You know pandas will use object type for string columns, but in new pandas, it can be StringDType, and it can be an arrow type (it is a mess I know). So in this case we want to treat them as the same type? In the current logic, we think they are different. That is why we saw those failed tests.

Gotcha that makes sense. Yeah strings should in theory be the same in that example.   On Fri, Nov 10, 2023 at 1:34 PM, Han Wang @.***> wrote:
no problem, actually think this is a very good case, i have found an example
You know pandas will use object type for string columns, but in new pandas, it can be StringDType, and it can be an arrow type (it is a mess I know). So in this case we want to treat them as the same type? In the current logic, we think they are different. That is why we saw those failed tests.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

no problem, actually think this is a very good case, i have found an example
You know pandas will use object type for string columns, but in new pandas, it can be StringDType, and it can be an arrow type (it is a mess I know). So in this case we want to treat them as the same type? In the current logic, we think they are different. That is why we saw those failed tests.
To generalize, datacompy needs to consider the comparison between original type and extension dtypes, do we think they are the same?
Examples are:
object vs StringDType
int vs Int
bool vs Bool

Yup those should be considered the same type IMHO.

We should probably list the added functionality for fugue in the docs. Right now only is_match and report are mentioned.

We should probably list the added functionality for fugue in the docs. Right now only is_match and report are mentioned.

Good call! Will add this in a bit.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: fdosani:x: Faisal DosaniFaisal Dosani seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@jdawang updated now


Add Responsive Font Size (RFS) for deskop viewports

Reviewed, just a fail on the test build I'm not sure why.

Error fixed


Update static-pod.md

Fixes Issue #31394

/sig node

Page preview: https://deploy-preview-31410--kubernetes-io-main-staging.netlify.app/docs/tasks/configure-pod-container/static-pod/#behavior-of-static-pods

LGTM (and also lol at commit message @chris-short )

/lgtm

/approve
Also taking LGTM from https://github.com/kubernetes/website/pull/31410#issuecomment-1022307327


[New Model][Format]: Support the HF-version of Pixtral

The model to consider.
vLLM supports mistral's "consolidated" format for the Pixtral model found at: https://huggingface.co/mistral-community/pixtral-12b-240910
However when HF implemented Pixtral in Transformers, they use a different format leveraging the existing Llava model structure. Model example: https://huggingface.co/mistral-community/pixtral-12b
HF PR reference: https://github.com/huggingface/transformers/pull/33449
Supporting the HF version means we can produce quantized versions of the model with LLM Compressor
The closest model vllm already supports.
No response
What's your difficulty of supporting the model you want?
Easy to moderate, all operations should already be implemented inside of vLLM
Before submitting a new issue...

[X] Make sure you already searched for relevant issues, and asked the chatbot living at the bottom right corner of the documentation page, which can answer lots of frequently asked questions.

Related issues:

https://github.com/vllm-project/vllm/issues/8566

Do you have any suggested work arounds at this time?

@Reichenbachian you can use the official mistral consolidated checkpoint with vllm if you want to use pixtral.
As for supporting the HF format, we are still waiting on someone to contribute the implementation.

Hey @mgoin we have a fine tuned version that has gone through the transformers library, so using the consolidated checkpoint isn't going to work for us unfortunately. If you can point me in the right direction though, I might be able to implement it

Otherwise, we may just retrain with Llama 3.2 vision. Sorry for the OOS question, but are you aware of any similar issues there?

Thanks @Reichenbachian we definitely want to have the implementation, for similar reasons. The key part that needs to be implemneted is registering the pixtral vision tower to the  _init_vision_tower function in llava.py
Llama 3.2 Vision is supported and should be fine for that usecase, but it is less optimized since the cross-attention architecture it uses is much less common than the Llava-style most other VLMs have been using.

I have started a draft here where we can load the weights properly. It still needs more work to properly perform inference https://github.com/vllm-project/vllm/pull/9036


Cluster monitoring stuck in installing state with templateversion not found error

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):
upgrade rancher-ha from v2.1.7 to v2.2.0
Result:

Other details that may be helpful:
instance A
E0327 08:46:54.797228 7 round_trippers.go:291] CancelRequest not implemented by *rest.tokenSourceTransport
E0327 08:46:54.797331 7 streamwatcher.go:109] Unable to decode an event from the watch stream: net/http: request canceled (Client.Timeout exceeded while reading body)
E0327 08:46:54.797361 7 round_trippers.go:291] CancelRequest not implemented by *rest.tokenSourceTransport
E0327 08:46:54.797433 7 streamwatcher.go:109] Unable to decode an event from the watch stream: net/http: request canceled (Client.Timeout exceeded while reading body)
W0327 08:49:45.017305 7 reflector.go:270] github.com/rancher/norman/controller/generic_controller.go:175: watch of *v1.Endpoints ended with: too old resource version: 469462021 (469483799)
W0327 09:01:29.032225 7 reflector.go:270] github.com/rancher/norman/controller/generic_controller.go:175: watch of *v1.Endpoints ended with: too old resource version: 469529654 (469559088)
2019/03/27 09:05:02 [ERROR] Failed to construct object on output: info=InvalidFormat 422: cpu=InvalidFormat 422: count=InvalidFormat 422: strconv.ParseInt: parsing "1k": invalid syntax
E0327 09:07:15.363290 7 round_trippers.go:291] CancelRequest not implemented by *rest.tokenSourceTransport
E0327 09:07:15.363290 7 round_trippers.go:291] CancelRequest not implemented by *rest.tokenSourceTransport
E0327 09:07:15.363290 7 round_trippers.go:291] CancelRequest not implemented by *rest.tokenSourceTransport
E0327 09:07:15.363358 7 round_trippers.go:291] CancelRequest not implemented by *rest.tokenSourceTransport
E0327 09:07:15.363448 7 streamwatcher.go:109] Unable to decode an event from the watch stream: net/http: request canceled (Client.Timeout exceeded while reading body)
E0327 09:07:15.363468 7 streamwatcher.go:109] Unable to decode an event from the watch stream: net/http: request canceled (Client.Timeout exceeded while reading body)
E0327 09:07:15.363452 7 streamwatcher.go:109] Unable to decode an event from the watch stream: net/http: request canceled (Client.Timeout exceeded while reading body)
E0327 09:07:15.363452 7 streamwatcher.go:109] Unable to decode an event from the watch stream: net/http: request canceled (Client.Timeout exceeded while reading body)
E0327 09:07:15.363505 7 round_trippers.go:291] CancelRequest not implemented by *rest.tokenSourceTransport

instance B
2019/03/27 09:30:43 [ERROR] AppController p-rj7s9/cluster-monitoring [helm-controller] failed with : catalogtemplateversions.management.cattle.io "system-library-rancher-monitoring-0.0.2" not found
2019/03/27 09:30:43 [ERROR] Failed to construct object on output: info=InvalidFormat 422: cpu=InvalidFormat 422: count=InvalidFormat 422: strconv.ParseInt: parsing "1k": invalid syntax
2019/03/27 09:30:43 [ERROR] AppController p-rj7s9/cluster-monitoring [helm-controller] failed with : catalogtemplateversions.management.cattle.io "system-library-rancher-monitoring-0.0.2" not found
2019/03/27 09:30:43 [ERROR] AppController p-rj7s9/cluster-monitoring [helm-controller] failed with : catalogtemplateversions.management.cattle.io "system-library-rancher-monitoring-0.0.2" not found
2019/03/27 09:30:43 [INFO] Catalog sync done. 40 templates created, 0 templates updated, 0 templates deleted
2019/03/27 09:30:43 [ERROR] AppController p-rj7s9/cluster-monitoring [helm-controller] failed with : catalogtemplateversions.management.cattle.io "system-library-rancher-monitoring-0.0.2" not found
2019/03/27 09:30:43 [ERROR] CatalogController aliyun [catalog] failed with : failed to update templates. Multiple error occurred: [failed to create template aliyun-kafka: namespaces "cattle-global-data" not found failed to create template aliyun-ack-knative-serving: namespaces "cattle-global-data" not found failed to create template aliyun-ceph: namespaces "cattle-global-data" not found failed to create template aliyun-ack-knative-build: namespaces "cattle-global-data" not found failed to create template aliyun-ack-springcloud-turbine: namespaces "cattle-global-data" not found failed to create template aliyun-ack-tensorflow-dev: namespaces "cattle-global-data" not found failed to create template aliyun-etcd: namespaces "cattle-global-data" not found failed to create template aliyun-exposecontroller: namespaces "cattle-global-data" not found failed to create template aliyun-ack-arms-pilot: namespaces "cattle-global-data" not found failed to create template aliyun-ack-istio: namespaces "cattle-global-data" not found failed to create template aliyun-nodered: namespaces "cattle-global-data" not found failed to create template aliyun-seldon-core: namespaces "cattle-global-data" not found failed to create template aliyun-jenkins-x-platform: namespaces "cattle-global-data" not found failed to create template aliyun-mysql-broker: namespaces "cattle-global-data" not found failed to create template aliyun-mariadb-broker: namespaces "cattle-global-data" not found failed to create template aliyun-ack-springcloud-configserver: namespaces "cattle-global-data" not found failed to create template aliyun-ack-springcloud-eureka: namespaces "cattle-global-data" not found failed to create template aliyun-ack-knative-sources: namespaces "cattle-global-data" not found failed to create template aliyun-ack-openmpi: namespaces "cattle-global-data" not found failed to create template aliyun-ack-tensorflow-serving: namespaces "cattle-global-data" not found failed to create template aliyun-catalog: namespaces
"cattle-global-data" not found failed to create template aliyun-glusterfs: namespaces "cattle-global-data" not found failed to create template aliyun-ack-consul: namespaces "cattle-global-data" not found failed to create template aliyun-ack-hyperledger-fabric: namespaces "cattle-global-data" not found failed to create template aliyun-zookeeper: namespaces "cattle-global-data" not found failed to create template aliyun-mysqlha: namespaces "cattle-global-data" not found failed to create template aliyun-rabbitmq-broker: namespaces "cattle-global-data" not found failed to create template aliyun-ack-tensorflow-training: namespaces "cattle-global-data" not found failed to create template aliyun-elasticsearch: namespaces "cattle-global-data" not found failed to create template aliyun-seldon-core-crd: namespaces "cattle-global-data" not found failed to create template aliyun-spark-broker: namespaces "cattle-global-data" not found failed to create template aliyun-vault: namespaces "cattle-global-data" not found failed to create template aliyun-ack-knative-init: namespaces "cattle-global-data" not found failed to create template aliyun-ack-springcloud-zipkin: namespaces "cattle-global-data" not found failed to create template aliyun-ack-springcloud-hystrix: namespaces "cattle-global-data" not found failed to create template aliyun-ack-springcloud-zuul: namespaces "cattle-global-data" not found failed to create template aliyun-redis-ha: namespaces "cattle-global-data" not found failed to create template aliyun-spark-oss: namespaces "cattle-global-data" not found failed to create template aliyun-ack-istio-remote: namespaces "cattle-global-data" not found failed to create template aliyun-ack-knative-eventing: namespaces "cattle-global-data" not found]
2019/03/27 09:30:43 [ERROR] AppController p-rj7s9/cluster-monitoring [helm-controller] failed with : catalogtemplateversions.management.cattle.io "system-library-rancher-monitoring-0.0.2" not found
2019/03/27 09:30:43 [ERROR] AppController p-rj7s9/cluster-monitoring [helm-controller] failed with : catalogtemplateversions.management.cattle.io "system-library-rancher-monitoring-0.0.2" not found
2019/03/27 09:30:43 [ERROR] AppController p-rj7s9/cluster-monitoring [helm-controller] failed with : catalogtemplateversions.management.cattle.io "system-library-rancher-monitoring-0.0.2" not found
2019/03/27 09:30:43 [ERROR] AppController p-rj7s9/cluster-monitoring [helm-controller] failed with : catalogtemplateversions.management.cattle.io "system-library-rancher-monitoring-0.0.2" not found
2019/03/27 09:30:43 [ERROR] AppController p-rj7s9/cluster-monitoring [helm-controller] failed with : catalogtemplateversions.management.cattle.io "system-library-rancher-monitoring-0.0.2" not found
2019/03/27 09:30:44 [ERROR] AppController p-rj7s9/cluster-monitoring [helm-controller] failed with : catalogtemplateversions.management.cattle.io "system-library-rancher-monitoring-0.0.2" not found
2019/03/27 09:30:44 [ERROR] AppController p-rj7s9/cluster-monitoring [helm-controller] failed with : catalogtemplateversions.management.cattle.io "system-library-rancher-monitoring-0.0.2" not found
2019/03/27 09:30:44 [INFO] Updating global catalog aliyun

Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI):v2.2.0
Installation option (single install/HA): HA

What

Version: v2.1.7 -> v2.2.0 Upgrade (HA setup)
No steps to reproduce were provided, but I did my best to test this from a QA perspective.
For what it's worth, and it may not be related, but on v2.1.7 I constantly get errors in the logs when trying to add the aliyun catalog show in the screenshot. It will never stop refreshing. When I review logs this is happening in a loop:
[rancher-b5756ff49-bn2pw] 2019/03/27 17:56:08 [INFO] Updating catalog aliyun
[rancher-b5756ff49-bn2pw] 2019/03/27 17:56:08 [INFO] Catalog sync done. 0 templates created, 2 templates updated, 0 templates deleted
[rancher-b5756ff49-bn2pw] 2019/03/27 17:56:08 [ERROR] CatalogController aliyun [catalog] failed with : failed to update templates. Multiple error occurred: [TemplateVersion.management.cattle.io "aliyun-seldon-core-crd-0.2.4-SNAPSHOT" is invalid: metadata.name: Invalid value: "aliyun-seldon-core-crd-0.2.4-SNAPSHOT": a DNS-1123 subdomain must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is '[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*') TemplateVersion.management.cattle.io "aliyun-seldon-core-0.2.4-SNAPSHOT" is invalid: metadata.name: Invalid value: "aliyun-seldon-core-0.2.4-SNAPSHOT": a DNS-1123 subdomain must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is '[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*')]

I can still see the apps when I launch the catalog though in v2.1.7.
I upgraded my v2.1.7 HA to v2.2.0 and successfully reproduced the issue described. I see essentially the same content in logs as well. cc: @sangeethah

As a workaround, when I click the refresh button on the Launch Apps page everything now shows up.

@davidnuzik Yes,it is shows up,but i did nothing.

By the way,Cluster Monitoring and Cluster Alert stuck on installing.
The log says failed with : catalogtemplateversions.management.cattle.io "system-library-rancher-monitoring-0.0.2" not found.
Should i create an another issue?

We are looking into the issue further. We'll create an issue for that or if we need more info let you know. Thanks for reporting.

@laoshancun are you in an airgap setup by chance?

@cjellick Rancher server has full Internet access.

@laoshancun  I upgraded Rancher from 2.1.7 to 2.2.0 and enabled cluster monitoring and I cannot reproduce this, don't have these errors in my logs. Are there any other steps you took to repro?

@mrajashree can you check the code to see what might cause cattle-global-data not being created?
Noticed this error failed to create template aliyun-kafka: namespaces "cattle-global-data" not found in the original comment's log. See if that might cause a follow up catalog not found error.

@mrajashree Let me think,i have created a global dns provider and global dns entry,and then remove it.

@laoshancun From your previous comments, you're able to view all templates am I right? The only issue was the error in the logs? Can you confirm if those errors keep showing repeatedly or you saw them only initially?

I can see all templates,but cluster-monitoring still stuck with installing.

2019/04/01 01:43:04 [ERROR] AppController p-rj7s9/cluster-monitoring [helm-controller] failed with : catalogtemplateversions.management.cattle.io "system-library-rancher-monitoring-0.0.2" not found
2019/04/01 01:43:04 [ERROR] AppController p-rj7s9/cluster-monitoring [helm-controller] failed with : catalogtemplateversions.management.cattle.io "system-library-rancher-monitoring-0.0.2" not found

@laoshancun Can you exec into your rancher server container and run kubectl get catalogtemplateversions system-library-rancher-monitoring-0.0.2 -n cattle-global-data and share the output

Changing the name of this issue since the original issue no longer exits

@davidnuzik @laoshancun said the templates showed up without doing anything, and I thought you reproduced the error in logs.
I just now read your comment saying you refreshed it, could have been some UI delay since the templates do show up. I'll leave it up to you and @sangeethah to change the title or open a new issue for that

Created issue #19301 to track the apps not showing up in catalog post upgrade (until you manually refresh catalog).

@mrajashree I have upgraded rancher to v2.2.1,it was not working too.
The resource type catalogtemplateversions does not exist.
root@Laoshancun:~# kubectl get catalogtemplateversions system-library-rancher-monitoring-0.0.2 -n cattle-global-data
error: the server doesn't have a resource type "catalogtemplateversions"

@laoshancun And how about this: In this system project from your local cluster, can you go to Namespaces and see if cattle-global-data is created?
Are you on rancher users slack channel? we could debug this faster there, couldn't find you by your Github username

@mrajashree
this cluster is a managed Kubernetes imported from Alibaba Cloud, and version is v1.11.5.

There is no cattle-global-data in system project.
The Rancher server is upgrade from v2.1.7,not from a alpha or rc version.
The kubeconfig file is download from rancher server.
I have created an account on slack with name laoshancun,but cannot find any channel with name rancher.

@laoshancun I think you're not checking the local cluster's system project for the global namespace.
Also I'm not sure I follow what you said about the slack account, but you can join this users channel https://rancher-users.slack.com/

@mrajashree
There is only one cluster on rancher server,but cluster name is not local.Is there any problem if the cluster name is not local?
The slack workspace require administrator's invatation.

@laoshancun To confirm again, this is an HA setup right? had you disabled the local cluster during installation by passing --set addLocal="false" to the helm install options?
Also I'm not sure why you're having problems joining slack users channel, that link should be enough, but can you provide your email address so we can send you an invite?

<EMAIL_ADDRESS>

@mrajashree
Rancher server is HA setup without --set addLocal="false",and Rancher running on the same cluster.

@laoshancun I've sent you the slack invite, please check it and join.
Also, if you did not set it to false explicitly, the local cluster should have been there. Did you have it prior to upgrade to 2.2?

@alena1108 From further discussion with @laoshancun on slack:

The underlying cluster used for rancher HA setup is not supported (We support RKE and he's using ACK from Alibaba cloud)
Prior to upgrade to 2.2, the rancher server was removed by mistake, and recovered only partially because local cluster went missing after that.
The overall state of this setup doesn't seem right, that's why we haven't been able to repro, maybe starting afresh with 2.2 is the best option?

Given that the set of steps described in https://github.com/rancher/rancher/issues/19206#issuecomment-481367971, something we don't support - running rancher on ali cloud, rancher cleanup being done only partially - I'm going to close the issue.
@laoshancun can you try running fresh rancher 2.2, and see if you encounter the issue? Please reopen the bug if you still see it.

OK,It's seems that i have to create new one.
THX,@alena1108 @mrajashree

我用腾讯云：	Failed to ensure catalog "catalog://?catalog=system-library&template=rancher-monitoring&version=0.0.2": failed to find catalog by ID "catalog://?catalog=system-library&template=rancher-monitoring&version=0.0.2": catalogtemplateversions.management.cattle.io "system-library-rancher-monitoring-0.0.2" not found

@lanmingle Is there any k8s management tools better than Rancher?


Upgrade to Django 2.1

After the Auth fiasco, we should check the rest of our code for 2.1 compatibility. The bottom of here lists removals between 2.0 and 2.1.

Now running on 2.1.7


Bugfix: 292 Show the admin toolbar on frontend when logged in as an agency editor.

Don't hide the admin bar if the user is an agency editor.
I've taken the opportunity to simplify this code so that anybody with permission to edit pages will see the admin bar. I've tested this against each of the user roles we use, and this behaves as expected:

Administrators, Global Editors, Agency Editors all see the admin bar.
Subscribers do not.

@marcincichon @RobertWDLowe please review?

looks good, @ollietreend

@ollietreend  looks good


Using SMARTS patterns to specify core atoms in mapping

Users have expressed the desire to allow the specification of the core atoms using SMARTS patterns. This should allow a higher flexibility when dealing with ligands for which users know the shared core but we don't have them in the exact same position, for example.

Releated https://github.com/choderalab/perses/issues/800


[main] Upgrade to latest dependencies

Cron -knative-prow-robot
/cc knative-sandbox/api-core-wg-leads
/assign knative-sandbox/api-core-wg-leads
Produced by: knative-sandbox/knobots/actions/update-deps
Details:
bumping knative.dev/pkg ea766de...980a337:
  > 980a337 Fix revive related linting issues (# 2131)

/approve
/lgtm


<EMAIL_ADDRESS>is a dependency and gets installed with Vue 2 projects

Hi! I was looking throughout a dependency graph of a Vue 2 + Vite project and noticed that the Vue 3 compiler chain was being included alongside the Vue 2 compiler chain, and npm was having issues properly de-duping the multiple conflicting dependencies.
I'm not quite sure why detective-vue2 has a dependency on the Vue 3+ compiler version.
It seems like this is best suited to be an optionalPeerDependency with a version range that includes 2.x and 3.x, not a real dependency
Could you fix the versioning?

Hi,
Because vue3 compiler can parse dependencies from the vue2 and vue3 SFC templates. Its not optional dependency.
I am not aware of how we could require vue2 or vue3 version of the package. If I remember correctly they also had different API so the code would need some adjustments as well.

The name is detective-vue2 because somebody else took detective-vue -name. Later this library was migrated to dependents -organisation and the name remained same


Review pull_request_template.md and create-list.md

create-list.md has at least 1 rule that's not in pull_request_template.md and could get missed by people making a PR:
Your awesome list should be named awesome-list-name"

I think these files could either be combined, or the PR template could be simplified (without removing rules), and create-list.md could have more details about each requirement.
I think if the PR template were more like a checklist or form, people would be able to follow it better.
I can give it a try, if you're ok with this suggestion.

Having one canonical list with all the rules sounds valuable to me

I think if the PR template were more like a checklist or form, people would be able to follow it better.

It used to be in checkbox form, but almost everyone checked stuff without actually adhering to it, so it ended up being a useless effort.

I think these files could either be combined, or the PR template could be simplified (without removing rules), and create-list.md could have more details about each requirement.

Good point. Fixed :)


Can't set FixedResolutionViewport in FlameGame

Current bug behavior
I first tried to set the viewport with camera.viewport = FixedResolutionViewport(...); but the camera getter is deprecated and says to use a World and CameraComponent.
I did as instructed and tried to set the viewport with cameraComponent.viewport = FixedResolutionViewport(...); but viewport is final. So then I tried to set the viewport in the constructor:
cameraComponent = CameraComponent(
  world: world,
  viewport: FixedResolutionViewport(
    Vector2(1000, 2000),
    clip: false,
  ),
);

but I get this error:
The argument type 'FixedResolutionViewport' can't be assigned to the parameter type 'Viewport?'.dart[argument_type_not_assignable](https://dart.dev/diagnostics/argument_type_not_assignable)
Expected behavior
An easy way to set a FixedResolutionViewport. If I'm doing something wrong, the correct code snippet should be added to the relevant documentation .
Steps to reproduce

I tried to set this up in zapp but it's using an old version of Flutter/Dart/Flame which doesn't have the deprecation, so I've just pasted the snippet here:
import 'package:flame/camera.dart';
import 'package:flame/game.dart';
import 'package:flutter/material.dart';

void main() {
  runApp(
    GameWidget(
      game: MyGame(),
    ),
  );
}

class MyGame extends FlameGame {
  late final world = World();
  late final CameraComponent cameraComponent;

  @override
  Future<void> onLoad() async {
    await super.onLoad();

    cameraComponent = CameraComponent(
      world: world,
      viewport: FixedResolutionViewport(
        Vector2(1000, 2000),
        clip: false,
      ),
    );
    addAll([cameraComponent, world]);
  }
}

Flutter doctor output

[✓] Flutter (Channel stable, 3.10.6, on Pop!_OS 22.04 LTS 6.4.7-surface, locale en_GB.UTF-8)
    • Flutter version 3.10.6 on channel stable at /home/ahanney/development/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision f468f3366c (3 weeks ago), 2023-07-12 15:19:05 -0700
    • Engine revision cdbeda788a
    • Dart version 3.0.6
    • DevTools version 2.23.1

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /home/ahanney/Android/Sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: /home/ahanney/.local/share/JetBrains/Toolbox/apps/AndroidStudio/ch-0/222.4459.24.2221.10121639/jbr/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✗] Chrome - develop for the web (Cannot find Chrome executable at google-chrome)
    ! Cannot find Chrome. Try setting CHROME_EXECUTABLE to a Chrome executable.

[✓] Linux toolchain - develop for Linux desktop
    • Ubuntu clang version 14.0.0-1ubuntu1.1
    • cmake version 3.22.1
    • ninja version 1.10.1
    • pkg-config version 0.29.2

[✓] Android Studio (version 2022.2)
    • Android Studio at /home/ahanney/.local/share/JetBrains/Toolbox/apps/AndroidStudio/ch-0/222.4459.24.2221.10121639
    • Flutter plugin version 74.0.2
    • Dart plugin version 222.4582
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[✓] VS Code (version 1.80.2)
    • VS Code at /usr/share/code
    • Flutter extension version 3.70.0

[✓] Connected device (1 available)
    • Linux (desktop) • linux • linux-x64 • Pop!_OS 22.04 LTS 6.4.7-surface

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

More environment information

Flame version: 1.8.1
Flutter 3.10.6
Platform affected: all

Log information

More information

Messing with the types a little reveals that we have two Viewport classes. Is this intentional?

'package:flame/src/camera/viewport.dart (via package:flame/camera.dart)'
'package:flame/src/game/camera/viewport.dart (via package:flame/game.dart)'

The FixedResolutionViewport is from the old API, you either need to use FixedSizeViewport or FixedAspectRatioViewport depending on what you want to achieve.
See the docs for more information: https://docs.flame-engine.org/latest/flame/camera_component.html#viewport

Oh I see, I was using https://docs.flame-engine.org/latest/flame/camera_and_viewport.html.

You can use CameraComponent.withFixedResolution to achieve the same results with the new camera system.


Add support for the new staking action

[OKR 2021Q4] All dev tools are updated to support the new staking action (Shared with Developer Platform)
Part of https://airtable.com/apprrAlfLE1QI6gao/tbl9h3jXf56F4lAql/viw1VzQz6DtES5vbR/rec7i5zgeHPtd6lv7?blocks=hide

Is there any more info on what this action would look like? Is this related to the new staking pool contract which is another OKR?

Based on updated Q4 OKRs, removed from NodeX Q4 board


Fix list layout

@bartvermeersch : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"


How to get a list of all symbols for an exchange?

Is it possible to get a list of all symbols trading at a given exchange?
Similar to what is presented here for example: https://eodhistoricaldata.com/exchange/TO
but in a nice dataframe format?

Ah, sorry, it is already implemented here:
https://github.com/EodHistoricalData/python-eodhistoricaldata/blob/d7080d32b02d46f5177ced602fb9bab51d36101a/eod_historical_data/data.py#L156


Timeout API rework

Big (but less so than the first version) API rework to make timeouts an opaque type, allowing for  their specification in alternate representations like 64 bit types, higher precision units and absolute/uptime values.
See #21305 for the RFC detailing the changes.

Update to include k_uptime_ticks(), which was in the RFC but been forgotten.  Also sneak in the extra test code for absolute timeouts which got dropped in the first version.

Rebased again around a minor collision.  Cleaned up the long line warnings from checkpatch.  Added the discussed doc changes and a few other minor things from review.  Fixed the elapsed() bug @nordic-krch  found.
Should be good to go.

I think it would be good to align timeout parameter documentation in kernel.h
For example k_thread_create delay is described as Scheduling delay, or K_NO_WAIT (for no delay).
but I noticed that here https://github.com/zephyrproject-rtos/zephyr/blob/698000f40c6e9f44fbc7753b259ae5e92ba90fc2/tests/kernel/threads/thread_apis/src/test_threads_spawn.c#L119 k_thread_create is called with K_FOREVER.
Some examples:

Non-negative waiting period to obtain a data item or one of the special values K_NO_WAIT and                K_FOREVER.
Delay before submitting the work item (no mention about special values)

Imo, they could all refer to k_timeout_t. Like:

Waiting period to obtain  a data item. See k_timeout_t for details.
Delay before submitting the work item. See k_timeout_t for details.
etc.

@andyross what about CI? Any chance to get results there?

Are those really something that should block merge?  The documentation as written is correct.  I agree it could be improved.  Can't we file an issue on that or just submit it personally post-merge?
As far as CI: no, PRs that touch this many tests don't complete.  Someone has to run it manually before merge.

Rebased

@andyross what about CI? Any chance to get results there?

@nordic-krch given the amount of the changes CI times out here, this will go through a manual run before merge. If you see any build/test issues related to this change, please raise them. Approval of this PR should not depend on CI passing.
The only alterntive is to raise the number of shippable minions  in .shippable.yml and let CI complete and then remove the CI change before the merge. @andyross You can do that if you feel like it.

just noticed this one:
/home/buildslave/src/github.com/zephyrproject-rtos/zephyr/drivers/timer/xlnx_psttc_timer.c:164:12: error: invalid operands to binary == (have 's32_t' {aka 'int'} and 'k_timeout_t' {aka 'struct <anonymous>'})
  164 |  if (ticks == K_FOREVER) {
      |            ^~

Correct some rebase crosses in built-but-not-run platform and one spot in tests/kernel/context that wasn't compatible and literally just merged.

there are 2 issues remaining:

loapic_timer.c
mcuboot. I've created PR for mcuboot: https://github.com/JuulLabs-OSS/mcuboot/pull/693
However, mcuboot part has some dependencies since #23625 required mcuboot changes which already got merged (https://github.com/JuulLabs-OSS/mcuboot/pull/682)

Fixed up the acrn failure (shouldn't acrn be built under sanitycheck if it's run under CI?  What broke there?)
Similarly: when did mcuboot move out of tree?  I don't even know how to run those tests, why are they in CI?

* mcuboot. I've created PR for mcuboot: [JuulLabs-OSS/mcuboot#693](https://github.com/JuulLabs-OSS/mcuboot/pull/693)

I do not think it is a good idea to be depending on upstream PRs, I would first submit this and make it work in the repo maintained in west and do the upstreaming in parallel, otherwise PRS will be stuck for a long time depending on how fast upstream reacts.

Rebase to resolve collisions with commit a4c919064973 and commit 84921c53b1b5 which snuck around this one into master.
Note I did run this through a full sanitycheck myself, just spot checked the relevant tests (kernel/threads and kernel/interrupt) kernel/common across the standard platforms.

* mcuboot. I've created PR for mcuboot: [JuulLabs-OSS/mcuboot#693](https://github.com/JuulLabs-OSS/mcuboot/pull/693)

I do not think it is a good idea to be depending on upstream PRs, I would first submit this and make it work in the repo maintained in west and do the upstreaming in parallel, otherwise PRS will be stuck for a long time depending on how fast upstream reacts.

What we've been doing for tightly-coupled projects like MCUboot is to send a PR upstream, point to that PR and remote in the west.yml in the zephyr PR, then merge the upstream MCUboot PR, then updated our MCUboot copy with the latest MCUboot, then point the west.yml in this PR to our MCUboot.
@nvlsianpu can help with that, or @nordic-krch or myself can do that

Er, excuse me: I did not run this through sanitycheck before pushing.
As far as that mcuboot PR, it shows merged now.  I'm hoping we're OK now.

DNM so we remember to remove the shippable hack.

One more to clean up another crossed issue with the new cavs_timer driver.  This also includes a build fix to an upstream bug with cc26x2r1_launchxl  that is merely unmasked here.  It's not a timer driver thing at all, but it's a simple and obvious patch.
Note that we're still seeing a failure in mcuboot though.  @nordic-krch , can you take a look at that?

OK, we're good now.  Just the one mcuboot failure left.

Fyi the cc26x2r1_launchxl issue is being addressed in #23929.

@andyross please rebase, the mcuboot build should be fixed now

Remaining issue with mcuboot in PR: https://github.com/JuulLabs-OSS/mcuboot/pull/700

Rebased (actually the only collision was that last patch to cc26x2r1_launchxl , which made mine unnecessary so I just removed it).


Add iyzico to Finance

[x] My submission is formatted according to the guidelines in the contributing guide
[x] My addition is ordered alphabetically
[x] My submission has a useful description
[x] The description does not have more than 100 characters
[x]  The description does not end with punctuation
[x] Each table column is padded with one space on either side
[x] I have searched the repository for any relevant issues or pull requests
[x] Any category I am creating has the minimum requirement of 3 items
[x] All changes have been squashed into a single commit

Thanks for your contribution @3rt4nm4n, but apparently this API is not free. There is a fee for each use. 😅
ref 1: https://www.iyzico.com/en/partner-solutions ref 2: https://www.iyzico.com/en/open-source

Sorry for the inconvenience my mistake.


[Event Request] Codeunit 91 "Purch.-Post (Yes/No)" - OnBeforeRunPreview

    [EventSubscriber(ObjectType::Codeunit, Codeunit::"Gen. Jnl.-Post Preview", 'OnRunPreview', '', false, false)]
    local procedure OnRunPreview(var Result: Boolean; Subscriber: Variant; RecVar: Variant)
    var
        PurchaseHeader: Record "Purchase Header";
        PurchPost: Codeunit "Purch.-Post";
        IsHandled: Boolean;
    begin
        IsHandled := false;
        OnBeforeRunPreview(Result, RecVar, IsHandled);
        if IsHandled then
            exit;

        with PurchaseHeader do begin
            Copy(RecVar);
            Ship := "Document Type" = "Document Type"::"Return Order";
            Receive := "Document Type" = "Document Type"::Order;
            Invoice := true;
        end;
        OnRunPreviewOnBeforePurchPostRun(PurchaseHeader);
        PurchPost.SetPreviewMode(true);
        Result := PurchPost.Run(PurchaseHeader);
    end;

    [IntegrationEvent(false, false)]
    local procedure OnBeforeRunPreview(var Result: Boolean; RecVar: Variant; var IsHandled: Boolean)
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


[Bug] {xct:Translate String} works only for currently open pages

Description
If user changes language with LocalizationResourceManager.Current.SetCulture(), all strings localized with {xct:Translate String} on currency open pages get updated to new language, but if user opens new page it will be localized with old language.
Steps to Reproduce

Open some page (page has language A)
Change culture to language B with LocalizationResourceManager.Current.SetCulture()
Open new page with some string localized with {xct:Translate String}
New page will be localized with language A instead of B

Expected Behavior
Newly opened pages respect current language
Actual Behavior
Newly opened pages localized using language app was opened with
Basic Information

Version with issue: 1.0.0
Last known good version:
IDE:
Platform Target Frameworks:

iOS:
Android: 10
UWP:

Android Support Library Version:
Nuget Packages:
Affected Devices:

Workaround
Manually localize all strings from the code
Reproduction imagery

Reproduction Link

I have seen this behavior in my own app too and have been working on a fix there. I will have to look at how I can implement this the easiest within the current XCT API without breaking it.

I have seen this behavior in my own app too and have been working on a fix there. I will have to look at how I can implement this the easiest within the current XCT API without breaking it.

I have seen this behavior in my own app too and have been working on a fix there. I will have to look at how I can implement this the easiest within the current XCT API without breaking it.

I thinks I found the cause for this issue. Thread.CurrentThread.CurrenUICulture is not persisted between pages. To fix it just for your app, you can store culture yourself and set Thread.CurrentThread.CurrenUICulture manually in constructor for each page.
I think it should work

I have seen this behavior in my own app too and have been working on a fix there. I will have to look at how I can implement this the easiest within the current XCT API without breaking it.

I thinks I found the cause for this issue. Thread.CurrentThread.CurrenUICulture is not persisted between pages. To fix it just for your app, you can store culture yourself and set Thread.CurrentThread.CurrenUICulture manually in constructor for each page.
I think it should work


Format of Pose in 0000000*.txt in DATAROOT/sample/sample1/pose/

Hi @zmurez ; Could you please discuss the format of data in each rgb corresponding pose's text file in  DATAROOT/sample/sample1/pose/ ?
For example : 00000001.txt contains
6.208292841911315918e-01 -2.021953016519546509e-01 7.574220299720764160e-01 5.417202062606811808e+00 -7.820422649383544922e-01 -9.244704246520996094e-02 6.163306236267089844e-01 4.002395458221435831e+00 -5.459773540496826172e-02 -9.749721288681030273e-01 -2.155191004276275635e-01 2.021697369813919121e+00 0.000000000000000000e+00 0.000000000000000000e+00 0.000000000000000000e+00 1.000000000000000000e+00

This is the 4x4 pose matrix. It is composed of the rotation R matrix and translation vector T: [R, T] (with T in meters). The camera extrinsics matrix (which maps world coordinates to camera coordinates) is the inverse of the pose matrix.
We use a right handed coordinate system where the world is z-up (gravity is along the z axis), while the camera is (x-right, y-down, z-forward, same as opencv)

@zmurez  could you please elaborate on the values present in the pose.txt file. What does each value stand for?

Here is a reference https://www.learnopencv.com/geometry-of-image-formation/
let P be the pose matrix in the .txt file, and K be the 3x3 intrinscis matrix. (note that inverse(P) is the extrinsics matrix)
Then for a point X=(x,y,z,1) in homogeneous world coordinates, it gets mapped to a pixel in the image q=(u,v,1) according to
K @ inverse(P)[:3,:] @ X
(where @ is matrix multiplication)
Hope this helps

Here is a reference https://www.learnopencv.com/geometry-of-image-formation/
let P be the pose matrix in the .txt file, and K be the 3x3 intrinscis matrix. (note that inverse(P) is the extrinsics matrix)
Then for a point X=(x,y,z,1) in homogeneous world coordinates, it gets mapped to a pixel in the image q=(u,v,1) according to
K @ inverse(P)[:3,:] @ X
(where @ is matrix multiplication)
Hope this helps

Hi, could you provide some intuition about why you exported the pose matrix with your camera, but not the extrinsic matrix directly? I am trying to do with ARKit...

@zmurez
Thank you for the information. Could you tell me how to convert bpy.context.scene.camera.matrix_world into the pose matrix for Atlas?
https://docs.blender.org/api/current/bpy.types.Object.html#bpy.types.Object.matrix_world


[WIP] Adaptive batch size

Still very much in progress, just opening the PR to track things.  If anyone cares at this point, this at least has my basic plan for implementing this in the docstrings, though I haven't done any of the actual work yet.

Preliminary result: looks like I can get BiDAF down to less than 45 minutes per epoch using this technique (@nelson-liu should be happy about that =) ).  This is a ~10x speedup, better than the 4-5x from my initial estimate.  It's possible it'll still crash on me before finishing the first epoch, but I'm optimistic.  The largest batches have somewhere around 400 instances in them.

wow, that's amazing! Great work @matt-gardner

It worked!  Final epoch time: ~34 minutes!

Ok, this is close to done.  I still need to add a test for the adaptive grouping, but other than that I think it's ready for review.

LGTM

Just a note for anyone who was following this thread:
You might not want to use as large a batch size as possible - this is kind of the whole point of SGD, anyway - you use minibatches, and the variance this causes in your gradient is actually helpful in avoiding local minima.  I ran BiDAF yesterday, fully optimizing GPU memory usage, and some batches had upwards of 400 instances in them.  This gave a running time of 33-34 minutes per epoch, but it also had poorer learning behavior, and only got to ~53% exact match on the dev set, where it should be reaching closer to ~60% (this is using one randomly picked dev annotation, not all three, so numbers are lower than the official evaluation).
I'm running it again, capping batch size at 60 (batches are still smaller for the largest instances, so it fits on the GPU without any truncation), and it's taking ~1.1 hours per epoch, but the learning behavior seems to be a bit better.  It hasn't finished yet, though.  I'll update this thread again when it does.

So the run with batch size 60 got to about 55% accuracy on span begin, and 58% on span end, which we could be optimistic and say translates to ~55% exact match, which is still a few percent below Min's implementation (which gets ~60% on this metric).  We haven't tuned any of the learning parameters, or dropout, or regularization, or anything, though.  The point is that the running time of this code is now a little faster than Min's, the code is much cleaner, and if someone wants to, you could tune it to get comparable performance.


path for new file textbox is not wide enough for some file names and it doesn't highlight selected text

Hi,
First of all, I would like to thank you for making this minimalistic theme. I really like it. One issue that I regularly face is this create new file issue. The textbox for entering name is not wide enough for some names, especially when the font size is large. Also there is no highlighting when I select text in this textbox.

I would also like to know what are the plugins that you use. Finding syntactical bugs is hard while using such monochromatic themes.


Think about making a new mini-module for Markdown

This is common for both Git READMEs, Jupyter and RMarkdown/Quarto, and could be optimised.

Make a small module with Markdown that can be linked from the relevant pages in the various tools. This module should not be on the schedule, and it just linked from the other tutorials.

See #220


Fixed #375

This change is

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Is there a plan to integrate mantine charts as well?

https://mantine.dev/charts/getting-started/

Hi @littlefishna
The charts are currently available, and will be in the next release of the docs.  If you want to get a head start, you can use the mantine docs as a reference.  For example, here is the stacked area chart
data = [
  {"date": "Mar 22", "Apples": 2890, "Oranges": 2338, "Tomatoes": 2452},
  {"date": "Mar 23", "Apples": 2756, "Oranges": 2103, "Tomatoes": 2402},
  {"date": "Mar 24", "Apples": 3322, "Oranges": 986, "Tomatoes": 1821},
  {"date": "Mar 25", "Apples": 3470, "Oranges": 2108, "Tomatoes": 2809},
  {"date": "Mar 26", "Apples": 3129, "Oranges": 1726, "Tomatoes": 2290}
]

dmc.AreaChart(
    h=300,
    dataKey="date",
    data=data,
    type="stacked",
    series=[
        {"name": "Apples", "color": "indigo.6"},
        {"name": "Oranges", "color": "blue.6"},
        {"name": "Tomatoes", "color": "teal.6"}
    ]
)


How can I validate if redis is used to enforce policies?

I have an action-chain and the following policy:
---
name: deploy_vm.concurrency
description: Limits the concurrent executions for the deploy_vm workflow
enabled: true
resource_ref: crv_vm.deploy_vm
policy_type: action.concurrency
parameters:
  action: delay
  threshold: 1

As documented at https://docs.stackstorm.com/reference/policies.html I had to add Redis or ZooKeeper to the Stack. I chose Redis. I configured a coordinator_url and when I execute two deploy_vm workflows at the same time the second one gets the status "delayed" and start running as soon as the first one is finished. So that works perfectly.
However I was trying to validate If I saw some data in Redis and was curious how the key was composed and what the value was. But during the executions I got the following response everytime:
<IP_ADDRESS>:6379> keys "*"
(empty list or set)
<IP_ADDRESS>:6379> keys "*"
(empty list or set)
<IP_ADDRESS>:6379> keys "*"
(empty list or set)
<IP_ADDRESS>:6379> keys "*"
(empty list or set)

So is Redis indeed used or is something else going on? How can I validate this?

I powered off the Redis server and I still get the same behaviour, that the second workflow execution gets the status delayed until the first is finished. So is the note from the documentation not correct (anymore)?
"The concurrency policy type is not enabled by default and requires a backend service such as ZooKeeper or Redis to work."

+1 I also have this question, I'm interested how to view the keys, values in redis.

Ok got my answer.  To get your keys/values (string types) in redis in a python terminal try this while your st2 executions are running:
>>> import redis
>>> r = redis.Redis("localhost", 6379)
>>> while True:
...     keys = r.keys()
...     if keys:
...         values = r.mget(keys)
...         kv = zip(keys, values)
...         print 'Dumping keys:'
...         print kv

Thanks for the followup @tim-ireland - will be useful for someone else searching for this in future


FP16 support for more architecture

https://github.com/NVIDIA/TransformerEngine/blob/522753f2f0fab0bbe3764a6b8819ac45d66d07fb/transformer_engine/common/fused_attn/fused_attn.cpp#L61
Hi, I want to know if the transformer engine supports fp16/bfp16 data type in the future for the ampere arch(sm80, sm86) or more architecture?
thanks!

Hi @CaramelFc, I assume you ask about the fused attention support specifically (as TE does support Ampere for other functionality). We expose multiple fused attention implementations - via Flash Attention (from Hazy Research) and via cuDNN. Flash Attention is exposed on the Python side (see: https://github.com/NVIDIA/TransformerEngine/blob/main/transformer_engine/pytorch/transformer.py#L558). The function you point to is the C++ side calling cuDNN implementation - right now it is very limited with exposing only short sequence length (<=512) FP8 attention implementation but we intend to increase its coverage (including 16-bit types support and Ampere) in the coming months.


WIP: Debugging OpenCL broadcasting tests' crashes

Thanks for submitting a pull request! Please remove this text when submitting.
Start by filling in the Summary, Tests, and Side Effects sections of this pull request and then work through the handy checklist at the bottom. If anything significant is missing, the pull request may be closed until it's ready. The full guidebook on how pull requests are reviewed is here: Code Review Guidelines.
Summary
Describe the contents of the pull request. The issue describes the problem that needs to be fixed, whereas the text here should help the reviewer figure out to what extent this pull request solves the issue.
Describe implementation details and any relevant information that would help the code reviewer understand this pull request. If there is anything special you need to draw someone's attention to, do it here. Link to any related pull requests or Discourse threads.
Tests
Describe the new tests with the pull request.
For bug fixes there should be a new test that would fail if the patch weren't in place (so that the bug could be caught if it comes up again).
For new features there should be at least one test showing the expected behavior and one test that demonstrates error handling. Be aware the reviewer will very likely ask for more tests than this, but it's a start.
Side Effects
Are there any side effects that we should be aware of?
Release notes
Replace this text with a short note on what will change if this pull request is merged in which case this will be included in the release notes.
Checklist

[ ] Math issue #(issue number)

[ ] Copyright holder: (fill in copyright holder information)
The copyright holder is typically you or your assignee, such as a university or company. By submitting this pull request, the copyright holder is agreeing to the license the submitted work under the following licenses:
- Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
- Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

[ ] the basic tests are passing

unit tests pass (to run, use: ./runTests.py test/unit)
header checks pass, (make test-headers)
dependencies checks pass, (make test-math-dependencies)
docs build, (make doxygen)
code passes the built in C++ standards checks (make cpplint)

[ ] the code is written in idiomatic C++ and changes are documented in the doxygen

[ ] the new changes are tested

The crash happens in program.build(). Looking at the arguments to this call, nothing looks wrong, so this must be a bug in the kernel compiler. There is not much we can do to fix this. We could try to update the OpenCL drivers on the mac that is failing. @serban-nicusor-toptal could you do that?
I will also try to simplify the generated kernels a bit in attempt to work around this.

Hey, current version is:   Platform Version                                OpenCL 1.2 (Jun 15 2018 20:52:09)
I will now try to update it.

Hey @t4c1 I think OpenCL 1.2 is the last version we can support on the mac as written in the Apple Support forums here please correct me if I'm wrong.

Yeah, OpenCL 1.2 is the latest OpenCL version we can support on MacOS. But can we maybe upgrade the GPU driver?

Mac has GPU drivers tied to the OS and I couldn't find an update for the Drivers, just some not needed ones.
Software Update Tool
Copyright 2002-2015 Apple Inc.

Finding available software
Software Update found the following new or updated software:
   * iTunes Device Support Update-
        iTunes Device Support Update ( ), 105805K [recommended] [restart]
   * iTunesX-12.8.2
        iTunes (12.8.2), 273614K [recommended]

Sorry I couldn't help, let me know if there's anything else I can do.

@rok-cesnovar @serban-nicusor-catena is it possible to identify which Jenkins machine / opencl version this is breaking for? Talking to @t4c1 it sounded like this may be a bug directly in the opencl impl when compiling programs

Could we use pocl on the mac instead of the default mac implementation? tmk most macs now don't come with OpenCL do they?

I think it comes by defaut, I've tried on a mac-mini using clinfo.
bew install clinfo
I'm getting OpenCL 1.2 (Jun 8 2020 17:36:15), the only thing I've installed was XCode Developer Tools from the Store. I've also found this in the opencl repo:
Type: GPU, CPU
Vendor: Apple
This OpenCL implementation is part of Mac OS X 10.6 (Snow Leopard) and
Max OS X 10.7 (Lion) operating systems. The OpenCL package has been
successfully tested with nVidia GPU devices (only single-precision is
supported on the devices available to us).
The OpenCL framework is part of the OS so no additional libraries need
to be installed.

Although I'm not sure if it's applicable for all versions, couldn't find anything on their forums.

It seems the workaround from #2215 does not for for the mac, so we can not use it for running OpenCL tests anymore.


Scim2 compliance test suite v1 backend

Purpose
This is back-end implementation of java library used in SCIM2 Compliance Test Suite. Resolves #8910

This PR is closing with #18
Will address the comments in #18 and merge the PR


[FEATURE] Added status code in error formatter

Closing because the reasons I outlined in https://github.com/webonyx/graphql-php/pull/869#discussion_r641364409


Feature add new apcu

The last commit '641af4e5444ce58263d46b68f7b8873a8399bd62' about APCU is ungraceful。
My commit contain APC and APCU。
it contains test cases,Although it's copied. Ha-ha
Finally,I deleted the old APC test cases.

@Jimdo any chance of getting this reviewed/merged? Be great to have PHP7+ support for APC

@gd I don't think this is going to be Merged as nobody's actively maintaining this project anymore.

@gdsmith I don't think this is going to be Merged as nobody's actively maintaining this project anymore.

So sad

@yuanshuli11 If you make it your own package, people can always use the two in conjunction with one another. I'd definitely use this :)

This project is dead, but I'm maintaining it under my employer - https://github.com/endclothing/prometheus_client_php. Feel free to submit the PR there.


Add Ptr variants to option/arguments where you can pass a pointers to existing variables

Instead of declaring new ones.
Fixes #83.

Coverage decreased (-0.3%) to 95.611% when pulling 0710d6a81b775870a434b8b583425b05be63634c on issue-83-ptrs into d9c41c7c90d99b076732ee72a657577128d610da on master.


question about running "training_Semantic3D.py"

Hi, @HuguesTHOMAS ,
When I run python training_Semantic3D.py, I got the following error:
Dataset Preparation
*******************
sg27_station10_rgb_intensity-reduced already done

birdfountain_station1_xyz_intensity_rgb already done

castleblatten_station1_intensity_rgb already done

castleblatten_station5_xyz_intensity_rgb already done

marketplacefeldkirch_station1_intensity_rgb already done

marketplacefeldkirch_station4_intensity_rgb already done

MarketplaceFeldkirch_Station4_rgb_intensity-reduced already done

marketplacefeldkirch_station7_intensity_rgb already done

sg28_Station2_rgb_intensity-reduced already done

stgallencathedral_station1_intensity_rgb already done

stgallencathedral_station3_intensity_rgb already done

stgallencathedral_station6_intensity_rgb already done

StGallenCathedral_station6_rgb_intensity-reduced already done

Preparing KDTree for all scenes, subsampled at 0.060
Traceback (most recent call last):
  File "training_Semantic3D.py", line 210, in <module>
    dataset.load_subsampled_clouds(dl0)
  File "/data/code9/KPConv/datasets/Semantic3D.py", line 289, in load_subsampled_clouds
    sub_labels = data['class']
ValueError: no field of name class

In line 289 of Semantic3D.py, data is considered as dict. However, when I debug into the code, data is actually a numpy array, and its shape is(1034819,). Could you give some suggestions to fix this issue?
THX!

Hi @amiltonwong ,
I found some mistakes in the Semantic3D.py file and corrected it. If you delete your data folders (except the original data folder), and start from the begining with the new Semantic3D.py file, it should work.
Best,
Hugues


[Feature/extensions] Add getSettings support for AD

Description
Copies the getSettings() method impelmentation from AnomalyDetectorPlugin to AnomalyDetectorExtension.  This is the third and final step to complete work already done in SDK #147 and and OS #4519 register AD settings with OpenSearch
Issues Resolved
Fixes SDK #159
Relates to SDK #132
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@dbwiddis build is failing

Failure is not associated with this PR.  Looks like package change/rename associated with ExtensionRestResponse from a previous PR merged to the SDK (and probably will change with my consumed param PRs under review).   How often does this project sync with those changes?

Updated the RestCreateDetectorAction (which was just a copy of hello world) with the new version of Hello World.  Tests should pass once both 4705 and 169 are merged.

169 and 4705 merged, should be able to re-run the tests now.

@saratvemulapalli can you take of it? I don't have access to rerun the test for AD.

Kicked them off.
Another hacky way to re-run them is to do a force push without any changes. ;)

Will probably have to do that.  I think the rerun is not pulling the updated branches.

Well now it's failing on something else entirely, probably something commented out that's still being tested.  I give up.

All green.
And this, kids, is why you pay attention to those little IDE flags that say you're using deprecated code. :-)

Don't merge yet.  Looks like I need to extend NodeRequest and not TransportRequest.  See #687 and #688

OK, ready to merge.  I'm getting confused by this same fix not working on main here, but I'm not sure it's broken (yet) on main here.


[Bug]: Should not export type for class declare

Package
@lucia-auth/oauth
Describe the bug

the customized provider

export class TcskAuth<_Auth extends Auth = Auth> extends OAuth2ProviderAuth<
  TcskUserAuth<_Auth>
> {
  private config: Config;

it will typings check issue
export type { OAuth2ProviderAuth, OAuth2ProviderAuthWithPKCE } from "./core/oauth2.js";

should be
export { OAuth2ProviderAuth, OAuth2ProviderAuthWithPKCE } from "./core/oauth2.js";

This isn’t a bug. OAuth2Provider is an internal API

but you have exported it , we can directly use
import {
  OAuth2ProviderAuth,
  ProviderUserAuth,
} from '@lucia-auth/oauth';

it’s confused, in fact sometimes we need to defined our customized oauth provider

The type is exported, but we recommend using the provided helpers instead for custom providers: https://lucia-auth.com/oauth/basics/oauth2/#oauth-helpers