Generating System.Uris with a configurable number of path segments

I need the option of generating System.Uris with a configurable number of path segments. As far as I'm informed, the UriGenerator currently only generates Uris in the form of protocol://host (and I'm after protocol://host/segment where number of segments = n).
Unless I missed something, it's not currently supported and I would like to contribute the feature a pull request.
Referring to UriGenerator

Thanks for registering the issue 👍
Honestly, I'm a bit in doubt whether we should change the default behavior of the core. The default set of builders is not supposed to cover all the needs and is suitable for the common cases only. When the default behavior is not suitable for clients, they are expected to customize the fixture. In your case, you can create the customized version (e.g. CustomUriGenerator) and inject it to the fixture:
fixture.Customizations.Add(new CustomUriGenerator());

In any case, let's clarify your particular needs before making the final decision. Do you need to change the default number of segments globally, or it's somehow local (e.g. you need Uris with variable number of segments in the same test)? Why the default Uri format doesn't work for you?
Thanks.

Thanks for the feedback. I guess the best approach is to keep the library as common as possible and then customize as suggested; however, what actually is considered common varies greatly. I guess the System.Uri type is sufficiently complex such that the scale tips in favor of customization.
Should be fairly easy to write a generator that uses CreateMany to create path segments.

I guess the best approach is to keep the library as common as possible and then customize as suggested; however, what actually is considered common varies greatly.

Well, the current strategy is to create values than pass most obvious expectations, so code which doesn't cares about the values "scrupulously" will not complain. OTOH if some code has more advanced expectations, we are fine to not satisfy them - it's exactly what customization API is designed for 😉

guess the System.Uri type is sufficiently complex

Exactly. And the variety of the possible forms is too large to predict all the possible expectations. That's why my vote is to keep the codebase as is in a favor of ad-hoc adjustments in clients' projects.
@moodmosaic Could you please also share your opinion as well?

Perhaps, you can do what you describe by supplying a custom scheme name? If not, could you provide some sample/pseudo code before putting effort on a proper pull request?

Closing this one due to inactivity. Please re-open if further assistance is required.

Got derailed .. and managed to return to the thread as I'm facing a similar requirement in another project. As previously suggested, I've implemented a specimen builder that generates a Uri with a configurable number of path segments.
It was debated if the stock UriGenerator could be refactored to cater for a wider set of use cases, and while it's a possible route, I'd personally argue that it's better to leave the default generators as simple as possible and costumize on a per project basis (the Uri class for instance has many aspects).
Considerations as follows:

Resolve the segments argument using the internal specimen builder
Expose the internal specimen builder as a constructor argument

Here's the source code:
/// <summary>
/// Creates new <see cref="Uri"/> instances.
/// </summary>
public class CompositeUriGenerator : ISpecimenBuilder
{
    private readonly ISpecimenBuilder builder = new UriGenerator();
    private readonly int segments;

    /// <summary>
    /// </summary>
    /// <param name="segments">Requested number of <see cref="Uri"/> segments.</param>
    public CompositeUriGenerator(int segments = 3)
    {
        if (segments <= 0) { throw new ArgumentOutOfRangeException(nameof(segments)); }

        this.segments = segments;
    }

    /// <summary>
    /// Creates a new specimen based on a request.
    /// </summary>
    /// <param name="request">The request that describes what to create.</param>
    /// <param name="context">A context that can be used to create other specimens.</param>
    /// <returns>
    /// The requested specimen if possible; otherwise a <see cref="NoSpecimen"/> instance.
    /// </returns>
    public object Create(object request, ISpecimenContext context)
    {
        if (context == null) throw new ArgumentNullException(nameof(context));

        if (typeof(Uri).Equals(request) == false) { return new NoSpecimen(); }

        var baseUri = this.builder.Create(request, context) as Uri;

        if (baseUri == null) { return new NoSpecimen(); }

        var path = string.Join("/",
            Enumerable
                .Range(0, this.segments)
                .Select(p => context.Resolve(typeof(string)) as string));

        return new UriBuilder(baseUri) { Path = path }.Uri;
    }
}


Code completion missing some of c++ keyword

Such as typename, using ...

Can you give more info? If I type "using" or "typename" at the top of an empty cc file, I see these completions.

Thanks for the quick reply. @sam-mccall
Code completion always gives typename as a code snippet.


Make PyMatting available on conda-forge

PyMatting can be made available on conda-forge by submitting a pull request to the staged-reciples repository. Detailed instructions can be found here.

Pymatting is now installable via conda forge. https://github.com/conda-forge/pymatting-feedstock
https://anaconda.org/conda-forge/pymatting

Thanks! It seems to have worked 👍


Write result duration into local database

Records the duration time of the one test to a local database.
Doesn't actually work yet - if you run pytest --benchmark it gets as far as creating a benchmark.db file locally before erroring with
E       sqlalchemy.exc.OperationalError: (sqlite3.OperationalError) no such table: test_run

This actually works now - you can run it with the benchmark, and then see the results with
from sqlalchemy import create_engine, text
from sqlalchemy.orm import sessionmaker

# Connect to the SQLite database
engine = create_engine('sqlite:///benchmark.db')

# Create a session
Session = sessionmaker(bind=engine)
session = Session()

# Query the existing table and print the results
query = text("SELECT * FROM test_run")
result = session.execute(query)
records = result.fetchall()

for record in records:
    print(record)

# Close the session
session.close()

which gives
(1, '30b586c2259e425ab833d87ae235f49c', 'test_quad_means[configs/local_single-threaded.yaml]', 'test_quad_means', 'benchmarks/test_array.py', None, None, None, '0.13.0', '2024.1.1.dev31+gdb9e448a.d20240203', '0.0.4', '3.1.2', '3.12.0.final.0', 'darwin', None, '2024-03-01 18:39:24.832665', '2024-03-01 18:39:26.787146', 1.9544808864593506, None, None, None, None, None, None)
(2, 'fde11a6e3d204982a3460ac3a7881acb', 'test_quad_means[configs/local_single-threaded.yaml]', 'test_quad_means', 'benchmarks/test_array.py', None, None, None, '0.13.0', '2024.1.1.dev31+gdb9e448a.d20240203', '0.0.4', '3.1.2', '3.12.0.final.0', 'darwin', None, '2024-03-01 18:43:49.368711', '2024-03-01 18:43:51.334510', 1.96579909324646, None, None, None, None, None, None)

Here you can see two entries in the test_run table, including timestamps for measuring test duration (i.e. the test_quad_means test took 1.95 then 1.96 seconds to run).

(There is a commit I forgot to push before merging this - see main for an actually-working version)


Add author/creator to rss feed

I started using Reeder and noticed you didn't have an author tag.  Finally had an excuse to contribute 😄 .
Atom specification requires email addresses to be supplied when
using <author>. Use Dublin Core <dc:creator> instead. Replace
& with and for proper rss validation.
If you don't mind having your email addresses in the rss feed we can use:
<author>
  <name> <%= post.developer.name %> </name>
  <email> <%= post.developer.email %> </email>
</author>

I'm assuming name and email exist due to their presence in validate_required.

I agree @tmock12 <dc:creator> seems better suited for this application– we don't want contributor's emails in the RSS feed. Great observation @codyjroberts.

Just let me know if you wan't me to squash 👍

Looks great @codyjroberts -- please squash into one commit and I will merge this.

Thank you!!! 👍


resolves dawgpath-110

hides coi graph from screenreaders, adds aria labels on abbreviations

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
394

Relevant Lines:
554

💛  - Coveralls


Provide source / project files for case

Thank you for providing such a great case for the redox keyboard. It's one of the most aesthetic and comfortable ones out there.
I tried to use the redox pcb with a kb2040 (which is slightly thicker) and the case doesn't fit. Furthermore peel-a-way sockets are nearly impossible to source for me.
Would you please provide the project files (openscad, fusion 360, freecad - any is ok) for the rev1.0 case? I need to have approximately 2 mm more clearance below the pcb to accomodate for the thicher kb2040 and a slightly thicker socket. Plus the TRRS port is a bit off on the PCB. Fixing these things would be easiest with a project file.
Slicing and patching the STEP files is quite a pain although I appreciate that you included those over STL files.

I'm afraid I've not saved the project files anywhere, it was done all in freecad but since it's kinda unreliable at times I thought the step files would be a better way of storing the info.
The offset trrs was on purpose as for the revision of the pcb that I got (the first one i think) the trrs socket was offset differently in both halves.
I'm sorry to not be able to provide the proyect files however if you do make any changes and want to fork or merge your  modifications with this repo feel free to open a pull request.
Thanks for the kind words and using the case.


Fix/issue 586

Codecov Report

Merging #618 into rusk-ipc will increase coverage by 0.40%.
The diff coverage is 58.55%.

@@             Coverage Diff              @@
##           rusk-ipc     #618      +/-   ##
============================================
+ Coverage     54.96%   55.37%   +0.40%
============================================
  Files           156      159       +3
  Lines          9689    10428     +739
============================================
+ Hits           5326     5774     +448
- Misses         3723     3945     +222
- Partials        640      709      +69

Impacted Files
Coverage Δ

pkg/config/consts.go
0.00% <ø> (ø)

pkg/core/consensus/collector.go
0.00% <0.00%> (ø)

pkg/core/consensus/header/header.go
24.75% <0.00%> (ø)

pkg/gql/handler.go
0.00% <ø> (ø)

pkg/gql/httpserver.go
65.11% <0.00%> (ø)

pkg/gql/query/mempool.go
31.03% <ø> (ø)

pkg/gql/query/transactions.go
63.86% <0.00%> (+2.07%)
:arrow_up:

pkg/p2p/peer/peer.go
69.93% <0.00%> (ø)

pkg/p2p/peer/responding/datarequestor.go
46.00% <ø> (ø)

pkg/p2p/wire/message/agreement.go
39.70% <0.00%> (-2.49%)
:arrow_down:

... and 53 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 841170e...e44dd33. Read the comment docs.


PUBG Finish Placement Prediction (Kernels Only)

项目描述：
So, where we droppin' boys and girls?
Battle Royale-style video games have taken the world by storm. 100 players are dropped onto an island empty-handed and must explore, scavenge, and eliminate other players until only one is left standing, all while the play zone continues to shrink.
PlayerUnknown's BattleGrounds (PUBG) has enjoyed massive popularity. With over 50 million copies sold, it's the fifth best selling game of all time, and has millions of active monthly players.
The team at PUBG has made official game data available for the public to explore and scavenge outside of "The Blue Circle." This competition is not an official or affiliated PUBG site - Kaggle collected data made possible through the PUBG Developer API.
You are given over 65,000 games' worth of anonymized player data, split into training and testing sets, and asked to predict final placement from final in-game stats and initial player ratings.
What's the best strategy to win in PUBG?  Let's let the data do the talking!
项目地址：https://www.kaggle.com/c/pubg-finish-placement-prediction
需整理或翻译的地址：https://www.kaggle.com/deffro/eda-is-fun

Finished.#6


Fix addProperty message

The following is printed to the screen when i run addProperty 1 address/Block 70B town/Bukit Batok type/Condo size/150 bed/3 bath/2 price/200000. But ideally, a simple message might be better
John Doe; Phone: 87431234; Email<EMAIL_ADDRESS>Address: Blk 123 Clementi Ave 3, #12-34; Remark: Looking for a 3-bedroom condo; Birthday: 1985-05-12; Remark: Looking for a 3-bedroom condo; Tags: [longTerm][buyer]; DateOfCreation: 2024-01-05; History: Date of Creation: 2024-01-05
[2024-02-15]:
  [2024-02-15] Meeting about 3-bedroom condo
; PropertyList: Property List:
Property at 123 Maple Street, Woodlands (Condo): 120.50 sqm, 3 bed, 2 bath - $850000.00
Property at 456 Elm Street, Bukit Timah (Landed): 250.00 sqm, 5 bed, 4 bath - $2200000.00
Property at Block 70B, Bukit Batok (Condo): 150.00 sqm, 3 bed, 2 bath - $200000.00

fixed


Disable posting to slack

I missed the ability to disable posting to Slack that was introduced in 6ec4bbe in response to #35. This PR brings it back, by letting a user set :slackistrano  to  false.

@r3trofitted Can you take a look at this? https://github.com/issues?utf8=✓&q=is%3Aopen+user%3Aphallstrom+sort%3Aupdated-desc+

😕 I not sure that I get what you'd like me to take a look at (?)
(But, thanks to your notification, I've seen and fixed the broken build, so thanks 👍)

@r3trofitted well despite the fact I pasted the completely wrong URL you figured out what I'd meant to say anyway.  Heh. Will review shortly.

@r3trofitted Sorry this took so long. 3.1.1 has been pushed to rubygems with this feature. Thanks!

🎉👍🙇


uefi: Add TryFrom u8 slice to DevicePathNode

Adds the capability to convert a byte slice to a DevicePathNode. This will be used to convert a byte slice to a DevicePath in a followup PR, which allows easier interaction with the data returned from UEFI variable services.

Checklist

[ ] Sensible git history (for example, squash "typo" or "fix" commits). See the Rewriting History guide for help.
[ ] Update the changelog (if necessary)

I assume you are going to rebase this onto #1199, right?


Tab Bar position absolute not visible

Code
import React, { Component } from 'react';
import {
  AppRegistry,
  StyleSheet,
  Text,
  View
} from 'react-native';
import { TabNavigator } from 'react-navigation'

class Index extends Component {
  render() {
    return <View style={styles.container}>
      <Text style={styles.welcome}>
        Welcome to React Native!
      </Text>
      <Text style={styles.instructions}>
        To get started, edit index.android.js
      </Text>
      <Text style={styles.instructions}>
        Double tap R on your keyboard to reload,{'\n'}
        Shake or press menu button for dev menu
      </Text>
    </View>
  }
}

const routeConfig = {
  index: {screen: Index}
}

const tabBarConfig = {
  tabBarOptions: {
    style: {position: 'absolute', flex: 1}
  }
}

export const AppNavigator = TabNavigator(routeConfig, tabBarConfig)

export default class ReactNavigationTest extends Component {
  render() {
    return <AppNavigator />
  }
}

Current Behavior
TabBarComponent won't display on Android. Setting fixed width, height, position doesn't change anything.
Expected Behavior
TabBarComponent should be visible. This worked fine in   1.0.0-beta.9. Doesn't work in  1.0.0-beta.10 or  1.0.0-beta.11.
Your Environment

software
version

react-navigation
1.0.0-beta.11

react-native
0.44.0

node

npm or yarn

I have the same problem too on iOS, tried this for a TabNavigator but the tabs disappear when I changed the position to 'absolute' from the default 'relative'
       tabBarOptions: {         style: {           position: 'absolute',           top: 100         }     }
Seems to be able to work when I used the default relative instead of absolute though.

Not sure if this really worked in older versions--because I had this same issue in older versions--but basically you can't absolute position it using the default navigators and tab bar components. Flexbox expects to flow the Views together in a certain way.
You're gonna have to accept that and make a custom TabBar. It's not that hard, especially for a TabNavigator (it's the StackNavigator that is very hard).
You have 2 options:

EASIER: create a custom TabBar and pass it to the tabBarComponent option. I suggest copy/pasting the code from React Navigation's implementation and then customizing. TIP: you can import from src/views/Etc.js. I.e. you can import anything.

HARDER: follow what this guy recently did where he made his own navigator using the default TabRouter:

https://github.com/react-community/react-navigation/issues/1686

class TabBar extends Component {
  render() {
    return <View style={{position: 'absolute', height: 100, width: 100, backgroundColor: 'red'}} />
  }
}

const tabBarConfig = {
  tabBarComponent: TabBar
}

We use a custom tab bar for our app. The result is the same. Its visible in 1.0.0-beta.9 but not in 1.0.0-beta.11.

are you using React Native Debugger? Inspect the component tree and see what View is the culprit. Then find where in the React Navigation source code the problem is, then compare beta 9 to beta 10 and provide a PR setting it back etc etc.

I have the same issue. By investigating, I found that the change that is causing the issue is the react-native-tab-view version bump. More precisely, the changes introduced in 0.0.63 from this commit.

was this fixed ? running into the same issue on android 1.0.0-beta.11

Pinging OP @mattjfong since this issue is quite old and related to an old version of the lib. If no answer I'll close in 7 days.

Hi there @mattjfong ,
In an effort to cleanup this project and prioritize a bit, since this issue had no follow up since my last comment I'm going to close it.
If you are still having the issue with the latest version (especially if it's a bug report) please check the current open issues, and if you can't find a duplicate, open a new one that uses the new Issue Template to provide some more details to help us solve it.


Readme.md states "only MQTT 3.1.1 supported" but code base seems to indicate MQTT 5 is also supported

Describe the bug
Readme.md states "only MQTT 3.1.1 supported" but code base seems to indicate MQTT 5 is also supported
To Reproduce
Steps to reproduce the behavior:

Go to 'Readme.md'
reconcile this with the tests found here: https://github.com/streamnative/mop/tree/master/tests/src/test/java/io/streamnative/pulsar/handlers/mqtt/mqtt5/hivemq

Expected behavior
not sure from reading the Readme.md which versions of MQTT are actually supported
Please explain or fix -- thanks

I'm not a maintainer so I can't speak definitively, but if you're looking at the main line you're seeing the current effort to support MQTT 5. I wouldn't be surprised if there are some MQTT 5 features that have landed in releases too, because some MQTT 5 spec differences only impact the broker and thus don't require a full working MQTT 5 implementation. There is an ongoing ticket to support MQTT 5, and that work is all in main line. The README on main is tracking what users expect to see, so it's still saying 3.1.1 support only, which is largely still true. I cannot speak to the timeline of these features, or whether or not certain MQTT 5 features have been pushed out in any releases. But as far as I know there is no MQTT 5 support in any release, and the effort in on going to support it.


Print mod source that called deprecated 'intllib.Getter'

Helps in debugging which mods need updated.

I just realized that I can set log output level to 'info' or 'verbose' which will show what mod was loading when the warning occurred. So this PR is not necessary, but in my opinion, may still be helpful.

d01a9c666f2d9f44a35562154ff86c7ce20e3e65
I reformatted the code a bit and added a few more checks (I'm a bit paranoid about debug).
Thanks!


Update TS and Node types versions for TypeScript Template

We need at least TS 4.1 to fix the occurrence of this issue in VS.
Two things

These versions were acquired via npm install --save-dev typescript @types/node. There is nothing special about them, they are just the latest published at this time. As long as the major version is ^4 we can change these to anything reasonable.
We should probably also update these versions for the other TypeScript* templates. Any reason not to?

Added JSTS first responders.


[Beam-10166] Improve execution time errors

Added a base skeleton for error message returned out of DoFns. Currently only for ParDo.
Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

retest this please

R: @lostluck

The licence failure is unrelated to this PR at least.

Run GoPortable PreCommit


No such module 'Jobs'

can't add as dependency to vapor 3 swift 4.1 xcode 9.3

I had the same, but I simply forgot to add it in the target dependency list
dependencies: [
.package(url: "https://github.com/vapor/vapor.git", from: "3.0.0"),
.package(url: "https://github.com/vapor/fluent-sqlite.git", from: "3.0.0"),
.package(url: "https://github.com/BrettRToomey/Jobs.git", from: "1.1.1")
],
targets: [
.target(name: "App", dependencies: ["FluentSQLite", "Vapor", "Jobs"]),
.target(name: "Run", dependencies: ["App"]),
.testTarget(name: "AppTests", dependencies: ["App"])
]


test: test proxy/gateway using existing integration tests

Instead of writing a bunch of proxy tests to check behavior that's already being tested, the integration package could be configured (possibly via build tag) to create proxies with every member and have the client connections go through the proxy.

CI is stable for integration tests. Still missing clientv3 tests, but that's testing local proxying, so can defer until 3.2.


Broken fuzzy search highlighting/ranking for trimable strings

VSCode Version: 1.28.2
OS Version: macOS 10.13.5

Fuzzy messy (https://github.com/Microsoft/vscode/issues/34088) search gives wrong highlights and inconsistent rankings when the quickpick entries start with some whitespace characters.
In the following GIF I'll be making two searches, in the first one the entries start with whitespace characters, in the second one they don't, there are some issues:

in the first search the wrong characters are highlighted. Notice also how removing the hyphen in vscode-todo causes the highlighted characters to go from vscode-todo to vsc and ode- for the vscode-todo-plus entry, this isn't happening if the entries don't start with whitespace characters.

In the second search you can see how removing the hyphen from vscode-todo leads to different rankings. And that hyphens obviously shouldn't matter since all entries have it in the same exact position.

IMHO the root cause of this is that for some reason proper fuzzy search is still not available everywhere, instead it looks like there are lots of special cases for supposedly improving the ranking. Shouldn't this be fixed sooner rather than later?

/duplicate https://github.com/Microsoft/vscode/issues/34088

I added your issue to our collection (https://github.com/Microsoft/vscode/issues/27317) since you give a good example but I still mark this as duplicate of https://github.com/Microsoft/vscode/issues/34088
I agree we should look into this, but I do not have time currently.


Fix buildup of multiple message event listeners

When loading/reloading multiple vimeo players, there is a build up of message event listeners on the global window object, because there is no detach on unmount.

I've just realised this PR would have fixed https://github.com/CookPete/react-player/issues/101 which I fixed with a quick commit.. Apologies @jackyzhen! Nice work!


Move test/Reflection tests to FileCheck instead of diff

This lets us test for lines specific to architecture, for example.
rdar://problem/26306303

@swift-ci Please smoke test Linux


Add missing snippets for declaration types (e.g. crud, api, apiNamespace)

Some of the declarations have snippets which are useful, and some don't.
https://github.com/wasp-lang/vscode-wasp/blob/main/snippets/wasp.json
We want to cover all of the declaration types to make using Wasp much easier.

We should also add a snippet for jobs.


LeetHub no longer pushing submissions with new Leetcode update

Please review previous closed issues before filling out a new one! Duplicate issues will be closed without comment.
Describe the bug
After the new UI changes to LeetCode, after submitting a problem it is no longer getting pushed to GitHub
To Reproduce
Steps to reproduce the behavior:

Make sure you are on the latest version of Leetcode
submit problem

Expected behavior
Push problem and README to specified repo

i'm also facing the same issue

Reverting to old version seems to be work around temporarily.
But the new UI is more productive.
https://github.com/QasimWani/LeetHub/issues/366

seconding this. Also had to revert back to old UI.
I'm okay with using the old UI, but seems like the new version will be rolling out and they will stop supporting the old UI.

Reverting to old version seems to be work around temporarily. But the new UI is more productive. #366

Yes, the new UI is more productive it needs less clicks.

how do you revert to the old UI?


Add statistics to archives

Description

This PR adds an archive.stats attribute that stores statistics about the archive. Individual stats may be accessed as properties of archive.stats, such as archive.stats.occupied.
Resolves #79
Resolves #33
Changelog Description

Add a stats attribute to archives that provides various metrics (#100)
TODO

[ ] Create stats object with update function
[ ] Add stats property
[ ] Compute (normalized) QD score
[ ] Use properties in tests
[ ] Use properties in tutorials and examples

Questions

Status

[x] I have read the guidelines in
CONTRIBUTING.md.
[x] I have formatted my code using yapf
[x] I have tested my code by running pytest (tox is run in CI/CD)
[ ] I have linted my code with pylint
[ ] This PR is ready to go

Closing since we will likely use various attributes on the archive itself instead of a separate object.


Create cube_all_in_one.yml

@jennydaman  Added this file to be able to use while trying to deploy using podman desktop. Please review

Solved in 1466bd9238512b171cca437d67daa8b02eac786a


Parse consent cookie value as JSON

https://trello.com/c/0PYrZ8Yy/323-cookie-settings-form-defaults-to-no-even-if-preference-set-to-yes
The Cookie Settings form was always defaulting to 'No', as the existing consent cookie value was not being parsed correctly (the test for it was returning a mocked response that was already a JSON object).
There is already a helper function to solve this very problem, getConsentCookie, which we weren't using 🤦♀

Tests WFM


Improve Activity View Interface

The existing Activity View take a quite of bit of clicking to view all of the information in an activity. To make it easier for the users to view the activity information, the Activity View Interface has been re-designed.
The following design shows the Activity Detail View

When user hovers over 'View in other languages', other language text will show up on in a tooltip.

When user clicks on 'Show more info', all the related information will be shown.

This enhancement has been completed.


Navigator cannot open files

Can see files from other person's computer, but cannot open them, disrupting the socket connection.


Treewide: use HTTPS for repo.or.cz and SF.net

Motivation for this change
https://repo.or.cz/ repo now uses a trusted Let's Encrypt certificate and http:// redirects to https://
+ replaced links to SourceForge still using http://
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build a2jmidid

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: a2jmidid
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: a2jmidid
Partial log (click to expand)
shrinking /nix/store/ij0c7gr3q3sdml9vk89ryiqiqhg8cx0s-a2jmidid-8/bin/a2jmidi_bridge
shrinking /nix/store/ij0c7gr3q3sdml9vk89ryiqiqhg8cx0s-a2jmidid-8/bin/j2amidi_bridge
gzipping man pages under /nix/store/ij0c7gr3q3sdml9vk89ryiqiqhg8cx0s-a2jmidid-8/share/man/
strip is /nix/store/rpbg8gmqxhz8g61p1plz5d2srs84pvmv-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/ij0c7gr3q3sdml9vk89ryiqiqhg8cx0s-a2jmidid-8/bin
patching script interpreter paths in /nix/store/ij0c7gr3q3sdml9vk89ryiqiqhg8cx0s-a2jmidid-8
/nix/store/ij0c7gr3q3sdml9vk89ryiqiqhg8cx0s-a2jmidid-8/bin/a2j: interpreter directive changed from "/bin/sh" to "/nix/store/n2hjbpkf4c0m48945ivxs3lmsczzw2rg-bash-4.4-p23/bin/sh"
/nix/store/ij0c7gr3q3sdml9vk89ryiqiqhg8cx0s-a2jmidid-8/bin/.a2j_control-wrapped: interpreter directive changed from "/usr/bin/env python" to "/nix/store/irrwwhym6v41phf2707jb8nccf6jzai3-python-2.7.15/bin/python"
checking for references to /build in /nix/store/ij0c7gr3q3sdml9vk89ryiqiqhg8cx0s-a2jmidid-8...
/nix/store/ij0c7gr3q3sdml9vk89ryiqiqhg8cx0s-a2jmidid-8

Related: #13809


Server Crashing after update and player data corrupted i think

https://bin.bloom.host/idomowuhef

It should be fixed with the new version of this mod. Thanks for reporting this issue.


fix(tests): restrict app name length to 24 characters

@bacongobbler FYI the same functionality was done by @lanej in https://github.com/deis/workflow/pull/189/files#diff-c970541257d7d853bf0c55878ff9b32cR31. I'm partial to the way you've done it here. @lanej do you mind removing that change from your PR?

@bacongobbler @lanej are both of you ok with merging this?

sure. im getting good at rebasing

looks like https://github.com/deis/workflow/commit/7c3fbe6d6c96886ee1d2fb6726471a873ab8a539 fixed this behaviour up so this is no longer necessary :)

ah @bacongobbler I forgot about that. thanks


[Spring Boot Project] Merge 2.7.x Spring-Boot project into Dubbo master

What is the purpose of the change
merge https://github.com/apache/dubbo-spring-boot-project into Dubbo master
base commit https://github.com/apache/dubbo-spring-boot-project/commit/4d3eabfe90dddbbffda062a22a2eb524340fe87c
discussing mail thread https://lists.apache.org/thread.html/rabf203174646f5cc332b0dd9c04ef95e02097b049b6133bf720a584e%40<dev.dubbo.apache.org>

Codecov Report

Merging #7418 (73c0c61) into master (b5c81d8) will decrease coverage by 0.08%.
The diff coverage is 56.36%.

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

LGTM


HDDS-5937. Inaccurate bucket info returned from bucket list command i…

…n Shell
What changes were proposed in this pull request?
There is a bug in the info being displayed from the Ozone Shell regarding the bucket layouts. The information displayed is inconsistent with the actual underlying behaviour.
The bucketLayout is being listed as LEGACY when you use the list command (which is incorrect) and is being shown as FILE_SYSTEM_OPTIMIZED when using the info command (which is the correct value)
bash-4.2$ ozone sh volume create /vol1
bash-4.2$ ozone sh bucket create /vol1/buck1 --type=FILE_SYSTEM_OPTIMIZED

bash-4.2$ ozone sh bucket list /vol1/
 [

{ "metadata" : { }

,
 "volumeName" : "vol1",
 "name" : "buck1",
 "storageType" : "DISK",
 "versioning" : false,
 "usedBytes" : 0,
 "usedNamespace" : 0,
 "creationTime" : "2021-11-02T05:55:35.311Z",
 "modificationTime" : "2021-11-02T05:55:35.311Z",
 "quotaInBytes" : -1,
 "quotaInNamespace" : -1,
 "bucketLayout" : "LEGACY"
 }]

bash-4.2$ ozone sh bucket info /vol1/buck1

{ "metadata" : { }

,
 "volumeName" : "vol1",
 "name" : "buck1",
 "storageType" : "DISK",
 "versioning" : false,
 "usedBytes" : 0,
 "usedNamespace" : 0,
 "creationTime" : "2021-11-02T05:55:35.311Z",
 "modificationTime" : "2021-11-02T05:55:35.311Z",
 "quotaInBytes" : -1,
 "quotaInNamespace" : -1,
 "bucketLayout" : "FILE_SYSTEM_OPTIMIZED"
 }

What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-5937
How was this patch tested?
Related unit tests + manual testing in shell

Thanks @JyotinderSingh for the contribution.
Can you please automate/add test case to verify the behavior. Below is one idea for test verification:
step-1) Add assert statement in TestRootedOzoneFileSystem.java - line no# 432
Assert.assertEquals(bucketLayout, ozoneBucket.getBucketLayout());
step-2) Presently TestRootedOzoneFileSystemWithFSO#testMkdirNonExistentVolumeBucket is ignored and this has been implemented now. Please delete this overridden method from TestRootedOzoneFileSystemWithFSO.java class, so that it will run along with other tests.

Thanks @JyotinderSingh for the contribution.
Can you please automate/add test case to verify the behavior. Below is one idea for test verification:
step-1) Add assert statement in TestRootedOzoneFileSystem.java - line no# 432 Assert.assertEquals(bucketLayout, ozoneBucket.getBucketLayout());
step-2) Presently TestRootedOzoneFileSystemWithFSO#testMkdirNonExistentVolumeBucket is ignored and this has been implemented now. Please delete this overridden method from TestRootedOzoneFileSystemWithFSO.java class, so that it will run along with other tests.

Thanks for the feedback @rakeshadr. I have added the assertion to verify bucket layout and unblocked testMkdirNonExistentVolumeBucket.


can't see what I am typing in terminal

Hi, I am new to this but after not being able to type anything into my terminal I have managed to change something in settings which means I now can but I can't see what I am typing. The cursor is moving but I can't see what is being written, when copy/pasting what is in the terminal I see that it is working fine just not showing me what is written.
Version: 1.74.0
Commit: 5235c6bb189b60b01b1f49062f4ffa42384f8c91
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_6) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/14.1.2 Safari/605.1.15
Embedder: codespaces

disable terminal.integrated.gpuAccleration as it's not supported on certain versions of safari
/duplicate #165416


Lookup by common name is returning 404

On the live server lookup by common name is returning 404 even though lookup by id is working and returning the common name that was requested.

The main issue was that the Lookup method was using "common_name" instead of "name", but on following up on this, there appears to be synchronization errors with both the name and countries index that need to be resolved.

The primary issue was that synchronizing the index to disk was in the shutdown method but the main goroutine was exiting before the shutdown routine could complete, meaning that the indices did not get saved to disk. I modified the Serve method to wait until the shutdown was complete by blocking on an error channel.
It was also a good opportunity to enhance the indexing process, and the index has to be repaired on the production server anyway. To that end, I beefed up the index data structures and added some tests for them, then added a Reindex method and a check if the indices were empty to perform the reindex. If it's a brand new server - fine, the reindex will iterate over nothing; but if the indices were incorrectly loaded at least they will be rebuilt on start.


GPU Management

Hey, sorry to bother again but I was wondering...
Would it be possible to run the server on a particular GPU? Can this be supported in a command line argument for the server?

Hi, I made an update, now you can specify which device the infer will run on.
To do this, write the -d flag and specify the device ( cpu, cuda, cuda:0 )


multi-select in graph and linear view

I do not know if it is just me, or if it is something on the to-do list, but selection of anything seems impossible?
What I mean is similar to selecting lines out of notepad, dragging to highlight/etc
Is it an issue or something that has yet to be implemented?

This is currently by design. We do plan on having some selecting mechanisms, but haven't decided exactly how yet.

Changed the subject to better match what we're tracking with this report.

This is complete for graph views on builds 1594 and newer. Leaving open to track linear view.

About merging #1325, please note that Ctrl + END key doesn't work in Graph view too.

@melomac Ctrl + END works in Graph view as of versions >=1611.

Closing this out -- there is one design limitation in that the selection only allows for single basic blocks for now. At some point we'll expand this, but this is the feature as we intend it to exist in 1.2.


Execute command when login failed [feature request]

I'd like to be able to set a command or action to be executed when a login attempt is failed. This can be used for things like ntfy (sends a notification to phone).

Howdy uses PAM to trigger during authentication, there are PAM modules that you can use to run a script on error


[BUG] Flaky test - testRestoreShardFromRemoteStore

  2> REPRODUCE WITH: gradlew ':server:test' --tests "org.opensearch.index.shard.IndexShardTests.testRestoreShardFromRemoteStore" -Dtests.seed=709E54060BABD98F -Dtests.security.manager=true -Dtests.jvm.argline="-XX:TieredStopAtLevel=1 -XX:ReservedCodeCacheSize=64m" -Dtests.locale=sr-ME -Dtests.timezone=Europe/Kiev -Druntime.java=17
  2> java.lang.AssertionError: expected:<0> but was:<2>
        at __randomizedtesting.SeedInfo.seed([709E54060BABD98F:DAD18613CF69A4F0]:0)
        at org.junit.Assert.fail(Assert.java:89)
        at org.junit.Assert.failNotEquals(Assert.java:835)
        at org.junit.Assert.assertEquals(Assert.java:647)
        at org.junit.Assert.assertEquals(Assert.java:633)
        at org.opensearch.index.shard.IndexShardTests.testRestoreShardFromRemoteStore(IndexShardTests.java:2710)

Run on windows CI against 2.4.

cc: @sachinpkale

Another instance - https://build.ci.opensearch.org/job/gradle-check/7152/testReport/junit/org.opensearch.index.shard/IndexShardTests/testRestoreShardFromRemoteStore_2/.
Having trouble reproing locally.

Let me take a look and fix it on priority.

Looks like a windows specific issue. I tried deleting in a loop and all files will get deleted. I think files are moving to pendingDeleteFiles and not getting purged until another check.
I've hit this with 1 file remaining. Usually write.lock and 2 files remaining write.lock and .cfe.

@sachinpkale Put up a PR to fix this, its not specific to remote store rather buggy delete logic on Windows.

Great! Thanks @mch2


README.md says plot_components.py when I think it should say plot_counts.py

python plot_components.py sample.png < sample.count

Thanks @bjpop! Fixed!


M3 1763 nick suite repair 0.38.0

Purpose
This addressed some issues around getting information and elements from the user table, and updated some expected message text.
To Test
yarn && yarn start new shell yarn selenium new shell yarn e2e:modified

Coverage remained the same at 27.882% when pulling 2b36753711f6921efee507fd10ca7894a10b9f37 on nwisnewski:M3-1763-nick-suite-repair-0.38.0 into 98beb10cdb8f310065b4c420095fe0ae3113890c on linode:develop.


Difference in wording of descriptions for in set of operators

In the Operators on Strings section of this document the wording for the Description column for the operators in, !in, in~, and !in~ is different. For in the description is "Equals to one of the elements" whereas for in~ the description is "Equals to any of the elements". The key difference here is using the word 'one' vs. the word 'any'. The word 'one' can be interpreted as exactly one or as any one while 'any' must be interpreted as any one. Assuming in does not operate differently than the other similarly named operators, this is a suggestion to update the wording to be consistent for this family of operators.

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

ID: 11df65bd-6d46-7a8d-de92-97d933d7f1b1
Version Independent ID: 51157ab2-6539-8ef8-75aa-89c1ad3026f7
Content: String operators - Azure Data Explorer
Content Source: data-explorer/kusto/query/datatypes-string-operators.md
Service: data-explorer
GitHub Login: @shsagir
Microsoft Alias: shsagir

Thanks for the feedback! We have assigned the issue to the content author to further review this and update the document as appropriate.

Hi @llavalleytw, you're absolutely right. Opening a PR now to fix that.
Thanks for helping us improve our documentation!
#please-close


Angular 4 Support

In the most recent changelog it mentions Angular 5 as a dependency.  Does this mean ng-sidebar will no longer work with versions prior to 5?
I'm using Angular 4 and can't seem to get it working with the example even after referencing #100

What's the error you're getting (if any)? npm should complain if you're installing something where the peer deps don't match up.

But yes, you'd need to use something below v7.0.0 if you're using Angular 4.

I updated my project to Angular 5 and it works great now.  Thanks for this it saves me a ton of work!

Just in case someone else has a similar problem this was the npm warning:
UNMET PEER DEPENDENCY +--<EMAIL_ADDRESS>+
Or something along those lines

Also the case if you get something similar when building the project:

ERROR in Metadata version mismatch for module /home/user/blabla/node_modules/ng-sidebar/lib/index.d.ts, found version 4, expected 3.

If you are using anything Angular 4, just install version 6.0.5 of ng-sidebar :)


Cross-compile instructions for TFLite XNNPACK delegate on ARM

Are there any instructions on how to cross-compile the XNNPACK delegate for ARM?
I've tried the following:
export CC=arm-linux-armeabief-gcc
export CPP=arm-linux-armeabief-g++
bazel-2.1.0 build //tensorflow/lite/delegates/xnnpack:xnnpack_delegate
ERROR: /home/andy/.cache/bazel/_bazel_andy/ff7061ef3fb000e53210886b26774f34/external/XNNPACK/BUILD.bazel:1654:1: C++ compilation of rule '@XNNPACK//:sse2_ukernels' failed (Exit 1)
arm-linux-gnueabihf-gcc: error: unrecognized command line option '-msse2'
Target //tensorflow/lite/delegates/xnnpack:xnnpack_delegate failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 26.516s, Critical Path: 14.13s
INFO: 28 processes: 28 local.
FAILED: Build did NOT complete successfully
Ubuntu 14.04
Bazel 2.1.0
GNU 4.9.3
Targeting Raspberry Pi 3B / Compute Module 3

@WillNichols726
Could you please refer to this link and let us know if it helps.

Tensorflow 2.1.0
I was able to use the like below to successfully cross-compile libtensorflow_lite.a
https://www.tensorflow.org/lite/guide/build_rpi
Do these instructions also compile the XNNPACK delegate? From what I could see rooting around in the Makefile at tensorflow/lite/tools/make, it didn't look like the XNNPACK delegate was included in the build, nor were there any options to include it.

We don't have XNNPACK support with ./tensorflow/lite/tools/make/build_rpi_lib.sh script yet.
We need to update tensorflow/lite/tools/make/Makefile to build XNNPACK.

I've just checked in ARM crossbuild toolchains (https://github.com/tensorflow/tensorflow/commit/4961f18733ca3967198393abf419e14476b4a85c)  for Bazel build.
You can build armhf, aarch64 targets (include XNNPACK) with the following commands.
$ bazel build --config=elinux_armhf  //tensorflow/lite/tools/benchmark:benchmark_model
$ bazel build --config=elinux_aarch64  //tensorflow/lite/tools/benchmark:benchmark_model


After Installation, whenever I go to insert mode, Error message :Failed to Source

Describe the bug
After installing kickstart following the guide, I.e cloning into my . files (MAC) I got multiple errors telling me that there were issues with my lua caches. Not sure why this error is happening as I cleared everything before installing.

To Reproduce

Install Kickstart
Open Nvim

Desktop

OS: Mac
Terminal: Iterm

Neovim Version

:version
NVIM v0.10.1
Build type: Release
LuaJIT 2.1.1720049189

Checkout issue #537. Removing the entirety of ~/.local/share/nvim/ and opening nvim backup allowed Lazy to install the proper versions.


Show open issue and PR counts per asset and link to them

Old tasks

Add GitHub label to assets schema so we can query open issues and pull requests
Query GitHub repo issues with that label when we fetch data and populate the data model with the results

Will this count get updated throughout the day with ISR

Render issue and PR counts in asset detail page and link out to the list of GH issues or PRs so you can see those details
Edge / empty states
Documentation on how to use these labels

New tasks

See comment

E.g. https://github.com/carbon-design-system/carbon/issues?q=is%3Aissue+is%3Aopen+label%3A"component%3A+combobox"

Do we only want to reply on labels? Or should we all query the component name in issue titles?
Carbon-charts treemap search example
https://github.com/carbon-design-system/carbon-charts/issues?q=is%3Aissue+is%3Aopen+treemap+in%3Atitle

Do we only want to reply on labels? Or should we all query the component name in issue titles?
Carbon-charts treemap search example https://github.com/carbon-design-system/carbon-charts/issues?q=is%3Aissue+is%3Aopen+treemap+in%3Atitle

I feel like we should query for the component name in the title as well - to capture all of the activity around that component. Thoughts?

@alisonjoseph You raise really good points and it makes me question a little bit the value-add of these versus the cost for us and the library maintainers.
I can definitely see these being useful metrics at the library-level, but I'm not convinced at the individual asset level. We'd be enforcing conventions on how people manage their issues and development process by forcing them to provide labels or certain keywords, which might not be in their best interest with how they prefer to manage their codebase and issues.
Also, if the labels are forgotten or are anything other than perfect, these counts will be off. Same thing would go for almost any algorithm we could devise to search for per-asset issues.
At the library-level, we could pull the package.json's bugs.url field to have them tell us where to look for issues.
At the time of writing, there's not a single PR in the entire carbon monorepo that has an asset label or other identifier, so for every single asset in the carbon monorepo, the PRs field will show up as zero or blank. The issues are prone to the same problem. Some of them have labels, some of them don't. Some have titles which make the affected component obvious, some don't. Some have titles which don't mention a component or they have no labels, but their content makes it apparent which component is relevant.
Monorepos present an additional challenge here too because PRs and issues are not often labeled in a meaningful way to distinguish between which package in the monorepo they're relevant to. So in the case of the carbon monorepo, each of its libraries would either show up as zero or "26" (the count of all PRs as the moment). npmjs.com has this same issue and does the latter (showing a count of all issues in the containing repo). https://www.npmjs.com/package/@carbon/react

After discussing with the platform squad devs, we determined:

We don't want to add GitHub labels to the schema because it's really time consuming to apply those, and even though the system squad does that in Carbon's monorepo, not all library maintainers do that
No need to render pull request counts anywhere. Ideally, PR review is prompt and count hovers near zero.
Nothing of value to render on library detail page because when multiple libraries come from the same repo, there's no good way to know which issues are relevant to which library
Per asset, query result count for open issues searching issue title, dashboard item heading "Open issues"
Per asset, query result count for discussions searching discussion title, dashboard item heading "Discussions"
For both of those, we're searching asset name not id, because the name can evolve but id needs to be stable
If result count is 0, show 0 and still out to it so people can then do additional issue and discussion searches in GitHub

One other thing we discovered - searching discussions is supported in the GitHub GraphQL API but not through OctoKit. We're currently using OctoKit. It doesn't make sense to use a totally new API right now, but in the near future we'll be able to when we build out our content service. Until then, the best thing we can do is render an en dash for discussion count and have that card take you to the repo's discussions pre-searched by the asset name.


Cannot choose custom persistence profiles with FAST/http template

Environment

FAST RPM Version: f5-appsvcs-templates-1.7.0-1.noarch
BIG-IP Version: BIG-IP <IP_ADDRESS> Build 0.0.2 Point Release 6
--

Summary
When deploying with bigip-fast-templates/http only the default persistence types are available

Steps To Reproduce
List the steps to reproduce the behavior:

Deploy->BigIP-Fast-Templates/http
Fill in Tenant, Application Name, IP address, etc
Select the check box for Persistence
Only the built-in persistent types are selectable

Expected Behavior
I expect to be able to select custom persistence profiles located within the Common or any other Partition
Actual Behavior
The drop down for persistence type only offers the out-of-the-box options

This fix for this issue was included with Release 1.10.

We just upgraded our test LTM to 1.11.0, and are getting an error when trying to choose a custom persistence profile.

Thanks for submitting this issue. We are now tracking it internally with ID MYSTIQUE-454.


CompilationDatabase_Error items have no comments

They are defined in the file, but not generated.

I do not think that this is need for a release


Tagged comments appear in My Requests Event Queue

As a ministry coordinator or IAO analyst
I want comments that I am tagged in to appear as an event in My Requests Event Queue
so that I do not miss comments in which I am tagged

Assumptions & Scope
What are the assumptions for this story?
When a user is assigned to a request they will see all event for that request on their My Requests Event Queue.
When a user is a watcher for a request they will see all event for that request on their Watching Event Queue.
When a user is tagged in a comment and they are not an assignee or a watcher for that request, the event (tagged user comment) will be displayed on the My Requests Event Queue.
The assumption is that tagged comments are important, so even though the user is not assigned to the request, the tagged comment should appear on their My Requests, the default Event Queue, so that it is not missed.
If a user is an assignee and they are tagged in a comment, the event will only appear in the My Requests queue once.
If a user is a watcher and they are tagged in a comment, the event will appear once in the Watching queue and once in the My Requests event queue.
What is IN scope?
Need to do some load testing for this story.
What is NOT in scope?
Acceptance Criteria
Scenario 1: Unassigned user is tagged in a comment

GIVEN a user is not assigned or a watcher for a request
WHEN the user is tagged in a comment
THEN the tagged comment will appear in the user's My Requests Event Queue
AND other events unrelated to the tagged comment will not appear in the user's My Requests Event Queue

Scenario 2: Reply to a comment where a user is tagged

GIVEN a user is not assigned or a watcher for a request
AND the user is tagged in a comment
WHEN there is a reply to the comment where the user is tagged
THEN the reply to the tagged comment will appear in the user's My Requests Event Queue
AND other events unrelated to the tagged comment will not appear in the user's My Requests Event Queue

Scenario 3: Assigned user is tagged in a comment

GIVEN a user is assigned but not a watcher for a request
WHEN the user is tagged in a comment
THEN the tagged comment will appear in the user's My Requests Event Queue
AND the tagged comment will only appear once in the user's My Requests Event Queue
AND all other events for the assigned request will appear in the user's My Requests Event Queue

Scenario 4: Unassigned user is tagged in a comment and also a watcher for the request

GIVEN a user is not assigned but is a watcher for a request
WHEN the user is tagged in a comment
THEN the tagged comment will appear in the user's My Requests Event Queue
AND other events unrelated to the tagged comment will not appear in the user's My Requests Event Queue
AND the tagged comment will also appear in the user's Watching Event Queue
AND all other events for the watched request will appear in the user's Watching Event Queue

Scenario 5: Reply to a comment where an unassigned user is tagged and also a watcher for the request

GIVEN a user is not assigned but is a watcher for a request
AND the user is tagged in a comment
WHEN there is a reply to the comment where the user is tagged
THEN the reply to the tagged comment will appear in the user's My Requests Event Queue
AND other events unrelated to the tagged comment will not appear in the user's My Requests Event Queue
AND the tagged comment will also appear in the user's Watching Event Queue
AND all other events for the watched request will appear in the user's Watching Event Queue

Dependencies? What is the impact of this dependency? (If so, link dependency in the ticket, make it visible in a team´s backlog)
Validation Rules? (If yes, list here)
Design
@xxx - please link the Design here
Definition of Ready

[ ] Is there a well articulated User Story?
[ ] Is there Acceptance Criteria that covers all scenarios (happy/sad paths)?
[ ] If there is a user interface, is there a design?
[ ] Does the user story need user research/validation?
[ ] Does this User Story needs stakeholder approval?
[ ] Design / Solution accepted by Product Owner
[ ] Is this user story small enough to be completed in a Sprint? Should it be split?
[ ] Are the dependencies known/ understood? (technical, business, regulatory/policy)
[ ] Has the story been estimated?

Definition of Done

[ ] Passes developer unit tests
[ ] Passes peer code review
[ ] If there's a user interface, passes UX assurance
[ ] Passes QA of Acceptance Criteria with verification in Dev and Test
[ ] Confirm Test cases built and succeeding
[ ] No regression test failures
[ ] Test coverage acceptable by Product Owner
[ ] Ticket ready to be merged to master or story branch
[ ] Developer to list Config changes/ Update documents and designs
[ ] Can be demoed in Sprint Review
[ ] Tagged as part of a Release
[ ] Feature flagged if required
[ ] Change Management activities done?

Hey @sumathi-thirumani-aot
Please see the AC above and let me know if I captured everything.

Closing: all ACs pass

Closing: all ACs pass in TEST


Unable to resolve the fs promise (node events)

Current behavior
I have upgraded Cypress to version 10.4 and cucumber-cypress-preprocessor to version 11.4.0. As this needed a bundler, I installed the ESbuild dependencies based on the examples given in the project.
Since then, I am receiving error when trying to run any test in the suite.
I tried using fs-extra dependency with no luck
I also tried replacing fs.readFileSync(file) with fs.promised.readfile using async and wait with no success.
I found a potential fix but not sure how to add it to my project: https://esbuild.github.io/getting-started/#bundling-for-node
Error: Build failed with 1 error:
cypress/support/commands/readXlsx.js:2:23: ERROR: Could not resolve "fs"
at failureErrorWithLog (/Users/abc/Test Automation Projects/functional-prj/node_modules/esbuild/lib/main.js:1624:15)
Desired behavior
I want the fs promise to work as before. though I had to add a bundler(Esbuild) for preprocessor 11.4 to work, I am still using node as the package manager.
Test code to reproduce
Cypress.Commands.add("readXlsx", (file, sheet) => {
const fs = require("fs");
const XLSX = require("xlsx");
const read = ({ file, sheet }) => {
    const buf = fs.readFileSync(file);
    const workbook = XLSX.read(buf, { type: "buffer" });
    const rows = XLSX.utils.sheet_to_json(workbook.Sheets[sheet]);

    return rows;
};

module.exports = {
    read,
};

});
Versions

Cypress version: 10.4.0
Preprocessor version:11.4.0
Node version:16.14.2

Checklist

[ x] I've read the FAQ.
[ x] I've read Instructions for logging issues.
[x ] I'm not using<EMAIL_ADDRESS>(package name has changed and it is no longer the most recent version, see #689).

You can’t use node modules in the Cypress runtime like that, you need to use a task which runs in a node process.

Can you please elaborate a little bit as the code was working fine with preprocessor version 4.0.

I can’t really see how this ever worked, in any version.

I can confirm this did work in the older versions. We were using node:path (https://nodejs.org/api/path.html) which we had to remove for now after upgrading to cypress 10 and the new cucumber plugin version.
We were also using the cypress-fail-on-console-error package(https://www.npmjs.com/package/cypress-fail-on-console-error) which I can't seem to get working using the esbuild bundler. It is using node:os (https://nodejs.org/api/os.html) which also is not working now.
Both worked with the older version.
Esbuild seems to give the following feedback:

The package "os" wasn't found on the file system but is built into node. Are you trying to bundle for node? You can use "platform: 'node'" to do that, which will remove this error.

The cypress-esbuild-preprocessor seems to have an option to give a platform config:
https://github.com/bahmutov/cypress-esbuild-preprocessor

If you want to pass your own ESBuild options

// cypress/plugins/index.js
const createBundler = require('@bahmutov/cypress-esbuild-preprocessor')
module.exports = (on, config) => {
  const bundler = createBundler({
    // any ESBuild options here
    // https://esbuild.github.io/api/
  })
  on('file:preprocessor', bundler)
}

And from the linked esbuild documentation:

By default, esbuild's bundler is configured to generate code intended for the browser. If your bundled code is intended to run in node instead, you should set the platform to node:

I have tried passing the platform configuration in cypress.config.js,
async function setupNodeEvents(on, config) {
  await preprocessor.addCucumberPreprocessorPlugin(on, config);

  on(
    'file:preprocessor',
    createBundler({
      plugins: [createEsbuildPlugin.default(config)],
      platform: 'node',
    }),
  );

  // Make sure to return the config object as it might have been modified by the plugin.
  return config;
}

but then I run into other errors.
The following error originated from your test code, not from Cypress.

> require is not defined

I'm really hoping it is just a matter of configuration and we can get this working so we can upgrade to your new plugin without compromises.

Can you please elaborate a little bit as the code was working fine with preprocessor version 4.0.
In CLI, it is suggesting to do add platform:node. Please see below: ✘ [ERROR] Could not resolve "fs"
cypress/support/commands/readXlsx.js:2:23:
  2 │     const fs = require("fs");
    ╵                        ~~~~

The package "fs" wasn't found on the file system but is built into node. Are you trying to bundle for node? You can use "platform: 'node'" to do that, which will remove this error.

You could possibly add it to cypress.config.js setup node events as a task.
const fs = require('fs');
const xlsx = require('node-xlsx').default;

async function setupNodeEvents(on, config) {

  on('task', {
    parseXlsx({ filePath }) {
      return new Promise((resolve, reject) => {
        try {
          const jsonData = xlsx.parse(fs.readFileSync(filePath));
          resolve(jsonData);
        } catch (e) {
          reject(e);
        }
      });
    },
  });

  // Make sure to return the config object as it might have been modified by the plugin.
  return config;
}

and run with cy.task (we added is as a command)
Cypress.Commands.add('parseXlsx', (inputFile) => {
  return cy.task('parseXlsx', { filePath: inputFile });
});

This works for us in this situation with the new cucumber plugin version.

You could possibly add it to cypress.config.js setup node events as a task.

I can chime in and say that this is the only way. At no point has anyone been able to do anything else. You simply can't run a node module in the browser. You might have been able to make a node module compile with earlier versions of browserify / webpack, but it just wouldn't behave as you'd expect.


allow path regex

option to allow prefix to be a regex.
fixes #302

The solution is probably not ideal but it works.
Initially I was going for keeping it envoy compatible adding a regex, but prefix is wired in too many places and making optional threw me to the lino's den.
The build fails due to some missing AWS credentials it seems.

@richarddli @kflynn is there anything else I would need to do here to get this merged ?

Running tests now, hoping to land this today. Thanks!


Add CHANGES.md, drop release- prefixes

Going forward, we're going to track changes to linkerd directly in this repo, in CHANGES.md.  We're also dropping the "release-" prefix that shows up in our tag names on the Github releases page.

:+1: :star:

:star:

I think I'd prefer that we wrap CHANGES.md at 80 or 100 chars...  not a real blocker :+1:

@olix0r Good suggestion -- have added wrapping as part of this branch.


Feature fosite 0.9.0 support

Coverage remained the same at 44.214% when pulling 7911623bca371cfb1fcb96ca7576043d48cf99ab on feature-fosite-0.9.0-support into e364da5e23ea8bdbd68654670d288effa71b1dae on master.


Code Not Working For Dataset with 38 classes

I am trying to run your model on my images dataset consisting of 38 classes(labels) but it is not working. The error came up is following:
InvalidArgumentError (see above for traceback): Cholesky decomposition was not successful. The input might not be valid.
[[Node: Cholesky = CholeskyT=DT_FLOAT, _device="/job:localhost/replica:0/task:0/cpu:0"]]
Can someone help.

I also encountered the same problem. Can someone help？


Decryption Error

An error was reported when using this tool. How to solve it? The error code is as follows
Unhandled exception. System.Security.Cryptography.CryptographicException: Padding is invalid and cannot be removed.
   at Internal.Cryptography.UniversalCryptoDecryptor.DepadBlock(Byte[] block, Int32 offset, Int32 count)
   at Internal.Cryptography.UniversalCryptoDecryptor.UncheckedTransformFinalBlock(Byte[] inputBuffer, Int32 inputOffset, Int32 inputCount)
   at Internal.Cryptography.UniversalCryptoTransform.TransformFinalBlock(Byte[] inputBuffer, Int32 inputOffset, Int32 inputCount)
   at System.Security.Cryptography.CryptoStream.ReadAsyncCore(Byte[] buffer, Int32 offset, Int32 count, CancellationToken cancellationToken, Boolean useAsync)
   at System.Security.Cryptography.CryptoStream.Read(Byte[] buffer, Int32 offset, Int32 count)
   at System.IO.Stream.CopyTo(Stream destination, Int32 bufferSize)
   at System.IO.Stream.CopyTo(Stream destination)
   at NPK3Tool.Extensions.ToMemory(Stream Stream) in C:\Users\Marcus\Documents\GitHub\MwareStuff\NPK3Tool\Extensions.cs:line 107
   at NPK3Tool.NPK.GetEntryTable(Stream Package) in C:\Users\Marcus\Documents\GitHub\MwareStuff\NPK3Tool\NPK.cs:line 282
   at NPK3Tool.NPK.Unpack(String Package, String OutDir) in C:\Users\Marcus\Documents\GitHub\MwareStuff\NPK3Tool\NPK.cs:line 229
   at NPK3Tool.Program.Main(String[] args) in C:\Users\Marcus\Documents\GitHub\MwareStuff\NPK3Tool\Program.cs:line 130

The game is Tokyo Necro(Jast USA)

Or your NPK is corrupted or you selected the wrong game, this is basically decryption error.
For be sure, use the KeyFinder tool to find your game key and try set it manually

KeyFinder

Or your NPK is corrupted or you selected the wrong game, this is basically decryption error. For be sure, use the KeyFinder tool to find your game key and try set it manually

Sorry, my mistake. I am using the Chinese version and the key is different from the English version. So can you add a Chinese version?

Or your NPK is corrupted or you selected the wrong game, this is basically decryption error. For be sure, use the KeyFinder tool to find your game key and try set it manually

I found this using the keyfinder tool, but I don't know how to use it

Or your NPK is corrupted or you selected the wrong game, this is basically decryption error. For be sure, use the KeyFinder tool to find your game key and try set it manually

Another issue is that I can extract the cg file, but after repackaging the NPK3 Packages, the game cannot recognize it. Even if I directly repackage it after extracting it, it cannot be done, let alone modify the file and repackage it again

Or your NPK is corrupted or you selected the wrong game, this is basically decryption error. For be sure, use the KeyFinder tool to find your game key and try set it manually

Then when I extract the repackaged file again, the following situation will occur

try do those steps https://github.com/marcussacana/MwareStuff/issues/30


don't use self

Reaching out to the window for these objects can cause conflicts.
Our app wraps vendor.js in an IFFE so these globals are aren't clobbered by offending third-party scripts. So requirejs, define, and require are all available in context, but not explicitly available on the window.
I'm hoping this edit can be merged in, since it shouldn't cause conflicts with upstream consumers.

👍  this looks good to me. I'll make a patch release now as well to unblock you 😄

Released with v0.1.2.

thanks!


Bulgaria map is not correctly displayed in Country Map chart

Hi,
I've tryed to create Country Map chart to locate some objects in Bulgaria and I've run across the following issues:
The codes of region/province/department, required by the the input when creating country map, are not in compliance with ISO-3166-2 in packages/superset-ui-legacy-plugin-chart-country-map/src/countries/bulgaria.geojson. Ok, then I changed the codes in my source DB table to be in the format like in the mentioned .geojson file and the Bulgaria map was displayed. Unfortunately - it is not correctly displayed - it is displayed like a small point in the upper corner of some rectangle.
I have looked at the DEMO  for superset-ui in Github - the issue thing there. When you select Bulgaria from the dropdown, only a point inside a rectangle is shown.
You can check it here:
https://apache-superset.github.io/superset-ui/?path=/story/legacy-chart-plugins-legacy-plugin-chart-country-map--basic

NOTE: the problem lies with superset-ui-legacy-plugin-chart-country-map/src/countries.js
@djok according to ba113c2056958ec0c0a84dafff74952377a45c53 you have added the bulgaria geojson map, can you comment?
@pkdotson according to 35e78ad895721e1e9807a4451bca1dc448e96e36 you have implemented controlPanel.ts could you also comment?

The problem is in 'file-loader!./countries/bulgaria.geojson', will prepare new geojson file

Related: https://github.com/apache/superset/issues/12987


Getting Started admonitions. #12495

Summary

Test plan

@mrazauskas sorry but can you look again at the most recent commit of that file?  I ran prettier before pushing this time.

All look good! Thanks.


host demos on penumbra-zone github

https://github.com/turbocrime/nextjs-penumbra-client-example
https://github.com/turbocrime/nextjs-penumbra-wasm-example

these should be independent repositories, not part of the monorepo

oh, didn't realize i could do this myself
https://github.com/penumbra-zone/nextjs-penumbra-client-example
https://github.com/penumbra-zone/nextjs-penumbra-wasm-example


Improve DX - dont minify dev build

Description
Dont minify for dev mode
Also, ditch the source maps (since its not minified!)
Discussed here: https://discourse.nodered.org/t/when-are-vuejs-methods-binded/93142/11?u=steve-mcl

Thanks @Steve-Mcl !
It would also be useful to put your npm run build:dev command somewhere.  I see now - for the first time - this page.  Not sure whether it should but put there, because that page is not part of the contribution documentation.  Even not sure why I have never found that readme page, because it contains a.o. the lint command (which I only found recently myself somewhere else).


Support Regex in IgnorePathsModifier

To allow setting ignored paths, add support for regex.
The following files should then be ignored by default:

*.swx
*.swp
*.DS_Store

Furthermore, provide an API in Sync to further specify these ignored paths

Prefer using glob patterns instead of regular expressions


Move "Concepts" to after Overview

[Enter feedback here] You have "Concepts" at the very end of SQL Edge.

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

ID: b7077b5a-b7a6-9319-9df6-bd8cf8b332dc
Version Independent ID: b06640ac-13e3-c3d1-a0da-2265f371236c
Content: Machine learning and AI with ONNX in Azure SQL Edge (Preview)
Content Source: articles/azure-sql-edge/onnx-overview.md
Service: sql-edge
Sub-service: machine-learning
GitHub Login: @dphansen
Microsoft Alias: davidph

Thanks for the feedback! We are currently investigating and will update you shortly.

@residentx10 Thank you for the feedback. We will look into it.
@YutongTie-MSFT I've sent off this issue to the SQL docs team.

#reassign:stevestein

@residentx10 - thanks for pointing this out. The SQL Edge table of contents is currently under review and will likely be updated soon.
Thanks for the feedback!
#please-close


Improve the repository in order to make it public

Structure of the README:

Description of the project (text + nice screenshot)
Motivations

User manual:

Quick start with docker
Detailed installation instruction without docker
Simple instructions for using the editor (create, edit, etc.)
Simple instructions for customizing the template

Add license file:

MIT license
Contribution guidelines (github template)
Community guidelines (github template)

Hi!, I've just tried the application. Is pretty neat!. Something I would add to the readme is that the application is at http://localhost:3000

Thank you! It's nice to see that you have been able to run the application. Lately, this project has been a bit left aside as we got busy with other projects. Out of curiosity, do you have a specific use case in mind? This is a topic we'd be happy to explore again.

I'm looking for something that shows me RDF using a UML-like diagram, entities with properties. connected with other entities. The simplest the better.
I want to show the RDF to a non-technical user mostly.

Cool! To some extent this was one of our goal with this editor. Most editors display the full rdf graph. We introduced the notion of template to make the graph more understandable to an non-technical editor familiar with a specific domain.
https://github.com/heigvd-software-engineering/semantic-flow/blob/main/documentation/semantic-flow-configuration.md

I like the idea of templates, or just diagrams with selected items on them, showing the whole RDF adds too much noise.
I can see the elements displayed in semantic-flow are drawn with React. I'm thinking perhaps of using them in a multi-purpose canvas, such as https://github.com/tldraw/tldraw


Feature Request: LISTEN support (ie. NotificationResponse)

the _MSG_NOTIFICATION_RESPONSE is defined in the code, but the message type is not implemented.
There are a few ways to implement this, but simply providing a conn.listen("channel") that returns a stream seems the most obvious and idiomatic to me.  the postgres listen command operates 'per-connection' and doesn't block, so I'm not sure how this would interact with pools (if at all).

Thanks for the feedback. I'm happy to take patches - but I am not planning on adding this at the moment.
Good point about the connection pools and listen. I guess you'd always want to have a separate connection (not from a pool) open to handle listen responses.

Thank you for the package, it's very helpfull,
and I'll also add a +1 for this feature, I was going to use it to create a pubsub in my  app but unfortunately there's no LISTEN support.
I take a look in the source code but I'm not sure if I'm able to implement this by myself.

Yet another +1 for this feature request.

I've forked it and added support for getting Notifications.
See https://github.com/aabluedragon/dart_postgresql.git
Works in a similar fashion to the Java PostgreSQL getNotification() method.

Wouldn't a Stream based API be more suiteable for Dart?

It would, but I haven't found a way to have the PostgreSQL connection just give you the received pending notifications without "asking" for it.
Java also has Streams, yet the official PostgreSQL driver also works that way over there.
You can wrap the call to getNotification() with an interval timer, and have it load the upcoming notifications to a stream.


ci: make sure build sees env vars

Investigating failure to set env vars noted in #1606.
DO NOT MERGE

Missed the FORCE_GOARCH case.


Light Gray tag missing from theming

What is happening?
The Light gray tag colour is missing from theming/variables.css and theming/colors.css
The light gray tag and board colour cannot be changed.
--theme--tag_default also seems to do nothing?
What should be happening?
Light gray tag should be able to be changed.
Screenshots (if applicable):

In a custom insert, I have set --theme--tag_default to #f00 (red), so it doesn't seem to be misnamed either
Operating system/browser name & version:
Windows 10 21H1
notion-enhancer and/or notion-repackaged version:
notion-repackaged v0.11.0
notion-enhancer configuration (enabled/disabled mods and their settings):

Pastel dark
Icon sets integration
Tweaks: thicker bold text, wrap tables to page width, hide "Type '/' for commands", custom insert css.
inside the custom insert:

:root.dark {
  --theme--tag_default: #f00 !important;
}

Integrated titlebar
Tray
Tabs
Outliner
Scroll to top
Indentation lines
Simpler databses
Topbar icons: all on
Truncated titles

I will make a pull req to fix this.

-_- i missed that because notion is inconsistent & doesn't use the light grey colour anywhere else
@blorbb the contents of the colors.css file is generated by https://github.com/notion-enhancer/repo/blob/dev/theming/_mapColors.js - your changes will either need to be made to a separate css file, or added to that script.


Login Element

Hi,
on osX El Capitan automute always disappears from the Login Elements. Every time I restart my mac.
Can you help me?

Sorry, I'm not sure what could be happening, it's working fine for me.


Add FakeExample package for testing use manifest functionality

I think this needs to be merged before #22, if we want #22 to use this package in testing.

Looks good :+1:


[GPII-3188]: First crack at smoke testing for GCP infra with Locust

To deploy Locust: rake sh['xk up live/dev/locust']
To start the swarm: rake sh['sh -c "TF_VAR_locust_swarm=1 xk up live/dev/locust"']
I don't like the way how chart handles task scripts. There is also an issue if you want to work with multiple hosts (i.e. preferences. and flowmanager.) within the same task set. But otherwise this approach is pretty flexible.

Typical "failed run":
Starting Locust swarm with 100 users and hatch rate of 10!
{"message": "Swarming started", "success": true}
USERS  RPS     STATUS
0	0	hatching
0	0	hatching
15	0	hatching
15	0	hatching
35	11	hatching
35	11	hatching
55	16	hatching
55	16	hatching
75	20	hatching
75	20	hatching
100	0	running
100	0	running
100	40	running
100	40	running
100	47	running
100	47	running
100	47	running
100	47	running
100	46	running
100	46	running
100	47	running
100	47	running
100	46	running
100	46	running
100	46	running
100	46	running
100	46	running
100	46	running
100	46	running
100	46	running
100	47	running
100	47	running
100	47	running
100	47	running
100	47	running
100	47	running
100	48	running
100	48	running
100	48	running
100	48	running
100	48	running
100	48	running
100	48	running
100	48	running
100	48	running
100	48	running
100	48	running
100	48	running
100	47	running
100	47	running
100	48	running
100	48	running
100	47	running
100	47	running
100	48	running
100	48	running
100	47	running
100	47	running
100	48	running
100	48	running

Swarming complete!
{"message": "Test stopped", "success": true}
Processing stats...

{"errors": [], "stats": [{"median_response_time": 82, "min_response_time": 23, "current_rps": 22.7, "name": "/preferences/carla", "num_failures": 0, "max_response_time": 618, "avg_content_length": 1549, "avg_response_time":<PHONE_NUMBER>8073654, "method": "GET", "num_requests": 1412}, {"median_response_time": 84, "min_response_time": 25, "current_rps": 27.0, "name": "/preferences/vladimir", "num_failures": 0, "max_response_time": 577, "avg_content_length": 1811, "avg_response_time":<PHONE_NUMBER>173913, "method": "GET", "num_requests": 1472}, {"median_response_time": 84, "min_response_time": 23, "current_rps": 49.7, "name": "Total", "num_failures": 0, "max_response_time": 618, "avg_content_length": 1682, "avg_response_time":<PHONE_NUMBER>0166436, "method": null, "num_requests": 2884}], "fail_ratio": 0.0, "slave_count": 3, "state": "stopped", "user_count": 0, "total_rps": 49.7}

Looks like max_response_time (618) is worse than desired (200)!
This is unacceptable!

Stats distribution:
"Name","# requests","50%","66%","75%","80%","90%","95%","98%","99%","100%"
"GET /preferences/carla",1433,82,100,120,130,190,250,320,370,618
"GET /preferences/vladimir",1492,85,110,130,140,190,240,300,370,577
"None Total",2925,84,100,120,140,190,240,310,370,618
Resetting stats...
ok

What is the end game for this - deploy locust into dev clusters, and run it against local GPII deployment every time TF runs?

@stepanstipl I am thinking of this as of separate step in CI pipeline (instead of current "smoke tests" that we have for AWS). We could start with deploying locust into the same cluster.  Run nightly benchmarks for stg and prd from dev. End game – have "utility" cluster, so we can benchmark from there?
@mrtyler As long as Locust deployed into the same cluster, it can swarm NodePort or ClusterIP targets with no issue. Locust can do OAuth.

@natarajaya

I am thinking of this as of separate step in CI pipeline

I like this. I think it would come after stg (and could potentially use stg as the cluster-under-test).

(instead of current "smoke tests" that we have for AWS).

 Something lightweight that runs as part of deployment gives us quick feedback on the health of that deployment.
A sort of compromise might be lightweight Locust tests that run during deployment (so we don't have to write and maintain scripts that run curl), and heavier performance-focused Locust tests that run in a separate step.
I think this is a good addition to the larger discussion of testing we've been trying to have for a few days :).

We could start with deploying locust into the same cluster. Run nightly benchmarks for stg and prd from dev. End game – have "utility" cluster, so we can benchmark from there?

I like doing it from dev in the short-term -- a nice environment that is destroyed and re-created regularly. We can add variables to the nightly build and use them to only run the benchmark step during nightly builds (not during triggered CI builds).
Longer term I like a "utility environment", probably the same environment where the CI worker lives.

@mrtyler As long as Locust deployed into the same cluster, it can swarm NodePort or ClusterIP targets with no issue.

Cool. Hitting the NodePort/ClusterIP directly is a little different from going through the public IP, Google LB, nginx-ingress, etc. I guess it depends on exactly what we're trying to test.

Locust can do OAuth.

That example code looks good.

This last batch of commits looks good.
I'm going to take another look at swarm.tf.

So, I think things look a bit better after refactoring.
To start preferences test:
rake sh['sh -c "xk down live/dev/locust && TF_VAR_locust_target_host=http://preferences.$TF_VAR_domain_name TF_VAR_locust_script=preferences.py xk up live/dev/locust"']
To start flowmanager test:
rake sh['sh -c "xk down live/dev/locust && TF_VAR_locust_target_host=http://flowmanager.$TF_VAR_domain_name TF_VAR_locust_script=flowmanager.py TF_VAR_locust_users=30 TF_VAR_locust_desired_total_rps=10 TF_VAR_locust_desired_median_response_time=200 TF_VAR_locust_desired_max_response_time=700 xk up live/dev/locust"']
I configured default thresholds so preferences test can pass without need to change anything. Interestingly, flowmanager shows much worse performance, so thresholds needs to be adjusted accordingly.

Added tests to CI config.
There is an issue with multiline scripts, I followed a workaround advice that supposed to work, but I have no way to verify.

LGTM!

Conceptually I'm not quite happy about mixing the provisioning of the locust swarm and actually running the tests. I would think that provisioning should be one thing, idempotent, handled with TF. and actually running the perf. test another. This woudl probably prevent, if I understand correctly above code, things like teraing down the cluster every time before running the test or non-idempotent TF runs.

@stepanstipl Thanks for review!
As the title of this PR states – it is just a first attempt to introduce Locust into our infra. I agree that there is definitely room for improvement! Maybe, once common code is merged, we can have common "utility" multi-purpose cluster to run smoke tests from. This would've resolved issue from your previous comment and allowed us to have better test results, "unstained" by Locust activity, always benchmark from different zone, etc.


⚠ ClamAV has degraded performance

In 6a411f2, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 72 ms

Resolved: ClamAV performance has improved in 6001a0b.


Title format in camera-ready version of ACL2022

Following the instruction, I protect some letters with curly braces when I fill the title box, but after submitting, I notice that the curly braces also show on the title of the openreview page.

The instruction says "These curly braces will not appear in the online conference program or in the proceedings. They will only appear in the BibTeX file ..."  So I wonder if it is ok that these curly braces also show in the openreview page.

This is not an ACLPUB2 issue. You should redirect the question to the ARR EiCs who are in charge of OpenReivew. Or is there someone else that should be emailed @crux82?


Transfer this repository to the pytransitions organization

@aleneum I've sent you an invite.
Please move this repository to the pytransitions organization.

Hi @thedrow,
sounds like a plan. Unfortunately I was too slow and the invitation expired. Could you send it again?

Done.

as far as i can tell i cannot move it (lack of permissions probably). but I invited you to pytransitions. As a member you should have the rights to create new repositories and thus also move the repo.


Non windows user can't build an app bootstrapped by a windows user

The manifest.yml file bootstrapped by a windows user will contain \ to define action paths. A non windows user can't build the app as an error is thrown due to that:
building actions..
    Error: ENOENT: no such file or directory, lstat

The workaround for a non windows user is to replace all \ with / in the manifest.yml.

The problem should be there https://github.com/adobe/generator-aio-app/blob/master/lib/ActionGenerator.js#L156  where the path will be written with backslashes if running the generators on windows.
We should enforce unix path style no matter what

It seems that the actions path has been already fixed.
I took care of the extensions paths in exchange.


After starting Rancher server, there is an admin account 1a1 that is visible

v0.49.0
Steps to Reproduce:

Launch Rancher Server
Go to accounts tab

Result: There is an admin account that has the ID of 1a1. This is confusing as it makes it seem like there is some admin floating around. This account remains even after authentication. If this is deactivated/deleted, will there be issues?
Expected Result: This admin account shouldn't be shown if it's only used internally.

That Account Is always there, not always used but not removed as if a user uses no auth but has apikeys then turns on auth we want apikeys to continue to work,

Also default project Belongs to 1a1

So as a new user, what would happen if i deactivate/delete it when auth is on, or before we turn on auth? I haven't done those test cases, but if Rancher is going to break because of that, we need to either hide it or disable the ability to remove.

I tried to deactivate the account (as the API allows you to do so) and it basically made my Rancher setup unusable. It got stuck in "Deactivating" and when I try to refresh, I just get "Error" page. We need to at least not allow users to deactivate/delete this account if we make it visible.

It depends on the setup you are using. If you are using a new setup it should be fine. But it it is a longrunning one that has upgraded it may have it so that the person logged in is actually 1a1.

It was a new set up.

Verified with v1.0.0-rc1 that 1a1 is no longer able to be removed if it's the last (and required) admin.


KAFKA-10456: Wrong description in kafka-console-producer.sh help

https://issues.apache.org/jira/browse/KAFKA-10456
Fix typo in description of ConsoleProducer.
More detailed description of your change,
if necessary. The PR title and PR message become
the squashed commit message, so use a separate
comment to ping reviewers.
Summary of testing strategy (including rationale)
for the feature or bug fix. Unit and/or integration
tests are expected for any behaviour change and
system tests should be considered for larger changes.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@omkreddy please review this minor change. Thanks.

@omkreddy Thanks for the review, merging to trunk.


Parallelization

Parallel acceleration of current linear methods.

Parallelization is implemented in this commit  and outperforms the faiss approach in medium data size + multi-cores environments.


Failed npm install, rimraf lib

Is there a way around this without updating node? My corporate environment is slightly behind the times and has slightly older versions of whitelisted apps, below is the most up-to-date they allow.
Failed at<EMAIL_ADDRESS>clean script 'rimraf lib dist es coverage'.
Below is the section of npm-debug that could be useful.
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'preclean', 'clean', 'postclean' ]
5 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>6 silly lifecycle<EMAIL_ADDRESS>no script for preclean, continuing
7 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>8 verbose lifecycle<EMAIL_ADDRESS>unsafe-perm in lifecycle true
9 verbose lifecycle<EMAIL_ADDRESS>PATH:
10 verbose lifecycle<EMAIL_ADDRESS>CWD: C:\Users\benjamin.petersen\Documents\GitHub\redux
11 silly lifecycle<EMAIL_ADDRESS>Args: [ '/d /s /c', 'rimraf lib dist es coverage' ]
12 silly lifecycle<EMAIL_ADDRESS>Returned: code: 1  signal: null
13 info lifecycle<EMAIL_ADDRESS>Failed to exec clean script
14 verbose stack Error<EMAIL_ADDRESS>clean: rimraf lib dist es coverage
14 verbose stack Exit status 1
14 verbose stack     at EventEmitter. (C:\Users\benjamin.petersen\AppData\Roaming\npm\node_modules\npm\lib\utils\lifecycle.js:239:16)
14 verbose stack     at emitTwo (events.js:87:13)
14 verbose stack     at EventEmitter.emit (events.js:172:7)
14 verbose stack     at ChildProcess. (C:\Users\benjamin.petersen\AppData\Roaming\npm\node_modules\npm\lib\utils\spawn.js:24:14)
14 verbose stack     at emitTwo (events.js:87:13)
14 verbose stack     at ChildProcess.emit (events.js:172:7)
14 verbose stack     at maybeClose (internal/child_process.js:817:16)
14 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:211:5)

My corporate environment is slightly behind the times and has slightly older versions of whitelisted apps, below is the most up-to-date they allow.

I’m not sure I’m following. What whitelisted apps?

Failed at<EMAIL_ADDRESS>clean script 'rimraf lib dist es coverage'.

Are you installing Redux for usage or development? npm install redux should not be running clean.

Closing as lacking the necessary information.