ROX-9134: add cache for compliance aggregations

Description
This PR is opssit approach to https://github.com/stackrox/stackrox/pull/7353 where we delete caching code.
This PR implements in memory cache for calculated aggregations.
The code for keeping aggregates was not implemented for postgres and not enabled for SAC (see: https://github.com/stackrox/stackrox/pull/301/files#r806715921).
To support SAC the cache key is built from compacted scope tree and query params.
Clean cache just after scan:

URL
code
type
source
size
time

graphql?opname=getAggregatedResultsAcrossEntity_NODE
200
xhr
main.97af8cf8.js:2
506 B
665 ms

graphql?opname=getAggregatedResultsAcrossEntity_NAMESPACE
200
xhr
main.97af8cf8.js:2
777 B
665 ms

graphql?opname=getAggregatedResultsByEntity_CLUSTER
200
xhr
main.97af8cf8.js:2
583 B
662 ms

graphql?opname=getAggregatedResultsAcrossEntity_CLUSTER
200
xhr
main.97af8cf8.js:2
592 B
662 ms

Reload the page:

URL
code
type
source
size
time

graphql?opname=getAggregatedResultsAcrossEntity_NODE
200
xhr
main.97af8cf8.js:2
499 B
60 ms

graphql?opname=getAggregatedResultsAcrossEntity_NAMESPACE
200
xhr
main.97af8cf8.js:2
778 B
67 ms

graphql?opname=getAggregatedResultsByEntity_CLUSTER
200
xhr
main.97af8cf8.js:2
581 B
54 ms

graphql?opname=getAggregatedResultsAcrossEntity_CLUSTER
200
xhr
main.97af8cf8.js:2
593 B
62 ms

Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
CI

Current dependencies on/for this PR:

master

PR #7378   👈

This comment was auto-generated by Graphite.

https://github.com/stackrox/stackrox/pull/7388


Output first n elements?

It'd be cool to just output the first n elements of the sorted md stream.

Could you kindly advise me on how we might be able to set or limit the number of items in a list that may have multiple depths? I would greatly appreciate your opinion on this matter. Thank you.


Added Mermaid to MarkdownIt editor

This is really a simple PR with addition of Mermaid into MarkdownIt editor, that BookStack is using.
It allows to create Mermaid diagrams via Markdown with specific syntax.
This is a preview of how it works:

This addresses #495 and #566 .

Thanks for offering this PR @cadavre but I'd really need my concerns listed here addressed before considering support.
In addition, I'm fairly sure that on the latest version of BookStack this will only show the mermaid output within the preview window, not within the the actual page output after save, unless I'm missing something?

Hah, I'm newbie to BookStack dev and looks like I went kind of too excited after Mermaid started to work in preview.  :/

The current use-case for UML/Mermaid is weaker since we now have draw.io which has a lot of overlap and is well-suited to the use-case of multi-ability environments. I can see why those technically inclined may like UML/Mermaid, kind of want to use it myself tbh, but it's focusing on a narrow audience with a custom text syntax, which kind of opposes the idea/spirit of the project.

Markdown itself sounds like "focusing on a narrow audience with a custom text syntax" in comparison to WYSiWYG. I can see nothing wrong in adding Mermaid which can be accessed only via MD.
I believe we could add Mermaid rendering same as it is done for draw.io – graphs can be exported to .png for browsing mode. If I follow up with this kind of implementation, would it be ok for you to merge it?

Markdown itself sounds like "focusing on a narrow audience with a custom text syntax" in comparison to WYSiWYG. I can see nothing wrong in adding Mermaid which can be "accessed" only via MD.

But now we're providing a narrow feature for an already narrowed audience, and this works against the alignment I'm intended to work towards with the editor to get to a place where you switch between modes easier.

I believe we could add Mermaid rendering same as it is done for draw.io – graphs can be exported to .png for browsing mode. If I follow up with this kind of implementation, would it be ok for you to merge it?

I'd imagine people would still want to edit via mermaid text, This is going to introduce some significant complications to achieve.  In addition it introduces some further complications if accepted in the editor markdown since you now have mis-alignment with the API unless you're able to do things back-end.
To be honest, I can't imagine an implementation I'd be happy to merge at this time, especially as we look to change up the editors later this year and align the editing modes. Will mean wasted effort if done now. I'd rather look to see what's required, after the editor work, to allow people to optionally add "Mermaid" via our customization options (Accepting the limitations/caveats).
I'll close off this PR for now though.

Any chance we can re-evaluate this?


abnormal adrenal gland zona fasciculata area

If this is a request for a new term, please provide us with the following information.

Preferred term label (e.g., asplenia)

increased adrenal gland zona fasciculata area
decreased adrenal gland zona fasciculata area

Synonyms (e.g., absent spleen)

enlarged adrenal gland zona fasciculata area
reduced adrenal gland zona fasciculata area

Textual definition (the definition should be understandable even for non-specialists. Include a PubMed ID or URL(s) to refer to any relevant article(s) that provides information about the suggested term).

an increase in size (two-dimensional area) of the wide middle zone of the adrenal cortex that produces cortisol (hydrocortisone)
a decrease in size (two-dimensional area) of the wide middle zone of the adrenal cortex that produces cortisol (hydrocortisone)

Any additional notes to clarify the meaning or use of the term. Please provide a PubMed ID or URL(s) for the source of this information.

These changes are determined histologically. PMID:36112420

Suggested parent term (Please look in the hierarchy of the MP using the MP Browser or other browsers such as OLS and find the place in the MP where the new term best fits)

abnormal adrenal gland zona fasciculata morphology (MP:0008294)

If you would like a nanoattribution, please indicate your ORCID id or provide a label such as MGI:sbello (organization, first initial-last name)

ORCID:0000-0001-5356-4174

http://purl.obolibrary.org/obo/UBERON_0002054

@slaulederkind Is there a reason to prefer area over size in the term label? Size is the more common term used in labels. Most terms using area refer to total cell area.

looking at annotations also adding
disorganized adrenal gland zona fasciculata

Added
abnormal size MP:0014284
increased size MP:0014286
decreased size MP:0014287
disorganized MP:0014285

I think size should still work for your annotations. Most of the existing MGI annotations tend to mention increased or expanded or reduced size, so I went with the more general term.


Update dependency typescript to v3

This PR contains the following updates:

Package
Type
Update
Change
References

typescript
devDependencies
major
^2.5.2 -> ^3.0.0
homepage, source

Release Notes

Microsoft/TypeScript
v3.1.6
Compare Source
This release contains a fix for a major performance regression.
(Thanks @ajafff!)
Download:

npm

v3.1.5
Compare Source
This release includes a fix for emit on code that uses const enum declarations.
Download:

npm
NuGet

v3.1.4
Compare Source
This release include a set of bug fixes reported for TypeScript 3.1.3 and TSServer plugin configuration support. For the complete list of fixed issues, check out the fixed issues query for TypeScript 3.1.3.
Download:

npm

v3.1.3
Compare Source
This release include a set of bug fixes reported for TypeScript 3.1.2. For the complete list of fixed issues, check out the fixed issues query for TypeScript 3.1.3.
Download:

npm

v3.1.2
Compare Source
This release include a set of bug fixes reported against TypeScript 3.1.1. For the complete list of fixed issues, check out the fixed issues query for TypeScript 3.1.2.
Download:

npm

v3.1.1
Compare Source
For release notes, check out the release announcement
For new features, check out the What's new in TypeScript 3.1.
For breaking changes, check out the Breaking changes in TypeScript 3.1 page.
For the complete list of fixed issues, check out the fixed issues query for Typescript 3.1.
Download:

npm
Visual Studio 2015 plugin
Visual Studio 2017 (Select new version in project options)
NuGet package

Special thanks to all contributors to this release:

Alexander Tarasyuk
Anders Hejlsberg
Andrew Casey
Andy Hanson
Benjamin Lichtman
Brian Terlson
@csigs
Dan Rollo
Daniel Imms
Daniel Rosenwasser
David Sherret
Elizabeth Dinella
Frederico Bittencourt
Jack Williams
James Keane
Jordi Oliveras Rovira
Klaus Meinhardt
Martin Probst
Matt McCutchen
Mine Starks
Minh Nguyen
Nathan Day
Mohamed Hegazy
Nathan Shively-Sanders
Ron Buckton
Rhys van der Waerden
Ryan Cavanaugh
Sam Lanning
Sheetal Nandi
Wenlu Wang
Wesley Wigham
Tim Schaub

v3.0.3
Compare Source
v3.0.1
Compare Source
For release notes, check out the release announcement
For new features, check out the What's new in TypeScript 3.0.
For breaking changes, check out the Breaking changes in TypeScript 3.0 page.
For the complete list of fixed issues, check out the issues fixed in TypeScript 3.0 RC and after TypeScript 3.0 RC.
Download:

npm
Visual Studio 2015 plugin
Visual Studio 2017 (Select new version in project options)
NuGet package

Special thanks to all contributors to this release:

Alexander Tarasyuk
Anders Hejlsberg
Andrew Casey
Andy Hanson
Armando Aguirre
Benjamin Lichtman
Dan Freeman
Daniel Gooss
Daniel Rosenwasser
David Sherret
David Staheli
Elizabeth Dinella
Eugene Timokhov
John Doe
Jordi Oliveras Rovira
Kerem Kat
Kevin Gibbons
Kitson Kelly
Klaus Meinhardt
Markus Johnsson
Martin Probst
Mateusz Burzyński
Matt McCutchen
Micah Zoltu
Mohamed Hegazy
Nathan Shively-Sanders
Nicu Micleușanu
Pi Lanningham
@rflorian
Ron Buckton
Ryan Cavanaugh
Sam Bostock
Sheetal Nandi
@styfle
Vimal Raghubir
Vyacheslav Pukhanov
Wenlu Wang
Wesley Wigham
@Zzzen

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Disabled by config. Please merge this manually once you are satisfied.
:recycle: Rebasing: Whenever PR becomes conflicted, or if you modify the PR title to begin with "rebase!".
:ghost: Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

This PR has been generated by Renovate Bot.

PR has been edited
:construction_worker: This PR has received other commits, so Renovate will stop updating it to avoid conflicts or other problems. If you wish to abandon your changes and have Renovate start over you may click the "rebase" checkbox in the PR body/description.


How many A100 are required for fine-tuning？

Hi @alibaba-oss @hitsz-zuoqi ，thanks for your fantastic work.
I wanna inquire that How many A100s are required for fine-tuning？

text2mv requires 1-4 gpus，while image2mv requires 8 gpus


which PyTorch version needed to make this build work? please make a clean environment.yaml so we can install it.

Hi, so I spent 3 days and I could pass the below error, I assume the requirements has conflict, so please fix it.

making attention of type 'vanilla' with 512 in_channels
Traceback (most recent call last):
  File "P:\ai\stable-diffusion-main\scripts\gradio_variations.py", line 143, in <module>
    fire.Fire(run_demo)
  File "C:\ProgramData\Miniconda3\envs\dsd\lib\site-packages\fire\core.py", line 141, in Fire
    component_trace = _Fire(component, args, parsed_flag_args, context, name)
  File "C:\ProgramData\Miniconda3\envs\dsd\lib\site-packages\fire\core.py", line 466, in _Fire
    component, remaining_args = _CallAndUpdateTrace(
  File "C:\ProgramData\Miniconda3\envs\dsd\lib\site-packages\fire\core.py", line 681, in _CallAndUpdateTrace
    component = fn(*varargs, **kwargs)
  File "P:\ai\stable-diffusion-main\scripts\gradio_variations.py", line 110, in run_demo
    model = load_model_from_config(config, ckpt, device=device)
  File "p:\ai\stable-diffusion-main\scripts\image_variations.py", line 26, in load_model_from_config
    model = instantiate_from_config(config.model)
  File "p:\ai\stable-diffusion-main\ldm\util.py", line 79, in instantiate_from_config
    return get_obj_from_str(config["target"])(**config.get("params", dict()))
  File "p:\ai\stable-diffusion-main\ldm\models\diffusion\ddpm.py", line 523, in __init__
    self.instantiate_cond_stage(cond_stage_config)
  File "p:\ai\stable-diffusion-main\ldm\models\diffusion\ddpm.py", line 581, in instantiate_cond_stage
    model = instantiate_from_config(config)
  File "p:\ai\stable-diffusion-main\ldm\util.py", line 79, in instantiate_from_config
    return get_obj_from_str(config["target"])(**config.get("params", dict()))
  File "p:\ai\stable-diffusion-main\ldm\modules\encoders\modules.py", line 209, in __init__
    self.freeze()
  File "p:\ai\stable-diffusion-main\ldm\modules\encoders\modules.py", line 212, in freeze
    self.transformer = self.transformer.eval()
  File "C:\ProgramData\Miniconda3\envs\dsd\lib\site-packages\torch\nn\modules\module.py", line 1185, in __getattr__
    raise AttributeError("'{}' object has no attribute '{}'".format(
AttributeError: 'FrozenCLIPImageEmbedder' object has no attribute 'transformer'

The environment.yaml was inherited from the base repo, I've deleted it. I'll update the requirements, but for the moment see:
https://github.com/justinpinkney/stable-diffusion/issues/15#issuecomment-1255932991

The environment.yaml was inherited from the base repo, I've deleted it. I'll update the requirements, but for the moment see:
#15 (comment)

Thanks, would you please mention the python version on your side too so we can setup the conda env?

I'm so bummed, yet another thing I try doing in Linux, dead-ended face-first into a cryptic error message (exactly as above), at the mercy of the developer as the only person that can decode the cryptic string of related packages that fail to mesh together.
It's a perfectly clean install of Ubuntu, having just brought up Torch to a point it can actually function (involved removing the default version of PyTorch and installing a CUDA 11.6 version of it).
Simply editing environment.yaml (as above) had no effect - pretty sure that info is read earlier in the setup process, and editing it after setup won't change/fix anything. My versions match those, though (torch-1.12.1+cu116, torchvision-0.13.1+cu116, and CUDA 11.7 - but I'm guessing newer is better? How would I even downgrade that?).

@FalconFour @mhnoni All your talk of requirements issues totally threw me off the scent. This was just a plain old bug, has nothing to do with requirements of anything. Was from the freeze method I'd added during some other work. Hopefully this works again now.

Thanks a lot!


Hub seems to be off line for some

I seem to be having a strange problem my hub has been very quite and i have just found out that its showing offline in beatsaber for some users. I have had users connect today just not a lot is there a reason it would show offline for some and not others. On the hub site its showing online and the server is running

There is a bug that causes the client not to disconnect after receiving the list of rooms, and this, coupled with the max connections limit (32 by default), leads to this behavior

so if i restart the hub will that clear the connections? because i have done that a few times but still i see very few connections

so if i restart the hub will that clear the connections?
Yeah, that should help


i18n: message catalogue update and clean-up

FIX Updates PO message catalogues and cleans them of duplicated
messages.  (closes #3455)

Adds setup.py Babel friendly statements.

Signed-off-by: Tibor Simko<EMAIL_ADDRESS>

:shipit:


change vertical coordinate from 'height' to 'z'

Fixes #1

I'm more comfortable allowing the user to pass in the Z axis name they would like to use rather than the hardcoded "height" value right now.  I set it up for that but never got around to adding the optional parameter because I didn't need it (COARDS was never the target compliance).


There are no DynamoDB resources configured in your project currently

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
No response
Amplify CLI Version
7.6.5
What operating system are you using?
Windows
Amplify Categories
function
Amplify Commands
add, pull
Describe the bug
Using Amplify Studio I created a model, then "saved and deployed" successfully. Then I ran "amplify pull", but when I try to create a function "amplify function add" I get the error "There are no DynamoDB resources configured in your project currently"
Maybe I am missing something, but when I deploy and then pull, shouldn't I have the updated ddb model in my project?
Expected behavior
I expect DynamoDB resources configured in my project.
Reproduction steps

Create model in Amplify Studio
amplify pull
amplify function add
get error "There are no DynamoDB resources configured in your project currently"

GraphQL schema(s)

# Put schemas below this line

Log output

# Put your logs below this line

Additional information
No response

@flgatormike Did you answer 'yes'  Do you plan on modifying this backend??

Closing due to a lack of followup, but we can revisit this issue if needed.


Upgrade i18n

Upgrade LinguiJS to v3 and clean up code accordingly.
Next.js i18n routing in a follow-up PR

@lucguillemot I've merged the latest commit of master into this, there were no conflicts except in the PO files which should be up-to-date now. Please review :)


feat(ingestion/tableau): optionally ingest multiple sites and create site containers

Description
With this PR, the Tableau ingestion can be configured to ingest multiple sites at once and to add the sites as containers. We have around 26 sites in the company, and it would be pretty cumbersome to manage 26 ingestions separately. Additionally, because we have so many sites and projects, we need the sites to show up as containers in Datahub.
To achieve this I introduced some new optional config properties: ingest_multiple_sites, add_site_container and site_name_pattern. All existing recipes should continue to work as-is.
A recipe could look something like this:
source:
  type: tableau
  config:
    # Coordinates
    connect_uri: https://tableau1-test.swisscom.com
    ingest_multiple_sites: True
    add_site_container: True
    site_name_pattern:
      allow: ["^Data.*$"]
    platform_instance: tableau_test

    # Credentials
    username: "${TABLEAU_USER}"
    password: "${TABLEAU_PASSWORD}"

sink:
  type: "datahub-rest"
  config:
    server: "http://localhost:8080"

And that's how it looks like after ingesting:

I already saw this asked a few times in the Slack channel as well:

https://datahubspace.slack.com/archives/CUMUWQU66/p1708101345375149
https://datahubspace.slack.com/archives/CUMUWQU66/p1699450650344249
https://datahubspace.slack.com/archives/C02GBGG90CU/p1688994637949829

Please let me know what you think and I will proceed with adding some documentation and tests to this.
Many thanks.
Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

I tried to fix the tests but for some reason when I run pytest tests/integration/tableau/test_tableau_ingest.py --update-golden-files most of the golden files get refactored to an empty list. Any idea what I'm doing wrong?

Hi @hsheth2
I think I now resolved all the open points on the PR.
As discussed offline, we keep the config properties and default values as-is and make it clear in the docs that this feature is currently only available for Tableau Server. This is because some API endpoints we use are currently not available for Tableau Cloud.
Also see my last comments regarding the refactoring and and some fixes.
What's currently still missing is tests and documentation. I might need some guidance with the tests, but I'll look into it as soon as you agree on the changes.
Please review and test the PR thoroughly again and let me know what you think.
Many thanks.


css-loader enable module generate error local name

System Info
codesandbox
Details
output name use contenthash, and when i set localIdentName: "[path][name]__[local]--[hash:base64:5]",, only local var  be used
___CSS_LOADER_EXPORT___.push([
    module.id,
    ".\\[contenthash\\] {\n    display: flex;\n}",
    ""
]);
___CSS_LOADER_EXPORT___.locals = {
    "box": "[contenthash]"
};

Reproduce link
https://codesandbox.io/p/sandbox/happy-frog-mkzmts?selection=[{"endColumn"%3A3%2C"endLineNumber"%3A816%2C"startColumn"%3A1%2C"startLineNumber"%3A809}]&file=%2Fdist%2Fmain%2Fmain.js
Reproduce Steps

pnpm i
node build.js

related to https://github.com/web-infra-dev/rspack/issues/3071

#3088 won't land in 0.1.12, but #3071 will, so localIdentName: "[path][name]__[local]--[xxhash64:hash:base64:5]", should works in 0.1.12, localIdentName: "[path][name]__[local]--[hash:base64:5]", should also works after #3088 is resolved

Could you provide a reproduction about the [internal]: Base64Error. error, so I can dig into this.

Could you provide a reproduction about the [internal]: Base64Error. error, so I can dig into this.

@ahabhgk https://codesandbox.io/p/sandbox/rspack-swiper6-forked-nhlly9?file=%2Fbuild.js%3A22%2C15
pnpm i
node build.js

cause by  path /src/components/TestA/index.tsx code import "swiper/swiper.min.css";


Getting failed to get k8s object, cause: failed to get k8s resulting object error cause: apiservices.apiregistration.k8s.io "v1beta1.external.metrics.k8s.io" not found

Description
Hi there,
We have deployed the "Helm Dashboard" on GCP's Cloud Run Service, and we are connecting to our Auto-Pilot GKE Cluster
through it, Dashboard is getting connected successfully, able to see the list of Helm Charts, but though the cloud run configurations like CPU and Memory are not getting utilised much still facing performance issue, and when tried to open any of the Helm Charts getting below error message:-
"failed to get k8s object, cause: failed to get k8s resulting object, cause: apiservices.apiregistration.k8s.io "v1beta1.external.metrics.k8s.io" not found"
Screenshots

Additional information
No response

Hello,
From your explanation, I did not understood what is "CPU and Memory are not getting utilised much" and how it relates to Helm Dashboard. Helm Dashboard has nothing to do with monitoring.
As for the error message, looks like some resource in your chart causes problems with retrieving its details from k8s. Current code of dashboard fails to display all the resources when one failed to retrieve the info. So I'll make it to say that gracefully. It will become a part of next release. Then, you will be able to tell resource is problematic.

HI Andrey,
Thanks for you quick response,
About the "cpu and memory utilization":- I mean to say as I have hosted the
container of "Helm Dashboard" tool over GCP's Cloud Run, so that hosted
container's resource utilization is not that much, but still "Helm
Dashboard" is taking too much time to load the details of my microservice's
Helm chart which are on an autopilot GKE cluster.
So just wanted to know the reason behind performance of "Helm Dashboard"
tool, or is it due to large numbers of Helm Charts that I am having on my
GKE cluster (Currently there are 387 helm charts)
It would be great if you can address this issue as soon as possible, and I
will be thankful to you.
Regards,
On Fri, Mar 24, 2023 at 11:07 PM Andrey Pokhilko @.***>
wrote:

Hello,
From your explanation, I did not understood what is "CPU and Memory are
not getting utilised much" and how it relates to Helm Dashboard. Helm
Dashboard has nothing to do with monitoring.
As for the error message, looks like some resource in your chart causes
problems with retrieving its details from k8s. Current code of dashboard
fails to display all the resources when one failed to retrieve the info. So
I'll make it to say that gracefully. It will become a part of next release.
Then, you will be able to tell resource is problematic.
—
Reply to this email directly, view it on GitHub
https://github.com/komodorio/helm-dashboard/issues/301#issuecomment-1483176946,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5KIE3RIUIZCH5TNWHRX7U3W5XLVDANCNFSM6AAAAAAWGIZBQE
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
[image: Profile Photo]
Aniket Bhatagalikar
Senior Cloud Reliability Engineer
M: +91-8087818134
[image: Searce Logo]
https://www.searce.com/home/?utm_source=email&utm_medium=emailsignature
We identify better ways of doing things
[image: LinkedIn] https://www.linkedin.com/company/searceinc/ [image:
Facebook] https://www.facebook.com/Searce.Inc/ [image: Twitter]
https://twitter.com/searce
The information contained in this communication is intended solely for the
use of the individual or entity to whom it is addressed and others
authorized to receive it. It may contain confidential or legally privileged
information. If you are not the intended recipient you are hereby notified
that any disclosure, copying, distribution or taking any action in reliance
on the contents of this information is strictly prohibited and may be
unlawful. If you have received this communication in error, please notify
us immediately by responding to this email and then delete it from your
system. Searce is neither liable for the proper and complete transmission
of the information contained in this communication nor for any delay in its
receipt.

It's hard to diagnose the exact reason for slowness. It would depend on performance of each request to Cloud Run, and also on the amount of the charts you have.
One of the workarounds is to limit the scope of namespaces via -n argument.

It's hard to diagnose the exact reason for slowness. It would depend on performance of each request to Cloud Run, and also on the amount of the charts you have.
One of the workarounds is to limit the scope of namespaces via -n argument.

@undera is it possible to analyse this somehow on our end?
How do you debug UX in the app?

The application is simple backend and JS frontend. You can open Chrome Developer tools' Network tab and investigate the timings of various XHR requests there. The one the would look the bulkiest - is likely to be a problem. Or there will be a lot of small requests in sequence that take time to load.


 Moved to raising an assertion and using imap_unordered() to catch the assertion and terminate the pool of child processes when there's an error
https://docs.python.org/3.3/library/multiprocessing.html#multiprocessing.pool.Pool.imap_unordered
Fixes #20
Moved dropping empty exon column in gene level table from report to stats script - Fixes #26

This change is

10/10 PR title


Update Ruby version to 3.3 for check-links.yml

sass-embedded requires ruby version >= 3.2.

check-links.yml works with Ruby 3.3 : https://github.com/endoflife-date/endoflife.date/actions/runs/9538773989/job/26288362279


"Error while loading rule" error

Describe the bug
I have VSCode set to format on save. I am getting an error in the output console when saving, and my file is not formatted.
Error: Error while loading rule 'react/no-did-update-set-state': Cannot read properties of undefined (reading 'uid')
Prior to today, the plugin formatted exactly how I wanted. I believe this has to do with the 1.79 update to VSCode, as I restarted today and received the auto update to 1.79. It seems to have broken Prettier. https://github.com/prettier/prettier-vscode/issues/3019 Documented in this issue, the Prettier plugin has a setting that will allow a workaround, by setting "prettier.prettierPath": "./node_modules/prettier". But the VSCode Prettier ESLint plugin does not seem to respect that setting.
To Reproduce
Steps to reproduce the behavior:

Update VSCode to version 1.79
Attempt to format a file using Prettier ESLint
See error in output console

Expected behavior
File is formatted using ESLint and Prettier.
Versions:

VS Code Prettier ESLint extension: v9.13.0
Visual Studio Code: 1.79.1

@jackolas-lock Could you please downgrade for now? The fix you described for vscode-prettier isn't working for everyone.

Yes, I have downgraded for now and the plugin works again. The issue I linked and the fix within only works for the Prettier plugin, it does not look like I can ask this plugin to use that path setting. I just linked for your reference.
Thank you for this plugin, it is very helpful!


include social media icon links in footer, remove few buttons from community resource page

Added social media and community resources' icons to the footer, and removed the operaterhub.io and rook.io buttons from the community resources page.

/assign @codificat

Fixes #193

Fixes #193
cc @fmuyassarov
I have added several icons to the footer including a github icon pointing to metal3-io github project page. Do you think its sufficient or should I add another explicit link/button to the main page?

Thanks @hroyrh , logos look nice.  I think the current state is good enough.
What do you think about making those logos little bit bigger or align to the center?

@fmuyassarov I have increased the icon size, let me know how it looks.

Thanks @hroyrh. It looks good me.
/lgtm

It looks like the travis CI job needs to be fixed first.

It looks like the travis CI job needs to be fixed first.

/retest

/retest

/ok-to-test
/retest

/ok-to-test
/retest

@fmuyassarov do I need to rebase this PR to add the ci fix or are we good to go

@fmuyassarov do I need to rebase this PR to add the ci fix or are we good to go

@fmuyassarov do I need to rebase this PR to add the ci fix or are we good to go

I think we need a rebase, because I assume we won't able to merge it even with lgtm+approve until all the CI checks passed.

@fmuyassarov do I need to rebase this PR to add the ci fix or are we good to go

I think we need a rebase, because I assume we won't able to merge it even with lgtm+approve until all the CI checks passed.

@fmuyassarov do I need to rebase this PR to add the ci fix or are we good to go

I think we need a rebase, because I assume we won't able to merge it even with lgtm+approve until all the CI checks passed.

Or you can just push empty commit with --amend.

@fmuyassarov do I need to rebase this PR to add the ci fix or are we good to go

I think we need a rebase, because I assume we won't able to merge it even with lgtm+approve until all the CI checks passed.

Or you can just push empty commit with --amend.

@hroyrh if you make your membership in the metal3-io GitHub org public, the bot will respond to your instructions. Right now it can't tell if you're a trusted person because it does not have admin rights to see the private org membership list. https://docs.github.com/en/free-pro-team@latest/github/setting-up-and-managing-your-github-user-account/publicizing-or-hiding-organization-membership

@hroyrh if you make your membership in the metal3-io GitHub org public, the bot will respond to your instructions. Right now it can't tell if you're a trusted person because it does not have admin rights to see the private org membership list. https://docs.github.com/en/free-pro-team@latest/github/setting-up-and-managing-your-github-user-account/publicizing-or-hiding-organization-membership

@dhellmann thanks! can you also take some time to see if this PR looks good

@dhellmann thanks! can you also take some time to see if this PR looks good

/approve

/approve

/lgtm

/lgtm


Updating for "Security vulnerability in WebRTC"

Regarding: https://groups.google.com/forum/#!topic/discuss-webrtc/pfwS9eeS23c, I see that the WebRTC 58 work (https://github.com/oney/react-native-webrtc/pull/249) has been updated so this should not be a concern for future builds of react-native-webrtc.
I have two questions regarding current react-native-webrtc releases:

Does the https://github.com/oney/react-native-webrtc/tree/rn-less-40 branch need to be updated or removed?
If the WebRTC 58 work still has some QA time before being merged in master and released on npm, is there a need to update the current master build / npm release?

I have not asked privately for the information on the WebRTC vulnerability, so I wanted to check if any of the contributors / maintainers of react-native-webrtc have and knew the answers on those questions?

Closing, I'm not sure at this point that my questions are relevant and don't want to cause alarm to people looking through the issues on this repo.


RYA-392-Datetime-Within

Description
Added a Function DateTimeWithinPeriod to determine whether two datetimes were within a
specified period of time of one another.
Tests
Added unit tests to test the Function and added integration tests to verify that the Function
is correctly utilized during query evaluation.
Links
Jira
Checklist

[ ] Code Review
[x] Squash Commits

People To Reivew
@jdasch @kchilton2

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-rya-master-with-optionals-pull-requests/512/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-rya-master-with-optionals-pull-requests/513/Failed Tests: 1incubator-rya-master-with-optionals-pull-requests/org.apache.rya:rya.indexing: 1org.apache.rya.indexing.mongo.MongoEntityIndexTest.org.apache.rya.indexing.mongo.MongoEntityIndexTest

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-rya-master-with-optionals-pull-requests/517/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-rya-master-with-optionals-pull-requests/518/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-rya-master-with-optionals-pull-requests/519/


Template system

Implemented ability to modify colors in the template file, using functions in util.py.
Currently, there is only support for the lighten_color, and darken_color function. To use them, use a substitution marker like this in your template:
{background.darken_color(20)}
Unfortunately, to keep backwards compatibility, you can not use float values as a function argument. Instead, you pass a number between 0 and 100 (or -0 and -100), that represents a percentage.

Please consider using a tried and true template system like Jinja2 which is extremely capable and would serve us well.


Update porch dev setup Readme

Add entry to setup APIServer config for linux env.
Some clean up

@efiacor I read through this now, it looks fine but let me run through it on a MacOS host tomorrow before approving.

tested the porch-dev-environment setup tutorial change by ssh'ing vscode into a linux vm. seems to correctly create the packages and connect to the api now.

/lgtm
/approve


impl(bigtable): support bigtable cookie in ReadRow(s)

Part of the work for #13447
This thing is like our StreamRange class, for bigtable rows. Before we create the stream, call RetryContext::PreCall(...). When the stream finishes (by returning a variant holding a Status), call RetryContext::PostCall(...).
Given the structure of the code, we need to hold the grpc::ClientContext in the class. Delete it at our earliest convenience.

This change is

Then use reset() instead?

+1 to unique_ptr::reset

Done.


LISTEN.moe app crashes when in background with phone locked

Please describe the problem you are having in as much detail as possible:
The app crashes when I minimize it and lock my phone.
Further details:
Logcat:
[ 05-23 14:29:39.367  1664:11047 W/ActivityManager ]
Foreground service started from background can not have location/camera/microphone access: service me.echeung.moemoekyun/.service.AppService

[ 05-23 14:29:39.368  1664:11047 W/ActivityManager ]
Background started FGS: Disallowed [callingPackage: me.echeung.moemoekyun; callingUid: 10274; uidState: CAC ; uidBFSL: n/a; intent: Intent { cmp=me.echeung.moemoekyun/.service.AppService }; code:DENIED; tempAllowListReason:<null>; targetSdkVersion:34; callerTargetSdkVersion:34; startForegroundCount:1; bindFromPackage:null: isBindService:false]

[ 05-23 14:29:39.368  1664:11047 W/ActivityManager ]
Service.startForeground() not allowed due to mAllowStartForeground false: service me.echeung.moemoekyun/.service.AppService

[ 05-23 14:29:39.395 29792:29857 W/AudioTrack ]
getTimestamp_l(1114): retrograde timestamp time corrected,<PHONE_NUMBER>7542 <<PHONE_NUMBER>3427

[ 05-23 14:29:39.404 29792:29828 E/AndroidRuntime ]
FATAL EXCEPTION: DefaultDispatcher-worker-2
Process: me.echeung.moemoekyun, PID: 29792
android.app.ForegroundServiceStartNotAllowedException: Service.startForeground() not allowed due to mAllowStartForeground false: service me.echeung.moemoekyun/.service.AppService
	at android.app.ForegroundServiceStartNotAllowedException$1.createFromParcel(ForegroundServiceStartNotAllowedException.java:54)
	at android.app.ForegroundServiceStartNotAllowedException$1.createFromParcel(ForegroundServiceStartNotAllowedException.java:50)
	at android.os.Parcel.readParcelableInternal(Parcel.java:5015)
	at android.os.Parcel.readParcelable(Parcel.java:4997)
	at android.os.Parcel.createExceptionOrNull(Parcel.java:3177)
	at android.os.Parcel.createException(Parcel.java:3166)
	at android.os.Parcel.readException(Parcel.java:3149)
	at android.os.Parcel.readException(Parcel.java:3091)
	at android.app.IActivityManager$Stub$Proxy.setServiceForeground(IActivityManager.java:6861)
	at android.app.Service.startForeground(Service.java:775)
	at me.echeung.moemoekyun.service.MusicNotifier.update(Unknown Source:270)
	at me.echeung.moemoekyun.service.AppService.update(Unknown Source:31)
	at me.echeung.moemoekyun.service.AppService.access$update(Unknown Source:0)
	at me.echeung.moemoekyun.service.AppService$onCreate$1.invokeSuspend$update(Unknown Source:0)
	at me.echeung.moemoekyun.service.AppService$onCreate$1.access$invokeSuspend$update(Unknown Source:0)
	at me.echeung.moemoekyun.service.AppService$onCreate$1$2.invoke(SourceFile:0)
	at me.echeung.moemoekyun.service.AppService$onCreate$1$2.invoke(SourceFile:0)
	at kotlinx.coroutines.flow.FlowKt__MergeKt$mapLatest$1.invokeSuspend(Unknown Source:50)
	at kotlinx.coroutines.flow.FlowKt__MergeKt$mapLatest$1.invoke(SourceFile:0)
	at kotlinx.coroutines.flow.FlowKt__MergeKt$mapLatest$1.invoke(SourceFile:0)
	at kotlinx.coroutines.flow.internal.ChannelFlowTransformLatest$flowCollect$3$1$2.invokeSuspend(Unknown Source:38)
	at kotlinx.coroutines.flow.internal.ChannelFlowTransformLatest$flowCollect$3$1$2.invoke(SourceFile:0)
	at kotlinx.coroutines.flow.internal.ChannelFlowTransformLatest$flowCollect$3$1$2.invoke(SourceFile:0)
	at kotlinx.coroutines.intrinsics.UndispatchedKt.startCoroutineUndispatched(Unknown Source:20)
	at kotlinx.coroutines.CoroutineStart.invoke(Unknown Source:27)
	at kotlinx.coroutines.AbstractCoroutine.start(Unknown Source:0)
	at kotlinx.coroutines.BuildersKt__Builders_commonKt.launch(Unknown Source:22)
	at kotlinx.coroutines.BuildersKt.launch(Unknown Source:0)
	at kotlinx.coroutines.BuildersKt__Builders_commonKt.launch$default(Unknown Source:12)
	at kotlinx.coroutines.BuildersKt.launch$default(Unknown Source:0)
	at kotlinx.coroutines.flow.internal.ChannelFlowTransformLatest$flowCollect$3$1.emit(Unknown Source:112)
	at kotlinx.coroutines.flow.FlowKt__ZipKt$combine$$inlined$combineUnsafe$FlowKt__ZipKt$2$2.invokeSuspend(Unknown Source:83)
	at kotlinx.coroutines.flow.FlowKt__ZipKt$combine$$inlined$combineUnsafe$FlowKt__ZipKt$2$2.invoke(SourceFile:0)
	at kotlinx.coroutines.flow.FlowKt__ZipKt$combine$$inlined$combineUnsafe$FlowKt__ZipKt$2$2.invoke(SourceFile:0)
	at kotlinx.coroutines.flow.internal.CombineKt$combineInternal$2.invokeSuspend(Unknown Source:295)
	at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(Unknown Source:11)
	at kotlinx.coroutines.DispatchedTask.run(Unknown Source:93)
	at kotlinx.coroutines.internal.LimitedDispatcher$Worker.run(Unknown Source:3)
	at kotlinx.coroutines.scheduling.TaskImpl.run(Unknown Source:2)
	at kotlinx.coroutines.scheduling.CoroutineScheduler.runSafely(Unknown Source:0)
	at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.executeTask(Unknown Source:14)
	at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.runWorker(Unknown Source:28)
	at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.run(Unknown Source:0)
	Suppressed: kotlinx.coroutines.internal.DiagnosticCoroutineContextException: [StandaloneCoroutine{Cancelling}@dde82d9, Dispatchers.IO]

[ 05-23 14:29:39.413  1664:11047 W/ActivityTaskManager ]
  Force finishing activity me.echeung.moemoekyun/.ui.MainActivity

[ 05-23 14:29:39.435  1664:11047 W/UserManagerService ]
Requested status bar icon for non-badged user 0

[ 05-23 14:29:39.534  1335: 7463 E/mediaserver ]
unlinkToDeath: removed reference to death recipient but unlink failed: DEAD_OBJECT

[ 05-23 14:29:39.535  1335: 7463 E/IPCThreadState ]
attemptIncStrongHandle(18): Not supported

[ 05-23 14:29:39.535 25359:25359 W/AudioMediaPlayerWrapper ]
The session was destroyed me.echeung.moemoekyun

[ 05-23 14:29:39.536  1664: 3454 W/InputManager-JNI ]
Input channel object 'f87c1af me.echeung.moemoekyun/me.echeung.moemoekyun.ui.MainActivity (client)' was disposed without first being removed with the input manager!

[ 05-23 14:29:39.539 25359:25386 W/bt_stack ]
[WARNING:device.cc(1720)] Device is not registered for now playing updates

[ 05-23 14:29:39.539 25359:25386 W/bt_stack ]
[WARNING:device.cc(1817)] Device is not registered for play position updates

[ 05-23 14:29:39.539 25359:25386 W/bt_stack ]
[WARNING:device.cc(1695)] Device is not registered for track changed updates

[ 05-23 14:29:39.553  1664: 1800 W/ActivityManager ]
setHasOverlayUi called on unknown pid: 29792

[ 05-23 14:29:39.554  1664: 1823 W/UsageStatsService ]
Unexpected activity event reported! (me.echeung.moemoekyun/me.echeung.moemoekyun.ui.MainActivity event : 23 instanceId : 107453097)

App version: 6.0.2
Android version: Upside Down Cake (14)
Device: Samsung Galaxy S20 FE 4G (r8q) (LineageOS 21 official)
Priority this should have – please be realistic and elaborate if possible: medium

Are you trying to play and then immediately backgrounding the app?

Are you trying to play and then immediately backgrounding the app?

I always wait for the song to start playing, or it doesn't start at all and the play button does nothing.

Does it still crash with v6.1.0?


Don't allow partitioned tables to have partitions with more than one versions created

Summary of the changes / Why this improves CrateDB
Fixes:
-- on 5.7.5:
cr> create table tbl (id int, o object(strict) as (a array(int))) partitioned by (id);
cr> insert into tbl values (1, {a=[1]});
cr> insert into tbl values (2, {a=[1, 2]});

-- upgraded to latest:
cr> ALTER TABLE tbl SET (number_of_replicas = 1); -- can be any alter stmts that modified the settings
cr> select version from information_schema.tables where table_name = 'tbl';
+-----------------------------------------+
| version                                 |
+-----------------------------------------+
| {"created": "5.10.0", "upgraded": null} |  -- should be 5.7.5
+-----------------------------------------+

cr> insert into tbl values (4, {a=[1,2,3]});
cr> select version from information_schema.table_partitions where table_name = 'tbl';
+--------------------------------------------+
| version                                    |
+--------------------------------------------+
| {"created": "5.10.0"}                      |  -- this should be 5.7.5
| {"created": "5.7.5", "upgraded": "5.10.0"} |
| {"created": "5.7.5", "upgraded": "5.10.0"} |
+--------------------------------------------+

alter stmt that alters settings removed version.created and then it was set to the latest version along the way.
Non-partitioned tables are not affected.
Checklist

[ ] Added an entry in the latest docs/appendices/release-notes/<x.y.0>.rst for user facing changes
[ ] Updated documentation & sql_features table for user facing changes
[ ] Touched code is covered by tests
[ ] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in the latest docs/appendices/release-notes/<x.y.0>.rst
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

The partitioned tables with multiple versions created caused bugs like https://github.com/crate/crate/issues/17150.

The partitioned tables with multiple versions created caused bugs like #17150.

It exposed such bug with removed settings but I think wrong version_created is a standalone issue.
Most likely this PR can fix/mask 17150 and such we can revert https://github.com/crate/crate/pull/17175 but as mentioned in https://github.com/crate/crate/issues/17150#issuecomment-2560942089, there are other things in metadata other then potentially removed settings, so we need address "outdated structure" with upgrade.
However, "version_created after ALTE RTABLE on partitioned tables" of course needs to be fixed

The partitioned tables with multiple versions created caused bugs like #17150.

It exposed such bug with removed settings but I think wrong version_created is a standalone issue.
Most likely this PR can fix/mask 17150 and such we can revert #17175 but as mentioned in #17150 (comment), there are other things in metadata other then potentially removed settings, so we need address "outdated structure" with upgrade.
However, "version_created after ALTE RTABLE on partitioned tables" of course needs to be fixed

Sorry had wrong link initially, fixed.


WIP: Fix deepcopy-ing of CFFs

Deepcopying a TTFont with a CFF table can result in an infinite recursion on Python 3 under certain circumstances.
@anthrotype found that CFFFontSet keeps a reference to the TTFont object that holds it to get at the glyph order at some point: https://github.com/fonttools/fonttools/blob/master/Lib/fontTools/cffLib/init.py#L34. Maybe this messes with deepcopy.

I was checking as well during my flight, but I don’t think it’s that. It has to do with the custom BaseDict.__getattr__ method, it seems to not play nicely with deepcopy. I need to study the code a bit more

Yeah, implementing
	def __deepcopy__(self, memo):
		from copy import deepcopy

		cls = self.__class__
		obj = cls.__new__(cls)
		for k, v in self.__dict__.items():
			if k == "otFont":
				obj.otFont = None
			else:
				obj.__dict__[k] = deepcopy(v, memo)
		return obj

on CFFFontSet doesn't help.

Stepping through this, it seems that BaseDict.__getattr__ is getting repeatedly called with name="rawDict", which executes self.rawDict.get(name, None) (rawDict being a PrivateDict), which in it's __getattr__ calls BaseDict.__getattr__, i.e. loops back.

Actually, there's infinite recursion even when I remove the __getattr__ on PrivateDict. Stopping on the first line of BaseDict's __getattr__ when name is rawDict, even accessing self.rawDict in the debugger causes infinite recursion.

Maybe because self.rawDict doesn't exist and as such triggers calling __getattr__ on itself, which we overrode to call ourselves.

I have no internet on my laptop so I just sent you a photo of a git patch via WhatsApp LOL 😂

We need to implement custom __getstate__ and __setstate__ for BaseDict so that both deepcopy and pickle work. I can pick up from here

fixed on master with https://github.com/fonttools/fonttools/commit/649dc49dbaf056545a117278190a9586d3eb9af5


Show open user projects too when displaying open projects

As title
Before:

After:

fix #26189

Alternatively, we can also combine projects2 and projects3 by removing the Type condition.
That's probably a better idea anyways, as there are more types than that.

That's probably a better idea anyways, as there are more types than that.

It may be that my previous code wasn't clear enough, leading you to think it should be changed to this title:Show open user projects too when displaying open projects.
I'm actually trying to fix https://github.com/go-gitea/gitea/issues/26189, The title of this should be：Fix unable to display individual-level project

The purpose of this PR is to fix Bug #26189. I have verified that this bug also exists in previous versions, so the PR should be backported to those earlier versions.


Modify activemq template to support subcollectives

The ActiveMQ XML template only supports the mcollective collective. Multiple subcollectives are not included. In de page http://docs.puppetlabs.com/mcollective/deploy/middleware/activemq.html#authorization-group-permissions there is a sample for how to adjust the template for subcollective support

given that we a) don't support middleware anymore, but b) support subcollectives, i'd say this is now fixed.


Sorting error

Click sort, unable to sort the sort field, click no response

System information:

Operating system (distribution) and version
DBeaver version
Additional extensions

Connection specification:

Database name and version
Driver name
Do you use tunnels or proxies (SSH, SOCKS, etc)?

Describe the problem you're observing:
Steps to reproduce, if exist:
Include any warning/errors/backtraces from the logs

Can't reproduce.
Please provide more details (what database, some screenshot would be very helpful)

无法重现。
请提供更多详细信息（什么数据库，一些截图将非常有帮助）
HIVE,If I press 1, then I press 2, I can't sort by 2, or sort at the same time

You have 3 columns included in ordering. If you want to order only by province_code then you need to remove ordering from two other columns.
You can do this by clicking on sort icon again or by opening custom filters dialog.

订购时包含3列。如果您只想通过province_code订购，那么您需要从其他两列中删除订购。
您可以通过再次单击排序图标或打开自定义过滤器对话框来执行此操作。

Can I just click sort without unclicking and click sort?

Yes. Press CTRL key when clicking on sort icon.


Frame in temp folder being used by another process + broken pipe

Trying to use the program results in this:

Running Python 3.9.12 btw

same issue


Gimme.mom

Entry type

New addition

App info
URL: gimme.mom
Manifest.json URL: https://gimme.mom/manifest.json
Name: Gimme
Description: A very simple and UX focused way to deposit tokens to earn Yield on them. Built on Yearn's vault
Icon (PNG, 180x180): https://gimme.mom/favicons/apple-icon-180x180.png
It's minified via https://tinypng.com: no
Homepage: https://gimme.mom
Twitter: https://twitter.com/built_by_mom
GitHub: https://github.com/BuiltByMom/gimme
Discord: N/A
App supports batching multiple transactions via Safe: yes
Supported networks

- Polygon
- _We expect to release a multichain version in the next two weeks_

Revision checks

[X] Used smart contracts were audited.
[X] You have implemented the app using the Safe Apps SDK
[X] Your Safe App includes a manifest.json file at the root with the required data
[X] The app can be loaded as a custom Safe App in the Apps section of https://app.safe.global.
[X] The app auto-connects to the Safe as a wallet
[X] It doesn't try to connect to the browser wallet (e.g. MetaMask)
[X] You are able to trigger and execute one transaction with a Safe.
[X] RPC requests are optimized (not triggering many requests in a very short time period).

Audit document

https://docs.yearn.finance/security/#third-party-audits
Code for review

https://github.com/BuiltByMom/gimme
Team information
Company: MOM
Official website: N/A
Point of contact: Major / Facu
Email/Telegram: @Majorfi / @saltyfacu

This one was reviewed and approved by the product team.
PS: @Majorfi you didn't get back to me in the email so found this issue just now.

I'm getting this error in the dev console.

I get the same error using your app as a stand alone in https://gimme.mom/earn, connecting my safe or my MM chrome extension.
Let us know if you are experiencing issues in your app currently

Hi there, sorry for taking so long to check this.
It does work fine now. Thanks!

@parius the app is approved from the QA side

the app is listed now (Polygon only, as requested)


GPT models' outputs are not fuzzy matched despite default behavior

labels from gpt models include misspellings, sentences (i.e., "this text can be classified as xxx").
this occurs when doing e.g, cls = Trooper("gpt4") and fuzzy matching is true by default.

this also happens when explicitly setting fuzzy matching as true cls = Trooper("gpt4", fuzzy_match=True)
labels are returned with 'around them and misspelled

#7 fixes this <3


Typescript: sort auto-complete results by priority

The auto-completion feature for Typescript in VS Code usually makes the wrong guess - and it looks like the results are merely ordered alphabetically?
Example:

In this case (as most of the time) it is far more likely I want the nearest matching variable connection rather than the global (window) method confirm or connect.
In PHP Storm or WebStorm, this works much better, and I've gotten used to typing extremely fast by relying heavily on it's ability to more accurately predict what I want after just two or three characters - I've learned to not even look at the auto-complete results before pressing TAB, and 95% of the time, it guesses exactly the thing I'm trying to type.
I think the Storm IDEs order the auto-complete results (in part) by vicinity?
That is, out of the results where the start of what I've typed matches, it orders those according to where I'm typing - variables in the closest scope to where I'm typing appear first, then variables in the parent scope, and so on, with results in the global scope being the least likely.
As a result, I'm learning not to trust auto-completion results, which is making me slower is Storm as well.
Could this be improved in VS Code, please?

@jrieken Could typescript fix this by returning proper sortText?

Yeah, set sortText and/or preselect. The sortText will be used when no prefix has been typed yet or when in the presence of a prefix multiple equal ranks exists (e.g sort by rank then sort by sortText-order)

@mjbvz @jrieken I'd like to work on this, I just play a bit with the code and try to get the overall understanding of that part of the code. I've made some changes to wordDistance (I'm not sure what the responsibility of wordDistance is! I just want to get it work and see how the result would be).
But I'm wondering to ask if this issue should be fixed on typescript or vscode!
Based on Matt's comment it should be fixed on typescript repo by changing getCompletionsAtPosition (I'm not sure actually just investigated a bit) or all places which is using CompletionInfo.
Also we could set sortText up based upon tsEntry.kind within MyCompletionItem on vscode repo.
Could you please explain a bit more on your comments?
Please Let me know if I'm thinking too crazy or I'm on track.

The wordDistance logic is new for the 1.28, see https://github.com/Microsoft/vscode-docs/blob/vnext/release-notes/v1_28.md#intellisense-locality-bonus. What changes did you have to make there, what is still missing?
Generally, "yes" this is something TypeScript should get right but the locality bonus feature is now here to compensate for that.

Btw, I'm wondering to ask if there is any issue regarding to sortText (^^) on TypeScript's repo.

we need to bring the sortText into the consideration in here.

We already do that. The initial sort order is defined by the sortText-property and later carried on as idx. There is potential for conflict with the word distance but that's implemented to be so that it doesn't conflict too much, e.g. the distance is the containing block not line/character distance

@jrieken Ah, Ok. Now I got :) Thanks.
So here we need to get  locality bonus feature to work.

@jrieken Do we need to do anything here at all? I'm confused!
It seems locality bonus feature has been implemented already!

:-) yeah, I think with the locality bonus we are in a pretty good spot. having said that, it's pretty language agnostics by design and more or less works by looking at brackets only. TypeScript, having all the knowledge, will likely be better at computing those orders but lets see how far we get here

Just wanted to show something strange I'm running into in the latest version:

I have an in-scope exact match that is somehow the last option? When I press . it auto-completes the first selection :S


Question about the read implementation

Hi there,
I was going throng the library and came across this
// ReadableSecureStorable lets us read the account from the keychain
let result = account.readFromSecureStore()

Is this really the desired functionality? Why would I want to read from the keychain when I already have the object? Maybe I am missing something important here

Yeah never mind, I was mixing 2 things, I should just "save data" when I want to store something specific and only use the GenericPasswordSecureStorable with actual account information. I was trying to mix this into an OAuthToken system

I'm afraid I'm as baffled as you about how reading is supposed to work. Why go to all the effort of wrapping up your credentials in an object only to throw that all away when you read from the keychain.

+1
Really need some explanation here too.

I think the example in the readme strives for simplicity at the expense of explicitness (which is probably a bad thing… would love to see a PR to improve the docs on this!)
A quick example (certainly not great code or anything) that might clear things up
// Let's say that Twitter accounts can be in one of two states: they can be unauthenticated (`MaybeATwitterAccount`) and authenticated (`TwitterAccount`).
// The latter has functionality that the former does not, like tweeting.

struct MaybeATwitterAccount: GenericPasswordSecureStorable, CreateableSecureStorable {
    let username: String
    let password: String

    let service = "Twitter"
    let account = "Twitter user"

    var data: [String: AnyObject] { return ["password": password] }

    func isValid() -> Bool { return true }
}

struct TwitterAccount: GenericPasswordSecureStorable, ReadableSecureStorable {
    let service = "Twitter"
    let account = "Twitter user"
    let username: String
}

// Maybe we get Twitter account credentials via a form in our app
let prospective = MaybeATwitterAccount(username: "_matthewpalmer", password: "my_password")
if prospective.isValid() { // Maybe we do some validation against the server
    try! prospective.createInSecureStore() // We know the account's ok, so we save it
}

// Some time later in our app
let lastUsername = "_matthewpalmer" // maybe we get this from user defaults (or even from the keychain)
let proper = TwitterAccount(username: lastUsername)

// Now we can get the password from the keychain (we probably wouldn't do this out here, it might be done inside a more useful method on the TwitterAccount itself, like a function called `tweet()` or something.
let password = proper.readFromSecureStore()!.data!["password"] as! String

I did notice in coming up with this example that it would have been slightly easier to cook up if the data requirement on CreateableSecureStorable wasn't a computed property but rather a function (or maybe even an argument to the createInSecureStore function). Let me know if that's an API change that would suit you better and I can roll it out in the next larger release.


Enhancement: Displaying of tasks in the UI

Account for tasks with long names
Fit more tasks into a panel (particularly for floating tasks)

@rachx  can review?

Resolved


Simple Pathfinding

We need some simple algorithm to find the shortest path from one Waypoint to another on the waynet. The algorithm should return the list of waypoints which need to be walked to in the correct order.
See the documentation on Waynets for more information.

Has been implemented in PR #31.


Remove version: OVRASTeam.OpenVR-AdvancedSettings version 5.8.6

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/OpenVR-Advanced-Settings/OpenVR-AdvancedSettings/releases/download/v5.8.6/AdvancedSettings-5.8.6-Installer.exe
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 911)


[FEATURE REQUEST] Add arm64 support to 'applications' Ansible role to install 'Keycloak'

Is your feature request related to a problem? Please describe.
Currently we only support the x86_64 architectures but we need to add support for arm64.
Describe the solution you'd like

We want the ability to install keycloak on K8s running on arm64 machines running CentOS.
Use the following multi-arch image: https://hub.docker.com/repository/docker/epiphanyplatform/keycloak
Only the latest versions we support: 9
Any requirements missing or not added in #2033 will be added for the effected roles.
Any other deployments available inside the application role should be blocked from installing on arm64.

Effected Ansible roles:

applications

Describe alternatives you've considered
None.
Additional context
No.
Extra info
Design document and resources:
https://github.com/epiphany-platform/epiphany/tree/develop/docs/design-docs/arm
Terraform scripts for ARM hardware on AWS:
https://github.com/mkyc/aws-arm-simple-setup
YAML for testing:
TODO

DoD checklist

[ ] Changelog updated
[ ] COMPONENTS.md updated / doesn't need to be updated
[ ] Feature has automated tests
[ ] Automated tests passed (QA pipelines)

[ ] apply
[ ] upgrade

[ ] Idempotency tested
[ ] Documentation added / updated / doesn't need to be updated
[ ] All conversations in PR resolved
[ ] Solution meets requirements and is done according to design doc
[ ] Usage compliant with license

✅ Tested, works fine, I was able to execute tests from our code in https://github.com/epiphany-platform/epiphany/tree/ARM/core/src/epicli/data/common/tests/spec

In addition need to verify that it's possible to install only supported apps. It's validated in preflight checks.

Assertion was added in preflight check to fail if there are unsupported applications enabled.
:heavy_check_mark: Tested on aarch64 instances.
2021-04-26T14:20:59.1105308Z[38;21m14:20:59 INFO cli.engine.ansible.AnsibleCommand - TASK [preflight : Check if enabled applications are supported for the architecture] ***
2021-04-26T14:20:59.2522900Z[31;21m14:20:59 ERROR cli.engine.ansible.AnsibleCommand - fatal: [ec2-1-1-1-1.eu-west-1.compute.amazonaws.com]: FAILED! => {
2021-04-26T14:20:59.2538102Z[38;21m14:20:59 INFO cli.engine.ansible.AnsibleCommand -     "assertion": "_unsupported_enabled_apps | length == 0",
2021-04-26T14:20:59.2555248Z[38;21m14:20:59 INFO cli.engine.ansible.AnsibleCommand -     "changed": false,
2021-04-26T14:20:59.2659123Z[38;21m14:20:59 INFO cli.engine.ansible.AnsibleCommand -     "evaluated_to": false,
2021-04-26T14:20:59.2670256Z[38;21m14:20:59 INFO cli.engine.ansible.AnsibleCommand -     "msg": "Following application(s) are not supported for aarch64 and cannot be installed: pgpool, pgbouncer, istio"
2021-04-26T14:20:59.2681909Z[38;21m14:20:59 INFO cli.engine.ansible.AnsibleCommand - }


[REFACTOR] std::string -> String Migration in IR nodes

This is an issue to track the progress of std::string -> String migration in IR Nodes.

[x] Merge initial example https://github.com/apache/incubator-tvm/pull/5438
[ ] Convert relay nodes
[ ] Convert base types
[ ] Convert tir nodes.

cc @icemelon9 @jroesch @zhiics please chime in if you would like to take a stab

NOTE, as in the intiial example, we will need to update the converter(via _update_from_std_str) to keep backward compatibility of the nodes

I like to contribute for this movement. May be we can share the workload, if @zhiics  already working on it. Please let me know.Thanks!

@ANSHUMAN87 Welcome to contribute. I've busy with other things recently. Thanks.


Trailing comma in an array field of annotation fails rewrite test

rewrite bom 2.6.4, wasn't an issue on 2.5.3
Test failures when there is a trailing comma in an annotation. If I remove it, tests pass.
java.lang.AssertionError: Source file was parsed into an LST that contains non-whitespace characters in its whitespace. This is indicative of a bug in the parser.
package com.playtika.services.infra.rewrite.spring;

import org.springframework.boot.test.context.SpringBootTest;

@SpringBootTest(
        properties = {
                "feign.circuitbreaker.enabled=true",
                "feign.circuitbreaker.group.enabled=true",
                "eureka.client.enabled=false"~~(non-whitespace)~~>,
        <~~},
        webEnvironment = SpringBootTest.WebEnvironment.NONE
)
public abstract class A {
}

So the comma at the end of properties in a test like this
    @Test
    void shouldChangePropertyKeyInAnnotationProperties() {
        rewriteRun(
                spec -> spec.recipe(new ChangePropertyKeyAnnotationAttributeValue(
                        "org.springframework.boot.test.context.SpringBootTest",
                        "properties", "feign.circuitbreaker.enabled",
                        "spring.cloud.openfeign.circuitbreaker.enabled")),
                java(
                        """
                                        package com.playtika.services.infra.rewrite.spring;

                                        import org.springframework.boot.test.context.SpringBootTest;

                                        @SpringBootTest(
                                                properties = {
                                                        "feign.circuitbreaker.enabled=true",
                                                        "feign.circuitbreaker.group.enabled=true",
                                                        "eureka.client.enabled=false", <-- THIS COMMA HERE
                                                },
                                                webEnvironment = SpringBootTest.WebEnvironment.NONE
                                        )
                                        public abstract class A {
                                        }
                                """
...
        ));
    }

Hi! We recently added validation indeed to flush out LST issues in our parsers. It's helpful to improve the parsers, but indeed might cause a test to fail until we circle back to the parser. I'd encountered this case as well and logged a test last week https://github.com/openrewrite/rewrite/commit/78a55a46056808b4b630e781f39bad112abd3d63
We can keep your issue open to see that parser issue resolved; in the meantime you can remove that trailing comma from the test. Hope that helps!

And perhaps good to note: This is only an internal modeling issue; your source code will still be reproduced faithfully even with that modeling mishap, as we have a separate check for that. So rest assured that any code changes will be just fine; it's only a matter of using compatible examples in tests until the parser correctly models the code.

Another test case which can be enabled again once this is fixed: https://github.com/openrewrite/rewrite/blob/33d8f7d42f9bd6749ab93171fad31289b0e2bc97/rewrite-java-test/src/test/java/org/openrewrite/java/format/RemoveTrailingWhitespaceTest.java#L60-L77


Update majorVersionChange scope

Do you think it would make sense to add the majorVersionChange flag as part of the recordChange workflow rather than having it as a global var in the EntityUpdater?
I am working on having custom minor/major changes in MlModel and having majorVersionChange = true; in the entitySpecificUpdate is giving me some issues, as I think the value gets changed globally in the updater and all tests are retuning MAJOR now.
Based on the Table Repository, I am trying to have something way simpler for the MlModel, as there is no need for validation:
// This is called in entitySpecificUpdate
private void updateAlgorithm(MlModel origModel, MlModel updatedModel) throws JsonProcessingException {
      majorVersionChange = true;
      recordChange("algorithm", origModel.getAlgorithm(), updatedModel.getAlgorithm());
    }

However, now all PUT tests are giving version 1.1. I was wondering about updating the recordChanges method to allow the majorVersionChange flag as an argument so that the logic flow happens atomically for each update.
Happy to hear your thoughts on this! Sorry I am missing something 🙏
Thanks!

cc @sureshms

@pmbrull, every time updateAlgorithm is called, you are setting majorVersionChange to true, irrespective of it has changed.
The following code will ensure only when algorithm is changed, you are setting majorVersionChange to true.
if (recordChange("algorithm", origModel.getAlgorithm(), updatedModel.getAlgorithm())) {
  majorVersionChange = true;
}

Note that major version will change when algorithm goes from null to new value, a value to null, and when the value is changed. See if this is the right behavior.

Thanks, @sureshms, you are totally right. Even when nothing changes, we are always going to updateAlgorithm from the updater. Completely missed that, thanks for the info.
I'll change this as you suggested, as it is the right behavior.
As always, many thanks 🙏


Fixups for macOS -- get rid of cruft and 1 tiny nit in osx.spec

Do you think you can squash these commits?

Sure!  I have no idea got to do that though.  Do you know  offhand how to do that and not destroy my repository?

I think git rebase --autosquash -I <commit hash> with the commit hash being the oldest you want to squash.

Oh no! What happened


refactor(fix): [Adyen] Fix bug in Adyen

Type of Change

[x] Bugfix
[ ] New feature
[ ] Enhancement
[ ] Refactoring
[ ] Dependency updates
[ ] Documentation
[ ] CI/CD

Description
Fixed bug:

Added support for 2 digits expiry year. (earlier it was accepting only 4digit exp year.)
In card 3DS, additional browser_info support was added as it was mandatory field for getting url.
Fixed Card Mandates for multi use mandate type. (for multiple payments from merchant side)
Fixed webhooks source verification.

Additional Changes

[ ] This PR modifies the API contract
[ ] This PR modifies the database schema
[ ] This PR modifies application configuration/environment variables

Motivation and Context

How did you test it?
Using Postman

Checklist

[x] I formatted the code cargo +nightly fmt --all
[x] I addressed lints thrown by cargo clippy
[x] I reviewed submitted code
[ ] I added unit tests for my changes where possible
[ ] I added a CHANGELOG entry if applicable

@swangi-kumari  please attach the screen shot of the response.


rsjx-mode

Hello, Would it be possible to add rjsx-mode to activate js icon in mode line?

seems to be fixed


Bring "phase filtering" filter forwards onto the summary view

Normally the annotation filters are present at the level which thy are applicable.
I.e most phenotype term and haploid/diploid are available on the summary view.
For GO,  all filters are hidden in the summary view. We should move the 'phase' filter forwards to the summary view, because phases are visible at this point.

This will be just a config change once this issue is fixed:

#1667.

Now that #1667 is fixed I've configured this for GO and for the qualitative gene expression annotation.

Nice, that is really useful on cdc2 page.
I will suggest some tweaking of the filter labels in the filter option....


RetryLoadBalancerInterceptor does not allow to retry on specific exceptions

Is your feature request related to a problem? Please describe.
I'd like to configure retries not by status code (as it's currently allowed), but by exceptions.
For example I'd like the call to be retried in case of HttpHostConnectException or ConnectTimeoutException, etc.
Describe the solution you'd like
I'd like to have a property that lets me configure the exception list (something similar is provided by Resilience4j), for example:
spring:
  cloud:
    loadbalancer:
      retry:
        retryableExceptions:
          - org.apache.http.conn.ConnectTimeoutException
          - java.net.ConnectException
          - java.net.UnknownHostException

Describe alternatives you've considered
Maybe I could wrap the call with some other library that allows this feature, like spring-retry or resilience4j but I guess it won't work properly because - for example - the last instance won't be filtered and such retry could hit the same (broken) one.

Hello, @danparisi. With LB retries, we assume that we do get a response before a retry. In the case you're describing, we would not be getting a response at all. It should be quite easy to implement. Will discuss this with the team and get back to you.

Gateway does this already in it's filter

@spencergibb yes but to be honest I'd like to ask for a feature request for the gateway too, you can find it HERE. Sorry for the off topic :)


[Docbug][Wrong SDK version]

Minor edits

Docs Build status updates of commit 521a336:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/resources/team-chat-member-api-changes.md
:bulb:Suggestion
View
Details

msteams-platform/resources/team-chat-member-api-changes.md

Line 19, Column 107: [Suggestion-docs-link-absolute] Absolute link 'https://docs.microsoft.com/microsoftteams/platform/bots/how-to/get-teams-context?tabs=dotnet#fetch-the-roster-or-user-profile' will be broken in isolated environments. Replace with a relative link.
Line 59, Column 24: [Suggestion-docs-link-absolute] Absolute link 'https://docs.microsoft.com/microsoftteams/platform/bots/how-to/get-teams-context?tabs=dotnet#fetch-the-roster-or-user-profile' will be broken in isolated environments. Replace with a relative link.
Line 59, Column 374: [Suggestion-docs-link-absolute] Absolute link 'https://docs.microsoft.com/dotnet/api/microsoft.bot.connector.conversationsextensions.getconversationpagedmembersasync?view=botbuilder-dotnet-stable&preserve-view=true' will be broken in isolated environments. Replace with a relative link.
Line 64, Column 24: [Suggestion-docs-link-absolute] Absolute link 'https://docs.microsoft.com/microsoftteams/platform/bots/how-to/get-teams-context?tabs=dotnet#get-single-member-details' will be broken in isolated environments. Replace with a relative link.
Line 64, Column 273: [Suggestion-docs-link-absolute] Absolute link 'https://docs.microsoft.com/windows/win32/ad/naming-properties#userprincipalname' will be broken in isolated environments. Replace with a relative link.
Line 67, Column 285: [Suggestion-docs-link-absolute] Absolute link 'https://docs.microsoft.com/microsoftteams/platform/bots/how-to/get-teams-context?tabs=json#get-single-member-details' will be broken in isolated environments. Replace with a relative link.
Line 69, Column 206: [Suggestion-docs-link-absolute] Absolute link 'https://docs.microsoft.com/microsoftteams/platform/bots/how-to/get-teams-context?tabs=json#tabpanel_CeZOj-G++Q_json' will be broken in isolated environments. Replace with a relative link.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 9f0548c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/resources/team-chat-member-api-changes.md
:bulb:Suggestion
View
Details

msteams-platform/resources/team-chat-member-api-changes.md

Line 19, Column 107: [Suggestion-docs-link-absolute] Absolute link 'https://docs.microsoft.com/microsoftteams/platform/bots/how-to/get-teams-context?tabs=dotnet#fetch-the-roster-or-user-profile' will be broken in isolated environments. Replace with a relative link.
Line 59, Column 24: [Suggestion-docs-link-absolute] Absolute link 'https://docs.microsoft.com/microsoftteams/platform/bots/how-to/get-teams-context?tabs=dotnet#fetch-the-roster-or-user-profile' will be broken in isolated environments. Replace with a relative link.
Line 59, Column 374: [Suggestion-docs-link-absolute] Absolute link 'https://docs.microsoft.com/dotnet/api/microsoft.bot.connector.conversationsextensions.getconversationpagedmembersasync?view=botbuilder-dotnet-stable&preserve-view=true' will be broken in isolated environments. Replace with a relative link.
Line 64, Column 24: [Suggestion-docs-link-absolute] Absolute link 'https://docs.microsoft.com/microsoftteams/platform/bots/how-to/get-teams-context?tabs=dotnet#get-single-member-details' will be broken in isolated environments. Replace with a relative link.
Line 64, Column 273: [Suggestion-docs-link-absolute] Absolute link 'https://docs.microsoft.com/windows/win32/ad/naming-properties#userprincipalname' will be broken in isolated environments. Replace with a relative link.
Line 67, Column 285: [Suggestion-docs-link-absolute] Absolute link 'https://docs.microsoft.com/microsoftteams/platform/bots/how-to/get-teams-context?tabs=json#get-single-member-details' will be broken in isolated environments. Replace with a relative link.
Line 69, Column 206: [Suggestion-docs-link-absolute] Absolute link 'https://docs.microsoft.com/microsoftteams/platform/bots/how-to/get-teams-context?tabs=json#tabpanel_CeZOj-G++Q_json' will be broken in isolated environments. Replace with a relative link.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 8ee2fc8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/resources/team-chat-member-api-changes.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 16f68f5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/resources/team-chat-member-api-changes.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Circular dependency between datafusion and datafusion-sqllogictest crates

Describe the bug
I was attempting to publish DataFusion 29, and I could not publish the core datafusion crate because it has a dev-dependency on datafusion-sqllogictest, which is not yet published.
   Packaging datafusion v29.0.0 (/home/andy/git/apache/arrow-datafusion/datafusion/core)
    Updating crates.io index
error: failed to prepare local package for uploading

Caused by:
  no matching package named `datafusion-sqllogictest` found
  location searched: registry `crates-io`
  required by package `datafusion v29.0.0 (/home/andy/git/apache/arrow-datafusion/datafusion/core)`

I cannot publish datafusion-sqllogictest either, because it depends on datafusion.
   Packaging datafusion-sqllogictest v29.0.0 (/home/andy/git/apache/arrow-datafusion/datafusion/sqllogictest)
   Verifying datafusion-sqllogictest v29.0.0 (/home/andy/git/apache/arrow-datafusion/datafusion/sqllogictest)
    Updating crates.io index
error: failed to verify package tarball

Caused by:
  failed to select a version for the requirement `datafusion = "^29.0.0"`
  candidate versions found which didn't match: 28.0.0, 27.0.0, 26.0.0, ...
  location searched: crates.io index
  required by package `datafusion-sqllogictest v29.0.0 (/home/andy/git/apache/arrow-datafusion/target/package/datafusion-sqllogictest-29.0.0)`

To Reproduce
Try publishing datafusion crates from source release.
Expected behavior
No response
Additional context
No response

It looks like the circular dependency was introduced in https://github.com/apache/arrow-datafusion/pull/7134
cc @tshauck @alamb

rust-analyzer plugin in VSCode also complains
[ERROR project_model::workspace] cyclic deps: datafusion_sqllogictest(Idx::<CrateData>(168)) -> datafusion(Idx::<CrateData>(100)), alternative path: datafusion(Idx::<CrateData>(100)) -> datafusion_sqllogictest(Idx::<CrateData>(168))

Sorry @andygrove
I think we can fix this by cutting the sqllogictest dependency on datafusion (essentially by moving the sqllogictest driver into the sqllogictest crate).
I can make a PR to do this later today

Sorry all 🙇... and thanks @alamb, you're certainly in a better position to fix this quicker. Happy to help out another way if needed.

https://github.com/apache/arrow-datafusion/pull/7284 is ready for review


[Improvement] Adjustments to notification emails

Just some thoughts about the notification emails that came to my mind:

[ ] Use a proven template, ref: https://github.com/leemunroe/responsive-html-email-template.
[x] In the table, include information about the best position so far, similar to the dashboard.
[ ] The value of the updated row should be the exact date instead of being relative because the email is stored "as seen."

The 2nd request has been added in 4c2f900


explicitly push after build

So that the compile step has access to this new container.
Otherwise, at first run, this build will fail; and in later runs it will always use the second latest version, as opposed to the latest version.
Note that the Cloud Build doc allows for this "double" push:
"If you want to store the image as part of your build flow and want to display the image in the build results, use both the Docker push command and the images field in your build config file."
https://cloud.google.com/build/docs/building/build-containers#store-images

LGTM


Update failed with hasManyThrough relationship

Laravel Version: v5.6.38
PHP Version: 7.2.8
Database Driver & Version: MySQL v5.7.23

Description:
I've made a relationship with hasManyThrough and it's working perfectly to fetch data. But, When I'm trying to update I get this error below.
Integrity constraint violation: 1052 Column 'customer_id' in field list is ambiguous (SQL: update `invoice_items` inner join `invoices` on `invoices`.`id` = `invoice_items`.`invoice_id` set `customer_id` = 2, `user_id` = 7, `updated_at` = 2018-09-22 21:47:15 where `invoices`.`order_id` = 17)

Steps To Reproduce:
Tables structures:
orders
- id
- user_id
- customer_id
- content
invoices
- id
- order_id
- user_id
- customer_id
- content
transactions
- id
- invoice_id
- user_id
- customer_id
- content

Relationships
public function transactions()
{
   return $this->hasManyThrough('App\Transaction', 'App\Invoice');
}

What i expected to work
$order->transactions()->update([
    'customer_id' => 100,
    'user_id' => 100
]);

I've found a temporary solution about this issue like this:
foreach ($order->transactions as $transaction) {
    $transaction->update($updateRules);
}

Specify the table name:
$order->transactions()->update([
    'transactions.customer_id' => 100,
    'transactions.user_id' => 100
]);

@standaniels Thank you. But, you know if I specified the table name but the problem still exists because the updated_at which can't be set by specific table name.
Have a look at this error:
SQLSTATE[23000]: Integrity constraint violation: 1052 Column 'updated_at' in field list is ambiguous (SQL: update `invoice_items` inner join `invoices` on `invoices`.`id` = `invoice_items`.`invoice_id` set `invoice_items`.`customer_id` = 2, `invoice_items`.`user_id` = 3, `invoice_items`.`updated_at` = 2018-09-23 21:33:54, `updated_at` = 2018-09-23 21:33:54 where `invoices`.`order_id` = 25)

I've experimented with below code by adding transactions.updated_at => now()->toDateTimeString(). Still no luck.

This is a more complex issue, see #13909.

You can use this workaround:
$order->transactions()->toBase()->update([
    'transactions.customer_id' => 100,
    'transactions.user_id' => 100,
    'transactions.updated_at' => now()
]);

Please close the issue.

Will be fixed in Laravel 5.8: #26031


Update package.json to resolve security issue

Fixes https://github.com/jfrog/bower-art-resolver/issues/30

@eyalbe4 is this something you can merge? It is currently blocking us from doing releases.

Thanks for this one @jorenbroekema.
It has just been released as part of version 2.0.10.


Demo..fuseki server

Hi,
Thank you for sharing.. very useful to learn how the Integration web, query, fuseki is done. I am interested to use it for modelling an open source autopilot system. Currently I am using excel that contains

objects (class id, o Id, attributes on a single spreadsheet
an other spreadsheet containing relationship information : link type name, parent,child and attributes).

I wrote a vba code that convert this data to a cypher script and the script is used to create the model in Neo4j as a graph database. Neo4j generate the ontological metamodel automcally after running the cypher script.
I would like to try to achieve the same with OML...I have a very large numbers of object and relationships. Is there a limitation using OML and this fuseki server? I am not an expert of opencaesar and curious to know if you think this would be feasible?  I need to find out how to generate oml by script reading an excel input. Any advice welcome.
Thank you
Regards

You need to generate from the Excel doc two artifacts: an OML vocabulary (metamodel) defining the terms, and an OML description (system model) using the terms. The generated vocabulary will be mostly syntactical. This is enough if you are only interested in running SPARQL queries against the assertions. But, if you also want to run the reasoner to generate logical entailments, to detect inconsistencies and simplify queries, you need to add DL semantics to the vocabulary by setting the DL flags on the terms and/or adding extra inference rules.
You will first need to create an OML project that adds at least core-vocabularies (OML standard library) as a dependency (in build.gradle). Then you can develop your Excel-to-OML generator and optionally invoke it from the Gradle script so it's orchestrated with the other build tasks.
For the generator itself, you can either develop a model-to-model transformation using the OML Java API, i.e., work at the abstract syntax level then ask the resulting resources to save to get the textual syntax (serialization).  Or, you could develop a model-to-text transformation to generate the textual syntax directly. OML supports a custom textual grammar (.oml), a XML-based one (.omlxmi) and a JSON-based one (.omljson).

The only limitation of the Fuseki server is that it runs on a single machine, it technically it is bound by that machine's resources/. However, practically, I did not run into this limitation yet even on a large (millions of triples) dataset.
For OML itself,  if you worked at the abstract syntax level, i.e., OML data loaded in memory, you are bound by the size of memory of course. However, since OML ontologies declare their imports (of other ontologies) explicitly, when you load an OML ontology in memory, it will cause only its import closure (its dependent ontologies) to load, not the entire dataset. So by fragmenting your dataset into multiple ontologies, and managing the import relationships between them, you can easily avoid running into this limitation.

Hi,
Thank you very much for the detailled explanation. I will try the approach
and share the link.
I think I posted in the wrong forum as this was more a question than issue.
Thanks for sharing this MBE framework. Very interesting as I am working on
open source autonomous systems and the goal will be to put the database
content accessible to the open source autopilot communities using this RDF
based repository.
On Sun, Mar 5, 2023, 6:30 PM Maged Elaasar @.***> wrote:

The only limitation of the Fuseki server is that it runs on a single
machine, it technically it is bound by that machine's resources/. However,
practically, I did not run into this limitation yet even on a large
(millions of triples) dataset.
For OML itself, if you worked at the abstract syntax level, i.e., OML data
loaded in memory, you are bound by the size of memory of course. However,
since OML ontologies declare their imports (of other ontologies)
explicitly, when you load an OML ontology in memory, it will cause only its
import closure (its dependent ontologies) to load, not the entire dataset.
So by fragmenting your dataset into multiple ontologies, and managing the
import relationships between them, you can easily avoid running into this
limitation.
—
Reply to this email directly, view it on GitHub
https://github.com/melaasar/kepler16b/issues/1#issuecomment-1455167305,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3SIULJR3M2XX4LDHJQQ35DW2TLUJANCNFSM6AAAAAAVPU27T4
.
You are receiving this because you authored the thread.Message ID:
@.***>

 Hi,  A little extra question about your answer:Ref: You wrote “. OML supports a custom textual grammar (.oml), a XML-based one (.omlxmi) and a JSON-based one (.omljson).”I was running some test excel to Neo4j test today using aura free cloud  and noticed that I can generate a report about the metamodel automatically with a cypher command (see pictur1 and the same metamodel can be exported as JSON file ( see attached json file “record” and picture “table json”). This made me remember your OML support omljson comment.Do you think this neo4j Json export (see attached file) could be converted automatically to oml using Opencaesar?  My code excel to neo4j  is in VBA and as I am more a system engineer than developer, if I can re use the neo4j Json directly in Eclipse, that would be great.  These picture are the metamodel that would corresponding to OML vocabulary right?  The data can be exported the same way. My workflow would become: excel -> Neo4j -> Neo4j Json metamodel export ->omljson->Openceasar        easier? Anyway thank you very much,Patirck     Sent from Mail for Windows From: Maged ElaasarSent: Sunday, March 5, 2023 6:13 PMTo: melaasar/kepler16bCc: Patrick F; AuthorSubject: Re: [melaasar/kepler16b] Demo..fuseki server (Issue #1) You need to generate from the Excel doc two artifacts: an OML vocabulary (metamodel) defining the terms, and an OML description (system model) using the terms. The generated vocabulary will be mostly syntactical. This is enough if you are only interested in running SPARQL queries against the assertions. But, if you also want to run the reasoner to generate logical entailments, to detect inconsistencies and simplify queries, you need to add DL semantics to the vocabulary by setting the DL flags on the terms and/or adding extra inference rules.You will first need to create an OML project that adds at least core-vocabularies (OML standard library) as a dependency (in build.gradle). Then you can develop your Excel-to-OML generator and optionally invoke it from the Gradle script so it's orchestrated with the other build tasks.For the generator itself, you can either develop a model-to-model transformation using the OML Java API, i.e., work at the abstract syntax level then ask the resulting resources to save to get the textual syntax (serialization). Or, you could develop a model-to-text transformation to generate the textual syntax directly. OML supports a custom textual grammar (.oml), a XML-based one (.omlxmi) and a JSON-based one (.omljson).—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>


fix pull up cte anchor should also pull up cte in apply (#28214)

pick from master
#28214
commit 4e50b2791ac349c4572a9ee9d0b7cd76503444de
Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall


Make credit maximum variable

You're opening a pull request? Awesome!
Please make sure though you did the following (tick off everything you
already did):

[ ] Run the tests and see them pass
[ ] Rebase your branch on top of develop
[ ] Include tests for features you introduced / bugs you fixed

This issue fixes #268

added the one real name commit to that PR

@BelphegorPrime Please undo the merge from develop into this branch (git reset --hard HEAD^), and rebase it onto develop (git rebase develop). One commit will “fall out”, since I cherry-picked it (the name→realname fix), and it is already included in develop.
Current graph:
6f865aa *   origin/develop origin/HEAD develop Merge pull request #285 from lukasjuhrich/frontend_tests
        |\
cf29a89 | * lukas/frontend_tests frontend_tests Test `/bustimes` reachability
de208d0 | * Add last missing `generic` frontend tests
3fb8071 | * Add missing template
4ae1dbd | * Extend `generic` frontend tests
5100888 | * changed name to real realname
e3d1ada | * Add simple tests for `generic` and `usersuite`
        |/
3be249c | *   marcel/develop Merge branch 'develop' into develop
        | |\
        | |/
        |/|
24b2243 * |   Merge pull request #283 from lukasjuhrich/gerok_200
        |\ \
5b5e026 | * | lukas/gerok_200 Fix PEP8
f7ad2f8 | * | Add test for unknown status logging
7c10c44 | * | Gerok: Don't raise on != 200 status codes
        |/ /
92b68f3 | * pep8 adjusted
ceb6c7b | * changed name to real realname
8fcc6bd | * doc change
adf6a2a | * changed the property to variables
b771388 | * marcel/sipa-268 max_credit is now as an separate parameter added to the graph generation function
c061520 | * WIP added max_credit and daily_credit to the other user models
0c55899 | * WIP maybe max_credit and daily_credit stuff
        |/

The reset undo's the unnecessary merge on top of marcel/develop, and the rebase will “move” the branches starting point from that lower commit to the current state of develop.

By the way, your git can display nice graphs as well if you pass some parameters to log:
git log --oneline --graph --decorate --all
You can make an alias out of it like git config alias.tree='git log […]', so you can just type git tree.

Hey, since I wanted to deploy, I rebased your changes on top of develop. They now are on https://github.com/agdsn/sipa/tree/marcel_develop. I resolved the merge conflicts. You just have to hard reset your branch (assuming you made no further changes) to origin/marcel_develop.

Closing, replaced by #287