Change default server_root to __dirname

From https://github.com/alexa-js/alexa-app-server/pull/72
See http://stackoverflow.com/a/18283508 for more info.

May this ticket be closed? See #84.

Closing via #84.


4541081D - Madden NFL 08

Validation

[X] I've read the FAQ.
[X] The Xenia build used is from a Canary branch. (not UWP/MLBS/AlexVS/master/pull requests, etc)
[X] This issue isn't for help (tech support) with Xenia, or a game.
[X] I've checked if this game doesn't already have a compatibility report.

Wiki page (optional)
No response
Xbox 360 Marketplace link (optional)
https://marketplace.xbox.com/en-US/Product/Madden-NFL-08/66acd000-77fe-1000-9115-d8024541081d
Xenia Canary version
https://github.com/xenia-canary/xenia-canary/commit/90fc07e4005127298cc36cb43134dc588c815b2e
Issues

player uniforms are corrupted.
some text in gameplay is corrupted.
occasionally commentator/crowd audio gets corrupted for a moment, then goes back to normal.

Log
xenia.zip
Screenshots

Labels
apu-garbage, backwards-compatibility-unsupported, gpu-drawing-corrupt, state-gameplay

Seems to be dependent on how close the camera is to a player judging from my own experimentation with the replay function. Could be something to do with texture streaming? I'm unsure, to be honest.

Once i click on the game it shows a black screen for a second and then crashes


Display of inscriptions that are published in a single publication

the grid goes off to the right. Needs to have 2 or max 3 columns

note: only in safari

Zak upgrade his Safari and see how to fix.

fixed


Update algorithm comments

Codecov Report

Merging #406 (3c81d91) into main (e124193) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #406   +/-   ##
=======================================
  Coverage   75.30%   75.30%
=======================================
  Files          79       79
  Lines        4050     4050
  Branches      691      691
=======================================
  Hits         3050     3050
  Misses        587      587
  Partials      413      413

Impacted Files
Coverage Δ

...vanyo/composelife/model/TemporalGameOfLifeState.kt
92.71% <ø> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Ordered Consumer

Push Ordered Consumer
Pull Ordered Consumer

https://github.com/nats-io/nats-architecture-and-design/blob/main/adr/ADR-17.md


missing package and prestart fix

"prop-types": "^15.5.8",
and prestart to use bash instead of sh since bash works on both linux and mac os

Looks like prop-types is already in the upstream.


CHEBI:63682 and CHEBI:63750, (13E)-labda-7,13-dien-15-yl diphosphate, more stereoconfiguration

Hi Gareth and Steve,
There is now more information available about the stereoconfiguration of (13E)-labda-7,13-dien-15-yl diphosphate, but I am not sure whether you would prefer to have a new entry, or just to amend the existing structure. I have attached the 'improved' structures so you can see the difference.
CHEBI_63682.txt
CHEBI_63750.txt
The new information is found in pmid:26814669
Cheers,
Kristian

Hi Kristian,
Seeing as the entries were created for a particular compound and its pH 7.3 form, I think it is fine to update the entries when more details about their structure is published.
I have updated the display names and IUPAC names of the entries to reflect the newly-reported stereochemistry, but retained the old display names as synonyms. PMID:26814669 has been added to the citations for both entries.
Best regards,
Gareth


Allow scrolling and copying error messages

Is your feature request related to a problem? Please describe.
When having a very large error message, it's not possible to scroll it, nor copying it to google the error message.
Describe the solution you'd like
Scrollable and copyable error messages
Describe alternatives you've considered
N/A
Additional context

Thanks @aabedraba for reporting this. You should be able to copy the entire text from the Notifications pane, click on the bell icon in the top right.
But, we should still make the notification scrollable and maybe add the same copy text button to the popup.

You should be able to copy the entire text from the Notifications pane, click on the bell icon in the top right.

Great! Thanks, didn't know that.
I'd still wish to be able to copy specific parts as copying the entire messages in some occasions might not be useful.
But adding the copy button to the popup is also nice :)
Thanks again

I'll close this issue as #725 has it all!


Retention Experiment: Hover Commands

PR to implement the Hover Command feature: https://github.com/sourcegraph/cody/pull/3585
Retention Experiment Doc: https://docs.google.com/document/d/1G5koKO6MoZflObkkcsjcYfexfSY8fWC50BoT4k-BmZU/edit?usp=sharing

Feature implementation completed in https://github.com/sourcegraph/cody/pull/3585
Status: Experiment in Progress
Feature is included in v1.12.0
The feature flag has been enabled for n% of dotCom users.

Status: Test Ended
Context: https://sourcegraph.slack.com/archives/C05AGQYD528/p1713973441091799


Restrict people who can approve reviews. This is to ensure code quality.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@ad508e1). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1762   +/-   ##
=========================================
  Coverage          ?   82.23%
  Complexity        ?     1977
=========================================
  Files             ?      291
  Lines             ?     9172
  Branches          ?      884
=========================================
  Hits              ?     7543
  Misses            ?     1334
  Partials          ?      295

Continue to review full report at Codecov.

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


Add support for multiple scripts

I think by adding multiple entry points in webpack we can build multiple scripts. But there needs to be a way to specify script specific headers, as all scripts may not be having same headers.

Thanks for suggestion. I'll think about it. :)

Maybe the comment is a little bit too late.... but!
If you have come out with any solutions, PR is always welcome.
Sorry for inconvenience, since I am not a full-time and keep working on another project these days.
Maybe I'll come back to this package in the future, but not now.

@momocow Hey, no inconvenience at all, in fact a great starting point for working with tampermonkey scripts.
I was browsing through tools which could help me for a project I was supposed to work on in the coming months - when I stumbled across your tool :)
The project is still WIP, and when I'll get to the tampermonkey script part, I'll make sure to try and find a solution to the wonderful tool you've developed! :)
Again, thanks for being a open sourceror :D

Finally I'm back to this project again and pick up a totally different solution for integrating userscripts and Webpack. That is, make the package a Webpack plugin instead of a project starter kit to leave everything flexible and configurable for developers.
This changes also solves the issue for multiple entries, as anyone who is interesting in this can take a look at the test for multiple entries.
@shivam-tripathi I'll close this issues (a year after it was opened 😂). Feel free to reopen it or file a new one if there is anything unsolved.

The plugin is now available as<EMAIL_ADDRESS>on NPM.


Speedup CI

Use vendoring to skip the "Updating crates.io index" step, which takes a lot of time. The vendored dependencies are put into a submodule. @ogoffart Do you have permissions to create a new repository in the corrosion-rs organization? Currently, I just created a repository for the submodule in my own namespace, but ideally it would be under the organizations' namespace.
Use --locked if a lock-file is present when invoking cargo metadata.

You should have gotten an invitation to get more right to the corrosion-rs organization.
I'm not a big fan of using a submodule for this purpose.
Can we not reach the same effect by using proper caching in the CI with the cache action?

You should have gotten an invitation to get more right to the corrosion-rs organization.

Thanks, I received the invitation

I'm not a big fan of using a submodule for this purpose.

Just to clarify, the submodule would only be used in CI and users would never have to initialize the submodule.

Can we not reach the same effect by using proper caching in the CI with the cache action?

We could use the cache action to cache the crates-io index, but this would cache the whole index which is huge - That is why it is taking so long to update the index in the first place. I'm not sure if it is possible / sensible to use the action to cache a subset of the index.
For Linux we could use a prebuilt docker image, but that doesn't speed up the macOS and windows jobs.
The vendoring approach has the advantage that it only contains the select few crates we actually need (cc for a build script and the dependencies of the generator crate), so cloning the submodule is very fast.
In the future this will hopefully be resolved by the sparse checkout over http feature the cargo team is working on, but until this stabilizes and we bump our MSRV it will likely still require a long time.

If you think it is the best approach, i trust your judgement.

The vendoring approach has the advantage that it only contains the select few crates we actually need

The cache should normally also only cache the needed crates.
(But it is true that the index is 300M, that's quite big, i hope future version of cargo optimizes that.)

Superseded by #282.

We could use the cache action to cache the crates-io index, but this would cache the whole index which is huge

It turns out that using the cache action to restore the cargo registry is much faster than whatever git does, so just caching the whole cache is actually a viable option.


Getting the 'image' tag to work

Hi there, thanks for the library! I'm running into issues getting an Image element to work. I've added 'file-loader' plugin to my .babelrc and the file path is successfully loaded and prints out within a componentDidMount() here's my code:
import React, {Component} from 'react';
import logo from '../img/logo.png';

class App extends Component {
  render() {
    return (
      <box width={'100%'} height={'100%'} style={{ bg: 'white'}}>
        <image
          top={40}
          left={40}
          file={logo} />
      </box>
    );
  }
}

Any ideas?

Ok I figured it out, and would just delete this but just in case anyone else is looking i replaced:
file={logo}

with:
file={__dirname + logo}


fix(panic): Stop panicking when handling inbound connection handshakes

Motivation
In PR #6969 which just merged yesterday, I introduced a panic bug when handling some inbound handshake errors.
Complex Code or Requirements
The lack of FuturesUnordered types made bugs like this easy to add and tricky to catch.
Solution

Fix the panic by handling errors directly on the futures
Fix a potential panic on shutdown by logging when tasks are cancelled, rather than panicking unconditionally
Prevent panics in other refactors by adding types to errors and FuturesUnordered

Related cleanups:

Remove a redundant timeout in outbound handshakes

Review
This could be an urgent fix if it causes panics in other tests or full syncs.
Reviewer Checklist

[ ] Will the PR name make sense to users?

[ ] Does it need extra CHANGELOG info? (new features, breaking changes, large changes)

[ ] Are the PR labels correct?
[ ] Does the code do what the ticket and PR says?

[ ] Does it change concurrent code, unsafe code, or consensus rules?

[ ] How do you know it works? Does it have tests?

@mergifyio refresh


⚠ Bofrost.lu has degraded performance

In 84e837a, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2148 ms

Resolved: Bofrost.lu performance has improved in 69f8a27 after 6 minutes.


scroll to the top💡

when going through different pages of projects. When clicked on 1st or 2nd or 3rd page, the page should scroll to the top to go through the projects. This way it gives good look to the website.
And clicking on the add your project button the page goes directly to the down this makes user to scroll to the top.

when going through different pages of projects. When clicked on 1st or 2nd or 3rd page, the page should scroll to the top to go through the projects. This way it gives good look to the website. And clicking on the add your project button the page goes directly to the down this makes user to scroll to the top.

could you share a demo?

when going through different pages of projects. When clicked on 1st or 2nd or 3rd page, the page should scroll to the top to go through the projects. This way it gives good look to the website. And clicking on the add your project button the page goes directly to the down this makes user to scroll to the top.

could you share a demo?

https://user-images.githubusercontent.com/117016540/233785399-5b9d7b01-f6b4-4632-8940-bb26031392d8.mp4
In this video you can see, clicking on add your project button loads the page directly to down.

when going through different pages of projects. When clicked on 1st or 2nd or 3rd page, the page should scroll to the top to go through the projects. This way it gives good look to the website. And clicking on the add your project button the page goes directly to the down this makes user to scroll to the top.

could you share a demo?

Untitled.video.-.Made.with.Clipchamp.mp4
In this video you can see, clicking on add your project button loads the page directly to down.

yeah +1. assigning to you

when going through different pages of projects. When clicked on 1st or 2nd or 3rd page, the page should scroll to the top to go through the projects. This way it gives good look to the website. And clicking on the add your project button the page goes directly to the down this makes user to scroll to the top.

could you share a demo?

Untitled.video.-.Made.with.Clipchamp.mp4
In this video you can see, clicking on add your project button loads the page directly to down.

yeah +1. assigning to you

Ok. I will be working on it.

Completed. Thanks for your contributions 🎉 ❤


Marriage abroad/reduce cyprus to 2 outcomes

This is a follow up to #3064.
Trello card
Factcheck
Expected changes

URL on GOV.UK

Reduce the Cyprus to 2 outcomes:

1 for opposite sex
1 for same sex

Before

After

Outcomes

https://smart-answers-pr-3066.herokuapp.com/marriage-abroad/y/cyprus/opposite_sex
https://smart-answers-pr-3066.herokuapp.com/marriage-abroad/y/cyprus/same_sex

LGTM 👍


"Gearpump" or "GearPump"

We use both of the terms now. "Gearpump" has around 300 occurrences and "GearPump" 40 occurrences. I think we should reach a consensus on which to use and use it from now on.

Please change all to Gearpump


Geocoding not working in 3D Tiles Terrain Classification demo

The 3D Tiles Terrain Classification demo uses the geocoder to set the initial view, but no longer works after https://github.com/AnalyticalGraphicsInc/cesium/pull/6876.
Is there anything that looks wrong about the code below?
var geocoder = viewer.geocoder.viewModel;
geocoder.searchText = 'Vienna';
geocoder.flightDuration = 0.0;
geocoder.search();

@mramato would you have a chance to look into this?

Good catch.  This is an easy one to fix.  Basically you are calling the geocoder before terrain is ready and we weren't correctly waiting on the ready promise in the new code I added.  This would (almost) never happen in normal usage because terrain initializes faster than the user performing a geocode.
I'll open a PR to fix it.


Tab completion broken for mpv

Tab completion for mpv shows an error message.
Current behavior
Typing mpv <TAB> displays the following error:
 mpv (eval):1: command not found: _mpv
(eval):1: command not found: _mpv
(eval):1: command not found: _mpv

Expected behavior
Available completions for the command or at least the file names in the current directory.
Relevant Zsh configuration (.zshrc)
export ZSH=$HOME/.oh-my-zsh
export EDITOR=vim
#export EDITOR="/usr/local/bin/mate -w"
# export HISTCONTROL=ignoreboth:erasedups
HISTORY_IGNORE="(ls|cd|pwd|exit|cd ..)"
. ~/.aliases

ZSH_THEME="robbyrussell"
HYPHEN_INSENSITIVE="true"
DISABLE_AUTO_UPDATE="true"
COMPLETION_WAITING_DOTS="true"
DISABLE_UNTRACKED_FILES_DIRTY="true"
plugins=(
  git brew node npm osx python pip
)

source $ZSH/oh-my-zsh.sh

export LANG=en_US.UTF-8

# Set Spaceship ZSH as a prompt
autoload -U promptinit; promptinit
prompt spaceship

Environment
Spaceship version: 3.6.0
Zsh version: zsh --version
zsh 5.6.1 (x86_64-apple-darwin17.7.0)
Zsh framework: oh-my-zsh
Zsh plugin manager: None
Terminal emulator: iTerm2
Operating system: macOS 10.13.6 (17G65)
Screenshot

Possible solution
none
googling for the error shows this: https://github.com/lukechilds/zsh-better-npm-completion/issues/7
Installation Method:
npm install -g spaceship-prompt

Spaceship is not doing anything about completions, it is only about rendering the prompt, look for problems with how the zsh completions are being set up for you (I assume via oh-my-zsh).

@maximbaz: thanks for the heads up! Looks like this is a duplicate of https://github.com/robbyrussell/oh-my-zsh/issues/2092.
Running upgrade_oh_my_zsh fixed the issue.


Remove unnecessary JavaScript function applyStyle()

This function iterated over elements and converted data-* attributes to
style attributes of the same name. Instead of using JavaScript, simply
assign these values as style attributes to begin with. This removes the
JavaScript execution time and reduces the number of JavaScript files
included with the project as well as downloaded at runtime.

Good find!


⚠ Bofrost.de has degraded performance

In c5ff1d3, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 961 ms

Resolved: Bofrost.de performance has improved in 2c9a004 after 34 minutes.


Fix nested lists in service doc strings.

Fixes boto/boto3#310
There were three issues at play:

We were completely disabling newlines when going through lists
The doc strings contain a vestigial space after each <li> and we also add a space after a list item opening, resulting in two spaces before content starts.
Nested lists need to be indented, but the top level list should not be.

I had some tests to fix, and I updated, but I'm gonna take a look at the output to make sure everything looks good.

Closing this because the approach changed wildly


For some people, when opening folders the background disappears when the animation is going, and then it reappears when the animation finishes

Stems from the solution to #17.
For the life of me I can't figure out what tweak is causing this. I have a hunch this is a 64-bit issue, because the only people who are reporting this have 5Ses... And if that's the case then I can't fix it :/

seems to be part of another issue. closing for now


Replace sync-request with nodejs-file-downloader

Task name: -
Description:

Replaced sync-request with nodejs-file-downloader because the package is not supported anymore
Replaced sync methods with async because nodejs-file-downloader and nodejs makes only async requests

Documentation changes required: N
Added unit tests: N
Attached related issue: N
Checklist:

[ ] Task version was bumped - please check instruction how to do it
[ ] Checked that applied changes work as expected

Looks that after we changed node version from 20.3.1 to 20.11.0, the npm version came with it updated from 9.6.7 to 10.2.4 in pipelines, and then sync-request library not supported any more.
(checking which npm version came with node: https://nodejs.org/dist/index.json)
The tests work fine if we're using npm 9.

/azp run


Apply large knowledge graphs to your methods.

If large knowledge graph is input as the training set, with 500,000 number of entities and 1,000,000 triples, is there any way to reduce the overhead? Is it necessary to extract subgraphs?

Hi, thank you very much for your attention to our work. It is quite likely that the current model cannot directly handle such a large-scale dataset, so extracting subgraphs may be a good solution. In fact, you can replace the KG embedding module of MKGL with any other model, as well as it can be trained on the given set.


Inline loading design fix

[ ] Inline loading implemented design is not the same as spec.

Current implementation on next.carbon site

Spec

[ ] Design spec is missing finished states (both success and error)
Current implementation on next.carbon site

Shixie propose:

Some explorations on this state:

[ ] Guideline says inline loader replaces button. This feels weird to me based on below implementation. Button is too big / prominent and our inline loader type is only 12px G70.

Refs IBM/carbon-components#1749

CC @dakahn

Spec & Sketchfile
https://ibm.box.com/s/quzwd4vj94mj6i8wjbxdjuxn1xt6wret

Closing because fixes mentioned in this issue are now consolidated in issue https://github.com/IBM/carbon-components/issues/1906


change dot to marker.

BEFORE:

AFTER:

QQ: Who requested this change? Are we sure that this change is needed?

:+1:

When the map switches from static to dynamic mode it looks like the hotel icon is removed.
This change will make sure that the hotel/center pin is consistent between the static and dynamic modes.


Question: Are there any sidecar configuration optimizations

Currently,istio distributes all the services in the cluster to Sidecar, which will cause a huge waste of sidecar resources. Of course, we can assign CR: Sidecar to deliver some service envoy to the designated envoy. However, in a complex and large clustered environment, it is difficult to determine which services are dependent on. Infrastructure teams are often unaware of the complex dependencies of business services. So Here I want to ask if istio has some configuration optimizations. Like subscription services, lazy loading, etc.?
thanks

The only mechanism we offer for this is Sidecar CR. We do not have lazy loading yet. Tracked in https://github.com/istio/istio/issues/19681

Closing as duplicate of https://github.com/istio/istio/issues/19681. thanks


fix(notifications): update for new api response

:tada: This PR is included in version 2.3.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


fix overrides of VPC in function.json.

The project's VPC needs to be deep copied before passing it to the
function, otherwise overrides modify the project's settings.
Fixes #597.

awesome, wonder if reflect has copying, I imagine we'll have this with some of the other props as well


NIFI-12231 FetchSmb supports Move and Delete Completion Strategies

Summary
NIFI-12231
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[x] Apache NiFi Jira issue created

Pull Request Tracking

[x] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[x] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[x] Pull Request based on current revision of the main branch
[x] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[x] Build completed using mvn clean install -P contrib-check

[x] JDK 21

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[x] Documentation formatting appears as expected in rendered files

The integration tests workflow failed on FetchSmbIT for the following tests:
FetchSmbIT.testCompletionStrategyDelete:114 expected: <false> but was: <true>
FetchSmbIT.testCompletionStrategyMoveWithCreatingDirectory:149 expected: <false> but was: <true>

Thanks for the heads-up @exceptionfactory!
I rerun the integration test on GitHub and got the same errors. So they are not intermittent.
However, I did not manage to replicate it in my environments. It worked on Mac and now I also set up a similar environment to GitHub Actions (same OS, Java and Docker versions) but no luck.
I pushed some changes in order to get more info and also modified the logic a bit (just trial-and-error approach).

@exceptionfactory I managed to fix the tests by changing the logic in FetchSmb.
There were 2 overlapping sessions to the SMB server: one for reading the file and the other for executing the completions strategy (move or delete) in commitAsync(). I moved the commit out of the TWR block of the first session, so the "read" session gets closed before the "move/delete" session starts.
Parallel sessions should not cause issues as the file handles are closed properly and it worked in my environments (ran flows connecting to Windows share and Samba share, ran integrations tests on Mac and Linux). Not sure what the docker environment is doing in Github CI but this change fixed it. I re-ran the integrations tests 3 times and all were green.


[Bug]: Allowlist (seemingly) Does Not Work

I'm trying the simplest case I can--am I doing something wrong here, or is this a bug?
Accounts:
   "*":
    Checks:
      "iam_avoid_root_usage":
        Regions:
          - "*"
        Resources:
          - "*"

 python3.11 ./prowler.py aws --log-level ERROR -b -w ../allowlist.yaml -c iam_avoid_root_usage

This report is being generated using credentials below:

AWS-CLI Profile: [default] AWS Filter Region: [all]
AWS Account: [<redacted>] UserId: [<redacted>]
Caller Identity ARN: [arn:aws:iam::<redacted>:user/<redacted>]

Executing 1 checks, please wait...

-> Scan completed! |▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉| 1/1 [100%] in 2.8s

Overview Results:
╭───────────────────┬─────────────────╮
│ 100.0% (1) Failed │ 0.0% (0) Passed │
╰───────────────────┴─────────────────╯

Account <redacted> Scan Results (severity columns are for fails only):
╭────────────┬───────────┬──────────┬────────────┬────────┬──────────┬───────╮
│ Provider   │ Service   │ Status   │   Critical │   High │   Medium │   Low │
├────────────┼───────────┼──────────┼────────────┼────────┼──────────┼───────┤
│ aws        │ iam       │ FAIL (1) │          0 │      1 │        0 │     0 │
╰────────────┴───────────┴──────────┴────────────┴────────┴──────────┴───────╯

Please, complete the following information:

Resource: (e.g. EC2 instance, Fargate task, Docker container manually, EKS, Cloud9, CodeBuild, workstation, etc.)
OS: [e.g. Amazon Linux 2, Mac, Alpine, Windows, etc. ]

Mac

Prowler Version [./prowler --version]:
3.1.0

Python version [python --version]:
3.11

Pip version [pip --version]:

22.3.1

Installation method (Are you running it from pip package or cloning the github repo?):

pip

Thank you for reaching us out @sinkr !
Could you execute it with --verbose to see the findings pls?

@sergargar thank you for the response, with --verbose, the output is this:
FAIL us-west-2: Root user in the account was last accessed 29 days ago.

Thank for the info @sinkr, the resources value admit only regex expressions (* is not a valid one), you can try with the following allowlist for this case:
Accounts:
   "*":
    Checks:
      "iam_avoid_root_usage":
        Regions:
          - "*"
        Resources:
          - "root"

However, we discovered that for example  .* is currently not accepted as a regex expression right now. We'll fix it shortly.

Hi @sergargar, this above does not work for me, nor does the fully-qualified ARN of the root account, either.
I will wait for your regex .* fix--thank you for your help and Prowler!

The ARN won't work since the resource is called <root_account> (you can see it in the output files).
https://github.com/prowler-cloud/prowler/pull/1734 will solve the regex .* issue.
Thank you!

Thank you @sergargar when I check out the branch remove-re.escape and have the Resources regex as .*, it still fails.
Allowlist:
Accounts:
   "*":
    Checks:
      "iam_avoid_root_usage":
        Regions:
          - "*"
        Resources:
          - ".*"

git branch
  master
* remove-re.escape

$ python3.11 ./prowler.py aws --log-level ERROR -b -w ../allowlist.yaml -c iam_avoid_root_usage --verbose

This report is being generated using credentials below:

AWS-CLI Profile: [default] AWS Filter Region: [all]
AWS Account: [<redacted>] UserId: [<redacted>]
Caller Identity ARN: [arn:aws:iam::<redacted>:user/<redacted>]

Executing 1 checks, please wait...

Check ID: iam_avoid_root_usage - iam [high]
        FAIL us-west-2: Root user in the account was last accessed 29 days ago.

-> Scan completed! |▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉| 1/1 [100%] in 2.6s

Overview Results:
╭───────────────────┬─────────────────╮
│ 100.0% (1) Failed │ 0.0% (0) Passed │
╰───────────────────┴─────────────────╯

Account <redacted> Scan Results (severity columns are for fails only):
╭────────────┬───────────┬──────────┬────────────┬────────┬──────────┬───────╮
│ Provider   │ Service   │ Status   │   Critical │   High │   Medium │   Low │
├────────────┼───────────┼──────────┼────────────┼────────┼──────────┼───────┤
│ aws        │ iam       │ FAIL (1) │          0 │      1 │        0 │     0 │
╰────────────┴───────────┴──────────┴────────────┴────────┴──────────┴───────╯

I think that is not working because you have a malformed yaml, try to add an indentation to the Accounts key like this:
Allowlist:
Accounts:
   "*":
    Checks:
      "iam_avoid_root_usage":
        Regions:
          - "*"
        Resources:
          - ".*"

Let us know if that's the issue @sinkr

Good catch, thank you @sinkr !!
https://github.com/prowler-cloud/prowler/pull/1735 solves the documentation and add the validator.


How to show the skills ?

As a user of Skillz
I want to see the lists of my skills in the alphabetical order
So that i can search easily in my refered/completed skills

It's currently available when you click on one of the 4 categories and then on the left. It's a descending order based on the skill's level and desired level.
Is it what you described (except for the order) ?

We did a test @benjaminbrgs and I.
When I click on "Practice" category, "my skills" are not listed in alphabetical order and it is disturbing.


add k8s 1.17.16, 1.18.14, 1.19.6

added versions:

1.17.16
1.18.14
1.19.6

/lgtm

/lgtm

could you also update the comments for supported versions?

could you also update the comments for supported versions?

lgtm

lgtm


std.file whitespace fix

I merged a comment only PR that added a litte bit of doc.
It containted whitespace at the end.
This PR removes that whitespace.

Auto-merge toggled on

thx


[FEATURE] Change Price Asset on TrueUSD / TN

Users need to be able to see the USD value on the pair and trade in this order.

New pair was opened, it works with Coinranking, Orders can be placed and made and etc etc


Shared more properties between TextDocument and CustomDocument

For #77131
Shares a few additional properties between TextDocument and CustomDocument.
We may want to consider this since CustomDocument is supposed to be subclassed.  Once we ship the custom editor API, we need to be careful about adding additional properties to CustomDocument so that our new properties don't collide with property in one of the subclasses
/cc @eamodio @bpasero

@mjbvz are we ready for untitled custom documents? Just curious if this has  been tested.

@bpasero CustomTextEditor should support untitled files. I'm making sure CustomDocument also supports saving them as well
Right now, from an extension's point of view, saving an untitled editor should be equivalent to:

Save as
Close the untitled document
Open the newly saved document

The proposed MoveCustomEditor apis would make this better so

Ok, based on discussions I am going to merge this work. I don't see much of downside to including these additions in the custom editor api finalization and suspect that if we don't add them now, we'd end up adding them later on


Changing data structures

This pull request

Changes the original data structure for NFA by the data structure developed by Juraj and Lukas.
The original data structure for NFA is completely removed - keeping both would need another work which is not now a priority.
The new data structure currently is not connected to VM (which is still work in progress), and does not support union_rename and check on disjoints set of states of two automata (which is not supported by our representation). Therefore tests related to these things are turned off".
The new data structure is not compatible with the original Pytonh binding. therefore its compilation is turned off.

I'm going to create issue for documentation since we need to merge this ASAP.

One last suggestion: Usually before finishing PR in Perun, we tend to run linter over the source code. It's usually easier to fix these issues while you are in the context of the PR and it might come handy in future to have minimal technical debt.

Maybe cppcheck or clang-tidy, but @ondrik will maybe know what is the best.

ok, i've run cppcheck over code and processed warnings. Unfortunately, there is no easy way to install clang-tide on macos.


Add build variant toggle for custom resources

This will allow anyone using the libhmiremoting.so patch described here: https://github.com/BimmerGestalt/AAIdrive/discussions/801 to make modifications to RHMI resources.

external.gradle is just used to copy resources from the original APK files to the assets/carapplications directory, and doesn't influence what gets packaged in the final APK. Any files in the assets directory gets packaged, so you can add your own resources manually without needing to change external.gradle.

I think this feature is too small to need an additional variant dimension. Adding another variant adds another keyword to all the build step names, and would be required for custom manifests or if we wanted to ship different assets in different builds. I believe just a simple Gradle property would be fine for this usecase.
Check out what I mean: https://github.com/BimmerGestalt/AAIdrive/compare/rhmi_custom_resources

I think this feature is too small to need an additional variant dimension. Adding another variant adds another keyword to all the build step names, and would be required for custom manifests or if we wanted to ship different assets in different builds. I believe just a simple Gradle property would be fine for this usecase. Check out what I mean: https://github.com/BimmerGestalt/AAIdrive/compare/rhmi_custom_resources

Agreed, thank you.


Improvements to BVH

The heart of any high-performance ray tracer lies in the acceleration structure for ray intersections. The construction and traversal of this acceleration structure has the biggest impact on performance and is the most important thing to optimize.
Currently, we using a SAH-based BVH as described in pbrt. We are traversing this BVH on the GPU naively based off ideas in pbrt. Both the construction and traversal of this BVH don't consider GPU architecture (memory bandwidth, ray coherence, etc), and GPU-specific designs contribute to around a 100% or 200% speedup in rendering time.
This tickets serves as a reference to recent papers, and will be updated on a regular basis. There are advantages and disadvantages to every design, and it remains an open question about which design is best suited for use in WebGL (versus CUDA or other GPU APIs).

2017 - Gayot et al. - GPU Ray Tracing using Irregular Grids
2017 - Ylitie et al. - Efficient Incoherent Ray Traversal on GPUs Through Compressed Wide BVHs
2018 - Gu et al. - Fast and Dynamic Construction of Bounding Volume Hierarchies based on Loose Octrees
2019 - Lin et al. - Dual-Split Trees

Does anyone have experience or input with recent developments towards gpu-tailored acceleration structures that might work well with the renderer?

Heya, I thought you might be interested. I have 2 implementations of BVH in my engine https://github.com/Usnul/meep. The engine in MIT licensed, one BVH implementation is compact and another one is dynamic. Build times are pretty fast for both. The dynamic BVH implementation has refitting, so you can move things around, rotate, scale etc. My code is based on a bunch of slightly older papers. If you're interested - I could pull it out into a separate module so both projects can benefit from it.

@Usnul, your work sounds interesting. A dynamic BVH is a really neat idea since it's a start for dynamic scenes, and that sounds like a road we'd like to explore eventually. At this point we're trying to optimize BVH traversal as much as possible for static scenes, so that's what I'd like to explore first.
Where in that engine is the BVH code located? Does this compact BVH implementation traverse particularly fast on the GPU? Is it based off any papers or is there somewhere I can learn more about it, aside from the code itself?

hey @jaxry , the representation is based on an intel paper, i don't recall it off the top of my head, i kept poor notes at the time. The idea is probably the same as what you're doing, you basically allocate space for a perfectly balanced tree, and then just fill in the space, every node has a pre-determined location. It's "compact" in the sense that it's binary, but it can be quite wasteful if the BVH is poorly balanced. It's fast to build and is fast to traverse though. My implementation is CPU-side, so it uses uint8, and the like for compactness, on the GPU you have to be a bit more tricky due to GLSL type constraints and available data transfer formats (textures, i guess?).
The code for all BVH stuff is located here:
https://github.com/Usnul/meep/tree/master/src/model/core/bvh2
The binary format BVH is here:
https://github.com/Usnul/meep/blob/master/src/model/core/bvh2/binary/IndexedBinaryBVH.js
Just had another look, turns out I have two implementations of binary format for BVH, one is compact and the other one is fast. So if you're interested in the compact one, it's here:
https://github.com/Usnul/meep/blob/master/src/model/core/bvh2/binary/BinaryBVH.js
Nothing revolutionary here, the code is pretty short too. It's been a number of years since I wrote that code, so it may take me a little time to recall details, but I'm here for any questions!


[Android] [Feature request] Tag and pinned item colours

First of all, I'm loving this app  🤩
Thanks for all your work.
I have a couple of requests.

I would love to have colours assigned to tags which would help with visual identification, and would help tagged items stand out a bit more. Even if colours are randomly assigned to each tag, it would be helpful.

It would also be great if the background of pinned items were a different colour to all the other clipped items.  It's a little difficult to see the little pin on each pinned item.
A visible line between each clipped item would also help to differentiate where an item begins and ends too.

Wouldn't this make UI a bit ugly? Too much color can make it difficult to focus on the actual clip item. I'll experiment & will let you know.
Instead of background I'll try to add a visual indicator that might differentiate the pinned item.
My UI was inspired by AOSP Android 10's Phone app, you can notice how the item expands on clicking just like the Phone app. They don't have any visual indicators to separate item yet still people understands it. Do we really need it?

It may be ugly. I don't know. I'd have to see it to make that call. If you have some idea that makes the tags stand out a little more, that'd be great. I'm using the dark theme and the tags are a dark gray colour. A little difficult to see.

Cool. Even just white pins (for the dark theme) so they stand out a bit better.

Some of my clips have spaces between the lines of text and sometimes the view seems cluttered and the differentiation between clips is not clear . Especially when you have a few of them together, they're not so clearly defined. I just thought a thin line between each might help.
It's ultimately up to you though.

I'll think UI for 1,3 later!
This is the design that I came up with to shown pinned items (this will also reflect in suggestions as well), check here.

Ok, cool.
The image looks good!
Thanks!

This comment is to let you know that for request 1 I tried having so many colors on tags looks pretty bizarre from UX point of view so I'm dropping it. Also for 3, the link mostly doesn't make sense (I talked with some internal testers & they said there is no problem differentiating clips so the line is not important).
This doesn't mean I'm never going to add it but in future if I get multiple requests like this I'll consider & will re-open this issue again.


[Feature] JSON Serializer Should Not Fail When Loading Object With Unknown Properties

Summary
Currently, the editor reports an error and the game probably crashes when a deserialized object is serialized that contains a non-existent property on the class.
We should make the JSON serializer fail silently, but parse the object nonetheless.
This makes the handling of deprecated properties much easier.

Closed by #178.


Feature/dapp switch eth balance according to account

Fixes #2237
Short description
In addition to the origin issue, I agreed with @taleldayekh that showing both ETH balances on the main account screen is another helpful addition for the user. This is ofc only the case if a Raiden account is in use.
Definition of Done

[x] Steps to manually test the change have been documented
[x] Acceptance criteria are met
[ ] Change has been manually tested by the reviewer (dApp)

Steps to manually test the change (dApp)

Connect to dApp with Raiden account enabled
Navigate to the account screen
Make sure that there are now two ETH balances shown
Navigate to the UDC screen and click withdraw
Make sure that the shown ETH balance relates to the Raiden account (probably zero if you haven't transferred something)
Repeat this without using a Raiden account and make sure the expected changes in the UI are there

We don't understand what CodeCov wants to tell us here. Looks pretty broken. The changed/added tests are pretty good. Therefore this gets merged now.


Dataflow Add Data to Dataverse • "All Data In That Table Will Be Overwritten"

Warning
If you use Power Query to add data to an existing table, all data in that table will be overwritten.

This sentence needs more context. As written I would never use the Dataflows feature.
I read the warning literally as "the entire table is dropped and replaced by the Dataflow". I think it's trying to say that based on the Primary key of the Dataflow, all records and columns mapped from the dataflow will upsert the table.
Also, in the screenshot there's a "Delete rows that no longer exist in the query output". Would be great if that was documented too, as it probably would add additional clarification to the warning message.
Thanks,
Joel

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

ID: 12795c71-e2fa-4ad0-abca-95fe5c5c5c75
Version Independent ID: 9b7d14ce-f391-db56-c68d-e508fca06e0d
Content: Add data to a table in Microsoft Dataverse by using Power Query - Power Query
Content Source: powerquery-docs/dataflows/add-data-power-query.md
Service: powerquery
GitHub Login: @mllopis
Microsoft Alias: millopis

Hi @JoelLeichty
Thanks for the feedback. the warning has been clarified with more context.


Add "click on picture" feature to show it at full resolution

That would help when some details are too small to be seen on a small picture.

@RaananW Can we close this now?
Also what is the process for closing issues that have been addressed? I can look through and ping you, but don't wanna put your notifications on blast if that's too much.


:sparkles: Managed cluster events inventory for global hub manager

Summary
Provide the events inventory for managed clusters. The cluster event from the managed hub:
apiVersion: v1
items:
- action: AvailableUnknown
  apiVersion: v1
  eventTime: "2024-05-07T12:33:27.502217Z"
  firstTimestamp: null
  involvedObject:
    apiVersion: cluster.open-cluster-management.io/v1
    kind: ManagedCluster
    name: globalhub-cluster1
    namespace: globalhub-cluster1
    resourceVersion: "747936"
    uid: 39c1aeff-0f51-48f0-a380-2b36defa3f59
  kind: Event
  lastTimestamp: null
  message: The managed cluster (globalhub-cluster1) cannot connect to the hub cluster.
  metadata:
    creationTimestamp: "2024-05-07T12:33:27Z"
    name: globalhub-cluster1.17cd34e8c8b27fdd
    namespace: globalhub-cluster1
    resourceVersion: "754506"
    uid: c3c1ea9d-183e-4e7c-bb44-537f316669a6
  reason: AvailableUnknown
  reportingComponent: registration-controller
  reportingInstance: registration-controller-cluster-manager-registration-controller-6794cf54d9-j7lgm
  source: {}
  type: Warning
- action: Available
  apiVersion: v1
  eventTime: "2024-05-07T12:34:07.319090Z"
  firstTimestamp: null
  involvedObject:
    apiVersion: cluster.open-cluster-management.io/v1
    kind: ManagedCluster
    name: globalhub-cluster1
    namespace: globalhub-cluster1
    resourceVersion: "754531"
    uid: 39c1aeff-0f51-48f0-a380-2b36defa3f59
  kind: Event
  lastTimestamp: null
  message: The managed cluster (globalhub-cluster1) is available now
  metadata:
    creationTimestamp: "2024-05-07T12:34:07Z"
    name: globalhub-cluster1.17cd34f21203e39b
    namespace: globalhub-cluster1
    resourceVersion: "754951"
    uid: d1383786-f39a-4460-ace0-7dba464ced62
  reason: Available
  reportingComponent: klusterlet-agent
  reportingInstance: klusterlet-agent-klusterlet-agent-9f7f5d4d8-644hj
  source: {}
  type: Normal
- action: AvailableUnknown
  apiVersion: v1
  eventTime: "2024-05-07T12:40:59.895652Z"
  firstTimestamp: null
  involvedObject:
    apiVersion: cluster.open-cluster-management.io/v1
    kind: ManagedCluster
    name: globalhub-cluster1
    namespace: globalhub-cluster1
    resourceVersion: "755295"
    uid: 39c1aeff-0f51-48f0-a380-2b36defa3f59
  kind: Event
  lastTimestamp: null
  message: The managed cluster (globalhub-cluster1) cannot connect to the hub cluster.
  metadata:
    creationTimestamp: "2024-05-07T12:40:59Z"
    name: globalhub-cluster1.17cd35521d72897b
    namespace: globalhub-cluster1
    resourceVersion: "758945"
    uid: bf52a104-63ae-420c-a0fb-24aaa109c552
  reason: AvailableUnknown
  reportingComponent: registration-controller
  reportingInstance: registration-controller-cluster-manager-registration-controller-6794cf54d9-j7lgm
  source: {}
  type: Warning
kind: List
metadata:
  resourceVersion: ""

Deprecated source filed in event: https://kubernetes.io/docs/reference/using-api/deprecation-guide/#event-v125
Related issue(s)
References # https://issues.redhat.com/browse/ACM-9676

/cc @clyang82 @ldpliu

/retest

/test sonarcloud


tmux-continuum showing as off

I've installed tmux-ressurect and tmux-continuum via plugins. In my .tmux.conf I have the following:
set -g @plugin 'tmux-plugins/tmux-continuum'
set -g @plugin 'tmux-plugins/tmux-ressurect'
set -g @continuum-restore 'on'
set -g status-right 'Continuum: #{continuum_status}'
set -g @resurrect-strategy-vim 'session'
set -g @resurrect-capture-pane-contents 'on'
Normal ressurection works via explicit keybindings for it. However, continuum seems to never turn on - After I added the status to the tmux line, I can see that its only ever set to 'Off' :/ Any hints?
I'm running Linux Mint, and Tmux v. 2.2.

I faced the same issue. The docs don't talk about it but auto save is off by default.
Once I add
set -g @continuum-save-interval '10'
in my tmux.conf and reload config, I am able to see the status as 10.

Thanks! That seems to have done it. This should probably be added to the readme =)

That's really confusing. Please add it to readme

@continuum-save-interval is (or should be) set to 15 by default
https://github.com/tmux-plugins/tmux-continuum/blob/6e58336c288958a3b1ff7773fb945770126db16e/scripts/variables.sh#L7-L8
If it's not set, then that's a bug that should be fixed. Can anyone provide repro steps?

1.default, fresh installation of tmux
2. add this to .tmux.conf:
`set -g @plugin 'tmux-plugins/tpm'
set -g @plugin 'tmux-plugins/tmux-sensible'
set -g @plugin 'tmux-plugins/tmux-resurrect'
set -g @plugin 'tmux-plugins/tmux-continuum'
set -g @continuum-restore 'on'
set -g status-right 'Continuum: #{continuum_status}'
run '~/.tmux/plugins/tpm/tpm'
`
3. reload plugins (prefix, shift+I)
4. source tmux.conf
without adding set -g @continuum-save-interval '1' status will be off and for me also not working (but might have worked after 15 minutes)

This is confusing. Is anyone fixing this?


Add an alternative parquet reader implementation

As discussed, I've implemented an alternative parquet reader library, and a Parquet to CQL converter on top of it.
I've briefly described all the files and their order in README.md.
The changes are not production ready. In particular, some compression algorithms are not implemented yet (that one's easy), there are no comprehensive tests, and most importantly, I'm not confident in the input validation, because right now, I validate the metadata ad-hoc - that is, I check whether a piece of metadata makes sense only when I'm about to use it. It's too easy to miss some crucial check (e.g. size >= 0) that way, and I don't trust that I haven't done just that somewhere. Nevertheless, I think the code is ready for review.

update: my fault, custom thrift still used stock headers, which caused it to use wrong offsets for member attributes.
As for yesterday's comment - one more thing to split from this big patch would be the cql_reader - ideally, file_reader should exist in its own patch, and the cql_reader should be an optional app. But, again, that's for later, the patch in its current state is good for review as well.

I have addressed most of the comments.
Besides that, I have revised my error handling in e23d38a - I made the error messages more descriptive, added some outer try...catch wrappers to add context to the errors (i.e. in which column invalid data was found), added some missing validity checks, and changed some integer types. (Sorry for making another big, non-atomic commit, I will be more disciplined about that next time).


Incompatible with webpack

This module is unfortunately currently incompatible with webpack. It produces the following error when built in a webpack environment:
define cannot be used indirect

The underlying issue is actually with json-schema, but because that module hasn't been updated since 2012 I might suggest choosing a different tool.

@morgante And what tool might you suggest?  As far as I can tell, json-schema still works just fine, even though nobody has changed it in three years.

is-my-json-valid seems to be updated quite regularly, so it might be worth trying.
You're right that it does seem json-schema is still mostly working, so it might be worth considering a fork if the issue isn't addressed soon.

Plus 1 .

getting this issue as well.

Because we use this validator in several latency-sensitive, internet-facing services, in order to switch implementations, we'd need to verify that there are no semantic changes (i.e., the new implementation rejects any invalid inputs rejected by the old implementation, and accepts valid inputs accepted by the old implementation) and no significant performance regressions (since existing implementations vary in latency by orders of magnitude).  At the very least, any semantic differences would need to be well-understood and documented for a major bump of this module.  We'd also need to normalize the reported errors to match the existing ones that this library emits.
It's a fair bit of work for someone to find a new implementation, integrate it into this module (translating errors), do the semantic testing, and do the performance testing.  If someone steps up to do that, we'd be happy to review the test results and take that PR, as long as it meets the above requirements.
I'm sorry that the json-schema dependency is affecting people's ability to use this module with webpack, but I don't believe any of the current maintainers has the interest or time to do that work ourselves, especially to work around what sounds like a deficiency in webpack.

you just have to bump the json-schema version

Fixed in version 1.3.1.


Joined columns not created with matching custom schema options

I have a many-to-one join between two string fields with the MySQL collation set to ASCII. Unfortunately the join column generated by SchemaTool doesn't inherit this collation because the incorrect metadata key options is passed instead of customSchemaOptions.
Changing line 673 of SchemaTool.php to:
$columnOptions['customSchemaOptions'] = $fieldMapping['options'];

fixes the issue and the migration diff tool no longer tries to convert my ASCII fields to the default UTF-8 collation.

@NathanBaulch could you please send us a failing test case that reproduces that behaviour (maybe even the fix)? It would help us a lot to identify and fix the issue you're describing.
You can find examples on https://github.com/doctrine/doctrine2/tree/388afb46d0cb3ed0c51332e8df0de9e942c2690b/tests/Doctrine/Tests/ORM/Functional/Ticket

Anything new on this issue? @lcobucci

@tok-amsiq seen https://github.com/doctrine/doctrine2/commit/657a54da84765858e7e825c97faf54cfe8ff0039?

@Ocramius briefly yes :)
But since it hadn't been reviewed, I was just curious about the progress. I have the same issue and used the columnDefinition as a solution, but if this worked it would of course be preferred.

@tok-amsiq if you have problems with this specific issue, please do provide a reproducible test case as suggested in https://github.com/doctrine/doctrine2/issues/6823#issuecomment-347095121

@Ocramius I figured @NathanBaulch already had with 657a54d, but I'll look more into it when I have the time!


Provide specific image path for html output

I'm working on a Sapper/Svelte PWA.

Sapper uses a template.html static html file to generate the SPA index.html
If i do
pwa-asset-generator mathr-icon.png ./static/images/app-icons -o false -m static/site.webmanifest -i src/template.html -f true

The image path is of the like:
<link rel="apple-touch-icon" sizes="120x120" href="../static/images/app-icons/apple-icon-120.png">

Where it should be
<link rel="apple-touch-icon" sizes="120x120" href="./images/app-icons/apple-icon-120.png">

Describe the solution you'd like
I would like an additional parameter, for example:  iff (for Images Folder Href) to set the path to use in the html generation.

Hi @pixeline, thanks for the issue. This makes sense, it can make the lib more flexible for the expected output. I will have a look at it soon.

:tada: This issue has been resolved in version 2.3.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[PLT-7109 + PLT-7050] Test Run state retain + View HTML Report

https://input-output.atlassian.net/browse/PLT-7109
https://input-output.atlassian.net/browse/PLT-7050

Summary by CodeRabbit

New Feature: Added a new route "/report/:uuid" to view certification results.
New Feature: Introduced CertificationResult component for displaying the result of a certification process with download functionality.
Refactor: Updated Dockerfile and related scripts for improved containerization process.
Refactor: Enhanced Certification component with better state management, error handling, and navigation.
Refactor: Modified useLocalStorage hook to include a function that removes stored value from local storage.
Chore: Cleaned up unused files and directories, updated .dockerignore, .gitignore, and GitHub workflows.
Bug Fix: Fixed image path in TimelineItem component.

I do have an subscription but I'm not allowed to start a TestRun

Abort a test run
pressing the IOG Home icon will bring you back to the previous state of the run, which is ok, but in the commit hash input we have autocompleted the Run ID, I guess, instead of empty or the previous commit hash

At some point the server crashed, and in the localStarage the userDetails is changed to undefined (I guess is related to the previous comment I added), now we can't load the page anymore, we have a blank page

Subscription is active but unable to start test run - resolved in https://github.com/input-output-hk/dapps-certification-web/pull/6
Blank page with isLoggedIn:true and userDetails:undefined in localStorage - unable to find exact cause of issue, is existent in master as well. Will be tracked.

Other state pertaining issues are resolved. Please re-review


Multicast discovery not working when wireguard device present

Cheers, ran into a rather weird issue just now. Starting mkchromecast as normal (both versions in AUR, doesn't matter):
Mkchromecast v0.3.9
Creating Pulseaudio Sink...
Open Pavucontrol and Select the Mkchromecast Sink.
Starting Local Streaming Server
[Done]
Selected backend: parec
Selected audio codec: mp3
Default bitrate used: 192k
Default sample rate used: 44100Hz.
PID of main process: 284407
PID of streaming process: 284414
 * Serving Flask app 'mkchromecast.audio' (lazy loading)
 * Environment: production
   WARNING: This is a development server. Do not use it in a production deployment.
   Use a production WSGI server instead.
 * Debug mode: off
 * Running on http://<IP_ADDRESS>:5000/ (Press CTRL+C to quit)
Traceback (most recent call last):
  File "/usr/lib/python3.9/site-packages/soco/discovery.py", line 212, in any_soco
    d for d in cls._instances[cls._class_group].values() if d.is_visible
KeyError: 'SoCo'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/bin/mkchromecast", line 311, in <module>
    mk()
  File "/usr/bin/mkchromecast", line 59, in __init__
    self.audio_linux()
  File "/usr/bin/mkchromecast", line 78, in audio_linux
    self.cc.initialize_cast()
  File "/usr/share/mkchromecast/mkchromecast/cast.py", line 118, in initialize_cast
    zone = soco.discovery.any_soco()
  File "/usr/lib/python3.9/site-packages/soco/discovery.py", line 215, in any_soco
    devices = discover(allow_network_scan=allow_network_scan, **network_scan_kwargs)
  File "/usr/lib/python3.9/site-packages/soco/discovery.py", line 124, in discover
    _sock.sendto(really_utf8(PLAYER_SEARCH), (MCAST_GRP, MCAST_PORT))
OSError: [Errno 126] Required key not available

So I'm guessing it's an issue with my network interface, as it's sitting on a bond interface and probably has some deeper issues trying to do multicast there. Since other apps (e.g. noson) appear to have no problem finding the SONOS device, I'm guessing it's something to either be fixed in soco… or Python. (welp.)
FWIW, the networki device for <IP_ADDRESS>/24 is just a simple bond device with
[NetDev]
Name=xxxx
Kind=bond

[Bond]
Mode=active-backup
MIIMonitorSec=1

Debugging hints greatly appreciated, a quick googly-go didn't reveal anything too obvious to try with regard to multicast with python on bond devices.

This is entirely consistent with the device not offering multicast as per iputils:
2: xxxx: <POINTOPOINT,NOARP,UP,LOWER_UP> mtu 1400 qdisc noqueue state UNKNOWN group default qlen 1000
    link/none
    inet <IP_ADDRESS>/23 scope global b1hetzner
       valid_lft forever preferred_lft forever

SoCo v0.24.1 is now released, which should address this issue. Please re-open if this is not the case.


Tightly coupled architectures >> update to new format

https://github.com/cncf/glossary/blob/main/definitions/tightly_coupled_architectures.md
Please convert entry to the property format. View the style guide for more details.

Hello @CathPag @JasonMorgan , I would like to work on this one please

@TakshakRamteke it's now in the new format but if you'd still like to work it it needs to be improved before we can mark it as completed. Let me know and I'm happy to assign it.

@JasonMorgan Yeah sure why not I'll be more than happy to work on this one, but could you please tell me what do we need to improve in this one ?

Term has been assigned


staking top section fully functional

Fixes #185
Description
This PR implements the fetching and preparing of the data in the REX store for the use of the Components.
Test scenarios
Screenshots
My current state in Telos Wallet:

The same account state in the Teloscan website:

Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have cleaned up the code in the areas my change touches
[x] My changes generate no new warnings
[x] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings
[x] I have removed any unnecessary console messages
[x] I have included all english text to the translation file and/or created a new issue with the required translations for the currently supported languages
[x] I have tested for mobile functionality and responsiveness
[x] I have added appropriate test coverage

💯


multi-select tags with bootstrap theme - value in using btn-primary css styling?

When using multi-select with the bootstrap theme, it is possible to select (click on) a tag in the list of selected items and highlight this:

However, when using the select2 theme, there is no visible change when selecting (clicking on) a tag in the list:

I'm not sure if there's any value in being able to select/highlight an item in the list, as when using the bootstrap theme, but if it doesn't serve any real purpose, it could be beneficial to remove the btn-primary styling, as well as for the sake of consistency across the bootstrap/select2 themes.
The source of this issue (if it is an issue) appears to be in https://github.com/angular-ui/ui-select/blob/fc42fdfb5c80c86a2d23c373184f5221eca28af0/src/bootstrap/match-multiple.tpl.html

Looks like it was already solved long time ago

I've noticed this too, and I'm looking for a way NOT to add btn-primary class to the item when clicking on it. Is there a way to prevent this?

Tested with v0.12.0 and the issue still seems to be there, i.e. when using multi-select with the bootstrap theme, it is still possible to select (click on) a tag in the list of selected items and highlight this item.

This breaks my site's multiselect box as I have overwritten CSS for class .btn-primary. Please fix this. Overriding the whole thing with other CSS rule for now.


Invalidate configuration cache upon changes to included build logic build

@bot-gradle test RFN

OK, I've already triggered ReadyForNightly build for you.

@bot-gradle test RFN

OK, I've already triggered ReadyForNightly build for you.

Perhaps also update the user manual chapter and release notes to let people know that included builds can be used for build logic (but not for task inputs yet).


Add docs for 'test-features'

@ras0219-msft edit:
Feature: https://github.com/microsoft/vcpkg-tool/pull/802

Converted this to draft for now until tool PR is merged.

It could make sense to briefly recommend a configuration with a compiler cache. There is a lot repetition in compilation commands over different feature combinations.

I have never used one. But maybe this should be an own article that then gets linked from here.


Fix #4269

I propose the way to fix #4269 and add a new test.

Codecov Report

Merging #4270 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #4270      +/-   ##
==========================================
+ Coverage   79.76%   79.76%   +<.01%
==========================================
  Files         106      106
  Lines      141579   141579
==========================================
+ Hits       112926   112931       +5
+ Misses      28653    28648       -5

Impacted Files
Coverage Δ

src/edit.c
85.13% <100%> (+0.03%)
:arrow_up:

src/gui_gtk_x11.c
48.27% <0%> (-0.64%)
:arrow_down:

src/window.c
86.33% <0%> (-0.07%)
:arrow_down:

src/terminal.c
80.5% <0%> (+0.03%)
:arrow_up:

src/channel.c
83.54% <0%> (+0.07%)
:arrow_up:

src/ex_cmds.c
81.77% <0%> (+0.09%)
:arrow_up:

src/ex_cmds2.c
84.96% <0%> (+0.09%)
:arrow_up:

src/screen.c
80.72% <0%> (+0.1%)
:arrow_up:

src/syntax.c
79.74% <0%> (+0.12%)
:arrow_up:

... and 1 more

Continue to review full report at Codecov.

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

I propose the way to fix #4269 and add a new test.

Looks like this handles exiting Insert mode with CTRL-C, but #4269 just
uses Esc.
I also don't understand the relation with the change in
test_bufline.vim.
--
Facepalm statement #4: "3000 year old graves?  That's not possible, it's only
2014!"
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

Sorry, I misunderstood... I confused #4269 with the similar problem I found:
test.vim
func StatusLine()
  return ''
endfunc
set statusline=%!StatusLine()
set laststatus=2

$ vim --clean -S test.vim
i<C-C>

"-- INSERT --" is not cleared when feeding Ctrl-C to leave insert mode.

I also don't understand the relation with the change in
test_bufline.vim.

Regarding "Test_setbufline_getbufline()":
By this patch adding test which creates a new buffer, the number of buffers in tests can reach 1234 so setbufline(1234, ...) succeeds against our expectation.
Thus should use the buffer number that cannot be, such as bufnr('$') + 1.


WIP: Fix client span finishes before server

Resolves #78
Do not merge yet! The issue is still present
Signed-off-by: Pavol Loffay<EMAIL_ADDRESS>

I find the same problem and I want to know when to merge and what is the real reason to cause this problem though I view the source and then I can not understand .Could you please tell me the real reason


Unable to clone project

hi i try to clone project to my local but getting error in multiple files one example as below:
Please make sure you have the correct access rights
and the repository exists.
fatal: clone of 'git@github.com:embeddedartistry/cppcheck-rules.git' into submodule path 'C:/arduino-logger/build/analysis/cppcheck/cppcheck-rules' failed
Failed to clone 'analysis/cppcheck/cppcheck-rules'. Retry scheduled
Cloning into 'C:/arduino-logger/build/analysis/vale/vale-styleguide'...
fatal: Could not read from remote repository.
thank you

Ah, this means you don't have SSH keys set up for GitHub, which the git@ checkout requires. I will get the submodule switched to https:// though.

Should work for you now, please confirm!


ASW labview

[x] complete purchase request
[x] order DSC module
[x] install labview on testing computer
[ ] test database function and work with Chase
[ ] Install labview on aws computer
[ ] acid addition component (3d lookup table)
[ ] more code...

Combined with another issue


DefaultRequester does not handle gzipped or deflated content

Some servers return gzipped or deflated content without asking (setting appropriate header), DefaultRequester does not decompress content and passes it to latter stages, which causes some trouble.

I don't know if I consider this a bug. The DefaultRequester is a very basic, rudimentary implementation that does not support a lot of things (for instance https is also not supported, even though it might work for some pages). If you really want to handle such content, you should implement the IRequester interface yourself.

Just to add an additional remark: I would have provided a better implementation, however, since AngleSharp is a PCL I only have access to the WebRequester (and even that is limited). Therefore platform specific implementations are far better. For an example if you go with the Microsoft HTTP Client libraries you'll already be in a better shape. Another library I can only recommend is EasyHttp.

I get your point, but I guess this will be an issue in future, because people are using libs to save time and can be annoyed when realize that they have to write multiple extensions to make it work properly. Don't get me wrong, no complains from my side, AngleSharp is best HTML parser compared to what I've seen before, but it's just a thing to think of :)
I've been using MS HTTP client in PSLs for some time, don't see any reason to use other alternatives.

If you know an easy way to support any of those things (e.g. supporting gzipped content) then any help would be appreciated. I do not exclude improving the requester, I just think it should not be the main purpose of AngleSharp (I will do other libraries on top of AngleSharp, that already implement something like that, but right now I can't focus on that).


feat(index.html): set lang of tag html to locale

Browsers will recognize this attribute to determine what language a website uses and provide corresponding services like automatically translating.
Maybe some spiders will also get infomartion about a website from this attribute?
However, I think it's necessary for users to use standard language abbreviations when internationalization if set this attribute to locale.

You are right.
But I have implemented it with a new scheme, so it conflicts with your PR and I have to close it.

More info see https://valaxy.yyj.moe/posts/i18n


Fix closing unclosed database when using forced upgrade

Fixes #81

Looks good to me. FYI, it does upgrade the tools versions required for building, but assuming whoever is compiling has an updated SDK, it should be fine.


PeoplePicker Id Mismatch with Sharepoint

While using principalTypes={[15]} (All) I'm getting a weird behaviour.
The id's I'm getting from PeoplePicker does not match with my list ID's.
If I choose Adele Vance and save. PeoplePicker gives Adele vance id 12, however in my sharepoint list it's saved as an existing group with ID 12

This is my PeoplePicker configs :
PeoplePicker disabled={this.state.isInPreview || (this.currentMappings && this.currentMappings.get("TargetUsers") === "-No Mapping-")} personSelectionLimit={1000} context={this.props.context} titleText={this.props.strings.properties.targetUsers} placeholder={this.props.strings.properties.targetUsersplaceholder} groupName={""} // Leave this blank in case you want to filter from all users ensureUser={true} onChange={(e) => { let target: number[] = []; e.forEach((elem) => { target.push(parseInt(elem.id)); }); this.setState({ targetUsers: { results: target } }); }} principalTypes={[15]} defaultSelectedUsers={this.state.targeted} resolveDelay={500}
Am I doing something wrong? or is there an Id mismatch between people picker and sharepoint?

Are you aware that user IDs are site specific?
For example, if you have a PeoplePicker on /sites/siteA selecting an user ID and fill it into a list on /sites/siteB, this will select a different user/group in the target.

Hello!
Thank you for answering , You are correct, when I use PeoplePicker on the same site collection the id's don't mismatch... however something else also happens.
I have a security group (SG1) and if I edit my item and add SG1 in the peoplepicker it works, and saves in the list... however, next time I load the page , the SG1 group doesn't appear as "Selected" on the PeoplePicker.

I can add it to the list via PeoplePicker, but after it's added , it doesn't appear on PeoplePicker (eventhough it's already on the list) .
Also... about the first issue... is there any work around or does the webpart needs to be on the same site collection as the list it consumes?
Thank you so much for the help

@DiogoBolt - are you talking about PeoplePicker from PnP Controls?
If yes - please, close this issue and open a new one in the appropriate repo:

PnP React Controls: https://github.com/pnp/sp-dev-fx-controls-react
PnP Property Controls: https://github.com/pnp/sp-dev-fx-property-controls


#52 update redis imports

Update imports to point redis to current repository.  They stopped using gopkg.in

Can be closed in favour of passing #54


Respect bound callbacks. Add greater test coverage

I want to support the following behavior:
app.push(action, params, reactComponent.method)

Up until now I was binding all push callbacks to app. That is problematic for this use case. Instead, I now use call inside of an anonymous function to respect Function::bind.
While I was at it, I also added test coverage to make sure that errors throw inside of eventually are not absorbed by promises. What a mind bender...

:+1:

This seems like a good change. :+1:


Write more about how String NO_CHANGE and String MINUS_NULL are "equivalent"

I've gotten questions about it from both @kevinb9n ("JSpecification - kevinb review copy") and @netdpb ("dpb questions about the JSpecify nullness spec").
(We could maybe avoid this if we change our rules for Augmented type of a type usage appearing in code and maybe Substitution / Applying a nullness operator to an augmented type to avoid ever generating one of those two augmented types. But even then, we'd likely want to address it in order to explain why that change in rules is safe.)

Note also that E NO_CHANGE and E MINUS_NULL may (or may not) end up being fully equivalent in cases like <E extends @NonNull Object>. I worry that rules to ensure that we produce E MINUS_NULL in that case could be complex or dangerous. Still, if we want to guarantee that we always produce MINUS_NULL when it's equivalent to NO_CHANGE, then that's another case that we'd need to handle[*].
Also, https://github.com/jspecify/jspecify/pull/651 touches on the fact that, beyond whatever changes we'd need to make for the rules for computing the Augmented type of a type usage appearing in code, we'd likely also need to make changes to how we compute the augmented type of a fresh type variable produced by capture conversion.
[*] Not that we couldn't aim for the more modest goal of always generating Foo MINUS_NULL for non-type-variable types Foo. But even that could be complicated by javac's questionable behavior with class Foo<E extends Bar>+Foo<? super Bar>, which lead to capture conversion that produces Foo<Bar> instead of the JLS-approved Foo<CAP#1>, where CAP#1 has both a super Bar bound and an extends Bar bound. So we'd still need to modify capture conversion to ensure that we produce Bar MINUS_NULL there even if we aren't otherwise touching capture conversion. "Luckily," I think we might want to touch capture conversion for this case, anyway, since we would eventually want to handle the case in which the super and extends bounds have different nullnesses (which can't both be carried over because javac didn't produce a type variable), probably by setting the nullness to UNSPECIFIED. Sorry, this is all horribly obscure, but I wanted to at least write it down as a note to my future self.


adding missing http definition

please add these files to angular type definitions

@alexeagle ?

@jeffbcross do you know what's up with the missing http typings?
@zebalu thanks for the help - we generate the .d.ts bundle from our source though, so we'll overwrite your files at the next release.

Thanks for the PR. looks like this is not needed any longer. closing as per @alexeagle's comment.


Stricter timeout options parsing

fix #752

What are we waiting for?

@stoivo the tests are red, for one thing

Pushed with no changed and now spec are green

Very nice suggestion. So much more strait forward. It looks like it sneaked in a , in raise ArgumentError, "#{long} must be numeric", unless normalized[long].is_a?(Numeric)

After taking a closer look, I think we should do parsing a bit differently:
KEYS = %i[read write connect].to_h { |k| [k, :"#{k}_timeout"] }.freeze

class << self
  def normalize_options(options)
    normalized = {}
    original   = options.dup

    KEYS.each do |short, long|
      if original.key?(short) && original.key(long)
        raise ArgumentError, "can't pass both #{short} and #{long}"
      end

      normalized[long] = original.key?(long) ? original.delete(long) : original.delete(short)
      raise ArgumentError, "#{long} must be numeric", unless normalized[long].is_a?(Numeric)
    end

    if original.size.positive?
      raise ArgumentError, "unknown timeout options: #{original.keys.join(', ')}"
    end

    if normalized.empty?
      raise ArgumentError, "no timeout options given"
    end

    normalized
  end
end

You should be able to only set one of the actions callbacks like HTTP.timeout :read => 125?

I like how it looks but rubocop this it is too high complexity. I think it's nice
normalize_options instead of parse_options

rubocop offences
be rubocop -a                                                                                15:37:07
lib/http/timeout/per_operation.rb:17:9: C: Metrics/AbcSize: Assignment Branch Condition size for normalize_options is too high. [<5, 23, 9> 25.2/17]
        def normalize_options(options) ...
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
lib/http/timeout/per_operation.rb:17:9: C: Metrics/CyclomaticComplexity: Cyclomatic complexity for normalize_options is too high. [10/7]
        def normalize_options(options) ...
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
lib/http/timeout/per_operation.rb:17:9: C: Metrics/MethodLength: Method has too many lines. [11/10]
        def normalize_options(options) ...
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
lib/http/timeout/per_operation.rb:17:9: C: Metrics/PerceivedComplexity: Perceived complexity for normalize_options is too high. [10/8]
        def normalize_options(options) ...
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
 80/80 files
80 files inspected, 4 offenses detected

@ixti ping

oh. sorry, totally forgot. I'm afk till Monday, will merge as soon as get back

I've added a commit that disables inline some cops for that method, but I believe I found a regression. Right now it's possible to set global timeout with:
HTTP.timeout({ global: 161 })

And now that I'm looking at all of this again, I think next major we should add breaking change and:

change #timeout signature to #timeout(type, options = nil)
type will be one of: :none, :global, :per_operation
signature of Global#initialize should be (value)
signature of PerOperation#initialize should be (connect: 0, write: 0, read: 0)

@tarcieri WDYT?

Sounds fine to me although it would be a good time to consider potential other breaking changes too

I was comparing this HTTP client to others in Ruby and one thing I would like to add for timeouts is that global and per-operation are not mutually exclusive. I may want a request to time out after 1 minute, but spend no more than 150ms on the TCP connection portion or spend no more than 10 seconds idle on a read or write operation.
An alternative interpretation is what HTTPX does, which is to treat read as a timeout reading the response, write as a timeout writing the request and global as a overall timeout.
In my experience, except for streaming, HTTP clients care about total request timeout and maybe connect timeout, but definitely not per-operation timeouts. Streaming cares about operation timeouts and only optionally cares about a total timeout.
Often clients measure time to first byte from the server as well as overall latency, so timeouts on both of those could be helpful.
I am happy to help implement this type of behavior for 6.X if that is the target version.

@misalcedo I opened #773 to track a potential redesign of the timeout subsystem for http 6.x

@stoivo any chance you could rebase this and get the tests green? We'd like to get it merged before a v6.0.0 release

I will try to take a look at this this week

I am sorry. I was motivated to fix this when I opened the MR. I have since moved on to use httpx and faraday. I don't have the extra time to look into this.


Fix documentation to reflect correct behavior of publish_pending

The cron job publishes the count of nodes required and not the count of
pending jobs.
Signed-off-by: Raghu Raja<EMAIL_ADDRESS>

I wonder if we shouldn't just air our dirty laundry and explain that SGE/Torque work one way and SLRUM works another.  We can say where we want to go as well, but I think it will be confusing if we don't explain how it works today.

@bwbarrett Agreed. It should be noted that SLURM was contributed to CfnCluster and not something originally developed as part of the first pass. SLURM is quite a different in how it works and that has resulted in a lot of challenges. Not sure it will ever work as well as SGE.


Unable to attach or mount volumes: unmounted volumes=[config-volume secrets-efulfill-inline appd-agent-repo-java kube-api-access-gbht9 log-volume], unattached volumes=[config-volume secrets-efulfill-inline appd-agent-repo-java kube-api-access-gbht9 log-volume]: timed out waiting for the condition

“HPA is not working since 18th of February.
ASK:  Why is the volume mounted correctly when using manual scale pods but using HPA it has an issue?
Describe for HPA has no error / events:
************* HPA *******************
Name:                    excess-cartons-cpu-hpa
Namespace:               ns-store
Labels:                  app.kubernetes.io/managed-by=Helm
Annotations:             meta.helm.sh/release-name: excess-cartons-ppd
meta.helm.sh/release-namespace: ns-store
CreationTimestamp:       Thu, 17 Feb 2022 04:49:25 +0000
Reference:               Deployment/excess-cartons
Metrics:                 ( current / target )
resource cpu on pods:  0 / 213m
Min replicas:            2
Max replicas:            4
Deployment pods:         2 current / 2 desired
Conditions:
Type            Status  Reason               Message

AbleToScale     True    ScaleDownStabilized  recent recommendations were higher than current one, applying the highest recent recommendation
ScalingActive   True    ValidMetricFound     the HPA was able to successfully calculate a replica count from cpu resource
ScalingLimited  True    TooFewReplicas       the desired replica count is less than the minimum replica count
Events:
Name:                       excess-cartons-memory-hpa
Namespace:                  ns-store
Labels:                     app.kubernetes.io/managed-by=Helm
Annotations:                meta.helm.sh/release-name: excess-cartons-ppd
meta.helm.sh/release-namespace: ns-store
CreationTimestamp:          Thu, 17 Feb 2022 04:49:25 +0000
Reference:                  Deployment/excess-cartons
Metrics:                    ( current / target )
resource memory on pods:<PHONE_NUMBER>6m / 255Mi
Min replicas:               2
Max replicas:               4
Deployment pods:            2 current / 2 desired
Conditions:
Type            Status  Reason               Message

AbleToScale     True    ScaleDownStabilized  recent recommendations were higher than current one, applying the highest recent recommendation
ScalingActive   True    ValidMetricFound     the HPA was able to successfully calculate a replica count from memory resource
ScalingLimited  False   DesiredWithinRange   the desired count is within the acceptable range
Events:
Current version of the CSI driver and AKV provider being used:
madhu@Azure:~$ helm ls -n ns-csi
NAME                                            NAMESPACE       REVISION        UPDATED                                 STATUS          CHART                                   APP VERSION
csi-secrets-store-provider-azure-1620916355     ns-csi          4               2021-05-16 13:04:57.654586501 +0000 UTC deployed        csi-secrets-store-provider-azure-0.0.18 0.0.14
Kubernetes version: kubectl get nodes -o wide and kubectl version output:
NAME                                 STATUS   ROLES   AGE    VERSION   INTERNAL-IP     EXTERNAL-IP   OS-IMAGE             KERNEL-VERSION     CONTAINER-RUNTIME
aks-nodepool02-37501463-vmss000002   Ready    agent   101d   v1.21.2   <IP_ADDRESS>           Ubuntu 18.04.6 LTS   5.4.0-1063-azure   containerd://1.4.9+azure
aks-nodepool02-37501463-vmss000003   Ready    agent   101d   v1.21.2   <IP_ADDRESS>           Ubuntu 18.04.6 LTS   5.4.0-1063-azure   containerd://1.4.9+azure
aks-nodepool02-37501463-vmss00001n   Ready    agent   101d   v1.21.2   <IP_ADDRESS>             Ubuntu 18.04.6 LTS   5.4.0-1063-azure   containerd://1.4.9+azure
aks-nodepool02-37501463-vmss00001o   Ready    agent   101d   v1.21.2   <IP_ADDRESS>            Ubuntu 18.04.6 LTS   5.4.0-1063-azure   containerd://1.4.9+azure
aks-nodepool02-37501463-vmss00001p   Ready    agent   101d   v1.21.2   <IP_ADDRESS>           Ubuntu 18.04.6 LTS   5.4.0-1063-azure   containerd://1.4.9+azure
aks-nodepool02-37501463-vmss00001x   Ready    agent   101d   v1.21.2   <IP_ADDRESS>           Ubuntu 18.04.6 LTS   5.4.0-1063-azure   containerd://1.4.9+azure
aks-nodepool02-37501463-vmss000023   Ready    agent   19d    v1.21.2   <IP_ADDRESS>           Ubuntu 18.04.6 LTS   5.4.0-1063-azure   containerd://1.4.9+azure
aks-nodepool02-37501463-vmss00002h   Ready    agent   10d    v1.21.2   <IP_ADDRESS>           Ubuntu 18.04.6 LTS   5.4.0-1063-azure   containerd://1.4.9+azure
aks-nodepool02-37501463-vmss00002k   Ready    agent   7d9h   v1.21.2   <IP_ADDRESS>            Ubuntu 18.04.6 LTS   5.4.0-1063-azure   containerd://1.4.9+azure
madhu@Azure:~$ kubectl version
Client Version: version.Info{Major:"1", Minor:"23", GitVersion:"v1.23.4", GitCommit:"e6c093d87ea4cbb530a7b2ae91e54c0842d8308a", GitTreeState:"clean", BuildDate:"2022-02-16T12:38:05Z", GoVersion:"go1.17.7", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"21", GitVersion:"v1.21.2", GitCommit:"802eff1fe87ad2dd737ebbe891f30500b88beb00", GitTreeState:"clean", BuildDate:"2021-11-15T08:35:41Z", GoVersion:"go1.16.5", Compiler:"gc", Platform:"linux/amd64"}
WARNING: version difference between client (1.23) and server (1.21) exceeds the supported minor version skew of +/-1

Hi All
Whenever we are trying to scale up pods through HPA we are getting mount failures and pods are getting terminated.
But if we try to scale manually pods are coming up and need your support to resolve the same

Thanks for opening the issue!
The driver and provider version you're currently running is not supported. 0.0.14 release is a year old and there have been multiple releases since then. The project is now GA and here is the list of supported releases: https://azure.github.io/secrets-store-csi-driver-provider-azure/docs/#project-status.
Please upgrade to the latest supported release.
If you still run into any issues with any of the latest releases, provide the following details in this issue:

SecretProviderClass yaml
Deployment yaml
Logs from driver and provider pods running on the same node as your app. Refer to our troubleshooting guide: https://azure.github.io/secrets-store-csi-driver-provider-azure/docs/troubleshooting/


Add DYNAMIC_COMPILE in testJobTemplate

Add DYNAMIC_COMPILE and its description after BUILD_LIST in testJobTemplate
https://github.com/adoptium/aqa-tests/blob/master/buildenv/jenkins/testJobTemplate#L330
booleanParam('DYNAMIC_COMPILE', false, '''WIP. It only works with functional tests atm.

Optional. Use DYNAMIC_COMPILE to automatically compile the required test material.

When DYNAMIC_COMPILE is on:

- if there is no BUILD_LIST input, DYNAMIC_COMPILE will automatically figure out which test material to compile based on the test target.

- if there is BUILD_LIST input, the BUILD_LIST will be further refined based on the test target.''')

Thanks @akanbifatimah . PR https://github.com/adoptium/aqa-tests/issues/3031 is delivered.


Dodata vest za Solaris Trolino 24 DPP

👷 Deploy Preview for balkantransport processing.
🔨 Explore the source changes: 02c001409738017e67eb004391025f5f9f641521
🔍 Inspect the deploy log: https://app.netlify.com/sites/balkantransport/deploys/62135f6a907c310008c87ee0


Sui wallet zeroed nfts deleted,  please advise

Steps to Reproduce Issue
Gone into wallet again today balanced zeroed and all minted nft missing, requested tokens again and this time only giving 0.3 instead of blocks of 50k sui please see attached photo , im finding everytme i close down for the day and then revisit next day wallet is zeroed and i have to start over again is this correct as its in its test stage or should this not be happing could you advise please as i dont want to keep sending messages if your aware of the issues with wallet and could you confirm if you want me to keep raising these issues/bugs with sui wallet kind regards starlew77<EMAIL_ADDRESS>e.g.

Call function .
Use return value and call .

Expected Result
Specify what outcome you expected should have resulted, but didn't.
e.g.
Expected  to return 42.
Actual Result
Specify what the actual unexpected outcome was.
e.g.
 returned 41.
System Information

OS:
Compiler:

logged back out and back in now have this message and wallet empty again please advise ?


use cargo workspace

The repo contains multiple rust crates, which can be referenced in a top-level Cargo.toml workspace file.

It appears bumping the compound-v2 substreams-ethereum = "~0.2.0" dependency to "0.2.2-rc" causes duplicate definitions in the comptroller.rs output. This is preventing the use of a workspace, since it ends up with a build conflict:
error: failed to select a version for `substreams-ethereum`.
    ... required by package `substreams-uniswap-v2 v0.1.0 (/Users/ryan/Projects/Messari/substreams/uniswap-v2)`
versions that meet the requirements `^0.2.2-rc` are: 0.2.2-rc.2, 0.2.2-rc.1

all possible versions conflict with previously selected packages.

  previously selected package `substreams-ethereum v0.2.0`
    ... which satisfies dependency `substreams-ethereum = "~0.2.0"` of package `compoundv2-substreams v0.1.0 (/Users/ryan/Projects/Messari/substreams/compound-v2)`

failed to select a version for `substreams-ethereum` which could resolve this conflict

Seems there may be some more work within substreams-ethereum-abigen to resolve the duplicate code generation within the comptroller code gen.

Remove one event to unblock on comptroller within the ABI.
Modify the ABI to remove the shadowed event signature.

NOTE: Streaming fast may need to modify the ABI codegen to handle shadow signatures.

love to see more standardization like this


hooks option in @Table decorator is broken

Given a model foo.ts
@Table({hooks: {
    beforeCreate: (instance, options) => {}
}})
export default class Foo extends Model<Foo> {}

An attempt to load this model will throw Error: Model not initialized: "Foo" needs to be added to a Sequelize instance before "addHook" can be called. during the model's own initialization process.
Sequelize ultimately calls this.Addhook() for every item specified in a Table's hooks options property, but the options are processed by sequelize before a model's isInitialized property is set to true by sequelize-typescript.
Either the hooks property should be removed from typings or overrideStaticFunctions should not apply to addHook, so that sequelize's built-in hooks option can be used as designed.  I suggest the latter.

I'm a fan of the latter, selfishly, because it makes a use case of mine slightly cleaner :)
In my opinion, the error-message-displaying override breaks Sequelize.  Sequelize internally calls this function, ultimately within sequelize.define, which as far as things go is pretty important.  Sequelize would run as expected if this override were not in place and Sequelize could call its own implementation.  For those who would rely on the usefulness of a Model not initialized warning, I would hope anyone who attempts to use an uninitialized model should find a useful error message scrolling across their screen well before they start tacking hooks onto its instance.
Additionally, I'd advocate against removing hooks from typings.  I feel the hooks option is the most convenient way of applying helper methods which are defined elsewhere.  The project of mine which brought this issue to light, for example, relies on a snowflake-like service for primary key generation within a particular table.  I've found it convenient to be able to pass a hook function by reference:
import { baz } from '../helpers/baz'

@Table({hooks: {
    beforeCreate: baz // I can pass a reference to the hook function
}})
export default class Foo extends Model<Foo> {}

Rather than have to wrap it, using sequelize-typescript's decorators:
import { baz } from '../helpers/baz'

@Table
export default class Foo extends Model<Foo> {
    @BeforeCreate
    static doBaz(instance, options) {
        baz(instance, options); // This feels unnecessarily verbose
    }
}

This could also be solved by calling addHook directly on an initialized model instance, but I'd consider it even worse because it would decouple a model's hooks from its definition.
Thoughts?

@RobinBuschmann that makes perfect sense to me.  Want a PR?

@wbk PRs are welcome 👍


Why AWQ and GPTQ model slower than ORI model in concurrency?

I've conducted a performance comparison using VLLM version 0.3.3 on an 8 A800 GPU machine, employing four GPUs for testing 10,000 address parsing data points with a concurrency of 500. The results are as follows: 1638s for GPTQ, 2025s for AWQ, and 1468s for the Original method. Surprisingly, both GPTQ and AWQ performed slower than the original setup.
blow is my cmd and pic:
GPT-1638s-CUDA_VISIBLE_DEVICES=4,5,6,7 python -m vllm.entrypoints.openai.api_server     --model /xxx/Qwen1.5-72B-Chat-GPTQ-Int4/     --tensor-parallel-size 4     --port 30142 --dtype half --max-model-len 2000 --gpu-memory-utilization 0.95 --enforce-eager --quantization gptq
AWQ-2025s-CUDA_VISIBLE_DEVICES=4,5,6,7 python -m vllm.entrypoints.openai.api_server     --model /xxx/Qwen1.5-72B-Chat-AWQ/     --tensor-parallel-size 4     --port 30142 --dtype half --max-model-len 2000 --gpu-memory-utilization 0.95 --enforce-eager --quantization awq
ORI-1468s-CUDA_VISIBLE_DEVICES=4,5,6,7 python -m vllm.entrypoints.openai.api_server     --model /xxx/Qwen_Qwen1.5-72B-Chat/     --tensor-parallel-size 4     --port 30142 --dtype half --max-model-len 2000 --gpu-memory-utilization 0.95 --enforce-eager

Perhaps due to the current inference process being bottlenecked by compute, while quantization methods are primarily aimed at optimizing memory access, there is additional overhead during the dequantization , leading to a decrease in overall throughput.


test/decoder.py:84:34: F821 Undefined name data

https://github.com/google/liblc3/blob/5f5251ab7ddc5e5637d2e5200a2526cb976d1413/test/decoder.py#L84
% ruff check --output-format=github --select=E9,F63,F7,F82
Error: test/decoder.py:84:34: F821 Undefined name `data`

% ruff rule F821
undefined-name (F821)
Derived from the Pyflakes linter.
What it does
Checks for uses of undefined names.
Why is this bad?
An undefined name is likely to raise NameError at runtime.
Example
def double():
    return n * 2  # raises `NameError` if `n` is undefined when `double` is called

Use instead:
def double(n):
    return n * 2

References

Python documentation: Naming and binding

Thanks!
this should fix it.


Add package unit tests into CI

Hi @vyahello may I try a shot to resolve the issue? With naive PR

Hey @4lovi4, sure go ahead, I appreciate your efforts. I've left a few comments before proceeding with the request, please have a look.


Extensible IdentityClient

Could it be possible to extend the default IdentityClient?
In our use case, I would like to use the TokenIssuer to sign a JWT token containing other claims other than the ones required by Backstage, and I would like to specify also different signing algorithms.
I did it with version 0.5 but with version 1.1.0 I cannot anymore since IdentityClient is a class with a private constructor and there is no interface that I could implement.
Feature Suggestion
The IdentityClient could extend an interface that I could implement with a custom one (same thing we are doing for the TokenIssuer).
Another solution could be to take some of the configurations from outside (e.g. the signing algorithm).
Possible Implementation
Maybe also a more structured "extension plan" of the tokens management could be a good idea, the suggestions above are just to enable customizations.
Context
Since we liked a lot the way Backstage was creating JWT tokens, we wanted to use them to send these tokens also to third-party systems by just adding other custom claims.
This is a very powerful feature if TokenIssuer and IdentityClient are extensible, but it seems that instead of making them more extensible the direction is to close them more (no interface, no public constructor, etc).
Maybe I am missing some other cleaner way to customize this behavior?

Yep this is something that we want to do but haven't focused our efforts on yet. Would you like to make a concrete implementation proposal to drive this forward?

Hi! :) I would like to help making a concrete implementation. But before that, was there any reason to remove the IdentityApi interface that I'm missing?

@manuelscurti The IdentityApi is alive an well, for example with this implementation over here


Beta diagnostics

Reworked current Gui presentation and definition diagnostics, and added serialization of all CE definitions to json.
Set the system environment variable 'SolastaCEDiagnosticsDir' to point to you project diagnostics folder, e.g.
"C:\Users\passp\source\repos\SolastaCommunityExpansion\SolastaCommunityExpansion\Diagnostics"
When you run the game all gui presentations and definitions will be saved in 6 files

If these files have changes you will see them before commit and can then check nothing's fup'ed.
Please give it a try. Check one of your definitions and confirm it's serialized correctly in CE-Definitions.json.  Check that if you screw something it shows up.

I am not familiar with this type of process so i cant offer much in the way of reviewing other than noting that TA are obviously comfortable having missing translations / guiPresentations so is there a need to track their omissions or us?

Not sure tracking TA's missing translations etc is of any benefit other than we can tell them, although surely they know already? But I quite often see 'Feature/&xyz' in the game so they do need to fix them.