Fixing up event handlers to break some cycles that can cause leaks

Summary of the pull request
Addressing an issue where a couple of events are contributing to the WidgetViewModel and the dashboard leaking through the cycles that get formed when non static event handlers are added.  Note that even with this change, the WidgetViewModel is leaking which is being investigated but this reduces the sources contributing to it leaking.
References and relevant issues
Contributes to ADO#44532978
Detailed description of the pull request / Additional comments
When instance level event handlers are added to an event, the instance that the event is from is kept alive for the lifetime of the event handler.  In the WidgetViewModel, we were holding onto an instance of the Widget model and setting up an event handler from it to subscribe to widget notifications.  This meant we had a cycle where the Widget model is holding onto a CCW for an event handler belonging to the WidgetViewModel while at the same time the WidgetViewModel is keeping the Widget alive.  The .NET GC cannot detect such cycles especially for non XAML scenarios and thereby it contributes to the view model leaking.  Addressed by introducing another class which is used to achieve weak event handlers.
On the dashboard page, we also had a couple of instance event handlers we were adding to events on static objects.  This meant the event handlers were keeping the first dashboard instance alive for the lifetime of the static object which is forever.
Validation steps performed
Validated scenarios continue to work, but note we still do have other things still causing the leak to happen.
PR checklist

[ ] Closes #xxx
[ ] Tests added/passed
[ ] Documentation updated

@manodasanW this has a merge conflict now

@manodasanW this has a merge conflict now

I plan on taking a look soon.  I might have another change to add here which is why I held off.

Closed and reopened to trigger new PR validation.

@manodasanW I rebased this onto the current main. Not sure if there was still more you wanted to do here.

@manodasanW do you think we could get this in, once the merge conflicts are fixed?


Fix multipart/form-data boundary parsing

I've spent several hours troubleshooting an issue where I couldn't get a simple plain python3 script to emulate a cURL request against haproxy-lua-acme, powered by haproxy-lua-http.
As it turns out, cURL uses boundaries like this:
----------------------e45e387473048941

while python's httplib uses boundaries like this:
6bc4c0b431e13fda6748fe624a1002c9

However, those boundaries are all prefixed by --, as per https://tools.ietf.org/html/rfc7578#section-4.1, to separate parts.
On cURL-style boundaries, M.request.parse_multipart does fine, but it chokes on python's dash-less boundaries and mixes up the boundary prefix and the boundary itself.
This PR aims to fix that. I've applied that patch to my httpd.lua and I've been able to do the same request using cURL as python3.
I am no Lua developper by any stretch of the imagination so do not hesitate to point out issues, even if the fix is small.

Hello there, nice catch!
In theory it should not matter since we only need to match that entire "boundary" appears[1] in the separating line, however, the code handling it is probably to fragile so we need to refactor it anyway. I suspect we might have problems with handling chunked parts as well (since headers can appear there too).
Is it too much to ask you for short steps for reproducing the issue. While your patch probably solves the issue, we might have other problems, so I'll investigate further.
[1] Per rfc2046
NOTE TO IMPLEMENTORS:  Boundary string comparisons must compare the
boundary value with the beginning of each candidate line.  An exact
match of the entire candidate line is not required; it is sufficient
that the boundary appear in its entirety following the CRLF.

I sadly haven't managed to simplify much the steps for reproducing, I've compared the two following behaviors:
$ curl -XPOST http://<IP_ADDRESS>:9011/acme/order -F<EMAIL_ADDRESS>-F<EMAIL_ADDRESS>-F 'domain=test.example.com'  -v

and
import requests
response = requests.post(
    'http://<IP_ADDRESS>:9011/acme/order',
    files={
        'account_key': ('account.key.pem', open('account.key.pem', 'rb'), 'application/octet-stream'),
        'domain_key': ('domain.key.pem', open('domain.key.pem', 'rb'), 'application/octet-stream'),
        'domain': (None, 'test.example.com', None),
    },
)
print(response.__dict__)

against an instance of the latest haproxy-lua-acme service on CentOS7 & haproxy 2.2 (package provided by IUS). Due to my limited knowledge of Lua and its haproxy integration, I'm afraid I'm not able to go one level deeper and test against a clean service without any lua-acme coupling.
Note: both account.key.pem and domain.key.pem were generated using openssl genpkey, but any non-empty file should do

That's good enough, you're using requests on the python side, which is the thing I need to know.
I'll be busy to handle it this week, probably early next week. I guess your use case is covered for now, but we'll keep this issue open until the fix is made (either your pull request, something more generic, or combination of the two).
Thanks!

This is fixed in the latest master, it got slightly complicated, but your PR (correctly checking for double dash) is an important part of it. Thanks again, hope it works now for python clients.


Feature DEVX-355

DEVX-355
Refactor example app

Released as part of https://github.com/primer-io/primer-sdk-ios/pull/529


Use NoCopy instead of Populate for volumes.

The default in Docker is to set populate, and the flag for turning this
off is nocopy in the -v syntax, we should match here.

LGTM

[Current coverage][cc-pull] is 54.77%

Merging [#1118][cc-pull] into [master][cc-base-branch] will decrease coverage by 0.12%

@@             master      #1118   diff @@
==========================================
  Files            77         77
  Lines         11923      11922     -1
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           6545       6530    -15
- Misses         4476       4487    +11
- Partials        902        905     +3

[][cc-pull]

Powered by Codecov. Last updated by [d6ba2cd...b0abf5e][cc-compare]
[cc-base-branch]: https://codecov.io/gh/docker/swarmkit/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/docker/swarmkit/compare/d6ba2cd369d6e87b46f90bdf64a34308b0d8109a...b0abf5e303217f4ecc7eeee3f620ed8364a96dc8
[cc-pull]: https://codecov.io/gh/docker/swarmkit/pull/1118?src=pr


My aimbot perfectly aims at the target but misses completly

i disabled spread and recoil.
Why does it miss? is csgo broken or what.
https://streamable.com/xqlmwz

How did you disable spread and recoil? I couldn't find that option.

i noticed the same thing

How did you disable spread and recoil? I couldn't find that option.

weapon_recoil_scale 0
weapon_accuracy_nospread 1

Thats in CSGO's console, That 100% doesn't affect Osiris workings. however that will change your spread and recoil in offline games, so make sure those are set back to default when using Osiris Aimbot in Offline games. Since both you @notgoodusename and @moonm00n are experencing this problem it could be another setting in Osiris. I would help more but I don't know enough about Osiris to help you fix the problem. Srry :P

I dont think I said this clear enough, but setting:

weapon_recoil_scale 0
and
weapon_accuracy_nospread 1
could atualy be creating the problem of "My aimbot perfectly aims at the target but misses completly". As Osiris might still be atempting to recoil control, but doesnt need to because of those two settings, therfore the recoil cortrol of Osiris would just be making you miss down and to the left and/or right (depending on the weapon and its recoil pattern).

So basically to fix this, put:

weapon_recoil_scale 0
and
weapon_accuracy_nospread 1
back to there original values.

I dont think I said this clear enough, but setting:

weapon_recoil_scale 0

and

weapon_accuracy_nospread 1

could atualy be creating the problem of "My aimbot perfectly aims at the target but misses completly". As Osiris might still be atempting to recoil control, but doesnt need to because of those two settings, therfore the recoil cortrol of Osiris would just be making you miss down and to the left and/or right (depending on the weapon and its recoil pattern).
So basically to fix this, put:

weapon_recoil_scale 0

and

weapon_accuracy_nospread 1

back to there original values.

Look at the rcs code it accounts for weapon_recoil_scale when pulling so that's not an issue.

welp then IDK what could cause it, also thx for the information @effex1337

I dont think I said this clear enough, but setting:

weapon_recoil_scale 0

and

weapon_accuracy_nospread 1

could atualy be creating the problem of "My aimbot perfectly aims at the target but misses completly". As Osiris might still be atempting to recoil control, but doesnt need to because of those two settings, therfore the recoil cortrol of Osiris would just be making you miss down and to the left and/or right (depending on the weapon and its recoil pattern).
So basically to fix this, put:

weapon_recoil_scale 0

and

weapon_accuracy_nospread 1

back to there original values.

Look at the rcs code it accounts for weapon_recoil_scale when pulling so that's not an issue.

no, there is a variable that adjust to the recoil so this doesnt make sense
const static auto weaponRecoilScale{ interfaces.cvar->findVar("weapon_recoil_scale") };
auto aimPunch{ localPlayer->aimPunchAngle() * weaponRecoilScale->getFloat() };
at aimbot.cpp

it could be the backtrack accidently shifting to the wrong tick? if you disable backtrack does it hit perfectly or still miss.

it could be the backtrack accidently shifting to the wrong tick? if you disable backtrack does it hit perfectly or still miss.

still misses

Lambda function you need:
        constexpr auto velocityExtrapolate {
            [](Entity* entity, const Vector& destination) constexpr noexcept {
                return destination + (entity->velocity() * memory->globalVars->intervalPerTick);
            }
        };

Example of usage:
                if (fov < bestFov) {
                    bestFov = fov;
                    bestTarget = config->aimbot[weaponIndex].velocityExtrapolation ? velocityExtrapolate(entity, bonePosition) : bonePosition;
                    bestAngle = angle;
                }

And you can look at my full re-implementation of Osiris aimbot at https://github.com/mdilai/JweegaCSGO/blob/master/Jweega/Hacks/Aimbot.cpp

Lambda function you need:
        constexpr auto velocityExtrapolate {
            [](Entity* entity, const Vector& destination) constexpr noexcept {
                return destination + (entity->velocity() * memory->globalVars->intervalPerTick);
            }
        };

Example of usage:
                if (fov < bestFov) {
                    bestFov = fov;
                    bestTarget = config->aimbot[weaponIndex].velocityExtrapolation ? velocityExtrapolate(entity, bonePosition) : bonePosition;
                    bestAngle = angle;
                }

And you can look at my full re-implementation of Osiris aimbot at https://github.com/mdilai/JweegaCSGO/blob/master/Jweega/Hacks/Aimbot.cpp

i have done just that on my project and i am using it in the video


snippet versioning

Snippet code versioning

In the future, you can sync your snippets with github gist and gist will take care of snippet code versioning :smile:


Implement array_except function

Is your feature request related to a problem or challenge?
Summary

Characteristic
Description

Function name:
array_except

Aliases:
list_except

Original function?:
No

Function Description:
Azure DataBricks: Returns an array of the elements in array1 but not in array2.   Spark SQL:  Returns all elements from col1 array but not in col2 array.

Sources:
Concept Azure Spark

Examples:
select array_except([1, 2, 3, 4], [2, 3, 1, 5]);
----
[4, 5]

select array_except([1, 2, 2, 3, 4], [1, 3, 1, 5]);
----
[2, 4]

Describe the solution you'd like
No response
Describe alternatives you've considered
No response
Additional context
No response

Plan to work on this


standardised nested date mapping

Proposed solution for various issues surrounding date fields, including:

Mixed precision levels which would complicate https://github.com/UUDigitalHumanitieslab/I-analyzer/issues/992
https://github.com/UUDigitalHumanitieslab/I-analyzer/issues/1114
The awkward UI of having four (!) date pickers for "earliest earliest date", "latest earliest date", etc.

I would propose using a single nested field mapping for all date fields. For instance, we can use four subfields:

exact: the exact day, if known
year: the exact year, if known
lower: lower bound for the date
upper: upper bound for the date

exact and year would not always be filled in, of course, but you could make it so the "less precise" values are inferred. (Either during data entry or by elasticsearch.) So in a corpus with exact dates, you could still request date.year, date.lower and date.upper.
Thus, a date filter for 1800-1900 would look something like the following:
{
	"range": {
		"date.lower": {
			"gte": "1800-01-01",
			"format": "yyyy-MM-dd"
		},
		"date.upper": {
			"lte": "1900-01-01",
			"format": "yyyy-MM-dd",
		}
	}
}

Which should work on any corpus with a date field, regardless of precision.
There is a broader question whether standardising corpus fields is a good idea (c.f. https://github.com/UUDigitalHumanitieslab/I-analyzer/issues/993), but then date fields are already "lightly" standardised by being referenced in the corpus metadata.

I think you would also need date.estimate or such for to make histograms. (If you only know a range, that is just the middle point.)

I like it!

How about we only introduce this date field for the corpora for which the estimates apply? That would mean we'd need to check in the frontend on whether or not the date field has subfields, but it would also save some reindexing hassle.

It would be the most urgent for those corpora, but the main advantage would be that you could have a unified query format for all corpora.  Especially relevant for #992

But since we currently do not have a unified format, it's not an issue to let a "new" mapping exist alongside a deprecated one. In the case of corpora like sweden-old, the new format would still be an improvement.

Some updates after experimentation: the field we probably want to use here is the Object field type. Other than a multifield, it allows throwing multiple values at it. So an indexing operation would become:
PUT testcorpus/_doc/1
{
  "date.year": "2010",
  "date.estimate": "2010-06-30",
  "date.earliest": "2001-01-01",
  "date.latest": "2020-12-31"
}

The query should always be performed on the date.estimate field, as we cannot pass multiple fields in a range query. It also makes more sense that way, as then we don't need to make up a date.earliest and date.latest field if we have an exact date. With an exact date, we would simply set estimate = exact, and if we only have a year field, we set estimate = yyyy-01-01, or something along those lines. And if we have earliest/latest fields, we set the estimate to a value in between. Does this sound like a good strategy?

Thanks for looking into this!

The query should always be performed on the date.estimate field, as we cannot pass multiple fields in a range query.

This seems like an issue to me. In the example you gave, if I'm searching for documents up to 2005, I would want that one to be included.  (I'm sure there are also cases where you would want stricter searching. You could include strict vs. lenient as an option to the filter.) Users currently have the option to search like that in corpora with date ranges.
That said, I don't think it's really a problem if a single Filter object in the frontend translates to two elasticsearch filters (i.e. one for the earliest field and one for the latest field). Our current implementation does assume that, but it should not be so difficult to adjust.

It also makes more sense that way, as then we don't need to make up a date.earliest and date.latest field if we have an exact date.

I suppose that would be true if you would not use earliest/latest for querying, as you propose.

I'm now moving away from the object field type. A multifield is not an option: using a date string as an input value and mapping it to an integer is not what multifields were designed for. Perhaps, an ingest pipeline with a custom analysis script which will map the input to year integer / date internally could make this work, but trying this will take time and may in the end amount to nothing.
The object field type would work, but comes with disadvantages:

the current date field validator fails, and the object field type is too loose to write a proper validation against
as stated above, it requires reindexing.

What I would propose instead is the following:

for all corpora with approximate dates, we introduce the following fields
date_earliest
date_latest
date
The description of the date field should be explicit about that it is an estimate. This field will solely be used for the frontend filter, whereas constructing the Elasticsearch filter will done by the backend. The backend will then check if there are date_earliest / date_latest fields present, and if so, do a composite must query which filters on these fields instead. Does this sound like a solution?
Advantage:
no need to reindex corpora with "normal" date fields
we can still introduce a date field across all indices

A problem which does remain is that Elasticsearch date fields cannot handle years lower than year 1. So for some of the inscription corpora (with year estimates 0 and lower), we still cannot provide a proper date filter.

In cross-corpus search, what would a query look like if it's filtering one corpus with exact dates and one corpus with date ranges?

Argh, good point, that. Didn't think of the cross-corpus situation: then we'd filter on date estimates in this case, which may not be what the user wants in all cases. That means we do indeed need date_earliest / date_latest fields on all corpora.

Yes, that was kind of why I proposed this.
Regarding re-indexing:

We won't build cross-corpus search anytime soon, and it would be fine to introduce a "new date field" alongside the old one.
I cannot imagine we'll get user-friendly cross-corpus search without reindexing most corpora. Stuff like making sure the same "concept" is under the same field name, or fields with the same name have the same mapping.

My proposal would then, still, be, to add new flat fields over adding a very complex "object type" field. I expect that retrieval will be faster that way, and we could even do this with an update script if we wanted.

I was also wondering about field naming conventions. We often choose the underscore for "Pythonic" reasons. Should we internally decide on a convention, or would this just make the application cumbersome to adapt for other developers? I can imagine field names such as date:later (date.later would mean Elasticsearch goes to assume object type), but right now, field names are a slug field, which means we cannot use anything but -_[Aa-Az][0-9] in the field names.

Btw, did you look at date_range fields?
Naming conventions are fine if you're managing your index from I-analyzer. Right now, you could also connect to an existing index, but ideas like this issue would make that problematic anyway.

Good find! This may be what we want, since it allows a simple date_range query. This means that the date field could have two different mapping types underlying, but that we can search over them in the same way.


Unable to resolve dependencies in NODE-PATH using DOCKER

Is this a bug report?
YES
Description
Hi, I'm using docker to develop my solution.
In Docker I install the node-modules insite /opt folder and add this node_modules folder to NODE_PATH and PATH, if I go into the Docker container and run node then execute something like
> const react = require('react');

The output will be a success, this will happen in every location of the docker container, not just the /opt
To reproduce this issue I will show you step by step how I making my project.
My DockerFile is:
FROM node:8.11
# Install dependencies to run react app
RUN yarn global add<EMAIL_ADDRESS>serve && mkdir /tmp/client

# define envirnment variables
ARG NODE_ENV=production
ENV NODE_ENV $NODE_ENV

# expose port to run it
ARG PORT=4000
ENV PORT $PORT
EXPOSE $PORT

# Install node modules and add them to the PATH and Node Path
WORKDIR /opt
COPY package.json yarn.lock ./
RUN yarn install --silent --production=true
ENV PATH /opt/node_modules/.bin:$PATH
ENV NODE_PATH /opt/node_modules/:$NODE_PATH

# Define variable to check if build is required
ARG ENVIRONMENT=$ENVIRONMENT

ENV NPM_CONFIG_LOGLEVEL warn

# Copy project
WORKDIR /app
COPY . /app

# Run the build if the build-arg ENVIRONMENT is production
RUN if [ $ENVIRONMENT="production" ] ; then echo BUILD $ENVIRONMENT && yarn run build ; else echo DEVELOPMENT BUILD; fi

Now if I ejecute the docker build with
$ docker build --build-arg ENVIRNMOENT=production -t my_react_project .

Actual behavior
I will get this error message
react-scripts build
Creating an optimized production build...
Failed to compile.

Module not found: Error: Can't resolve 'react' in '/app/src'

error An unexpected error occurred: "Command failed.
Exit code: 1
Command: sh
Arguments: -c react-scripts build
Directory: /app
Output:
".

Now, If you go into the docker image running
$ docker run --rm -it my_react_project bash

Now you will be into something like
root@033a87fa9eae:/app#

Now you can execute node and test importing react, it will works
$ node

> const react = require('react');
undefined
> react
{ Children:
   { map: [Function: mapChildren],
     forEach: [Function: forEachChildren],
     count: [Function: countChildren],
     toArray: [Function: toArray],
     only: [Function: onlyChild] },
  Component: [Function: ReactComponent],
  PureComponent: [Function: ReactPureComponent],
.
.
.

Environment
As you can see my platform is
NODE: 8.11
yarn: 1.5.1
Reproducible Demo
You can clone this repo https://github.com/camiloperezv/login-template-react and reproduce the bug there.
I found this issue related https://github.com/facebook/create-react-app/issues/2230
Regards

Hi, we're not experts in Docker or this setup -- why are you installing react-scripts globally?
This seems better suited for Stack Overflow. Good luck!

I'm having the exact same issue with the same symptoms. It seems like npm test is not picking up NODE_PATH correctly even though it is set properly in the environment. I can't tell where the problem lies; it seems like it is somewhere between node, react-scripts and jest.
I can't imagine why the Docker environment would cause any problems. If I hack up the jest-resolve/build/index.js with debug output it clearly shows that NODE_PATH is not making it through:
  resolveModuleFromDirIfExists(dirname, moduleName, options) {
    const paths = (options && options.paths) || this._options.modulePaths;
    const moduleDirectory = this._options.moduleDirectories;
    const key =
      dirname + (_path || _load_path()).default.delimiter + moduleName;
    const defaultPlatform = this._options.defaultPlatform;
    const extensions = this._options.extensions.slice();
    console.log('debug', JSON.stringify({
        paths: paths,
        moduleDirectory: moduleDirectory,
        key: key,
        defaultPlatform: defaultPlatform,
        extensions: extensions,
        nodePaths: nodePaths,
        NODE_PATH: process.env.NODE_PATH,
        dirname: __dirname,
    }, null, 2));

This leads to the output:
debug {
  "moduleDirectory": [
    "node_modules"
  ],
  "key": "/opt/app/client/src:react",
  "extensions": [
    ".web.js",
    ".js",
    ".web.ts",
    ".ts",
    ".web.tsx",
    ".tsx",
    ".json",
    ".web.jsx",
    ".jsx",
    ".node"
  ],
  "nodePaths": null,
  "NODE_PATH": "",
  "dirname": "/modules/node_modules/jest-resolve/build"
}

The specific command I run doesn't seem to matter much between:

npm test
NODE_PATH=/modules/node_modules npm test
NODE_PATH=/modules/node_modules react-scripts test

Other notes and observations:

If I run node and then output the NODE_PATH it works as expected
jest-resolve itself is installed in /modules/node_modules/ and is loaded correctly -- I assume by node itself
The reason for moving the node_modules to /modules/node_modules is a detail of the Docker environment but is necessary in order to build the proper node-sass binary and keep the development environment sane
The issue is specific to using create-react-app. This pattern worked for me before I switched to create-react-app. (I was using webpack directly before and did have to add some configuration to make it happy.)
Trying to install jest-resolve globally did not fix the problem

My package versions:
root@03cc55342435:/modules/node_modules# npm ls react jest jest-resolve react-scripts
<EMAIL_ADDRESS>/modules
+--<EMAIL_ADDRESS>`--<EMAIL_ADDRESS>  +--<EMAIL_ADDRESS>  | `--<EMAIL_ADDRESS>  |   +--<EMAIL_ADDRESS>  |   | `--<EMAIL_ADDRESS>deduped
  |   +--<EMAIL_ADDRESS>  |   | `--<EMAIL_ADDRESS>deduped
  |   `--<EMAIL_ADDRESS>  |     `--<EMAIL_ADDRESS>deduped
  `--<EMAIL_ADDRESS>

Alright, I figured it out. The issue is that react-scripts is explicitly filtering out absolute paths from NODE_PATH. You have to change it to a relative path and then it works as expected. In my case, that was changing /modules/node_modules to ../../../modules/node_modules. Ironically, it then normalizes the path back to /modules/node_modules.
I'm not sure why it wants to filter out absolute paths. This is a pretty standard docker setup due to how volume mapping local development environments is so wonky.


Subscription oauth v2

Task/Issue URL: https://app.asana.com/0/1205842942115003/1207991044706235/f
iOS PR: https://github.com/duckduckgo/iOS/pull/3480
macOS PR: https://github.com/duckduckgo/macos-browser/pull/3580
What kind of version bump will this require?: Major
CC: @miasma13
Description:
This PR introduces the use of OAuth V2 authentication in Privacy Pro Subscription.
The code changes are comprehensive due to the paradigm changes between the old access token lifecycle and the new JWT lifecycle.
The Subscription UI and UX should be unchanged.
Steps to test this PR:
Test all Privacy Pro Subscription features and UX, more details here

Internal references:
Software Engineering Expectations
Technical Design Template

There's basically no PIR stuff in BSK at the moment


fix an error when delete volume

What this PR does / why we need it:
There is no error return to the cli when volume whoes profile id or pool id is empty was deleted.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

Coverage remained the same at 43.87% when pulling b1e4204b37148d6a21c8c4d7703fe6c97ef90f5f on lasshpanzer:deleteVolumeError into ea683dd450612f422c2a7eb910da1b3db2c0071e on opensds:development.


feat(metrics): Add remote IP label to containerssh_ssh_current_connections

Please describe what you would like to see in ContainerSSH
The remote server IP in ssh-proxy backend may change with the config webhook. We cannot get the current connections for a specific server. Thus I propose to add a new label server-ip to the metric containerssh_ssh_current_connections
Please describe your use case
I will create multiple ssh backends. And use them in ssh-proxy mode. The server IP is updated by the config server webhook. And I wanna know how many connections are alive in the server.

SGTM.


ROX-14398: Remove "Scope Manager" default Role and PermissionSet

Description
A detailed explanation of the changes in your PR.
Feel free to remove this section if it is overkill for your PR, and the title of your PR is sufficiently descriptive.
Checklist

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

If any of these don't apply, please comment below.
Testing Performed
TODO(replace-me)
Use this space to explain how you tested your PR, or, if you didn't test it, why
you did not do so. Valid reasons include, for example, "CI is sufficient",
"No testable changes". Feel free to attach JSON snippets, curl commands,
screenshots.
In addition to reviewing your code, reviewers must also review your testing
instructions and make sure they are sufficient.

Current dependencies on/for this PR:

master

PR #5848

PR #5849

PR #5850

PR #5851   👈

This comment was auto-generated by Graphite.

/retest


x/tools/cmd/goimports doesn't reorder standard library imports if they're already in separate blocks

What version of Go are you using (go version)?

$ go version
go version go1.17.5 linux/amd64

Does this issue reproduce with the latest release?
Yes (1.17.5 is the latest release, as of filing this issue)
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/dylan/.cache/go-build"
GOENV="/home/dylan/.config/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/dylan/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/dylan/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="go1.17.5"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/dev/null"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build3462334434=/tmp/go-build -gno-record-gcc-switches"

What did you do?
Use goimports to reorder imports, expecting to see standard library dependencies placed above other dependencies.
main.go:
import (
    _ "github.com/google/uuid"

    _ "time"
)

Command run: goimports main.go
What did you expect to see?
I expected to see
import (
    _ "time"

    _ "github.com/google/uuid"
)

following the convention that standard library imports are placed first. (This convention is followed by goimports when stdlib dependencies are mixed in with another block, as well as being mentioned on GitHub at https://github.com/golang/go/wiki/CodeReviewComments#imports)
What did you see instead?
No changes:
$ goimports main.go
import (
        _ "github.com/google/uuid"

        _ "time"
)

Other notes

The last comment on https://github.com/golang/go/issues/31646 mentions running into this same issue.
It's not clear what goimports is supposed to do from the documentation. If this is working as designed, it'd be helpful to note that in the docs.

Duplicate of #20818


New Component: HandleServerDownOrRestarted

I have an idea for a simple component without any gui but only javascript.
Preface:
When i restart, stop or reload my server on the client side the page can't be reloaded
and the user is currently not informed that the server is down at the moment.
So i tried the following to handle this special case which could also be
an improvement for other users:
1.) Try to ping the server via AJAX like the ping in the speedtest component.
2.) On Error remember that the server is currently offline and show a sidebar panel in fullscreen mode that the server is currently offline.
Retry with the ping as described on 1.)
3.) On Success check if the server was offline before. If so, now redirect to login page. If the server was not offline before retry with the ping as described on 1.)
Here is the sidebar i used in a general template:

							<p:outputLabel value="Server is currently offline." escape="false"/>

And here is the javascript which will do the action:
<h:outputScript>
	var SERVERPINGTIME = 30*1000; // in ms
	var SERVERTIMEOUTTIME = 10*1000; // in ms
	var isServerOnline = true;
	var ServerPingTimeout;
	var RedirectURL = "/index?faces-redirect=true";
	// Check server state
	function CheckServerState() {
		console.log("Checking current server state...");
		// Try to ping server and get result or timeout:
		$.ajax({
			url: "/dummy.html" + "?id\x3d" + "1",
			async: false,
			cache: false,
			type: "GET",
			success: function(a) {
				OnServerOnline();
			},
			error: function(a) {
				OnServerOffline();
			},
			timeout: SERVERTIMEOUTTIME
		});
	}
	function OnServerOnline() {
		// Redirect to start page
		if (isServerOnline == false) {
			console.log(" Server is online again, try to redirect to '"+RedirectURL+"'");
			window.location.href = RedirectURL;
		}
		// Re-Check
		if (isServerOnline == true) {
			console.log(" Server is online ...");
			ServerPingTimeout = setTimeout("CheckServerState()", SERVERPINGTIME);
		}
		isServerOnline = true;
	}
	function OnServerOffline() {
		console.log(" Server is now offline - retry ...");
		isServerOnline = false;
		// Show information that server is offline to user:
		PF('SideBarServerOffline').show();
		// Re-Check
		ServerPingTimeout = setTimeout("CheckServerState()", SERVERPINGTIME);
	}
	// Execute script on page ready
	$(document).ready(function()
	{
		// Install a timer to check the server's current state
		clearTimeout(ServerPingTimeout);
		ServerPingTimeout = setTimeout("CheckServerState()", SERVERPINGTIME);
	});
</h:outputScript>

The only thing here todo is to create a new component which
is handling the parameters like SERVERPINGTIME, SERVERTIMEOUT, RedirectURL, ...
because it's currently tested in production and it works as it should ...

@ssibitz @balta3 are either one of you going to submit a PR for this if not I will close the ticket.

No updates from the OP or any follow up so closing for now. Will reopen if someone donates a PR.


Bump-up Mypy to 1.11.2

Description
Bump-up Mypy to 1.11.2
Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library
[x] Bump-up library or tool used for development (does not change the final image)

/lgtm

/approve

/override "Red Hat Konflux / ols-enterprise-contract / lightspeed-service"

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

Project coverage is 94.75%. Comparing base (229391e) to head (9192721).
Report is 14 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1501      +/-   ##
==========================================
- Coverage   94.76%   94.75%   -0.02%
==========================================
  Files          59       59
  Lines        2581     2631      +50
==========================================
+ Hits         2446     2493      +47
- Misses        135      138       +3

see 1 file with indirect coverage changes


Feature: margins

Codewars is not showing new pushed commits from branch after failed test. Close this one, and create new pull request.


Added bg remover

#896
added bg remover

@Aditijainnn I accept this PR, however there is a rule that I address in the guidelines. The amount of changes contributors can make to the website is unlimited, but they can only add 150 points to projects. Thus, if you add any more projects after that, I will not be able to award you points, but if you make as much of an effort to contribute to the website as you can, I will.


bazzite-deck: Digital PCM Volume is lower than SteamOS default

SteamOS Default ucm2 settings for the acp5x seem to have it output a bit louder than the defaults we pull from jupiter-hw-support.  If we can adjust the following params in Hifi-Upstream.conf and HiFi.conf it should match up properly.
cset "name='Left Digital PCM Volume' 817" -> cset "name='Left Digital PCM Volume' 913"
cset "name='Right Digital PCM Volume' 817" -> cset "name='Right Digital PCM Volume' 913"

Addressed in https://github.com/ublue-os/bazzite/commit/f82baf75fbe5cb1f94614178d43608019e507f7c
Thank you so much!

Tested and the setting doesn't seem to have changed after receiving the update - on my Steam Deck I have another file in that directory with similar settings on it but I'm unsure which one ALSA is actually using.
/usr/share/alsa/ucm2/conf.d/acp5x/Valve-Jupiter-1.conf symlinked to /usr/share/alsa/ucm2/AMD/acp5x/acp5x.conf.
Can anyone confirm that they also have this present on their system?

That file appears to be from upstream, additionally I found a syntax error in Valve's own and fixed that with a patch. Latest build removes that file and symlinks it to the location Valve uses on their end,

It definitely took effect this time.  It seems to be falling back to a different configuration profile rather than using the ones intended.
This is what I get from alsactl info 2:
#
# Sound card
#
- card: 2
  id: acp5x
  name: acp5x
  longname: Valve-Jupiter-1
  driver_name: acp5x
  mixer_name:
  components:
  controls_count: 80
  pcm:
    - stream: PLAYBACK
      devices:
        - device: 0
          id: Playback/Capture nau8821-hifi-0
          name:
          subdevices:
            - subdevice: 0
              name: subdevice #0
        - device: 1
          id: CS35L41 Stereo Playback multicodec-1
          name:
          subdevices:
            - subdevice: 0
              name: subdevice #0
    - stream: CAPTURE
      devices:
        - device: 0
          id: Playback/Capture nau8821-hifi-0
          name:
          subdevices:
            - subdevice: 0
              name: subdevice #0

Working on my end now as of the latest build with the values you suggested.


Add extra padding before the start of each line in the editor #1875

fixes #1875
removed unnecessary files and changes.

Hi @sanariaz154 thanks for the PR! As a helpful github.com administration issue, can you add the line "fixes #1875" to your PR comment? That way github will set up a crosslink between the PR and the associated issue, and landing the PR will then automatically close the issue with it.

Hi @sanariaz154 - thank you for the PR. I have some feedback

Please put your style changes into bramble_overrides.css

brackets_codemirror_override.less was created by the Brackets team and I worry that if they update it in a new version, we may lose our changes.
We have full control over bramble_overrides.css is our own file, so I think it's the better choice

Your change also introduces extra spacing before the line numbers in the editor, which we don't want, here's a side by side...

Let me know if you can see the different I'm talking about

In the issue related to this you mentioned that changes to bramble_overrides.css aren't working. I was able to make changes in that file to get a similar effect to your changes. Are any CSS changes in that file working for you?
Thanks!

@sanariaz154 in case it's not obvious, you can run Brackets with npm start and then navigate to http:localhost:8000/src/hosted.html and you'll get just the editor without Thimble.  When I'm working on fixes just in Brackets, I tend to debug that way, which makes it a bit easier.
@flukeout you mention a "side by side" but didn't attach it, fyi.

Updated with a screengrab.

@flukeout    I understand  what you are saying I didn't notice the space before linenumbers before.

I was getting this error  when I did changes in bramble-overrides.css
 FileError:"http://localhost:8000/src/styles/bootstrap/utilities.less"  was'nt found(0)
but know the error is not showing and issue is resolved .

@humphd i was also running Brackets this way .
Thanks

Testing now, thank you!

Thank you for updating the PR, here is some feedback

Please remove any changes to brackets_codemirror_override.less it should not be a part of this PR
Please fix up your whitespace and indentation in your code, it should match the rest of the file and be consistent, your changes currently look like this...

Please indent and space the code consistently.
Lastly, in your PR, the DIV that acts as a click target for the code-folding arrows doesnt line up with the arrows anymore, take a look here, I'm highlighting it in the DOM

It has stayed a bit to the left, so clicking on some parts of the arrows doesn't work. To fix this, here's what you can add to your PR, this will move the click area to the right by 8 pixels and line it up with the arrows...
.CodeMirror-foldgutter {
  transform: translateX(8px);
}

.CodeMirror-foldgutter:after {
  display: none;
}

Thank you!

OK, testing now!

Ok, great! Works well as far as I can tell so we'll merge it! @humphd has had trouble with this kind of change in the past, so let's all keep an eye on this to see if causes any unexpected problems.
Thank you again @sanariaz154!

Thanks a lot, @sanariaz154.  Looking forward to having you work on other bugs with us.  Appreciate the contribution.

Thank you @flukeout  @humphd . you've been very supportive .I would like to work more ,I've searched from thimble.mozilla.org's issues labeled as good-first-bug but most of them are assigned. Can you please suggest me some other good first issues which are not yet assigned to anyone?

#889 is one you could do.  It's assigned, but I don't think he's working on it.  If you are interested, I'll ask him to let you do it.  https://github.com/mozilla/thimble.mozilla.org/issues/1251 is another one you might be good at, since it also deals with CSS in Brackets, but would be a bit more challenging.  https://github.com/mozilla/thimble.mozilla.org/issues/1588 is another I'd like to see, where you can right-click and "Beautify" your code to make it clean and tidy.


Move to Multiformats?

Hey Kyle!
Thanks so much for this. As you may know, we recently created the Multiformats organization to be a home for all of the multiformats - multiaddr, multihash, etc. Would you be interested in moving this repository to that organization? You would still have admin rights on the repository, but it would be a part of a wider organization. This would mean more relevant eyes on it (most likely) and better cross-repository issue tracking. We'd also add a line mentioning you as the original author, and of course your commits would stay the same.
Of course, keeping it on your profile is also cool; we'll still link to this from the main multihash repository at github.com/multiformats/multihash.
Thanks for taking the time to read this. 👍
Tracking issue, here: https://github.com/multiformats/multiformats/issues/4.

That sounds good to me. What is the next step here?

Added you as a member of the org. You can transfer it over, now, and I'll re-add you as an admin for this repo. Sound good?

I'm not seeing myself on the list, so I can't transfer it over (by transfer do you mean fork, or init as a new repo?)

Hmm, weird. I mean move the ownership over; see https://github.com/neocities/ruby-multihash/settings, and click "Transfer Ownership".

"You don’t have admin rights to multiformats"

Weird; I've added you twice to the Multiformats organization. Are you getting my invites in your inbox?

Nope, it was in a really weird place on their site! Okay it should work now, standby

Done.

Nice! I added you as an admin. What's next is for me to standardize the Readme, and link this from multiformats/multiformats.
Thanks Kyle!


Question: Where is the ui for this app?

Sorry to bother you but I just want to know where can I find the UI code for this app.
Side Note: This app is super cool and I love it, when I get my RPI back I would definitely test this app

Hi,
You can find UI code under lib/screens and then just look what you want under each folder (naming is bad right now, sorry). And there is also lib/components for reusable flutter components.

Thank you


Wrong item play/stop after drag-drop sorting

In some cases when items are deleted and/or sorted on the rundown the play/cont/stop commands effect the neighbouring item instead.
The root cause of this is the identification mechanism in the rundown view and it will need to be re-written to be based on UUID's, planned for 1.1.
Workaround: reload the view (F5 / Ctrl+R) after delete or sort.

Fixed in 1.0.4


cannot convert type string to type dynamic

Description
I manually translated these lines from spanish, sorry if they look odd
...dev\ioserver\Microsoft.NET.Sdk.Razor.SourceGenerators\Microsoft.NET.Sdk.Razor.SourceGenerators.RazorSourceGenerator\Pages_Reports_DeviceGroupReport_razor.g.cs(468,145): error CS0400: The type or namespace 'dynamic' does not exist in the namespace global (are you missing an assembly reference?) [...dev\ioserver\AdminDashboard.Server.csproj]
...dev\ioserver\Pages\Reports\DeviceGroupReport.razor(137,72): error CS1503: Argument 1: can't cast 'string' to 'dynamic' [...dev\io\ioserver\AdminDashboard.Server.csproj]

Configuration

Which version of .NET is the code running on?
7.0.100-preview.4.22252.9
What OS and version, and for Linux, what distro?
Win 10 pro
What is the architecture (x64, x86, ARM, ARM64)?
x64
Do you know whether it is specific to that configuration?
no
If you're using Blazor, which web browser(s) do you see this issue in?
compile time

Regression?
this works on 6.0.300

Other information
the dynamic parameter I assign to is a blazor component's parameter

Fixed by https://github.com/dotnet/razor/pull/6815


UI refactoring / bugfixes tasks

[x] fix search query subtitle wrong color

[x] fix search album info height for saved in library case

[x] fix missing search status loader when switching between queries using address bar
[x] refactor containers to react modules
[x] refactor components to react modules
[x] write tests for components (except pages)

[ ] center fallback message on the "latest albums" page


Mongo Support

Add logic to support MongoDB installations.

This will be supported when OSGi Configurations are available.


Redirect to login page after successfully activating account

Fixes #4439

Codecov Report

Merging #4447 into master will increase coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #4447      +/-   ##
==========================================
+ Coverage   20.05%   20.06%   +<.01%
==========================================
  Files         153      153
  Lines       30769    30769
==========================================
+ Hits         6171     6174       +3
+ Misses      23654    23652       -2
+ Partials      944      943       -1

Impacted Files
Coverage Δ

routers/user/auth.go
0% <0%> (ø)
:arrow_up:

modules/process/manager.go
73.91% <0%> (+4.34%)
:arrow_up:

Continue to review full report at Codecov.

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


Parsing Error for Example Code

Hello everyone, thanks for the amazing tool. I have an issue when I try to run the example given in the following link.
https://lmql.ai/docs/language/overview.html#dynamically-reacting-to-model-output
I am getting the following error. (I believe it is the commit number of the version 9a59739d980a88e64687689b7cecee6cbaf79b78 (From the right top corner))
RuntimeError: parsing error: Failed to parse where clause of the query (unexpected indent):

	   STOPS_AT(FURTHER_ANALYSIS, ".") \

Hi hiroo, can you share more information about your setup? Are you running in the Playground or from within Python?
From the error message it looks like when copying, the indentation of the code may have gotten misaligned. Note that LMQL has the same indentation rules that Python does.

I just tried again and yes, it was the indentation problem. Sorry for alarming up. Closing the issue.


feat: Remove streaming.parsePrftBox config

The site code is changed to use a parser that is always used and thus we avoid having a configuration that is not necessary.

Would you say we would need to have a deprecation warning for this removed config value in shaka.Player.configure? Or was it too recent for that, I'm not sure.

Since before the event was only fired when the configuration was true (default false). And now we always fire the event. It's not a problem

Would you say we would need to have a deprecation warning for this removed config value in shaka.Player.configure? Or was it too recent for that, I'm not sure.

Since before the event was only fired when the configuration was true (default false). And now we always fire the event. It's not a problem

I was thinking more so that we don't create an error log for people who are currently setting the configuration value to true. It can say that the event is now fired without needing a configuration value, perhaps.
Maybe that's not really necessary though.

Would you say we would need to have a deprecation warning for this removed config value in shaka.Player.configure? Or was it too recent for that, I'm not sure.

Since before the event was only fired when the configuration was true (default false). And now we always fire the event. It's not a problem

I was thinking more so that we don't create an error log for people who are currently setting the configuration value to true. It can say that the event is now fired without needing a configuration value, perhaps. Maybe that's not really necessary though.

Done!


Can't access WSL2-Ubuntu-20.04's file system by "\wsl$Ubuntu-20.04"

I installed the Ubuntu-20.04 of WSL2 through Microsolft Store.
Then in the subsystem:
"sudo apt update"
...
"sudo apt upgrade"
...
After that,
the explorer of windows10 will always be loading state when I type "\\wsl$Ubuntu-20.04" in its search box,
seems that it can't find the Ubuntu-20.04's files.
However,
I can successfully access the Ubuntu-18.04's files by typing "\\wsl$Ubuntu-18.04"
Is it a Bug of WSL2-Ubuntu-20.04?

I need to access my distros via \\wsl$\Ubuntu-20.04. Note the slash between the dollar sign and the distro name. Does that fix your issue?

What is the output, from a powershell or CMD prompt:
C:\> cmd.exe /c ver
C:\> wsl.exe -l -v

The prompts are like the picture showing.------------------ 原始邮件 ------------------
@.>
发送时间: 2021年6月8日(星期二) 上午6:22
@.>;
<EMAIL_ADDRESS>主题: Re: [microsoft/WSL] Can't access WSL2-Ubuntu-20.04's file system by "\wsl$Ubuntu-20.04" (#7039)

@fetherolfjd
It didn't work...

@therealkenc
The prompts are as follows:
C:> cmd /c ver
Microsoft Windows [version 10.0.19042.985]
C:\ wsl -l -v
NAME            STATE           VERSION

kali-linux      Stopped         2
Ubuntu-18.04    Stopped         2
Ubuntu-20.04    Running         2

Is it a Bug of WSL2-Ubuntu-20.04?

All things being equal, WSL proper doesn't know one distribution from the next. And the distributions (f.e. Ubuntu-20.04) isn't responsible for sharing \\wsl$\Ubuntu-20.04.

If kali-linux and Ubuntu-18.04 both work, but Ubuntu-20.04 "always be loading state", one thing to try on a lark is install Ubuntu (sic) from the store and see how that fares.


Fix addRequestTarget not affect anything.

When Sign and Verify, (request-target) are ignored.
Because func addRequestTarget() receive Buffer b as call by value,
so inner processing don't affect b in signatureString().
Tests response ok, because signing and verifing takes same mistake.
I don't have good idea to fix them now.

Thanks for this!


cannot download files greater than 15gb

everytime i add a torrent greater than 15 gb it downloads all the way but in my drive it shows a size of 0B. on rechecking the torrent everything seems fine. is it because i have a free google drive account(15gb storage)?

Yes you can't download files bigger than your Google Drive size but I tried downloading file bigger than 15GB I also use free version of GDrive but only some of the files were not existed. This is weird I'm sorry but I actually don't know how to fix it.

I've tried again and when my drive full it stopped for a while and resumed after that. But I can see downloaded files in my drive. Maybe try terminating your session and downloading again. Sorry for taking your time.
To terminate session: Runtime > Manage Sessions > Terminate Session

you need an unlimited gdrive account or an edu account for unlimited gdrive storage.

@rhyzq Yes this is right but the problem is he mentions his drive is empty. This isn't what should've happened. I tried downloading bigger torrent than my GDrive limit even it stopped downloaded files were still there

@FKLC @Grainlord  it happens for me too sometimes, i just restart all runtime(runtime>reset all runtimes)  then start that torrent again and fastresume will check the file, or u can just delete the 0b file then redownload the torrent.

@rhyzq thank you for answer. so @Grainlord can you please try @rhyzq method first then my method and inform us so i can put this question to FAQ.

@FKLC tried both the methods, any file greater than 15gb doesnt show in the drive.

try to download using unlimited gdrive account, u can google how to make it.

this is the prove that u can download a file more than 15gb using google colab but i am using google drive unlimited account

@FKLC @rhyzq Got a .edu acoount now. Having no issues.
But what if you can make it so that instead of downloading torrents to my own cloud it downloads it to a specific .edu account which is monitored by a script which deletes data older than 2 days or something? So like a free unlimited cloud for everyone to access. idk its just a thought.

Queueing is not available but you can download in parallel. Just run adding magnet link cell as many times you want
Sorry but we can't delete files older than 2 days because it needs to be checked everyday and you can't hold Colab servers online.
If all of your questions are answered. Please close the issue.

ok, thanks


Make useHandleFormikSubmission call onError

make useHandleFormikSubmission call onError
rearrange hook arguments (BREAKING CHANGE)

Sorry @mmalfertheiner I forgot to push the changes for the documentation before, that's why i requested a new review...


NEW: ISO 1996:2 2007. Objective method for assessing audibility of tones

This adds an implemention of ISO 1996:2 2007, annex C.
To do:

[ ] examples
[ ] tests

yield from is only available in python >= 3.4.

Superseded by #169


Fix in-page links on ToS

The sidebar links weren't working on the legal notices. It turns out that this page was using TableOfContentsSidebar where as every other page was using NestedTableOfContentsSidebar. I consolidated this usage and deleted the TableOfContentsSidebar.
Existing
https://roadie.io/legal-notices/terms-of-service/v4/
ToS and DPA headings run into each other and in-page links don't work.
With change
https://deploy-preview-1022--roadie.netlify.app/legal-notices/terms-of-service/v4/

https://app.shortcut.com/larder/story/14888


[Bug]:  Scrollbar cannot be dragged when website is in dark mode

Give a brief about the bug ✍
The scrollbar doesn't work properly, as in you cannot drag it up or down, when dark mode is in use. The dragging only works when the light mode is on.
https://github.com/kunjgit/GameZone/assets/141991095/899a603f-fe29-414f-b80c-090bf08d188c
What is the expected behavior? 🤔
The scrollbar should function irrespective of the display settings and user should be able to drag it up or down no matter what light settings are set.
Provide step by step information reproduce the bug 📄

Go to the website.
If dark mode is one, try to drag the scrollbar.
Now switch to light mode and try dragging the scrollbar again.

Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @techy4shri !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign

@kunjgit hey there! I just checked and the issue seems to be no longer present on the website. How come this happened? Please look into this. How can someone else work on ana issue I raised?

Hey @techy4shri ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Crawler is connecting then disconnecting??

Using river web 2.4.0
My es version:
{
name: "LinuxGrants",
cluster_name: "LinuxOER",
version: {
number: "2.4.0",
build_hash: "ce9f0c7394dee074091dd1bc4e9469251181fc55",
build_timestamp: "2016-08-29T09:14:17Z",
build_snapshot: false,
lucene_version: "5.5.2"
},
tagline: "You Know, for Search"
}
Log shows:
2017-06-27 12:18:34,478 [main] INFO  Connected to xxx.xxx.xxx.xxx:9300
2017-06-27 12:18:34,712 [Crawler-836317c4-95e9-485a-9c1a-935b2dea7117-1] INFO  Crawling URL: http://www.xxxxxxxxxxxx.com/
2017-06-27 12:18:34,747 [Crawler-836317c4-95e9-485a-9c1a-935b2dea7117-1] INFO  Checking URL: http://www.xxxxxxxxxxx.com/robots.txt
2017-06-27 12:18:34,809 [Crawler-836317c4-95e9-485a-9c1a-935b2dea7117-1] INFO  Redirect to URL: http://www.xxxxxx.com/
2017-06-27 12:19:06,012 [Thread-0] INFO  Disconnected to LinuxOER: xxx.xxx.xxx.xxx:9300
I tried changing to https and still the same thing. Any help?

What is the crawl config?


broken apt-get packages on ubuntu

Describe the bug
Since an hour ubuntu-latest as well as ubuntu-18.04 fails to execute the following command:
sudo apt-get install -y -qq build-essential gcc-multilib libsdl2-dev:i386 libsdl2-mixer-dev:i386
Relevant script down to failing step:
name: Build Workflow
on:
  push:
    branches:
    - master
    paths-ignore:
    - '**.md'
    - '**.yml'
    - '**/doc'

  pull_request:
    types:
    - opened
    - synchronize
    - reopened
    paths-ignore:
    - '**.md'
    - '**.yml'
    - '**/doc'

jobs:
  Linux-i386:
    runs-on: ubuntu-18.04

    env:
      BUILD_DIR: build
      BUILD_TYPE: Debug

    steps:
    - name: Setup Dependencies
      run: |
        sudo dpkg --add-architecture i386
        sudo apt-get update -y -qq
        sudo apt-get install -y -qq build-essential gcc-multilib libsdl2-dev:i386 libsdl2-mixer-dev:i386

Error log:
 Setup Dependencies
    BUILD_TYPE: Debug
Run sudo dpkg --add-architecture i386
  sudo dpkg --add-architecture i386
  sudo apt-get update -y -qq
  sudo apt-get install -y -qq build-essential gcc-multilib libsdl2-dev:i386 libsdl2-mixer-dev:i386
  shell: /bin/bash -e {0}
  env:
    BUILD_DIR: build
    BUILD_TYPE: Debug
E: Unable to correct problems, you have held broken packages.
##[error]Process completed with exit code 100.

Area for Triage:

Maybe Packages area?
Question, Bug, or Feature?:

Bug
Virtual environments affected

[ ] macOS 10.15
[x] Ubuntu 16.04 LTS
[X] Ubuntu 18.04 LTS
[ ] Windows Server 2016 R2
[ ] Windows Server 2019

Expected behavior
The command worked for many months and started failing just an hour ago.
Working build.
Failing build
Actual behavior
The issue occurs when additional packages are tried to be installed onto clean ubuntu VM. Just use the provided action script from the report. It should be possible to reproduce the issue with it.

Hello, @klei1984
I can't reproduce this behavior on the same image - Version: 20200512.2. Could you please run 2-3 times to confirm what the issue is not transient?

Hello @al-cheb
I re-ran the builds, about 5-6 of them, that failed yesterday. Now the Linux-i386 jobs worked without problems.
The virtual environment reported today by the logs is:
  Environment: ubuntu-18.04
  Version: 20200518.1

Yesterday the failed logs were produced by:
  Environment: ubuntu-18.04
  Version: 20200512.2

I am not sure how to specify to use the version in question for re-runs.

@klei1984 , Feel free to open the thread if you have any concerns.


New semester started but viewer still shows nothing

Spring 2024 semester begins today, but the viewer is still not showing that I have any classes on my schedule (likely stuck on winter 2023-24 still).
In addition to updating to the new semester automatically, it might be nice to have the option to pick the semester you want to view manually in order to reduce the possibility of issues like this

Feel free to open a PR similar to https://github.com/TheTripleV/uiuc-course-calendar/pull/4 updating the year and season and I'll approve.
Otherwise, I'll get to it in a few hours

I merged it in. It should have updated now.

#6


Support building rkt inside rkt container

This adds some new scripts and documentation details on how to use acbuild to create an ACI which has all build-time dependencies for compiling rkt, as well as how to use said image to compile rkt and output the result into a volume.
Vagrant still builds in the VM, not in a container, it would be slower to do it in a container because the image would need to be built first, but once we can pull in the builder image, it should be switched to use that & rkt for builds. Also, when rkt is being built in vagrant, the it now logs to /vagrant/rkt-build.log, because vagrant was printing the wget progress bar terribly (one line per dot blink), so if it looks like nothing is happening during the build, that's why.
Fixes #844

Awesome!
This could be used to build future releases of rkt.

@alban
Changed GIDs to 0 and linked to the acbuild bug in the docs.

@iaguis Addressed the comments.

It's unfortunate that after building rkt in rkt, ./configure has root:root owner because if you try to build again normally, you'll get a permission denied error.

@iaguis Any suggestions on how to fix it? I can try modifying the user/group but I don't think thats much better since they're basically arbitrary.

@iaguis Any suggestions on how to fix it? I can try modifying the user/group but I don't think thats much better since they're basically arbitrary.

Also: I'll be fixing this all up over the weekend, just haven't had time during the week.

@iaguis Cloning the repo kinda sucks if your trying to use this for local dev though, it makes testing changes you made locally difficult.

Yeah, that's annoying...

Also: I'll be fixing this all up over the weekend, just haven't had time during the week.

Thanks!

Ping?

@iaguis Sorry was gone this weekend, I'll take another pass at this this week.

@ecnahc515 bump

@ecnahc515 do you have time to continue this?

@alban Haven't had a lot of free time lately, so if someone wants to take this over that would be fine, but I can also try to make time for it.

I'd like to see this for 0.13.0. @ecnahc515 think you'll have time within the next 2 weeks? otherwise please unassign and mark as help wanted

@jonboulle I could probably take the time today/tomorrow to do this i think.

superb, thanks!
On Wed, Nov 25, 2015 at 11:19 AM, Chance Zibolski<EMAIL_ADDRESS>wrote:

@jonboulle https://github.com/jonboulle I could probably take the time
today/tomorrow to do this i think.
—
Reply to this email directly or view it on GitHub
https://github.com/coreos/rkt/pull/1681#issuecomment-159705737.

@ecnahc515 are you going to be able to get to this or should we reassign to someone else?

Okay, so I've gotten this up to date, rebased on master, and addressed the issues. I had to make other changes, but everything should be essentially how it worked before.
I had a few questions regarding the acbuild-rkt-builder.sh script and ACI(s) it produces. Firstly, I've set this up to work with Debian Sid and Fedora 22 builder base OSes. I don't know if the base OS name should be included in the ACI name, and I also wonder if I should output an ACI with the OS name in the file name.
Ex, currently it produces rkt-builder.aci with a name of coreos.com/rkt/builder. Maybe it makes sense to have rkt-builder-debian-22.aci with a name of coreos.com/rkt/builder-debian-22. I'm not sure. I just wanted to have two builders in case people want to test building rkt on the two main packaging platforms.
Just a note, if you attempt this right now, you'll need to run acbuild on master with https://github.com/appc/acbuild/pull/134 applied as well.
CC @jonboulle @alban @iaguis

@krnowak I think I've addressed all your comments. I can rebase the last commits once everything looks good to merge.

LFAD.

@ecnahc515: Thanks for your work!


Find matching configuration file for a model

Dear authors, thank you for your work and for your repo.
Is there a simple way how to find matching configuration .py file (from /configs folder) for a particular .pth file?
E.g., I am looking for the configuration files for models:
vfnet_r50_dcn_ms_2x_47.8.pth
vfnet_r2_101_dcn_ms_2x_51.1.pth
For ResNet 50 I have tried vrefnet_r50_fpn_1x_coco.py and vrefnet_r50_fpn_mstrain_2x_coco.py giving me mAP 0.91. For R2 I have tried vfnet_r2_101_fpn_mstrain_2x_coco.py, got error 'pth file is not valid checkpoint file'.
Thank you for your answer in advance.
Best regards
MV

Hi @Engin1980, thanks for interesting in our work.
vfnet_r50_dcn_ms_2x_47.8.pth -> vfnet_r50_fpn_mdconv_c3-c5_mstrain_2x_coco.py
vfnet_r2_101_dcn_ms_2x_51.1.pth -> vfnet_r2_101_fpn_mdconv_c3-c5_mstrain_2x_coco.py
The rule is : dcn -> mdconv,  ms -> mstrain.
By the way, if you use vfnet_r2_101_fpn_mdconv_c3-c5_mstrain_2x_coco.py and corresponding model file in inference, please comment delattr(self, 'conv2') in res2net.py.


Routes?

I was wondering whether it would be good/useful to have routes.
The difference to "old" routes eg stars is mainly that you don't have transitions and different phrasing. You would clear like r(r) BALAD3K or something like that. The general use, I suppose, would be stars that are different to US stars and routes that are more complex or routes in enroute control.
If possible we should add the ability to add more complex procedures to a route, like hold, if (functions), land, different waypoints (ones you have to overfly, ones you don't), as well as go around procedures (maybe the plane could even call the ATC and tell them!). I think most of them are pretty obvious except for if functions. i think they will be used for altitude checking (holding if not below/above a certain altitude) or activation of go a round procedures if ILS wasn't caught or something like that.
As I don't have the skill to work on this project, I would have to ask someone else to do so!
Is there anyone interested in working on these?
What do you think about about this, is is possible and worth the work or is there anything to add?

@jakobeng1303, yes it would be good/useful to have routes and it something which is being worked on.
As of right now, there is a route command in the game which can be used for giving an aircraft a list of procedures to follow.  It is limited to assigning a sequence of direct to fix, Airway segments, STARs, and SIDs.  No airport has yet implemented airways though.
The implementations of SIDs and STARs include the ability to specify altitudes and speeds along the procedure.  However as controller you don't have the ability to access that functionality, #315 had some relevant discussion.
As we implement the various features they are generally being done in a way that any procedure can make use of them.  Unfortunately we don't have a lot of programmer time working on the game right now so progress will be slow.  I get the impression you don't think of yourself as a coder.  If you have any inclination to program though, I would be happy to review your code and help you learn.  Other useful things can be thinking through how exactly the features should work (for example if the controller tells an aircraft their route is XYZ then is the aircraft also clear to fly all of XYZ or will further clearances be required?)


Feature request: Group bar chart by dataset

I'm fairly certain this doesn't already exist but apologies if it does.
It would be very nice to have another option for bar charts to 'group by dataset' so the dataset labels are shown on the axis and the legend shows bar labels, a bit like jqplot:

Many thanks.

@tannerlinsley Do the grouped datasets work this way?

Hover mode dataset is on the way. This will show tooltips by dataset.

Is this present in 2.1 ?

Closing as this feature appears to be already implemented in the current version.
Demo: http://codepen.io/zachpanz88/pen/ORQpEO


logoff() runs GET "account/logout" from IDP and receives 404 response

Executing OidcSecurityService.logoff() from my client (ASPNET MVC Core 2.0 Angular 4.3.6) ends up making this Get Request to my IDP (ASPNET MVC Core 1.1.2/IdentityServer4 1.5.2): "https://localhost:44333/account/logout?logoutId=ab01767bb9e09988434df8a7bc32e260". My IDP has only one "account/logout" and is a Post method. The debug output log shows a 404 response code.
The client setupModule has post_logout_redirect_uri = 'https://localhost:44345/signout-callback-oidc' and IDP has PostLogoutRedirectUris = new List() { "https://localhost:44345/signout-callback-oidc" }.
I was expecting my IDP redirecting back to my client in a state before logging in.
Am I missing something? :-\

It could/should redirect back to your client which logged in, At present , you need to click a link in the view. This could be removed. I look at this fully, after I update to .NET Core 2.0

Thank you again for your prompt response! 👍 .. :)
Sorry to ask but, what do you mean by "click a link in the view"? What view? Could you please elaborate on this view you mention? :-\
By the way, I created another client with ASPNET MVC Core 1.1.2/Angular 4.3.6 and it behaves the same.
The issue here, I believe, is that my IDP does not have a logout method to be called with an HTTP GET request, which is what your function "OidcSecurityService.logoff()" does. The logout method (generated out of the box when creating the client with ASPNET MVC Identity Authentication) is called with a POST Request. So that tells me I need to code an HTTP GET "account/logout" in my IDP.  To learn if I may be correct, I checked this sample link "https://github.com/IdentityServer/IdentityServer4.Quickstart.UI/blob/release/Quickstart/Account/AccountController.cs" and sure enough found this GET method:
[HttpGet]
public async Task<IActionResult> Logout(string logoutId)
{
    var vm = await _account.BuildLogoutViewModelAsync(logoutId);
    if (vm.ShowLogoutPrompt == false)
    {
        // no need to show prompt
        return await Logout(vm);
    }
    return View(vm);
}

When added to my code, your "OidcSecurityService.logoff()" function finally reaches this GET "account/logout" method (I probably need to declare the "AccountService _account" object, but I am not sure though).
Am I going the right path or am I making things more complicated than necessary?
Again, can you please elaborate what you meant by "click a link in the view" to see if your approach is enough? I really appreciate your time and effort on this. You are really, really awesome! :)

Hello Damienbod. Do you have any new comments from this issue I posted some days ago? :)


How to use this amazing plugin with listview?

As you know list view duplicate element by loop.
So, how can add the loop inside you plugin?
My listview is render inside the plugin  "ScrollableTabView"
Example code:
<ScrollableTabView
  style={styles.container}
  renderTabBar={() =>
    <DefaultTabBar backgroundColor='rgba(255, 255, 255, 0.7)' />
  }
  tabBarPosition='bottom'>
  <ListView
    style={{backgroundColor:'yellow'}}
    tabLabel='iOS'
    dataSource={this.state.dataSource}
    renderRow={(rowData) => <Text>{rowData}</Text>}/>
</ScrollableTabView>

I think you can't. The best thing you can do is to use a <BlurView /> for every single row in your ListView.


CSS - Question expansion: What are the different ways to visually hid…

Question expansion:

What are the different ways to visually hide content (and make it available only for screen readers)?

Many thanks! 🎉


Test commit

Signed-off-by: faizanahmad055<EMAIL_ADDRESS>

@faizanahmad055 Image is available for testing. docker pull stakater/workshop-excercise:SNAPSHOT-PR-20-cd7465ea

@faizanahmad055 Image is available for testing. docker pull stakater/workshop-excercise:SNAPSHOT-PR-20-cf7c4f72


runtime: Go 1.6.1 TestGDBPython is failing on linux amd64

Please answer these questions before submitting your issue. Thanks!

What version of Go are you using (go version)?

What operating system and processor architecture are you using (go env)?

What did you do?
If possible, provide a recipe for reproducing the error.
A complete runnable program is good.
A link on play.golang.org is best.

What did you expect to see?

What did you see instead?

I am the maintainer of Go on Gentoo Linux, and I have gotten a couple of reports of this test failing. I checked on my system, and it fails like this:
--- FAIL: TestGdbPython (0.65s)
runtime-gdb_test.go:42: gdb version 7.10
runtime-gdb_test.go:148: info goroutines failed: END
BEGIN print mapvar
No symbol "mapvar" in current context.
FAIL
FAIL	runtime	13.576s
Any help would be appreciated.

If you put a trivial "hello, world" in program in hello.go, what is the output of
go build hello.go
gdb hello
start
info goroutines

On Mon, Apr 18, 2016 at 03:23:00PM -0700, Ian Lance Taylor wrote:

If you put a trivial "hello, world" in program in hello.go, what is the output of
go build hello.go
gdb hello
start
info goroutines

william@linux1 hello $ cat hello.go
package main
import "fmt"
func main() {
fmt.Println("Hello world")
}
william@linux1 hello $ go build hello
william@linux1 hello $ gdb hello
GNU gdb (Gentoo 7.11 vanilla) 7.11
Copyright (C) 2016 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later http://gnu.org/licenses/gpl.html
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.  Type "show copying"
and "show warranty" for details.
This GDB was configured as "x86_64-pc-linux-gnu".
Type "show configuration" for configuration details.
For bug reporting instructions, please see:
https://bugs.gentoo.org/.
Find the GDB manual and other documentation resources online at:
http://www.gnu.org/software/gdb/documentation/.
For help, type "help".
Type "apropos word" to search for commands related to "word"...
Reading symbols from hello...done.
warning: File "/usr/lib64/go/src/runtime/runtime-gdb.py" auto-loading has been declined by your `auto-load safe-path' set to "$debugdir:$datadir/auto-load".
To enable execution of this file add
add-auto-load-safe-path /usr/lib64/go/src/runtime/runtime-gdb.py
line to your configuration file "/home/william/.gdbinit".
To completely disable this security protection add
set auto-load safe-path /
line to your configuration file "/home/william/.gdbinit".
For more information about this security protection see the
"Auto-loading safe path" section in the GDB manual.  E.g., run from the shell:
info "(gdb)Auto-loading safe path"
(gdb) start
Temporary breakpoint 1 at 0x401000: file /home/william/go/src/hello/hello.go, line 4.
Starting program: /home/william/go/src/hello/hello
[New LWP 5764]
[New LWP 5765]
[New LWP 5766]
[New LWP 5767]
Thread 1 "hello" hit Temporary breakpoint 1, main.main ()
at /home/william/go/src/hello/hello.go:4
4	func main() {
(gdb) info goroutines
Undefined info command: "goroutines".  Try "help info".
(gdb) quit
A debugging session is active.
Inferior 1 [process 5760] will be killed.

Quit anyway? (y or n) y
william@linux1 hello $ exit
exit

Well, that didn't work, because of this:
warning: File "/usr/lib64/go/src/runtime/runtime-gdb.py" auto-loading has been declined by your `auto-load safe-path' set to "$debugdir:$datadir/auto-load".

Try the same thing, but this time run
gdb -iex 'add-auto-load-safe-path /usr/lib64/go/src/runtime' hello

william@linux1 hello $ gdb -iex 'add-auto-load-safe-path /usr/lib64/go/src/runti me' hello
GNU gdb (Gentoo 7.11 vanilla) 7.11
Copyright (C) 2016 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later http://gnu.org/licenses/gpl.html
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.  Type "show copying"
and "show warranty" for details.
This GDB was configured as "x86_64-pc-linux-gnu".
Type "show configuration" for configuration details.
For bug reporting instructions, please see:
https://bugs.gentoo.org/.
Find the GDB manual and other documentation resources online at:
http://www.gnu.org/software/gdb/documentation/.
For help, type "help".
Type "apropos word" to search for commands related to "word"...
Reading symbols from hello...done.
Loading Go Runtime support.
(gdb) start
Temporary breakpoint 1 at 0x401000: file /home/william/go/src/hello/hello.go, line 4.
Starting program: /home/william/go/src/hello/hello
[New LWP 10142]
[New LWP 10143]
[New LWP 10144]
[New LWP 10145]
Thread 1 "hello" hit Temporary breakpoint 1, main.main ()
at /home/william/go/src/hello/hello.go:4
4	func main() {
(gdb) info goroutines

1 running  runtime.systemstack_switch
2 waiting  runtime.gopark
3 waiting  runtime.gopark
4 waiting  runtime.gopark
(gdb) quit
A debugging session is active.
Inferior 1 [process 10138] will be killed.

Quit anyway? (y or n) y
william@linux1 hello $ exit

Thanks.  Unfortunately I have no idea why the test is failing.  Someone with access to the failing system is going to have to debug this.

Hi,
Now there is more information about what is going on in the Gentoo bug, see
https://bugs.gentoo.org/show_bug.cgi?id=581072#C1

I experienced the same issue on my amd64 Ubuntu 14.04 machine. I was able to get around it using the following patch:
--- a/src/runtime/runtime-gdb_test.go
+++ b/src/runtime/runtime-gdb_test.go
@@ -98,6 +98,7 @@ func TestGdbPython(t *testing.T) {
    args := []string{"-nx", "-q", "--batch", "-iex",
            fmt.Sprintf("add-auto-load-safe-path %s/src/runtime", runtime.GO

         "-ex", "set startup-with-shell off",
          "-ex", "info auto-load python-scripts",
          "-ex", "br main.go:10",
          "-ex", "run",

I must admit I don't understand "startup-with-shell" is set to "on" on my machine.

Actually, I had to add the same line twice to pass all the tests:--- a/src/runtime/runtime-gdb_test.go
+++ b/src/runtime/runtime-gdb_test.go
@@ -98,6 +98,7 @@ func TestGdbPython(t *testing.T) {

        args := []string{"-nx", "-q", "--batch", "-iex",
                fmt.Sprintf("add-auto-load-safe-path %s/src/runtime", runtime.GO
+               "-ex", "set startup-with-shell off",
                "-ex", "info auto-load python-scripts",
                "-ex", "br main.go:10",
                "-ex", "run",
@@ -226,6 +227,7 @@ func TestGdbBacktrace(t *testing.T) {

        // Execute gdb commands.
        args := []string{"-nx", "-batch",
+               "-ex", "set startup-with-shell off",
                "-ex", "break main.eee",
                "-ex", "run",
                "-ex", "backtrace",

Submitted the patch for review: https://go-review.googlesource.com/#/c/23142/


IOS - Remove admin user after the invite new member in WS

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Create new WS(profile-> worspace-> create new WS)
Then click on members
Click on invites
Select some email and invite
Observe the email list

Expected Result:
Admin email should not be gone
Actual Result:
After inviting the users, i not able to see admin user in list of members list.
When i am go back and again go to member list then it is showing.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[x] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.36-5
Reproducible in staging?: n/a
Reproducible in production?: n/a
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/49875d5a-47bb-4d47-8d49-a77889358c1b
Expensify/Expensify Issue URL:
Issue reported by: @niravkakadiya25
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1688786931252889
View all open jobs on GitHub

I cannot reproduce this!

Commented in slack, neither me nor applause can repro. closing!


Missing OracleLinux.yml or override method for running CIS on Oracle Linux.

Describe the Issue
This issue goes back to  https://github.com/ansible-lockdown/RHEL7-CIS/issues/96 and the PR that was accepted.
At first read, I thought I could set the os check to false and let it work with OracleLinux.  Soon I discovered the in tasks/main.yml line https://github.com/ansible-lockdown/RHEL7-CIS/issues/96 that it doesn't escape the fact that there needs to be OracleLinux.yml file in vars directory for it to proceed or add a toggle there at line https://github.com/ansible-lockdown/RHEL7-CIS/issues/96 in tasks/main.yml not to use it.
I think that would mean a rule for running yum update would need to be disable or set to false.
Is there another way to get RHEL7_CIS and RHEL8_CIS to run on OEL 7 or OL7 and OEL8 or OL8?
Possible Solution
Add a OracleLinux.yml file or a method to allow me to override this need for a OS dependency file.

Are you reffering to PR #307 ?
Is it a workaround to add an empty OracleLinux.yml and to disable rule rhel7cis_rule_1_2_1 ?

hi
I have added the required files for rhel8-cis and should now be in devel branch. We are getting round to the rhel7-cis after we have caught up on all new vendor benchmarks and client priorities. RHEL7 unfortunately wasn't one of those.
If someone is able to get the file details for oracle on rh7 and test happy to bring that PR in. Hoping over the next couple of weeks we should be able to get on this one.
Apologies for the delay.
Regards
uk-bolly

Hi uk-bolly,
Thank you!!!


nao_sensor_msgsConfig fails build

Ubuntu 22.04, ROS2 Rolling
When building the package i got the following error

Do you have the same issue?

I cannot reproduce this issue. The error is saying it can't find nao_sensor_msgs installed, which is a binary dependency. Did you ensure you have dependencies installed correctly?
In your workspace:
rosdep install --from-paths src -i

Thanks for the reply. I missed this command in my setup.


stream-metadata: remove cache for 404 (stream not found/image not found)

We create the space on-chain before creating the stream.
With the 404 cache, we could accidentally hit /space/{spaceAddress}/image before the stream creation, causing and error and caching it. We don't want this for now.

Let’s ignore the warnings for now, let’s check the failed to get image logs too


[Question] Is it possible to autocomplete on TAB?

Describe the question
I'd like to know if it's possible to autocomplete local functions on TAB.
Expected behaviour
When I type the name of the function and choose a suggestion, I'd like to complete the function signature (parentheses and arguments).
Current behaviour
When I type the start of the function and choose a suggestion with TAB, it only completes the name of the function. It shows the signature but after hitting TAB, nothing happens.
Screenshots

Environment (please complete the following information):

OS: Ubuntu 18.04
Is WSL remote? No
Client: VSCode

Version: 1.62.0
Commit: b3318bc0524af3d74034b8bb8a64df0ccf35549a
Date: 2021-11-03T15:18:44.904Z
Electron: 13.5.1
Chrome: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Linux x64 4.15.0-161-generic

Extension: sumneko.lua-2.4.7

Provide logs
file_home_schica_CVEScannerV2.log
service.log

Lua.completion.callSnippet

Thank you very much, sir!


Replace maxage in load with more detailed cache option

Describe the problem
Pages can return a maxage property from load which a) sets a Cache-Control header on the server response, and b) causes the returned value to be cached in memory so that navigating back to the page won't result in a repeated load.
This mostly works, but has limitations:

determining whether to add public is based on a heuristic (no fetch without credentials: 'omit', and no session access) that doesn't cover every edge case
there's nowhere to express revalidate
some people want to be able to express s-maxage rather than maxage, though I'll confess I don't fully understand why

Describe the proposed solution
Replace maxage with a cache object:
<script context="module">
  /** @type {import('@sveltejs/kit').Load} */
  export async function load() {
    return {
      props: {...},
      cache: {
        maxage: 3600,
        private: false // if unset, uses existing heuristic
      }
    };
  }
</script>

In future, we could add revalidate and smaxage if those prove necessary.
Alternatives considered
No response
Importance
nice to have
Additional Information
No response

Hey @Rich-Harris -- would you like me to tackle this one? Happy to grab it if no one else is currently working on it.

That would be great @tcc-sejohnson, thank you!

@Rich-Harris
These changes will cause merge conflicts with the existing pull request I have open (#4515) -- the typedef for LoadInput has completely moved and changed to include the ErrorLoadInput properties.
If you're bullish on that change, I can base the current changes off of the tip of my feature branch to make the eventual merge easier. If you're bearish on it or think it needs a significant rework, I can base the changes off of current master and we can just deal with the merge conflicts when we get to them.

Disregard previous -- I see the PR has been merged. I'll work on this over the weekend and report back with any questions.

@Rich-Harris
Completion candidate: #4690
I exhaustively updated the tests to cover every edge case I could think of.

@Rich-Harris
Glad to see it all worked out (other than the snafu with the doc site -- oops). Just wanted to let you know this issue is still open. I can't close it :)

some people want to be able to express s-maxage rather than maxage, though I'll confess I don't fully understand why

@Rich-Harris I might be missing something obvious but, isn’t s-maxage the only way to cache on the proxy (CDN, Vercel for example)? I do this often on pages that don’t require real time updates since even a 5m edge cache can save a lot of computational power. It’s both a cost effective and performant (UX wise) approach.

@Rich-Harris

some people want to be able to express s-maxage rather than maxage, though I'll confess I don't fully understand why

I was just reading through the Vercel documentation and I found out why:

Recommended Cache-Control
We recommend that you set your cache to have max-age=0, s-maxage=86400, with changing 86400 to the amount of seconds you want your response to be cached for.
This recommendation will tell browsers not to cache and let our edge cache the responses and invalidate when your deployments update, so you never have to worry about stale content.

@Rich-Harris

some people want to be able to express s-maxage rather than maxage, though I'll confess I don't fully understand why

I was just reading through the Vercel documentation and I found out why:

Recommended Cache-Control
We recommend that you set your cache to have max-age=0, s-maxage=86400, with changing 86400 to the amount of seconds you want your response to be cached for.
This recommendation will tell browsers not to cache and let our edge cache the responses and invalidate when your deployments update, so you never have to worry about stale content.

Similar scenario with Cloudflare specific cache headers. I ended up handling cache headers in hooks to get the behavior I need.

Similar scenario with Cloudflare specific cache headers. I ended up handling cache headers in hooks to get the behavior I need.

Do you have a code example for this?

Similar scenario with Cloudflare specific cache headers. I ended up handling cache headers in hooks to get the behavior I need.

Do you have a code example for this?

There's a bunch of ways, depending on the use case.
One generic approach I do where I need to cache some routes on Cloudflare based on the route type:
// Can use routeId instead of the pathname
if(event.url.pathname.indexOf('/pathtocache') === 0) {
    response.headers.set('cloudflare-cdn-cache-control', 'max-age=1600')
}

Another use case is where I cache articles but I don't want to cache draft articles:
// hooks
const bypassCacheTestArticle = response.headers.get('cache-control')?.includes('testarticlebypasscache')
if (bypassCacheTestArticle) {
    // remove the header we use for the hack
    response.headers.set('cache-control', 'no-cache')
    response.headers.set('cloudflare-cdn-cache-control', 'no-cache')
} else {
    response.headers.set('cloudflare-cdn-cache-control', 'max-age=600')
}

// route
// return the header inside load when the article is in draft mode
response.cache = { maxage: 'testarticlebypasscache' };

didn't we have setHeaders already ? why do we need this cache object ?
we already have this
export const load: PageServerLoad = async (event: any) => {
	const token = event.locals.token
	try {
		const res = await getDataServer(api.getOrders, token)
		event.setHeaders({
			'cache-control': 'max-age=5',
		})
		return res
	} catch (error) {
		return {}
	}
}


// @ts-check on HTML inline script does not respect tsconfig.json

🔎 Search Terms
ts-check inline script tsconfig.json
🕗 Version & Regression Information
Typescript version 5.3.3
⏯ Playground Link
No response
💻 Code

This is index.html

This is index.js

This is project structure

This is tsconfig.json

{
  "compilerOptions": {
    "target": "es5",
    "lib": [
      "dom",
      "esnext"
    ],
    "module": "esnext",
    "esModuleInterop": true,
    "forceConsistentCasingInFileNames": true,
    "strict": true,
    "skipLibCheck": true
  },
  "files": [
    "index.js",
    "index.html"
  ]
}

🙁 Actual behavior
I use @ts-check in index.html's html inline script, but it seems that @ts-check  fails to read my tsconfig.json. There is an error in index.html's inline script but not int index.js. Their code is exactly the same.
🙂 Expected behavior
The script in html inline script should have same behavior as index.js.
Additional information about the issue
microsoft/TypeScript#25809

Yes html files cannot belong to a jsconfig project. This falls under #26338


Vagrant box is too big

The base bento/ubuntu-14.04 box is 517MB.
When I export my box (which is almost exactly the basic provisioned box) the size is... 2271MB
:cry:

Possible steps to take
[ ] Don't install recommended packages with apt-get
[ ] Remove build-time dependencies for Consular (and other python packages).
[ ] Check what packages the bento base image installs and check if those can be removed.
[ ] Investigate debootstrap

An almost completely freshly provisioned and then boxed VM as of 083fd93 is ~1750MB


Error loading saved standalone mzmine file

Basic information
Windows 10 (64bits) operating mzmine 3.9.
What happened
When trying to open a standalone saved file, it prompts the following error and does not perform the loading:
2024-09-20 11:15:47 FINEST io.github.mzmine.gui.MZmineGUI displayMessage Error - Could not identify the data type needed for import of n files=17 - null
2024-09-20 11:15:47 SEVERE io.github.mzmine.modules.io.import_rawdata_all.AllSpectralDataImportModule runModule Could not identify the data type needed for import of n files=17.  C:\Users\jfmon\AppData\Local\Temp\mzmine_msdatafiles_temp921199070776519868\msdatafiles\inmic06.raw
Nonetheless the original raw data files, that were read in the saved version, remain in the same folder (they are in a OneDrive synchronized folder though), and this error has only recently appeared.
Thanks in advance and best regards.

Hi @jfmonbulusal,
in a standalone project, the raw files are cpoied into the project file to make it portable. If an error occurs during loading/saving (e.g. mzmine closed or no more disk space) then the copying does not work and the project file is corrupted (this is why we recommend to set the temp folder to a place with enough space). In your case with all the files in a constant position on a onedrive, a flexible project is recommended.
What you can try to fix the project is:

open the .mzmine project as a zip folder (e.g. 7zip). Check if all the files are present. if not, copy them there manually.

or:

open the project as a zip, open the "raw_data_import_batch.xml" and edit all the file paths to the raw files to the onedrive folder, delete the "STANDALONE" file

(i recommend the first option)

Thank you doubly since both approaches worked perfectly. I realised the raw files not present in the standalone file once I opened as a zip (even though my SSD disk should have had enough space... probably, I closed the interface before fully saved). Best regards.


Futures 0.3

It was actually a lot easier than I thought to convert from Futures 0.2 to 0.3 (it required very little code changes).
Breaking changes:

Now uses Futures 0.3 rather than 0.2 (which has various breaking changes of its own).
PromiseFuture::spawn_local has been changed to spawn_local
PromiseFuture::print_error_panic has been changed to print_error_panic

New additions:

It's now possible to use async + await! + join! + try_join! syntax :tada: (I added a promise example which demonstrates this).
New unwrap_future function (which internally uses print_error_panic)

The breaking changes and new additions only affect people who are using the experimental_features_which_may_break_on_minor_version_bumps feature, so this does not require a major version bump.

It's easiest to review the changes with whitespace disabled.

Note: Futures 0.3 provides a compatibility shim so that it's possible to use 0.1 Futures with 0.3. However it doesn't provide a shim from 0.2 to 0.3. Perhaps that could be added in?

Okay, I tried various things, and I wasn't able to get the example working with Stable or Beta Rust.
And even without the futures example, Stable and Beta still break (why? they shouldn't be building with the experimental_features_which_may_break_on_minor_version_bumps feature)
I'm pretty stumped here, doesn't the Travis scripts have support for running some examples only on nightly? I tried that, but it still doesn't seem to be working.

And even without the futures example, Stable and Beta still break (why? they shouldn't be building with the experimental_features_which_may_break_on_minor_version_bumps feature)

You mean even if you completely delete that example? Is cargo still generating an error for a dependency which is gated with a disabled feature flag?

@koute Yes:
https://travis-ci.org/koute/stdweb/builds/430253300?utm_source=github_status&utm_medium=notification
Even with the futures example completely deleted, and everything feature-gated behind futures-support and nightly.

Oh, I think I figured it out: the drag example uses experimental_features_which_may_break_on_minor_version_bumps.
And Travis always uses the ci/run_tests.sh file which is defined in master, so I can't fix it to exclude examples/drag and examples/futures from Stable/Beta Rust.

(The tests pass, but it's not ready to merge yet, I still need you to exclude the examples/drag and examples/future from the Stable/Beta Rust on Travis).

Actually, the drag example doesn't seem to be using futures at all, I wonder why it uses the experimental_features_which_may_break_on_minor_version_bumps feature...?

I got everything working, except I had to remove futures from the Cargo.toml, which means that anybody trying to run the example will get this error:
error: error: current package believes it's in a workspace when it's not:
current:   C:\Users\Pauan\Shared Folders\NixOS\stdweb\examples\futures\Cargo.toml
workspace: C:\Users\Pauan\Shared Folders\NixOS\stdweb\examples\Cargo.toml

this may be fixable by adding `futures` to the `workspace.members` array of the manifest located at: C:\Users\Pauan\Shared Folders\NixOS\stdweb\examples\Cargo.toml

I had tried adding futures to NIGHTLY_EXAMPLES, but it didn't seem to help. I'm not sure how the Travis scripts work, so I can't fix that.
This is ready to merge, except for the above mentioned error.

@koute Ping.

Apologies for the delay!
Looks mostly good to me! Thanks! I'll try to do something about the example.

Hmm... it looks like the futures crates are broken on the current nightly (4141a4079 2018-09-25):
error[E0432]: unresolved import `core::pin::PinMut`
 --> /home/kou/.cargo/registry/src/github.com-1ecc6299db9ec823/futures-core-preview-0.3.0-alpha.6/src/future.rs:4:5
  |
4 | use core::pin::PinMut;
  |     ^^^^^^^^^^^^^^^^^ no `PinMut` in `pin`
error[E0432]: unresolved imports `core::future::FutureObj`, `core::future::LocalFutureObj`, `core::future::UnsafeFutureObj`
 --> /home/kou/.cargo/registry/src/github.com-1ecc6299db9ec823/futures-core-preview-0.3.0-alpha.6/src/future.rs:6:32
  |
6 | pub use core::future::{Future, FutureObj, LocalFutureObj, UnsafeFutureObj};
  |                                ^^^^^^^^^  ^^^^^^^^^^^^^^  ^^^^^^^^^^^^^^^ no `UnsafeFutureObj` in `future`
  |                                |          |
  |                                |          no `LocalFutureObj` in `future`
  |                                no `FutureObj` in `future`. Did you mean to use `Future`?

error[E0432]: unresolved import `core::pin::PinMut`
 --> /home/kou/.cargo/registry/src/github.com-1ecc6299db9ec823/futures-core-preview-0.3.0-alpha.6/src/stream/mod.rs:5:5
  |
5 | use core::pin::PinMut;
  |     ^^^^^^^^^^^^^^^^^ no `PinMut` in `pin`

...so I guess we first need to wait for them to be updated?
On the bright side the editions are now stabilized it seems.

@koute Yeah, there's some (extremely recent) breaking changes that need to land first: https://github.com/rust-lang-nursery/futures-rs/issues/1265
If editions are stabilized, then we can probably re-enable the futures example?

If editions are stabilized, then we can probably re-enable the futures example?

Yep. Although only after they're actually on stable. (They're only stable on nightly.) Until then I'll just copy that example over to a separate directory and make it nightly-only.


New dynamo tables

Trainspace DB Table
What user problem are we solving?
DDB table for trainspaces
What solution does this PR provide?
DDB table
Testing Methodology
used ipython + main method
Any other considerations
make sure it works

@farisdurrani @dwu359 thoughts?

@farisdurrani found the problem with the build check. the problem was in test_send_email.py and the attachment file path was invalid due to nextJS upgrade. I adjusted the test attachment array accordingly

@karkir0003 fixed the pydocs, see if it's okay

Tested and worked

@farisdurrani


Issue#423

Related Issuse or bug

This CNN model for facial features and face detection

Fixes: #423
Proposed Changes

Building the whole CNN model from scratch

awesome implementation