Update Go dependencies in preparation for v2.33

@juliusv & @Nexucis , please note that I haven't touched the React dependencies. I assume that's done by those various individual PRs I have seen, and that it is currently not a responsibility of the release shepherd. If that's wrong, or if there is any React dependency update you would like to get in before the release, please let me know.

It looks like there is an issue with the k8s dependencies. @SuperQ rollback them with https://github.com/prometheus/prometheus/commit/a78a3cea0209e11fe9845a11571def1e0065b07f 6 days ago.

The issue seems to only happen with Go 1.16.
Is it OK to require Go 1.17?
Or should I keep the older K8s version?
@SuperQ ?

I see the discussion in #10097. Which would point towards keeping 1.16 happening. However, locking a dependency to an older version causes trouble (as proven here). Perhaps it could be cleanly done with a replace directive?
But since Go 1.18 will be released soon, perhaps it's OK to bump the minimum requirement  to 1.17 now?

I added a commit to bump the minimum Go requirement to 1.17. Please let me know if you prefer keeping 1.16 and the weird state with the k8s library (in which case I'll play with replace to lock it down properly).

I need to fix the go mod tidy inconsistency. My local Go is too old, I guess.

Apparently, there is no clean way of locking a dependency with Go modules.
So it's either weird state and allow Go 1.16 or clean state and require Go 1.17.

I think we wanted to keep compatibility with at least 1.16 until we are satisfied with 1.18. Supporting at least one older compiler minor version (as Go itself does).
We shouldn't need to do these manual bump PRs anymore, as we have dependabot to do these for us.
My recommendation is to rollback and keep the k8s.io at v0.22.4.

We shouldn't need to do these manual bump PRs anymore, as we have dependabot to do these for us.

That's news to me. Could you update https://github.com/prometheus/prometheus/blob/main/RELEASE.md#0-updating-dependencies ?

I guess we still need the manual bump for non-semver updates (those with +incompatible and v0.0.0 in the version specifier).

Now updated the PR to update normally but then manually set the following versions:

k8s.io/api v0.22.4
k8s.io/apimachinery v0.22.4
k8s.io/client-go v0.22.4

Let's see if it passes the tests.

:tada: It passed!

npm outdated shows me the following outdated react dependencies: (@juliusv @Nexucis I always feel a bit lost when it comes to React. Are those below OK or do we need to update before I cot the release?)
Package                              Current    Wanted   Latest  Location                                                                Depended by
@codemirror/autocomplete              0.19.9   0.19.10  0.19.10  node_modules/@codemirror/autocomplete                                   react-app<EMAIL_ADDRESS>@codemirror/autocomplete              0.19.9   0.19.10  0.19.10  node_modules/@codemirror/autocomplete  <EMAIL_ADDRESS>@codemirror/basic-setup               0.19.0    0.19.1   0.19.1  node_modules/@codemirror/basic-setup  <EMAIL_ADDRESS>@codemirror/commands                  0.19.5    0.19.6   0.19.6  node_modules/@codemirror/commands                                       react-app<EMAIL_ADDRESS>@codemirror/highlight                 0.19.6    0.19.7   0.19.7  node_modules/@codemirror/highlight                                      react-app<EMAIL_ADDRESS>@codemirror/highlight                 0.19.6    0.19.7   0.19.7  node_modules/@codemirror/highlight  <EMAIL_ADDRESS>@codemirror/history                   0.19.0    0.19.2   0.19.2  node_modules/@codemirror/history                                        react-app<EMAIL_ADDRESS>@codemirror/search                    0.19.3    0.19.5   0.19.5  node_modules/@codemirror/search                                         react-app<EMAIL_ADDRESS>@codemirror/view                     0.19.27   0.19.39  0.19.39  node_modules/@codemirror/view                                           react-app<EMAIL_ADDRESS>@codemirror/view                     0.19.27   0.19.39  0.19.39  node_modules/@codemirror/view  <EMAIL_ADDRESS>@forevolve/bootstrap-dark              1.1.0     1.1.0    2.1.0  react-app/node_modules/@forevolve/bootstrap-dark                        react-app<EMAIL_ADDRESS>@types/chai                           4.2.22     4.3.0    4.3.0  node_modules/@types/chai  <EMAIL_ADDRESS>@types/enzyme                        3.10.10   3.10.11  3.10.11  node_modules/@types/enzyme                                              react-app<EMAIL_ADDRESS>@types/jest                           27.0.3    27.4.0   27.4.0  node_modules/@types/jest                                                react-app<EMAIL_ADDRESS>@types/jquery                          3.5.9    3.5.11   3.5.11  node_modules/@types/jquery                                              react-app<EMAIL_ADDRESS>@types/node                         16.11.12  16.11.19   17.0.8  node_modules/@types/node                                                react-app<EMAIL_ADDRESS>@types/node                         16.11.12  16.11.19   17.0.8  node_modules/@types/node  <EMAIL_ADDRESS>@types/react                         17.0.36   17.0.38  17.0.38  node_modules/@types/react                                               react-app<EMAIL_ADDRESS>@typescript-eslint/eslint-plugin       5.8.0     5.9.0    5.9.0  module/codemirror-promql/node_modules/@typescript-eslint/eslint-plugin  <EMAIL_ADDRESS>@typescript-eslint/parser              5.8.0     5.9.0    5.9.0  module/codemirror-promql/node_modules/@typescript-eslint/parser  <EMAIL_ADDRESS>@wojtekmaj/enzyme-adapter-react-17     0.6.5     0.6.6    0.6.6  node_modules/@wojtekmaj/enzyme-adapter-react-17                         react-app<EMAIL_ADDRESS>bootstrap                              4.6.1     4.6.1    5.1.3  node_modules/bootstrap                                                  react-app<EMAIL_ADDRESS>eslint                                 8.4.1     8.6.0    8.6.0  module/codemirror-promql/node_modules/eslint<EMAIL_ADDRESS>eslint-config-react-app                6.0.0     6.0.0    7.0.0  react-app/node_modules/eslint-config-react-app                          react-app<EMAIL_ADDRESS>eslint-plugin-import                  2.25.3    2.25.4   2.25.4  node_modules/eslint-plugin-import<EMAIL_ADDRESS>fsevents                             MISSING     2.3.2    2.3.2  -                                                                       react-app<EMAIL_ADDRESS>mocha                                  9.1.3     8.4.0    9.1.3  module/codemirror-promql/node_modules/mocha<EMAIL_ADDRESS>react-resize-detector                  6.7.6     6.7.8    7.0.0  node_modules/react-resize-detector                                      react-app<EMAIL_ADDRESS>react-router-dom                       5.3.0     5.3.0    6.2.1  react-app/node_modules/react-router-dom                                 react-app<EMAIL_ADDRESS>react-scripts                          4.0.3     4.0.3    5.0.0  react-app/node_modules/react-scripts                                    react-app<EMAIL_ADDRESS>reactstrap                            8.10.0    8.10.1    9.0.1  react-app/node_modules/reactstrap                                       react-app<EMAIL_ADDRESS>sanitize-html                          2.6.0     2.6.1    2.6.1  node_modules/sanitize-html                                              react-app<EMAIL_ADDRESS>sass                                  1.44.0    1.44.0   1.46.0  node_modules/sass                                                       react-app<EMAIL_ADDRESS>ts-loader                              7.0.5     7.0.5    9.2.6  node_modules/ts-loader<EMAIL_ADDRESS>typescript                             4.5.2     4.5.4    4.5.4  node_modules/typescript                                                 react-app<EMAIL_ADDRESS>typescript                             4.5.2     4.5.4    4.5.4  node_modules/typescript<EMAIL_ADDRESS>

I tried to update the packages above, but with my not-existing React knowledge, I predictably ended up with error messages I do not understand.

I'll merge this, as it is titled "update Go dependencies", but please any of the React experts, let me know if there are updates needed for React packages before I cut the release.

we should be good for the react update @beorn7 :).


Allow setting value for prompt

Discord's OAuth has a parameter prompt which allows you to control if a user is displayed the consent screen or not.

 prompt controls how the authorization flow handles existing authorizations. If a user has previously authorized your application with the requested scopes and prompt is set to consent, it will request them to reapprove their authorization. If set to none, it will skip the authorization screen and redirect them back to your redirect URI without requesting their authorization.

Allowing devs to set this value would be useful for websites that might have frequent traffic from consistent users.

This is really important!

If it's so important, make a Pull Request, otherwise stop spamming the Comments thanks.

Resolved with #21


Add dtype keyword parameter to MergeFeatureTask

Codecov Report

Merging #394 (4d99e9f) into develop (82074d3) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff            @@
##           develop     #394   +/-   ##
========================================
  Coverage    82.87%   82.87%
========================================
  Files           54       54
  Lines         5174     5174
========================================
  Hits          4288     4288
  Misses         886      886

Impacted Files
Coverage Δ

core/eolearn/core/core_tasks.py
98.10% <100.00%> (ø)

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 82074d3...4d99e9f. Read the comment docs.


docs: add a PR template

This PR is to add this PR template which is used as below:
Description:
Update the PR template to the one that was agreed by the IO tribe and that will be applicable to all related repositories, this one included.
Checklist:

[x] My PR follows the contribution guidelines of this project
[x] My PR is free of third-party dependencies that don't comply with the Allowlist
[] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[] I have added tests that prove my fix is effective or that my feature works
[x] I have checked the PR title to follow the conventional commit specification

Let's follow-up on the contribution guideline link to add.


Adds rules for norminette in settings

Please remove this last commit, atom already bump the version itself

Delete any bumping thing you have done, please !

Thanks 👍
Bumping to 0.3.0 https://github.com/Kerollmops/atom-norminette-linter/commit/2197d976618b64db4eac04d4c270ce426992a978


Themes

As a user, I want to have a variety of different displays to choose from so that I can have different options for the appearance of the application.

[x] 4 different themes for the application.
[x] The themes should be based on Disney movies.
[x] The themes should not have too bright colors.
[x] The themes should not confuse the user.
[x] The other features displayed should be modified according to the different themes.

This user story was missing from Github however we added it now. This is a closed issue.


Request for Correspondence Baseline Code

Hi there,
I hope you're doing well! I was wondering if you could share the code for the baselines for EgoExo4D, as it would be extremely helpful for our experiments in the following work.
Thank you in advance!

They are available here: https://github.com/EGO4D/ego-exo4d-relation
We will improve our documentation page to highlight this more cleary.
Please close the issue if this resolves your question

They are available here: https://github.com/EGO4D/ego-exo4d-relation
We will improve our documentation page to highlight this more cleary.
Please close the issue if this resolves your question

Thanks for your help!


adding cobertura to travis

Codecov Report

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

@@            Coverage Diff            @@
##             master      #27   +/-   ##
=========================================
  Coverage          ?   74.96%
=========================================
  Files             ?       30
  Lines             ?      659
  Branches          ?       73
=========================================
  Hits              ?      494
  Misses            ?      153
  Partials          ?       12

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 686fd54...3f45950. Read the comment docs.


Remove this paper.

Description
Please remove this paper from arXiv as there is some unclear and incorrect estimates.
(Optional:) Please add any files, screenshots, or other information here.
No response
(Required) What is this issue most closely related to? Select one.
Formula
Internal issue ID
e86f0a33-4a88-4220-8e6a-d2466c1b1807
Paper URL
https://arxiv.org/html/2405.08360v1
Browser
Safari/605.1.15
Device Type
Desktop

Thank you for the report. Please contact the document authors directly for content corrections.
This repository only tracks reports on the experimental HTML format.


add await keyword

https://github.com/scaffold-eth/scaffold-eth/blob/master/packages/hardhat/deploy/00_deploy_your_contract.js#L34
I'm a newbie of js, when I want to transferowner, I have to add await for this line.

yes you do, good catch.

I fixed this on master


ci: using ubuntu-20.04 by default

Signed-off-by: Jintao Zhang<EMAIL_ADDRESS>
Type of change:

[x] Bugfix
[ ] New feature provided
[ ] Improve performance
[ ] Backport patches

What this PR does / why we need it:

GitHub Actions is modifying its default system version, which may cause CI to fail since cgroup v2 will be used in the latest system, but not all Kubernetes versions support it.
Setting to ubuntu-20.04 will use cgroup v1, which is more general.
ref: https://github.blog/changelog/2022-11-09-github-actions-ubuntu-latest-workflows-will-use-ubuntu-22-04/
Pre-submission checklist:

[ ] Did you explain what problem does this PR solve? Or what new features have been added?
[ ] Have you added corresponding test cases?
[ ] Have you modified the corresponding document?
[ ] Is this PR backward compatible? If it is not backward compatible, please discuss on the mailing list first

Codecov Report

Merging #1504 (943571a) into master (768a35f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1504   +/-   ##
=======================================
  Coverage   41.47%   41.47%
=======================================
  Files          85       85
  Lines        7394     7394
=======================================
  Hits         3067     3067
  Misses       3974     3974
  Partials      353      353

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Change RecipeRef type to *corev1.LocalObjectReference

Further the reference to the recipe itself can be a LocalObjectReference or a TypedLocalObjectReference which would handle this issue/problem as well #105

Originally posted by @ShyamsundarR in https://github.com/RamenDR/ramen/pull/675#discussion_r1108904187
Making KubeObjectProtection API changes now when fewer are affected and before support for older versions is required.
Additional changes:

Remove duplicated VRG name from log entries
Remove duplicated vrg.spec.kubeObjectProtection not nil checks
Capitalize first letter in some log messages

@hatfieldbrian rebase when you get a chance...


Fix typo & delete unnecessary space

typo: Github -> GitHub
delete unnecessary space: As Markdown syntax, there was a half-width space that did not affect the display, so I deleted it

@changeworld : Thanks for your contribution! The author, @kraigb, has been notified to review your proposed change.

#sign-off


Feature de/serialization. DO NOT MERGE

Feature de/serialization is triggered from the FeatureModel, when the
model is being serialized itself.

Features are de/serialized from/to a subfolder of the project folder.
With the DefaultFeatureModel, this subfolder is 'Features'.

Each feature is saved to an individual file, named with the key of the
feature + '.raw'.

Specialized classes, derived from the interface FeatureSerializer,
are in charge of the de/serialization of individual features. There is
one specific FeatureSerializer for each feature. This is necessary
since we do not know what is the type of values used by a feature.

Concrete FeatureSerializer implementations are provided by the
FeatureComputers. Since these classes (normally written by 3rd party
users) are the ones that know how to generate a feature, they also know
how to de/serialize them.

So the model needs to know what are the FeatureComputers when saving
and loading. Since they are provided by SciJava plugin discovery mechanism
(within the FeautureComputerService), the model needs to access a SciJava
Context. Now each Model instance has its own context (lightweight,
only instantiated with the MamutFeatureComputerService). The context
instance has then been removed from the MainWindow class, and the later
uses the context of the model it manages.

Let's redo this after the changes in the Feature framework are merged.


Refactor FOM to use FC-API

Not a priority at this time. The team decided that instead of spending time refactoring FOM to use a forest-client API, we would just run an ETL job to get the forest client data into the FOM database nightly.


Query API v2 for images modified within a range

Hello! I'm one of the maintainers of Openverse, a search engine for openly licensed media on the web. Phylopic is one of the sources of CC images we ingest regularly, and unfortunately our ingestion broke with the move to the API v2.
In our exploration regarding what it would take to migrate (issue here: https://github.com/WordPress/openverse-catalog/issues/998), it doesn't look like there's a way to query images which were modified within a certain date range. The previous URLs looked something like http://phylopic.org/api/a/image/list/modified/2023-02-12/2023-02-13 (and documentation to the method in question was here http://phylopic.org/api/#method-image-time-range, although that link no longer works after the v2 migration).
I may be misinterpreting what's possible with the new API, and forgive me if this is the wrong place to ask, but - is it possible using v2 to query images by modification date? It seems like that information might be captured on the Image model itself, but I wasn't able to find a field reference for it in the API (I'm also no TypeScript aficionado :sweat_smile:). We'd love to continue updating our index in this fashion if it's possible to do so!

The data has been fixed up for legacy images.
Here is a proposal for API changes to add new filters. Does this look like it would address the issue? https://github.com/keesey/phylopic/blob/feature/date-filters/apps/api-docs/public/v2/openapi.yaml

Thanks for getting back to us @keesey! The spec you outlined in #8 looks like exactly what we would ideally use 😄 Let us know if there's any way we can help or test out the new endpoints!

Cool, I'll let you know when it's done. (Might not be for a little while.)

I lied, I finished it already.
Documentation: http://api-docs.phylopic.org/v2/
Sample URL: https://api.phylopic.org/images?filter_modified_after=2023-01-01T00:00:00.000Z


Fix multi-collection format in fakes

Multi-collection format was being treated as a CSV which is incorrect. Add first-class support for parameter collection formats instead of relying on type inspection and optional field heuristics.

One thing I wonder if you have take into account: if query param is empty, empty judge is needed before split.

Opened https://github.com/Azure/autorest.go/pull/1035 to fix splitting on an empty value.

After this fix, we still met the following problem with armcdn:


Potential bug regarding: "All your test should be compiling for Birdie to work!"

I'm getting this with birdie snapshot testing:
❌ The test file "./test/qcheck/gen_int_test.gleam"
 is not compiling.
All your test should be compiling for Birdie to work!

The project compiles fine with gleam check and gleam test, just not with the gleam run -m birdie.
Some relevant versions:
gleam 1.1.0
name = "birdie", version = "1.1.4"
name = "glance", version = "0.11.0"

I have also tried the gleam update , then gleam clean, then tried the tests again, but it didn't fix the problem.

I found a small reproducible example. If this code is in a test file:
pub fn x__test() {
  let x = case Error(-1) {
    // If you change this case to Error(2) it works.
    Error(-2) -> Error("This branch breaks stuff")
    _ -> Ok(0)
  }

  should.equal(x, Ok(0))
}

For some reason the negative int in the Error in a case statement breaks things.  If you switch it to Error(2) it works okay. Also, you can have Error(-2) in any other location in the case thing, but just not negative numbers on the thing that is being matched.

Hello! Thanks for reporting this!! It's a bug in glance, I'll try and contribute a fix, anyways I'll make sure birdie can gracefully handle any of these kind of problems in the next release


Add option to completely disable help row

This adds an option to always hide the help row, independently from the mode or window height.

Hey! Thank you for contributing 😄
Just the one comment but looks good otherwise. Would you also be able to update the docs please?

Sure thing, will do!

All done. Tell me if there's anything else you need!


HTTP Strict Transport Security (HSTS) is not implemented

Expected Behavior
Uchiwa supports HTTP Strict Transport Security (HSTS) as a mechanism for protecting against protocol downgrade attacks and cookie hijacking.
Current Behavior
Uchiwa does not implement HSTS policy mechanism.
Context
Lack of HSTS headers over HTTPS connections leaves Uchiwa instances vulnerable to protocol downgrade attacks and cookie hijacking.
See https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Strict-Transport-Security for reference.
Your Environment

Uchiwa version used: 1.3.1
Sensu version used:
Operating System and version (e.g. Ubuntu 14.04):

Per Simon, should be easy to implement.


Lock one of n tabs

Hi, can someone help me understand - how to lock one tab by condition if I have for example three tabs? Other tabs must be unlocked...

Can it be helpful https://github.com/skv-headless/react-native-scrollable-tab-view/issues/629?

Thanks for answering, I just solved this problem by this approach:
<ScrollableTabView           initialPage={1}           renderTabBar={() => <View />}           ref={(tabView) => { this.tabView = tabView }}           contentProps={{bounces: false}}           onChangeTab={this.handleChangeTab}         >           <FirstTab />           <SecondTab />           {!condition && <ThirdTab />} </ScrollableTabView>

Great :+1:


docs: property instead of method

The documentation of "Checking if an entry is published:" says entry.is_published(), but it's a property: entry.is_published

Merged


Update integrationTests.md

fix grammar issues

Hello @ParsaMehdipour ... These changes wouldn't be consistent with our style manual, so we won't be able to accept them. However, thanks for the suggestions.

Got it
Thanks


In-cluster Registry framework updates

Follow up of #7813.
Fixed all comments, rename local registry to in-cluster registry. Removed killall calls, and updated the CLI, to keep running until we hit ctrl-c. I called this helper, as we could add multiple things that should help devs in this, not just the registry.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers: nmittler, sebastienvas
If they are not already assigned, you can assign the PR to them by writing /assign @nmittler @sebastienvas in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/OWNERS
tests/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@sebastienvas: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
4c1acf2bbc7bd915b6e821f6cfe3c05b1d2c3f4d
link
/test e2e-bookInfo-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@nmittler PTAL

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: nmittler
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: sebastienvas
If they are not already assigned, you can assign the PR to them by writing /assign @sebastienvas in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/OWNERS [nmittler]
tests/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: hklai, nmittler, sebastienvas
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/OWNERS [hklai,nmittler]
tests/OWNERS [hklai,nmittler,sebastienvas]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

New changes are detected. LGTM label has been removed.

@sebastienvas: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth-mcp.sh
ae30fa8054259be274425dd1375f532af8976970
link
/test e2e-mixer-no_auth-mcp

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: hklai, nmittler, sebastienvas
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/OWNERS [hklai,nmittler]
tests/OWNERS [hklai,nmittler,sebastienvas]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@sebastienvas: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth-mcp.sh
27fac4ccd0004c41985d598bdde7d7fad107e5a7
link
/test e2e-mixer-no_auth-mcp

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Street Network Matrix node to use harmonized distance settings

https://github.com/spatial-data-lab/knime-geospatial-extension/issues/147

revised, see the pull request


Add test for rasterfile centroids.

When creating many-to-many entries in a test factory, somehow the post commit hooks are processed before the factory saves  the model.  The factory resaves the model (internally in factory_boy after processing), but, since this hasn't refreshed the model from the post commit hook changes, obliterates those changes.  Currently, I've worked around this issue in the RasterEntryFactory by manually running the post commit hook.  We'll have to dig deeper into factory_boy to figure out how to avoid needing to do this.
I think we should merge this as it stands.

@banesullivan This now changes some of the TaskEventMixin code.  It makes the test factory less hacky, though, and it will work in a synchronous environment (whereas, before, it would not).
I think this is ready for review and merging.


synchronize ecs-logging spec

What
ECS logging specs automatic sync
Why

@v1v Did https://github.com/elastic/ecs-logging/blob/master/.ci/Jenkinsfile#L175-L186 not work to fill in the "Why" here?

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Build Cause: Pull request #74 opened

Start Time: 2021-03-24T16:14:03.408+0000

Duration: 7 min 3 sec

Commit: 648de5403774f11964703167ecd431695b8db983

Trends :test_tube:

:/ env variable was changed from singular to plural and the reference was not updated :/
https://github.com/elastic/ecs-logging/blob/aaf458815b3991afae7d3a0872d960e51007db7e/.ci/Jenkinsfile#L180


[23.11] nwjs: 0.54.1 -> 0.83.0

Description of changes
This introduced important security fixes to nwjs, which includes a chromium engine.
Backport of: #262424 and #276397
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 276962 run on x86_64-linux 1

10 packages built:

betaflight-configurator
emuflight-configurator
gridtracker
inav-configurator
nwjs
nwjs-sdk
onlykey
pinegrow
pinegrow6
popcorntime


ui: channels should be grouped/collapsed based on the node key/id

Today in the network, it's possible to open up many channels to a single node. As a result, if a loop is initiated that ends up traversing that node, we don't have direct control over which channel is used, since the end node is able to select which channel it wants to use. Therefore any operations which select one channel (out of many) for a single node, should also grow to select all the other channels for that node.
There're a few ways to do this, and this may require another brief UI iteration. One idea that pops into mind is to collapse (group) all channels belonging to a target node in the UI. This grouped channel would appears as though it's a single channel, as we would sum up all balance information for that groups of channels.

This is done in Terminal Web UI. Gonna close this issue given that.


feat: Add support for rounded corners in diamond

Fixes #4266
PS: This PR might needs a few more changes to support edge selection for diamond

I assume this was fixed in the 2nd commit. Let's ship!

Yep, Thanks for guiding me to the correct function 😄


method similar to rails' content_for

http://api.rubyonrails.org/classes/ActionView/Helpers/CaptureHelper.html
This lets one do something similar to
# post/show.ecr
<% content_for(:title, "About") %>

It's also useful for generating Meta tags per post:
# layouts/main.ecr
<meta
  property="og:title"
  content="<%= content_for?(:title) ? yield(:title) : "Default"
/>

How tough would this be to implement in Kemalyst?

not sure how hard it would be.  I will look into it.

One way to handle this:
# post/show.ecr
<% context.params["title"] = "About" %>

and then in the layout:
<title><%= context.params["title"]? || "Default" %></title>


@conda-forge-admin, please add user @cmutel

Additional comment:
No response

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added user @cmutel in conda-forge/presamples-feedstock#3.


Added clearTimeout for the setCalendarMonthGridHeight timeout

I know that this might seem like a trivial PR, but in our application we are getting " Can't call setState (or forceUpdate) on an unmounted component. This is a no-op, but it indicates a memory leak in your application. To fix, cancel all subscriptions and asynchronous tasks in the componentWillUnmount method." warnings because we aren't clearing the timeout.

Coverage decreased (-0.04%) to 85.033% when pulling e1a038ec74e48c68bda1bc49a0207815bd5c2f23 on sheldonj:master into 94262344f9f70ce362553379c99e6c9947daa2e9 on airbnb:master.

Hi @majapw is there any extra work you need to get this merged?

Looks great! Mostly my difficulty in giving this repo the time it needs right now. :)


.option (no)?relax and label differences

GNU assembler emits a pair of R_RISCV_ADD32 and R_RISCV_SUB32 for 32-bit label differences. This assembler construct is useful for metadata and the desired behavior should be documented somewhere.
.option push
.option norelax
.section .text,"ax",@progbits
lo:
  ret
hi:
.option pop

.data
.long hi - lo

Currently both GNU assembler and LLVM integrated assembler emit R_RISCV_ADD32 R_RISCV_SUB32 regardless of the relax/norelax state. I think it will be useful to have a way to emit a constant in the norelax case to make relocatable object files smaller. This will help clang -g:
.section        .debug_info,"",@progbits
...
.word   .Lfunc_end0-.Lfunc_begin0

If 32-bit label differences always lead to R_RISCV_ADD32 R_RISCV_SUB32, it will be cumbersome to get rid of the relocations in -mno-relax mode. An object file producer can do some hard work emitting a constant but this isn't feasible for an assembly file producer.
The documentation may specify:

the symbols must be STB_LOCAL. Currently the assemblers don't report an error even if this output may not make sense as linked output.
the behavior when the labels are not in a SHF_EXECINSTR section.

@compnerd

I have a much better understanding of label differences now and think this is probably out of the scope of an assembler manual. https://maskray.me/blog/2021-03-14-the-dark-side-of-riscv-linker-relaxation#label-differences-related-to-text-section-symbols
When my https://reviews.llvm.org/D155357 ([RISCV] Allow delayed decision for ADD/SUB relocations) lands, .long hi - lo in the case will generate no relocation.
GNU assembler conservatively generates relocations because lo and hi are in a text section (TC_FORCE_RELOCATION_SUB_SAME in gas/write.c:fix_segment returns true).


Added selected items from Chromium What's New 95

This PR contains experiments for consideration & (mostly offline) discussion:

Granular linking to Chromium page (per-section/per-feature).
Consolidating several links.
Clarifying what is and is not from Chromium (sections/features).
Freshening the page structure/layout/sequencing

Consolidating the Download info.

Rendered page for review:
https://review.docs.microsoft.com/en-us/microsoft-edge/devtools-guide-chromium/whats-new/2021/10/devtools?branch=pr-en-us-1533#announcements-from-the-chromium-project
Existing live page

Docs Build status updates of commit 94b90d2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/devtools-guide-chromium/whats-new/2021/10/devtools.md
:white_check_mark:Succeeded
View

microsoft-edge/devtools-guide-chromium/whats-new/includes/edge-whats-new-note.md
:white_check_mark:Succeeded
View

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

Docs Build status updates of commit c2503a0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/devtools-guide-chromium/whats-new/2021/10/devtools.md
:white_check_mark:Succeeded
View

microsoft-edge/devtools-guide-chromium/whats-new/includes/edge-whats-new-note.md
:white_check_mark:Succeeded
View

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

Docs Build status updates of commit a7ee83f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/devtools-guide-chromium/whats-new/2021/10/devtools.md
:white_check_mark:Succeeded
View

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

Docs Build status updates of commit ae8121a:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: RuningBuildFailed] Some unexpected errors happened when running build, please open a ticket in https://aka.ms/SiteHelp and include the error report for our team to troubleshoot

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit cc1ef7a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/devtools-guide-chromium/whats-new/2021/10/devtools.md
:white_check_mark:Succeeded
View

microsoft-edge/devtools-guide-chromium/whats-new/media/2021/10/command-menu-filenames-bold-icons.png
:white_check_mark:Succeeded
View

microsoft-edge/devtools-guide-chromium/whats-new/media/2021/10/flattened-list-of-properties.png
:white_check_mark:Succeeded
View

microsoft-edge/devtools-guide-chromium/whats-new/media/2021/10/improved-display-properties.png
:white_check_mark:Succeeded
View

microsoft-edge/devtools-guide-chromium/whats-new/media/2021/10/snippets-sorted-alphbetically.png
:white_check_mark:Succeeded
View

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

Docs Build status updates of commit aa2b4ee:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/devtools-guide-chromium/whats-new/2021/10/devtools.md
:white_check_mark:Succeeded
View

microsoft-edge/devtools-guide-chromium/whats-new/media/2021/10/command-menu-filenames-bold-icons.png
:white_check_mark:Succeeded
View

microsoft-edge/devtools-guide-chromium/whats-new/media/2021/10/flattened-list-of-properties.png
:white_check_mark:Succeeded
View

microsoft-edge/devtools-guide-chromium/whats-new/media/2021/10/improved-display-properties.png
:white_check_mark:Succeeded
View

microsoft-edge/devtools-guide-chromium/whats-new/media/2021/10/snippets-sorted-alphbetically.png
:white_check_mark:Succeeded
View

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

Docs Build status updates of commit f0d08aa:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/devtools-guide-chromium/whats-new/2021/10/devtools.md
:white_check_mark:Succeeded
View

microsoft-edge/devtools-guide-chromium/whats-new/media/2021/10/command-menu-filenames-bold-icons.png
:white_check_mark:Succeeded
View

microsoft-edge/devtools-guide-chromium/whats-new/media/2021/10/flattened-list-of-properties.png
:white_check_mark:Succeeded
View

microsoft-edge/devtools-guide-chromium/whats-new/media/2021/10/improved-display-properties.png
:white_check_mark:Succeeded
View

microsoft-edge/devtools-guide-chromium/whats-new/media/2021/10/snippets-sorted-alphbetically.png
:white_check_mark:Succeeded
View

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


Server port is still in use after killing the application.

Hard to figure out what's going on in the diff since it's moving so quickly, could you spell it out for me please? Steps to reproduce, etc.

I ran the application, and then killed it. The port is still going to be in use after doing this, so you won't be able to run the application again.

@maxknivets Please include the actual code (what's inside server.py?) and commands you ran. Basically make it as easy as possible for other people to reproduce what you did, without trying to guess.
Here's me failing to reproduce:
$ cd examples/python
$ python -m gohttp

And in another terminal:

Alright, I see.

The server.py file:

https://gist.github.com/maxknivets/6c18935505f45121c278b211774e02c6

Python version:

My current python version is 3.6.7, but the identical issue also happens on 2.7.15rc1.

Commands I ran:

$ cd examples/python
$ python3 server.py

OS:

My OS is Linux (Ubuntu 18.04).
Hope this helps!


Add HashMap::retain method

Hey 👋 I was trying out replacing dash map in my application with papaya to see if it performs better and noticed a couple missing methods from std::collections::HashMap, namely for this issue the retain method which drains entries that fail a condition.
https://doc.rust-lang.org/stable/std/collections/hash_map/struct.HashMap.html#method.retain

Thanks for the issue, released in 0.1.4.
Note that if your use of retain is latency-sensitive you may benefit from ResizeMode::Blocking. If this poses a problem please let me know — there are alternative implementations of retain that can be considered.


Features extraction

Hi wizyoung,
There's a way to extract features from the last convolutional layer for each bounding box?
Thanks

@tommyz94 what's the meaning of extract feature? Get weights of layer?

I mean the features extracted by the last convolutional layer. For example in VGG16 after all convolutional phase I can obtain a vector of dimension 4096 that describes the image  at an high-level.

@tommyz94 Yes it is possible, you can hook out weights of layer and save it, from you kept

@tommyz94 Yes it is possible, you can hook out weights of layer and save it, from you checkpoint

I mean the features extracted by the last convolutional layer. For example in VGG16 after all convolutional phase I can obtain a vector of dimension 4096 that describes the image at an high-level.

Do u have it?


distutils.core.setup was never called

Attempting to run pip-compile on a project results in RuntimeError: 'distutils.core.setup()' was never called -- perhaps 'setup.py' is not a Distutils setup script?
Environment Versions

macOS
Python version: 3.7.0
pip version:  10.0.1
pip-tools version: 2.0.2

Steps to replicate
draft $ cat > setup.py
__name__ == '__main__' and __import__('setuptools').setup()
draft $ python -m piptools compile

Expected result
pip-tools should produce the pinned requirements.txt for the project, such as it does for a simpler setup.py.
draft $ cat > setup.py
__import__('setuptools').setup()
draft $ python -m piptools compile
#
# This file is autogenerated by pip-compile
# To update, run:
#
#    pip-compile --output-file requirements.txt setup.py
#

Actual result
Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/piptools/__main__.py", line 16, in <module>
    cli()
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/click/core.py", line 722, in __call__
    return self.main(*args, **kwargs)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/click/core.py", line 697, in main
    rv = self.invoke(ctx)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/click/core.py", line 1066, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/click/core.py", line 895, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/click/core.py", line 535, in invoke
    return callback(*args, **kwargs)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/piptools/scripts/compile.py", line 166, in cli
    dist = run_setup(src_file)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/distutils/core.py", line 227, in run_setup
    script_name)
RuntimeError: 'distutils.core.setup()' was never called -- perhaps 'setup.py' is not a Distutils setup script?

The issue is probably that other installers will invoke setup.py as a script with __name__ set to "__main__", but apparently that's not the case when pip-tools invokes it.
Many projects guard on __name__. I do it as a matter of course in my projects. I do that so that tools like flake8 and doctest runners can import and inspect the module without executing it.
Obviously, you can't depend on everyone guarding on __name__, but you should be able to set it when loading the module so the module loads in the same context as it would if someone were to run it.

At the point where setup.py is executed, __name__ == 'builtins'.

Change setup.py to __name__ in ['__main__', 'builtins'] and __import__('setuptools').setup() works around the issue.

To support Python 2 invocation of pip-compile, you need __name in ['__main__', 'builtins', '__builtin__'].

Hello @jaraco,
I'll close this since you've found a workaround. Please let us know if you have any other issues. Thanks for the issue!


Investigate Material dependencies on Android 13; transitions are a bit odd

I noticed that if you launch the app and navigate to MyMints the first time, it comes diagonally from top left but if you repeat this then there is a different animation.


PTI + style CLIP

Thank you for your great work @danielroich
this repo is the implementation of StyleCLIP's global direction methods text-based manipulation using Pivot tuning inversion
https://github.com/soushirou/StyleCLIP-pytorch

More than just a thumbs up, I want to say thank you @soushirou !


Does the buffer source include only current one or other buffers as well?

FAQ

[X] I have checked the FAQ and it didn't resolve my problem.

Issues

[X] I have checked existing issues and there are no open or closed issues with the same problem.

Neovim Version
NVIM v0.6.1 Build type: Release LuaJIT 2.0.5 Compiled by builduser  Features: +acl +iconv +tui See ":help feature-compile"     system vimrc file: "$VIM/sysinit.vim"   fall-back for $VIM: "/usr/share/nvim"  Run :checkhealth for more info
Minimal reproducible config
if has('vim_starting')
set encoding=utf-8
endif
scriptencoding utf-8
if &compatible
set nocompatible
endif
let s:plug_dir = expand('/tmp/plugged/vim-plug')
if !filereadable(s:plug_dir .. '/plug.vim')
execute printf('!curl -fLo %s/autoload/plug.vim --create-dirs https://raw.githubusercontent.com/junegunn/vim-plug/master/plug.vim', s:plug_dir)
end
execute 'set runtimepath+=' . s:plug_dir
call plug#begin(s:plug_dir)
Plug 'hrsh7th/nvim-cmp'
Plug 'hrsh7th/cmp-buffer'
Plug 'hrsh7th/cmp-nvim-lsp'
Plug 'hrsh7th/vim-vsnip'
Plug 'neovim/nvim-lspconfig'
call plug#end()
PlugInstall | quit
" Setup global configuration. More on configuration below.
lua << EOF
local cmp = require "cmp"
cmp.setup {
snippet = {
expand = function(args)
vim.fn"vsnip#anonymous"
end,
},
mapping = {
[''] = cmp.mapping.confirm({ select = true })
},
sources = {
{ name = "nvim_lsp" },
{ name = "buffer" },
},
}
EOF
lua << EOF
local capabilities = require('cmp_nvim_lsp').update_capabilities(vim.lsp.protocol.make_client_capabilities())
require'lspconfig'.cssls.setup {
capabilities = capabilities,
}
EOF
Description
Not necessarily a bug.
I'm not sure about the intended behavior regarding the buffer source:
It gives auto-completion suggestions for the words in the same buffer, but not for those in other opened buffers.
Is it the intended behavior for the buffer source, or there is a separate one for completions from other buffers?
Steps to reproduce

Copy the provided Minimal reproducible config
Run nvim init.vim, wait for plugin installation
Check for completion in same and other buffers

Expected behavior
Gives completion suggestions from other buffers.
Actual behavior
Only completions from same buffer.
Additional context
No response

This isn't the write repository. buffer source is separate to cmp and your question is answered in its readme: https://github.com/hrsh7th/cmp-buffer#get_bufnrs-type-fun-number

Thank you. @gegoune is right.

This isn't the right repository. buffer source is separate to cmp and your question is answered in its readme: https://github.com/hrsh7th/cmp-buffer#get_bufnrs-type-fun-number

Thanks both! Got it solved.


can not debug with oracle

can not debug with oracle ,is it not supported?

DBeaver doesn't support debug of server-side code. Yet.


header data?

Is it possible to get header data with your package for all requests? I'm trying to make sure I check headers to get
X-Ratelimit-Limit and X-Ratelimit-Usage listed here: https://strava.github.io/api/#rate-limiting so I know how to adjust on my end.

Hi,
access to header data is now available in version 1.2.0.
Just use the getResponseHeaders() or getResponseHeader($header) methods.
Marcus


Add support for weaviate multi tenancy

This PR adds the option to specify a tenant in the weaviate client configuration if the weaviate class uses the new multi-tenancy option.

Thanks for this! One question


[Remote Segment Store] Delete merged away segment files from remote store in periodic/async flow

Describe the solution you'd like
Lucene segments are immutable files. Once segments are created, they can be deleted due to various flows, one of the flows could be segment merge. These merged away segments are deleted from local filesystem. To keep remote segment store in-sync with the local filesystem, we need to delete these segments from remote segment store as well.
The delete flow need not be in sync and can be done in periodic manner. But to keep the data integrity, we need to mark the files as deleted.

I'd imagine we could reuse these incremental updates as snapshots vs needing a separate snapshot/restore flow.  Should we provide some way of marking specific incremental uploads so they are not deleted?


Make tag,key_ring,key optional for release workflow.

Signed-off-by: Kenny Leung<EMAIL_ADDRESS>
Summary

Make tag,key_ring,key optional for release workflow.
This is make it possible to cut releases completely automatically.
The tag will now increment the patch value by one if the release tag is
not specified.
For example:

1.5.9 => 1.5.10

Keyring and Key value from GCP project are now hardcoded.
Will remove the input key_ring and key when it dependent workflows are
updated to no longer specify them.
Ticket Link

Fixes
towards fixing https://github.com/sigstore/public-good-instance/issues/50
Release Note

@cpanato please review.

for sigstore/sigstore we don't have a cloudbuild, we just tag the repository and create a GitHub release. i think I did not pay attention much here #325 :(
maybe for sigstore/sigtore we can have a workflow just to create the GH and push the changelog

Agree. sigstore/sigstore will need it's own release workflow.
This workflow was meant for reuse in cosign, rekor and fulcio.

for sigstore/sigstore we don't have a cloudbuild, we just tag the repository and create a GitHub release. i think I did not pay attention much here #325 :(
maybe for sigstore/sigtore we can have a workflow just to create the GH and push the changelog

Agree. sigstore/sigstore will need it's own release workflow. This workflow was meant for reuse in cosign, rekor and fulcio.

ahh true, sorry Monday.... 🤦


Add interactive language and platform buttons to Getting Started page

Currently, the Getting Started page has examples for a few languages with the pack CLI. These are all visible on the page when it loads. A user can scroll through the examples to find the one they want.
The figma shows a UI with two sets of buttons that allow a user to select which language and platform they want to see an example for.

When the page loads, a default language (probably Java) and platform (pack CLI) should be selected. Only the Java + pack CLI getting started guide should be visible on the page. When a user selects a different language and/or platform, a new guide should replace the previous one.
Not in scope for this issue:

Writing net-new Getting Started content for more languages and/or platforms

In scope for this issue:

Implementing two sets of buttons (one for languages, one for platforms), even though there is currently only one platform option (pack CLI)

Showing a single platform button, prior to us creating new Getting Started content for different platforms, it feels like it may be confusing to show that single button.
Is it possible to gate the display of these buttons based on either the number of platforms implemented, or a feature flag to hide all of the buttons.


Failing e2e tests - scorecard-bash on ossf-tests/scorecard-action

Matrix: {
"publish_results": true,
"results_format": "sarif"
}
Repo: https://github.com/$GITHUB_REPOSITORY/tree/$GITHUB_REF_NAME
Run: https://github.com/$GITHUB_REPOSITORY/actions/runs/$GITHUB_RUN_ID
Workflow name: $GITHUB_WORKFLOW
Workflow file: https://github.com/$GITHUB_REPOSITORY/tree/main/.github/workflows/scorecards-bash.yml
Trigger: $GITHUB_EVENT_NAME
Branch: $GITHUB_REF_NAME

Test issue.


FEAT: Adding terminal feature

Terminal is a command for the cloud-platform cli to enable setup of a users shell to include the right environment variables for awscli, kubectl and terraform to make sure that workspaces and cluster are both the same.
The script does require a user to have aws config and credentials place in a particular directory. the kubecfg uses the awscli to set context as well as update the config under .kube, if missing it will create the config, dependant on aws credentials

linked ticket
https://app.zenhub.com/workspaces/cloud-platform-team-5ccb0b8a81f66118c983c189/issues/ministryofjustice/cloud-platform/3612

Is this ready for review, or are you still working on it?

I'm going to close this, but feel free to re-open it when it comes time to revisit it.


change error display length on all line instead of first char

get the line size of the document to display error on full line
fix https://github.com/swordev/phpstan-vscode/issues/6

Thanks for your merge request!
What happen if you have hundreds or thousands of errors? The async function vscode.workspace.openTextDocument will be called a lot of times.
Possible bottleneck?


PDF character randering problem

Attach (recommended) or Link to PDF file here:
Configuration:

Web browser and its version:"{"http_parser":"2.7.0","node":"7.4.0","v8":"5.6.326.50","uv":"1.10.1","zlib":"1.2.8","ares":"1.10.1-DEV","modules":"53","openssl":"1.0.2j","electron":"1.6.2","chrome":"56.0.2924.87","atom-shell":"1.6.2"}"

Operating system and its version: win10

PDF.js version: 2.5.207-es5

Is a browser extension:

Steps to reproduce the problem:
This is happening randomly. Sometimes, after opening viewer again and again, It is displayed correctly.
What is the expected behavior? (add screenshot)

What went wrong? (add screenshot)

Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):

Attach (recommended) or Link to PDF file here:

This part of the ISSUE_TEMPLATE is mandatory, in order for this (or any other) issue to be actionable/valid.

Web browser and its version:"{"http_parser":"2.7.0","node":"7.4.0","v8":"5.6.326.50","uv":"1.10.1","zlib":"1.2.8","ares":"1.10.1-DEV","modules":"53","openssl":"1.0.2j","electron":"1.6.2","chrome":"56.0.2924.87","atom-shell":"1.6.2"}"

Huh, that doesn't really look like a browser but rather Node.js!?

Finally, please see https://github.com/mozilla/pdf.js/blob/master/.github/CONTRIBUTING.md (emphasis mine):

If you are developing a custom solution, first check the examples at https://github.com/mozilla/pdf.js#learning and search existing issues. If this does not help, please prepare a short well-documented example that demonstrates the problem and make it accessible online on your website, JS Bin, GitHub, etc. before opening a new issue or contacting us in the Matrix room -- keep in mind that just code snippets won't help us troubleshoot the problem.


Search functionality is broken

When looking at the console.log, I can see that you're attempting to use GitHub's Code Search API, but a 401 Unauthorized is received.
Indeed, since April 2023, all GitHub Code Search API requests must be authenticated: https://github.blog/changelog/2023-03-10-changes-to-the-code-search-api/

Thanks! Had to move to using a Vercel function to call the GitHub API with a token


[FEEDBACK] Vulkan Game Engine Tutorial 02, Capital L and small l : Brendan Galea

Hi,
I just wanted to give some feedback on Tutorial 02.
I thought I should share it with everyone, just in case they made the same mistake I did.
I made a small mistake with capitalization and the compiler error message will be very confusing.
In 19:15 of your video, you want to create an instance of LvePipeline.
However, if you accidentally type in "LvePipeline" instead of "lvePipeline", you will create a member function instead of creating an instance.
Severity	Code	Description	Project	File	Line	Suppression State
Error	C2059	syntax error: 'string'	vlkGameEngine	C:\Users\user\source\repos\vlkGameEngine\vlkGameEngine\first_app.h	20

Thanks.

Once it's set as a member function, it won't be able to change back to an instance.

Thanks for sharing! Please leave future comments like this on the youtube videos itself! Peopline are more likely to see them there (including me)


chore(net): Add proptest roundtrip to rlp  types

Adds arbitrary/proptest::arbitrary traits to types that derive compact and rlp traits.
Adds auto tests to both. If a type has both traits, then it tests both inside the same proptest! closure.
Two types needed manual arbitrary impl: Chain and Capability since they depend on outside types.
One type needed manual arbitrary impl: TransactionSigned because the hash field is recalculated on decode, which wont match the random hash field generated by arbitrary.
reth-primitives:  change ethers-core::{H512,  U128}  to own implementation for H512 and ruint::U128
change a couple more bytes::Bytes to reth-primitives::Bytes
better UX for main_codec and derive_arbitray. Left some comments with examples below.

@joshieDo can merge pending conflict

So added an arg for proptest cases, and limited some to fewer cases - since it was really exploding from all the vec^N. There are 2 types that even with just one case it's taking a lot of time.
Also made the default number of cases to 256 (same as proptest). Open to increase it, since the problematic cases should already be adjusted.

Codecov Report

Merging #829 (7b13329) into main (75fab00) will increase coverage by 0.16%.
The diff coverage is 83.83%.

@@            Coverage Diff             @@
##             main     #829      +/-   ##
==========================================
+ Coverage   73.55%   73.71%   +0.16%
==========================================
  Files         280      282       +2
  Lines       29898    30059     +161
==========================================
+ Hits        21991    22159     +168
+ Misses       7907     7900       -7

Flag
Coverage Δ

unit-tests
73.71% <83.83%> (+0.16%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

crates/common/rlp/src/decode.rs
91.09% <ø> (+0.55%)
:arrow_up:

crates/common/rlp/src/encode.rs
97.29% <ø> (+0.24%)
:arrow_up:

crates/net/network/src/message.rs
40.77% <ø> (ø)

crates/primitives/src/bloom.rs
98.18% <0.00%> (+4.43%)
:arrow_up:

crates/primitives/src/lib.rs
100.00% <ø> (ø)

crates/primitives/src/peer.rs
63.15% <ø> (ø)

crates/primitives/src/chain.rs
90.81% <50.00%> (-2.60%)
:arrow_down:

crates/primitives/src/hex_bytes.rs
93.39% <50.00%> (ø)

crates/primitives/src/transaction/mod.rs
88.57% <59.45%> (-1.27%)
:arrow_down:

crates/net/eth-wire/src/capability.rs
62.13% <64.28%> (+0.33%)
:arrow_up:

... and 39 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


fix dump with document reference

In the previous version. If there contains any DocumentReference fields. call to instance.dump() will throw an error Converting object to an encodable object failed: Instance of 'MockDocumentReference'
This pull request is fixing it.

Thanks for contributing a PR. I'll review it tomorrow. In the meantime,  would you mind formatting the code?
https://flutter.dev/docs/development/tools/formatting

We've reformatted the code

Thanks for proposing the PR and writing exhaustive tests. Can you try modifying myEncode instead? The tests should still work fine after that.

Feel free to reopen if you plan to fix it.


No mobile support?

 Are there settings or compatibility issues to keep in mind on mobile? We don't want to force full screen like most mobile web video solutions.

@tommybananas Last time I checked mobile support was spotty until vendors stabilize webgl. Thatm may not be the case today. You can set webgl: false as an option to the Axis constructor which will force rendering to occur on canvas directly, but there are no guarantees.

@tommybananas We're addressing this in version 2.0. See #102


Toc tos

issues: #46 #59

initial terms of service page
fix some darkmode colours
move mobile menu to app.vue level
create table of contents with intersection observer for docs page
add previous/next buttons for docs page
refactor layouts to keep header, aside and table of contents fixed

/gcbrun

Temporary Url for review: https://apigw-dev--pr-60-c7vynk3q.web.app

/gcbrun

Temporary Url for review: https://apigw-dev--pr-60-c7vynk3q.web.app


Fenecon  PV Konfiguration

Describe the bug
Hallo zusammen,
ich hab ein Problem mit dem Fenecon  und der Konfiguration.

Steps to reproduce
evcc configure

Advanced mode (Ask more details, requires technical know-how)
Configure a single device (has to be added manually to a configuration file!)
PV inverter (or corresponding meter)
FENECON
Please provide the following settings:

IP address or hostname [? for help] <IP_ADDRESS>
Password [? for help] ****
Configuration details
In which mode should this process guide you through the configuration? Standard mode (As simple and quick as possible)

- Setup meters (house installation)

Choose one of the following integrated PV systems, or 'My device is not in this list' if you have none of them

 Choose an integrated PV system: FENECON

Please provide the following settings:

 IP address or hostname <IP_ADDRESS>

Testing the grid configuration of FENECON ...

Testing the pv configuration of FENECON ...
   Error: strconv.ParseFloat: parsing "<nil>": invalid syntax

 Testing of the pv configuration of FENECON failed. Do you want to add pv anyway? (y/N)
 Testing of the pv configuration of FENECON failed. Do you want to add pv anyway? No

Testing the battery configuration of FENECON ...

FENECON was successfully added.

 Do you want to add a PV inverter (or corresponding meter)? No

 Do you want to add another battery inverter (or corresponding meter)? Yes

- Configuration battery inverter (or corresponding meter)

 Choose a battery inverter (or corresponding meter): FENECON

Please provide the following settings:

 IP address or hostname <IP_ADDRESS>

Testing the battery configuration of FENECON ...

FENECON FENECON was successfully added.

 Do you want to add another battery inverter (or corresponding meter)? (y/N)

Log details
- Configuration PV inverter (or corresponding meter)

 Choose a PV inverter (or corresponding meter): FENECON

Please provide the following settings:

 IP address or hostname <IP_ADDRESS>
 Password [? for help] ****

Testing the pv configuration of FENECON ...
   Error: strconv.ParseFloat: parsing "<nil>": invalid syntax

What type of operating system are you running?
 Negative values for sell-to-grid; positive for buy-from-grid","unit":"W","value":746}
Power: 746W
pv2
[http  ] TRACE 2024/01/04 12:21:13 GET http://<IP_ADDRESS>/rest/channel/_sum/ProductionActivePower
[http  ] TRACE 2024/01/04 12:21:13 {"address":"_sum/ProductionActivePower","type":"INTEGER","accessMode":"RO","text":"Total production; always positive","unit":"W","value":null}
Power: strconv.ParseFloat: parsing "": invalid syntax
battery3
[http  ] TRACE 2024/01/04 12:21:13 GET http://<IP_ADDRESS>/rest/channel/_sum/EssDischargePower
[http  ] TRACE 2024/01/04 12:21:13 {"address":"_sum/EssDischargePower","type":"INTEGER","accessMode":"RO","text":"Actual AC-side battery discharge power of Energy Storage System. Negative values for charge; positive for discharge","unit":"W","value":-19}
[http  ] TRACE 2024/01/04 12:21:13 GET http://<IP_ADDRESS>/rest/channel/_sum/EssSoc
[http  ] TRACE 2024/01/04 12:21:13 {"address":"_sum/EssSoc","type":"INTEGER","accessMode":"RO","text":"Range 0..100","unit":"%","value":0}
Power: -19W
Soc:   0%
battery4
[http  ] TRACE 2024/01/04 12:21:13 GET http://<IP_ADDRESS>/rest/channel/_sum/EssDischargePower
[http  ] TRACE 2024/01/04 12:21:13 {"address":"_sum/EssDischargePower","type":"INTEGER","accessMode":"RO","text":"Actual AC-side battery discharge power of Energy Storage System.  Negative values for sell-to-grid; positive for buy-from-grid","unit":"W","value":746}
Power: 746W
pv2
[http  ] TRACE 2024/01/04 12:21:13 GET http://<IP_ADDRESS>/rest/channel/_sum/ProductionActivePower
[http  ] TRACE 2024/01/04 12:21:13 {"address":"_sum/ProductionActivePower","type":"INTEGER","accessMode":"RO","text":"Total production; always positive","unit":"W","value":null}
Power: strconv.ParseFloat: parsing "": invalid syntax
battery3
[http  ] TRACE 2024/01/04 12:21:13 GET http://<IP_ADDRESS>/rest/channel/_sum/EssDischargePower
[http  ] TRACE 2024/01/04 12:21:13 {"address":"_sum/EssDischargePower","type":"INTEGER","accessMode":"RO","text":"Actual AC-side battery discharge power of Energy Storage System. Negative values for charge; positive for discharge","unit":"W","value":-19}
[http  ] TRACE 2024/01/04 12:21:13 GET http://<IP_ADDRESS>/rest/channel/_sum/EssSoc
[http  ] TRACE 2024/01/04 12:21:13 {"address":"_sum/EssSoc","type":"INTEGER","accessMode":"RO","text":"Range 0..100","unit":"%","value":0}
Power: -19W
Soc:   0%
battery4
[http  ] TRACE 2024/01/04 12:21:13 GET http://<IP_ADDRESS>/rest/channel/_sum/EssDischargePower
[http  ] TRACE 2024/01/04 12:21:13 {"address":"_sum/EssDischargePower","type":"INTEGER","accessMode":"RO","text":"Actual AC-side battery discharge power of Energy Storage System.  Negative values for sell-to-grid; positive for buy-from-grid","unit":"W","value":746}
Power: 746W
pv2
[http  ] TRACE 2024/01/04 12:21:13 GET http://<IP_ADDRESS>/rest/channel/_sum/ProductionActivePower
[http  ] TRACE 2024/01/04 12:21:13 {"address":"_sum/ProductionActivePower","type":"INTEGER","accessMode":"RO","text":"Total production; always positive","unit":"W","value":null}
Power: strconv.ParseFloat: parsing "": invalid syntax
battery3
[http  ] TRACE 2024/01/04 12:21:13 GET http://<IP_ADDRESS>/rest/channel/_sum/EssDischargePower
[http  ] TRACE 2024/01/04 12:21:13 {"address":"_sum/EssDischargePower","type":"INTEGER","accessMode":"RO","text":"Actual AC-side battery discharge power of Energy Storage System. Negative values for charge; positive for discharge","unit":"W","value":-19}
[http  ] TRACE 2024/01/04 12:21:13 GET http://<IP_ADDRESS>/rest/channel/_sum/EssSoc
[http  ] TRACE 2024/01/04 12:21:13 {"address":"_sum/EssSoc","type":"INTEGER","accessMode":"RO","text":"Range 0..100","unit":"%","value":0}
Power: -19W
Soc:   0%
battery4
[http  ] TRACE 2024/01/04 12:21:13 GET http://<IP_ADDRESS>/rest/channel/_sum/EssDischargePower
[http  ] TRACE 2024/01/04 12:21:13 {"address":"_sum/EssDischargePower","type":"INTEGER","accessMode":"RO","text":"Actual AC-side battery discharge power of Energy Storage System.

Ja, mit sowas wie z. B. (.value // 0)

Konfiguration läuft jetzt :-)
Vielen Dank euch


Adding extraction of videoID from youtube videos url which contain '/live/'

Solved #5416
Added an extra conditional statement for extracting 'videoID' from youtube video urls containing '/live/'

Sure, I am not familiar with adding tests but I'll look into this.

@ericof I have added the tests could you please go through them once as this my first time writing jest tests so if there is something wrong do let me know.
Also I have moved the videoDetails extraction code to a separate function and there was also a issue to video's thumbnails not begin displayed so I also changed the logic to bit to get the correct thumbnail URLs.

The first screenshot is from https://demo.plone.org/ where the videos in which the url contains "list" or ".be" dont have their thumbnails displayed, the second one is from my local server. There was a problem with the extraction of the videoID from the urls with "list" and ".be".


.jsbeautifyrc file support

Hey!
As I understood, your plugin takes JSBeautify setting only from VSCode settings, but VSCodeBeautify plugin allows you to use .jsbeautifyrc file. Is it possible to add support for this file in your plugin as well?
PS. I can't add html.format.inline option to VSCode settings now (seems like they didn't add it yet), but I can use inline property in .jsbeautifyrc file.

@f3oall is the inline property valid on the HTML portion of JS Beautify?
Either way I've been wanting to support .jsbeautifyrc files anways so reguardless of if the inline will work on not how you expect, this will be in my next release.

Well, seems like I misunderstood the configuration options of JS Beautify. I opened similar issue in VSCode Beautify repo, and basing on the answer seems like html.format.inline won't be ever implemented. But inline is still valid option in jsbeautify.rc. So it would be cool if you include its support in your next release. Thanks!

It's close to being done actually


cant use kickstart.nvim

Cant use kickstart.nvim its gives me this error
Error detected while processing /home/sepehr/.config/nvim/init.lua:
E5113: Error while calling lua chunk: /home/sepehr/.config/nvim/init.lua:165: attempt to index field 'keymap' (a nil value)
stack traceback:
/home/sepehr/.config/nvim/init.lua:165: in main chunk
Press ENTER or type command to continue
what do i do
The machine I'm using
Hardware model Sony Corporation VPCEA2KGX
The os i'm using Pop!_OS 22.04 LTS
let me know if you want anymore info.

What version of Neovim are you using? And are you not accidentally using vim rather than (Neovim) nvim?
To see what version of neovim you can do nvim --version

+1 Facing same error
Fresh install of neovim using sudo apt install neovim
`neovim --version
NVIM v0.6.1
Build type: Release
LuaJIT 2.1.0-beta3
Compiled by<EMAIL_ADDRESS>Features: +acl +iconv +tui
See ":help feature-compile"
system vimrc file: "$VIM/sysinit.vim"
fall-back for $VIM: "/usr/share/nvim"
Run :checkhealth for more info
`

You are running v0.6.1; latest stable version as of writing this is v0.10.2
Kickstart.nvim targets only the latest 'stable' and latest 'nightly' of Neovim.
Update accordingly and it will no longer be an issue.

Hello! Same issue here. Running v.0.10.2 on windows via chocolatey.
Error detected while processing C:\Users\<name>\AppData\Local\nvim\init.lua:
E5113: Error while calling lua chunk: C:\Users\<name>\AppData\Local\nvim\init.lua:230: module 'lazy' not found:
        no field package.preload['lazy']
        no file '.\lazy.lua'
        no file 'C:\tools\neovim\nvim-win64\bin\lua\lazy.lua'
        no file 'C:\tools\neovim\nvim-win64\bin\lua\lazy\init.lua'
        no file '.\lazy.dll'
        no file 'C:\tools\neovim\nvim-win64\bin\lazy.dll'
        no file 'C:\tools\neovim\nvim-win64\bin\loadall.dll'
stack traceback:
        [C]: in function 'require'
        C:\Users\<name>\AppData\Local\nvim\init.lua:230: in main chunk
Press ENTER or type command to continue

@skalixur yeah, this is a different error. If still exists, could you please create a new issue?

Hi! I`m using nvim 0.10.2
E5113: Error while calling lua chunk: vim/loader.lua:0: attempt to call upvalue '' (a nil value)
stack traceback:
        vim/loader.lua: in function 'cache_file'
        vim/loader.lua: in function 'read'
        vim/loader.lua: in function 'loadfile'
        vim/loader.lua: in function <vim/loader.lua:0>

The OP has gone radio silent and pretty clearly seems to be running an older version.
Closing. Please feel free to reopen if you can prove you're running a more current compatible version.


Laravel Linter

This pull request includes changes and recommendations for crafting your application "The Laravel Way". Feel free to commit any additional changes to the shift-81403 branch.
Before merging, you need to:

Checkout the shift-81403 branch
Review all pull request comments for additional changes
Thoroughly test your application (no tests?, no CI?)

Much of the lint detected may be automatically fixed by running the Laravel Fixer or tasks within the Shift Workbench.

:x: The following configuration files differ from the Laravel defaults. If you are modifying configuration values directly, consider using an environment variable instead to keep these files maintainable. You should compare your configuration files against the latest version and merge any updates.

[ ] config/app.php
[ ] config/auth.php
[ ] config/database.php
[ ] config/filesystems.php
[ ] config/services.php

:warning: The following files reference Laravel facades through the global namespace. For example, you're referencing \DB or importing use DB. Instead import Illuminate\Support\Facades\DB and reference DB.
While global references are allowed through aliases, you should import the facade explicitly. This can improve clarity not only for developers, but static analysis used by your IDE.

[ ] app/Http/Controllers/PrinterController.php
[ ] app/Http/Livewire/Customers/PayDue.php
[ ] app/Http/Livewire/Language/Create.php
[ ] app/Http/Livewire/Language/Edit.php
[ ] app/Http/Livewire/Language/EditTranslation.php
[ ] app/Http/Livewire/Language/Index.php
[ ] app/Http/Livewire/Settings/Index.php

:information_source: Starting with Laravel 8, the model property within factories may be removed when your models are stored within a conventional location such as app/Models. Unless your factory is for a model outside of this location, you may remove this property.

:warning: Shift detected Eloquent calls in the following views. You should keep views from interacting directly with your models by passing the necessary data from the controller.

[ ] resources/views/admin/purchases/edit.blade.php
[ ] resources/views/admin/purchasesreturn/create.blade.php
[ ] resources/views/admin/purchasesreturn/edit.blade.php
[ ] resources/views/admin/quotation/create.blade.php
[ ] resources/views/admin/quotation/edit.blade.php
[ ] resources/views/admin/quotation/quotation-sales/create.blade.php
[ ] resources/views/admin/reports/purchases-return/index.blade.php
[ ] resources/views/admin/reports/purchases/index.blade.php
[ ] resources/views/admin/reports/sales-return/index.blade.php
[ ] resources/views/admin/reports/sales/index.blade.php
[ ] resources/views/admin/sale/edit.blade.php
[ ] resources/views/admin/salesreturn/create.blade.php
[ ] resources/views/admin/salesreturn/edit.blade.php
[ ] resources/views/components/language-dropdown.blade.php
[ ] resources/views/livewire/pos/index.blade.php

:warning: The following controllers contain actions outside of the 7 resource actions (index, create, store, show, edit, update, destroy). For more details, review the docs or watch Cruddy by Design to see if you may rework these into resource controllers.

[ ] app/Http/Controllers/BarcodeController.php
[ ] app/Http/Controllers/CustomersController.php
[ ] app/Http/Controllers/ExportController.php
[ ] app/Http/Controllers/HomeController.php
[ ] app/Http/Controllers/ReportsController.php
[ ] app/Http/Controllers/SettingController.php
[ ] app/Http/Controllers/SuppliersController.php

:warning: Shift found inline validation in the following controllers. Unless you require this level of control, you should use a Form Request to encapsulate this validation logic and keep your controller clean. You may automate this conversion with the Laravel Fixer.

[ ] app/Http/Controllers/AdjustmentController.php
[ ] app/Http/Controllers/Auth/ConfirmablePasswordController.php
[ ] app/Http/Controllers/Auth/NewPasswordController.php
[ ] app/Http/Controllers/Auth/PasswordResetLinkController.php
[ ] app/Http/Controllers/Auth/RegisteredUserController.php
[ ] app/Http/Controllers/PurchasePaymentsController.php
[ ] app/Http/Controllers/PurchaseReturnPaymentsController.php
[ ] app/Http/Controllers/SaleReturnPaymentsController.php

:warning: To improve performance by caching your configuration, Laravel recommends you only use the env() helper within configuration files and use config() everywhere else. Shift found potential uses of env() in:

[ ] resources/views/admin/settings/index.blade.php

:information_source: Shift detected string actions being used within your routes. Starting with Laravel 8, the preferred convention for registering routes is to use static class references.

:information_source: Shift detected your database migrations are using custom classes. Starting with Laravel 9.0, database migrations now use anonymous classes. This avoids class name collisions and streamlines migrations.

:warning: Laravel uses PSR-4 autoloading for the app/ folder. While you may configure your application to load these files within composer.json, all files under the app folder should be classes with a PSR-4 compliant namespace.
The following PHP files are missing a namespace:

[ ] app/Helpers/helpers.php


Define ABSL_LEGACY_THREAD_ANNOTATIONS

Define ABSL_LEGACY_THREAD_ANNOTATIONS in CMake pipeline so cartographer continues to build with absl version 20230125.1 and greater: https://github.com/abseil/abseil-cpp/releases.

I've tested this by running:
make cartoclean && make buf && make buildcarto && make installcarto

It compiles just fine on both.
I then ran cartogropher in mapping / updating mode on both mac & linux with those compiled grpc servers & they appeared to work on both platforms. I updated the map for about 5 min & rendered the map in the RDK control UI.

Yes that is correct.

To clarify, with this change, we will no longer need to pin a version of abseil, is that correct?

Yes that is correct.


refactor(args): Abstract level vs version args

Besides separating concerns, this is a step towards a potential solution
to #222, see discussion in #246.  Now we are preserving whether the user
used an absolute or relative version.

Sorry for late. The patch looks good to me


Mobile Chrome triggers animations only after scroll stop

On desktop and mobile safari all works nice but not on mobile chrome. I think chrome does render the scrolling different?
When I scroll down, the content blocks should appear with a fade effect. But instead, it appears only if the scrolling has completely stopped. In mobile Safari the effect shows up, when the block is entering the viewport. This is correct.
Is there any workaround for showing the animations on mobile chrome like on mobile safari?

@micrdy Have you been able to find a fix for this? I am having the same issue with no luck on a fix.
Thanks

No,  unfortunately not. If have disabled the animation on mobile.


BC Requirements for revocation

I've produced these requirements several times in different forums and recently emailed them to someone. I thought they might be a useful starting point for revocation requirements in this repo.  Worst case, we close the PR without merging :-).
Some of the requirements could be supplemented by the results of the survey done by @coffeethecup about how big things should be in different contexts -- e.g. how much space a credential should take up (including revocation data) on a mobile wallet app.
Hope this generates some discussion.

Hi @swcurran I included privacy requirements in the assessment xls. Thanks for mentioning it again.

comments captured here https://github.com/decentralized-identity/revocation/pull/22 closing this one


Packet truncation issue

Hi Troy!
Your -s parameter seems to not respect Standard libpcap truncation because my network probe does not accept it.
How do you truncate the packets ?
Thx in advance!

If you’re transmitting truncated packets, that will confuse anything that expects well formed packets on the wire. Is it an option in your environment to raise the MTU so you don’t need to truncate?

On Mar 15, 2018, at 3:53 AM, nirlya<EMAIL_ADDRESS>wrote:
Hi Troy!
Your -s parameter seems to not respect Standard libpcap truncation because my network probe does not accept it.
How do you truncate the packets ?
Thx in advance!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/troydhanson/fluxcap/issues/2, or mute the thread https://github.com/notifications/unsubscribe-auth/AA0S6IksuCCTh0oBmoMEua9JgB1Vm5erks5teXU7gaJpZM4SrEyS.

Sadly not. In fact, the fluxcap GRE tunnel does not fragment packets before send it and truncate it seems my only solution.

Did you disable offloading on the original capture interface? By default the NIC hardware coalesces packets but this causes havoc when your intention is to retransmit them.
You’re right about the GRE tunnel lacking fragmentation support. That could be a useful feature.

On Mar 15, 2018, at 10:13 PM, nirlya<EMAIL_ADDRESS>wrote:
Sadly not. In fact, the fluxcap GRE tunnel does not fragment packets before send it and truncate it seems my only solution.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

 The GRE tunnel adds something like 24 bytes to each packet. So, I think it captures a large packet, and then GRE encapsulation makes it larger yet. Then it exceeds the MTU on the LAN, when you try to transmit it.
I think the choices are (1) to increase the LAN MTU (to the capture MTU plus 100 or so), but you said this not an option; (2) truncate using -s 1476 (or so), but then the downstream tools need to expect some packets to be truncated after the GRE wrapper is removed; or (3) to add fragmentation support to the fluxcap GRE encapsulator.
I can think about the third option, but I can make no promises.

Reply to this email directly, view it on GitHub https://github.com/troydhanson/fluxcap/issues/2#issuecomment-373423224, or mute the thread https://github.com/notifications/unsubscribe-auth/AA0S6N6Rn4DvcR77z5BMk6VV4rJchEWMks5teoy_gaJpZM4SrEyS.

I added support for IP fragmentation in the GRE encapsulation modes. There is no longer a need to use the -s truncation option. Instead it fragments automatically when needed. Testing looked good so far. Thank you for suggesting the feature.

Also note the feature is on the branch gre-frag-support in commit 2f34e7c910e13b850019d42dd8e2bc2e305bff65. Please do not use master, as it's a mess right now.

Awesome, it works well with my tests !
Also I disabled Offload on the gretap receiver interface.

Great, glad to hear it!


Logs handling API - add javadoc

I was creating my own logger, and I would have appreciated to find some Javadoc on the getters.
The text I wrote is based on what can be found in the README.adoc at the root of the repository.

Not sure "rending" is the term we want to use 😆 The API moved from "rendering" to "converting", I think it would be better.

I have changed "render" into "convert" in the javadoc and in the readme (only the LogHandler section)

Thanks! 🎉


tagging schema for craft=atelier

OSM Tag(s)
craft=atelier
How would you like this tag to see supported?
As a Preset
Label
atelier
Aliases
art studio
Terms
No response
Link to OSM Wiki page
https://wiki.openstreetmap.org/wiki/Tag:craft%3Datelier
Status of the Tag
In Use
Usage of the tag
517
Replaces other Tag?
No response
Regional Tag?
It's mostly used in Europe
Further Information
No response

Thanks for the suggestion.
Does anyone see any reason not to add this?
It does have a relative low usage of ~500+ but looks well defined.
The wiki page is from 2019 so it had time to collect feedback.
And the wiki page discussions don't sound like blockers.


refactor(autoware_point_types): prefix namespace with autoware::point_types

Description
prefix namespace with autoware
Related links
Parent Issue:

https://github.com/autowarefoundation/autoware/issues/5077
https://github.com/autowarefoundation/autoware/issues/4569

How was this PR tested?
Notes for reviewers
None.
Interface changes
None.

Effects on system behavior
None.

Once the maintainer of autoware_point_types approves, I'll also approve the localization module.

@drwnz @mojomex  :pray:

@xmfcx thanks!


Add missing license headers.

we should try to fix the builds as @jimmidyson mentioned in https://github.com/redhat-ipaas/ipaas-rest/pull/167#issuecomment-285849829 so that we can start to make proper merge builds again.

[merge]


Fix IndexOfSetBit for the case of a negative pow2 input parameter.

Before the fix, the while loop would not execute at all because pow2 was negative, and thus the return value was 0.
The fix is to remove the test for pow2 > 0 from the while condition, but test for the special case pow2 == 0 before going into the loop, otherwise an infinite loop would result.
The call sites never call this routine with a 0 parameter, but the routine should be robust to a 0 input parameter nonetheless.

Thanks @PeterSolMS for the fix.  It looks like the issue here is that there was an assumption in the previous implementation that the input value would not be negative.  However, the failing test cases pass negative values, for example 0xFFFFFFFFC0000001, which with the previous implementation returned 0, but with the proposed implementation returns 1.
I want to make sure that the negative values are expected and that this isn't a matter of missing a cast to a smaller data type.  I don't expect that with your change it will impact things either way, but I want to make sure that there isn't another bug lurking here.
As far as the failing tests, these are in-fact from your change, but they are expected.  You are changing the output results of processing a trace and these TraceEvent tests are making sure that nothing changes.  To fix this, you'll just need to update the baselines for these tests and make sure that they are correct.  If you run the tests locally in VS, they will show you where the baseline and actual txt file results live and you can diff them and/or copy the actual txt file over top of the baseline and commit it.

the usage of this method means it will always take a negative number because if it's none 0 it means by definition the MSB is set. so this method never worked 'cause it always returned 0. returning 1 is correct in this case.
as far as whether to handle the 0 input, it's ok to have it return -1, or perhaps a bit better - you could hoist the not_specified value for various enums and have it return that instead, that way we don't need to filter out 0 first before calling this method. it requires a bit more code change so it's up to you.

Ok, great.  Thanks @Maoni0.  I'm ok with either option you propose.

I removed the checking from the callsites and integrated it into IndexOfSetBit, and I updated the baseline files.


DuckDB: Open parameter shouldn't be required for data connector

Describe the bug
The open param is currently needed for all DuckDB functions.
datasets:
  - from: duckdb:database.schema.table
    name: my_dataset
    params:
      open: path/to/duckdb_file.duckdb

  - from: duckdb:read_csv('test.csv', header = false)
    name: from_function
    params:
      open: path/to/duckdb_function_file.duckdb
```yaml

This shouldn't be required, the runtime can create a duckdb db on demand if the param isn't specified.

This has already been fixed.  Resolving.


Typescript compiler allows me to use non-existent getter

Code
class Foo {
  constructor() {
    const x = this.bar;
  }
  set bar(value: boolean) {

  }
  //get bar() {
    //throw new Error('error');
  //}
}
const y = new Foo();

Expected behavior:
The code above doesn't compile
Actual behavior:
The code above compiles
This might be a duplicate bug, sorry I couldn't find anything that resembles this.

Please search before logging new issues. #11596

Thanks! Sorry about that.
For more context:
We were trying to refactor @Input() type to @Input() set type, forgot to change all this.type instances. As a result, error was hard to debug.. since this.type is undefined. i assumed  that would be a compile time error..


Rename color variables to make more sense

To be done after https://github.com/pypa/warehouse/pull/3433 is merged.

Rename primary-color to highlight-color
Rename brand-color to primary-color
Rename brand-color-soft to primary-color-light

Good First Issue: This issue is good for first time contributors. If you've already contributed to Warehouse, please work on another issue without this label instead. If there is not a corresponding pull request for this issue, it is up for grabs. For directions for getting set up, see our Getting Started Guide. If you are working on this issue and have questions, please feel free to ask them here, #pypa-dev on Freenode, or the pypa-dev mailing list.

I'd like to work on this one.

@nitinprakash96 Please go ahead! Let us know how it's going.


React-Native animations

Use this library a lot for web projects (thank you for making!) — curious about the claims for react-native use — I'm assuming that animations are not run on the UI Thread (and instead run on the JS Thread).
If that is the case, the library isn't applicable to most mobile animations needs. Just a suggestion, since it was a little confusing when I used it on a react-native project.

Best explanation of easing function
https://www.selectionjs.com/animation


ttrt add support for initialization function + ident check

New tensor initialization functions, for certain kinds of tests it's useful to initialize with a range of values vs random.  In the future we could add more like uniform distribution.
Some tests, like for data movement, can be implemented to perform a trivial golden check where the input exactly matches the output. Add an identity flag which does this check.

@tapspatel, feel free to commandeer this PR if you want to do it another way.  I had made these changes locally for a test I'm bringing up, since I had already done the work I figured I'd just post a PR with what I had.

@tapspatel, are you good with any of this change?

changes look good!


Automate the Alluxio integration test

In https://github.com/NVIDIA/spark-rapids/issues/6121, we'll create the notebooks on Databricks to test Alluxio manually.
We should create the CI job to run the test automatically.
And check if it's possible to run Alluxio test internally,

CI job is created and connected with nightly build on internal CI server


[google_sign_in] iOS user get stuck on plugin auth screen if OAuth user type is internal

We have inner application and OAuth user type mode set to internal on Google Cloud Platform. It means that account that does't belong to organization workspace can't sing in. But there is a problem if an iOS user singed in with such account and it was the only one, then he will get stuck on the plugin authorization screen: he can't add a new account and always sing in to previous account when click to sing in.
Steps to Reproduce

git clone https://github.com/umbrellait-timur-khismatullin/google_sign_in_issue.git
run on iOS Simulator
tap to sign in
sing in with Google account
try to sign in again

Firebase configuration file added to the project. Auth mode set to internal.

Logs
flutter run --verbose
https://pastebin.com/jkC80eUn
flutter analyze
Analyzing google_sign_in_issue...
No issues found! (ran in 2.6s)

flutter doctor -v
[✓] Flutter (Channel stable, 3.0.2, on macOS 11.6 20G165 darwin-x64, locale ru-RU)
    • Flutter version 3.0.2 at /Users/timurkhismatullin/development/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision cd41fdd495 (2 weeks ago), 2022-06-08 09:52:13 -0700
    • Engine revision f15f824b57
    • Dart version 2.17.3
    • DevTools version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/timurkhismatullin/Library/Android/sdk
    • Platform android-33, build-tools 30.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7590822)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2020.3)
    • Android Studio at /Applications/Android Studio AF.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)

[✓] Android Studio (version 2021.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7590822)

[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 61.2.4
    • Dart plugin version 212.5080.8

[✓] VS Code (version 1.67.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.40.0

[✓] Connected device (4 available)
    • UmbrellaIT-iPhoneXR.white (mobile) • 00008020-000D04E00169002E            • ios            • iOS 14.8 18H17
    • iPhone 13 Pro Max (mobile)         • 11217F52-3E85-4AF7-A09A-2AC9052AFE92 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-15-2 (simulator)
    • macOS (desktop)                    • macos                                • darwin-x64     • macOS 11.6 20G165 darwin-x64
    • Chrome (web)                       • chrome                               • web-javascript • Google Chrome 103.0.5060.53

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

Video from original app
https://user-images.githubusercontent.com/84508364/175493781-a99cbd97-e0db-4aa1-b1bb-2151ea4f63ac.MP4

The solution of this issue is in native library and may look like this:
https://github.com/google/GoogleSignIn-iOS/compare/main...umbrellait-timur-khismatullin:fix-internal-mode-reauth-issue

Hi @umbrellait-timur-khismatullin
It seems only happens with Gsuite account. I can not reproduce this issue due to no google workspace account now.
Labeling this issue for further investigation from the team.

The solution of this issue is in native library and may look like this: google/GoogleSignIn-iOS@main...umbrellait-timur-khismatullin:fix-internal-mode-reauth-issue

I see  https://stackoverflow.com/a/58413506 which I guess you found based on your patch.  It would have to be fixed in GoogleSignIn-iOS, and they would have more context for this.  I see https://github.com/google/GoogleSignIn-iOS/issues/112 which is probably similar.