Potential security risk by exposing API key

Currently if you have an object of the LilyWeight class the apiKey class field is accessable by any code that has a reference to the object.
This is a security regression caused by #4
I recommend fixing this by using private class fields.
I can make a PR for this later but right now I am busy with Final exams

Hello, sorry for the late reply. I was busy with finals as well.
I'm not really sure how this would be possibly exploited. If you already have arbitrary code execution, why not just read the env/config value instead of looking at the LilyWeight class?

I'm not really sure how this would be possibly exploited.

I agree that is not a big risk but it seems like it would still be a good idea to make the key private


Google Translate causes bindings to break

Hi,
I have been investigating a problem that some of my users have reported. After weeks of investigation I have tracked down the problem to Google translate.
I know this is technically a Google translate issue, but as Angular is a Google product and I'm not sure how to report a bug with Google translate, I thought I'd start here.
The problem occurs when someone tries to translate a page using Google translate. It seems that the extension replaces HTML with translated text which of course breaks the 2 way binding.
After some investigation I see I can add a class "notranslate" to each of the elements that have a binding, but this seems quite a messy solution. The obvious solution would be for Google translate to ignore all elements with a class "ng-binding", but I'm sure Google can find a more elegant solution.
Darren

It is difficult to imagine a good solution to let them work together...
Even if google translate waits for the text output by angular before translate, instead of translating angular expressions in {{}}, after that angular would compare the translated version which would no more be in synch with the model and erases the translation...
We write a big multi-language application, and the translation are encoded in our database.
Angular is marvelous for that:the user can change his/her language setting at any time using the application and the angular bindings makes all the display adapted, without any google translate.

Thanks for your comment. Our application is written in English but some users translate it into their native language.
Let me be more specific. An example would be

Bonus page in {{ data.bonus }}

Where data.bonus is an integer.
Google translate translates the text and translates the number and loses the binding. If I add "notranslate" as a class to the  element, then the text is translated and the binding remains intact which is the desired outcome.
The hack that I've put in place is on $viewContentLoaded - $(".ng-binding").addClass("no translate") and I'm hoping that this adds the class before translate does its stuff.

I'm not sure if this is solvable completely, either by GT or Angular. I don't see how GT is designed to translate dynamic websites. For me, it simply makes a static copy of a site where all translatable strings are translated.
For example, if you have a string like You have selected 1 image, GT would translate it. But if the user selects > 1, you'd have You have selected 2 images, and the translation would not update, wouldn't it?
That said, I believe your case where number bindings get destroyed can be fixed by using:
Bonus page in

@Narretz I think you mean Bonus page in

@wesleycho You are right, thanks!

There is not a lot that can be done if the page is sliced and diced by an external tool. Closing this, as there is a workaround, and there is not a lot that can be done within angular

For those that come from Angular >= 2, the solution propose by @Narretz also work , in high versions of Angular
Original:
foo {{ name }}

Working example:
foo {{ name }}

For those that come from Angular >= 2, the solution proposed by @Narretz also work in high versions of Angular
Original:
foo {{ name }}

Working example:
foo {{ name }}

KevinGuancheDarias This works wonderfully!

For those that come from Angular >= 2, the solution proposed by @Narretz also work in high versions of Angular
Original:
foo {{ name }}

Working example:
foo {{ name }}

KevinGuancheDarias This works wonderfully!


scatterplot implementation

Does anyone have a working example of a scatterplot chart like this example:
http://c3js.org/samples/chart_scatter.html

Check this one

Thanks @random-one this is helpful


Remove workaround for SIGINT

It's no longer needed with the latest Dart SDK roll thanks to the awesome work of @zanderso.


status and filled amount of order not match

Filled amount is 15.71% but status is Filled

data from server
amount: "1000000000000000000"
baseToken: "0x4d7ea2ce949216d6b120f3aa10164173615a2b6c"
createdAt: "2020-02-18T07:48:45.073Z"
exchangeAddress: "0x0d3ab14bbad3d99f4203bd7a11acb94882050e7e"
filledAmount: "157180000000000000"
hash: "0xcce7d963de4a9f1b051d4ad2949f40ac21b9b48f49a7e28b3667f27553e848a8"
key: ""
nonce: "0"
orderID: "0"
pairName: "BTC/USDT"
pricepoint: "980000000000"
quoteToken: "0xd9bb01454c85247b2ef35bb5be57384cc275a8cf"
side: "SELL"
status: "FILLED"
type: "MO"
updatedAt: "2020-02-18T07:48:45.073Z"
userAddress: "0xb4f931a6680dbdf339bc5f91d45348de1e21e66e"


Add support for enum DB type

I'm using native db enum type in my app and was having this error:
ROM::SQL::UnknownDBTypeError: Cannot find corresponding type for enum
This solved my problem. Please suggest if there should be any changes.

Thanks :)
@flash-gordon wdyt about this?

Hey @vladra
DB-specific types should go to corresponding inferrers, see this one as an example.
PG's enums require more work. First of all, String :enum creates a column with type text and name enum, not a column with enum type. See sequel's sources for how this should be fixed.  Also, I think we should add possible enum values to type's meta, something like this type.meta(values: enum_values).


feat(policy): 1659 implement transaction support in policy service

fix #1659

This PR will be left as a draft for the foreseeable future as an example of a simple transactions implementation.  At the moment there is not enough time to dedicate to the effort, as there are higher priorities such as the config service.  We will need to go through every service to be sure that this approach is feasible.


Windows: node-gyp sets wrong include path

My addon is on drive d:\ and my home dir is on drive c:.
This makes node-gyp set the include path as follows:
/I"..\..\..\..\..\C:Users\Bert Belder\.node-gyp\0.7\src" /I"..\..\..\..\..\C:Users\Bert Belder\.node-gyp\0.7\deps\uv\include" /I"..\..\..\..\..\C:Users\Bert Belder\.node-gyp\0.7\deps\v8\include"

You should use path.resolve instead of path.join btw. But that alone does not fix the issue.

Thanks for the report Bert. These are exactly the kind of kinks I wanna iron out. However, I don't know if there's anything I can do about it. It seems to be a bug in either gyp, gyp_addon, or the addon.gypi file.
I tried running gyp_addon and msbuild manually, not using node-gyp at all and the same thing happens :(
Thanks for the tip on path.resolve() though, I'll be sure to switch those over. Where do we go from here though?

Should be fixed in node-gyp v0.2.2. Closing.


advanced docs custom linear solver

similar to LinearSolve

You just pass a LinearSolve.jl algorithm now, done.


iPA or Xcode Project Not Working!

The iPA or Xcode Project isn't working for me. Plz help. For the iPA, it says there is no Info.plist

Read the description. Compiling isn't fully working.

How bout the iPA. Isn’t it already compiled?

Yes. but it's missing a lot of things

Should be fixed!


Deprecated call async_forward_entry_setup

Problem
Logger: homeassistant.helpers.frame
Source: helpers/frame.py:184
First occurred: 01:27:18 (11 occurrences)
Last logged: 01:27:49
Detected that custom integration 'tplink_deco' calls async_forward_entry_setup for integration, tplink_deco with title: http://<IP_ADDRESS> and entry_id: 01J2CMRZZ59RQGH8WADMCRJ00N, which is deprecated and will stop working in Home Assistant 2025.6, await async_forward_entry_setups instead at custom_components/tplink_deco/init.py, line 166: config_entry.async_create_task(, please create a bug report at https://github.com/amosyuen/ha-tplink-deco/issues
TP-Link Deco Version
v3.6.3
Router Model
n/a
Debugging Information
Logging should suffice.

I get the same message but also get a second message saying it will stop working in 2025.1:
2024-07-12 21:10:39.504 WARNING (MainThread) [homeassistant.helpers.frame] Detected that custom integration 'tplink_deco' calls async_forward_entry_setup for integration, tplink_deco with title: <IP_ADDRESS> and entry_id: 760741b0e8f7b7503e54fb7c71a67a81, which is deprecated and will stop working in Home Assistant 2025.6, await async_forward_entry_setups instead at custom_components/tplink_deco/__init__.py, line 166: config_entry.async_create_task(, please create a bug report at https://github.com/amosyuen/ha-tplink-deco/issues
2024-07-12 21:10:39.554 WARNING (MainThread) [homeassistant.helpers.frame] Detected code that calls async_forward_entry_setup for integration tplink_deco with title: <IP_ADDRESS> and entry_id: 760741b0e8f7b7503e54fb7c71a67a81, during setup without awaiting async_forward_entry_setup, which can cause the setup lock to be released before the setup is done. This will stop working in Home Assistant 2025.1. Please report this issue.

tp-link deco version 3.6.2

When will the fix be published??

I've removed the TP Link Deco integration now.  It wasn't adding a great deal to HA, the sensors were filling up the database and I wasn't really using it

So then you should immediately grant publish option to those submitting fixes instead of not approving them.

So then you should immediately grant publish option to those submitting fixes instead of not approving them.

I think you are confused, I am (was) a co-user of the integration, I have no control over the codebase.  I just commented that whilst I had in the past seen the same code issue I had now stopped using the integration.
If there is a code fix out there then it's easy enough to swap to that author's fork if the original developer has stopped maintaining it


merge from arlac77/template-github,arlac77/template-kronos-component

package.json

fix(engines): add >=14.13.0 remove >=14.12.0 (engines.node)

.github/workflows/ci.yml

chore(deps): add 14.13.0 remove 14.12.0 (jobs.build.strategy.matrix.node-version)

.github/workflows/update_readme_api.yml

chore: add 14.13.0 (jobs.update_readme_api.steps.with.node-version)

Codecov Report

Merging #763 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #763   +/-   ##
=======================================
  Coverage   91.03%   91.03%
=======================================
  Files          12       12
  Lines         792      792
  Branches       94       94
=======================================
  Hits          721      721
  Misses         71       71

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 c0c5584...a727cb6. Read the comment docs.


Making options text full width of parent

Can I check how do I make it so that the option takes up 100% of the parent, or 100% subtracting the width of the SVG? I tried flex: 1, width: '100%' but neither worked.
The rationale is so that the user can press anywhere along the container to select the option, rather than having to press on the text.

@teo-jun-xiong try flexGrow: 1, otherwise share sample code.


[BUG] [7.3.0] Breaking change in txAdmin:events:playerWarned event

Since the recent beta release, I noticed a breaking change in the txAdmin:events:playerWarned event. Previously in 7.2.2, I was using the following code:
AddEventHandler("txAdmin:events:playerWarned", function(data)
    local message = "**"..data.author.."** WARNED "..GetPlayerName(data.target).." REASON: `"..data.reason.."`"
    DiscordMessage(message)
end)

However, after the update, I had to modify the code to:
AddEventHandler("txAdmin:events:playerWarned", function(data)
    local message = "**"..data.author.."** WARNED "..GetPlayerName(data.target**NetId**).." REASON: `"..data.reason.."`"
    DiscordMessage(message)
end)

This change was necessary to get the event to work, as the data.target seems to have been replaced by data.targetNetId.
Expected behavior:
The txAdmin:events:playerWarned event should maintain backward compatibility with existing code by either supporting data.target or providing clear documentation on the required changes for forward compatibility.
Suggested fix or improvement:
If this is an intended change, please document the transition clearly to ensure backward compatibility or provide guidance on how developers should adapt their code. Otherwise, reintroducing support for data.target would prevent this issue in future updates.
playerKicked currently still works with data.target.

Thanks for reminding me I need to call out the change in the announcements.

I closed it once I spotted it in the docs. Sorry for the false alarm xD
On Wed, Oct 9, 2024 at 07:48 tabarra @.***> wrote:

Thanks for reminding me I need to call out the change in the announcements.
—
Reply to this email directly, view it on GitHub
https://github.com/tabarra/txAdmin/issues/992#issuecomment-2402229103,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2F3BVMYT2J3AMHSYMKYBG3Z2UQZTAVCNFSM6AAAAABPT5A7Q2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBSGIZDSMJQGM
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


Removed facade

The RouteAttributes class and the package facade are empty currently, and don't serve a purpose. I guess they are leftovers from the package skeleton.
This is giving me trouble with the ide-helper package:
> @php artisan 'ide-helper:generate'
Exception: Target class [laravel-route-attributes] does not exist.
Skipping \Spatie\RouteAttributes\RouteAttributesFacade.
Exception: Target class [laravel-route-attributes] does not exist.
Skipping \Spatie\RouteAttributes\RouteAttributesFacade.

As there's no reason to keep those classes around, and they likely aren't useful to what this package does, I'd like to remove them.

(Trouble as in annoying log messages)

Thanks!


Flask segmentation fault error AGAINNN

Oh god... increasing ulimit doesn't help - chatgpt suggests db connection pooling.

could solve by using pooling db connection....


replace exports with module.exports

I have made the suggested changes. can you please review it.I did verify the exported objects before and after the change was made.
Issue #82

https://github.com/beagleboard/bonescript/issues/24


specify mongodb version 4.0 because mongodb 5.0 fail to run on VM

Hello @free5gc-org, we had this issue before https://github.com/free5gc/free5gc-compose/issues/30 and I don't think that we can stick to an old version of mongodb for too long. I was wondering if FerretDB https://github.com/FerretDB/FerretDB would be a good alternative.

@abousselmi
Already revert it and update the https://github.com/free5gc/free5gc/wiki/Installation


CouchDB sl-users db is public with no _security doc on create

Similar to the situation talked about on issues #174 and #179, the sl-users db has no _security document when created so it can be accessed by any _users session user.  I think it should maybe have a Members role of admin added when it is created in the _security doc.  I tested it out manually by adding the admin role to Members -> Roles and it seems to solve the problem.
To see the problem in action, just grab a dbURL from a user session and replace the user private db like supertest$john with sl-users and you should be able to get access with curl or postman.  You can do _all_docs or _all_docs?include_docs=true as the endpoint and see everything.
A bit surprised that this has not been patched by now, but maybe there is some other configuration that I missed?

Woah! Never noticed that.
I have never really setup superlogin in a complete production environment to actually look into security concerns but in all of my simple demos that I used superlogin for, I changed the prefix of the users DB. So, I guess it would be hard for someone to guess that maybe? 😄

It looks like for CouchDB you can set a default Admin and Member _security document, but that seems to be only for the user dbs and not for sl-users, so I will need to look into this a bit more to see if there is a way to do this in the config https://github.com/colinskow/superlogin/blob/master/config.example.js .

No go on using the defaultSecurityRoles config option for sl-users the code is not there to support that.  I think this is a pretty big security issue and FWIW, security is mentioned in the README.md, but it indicates that...

To block anonymous reads across all databases you can set  require_valid_user  to  true  under  [couch_httpd_auth]  in your CouchDB config.

Which is true for anonymous reads, but it does not handle authenticated _user tokens which are readily available to all logged in users and with that token you can have access to every bit of data in the sl-users db within a matter of seconds.
To harden the sl-users db here are some options:

Change the name to something other than sl-users, add "admin" to the Permissions _security doc Members -> roles.  Name change in the config, Permissions would be a manual process.
Same thing as 1. but modify code to automatically add the _security doc role.
Haven't tried it, but you can leave the config dbServer.userDB property blank '' then provide pass in a PouchDB object to the SuperLogin constructor, so the superlogin userDB will be managed in the servers PouchDB.  This seems like a possibility for production as long as everything is working, as far as I know there is nothing like Fauxton for the PouchDB instance running in node, so it could get curly or postmanny if things go south on you.

I think 1. is the easiest thing to do, just have to remember to manually add the role anytime a new instance of superlogin is initialized or reinitialized.  Would welcome any comments here with this is fresh before I close the issue.

Closed - Going with option 1. the manual route, just need to make it part of the setup checklist. Some mention of this could be helpful in the README.md, but it seems like for now this project is not actively merging PRs.


Peer to Peer review week 3 - Updates on Portfolio

I kindly suggest you use a different unit of measure for your texts.
I kindly suggest you align your page to achieve better visual aesthetics.

Hello, @mpa-mxiang  good job with you. To improve your code I think that you can review the following points:
1 - Tha navbar is not accessible through navigation with the keyboard. Maybe you can review the w3c guide about it https://www.w3.org/WAI/test-evaluate/preliminary/#interaction
2- Maybe you are losing something important as a requirements project, the cards must be changed when are hover it. It's possible that you check again the Figma template and try to implement it, to improve your portfolio webpage, stick to the Figma template and practice one good characteristic with CSS.

Fixed above issues ✅


Typo(s?) in readme

Hello,
I noticed at least one typo in the readme. Assuming this readme is static (not sure if it's auto-generated from somewhere), I could fix it? Just curious how / if the readme is generated, otherwise I'll create a PR to fix the typo :)

the readme is static, you can fix and create a PR. :)

Sounds good! Thank you.


Followers in wrong area

Describe the bug
When you exit the museum, your followers are spawned in the area to the right between the Blue Palace and museum.  This area is behind a fence, so has no pathing to reach you.  Eventually the followers will teleport to reach you when you leave the area.
To Reproduce
Steps to reproduce the behavior:

Go to Dragonborn Museum in Solitude.
Enter the building and then exit.
Look to your right to see your followers struggling to reach you.
See error

Expected behavior
Normally followers spawn behind or next to you when you go through a door.
Screenshots

Save
OutsideMuseum.zip
Crash log
No crash, just weird.
Desktop (please complete the following information):
OS: Microsoft Windows 11 Pro Insider Preview v10.0.25158
CPU: AuthenticAMD AMD Ryzen 9 5950X 16-Core Processor
GPU #1: Nvidia GA102 [GeForce RTX 3090]
GPU #2: Microsoft Basic Render Driver
PHYSICAL MEMORY: 11.33 GB/31.91 GB

Antivirus [Windows Defender]

Additional context
The terrain outside of the museum is pretty jacked up, so maybe that is part of the problem.

This is likely caused by me forgetting to regen NAVI. Will have to check on 2.0-5


[Enhance] Supplying the model for lyft trained by regnet

Supplying the model for lyft trained by regnet with FPN architecture
Supplying the model for lyft trained by regnet with SECFPN architecture

May also add Lyft here?

May also add Lyft in the first line of README?
We implement RegNetX models in 3D detection systems and provide their first results with PointPillars on nuScenes and Lyft datasets.

Thanks for your advice. I have updated the readme.


Fix path settings of grafana ingresss is incorrect

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes https://github.com/pipe-cd/pipecd/issues/4422
Does this PR introduce a user-facing change?:
NONE

Looks like Grafana chart changed its behavior since our current stg works with the previous configuration 🤔 Lets update and see how it works on dev 👍


docs: add Japanese README file

I created Japanese translated README.

Hey @eltociear - Thanks for the PR! This will be helpful. We will review this quickly and merge it.


Failing on nuxt build

Environment

node- 14.20.0
nuxt - 2.16.3
@nuxt/bridge - npm<EMAIL_ADDRESS>@nuxt/bridge-edge - 3.0.0-28071266.dab55db

Reproduction
application is running on development mode. But failing on build.
Describe the bug
✔ Generated public .output/public                                                                                nitro 4:09:34 PM
ℹ Initializing prerenderer                                                                                nitro 4:09:34 PM

 FATAL  chunkInfo.moduleIds.at is not a function                                                                                16:09:36

  at chunkFileNames (node_modules/@nuxt/bridge/node_modules/nitropack/dist/shared/nitro.28465d4b.mjs:1546:58)
  at Chunk.getPreliminaryFileName (node_modules/@nuxt/bridge/node_modules/rollup/dist/es/shared/node-entry.js:15783:74)
  at Chunk.render (node_modules/@nuxt/bridge/node_modules/rollup/dist/es/shared/node-entry.js:15842:42)
  at node_modules/@nuxt/bridge/node_modules/rollup/dist/es/shared/node-entry.js:17480:72
  at Array.map (<anonymous>)
  at renderChunks (node_modules/@nuxt/bridge/node_modules/rollup/dist/es/shared/node-entry.js:17480:53)
  at Bundle.generate (node_modules/@nuxt/bridge/node_modules/rollup/dist/es/shared/node-entry.js:17702:19)
  at async node_modules/@nuxt/bridge/node_modules/rollup/dist/es/shared/node-entry.js:26543:27
  at async catchUnfinishedHookActions (node_modules/@nuxt/bridge/node_modules/rollup/dist/es/shared/node-entry.js:25676:16)
  at async _build (node_modules/@nuxt/bridge/node_modules/nitropack/dist/shared/nitro.28465d4b.mjs:2313:5)
  at async prerender (node_modules/@nuxt/bridge/node_modules/nitropack/dist/shared/nitro.9ad1b522.mjs:44:3)
  at async node_modules/@nuxt/bridge/dist/shared/bridge-edge.c2605ac0.mjs:403:7
  at async Nuxt.callHook (node_modules/hable/dist/hable.js:1:959)
  at async Builder.build (node_modules/@nuxt/builder/dist/builder.js:251:5)
  at async Object.invoke (node_modules/@nuxt/bridge-edge/node_modules/nuxi/dist/chunks/build.mjs:56:5)
  at async _main (node_modules/@nuxt/bridge-edge/node_modules/nuxi/dist/cli.mjs:48:20)

Additional context
No response
Logs
No response

It was an issue for the node version. Had to move version 14 to 16.

if you want run at node 14 , you can do like this, put this code at head of nuxt.config.js
if (!Array.prototype.at) {
  Array.prototype.at = function (index) {
    if (index >= 0)
      return this[index]

    return this[this.length + index]
  }
}

perfect for me! Thanks


fix typeerror because _debug is boolean

The lib did not work out of the box because _debug was false in my case.
This fixes the problem by checking if _debug is a function before calling it.

IMHO if you're already digging into the code to discover the undocumented debug option in the first place, I think it's safe to assume you'll typically either leave it unset or you will set it to a function.


Update version of yew in web_app example from 0.18 to 0.20

Pull Request
Related issue
Fixes #402
What does this PR do?

Update version of yew in web_app example from 0.18 to 0.20.
Modified with reference to the following.

https://yew.rs/docs/migration-guides/yew/from-0_18_0-to-0_19_0
https://yew.rs/docs/migration-guides/yew/from-0_19_0-to-0_20_0

I have tested the operation in my local environment.

PR checklist
Please check if your PR fulfills the following requirements:

[x] Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
[x] Have you read the contributing guidelines?
[x] Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!

bors merge


Typo in documentation

Describe the bug
In section "Metro configuration updating" in the code block for the metro.config.js file the blackList is typed as blockList but in the example project it is named as blackList
https://kirillzyusko.github.io/react-native-bundle-splitter/docs/metro
Code snippet
const resolve = require('path').resolve;
const fs = require('fs');

// Update the following line if the root folder of your app is somewhere else.
const ROOT_FOLDER = resolve(__dirname, '.');

const config = {
  transformer: {
    getTransformOptions: (_, { platform }) => {
      let modulePaths = [];
      const moduleMap = {};

      if (platform === 'android') {
        modulePaths = require('./packager/modules.android');
      } else {
        modulePaths = require('./packager/modules.ios');
      }

      modulePaths.forEach(path => {
        if (fs.existsSync(path)) {
          moduleMap[resolve(path)] = true;
        }
      });

      return {
        preloadedModules: moduleMap,
        transform: {
          inlineRequires: {
            // `blacklist` for RN < 0.64
            blockList: moduleMap, //// It should be blackList: moduleMap
          },
        },
      };
    },
  },
  projectRoot: ROOT_FOLDER,
};

module.exports = config;

@swizes I think it's because I haven't updated example application for a long time 😅
This property should be named as blacklist if you are running RN < 0.64, and blockList for RN >= 0.65


Fix msvc build warnings

There are some msvc build warnings in current xsimd code. It may block
adoption of xsimd by some projects (e.g., apache arrow) if they treat
compiler warnings as errors. As xsimd is header only, it's not easy to
build xsimd separately to ignore the warnings.

Thanks!


do you have to install the plugins yourself

when trying to use the config, it gives me a bunch of errors for plugins that aren't installed.
do i have to install them myself?

some plugins that i used with dev = true  in my local .just remove dev = true


Red sea

Merging Red Sea case history.

Final comments from Daniele and Doug addressed.

Travis fails due to broken links associated with other case histories.

If those links were broken, how did they pass Travis test and get merged into master in the first place?
This happens every time. Can we make adjustments to the test so it doesn't happen in the future?


Website: Types getting stripped out of docs

In the drawRawPoints documentation, the following dartdoc:
  /// See also:
  ///
  ///  * [drawPoints], which takes `points` as a [List<Offset>] rather than a
  ///    [List<Float32List>].

Gets rendered as :

See also:
drawPoints, which takes points as a List rather than a List.

Similarly for drawPoints:
/// See also:
///
///  * [drawRawPoints], which takes `points` as a [Float32List] rather than a
///    [List<Offset>].

Produces:

See also:
drawRawPoints, which takes points as a Float32List rather than a List.

It looks like the markdown parser strips the type declaration out, assuming they are HTML tags [the same would happen in this github view if they weren't wrapped in a code block]
I noticed it in these two docs since I was looking at them and "takes points as a List rather than List" line stood out, but I suppose it's safe to assume it happens in other places as well.

Might be a dup of https://github.com/dart-lang/dartdoc/issues/1520#issuecomment-337624691

Also probably related to https://github.com/dart-lang/dartdoc/issues/1529 and https://github.com/dart-lang/dartdoc/issues/1527.
I'm going to close this since these are open issues with dartdoc.
cc @gspencergoog


Migrate from Gradle Enterprise Gradle Plugin to Develocity Gradle Plugin

This change migrates Kafka to use the Develocity Gradle Plugin. The existing Gradle Enterprise Gradle Plugin will no longer be supported when ge.apache.org is updated to 2024.3.
These changes will not functionally change how build scans are published to ge.apache.org.
This will resolve https://issues.apache.org/jira/browse/KAFKA-17352
Committer Checklist (excluded from commit message)

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

Some of the CI pipeline has finished and I confirmed the build scans got uploaded. Here's one https://ge.apache.org/s/gfkhbv3sudpey


object-fit polyfill not working in IE 11

Seems like the object-fit plugin demo provided readme ( https://jsfiddle.net/trixta/x2p17f31/ ) is not working in IE 11. Images are stretched, did anything changed in the markup?

Thx for your bug report. This demo issue was produced by our new module pattern for plugins (introduced by 4.0.0-rc1). In reality it is very unlikely to happen. (i.e.: it does not happen if you use ES6 or CommonJS modules and it also does not happen if you manually concat/include the object-fit plugin after the lazySizes core script.)
In general I expect everybody uses build systems now. I also fixed it in version 4.0.0-rc3.


Fix helm version checking

Noticed this incorrect algorithm while browsing the changelog.  Should really use the built in semver package, but my golang-fu wasn't up to making that big a change...

@tyrken Hey! This helped me realize our test suite wasn't sufficient to catch it, and resulted in #1490 that uses the semver library for parsing it in a more robust way.
I'm closing this as included in #1490. Thanks again for your contribution ☺


Clean response API

On an ApplicationCall there are

response property which is ApplicationResponse
respond property which is ResponsePipeline
respond function which executes a subject on a pipeline

We need to figure our better non-confusing APIs.

At first, I've moved respond: RespondPipeline to ApplicationResponse.pipeline. So now we have response and respond() in call.


Advanced search

Features:

Use useExtendedSearch to enhance your search experience
Add a switch to enable / disable fuzzy search (By default, it's on)
Add a slider to customize the threshold of fuzzy search (By default, it's 0.6)

Merge to #34


Show all properties instead of fragments

Hello! Thanks for your library, it's really great! But I have one question with which I'm struggling. I don't want to show in my examples a ...Fragment properties, but every property from the model. Is that possible to achieve?

Look at this example, instead of ...OrderResponseFragment I would like to have all properties from this response.
Thanks!

I must really say I agree here. My life without this feature is really dull. I tried to move on but I cant...

Yes, I like this, too, as an option, but perhaps not the default.
The tricky part is that depending on your schema - specifically the number of fields on the return type - the output when expanding recursively deep can be very long...too long...and that's why by default it will only expand the top-level depth. I think it works well for people as the default.
However:

I'll noodle on ways to override the depth option either overall, or for specific queries/mutations/examples...but I'm not sure the latter will feel very good, and the former will probably make some situations better, but some worse.
This example looks like a "pagination" scenario, which I think does make a bit more sense to expand deeper in that case. At least the 'items' field could/should be expanded, IMO. Perhaps I can add a way/option to

Tell SpectaQL what your "pagination" scheme looks like.
Expand the "Items" (or other pagination fields) when a pagination type is detected.

Thanks for your response. I think option for configure it globally will be the best, and then for example you can add a "+" button at the left side of properties which could be expanded, like items or pageInfo in my case. Then it could be dynamically expanded by user on click (collapsed or uncollapsed by default, also configurable).
About the recursively deep expanding, then you can for example expand only 2 levels of it, and then in 3rd level just paste what's now here - a fragment of model with spread operator. It will be obvious then, that's it a recursion in property.
Please take a look on this: https://rejestr.io/api/info/v1/dokumentacja#operation/searchKrsOrgs - it's documentation based on Redocly. On the right side you can see that properties are expandable by user interaction and you have also option to collapse/expand all.

One more thing - it will be cool, when there will be possibility to see all return properties in model at the query level. So you don't need to click on model name, scroll down and then return, but on query level you see all return properties with their proper description. Like it's also in Redocly:

Regards
Adrian

Support for a fieldExpansionDepth option now exists and is out in 1.1.0.
Closing.


About Training Time

Hi authors! Thanks for your excellent work. I encounter low training efficiency issues. The second Instruction finetuning stage takes about 6 days on my 8*A100 (40G) GPUs, utilizing only Town01 data downloaded from openxlab. I noticed you mentioned that Instruction finetuning takes around 3 days for the visual encoder on 8x A100 (80G). If you utilize all (Town01-Town07,Town10) data during the finetuning stage? and what could be the possible reasons on my machine?

Hi!
Sorry for the late reply. Maybe you can check the disk speed? It will significantly affect the training efficiency. I recommend moving the training data to the SSD.


revert breaking change of vault tls path, mounts and env vars

partial revert of; https://github.com/banzaicloud/bank-vaults/commit/7338457003d7f019cbd96626d0073fa99bb6c128
does not seem necessary, not sure if this breaks something else but all of the doco (https://github.com/banzaicloud/bank-vaults/search?q=%2Fvault%2Ftls%2Fca.crt&unscoped_q=%2Fvault%2Ftls%2Fca.crt) and implementations is without the '-env', etc.

Yes, this currently breaks the capability of mounting "vault-tls" volume to vault itself, next to its own "vault-tls" volume, which would like to use the transit engine. :) I would reverse the question, how this breaks other stuff?

well check the search link; https://github.com/banzaicloud/bank-vaults/search?q=%2Fvault%2Ftls%2Fca.crt&unscoped_q=%2Fvault%2Ftls%2Fca.crt
look at all the references to that path. I have everything setup as that path, so using 0.5 all of my deployments failed.

I have added some code paths to handle this situation more gracefully (instead of introducing another breaking change ;) )
If there is a vault-tls volume mount already, we use the vault-env-tls volume mount name and mountPath, so all the old way and the new way will work again.

@jurgenweber please check this version the transit unseal scenario works again, hopefully, your deployments are too.

https://github.com/banzaicloud/bank-vaults/releases/tag/0.5.1-rc.1

I have added some code paths to handle this situation more gracefully (instead of introducing another breaking change ;) )
If there is a vault-tls volume mount already, we use the vault-env-tls volume mount name and mountPath, so all the old way and the new way will work again.

yeah, that was the sort of solution I was going to suggest today if you didn't manage to get around to it.

i'll give it a whirl today.


WIP: Document component keyboard shortcuts (a11y)

This PR tracks the progress of adding the documentation for all the components keyboard shortctus, it is still a WIP
components

[x] ListBox
[x] Accordions
[ ] Avatars  (no keyboard interactions)
[ ] Alerts  (no keyboard interactions)
[ ] Breadcrumbs (no keyboard interactions)
[ ] Conic Gradient (no keyboard interactions)
[x] Data Tables
[ ] Menu
[ ] Progress Bar
[ ] Progress Radial
[ ] Radio Groups
[ ] Range Slider
[ ] Slide Toggle
[ ] Stepper
[ ] Tabs
[ ] Tooltips

utilities

[ ] Drawers
[ ] Dialogs
[ ] Toasts
[ ] Lightswitch

Looking good with the addition of the data table and accordion. Good catch on the lastIndex fix for the table. FYI this is one of several features being reworked in the near future:

https://github.com/Brain-Bones/skeleton/issues/254

Given there's a lot changes coming to these, it may be better to let me tackle them as I rework the feature and docs myself. For example, I'm working on the Menu now and the changes are pretty substantial. We're moving from a Svelte Component to Tailwind Elements + Svelte Action:

https://github.com/Brain-Bones/skeleton/issues/196

Just want to ensure you're not putting in effort on something that's going to be thrown out and rebuilt soon! Plus this might make the merge a but funky.

I love the new actions API, will read more about that but I think you make a point, it is more efficient/reasonable if you document the shaking parts.

Thanks, still plenty to figure out but the concept is solid. Yeah no worries, I'll handle the docs listed in that link above. Feel free to prune those from your todo list or move them to their own category for now. Whatever works!

FYI I've applied a11y info and keyboard interactions to the upcoming Menu update. Note that Tooltip are still marked "work in progress" for the ARIA guidelines, so I may revisit that in the future.

@benzara-tahar Per my post in Discord I'll be away this weekend for my birthday. If you need anything before Monday please reach out to a moderator on Discord. Have a nice one and catch you next week!

I went through all the items in the checklist that aren't marked  with (leave this to endigo),   had to do a small fix  for the SlideToggle interaction inorder to make it work.
Also Happy birthday btw @endigo9740 :)

Great job @benzara-tahar I'm just getting caught up with everything over the weekend, but will review this asap. FYI make sure to tap "ready for review" to make a formal declaration when you're ready for me to take a look! I'll grab for it now.

@benzara-tahar I've completely my review. There was a few bit of text that I adjusted, but nothing major. I think with this we're good to merge. Great job with this, very clearly done! You should appear as a contributor on GitHub and our homepage very soon :)


Mailto urls with query parameters incorrectly converted to file urls

Lychee seems to incorrectly convert mailto: links with query parameters into file:// links.
mailto:user@example.com?subject=Hello becomes<EMAIL_ADDRESS>The actual link to check (when checking emails) is<EMAIL_ADDRESS>

Can you provide some more info here? What's the input file format you're using? Can you provide a minimal failing test case?
I tried it locally, but I can't reproduce:
echo "this is a mailto:user@example.com?subject=Hello link" | lychee -- --dump -
<EMAIL_ADDRESS>

Also see https://github.com/lycheeverse/lychee/pull/655.

Can you provide some more info here? What's the input file format you're using? Can you provide a minimal failing test case?

I was running this on the output of a Zola static website. The actual email address was different.
I'll try to put a test case together that shows off the behavior. I was running with the --offline flag. My intuition is that this doesn't happen with --dump.

Actually, it does show the same behavior with --dump. The error occurs when reading from a full HTML file. I could not reproduce it by echoing just the p tag on line 10 from the attached files to lychee.
$ lychee ~/Desktop/lychee-test-working.html --dump
<EMAIL_ADDRESS>
$ lychee ~/Desktop/lychee-test-broken.html --dump
<EMAIL_ADDRESS>
A .txt extension has been added to the files since apparently you cannot attach HTML files to GitHub issues.

lychee-test-working.html.txt
lychee-test-broken.html.txt

P.S. I'll take a look at the code and try to issue a pull request with the fix, but that may not happen for a bit as I have a backlog of other tasks that I need to work through first. ☹

@mre There's a mailto link in this Code of Conduct that is being flagged.
The contents of the link is mailto:abuse@exercism.org?subject=%5BCoC%5D

Update, I see now we are behind. We are using version 1.4.1. I'll update and see if that fixes things for us.

@mre I hope my updates above were helpful in seeing this issue was affecting a repo.

I noticed version 1.5.2 was released, re-ran the workflow and the issue still persists.


Migrate TechDocs to new Frontend System

This is part of the work related to #18372. After this PR TechDocs should be configurable through config using the new Frontend System.
:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[ ] All your commits have a Signed-off-by line in the message. (more info)

Changeset added, but running into the following errors in the app-next. A bit suprised about the fetchApiRef, but expected the error with the catalog.
Question: Should I add it to the app-next even though it's not functioning yet?

Blocked by finishing the routing system implementation for now

Nice! Are you looking into camila's comment about configSchema too?

Did not include it yet - let's change it when adjusting the routeRef :)


phimmedia.tv

@adguard-bot commented on Sat Jun 08 2019
Issue URL (Ads)
http://phimmedia.tv
Comment

Dear Developers,
Adguard for Windows failed to block any ads on phimmedia.tv site. I enabled the ABPVN filter and it still doesn't work either. However, the Adguard adblocker extension for Edge browser does work. I used the same exact settings and filters for both. What is wrong with the new version 7.0 of Adguard PC? My computer is using the latest Windows 10 version May 2019 Update. Please look into this issue and find a fix soon for Adguard PC.
I had sent an email but told I have to report it here. So I have to do it again. The picture I uploaded was after I uninstalled Adguard PC and use the Adguard Adblocker extension for Edge. The Edge extension works perfectly at removing all ads on this website as you can see there are no ads.
Thanks

Screenshots

Screenshot 1

System configuration

Information
value

Platform:
Win

AdGuard version:
7.0

Browser:
Edge

AdGuard driver:
WFP

Filters:
AdGuard Base,AdGuard Tracking Protection,AdGuard Social Media,AdGuard Annoyances,Malware Domains,Spam404,ABPVN List,NoCoin Filter List

It seems that AdGuard's JS is not injected (there is no local.adguard.org in source code).

Works for me:

Does site have AG certificate?
Can you please send debug log?

Ah ok, local.adguard.org is really missing. I'll check this

Merged into https://github.com/AdguardTeam/CoreLibs/issues/965


Update unit as  inference/sec

As a user, I am trying to enable the unit self-explanatory.
Usually, we see unit format as   /. Example: X images/sec,  X frames/sec or etc where images or frames helping user to understand , what the value for.
Below Rate unit, until, I read doc or check with owner, I am not sure what it means.  Just for a layman, if we keep like below, they might not get it. Rate: X/sec
But, if we keep, X inference/sec, it will help clearly to know what this value for.  The Rate is  how many inferences of the given inputs can be processed per second so X inference/sec will help user for the same.
For advance user, it /sec might be ok but I am trying to cover all.

Hi @causten @umangyadav
Please feel free to merge as I don't have the permission.
After that need to put the same in master and later 6.0 release branch

This formatting looks wrong. Before it was /sec, which is fine to not have a space between the number. But now we need to modify it as such:
    os << "Rate: " << rate * batch << " inferences/sec" << std::endl;

This formatting looks wrong. Before it was /sec, which is fine to not have a space between the number. But now we need to modify it as such:
    os << "Rate: " << rate * batch << " inferences/sec" << std::endl;

I agree...
Batch size: 16
Rate: 1337.84inferences/sec
Total time: 11.9596ms
Total instructions time: 12.9021ms
Overhead time: 0.0595123ms, -0.942473ms
Overhead: 0%, -8%

Honestly it would parse a lot easier if it looked like...
Batch size: 16
Rate (inf/sec): 1337.84
Total time (ms): 11.9596
Total instructions time (ms): 12.9021
Overhead time (ms): 0.0595123, -0.942473
Overhead: 0%, -8%

Honestly it would parse a lot easier if it looked like...
Batch size: 16
Rate (inf/sec): 1337.84
Total time (ms): 11.9596
Total instructions time (ms): 12.9021
Overhead time (ms): 0.0595123, -0.942473
Overhead: 0%, -8%

I like that better. also i'll need to change my script to reflect this change now. I'll do that now.
https://github.com/ROCmSoftwarePlatform/YModel-Scripts/blob/90bc6e1dc1b600bc062cd3d55c0791ec6387c334/run_ymodel.sh#L147

Rate (inf/sec): 1337.84

This also looks fine. Can we get this merged in develop, master and release 6.0 branch.

After that we need to also modify the our automation code.
cc: @umangyadav @causten


fix(community): ElasticVectorSearch: exclude metadata filters not working due changing to term instead of terms

The fix merged in #6536 broke our code due to the following error:
parsing_exception: [term] query does not support array of values
As we also need to allow an array, I made some changes to allow both term and terms.

Hey sorry about this and the delay here - will test and have a look.
It might be nice to add an integration test here to prevent breakages in the future.

Added in https://github.com/langchain-ai/langchainjs/pull/6688 and will merge there. Thank you very much!

Merged in #6688, thank you again!

Hey sorry about this and the delay here - will test and have a look.
It might be nice to add an integration test here to prevent breakages in the future. I can look into it.

No problem at all, thanks for reviewing and merging!
Have a nice day


docs: remove the version from the component

This PR relates to and is explained in https://github.com/modelix/modelix.docs/pull/18

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


WXCInkCanvas: Access violation reading location exception

When I try to add WXCInkCanvas on a view, I get Access violation reading location exception
WXCInkCanvas*inkCanvas = [WXCInkCanvasmake];
UIView *testView = [[UIView alloc] init];
[testView addSubview: inkCanvas];

Same issue with WXCInkToolbar.
Please advice how to work with InkCanvas & InkToolbar in WinObjC project.

Xaml controls cannot be treated simply as UIView objects. What you may be looking for is the initWithFrame constructor for a UIView. I would recommend trying something like...
WCXInkCanvas *inkCanvas = [WCXInkCanvas make]; CGRect frame = CGRectMake(0, 0, 500, 500); UIView *inkCanvasView = [[UIView alloc] initWithFrame:frame xamlElement:inkCanvas]; [mainView addSubview: testView];
Let me know how this works out for you. Thanks!

Hi, @MSFTFox  Thanks a million! It works!
Here is more detailed code snippet, which works for me:
        WXCInkCanvas *inkCanvas = [WXCInkCanvas make];
	CGRect frame = CGRectMake(0, 0, 500, 500);
	UIView *inkCanvasView = [[UIView alloc] initWithFrame:frame xamlElement:inkCanvas];
	[_mainView addSubview: inkCanvasView];

	WXCInkToolbar *inkToolbar = [WXCInkToolbar make];
	inkToolbar.targetInkCanvas = inkCanvas;
	inkToolbar.initialControls = WXCInkToolbarInitialControlsAll;
	UIView *inkToolbarView = [[UIView alloc] initWithFrame:frame xamlElement:inkToolbar];
	[_mainView addSubview: inkToolbarView];

	_inkPresenter = inkCanvas.inkPresenter;
	_inkPresenter.inputDeviceTypes = WUCCoreInputDeviceTypesMouse | WUCCoreInputDeviceTypesPen | WUCCoreInputDeviceTypesTouch;

Wonderful! I'm glad to hear it!
I'll close this issue then.


Added my name

Hello

Hello @koolmili
Can you please do these steps
open git bash
then do these steps( if you need help in any step ping me back)
cd git-learning
git checkout -b newbranch3
go to folder (git-learning) and create a new file yourname.html and add your name there
now come back to git shell
git status to check the status
git add .
git commit -m "new file added"
git push origin newbranch3
then enter your username and password
come back to this website and do comare and pull request.
Cheers,
Shahbaz

thksss :)
On Mon, Jan 25, 2016 at 9:24 AM, Md Shahbaz Alam<EMAIL_ADDRESS>wrote:

Hello @koolmili https://github.com/koolmili
Can you please do these steps
open git bash
then do these steps( if you need help in any step ping me back)
cd git-learning
git checkout -b newbranch3
go to folder (git-learning) and create a new file yourname.html and add
your name there
now come back to git shell
git status to check the status
git add .
git commit -m "new file added"
git push origin newbranch3
then enter your username and password
come back to this website and do comare and pull request.
Cheers,
Shahbaz
—
Reply to this email directly or view it on GitHub
https://github.com/shahbaz17/git-learning/pull/8#issuecomment-174387333.


Make OPENSSL_malloc_init() a no-op

Making this a no-op removes a potential infinite loop than can occur in
some situations.
Fixes #2865

No one seems to know why this would ever be needed. I can't think of a good reason, and making it a no-op fixes the issue in #2865. If anyone can suggest why this shouldn't be done, please let me know!

Pushed. Thanks.


'Unexpected value 'undefined' exported by the module' * ... in generated typings file. When importing custom library. #10967

the same issue as this one
https://github.com/angular/angular-cli/issues/10967
facing it now
consider reopening the original or handling it here

The referenced issue was opened against a much older version of Angular than the current version (6.0 vs 11.2) and there have been significant changes since version 6.0.
If encountering a similar issue, please provide the information present in the issue template including the output of ng version and a minimal reproduction if possible.


chore: merge dev into master

Coverage decreased (-3.2%) to 76.218% when pulling 3cc5ea8cb1490feebbb4941ffabd1f2c38e3e839 on dev into 342a257bbd856e8ec5f361a026d9f561b11076aa on master.


What providers should be added next?

At the moment CloudProxy supports AWS and DigitalOcean, which is enough for my own personal use case. I'm keen to hear if there is interest in other providers being supported, please share here and I will prioritise. Otherwise, new features will be prioritised for now.

Google cloud and Azure

Hetzner cloud has a pretty good API, very competitive pricing (way cheaper than AWS) and good connectivity to Frankfurt, Germany

Linode and Google Cloud are the top options for me.

How about Cloudflare Workers?

Hetzner cloud has a pretty good API, very competitive pricing (way cheaper than AWS) and good connectivity to Frankfurt, Germany

Hetzner has now been added (I'm also a big fan of Hetzner).
Azure is next on the list.

How about Cloudflare Workers?

Cloudflare Workers, I'm not familiar with, my understanding is it's more serverless. Though I will investigate.

Google Cloud has now been added.


SizedBox Issue

/C:/src/flutter/flutter/packages/flutter/lib/src/widgets/basic.dart:2304:44: Error: The getter 'width' isn't defined for the class 'SizedBox'.

'SizedBox' is from 'package:flutter/src/widgets/basic.dart' ('/C:/src/flutter/flutter/packages/flutter/lib/src/widgets/basic.dart').
package:flutter/…/widgets/basic.dart:1
Try correcting the name to the name of an existing getter, or defining a getter or field named 'width'.
properties.add(DoubleProperty('width', width, defaultValue: null, level: level));

Hi @sunsj1,
Can you please provide your flutter doctor -v and flutter run --verbose logs, and a minimal complete reproducible code sample?
Thank you


aws credentials in the app not working to escalate more ... !

found aws credentials but not working to dig deeper

Thank you for reporting this issue. In the next update on April 4th a new AWS S3 bucket will be created for this flag.

This is fixed with the latest update.


there is no 'calender' icon to add

when I was building a webpage for my application, I mentioned
but it never appeared

mif-calendar


Accesspoint not being broadcast

I've used two Android phones, one Macbook, one Windows laptop and a Kobo e-reader. None show the 'Google Free WIFI' network.
When launching the portal, I notice that the Flipper display shows the entire Evil-Portal 'Index.html' file's content. I have Evil Portal as part of the recent Hidden Master release, I did not install it manually. I've been using Marauder on my ESP32 board for a while now, so I know that it's got the right firmware and working correctly. Maybe just a typo somewhere along the HTML code?
I've uploaded a short youtube video demonstrating what I'm seeing on my flipper, link: evil portal bug?
I've attached the logs generated by Evil-Portal
log_0.txt
log_1.txt
log_2.txt
log_3.txt
log_4.txt
log_5.txt
log_6.txt

You need to flash evil portal firmware to the esp32.

Look under Releases. There is a full written Tutorial for flashing and the zip archive you're looking for.

You won't be able to run Marauder anymore since that requires it's specific firmware on the esp. If you want to run Marauder again you need to flash its firmware again. However there is a tool called "ESP Flasher" which allows you to flash the esp from the flipper itself this way you can do it on the go.

I don't think those steps listed above are quite accurate, given this instruction for people running Marauder.... https://github.com/justcallmekoko/ESP32Marauder/wiki/evilportal  Correct me if I'm wrong? I was going strong until I got up to the index.html part, and yet I'm still getting a "could not find /index.html. Use stopscan..." There are a handful of sources on here specifying different things.

I have this same issue. I've definitely got the Marauder with Evil Portal firmware loaded (and have tried the Evil Portal only release firmware)
All my configs are in place - the ap.config.txt and the index.html. Both are recognized when starting the portal. The board can scan and capture no problem. But starting evil portal loads the configs, says web server running - but is broadcasting not a single thing from the radio.
I reloaded Black Magic and confirmed the board isn't broken - it transmits and receives just fine. But for whatever reason, the portal AP never actually stands up.


License

Codecov Report
Patch and project coverage have no change.

Comparison is base (64f138a) 90.05% compared to head (9dc87f6) 90.05%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main       #4   +/-   ##
=======================================
  Coverage   90.05%   90.05%
=======================================
  Files          11       11
  Lines         191      191
  Branches       31       31
=======================================
  Hits          172      172
  Misses         10       10
  Partials        9        9

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Revert "chore(apps/prod/tekton/setup): bump tekton operator to v0.61.0"

Reverts PingCAP-QE/ee-ops#836

/approve


Add optional version input

Add option to optionally specify a specific Conan version
Fixes #1

Looks great! I didn't see I'm sorry, I don't get a lot of PRs 😅


Experiment with  scapulothoracic model

I want to try to use the main.py as-is on the scapulothoracic model (no muscles). I expect the ID check to fail since inverse dynamics does not account for point constraints such as the acromioclavicular (AC) one.

Even before arriving to the ID check, the code fails as it does not support many of the joints in the model (like CustomJoint, ScapulothoracicJoint and so on).

I was able to modify Antoine's code to generate a functioning external function that captures symbolically the behaviour of ID on a simple pendulum model.
For what concerns the scapulothoracic joint, there is probably more work to do since the OpenSim version that is shipped with the current OpenSimAD setup instructions appears not to include it yet.


WIP: Incoporate Sebolabs curl fix, with fixes to the fix, and some upgrades

I actually feel this is more of a problem for CI systems. With the proliferation of dockerized builds mktemp needs the mount privilege. Is this something you'd consider reverting?


Copy some entries from an archive to another archive

I would like to copy some entries from a file to another file, which is the best way to do it?

You can instantiate two instances of Archive. One with accessMode: .read and the other with .update.
Then you can iterate over the "read" archive (for entry in readArchive) and add entries to the "update" archive (updateArchive.addEntry(...)).
To avoid filesystem roundtrips, I'd use the closure-based API . You can pass the data that you read via archive.extract(entry, consumer: { ... } to the closure based variant of .addEntry.


Extended APIs delegate merge

Handlers for three different exit reasons have been added. The implementation right now is working,
but has alot of room for improvement I think. For starters, it should probably try to reuse some functionality across similar reasons (e.g. rdmsr and wrmsr). However, I think we should wait until more handlers are written before we try to generalize it.
Now, each exit reason is represented as a small class. The following pseudocode shows their basic function
auto key = make_key(vmcs)
auto handler = handler.at(key);
handler(vmcs);

What differs between the classes (and what I think should be generalized) is what makes up a "key". I view it as basically a snapshot of additional state/information about the exit e.g. right now, the wrmsr
key is the msr address (vmcs->save_state()->rcx), but it very well could be the address and the value written. This would allow for entities interested in wrmsrs to not have to worry about exceptional values written to the msr (and perhaps an exception susbsystem would only subscribe to exceptional write values).
What are some other ways/ideas that we could do this? Should we do this? The example code is mainly in exit_handler/{rdmsr,wrmsr,cr_access}.{h,cpp}. They are created in the vcpu_factory.cpp

I will start work on this tomorrow


6L search and mods orders

Can we have a quick way (like a button) to chose 6L instead of click all 5 linked line?
Also can you separate normal mod and psuedo mod like trade marco?

All sockets/links can be selected at once by clicking while holding shift. See: https://github.com/Kyusung4698/PoE-Overlay/wiki/Evaluate#sockets
This is an intended behaviour to keep the clutter small. Therefore all mods which are aggregate as pseudo will be removed from the view. But i may add a "toggle hidden mods" in the future.

The hidden mods issue will be continued here #224.


ValueError: Invalid header value b'Bearer

I would like to try your code.
I put my api key in openaiapikey.txt in the repository and run the follows, it gave me an error:
$ python hello_world.py
Traceback (most recent call last):
  File "hello_world.py", line 29, in <module>
    response = gpt3_completion(prompt)
  File "hello_world.py", line 22, in gpt3_completion
    stop=stop)
  File "/opt/anaconda3/lib/python3.7/site-packages/openai/api_resources/completion.py", line 25, in create
    return super().create(*args, **kwargs)
  File "/opt/anaconda3/lib/python3.7/site-packages/openai/api_resources/abstract/engine_api_resource.py", line 122, in create
    request_timeout=request_timeout,
  File "/opt/anaconda3/lib/python3.7/site-packages/openai/api_requestor.py", line 179, in request
    request_timeout=request_timeout,
  File "/opt/anaconda3/lib/python3.7/site-packages/openai/api_requestor.py", line 363, in request_raw
    timeout=request_timeout if request_timeout else TIMEOUT_SECS,
  File "/opt/anaconda3/lib/python3.7/site-packages/requests/sessions.py", line 533, in request
    resp = self.send(prep, **send_kwargs)
  File "/opt/anaconda3/lib/python3.7/site-packages/requests/sessions.py", line 646, in send
    r = adapter.send(request, **kwargs)
  File "/opt/anaconda3/lib/python3.7/site-packages/requests/adapters.py", line 449, in send
    timeout=timeout
  File "/opt/anaconda3/lib/python3.7/site-packages/urllib3/connectionpool.py", line 672, in urlopen
    chunked=chunked,
  File "/opt/anaconda3/lib/python3.7/site-packages/urllib3/connectionpool.py", line 387, in _make_request
    conn.request(method, url, **httplib_request_kw)
  File "/opt/anaconda3/lib/python3.7/http/client.py", line 1252, in request
    self._send_request(method, url, body, headers, encode_chunked)
  File "/opt/anaconda3/lib/python3.7/http/client.py", line 1293, in _send_request
    self.putheader(hdr, value)
  File "/opt/anaconda3/lib/python3.7/http/client.py", line 1230, in putheader
    raise ValueError('Invalid header value %r' % (values[i],))
ValueError: Invalid header value b'Bearer sk-4XXXXXXXXXXXXXXcpi\n'

Could you help?

Replace the following on line 9
openai.api_key = open_file('openaiapikey.txt')
with
openai.api_key_path = 'openaiapikey.txt'
then it works great


Fix big.Int (incorrectly) marshals to a number instead of a string

Task
Currently, big.Int incorrectly maps to integer instead of string. This is a known "feature" of golang (see here).
A possible fix is to wrap the big.Int into a type and marshal said type to a string.
Acceptance criteria

Big int (or wrapped type) correctly marshals to string.

This issue only affects debugapi for now.


On Android, a Toast can only show a maximum of 2 lines

Summary
On Android we can show a Toast with a message. However, the message is cut off after two lines. On iOS all lines are shown.
I don't see a reason why this needs to be cut off.

API Changes
The API does not need to change. The Implementation only
Intended Use Case
For example when I display a message with some additional error information.

This is actually by design since the snackbar doesn’t really resize well on all size factors for droid

Ok, thanks for your fast answer and your work. I appreciate it.


Custom node installation location in .nvmrc

Feature request:
Could nvmrc point to a local path where a custom node version is installed? When nvm would be run it would simply witch the path to that location.
This would be useful when using https://github.com/eirslett/frontend-maven-plugin which downloads and installs a version of node as specified in maven's pom.xml.

Other than system, nvm manages its own node versions.
If you want a maven plugin to install a version of node for you, you may want to configure the installation location to be nvm_version_path 4.5.6 (or whichever version); however it'd be better to let nvm do the installing.

I would love to leave the install tonvm, but our build process is driven by maven and it does the installation into a directory within the repo.
It would be great if we could use nvm in some way to set the proper path to the custom location so that we could use nvm and node as we are used to from javascript projects.

Instead of using maven to install node, could you configure maven to run arbitrary commands?

That should be possible. It would have to be separate profile and hence would be different build than process than CI does. But that should be ok. Are you suggesting to set up a .nvmrc to the same version as maven's pom.xml specifies and simply call nvm use from maven?

Yes.


net/haproxy: switch stats to native JSON data format

From the release notes of HAProxy 1.8.0:

JSON stats (Simon Horman) : the stats socket's "show stat" and "show info"
output can now be emitted in a structured JSON format which is more
convenient than CSV for some modern data processing frameworks.

Since this would only be a cosmetical change (that would still require some effort), I've decided to scrap this idea.


Unable to Launch TabBar on Android

Hi, I just started implementing react-native-router-flux(Lovin it - beer for you @aksonov ) for my app. It works like a charm for my IOS version. Trying the basic setup on Android, and I cant get to the TabBar view. Getting the following error. I looked up for the documentation for  Navigator  of React Native Android, and it find anything different when compared to iOS. Did anyone run into this issue?
Thanks in advance.

Try to debug in Chrome dev tools (set breakpoint for exception). For now i don't see what code of my component could raise it (stacktrace doesn't show it as well)

Not entirely sure, if this info is what you're looking for or not .. this is the exception. And, this is the error from the call stack - https://gist.github.com/chai2/7e4dc8d672f285b580e5

No, it is useless. If you are sure that the exception raised from my component, you have to find the line in the component code. Set breakpoints on all 'navigator' code and find out where the exception happen.

Closing it for now


Make pkgs vars default

Remove packages (env.pkgs.) from all.yaml.template and use defaults instead.

@veera-damisetti Pls review this PR. I don't know why I picked up that amount of commits but nevertheless the PR is good to go. The packages parameters are removed and substituted with default values. Nevertheless user could add specific pkgs_xxx to all.yaml file. The documentation is updated accordingly.

@isumitsolanki  shall we sync offline and review this PR
thanks

@veera-damisetti sure , we'll do in sometime.

@AmadeusPodvratnik , @veera-damisetti , Looks good to me.

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


Problem building docker image

@Baschdl: When trying to build your docker image, I get the following stack trace:
Step 12 : RUN python server/manage.py migrate
 ---> Running in d4a8f66ecac8
Traceback (most recent call last):
  File "server/manage.py", line 8, in <module>
    execute_from_command_line(sys.argv)
  File "/usr/local/lib/python2.7/site-packages/django/core/management/__init__.py", line 385, in execute_from_command_line
    utility.execute()
  File "/usr/local/lib/python2.7/site-packages/django/core/management/__init__.py", line 354, in execute
    django.setup()
  File "/usr/local/lib/python2.7/site-packages/django/__init__.py", line 21, in setup
    apps.populate(settings.INSTALLED_APPS)
  File "/usr/local/lib/python2.7/site-packages/django/apps/registry.py", line 108, in populate
    app_config.import_models(all_models)
  File "/usr/local/lib/python2.7/site-packages/django/apps/config.py", line 202, in import_models
    self.models_module = import_module(models_module_name)
  File "/usr/local/lib/python2.7/importlib/__init__.py", line 37, in import_module
    __import__(name)
  File "/usr/local/lib/python2.7/site-packages/social/apps/django_app/default/models.py", line 1, in <module>
    from social_django.models import AbstractUserSocialAuth, UserSocialAuth, Nonce, Association, Code, DjangoStorage
ImportError: No module named social_django.models

@jonico We will look into this

@jonico Sorry for the inconvenience, apparently there were some breaking changes between dependency upgrades. It should work now. Have a nice weekend!


Chrisda to Master

Docs Build status updates of commit f79bfb0:
:clock10: Full build: the average full build time is 13 min(s) 30 sec(s), this is based on the last 1000 build(s) for this repository.

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

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/policy-and-compliance/Set-LabelPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 61c5949:
:clock10: Full build: the average full build time is 13 min(s) 39 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 61c5949:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/policy-and-compliance/Set-LabelPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


src/app: fix path to deny-request-copy component

References

Fixes https://github.com/DSpace/dspace-angular/issues/2351

Description
The themed-deny-request-copy.component erroneously includes the custom theme's deny-request-copy component instead of its own. This appears to work, but breaks when the custom theme is missing (and of course the application should not be relying on the custom theme at all).
Instructions for Reviewers
Build the application and see if the deny request copy functionality is working as expected (including the themed component in the custom theme).
List of changes in this PR:

Fix path to deny-request-copy.component

Checklist

[x] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & specs/tests), or I have provided reasons as to why that's not possible.
[x] My PR passes ESLint validation using yarn lint
[x] My PR doesn't introduce circular dependencies (verified via yarn check-circ-deps)
[x] My PR includes TypeDoc comments for all new (or modified) public methods and classes. It also includes TypeDoc for large or complex private methods.
[x] My PR passes all specs/tests and includes new/updated specs or tests based on the Code Testing Guide.
[x] If my PR includes new libraries/dependencies (in package.json), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.
[x] If my PR includes new features or configurations, I've provided basic technical documentation in the PR itself.
[x] If my PR fixes an issue ticket, I've linked them together.

I don't use the request a copy feature so I can't test this other than building and seeing if anything is broken. @Leano1998 could also take a look since he found the issue and might be using request a copy?

@mwoodiupui : This might be a quick test for you, as it appears this bug was accidentally in #2245 .

I tested the change and it works good for me.

@alanorth :  We talked about how to sync "dspace-7_x" and "main" in today's dev meeting: https://wiki.lyrasis.org/display/DSPACE/2023-07-06+DSpace+Developers+Meeting#id-20230706DSpaceDevelopersMeeting-Notes
We decided that the rule should be to have two PRs (one for each branch).  But, for small fixes, it's OK to make an exception and just apply the fix to either branch...and I (or another Committer) can sync it to the other one.
So, to answer your question, yes you can merge into "dspace-7_x", but I'd appreciate it if you could then port to main (I just added that flag as a reminder).  Once done, it can be merged immediately to main and the port to main flag removed.


Implement an rsync receiver, too

Currently, we only implement a sender (for serving files), but a receiver would be neat, too (for downloading files).
Left to do:

[x] implement receiver protocol
[x] write files to disk
[x] only transfer deltas
[ ] cross-system user/group id/name mapping

https://github.com/gokrazy/rsync/commit/d5a5b3defb3b1b49e795200e215823cd797371b6 lays the foundation.
Next up is actually writing files to disk.

https://github.com/gokrazy/rsync/commit/d1c307d7a3db853abb5b39de3a206303c4936f4f writes files to disk.
Next up is generating checksums / transferring deltas

@stapelberg Is this ready to go yet? This would be great for my backup/restore functionality 😅 I'm happy to help test your work!

@stapelberg Is this ready to go yet? This would be great for my backup/restore functionality sweat_smile I'm happy to help test your work!

The status in the commit messages and first comment of this issue is accurate. A receiver exists (so feel free to try it out!), but it doesn’t map user/group ids/names across machines.

@stapelberg Thanks! I actually did, however I decided to go with a completely different solution as I wasn't able to get what I had hoped working with gokr-rsyncd -- I think what's missing is (which is stated on the README) is receiver support in the daemon/server itself 🤔

Yes, the daemon doesn’t accept uploads yet.

@stapelberg Hi! Can I already use this package to sync local directory on windows with linux server?

Depends on what kind of synchronization we’re talking about.
If you want to download files from a Linux server, yes.
If you want to do bi-directional synchronization: no. As I wrote in the previous message, the daemon does not accept uploads yet.


rename property cassandraTelemetry property [K8SSAND-1252]

What this PR does:
Renames the property to just telemetry
Which issue(s) this PR fixes:
Fixes #411
Checklist

[ ] Changes manually tested
[ ] Automated Tests added/updated
[ ] Documentation added/updated
[x] CHANGELOG.md updated (not required for documentation PRs)
[x] CLA Signed:  DataStax CLA

Why are we doing this? We have previously discussed it and concluded that it is a bad idea given that Stargate and Reaper properties are also held within the cassandra field.
Has that rationale changed since we had this discussion?

I think it  is clear from the parent/child relationships or nesting of properties that cassandraTelemetry or telemetry applies to Cassandra. Other than the CassandraConfig property, we don't prefix any of the other property names with cassandra.
@jdonenine thoughts?

I think it is clear from the parent/child relationships or nesting of properties that cassandraTelemetry or telemetry applies to Cassandra. Other than the CassandraConfig property, we don't prefix any of the other property names with cassandra.

Given that other fields of the parent include Stargate, I disagree in this case...

A trimmed down example for comparison.
before:
apiVersion: k8ssandra.io/v1alpha1
kind: K8ssandraCluster
metadata:
  name: test
  namespace: k8ssandra-operator
spec:
  cassandra:
    datacenters:
    - config:
        jvmOptions:
          heapSize: 384Mi
      metadata:
        name: dc1
      size: 1
    serverVersion: 3.11.11
    cassandraTelemetry:
      prometheus:
        enabled: true
  stargate:
    size: 1
    telemetry:
      prometheus:
        enabled: true

after:
apiVersion: k8ssandra.io/v1alpha1
kind: K8ssandraCluster
metadata:
  name: test
  namespace: k8ssandra-operator
spec:
  cassandra:
    datacenters:
    - config:
        jvmOptions:
          heapSize: 384Mi
      metadata:
        name: dc1
      size: 1
    serverVersion: 3.11.11
    telemetry:
      prometheus:
        enabled: true
  stargate:
    size: 1
    telemetry:
      prometheus:
        enabled: true

Example of the problem I'm talking about
Before
apiVersion: k8ssandra.io/v1alpha1
kind: K8ssandraCluster
metadata:
  name: test
  namespace: k8ssandra-operator
spec:
  cassandra:
    datacenters:
    - config:
        jvmOptions:
          heapSize: 384Mi
      metadata:
        name: dc1
      size: 1
      cassandraTelemetry:
        prometheus:
          enabled: true
      stargate:
        telemetry:
          prometheus:
            enabled: true

After
apiVersion: k8ssandra.io/v1alpha1
kind: K8ssandraCluster
metadata:
  name: test
  namespace: k8ssandra-operator
spec:
  cassandra:
    datacenters:
    - config:
        jvmOptions:
          heapSize: 384Mi
      metadata:
        name: dc1
      size: 1
      telemetry:
        prometheus:
          enabled: true
      stargate:
        telemetry:
          prometheus:
            enabled: true

I think it is clear from the parent/child relationships or nesting of properties that cassandraTelemetry or telemetry applies to Cassandra. Other than the CassandraConfig property, we don't prefix any of the other property names with cassandra.
@jdonenine thoughts?

So, yaml still sucks...
Say we look at it like this...
spec.cassandra.datacenters[0].telemetry.prometheus.enabled = true

and
spec.cassandra.datacenters[0].stargate.telemetry.prometheus.enabled = true

I think in an ideal world, simply for consistency sake, I'd actually like to see cassandra represented as a level in the datacenter or stargate removed from it, like @Miles-Garnsey mentions.
In lieu of either of those things happening I don't think we need the special prefix, we don't prefix size with cassandraSize or whatever.
but we're talking semantics, I honestly don't really find either terribly abusive.

Whether its declared at the cluster or at the dc-level, I think it's pretty clear. Consider the size property for comparison. We're not calling it cassandraSize. This is the same for all other properties at the DC level. Currently we have:

serverImage
size
stopped
serverVersion
jmxInitContainerImage
config
resources
racks
networking
storageConfig
stargate
mgmtAPIHeap
cassandraTelemetry
softPodAntiAffinity
tolerations

In terms of naming cassandraTelemetry stands out as being less consistent with naming IMO.

Merge if you like @jsanda - but I will send every user who has a question on this topic in your direction... Until the end of time 😈

but I will send every user who has a question on this topic in your direction... Until the end of time

🤣
Let's hold off and let others chime in if they are so inclined.

I'm +1 on just using telemetry.
It looks understandable to me that cassandra.telemetry is telemetry for Cassandra and cassandra.stargate.telemetry is telemetry for Stargate 🙂

And I'm +1 using just metrics like everyone else in the industry..


Decide on a better long-term graphql server implementation

We are currently using graphql-go for our server in cloak because it is a mostly functioning implementation that supports dynamic loading of schemas (as opposed to gqlgen which is a high-quality implementation that doesn't support dynamic loading of schemas).
However, we have found a few bugs already with graphql-go:

aliases don't get resolved in parallel
introspection results are not sorted, change order every query

More may easily be lurking. While it is maintained to some extent, it's not particularly active (i.e. see https://github.com/graphql-go/graphql/issues/577)
Some alternative options:

Fix graphql-go upstream or fork it if strictly needed
Find a better server written in Go that supports dynamic loading of schemas (i.e. not gqlgen).

https://github.com/wundergraph/graphql-go-tools may be worth another look

Update (or, if needed, fork) a high-quality implementation like gqlgen to support our dynamic use case better
Use the typescript Apollo implementation.

Pros: High-quality, supports dynamic stuff we need.
Cons: not in go, which is okay because we can run it in buildkit but still complicates architecture and maintenance.

(Utter last resort, only feasible in the long term) Make our own go server??

cc @aluzzardi

We went with forking graphql-go for now, closing since we have no further plans at this time


The Linux Command Line (Chinese Edition)

http://billie66.github.io/TLCL/
The hybrid version in Chinese and English
http://billie66.github.io/TLCL/book/

Thanks!


Fix usage-count workflow.

Codecov Report

Merging #129 (15c9499) into master (8a41091) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #129   +/-   ##
=======================================
  Coverage   84.14%   84.14%
=======================================
  Files          15       15
  Lines         473      473
  Branches       49       49
=======================================
  Hits          398      398
  Misses         75       75

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 8a41091...15c9499. Read the comment docs.

Codecov Report

Merging #129 (15c9499) into master (8a41091) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #129   +/-   ##
=======================================
  Coverage   84.14%   84.14%
=======================================
  Files          15       15
  Lines         473      473
  Branches       49       49
=======================================
  Hits          398      398
  Misses         75       75

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 8a41091...15c9499. Read the comment docs.


fix bug on page reload

Uncaught TypeError: util.events.destructor is not a function

Thanks for finding and fixing this! This is such a simple change that I'm just going to accept it without a test. If you feel up for adding a test at some point (probably just testing that util.events.destructor can be called, and does remove event handlers) that would be awesome.


Export to PDF

Great extension! 😊
I was hoping for a way to export a notebook to PDF similar to Jupyter: Export to PDF.
Even after installing the Jupyter-Extension there is no command for exporting PDFs for nnb-Files.
Could you add this functionality?
All the best.

I have written a command line tool in typescript/deno which converts the output parts of a notebook to markdown:
https://github.com/aheissenberger/notebook-to-markdown
I think this is much better as it allows to use existing tools in the vscode ecosystem to create PDFs, HTML,...
I tried to create a more integrated solution with typescript-notebook, but failed to get the required packages installed on my M1 based MacOS - I fail with tensor flow package and after deleting the lock file with many more dependency problems :-(
Maybe someone with a "normal" system can copy my code as it is only a couple of lines.


Cannot install CF CLI error: gpg: no valid OpenPGP data found

Please fill out the issue checklist below and provide ALL the requested information.

[x] I reviewed open and closed github issues that may be related to my problem.
[x] I tried updating to the latest version of the CF CLI to see if it fixed my problem.
[ ] I attempted to run the command with CF_TRACE=1 to help debug the issue.
[ ] I am reporting a bug that others will be able to reproduce.

Describe the bug and the command you saw an issue with
Provide details on what you were trying to do (and why).
Install CF CLI during deployment workflow.
What happened
A clear and concise description of what happen.
Installation of CF CLI fails with error: gpg: no valid OpenPGP data found.
Expected behavior
A clear and concise description of what you expected to happen.
CF CLI installation works.
Exact Steps To Reproduce
Steps to reproduce the behavior; include the exact CLI commands and verbose output:
wget -q -O - https://packages.cloudfoundry.org/debian/cli.cloudfoundry.org.key | sudo apt-key add -
echo "deb https://packages.cloudfoundry.org/debian stable main" | sudo tee /etc/apt/sources.list.d/cloudfoundry-cli.list &&
sudo apt-get -y update &&
sudo apt-get -y install cf8-cli
Provide more context

ubuntu-20.04
cf8-cli

Following the workaround from here works, but I feel it's hacky: https://answers.sap.com/questions/13210328/installing-cf-cli-on-ubuntu-20041.html
I can install CF CLI v8 I add --no-check-certificate as below:
wget --no-check-certificate -q -O - https://packages.cloudfoundry.org/debian/cli.cloudfoundry.org.key | sudo apt-key add -
Can you please have a look at the issue anyway?

Maybe connected to https://github.com/cloudfoundry/cli/issues/2603 ?


Chrome Store Extension Not Found

I tried to install the extension to chrome earlier today (July 12, 2021), but I couldn't find in the webstore, and the google earch link goes to a non-existent page
Broken link
https://chrome.google.com/webstore/detail/saka-key/hhhpdkekipnbloiiiiaokibebpdpakdp?hl=en
Chrome Web Store Search Screenshot

Thanks in advance.

Same here, has the extension been removed? I was unable to load it on a new system

Same here :(
Chrome version: Version 92.0.4515.107 (Official Build) (64-bit)
OS: Linux Mint 20.1
I don't think the OS matters but  I add it just in case

For what it's worth, it looks like it violates Google's Web Store policy.

The extension was disabled for me automatically, but if I enable it manually it still works.
Curious what the problem is.

Today disabled on Microsoft Edge with the note that it is disabled because of malware. Impossible to force-enable it back.
Extension got probably flagged because it "listens" to keystrokes in order to make it work? Well, too bad, Lazyfox is the last resort now. Goodbye Chrome, Goodbye Edge :-)

Can anyone confirm it definitely hasn't been compromised etc? I'm paranoid after recent events with the great suspender etc

Would anyone know how can you re-enable the extension on Edge? It seems like once it got blocked, you cannot reactive it unlike in other chromium browsers.

Might not be able to re-install it

I wonder what happened that it started getting flagged in all those stores
also, i tried forcing enable it on Vivaldi - but the extension doesnt work properly anymore

Just shows how bad the removal process is. Google has also removed the extension from my browser, without any notice, or even a message in-browser. Its as if this addon never existed. I don't mind removal of something malicious but jeez, tell us if this was compromised and if we need to do anything
In the meanwhile, I've found Vimuim extension to be a great, close fit with a large install base

Would anyone know how can you re-enable the extension on Edge? It seems like once it got blocked, you cannot reactive it unlike in other chromium browsers.

Is there any way to see what triggered the removal? Any reason to think this extension is actually a risk?

The Extension was removed from Chrome and Edge, I found an alternative, it's called Vimum, it works on edge of course.
https://chrome.google.com/webstore/detail/vimium/dbepggeogbaibhgnhhndojpepiihcmeb?hl=en
This is my current configuration which is inspired by the default Saka Key configuration, it's really easier to customize, and I found it more effective with hidden/visible html elements and overlays.
unmapAll
map d scrollDown
map s scrollUp
map D scrollPageDown
map S scrollPageUp
map fb LinkHints.activateModeToOpenInNewTab
map ff LinkHints.activateMode
map [ previousTab
map ] nextTab
map xx removeTab
map cc goBack
map vv goForward

I did unmapAll optionally, because I was annoyed by the behavior of default keys
Hope this helps until Saka Key is -- ever -- back

Thanks @ahmed-alamer , this is a great idea to Saka-fie the Vimium :-)
I wrote email to on of the developers, if I ever get response, I will let you know.
For the advanced users - Saka Key can still be packed and installed manually in the developer mode, have anybody tried it?

I have added j and k, which I use more often for some reason :-)
unmapAll
map d scrollDown
map s scrollUp
map j scrollDown
map k scrollUp
map D scrollPageDown
map S scrollPageUp
map fb LinkHints.activateModeToOpenInNewTab
map ff LinkHints.activateMode
map [ previousTab
map ] nextTab
map xx removeTab
map cc goBack
map vv goForward

Thanks @ahmed-alamer @athelas64 too for the tips.
Well, I did a bit tests of a Sake-fied config of Vimium on Chrome. It's ok for the new or advanced users, but some functions of Saka Key aren't there.

No way to open a link in a new window.
No way to just get the focus on a link w/o activating it. As for example only to see the URL behind.

It's possible to re-enable the installed Saka Key extension on my Chrome browser (no dev mode). Then it works for some time, but later, maybe on the next update check, it get automaticaly disabled and I have to enable it again.

Thanks @ahmed-alamer @athelas64 too for the tips.
Well, I did a bit tests of a Sake-fied config of Vimium on Chrome. It's ok for the new or advanced users, but some functions of Saka Key aren't there.

No way to open a link in a new window.
No way to just get the focus on a link w/o activating it. As for example only to see the URL behind.

It's possible to re-enable the installed Saka Key extension on my Chrome browser (no dev mode). Then it works for some time, but later, maybe on the next update check, it get automaticaly disabled and I have to enable it again.

I don't think opening a link in a new window is supported, there is a guide on the options page if you click on show available commands

Thanks @ahmed-alamer @athelas64 too for the tips.
Well, I did a bit tests of a Sake-fied config of Vimium on Chrome. It's ok for the new or advanced users, but some functions of Saka Key aren't there.

No way to open a link in a new window.
No way to just get the focus on a link w/o activating it. As for example only to see the URL behind.

It's possible to re-enable the installed Saka Key extension on my Chrome browser (no dev mode). Then it works for some time, but later, maybe on the next update check, it get automaticaly disabled and I have to enable it again.

I don't think opening a link in a new window is supported, there is a guide on the options page if you click on show available commands

In Saka Key there is Link Hint to open in new widow with "fn", but not such command in Vimium. As a workaround in Vimium is the command 'moveTabToNewWindow'. However a link hint command to focus a link isn't existend in Vimium.

so this is basically dead for chrome?

@pureooze Any change we could get your help here? I see you are the most active member of the org lusakasa.

@adamryman I believe the issue is with the permissions that saka key is using. Seems google doesn't like the usage of the tabs permission. I have no experience with the saka key codebase so I am not sure if its needed or can be easily removed. If someone wants to take on that work I can help push it through to the respective stores.

@pureooze https://github.com/lusakasa/saka-key/pull/334
let me know if that does the trick 🤞

I am testing/reviewing the PR written by @ceuk. Let's get back in the chrome store.

Hi @marcoep, nothing has happened in the last few months. It seems the maintainers have become busy, which is very understandable these days.
I called them out directly in PR #334 a little bit ago, perhaps we will hear back.
If we don't hear back in the next month or so, perhaps we can create a community fork of this repo. Then we could submit it to the chrome web store under a different name. The LICENCE of saka-key is MIT, so no worries there.
Though I hope that our friendly maintainers are able to get back to us soon. Then we would not need to fork this repo to get a saka-key based extension back in the chrome store.

I'm using Vimium https://chrome.google.com/webstore/detail/vimium/dbepggeogbaibhgnhhndojpepiihcmeb with these custom mappings:
unmapAll
map d scrollDown
map s scrollUp
map D scrollPageDown
map S scrollPageUp
map fb LinkHints.activateModeToOpenInNewTab
map ff LinkHints.activateMode
map [ previousTab
map ] nextTab
map xx removeTab
map cc goBack
map vv goForward

I don't have a whole lot of time available to maintain this but I am willing to create a community fork with others.
I've noticed my in-browser ad blocker (Vivalidi) has broken Saka key so maybe something fishy is going on with the published extension but I haven't investigated that fully.