Undeprecate load locations

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

There was a CHANGES entry related to this change which is now misleading. I think the references to the deprecation should be removed from that entry.

I don't think it is appropriate to ever edit a CHANGES entry - I could add a new one indicating the undoing of the deprecation if you would like - but editing the change history seems like a slippery slope to go down.

I don't think it is appropriate to ever edit a CHANGES entry - I could add a new one indicating the undoing of the deprecation if you would like - but editing the change history seems like a slippery slope to go down.

I totally disagree. If it were in a published release I would agree. Once published the CHANGES record should be an accurate reflection of all the changes that have occurred since the last release. People should be able to read that list to gain an understanding of important differences between releases. Contradictory entries would be very confusing. The CHANGES document is not a record of the history of the development. That is what the git log is for. It is a list of all the CHANGES since the last release. If we alter those changes before we release then we should absolutely feel free to edit the CHANGES document.

Commit added - wasn't thinking correctly about the CHANGES context.

I frankly don't understand why we should maintain the functions that are replaced.  Do we want to retain them "permanently"?  Why?
Deprecation is, after all, only a marker that things will eventually go away, with a compiler warning thereof.  NOT deprecating can be seen as a promise that the old functions will continue to be maintained "indefinitely".

Because we have users who use those APIs and get zero benefit from changing from one name to a slightly different one as views change over time. It was documented and the functionality remains.
Depreciation warnings should be for things that need to change where users need to migrate.
Swapping one name for another and adding a NULl check around the new version is a pointless exercise for the user.
This one is unlike the majority of the others that have been deprecated where we want the users to move off to a better alternative so we can clean up or restructure or get users out of out internals. This certainly isn't that situation.

Swapping one name for another

There's a little bit more than a mere name change in this API change.

This one is unlike the majority of the others that have been deprecated where we want the users to move off to a better alternative so we can clean up or restructure or get users out of out internals. This certainly isn't that situation.

And this is where you and I disagree.  I do think that the itemized functions are the better alternative, and very clearly are the only choice as soon as new location forms (such as a URI for use with OSSL_STORE functionality) is to be passed.

I have no issue with adding the itemised functions. But deprecating something that is widely used is not the right approach - this is not what we do to our users without a very good reason - and that isn't this case.  It doesn't mean you force users to change to calling two other functions simply because there are now other options which they user would also have to call.

So pile the functions on, keep the "replaced" ones permanently (as far as we can see for the moment), is that the basic message?  ...  within the scope of these fairly simple things, there are other parts of OpenSSL where I assume we can justify the need much more strongly.
Boy, this will be messy, but ok, I can align with that line of thinking.  There are bigger fish :wink:

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

.... and close


Add debian packaging

For now this is just the basic cargo-deb config.
@ximon18 can you give some pointers regarding the packaging, changelog, GH actions and publishing this in our repository along side the krill and routinator packages?
(not urgent, next week is fine)

Recently for NLnetLabs/ROVer I copied and adjusted Cargo.toml entries, debian/ directory and GH workflow from the other projects (Routinator or Krill) and edited it. Had it working within an hour or so. We can do the same here.


read_json() got an unexpected keyword argument 'layout'

Hello every body,
my question is maybe very simple but it is problem for me while i have no idea of programming -_- . so any help or tips will be appreciated , thanks !
i have a very simple code that must covert a json file into spss for me :
import spss_converter
as_spss = spss_converter.from_json('/home/user/file.json',
target = '/home/user/my-spss-file.sav',
layout = 'table')
but if i run the code with :    python3 spss.py
i get the following error:
TypeError: read_json() got an unexpected keyword argument 'layout'
I run it on Ubuntu ( Python 3.8.10 )
Thanks for your help !

Hi @mazbeh : Apologies, it looks like there's an error in the documentation. Rather than layout = 'table', you should pass the argument orient = 'table'.
If you take a look at the documentation for the .from_json() function (see here), you'll find that it only expects:

as_json - the location of your JSON file
target - where you want your SPSS file to go
compress - a True/False flag as to whether you want the data in ZSAV (compressed) format or standard SAV (uncompressed) format

All other keyword arguments - like in this case orient - are passed directly to the pandas.read_json() function, so you can look at those options in the Pandas documentation here.
I'll fix the SPSS Converter documentation and re-release shortly.

Looking through the API and thinking about it a little more, I realized that it will be more consistent with the .to_* functions if I explicitly add support for the layout parameter to .from_json() and .from_yaml(). So that's what I will do, and I'll re-release a new version later today. In the meantime, you can feel free to use orient = 'table'` rather than layout = 'table'`` and it should work fine for you.
I'll re-close this issue when the next version is released.

Okay, I've released v.0.2.0 which addresses this issue.


As a developer, I need to know when I've introduced code that significantly reduced the performance of cellxgene.

Story: As a developer, I need to know when I've introduced code that significantly reduced the performance of cellxgene. A "selling" point of cellxgene is that scientists can interact with their datasets at interactive speed. As a developer, I need to know when I have detracted from that experience so that I don't spend time later down the line debugging the performance issues which are sometimes tricky to find.
Need: As a scientists I want to continue having interactive speed of cellxgene even as my datasets get larger to up to a few million cells.
TL;DR: Introduce a performance test suite to cellxgene.

Closing since all issues have been addressed.


0.5.0 overhaul

Previously: #1119, #1184.
Resubmitting to set me as an author again.

Codecov Report

Merging #1185 (32cd765) into develop (cca9786) will decrease coverage by 0.75%.
The diff coverage is 69.92%.

:exclamation: Current head 32cd765 differs from pull request most recent head 8477b0c. Consider uploading reports for the commit 8477b0c to get more accurate results

@@             Coverage Diff             @@
##           develop    #1185      +/-   ##
===========================================
- Coverage    59.24%   58.49%   -0.76%
===========================================
  Files           98       82      -16
  Lines         4498     4115     -383
===========================================
- Hits          2665     2407     -258
+ Misses        1833     1708     -125

Impacted Files
Coverage Δ

...uiggle-lang/src/rescript/Reducer/Reducer_Value.res
39.32% <0.00%> (ø)

...ckages/squiggle-lang/src/rescript/FR/FR_Danger.res
62.96% <30.43%> (ø)

packages/squiggle-lang/src/rescript/FR/FR_Date.res
36.14% <36.14%> (ø)

...kages/squiggle-lang/src/rescript/FR/FR_Scoring.res
65.00% <60.00%> (ø)

...e-lang/src/rescript/ForTS/ForTS_ReducerProject.res
61.53% <60.00%> (-2.47%)
:arrow_down:

...ript/FunctionRegistry/FunctionRegistry_Helpers.res
57.14% <63.09%> (+2.97%)
:arrow_up:

...s/squiggle-lang/src/rescript/FR/FR_GenericDist.res
63.21% <63.21%> (ø)

...gle-lang/src/rescript/Reducer/Reducer_Bindings.res
64.70% <64.70%> (ø)

...gle-lang/__tests__/Reducer/Reducer_TestHelpers.res
52.63% <66.66%> (-8.24%)
:arrow_down:

...le-lang/src/rescript/Reducer/Reducer_Namespace.res
66.66% <66.66%> (ø)

... and 45 more

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


HelloSign.get_signature_requests does not work as expected

HelloSign.get_signature_requests is supposed to take an array of signature_ids and return the status of those. But the actual behavior is that it returns the status of all documents in my hellosign account.
For eg:  Following code fails since I do not have '42383e7327eda33f4b8b91217cbe95408cc1285f' in my account. This is from example in the documentation. But if I use HelloSign.get_signature_requests it returns all data in my account.
pry(main)> signature_request = HelloSign.get_signature_request :signature_request_id => '42383e7327eda33f4b8b91217cbe95408cc1285f'
HelloSign::Error::NotFound: Server responded with code 404
Request URI: https://api.hellosign.com/v3/signature_request/42383e7327eda33f4b8b91217cbe95408cc1285f

Hi @nishabn -
Actually, the get_signature_requests method will list the signature requests you've sent through the API, and can be called without any arguments.
Optionally, you can supply a parameter hash specifying a :page of results to fetch, and a :page_size - parameters that are useful when you have large numbers of results.
We don't currently have an endpoint that checks the status of multiple signature requests as you mention -- you'd have to either get all your requests with the above function and filter, or iterate and use get_signature_request (note the singular "request") to check each.
I'd be curious to know where you saw a reference to using get_signature_requests to check status, and providing an array as parameters, as I don't think this has ever been supported. Please let us know -- we be able to clear up any documentation that's confusing around that.
Thanks much!
//Derek
[Engineer @HelloSign]

Ah  I see. The documentation here https://github.com/HelloFax/hellosign-ruby-sdk has reference to some thing like that. The comment says specific request but the api method used is get_signature_requests.

you'd have to either get all your requests with the above function and filter, or iterate and use get_signature_request (note the singular "request") to check each

Well yes, but would be nice to have array of signature_ids. Given that we might have lot of forms, doesn't scale to iterate through each of them. Same applies to get_signature_request as well. Do you have plans to support something like that ? How easy/difficult is to support such an endpoint ?
-Nisha

Hi @nishabn -- sorry for the slow response.
Offhand I don't think it makes sense for us to create such an endpoint, since you can just use get_sginature_requests as in the example in Usage to grab all of your signature requests, and just filter to the ones you want locally.
The only use case I could see would be if you had a very, very large number of requests (too many to retrieve all of them at once) and wanted to retrieve a small, specific subset of them -- is this what you're experiencing?
Let me know and I can check into it.


OpenAPI Tool Omits Types Defined in Modules from the Generated Specification

Description:
When generating the OpenAPI spec for the service that has input parameters defined in a sub-module, it completely omitted the types defined in the sub-modules in the resulting specification file.
Steps to reproduce the issue can be found here: https://discord.com/channels/957996897782616114/1151957139158478898/1151957139158478898

Fixed via https://github.com/ballerina-platform/openapi-tools/pull/1564


Hive CRUD smoke tests

Added Hive and Hive2 tests which covers:

Oozie Hive and Hive2 actions
Creation of Database, Tables in Hive using JDBC and thrift server.
Insertion of data into Hive table,
Clean up of test data once test is completed.
Run tests in parallel.

Though not your sins, please run rubocop one more time as I still see some stuff in what you are touching that could be cleaned up.
As to the fork to run the CRUD test twice, simultaneously; why run the same test twice on two differently named databases?

Looks good. Reviewed.


Conversion of a Varchar datatype error on backfill

args provided
--vars "{start_date: '2021-01-01', stop_date: '2024-11-31', backfill: true}"
  ('22007', '[22007] [Microsoft][ODBC Driver 17 for SQL Server][SQL Server]The conversion of a varchar data type to a datetime data type resulted in an out-of-range value. (242) (SQLExecDirectW)')

Came from this line of code:
WITH data AS (
            SELECT

                '2021-01-01' AS start_timestamp, -- in backfill mode we want to start with this date
                COALESCE(nullif('2024-11-31','none') ,
                         CURRENT_TIMESTAMP ) as stop_timestamp

        )
        SELECT
            start_timestamp,
            stop_timestamp,
            datediff(
        month,
        start_timestamp,
        stop_timestamp
        )  + 1   AS num_periods
        FROM data

Root cause:
SELECT
COALESCE(nullif('2024-11-31','none') ,
                         CURRENT_TIMESTAMP )

Suggestion -> edit that.
Version:
dbt 1.8.
ibtp = 0.2

Seems to be somewhat ODD internal synapse behaviour in that the first of these examples doesn't error, but the 2nd does...
SELECT COALESCE(NULL, '2024',1)

SELECT COALESCE(NULL, '2024-11-31',1)

(note the bug remains when you put a valid date in!)


Bundling for lazy loading is broken: No module factory available

After upgrading CLI to beta.24 my app that uses lazy loaded module started giving me errors during ng serve:
NG Live Development Server is running on http://lo 60% building modules 515/611 modules 96 active .../compiler/src/i18n/serializers/xtb.jsError: No module factory available for dependency type: ContextElementDependency
at Compilation.addModuleDependencies (/Users/yfain11/Documents/Farata/_Training/courseware/angular2/handouts/12-hr-intro/unit2/router-samples/node_modules/webpack/lib/Compilation.js:188:20)
at Compilation.processModuleDependencies (/Users/yfain11/Documents/Farata/_Training/courseware/angular2/handouts/12-hr-intro/unit2/router-samples/node_modules/webpack/lib/Compilation.js:177:7)
at /Users/yfain11/Documents/Farata/_Training/courseware/angular2/handouts/12-hr-intro/unit2/router-samples/node_modules/webpack/lib/Compilation.js:317:12
at /Users/yfain11/Documents/Farata/_Training/courseware/angular2/handouts/12-hr-intro/unit2/router-samples/node_modules/webpack/lib/Compilation.js:124:4
at Array.forEach (native)
at callback (/Users/yfain11/Documents/Farata/_Training/courseware/angular2/handouts/12-hr-intro/unit2/router-samples/node_modules/webpack/lib/Compilation.js:123:12)
at /Users/yfain11/Documents/Farata/_Training/courseware/angular2/handouts/12-hr-intro/unit2/router-samples/node_modules/webpack/lib/Compilation.js:147:10
at ContextModule. (/Users/yfain11/Documents/Farata/_Training/courseware/angular2/handouts/12-hr-intro/unit2/router-samples/node_modules/webpack/lib/ContextModule.js:93:3)
at /Users/yfain11/Documents/angular-cli/packages/@ngtools/webpack/src/utils.ts:11:13
at /Users/yfain11/Documents/angular-cli/packages/@ngtools/webpack/src/plugin.ts:196:124
at ContextModule.resolveDependencies (/Users/yfain11/Documents/angular-cli/packages/@ngtools/webpack/src/utils.ts:5:9)
at ContextModule.build (/Users/yfain11/Documents/Farata/_Training/courseware/angular2/handouts/12-hr-intro/unit2/router-samples/node_modules/webpack/lib/ContextModule.js:74:7)
at Compilation.buildModule (/Users/yfain11/Documents/Farata/_Training/courseware/angular2/handouts/12-hr-intro/unit2/router-samples/node_modules/webpack/lib/Compilation.js:127:9)
===
I downgraded CLI to beta.22-1 in devDependencies in package.json. The error remained the same.
After downgrading to beta.21 the ng serve and the app work fine again.

I'm going to have to ask you to fill in the issue template, complete with repro steps. There just isn't much I can figure out just by looking at that log.

To reproduce, download this project:
https://github.com/Farata/angular2typescript/tree/master/chapter3/cli_version/router-samples
ng serve builds a simple app with the lazy loading of one module (LuxuryModule), and it runs fine. Currently it uses Angular CLI beta.21.  Change the version of Angular CLI to beta.24 in package.json and do npm install.
Try ng serve again, and you'll see this error.

I had same issue to get it working I had to run it with npm start instead of ng serve

@viltsu this definitely isn't fix as npm start is just alias for ng serve, nothing more.

Same error for me. Any news ?

After reporting this issue I split the bootstrap code (main.ts) and the app.module.ts into separate files and this error was gone.

you can try
https://github.com/angular/angular-cli/issues/4246#issuecomment-369477979


utils_13_diagnosis_sbc is missing

I am following the Simulation-based Calibration in SBI.
module utils_13_diagnosis_sbc seems is missing.

it's actually here: https://github.com/mackelab/sbi/blob/main/tutorials/utils_13_diagnosis_sbc.py


Integrating data tracing in e2e tests

Problem
The inclusion of tracing data is essential for the testground sanity test, primarily for precise bandwidth utilization comparisons. Currently, our e2e tests do not incorporate this feature.
Acceptance Criteria
This issue aims to implement the required modifications to the e2e tests, enabling them to trace and either download data from the Knuu instances or upload it to an external server.

to update this, we are just waiting to merge some existing work that integrates the tracing data. PR incoming


Provided keys by a plugin are inferred with type unknown

Environment
Nuxt project info:

Operating System: Linux
Node Version:     v18.20.3
Nuxt Version:     3.13.0
CLI Version:      3.13.1
Nitro Version:    2.9.7
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      compatibilityDate, devtools, future
Runtime Modules:  -
Build Modules:    -

Reproduction
https://stackblitz.com/edit/github-6knne9?file=app%2Fcomposables%2FuseHello.ts
Describe the bug
Issue
NuxtApp injections are not inferred correctly
// alternatively, you can also use it here
const { $hello } = useNuxtApp()

// $hello: unknown
</script>

Screenshots

Additional context
No response
Logs
No response

We are facing the same problem trying to upgrade to 3.13.0

A similar issue was already raised https://github.com/nuxt/nuxt/issues/28693 I believe the fix is still to be released

This is already resolved in the latest version of nuxt/devtools:

A hotfix for libraries that haven't updated to the latest module-builder is due to be released in v3.13.1.

Thank you

Hey @danielroe we still have this problem and I could reproduce it by adding the @nuxt-alt/auth module to the repro project indeed

I don't think they use module-builder but should we still expect this to be resolved with nuxt 3.13.1 or do you think I should open an issue on the module repo?

Yes, you should open an issue upstream, but the next Nuxt patch should hotfix it for now.


AS Arena Bug/Abuse

Hi,
There is a way to abuse arena if you don`t level up. I believe that this is only relevant for academy students as later on exp requirements are higher.
If a player does not click on profile button it leads to player not lvling up. User can stay lvl1 and just keep beating lvl1 arena opponent. With a little training user can soon 1hit enemy and grind faster to genin, roll BL, if unlucky, start again. I guess this makes road to genin faster than intended.

Level holding is an intended feature. You should be able to be 5 levels below your max level. This can be done by not visiting your profile or toggling the level hold feature in user settings.

Yes, but you can be level 1, while you have enough experience for level 10. Doesn`t this mean that you are 10 levels below max level?

This is still an (small) issue. It's possible to hold levels at a greater value than intended.
Proposed solution:

Level up on any page

Convert level up messages from raw html header to notification


GitHub becomes GitHub Desktop

Thank you for the contribution. It was merged directly as 890ae85c93f9df17fd9fc3e7b5c35c6e538e2e95 to keep commit history cleaner. Your contribution is still credited to you.


shutdown_and_then_consume_events API change

Changed the signature of shutdown_and_then_consume_events to allow for ignoring warnings or errors.
Happy for better suggestions regarding the API as long as functionality remains the same.
pub async fn shutdown_and_then_consume_events(
    self,
    expected_errors: Option<&[EventMatcher]>,
    expected_warnings: Option<&[EventMatcher]>
) -> Events

I believe you can already do this via EventMatcher.
e.g.
# allow any warnings
shutdown_and_then_consume_events(&[EventMatcher::new().with_level(Level::Warn)])

# allow any errors
shutdown_and_then_consume_events(&[EventMatcher::new().with_level(Level::Error])

# allow any errors or warnings
shutdown_and_then_consume_events(&[EventMatcher::new())

Does this work for you?


Handle js localStorage exceptions

This patch should fix https://github.com/go-humble/locstor/issues/2

@albrow In case you didn't notice - I updated the PR last weekend.  Hopefully it's OK now, let me know if you need more changes.

@McKael thanks for updating this PR. Unfortunately my main development computer has been out of commission for a while and I am unable to test this change. I'll likely be up and running again in the next week or so.

:+1: for this PR

@McKael sorry for the delay. I was able to get my dev environment running again and verified the tests still pass. I can't think of a way an automated way to test the behavior when local storage is not supported, but I think it's okay to go ahead and merge for now.
Thanks again!


Binary Flashing guide

Hello,
I was trying to flash the binaries to my watchy but can't manage to do so. Could you update the readme with some intructions on how to setup the project.
I've tried to upload it following this guide esp-idf but didn't manage to make it work as I keep getting Cmake issues.

Don't worry i figured it out, for some reason the .gitmodules didn't download all the dependencies.


Feature Request - Updated to 1.10.2!

@LakMoore
If you are having issues getting started contact this mod author and ask modding questions! Just try and be patient as he tends to do a lot at a time.

Please don't contact me the message was for the creator of this mod not you @majix560

What I don't like is that you kept the suggestion that @LakMoore contact Thut when you edited it after Thut told you not to.

Not sure what is going on up there ^^^ but I can say that I have no current plans to update this 1.10.2.
I would like to do so but have no time to complete the work.  If someone wants to fork this git and either do the work or make a start that is fine and I may even chip in but I don't want to get your hopes up for me to do it any time soon.  Sorry.


Exclude polyfills from build

Hi,
could you exclude the polyfills from the build, It is recommended in official vue docs.
It could reduce the size by quite a bit.

Closed because it was fixed in d09270143ab3f44df422a14c0a83b73f3ebe81c3


BUG: Dark Mode has various differences in style

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
The style of site is not good in visual when mode is changed from light to dark
@anuragverma108 kindly assign this issue to me
Added Screenshots for reference :

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@anuragverma108 assign this issue to me

@anuragverma108 assign this issue to me
After correction:

assigned!

@huamanraj sir the time I raised this issue there was one less section, addition of a new section changed the color property of other section therefore I made changes accordingly.
Adding screenshot for reference:
Before:

After:

if all okay?
can i raise the pull request
In addition to this I tried to improvised the navbar
before:

After:

@huamanraj sir the time I raised this issue there was one less section, addition of a new section changed the color property of other section therefore I made changes accordingly.
Adding screenshot for reference:
Before:

After:

if all okay?
can i raise the pull request
In addition to this I tried to improvised the navbar
before:

After:

awesome work @ChhaviRohilla 👏!
raise a PR and dont forget to do changes in the latest codebase other it can lead to conflicts!
also tag me in PR!


Submit group YxBanpu

design PLC  => design PCB
.DS_Store should be removed.
Link to EasyEDA project should be included.
Sonnet Simulation files should be included.
Any related files should be included.

The legend should be included or the figure caption should explain the cases associated with colors in the image in step 3. You have not got the PCB back, have not measured the PCB. What is "the real thing"? Have you place the order? Include the link to your project so I can check your PCB design. design PLC => design PCB .DS_Store should be removed. Link to EasyEDA project should be included. Sonnet Simulation files should be included. Any related files should be included.

Already deleted .DS_Store


Ensure that required symbols are available to FFI even when the final binary   is linked with -dead_strip

[X] I’ve reviewed the contributor guide and applied the relevant portions to this PR.

Contribution guidelines:

See our contributor guide for general expectations for PRs.
Larger or significant changes should be discussed in an issue before creating a PR.
Contributions to our repos should follow the Dart style guide and use dart format.
Most changes should add an entry to the changelog and may need to rev the pubspec package version.
Changes to packages require corresponding tests.

Note that many Dart repos have a weekly cadence for reviewing PRs - please allow for some latency before initial review feedback.

coverage: 88.823% (+0.01%) from 88.809%
when pulling a11dbe924d79a77f4bc3890ca0c77e9832b39fa7 on brianquinlan:make_symbols_visible
into 4ce481621c224dc5b7d86e2589c612f0624f0e44 on dart-lang:main.

We'll probably need to revisit this when we implement tree shaking. This is fine for now though.
cc #1099

I'm going to check that this works in a real app before merging.

I'm going to check that this works in a real app before merging.

It looks like the missing symbols problem has been replaced by https://github.com/dart-lang/http/issues/1413 but this PR is fine.


[#3498] Add prone separately for sleeping/unconscious

Closes #3498.
Adds a riders property for add-on conditions that should be applied separately.

Seems a bit niche for the added complexity. Are there any other places we would use this?

Seems a bit niche for the added complexity. Are there any other places we would use this?

If the question is whether adding this is worth it, I personally find myself adding the prone condition separately, on top of applying Unconscious when hitting zero hp, because I know (but others might not know) that the Prone condition would otherwise (incorrectly) remove itself when someone slaps a healing potion in my face. I think it is a worthwhile QOL feature.
As to why I decided to generalize it in this way? I cannot think of reasons as to why not.

Definitely something I found myself wanting when playing last weekend when I knocked my players unconscious about a dozen times :).


Show balance of accounts

We should show the free balance of all the important accounts, on the same line:

Pure proxy
Multisig

To ease things up we can use api.derive.balances.account and formatBnBalance already in use in the code base.

hi I like to work on this issue, can you clarify where do you want to show the balance of accounts

It should be visible for Pure and Multi accounts. On the right after their name/address.

is this look ok, in terms of implementing the accounts balance, do i need to also show the balance under the delegate

Thanks, we can do something in terms of design, but it's a start.

the balance should only be visible for the multisig account, not the signatories.
it should only be visible on the left, not in the transaction list (and it shouldn't replace the name or the address)
please see the balance implementation, using formatBnBalance to show a meaningful information.


Fix GHD not opening repo in file manager

shiftkey/desktop#870

Before looking for the bug, I'll test the original dotfiles on a new user. If the issue persists there, I'll move this issue to upstream.
I doubt the original has this bug, but it's worth testing

Surprisingly, the bug also happens when installing the original .files.
Moving issue...

This is a bug in the GHD AppImage. won't-fix


iOS: check('mediaLibrary') always return undetermined

Describe your environment

React-Native: 0.54.4
Platform: iOS
Device iPhone 6S+
OS: 11.3.1
react-native-permissions: 1.1.1
Devtools: Xcode 9.3.1

How to repeat issue and example

Check permission. It always returns undetermined
Try request permission. It return authorized if user allow or deny, if user deny request OR blocked permission in settings.

  Permissions.check('mediaLibrary')
    .then((accessCheckResult) => {
      console.log('[permissionHelper] check:', accessCheckResult);
      Permissions.request('mediaLibrary')
        .then((requestPermissionResult) =>
          console.log('[permissionHelper] request:', requestPermissionResult))
    });

So, how to determine is user deny request or blocked permission in settings earlier ?
Solution
No

Hello, I am also facing this issue. Does anyone have solution?

I am closing this since this issue does not exists anymore with the 2.0.0 release.


Allow reusage / caching of popper instance for different references

Allow modifing the references of a popper instance (within a plugin).

I will reuse the popper instance and modify the reference.
This could be possible though a plug modifing the state.elements.reference, but does not work because state.scrollParents calculation uses the original reference argument, instead the parameter of the state.

This must be fixed by the core, to support 3rd party modifiers to update the reference properly.
Two workaround exsists:

also update state.scrollParents within the plugin => disadvantage: to be save, setOption must also be overwritten by the plugin to prevent setOption calls with the initial reference
wrap the element in a virtual Element => disadvantage: boilerplate

Alternative implementation:
Add an updateReference/setReference Methode to the instance. This will be the cleanest solution and needs no 3rd party plugin.

It is feasible. This is how it is currently done, because we have no other choice.
We use popper to display many shortcuts for different elements on pressing ALT. So we caching the elements and the logic behind, but not the popper instances.
Another option is to wrap the reference outside of popper with a virtual element. But this is more unnecessary code as the modifications of popper js.
The reference within popper is saved within the state here
https://github.com/popperjs/popper-core/blob/2e909c71b2d09582a373aaa697152012016735ad/src/createPopper.js#L63-L66
and here by setOptions
https://github.com/popperjs/popper-core/blob/2e909c71b2d09582a373aaa697152012016735ad/src/createPopper.js#L91-L98
And used here ...
https://github.com/popperjs/popper-core/blob/2e909c71b2d09582a373aaa697152012016735ad/src/createPopper.js#L175
https://github.com/popperjs/popper-core/blob/2e909c71b2d09582a373aaa697152012016735ad/src/createPopper.js#L235
https://github.com/popperjs/popper-core/blob/2e909c71b2d09582a373aaa697152012016735ad/src/createPopper.js#L277
If setOptions can also updates the state.reference and state.scrollParents.reference uses the state.reference instead of the initial reference parameter, I think it should work.

Destroying and recreating the popper instance is a fast operation, which means you don't need to cache. Updating the reference in this case would be virtually equally as expensive as destroying & re-creating because it needs to undergo the same steps anyway. Is this to do with the speed or just the setup of your code?

We have no trouble. It's only a speed optimization request.
In the following code the comp reference can change. This is the only reason, why I will update the reference within popper.
Hiding instead of destroying can be done with a modifier if necessary and if the reference in popper is updateable.
Code Snippet: showing mnemonics/shortcuts
mnemonicsElements.forEach((comp) => {
    const { mnemonic } = comp.dataset;
    const mnemonicElem = this.#updateMnemonic(mnemonic);
    this.#popperInstances.push(
        createPopper(comp, mnemonicElem, {
            placement: 'top-start',
            modifiers: [hide],
        }),
    );
});

Code Snippet: hiding mnemonics/shortcuts
this.#popperInstances.forEach((popper) => popper.destroy());
this.#popperInstances = [];

Hi, we're closing all feature requests related to Popper as it is entering low maintenance mode. All feature requests should now relate to the new package, Floating UI!


Remove AdditionalProperties dictionary interface

Description

Fixes the parts of https://github.com/Azure/autorest.csharp/issues/1232 that we are hashing out now.
Remove AdditionalProperties's IDictionary interface and make the field public

Checklist
To ensure a quick review and merge, please ensure:

[x] The PR has a understandable title and description explaining the why and what.
[x] The PR is opened in draft if not ready for review yet.

If opened in draft, please allocate sufficient time (24 hours) after moving out of draft for review

[x] The branch is recent enough to not have merge conflicts upon creation.

Ready to Land?

[ ] Build is completely green

Submissions with test failures require tracking issue and approval of a CODEOWNER

[ ] At least one +1 review by a CODEOWNER
[ ] All -1 reviews are confirmed resolved by the reviewer

Override/Marking reviews stale must be discussed with CODEOWNERS first

Hey @pakrym - Not sure what you meant by your "might need to change it, we have a real property" comment on the commit.

I suspect you might need to fix some tests.

Moving back to draft as a test failure showed me that it is incomplete.

Don't land this until after the next next autorest bump.

Wait for next week.

Decided to land this week and not land bump PR until next.


Command line --mailStore option doesn't work.

java -jar dumbster-<IP_ADDRESS>-SNAPSHOT.jar --mailStore=mailstore 4039

Above command results into help text. It should start dumbster with mailstore.
We want to use dumbster by running with command line. And want to retrieve email from the mailstore.

Note that that isn't what the --mailstore option means. The option is about what implementation class dumbster should use for its mailstore, not a file where there are a bunch of email messages already stored.


_run_hook_from_repo_dir should not remove files if they already existed before

The current behavior makes --overwrite-if-exists remove all the files (undesirable) if some hook fails.

I'm not sure if this is the same issue, but I have had directories with content already in them completely disappear twice when using --overwrite-if-exists.  The last example:
$ cookiecutter --overwrite-if-exists devops-pyapp
project_namespace [filebrowser]:
project_dirname [filebrowser-src]:
python_version [3.5]:
python_tox_envname [py35]:
python_executable [python3.5]:
gh_repo_path []:
gh_repo_name []:
circle_badge_token []:
codecov_api_token []:
codecov_badge_token []:
Unable to create file 'tox.ini'
Error message: 'dict object' has no attribute 'python_tox'
Context: {
    "cookiecutter": {
        "circle_badge_token": "",
        "codecov_api_token": "",
        "codecov_badge_token": "",
        "gh_repo_name": "",
        "gh_repo_path": "",
        "project_dirname": "filebrowser-src",
        "project_namespace": "filebrowser",
        "python_executable": "python3.5",
        "python_tox_envname": "py35",
        "python_version": "3.5"
    }
}

After this happens, my filebrowser-src directory is completely wiped out.  I've been forced to restore from backups b/c I had local-only configuration files get deleted that I didn't want to have to recreate.
IMO, this is really bad.  I want to use CC to add devops files to already existing application repos.  But my devs run the risk of deleting their app directory if something in the config goes wrong, that's a hard sell for me.
I can work on a PR if someone can confirm _run_hook_from_repo_dir is the problem and commit to reviewing and merging the PR.  With 33 open PRs, I'm hesitant to start a new one.  That's not a criticism, I realize that's how OSS goes sometimes.  I just don't want to spend on the time on the PR until this issue is acknowledged.
Thanks.

@rsyring I don't think that the rmtree in generate. _run_hook_from_repo_dir is the culprit in your scenario.  The code is raising jinja2.exceptions.UndefinedError, so the problem looks to be here:
https://github.com/audreyr/cookiecutter/blob/master/cookiecutter/generate.py#L352.
Introduced in #598.

@michaeljoseph interesting...I made the following adjustment to cookiecutter locally and thought that this fixed the bug: https://github.com/rsyring/cookiecutter/commit/a14570c5c36400f441a377622ef8f6aa1ab35234
But I can revisit.

The error message reads
Unable to create file 'tox.ini'
Error message: 'dict object' has no attribute 'python_tox'
Context: {
    "cookiecutter": {
        "circle_badge_token": "",
        "codecov_api_token": "",
        "codecov_badge_token": "",
        "gh_repo_name": "",
        "gh_repo_path": "",
        "project_dirname": "filebrowser-src",
        "project_namespace": "filebrowser",
        "python_executable": "python3.5",
        "python_tox_envname": "py35",
        "python_version": "3.5"
    }
}

Cookiecutter can't find a variable named python_tox in the context and exits gracefully by cleaning up the directory it was supposed to generate to. At this point this is implemented using a rmtree rather than removing each file and directory that has been created. Changing this may go hand in hand with #633.

Would it make sense to handle this case making the overwrite in a secondary step (i.e., writing into a temp dir first, before overwriting to destination)?
This would be simpler than tracking every file that was created.
Another idea would be to prompt for confirmation before deleting when the directory already existed before.

writing to a temp directory and then copying to the new location does seem like it might be a good idea but outside the scope of this issue IMO as it would be a larger architectural decision IMO.
Promoting to delete IMO doesn't win much.  If the directory existed, it seems better to me to simply leave it alone and alert the user to the fact that a partial update occurred.  Let them deal with it.


fuzzing: check our Equals() functions as well

Just minor improvement, mostly future proofing for
https://github.com/istio/istio/issues/52446 if we do it

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio
d316f01242ee499b48c606922976618cb0b1a0aa
link
true
/test gencheck

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

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio
d316f01242ee499b48c606922976618cb0b1a0aa
link
true
/test gencheck

lint_istio
d316f01242ee499b48c606922976618cb0b1a0aa
link
true
/test lint

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

@wulianglongrd updated to use mutate struct. Thanks!!

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
808923841d21d39dbc29100f90fc6bfb77651939
link
true
/test lint

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

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
808923841d21d39dbc29100f90fc6bfb77651939
link
true
/test lint

unit-tests_istio
808923841d21d39dbc29100f90fc6bfb77651939
link
true
/test unit-tests

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

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
808923841d21d39dbc29100f90fc6bfb77651939
link
true
/test lint

unit-tests_istio
808923841d21d39dbc29100f90fc6bfb77651939
link
true
/test unit-tests

unit-tests-arm64_istio
808923841d21d39dbc29100f90fc6bfb77651939
link
true
/test unit-tests-arm64

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

The test failed because I changed the parameter type in the recent PR.

fuzzDeepCopy: we only have a DeepCopy to test
fuzzDeepCopyEqual: we want to test DeepCopy and Equals
Not all types implement Equals


[designspaceLib] Default font not set as described in spec

The designspace spec (see fonttools/Doc/source/designspaceLib/) says "The source with the copyInfo flag indicates this is the default font." This matches what I have always thought, that you add the child element  to a source file in order to make it be the default source font for a variation font.
However, poking around in varLib.init.py::load_designspace() and experimenting with design space files shows that the default font is now set by setting the  'default' attribute of the axis elements. The source font element whose location matches the axis default attributes is set to be the default font. I am fine with this, but we then need to update the design space spec. Or am I missing something?

"The source with the copyInfo flag indicates this is the default font."

I think that's a remnant from MutatorMath era.

The source font element whose location matches the axis default attributes is set to be the default font.

We indeed need to update the spec. The default font is the netural master, which is at default coordinate along all the axes.

Spec has been updated: https://github.com/fonttools/fonttools/tree/master/Doc/source/designspaceLib#82-older-versions


point_near_pole notice is missing from RULES.md

The point_near_pole notice is missing from RULES.md. It is related to PointNearPoleNotice.java and LatLonGeneratorValidator.java in the validator.

Nice catch!
Fixed in #1211, just need your review :)


refactor: merge base modules with -infinity modules

In a similiar fashion to what we previously did with -coroutines modules, we now merge -infinity modules with their respective bases. This should allow us to re-use most classes across higher level constructs, as well as to converge on a single module that deals with Infinity-related code - :sdk-infinity, which has now becomes the base.

Does it mean there is only one sdk-infinity module now? Any plans on merging that with sdk-conference as well?

No modules were removed in this PR. But the overall direction would be to remove sdk-api/sdk-conference/sdk-registration (and all -coroutines & -infinity modules). Right now sdk-infinity is a base of sdk-api, on which both sdk-conference and sdk-registration depend on.


StackedLabel

Good day everyone. I use StackedLabel from source examples Mapsui.

I am interested in drawing this without circles. I wanna just rectangular outline. Сan anyone know how to implement this? below is some of the code to create style.

I understand this is confusing. The VectorStyle applies to all features in the Layer, which consists of the points and the rectangles added as polygon. You could filter out the points within the GetStyle method of the ThemeStyle. So you need to replace the VectorStyle with a ThemeStyle.

@VladGluschenko Did you manage to solve this?

I have postponed this question for now. But I'll get back to this soon


I'm having trouble going past Instagram home page. It doesn't proceed to URL

DevTools listening on ws://<IP_ADDRESS>:52313/devtools/browser/91b1ac12-ee5b-4523-a7b3-665253460d20
Traceback (most recent call last):
File "C:\Users\am1460\Downloads\InstagramScraper\scrape_likes.py", line 7, in
Driver, args = scraper.setup()
^^^^^^^^^^^^^^^
File "C:\Users\am1460\Downloads\InstagramScraper\src\likes_scraper.py", line 53, in setup
driver = chrome_driver.make_IG_access_w_creds(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\am1460\Downloads\InstagramScraper\scripts\chrome_driver.py", line 75, in make_IG_access_w_creds
wait.until(
File "C:\Users\am1460\Downloads\insta-scrapper\myenv\Lib\site-packages\selenium\webdriver\support\wait.py", line 95, in until
raise TimeoutException(message, screen, stacktrace)
selenium.common.exceptions.TimeoutException: Message:
Stacktrace:
Backtrace:
GetHandleVerifier [0x007BA813+48355]
(No symbol) [0x0074C4B1]
(No symbol) [0x00655358]
(No symbol) [0x006809A5]
(No symbol) [0x00680B3B]
(No symbol) [0x006AE232]
(No symbol) [0x0069A784]
(No symbol) [0x006AC922]
(No symbol) [0x0069A536]
(No symbol) [0x006782DC]
(No symbol) [0x006793DD]
GetHandleVerifier [0x00A1AABD+2539405]
GetHandleVerifier [0x00A5A78F+2800735]
GetHandleVerifier [0x00A5456C+2775612]
GetHandleVerifier [0x008451E0+616112]
(No symbol) [0x00755F8C]
(No symbol) [0x00752328]
(No symbol) [0x0075240B]
(No symbol) [0x00744FF7]
BaseThreadInitThunk [0x76A96BA9+25]
RtlGetFullPathName_UEx [0x76FB8F9F+1215]
RtlGetFullPathName_UEx [0x76FB8F6D+1165]

Hi, judging by the error,most probably something on the html on the page has changed.
Can you provide a more extensive description of the issue?

The command that gave that error;
The page are you trying to scrape.


webapp: Add support to sync a cert to the latest version on AKV

On a webapp when we import a cert from AKV - there is no way to force sync the app to use the latest version from AKV. Portal has 'Sync' option - if a customer doesn't want to wait for our background sync job to be completed.

webapp


feat: redirect to previous path if its a 404

This is an MVP, not the perfect result, but much better than what we currently have (which is nothing)

I like it!
In future I think we can have some sort of combination of this logic, and a suitably populated 404 page with onward links to likely locations the user intended... but for now, this would be a valuable improvement

our current 404 page is terrible (just an empty page that says "Not Found"), and since our URLs are structured hierarchically, going to the next possible parent makes the most sense.
One example also is for some reason google indexed https://docs.deno.com/api/node/streams, which doesnt even exist, so going to the parent in this case is a better experience

I still think overriding the "correct" response is hazardous and unexpected behavior. Even if it works sometimes, it probably won't others, and that creates a confusing experience. (I also don't think it's probably a good thing that we make it impossible to even hit a 404, for a few reasons.)
I also notice that if the first redirect doesn't hit an existing page, it creates a "too many redirects" error.
Can we write some kind of "is this what you were looking for?" logic into the 404 page that pattern matches, or something along those lines? That seems much more in line with expected behavior. I just think this seems like a pretty haphazard solution.

Happy to take a stab at this, if it helps

I think it may lead users to be confused about what just happened if there isn't something on the page telling them they were redirected due to a 404.
Like if I clicked a link and it just loaded the exact same page I'm on, I'd think this page is broken and not that the page I'm trying to go to doesn't exist. It would feel like a buggy experience.

sure @josh-collinsworth, go ahead; i probably wont have have time to work on this anytime soon


Warning user that the active machine was restarted and may have a new IP

Fixes #1298
Signed-off-by: Dave Henderson<EMAIL_ADDRESS>

/cc @ehazlett @nathanleclaire @aperepel

Cool idea.
Why not store the value of DOCKER_HOST before any action is taken (you can also use IsActive to check if the machine you are restarting is active IIRC), and then check to see if it has been invalidated after the action is taken?  I feel like if it is printed every time users will grow immune to it and it will defeat the purpose of having it.

@nathanleclaire - Good idea. I don't see any IsActive function... were you thinking of Machine.GetActive?
I've mocked something up but there's some work to do before it'll work properly.

Yep, GetActive is what I was thinking of.  Then you'd actually check the URL against what is existing in DOCKER_HOST.

+1

Ok, I've fixed this all up so it's actually working now, and squashed/rebased.

So revisiting this issue, perhaps we could simply not get too clever and suggest to the user on start and restart a note that the IP may have changed and they may need to re-eval ?

@nathanleclaire - I'm inclined to agree... that'd certainly have less moving parts to break ;P Plus, I just realized that my change didn't have any support for start, so that'd be missed too.
I'll de-complicate this soon...

@nathanleclaire PTAL

Nice.  Nitpick, but the Infof is superflous- you only need log.Info.

Heh. Was thinking "I need to change this to Info" as I was writing that... Looks like I forgot :P
Will fix when I get back to a computer :)

@nathanleclaire - ok, fixed :)

Nice LGTM thanks!


unit tests

Hi! I was looking this repo for some kind of tests for the differente resamplers against expected outcomes and didn't find any... I am using Rubato on my R package and would like to use these tests in R to assert I am doing everything ok when using Rubato. Do you happen to have these tests to share? Thank you in advance!

There are tests for individual components like the sinc interpolator, but not for the full resampler. I haven't gotten around to writing those yet. It becomes a bit tricky because I want to be able to catch very very small errors.
What I do instead is to use the different examples to resample a sine wave, and analyze the result with examples/analyze_result.py. That makes it very easy to see if there is a problem. There is also a python script for generating suitable sine waves to use.

Thank you! Gonna take a look.


Bug: Error while calling lua chunk

Hello,
thanks you for this great plugin, I was searching for such thing for some time now.
Anyway, I wasn't able to initialize it properly. When I added the installation part to the nvim config, I was able to install it via PLUG manager:
        Plug 'nvim-lua/plenary.nvim'
        Plug 'folke/todo-comments.nvim'

        lua <<EOF
          require("todo-comments").setup {
            -- your configuration comes here
            -- or leave it empty to use the default settings
            -- refer to the configuration section below
          }
EOF

But, when I ran the neovim, I received this error:
Error detected while processing /Users/mcibulka/.config/nvim/init.vim:
line  423:
E5108: Error executing lua [string ":lua"]:1: module 'todo-comments' not found:
        no field package.preload['todo-comments']
        no file './todo-comments.lua'
        no file '/Users/runner/work/neovim/neovim/.deps/usr/share/luajit-2.1.0-beta3/todo-comments.lua'
        no file '/usr/local/share/lua/5.1/todo-comments.lua'
        no file '/usr/local/share/lua/5.1/todo-comments/init.lua'
        no file '/Users/runner/work/neovim/neovim/.deps/usr/share/lua/5.1/todo-comments.lua'
        no file '/Users/runner/work/neovim/neovim/.deps/usr/share/lua/5.1/todo-comments/init.lua'
        no file './todo-comments.so'
        no file '/usr/local/lib/lua/5.1/todo-comments.so'
        no file '/Users/runner/work/neovim/neovim/.deps/usr/lib/lua/5.1/todo-comments.so'
        no file '/usr/local/lib/lua/5.1/loadall.so'
Press ENTER or type command to continue

The TODO plugin didn't work. But when I reload the nvim config, it didn't gave me any error and the this plugin worked fine.
Here is the output of nvim --version:
❯ nvim --version
NVIM v0.5.0-dev+1321-gc57a85d53
Build type: Release
LuaJIT 2.1.0-beta3
Compilation: /Applications/Xcode_12.4.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/cc -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1 -O2 -DNDEBUG -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wshadow -Wconversion -Wmissing-prototypes -Wimplicit-fallthrough -Wvla -fstack-protector-strong -fno-common -fdiagnostics-color=always -DINCLUDE_GENERATED_DECLARATIONS -D_GNU_SOURCE -DNVIM_MSGPACK_HAS_FLOAT32 -DNVIM_UNIBI_HAS_VAR_FROM -DMIN_LOG_LEVEL=3 -I/Users/runner/work/neovim/neovim/build/config -I/Users/runner/work/neovim/neovim/src -I/Users/runner/work/neovim/neovim/.deps/usr/include -I/Applications/Xcode_12.4.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.1.sdk/usr/include -I/Library/Frameworks/Mono.framework/Headers -I/Users/runner/work/neovim/neovim/build/src/nvim/auto -I/Users/runner/work/neovim/neovim/build/include
Compiled by<EMAIL_ADDRESS>
Features: +acl +iconv +tui
See ":help feature-compile"

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/share/nvim"

Run :checkhealth for more info

Could somebody look at this error please?
Thank you

It only works on neovim 0.5. your second error shows you didn't properly install the plugin

Yes, you're right, I'm sorry, I noticed in the middle of writing that I was not using nightly build, I installed it and tried it there, thats where I got that second error, and king of forgot about the first half. I have rewritten it accordingly.

The second error means you don't have the plugin installed.

If you installed nighty afterwards, you probably also need to reinstall the plugs

I tried removing the installation statement, running PlugClean, adding it back, running PlugInstall, but it didn't help. Still the same error on startup, and when I run :source ~/.config/neovim/init.vim, this plugin starts to work.
Am I missing some installation steps?

~/.config/neovim/init.vim This path is wrong?
It should be nvim, not neovim

That was typo ... I am not running it manually, but I have shortcut, it definitely is not because of wrong path, as it doesn't work when I start new instance of neovim

I'm sorry, but if you get this error:
E5108: Error executing lua [string ":lua"]:1: module 'todo-comments' not found:
Then that can only mean you didn't install the plugin properly.
The error literally means that the plugin could not be found on your plugins path.

Closing this issue as it seems to be a problem related to configuring nvim.

I experienced this bug too. It disappeared when I had moved Lua setting chunk at the end of init.vim.
I use Vundle and place plugin setting in init.vim after Plugin '...' line for better readability. For example:
filetype off                                        " required for Vundle
" set the runtime path to include Vundle and initialize
set rtp+=~/.vim/bundle/Vundle.vim
call vundle#begin()
Plugin 'VundleVim/Vundle.vim'                       " Vundle manage Vundle

Plugin 'folke/todo-comments.nvim'
" {{{
lua << EOF
  require("todo-comments").setup {
  }
EOF
" }}}

" All of your Plugins must be added before the following line
call vundle#end()            " required
filetype plugin indent on    " required

Placing lua chunk inside vundle#begin() and vundle#end() was the reason that caused this bug.

@aborzunov that makes sense. It's only after the end that the plugin is effectively added and when you can do the require

Was getting the same issue.
The issue, at least for me, was that I was assuming that configuration could happen before the plugin was loaded bc most, if not all of my plugin configuration, is setup before they're loaded.
A little strange to me but I'm sure there's a reason for this.
Thanks for the plugin! Looking good so far.

Was getting the same issue.
The issue, at least for me, was that I was assuming that configuration could happen before the plugin was loaded bc most, if not all of my plugin configuration, is setup before they're loaded.
A little strange to me but I'm sure there's a reason for this.
Thanks for the plugin! Looking good so far.

How could you execute a setup function from a module that you require that is not loaded? That's just Lua. If you don't understand that, there's a number of good lua guides out there to get you started.

A little strange to me but I'm sure there's a reason for this.

Ah it's how Lua configures modules that makes more sense. I'm not familiar with it but I might take a look in the new year! Thanks again :+1:


Remove ES6 Model class documentation

As we don't want to support this feature, we should remove this piece of documentation.

Coverage remained the same at 96.154% when pulling 65c68ac228dbd3d76c36e9e64c128a5df5667107 on remove_model_es6_docs into 1b0730aaf0bd9e74ed37ecd478fcc35b5654875e on master.


input[attr] specificity

Hello,
Wondering if normalize can drop the input qualifiers for the inputs with attributes? As it is now, input[type="checkbox"] is more specific than a single class, so it can't be overridden with a class, whereas [type="checkbox"] is not. It should accomplish the same thing with less specificity, right?
My specific use case is overriding 0 padding on a checkbox from this:
https://github.com/necolas/normalize.css/blob/2bdda84272650aedfb45d8abe11a6d177933a803/normalize.css#L334-L338

I know padding on a checkbox input isn't super valuable (or even works in most browsers), but the overall idea might be...

There's already a discussion in an issue (#187) that I opened a while ago.
Sadly I deviated the topic —and FWIW I have changed my mind since then—, but could be useful as a reference. Hope it helps.

Oh, I didn't find it in my searches. That pretty well covers it...its a pretty weak use case. Personally I have no problem editing my file if I need it.

Following the threads, it seems it was decided that the potential performance impact was not worth the reduced selector weight. I might have leaned the other way, but any moderate developer should catch this if it ever creates a problem for them.

While I strongly believe that libraries shouldn't have —or should try to avoid as much as possible— selectors with a specificity higher than [0,0,1,0], in this case —and considering the current code-base— I agree that considering the performance impact that this would have is not really worth it to reduce the specificity.


Smart Switches not working anymore with 2.12.02 / 12.7.2021

A few days ago I still had objects for the "click" states of my AQARA Smart Switches - single press, double press, long press.
Now this entry points have been vanished.
Reconnecting and syncing my Smart Switches only adds this objects:

In the Log I can see the following warning:
deconz.0 | 2021-07-16 09:41:51.615 | warn | Unknown command: send
I am on Phoscon version 2.12.02 / 12.7.2021
Any idea?
Thanks!

Ok so I manually flashed the latest firmware.
Now it seems to work,
Shouldn't the APT-GET UPDATE do this?

I don't know. You can ask this on phoscon forum -> https://forum.phoscon.de/


PowerShell documentation for AzIoT

Is there PowerShell documentation for connecting a Rasberry Pi to Azure IoT Hub?

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

ID: 773d535b-29c4-20ca-d260-1d6c20fc43dd
Version Independent ID: b38b426a-6f56-026c-be29-6aee30586170
Content: Connect Raspberry Pi to Azure IoT Hub using C
Content Source: articles/iot-hub/iot-hub-raspberry-pi-kit-c-get-started.md
Service: iot-hub
GitHub Login: @wesmc7777
Microsoft Alias: wesmc

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

@Ayanmullick
Where you are trying to connect from, Windows/Linux or RaspberryPI?
Have you visited the following documentation,
Az.IotHub
AzureIoT

I want to send the sensor telemetry data to Az.IoTHub from Pi. I'd use Powershell 7 to set it up on the latest version of Raspbian on Raspberry Pi 4.

@Ayanmullick  Could you please add your case scenario as Azure IoT Feedback Forum new request.

This is just a PowerShell documentation request and not a Product feedback or feature request.
I think Microsoft should provide documentation for their native language [PowerShell] if you can provide the same for other languages like Java, NodeJS, Python etc.

@Ayanmullick Thank you for the  Azure IoT Feedback request. Please follow PowerShell documentation for AzIoT feedback request for future updates.


Bypassing the verifier

One can easily bypass the Alexa verifier by simply not sending the signaturecertchainurl header because of https://github.com/tejashah88/alexa-verifier-middleware/blob/master/container.js#L11. Meaning, somebody can still invoke the skill without any of the checks. I don't think that's desirable as one would want to fail any requests that are invalid. So, I think that check should be removed.

@jgawor Thank you for bringing that up. Unfortunately, I'm not very knowledgeable as to why this code is what it is. The original alexa-verifier module did have an example for using with express here, before it was moved to this module. @mreinstein might be able to explain more about this.

@jgawor Actually, looking back at the code, it does make sense that request should not go through if the signaturecertchainurl header is not present and it is indeed an Alexa request. I guess the only caveat is that any non-Alexa requests to the server will fail. I'll run some tests later today.

One can easily bypass the Alexa verifier by simply not sending the signaturecertchainurl header

This was done originally because it wasn't clear to me if there was a way to run this middleware only for alexa related express routes. I've found that people tend to mess this up if it's just based on declaring middlewares/routes in the right order.
Keep in mind the original goal was not to have perfect security, but to at least handle the cases that Amazon checks for when reviewing Alexa skills. This module accomplishes that. (I think...I haven't published an alexa skill in a while but last time I tried using this it passed.)
That said, I'm always open to improving the security where practical.   Here's what I think we should do:

add an optional parameter to to the options passed to the middleware. something like strictHeaderCheck and have it be false by default. Setting this to true would reject any requests lacking the header
add some documentation about this mode, and show an example of how to do this. I'd suggest we encourage people to group alexa calls to 1 or more express routers, and then attach this verifier middleware to that. example:

const app = express()

const alexaRouter = express.Router()
alexaRouter.use(alexaVerifierMiddleware({ strictHeaderCheck: true }))

// routes that handle alexa traffic are now attached here. since this is
// attached to a router mounted at /alexa,  this endpoint will be accessible
// at  /alexa/weather_info
alexaRouter.get('/weather_info', function(req, res) { ... })

app.use('/alexa', alexaRouter)

app.listen(3000)

thoughts?

I've just merged the changes here. Let me know if this is sufficient enough before I publish a new version.

I would rather see the strictHeaderCheck to default to true to be on the safe side but this will work too. Thanks!

Just made the changes and will be pushing to a new version! Thanks!

so a few comments:

I would rather see the strictHeaderCheck to default to true

@jgawor I agree that this would be ideal, but this is a breaking change. It's going to change the current behavior of the module. (the current behavior is essentially the same as strictHeaderCheck: false.
@tejashah88  this line:
module.exports = function alexaVerifierMiddleware(options)

you need to check that options is actually defined. The current behavior of anyone using this middleware is to pass nothing as an options object.


adr: Add -batch-number label/taint for AMS

adr: Add -batch-number label/taint for AMS

Thanks @lucianvlad . I see two files now in the PR, ADR-0034 and ADR-0035. They are both similar, but ADR-0035 seems to have all fixes, so I assume you meant to keep only that one.
As it stands in ADR-0035, the label is harmonized now. However, I'm still confused. Didn't we agree to elastisys.io/batch-number, i.e., the same label for all AMS-es? So Nodes would have:
elastisys.io/node-type=postgresql
elastisys.io/batch-number=1

and NOT:
elastisys.io/node-type=postgresql
elastisys.io/postgresql-batch-number=1

I'm getting old so please remind me. :smile:

Thanks @lucianvlad . I see two files now in the PR, ADR-0034 and ADR-0035. They are both similar, but ADR-0035 seems to have all fixes, so I assume you meant to keep only that one.
As it stands in ADR-0035, the label is harmonized now. However, I'm still confused. Didn't we agree to elastisys.io/batch-number, i.e., the same label for all AMS-es? So Nodes would have:
elastisys.io/node-type=postgresql
elastisys.io/batch-number=1

and NOT:
elastisys.io/node-type=postgresql
elastisys.io/postgresql-batch-number=1

I'm getting old so please remind me. smile

I will remove the 0034 file.
As for elastisys.io/postgresql-batch-number=1 or elastisys.io/batch-number=1 it seems we are still debating this in one of the PR's but I think you are remembering correctly :)
I will change to what we decide in the PR debate :D

debating

I will fix 0034 as I thought that Andre used 0034 on his tekton adr ...


Operator Investigation | Run W&B platform on Microk8s and prepare instructions

Related to the VM image project, we would like to see if it's possible to run the k8s operator-based W&B platform on Microk8s. If yes, we would like to understand the performance and scalability constraints of that, in terms of how large a single-node deployment could that sustain and what are its high-level breaking points. If it works for small (4 vCPU, 32 GB memory) to intermediate (32 vCPU, 256 GB memory) sized deployments, we would like to try and offer this as a v1 solution to customers who need capabilities unlocked via the operator but can't / don't want to use k8s directly, and see how far we can get with this.
Ask in this issue is to do a PoC on end to end deployment of k8s operator-based W&B platform on Microk8s, starting with local machine but then on a EC2 VM, and record the relevant instructions which could be shared with customers. When testing on AWS, try on at least three different sizes of VMs, and see how much memory Microk8s is able to provide to the application.

#Microk8s Installation and wandb-operator chart deployment:
https://docs.google.com/document/d/1OjF7hZK4PSdmAi_3yk4hC6Xfz6PMo0xv6aJ_g80CDQI/edit?usp=sharing


Does this work with django 2/3 and python 3+?

Hi, just wondering if this works with newer versions of django and pipedrive..?

Nevermind, I see the commit


IRC servers with passwords

I know IRC support is a work-in-progress but just wanted to request support for using IRC servers that require passwords. Great project btw!

Should support password protected servers (works with registered nicks). Credentials are as follows:
irc.freenode.net:7000:1 for SSL enabled host (w/o custom nick and password)
somenick@irc.freenode.net:7000:1 for SSL enabled host + custom nick (w/o password)
somenick@irc.freenode.net:7000:1:s0m3p4ssw0rd for SSL enabled host + custom nick + password
launching bot (example with credentials above):
torpedobot -ircapikey somenick@irc.freenode.net:7000:1:s0m3p4ssw0rd
Please drop me a line if that worked so I can close this and #11
Thanks.

Wow @tb0hdan -- that's great! I'll test it now and report back.

Works great, thanks again for the fast turn around!


feat: add protobuf 3.22.5 release

update protobuf with release 3.22.5

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


Change String Identifier to Int Identifier

Here https://github.com/nlopes/slack/blob/8ac1139f0b5b26ddb4a5c05b727e7ce155cc37f4/slack.go#L15 Instead of using %s as string identifier we can use %d identifier as here https://github.com/nlopes/slack/blob/8ac1139f0b5b26ddb4a5c05b727e7ce155cc37f4/misc.go#L200
time is in int.

i don't have access to push the branch to this repo.

correct you have to open a PR.

@hitesh-goel plenty of tutorials for github to get started. feel free to google around for guides.


bug: operation running outside ngZone on native and mobile web

Bug Report
Ionic version:
[x] 4.x
Current behavior:
I have a parent component with the following template:
 <app-graph-grouped-vertical
      [results]="graphRecords"
      [series]="graphPredictions"
      [seriesColors]="predictionColors"
      [xAxis]="xAxis"
></app-graph-grouped-vertical>

And, inside GraphGroupedVerticalComponent, I've got the following code:
export class GraphGroupedVerticalComponent extends GraphBaseComponent<GroupedResultDto>
  implements OnChanges, OnInit, OnDestroy, DoCheck {
...
 @Input() results!: GroupedResultDto[]

 ngOnChanges(changes: SimpleChanges){console.log('Triggered')}
...

}

In the parent component, I subscribe to a store and update the property graphRecords whenever a new value triggers the subscription. The problem is that when I'm in a computer browser, this works perfectly, and NgZone.isInAngularZone() will return true.
However, when I use the app in a mobile web browser, or compiled into a native app (physical phone or emulator) this code will run outside ngZone and I'm forced to use ngZone.run().
This happens involuntarily and I haven't found anything in the Docs related to this issue, so in my opinion, it is a bug.
Expected behavior:
This operation should run inside ngZone without necessarily forcing it.
Steps to reproduce:
Unfortunately, I cannot give a complete example of this because it is a private project but, if needed, ask and I will try and give you all the possible information.
Ionic info:
Ionic:

   Ionic CLI                     : 5.4.6
   Ionic Framework               : @ionic/angular 4.11.4
   @angular-devkit/build-angular : 0.803.18
   @angular-devkit/schematics    : 8.3.18
   @angular/cli                  : 8.3.18
   @ionic/angular-toolkit        : 2.1.1

Capacitor:

   Capacitor CLI   : 1.3.0
   @capacitor/core : 1.3.0

Cordova:

   Cordova CLI       : 9.0.0<EMAIL_ADDRESS>   Cordova Platforms : none
   Cordova Plugins   : no whitelisted plugins (2 plugins total)

Utility:

   cordova-res : 0.8.1
   native-run  : not installed

System:

   NodeJS : v10.16.0 (/Users/javier.marzan/.nvm/versions/node/v10.16.0/bin/node)
   npm    : 6.11.2
   OS     : macOS Mojave

Thanks for the issue. Are you able to provide a simplified reproduction without any of your project's confidential information?

Sorry for the late response. I was fiddling a bit, and I think it can be a problem with ngx-charts, as they use ngZone within their graph update methods, and this bug only occurs in screens where we are using their charts, so I think it's their problem.
I will post a new issue if this happens in a screen without a graph, but I think it's okay for now. Thanks for answering to the issue anyway :)

Thanks for the follow up. I will close this issue, but feel free to open a new issue later if you feel this is a bug in Ionic Framework. Thanks!


How evaluation being done without storing quantized weights?

I would like to know how the quantized model's performance is evaluated while keeping this line blank?
How are the quantized weights being considered while evaluating the performance of the model since the value of quantizers here is empty?

Hello, @tahmiddialpad,
Thank you for interest to SpQR compression.
When the quantize() function is called in line 217(https://github.com/Vahe1994/SpQR/blob/e75c55bbc1c96f8411bc58be6d8f44596de44f84/main.py#L217) it quantizes and then immediately dequantizes the model weights. See lines 178-189 in spqr_engine.py(https://github.com/Vahe1994/SpQR/blob/e75c55bbc1c96f8411bc58be6d8f44596de44f84/spqr_engine.py#L178-L189).
The line that you mentioned would return quantizers necessary to save the compressed model. This saving functionality is not yet implemented.

Hello, @tahmiddialpad,
Thank you for interest to SpQR compression.
When the quantize() function is called in line 217 it quantizes and then immediately dequantizes the model weights. See lines 178-189 in spqr_engine.py().
The line that you mentioned would return quantizers necessary to save the compressed model. This saving functionality is not yet implemented.


Removing the 3-column skeleton and supporting CSS

Looking at this project again, after a bit of a break, it seems like removing the "skeleton markup and styling for a 3-column website" from this theme and replacing it with a bare-bones layout that does what Sphinx's basic theme does (i.e. no-styling content placement) would be ideal.
Having tried to work with this in Lutra, it's actually quite annoying.... There's no easy way to tell your IDE that "hey, this project actually also uses classes and content from that other set of files in that virtual environment/local directory", and trying to figure out "hey, what thing controls the layout of that" is a non-trivial piece spread across two projects.
/cc @choldgraf @AakashGfude -- I believe both pydata and book themes were thinking of using this skeleton and experimenting with it. Any thoughts based on the work done around there?

Hmm - honestly, I don't think I'm a good person to ask about this. I haven't had time to dig into the basic-ng theme and I won't have time to do much development going forward, though am happy to support others in these efforts. I think the biggest benefit from this theme so far has been the standardization that it has catalyzed for the pydata theme. We've renamed most things to use the structure and naming of this theme, but thus far have not made the actual switch. I know @aakashgfude was trying this out but hasn't picked it up in a little while.
If the skeleton / structural thing was gone, then what is the primary value proposition of this theme?

The components -- which serve as shared implementations of common web design patterns.

@pradyunsg @choldgraf I think the 3-column skeleton worked nicely for the themes of pydata and sphinx-book-theme, as both of them have the same structure. The classes and components were being inherited from what i remember working on this PR: https://github.com/pydata/pydata-sphinx-theme/pull/840 . And we did adjust some of the structural things in pydata-sphinx-theme to be in sync with sphinx-basic-ng skeleton.

We can start working with the component inheritance of sphinx-basic-ng in pydata-sphinx-theme. Now that sbt has been updated to pydata 0.10 . As mentioned in this comment https://github.com/pydata/pydata-sphinx-theme/pull/840#issuecomment-1197545667. And see how we go?

Sounds reasonable!
I'll close this out since... well, at the very least, we'll defer this until we have a better idea. :)

Ya agree - so for now if we depend on this base theme, it should just be for the reusable components and not for the skeleton.


Failed to render application: ReferenceError: jQuery is not defined

I was trying the application. I am using Angular 4.1.0 and Jquery 3.2.1. Everything runs fine with the Angular CLI. But When i try ./node_modules/.bin/ng-render. I get an error:
Failed to render application: ReferenceError: jQuery is not defined
Any suggestions?

Yes, can you run ng-render with the --debug flag and paste the entire call stack that it prints out? Probably whichever .ts module is referencing jQuery is not importing it.

clbond,
Please check the print out:
[info] Rendering application from source (working path: /var/folders/qw/fy6pdj413xn4lvnqt__2lyw40000gn/T/f36ba326c3a83) =
[error] Failed to render application: ReferenceError: jQuery is not defined
at Object.module.exports.installedModules (/private/var/folders/qw/fy6pdj413xn4lvnqt__2lyw40000gn/T/f36ba326c3a83/0.js:90870:13052)
at webpack_require (/private/var/folders/qw/fy6pdj413xn4lvnqt__2lyw40000gn/T/f36ba326c3a83/0.js:21:30)
at Object. (/private/var/folders/qw/fy6pdj413xn4lvnqt__2lyw40000gn/T/f36ba326c3a83/0.js:29702:116)
at webpack_require (/private/var/folders/qw/fy6pdj413xn4lvnqt__2lyw40000gn/T/f36ba326c3a83/0.js:21:30)
at Object.module.exports.c (/private/var/folders/qw/fy6pdj413xn4lvnqt__2lyw40000gn/T/f36ba326c3a83/0.js:30079:76)
at webpack_require (/private/var/folders/qw/fy6pdj413xn4lvnqt__2lyw40000gn/T/f36ba326c3a83/0.js:21:30)
at Object. (/private/var/folders/qw/fy6pdj413xn4lvnqt__2lyw40000gn/T/f36ba326c3a83/0.js:36204:96)
at webpack_require (/private/var/folders/qw/fy6pdj413xn4lvnqt__2lyw40000gn/T/f36ba326c3a83/0.js:21:30)
at Object. (/private/var/folders/qw/fy6pdj413xn4lvnqt__2lyw40000gn/T/f36ba326c3a83/0.js:33020:72)
at webpack_require (/private/var/folders/qw/fy6pdj413xn4lvnqt__2lyw40000gn/T/f36ba326c3a83/0.js:21:30)
at Object. (/private/var/folders/qw/fy6pdj413xn4lvnqt__2lyw40000gn/T/f36ba326c3a83/0.js:93402:18)
at webpack_require (/private/var/folders/qw/fy6pdj413xn4lvnqt__2lyw40000gn/T/f36ba326c3a83/0.js:21:30)
at /private/var/folders/qw/fy6pdj413xn4lvnqt__2lyw40000gn/T/f36ba326c3a83/0.js:67:18
at Object. (/private/var/folders/qw/fy6pdj413xn4lvnqt__2lyw40000gn/T/f36ba326c3a83/0.js:70:10)
at Module._compile (module.js:571:32)
at Object.Module._extensions..js (module.js:580:10)
at Module.load (module.js:488:32)
at tryModuleLoad (module.js:447:12)
at Function.Module._load (module.js:439:3)
at Module.require (module.js:498:17)
at require (internal/module.js:20:19) =

I think you are missing an import jQuery = require('jquery') somewhere


samples/bluetooth/peripheral on efr32xg24_dk2601b throw error: bt_settings: settings_subsys_init failed (err -45)

Describe the bug
I have successfully run samples/bluetooth/observer, but running samples/bluetooth/peripheral gives an error, indicating that the settings_subsys_init  call has an error, with an error code of -45.
To Reproduce
Steps to reproduce the behavior:

west build -p=always -b efr32xg24_dk2601b samples/bluetooth/peripheral
west flash
use picocom connect to the board
See error

Expected behavior
Be able to successfully run samples/bluetooth/peripheral and provide the functions expected by the program
Logs and console output
 <inf> flash_gecko: Device flash-controller@50030000 initialized
*** Booting Zephyr OS build zephyr-v3.5.0-3466-gb702c8e8f01f ***
[00:00:00.000,000] <err> bt_settings: settings_subsys_init failed (err -45)
Bluetooth init failed (err -45)

Environment (please complete the following information):

OS: Ubuntu22.04
SDK: Zephyr SDK
board: xG24-DK2601B Silicon Labs
zephyr: commit b702c8e8f01f09cb73ef554b129db0773f9f98a5 (HEAD -> main, origin/main, origin/HEAD)
modules/hal/silabs: 7a6ba7c

Additional context
I tested samples/bluetooth/observer and it works fine.
west build -p=always -b efr32xg24_dk2601b samples/bluetooth/observer
west flash
output:
* Booting Zephyr OS build zephyr-v3.5.0-3466-gb702c8e8f01f ***
Starting Observer Demo
Started scanning...
Exiting main thread.
Device found: DE:CD:2F:AD:F3:F2 (public) (RSSI -72), type 0, AD data len 28
Device found: D0:B4:5D:53:EC:DE (public) (RSSI -71), type 0, AD data len 22
Device found: A4:C1:38:C1:2B:48 (public) (RSSI -99), type 0, AD data len 21

I don't think this is Bluetooth related, rather something to do with the support for non-volatile storage on this specific board. Error number 45 is EDEADLK, and the only storage-related code I managed to find using that error is this:
https://github.com/zephyrproject-rtos/zephyr/blob/2312897055d19ce11ddfa496052e400afb8893f4/subsys/fs/nvs/nvs.c#L739-L743

@HonestQiao, this error typically appears when trying to use nvs storage on a area that is not erased before the first use. Erase the storage area and retry,

Can you try erasing the entire flash of device and re-uploading the software?

Hi @HonestQiao! I've tried to reproduce this issue but wasn't able to with the instructions that you had provided:
*** Booting Zephyr OS build zephyr-v3.5.0-3838-g8ca40e5ebfb6 ***
[00:00:00.010,000] <inf> fs_nvs: 8 Sectors of 8192 bytes
[00:00:00.010,000] <inf> fs_nvs: alloc wra: 0, 1fe8
[00:00:00.010,000] <inf> fs_nvs: data wra: 0, 0
[00:00:00.020,000] <wrn> bt_hci_core: Controller to host flow control not supported
[00:00:00.021,000] <wrn> bt_ecc: ECC HCI commands not available
[00:00:00.021,000] <inf> bt_hci_core: No ID address. App must call settings_load()
Bluetooth initialized
[00:00:00.023,000] <inf> bt_hci_core: Identity: 58:8E:81:63:09:5A (public)
[00:00:00.023,000] <inf> bt_hci_core: HCI: version 5.3 (0x0c) revision 0x0000, manufacturer 0x02ff
[00:00:00.023,000] <inf> bt_hci_core: LMP: version 5.3 (0x0c) subver 0x0000
Advertising successfully started
Indicate VND attr 0x802ff78 (UUID 12345678-1234-5678-1234-56789abcdef1)


refactor(packages/dds/merge-tree): clean up no-param-reassign violations

#990
Clean up no-param-reassign from "packages/dds/merge-tree/**"

@atiqueahmedziad Thank you!


fix(test): remove timestamp when doing assert.Equal

IMPORTANT: Please do not create a Pull Request without creating an issue first.
Problem:

Solution:

Related Issue:
#3674 similar to #2101
Test plan:

@mergifyio backport v1.1


Issues when using with realsense

These issues are related to 'raw_image_pipeline_node.launch' launch files

If you set input_type to color and turn off all the image processing options (e.g., debayer/enabled, flip/enabled), the topics of debayered and color would still be published
And if you set input_type to color to other values, even if you turn on debayer/enabled, there is still nothing output

(I think this issue is related to the above one) If I set flip/enabled to true and turn off other options, the image would still be debayered. (Here the  input_type is set to color)

topics with postfix slow do not publish messages at a slow rate

Need feature to handle 16bit image (depth image)

Hi @SDADEEC , I added some changes to the branch fix/issue-9. Can you check if they work for you?
You can try the following settings for depth images:
  <!-- General options-->
  <arg name="camera_name"                      default="realsense_depth"/>
  <arg name="input_topic"                          default="/camera/aligned_depth_to_color/image_raw"/>
  <arg name="input_type"                           default="depth"/>

  <arg name="output_prefix"                        default="/camera_corrected"/>
  <arg name="output_frame"                         default="passthrough"/>

  <arg name="launch_prefix"                        default=""/>

  <arg name="output_encoding"                      default="passthrough"/>   <!-- Format of the published image BGR and RGB supported-->
  <arg name="skip_number_of_images_for_slow_topic" default="5"/>     <!-- To publish images at a slower rate -->
  <arg name="use_gpu"                              default="false"/>     <!-- To publish images at a slower rate -->
  <arg name="debug"                                default="false"/> <!-- to export debug data to /tmp -->

  <arg name="disable_compression_plugins"          default="true"/>     <!-- To avoid publication of theora or compressedDepth topics -->

  <!-- Modules -->
  <arg name="debayer/enabled"                      default="false"/>
  <arg name="flip/enabled"                               default="true"/>
  <arg name="white_balance/enabled"           default="false"/>
  <arg name="color_calibration/enabled"       default="false"/>
  <arg name="gamma_correction/enabled"   default="false"/>
  <arg name="vignetting_correction/enabled"        default="false"/>
  <arg name="color_enhancer/enabled"         default="false"/>
  <arg name="undistortion/enabled"              default="false"/>

  <!-- Camera -->
  <!-- Debayer  -->
  <arg name="debayer_encoding"                     default="auto"/>  <!-- auto, bayer_bggr8, bayer_gbrg8, bayer_grbg8, bayer_rggb8-->
   <!-- Flip  -->
  <arg name="flip/angle"                           default="270"/>  <!-- 90 (clockwise), 180, 270 (counter-clockwise) -->

An example of the output:

For color, you only need to change these lines:
  <arg name="input_topic"                          default="/camera/color/image_raw"/>
  <arg name="input_type"                           default="color"/>
  ...
  <arg name="output_encoding"                default="RGB"/>

The output should look like this

5. (ReadMe update) Need to add glog_catkin package to workspace to build

Regarding this part, this was in the README already: https://github.com/leggedrobotics/raw_image_pipeline#requirements-and-compilation
Perhaps I can make it more visible or move it upwards in the README

Closing this as it was merged in #11


📝 Add external link: How to Create A Fake Certificate Authority And Generate TLS Certs for FastAPI

How to Create A Fake Certificate Authority And Generate TLS Certs for FastAPI

Great, thanks for your contribution! :cake:


Any plans on making this work for UWP?

Awesome package.
But are there any plans on making this work for UWP?
Each time I try to connect to a device, or disconnect, or run device discovery, it fails.
Thanks,

Hi @dgerchcovich,
I'm the friendly issue checker.
It seems like (100.00 %) you haven't used our issue template :cry: I think it is very frustrating for the repository owners, if you ignore them.
If you think it's fine to make an exception, just ignore this message.
But if you think it was a mistake to delete the template, please close the issue and create a new one.
Thanks!

This is a duplicate of https://github.com/dotnet-bluetooth-le/dotnet-bluetooth-le/issues/14.
@dgerchcovich I think you can close this one. I didn't try but https://github.com/dotnet-bluetooth-le/dotnet-bluetooth-le/issues/14 suggests that UWP should be working from version 2.2.0 onwards.

This is a duplicate of #14.

True.

I didn't try but #14 suggests that UWP should be working from version 2.2.0 onwards.

There has never actually been a final 2.2.0 release, but UWP support has been available since the 2.2.0-pre releases, and it is part of the stable release 3.0.0 now.
However, it may well be that individual features do not work fully yet. If you find any issues with the UWP support in 3.0.0, please open a new issue. I'm closing this one.


Add create_time field

create_time describes when the task was created, which is very useful for finding tasks later, and for sorting tasks in a dashboard.
I went with create_time in order to by consistent with the start_time and end_time naming.

👍

👍 (creation_time sounds better)

Changed to creation_time


Run as non-root

Our new kubernetes security doesnt allow root containers, and is unhappy about the roboprop image because it doesn't set a non-root user.

@petterroea can you do a regular merge into main when you do this. Otherwise fast forwards get confused when putting dev back in line with it.

@tomonorman Ok sorry, no problem.

danke danke, nothing to apologise for!


Using OrbitDB to create a distributed website with user functionality

I'm trying to understand how to use OrbitDB to create a distributed website similar to social media. The plan is for the entire site to run within the browser, the JavaScript files embedded in the HTML will be doing all the work. I already got the base setup running using the browser example provided by Orbit, which allows my browser to read and write a database through IPFS without having to run extra software in the background.
The next step is for me to understand how I'm going to use and structure the Orbit database to create a functional user system, allowing people to create an account and make post under it while other users can add posts to playlists and comment on them. I'm still new to Orbit and have a hard time understanding the exact structure of the functions I'll need to use.
When an user registers, they're given the name of the new database as well as its password. That database must have 4 overall functionalities: Parts only the owner can read (eg: private messages), parts only the owner can modify (eg: creating posts, changing profile information), parts that anyone may read (eg: the contents of posts, profile information), and parts that anyone may modify (eg: comments on your posts). Parts that are readable and / or writable only to the owner must be protected by the password / key and securely encrypted to anyone who doesn't have it... parts that are readable and / or writable to anyone must be public.
So first I will need functions that let the user create an account (sub-database) protected by a password or private key... after that I'll need functions that can read and write various data and fields in this database, some only accessible to the owner while others to anyone else. For instance, if you're looking at a post made by an user, the site script will need to tell Orbit "get data from orbitdb/the_site/the_user/the_post". Ideally posts and profiles can be stored as json files, from which you read parameters like the date or content or attachments as variables in the object... if not I'm also fine with the database storing each value as an individual boolean / float / integer / string.
Can anyone please point me to any examples that can make getting started easier? If someone already attempted such a project, I'd like to see a simple showcase of it. Otherwise I'd like a list of all the OrbitDB function calls I'll be using in my site scripts for each of those functionalities. Thank you.

https://github.com/orbitdb/orbit-db/blob/master/GUIDE.md#access-control
I've been reading this section of the GUIDE.md to better understand how securing write access to a database works. I now get the general functionality, but also see huge issues to overcome.
Databases can't be secured using an username + password or public + private key pair as I had hoped. Security is based on database ownership, which is itself determined by a key. You obtain this key using orbitdb.key.getPublic('hex') which outputs the public key of your OrbitDB instance. I have several questions about this key and how it's used.
First of all, I wish to know whether it's the same as your IPFS peer key. Is the Orbit public key essentially the key that identifies your IPFS daemon in the swarm?
Second and most importantly, I'd like to know how device portability can be achieved with such a model. Say I create a secure database using one web browser on one of my computers. Now I wish to modify it from another computer, or even another web browser on the same computer. The keys will not match! It would be ridiculous to ask users to create an account for every browser and device they use. There are only two solutions: Either users can mirror their identifying key across all browsers and devices (but then anyone could do it and gain access) or the device that created the database can give write access to other keys (without requiring a new database to be created). What is the solution to this problem?

Thinking about it now, there may actually be some good in this system: Imagine users never having to register with an username and password at all, but automatically being recognized by their account when viewing it from a device that belongs to them. This is much more convenient while also remaining pretty much secure.
But for that model to work, devices need to be able to grant or revoke access to other devices. I understand you can update the access field of an Orbit database, however that requires generating a full copy of the whole database. Apart from that meaning that peers have to re-download all of the data, pointers to the old database would become invalid and you essentially have to update several external entries. There must be a more flexible way to add or remove access keys in an existing database!

howdy@MirceaKitsune! Yes, dynamic access permissions are definitely needed and are under active development.
As you noted, if you just want databases that only the owner can edit, or anyone can edit, you can already accomplish this goal with OrbitDB. You can also currently grant access to a group of people, if you know them and have access to their OrbitDB keys at the time of creating the database.
To change devices, you will have two options, after the dynamic access permission branch lands. You will be able to govern access with another Orbit database, or with a smart contract.
You should also be able to store your data essentially as JSON, using a key-value database, for instance.
It would be great to have a starter project, the OrbitDB equivalent of create-react-app, but this is not there yet.
I hope this helps, please keep asking questions - here or on #orbitdb on IRC is the best place to ask.

@coyotespike Thank you for clarifying! I'm looking forward to this branch... this functionality is essential for a secure yet portable account system to be possible through Orbit.
I'd like to know where I can try out this branch, and how far the development of this feature has gotten at this stage. Can this feature be tested anywhere at the moment?
If so I would also like to know whether its API is documented anywhere. The existing API.md obviously doesn't include it, and I'd like to see exactly how this will work so I can plan in my mind how I'm going to use it. Being able to use one database to establish access rights to another database will be very useful, for instance allowing comments on posts which are stored in their own database but should only be editable by the owner of the poster account.

Yes! Check out the proto/dynamic-access branch in this repo. That has tests and basic functionality for an Orbit access controller (to be clear, OrbitDB currently uses an IPFS access controller). The contract access controller is still WIP.
There is no API documentation yet, sadly.

@coyotespike Should we open an issue to add API docs for that?

Yeah, I think eventually, maybe wait a bit for the branch to get closer to being merged though.

@filips123 Custom ACLs and dynamic access are now part of the 0.20.0 branch, which is now, finally in RC!
You can use npm i -S orbit-db@rc3 to get the latest code but keep an eye on releases if you're using the release candidate! They will be changing quick.

@aphelionz When will they be stable?

We're in RC now which you know means testing, finding bugs, fixing bugs, repeat. I'm hesitant to give a stable date but we will be announcing on Gitter and here as soon as we can be concrete.


Detection results points rotated on iOS

Hi!
Was just updating to the latest version and have run into an issue on iOS.
it looks like the coordinates for the results are (not?) rotated, tested this on iphone 12 and 15 pro and ios 17.5 and 18 beta. Works good like before on android.

The recent Vision Camera version added support for orientation: https://github.com/mrousavy/react-native-vision-camera/pull/2932#issuecomment-2154196434
I have to rotate the camera frame to match the preview. It is tested using vision camera v4.3.2.

Thanks for the quick response.
Appologies, I thought I posted my versions but yeah I'm running vision-camera v4.3.2 too.
I can't see where you rotate the camera frame in the example app? Can you please send a link or what am I missing 🙈

Well, it is done in the plugin's native code. You'd better lock the orientation and try again.
https://github.com/tony-xlh/vision-camera-dynamsoft-document-normalizer/blob/b80154bfd17329d9a9e88143a44899352a6352f7/ios/DetectionFrameProcessorPlugin.swift#L40-L49

Aha, so it is handled in the native code and I should not have to change the frameprocessor or style?

You don't have to change the js code.

you mean this? https://github.com/tony-xlh/vision-camera-dynamsoft-document-normalizer/blob/main/example/src/components/Scanner.tsx#L50-L72

before it was "if Platform.Android" but now it checks frame width and frameheight instead...

I did notice this and I have made the same change I think 🤔  But if this works I must have messed up something 🙈

You can skip the calculation of frame width and frame height first since we've set a fixed resolution: 1920x1080. Then the frame width should be 1080 and the frame height should be 1920.

okey... this is working for me
		} else if frame.orientation == UIImage.Orientation.up {
            degree = 90.0;


Limit multiple builds to release or nightly source

Release or nightly source supports multiple SDKs
Signed-off-by: Sophia Guo<EMAIL_ADDRESS>

Grinder
https://ci.adoptopenjdk.net/view/work in progress/job/Grinder_Platforms/29/console
https://ci.adoptopenjdk.net/view/work in progress/job/Grinder_Platforms/28/console