[Feature Request] bind a shortcut to workbench.action.files.openLocalFile with an overridden files.simpleDialog.enable

Hello. I wonder a way to bind 2 different shortcuts to open files - one in a simple dialog and the other in a system dialog.
Here's my expected usage:

when I want to open files inside a current folder, Ctrl+O and a simple dialog is fastest

Ctrl+P will search in all known folders, so it requires more characters to exactly locate a wanted file

Windows' system open-file dialog is somehow heavy and then much slower than a simple dialog or Ctrl+P

when I want to open a file in an outside folder, the simple dialog in VSCode is not so convenient

it's very easy to jump among deep folders using a mouse on a Windows' system open-file dialog

I've installed some helper softwares for explorer.exe, like Listary which provides:

a much more powerful whole-system quick search panel

a way to add keyboard shortcuts to jump to configurable folders and files at once

However, up to now there seems no way to override the files.simpleDialog.enable in a new key binding item, which makes my experience not perfect as I once expected.
====
Here're some screen snapshots of Listary panel injected into a system dialog:

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


'version' command | Unknown argument "skipBumpOnlyReleases"

Describe the bug
When using lerna-lite version argument '--skip-bump-only-releases' https://github.com/lerna-lite/lerna-lite/blob/main/packages/version/README.md#--skip-bump-only-releases, lerna-lite version does not execute, but fails with:
ERR! lerna-lite Unknown arguments: skip-bump-only-releases, skipBumpOnlyReleases

Using '--skip-bump-only-release' works fine, though.
Related issue #703 => pr #721
Expectation
lerna-lite version executes fine with '--skip-bump-only-releases'.
Reproduction
$ npx -y -p conventional-changelog-conventionalcommits@7 -p<EMAIL_ADDRESS>-p<EMAIL_ADDRESS>\
  lerna version --yes --skip-bump-only-releases --dry-run

ERR! lerna-lite Unknown arguments: skip-bump-only-releases, skipBumpOnlyReleases

Lerna config and logs
No response
Environment Info
$ npm -v
10.8.1
Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion.
[X] The provided reproduction is a minimal reproducible example of the bug.

oh yeah it was originally singular and I rename the option as plural everywhere else but missed the CLI one. Would you like to create a PR to fix it?
https://github.com/lerna-lite/lerna-lite/blob/8d028847e8237a9b8ab94e36b23050aae219629c/packages/cli/src/cli-commands/cli-version-commands.ts#L284-L287

I'm afraid my TypeScript knowledge is insufficient to make good changes in such an extensive project.

it's really just a typo and it is just missing a single character, we can even fix it by using GitHub directly. There's no actual tests for the CLI command itself which is why it wasn't caught earlier (the command has unit tests but not directly from the CLI).


chore: fix typo in archive.py

This PR introduces:

neigbors -> neighbors

Checks

[x] a clear description of the PR has been added
[ ] sufficient tests have been written
[ ] relevant section added to the documentation
[ ] example notebook added to the repo
[ ] clean docstrings and comments have been written
[ ] if any issue/observation has been discovered, a new issue has been opened

Future improvements
[List here potential observations made and/or improvements that could be made in the future. If relevant, open issues for those.]

Thanks for pointing out the typos.


add simd-json and json-rust

@Diggsey I added two more libraries to the benchmark, but it seems like on simd there is some error.

Thanks for the PR! I'll try to look at this ASAP but feel free to bug me if I don't get around to it in the next few days.

I looked into this, and there are two reasons why it errors:

There is some lazily initialized static data from the ahash crate (simd_json -> halfbrown -> hashbrown -> ahash) which was being counted as a leak. I resolved this by forcing this to be initialized outside of the tracked region.

There is a soundness bug in simd_json's AlignedBuf implementation, which triggered mockalloc to raise an error: https://github.com/simd-lite/simd-json/issues/213

@Diggsey it seems like the bug was fixed in simd_json

This is the fix for the mockalloc issue:
 fn test_simd_decode(data: &[u8]) -> (AllocInfo, AllocInfo) {
+    // `simd_json` uses `hashbrown` which uses `ahash` as the default hasher,
+    // which lazily initializes some static data. Force the initialization to
+    // happen early, or else it will be considered a memory leak.
+    let _ = simd_json::OwnedValue::Object(Default::default());

The new dependencies should also be dev dependencies.
I'd be open to a PR in future to make serde_json a dev dependency too - currently it's only used for its Error type.

@Diggsey do you think we should move ahead with this PR?

Happy to merge if the errors are fixed and those dependencies are made dev-dependencies.


TS should use narrowed type when extracting method

See this example:

I would have wanted:
private async migrateWorkspace(to: IAnyWorkspaceIdentifier) {

@bpasero Can you share a link to code that demonstrates this problem?

function migrate(foo: string | boolean) {
  if (typeof foo === `string`) {
    const bar = foo;
    console.log(bar);
  } else {
    console.log(foo);
  }
}

When you extract the block in typeof foo === string, it should put foo: string into the method signature.

Hm, maybe this is a case of then the interfaces are identical in shape:
interface IFoo {
	id: string;
}

interface IBar {
	id: string;
}

function isFoo(obj: any): obj is IFoo {
	return typeof obj === 'string';
}

function migrate(foo: IBar | IFoo) {
	if (isFoo(foo)) {
		const bar = foo;
		console.log(bar);
	} else {
		console.log(foo);
	}
}

Thanks! Yes that appears the be the issue. I opened https://github.com/microsoft/TypeScript/issues/49576 yesterday based on this but could not reproduce the original issue you showed, so thank you for the more minimal repo.
However I think this is expected since isFoo does not narrow the type as you'd expect in the example. If you hover over bar,  it's type is still Foo | IBar
As soon as you add a new property to Foo, then the type check works as expected


[Question] Is transformers.js expected to work with react native?

I've naively been trying to run the transformers js library via react native on android.
Note that onnxruntime-react-native explicitly supports react native, however the transformers.js package depends only on onnxruntime-web and onnruntime-node.
Importing the transformers.js works fine, however as I try to load a model, I receive the error import.meta is currently unsupported from transformers.js.
It would be super convenient to be able to use pipes directly without needing to interface without onnxruntine-react-native directly! If not supported yet, what would need to be done?

Hi there! 👋 React Native is not officially supported (yet), but we do aim to get it working in the future! Fortunately, there is an open fork/PR here: https://github.com/xenova/transformers.js/pull/118, and it seems to works!

Thanks for the info! Looking forward to it.


{!! $variable !!} is the syntax to output unescaped data of laravel not supported

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: latest
OS Version: windows 10 - latest

Steps to Reproduce:

VS Code - {
"resource": "/d:/xamp/htdocs/all-accounting-soft/accounting-new/resources/views/partials/auth/head.blade.php",
"owner": "generated_diagnostic_collection_name#1",
"severity": 8,
"message": "Non-null assertions can only be used in TypeScript files.",
"source": "javascript",
"startLineNumber": 36,
"startColumn": 64,
"endLineNumber": 36,
"endColumn": 74
}

on line no. 36 ->
var aka_currency = {!! !empty($currency) ? $currency : 'false' !!};
It is correct on code for laravel blade. It is showing error here.
If format on save is tick, it will be modified and the the application will break and wont work at all.
so need to untick on format on save.
on format on save, it changes like this - {
!!!empty ....
!!
}
and does not work.
and this -> working code -
var flash_notification = {!! (session()->has('flash_notification')) ? json_encode(session()->get('flash_notification')) : 'false' !!};

changes like this and wont work.
var flash_notification = {
!! (session() - > has('flash_notification')) ? json_encode(session()->get('flash_notification')) : 'false' !!
};

even (session()->has is separated.

You must install a laravel blade syntax highlight extension to get it working

You must install a laravel blade syntax highlight extension to get it working

Non-null assertions can only be used in TypeScript files.javascript
Non-null assertions can only be used in TypeScript files.javascript
Non-null assertions can only be used in TypeScript files.javascript
Property assignment expected.javascript
I installed *laravel blade syntax highlight extension, still same error.

@diego1araujo
Thanks and Glad it worked after I re-started after installing the blade ... extensions. I installed 3 blade extensions.
Before, I had to turn off formatter to save on blade and turn on again for others.


FetchListBySPAsync function ReturnType not used

On FetchListBySPAsync function ReturnType object model is not being used to return ResultSet

It's solved in the latest version.


embedsMany objectId problem

saving embedded models is working but when I get the parent model and try to get the child models I get the ids as -when logging them-
id: {'$id': '53f9e034fa0da4eb426692cf'} as an instance.

Thanks @lucknerjb this solves the problem. I think the issue can be closed..?!

Thanks a lot. solved :)

@lucknerjb I still have this problem when using nested embedsMany relationships. Do you have an solution for this?
For example:
Form ==> embedsMany ==> Fields ==> embedsMany ==> Subfields

@JoeyHoutenbos did you ever find a solution for this?

@digitalturtle I ended up overriding/extending the toArray-function in my BaseModel:
public function toArray()
{
    $data = parent::toArray();

    // Transform IDs
    array_walk_recursive($data, function(&$item, $key) {
        if ('_id' === $key && is_object($item) && get_class($item) === 'MongoId') {
            $item = (string)$item;
        }
    });

    return $data;
}

Be aware that this uses the deprecated MongoId (http://php.net/manual/en/class.mongoid.php).


Duplicate GlobalKey

The relevant error-causing widget was:
TextFormField file:///G:/flutter_app/poem/lib/widgets/poem_input_widget.dart:111:23
When the exception was thrown, this was the stack:
#2      TextInputConnection.setStyle (package:flutter/src/services/text_input.dart:742:12)
#3      EditableTextState.didUpdateWidget (package:flutter/src/widgets/editable_text.dart:1162:29)
#4      StatefulElement.update (package:flutter/src/widgets/framework.dart:4396:58)
#5      Element.updateChild (package:flutter/src/widgets/framework.dart:2977:15)
#6      SingleChildRenderObjectElement.update (package:flutter/src/widgets/framework.dart:5452:14)
...
════════════════════════════════════════════════════════════════════════════════════════════════════
════════ (2) Exception caught by widgets library ═══════════════════════════════════════════════════
Duplicate GlobalKey detected in widget tree.
════════════════════════════════════════════════════════════════════════════════════════════════════

Hi @ryz418
The error is self descriptive,
This platform is not meant for assistance on personal code.
Please see https://flutter.dev/community for resources and asking questions like this,
you may also get some help if you post it on Stack Overflow.
Closing, as this isn't an issue with Flutter itself,
if you disagree please write in the comments and I will reopen it
Thank you

Hi, VladyslavBondarenko.
Thanks for reply.
I want to slove this error.


Update server-side-apply.md

Description of changes
Currently we have 2 sections which are identical, causing confusion. I suggest keeping only one.
Custom resources and Server-Side Apply
Server-Side Apply for custom resources
Created issue for the same.

@Pavaningithub Thanks you for your contribution.
Could you please sign the CLA before the PR can be reviewed.
You can follow the steps documented here: CLA Steps


Test failing after release update

We have some test failing with Preflight Test: Failure - Installed operator version 1.10.0-SNAPSHOT does not match expected version (1.11.0-SNAPSHOT) see https://github.com/apache/camel-k/runs/8185438314?check_suite_focus=true
I think we've seen this already in the past when moving to a new release. I think we should change the update procedure to include a make codegen as soon as we update the makefile.

cc @oscerd


Allow inference for spread operator with variable length arrays

Purpose
$subject.
Fixes #36064
Approach
N/A
Samples
N/A
Remarks
N/A
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[x] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Reviewing


htlcswitch: interceptor watchdog

Fixes a left-over of the htlc interception api implementation that prevents channels from force-closing if a connected interceptor client does not resolve or resume an htlc in time.
Original mention: https://github.com/lightningnetwork/lnd/pull/6212#pullrequestreview-866089355

@carlakc: review reminder
@joostjager, remember to re-request review from reviewers when ready

@Crypt-iQ do you want to assign a second Lightning Labs reviewer to this?

Yes, agreed that adding a height to the interceptor request rpc message is useful. Even just during development. Added.

@joostjager, remember to re-request review from reviewers when ready

@joostjager, remember to re-request review from reviewers when ready

@joostjager, remember to re-request review from reviewers when ready

@yyforyongyu: review reminder

I think this just needs a rebase then it can land.

Rebased

@yyforyongyu: review reminder

GitHub shows a conflict, but doesn't say in which file... Can you try rebasing please?

Done. Problem was the release notes.


Add 64-bit ARM (aarch64) executable for SPV wallet

Pull Request Checklist

[x] 📖 I created my PR using provided CODE_STANDARDS.
[x] 📖 I have read the short Code of Conduct: CODE_OF_CONDUCT.
[x] 🏠 I tested my changes locally.
[ ] ✅ I have provided tests for my changes.
[x ] 📝 I have used conventional commits.
[ ] 📗 I have updated any related documentation.
[ ] 💾 PR was issued based on the Github or Jira issue.

Related Tickets & Documents

Closes #[issue-number] or Related Issue #[issue-number]

Feature/Issue Details
Added a cross-compiled Arm64 bit architecture executable in arm64 folder. Simply chmod+x ./arm64/spv-wallet-arm64 and run ./arm64/spv-wallet-arm64 to start spv wallet. Modify config.yaml same as go implementation.

Potential security risk, nit merging this without a conversation and testing. Any further information you can give would be helpful.

We won't store binary in the repository, there are better ways to deal with binaries, like adding them to release, as this will sync binary with real code version.
With such an idea (especially when it is providing some binaries, that couldn't be reviewed), please first start with creating and issue, where we can discuss solutions.


[L5] MethodNotAllowedHttpException on a existing route.

I'm developping a public API since a couple of month under Laravel 5 and ran into a odd issue : All the methods PATCH, PUT and DELETE aren't working.
My local env : Ubuntu 14.04 LTS, PHP 5.6 (also tried with 5.5), Nginx 1.6.2, and latest L5 commit (https://github.com/laravel/framework/commit/7e0a6472e7cfed16d3635b0da58d14f14a42c2a3).
I'm using Postman extention to test my API and set the requests like this :
POST /v1/feeds/ HTTP/1.1
Host: api.metanews.local
Cookie: laravel_session=eyJpdiI6Iit2M283bnQxcmRcL0F1TjNPa2g3OFwvQT09IiwidmFsdWUiOiJDbUQ0czJkTnV0OXBZNFQrNlN6S1Q4OGZKUlBWVERBRXE3MHhGa1ZDYUVWOG1VV1JMVUFGTEE0ZEpZR0tSOXk4UFl0b1lKZUlIME1RdWFmenBwRkdJQT09IiwibWFjIjoiNTVlYWQ3NjdhMTc4N2Y0MDE3YzY4Y2VkZDdjMmYzMGRhMzE4MDY2ODk3NjcxMWM3Zjg0NjAwOTk0Zjk1Mjc4YSJ9
X-Tenant-Key: stringexemple
Authorization: Basic stringbase64encoded
Cache-Control: no-cache
Postman-Token: fe87f096-09cd-d4c2-6e3d-eb6f1ccb7790
Content-Type: application/x-www-form-urlencoded

_method=PATCH

I also tried with curl, and had the same issue...
This kind of requests used to work. I didn't change my router.php since and nothing has change in the laravel's Router layer at my knowledge....
Routes are well-listed with the php artisan route:list (and I cleared any route cache just to be sure) but HTTP requests != (GET|POST) keeps failing.
My sample routes.php
<?php

$router->group(['prefix' => 'v1', 'namespace' => 'V1', 'middleware' => ['tenant', 'auth.basic', 'authorize']], function($router) {
    $router->get('feeds/trashed', ['uses' => 'FeedsController@trashed', 'as' => 'v1.feeds.trashed']);
    $router->patch('feeds/restore/{feeds}', ['uses' => 'FeedsController@restore', 'as' => 'v1.feeds.restore']);
    $router->resource('feeds', 'FeedsController', [
        'only' => ['index', 'show', 'store', 'update', 'destroy']
    ]);
});

php artisan route:list output
| Method      | URI                               | Name                  | Action                                                                             |
| ----------------- | ------------------------------------ | -------------------------- | -------------------------------------------------------------------------------------- |
| GET|HEAD | v1/feeds/trashed            | v1.feeds.trashed  | Metanews\Http\Controllers\V1\FeedsController@trashed |
| PATCH       | v1/feeds/restore/{feeds} | v1.feeds.restore  | Metanews\Http\Controllers\V1\FeedsController@restore  |
| GET|HEAD | v1/feeds                         | v1.feeds.index    | Metanews\Http\Controllers\V1\FeedsController@index     |
| POST         | v1/feeds                         | v1.feeds.store     | Metanews\Http\Controllers\V1\FeedsController@store     |
| GET|HEAD | v1/feeds/{feeds}              | v1.feeds.show    | Metanews\Http\Controllers\V1\FeedsController@show     |
| PUT           | v1/feeds/{feeds}              | v1.feeds.update  | Metanews\Http\Controllers\V1\FeedsController@update  |
| PATCH      | v1/feeds/{feeds}               | v1.feeds.update  | Metanews\Http\Controllers\V1\FeedsController@update  |
| DELETE     | v1/feeds/{feeds}              | v1.feeds.destroy  | Metanews\Http\Controllers\V1\FeedsController@destroy |

And debugger tells me :
MethodNotAllowedHttpException in RouteCollection.php line 207:

in RouteCollection.php line 207
at RouteCollection->methodNotAllowed(array('GET', 'HEAD', 'POST')) in RouteCollection.php line 194
at RouteCollection->getRouteForMethods(object(Request), array('GET', 'HEAD', 'POST')) in RouteCollection.php line 142
at RouteCollection->match(object(Request)) in Router.php line 702
at Router->findRoute(object(Request)) in Router.php line 628
at Router->dispatchToRoute(object(Request)) in Router.php line 604
at Router->dispatch(object(Request)) in Kernel.php line 155
at Kernel->Illuminate\Foundation\Http\{closure}(object(Request))
at call_user_func(object(Closure), object(Request)) in Pipeline.php line 141
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in ShareErrorsFromSession.php line 55
at ShareErrorsFromSession->handle(object(Request), object(Closure)) in Pipeline.php line 125
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in StartSession.php line 53
at StartSession->handle(object(Request), object(Closure)) in Pipeline.php line 125
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in AddQueuedCookiesToResponse.php line 36
at AddQueuedCookiesToResponse->handle(object(Request), object(Closure)) in Pipeline.php line 125
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in EncryptCookies.php line 40
at EncryptCookies->handle(object(Request), object(Closure)) in Pipeline.php line 125
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in CheckForMaintenanceMode.php line 42
at CheckForMaintenanceMode->handle(object(Request), object(Closure)) in Pipeline.php line 125
at Pipeline->Illuminate\Pipeline\{closure}(object(Request))
at call_user_func(object(Closure), object(Request)) in Pipeline.php line 101
at Pipeline->then(object(Closure)) in Kernel.php line 108
at Kernel->sendRequestThroughRouter(object(Request)) in Kernel.php line 83
at Kernel->handle(object(Request)) in index.php line 53

After some research, it seems that the check function in \Illuminate\Routing\RouteCollection has an odd behavior
<?php
/**
     * Determine if a route in the array matches the request.
     *
     * @param  array  $routes
     * @param  \Illuminate\http\Request  $request
     * @param  bool  $includingMethod
     * @return \Illuminate\Routing\Route|null
     */
    protected function check(array $routes, $request, $includingMethod = true)
    {
        return array_first($routes, function($key, $value) use ($request, $includingMethod)
        {
            return $value->matches($request, $includingMethod);
        });
    }

The $value->matches returns true if a declared route is matched, and with some dd(), I discovered that it returns true at some point, but the check function returns null...

I'd imagine it's because you're trying to send a PATCH to /v1/feeds/ instead of v1/feeds/{feeds}...

Woah, I'm so blind & dumb.... thanks anyway.

@devonzara @Charkhan Wow, I just did the exact same thing. Thanks you guys for this.

I ran into same issue...it persist even after appending the id at the end...please help me out...thanks in advance

Hola, tengo el mismo problema solo me deja utilizar y consumir los metodos GET y POST pero el PUT y el DELETE no me deja, que hago


Merge button screws up mod load order

Merge button reverses the order of the mods in the merged module they should go.
The merge button needs to be removed from the menu by default and an option to add it in or just removed completely.

Merge button reverses the order of the mods in the merged module they should go.
The merge button needs to be removed from the menu by default and an option to add it in or just removed completely.

This feature is still experimental - as it is stated here, on GitHub page. The smarter move would be to fix it's behavior - not to disable it completely ;-)

Even so, experimental features shouldn't have a big button on the main window of the exe right underneath START MOD.

Even so, experimental features shouldn't have a big button on the main window of the exe right underneath START MOD.

Fair point!

What is exactly reversed? I have just checked a test merge and there indeed the DXP2 and W40k entries were reversed.
Pre merge
RequiredMod.1 = DXP2
RequiredMod.2 = W40k

Post merge
RequiredMod.1 = W40k
RequiredMod.2 = DXP2

However everything else from the Merge list was applied properly from top to bottom.

You may fix that for now, I think - it still may cause some troubles ;-)


Research automatic version changes for npm publishes

Overview
Let's look into GitHub Actions or custom nx processes for us to be able to automate bumping any package versions thats are deploying to npm
Go ahead and add a build step to check and bump versions on master

@alexkeating @skuhlmann -- didn't add much detail but wanted to create something for this

@earth2travis
Copy Paste from the Docs for the GitHub Action.
It should work with the Mono Repo.
Using Multiple Versions in the Same Repository
It is possible to create additional versions for multiple project co-existing in one repository, for example you may have a Helm chart, database migration, or simply be hosting multiple projects in the same repository and want them to be versioned independently. There are a few settings that can be used to accomplish this:

First, you can set the change_path input to specify a path that will be inspected for changes. Commits which do no change any files in this path will not increase the increment output. In addition, if there are no changes in a given commit with this path specified, the changed value will be false.

Second, the input namespace can be set to create an additional named version. If this value is set, it will be appended (separated by a hyphen) to the end of tags for the version, and only tags with this value appended will be considered when determining the version. The namespace will be pruned from the string output as "version" within the action.

Finally, set different values for major_pattern and minor_pattern than the other projects in order to be able to mark these commits independently.

To use secondary versions in a workflow, simply create additional steps in a job referencing semantic version multiple times. For example, a project tagged like v1.2.3+0-db could be configured like this:

- name: Application Version
  id: version
  uses<EMAIL_ADDRESS>  with:
    change_path: "src/service"
- name: Database Version
  id: db-version
  uses<EMAIL_ADDRESS>  with:
    major_pattern: "(MAJOR-DB)"
    minor_pattern: "(MINOR-DB)"
    change_path: "src/migrations"
    namespace: db


Normal login after fail login

i have a problem in login with phone.
When login fails, the basic (keycloak theme) login page is displayed.
thanks
https://user-images.githubusercontent.com/113419422/189836149-083ec308-94bb-4599-b3ca-8f00931a44d0.mp4

answer : supportPhone in login.ftl in post request is false

your flows config  is wrong, refer readme

@cooperlyt I used version keycloak-phone-provider-21.0.2_phone-2.3.3 and got the same error. I have configured as readme.

Hi @activekey, How did you fix this?

Hi any answer for this? I have same experience for this

@tungboom @cooperlyt @activekey do you fix on this issue?

@GLY-JeffreyTCK I have customized it with another theme and removed supportPhone. It's not necessary for me

@tungboom ok thanks for your reply. Finally i able find another solution to override challenge function to set supportPhone to true in PhoneUsernamePasswordForm.java


Consider a more robust exec approach

exec fails with simple (an natural) approaches:
import cadquery as cq

l = 1
w = 2
h = 3

def box():
    return cq.Workplane().box(l, w, h).edges().fillet(0.1)

show(box())

fails with
  File "<string>", line 9, in box
NameError: name 'cq' is not defined

So let's change it (just for the test, I would never program like that):
l = 1
w = 2
h = 3

def box():
    import cadquery as cq

    return cq.Workplane().box(l, w, h).edges().fillet(0.1)

show(box())

Now, this fails with
  File "<string>", line 9, in box
NameError: name 'l' is not defined

exec just doesn't play well with global variables
Of course, this works:
def box(l=1, w=2, h=3):
    import cadquery as cq

    return cq.Workplane().box(l, w, h).edges().fillet(0.1)

show(box())

However I would need to adapt my style of coding - especially for quick hacks - to the limitations of exec.
Maybe there is a more robust way?

It should be fixed in the last version of pip package / docker image.

It behaves pretty strange:
I start with
l = 1
w = 2
h = 3

Then I change l, save, change w, save and change h and save. Looks good up to now.
However, when I press save then without changes, the view show kind of a random view of the four views I have created with my process, see animated gif, where I just press save before the view changes.

I cannot not reproduce this. Do you use the docker version of cadquery-server or the pip-one?

I use pip. I anyhow have cadquery installed and don't need the whole docker infrastructure on my Mac

... and the pip version allows to load step files. Not sure this is supported with docker where the interpreter runs in the container

It would be great if you could get rid of the server component. My viewer approach uses zmq. There is a version for Javascript: https://github.com/zeromq/zeromq.js It seems that the server component in javascript or typescript running in VS Code can receive messages from python zmq lib.
Then one would only need a show function that does the tessellation and then sends the tessellated arrays via zmq to VS Code.
Just an idea, since for me the server component is as inconvenient as my approach with the separate viewer.
And, with the zmq approach, one could use the Python debugger in VS Code to do step-by-step debugging of the code and every now and then call show to check the interims results.
Just my 2 cent ...

I use pip. I anyhow have cadquery installed and don't need the whole docker infrastructure on my Mac

Yes, this is totally legit, I just want to understand how to reproduce the bug, I don't have it on the docker version.

Then one would only need a show function that does the tessellation and then sends the tessellated arrays via zmq to VS Code.

The solution with zmq is interesting. I made the choice of the server part because of these two benefits:

avoid complex installation: for many end-users, installing cadquery can be difficult, so I liked the idea to run a docker container that "just work";
reduce the rendering delay: since the cadquery server already loaded the cadquery module.

The first point could be eventually solved with https://github.com/CadQuery/cadquery/issues/1048.
I don't see how to adress the second point with the zmq architecture. Do you?

avoid complex installation

I agree, installing CadQuery is not always fun. I have spent quite some time on it in the past. So the docker approach is good, but is a little bit inflexible: CadQuery doesn't have a short term release strategy (2.1. is more than 1 year old and everyone works with the latest master) and with your docker container I am restricted to the version you provide. I cannot quickly pip install the git repo (If OCP doesn't change, pip of Cadquery repo is sufficient).
By the way, can I load e.g. step files with the docker version? Or save STL results for my 3d printer?
I personally don't really trust the exec approach in Python due to the locals, globals handling. To me it feels like one can mess up things easily with unclear die effects. Maybe it is just because I don't understand it enough ...

reduce the rendering delay: since the cadquery server already loaded the cadquery module.

I understand this issue. I solved it in my workflow by working in Jupyter. I import cadquery only once for a session and even better, really complex interims results can be calculated once and then used in the cells below without recalculation. Big benefit for my workflows. I don't want to open the IDE vs. Notebook discussion. Just explain my approach to avoid the loading times (latency is an issue in general, so my latest version of Jupyter CadQuery - nearly ready - will have caching for bounding box calculations and tessellation results and uses multiprocessing for large CAD assemblies. That is a much bigger performance booster than avoiding the import of cadquery. Note this is only available in Jupyter-CadQuery not in three-cad-viewer).
But agreed, it is a difficult decision, and one has to go one route ...

By the way, can I load e.g. step files with the docker version? Or save STL results for my 3d printer?

Yes, using a docker volume: docker run --rm -d -v $(pwd):/home/cq -p 5000:5000 cadquery/cadquery-server, but I agree it's not very practical. But anyway cadquery can now be installed with pip, so use of docker is not really necessary.
I agree that the server and exec approach is not perfect.
I could eventually give the choice between using the cadquery server or not.

One way to overcome the latency is to use "shift-enter" approach in VS Code.
Instead of just saving (very convenient with cadquery-server), one could use Ctrl-A + Shift-Enter (or Cmd-A + Shift-Enter on the Mac). This will incrementally run the code in VS Code in the attached Python interpreter.
Not as easy as your cadquery-server approach, but no exec needed and same speed. Additionally, one could leverage pre-calculated objects by reusing the variables and just selecting the rest of the code before pressing Shift Enter.
Again, no optimal solution, however you had asked about ideas how to avoid the import latency.
However, both the "cadquery-server" and the "shift-enter" approach do not allow step-by-step debugging.

At least from a debugging perspective, I don't feel that the import cadquery is a big issue (at least on my laptop, which admittedly is a fast machine)

Also a "Run without Debugging" is really fast on my laptop:

Given that experience, on my laptop I definitely would prefer to work without cadquery-server and be able to properly
debug my code step by step

I made some work on this!
It consists on a huge change on the cadquery-server behavior.
The exec approach is not used anymore, instead the server executes the CadQuery code and renders the model using three-cad-viewer. It includes a file watcher that reloads the Python code and update the web page when the file is updated.
This approach allows to debug the code in VSCode.
It also makes the project compatible with any IDE and any web browser.
And also allow a user to display the model in an other screen, or even in an other computer on the same local network.
In VSCode, the web page can be displayed using LivePreview extension (ctrl+shift+P -> Simple Browser: Show).
I moved it in a dedicated repository and can be installed with pip install cadquery-server (it doesn't include cadquery itself, which is an extra-dependency).
 :)

Hi @roipoussiere
having been away on vacation for the last two weeks, I how wanted to give it a try.
Unfortunately minify-html 0.10.0 doesn't seem to exist for Mac M1, only 0.9.2.
So I haven't managed to install it. Is 0.10.0 necessary?

https://github.com/wilsonzlin/minify-html/commit/29e7b1c18705cbb534c2d8cabe11f66731524ac0 doesn't sound promising

Cloned it and gave it a try:

On a Mac, port 5000 is used by ControlCenter, hence I use -p parameter
Somehow on my machine it doesn't detect code changes. I use a file test.py and started with cq-server -p 5001 -m test.py. However, when I change and save test.py nothing happens and I have to refresh the browser
I don't understand how I could use step-by-step debugging in VS Code

So I haven't managed to install it. Is 0.10.0 necessary?

I think you can use minify html 0.9.2. And it's only used for export when --minify option is passed to cli.

I don't understand how I could use step-by-step debugging in VS Code

In fact I tried myself and it's not possible currently. The issue is reported here: https://github.com/roipoussiere/cadquery-server/issues/27. The idea is to actually execute the script when using the debugger, instead of reloading it using cq-server.

@roipoussiere Just for information:
The main reason for me to switch to and IDE like VSCode is step-by-step debugging. I have now created my own VSCode extension, which is loosely coupled between Python and the Typescript VSCode extension: The extension starts a web server on port 3939 and the Python show command sends tessellated objects to this web server via HTTP POST. No external server that needs to be run separately.

The CadQuery viewer extension for VS Code can be installed from a vsix file and started via "Ctrl-k v" / "Cmd-k v" (check CadQuery settings in VS Code)
The show command can be installed by installing a wheel, show_object would need to be improved (args)

Both vsix and wheel can be found in https://github.com/bernhard-42/vscode-cadquery-viewer/releases at the moment.
This gif shows both, using "Run in Python Terminal" (very fast for small experimental code snippets) and proper "Debugging" with VSCode debugger:


Update CODEOWNERS in staging branch

Summary of the pull request
References and relevant issues
Detailed description of the pull request / Additional comments
Validation steps performed
PR checklist

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

/azp run


GoogleHome not refresh air conditioner entities

I am using v.0.14.0beta7 and I have tested with stable version v.0.13 but if I try to use GoogleHome
"Hey Google, please poweroff air conditioner in the kitchen"
The air conditioner power off but the enty in HomeAssistant not refresh, result always in ON
Other info:
HA Core: 2024.7.3
Supervisor: 2024.06.2
IPv4 static
IPv6 disabled

I have the same issue, i got home automations on them Homeassistant keeps telling me air conditioner is on but its off

It seems that sometimes the state is not refreshed until you make another change

from version 0.14.0 all sensors will not update autonomously, this prevents normal use of any automation, until the previous release it worked perfectly

Yes, I have the same problem with my wash machine on 0.14.0. all sensors will not update autonomously

For me I also noticed that the version 0.14 does not update the state of Air conditioner when I change in the remote.
I reverted back to 0.13 to use pooling

check https://github.com/Andre0512/hon/issues/269


Pin ansible galaxy installs

Description of the change:
This PR is part of #4237 to work towards a more user-friendly way of doing reproducible local builds
This PR pins versions of Ansible collections installed through ansible-galaxy
Motivation for the change:
Answers https://github.com/operator-framework/operator-sdk/issues/4237#issuecomment-770829132
Checklist
If the pull request includes user-facing changes, extra documentation is required:

[ ] Add a new changelog fragment in changelog/fragments (see changelog/fragments/00-template.yaml)
[ ] Add or update relevant sections of the docs website in website/content/en/docs

There's a yaml linting error at https://github.com/operator-framework/operator-sdk/runs/1902574263?check_suite_focus=true#step:5:924 but it's unclear to me what file triggered the error - the yamllint output doesn't say

https://github.com/operator-framework/operator-sdk/pull/4529/checks?check_run_id=1902574007#step:5:323 is a different error. It produces an error on any diff in the Go code? So how does the process work on changes in the Go code?

/retest

@jberkhahn @estroz How does one solve this "sanity" lock? It's obvious that https://github.com/operator-framework/operator-sdk/pull/4529/checks?check_run_id=1905731587#step:5:309 will produce a diff; it's intentional.

@reinvantveer can you rebase with master after #4594, run make generate, then push? Then lgtm.

That seems to have solved it.
BTW there appears to be a related MR at #4594


Laravel 5: "The model option must be the string name of a valid Eloquent model"

What the title says. I have an User.php class under App/Models. Then the administrator.php under config/, and the settings folder and users.php in config/admin.
My paths are:
'model_config_path' => config_path() . '/admin',
'settings_config_path' => config_path() . '/admin/settings',
'home_page' => 'users',

My users.php has this:
return array(
  'title' => 'Users',
  'single' => 'user',
  'model' => 'User',

However when I try to access the home_page, I get the error:
There are problems with your users config: The model option must be the string name of a valid Eloquent model
I'm using my User model everywhere in the application, and if I die() while inside users.php that die() is called as it was supposed to. Any ideas?

Change 'model' to 'App/Model/User'. Your model is in this namespace.

Yes - silly me. Old code dies hard and I'm still finding my way around L5's namespace conventions. Thanks!

hello,my User.php is under app, but when I change "model" to "app/User",it still went wrong, "There are problems with your user config: The model option must be the string name of a valid Eloquent model"
i have no idea how to work it out .....please save me   T.T

It should be "App\ User" but not "app/User"


GPU detection priority changed

As of the last update, the interface no longer shows my RTX 3090 as the GPU. It instead shows the integrated graphics of my Intel CPU.
Would it be possible to change the GPU detection priority, or to give the user an option to choose which GPU to display?
I see that another identical issue was marked as closed, however the issue still persists.

@alexissorokine I've updated. Can you check if it's working now ?

It's working for me now :)
I'm closing the issue.


Introduce a "run mode"

We use the autoscaler to turn off certain deployments over the weekend. It seems wasteful to keep a process in our cluster that constantly consumes 100MB of memory just for doing its work on Fridays. So, we have set up the autoscaler as a CronJob, which required some minor changes to the startup code.

Looks good thanks @dirk-olmes


Add version check to 3.0 parser to prevent 2.0 docs parsed into OpenAPI

In some conditions 3.0 parser can produce OpenAPI object and it bypasses the 2.0 converter
This lines sometimes produce non null object from v2 documents
OpenAPI api = mapper.readValue(swaggerAsString, OpenAPI.class); result.setOpenAPI(api);

Thank you--can you please add a test to catch the failing condition?

Added test to check that v3 parser returns null if v2 document is supplied

I've looked into this--the correct way to handle the version check during parsing is inside the v3 parser after turning into a JsonNode.  Please see upcoming PR.

@fehguy Hey, I though above using JsonNode at first, but did streaming parsing approach to optimize parsing to skip over children if openapi key is found. JsonNode would parse entire document each time. I think it might be a bit slower with large documents. And in most documents this key is at the top


:four_leaf_clover: Proposal: Plugin github-repo-scaffolding-golang Enhancement (sub-task of #513)

What would you like to add? Why is this needed?
See #513 for the details

Background
Current config:
tools:
- name: github-repo-scaffolding-golang
  instanceID: default
  dependsOn: []
  options:
    owner: YOUR_GITHUB_USERNAME
    org: YOUR_ORGANIZATION_NAME
    repo: YOUR_REPO_NAME
    branch: main
    image_repo: YOUR_DOCKERHUB_USERNAME/YOUR_DOCKERHUB_REPOSITORY

The biggest issue I have with it is that the config example is a bit confusing. Why put both "org" and "owner" at the same time? In fact, only one is required, but from the sample config, it's not straightforward until you read all the comments.
Another issue is the "image_repo" part, what does image repo have anything to do with GitHub repo scaffolding?
Proposals

better documentation, usage example, so that users know what exactly each parameters are
better param naming
maybe put org/owner in a more clear way.

Example:
  org: yes (default no)
  owner: name(can be the name of the owner or the org)
  repo: xxx

Not the best idea, but maybe it's more clear.
A better design is needed here.
Be Brave, Be Imaginative
Anything that isn't following the clean code best practice, refactor it.
Any feature you think would add value to end-users, add it.
Any configuration refactors / documentation updates that would improve the end users' quality of life, do it.
Be brave when deleting unnecessary code, and be imaginative when adding new features.

assgin me

I want to try it. @daniel-hutao @IronCore864
Only for org and owner fields, my thoughts following:

remove org and owner fileds, add new user filed.

This is because github username is a unique identifier, it can only be an authorized user or an organization.
We can identify its type internally, then call the github restful interface to create a repository. With the help of UsersService.Get interface, it is easy to know whether it is a user or an org.

Hi, @tryturned I'm really sorry to reply to you so late. This plugin has been refactored. Should we close this issue now? @IronCore864


fix: indicate peer dependency

Coverage remained the same at 100.0% when pulling daddcba2020cb981fef89c3887bc9fc0c817ff90 on fix-add-peer-dependency into 96541c5d9f7b179c4559a730403da3b71344b7ac on main.

:tada: This PR is included in version 1.2.33 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Fix visual overflow bug in inline code blocks within blockquotes

https://hydephp.com/posts/creating-a-new-hyde-site-from-scratch

Fixed in https://github.com/hydephp/develop/pull/1525


Channel Management

Description
Container to collect ideas, screen flows, etc. for an improved channel management
Acceptance criteria

Tasks

[ ]

Roy's mockups for channel management proposal
All channels for a selected token are visualised as cards. The user shifts between channels by swiping through the cards.
Underneath is a list with the transaction history for the coresponding channel. Similar to what is currently being discussed in issue #426 .
A main menu is added (the + button in the bottom)

The user can "flip" each card (channel) and access options for closing and settling the channel.

The user performs payments and deposits by accessing those options from the main menu.

@taleldayekh and @christianbrb
Proposal:
First step towards the card design. Channels can be shifted left and right.

Questions:

Do we want to keep the automatic selection of the channel with the highest capacity?

We think it is nicer, if the channel

What is the capacity above „TestToken“ on the top?

If it is the total capacity of all channels we keep it

Currently, the channel actually used for transfer is either direct channel (if available) or the one informed by PFS, which can then be selected as per the available routes.
The auto selection is only to show relevant capacity. We need to show a single one, because there's no point into summing them if you can't make a single transfer that big, so we show the highest one which is the one with highest capacity and chance of being picked by PFS.
Notice we didn't put much effort into improving on this because usually it shouldn't matter much, as we expect the common case to be a single channel per token network, so it's the same used for capacity above and the only one pickable by PFS.

The user can "flip" each card (channel) and access options for closing and settling the channel.

What are steps in the UI to close/settle a channel from the slider?

If we, as a first step, only are moving the current channels to the transfer view then perhaps the best is to keep the close/settle as it is in the dropdown for each channel.

Good collection an discussion, but not actionable. Closing.


[NIO] Fix inconsistent failure messages.

The duplicate of https://github.com/GoogleCloudPlatform/google-cloud-java/pull/2096. For some reason travis cannot build the old PR without errors.

LGTM 👍


Find Minimum distance of each node using SSSP on trees by DFS in C

Have you read the Contributing Guidelines on Pull Requests?
YES.
Description
Given total number of nodes in an undirected graph
numbered from 1 to n [ number of nodes] and total number of edges in the graph.
Find  minimum distance of each node using SSSP (Single Source Shortest Path) on trees
by DFS.
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
Fixes #6625

Kindly review @ankitaggarwal23 @HarshCasper
:)


not work coz private mode

all link not work for search, because geckodriver run as private mode, it's mean facebook and other can search image without login first..

If you supply a custom FB Profile List via the --skipfb --facebookList <filename> options, those profiles must be public since we dont login to FB.
So thats not a problem. Make sure that those profiles are public.


Test utility for configuring test LDAP directories

Here's the pull request to address https://github.com/wstrange/dartdap/issues/34.
It turned out better than I thought. Someone can clone the repository and (after running "pub get") can immediately run "pub run test" and it will work - without needing to setup an LDAP directory for testing. It will use the default config and skip all the tests that require a directory. But if they go to the extra effort of setting up an LDAP directory, additional tests can run.
I've also removed the hard-coded "dc=example,dc=com" from the tests, so any LDAP entry can now be used for testing. Just specify the "testDN" to use in the config file. It is all described in the test/README.md.
All of the tests pass, except for the one that that expects the directory to have already been "populated with 2000 sample users".

Really nice!
I think you have merge privs and can push to pub? Let me know if you need any assistance


Update test to show that generated code for field of type enum? does not compile

This draft update the Struct test to reproduce #3663.

Incorporated in #3666.


[raymath] API inconsistencies

Currently there are few functions in raymath that look inconsistent with the rest of the API. Most of the API is built around the idea of returning a new Vector, Matrix or Quaternion. However some of the functions modify the input instead.
I'm aware that I could write my own functions to do that, but thought maybe some other people are affected as well. Similarly, not sure whether this improvement is worth breaking the API.
An example (Note: original code was using raylib-go, so sorry for any C errors):
auto hingePointMin = (Vector3){20, 0, 0};
auto hingePointZero = (Vector3){20, 0, 0};
auto hingePointMax = (Vector3){20, 0, 0};

Vector3Transform(&hingePointMin, MatrixMultiply(transform, MatrixRotateZ(hinge->Zero+hinge->Range.Min)));
Vector3Transform(&hingePointZero, MatrixMultiply(transform, MatrixRotateZ(hinge->Zero)));
Vector3Transform(&hingePointMax, MatrixMultiply(transform, MatrixRotateZ(hinge->Zero+hinge->Range.Max)));

DrawLine3D(hingeCenter, hingePointMin, Red);
DrawLine3D(hingeCenter, hingePointZero, Green);
DrawLine3D(hingeCenter, hingePointMax, Blue);

Which could be:
DrawLine3D(hingeCenter,
	Vector3Transform((Vector3){20, 0, 0},
		MatrixMultiply(transform, MatrixRotateZ(hinge->Zero+hinge->Range.Min))),
	Red);
DrawLine3D(hingeCenter,
	Vector3Transform((Vector3){20, 0, 0},
		MatrixMultiply(transform, MatrixRotateZ(hinge->Zero+hinge->Range.Center))),
	Green);
DrawLine3D(hingeCenter,
	Vector3Transform((Vector3){20, 0, 0},
		MatrixMultiply(transform, MatrixRotateZ(hinge->Zero+hinge->Range.Max))),
	Blue);

Specific functions that look like that in raymath:
RMDEF void Vector2Scale(Vector2 *v, float scale);
RMDEF void Vector2Negate(Vector2 *v);
RMDEF void Vector2Divide(Vector2 *v, float div);
RMDEF void Vector2Normalize(Vector2 *v);

RMDEF void Vector3Scale(Vector3 *v, float scale);
RMDEF void Vector3Negate(Vector3 *v);
RMDEF void Vector3Normalize(Vector3 *v);
RMDEF void Vector3Transform(Vector3 *v, Matrix mat);

RMDEF void MatrixTranspose(Matrix *mat);
RMDEF void MatrixInvert(Matrix *mat);
RMDEF void MatrixNormalize(Matrix *mat);

RMDEF void QuaternionNormalize(Quaternion *q);
RMDEF void QuaternionInvert(Quaternion *quat);
RMDEF void QuaternionTransform(Quaternion *q, Matrix mat);

There of course are few functions that have multiple out parameters, so not sure what to do about those.

Hi @egonelbre! Thanks for the review!
I defined some functions that way for convenience but I agree it breaks consistency with other functions. It also requires some changes all around the code...
raymath is already being reviewed... Maybe it's a good time to change that for the upcoming raylib v2.0?
@a3f, what do you think?

Maybe it's a good time to change that for the upcoming raylib v2.0?

I agree.

Complete review of raymath implemented in commit https://github.com/raysan5/raylib/commit/61e0e4b4f37cc66135445bc87af7c92399fa69ee
It could possible break some examples... need to review that.

I took a quick skim... and is keeping the statements on multiple lines intentional?
For example:
// 1.
result.normal = Vector3CrossProduct(edge1, edge2);
result.normal = Vector3Normalize(result.normal);

// 2.
Vector3 axis = (Vector3){ x, y, z };
axis = Vector3Normalize(axis);
matRotation = MatrixRotate(axis, angleDeg*DEG2RAD);

Instead of:
// 1.
result.normal = Vector3Normalize(Vector3CrossProduct(edge1, edge2));

// 2.1
Vector3 axis = Vector3Normalize((Vector3){ x, y, z });
matRotation = MatrixRotate(axis, angleDeg*DEG2RAD);

// 2.2
matRotation = MatrixRotate(
	Vector3Normalize((Vector3){ x, y, z }),
	angleDeg*DEG2RAD);

is keeping the statements on multiple lines intentional?

No, just the result of a quick editing... your approximation looks better!

Reviewed in commit https://github.com/raysan5/raylib/commit/375adf86a61be8f3b23c253672dfd3c8df805784


Bold text not working

I can't understand why this cannot be parsed, there are no spaces, and I'm not sure if it's in Chinese?
babelmark

This is the relevant section of the commonmark spec: https://spec.commonmark.org/0.30/#emphasis-and-strong-emphasis

A right-flanking delimiter run is a delimiter run that is (1) not preceded by Unicode whitespace, and either (2a) not preceded by a Unicode punctuation character, or (2b) preceded by a Unicode punctuation character and followed by Unicode whitespace or a Unicode punctuation character. For purposes of this definition, the beginning and the end of the line count as Unicode whitespace.

It's essentially saying that the closing ** (right-flanking) have some rules around what characters can be present just before and just after it.
In your case, you have a 。 ("ideographic full stop") character right before the **.
This character is a punctuation character, so if you want the ** to be considered, it must also be followed by a whitespace or punctuation.
If you insert a space right after **, you should see it working again.


Migrate repository to AztecProtocol

This repository is a boilerplate for the E2E workflow covering:

NoirJS compile
NoirJS execute
bb.js prove
bb.js Solidity verifier (generation + deploy)
Solidity verify

Following https://github.com/noir-lang/noir/issues/4960, Noir covers up to Step 2 of the workflow and proving backends tackle the rest.
This repository should hence be migrated to AztecProtocol, where it can integrate and test with bb.js more easily.
Open questions

Should this migrate as a standalone repository in AztecProtocol, or into AztecProtocol/aztec-packages?

Latter enables CI tests with bb.js changes, but would it negatively impact DevEx of the boilerplate itself (e.g. harder to clone)?

@signorecello looks like this has been migrated. Do we still want/need to use this repo for testing around steps 3-5 mentioned above? Are those being tested elsewhere?

bb.js tests should already exist in aztec-packages (or should exist there if not), we can close this as completed. Thanks for the work!


Biotech Expansion - Mammalia + Mythic -Support

All xenotypes in both mods are now supported.

Thank you !


how do I read the value returned by GET command?

`use std::str::FromStr;
use redi_rs::RedisPool;
#[tokio::main]
async fn main() -> Result<(), Box> {
let mut pool: RedisPool = RedisPool::from_str("localhost:6379").unwrap();
let _ = pool.max_connections(10).establish_pool().await?;

pool.send_command("SET foo bar").await?;

pool.send_command("GET foo").await?; // how do I get "bar" ?

Ok(())

}`

Hi Raul, there is no such feature since I did not implement it. To be honest, this was a side project I made while I was learning Rust, so this should not be used in production whatsoever.
To answer your question send_command function returns connection. I did not implement any function to read the incoming data but since this function returns the connection that was taken from the pool, you can read from it.
Here is the example code for such functionality: https://github.com/roku-on-it/redi-rs/blob/master/examples/read_from_stream.rs
 let _ = pool.max_connections(10).establish_pool().await?;

    let redis_connection: Rc<RefCell<RedisConnection>> = pool.send_command("SET foo bar").await?;

    let mut redis_connection_guard = redis_connection.borrow_mut();

    let mut buffer = [0; 1024];

    redis_connection_guard.stream.read(&mut buffer).await?;

    println!("Response: {}", String::from_utf8_lossy(&buffer)); // Response: +OK

    let redis_connection: Rc<RefCell<RedisConnection>> = pool.send_command("PING").await?;

    let mut redis_connection_guard = redis_connection.borrow_mut();

    let mut buffer = [0; 1024];

    redis_connection_guard.stream.read(&mut buffer).await?;

    println!("Response: {}", String::from_utf8_lossy(&buffer)); // Response: +PONG

got it, thank you


Arabic Language output is reversed

Hello,
I'm new with tesseract-ocr x3, I'm using the command line to ocr arabic tif. but all words positions are reversed (opposite).
Any help you can provide to reverse ara lang in tesseract x3.
I'm using cygwin to run tesstrain.sh but till now ocr output is reversed.
Many Thanks in Advance.

This is duplicate of https://github.com/tesseract-ocr/tesseract/issues/212

@zdenop, please close this issue.


connections option is being ignored

For some reason, the widget is ignoring the connections option and not showing any social login buttons.
import Auth0LockPasswordless from 'auth0-lock-passwordless'

var lock = new Auth0LockPasswordless(process.env.AUTH0_CLIENT_ID, process.env.AUTH0_DOMAIN)

lock.magiclink({
  primaryColor: "#000000",
  connections: ["github", "twitter"],
  dict: {
    welcome: "Log in"
   }
})

Any ideas?

Figured it out, to show social buttons I should've used socialOrMagiclink instead.


Swashbuckle 4.1 - Authorizations not appearing in SwaggerApi

Hi,
In our SwaggerConfig, we have set Basic Authorization using.
c.Authorization("basic", new Authorization { Type = "basicAuth", PassAs = "header" });
When I set a break-point and look at the swaggerApi object passed into the onComplete function for SwaggerUI, I see that authSchemes, authorizationScheme and authorizations are null or undefined.
... works as expected for 5.0, with authSchemes being populated.

More information ... appears to only be affecting services with multiple APIs. If I have a services with only a single API then the objects are being populated correctly.

Confirming in the morning if it also happens with later versions of Swashbuckle

Did you get to the bottom of this?

Closing for now due to inactivity, please re-open if you're still experiencing this issue


(Figma component) When replacing a list icon button with a toggle control, the toggle knob is incorrectly positioned

Please provide a reproduction of the bug:
Add a list component (I'm using 01. Single-Line / 01. Standard / ⚪ C. With Icon - Light), select the trailing icon which should be a 24x24 square frame, replace it with a non-square component like a toggle control.
Actual behavior:
Knob is centered
Expected behavior:
Knob is on one of the edges of the track, not centered.
Priorities and help requested:
Are you willing to submit a PR to fix? Yes
Requested priority: Normal
Products/applications affected: Figma design library

which fluent version are you using?
this issue doesn't seem to repro now. Please feel free to reopen if it's still happening.
Closing this issue for now!
Thanks!


local_exp has only one feature index but many weights

Using the following code with the explainer,
explanation = LimeH2OHI3G.explain_instance(dataRow, h2o_drf_wrapper.predict_proba, num_samples=100, num_features=6)   explanationlist = explanation.as_list()
I get this error TypeError: only size-1 arrays can be converted to Python scalars   originating from this line in as_list() : ans = [(x[0], float(x[1])) for x in ans]
digging a bit i found out that local_exp in explanation has 6 wights but only one value for the feature indexes. I tried many combinations of num_features and num_samples but no luck.

Can you paste the line you used to initialize the explainer?

I am loading the explainer from a file as it is used in a separate environment that the training one.
with open('/LimePredict/XLimeH2OHI3GModel', 'rb') as f:
        LimeH2OHI3G=dill.load(f)
        explanation = LimeH2OHI3G.explain_instance(dataRow, h2o_drf_wrapper.predict_proba,num_samples=10, num_features=6)
        explanationlist = explanation.as_list()

Ti initialize it i am using this line
explainerH2OHI3G = lime.lime_tabular.LimeTabularExplainer(XLimeH2OHI3G ,mode='classification', feature_names = feature_names05, class_names = ['0','1'],
                                                   categorical_features=categorical_features05,
                                                   categorical_names=['FAC_CODE'], discretize_continuous=False)

I'm having a hard time reproducing it. Can you generate any explanations with this explainer?
exp.local_exp should be a dict of labels to list of (feature, value) tuples.

Thank you for investigating it. To be more specific, the explainer work in my development environment, on JupiterLab and a running H2O cluster.
But as I said when using it on a separate environment by loading it and executing it against remote H2O server through API, it haven't generated any valid explanation.


Sb 11168

Codecov Report

Merging #475 into release-1.15 will not change coverage.
The diff coverage is n/a.

@@               Coverage Diff               @@
##             release-1.15     #475   +/-   ##
===============================================
  Coverage           40.72%   40.72%
  Complexity            775      775
===============================================
  Files                 110      110
  Lines                7637     7637
  Branches              859      859
===============================================
  Hits                 3110     3110
  Misses               4187     4187
  Partials              340      340

Impacted Files
Coverage Δ
Complexity Δ

...n/java/org/sunbird/common/models/util/JsonKey.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


WIP: ATS-707: PoC using local (latest) only - untested hence do not merge ;-)

note: for reference only - will be updated to use 2.2.0-A1 once available

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

JFYI  (cc @alandavis @eknizat) I just tried legacy only (pointing to latest all-in-one) but unfortunately that does not work, at least with current "alfresco/alfresco-content-repository-community:6.2.0-ga" due to:

alfresco-transform-aio_1  | 2020-04-08 08:10:17.350 ERROR 1 --- [nio-8090-exec-8] o.a.transformer.TransformController      : Request parameter 'targetMimetype' is missing
alfresco-transform-aio_1  |
alfresco-transform-aio_1  | org.springframework.web.bind.MissingServletRequestParameterException: Required String parameter 'targetMimetype' is not present
alfresco-transform-aio_1  | 	at org.springframework.web.method.annotation.RequestParamMethodArgumentResolver.handleMissingValue(RequestParamMethodArgumentResolver.java:204) ~[spring-web-5.2.5.RELEASE.jar!/:5.2.5.RELEASE]

For reference, here's the diff to force testing of legacy urls:

            -Dlocal.transform.service.enabled=false
            -DlocalTransform.transform-aio.url=
            -DlocalTransform.pdfrenderer.url=
            -DlocalTransform.imagemagick.url=
            -DlocalTransform.libreoffice.url=
            -DlocalTransform.tika.url=
            -DlocalTransform.misc.url=

            -Dlegacy.transform.service.enabled=true
            -Dalfresco-pdf-renderer.url=http://alfresco-transform-aio:8090/
            -Djodconverter.url=http://alfresco-transform-aio:8090/
            -Dimg.url=http://alfresco-transform-aio:8090/
            -Dtika.url=http://alfresco-transform-aio:8090/
            -Dtransform.misc.url=http://alfresco-transform-aio:8090/

It would be great if we could promote https://github.com/Alfresco/acs-community-deployment/tree/feature/repo-4710_remove_legacy_transforms to master (as part of next release of ACS Community) ?

I will close this PR without merge since it is now superseded by https://github.com/Alfresco/acs-community-deployment/pull/105.


plugin-v2 improvements

This issue is for documenting improvements to net-plugin (Docker plugin v2).

Avoid retrying to connect to removed from Swam peers (context: https://github.com/weaveworks/weave/pull/2929#discussion_r115210753).

Extend the smoke test. Test multicast, encryption, weave status output.


Getting 500 exceptions while running app locally

This is the first time I am trying to run the app locally. I cloned the repo and ran bower, gulp etc. I can start the app but all rest calls made by the code for e.g. below, throw a 500 exception
https://public-rest3.bullhornstaffing.com/rest-services/{{my_corp_token}}/search/JobOrder?query=(isOpen:1)&fields=id,title,publishedCategory(id,name),address(city,state),employmentType,dateLastPublished,publicDescription,isOpen,isPublic,isDeleted&count=20&sort=-dateLastPublished&start=0
comes back as
{
errorMessage: "An internal error has occurred",
errorMessageKey: "errors.internalError"
}

The app is running under localhost:3000

As there are steps to ensure the public API can be used, I wanted to check, have you contacted Bullhorn Support to enable use of your career portal?  That is a prerequisite action that has to be done for your jobs to display.

Someone from BH support is on this case but they haven't mentioned anything about enabling public api's yet. I have access to the control panel, I don't see public api section anywhere. There is a Web services API section with key etc. but I think that is for server to server calls.

Hello! I communicated with the representative assisting you in Bullhorn Support.  If you retry the request it should work properly for you without the errors.  They will be following up with you via email shortly. Please let me know if you have any more questions.


Combo field with taginfo for road networks

Added a variation of the combo field for road networks. If the field’s value is blank or shares a prefix with the current country code, search taginfo for values beginning with the country code and a colon.

network values for road routes generally begin with the country code (either uppercase or lowercase) followed by a colon. The one big exception is Ontario: 2,023 are relations tagged network=ca_on_county, so overall there are 1,500 elements tagged network=CA:* versus 2,429 tagged network=ca_*. On the other hand, 2,478 bus stops are tagged network=co:dc:sitp, which does begin with the country code and a colon. Also, a few popular values in Europe are unqualified letters. Despite these exceptions, I think it’s reasonable to suggest values beginning with the country code and a colon; if local standards differ, the user is free to enter any other value.
/cc @bhousel

This same field could be used for cycle_network, where there is even stronger consensus on prefixing networks with the country code and a colon.

Added the same functionality to the cycle_network field.
I also noticed taginfo is returning matches anywhere in the tag values, not just prefix matches. Here’s what the cycle_network field looked like in Canada:

So I fixed that by filtering after the fact:

I played around with this and I think it's great, thanks @1ec5 👍
Aside: surprised that US:TX:FM ranks so highly.

Yes, by sheer number (and mileage), the Texas Farm to Market and Kentucky Route networks are huge. Unfortunately, there isn’t a good way to rank the U.S. networks by importance just by doing string processing.

network values for road routes generally begin with the country code (either uppercase or lowercase) followed by a colon.

This really varies. When I looked previously colons were mainly used in the US, but usage has shifted somewhat.
Of course, the most common network tag values are lwn, rwn, rcn, and lcn.

When I looked previously colons were mainly used in the US, but usage has shifted somewhat.

Colons are used almost everywhere these days. In the past, the U.S. was unique for using colons to delimit a sort of “reverse DNS” rather than the older XY:national scheme. But other countries have adopted the U.S. scheme too (Brazil, Canada, India, Italy, and Slovenia the last time I checked).

Of course, the most common network tag values are lwn, rwn, rcn, and lcn.

iD uses a different field to represent the network tag for non-road route relations: #3281. A typical user should never see the letters “lwn”. With this PR, the separate cycle_network field pulls the most common values for that tag.


Don't shadow built-in symbols

Description
Don't shadow built-in symbols
Next step in separate PR: enable this check on CI
Type of change

[x] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library

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

Project coverage is 91.89%. Comparing base (4f81c36) to head (21dfbeb).
Report is 22 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #787      +/-   ##
==========================================
- Coverage   94.67%   91.89%   -2.79%
==========================================
  Files          53       55       +2
  Lines        1935     2010      +75
==========================================
+ Hits         1832     1847      +15
- Misses        103      163      +60

Files
Coverage Δ

ols/app/endpoints/ols.py
96.87% <ø> (ø)

ols/app/models/config.py
95.93% <100.00%> (+0.02%)
:arrow_up:

ols/src/query_helpers/docs_summarizer.py
100.00% <ø> (ø)

... and 8 files with indirect coverage changes

/retest

/unhold

/retest

/retest

/lgtm

/unhold

/approve

/retest

/approve


Gateway API support

The ingress API was never complete or flexible enough that many ingress controllers implemented their own custom resource to meet more end user use cases (Gloo, Kong, etc.).  But this also locks your deployment to support a particular ingress controller and porting such app to AGIC difficult or impossible.
The Kubernetes ingress API specification is planned to be replaced by the more complete Gateway API:  https://gateway-api.sigs.k8s.io/
Several ingress controllers in the CNCF space already have initial support for the Gateway API (see https://gateway-api.sigs.k8s.io/implementations/).  This feature request is to have AGIC support for this Gateway API.

@siegenthalerroger Azure did but it is implemented in a new product, still in preview:  https://learn.microsoft.com/en-us/azure/application-gateway/for-containers/overview


confusing error message from front-end related to mispatched function types

void foo<T extends num>() {
  print(T);
}

void bar(void Function<T extends String>() f) {
  f();
}

main() {
  bar(foo);
}

CFE reports:
Error: The argument type '<T extends dart.core::num = dart.core::num>() → void' can't be assigned to the parameter type '<T extends dart.core::String = dynamic>() → void'.
Try changing the type of the parameter, or casting the argument to '<T extends dart.core::String = dynamic>() → void'

These dart.core::num = dart.core::num and dart.core::String = dynamic are extremely confusing because they don't map to any valid Dart syntax.

This encapsulates a lot of our problems (except the #lib1 #lib2 issues).
Ideally we'd get produce something like void Function<T extends num>() can't be assigned to void Function<T extends String>() because the bound String can't be assigned to the bound num.
We'll try to make incremental improvements when we can, and embark on a very concerted effort to improve errors after Dart 2.1.

Fixed by c959bd98afb1d24cd944388040fa913a19021620.


Nordland dataset

Hi, I am unable to use the download_nordland.py since the link for downloading the dataset in the python file that hosts the nordland dataset is not working. Is the dataset hosted by AnyLoc available at different link?
Error:
raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: HTTPSConnectionPool(host='cloudstor.aarnet.edu.au', port=443): Max retries exceeded with url: /plus/s/8L7loyTZjK0FsWT/download?path=%2F&files=summer.tar.gz (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x0000021D03DC58B0>: Failed to establish a new connection: [WinError 10060] A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond'))

Hi @nobeldang, can you please try the following download links:
wget -O summer.tar.gz "https://universityofadelaide.app.box.com/index.php?rm=box_download_shared_file&shared_name=zkfk1akpbo5318fzqmtvlpp7030ex4up&file_id=f_1424421870101"

wget -O winter.tar.gz "https://universityofadelaide.app.box.com/index.php?rm=box_download_shared_file&shared_name=zkfk1akpbo5318fzqmtvlpp7030ex4up&file_id=f_1424424688104"

wget -O cleanImageNames.txt "https://universityofadelaide.app.box.com/index.php?rm=box_download_shared_file&shared_name=zkfk1akpbo5318fzqmtvlpp7030ex4up&file_id=f_1424408901067"

Hi @oravus, thank you. This works.
Closing this issue,


[BUG] Text in installer is undercut

Describe the bug
text in installer is undercut

Environment (please complete the following information):

OS and version: Windows 10
Locale: English (UK)
Disassembler and version: v8 (from github)

Note: I think this needs to be fixed in the Wix Toolset installer template.


Make report export show up in new tab

From Sebastian,
"Btw, I know this is not something you are expliciting working on, but I was looking at the Rerpot Export functionality, and even though it looks good, I think it may be a good idea to open that exported Report page in a new tab, instead of the same one. You lose navigation other than the browser buttons in that case, and opening a print version in a separate tab is a behavior I've seen around, including in Gmail."

Addressed in #434


Rule tags - allow hyphens?

YARA 3.5.0
Noted in YARA that rules specifying tags containing a hyphen are invalid. I understand this is due to them following the same restrictions as rule identifiers which follow C conventions for identifier naming.
Sample rule headers and errors:
rule foo : bar baz-qux

/rules/main.yar(1816): error: syntax error, unexpected '-', expecting '{'

Noticed too a hyphenated tag name causes the invalid tag name to match an existing tag if the part before the hyphen matches a valid tag.
rule foo : bar bar-qux

/rules/main.yar(1816): error: duplicated tag identifier "bar"

We use a metadata taxonomy where hyphenated labels are commonplace and I was wondering if this is a necessary restriction in YARA, or can this behavior be changed to allow hyphens in the rule tags?
Possibly same request for the rule identifier - just curious if it's feasible to loosen restrictions on identifiers and allow hyphens in the rule identifier?

The reason for not allowing hyphens in identifiers and tags is because it is used as the minus operator. They could be accepted for tags, as tags can't appear in conditions, but for rule identifiers it introduces problems for the parser, you can't tell if foo-bar is a single identifier or if it means foo minus bar. That's why must programming language don't accept hyphens in identifiers.
In order to keep things simple and coherent I prefer not allowing hyphens in tags and handle them as standard identifiers. There are also some ideas about allowing tags to be used in conditions, and allowing hyphens in tags could be a problem in the future for implementing such features.

Understood, thanks for the explanation.


Helm support on odo

/kind feature
Goal
Developers should be able to use services such as databases, caches, messaging that are installed as Helm charts to the cluster to link with their applications.
Non-goals
Ability to install helm charts using odo is not a primary goal with this enhancement.
Creating helm charts for the application developed is NOT a goal.
Use cases
In order of priority
Link a service installed using a helm chart
odo link command should be able to link a service installed using a helm chart.
List services installed using helm
odo service command should be able to list the helm based installed services.
Restore development state from devfile
odo push should be able to restore the development environment including installation of the required helm components.
Discussion
This enhancement purposely does not include the installation of helm charts via odo. Helm has a very comprehensive CLI and tooling that is familiar for developers. odo should not try to replace those functions provided by the helm CLI but instead complement them on a multi-container, cloud native application development scenario.

Pulling into 2.1 milestone.  This work cannot be started until the refactor work is complete.  Will check back at end of January to re-evaluate status and 2.1 goal.

an open question to the team and maybe even @Gorkem Ercan?
There are operators that internally use helm or like operators can be driven using helm - so wouldn’t that be enough in terms of providing helm support?
issue - https://github.com/openshift/odo/issues/4236
We are also in discussion about allowing an easy way to install operators ( i.e. install using admin rights ) in odo then wouldn’t someone just use a cockroach db helm based operator - install it and use it ? and our current linking feature to that operator would work upfront?

I would say that this won't be enough. This operator uses helm internally, but it is still an operator on the outside.
The point of adding Helm support to odo is to benefit from the Helm ecosystem.
We want users to be able to do just helm install mysql than odo link mysql and have everything ready for the application to connect to the database.

We want users to be able to do just helm install mysql than odo link mysql and have everything ready for the application to connect to the database.
but this a very specific chart that would only have one service which we can detect and link to - that will not be the case always.

I would also like to ask how hard is it to convert a mysql chart to run as an operator using the "helm backed operator strategy of operator sdk"?

but this a very specific chart that would only have one service which we can detect and link to - that will not be the case always.

The same problem applies to the operator backed services. There is nothing that would guarantee that resources created by an operator include Service.

I would also like to ask how hard is it to convert a mysql chart to run as an operator using the "helm backed operator strategy of operator sdk"?

What problem would this solve?

We want users to be able to do just helm install mysql than odo link mysql and have everything ready for the application to connect to the database.

+1

I agree with @kadel. The goal is to be able to support Helm. Helm operator instructions is a mile long, we can not expect developers to  go through it because they wanted to deploy a DB during development.

Implementation of this should not be that complicated.
odo service list should be extended with list of installed charts.
We can check the Helm source code to check how helm list does that.
Or if we are saying that we need Service in order to create a link we can use labels and annotations that Helm is labeling all objects.
For example to list Services that belong to Helm chart you could do something like this
▶ kubectl get svc -l app.kubernetes.io/managed-by=Helm -o jsonpath="Service.name | Helm.release-name{'\n'}{range .items[*]}{.metadata.name}  |  {.metadata.annotations.meta\.helm\.sh\/release-name}{'\n'}{end}"

Service.name | Helm.release-name
mongodb  |  mongodb
my-mysql  |  my-mysql
my-mysql-slave  |  my-mysql

Now we know both the Helm release name and Service.
So now odo link my-mysql can easily generate  ServiceBinding or in this case two ServiceBindigs as there are two Services assuming that SBO can create two SBs for one Deployment (@sbose78 ?). If not users will have to pass extra parameters to odo link narrowing it down to just one Service.

Started putting together how this should work and how it can be implemented.
https://github.com/openshift/odo/pull/4413
It looks like there might be some blockers. Need to investigate further.

I've looked into doing linking using Service Binding Operator. The outcome of my research is that it is not possible to construct ServiceBinding in a way that would allow linking to Helm Chart.
On problem is that there is no standard way how Helm Charts are exposing information about how to connect to the provided service. One way to approach this could be assuming that any Service and Secret created by Helm Chart is providing connect information.
The other problem is Service Binding Operator.
Currently, it is not possible to directly specify Secret or Service as a binding service in ServiceBinding https://github.com/redhat-developer/service-binding-operator/issues/872
We would have to modify resources created by Helm and add annotations and ownerReferences to make SBO work with Helm Charts.
We either need to wait for SBO to implement the necessary functionality or try to implement linking between the odo component and Helm charts ourselves inside odo.

I would like to jump on the issue with slight different variation.
Currently, ODO recognizes services as Operator Backend Backends which are basically CR + deployments.
Service Binding Operator abstracts from that model and will allow to bind any CR - even the one that doesn't have particular deployment.
When considering adding helm it will be good to see if ODO can provide flexible way to define what is service by using labels etc.
For example
app.kubernetes.io/component="external-service"
app.kubernetes.io/managed-by="some-oass"

This will not only support help but also "Other" services.


Update README.md

modify the info of the step machine learn

Changes

Context

Documentation (please check one with an [x])

[ ] I have updated the documentation, or
[ ] No documentation update is required

How I've tested my work (please select one)
I have verified these changes via:

[ ] Code inspection only, or
[ ] Newly added/modified unit tests, or
[ ] No unit tests but ran on a real repository, or
[ ] Both unit tests + ran on a real repository

Code review test - do not rewrite the organization's settings.


Add FASTA export

This was suggested on #741 and would be useful for downstream projects (e.g., ananas).

To clarify, this is for writing the sequences from an RDD[AlignmentRecord] out as FASTA format?  Seems easy enough, perhaps I should tackle it.  Any suggestions to the format of the description line?

I assumed this would be RDD[NucleotideContigFragment] → .fa file.. then the contig names would be all you'd have to write.

Update: NucleotideContigFragment also contains a description field!

But yea if you want to tackle this that would be great @heuermh

Got it, thanks

Agreed; it'd be great for you to take this on @heuermh! It might also be good to write out the sequence dictionary while you do this, since those are popular to carry around.
There are a few nuances around how to actually do this, since genome assemblies and assembled contigs will have different dataset characteristics. E.g., with a finished assembly like hg19, it'd be OK to write each contig as a single partition file. With unfinished assembled contigs, you probably would have too many contigs to want to write each as a partition.

It might also be good to write out the sequence dictionary while you do this

If he's writing a .fa file, where would the SequenceDictionary go?

@ryan-williams You could either write a SequenceDictionary per partition (as an accompanying file) or write a single SequenceDictionary for all files. I think the former is more correct. This would not be an ADAM SequenceDictionary but a Picard/SAMtools *.dict.

I've created a pull request for a new adam2fasta command in #816.
Some possible points for conversation:

I haven't done anything with SequenceDictionary as mentioned above
I'm keying sequences by contig.contigName rather than ReferenceRegion
A reduceByKey depends on the sort order from an earlier sortBy, and I'm not sure that is ok
I'm performing a coalesce(1) before writing instead of using partitioning as mentioned above


How to convert fastflow's onnx model to tensorrt model that supports dynamic batch??

What is the motivation for this task?
I was trying to convert the fastflow model to a tensorrt model that supports dynamic batch, but there was an error when onnx converted the trt model。
Here are my execution steps：
1、Turn onnx into dynamic batch
    torch.onnx.export(
        model.model,
        torch.zeros((1, 3, *input_size)).to(model.device),
        str(onnx_path),
        opset_version=11,
        input_names=["input"],
        output_names=["output"],
        dynamic_axes={"input": {0: "batch_size"}}, # add this line to support dynamic batch
    )

2、export engine
# input is the input name, 1, 4, 8 to be specified manually, and 256 is the output size
trtexec --onnx=model.onnx --saveEngine=model.engine --minShapes=input:1x3x256x256 --optShapes=input:4x3x256x256 --maxShapes=input:8x3x256x256

3、the following is the error message
`
&&&& RUNNING TensorRT.trtexec [TensorRT v8403] # trtexec --onnx=model_dynamic_batch.onnx --saveEngine=model_dynamic_batch.engine --minShapes=input:1x3x256x256 --optShapes=input:4x3x256x256 --maxShapes=input:8x3x256x256
[08/22/2023-18:13:35] [I] === Model Options ===
[08/22/2023-18:13:35] [I] Format: ONNX
[08/22/2023-18:13:35] [I] Model: model_dynamic_batch.onnx
[08/22/2023-18:13:35] [I] Output:
[08/22/2023-18:13:35] [I] === Build Options ===
[08/22/2023-18:13:35] [I] Max batch: explicit batch
[08/22/2023-18:13:35] [I] Memory Pools: workspace: default, dlaSRAM: default, dlaLocalDRAM: default, dlaGlobalDRAM: default
[08/22/2023-18:13:35] [I] minTiming: 1
[08/22/2023-18:13:35] [I] avgTiming: 8
[08/22/2023-18:13:35] [I] Precision: FP32
[08/22/2023-18:13:35] [I] LayerPrecisions:
[08/22/2023-18:13:35] [I] Calibration:
[08/22/2023-18:13:35] [I] Refit: Disabled
[08/22/2023-18:13:35] [I] Sparsity: Disabled
[08/22/2023-18:13:35] [I] Safe mode: Disabled
[08/22/2023-18:13:35] [I] DirectIO mode: Disabled
[08/22/2023-18:13:35] [I] Restricted mode: Disabled
[08/22/2023-18:13:35] [I] Build only: Disabled
[08/22/2023-18:13:35] [I] Save engine: model_dynamic_batch.engine
[08/22/2023-18:13:35] [I] Load engine:
[08/22/2023-18:13:35] [I] Profiling verbosity: 0
[08/22/2023-18:13:35] [I] Tactic sources: Using default tactic sources
[08/22/2023-18:13:35] [I] timingCacheMode: local
[08/22/2023-18:13:35] [I] timingCacheFile:
[08/22/2023-18:13:35] [I] Input(s)s format: fp32:CHW
[08/22/2023-18:13:35] [I] Output(s)s format: fp32:CHW
[08/22/2023-18:13:35] [I] Input build shape: input=1x3x256x256+4x3x256x256+8x3x256x256
[08/22/2023-18:13:35] [I] Input calibration shapes: model
[08/22/2023-18:13:35] [I] === System Options ===
[08/22/2023-18:13:35] [I] Device: 0
[08/22/2023-18:13:35] [I] DLACore:
[08/22/2023-18:13:35] [I] Plugins:
[08/22/2023-18:13:35] [I] === Inference Options ===
[08/22/2023-18:13:35] [I] Batch: Explicit
[08/22/2023-18:13:35] [I] Input inference shape: input=4x3x256x256
[08/22/2023-18:13:35] [I] Iterations: 10
[08/22/2023-18:13:35] [I] Duration: 3s (+ 200ms warm up)
[08/22/2023-18:13:35] [I] Sleep time: 0ms
[08/22/2023-18:13:35] [I] Idle time: 0ms
[08/22/2023-18:13:35] [I] Streams: 1
[08/22/2023-18:13:35] [I] ExposeDMA: Disabled
[08/22/2023-18:13:35] [I] Data transfers: Enabled
[08/22/2023-18:13:35] [I] Spin-wait: Disabled
[08/22/2023-18:13:35] [I] Multithreading: Disabled
[08/22/2023-18:13:35] [I] CUDA Graph: Disabled
[08/22/2023-18:13:35] [I] Separate profiling: Disabled
[08/22/2023-18:13:35] [I] Time Deserialize: Disabled
[08/22/2023-18:13:35] [I] Time Refit: Disabled
[08/22/2023-18:13:35] [I] Inputs:
[08/22/2023-18:13:35] [I] === Reporting Options ===
[08/22/2023-18:13:35] [I] Verbose: Disabled
[08/22/2023-18:13:35] [I] Averages: 10 inferences
[08/22/2023-18:13:35] [I] Percentile: 99
[08/22/2023-18:13:35] [I] Dump refittable layers:Disabled
[08/22/2023-18:13:35] [I] Dump output: Disabled
[08/22/2023-18:13:35] [I] Profile: Disabled
[08/22/2023-18:13:35] [I] Export timing to JSON file:
[08/22/2023-18:13:35] [I] Export output to JSON file:
[08/22/2023-18:13:35] [I] Export profile to JSON file:
[08/22/2023-18:13:35] [I]
[08/22/2023-18:13:35] [I] === Device Information ===
[08/22/2023-18:13:35] [I] Selected Device: NVIDIA GeForce RTX 3060 Ti
[08/22/2023-18:13:35] [I] Compute Capability: 8.6
[08/22/2023-18:13:35] [I] SMs: 38
[08/22/2023-18:13:35] [I] Compute Clock Rate: 1.665 GHz
[08/22/2023-18:13:35] [I] Device Global Memory: 8191 MiB
[08/22/2023-18:13:35] [I] Shared Memory per SM: 100 KiB
[08/22/2023-18:13:35] [I] Memory Bus Width: 256 bits (ECC disabled)
[08/22/2023-18:13:35] [I] Memory Clock Rate: 7.001 GHz
[08/22/2023-18:13:35] [I]
[08/22/2023-18:13:35] [I] TensorRT version: 8.4.3
[08/22/2023-18:13:35] [I] [TRT] [MemUsageChange] Init CUDA: CPU +483, GPU +0, now: CPU 16772, GPU 1205 (MiB)
[08/22/2023-18:13:36] [I] [TRT] [MemUsageChange] Init builder kernel library: CPU +368, GPU +104, now: CPU 17322, GPU 1309 (MiB)
[08/22/2023-18:13:36] [I] Start parsing network model
[08/22/2023-18:13:36] [I] [TRT] ----------------------------------------------------------------
[08/22/2023-18:13:36] [I] [TRT] Input filename:   model_dynamic_batch.onnx
[08/22/2023-18:13:36] [I] [TRT] ONNX IR version:  0.0.7
[08/22/2023-18:13:36] [I] [TRT] Opset version:    13
[08/22/2023-18:13:36] [I] [TRT] Producer name:    pytorch
[08/22/2023-18:13:36] [I] [TRT] Producer version: 1.12.0
[08/22/2023-18:13:36] [I] [TRT] Domain:
[08/22/2023-18:13:36] [I] [TRT] Model version:    0
[08/22/2023-18:13:36] [I] [TRT] Doc string:
[08/22/2023-18:13:36] [I] [TRT] ----------------------------------------------------------------
[08/22/2023-18:13:36] [W] [TRT] onnx2trt_utils.cpp:369: Your ONNX model has been generated with INT64 weights, while TensorRT does not natively support INT64. Attempting to cast down to INT32.
[08/22/2023-18:13:36] [W] [TRT] onnx2trt_utils.cpp:395: One or more weights outside the range of INT32 was clamped
[08/22/2023-18:13:36] [E] [TRT] ModelImporter.cpp:773: While parsing node number 185 [Slice -> "onnx::Add_407"]:
[08/22/2023-18:13:36] [E] [TRT] ModelImporter.cpp:774: --- Begin node ---
[08/22/2023-18:13:36] [E] [TRT] ModelImporter.cpp:775: input: "onnx::Slice_379"
input: "onnx::Slice_401"
input: "onnx::Slice_1636"
input: "onnx::Slice_1637"
input: "onnx::Slice_406"
output: "onnx::Add_407"
name: "Slice_185"
op_type: "Slice"
[08/22/2023-18:13:36] [E] [TRT] ModelImporter.cpp:776: --- End node ---
[08/22/2023-18:13:36] [E] [TRT] ModelImporter.cpp:779: ERROR: builtin_op_importers.cpp:4067 In function importSlice:
[8] Assertion failed: (axes.allValuesKnown()) && "This version of TensorRT does not support dynamic axes."
[08/22/2023-18:13:36] [E] Failed to parse onnx file
[08/22/2023-18:13:36] [I] Finish parsing network model
[08/22/2023-18:13:36] [E] Parsing model failed
[08/22/2023-18:13:36] [E] Failed to create engine from model or file.
[08/22/2023-18:13:36] [E] Engine set up failed
&&&& FAILED TensorRT.trtexec [TensorRT v8403] # trtexec --onnx=model_dynamic_batch.onnx --saveEngine=model_dynamic_batch.engine --minShapes=input:1x3x256x256 --optShapes=input:4x3x256x256 --maxShapes=input:8x3x256x256
`
Describe the solution you'd like
1、The wrong message is that there is a problem with Slice op，but I don't know how to do it.
2、How can fastflow be modified to support dynamic batch ?
Additional context
No response

I'm not sure if anyone from our team is familiar with TensorRT. We would welcome the community support on this.
I'm moving this to a Q&A for now. If it is a really issue, we could convert it back to an issue here.

Thanks response!
I exported the onnx model and use onnxsim.simplify it to convert it to engine model, this could be a reference
    onnx_path = export_path / "model.onnx"
    torch.onnx.export(
        model.model,
        torch.zeros((1, 3, *input_size)).to(model.device),
        str(onnx_path),
        opset_version=11,
        input_names=["input"],
        output_names=["output"],
        dynamic_axes={"input": {0: "batch_size"}},  # add this line to support dynamic batch
    )

    # 简化模型simplify
    model_ = onnx.load(onnx_path)
    model_simp, check = simplify(model_)
    assert check, "Simplified ONNX model could not be validated"
    onnx.save(model_simp, onnx_path)


Option to Zoom and follow cursor in one command

Is it possible to zoom and follow using same shortcut key?

Technically you can bind zoom and follow to the same action. By default, follow is enabled so you don't need to bind anything if you always want following when zoomed in.

ok, thanks


how to pull down to get more data #question

Please how to pull down to get more data

See #525, #840 and #1114.


Simplify the LFS GC logger usage

Remove unnecessary if opts.Logger != nil checks.

For "CLI doctor" mode, output to the console's "logger.Info".
For "Web Task" mode, output to the default "logger.Debug", to avoid flooding the server's log in a busy production instance.

https://github.com/go-gitea/gitea/labels/kind%2Frefactor?


xfce4 on archlinux

This is the output I get when I run it.  I am on arch, so 'python' is python3 (version included below).
$ python himawaripy.py
Updating...
Latest version: 2016/02/04/16:20:00 GMT
Downloading tiles: 16/16 completed
Downloaded
Property "/backdrop/screen0/monitor0/image-path" does not exist on channel "xfce4-desktop". If a new property should be created, use the --create option.
Done!
$ python
Python 3.5.1 (default, Dec  7 2015, 12:58:09)
[GCC 5.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.

Can you please paste the output of the following command:
xfconf-query --channel xfce4-desktop --list

$ xfconf-query --channel xfce4-desktop --list
/backdrop/screen0/monitor0/workspace0/color-style
/backdrop/screen0/monitor0/workspace0/image-style
/backdrop/screen0/monitor0/workspace0/last-image
/backdrop/screen0/monitor0/workspace1/color-style
/backdrop/screen0/monitor0/workspace1/image-style
/backdrop/screen0/monitor0/workspace1/last-image
/backdrop/screen0/monitor0/workspace2/color-style
/backdrop/screen0/monitor0/workspace2/image-style
/backdrop/screen0/monitor0/workspace2/last-image
/backdrop/screen0/monitor0/workspace3/color-style
/backdrop/screen0/monitor0/workspace3/image-style
/backdrop/screen0/monitor0/workspace3/last-image
/backdrop/screen0/monitorHDMI-0/workspace0/color-style
/backdrop/screen0/monitorHDMI-0/workspace0/image-style
/backdrop/screen0/monitorHDMI-0/workspace0/last-image
/backdrop/screen0/monitorHDMI-0/workspace1/color-style
/backdrop/screen0/monitorHDMI-0/workspace1/image-style
/backdrop/screen0/monitorHDMI-0/workspace1/last-image
/backdrop/screen0/monitorHDMI-0/workspace2/color-style
/backdrop/screen0/monitorHDMI-0/workspace2/image-style
/backdrop/screen0/monitorHDMI-0/workspace2/last-image
/backdrop/screen0/monitorHDMI-0/workspace3/color-style
/backdrop/screen0/monitorHDMI-0/workspace3/image-style
/backdrop/screen0/monitorHDMI-0/workspace3/last-image
/desktop-icons/show-thumbnails
/desktop-icons/style
/last/window-height
/last/window-width

Erreur is reproduced here.
Latest Arch/Xfce

So according to what I found on the Internet, there must be property
/backdrop/screenX/monitorX/image-path

that we can change to change the background. I don't know if are there any other way around to do this, and I can't try it myself, since I don't have XFCE installed.
I'd be glad if you can come up with a solution. :)

There is no longer an image-path property. Backgrounds are now set by changing the last-image property, as odd as that may seem. Needed to change
 /backdrop/screen0/monitor0/workspace0/image-path

to
 /backdrop/screen0/monitor0/workspace0/last-image

rbong, that doesn't work for me.
$ python himawaripy.py
Updating...
Latest version: 2016/02/05 19:50:00 GMT
Property "/backdrop/screen0/monitor0/last-image" does not exist on channel "xfce4-desktop". If a new property should be created, use the --create option.
Done!

OK, I see that you have added workspace0 to the path. When I try that, himawaripy.py runs cleanly. I get on error message, but my background doesn't change.

You also need to add /worspace0 after /monitor0
 /backdrop/screen0/monitor0/workspace0/last-image

What is the error message?

There is no error message, and my  background doesn't change.
$ python himawaripy.py
Updating...
Latest version: 2016/02/05 19:50:00 GMT
Downloading tiles: 16/16 completed
Done!

Are you sure you're using monitor0, and not monitorHDMI-0? Also, are you on workspace0? The desired value should be somewhere in the command
xfconf-query --channel xfce4-desktop --list | grep last-image

I am apparently using monitorHDMI-0. It is working for me now!

That's great!


Wrapping of the index exports is inconsistent

Some are exported bound and others are raw builtin values (if they exist).
This may get tricky (an exception to consistency) for things like ./Date.

For packages that contain multiple shims, the individial indexes don't have properties decorated onto them, so a binding isn't required.

Thanks for the clarification. That makes sense.