feat: notes without assets

Closes #596.
Adding the ability to create notes without assets.
The implementation is a bit naive, but I'm happy to change it. Basically, I added a new procedure in tx.masm - create_note_without_asset.  This leads to some code duplication. I can refactor common functions or modules out.
We could also change our normal procedure tx::create_note. The problem is, though, that the TransactionKernel cannot see if there is a malformatted asset or no asset.
There are several other ways to implement this:

We could introduce a special ASSET that we push if the user wants to create a note without an asset.
We could introduce a flag when calling tx::create_note, but then, all other procedures, from the wallet to the kernel API, also need to change.

Have you considered also doing https://github.com/0xPolygonMiden/miden-base/issues/563 ? Another alternative would be to give an asset counter and a list of assets. I'm not sure how to provide the list of assets though, AFAIK we can only have 16 elements in the stack, so it would need to go somewhere else, one possibility is to push them to the advice provider and then give the hash of the assets to the create_note call.

That is an excellent point; maybe I should rather work on https://github.com/0xPolygonMiden/miden-base/issues/563.
One problem is that tx::create_note calls syscall.create_note. This changes the context, and we can only add max two ASSETs since the procedure already takes [tag, note_type, RECIPIENT].
That means, instead of providing the full ASSET, we should provide the ROOT of a tree of ASSETS that go into a specific note. If we add a num_of_assets to the create_note procedure, we can iterate over the tree and use numbers as KEYs.  At KEY 0 we can store the note_idx to ensure we put the correct ASSETs into the right note.
But the overall complexity grows quite a bit. I mean, you need to define before the transaction which assets you want to send with which note. That has implications for the note_args use case as well.
#! Creates a new note and returns a pointer to the memory address at which the note is stored.
#!
#! Inputs: [num_of_assets, ROOT, tag, note_type, RECIPIENT]
#! Outputs: [ptr]
#!
#! num_of_assets is how many different assets the note should contain
#! ROOT is the root of the tree of all ASSETs to send stored in memory
#! tag is the tag to be included in the note.
#! note_type is the storage type of the note
#! RECIPIENT is the recipient of the note.
#! ptr is the pointer to the memory address at which the note is stored.
export.create_note
    syscall.create_note
    # => [ptr, ZERO, ZERO, 0]

    # clear the padding from the kernel response
    movdn.8 dropw dropw swap drop
    # => [ptr]
end

Maybe the way to go here is to change how create_note works so that it takes no assets. Then, we could add assets one by one to the note through something like #228.

that way we could address #563 #228 and #596.
That might be the easiest to simply redefine:
#! Creates a new note and returns a pointer to the memory address at which the note is stored.
#!
#! Inputs: [tag, note_type, RECIPIENT]
#! Outputs: [ptr]
#! ...
export.create_note
    syscall.create_note
    # => [ptr, ZERO, 0]
end

#! Adds an asset to a note
#!
#! Inputs: [note_idx or note_ptr, ASSET]
#! Outputs: [ptr]
#! ...
export.add_asset_to_note
    syscall.add_asset_to_note
    # => [ptr, ZERO, 0]
end

That might be quite a change. Let me check.

Closing this in favor of https://github.com/0xPolygonMiden/miden-base/pull/686 and #674


Add to homebrew

It would be nice to install NPS Browser from brew or brek cask.

this isn't an NPS Browser issue. feel free to issue a PR to homebrew-cask, though i doubt they'd approve it.
https://github.com/Homebrew/homebrew-cask/blob/master/doc/development/adding_a_cask.md#finding-a-home-for-your-cask


Custom manifest filename

Laravel Vite Plugin Version: 0.5.2
Laravel Version: 9.19
Node Version: 16.16.0
NPM Version: 8.11.0
Vite : 3.0.3
Host operating system: Windows 10
Web Browser & Version: Brave 1.41 (Chromium 103)
Running in Sail / Docker: No
Description
Vite allows setting a custom manifest filename https://vitejs.dev/config/build-options.html#build-manifest because the default filename may conflict if you are building a PWA or Chrome extension or whatever
This option is overridden by this plugin (maybe here? https://github.com/laravel/vite-plugin/blob/main/src/index.ts#L105) and always overrides the manifest.json file
Config
import { defineConfig } from 'vite';
import laravel from 'laravel-vite-plugin';

export default defineConfig({
    build: {
        manifest: 'vite-manifest.json',
        emptyOutDir: false,
        outDir: "public/",
    },
    plugins: [
        laravel({
            buildDirectory: "./",
            input: [
                'resources/css/app.css',
                'resources/js/app.js',
            ],
        }),
    ],
});

Output with laravel plugin
> build
> vite build --config vite.config.js

vite v3.0.3 building for production...
✓ 58 modules transformed.
public/manifest.json             0.23 KiB
public/assets/app.4ee680d5.css   0.02 KiB / gzip: 0.04 KiB
public/assets/app.d81e8d40.js    90.63 KiB / gzip: 33.07 KiB

Output without laravel plugin
> build
> vite build --config vite.config.js

vite v3.0.3 building for production...
✓ 58 modules transformed.
public/vite-manifest.json        0.23 KiB
public/assets/app.4ee680d5.css   0.02 KiB / gzip: 0.04 KiB
public/assets/app.f223d706.js    90.63 KiB / gzip: 33.07 KiB

Hi @arthvrian,
If we were to do this, we'd also need to provide a way to specify the custom manifest location on the Laravel side of things too.
It should be fairly straight-forward, especially after https://github.com/laravel/framework/pull/43442 is merged.
Thoughts @timacdonald?

@arthvrian are you actually experiencing this conflict? Is there any specific reason you are trying to put things in the root /public directory?
If you are doing this, isn't it also deleting you index.php, robots.txt, favicon.ico` and everything else that is static in the public directory? I would imagine doing this renders your Laravel application un-usable as it can no longer receive requests?

@jessarcher you are right
@timacdonald I'm just trying to mimic the behaviour of -mix- using -vite-, taking advantage of the speed of -vite- to compile, but leaving the resulting files in the same path that -mix- does/did, so I don't have to modify (for now, a lot of) blade templates
After a few tries, I realized that it's much easier to do it with the plain -vite- config, without involving the laravel plugin (in a new test project, I'll try the actual target project later), but it would be a good enhancement for the plugin
For your last question, that's what this -vite- option emptyOutDir is for, update your compiled assets without removing anything from external sources

It sounds like you've got a workaround, so I will close this for now. I'm not keen on adding a new configuration option to this plugin and to the framework to mimic Mix. We can always revisit it if another use case requires a different manifest name.


reduce allocations

Reduce allocations by storing content as TagHelperContent on ScriptBlock rather than string and using methods accepting IHtmlContent
Add benchmarks and unit tests
Add sample project
Fix instructions on how to add library

Results of benchmark that compares Process method execution on ScriptRenderTagHelper

BenchmarkDotNet=v0.10.13, OS=ubuntu 17.04
Intel Core i7-7700 CPU 3.60GHz (Kaby Lake), 1 CPU, 8 logical cores and 4 physical cores
.NET Core SDK=2.1.300-preview1-008174
  [Host]     : .NET Core 2.1.0-preview1-26216-03 (CoreCLR 4.6.26216.04, CoreFX 4.6.26216.02), 64bit RyuJIT
  DefaultJob : .NET Core 2.1.0-preview1-26216-03 (CoreCLR 4.6.26216.04, CoreFX 4.6.26216.02), 64bit RyuJIT

Method
StringLength
Mean
Error
StdDev
Gen 0
Gen 1
Gen 2
Allocated

Process
1000
1.860 us
0.0348 us
0.0291 us
2.6913
-
-
11.03 KB

NewProcess
1000
1.186 us
0.0385 us
0.1059 us
1.6327
-
-
6.7 KB

Process
10000
8.066 us
0.1608 us
0.3427 us
23.1476
-
-
95 KB

NewProcess
10000
4.646 us
0.0926 us
0.1670 us
13.5117
1.9302
-
55.52 KB

Process
100000
98.133 us
1.8977 us
2.5976 us
249.8779
249.8779
249.8779
798.13 KB

NewProcess
100000
52.534 us
1.0452 us
1.2036 us
124.9390
124.9390
124.9390
407.09 KB

Thanks for your comments.
I reverted the change that put all captured contents under one script tag.
Here are new benchmarks after the change (doesn't contain figures for old implementation since they were presented in the description)

BenchmarkDotNet=v0.10.13, OS=ubuntu 17.10
Intel Core i7-7700 CPU 3.60GHz (Kaby Lake), 1 CPU, 8 logical cores and 4 physical cores
.NET Core SDK=2.1.300-preview1-008174
  [Host]     : .NET Core 2.0.6 (CoreCLR <IP_ADDRESS>, CoreFX 4.6.26212.01), 64bit RyuJIT
  DefaultJob : .NET Core 2.0.6 (CoreCLR <IP_ADDRESS>, CoreFX 4.6.26212.01), 64bit RyuJIT

Method
StringLength
Mean
Error
StdDev
Gen 0
Gen 1
Gen 2
Allocated

Process
1000
1.373 us
0.0274 us
0.0243 us
1.6689
-
-
6.84 KB

Process
10000
4.420 us
0.0882 us
0.2369 us
13.5727
-
-
55.66 KB

Process
100000
45.170 us
0.8774 us
1.1713 us
124.9390
124.9390
124.9390
407.22 KB


feat: Merge v0.7-release to main

Features
Merge v0.7-release to main

Codecov Report
Attention: Patch coverage is 92.59259% with 4 lines in your changes are missing coverage. Please review.

Project coverage is 81.18%. Comparing base (c7b197f) to head (283c7b4).
Report is 389 commits behind head on main.

Files
Patch %
Lines

metagpt/provider/google_gemini_api.py
40.00%
3 Missing :warning:

metagpt/software_company.py
83.33%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1063      +/-   ##
==========================================
- Coverage   81.19%   81.18%   -0.02%
==========================================
  Files         226      226
  Lines       12477    12483       +6
==========================================
+ Hits        10131    10134       +3
- Misses       2346     2349       +3

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

What is the relationship between this PR and https://github.com/geekan/MetaGPT/pull/983?
There are currently many conflicts


Fix incorrect bottom-padding spacing in content section

Fixes

Fixes #953 by @panchovm
Description
Added class padding-bottom-small to content section
Checklist

[X] My pull request has a descriptive title (not a vague title like Update index.md).
[X] My pull request targets the master branch of the repository.
[X] My commit messages follow best practices.
[X] My code follows the established code style of the repository.
[X] I added tests for the changes I made (if applicable).
[X] I added or updated documentation (if applicable).
[X] I tried running the project locally and verified that there are no
visible errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

The problem with this fixed 64px padding is that the size of this area depends a lot on the image resolution. With this 64px padding, some cases end up with too much empty space.

I think a padding-bottom-normal (16px) works best for most cases (big and small images)

@panchovm

Oh yes yes. I am aware of that situation. That's why I reviewed it with a full-width image, just to see how padding/margin was applied.

Hi @panchovm I wanted to follow-up with you here. Do you think the 64px of padding recommended by this PR is acceptable or too tall? I couldn't tell what you and Breno decided.

I still believe that 64px is reasonable to separate the image's information from the tag and related images section (see this reference). That is the first point of this request. But I do agree that it should depend on which section is taller.


default parameters as variables triggers no-unused-vars incorrectly

"use strict";

var bar = 1;

function myFunc({foo=bar}) {}

module.exports = myFunc;

gives a bar is defined but never used no-unused-vars error.
Using eslint 0.21.2 and babel-eslint 3.1.7

Does eslint give you an issue without using babel-eslint as the parser?

@bobbyrenwick can you reproduce with the normal parser?

Closing as I can reproduce with regular parser.


fix: Concourse build badge URL

How to categorize this PR?

/area documentation
/kind cleanup
What this PR does / why we need it:
Pipelines were moved to different concourse-team, thus URLs changed.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

NONE

The flake will be fixed by https://github.com/gardener/gardener-extension-registry-cache/pull/82.

/test pull-gardener-extension-registry-cache-e2e-kind


Use travis container-based infrastructure

See http://docs.travis-ci.com/user/migrating-from-legacy/

Does the Travis run on this PR use the new config?

Does the Travis run on this PR use the new config?

Yes. See https://travis-ci.org/google/closure-library/builds/73896026. "This job is running on container-based infrastructure".


Add derivation to build unison trunk with cabal

I have found a way to build unison trunk with a current nixpkgs.
Sadly because of the random >= 1.2.0 constraint you get a lot of cache misses.
Some of the packages we depend on needed some fixing and are marked broken in nixpkgs.
I can can churn through them and fix them in the next days in nixpkgs. (I am a nixpkgs haskellPackages maintainer.)
I would in principle like very much to get this back into nixpkgs (and could maintain it there) but I am not really inclined to do so as long as we depend on these haskeline and megaparsec forks.
I am also blatantly overriding the containers >= 6.3.0 constraint here. I am sure it has a reason, but it compiles without that. (Und bumping containers would again mean rebuilding 160 packages.)

Thanks again! This looks good to me. It's been running locally for me for a long time now but seems to be on the right track. Once it finishes I'll poke around a little.
This probably currently takes too long to build to test it as part of the CI action. In the longer term, maybe the Unison build can be tweaked a little to hit caches better, or maybe I/we can look into getting a cachix cache set up.

@ceedubs 95% of the rebuilds are caused by the fact that I had to override random == 1.2.0 where random == 1.1.0 is currently in stackage nightly (and nixpkgs). There is an open issue for that at stackage and as soon as they bump and that propagates to our nixpkgs pin we will have a lot more cache hits here.

I am encountering a lot of bugs in unison. I don‘t have enough experience to tell if this has something to do with the build. I am looking into bumping containers to achieve a higher fidelity to the stack build.

Hm, currently no joy … I run into weird problems.
The only sane solution to getting the newer container version would be to wait until nixpkgs migrates to ghc 9.0 …

@maralorn my laptop bit the dust last night, so I haven't gotten a chance to try this out. From what you are saying it sounds like it at least partially works, but needs to build a lot from source; is that right? Since this is a new attribute and not a replacement of the existing unison-ucm, personally I'm fine merging this in its current form and improving it over time. It might just be good to add a comment on the attribute warning that it might not be stable. What do you think?

Sad to hear about your laptop.
The rest seems fine to me.
Sadly when used with '--new-runtime' you get crashes for example when using Map.lookup …
That’s why I now actually installed unison declaratively with stack.

Thanks again @maralorn! I'm going to go ahead and merge this, knowing that there are improvements that may come in the future.


Doesnt work with pagination

@helixaros Please provide a code example of what you are currently doing and what you expect to happen.

Could be wrong, but if I create a table and add it as part of a ListView that uses pagination, would pagination be applied on the table ?
Also if I use a class based TemplateView how can I enable pagination to be applied on the table?

Could be wrong, but if I create a table and add it as part of a ListView that uses pagination, would pagination be applied on the table ?

If the pagination uses the url query parameters to access the pages, then yes, the pagination will be applied to the table.

Also if I use a class based TemplateView how can I enable pagination to be applied on the table?

You declare the table with a pre-paginated queryset, usint the  object_list.
class TableSort:
    def __init__(
        self,
        request: HttpRequest,
        object_list: QuerySet | list,
        fields: list = ALL_FIELDS,
        exclude: list = None,
        column_names: None | dict[str, str] = None,
        field_order: None | list[str] = None,
        sort_key_name: str = "o",
        table_css_clases: str = "table",
        table_id: str = None,
        **kwargs,
    ):


[CSS Isolation] More in depth explanation about the ::deep pseudoelement

The section on the ::deep pseudo selector could use a more in-depth explanation about how deep works. There is no mention to how ::deep affects the transformation of the CSS rule and that makes it incredibly hard to reason about.
We should include a section that describes what ::deep does with a sample "before" and "after" ::deep at several locations in the rule.

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

ID: 2703a48e-8b35-53fc-ac36-cfb395833077
Version Independent ID: 39d464d9-aca6-34fa-7e52-5fde2e393ba7
Content: ASP.NET Core Blazor CSS isolation
Content Source: aspnetcore/blazor/components/css-isolation.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @daveabrock
Microsoft Alias: riande

Hey @guardrex - I see you've got needs-more-info but let me know if you would like me to assist with this update.

Sure, thanks @daveabrock. I think 'needs more info' was a stray click ... looks like I added and then removed it immediately.

OK, give me a few weeks. I should be able to dig into it a little more.

Hey @javiercn - I'm working on this improvement but would like some clarification on other scenarios you are looking for, in addition to what's covered in the section, Child component support.
In that section, I use the ::deep combinator to illustrate how you can inherit identically named styles from parent components. What other scenarios or details are you looking for? Are you looking to expand with a more complex example with multiple children?

Checking in on this one @javiercn ...
:point_up: @daveabrock is asking if that section at Child component support is enough to cover your ask? If not, what else are u requesting for ::deep?

@daveabrock ... Javier is very 🏃😅 with a large workload. We'll circle back around to this later, which might even be post 6.0 GA.


MCD code

Can you share the mcd calculate code? I found some mcd methods, but the results are wrong

Hi,
We don't provide it.


New "Function" syntax and current release status

It might make sense to document somewhere that in stable it is almost completely unusable (due to analyzer crashes, and due to it not being implemented in the VM), and even in dev there are a lot of quality-of-life issues that make using it over the old typedef unsuitable:
EDIT: Re-organized the list of issues as-of 2017/10/15.
Still broken:

https://github.com/dart-lang/sdk/issues/30732
(VM: New typedef doesn't support annotations on parameters)
https://github.com/dart-lang/sdk/issues/30596
(Analyzer: no analyzer syntax error when using the old function type syntax inside a function
type using the new syntax)
https://github.com/dart-lang/sdk/issues/30759
(Analyzer: Implement missing methods and new functions quick fix)
https://github.com/dart-lang/sdk/issues/30858
(Analyzer: Using function types inside of <...>)

Not specific to the new syntax:

https://github.com/dart-lang/sdk/issues/30379
(Analyzer function type parameters are not correctly scoped)
https://github.com/dart-lang/sdk/issues/30180
(Analyzer does not fail on invalid generics use)

Fixed or otherwise closed:

~~https://github.com/dart-lang/sdk/issues/30729~~ (Analyzer: crashes with new typedef + named arguments)
~~https://github.com/dart-lang/sdk/issues/30758~~
~~https://github.com/dart-lang/sdk/issues/30595~~
~~https://github.com/dart-lang/sdk/issues/30375~~
~~https://github.com/dart-lang/sdk/issues/30760~~
~~https://github.com/dart-lang/sdk/issues/31107~~ (Analyzer: Using function types inside of named initializing formals)
~~https://github.com/dart-lang/sdk/issues/29665~~ (Analyzer crash using a function type as a bound on a class type argument)

Some of these are admittedly P2, others have workarounds, and some are P1.
Did we use this feature at all in our libraries or SDK before releasing?
/cc @anders-sandholm @mit-mit @kevmoo

I fixed analyzer resolution & DDC recently, FYI. I haven't had a chance to go through and close out old bugs though. So it's worth retesting any older ones.

Thanks! I think engineering effort here (and responsiveness!) has been awesome, my main issue has been with release management and verification :)

Thanks! I think engineering effort here (and responsiveness!) has been awesome, my main issue has been with release management and verification :)

Yeah, agreed. I thought it worked already, so I used it in a test case and was really surprised by crashes. That's how I ended up implementing Analyzer and DDC support :). It's definitely in a confusing state still (I'm not sure the state of it in other implementations, for example the Dart Formatter seems to crash on the syntax).

@anders-sandholm @kevmoo - what's the right way to track this?

#30146 is specific to the new syntax.

@matanlurey Use of the new function syntax outside of a typedef was not handled properly in dartdoc and I didn't notice until something broke our dartdoc asserts in https://github.com/dart-lang/dartdoc/issues/1505.  That something was https://github.com/dart-lang/sdk/commit/38bf70d7ac4ea695adc65ae3d7d0fa083b4dcd46, specifically, the use of the Function syntax here:
https://github.com/dart-lang/sdk/blame/38bf70d7ac4ea695adc65ae3d7d0fa083b4dcd46/sdk/lib/async/zone.dart#L573
Since our asserts would have started firing sooner otherwise, I can say with some confidence that we haven't used the Function syntax in this manner inside the SDK until that CL.

Another issues with this – caused by rolling out support in pkg/http2 - https://github.com/dart-lang/http2/issues/14

Added two other bugs that seem to belong here

I closed
#29665
(Analyzer crash using a function type as a bound on a class type argument), seems to be fixed in latest.
Also looks like I should not have added #30147
(Analyzer: crash with partial application of generic function typed function), its actually not related to generic function types but generic methods in general.

https://github.com/dart-lang/sdk/issues/30858 is still open because the "fix" caused a secondary issue. It's a pretty deep issue related to synthetic generic element types which I've not had much luck with. Latest solution involved turning off an assertion and is up for review: https://dart-review.googlesource.com/c/sdk/+/20481

I haven't run into any problems here for a while. Thanks!


Visual Studio code- setup was unable to automatically close all applications

I am getting below error while upgrading my visual code to Ver: 1.95.1

Could anyone please help?

Hi @aayushkamal, are you still seeing the error? If so, I'd recommend pressing "Cancel installation", closing all open programs (including checking for and terminating OpenConsole.exe in the Task Manager), opening VS Code, and trying the update again. Let me know if that helps.


The upwards rounding for the y-axel of the graph

example case:
insert data set of values between: 44,7 - 55.
the rounding of 44.7 to 45 hides the lowest point of the graph line.
It should alwaoys round downwards, so in this case to 44.

at least for the min value

have you found a solution?

Not really.
But the fix should be something like, every integer, no mather the decimal value, should get a y-axel label. The math.round function does not implement this =)

As a generic FYI — I am no longer able to maintain this library. I recommend checking out victory-native as it's much more maintained.


Support literate coffeescript

Would fix #25

@dgageot
If I right understanding the problem then settings like this should fix the problem
 coffeePreprocessor: {
      options: {
        bare: true,
        sourceMap: false,
        literate: true
      },
     transformPath: function(path) {
        return path.replace(/\.litcoffee$/, '.js');
      }
    },

Thanks


Whitespace

This will strip spaces and tabs from a trailing empty line in raw text. The intended effect is that the indentation of Twig tags in a template will be removed from the result text.
This patch is in response to an old thread on the mailing list concerning whitespace control. Some objections were raised that this might not be in the interest of all users, but I wanted to offer it regardless.

Don't we have the trimming tags using - for that?
{% block foo %}
    {%- for i in 1..10 %}
        {# ... #}
    {%- endfor %}
{% endblock %}

The trimming tags will consume the previous newline. I just wanted to consume whitespace up to, but not including, the previous newline. For example:

  {%- for i in ... %}

lorum

  {%- endfor %}

gives me

lorum

lorum

lorum
  ...

instead of

lorum

lorum

lorum

  ...

Since I would want this behavior always, I didn't invest the time necessary to figure out how to do this with a custom tag. If someone wants to do it themselves, or has some instructions, I'm all ears.


If a debug adapter fails to send a terminated event, VS Code gets stuck in debug mode

See: https://github.com/microsoft/vscode/issues/100368#issuecomment-654354230 VS Code can get stuck in a state where it thinks a debug session is going, but trying to end the session results in an error "No debug adapter, cannot send 'terminate'".
Might only happen with multiple parent/child sessions. Can be reproed perfectly by commenting out the two dap.terminated() calls in binder.ts within js-debug.

Thanks for creating this. I acknowledge that VS Code relies on the adapter sending a terminate event to nicely dispose the debug session and thus end the debug mode.
What we could do to improve this:

[ ] Pressing the red stop button by the user should always terminate the session, even if the terminate can not be sent

I am open for other ideas to improve this.
Unassigning you Connor since I can tackle this on the VS Code side. Just let me know if you have additional ideas here.

Pressing the red stop button by the user should always terminate the session, even if the terminate can not be sent

In the case when the debug adapter already stopped, I think this is good behavior

@connor4312 I have pushed a potential fix for this.
In case the user hits the stop or disconnect we fire a "fake" terminated event to clear out all the state.
Since I could not reproduce this issue reliably I would ask you to try it out since you mentioned that you have good repro steps. So can you please try it out and let me know how it goes?
Thanks!

Thanks! Using it for the last week (and in particular looking into a self-host issue where this happened 100% of the time) think this is a good change. It's still a problem in the debug adapter but VS Code no longer gets stuck in the bad state.

Great, thanks for letting me know.


Anuragd/multi_gpu_device Fixed multi-gpu scenario with Python set_device API support

Description
Fix for multi-gpu configuration where the compilation of graph fails.
Root cause: Implicit typecasting of cudaSucess as boolean. Incorrect cuda context before setting the device ID (ConversionCtx)
Fixes #356
Type of change
Please delete options that are not relevant and/or add your own.

Bug fix (non-breaking)

Checklist:

[x] My code follows the style guidelines of this project (You can use the linters)
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas and hacks
[x] I have made corresponding changes to the documentation
[x] I have added tests to verify my fix or my feature
[x] New and existing unit tests pass locally with my changes

Due to multiple conflicts with the docs/ I have moved the PR as a new PR  (with address to review comments)
Please refer https://github.com/NVIDIA/TRTorch/pull/371
Closing this PR.


Add support for editing a space view's space origin

What
This PR adds the possibility to edit a space view's space origin.
TODO:

[x] Are we happy with the general behaviour of this thing?
[x] Should this be feature gated?

https://github.com/rerun-io/rerun/assets/49431240/37571532-0326-4e43-98ee-8d0e537d9235
Follow-up issues:

https://github.com/rerun-io/rerun/issues/4849

Checklist

[x] I have read and agree to Contributor Guide and the Code of Conduct
[x] I've included a screenshot or gif (if applicable)
[x] I have tested the web demo (if applicable):

Using newly built examples: app.rerun.io
Using examples from latest main build: app.rerun.io
Using full set of examples from nightly build: app.rerun.io

[x] The PR title and labels are set such as to maximize their usefulness for the next release's CHANGELOG

PR Build Summary
Docs preview
Examples preview
Recent benchmark results
Wasm size tracking

I expect to be able to arrow down to select one of the suggestions. Having to click on it feels broken.

I find this interaction really surprising. If I click on an editable field I expect to be able to edit it, not that it will be a link.
We should remove the edit button and make it behave more like an editable field. The design should at the same time make it clear when other elements are in fact links and if that isn't clear then we need to address that instead

I'm noticing a problematic interaction with heuristics we probably need to figure out.
In the SFM example, if I click the original space-view / and change it's origin to /camera, then it appears the heuristics re-create the / space-view again.

I'm noticing a problematic interaction with heuristics we probably need to figure out.
In the SFM example, if I click the original space-view / and change it's origin to /camera, then it appears the heuristics re-create the / space-view again.

We probably shouldn't be adding new views once the user has modified the blueprint. Once they have, we could consider highlighting the plus button and suggesting the top three views that would be added if we were to reset the blueprint

I'm noticing a problematic interaction with heuristics we probably need to figure out.
In the SFM example, if I click the original space-view / and change it's origin to /camera, then it appears the heuristics re-create the / space-view again.

Is that blocking for this PR?


fcb: start using errno codes

Switch form using privater FCB error codes to
errno codes. FCB private codes convention were compatible
with <errno.h> codes:

0 mean success
negative values mean errors
similar error types.
There was no sense to kept private FCB error codes.

merge after https://github.com/zephyrproject-rtos/zephyr/pull/17425 as need to remove codes from fcb.h
Signed-off-by: Andrzej Puzdrowski<EMAIL_ADDRESS>

cc @carlescufi

@nvlsianpu #17425 is merged.

@nvlsianpu is this needed for 2.0?

@ioannisg Not need - but still wort to merge, this is cleanup patch IMO.

@nvlsianpu there's a typo ENSPACE

@carlescufi ^^ fixed


Dependencies not showing on GitHub Enterprise Cloud repo

I'm not seeing any dependencies show up on the Dependency Graph. This is a private repository in GitHub Enterprise Cloud.

Our GitHub actions steps include:
    steps:
      - name: Checkout code
        uses: actions/checkout@v3

      - name: Calculates Gradle dependencies and submit to GitHubs Dependency Submission API
        # Currently the action has to be run on the default branch.
        # Running it as part of a PR won't update the Dependency graph of your projects Insights.
        if: ${{ github.ref == 'refs/heads/main' }}
        uses<EMAIL_ADDRESS>        with:
          gradle-build-configuration: "testCompileClasspath"

The workflow run is showing:

Running gradlew dependencies shows lots of dependencies for testCompileClasspath.
We have the Dependency Graph enabled:

Any thoughts on what we're doing wrong?

It's worth noting that this requirement is only in place for the --property filter on the properties task. (to only retrieve particular properties)
We do this so we don't have to write a properties output parser to filter out unwanted properties -> and more importantly the action should not request any information it is not meant to see -> request as little as needed.
In the action (highlighting for self-audit reasons) we use the properties task to retrieve the project name and buildFile path for modules:
https://github.com/mikepenz/gradle-dependency-submission/blob/main/src/gradle.ts#L91-L107

I'm still not seeing anything. Do I need to wait a certain amount of time for them to show up?
The debug log is:
##[debug]Evaluating condition for step: 'Calculates Gradle dependencies and submit to GitHubs Dependency Submission API'
##[debug]Evaluating: (success() && (github.ref == 'refs/heads/main'))
##[debug]Evaluating And:
##[debug]..Evaluating success:
##[debug]..=> true
##[debug]..Evaluating Equal:
##[debug]....Evaluating Index:
##[debug]......Evaluating github:
##[debug]......=> Object
##[debug]......Evaluating String:
##[debug]......=> 'ref'
##[debug]....=> 'refs/heads/main'
##[debug]....Evaluating String:
##[debug]....=> 'refs/heads/main'
##[debug]..=> true
##[debug]=> true
##[debug]Expanded: (true && ('refs/heads/main' == 'refs/heads/main'))
##[debug]Result: true
##[debug]Starting: Calculates Gradle dependencies and submit to GitHubs Dependency Submission API
##[debug]Loading inputs
##[debug]Evaluating: github.token
##[debug]Evaluating Index:
##[debug]..Evaluating github:
##[debug]..=> Object
##[debug]..Evaluating String:
##[debug]..=> 'token'
##[debug]=> '***'
##[debug]Result: '***'
##[debug]Loading env
Run<EMAIL_ADDRESS>::group::📘 Reading input values
📘 Reading input values

::set-output name=snapshot::{"detector":{"name":"mikepenz/gradle-dependency-submission","url":"https://github.com/mikepenz/gradle-dependency-submission","version":"0.7.0"},"version":0,"job":{"correlator":"run_features--testCompileClasspath","id":"[2](https://github.com/myorg/eoss-integration-features/runs/8080097051?check_suite_focus=true#step:3:2)951816476"},"sha":"0c0c9ae11a54457fdfbb6c2[3](https://github.com/myorg/eoss-integration-features/runs/8080097051?check_suite_focus=true#step:3:3)9c68d[4](https://github.com/myorg/eoss-integration-features/runs/8080097051?check_suite_focus=true#step:3:4)d160096112","ref":"refs/heads/main","scanned":"2022-08-29T21:1[5](https://github.com/myorg/eoss-integration-features/runs/8080097051?check_suite_focus=true#step:3:5):52.734Z","manifests":{}}
##[debug]='{"detector":{"name":"mikepenz/gradle-dependency-submission","url":"https://github.com/mikepenz/gradle-dependency-submission","version":"0.7.0"},"version":0,"job":{"correlator":"run_features--testCompileClasspath","id":"295181[6](https://github.com/myorg/eoss-integration-features/runs/8080097051?check_suite_focus=true#step:3:6)4[7](https://github.com/myorg/eoss-integration-features/runs/8080097051?check_suite_focus=true#step:3:7)6"},"sha":"0c0c9ae11a54457fdfbb6c239c6[8](https://github.com/myorg/eoss-integration-features/runs/8080097051?check_suite_focus=true#step:3:8)d4d1600[9](https://github.com/myorg/eoss-integration-features/runs/8080097051?check_suite_focus=true#step:3:9)6[11](https://github.com/myorg/eoss-integration-features/runs/8080097051?check_suite_focus=true#step:3:11)2","ref":"refs/heads/main","scanned":"2022-08-29T21:15:52.734Z","manifests":{}}'
Notice: Submitting snapshot...
Notice: {
    "detector": {
        "name": "mikepenz/gradle-dependency-submission",
        "url": "https://github.com/mikepenz/gradle-dependency-submission",
        "version": "0.7.0"
    },
    "version": 0,
    "job": {
        "correlator": "run_features--testCompileClasspath",
        "id": "2951816476"
    },
    "sha": "0c0c9ae11a54457fdfbb6c239c68d4d1600961[12](https://github.com/myorg/eoss-integration-features/runs/8080097051?check_suite_focus=true#step:3:12)",
    "ref": "refs/heads/main",
    "scanned": "2022-08-29T21:[15](https://github.com/myorg/eoss-integration-features/runs/8080097051?check_suite_focus=true#step:3:15):52.734Z",
    "manifests": {}
}
::endgroup::

Notice: Snapshot successfully created at [20](https://github.com/myorg/eoss-integration-features/runs/8080097051?check_suite_focus=true#step:3:20)22-08-29T[21](https://github.com/myorg/eoss-integration-features/runs/8080097051?check_suite_focus=true#step:3:21):15:53.[46](https://github.com/myorg/eoss-integration-features/runs/8080097051?check_suite_focus=true#step:3:48)3Z
##[debug]Node Action run completed with exit code 0
##[debug]Finishing: Calculates Gradle dependencies and submit to GitHubs Dependency Submission API

Oh I am sorry, I should have seen this directly. You are not configuring any module.
As such this will be length of 0: https://github.com/mikepenz/gradle-dependency-submission/blob/main/src/main.ts#L22
Ultimately not doing any work: https://github.com/mikepenz/gradle-dependency-submission/blob/main/src/main.ts#L61-L72
If you want the root module, please use : as module spec.
The README for example shows it like this:
- name: Root Gradle Dependency Submission
  uses: mikepenz/gradle-dependency-submission@{latest}
  with:
    use-gradlew: false
    gradle-build-module: |-
      :
    gradle-build-configuration: |-
      compileClasspath

Definitely need to add back to fail the build for this case.

Thanks, that's working now @mikepenz.
Maybe it could default to : if no module is specified?
Next, I have to figure out how to get our internal repositories to not show up as FAILED :)

@nigelcharman for the FAILED one. how does the dependencies output look like for that one?

I've just realised the step doesn't have the secrets to access the internal repositories. I've added those now and will let you know if it's fixed once the PR is merged.

Released v0.7.1 which has a log and falls back to : if not provided. thanks again for the report.

I can confirm that the internal dependencies which previously showed as FAILED are picked up correctly with the credentials supplied to the workflow


document-node(...) typed parameters

Hello,
We use XSLT we parameters typed as document-node() :
<xsl:param name="BO.CONF" required="true" as="document-node(element(conf))"/>

In the same XLST, we have functions that use this as a typed parameter :
  <xsl:function name="conf:getBoConf" as="element(els-conf:conf)" cache="true" visibility="public">
    <xsl:variable name="boConfDoc" select="$BO.CONF" as="document-node(element(els-conf:conf))"/>
    <xsl:sequence select="$boConfDoc/els-conf:conf"/>
  </xsl:function>

If we want to unit-test this XSLT, we must define such a parameter. In xspec, that's done like this :
<x:param name="BO.CONF">
    <conf>
        <variable name="BASEXML_PUBLI_USER" value="flash-rest-writer" as="xs:string"/>
    </conf>
</x:param>

XSpec compiler translates this as
<xsl:variable name="BO.CONF-doc" as="document-node()">
  <xsl:document>
    <conf xmlns="http://www.lefebvre-sarrut.eu/ns/els/conf">
      <variable name="BASEXML_PUBLI_USER" value="flash-rest-writer" as="xs:string"/>
    </conf>
  </xsl:document>
</xsl:variable>
<xsl:param name="BO.CONF" select="$BO.CONF-doc/node()"/>

So it defines an un-typed parameter BO.CONF, which - according to XSLT 2 specification - should be considered as a document-node().
If I have a scenario that call conf:getBoConf function, I have an error :
[ERROR] Required item type of value of variable $boConfDoc is document-node(element(conf)); supplied value has item type element(conf)

How could I write such a scenario, to test my XSLT ?
Thanks in advance,
Christophe

When you have
<x:param name="foo">
  <bar />
</x:param>

XSpec first wraps <bar> in a wrapper document node ($foo-doc) and then evaluate @select in the context of $foo-doc.
If you don't have @select, child::node() is assumed.
So in the above x:param, $foo is $foo-doc/child::node(), hence <bar> (i.e. element(bar)).
If you want to select the wrapper document node itself, you can specify it so in @select:
<x:param name="foo" select="self::document-node()">
  <bar />
</x:param>

Or one may prefer this:
<x:param name="foo" select="/">
  <bar />
</x:param>

I've seen this kind of questions for several times. So I updated Wiki to describe the behavior in a little more detail.
Although this behavior isn't applicable only to function calls, I updated the Function Scenarios section just because there was no other example to update.

Closing this issue, assuming that the update Wiki matches the problem.
@cmarchand
If the updated Wiki isn't clear enough to resolve the problem, please let us know.


Join/Part events doesnt work

Hi, I was trying to catch Join and Part event so I can keep a track of every user watching my stream but looks like those events has been removed from twitch's API.
Is there any workaround to keep catching those events ?
Thanks in advance.

If using TWITCHCLIENT 1 doesn't work for you, I recommend to use the internal Twitch API to keep a track of who is watching a stream. There is an example available in the documentation here.

Hey Schmoopiie, thanks for your fast answer
What I'm really trying to do is to track time watched for every user in the stream. I thought in doing the way you said but, I would have to process viewers list every minute or so, in order to keep it clean and accurate.
I find this very overprocessing and it may consume a lot of server resources, that's why I feel like I need those events. I dont know how you see it

This post explains why I do not recommend the use of TWITCHCLIENT 1 when you want to track viewers. He said: "Keep in mind that we plan to deprecate the existing protocols (aka TWITCHCLIENT), but have no ETA when old protocol versions will no longer be supported." - Of couse we don't know when it will be deprecated but I'm not taking any chances.
I will try using tc 1 tomorrow to see if something is wrong with it, will keep you updated here.
Thanks!

Thanks again Schmoopiie. I have been trying with channels with less than 1k viewers

Are you using tc: 1 in your options ? By default the module is using TWITCHCLIENT 4 which is not what you want if you want to catch all the joins and parts. If you are not sure, check the configuration options.

I just tried the module to get joins and parts and it is working properly. I guess you are not using TWITCHCLIENT 1, hence why you can only see the client joining/leaving a channel. I am providing you the piece of code I used to investigate your issue.. Keep in mind that joins and parts are delayed and sent in batch.
var irc = require('twitch-irc');

var clientOptions = {
    options: {
        debug: true,
        debugIgnore: ['ping', 'action', 'chat'],
        tc: 1
    },
    identity: {
        username: 'Your-username',
        password: 'oauth:token'
    },
    channels: ['lirik']
};

var client = new irc.client(clientOptions);

client.connect();

client.addListener('join', function (channel, username) {
    console.log('[JOIN] ' + channel + '> ' + username);
});

client.addListener('part', function (channel, username) {
    console.log('[PART] ' + channel + '> ' + username);
});

Hi there,
I was using default option, changed it to 1 and it worked. Thank you for your time.


Tarik should use chain of thought to gather context to satisfaction and then plan work

In order to fulfill this, most likely need vector embeddings database. Investigate this and use of CoT

https://twitter.com/hwchase17/status/1645442763536891904?s=46&t=dBFhB45pygQXUYC119egUA
Above demonstrates how BabyAGI when given a task, will prompt itself to create a task list and work through it. Could be inspirational? Check their codebase and how they prompt

multiple phases (i.e. Skeleton, consider breaking down into subtasks)
Borrow from GANs? Tarik adversary that just criticizes? Weight higher initially then trend towards zero?


Having a compatible PSR-11 container instance.

Laravel Version: 5.x

Description:
After this comment I want to get started a plan to allow for a PSR-11 compliant wrapper around the Illuminate container.
As there is an increase in the number of packages I've seen being developed which make use of PSR-11, 450 of them registered on packagist as a dependant. I think it would be a good time for Laravel to have a wrapper implemented in the framework.
Other projects such as Zend Service Manager have this built into there service manager. I my self have had to make a temporary patch to a project in order to satisfy this requirement when using Laravel.

This is not a bug. Open an issue in the laravel/internals repo

Thanks @Dylan-DPC I was unaware of this. laravel/internals#550


Update dependecies

Coverage remained the same at 90.625% when pulling ab62b8b51188e8cf9a49ae66c46350aca50184ce on update_dependecies into bb2478ee75e585a6177403e80f526b1505802daa on master.


UI not updating GRIN NODE data

Hi @maoxs2,
First of all thanks a lot for your work and support,
Pool is not throwing any error other than

TypeError: pool.mined_blocks is undefinedindex.js:18:21
onload http://<IP_ADDRESS>/js/index.js:18

which refers to:
for (i = 0; i < pool.mined_blocks.length; i++) {
And I suspect it is blocking the execution of API calls. Which is why no data is being rendered. I have checked API is working fine. Can you explain what I may have done wrong?

Also, miner's stats are not being updated when accessed from the remote browser at http://ipaddress/miner.html. This exception is thrown in consol.

Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource at http://ipaddress:3330/miner/. (Reason: CORS header ‘Access-Control-Allow-Origin’ missing).

I am using caddy webserver Caddy file.

I have bypassed this loop, grin node data is working. But I couldn't fix this exception. Maybe it has something to do with this issue CORS

mined_blocks is migrated to the new api http://api_server:api_port/blocks
check this commit
https://github.com/maoxs2/open-grin-pool/commit/b91646d8a8f3b64e2618b4e5d527212ef72ac6c2


Installation with VAM

https://github.com/MarcWeber/vim-addon-manager#recommended-setup

OK, thanks!
But, you know, there are lots of plugin managers for Vim, and listing every method will expand our README into horrendous long document, and make it unreadable :)


No shape mismatch error with different shapes of outputs and targets with tf.keras.losses.BinaryCrossentropy()

Please go to TF Forum for help and support:
https://discuss.tensorflow.org/tag/keras
If you open a GitHub issue, here is our policy:
It must be a bug, a feature request, or a significant problem with the documentation (for small docs fixes please send a PR instead).
The form below must be filled out.
Here's why we have that policy:.
Keras developers respond to issues. We want to focus on work that benefits the whole community, e.g., fixing bugs and adding features. Support only helps individuals. GitHub also notifies thousands of people when issues are filed. We want them to see you communicating an interesting problem, rather than being redirected to Stack Overflow.
System information.

Have I written custom code (as opposed to using a stock example script provided in Keras): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):  Windows 11
TensorFlow installed from (source or binary):                     pre installed on Google Colab
TensorFlow version (use command below): 2.9.2
Python version: 3.8.16
Bazel version (if compiling from source): None
GPU model and memory: None
Exact command to reproduce:

import numpy as np
import tensorflow as tf
fake_preds = np.random.rand(10,768)
fake_labels = np.random.rand(10,1)
loss1 = tf.keras.losses.BinaryCrossentropy(from_logits=True)
print(loss1(fake_labels, fake_preds).numpy())
ValueError: logits and labels must have the same shape, received ((10, 768) vs (10, 1)).
loss2 = tf.keras.losses.BinaryCrossentopy(from_logits=False)
print(loss2(fake_labels, fake_preds),numpy())
0.998 (some value for random labels and preds), works well!
You can collect some of this information using our environment capture script:
https://github.com/tensorflow/tensorflow/tree/master/tools/tf_env_collect.sh
You can obtain the TensorFlow version with:
python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"
Describe the problem.
We're using  a shape mismatch between the outptus and targets of a model.
for example, outputs.shape = (10,768)   # assuming predictions of some model (10 examples)
targets.shape  = (10,1)       # fake labels for 10 examples
When trying to compute the loss tf.keras.losses.BinaryCrossentropy(from_logits=False).
setting the argument "from_logits" = False IS NOT THROWING AN ERROR when there is a shape mismatch and which is not allowable as per keras documentation.
Describe the current behavior.
inputs = np.random.rand(10,768)
targets = np.random.rand(10,1)
loss = tf.keras.losses.BinaryCrossentropy(from_logits=False)
print(loss)
= 0.95677 ( some value based on the random_state)
Describe the expected behavior.
It should throw the same "Value Error" as it does for the same inputs and targets when using
loss = tf.keras.losses.BinaryCrossentropy(from_logits=True)
ValueError: logits and labels must have the same shape, received ((10, 768) vs (10, 1)).
Contributing.

Do you want to contribute a PR? (yes/no): No
If yes, please read this page for instructions
Briefly describe your candidate solution(if contributing):

Standalone code to reproduce the issue.
Provide a reproducible test case that is the bare minimum necessary to generate
the problem. If possible, please share a link to Colab/Jupyter/any notebook.
import numpy as np
import tensorflow as tf
import random
loss1 = tf.keras.losses.BinaryCrossentropy(from_logits=True)
loss2 = tf.keras.losses.BinaryCrossentropy(from_logits=False)
fake_preds = np.random.rand(2,3)
fake_labels= np.random.rand(2,1)
print(loss2(fake_labels, fake_preds).numpy())
prints some value ex. 0.83954634
print(loss1(fake_labels,fake_preds).numpy())
----> 4 loss1(fake_labels, fake_preds)

Source code / logs.
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached. Try to provide a reproducible test case that is the bare minimum necessary to generate the problem.

@SuryanarayanaY I was able to replicate the issue here. Thank you!

I can work on this and fix that if someone hasn't started yet.

Hi @rahul-ohlan ,
Thanks for sharing your observation. When from_logits=False all the values of logits will be converted to single probability value because you are using Binary Cross-entropy hence the last dimension will be reduced to 1 anyway.When from_logits=True the logit values will be directly taken for comparison with Labels and shape-mismatch error will be raised.
But the question is whenever you want to use Binary Cross entropy you need to have One neuron at the last layer of neural network.Anything >1 at last layer as neuron will not allow the model to train with from_logits=False or from_logits=True.You may check the attached gist here.
AFAIK you can create a model which can generate y_pred shape as (x,768) but you can't train this model for Binary classification with from_logits=False or from_logits=True.
If you come across any such use case please confirm then it will be analysed and needful action can be taken.


'record' is of type 'unknown'

Describe the bug
My code does not compile because of the following error:
Failed to compile.

#15 5.452
#15 5.452 ./src/pages/tenant/index.tsx:141:56
#15 5.452 Type error: 'record' is of type 'unknown'.
#15 5.452
#15 5.452   139 |        fetching={isLoading}
#15 5.452   140 |        sortStatus={sortStatus}
#15 5.452 > 141 |        onRowClick={(record) => {router.push(/tenant/${record.uuid})}}

This error also trigger on all renders.

<DataTable
minHeight={450}
totalRecords={dataTenant?.pagination?.count}
recordsPerPage={pageSize}
page={page}
withBorder
fetching={isLoading}
sortStatus={sortStatus}
onRowClick={(record) => {router.push(/tenant/${record.uuid})}}
onPageChange={(p) => setPage(p)}
columns={[{ accessor: 'name', title: 'Name', sortable: true },
{ accessor: 'created_at', title: 'Created', sortable: true, width: 200 , render: (created_at) => Moment({created_at}).format('yyyy-MM-DD')},
{ accessor: 'modified_at', title: 'Modified', sortable: true, width: 200 }]}
records={dataTenant?.results}
onSortStatusChange={setSortStatus}
recordsPerPageOptions={PAGE_SIZES}
onRecordsPerPageChange={setPageSize}
idAccessor="uuid"
/>

Does it have something to do that my id is not a number but a uuid?

No, it shouldn't.
But I believe your dataTenant?.results may not be properly typed:
records={dataTenant?.results}

No, it shouldn't. But I believe your dataTenant?.results may not be properly typed:
records={dataTenant?.results}

Thank you for your fast reply.
I am new to Next.js , so maby its my mistake. My code for populating dataTenant:
const { dataTenant, isLoading, isError } = getTenants()
function getTenants() {
  const params = new URLSearchParams();
  params.append("page_size", pageSize.toString());
  params.append("page", page.toString());
  params.append("ordering", sortStatus.columnAccessor.toString() + " " + sortStatus.direction.toString());
  //const params = 'page_size=2'
  const { data, error,isLoading } = useSWR('/api/ext/v1/admin/tenant?'+params.toString(), fetcher, {keepPreviousData: true});
  return {
    dataTenant: data,
    isLoading: isLoading,
    isError: isError
  }
}

Json from API:
{     "pagination": {         "count": 56,         "current": 1,         "end_index": 2,         "next": 0,         "previous": 0,         "start_index": 1,         "total_pages": 28     },     "results": [         {             "created_at": "2023-05-14T19:32:14.663506Z",             "modified_at": "2023-05-14T19:32:14.663506Z",             "name": "46",             "uuid": "6d6c6bbb-7cf1-48ee-aaa4-d99ca2782010"         },         {             "created_at": "2023-05-14T19:33:10.889489Z",             "modified_at": "2023-05-14T19:33:10.889489Z",             "name": "47",             "uuid": "0cf4cf13-a082-4a1f-b089-54f39ae66869"         }     ] }
The question mark is there because if SWR loads the data, dataTenant is null.

No, it shouldn't. But I believe your dataTenant?.results may not be properly typed:
records={dataTenant?.results}

Thank you for your fast reply.
I am new to Next.js , so maby its my mistake. My code for populating dataTenant:
const { dataTenant, isLoading, isError } = getTenants()
function getTenants() {
  const params = new URLSearchParams();
  params.append("page_size", pageSize.toString());
  params.append("page", page.toString());
  params.append("ordering", sortStatus.columnAccessor.toString() + " " + sortStatus.direction.toString());
  const { data, error,isLoading } = useSWR('/api/ext/v1/admin/tenant?'+params.toString(), fetcher, {keepPreviousData: true});
  return {
    dataTenant: data,
    isLoading: isLoading,
    isError: isError
  }
}

Json from API:
{
    "pagination": {
        "count": 56,
        "current": 1,
        "end_index": 2,
        "next": 0,
        "previous": 0,
        "start_index": 1,
        "total_pages": 28
    },
    "results": [
        {
            "created_at": "2023-05-14T19:32:14.663506Z",
            "modified_at": "2023-05-14T19:32:14.663506Z",
            "name": "46",
            "uuid": "6d6c6bbb-7cf1-48ee-aaa4-d99ca2782010"
        },
        {
            "created_at": "2023-05-14T19:33:10.889489Z",
            "modified_at": "2023-05-14T19:33:10.889489Z",
            "name": "47",
            "uuid": "0cf4cf13-a082-4a1f-b089-54f39ae66869"
        }
    ]
}

The question mark is there because if SWR loads the data, dataTenant is null.

I see.
The useSWR hook doesn't know how to type your data, but since it's a generic, you can easily type it.
See here.
I'm going to close this, since it's not related to Mantine DataTable.

 Thank you for the support. Got it working.

Hey @Nefkens, thanks for the kind words, but I am no hero, I assure you! ;-)
I'm glad you got it working.


Rebuild when?

Plan on rebuilding the IDM with an updated tech stack. Planning to use:
Typescript
React
ExpressJS
MongoDB
Vite
Stack subject to change as I learn more tools

Started rebuild in #125 Will keep issue open until rebuild is completed.

First a successful Vite build in #127 . Pages that build: Home, contact, translator...Country selection page and individual country pages wont build until progress on backend is made

Making good progress. Translator redone in #143. Plan to us TTS for output translation looking less likely. Might need to use googles TTS

Going to attempt deployment through Vercel instead of deploying frontend to Netfliy and backend to Render

Successful rebuild and deployment in #157


fixed-data-table-2  compatibilty?

I no longer work on this. You're more than welcome to submit a pull request if you wish.


Wrong translation language specified for even1259

The metadata at https://sharedocs.huma-num.fr/wl/?id=s947TcfRfDZR643QURdyncdku4EmeKyb&fmode=download says even1259 has translations in English, but they appear to be Russian.

Thanks for pointing this out. The entry will be corrected with the next major update, along with a series of other improvements.


Post processing / diagnosis of ocp solutions

Gives more detailed information about a solution

[ ] sort constraints (pointwise and path) and display violation and multipliers (use norm for path constraints). Plots ?
[ ] We can't for the moment recorver the boundary conditions
[ ] norms 1 and 2 of the control (component wise ?). NB. Can be displayed also in callback
[ ] display running cost w.r.t time

Also provide a comparison tool for multiple solutions

[ ] plot functions on same graphs
[ ] display norm of errors between variables (objective also)

@ocots Is it possible to add savefig for plots? I am assuming that Plots.jl is imported, so I guess it is just a matter of re-exporting savefig from the standard backend.


Segmented options

Hi, this post is more discussion than issue. It's related to what was said before on issue #13.
Currently brains, when created take options hiddenLayers & learningRate :
var net = new NeuralNetwork({
   hiddenLayers: [4],
   learningRate: 0.6
});

And when trained, take data and options errorThresh, iterations, log & logPeriod :
net.train(data, {
   errorThresh: 0.004,
   iterations: 20000,
   log: true,
   logPeriod: 10
});

What I'm asking is this, given that one cannot iteratively train a these neural networks, why are there 2 points at which different options are provided to the neural network?
Does it not make sense to declare all options/setting in one place, ie on creation of the neural network?
Perhaps something like:
var net = new NeuralNetwork({
   hiddenLayers: [4],
   learningRate: 0.6,
   errorThresh: 0.004,
   iterations: 20000,
   log: true,
   logPeriod: 10
});

net.train(data);

Chris

@tlhunter see: https://github.com/harthur/brain/pull/63


New view like timeline dependency (Gantt chart) without time axis (like a graph)

Hi, I just came across your interesting repository. I think your project is awesome.
I am wondering if you are planning to make a graph-like view for Notion database. I found sometimes it would be great if we could use some property, like relation property, and see what pages are linked via such relation as a graph view. I think the demo you showed on your README can do that, can't it?
Also, are you going to publish this work as an integration in the near future?

so firstly the readme is outdated. I need to update that first.
But currently it can only query child pages, mentions and href links. I intentionally disabled child database to be queried as links however i could add an option to enable it.
Im not quite sure what you mean by "relation property" but how I understand it you would like to have every property to be a node and all pages that share that property to connect to it?
This could for sure be done, however im currently somewhat busy so idk when ill have time to do this. Feel free to look at the code and tinker for yourself in the meantime.


esm() breaks console.*() functions in esm-styled nuxt.config.js

Version
v2.11.0
Steps to reproduce
1.Create nuxt.config.js like the following:
export default {
  vue: {
    config: {
      errorHandler(error) {
        console.error(error);
      },
    },
  },
}

2.Check the compiled file: .nuxt/client.js
What is expected ?
Object.assign(Vue.config, {"errorHandler":function(error) {
        console.error(error);
      },"silent":false,"performance":true})

It should work correctly.
What is actually happening?
Object.assign(Vue.config, {"errorHandler":function(error) {
        _xxx.g.console.error(error);
      },"silent":false,"performance":true})

Reference error occurs at _xxx. It seems that the prefix originally exists as _.g and has been transformed into _xxx.g by webpack.
Additional comments?
nuxt.js/config.js at 5fa768373da1adfd8c76145b2ec95b7824af93b4 · nuxt/nuxt.js
options = esm(module)(nuxtConfigFile) || {}

I guess that the usage of esm() is wrong. Also there are the test that confirms prefixing global console and reflect.
esm/compiler-tests.mjs at a3b8290074ccd2a708ed9b985a2f89d4f4cdd026 · standard-things/esm
it("should transform `console` references and calls with identifier arguments", () => {
  const lines = [
    "console",
    "console.a(b)",
    'console.a("b", c)',
    'console["a"](b)',
    "new console.Console(a)",
    "class C extends console.Console {}",
    "const a = { console }",
    "const a = { [console]: console }",
    "const a = { console() { console } }",
    "const a = () => console"
  ]

  const compiled = [
    "_.g.console",
    "_.g.console.a(b)",
    '_.g.console.a("b", c)',
    '_.g.console["a"](b)',
    "new _.g.console.Console(a)",
    "class C extends _.g.console.Console {}",
    "const a = { console:_.g.console }",
    "const a = { [_.g.console]: _.g.console }",
    "const a = { console() { _.g.console } }",
    "const a = () => _.g.console"
  ]

  lines.forEach((line, index) => {
    const code = [
      'import a from "a"',
      line
    ].join("\n")

    for (const sourceType of modernTypes) {
      const result = Compiler.compile(code, { sourceType })
      const actual = result.code.split("\n").pop()

      assert.strictEqual(actual, compiled[index])
    }
  })
})

This bug report is available on Nuxt community (#c10136)

This looks like an issue in esm. If I remove all imports from my nuxt.config it generates correctly.
Minimal reproduction:
// config.es.js
import path from 'path'

export default {
  errorHandler(error) {
    console.error(error);
  }
}

// load.cjs
#!/usr/bin/env node

const esm = require('esm')

const config = require.resolve('./config.js')

console.log(esm(module)(config).default.errorHandler.toString())

@pimlie

This looks like an issue in esm.

Did you see my explanation?

Also there are the test that confirms prefixing global console and reflect.

@mpyw Did you see and try my reproduction?

@pimlie Sure. What do you think about the tests on esm() repository?

@mpyw How are we using esm() wrong according to you?

It seems that we need to provide original global variables on namespace _.g.

The esm docs doesnt seem to mention this at all? Did you find anything in the docs? Can you show that? Or is proving global vars only given in by looking at those tests?

No. There is no such document anywhere. But as far as considering the tests, it's obvious that we need to do so. esm looks preventing global variables (undefined variables in each file) from making collisions. ...Yes, the esm document is so unkind for developers.

Ah right, thanks now I understand.
I think the issue is that the prefix just doesnt exist after the Nuxt build. What happens is:

load nuxt.config, esm adds global prefix
build nuxt.js

esm global prefix is available
but nuxt serializes the errorReporting function prefix into the Nuxt vue-app template

after nuxt has been build, it executes the errorReporting method that was added to the vue-app template which still uses that esm global prefix. But as we are now running in a complete new context without esm, that prefix just doesnt exists anymore

@pimlie So nice explanation, thanks!


Run formatting/static checks/tests in ci

@wc117 & @rootedbox What's blocking review and merge? Over 1 month old.

@taariq The changes to .github should be copied and the formatters/linters rerun to reproduce all other changes in this PR. This is a change we should adopt either way.

@measure-fi anytihng else we need to do to merge this?

Defunct in favor of https://github.com/palomachain/paloma.py/pull/11


Not worked on latest GoLand

The 1.7.0 of this plugin required 233.11799.241, but the latest GoLand is only 233.11799.228

the same error


Cant get to work with Laravel 5.4

ErrorException in Builder.php line 14:
Declaration of Sgpatil\Orientdb\Eloquent\Builder::has($relation, $operator = '>=', $count = 1, $boolean = 'and', ?Sgpatil\Orientdb\Eloquent\Closure $callback = NULL) should be compatible with Illuminate\Database\Eloquent\Builder::has($relation, $operator = '>=', $count = 1, $boolean = 'and', ?Closure $callback = NULL)
How do I get this to work, was using Neo4j but I want to try out OrientDb

Clean install of Laravel 5.4
ErrorException in Connection.php line 20:
Declaration of Sgpatil\Orientdb\Connection::getConfig($option, $default = NULL) should be compatible with Illuminate\Database\Connection::getConfig($option = NULL)
What needs to be resolved?

Hi @1jason1 , Thank you for using this library. I have made some changes in Connection.php to resolve the getConfig() problem. Can you please retry it by installing fresh copy. Also the project is still in development mode so you may get unexpected bugs while using. Let me know if this solved your problem.

Notes from clean install:
Generating optimized class loader
The compiled services file has been removed.

php artisan key:generate
Application key [base64:OvqjnxvpRHYNmQXph0jHhv9Tfr2TX3inO1MZ1W6Ym6w=] set successfully.
Application ready! Build something amazing.
jasonf /srv/www$ cd test2
jasonf /srv/www/test2$ composer require sgpatil/oriquent
Using version v<IP_ADDRESS> for sgpatil/oriquent
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Package operations: 2 installs, 0 updates, 0 removals

Installing sgpatil/orientdb-php (v0.0.3) Downloading: 100%
Installing sgpatil/oriquent (v<IP_ADDRESS>) Downloading: 100%
Writing lock file
Generating optimized autoload files
Warning: Ambiguous class resolution, "Sgpatil\Orientphp\Command\CreateClass" was found in both "/srv/www/test2/vendor/sgpatil/orientdb-php/src/Sgpatil/Orientphp/Command/CreateClass.php" and "/srv/www/test2/vendor/sgpatil/orientdb-php/src/Sgpatil/Orientphp/Command/MakeClass.php", the first will be used.

Illuminate\Foundation\ComposerScripts::postUpdate
php artisan optimize
Generating optimized class loader
The compiled services file has been removed.

Also when trying to register a new user via http://test2.dev/register
Form is displayed no problem, after submitting the form get the following error.
<EMAIL_ADDRESS>On Apr 5, 2017, at 8:20 AM, Sumit Patil <notifications<EMAIL_ADDRESS>wrote:
Hi @1jason1https://github.com/1jason1 , Thank you for using this library. I have made some changes in Connection.php to resolve the getConfig() problem. Can you please retry it by installing fresh copy. Also the project is still in development mode so you may get unexpected bugs while using. Let me know if this solved your problem.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/sgpatil/oriquent/issues/36#issuecomment-291858717, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFUe08NYPqfZwnsdDKFJ85EgfMXAIQrQks5rs5UjgaJpZM4MyNNS.

Hello @1jason1 , can you please try branch dev-master as I have made changes in that branch only.

Installed dev-master.
Error after submitting the register form.
[cid:03A73769-63B3-47F3-BE82-7979D0A590FA]
On Apr 6, 2017, at 2:46 AM, Sumit Patil <notifications<EMAIL_ADDRESS>wrote:
Hello @1jason1https://github.com/1jason1 , can you please try branch dev-master as I have made changes in that branch only.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/sgpatil/oriquent/issues/36#issuecomment-292095693, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFUe01N_trGrmLKVo0SOTWdF2OaC2V-Sks5rtJhWgaJpZM4MyNNS.

I’ve had to take a break from this now picking it back up. Still having issues getting it to work.
I have a valid user created but when I try to login I get the following error, please help.
[cid:3403BC8C-89D5-44BF-85F2-0F3470A219D5]
[cid:24630D22-0AB8-481F-BC2B-BFDCD51753D0]
On Apr 6, 2017, at 9:21 PM, Jason Forgey <jforgey<EMAIL_ADDRESS>wrote:
Installed dev-master.
Error after submitting the register form.
<PastedGraphic-3.png>
On Apr 6, 2017, at 2:46 AM, Sumit Patil <notifications<EMAIL_ADDRESS>wrote:
Hello @1jason1https://github.com/1jason1 , can you please try branch dev-master as I have made changes in that branch only.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/sgpatil/oriquent/issues/36#issuecomment-292095693, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFUe01N_trGrmLKVo0SOTWdF2OaC2V-Sks5rtJhWgaJpZM4MyNNS.

Hi @1jason1 , Actually I have tried the same code on fresh installation and everything is running properly in my system. Can you please tell me your system configuration like laravel version, Orientdb version, OS, Server, php version, screen shot of error message etc. This may help me to figure out the problem.

Thanks for the response.
Here are my settings.

orientdb-community-2.2.17
Laravel Framework 5.4.19
MacOS (Sierra) 10.12.4
Apache via MAMP Pro v4.1.1
PHP 7.1.1

Snip of my app.php
/*

Package Service Providers...
*/
Laravel\Tinker\TinkerServiceProvider::class,
Sgpatil\Orientdb\OrientdbServiceProvider::class,

This is my User Model:
namespace App;
use Illuminate\Notifications\Notifiable;
use Illuminate\Auth\Authenticatable;
use Sgpatil\Orientdb\Eloquent\Model AS Model;
use Illuminate\Auth\Passwords\CanResetPassword;
use Illuminate\Foundation\Auth\Access\Authorizable;
use Illuminate\Contracts\Auth\Authenticatable as AuthenticatableContract;
use Illuminate\Contracts\Auth\Access\Authorizable as AuthorizableContract;
use Illuminate\Contracts\Auth\CanResetPassword as CanResetPasswordContract;
class User extends Model implements
AuthenticatableContract,
AuthorizableContract,
CanResetPasswordContract
{
use Notifiable, Authenticatable, Authorizable, CanResetPassword;
/**
 * The attributes that are mass assignable.
 *
 * @var array
 */
protected $fillable = [
    'name', 'email', 'password',
];

/**
 * The attributes that should be hidden for arrays.
 *
 * @var array
 */
protected $hidden = [
    'password', 'remember_token',
];

}
Here is the error I get when trying to register a new user
ErrorException in Builder.php line 14:Declaration of Sgpatil\Orientdb\Eloquent\Builder::has($relation, $operator = '>=', $count = 1, $boolean = 'and', ?Sgpatil\Orientdb\Eloquent\Closure $callback = NULL) should be compatible with Illuminate\Database\Eloquent\Builder::has($relation, $operator = '>=', $count = 1, $boolean = 'and', ?Closure $callback = NULL)
[cid:9141C812-3256-46A2-B9EA-320990BB1450]
On Apr 20, 2017, at 6:40 AM, Sumit Patil <notifications<EMAIL_ADDRESS>wrote:
Hi @1jason1https://github.com/1jason1 , Actually I have tried the same code on fresh installation and everything is running properly in my system. Can you please tell me your system configuration like laravel version, Orientdb version, OS, Server, php version, screen shot of error message etc. This may help me to figure out the problem.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/sgpatil/oriquent/issues/36#issuecomment-295699301, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFUe0xLBVD7ntYwSFQysoQFKcX7iuOJLks5rx0QrgaJpZM4MyNNS.

Also, I get the following Warning when doing a Composer Update and at the point where composer is "Generating optimized autoload files" I get this Warning.

Warning: Ambiguous class resolution, "Sgpatil\Orientphp\Command\CreateClass" was found in both "/srv/www/test2/vendor/sgpatil/orientdb-php/src/Sgpatil/Orientphp/Command/CreateClass.php" and "/srv/www/test2/vendor/sgpatil/orientdb-php/src/Sgpatil/Orientphp/Command/MakeClass.php", the first will be used.

I was also having this issue but it looks like the updates on dev-master fixed the issue. As @1jason1 mentioned, there is also the composer Ambiguous class resolution warning. Should we open a new issue for this?

@rickvschmidt How did you get the errors to go away for Laravel 5.4? Not sure if you follow my config setup in prior posts but do I have my configs setup correctly? Are you able to register users and login via the auth package that Laravel provides by default?
In my composer.json in my require section I'm using the following, is this correct to pull the version your using?
"require": {         "php": ">=5.6.4",         "laravel/framework": "5.4.*",         "laravel/tinker": "~1.0",         "sgpatil/oriquent": "dev-master"     },

@1jason1 Not exactly yet, though I must admit that I am new to this package altogether. I have the same composer config as you do. I at least got past the original error you got while viewing the default Laravel view. I am currently having issues running any migrations... for me it is complaining that the database doesn't exist, while I am able to make a new model via Artisan orient:create

@sgpatil I am unable to run any migrations. orient:install works fine, and is able to write to the DB, I am also able to call orient:create without issue. However, when I try to run a migration, I get the following error with the same configuration file setup... so I know the database exists and the drivers are able to connect. This running your latest dev-master branch and I am also on Laravel 5.4
[Sgpatil\Orientphp\Exception]   database 'rick' does not exist. Please check database configuration. #0 /Users/Rick/code/orient-db-test/ven   dor/sgpatil/orientdb-php/src/Sgpatil/Orientphp/Transport.php(114): Sgpatil\Orientphp\Transport\Curl->makeRe   quest('GET', '/server', NULL)   #1 /Users/Rick/code/orient-db-test/vendor/sgpatil/orientdb-php/src/Sgpatil/Orientphp/Command.php(67): Sgpat   il\Orientphp\Transport->get('/server', NULL, NULL)   #2 /Users/Rick/code/orient-db-test/vendor/sgpatil/orientdb-php/src/Sgpatil/Orientphp/Client.php(904): Sgpat   il\Orientphp\Command->execute()   #3 /Users/Rick/code/orient-db-test/vendor/sgpatil/orientdb-php/src/Sgpatil/Orientphp/Client.php(482): Sgpat   il\Orientphp\Client->runCommand(Object(Sgpatil\Orientphp\Command\GetServerInfo))   #4 /Users/Rick/code/orient-db-test/vendor/sgpatil/orientdb-php/src/Sgpatil/Orientphp/Client.php(505): Sgpat   il\Orientphp\Client->getServerInfo()   #5 /Users/Rick/code/orient-db-test/vendor/sgpatil/orientdb-php/src/Sgpatil/Orientphp/Command/ExecuteBatchQu   ery.php(82): Sgpatil\Orientphp\Client->hasCapability('cypher')   #6 /Users/Rick/code/orient-db-test/vendor/sgpatil/orientdb-php/src/Sgpatil/Orientphp/Command.php(63): Sgpat   il\Orientphp\Command\ExecuteBatchQuery->getPath()   #7 /Users/Rick/code/orient-db-test/vendor/sgpatil/orientdb-php/src/Sgpatil/Orientphp/Client.php(904): Sgpat   il\Orientphp\Command->execute()   #8 /Users/Rick/code/orient-db-test/vendor/sgpatil/orientdb-php/src/Sgpatil/Orientphp/Client.php(237): Sgpat   il\Orientphp\Client->runCommand(Object(Sgpatil\Orientphp\Command\ExecuteBatchQuery))   #9 /Users/Rick/code/orient-db-test/vendor/sgpatil/orientdb-php/src/Sgpatil/Orientphp/Batch/Query.php(58): S   gpatil\Orientphp\Client->executeBatchQuery(Object(Sgpatil\Orientphp\Batch\Query))   #10 /Users/Rick/code/orient-db-test/vendor/sgpatil/oriquent/src/Sgpatil/Orientdb/Schema/OrientdbBuilder.php   (15): Sgpatil\Orientphp\Batch\Query->getResultSet()   #11 /Users/Rick/code/orient-db-test/vendor/sgpatil/oriquent/src/Sgpatil/Orientdb/Migrations/DatabaseMigrati   onRepository.php(137): Sgpatil\Orientdb\Schema\OrientdbBuilder->hasTable('migrations')   #12 /Users/Rick/code/orient-db-test/vendor/sgpatil/oriquent/src/Sgpatil/Orientdb/Migrations/Migrator.php(37   8): Sgpatil\Orientdb\Migrations\DatabaseMigrationRepository->repositoryExists()   #13 /Users/Rick/code/orient-db-test/vendor/sgpatil/oriquent/src/Sgpatil/Orientdb/Console/Migrations/Migrate   Command.php(102): Sgpatil\Orientdb\Migrations\Migrator->repositoryExists()   #14 /Users/Rick/code/orient-db-test/vendor/sgpatil/oriquent/src/Sgpatil/Orientdb/Console/Migrations/Migrate   Command.php(61): Sgpatil\Orientdb\Console\Migrations\MigrateCommand->prepareDatabase()   #15 [internal function]: Sgpatil\Orientdb\Console\Migrations\MigrateCommand->fire()   #16 /Users/Rick/code/orient-db-test/vendor/laravel/framework/src/Illuminate/Container/BoundMethod.php(29):   call_user_func_array(Array, Array)   #17 /Users/Rick/code/orient-db-test/vendor/laravel/framework/src/Illuminate/Container/BoundMethod.php(87):   Illuminate\Container\BoundMethod::Illuminate\Container\{closure}()   #18 /Users/Rick/code/orient-db-test/vendor/laravel/framework/src/Illuminate/Container/BoundMethod.php(31):   Illuminate\Container\BoundMethod::callBoundMethod(Object(Illuminate\Foundation\Application), Array, Object(   Closure))   #19 /Users/Rick/code/orient-db-test/vendor/laravel/framework/src/Illuminate/Container/Container.php(531): I   lluminate\Container\BoundMethod::call(Object(Illuminate\Foundation\Application), Array, Array, NULL)   #20 /Users/Rick/code/orient-db-test/vendor/laravel/framework/src/Illuminate/Console/Command.php(182): Illum   inate\Container\Container->call(Array)   #21 /Users/Rick/code/orient-db-test/vendor/symfony/console/Command/Command.php(262): Illuminate\Console\Com   mand->execute(Object(Symfony\Component\Console\Input\ArgvInput), Object(Illuminate\Console\OutputStyle))   #22 /Users/Rick/code/orient-db-test/vendor/laravel/framework/src/Illuminate/Console/Command.php(167): Symfo   ny\Component\Console\Command\Command->run(Object(Symfony\Component\Console\Input\ArgvInput), Object(Illumin   ate\Console\OutputStyle))   #23 /Users/Rick/code/orient-db-test/vendor/symfony/console/Application.php(826): Illuminate\Console\Command   ->run(Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOut   put))   #24 /Users/Rick/code/orient-db-test/vendor/symfony/console/Application.php(190): Symfony\Component\Console\   Application->doRunCommand(Object(Sgpatil\Orientdb\Console\Migrations\MigrateCommand), Object(Symfony\Compon   ent\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput))   #25 /Users/Rick/code/orient-db-test/vendor/symfony/console/Application.php(121): Symfony\Component\Console\   Application->doRun(Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Outp   ut\ConsoleOutput))   #26 /Users/Rick/code/orient-db-test/vendor/laravel/framework/src/Illuminate/Foundation/Console/Kernel.php(1   23): Symfony\Component\Console\Application->run(Object(Symfony\Component\Console\Input\ArgvInput), Object(S   ymfony\Component\Console\Output\ConsoleOutput))   #27 /Users/Rick/code/orient-db-test/artisan(35): Illuminate\Foundation\Console\Kernel->handle(Object(Symfon   y\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput))   #28 {main}

I simply get
[ReflectionException]
Class db does not exist

@rickvschmidt, Is your .env file is configured properly ? It should be like
DB_CONNECTION=orientdb
DB_HOST=<IP_ADDRESS>
DB_PORT=3306
DB_DATABASE=orientdb_database
DB_USERNAME=orientdb_username
DB_PASSWORD=orientdb_password

@sgpatil Yes, I have changed it. For some reason it requires Root access in order to properly work (not sure if this is by design, though I find it strange that admin access is not adequate). Once everything is switched over to root, the database not found error goes away, but then I get the error that @1jason1 gets (first post in this thread) when trying to use a basic auth page produced by Laravel. It appears there are some updates to be done to make the query builder work with Laravel 5.4.

So I've tried all I can to get this driver to work with Laravel 5.4, I really like OrientDb vs Neo4j. I have a large social media type project that would work better under OrientDb but there just doesn't seem to be the right support for drivers yet for Laravel. Can anyone point me in the direction of another driver option before I have to go the Neo4j route?

Sorry @1jason1, Unfortunately there are no perfect orientdb driver for laravel. You may contribute to this to build that one, PRs are always welcome. Due to ongoing projects I am not able to give full time to this project. Even I also wanted such a driver for one of my project which I have built using Neo4j (NeoEloquent Driver). Hope to build it better.

@sgpatil I didn't mean to come off rude in my comment so I apologize if I did and thank you for the work you have done on this. I would like to help out, do you have any documentation on a recommend path to making the driver?


source: support create materialized source x with paused

Sometimes users might need to wait hours before a materialized source has been fully ingested. We can create sources with paused state, and recover them when users have created all materialized views they need.
Alternative: support create materialized view on in-progress MV

Sounds weird to me 🥵


[WIP] Port realtime servo tutorial

Description
From issue #39, this ports "Realtime Arm Servoing" for MoveIt2.
This leans heavily on the demos from moveit_servo/doc/ which deviate significantly from the ROS1 tutorials.
I also added a way to test servoing with the keyboard, based off the turtlesim teleop demo over in ROS tutorials.
Open questions from me

Most of the code lives in "demos" in the moveit_servo code, should those demos be brought to this repo instead?
Similar to above, should the moveit_servo/doc/ directory be removed from the core code there? If I recall, I had originally put it there because the moveit2 tutorials were not off the ground yet
How much of the background or "Servo Overview" stuff should actually be included?

Checklist

[ ] Required by CI: Code is auto formatted using clang-format
[ ] While waiting for someone to review your request, please consider reviewing another open pull request to support the maintainers

I think the consensus is to move as much of the demos/examples/docs from moveit2 to here in moveit2_tutorials. Unless @AndyZe disagrees, I will work on moving the files over and will open a PR removing them from the other repo.

I think the consensus is to move as much of the demos/examples/docs from moveit2 to here in moveit2_tutorials. Unless @AndyZe disagrees, I will work on moving the files over and will open a PR removing them from the other repo.

Yes! Sorry I totally forgot to write here after the meeting.

I moved the documentation from the source to tutorials and updated the RST to reflect
I also added the C++ interface code, but decided to modify it. It had been creating a class that inherited from rclcpp::Node and using that through the demo. My understanding is that this is the recommended ROS2 design. However for the tutorial, I think it made the code hard to read/understand so I moved to putting everything in the main function and using a few global variables. Please let me know if that was a bad idea and I can revert it

@AdamPettinger could you resolve that conflict and I think this will be ready to merge.


Create a Web API: Update prerelease specific steps after .NET 7 GA

After .NET 7.0 GA, for the Visual Studio for Mac instruction, remove the following steps from prerelease to GA:
For the VS for Mac section detailing how to install packages from the package manager, remove this step:

Select Include prereleases.

For the VS for Mac section on installing scaffolding packages and tools using CLI:
Remove the "-v 7.0.0-*" from each of the add package commands.

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

ID: 583ae3e7-5915-f22b-e54c-6c100cf2a7da
Version Independent ID: 95f2549e-29f9-e31a-b4d0-432b971643ff
Content: Tutorial: Create a web API with ASP.NET Core
Content Source: aspnetcore/tutorials/first-web-api.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Associated WorkItem - 33254

Verified this has been fixed and reviewed the package instruction.


Add concurrent execution support to Executor

To answer your earlier question: yes, the tests pass under TSAN.


fix(twig-component): fix  syntax with empty attribute, close #925

Q
A

Bug fix?
yes

New feature?
no

Tickets
Fix #925

License
MIT

This PR is a proposal for fixing the issue with Twig Components when using the <twig:Component> syntax with empty attributes.

Thanks a lot! We completely forget this issue! 😅

This is awesome - thank you @Kocal!


Feature: create search ink for error

This PR creates a search link for StackOverflow similar problems.

:+1:
Published in version 2.1.23. Will be available in a few hours.
Thanks!

Sir,  I, sorry for the confusion I may have created, I am fairly new at
using and understanding the services provided in and on the internet. I
have been learning as I go, at times its been quite frustrating. I will
have things running smoothly and then I would have problems, I would read
the error message and see that somehow the extensions would be disabled and
try to reinstall them. in the beginning they would be reinstalled and then
be disabled again. Then the requests would be unable to reinstall. Then I
couldnt access certain places because the cookies were blocked in
programs.Last time I was trying to access my email and I was having
problems so I went into setting and when I saw the problem , I could not
access plus and had to go through the protocol for access, once again I
endured more problems , now the cookies were blocked there as well. I was
following the options prompt to fix the error in notifications. You can see
my spellings not the greatest , I downloaded a spell correct program that
worked perfectly out side of plus, but not at all in plus, infact perfectly
spelt words would be changed after I would send the post. Those are the
issues Ive been dealing with for quite some time, so my apologizes for any
inconvenice I may have caused. I just upgraded to Windows 10 and I hope
that improves my situation. Thank you  Adam Gerard
On Thu, Sep 17, 2015 at 11:27 AM, Sergey<EMAIL_ADDRESS>wrote:

Merged #26
https://github.com/barbushin/javascript-errors-notifier/pull/26.
—
Reply to this email directly or view it on GitHub
https://github.com/barbushin/javascript-errors-notifier/pull/26#event-412412989
.

It's okay, don't worry :) there was missing icons in JEN repo, just merge my pull request to get fixed version https://github.com/GabrielJMJ/javascript-errors-notifier/pull/1


Unexpected panic since client.mu.Rluck

Please use the following questions as a guideline to help me answer
your issue/question without further inquiry. Thank you.
Which version of Elastic are you using?
[ ] elastic.v7 (for Elasticsearch 7.x)
[ x] elastic.v6 (for Elasticsearch 6.x)
[ ] elastic.v5 (for Elasticsearch 5.x)
[ ] elastic.v3 (for Elasticsearch 2.x)
[ ] elastic.v2 (for Elasticsearch 1.x)
Please describe the expected behavior
never panic
panic: runtime error: invalid memory address or nil pointer dereference [recovered]
	panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x50 pc=0x7754ea]
goroutine 3775 [running]:
panic(0x28c85c0, 0x52c6a90)
	golang/go/src/runtime/panic.go:565 +0x2c5 fp=0xc00d419c28 sp=0xc00d419b98 pc=0x43b6e5
runtime.call256(0x0, 0x2e9a590, 0xc00c2013b0, 0x9800000098)
	golang/go/src/runtime/asm_amd64.s:522 +0x55 fp=0xc00d419e50 sp=0xc00d419d40 pc=0x4698a5
panic(0x28c85c0, 0x52c6a90)
	golang/go/src/runtime/panic.go:522 +0x1b5 fp=0xc00d419ee0 sp=0xc00d419e50 pc=0x43b5d5
runtime.panicmem(...)
	golang/go/src/runtime/panic.go:82
runtime.sigpanic()
	golang/go/src/runtime/signal_unix.go:390 +0x411 fp=0xc00d419f10 sp=0xc00d419ee0 pc=0x4515e1
sync.(*RWMutex).RLock(...)
	golang/go/src/sync/rwmutex.go:48
gopkg.in/olivere/elastic%2ev6.(*Client).PerformRequest(0x0, 0x340e840, 0xc00d3b5200, 0x2db3fb4, 0x4, 0xc00d39d560, 0x1a, 0xc00d4331d0, 0x28316a0, 0xc00d433200, ...)
	/home/ubuntu/jenkins/workspace/go/pkg/mod/gopkg.in/olivere/elastic.v6@v6.2.10/client.go:1252 +0xea fp=0xc00d41a1c0 sp=0xc00d419f10 pc=0x7754ea
gopkg.in/olivere/elastic%2ev6.(*SearchService).Do(0xc00c640a90, 0x340e840, 0xc00d3b5200, 0xa, 0xbf6bae7d0f64eec7, 0xd0824062c)
	/home/ubuntu/jenkins/workspace/go/pkg/mod/gopkg.in/olivere/elastic.v6@v6.2.10/search.go:401 +0x143 fp=0xc00d41a318 sp=0xc00d41a1c0 pc=0x7d3d63

Please describe the actual behavior
unexpect panic
Any steps to reproduce the behavior?
no. just happen once.

First of all, please use a valid import path. Use github.com/olivere/elastic for v6 as described here.
Then you accessed this line with a *elastic.Client of nil, hence the 0x0 in (*Client).PerformRequest(0x0, . Please make sure you use a valid *elastic.Client instance.


Rebuild the page once a week & Personal Access Token is no longer needed

This PR makes sure that the page is rebuilt at least once a week for updated crate or repo descriptions etc.
Also the personal access token is no longer needed and makes it easier to test it on forks.
see https://github.com/crazy-max/ghaction-github-pages/issues/1#issuecomment-623202206

Personal Access Token is not required anymore to trigger a page build on a public repository. Nothing official from GitHub but everything seems to work for me on several repositories with the built-in GITHUB_TOKEN (public and private).

Thanks!


InitSpace does not support tuple structs

If we try to derive InitSpace on a tuple struct, it will fail with the error Please use named fields in account structure. This is annoying, but the fact that the code seems to intentionally reject this case makes me wonder if there's a reason behind intentionally not supporting this?
Is there one or would you be open to a PR fixing this @acheroncrypto?

This is annoying, but the fact that the code seems to intentionally reject this case makes me wonder if there's a reason behind intentionally not supporting this?

It was intentional because Anchor did not support tuple structs during that time. It was added in v0.30 (https://github.com/coral-xyz/anchor/pull/2824).

Is there one or would you be open to a PR fixing this @acheroncrypto?

I don't see any reason to not support this, go ahead.

Added in https://github.com/coral-xyz/anchor/pull/3084


update role for ceph scaleout

Adding a new role that restarts the ceph services in a serialized manner after ceph expansion. The noout option is set when restarting ceph osd nodes in order to prevent the cluster from re-balancing. Also updating the sync_interval field for ceph.conf.

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist


Doesn't work or exist

I have 2 alternatives in the  save menu.
"Save as query", which takes me to a form that  can be filled in, but not submitted.
Apparently because I don't have the rights.
No one knows how/where to change those rights.
"Save as function", which is disabled.
No clue why.
Why does it need to be THAT complicated just to save a query?
Why cant I simply bookmark the page in my browser.
It's literally ONE click.
And its how searches  usually  work on other sites.
If the url and title of the page was updated to reflect the current query, perhaps when you run it.
It would be the simplest thing in the world to save a query, just one click.
Now I have to use the share/copy link to query -function, and its  like Russian roulette getting it working.
Getting to the point where you have a 100% verified working findable bookmark to a query, takes dozens and dozens of steps, and about half an hour of work on average.

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

ID: b25cda55-9c4a-cbcd-d66a-44c9b58375eb
Version Independent ID: a4adea02-d3ac-ba36-752c-dcb34f187411
Content: Save a query in Azure Monitor Log Analytics (preview) - Azure Monitor
Content Source: articles/azure-monitor/logs/save-query.md
Service: azure-monitor
Sub-service: logs
GitHub Login: @guywi-ms
Microsoft Alias: guywild

@MatsSvenssonNewseed - Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can review and update accordingly.

@MatsSvenssonNewseed Thanks for your feedback! We've created a work item to follow up and address this issue. I'm going to go ahead and close this issue now. Thanks again for your dedication to making our product better👍
#please-close


[Improvement] Add XDEBUG_SESSION_START to docs

Nice that xdebug comes preinstalled, i think it's worth mentioning in the docs that to debug any api route, e.g. with Postman, you can set a header XDEBUG_SESSION_START=PHPSTORM or append it as a get parameter. It will trigger a PHPStorm debug session.
Of course if there is any other option for it, i couldn't get the remote_autostart/connect_back_on to work.

Thanks for giving us the idea.
It's now included in the dockware docs.
https://dockware.io/docs#debugging


Completing a task via keyboard shortcut (CMD + L) does not append the completion date

Please check that this issue hasn't been reported before.

[X] I searched previous Bug Reports didn't find any similar reports.

Expected Behavior
When I use the trackpad/mouse pointer to click on a task checkbox and complete the task, the date at which this task was completed is appended to the task
Current behaviour
When I use Obisidian's keyboard shortcut for completing the task (CMD + L), the task gets completed but the completion date is not appended.
Steps to reproduce
Type CMD + L while the caret is on a task.
Which Operating Systems are you using?

[ ] Android
[ ] iPhone/iPad
[ ] Linux
[X] macOS
[ ] Windows

Obsidian Version
1.15.12
Tasks Plugin Version
7.2.0
Checks

[X] I have tried it with all other plugins disabled and the error still occurs

Possible solution
Unknown

Current behaviour
When I use Obisidian's keyboard shortcut for completing the task (CMD + L), the task gets completed but the completion date is not appended.

Yes that's right. Obsidian doesn't know anything about the Tasks completion date emoji.
You can find the name of the Tasks command to re-bind Cmd+L to in the Installation notes.

Can you please kindly confirm that this works for you so that I can close the ticket?

No action needed....

I've closed this as I don't believe any action on Tasks is needed...
If for some unexpected reason you find that the info above doesn't work for you, do let  me know, giving more info, and I will of course re-open the ticket.


Fix Auth_url for AAD env (stack) (#6500)

This checklist is used to make sure that common guidelines for a pull request are followed.

[na] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[na] I adhere to the Command Guidelines.

//CC @troydai

Codecov Report

Merging #6504 into rc38 will not change coverage.
The diff coverage is n/a.

@@         Coverage Diff          @@
##           rc38   #6504   +/-   ##
====================================
  Coverage     0%      0%
====================================
  Files        11      11
  Lines       133     133
  Branches      9       9
====================================
  Misses      133     133

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 8f9d542...5368b26. Read the comment docs.


Incorrect results when evaluate OnePiece on RefCOCO+

Hi, I evaluate OnePiece following the guidance of https://github.com/OFA-Sys/ONE-PEACE/blob/main/one_peace/README.md with the weights of finetune_refcoco+.pt on dataset/refcoco+/val.tsv  on a single GPU and using fp16, but the results are:
INFO:one_peace.evaluate:{'iou_acc': 0.29689533370514964, 'score_sum': 3194.0, 'score_cnt': 10758}
I don't know how to solve this issue, do you have any suggestions?
Thank you!

@Yunpeng-Qi Hi, I tested the latest code and it works fine. Can you double-check that you've updated your code and used the correct checkpoint?
INFO:one_peace.evaluate:{'iou_acc': 0.8874326082915039, 'score_sum': 9547.0, 'score_cnt': 10758}