Pass app into group callable

As discussed in #1976, route group callables are bound to App, making the use of closures easy, but App is not passed to groups on invoke so an invokable class will not work.
I chose to remove the closure binding altogether as passing App through on invoke negates the need to do this anymore. The tests weren't even using the feature...

Coverage decreased (-0.02%) to 94.118% when pulling 562a683030ddbd00af7a33c61ddea689ea6b1a75 on geekish:feature/group-callable into 72eea92845b090ce8c8f56702a571bbbf7fdb01f on slimphp:4.x.

I am happy with this change getting added into Slim 4, @geekish has also provided example code to show the use of this feature, which current implementation doesn't allow.
https://gist.github.com/geekish/3a1cf866aec0a6ee1935007cbf915d78

Coverage decreased (-0.02%) to 94.118% when pulling de333a6ae755dd73f6aac36d3753cd3779b713f2 on geekish:feature/group-callable into 72eea92845b090ce8c8f56702a571bbbf7fdb01f on slimphp:4.x.

I need to think about it more. The idea of passing App around strikes me as a code smell.
I understand the desire to fix the problem with route groups, and registering routes and what not.
I think it would make more sense to only pass through the Router object.
//cc @akrabat

@geggleto we can't pass the Router instead, because the Router doesn't carry these methods, these are helper methods [get, post, put, delete, map] that are created inside of App.

@silentworks maybe they should be on the router and not the app ;)

I would definitely prefer that we pass Router through instead and get rid of the helper methods on App, but that would be out of the scope for this PR.
Besides that, I don't see how passing App through is any smellier than binding the closure to it; that kind of magic seems pretty smelly to me.

I don't like this PR, there is no reason to add App to the route group, routing and application are separate packages.

@sharifzadesina we are already doing this in the form of closure binding. So I am not sure why there is fuss about this PR.

@silentworks of course you are, and I know this will sequent injection of the App class, but this is not good design, imagine router without App class, what will happen? your router will be useless.

@sharifzadesina so the point you are making has nothing to do with this PR. Someone else should send another PR to fix the issue you are concerned about.

@silentworks Maybe, but by this PR you are making the bad design bigger and bigger. anyway this is just my idea.

@sharifzadesina no we aren't making it bigger, it still the same magnitude as it was before this PR. This PR however is making the use more consistent.

For past commenters and future commenters on this issue, I want to make it clear that coming with an argument about design decision has no effect on this PR, this PR was created after the design decision was taken, this PR is just making it more clear as to how you create routes in route groups.
I find it distasteful that many have commented about bad design on a PR by a first time contributor that has/had no effect on the bad design decision that was taken more than a year ago. If you wish to fix this bad design decision, raise a new issue or send a PR, please don't make this PR out to be what its not.

@geekish I think we should revisit the groups in a different way. :)

One side effect of removing the binding is that it will be that much harder to upgrade from 3 to 4. Is the fact that you don't like the closure binding feature of PHP a reason to remove it?

I think its more of consistency and my IDE would know how to give me autocomplete hint based on $app than currently where I get none based on $this.

I'm happy to pass in $app to the closure / __invoke(), I'm just questioning whether we should remove the binding of $app to  $this for closures.
I think that smoothing out upgraders from 3 warrants it.

Note that we could also do this for 3.x to provide a forward migration path.

Coverage decreased (-1.1%) to 93.035% when pulling f9995d22d96e6ae8f30448a34e87c96b9c6c4cc2 on geekish:feature/group-callable into 72eea92845b090ce8c8f56702a571bbbf7fdb01f on slimphp:4.x.

I've re-added the closure binding, but didn't add a trigger_error - I agree there needs to be some way to notify the user about the deprecation but that would trigger on every route using a closure whether the user utilizes the bind or not.

thank you @geekish  :)

Any reason we can't do this for Slim 3 too?


⚠ Shadow Does Streams has degraded performance

In 831f324, Shadow Does Streams (https://stream.everythingbagel.me) experienced degraded performance:

HTTP code: 200
Response time: 3148 ms

Resolved: Shadow Does Streams performance has improved in ec98ee6 after 43 minutes.


ENH: Filling of missing data points

This PR addresses #321

TODO:

[ ] Option for filling undefined data points should not be shown for nonuniform line scans
[ ] Test nonuniform line scans that to not have the interpolate_undefined_data pipeline function.

Hi @mcrot - I think this is ready for your review


Correctly handle filtered-out latest versions

Basically, in the case that there are two versions of a crate:

foo-1
foo-2

cargo vendor will write directories named:

vendor/foo-1
vendor/foo (meaning foo-2)

We weren't handling the case where the newer version of these was filtered out.
To correctly handle this, we need to always gather an unfiltered view to know what cargo vendor will do, and then build up a mapping from package IDs to that name.
Closes: https://github.com/coreos/cargo-vendor-filterer/issues/75

CI / test-crates<EMAIL_ADDRESS>(pull_request) Successful in 2m

:tada:


Updated OnlinePstnGateway with new parameters

Added descriptions for the 2 parameters Description and GatewayLbrEnabledUserOverride
Add a sentence to description for FailoverResponseCodes
Changed order of parameter description
Changed applicable to be Microsoft Teams

Docs Build status updates of commit 38c1575:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/New-CsOnlinePSTNGateway.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Set-CsOnlinePSTNGateway.md
:white_check_mark:Succeeded
View (skype-ps)

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

@dariomws @tseward @get-itips

@JensTrier Please review. Thanks

Docs Build status updates of commit 3361c23:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/New-CsOnlinePSTNGateway.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Set-CsOnlinePSTNGateway.md
:white_check_mark:Succeeded
View (skype-ps)

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

Updated both new and set to address the comments.

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

File
Status
Preview URL
Details

skype/skype-ps/skype/New-CsOnlinePSTNGateway.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Set-CsOnlinePSTNGateway.md
:white_check_mark:Succeeded
View (skype-ps)

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

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

File
Status
Preview URL
Details

skype/skype-ps/skype/New-CsOnlinePSTNGateway.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Set-CsOnlinePSTNGateway.md
:white_check_mark:Succeeded
View (skype-ps)

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

@get-itips

@tiburd Please merge this PR. Thanks

Docs Build status updates of commit 6633dd6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/New-CsOnlinePSTNGateway.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Set-CsOnlinePSTNGateway.md
:white_check_mark:Succeeded
View (skype-ps)

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


Added Travis build image and sample topology screenshot to the README

Added the Travis build status to the README as well as a sample topology screenshot to better portray the purpose of Streamflow.

Self-merging simple README changes


[queryParamsStore] enhancements

Since your project is AMAZING, I was wondering what do you think about creating a "kit for Kit" with Svelte 5 that allows us to save in the page URL any POJO object or scalar type we need with the serializer we need (for example I can use JSON.stringify() and JSON.parse() for now).
I described something similar here.
This would be amazing for lists filters for example:

I have a players list on page /players

I order the list by created_at DESC

the kit saves this filter in the URL: /players?order_by=created_at_DESC (just an example)

I filter players form team_id: 123

the kit saves this new filter in the URL: /players?order_by=created_at_DESC&team_id=123 (just an example)

I go back with the browser and the URL is now again: /players?order_by=created_at_DESC (just an example)

I reload the page now and since the URL is this the list is ordered like before the reload

I hope you get the idea.
I now can manually add this using in each place I need it url.searchParams.set and afterNavigate but I would like to have an helper to be used everywhere.
Again, thanks for your work!

Thanks for the kinds words @frederikhors 😊.
If I'm understanding correctly, It sounds like what you are looking for is query string management with different strategies.  Svelte UX has queryParamsStore (manage multiple params) as well as queryParamStore (single param) with different types/strategies.  There is a pretty robust test suite and docs to see the differences, but maybe I'm misunderstanding what you are looking for.
@paoloricciuti also has the wonderful https://github.com/paoloricciuti/sveltekit-search-params which I haven't looked closely at yet, but I think it's similar and likely has better SvelteKit integration (I created queryParamsStore for SPAs before SvelteKit existed)

Thanks for your answer.
I tried queryParamsStore and it works good (thanks again for your work!) but does not all I need.
I created a small reproduction: see the file /svelte-ux/+page.svelte:

since I'm using new Svelte 5's $props() I'm assigning:
const filters = queryParamsStore({
  defaults: {
    condition: {},
    searchText: undefined,
  },
  paramTypes: (key) => {
    switch (key) {
      case "condition":
        return "object";
      case "searchText":
        return "string";
    }
  },
  page,
  goto,
});

let { pagination = { page: 1, size: 10 }, condition = $filters.condition } = $props();

and obviously if I assign something to condition.name = "bob" it doesn't work

I cannot assign to $filters.condition neither

The same goes with searchText (simple string): if I assign it as default props value and I bind it with an input it doesn't work

These are very useful features especially with Svelte 5.
I'm already talking with @paoloricciuti here about this.

Thanks for the added context and discussion.  If i understand the problem you want:

Fine-grained reactivity (for each param) with the ability to bind each parameters, including nested properties (ex. condition.name or pagination.page
You want to set an initial default value (constant) or default to a props value, but will be overridden by the query param value when set, but used anytime it is not defined in the query string.

You MIGHT be able to use queryParamStore for each parameter (and thus a store for each) instead of managing all params with a single queryParamsStore.
I haven't tried testing this and rather buried with other tasks ATM.  I plan to migrate all Svelte UX stores to Svelte 5 with runes but it will take some time.
import { queryParamsStore } from 'svelte-ux';
import { page } from '$app/stores';
import { goto } from '$app/navigation';

let { pagination, condition } = $props();

const paginationParam = queryParamStore({
  name: 'pagination',
  default: pagination ?? { page: 1, size: 10 },
  paramType: 'object',
  page,
  goto,
});

const conditionParam = queryParamStore({
  name: 'condition',
  default: condition,
  paramType: 'object',
  page,
  goto,
});

<input type="text" bind:value={$conditionParam.name} />

I did try to update your SvelteLab example with the following...

condition: {JSON.stringify($conditionParam)}

Search player by name:

<input type="text" bind:value={$conditionParam.name} />

{#each players as player}
	name: {player.name} - gender: {player.gender}

{/each}

but it didn't work

I will try in a moment. I think the pushState() has a bug and needs setTimeout for now.
But in the meantime, I opened https://github.com/sveltejs/kit/discussions/12460 right now.
That code works (not so performant or goo) and is very repetitive, but it's a start for me.
Do you think the code you posted can do the same?

TBH I'm not sure.  queryParamStore does not use Svelte 5 $effect and just calls goto from $app/navigation based on store.set being called.

I updated https://www.sveltelab.dev/4vnxej0ewwtlwe9?files=.%2Fsrc%2Froutes%2Fsvelte-ux%2F%2Bpage.svelte.
Condition is not persisted in the URL at all.
😢

I updated https://www.sveltelab.dev/4vnxej0ewwtlwe9?files=.%2Fsrc%2Froutes%2Fsvelte-ux%2F%2Bpage.svelte.
Condition is not persisted in the URL at all.
😢

Look in your console 😉.  This was the error I was referring to.

You're right. I don't know what to do.
I'm shocked that I haven't found anything similar to what I need anywhere on the web and yet it's something that should be very useful to a lot of people!

I think rolling something custom like you have in https://github.com/sveltejs/kit/discussions/12460 is your best choice right now.
Svelte UX's plan for Svelte 5 is to get it compatible with Svelte 3-5 which is helping to tease out some Svelte 5 regressions.  I'm fairly close on getting the full docs running on Svelte 5 (it does locally, but currently fails some svelte-check and I had to remove sveld generated API docs until I find a solution).  I think the root issue is the AST changing.  I'm considering parsing the AST myself, but just haven't found the bandwidth.  There are some additional regressions I've seen while using the docs locally I need to identify a small reproduction and report as well.  Migrating the LayerChart docs to Svelte 5 is also a priority.
I already have Github Analysis and Strava Analysis running on Svelte 5, but know there is at least an issue with DateRangeField in some cases I need to investigate / report.

LayerChart's simplified charts is my current priority when I have time for open source, which ebbs and flows.


gperftools 2.6.3 build failure at OS X 10.11.6 (El Capitan) with clang-703.0.31 (Apple LLVM version 7.3.0)

I hit an issue with building gperftools 2.6.3 at OS X 10.11.6:
/bin/sh ./libtool  --tag=CXX   --mode=link g++ -DNO_TCMALLOC_SAMPLES -D_THREAD_SAFE  -DNDEBUG -Wall
-Wwrite-strings -Woverloaded-virtual -Wno-sign-compare -fno-builtin-malloc -fno-builtin-free -fno-builtin-realloc -fno-builtin-calloc -fno-builtin-cfree -fno-builtin-memalign -fno-builtin-posix_memalign -fno-builtin-valloc -fno-builtin-pvalloc  -Wno-unused-result -fsized-deallocation    -g -fno-omit-frame-pointer -O2   -stdlib=libc++ -version-info 9:1:5 -no-undefined  -stdlib=libc++ -o libtcmalloc_minimal.la -rpath /Users/aserbin/Projects/kudu/thirdparty/installed/uninstrumented/lib src/libtcmalloc_minimal_la-tcmalloc.lo    libtcmalloc_minimal_internal.la -lc++ -lc++abi
libtool: link: g++ -dynamiclib  -o .libs/libtcmalloc_minimal.4.dylib  src/.libs/libtcmalloc_minimal_la-tcmalloc.o   -Wl,-force_load,./.libs/libtcmalloc_minimal_internal.a  -lc++ -lc++abi  -g -O2 -stdlib=libc++ -stdlib=libc++   -install_name  /Users/aserbin/Projects/kudu/thirdparty/installed/uninstrumented/lib/libtcmalloc_minimal.4.dylib -compatibility_version 10 -current_version 10.1 -Wl,-single_module
Undefined symbols for architecture x86_64:
"operator delete(void*, unsigned long)", referenced from:
TCMallocImplementation::~TCMallocImplementation() in libtcmalloc_minimal_la-tcmalloc.o
SbrkSysAllocator::~SbrkSysAllocator() in libtcmalloc_minimal_internal.a(libtcmalloc_minimal_internal_la-system-alloc.o)
MmapSysAllocator::~MmapSysAllocator() in libtcmalloc_minimal_internal.a(libtcmalloc_minimal_internal_la-system-alloc.o)
DevMemSysAllocator::~DevMemSysAllocator() in libtcmalloc_minimal_internal.a(libtcmalloc_minimal_internal_la-system-alloc.o)
DefaultSysAllocator::~DefaultSysAllocator() in libtcmalloc_minimal_internal.a(libtcmalloc_minimal_internal_la-system-alloc.o)
__GLOBAL__sub_I_symbolize.cc in libtcmalloc_minimal_internal.a(libtcmalloc_minimal_internal_la-symbolize.o)
SysAllocator::~SysAllocator() in libtcmalloc_minimal_internal.a(libtcmalloc_minimal_internal_la-malloc_extension.o)
The configure script was run with the following essential flags:
--enable-frame-pointers
--enable-heap-checker
--with-pic
Some information about the system:
$ sw_vers
ProductName:  Mac OS X
ProductVersion: 10.11.6
BuildVersion: 15G1611
$ g++ --version
Configured with: --prefix=/Applications/Xcode.app/Contents/Developer/usr --with-gxx-include-dir=/usr/include/c++/4.2.1
Apple LLVM version 7.3.0 (clang-703.0.31)
Target: x86_64-apple-darwin15.6.0
Thread model: posix
InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin
While digging in and compiling the test that the configure scripts uses to determine whether -fsized-deallocation is supported by the host compiler, I found some surprises; see below.
test.cc
`#include
int main () {
(::operator delete)(0, 256);
return 0;
}`
As it turned out, the configure scripts runs the test with '-02' flag, and in my case it produces no errors:
$ c++ -fsized-deallocation -O2 test.cc
$ echo $?
0
Even with -01 it passes:
$ c++ -fsized-deallocation -O1 test.cc
$ echo $?
0
However, if compiled without optimizations and built-in function replacement (actually, either one suffices):
dhcp-10-16-1-175:tmp$ g++ -fsized-deallocation -O0 -fno-builtin test.cc
Undefined symbols for architecture x86_64:
"operator delete(void*, unsigned long)", referenced from:
_main in test-c91491.o
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
$ echo $?
1
Also, the AC_COMPILE_IFELSE used by configure.ac for that test, but the error comes from the linkage phase.  Maybe, it's worth replacing it with AC_LINK_IFELSE?
Overall, the zipped patch that un-breaks the build of gperftools-2.6.3 on my laptop is attached.
gperftools-sized-alloc-build-fix.diff.zip

Thanks for report. Very "nice" finding (and not nice for apple screwing us more).
I am a bit reluctant to have -fno-builtin added to flags for testing compiler features. There is chance that we'll be dealing with compiler that doesn't know -fno-builtin but does have features we're testing for.
I like idea of AC_LINK_IFELSE. And we can change test for operator delete to ensure that compiler doesn't optimize it away. I.e. like this:
int main () {   static void (* volatile ptr)(void *, size_t) = ::operator delete;   (*ptr)(0, 256);   return 0; }
Can you please try something like that on your system? E.g. https://godbolt.org/g/N6qVGG

Adding -O0 is enough, actually.  I added both -O0 and -fno-builtin 'just in case' while playing with that test.
Yep, the new test works better in my case.  I guess it prevents the compiler from making corresponding optimizations.  The code at https://godbolt.org/g/N6qVGG fails to compile (as expected) for all optimization levels -O{0,1,2,3,s}.  Below is the output for -O0 and -O2, plus the output from 'c++ --version':
$ c++ -fsized-deallocation -O0 test1.cc
Undefined symbols for architecture x86_64:
"operator delete(void*, unsigned long)", referenced from:
main::ptr in test1-188497.o
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
$ c++ -fsized-deallocation -O2 test1.cc
Undefined symbols for architecture x86_64:
"operator delete(void*, unsigned long)", referenced from:
main::ptr in test1-8f1c91.o
ld: symbol(s) not found for architecture x86_64
$ c++ --version
Apple LLVM version 7.3.0 (clang-703.0.31)
Target: x86_64-apple-darwin15.6.0
Thread model: posix
InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin

Should be fixed. Please let me know if not.


Side menu appear when we navigate back.

Side menu appear when we navigate back previous route even isOpen is false. I get the small glimpse of it.
My menu background color is black (Even also tried making it white).
Any solution developers ;)

Seems to be related to https://github.com/react-native-fellowship/react-native-side-menu/issues/172

See #172


Search attribute checker

What changed?

Why?

How did you test it?
Valid Case:

False Case:

Potential risks

Release notes

Documentation Changes

Pull Request Test Coverage Report for Build 0189adeb-2f86-46d8-984e-691fa9ca7897

5 of 78 (6.41%)  changed or added relevant lines in 1 file are covered.
39 unchanged lines in 10 files lost coverage.
Overall coverage increased (+0.03%) to 56.953%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

tools/cli/domainCommands.go
5
78
6.41%

Files with Coverage Reduction
New Missed Lines
%

common/task/weightedRoundRobinTaskScheduler.go
2
89.64%

service/history/execution/mutable_state_builder.go
2
68.6%

service/history/handler.go
2
47.28%

common/task/fifoTaskScheduler.go
3
84.54%

host/taskpoller.go
3
72.5%

common/types/history.go
4
45.35%

common/types/mapper/thrift/shared.go
4
63.2%

service/history/historyEngine.go
4
68.79%

service/history/decision/task_handler.go
5
72.33%

service/history/task/transfer_standby_task_executor.go
10
86.21%

Totals

Change from base Build 018999a3-1486-424f-835f-83edd082d3a5:
0.03%

Covered Lines:
87218

Relevant Lines:
153140

💛  - Coveralls


match patterns

example enum:
enum E {
    a,
    b { c: u32 },
}

there is a lot of
match e {
    &e::a => ... ,
    &e::b { ref c } => ... ,
}

that can be replaced by
match *e {
    e::a => ... ,
    e::b { ref c } => ... ,
}

where e is a reference to this enum
Note: this needs to be ref c because e is just borrowed.
and there are a lot of matches on enums:
match e {
    e::a => ... ,
    e::b { .. } => ... ,
}

that can be replaced easily by:
use self::E::*;
match e {
    a => ... ,
    b { .. } => ... ,
}

where e is an value of this enum
NOTE: if you only use C-like enums, then the use line can be omitted.
If i have some spare time, there will be a PR for this in a few days.

Closing, because this is not going to be fixed, might be minded for future code


failed to build

I've installed firebase plugin in my ionic4 app and got error while building with this exception message:
BUILD FAILED in 9s
cmd: Command failed with exit code 1 Error output:
D:\ionicapp\platforms\android\app\src\main\java\org\apache\cordova\firebase\FirebasePluginMessagingService.java:128: error: constructor Builder in class Builder cannot b
e applied to given types;
            NotificationCompat.Builder notificationBuilder = new NotificationCompat.Builder(this, channelId);
                                                             ^
  required: Context
  found: FirebasePluginMessagingService,String
  reason: actual and formal argument lists differ in length
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
1 error

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app:compileReleaseJavaWithJavac'.
> Compilation failed; see the compiler error output for details.

my plugin version is 2.0.5
my config.xml looks like this:
<widget>
    ...
    <plugin name="cordova-plugin-firebase" spec="2.0.5" />
    <engine name="android" spec="7.1.4" />
</widget>

I've run in to the same issue, I'm using:
Cordova 8
Cordova Android 7.1.4
I've tried using:
cordova plugin add cordova-android-firebase-gradle-release  --variable FIREBASE_VERSION=15.+ --save
Also FIREBASE_VERSION=+
Any thoughts?
Thank you

Same problem here.
Cordova 8
Cordova Android 7.1.4
I added the necessary files
cordova-android-play-services-gradle-release and cordova-android-support-gradle-release
But no hope.


Ability to read request headers during "WebSocket upgrade"

In my application I need an ability to read custom headers from websocket handshake.
Possible workarounds discussed here: https://github.com/boostorg/beast/issues/1442 https://github.com/boostorg/beast/issues/501 https://github.com/boostorg/beast/issues/751
With current implementation developer needs to create both http session and websocket session. This is an overhead for websocket-only server.
In other libraries this issue is solved by saving http request for each connection, some examples:

Websocket++
QtWebsocket

However, saving upgrade request directly is not compatible with boost::beast design.
My idea is to make an overload of async_accept function similar to this one. But instead of passing already created request, pass a reference where request will be stored after handshake.

saving upgrade request directly is not compatible with boost::beast design.

Why not? Just read the request yourself using Beast (e.g. beast::http::read) and then if it is a WebSocket Upgrade request (i.e. beast::websocket::is_upgrade returns true) call the overload of accept which lets you pass the message:
https://www.boost.org/doc/libs/1_71_0/libs/beast/doc/html/beast/ref/boost__beast__websocket__stream/accept/overload6.html
This is demonstrated in the advanced examples:
https://github.com/boostorg/beast/blob/6d614cf9a8c6d81258e309682621286bf9d74ec3/example/advanced/server/advanced_server.cpp#L482

Just to clarify everything, is it true that these two snippets work similar?
Standart accept:
ws_.async_accept([this](beast::error_code ec)
{
    if (ec) return;
    OnHandshake();
});

Accept with saving request:
// _req is a class member
flat_buffer read_buffer;

http::async_read(ws_.next_layer(), read_buffer, req_, [this](beast::error_code ec)
{
    if (ec) return;
    ws_.async_accept(_req, [this](beast::error_code ec)
    {
        if (ec) return;
        OnHandshake();
    });
});

Yes that looks right.


finetune_300.tar

Unable to decompress file finetune_300.tar
If i try to decompress, it is not a compressed file.
What should i do if i try to decompress?

Hi,
Sorry about that. I did not explain it clearly. Even though the suffix is 'tar', but it is not a compressed file. You don't have to decompress it. You can directly load it using torch.load().

Aha! Thank you for the explanation!


Layout with Variable Size Tiles

This is a layout challenge, not a TilesFX issue, but I'm hoping you could make a recommendation.
I built a simple dashboard app with TilesFX, and most of the tiles are simple square metrics and the FlowGridPane works great -- although I didn't notice it until recently and have been using a FlowLayout instead.
My problem is that I have some custom tiles (histogram/line charts) which span multiple cells. This works fine with a FlowLayout since it simply wraps to the next line, but FlowGridPane doesn't support spanning. I see that there are some comments about col/row span in FlowGridPane::relayout, so I'll probably experiment with that.
Another problem that I had is that the FlowLayout automatically redistributes the number of columns based on the window size. So people can have narrow / short dashboards by resizing the window. I'll look into this as well, but I'm not sure where to start.

Could you provide a code snippet that shows the problem so that I can play around with it?

I'm using Kotlin/TornadoFX and am not exactly sure how this maps to standard JavaFX. Basically adding some tiles with a preferred size to a minimally configured FlowPane.
    fun makeTile(w: Int, h: Int) = TileBuilder.create()
        .skinType(Tile.SkinType.CUSTOM)
        .prefSize(w.toDouble(), h.toDouble())
        .roundedCorners(true)
        .title("hello tile world")
        .build()

    val sizes = listOf(
            Pair(200,200), Pair(200,200), Pair(200,200), Pair(200,200), Pair(200,200),
            Pair(405,200), Pair(405, 200), Pair(200,200),
            Pair(200,200), Pair(200,200), Pair(200,200), Pair(200,200), Pair(200,200)
            )

    override val root = flowpane {
        hgap = tileHgap()
        vgap = tileVgap()
        prefWidth = tileWidth(5)
        prefHeight = tileHeight(3)

        sizes.forEach {
            add(makeTile(it.first, it.second))
        }
    }

This is the main layout I'm targetting. If I could do this with the GridFlowLayout I'd be very happy.

These other two are nice, my system has several networks so it's common to have multiple instances of the dashboard open and some users like to resize the window to fit their preferred layout.

Concretely, my multi-cell tiles are simple custom tile where I've embedded charts:

Well that's not to hard, you simply should use a GridPane and set the column span for the wide nodes to 2. Here is an example (it's plain JavaFX but should be no big difference to TornadoFX):
`public class LayoutDemo extends Application {
private GridPane pane;
@Override public void init() {
    pane = new GridPane();
    pane.setHgap(5);
    pane.setVgap(5);

    for (int i = 0 ; i < 5 ; i++) {
        Tile tile = createTile();
        pane.add(tile, i, 0);
    }
    Tile tile1 = createWideTile();
    pane.add(tile1, 0, 1);

    Tile tile2 = createWideTile();
    pane.add(tile2, 2, 1);

    Tile tile3 = createTile();
    pane.add(tile3, 4, 1);

    for (int i = 0 ; i < 5 ; i++) {
        Tile tile = createTile();
        pane.add(tile, i, 2);
    }
}

@Override public void start(Stage stage) {
    Scene scene = new Scene(pane);

    stage.setTitle("Layout Demo");
    stage.setScene(scene);
    stage.show();
}

@Override public void stop() {
    System.exit(0);
}

private Tile createTile() {
    return TileBuilder.create()
                      .skinType(SkinType.TEXT)
                      .prefSize(200, 200)
                      .title("Hello World")
                      .build();
}
private Tile createWideTile() {
    Tile tile = TileBuilder.create()
                           .skinType(SkinType.TEXT)
                           .prefSize(405, 200)
                           .title("Hello Wide World")
                           .build();
    GridPane.setColumnSpan(tile, 2);
    return tile;
}

public static void main(String[] args) {
    launch(args);
}

}`


Installation error

While running the sudo bazel build -c opt //tensorflow/tools/pip_package:build_pip_package, I get the following error:
./tensorflow/python/client/tf_session_helper.h:6:10: fatal error: 'numpy/arrayobject.h' file not found
#include "numpy/arrayobject.h"
         ^
1 error generated.

However, I installed numpy from the source listed and it worked perfectly. Any suggestions?

I forgot to add I am on Mac OS X Mavericks

Edit:
By updating the file ./tensorflow/python/client/tf_session_helper.h to include the actual path of the numpy include directory of the computer, I have managed to get a new error,
bazel-out/local_darwin-opt/bin/tensorflow/python/pywrap_tensorflow.cc:3609:10: fatal error: 'numpy/arrayobject.h' file not found
#include "numpy/arrayobject.h"
         ^
1 warning and 1 error generated.

I have tried to edit this file the same way, but I cannot because it is just a symlink to a tmp direcotry, so it is regenerated each time.
Any ideas?

I removed the directory, reinstalled everything from scratch, then got the 1st error:
INFO: From Compiling tensorflow/python/client/tf_session_helper.cc:
In file included from tensorflow/python/client/tf_session_helper.cc:1:
./tensorflow/python/client/tf_session_helper.h:6:10: fatal error: 'numpy/arrayobject.h' file not found
#include "numpy/arrayobject.h"
         ^
1 error generated.

What next?

Any chance this is related to https://github.com/tensorflow/tensorflow/issues/187 ? (are you using anaconda python?

No, I installed numpy directly from source, and it is located in the right directory. The directory tree to the file the installer can't find is as follows:
/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/numpy/core/include/numpy/arrayobject.h

Hm, I'm not entirely sure what the answer is here.  https://github.com/BVLC/caffe/issues/1284 seems to suggest a similar problem, but the solution to 'reinstall numpy' doesn't really apply here.  Hopefully someone from the community can chime in and help.

We look for the numpy headers here:
/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/numpy/core/include
which is different from your setup.  Try changing this line accordingly.

Works great for first step. Waiting for next two to execute. Thanks ebrevdo!

Installation completely worked. Thanks!

hey @ebrevdo, getting this issue from the master branch right now
is there an analogous fix for it there or should I compile a different release?

having the following issue on release branches 1.8 - 1.10
/tensorflow/third_party/python_runtime/BUILD:5:1: no such package '@local_config_python//':

and having numpy import errors on 1.11 and 1.12 and master


Adds CSV Reporter for Output :#6

PR to Support CSV Output for Filter Results #6
https://github.com/safedep/vet/issues/6

@shivamsk Please ensure your branch is building. It seems there are syntax error in your branch.


mui5 compatible version

https://github.com/GemunIon/mui-rte

Thanks! Version 2.0.0 added compatibility with MUI (Material-UI v5). Check it here.


Add Debian 12 compatibility

Summary
Provide a detailed description of all the changes present in this pull request.
Additional Context
Add any additional context about the problem here.

[ ] Root cause and the steps to reproduce. (If applicable)
[ ] Thought process behind the implementation.

Related Issues (if any)
Mention any related issues or pull requests.
Checklist

[ ] 🟢 Spec tests.
[ ] 🟢 Acceptance tests.
[ ] Manually verified. (For example puppet apply)

Looks like no acceptance tests run on Debian 12, but I have no idea how that would happen.

As we do not have the support for debian 12 need to revert will get back on this...


App set-up

In this PR, we did the following ;

[ ] Set-up React app
[ ] Set-up linters
[ ] Add relevant packages

Back-end link : https://github.com/skabeo/backend_capstone/pull/17

Hi @yegonkimutai
Your project is complete! There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉


Adapt for other centers and experiments

I really like the intake features in combination with xarray. I wonder how your package could be applied on a different supercomputer.
I'm from MPI-M, Germany and would like to use intake-esm, collections, ... to load our large ensembles.
Do you see a way how intake-esm could be used at a different supercomputer? I think I get the rough concept, but its very complicated and seems to me very much tailored to the NCAR facilities.
But maybe a minimal working example would already be enough to understand the concept in order to adapt it to a new experiment. Right now I dont even know where to start...

I hope NCAR can be helpful for explaining things to you, @aaronspring , and they probably understand your use case better than I do; but still feel free to raise issues on Intake if you find bugs or unclear situations.

@aaronspring thanks for your interest in intake-esm. There is no inherent dependency on being on the NCAR machine, so you should be able to get this working at MPI. It will take a bit of effort, but we are happy to help and I think this would be useful exercise to help iron out some shortcomings in generality.
I think it's important to distinguish between building catalogs and using catalogs. The build process yields a database that includes files locations and their attributes (as a pandas dataframe). intake-esm builds catalogs by crawling directories and identifying files according to a filename-pattern match (for CESM) or the directory structure (CMIP). The database could be generated by other means too, for instance by connecting to a run database and filling the necessary fields.
You can look at
https://github.com/NCAR/intake-esm/blob/master/tests/cesm1-le_collection-input.yml
which provides an example input file used for building a catalog of the CESM1 Large Ensemble. You'll note that the locations fields specify paths; in the case of CESM, these locations are examined to find files that include the "case" string for each ensemble member. Logic in the code enables identifying the files, also relying on information in config.yaml where the types of collections we support are defined.
In order to support your use case, I think we need to understand whether your data fits into our paradigm in a general sense. For instance, for CESM, we identify files by matching patterns that look like this:

{casename}.{data_stream}.{variable}.{date_string}.nc

What do you think?

hi guys,
thanks for the effort.
All files follow this structure:
MPI-GE/rcp85/rcp850001/outdata/mpiom/rcp850001_mpiom_data_3d_mm_20090101_20091231.nc

{casename} _ {data_stream} _ {date_string}.nc
mpiom is our pop
all cases have biogeochemistry

However, all variables from a datastream are in the same file. Therefore concatinating these takes ages. So far we mostly use cdo for postprocessesing.
I adapted config.yaml and created a new mpi-ge collection. tried to create a PR to compare but doesnt work.
remote: Permission to NCAR/intake-esm.git denied to aaronspring.
fatal: unable to access 'https://github.com/NCAR/intake-esm/': The requested URL returned error: 403

tried to create a PR to compare

You probably want to fork the repo to your account, push changes there and create the PR from that branch.

@aaronspring, yes please create a fork and submit a PR from there.
I think further changes in the code will be required; we can help with that.
For testing and development purposes, it might be helpful to create a file list. Perhaps as follows.
find MPI-GE/rcp85/rcp850001/outdata -name "*.nc" > mpi_filelist.txt

We can build collections from such files, removing the need to actually be on the machine of interest to test and develop (an example).
cc @andersy005


do create only python projects?

Is it supported only python as new project language, or others instead?

enh

With my question, I mean if this gpt-engineer project can generate projects that are not only in Python but also in other languages. I didn't mean if this specific gpt-engineer project will be done in other languages.

With my question, I mean if this gpt-engineer project can generate projects that are not only in Python but also in other languages. I didn't mean if this specific gpt-engineer project will be done in other languages.

It's using the GPT API so you could do probably any language, the program itself is written in python but that's it

Thanks for the clarification. Congrats for the project, it's very useful.
Daniele Mariotto
Il mer 14 giu 2023, 4:47 PM Yevgeny Radchenko @.***> ha
scritto:

With my question, I mean if this gpt-engineer project can generate
projects that are not only in Python but also in other languages. I didn't
mean if this specific gpt-engineer project will be done in other languages.
It's using the GPT API so you could do probably any language, the program
itself is written in python but that's it
—
Reply to this email directly, view it on GitHub
https://github.com/AntonOsika/gpt-engineer/issues/26#issuecomment-1591377058,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADGNDQSXMW2XZ7DTPTT6DADXLHFI5ANCNFSM6AAAAAAZGFBMZA
.
You are receiving this because you authored the thread.Message ID:
@.***>


error

when dump give me error
[CRITICAL] error: <urlopen error EOF occurred in violation of protocol (_ssl.c:992)>

need more info, this seems an exception being handled and not a error. running again should fix the issue if not comment here again.


cannot import name '_sentencepiece' from partially initialized module 'sentencepiece' (most likely due to a circular import)

Hi, I'm encountering the following error while following the usage of the command. Could anyone kindly help me resolve this issue? I would really appreciate any assistance. Thank you!
Below are the details of my OS and environment that might be related to the problem:
window 11
python 3.10.14
torch                       2.1.2+cu121
transformers                4.38.2
sentencepiece               0.2.0
Traceback (most recent call last):
File "C:\Users\user\miniconda3\envs\whisper_diart\lib\site-packages\transformers\utils\import_utils.py", line 1390, in get_module
return importlib.import_module("." + module_name, self.name)
File "C:\Users\user\miniconda3\envs\whisper_diart\lib\importlib_init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1050, in _gcd_import
File "", line 1027, in _find_and_load
File "", line 992, in _find_and_load_unlocked
File "", line 241, in _call_with_frames_removed
File "", line 1050, in _gcd_import
File "", line 1027, in _find_and_load
File "", line 1006, in find_and_load_unlocked
File "", line 688, in load_unlocked
File "", line 883, in exec_module
File "", line 241, in call_with_frames_removed
File "C:\Users\user\miniconda3\envs\whisper_diart\lib\site-packages\transformers\models_init.py", line 15, in
from . import (
File "C:\Users\user\miniconda3\envs\whisper_diart\lib\site-packages\transformers\models\mt5_init.py", line 29, in
from ..t5.tokenization_t5 import T5Tokenizer
File "C:\Users\user\miniconda3\envs\whisper_diart\lib\site-packages\transformers\models\t5\tokenization_t5.py", line 24, in
import sentencepiece as spm
File "C:\Users\user\miniconda3\envs\whisper_diart\lib\site-packages\sentencepiece_init.py", line 10, in
from . import _sentencepiece
ImportError: cannot import name 'sentencepiece' from partially initialized module 'sentencepiece' (most likely due to a circular import) (C:\Users\user\miniconda3\envs\whisper_diart\lib\site-packages\sentencepiece_init.py)
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "C:\Share\Users\kevin\whisper-diarization-main\diarize.py", line 8, in
from ctc_forced_aligner import (
File "C:\Users\user\miniconda3\envs\whisper_diart\lib\site-packages\ctc_forced_aligner_init_.py", line 2, in
from .alignment_utils import (
File "C:\Users\user\miniconda3\envs\whisper_diart\lib\site-packages\ctc_forced_aligner\alignment_utils.py", line 9, in
from transformers import AutoModelForCTC, AutoTokenizer
File "", line 1075, in _handle_fromlist
File "C:\Users\user\miniconda3\envs\whisper_diart\lib\site-packages\transformers\utils\import_utils.py", line 1380, in getattr
module = self._get_module(self._class_to_module[name])
File "C:\Users\user\miniconda3\envs\whisper_diart\lib\site-packages\transformers\utils\import_utils.py", line 1392, in _get_module
raise RuntimeError(
RuntimeError: Failed to import transformers.models.auto because of the following error (look up to see its traceback):
cannot import name 'sentencepiece' from partially initialized module 'sentencepiece' (most likely due to a circular import) (C:\Users\user\miniconda3\envs\whisper_diart\lib\site-packages\sentencepiece_init.py)

I reinstalled the environment, following the steps mentioned by [kirahman2] below, but I am still encountering the same FileNotFoundError as mentioned in this issue.
conda create -n whisper_diar2 python=3.10 --yes
conda activate whisper_diar2
pip install --upgrade pip
conda install -c conda-forge jupyterlab --yes
conda install ipykernel --yes
python -m ipykernel install --user --name whisper_diar2 --display-name "whisper_diar2"
pip uninstall torch --yes
conda install pytorch torchvision torchaudio pytorch-cuda=12.1 -c pytorch -c nvidia --yes
pip install cython
winget install ffmpeg
pip install -r requirements.txt
https://github.com/MahmoudAshraf97/whisper-diarization/issues/188#issuecomment-2277862623
pip install --upgrade Cython setuptools wheel
https://github.com/MahmoudAshraf97/whisper-diarization/issues/202
pip uninstall huggingface_hub --yes
pip install huggingface-hub==0.20.3

if you are using windows, make sure your program is running with sufficient permissions, I couldn't reproduce the error at #212 using windows 11 so I can't say for sure what is the solution


feat(shim): support archive layers

Closes https://github.com/spinkube/containerd-shim-spin/issues/28
Depends on https://github.com/fermyon/spin/pull/2523
Draft for now as 2523 is based on Spin's main branch which has since bumped wasmtime to 21.0.1.  Can either wait for that same bump to occur in this project (presumably in tandem with pinning to the next Spin release with said version) or can explore backporting 2523 to the v2.5 branch and cutting a patch release with it.
TODO

[x] Depends on PR(s) bumping spin + co repos to next release tag and wasmtime to 21.0.1 in Cargo.toml.  Rebase after this lands.

[x] https://github.com/spinkube/containerd-shim-spin/pull/144

[x] decide on merge approach (avoid wasmtime/spin crates bump or wait for them to land)

I think waiting for the upcoming 2.6 Spin release and bumping wasmtime to suit will be fine; no big hurry to get this in.

[x] add test app/test case

In https://github.com/spinkube/containerd-shim-spin/pull/121/commits/3f3d2919604b7424545b24d1bf9dbb6bf352ff94 I added a new spin-static-assets app and incorporated into the integration tests, to cover the archive layer functionality.  I haven't had success running the integration tests locally, so leaning on CI here to check.  I'm also happy to relegate this commit (or general approach if adjustments are requested) as a follow-up instead.

Thanks, all.  Please merge when ready.


‘Report issue’ button on ‘Running extensions’ causes tens of duplicate issues

VSCode Version: 1.31.1
OS Version: macOS 10.14.3

Steps to Reproduce

Open Running extensions

Press Report Issue. You are now just one step away from creating an army of these: https://github.com/prettier/prettier-vscode/issues?utf8=✓&q=is%3Aissue+Extension+causes+high+cpu+load.

Expected behaviour
I'd like VSCode to search for Extension causes high cpu load in the repo and show Report issue button only if no open issues are found. Otherwise, I'd like it to take me to the exiting issue so that I could +1 it or add a comment. I remember Atom following this logic when there is an unhandled exception coming out of an extension.

I would close any of these which don't have a profile attached, but personally I find it annoying when I am investigating one issue involving high cpu, and many other people with different issues pile on just because their issue also involves high cpu.

Yes, this is very good feedback. I have also thought about it and I feel bad for extensions that get spammed this these issues. Totally +1 for closing issues without profiles and also +1 for making performance fixes but by now I think that we should collect perf issues and file them

https://github.com/Microsoft/vscode/issues/71290 and https://github.com/Microsoft/vscode/issues/71381 are both issues with extensions we ship. Ironically we don't encourage users to file issues for them... I would love to be able to simple collect those profile on our behalf but I am unsure if telemetry is fit for that

We should look at total_count of this https://api.github.com/search/issues?q=is:issue+in:title+repo:prettier/prettier-vscode+state:open+Extension+causes+high+cpu+load

I thought one could get the same result via the GitHub's GraphQL endpoint and thus avoid massive overfetching, but unfortunately the only works for authenticated users 😢
https://api.github.com/graphql?query={search(type: ISSUE, query: "repo:prettier/prettier-vscode state:open in:title Extension+causes+high+cpu+load"){issueCount}}
Query
{
  search(type: ISSUE, query: "repo:prettier/prettier-vscode state:open in:title Extension+causes+high+cpu+load") {
    issueCount
  }
}

Result
{
  "data": {
    "search": {
      "issueCount": 29
    }
  }
}

You can try this out in the Github GraphQL API explorer. Shame that the authentication requirement renders use of GraphQL inapplicable in this case, but it might be useful elsewhere 🤷♂

We now show existing issues when some exist. We also show a modal message to remind folks to upload cpu-profiles

Many thanks @jrieken! Hope this reduces the number of duplicate issues created in repos like prettier-vscode :)

Still quite some new issues in the last two weeks. At the time of writing, six of them are from VSCode version 1.33.

This change is for 1.34 and active in 1.34-insider, not yet in stable

I've been getting many more of these (again without profiles, and no response when I ask for them), so good to see improvements! (I'm pretty sure they're all the same thing, the module loading due to no bundling).
FWIW - I think the message is a little misleading, since if the issue is module loading (which I suspect is most), there's no high CPU, it's just blocked sync file access which trips the 3s timer.


support for scala 2.12.8 is needed

i whanted to adopt your library for my codebase but i need 2.12 scala(

Supported Scala 2.12.10 since 0.2.0 version.


Fat snapsha support

Off to the side, do the post-processing of sha-based image creation - this is triggered by the pipeline's quay-retag branch .travis.yml, and it in turn triggers the release repo by dropping the fat manifest there (implemented in release's .travis.yml).

Goodness knows I'm scared!


Add cycle amount property

This will allow it to shift more than once per cycle

Done! Set it using MarqueeTitle.shifts = shiftAmount


Prisma generate get's stuck after adding nexusPrisma generator block to prisma schema

Screenshot
.
Description:
I have a blitz.js project and I am using the default graphql-apollo-server recipe and after running blitz prisma generate it gets stuck after printing:
Environment variables loaded from .env
Prisma schema loaded from db/schema.prisma

This also happens if I manually install nexus-prisma and add the generator block to my schema.
I am using nexus-prisma version: 0.35.0

Posted a solution here: https://github.com/prisma/nexus-prisma/issues/186#issuecomment-1233163091


Issue #567: GDS renderer missing precision argument in boolean subtra…

…ction of interior polygon boundaries

What are the issues this pull addresses (issue numbers / links)?
Did you add tests to cover your changes (yes/no)?
Did you update the documentation accordingly (yes/no)?
Did you read the CONTRIBUTING document (yes/no)?
Summary
Details and comments

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

Good catch.  There is same bug in different file.  I made an new issue (#569) and will fix that one independently of this pull request.

@JeremyDrysdale-ibm  or @marcolincs , since there are two tests in CI that are required, could you please merge this pr?


(io.js) npm install failed on iojs

Hi there,
I couldn't get it installed in iojs 1.6.2 but it run flawlessly in node 0.12.1
Does it have something to do with v8?
../src/fibers.cc:140:3: error: ‘SetResourceConstraints’ is not a member of ‘v8’
v8::SetResourceConstraints(isolate, constraints);
^
../src/fibers.cc:140:3: note: suggested alternative:
../src/fibers.cc:139:7: note:   ‘uni::SetResourceConstraints’
void SetResourceConstraints(Isolate* isolate, ResourceConstraints* constraints) {
^
make: *** [Release/obj.target/fibers/src/fibers.o] Error 1
make: Leaving directory /home/myhome/react-isomorphic-starterkit/node_modules/fibers/build' gyp ERR! build error  gyp ERR! stack Error: make` failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/home/myhome/iojs-v1.6.2-linux-x64/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:267:23)
gyp ERR! stack     at emitTwo (events.js:87:13)
gyp ERR! stack     at ChildProcess.emit (events.js:169:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (child_process.js:1044:12)
gyp ERR! System Linux 3.16.0-33-generic
gyp ERR! command "/home/myhome/iojs-v1.6.2-linux-x64/bin/iojs" "/home/myhome/iojs-v1.6.2-linux-x64/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild" "--release"
gyp ERR! cwd /home/myhome/react-isomorphic-starterkit/node_modules/fibers
gyp ERR! node -v v1.6.2
gyp ERR! node-gyp -v v1.0.2
gyp ERR! not ok
Build failed

Thanks for reporting this. I think this node-fibers issue is related: https://github.com/laverdet/node-fibers/issues/203
For now I suggest only using the starterkit with node. I'm also researching the Relay-approach for fetching data with higher-order components: https://github.com/RickWong/react-transmit It's a cleaner solution so the starterkit wouldn't need the react-async and fibers dependencies anymore.

OK, react-async and its fibers dependency are gone. The starterkit now uses React Transmit for data fetching and isomorphism.
I've tested io.js support again. It worked like a charm.


Add pip Installation for fastapi and uvicorn

Package name:

fastapi-pip python3-fastapi
uvicorn-pip python3-uvicorn

Package Upstream Source:
https://pypi.org/project/fastapi/
https://pypi.org/project/uvicorn/
Purpose of using this:
To run a restful api server in python, with uvicorn fast ASGI server implementation. Although uvicorn is avail via debian, will prefer api server is running as near "latest", hence using pip. If we should follow the convention, I can make the change to python3-uvicorn.
Also, not sure for fastapi, should we can mix the installation from various sources (e.g. debian pip, ubuntu : python-* ....)
Links to Distribution Packages
Fastapi

Debian: NOT AVAILABLE https://packages.debian.org/bullseye/python3-fastapi
Ubuntu: https://packages.ubuntu.com/impish/python3-fastapi   (Not avail in focal)
Fedora: https://src.fedoraproject.org/rpms/python-fastapi
Arch: https://archlinux.org/packages/community/any/python-fastapi/

Uvicorn

Debian: https://packages.debian.org/bullseye/python3-uvicorn
Ubuntu: https://packages.ubuntu.com/focal/python3-uvicorn
Fedora: https://src.fedoraproject.org/rpms/python-uvicorn
Arch: https://archlinux.org/packages/community/any/uvicorn/

Thanks for reply. I have updated it accordingly.
I noticed that the python3-fastapi is only available in Ubuntu 21 Impish and above. What's the convention if a user is using ubuntu 20.04 focal, and decides to use rosdep to download this package?

I noticed that the python3-fastapi is only available in Ubuntu 21 Impish and above. What's the convention if a user is using ubuntu 20.04 focal, and decides to use rosdep to download this package?

Good question.  The more usual way (which will allow CI to pass here) is to set the key for the distributions that are still supported, but don't have the package to null, which the tools will ignore.  So in the case of fastapi, this would look something like:
python3-fastapi:
  arch: [python-fastapi]
  debian:
    '*': [python3-fastapi]
    buster: null
  fedora: [python-fastapi]
  ubuntu:
    '*': [python3-fastapi]
    bionic: null
    focal: null

It's also possible to use a pip key for the distributions that don't currently support it.  That said, we still can't release packages onto the buildfarm that depend on that key; we only support system packages on the buildfarm.  If you'd still like to have the pip keys for local, source use, you can do something like:
python3-fastapi:
  arch: [python-fastapi]
  debian:
    '*': [python3-fastapi]
    buster:
      pip:
        packages: [fastapi]
  fedora: [python-fastapi]
  ubuntu:
    '*': [python3-fastapi]
    bionic:
      pip:
        packages: [fastapi]
    focal:
      pip:
        packages: [fastapi]

Does that make sense?

@youliangtan friendly ping

Thanks @clalancette for the suggestion. I think supporting other distributions will be handy for our application. I have implemented the second method for python3-fastapi and python3-uvicorn.


Updating tremolo throws depth and frequency is read-only

Describe the bug
Updating the tremolo effect throws:

Uncaught TypeError: "depth" is read-only

and

Uncaught TypeError: "frequency" is read-only

To Reproduce
I have something like this:
// manage tone.js
const effects = {
    chorus: undefined,
    distortion: undefined,
    reverb: undefined,
    tremolo: undefined,
}

const preload = () => {
    effects.reverb = new Tone.Reverb({
      wet: 1,
    });
    effects.tremolo = new Tone.Tremolo({
      wet: 1,
    });
    effects.chorus = new Tone.Chorus({
      frequency: 2.5,
      delayTime: 0.5,
      depth: 1,
      feedback: 0.3,
      wet: 1,
    });
    effects.distortion = new Tone.Distortion({
      distortion: 1,
      wet: 1,
    });
    // ...
}

const selectedEffects = ['chorus', 'tremolo'] // to respect the order to chain the effects

export const updateEffects = (effectType, effectKey, newVal) => {
  const effect = effects[effectType]; // get effect to be updated
  effect[effectKey] = newVal; // update the value
};

// user interface
const updateReverbDecay = (val) => {
  updateEffects('reverb', 'decay', val); // works as fine
};

const updateTremoloDepth = (val) => {
  updateEffects('tremolo', 'depth', val); // FIXME: depth is read-only
};

Expected behavior
Tremolo depth will accept a value that is between [0, 1]
Tremolo frequency will accept a value
What I've tried
Updating other effects with the same method like: chorus and delayTime, chorus and depth, distortion and distortion, reverb and decay works as expected.

solved.
Had to add .value notation to set the depth or feedback value to the tremolo effect.


Svg is not loading in production

Hi,
SVG is not loading in production somehow. Development is fine. Here is what I see in Network log:
/assets/swipebox-icons-54ac5cc2e33d7110ec5fee2d050f46e8.png – 200
/assets/swipebox-icons-54ac5cc2e33d7110ec5fee2d050f46e8.svg – 404

When I compile assets manually the hash is different:
I, [2015-03-06T10:02:43.800675 #39826]  INFO -- : Writing
/[...]/assets/swipebox-icons-54ac5cc2e33d7110ec5fee2d050f46e8.png
I, [2015-03-06T10:02:43.801717 #39826]  INFO -- : Writing
/[...]/assetsswipebox-icons-2a000235eb3b307f2aa9f596c043facd.svg

But it tries to load the wrong one:

Please advise. Thanks.

I see why, I had to patch swipebox and this is what's causing it:
          bg = bg.replace( 'png', 'svg' );

You're clearly not using my gem :confused:
I take care with this with https://github.com/mrfoto/swipebox/blob/master/vendor/assets/javascripts/swipebox.js#L167-L169

I do use your gem, but had to patch the original swipebox.js to get rid of duplicates https://github.com/brutaldesign/swipebox/issues/210
Thought your css would override the original styling. But no worries, I just disabled svg for now.

Why? My gem includes the patched js :confused:

I have links to images repeated a few times on the page. So I had to prevent duplicates in gallery.


Update CollectionsSection.tsx

Add an id to the collections section to make it easier to link to (i.e., https://researchequals.com/#collections).

This doesn't work as of right now because of the way the page loads. Merging for completeness anyway.


Add docs for Management and RBAC Batch API.

Currently, we added batch API to some languages already:
Go: https://github.com/casbin/casbin/issues/386
Node.js: https://github.com/casbin/node-casbin/issues/105
But we don't have docs. Can anyone work on a new "Batch API" page? @nodece @GopherJ @DivyPatel9881 @kingiw @hackerchai

I will work on this.


[Bug]: When the current account does not have the task, an error message will appear when entering through the sharing link

ODC version
odc423
OB version

What happened?

What did you expect to happen?
Jump to login page
How can we reproduce it (as minimally and precisely as possible)?

Copy a task
Open a link in an account without changing tasks

Anything else we need to know?
No response
Cloud
No response

Fixed In Commit c1fede16e3ffc7523a89325c550087a46ad09534
------
xiaokang
Wed Dec 20 11:21:05 2023 +0800

pass


Make petclinic working in functional mode

Including with Spring MVC annotation infra if possible. cc @aclement @dsyer

@dsyer Should we close that one?

Your call. It’s not working with @MVC yet.

Does not make sense anymore due to #347, this issue is superseded by #386.

Does not make sense anymore due to #347, this issue is superseded by #386.


Reduce readings of training data files during training

Rasa version: 2.0.0rc3
As mentioned in the forum, Rasa Open Source reads training data file and Domain more than once during training, whilst the files don't change.
Even though that doesn't slow down things, it produces confusing logs and not optimal usage of resources anyway.
We should reduce the reading of the training data files.

fixed by https://github.com/RasaHQ/rasa/pull/6938


Recent Book

It will be nice if the API can call Recently added. The latest new and updated eBooks.
Thanks in advance & for this great project.

Hello, I don't know how to get recently updated books (except by downloading the full contents of every book everyday and comparing each to the last downloaded version, which I don't want to do, because it takes a lot of time and space).
However, I could add an option to sort by descending ID. Since newer books seem to get larger IDs, I would expect that to show the most recently added ones first. Would that help?

Yeah, I think that is a good idea!
Thanks!

OK, I added a sort parameter. /books/?sort=descending should do it now. Let me know if you have any issues.

Thanks for adding the new parameter!, but I noticed something, I do not know if to open a new issue or not.
Looks like the new parameter /books/?sort=descending  is not getting the book covers and download count.
Example: https://www.gutenberg.org/ebooks/67200 in the API those not get the image
Thanks!

Yes, unfortunately, this is a limitation of the approach that the Project Gutenberg website asks people to take: its downloadable catalog data, which Gutendex uses, seems to be missing some cover images and download counts that appear on Project Gutenberg.
However, for cover images, you might have success with this URL format:
https://www.gutenberg.org/cache/epub/<id>/pg<id>.cover.<small|medium|large>.jpg

Example: https://www.gutenberg.org/cache/epub/67200/pg67200.cover.medium.jpg

Thanks!


Add to bug report template.

This might be more appropriate for website projects than the design system but I think formal reports would be helpful. Devs, what do you think of using whatsmybrowser.org? I've found it helpful in the past.

@chachasikes Yeah try it! Here's my result whatsmybrowser.org/b/SHRBW

Maybe I should make the instructions in the readme a little clearer?


Support for validating signed cookies

http://expressjs.com/en/4x/api.html#req.signedCookies

Implemented by https://github.com/Hilzu/express-openapi-validate/commit/cca5e67212a2f884e2fce711f4887e01a231df72


feat: CS-809 - UI component - Microsite Logo

Storybook entry with consistant docs

:tada: This PR is included in version 1.0.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Convert strongly typed object to document

It' from the doc https://github.com/yojimbo87/dictator#convert-document-to-strongly-typed-object
When this code is used :
var dummy = new Dummy();
dummy.Foo = "string value";
dummy.Bar = 12345;
var document = Dictator.ToDocument(dummy);
but on "var dummy = new Dummy();",
I get CS0246  C# The type or namespace name could not be found (are you missing a using directive or an assembly reference?)
Even if I declared "using Arango.Client;"
I tried to download dictator fro github, reference it into my project and using it but it conflicts with Arango.Client.
Any idea ?

Hi, Dummy is just sample POCO with two properties (Foo as string and Bar as integer value). It serves as an example of strongly typed object conversion.

Yes, i realized and use my poco just after
As a beginner in c# and arango, there is a delay to understand... :)
Thank you for taking time to answer


Unbiased uri utils

Another step towards https://github.com/microsoft/vscode/issues/93368

@bpasero with this change a few of these tests fail: https://github.com/microsoft/vscode/blob/f7dde7f949eec423ca0b0326338f6334a484ca16/src/vs/platform/backup/test/electron-main/backupMainService.test.ts#L694
I am not sure if the test is to blame of if the backup main service should use a different compare logic (which it shouldn't because it trust documents from the renderer, right?)

I believe we could have the IURIIdentityService also in electron-main, quite sure the IFileService is also there which could tell which IExtUri to use.

Really? How does the synchronisation of file system providers etc work across processes? I'd rather opt for a local change in your service then. Basically copy (or keep) the case-ignore logic as of today use that in the service then.

@jrieken on electron-main there is only the local disk based file system provider where we currently hardcode that mac & windows are case-insensitive. I was not suggesting that electron-main has the same file service as renderers.

Ah, understood. I think we are basically on the same page. There is ExtUri implementation which takes a function to decide if a scheme means case ignore or not. You can use that - plain or wired  into the (main side) file service.

@jrieken I pushed my changes to use a custom ExtUri for selected electron-main pieces to master and merged that change back into this branch. At least the backup main service tests should be fine then.

Thanks @bpasero. I'll see what else is going wrong here...

@jrieken Removed the usage of biased utility in fileUserDataProvider with extUri created using the provider's capabilities.
I could not use IUriIdentityService because fileUserDataProvider is created along with initial services where IUriIdentityService is not available. Hence created extUri by checking the data provider's capability, which makes it unbiased - d6d6fb2cbf563b641ff7d86b1af5fd2d5fd66e25

Thanks @sandy081


custom transport RS458 UART for portenta machine control

Description:
I create a new transport mechanism to communicate through RS485 in  Portenta machine control H7, and i run ros-agent in jetson.
below the rs485_transport.cpp :
#include <Arduino.h>
#include <Arduino_MachineControl.h>

extern "C"
{
  #include <stdio.h>
  #include <stdbool.h>
  #include <sys/time.h>

  int clock_gettime(clockid_t unused, struct timespec *tp) __attribute__ ((weak));
  bool uart_rs485_transport_open(struct uxrCustomTransport * transport) __attribute__ ((weak));
  bool uart_rs485_transport_close(struct uxrCustomTransport * transport) __attribute__ ((weak));
  size_t uart_rs485_transport_write(struct uxrCustomTransport * transport, uint8_t *buf, size_t len, uint8_t *errcode) __attribute__ ((weak));
  size_t uart_rs485_transport_read(struct uxrCustomTransport * transport, uint8_t *buf, size_t len, int timeout, uint8_t *errcode) __attribute__ ((weak));

  #define micro_rollover_useconds<PHONE_NUMBER>

  int clock_gettime(clockid_t unused, struct timespec *tp)
  {
    (void)unused;
    static uint32_t rollover = 0;
    static uint32_t last_measure = 0;

    uint32_t m = micros();
    rollover += (m < last_measure) ? 1 : 0;

    uint64_t real_us = (uint64_t) (m + rollover * micro_rollover_useconds);
    tp->tv_sec = real_us / 1000000;
    tp->tv_nsec = (real_us % 1000000) * 1000;
    last_measure = m;

    return 0;
  }

  bool uart_rs485_transport_open(struct uxrCustomTransport * transport)
  {
    // Set the PMC Communication Protocols to default config
    machinecontrol::comm_protocols.init();
    // Enable the RS485/RS232 system
    machinecontrol::comm_protocols.rs485Enable(true);
    // Specify baudrate, and preamble and postamble times for RS485 communication
    machinecontrol::comm_protocols.rs485.begin(115200, 0, 500);

    machinecontrol::comm_protocols.rs485.receive();
    return true;
  }

  bool uart_rs485_transport_close(struct uxrCustomTransport * transport)
  {
    machinecontrol::comm_protocols.rs485.end();
    return true;
  }

  size_t uart_rs485_transport_write(struct uxrCustomTransport * transport, uint8_t *buf, size_t len, uint8_t *errcode)
  {
    (void)errcode;
    machinecontrol::comm_protocols.rs485.noReceive();
    machinecontrol::comm_protocols.rs485.beginTransmission();
    size_t sent =  machinecontrol::comm_protocols.rs485.write(buf, len);
    machinecontrol::comm_protocols.rs485.endTransmission();
    return sent;
  }

  size_t uart_rs485_transport_read(struct uxrCustomTransport * transport, uint8_t *buf, size_t len, int timeout, uint8_t *errcode)
  {

    (void)errcode;
    machinecontrol::comm_protocols.rs485.receive();
    machinecontrol::comm_protocols.rs485.setTimeout(timeout);
    if (machinecontrol::comm_protocols.rs485.available())
      return machinecontrol::comm_protocols.rs485.readBytes((char *)buf, len);
    else
      return 0;

  }
}

// ---- Build fixes -----

// TODO: This should be fixed
#if defined(ARDUINO_TEENSY31) || defined(ARDUINO_TEENSY32) || defined(ARDUINO_TEENSY35) || defined(ARDUINO_TEENSY36)

extern "C" void _write(){
}

#endif
// ----------------------

Version :
i use foxy version of micro-ROS

Steps to reproduce the issue
when i run ros2 run micro_ros_agent micro_ros_agent serial --dev /dev/ttyTHS0 -b 115200 -v6, node doesn't display

Thanks !

Check your uart_rs485_transport_read method, it looks like the board is not getting the response message.
You can see on the agent how board is sending a session establish message periodically, as its not getting the confirmation.

thank you for your response,
yes i check it, but now when i run the agent nothing showing up in log !:

How are you using rmw_uros_set_custom_transport?
Check that the first argument is set to true to enable transport framing, as in the default method:
static inline void set_microros_transports(){
	rmw_uros_set_custom_transport(
		true,
		NULL,
		arduino_transport_open,
		arduino_transport_close,
		arduino_transport_write,
		arduino_transport_read
	);
}

More info about this here.
Apart from that detail, I am afraid we can't help here without more info. Try to debug your custom transport methods and check that they are actually sending/receiving the expected payloads.

I test it, but it doesn't work,
I think the problem related by the encoding and decoding, because when i tried a sample code without uROS, just send and receive and some problem for decoding appeared !
I modified the microros library to support native Ethernet transport in portenta machine control, and it works now
I already created a pull request for that
you can find it in #852
Thanks

Okey! I am closing for now, reopen if you encounter problems on uROS side.

I think the problem related with encoding and decoding,


Can not clear auth token for one extension if not sign out my account completely

Issue Type: Bug
I have mutiple extensions using github account: Settings Sync, GHPRI and GitHub Issue Notebook. I often get Bad credential error with GitHub Issue Notebook. I guess it's because of SSO so I need to reauthenticate.

Manage Trusted Extension
Uncheck GitHub Issue Notebook
Run query
Allow access to the GitHub account
It's using the old token and still leads to the same error

The only way to workaround this is sign out my GitHub account completely. However that means I need to three times of authentication (dialog, and then open a page in browser, come back. repeat three times), which is also bad.
IIRC, extensions under the same github account are using different tokens, but currently I didn't see a way to clear the token.
VS Code version: Code - Insiders 1.51.0-insider (2ed16d0ce8713a5dfed9729071141be0ccf7fbec, 2020-10-14T05:48:54.391Z)
OS version: Darwin x64 19.5.0

It's much worse. I signed out of my GitHub account, reload the window, open a GitHub issue notebook file, and then it shows "allow the extension to GitHub account" three times, even if every time I say allow, and finish the authentication in the browser.

I'm sorry this was such a bad experience!
I think we should delete credentials that return the Bad credentials response so that it's not necessary to sign out completely and remove all tokens. This still creates an annoying experience of having to sign in again, but at least it would only be for one thing. I'm not sure yet how to avoid getting into the Bad credentials state, I think this is because the number of tokens GitHub will issue to an OAuth app is limited, and it will start to revoke older ones when you request new ones.

What is Manage Trusted Extension here? I don't see a command with that name

@roblourens

To start, I don't have any Bad Credential Error. But, if I follow the steps, this is how it looks now:

Click on Manage Trusted Extensions
Uncheck GitHub Issue Notebook
Run query in a notebook
Allow access to the GitHub account
I'm not redirected to the browser, instead it just signs me in and the query runs.

Is this the new expected behaviour? @rebornix @RMacfarlane

This is expected behavior, yes, but isn't related to this fix.
Steps to verify: (these are assuming mac, I'm not sure if all of these operations are possible using Windows Credential Manager or gnome keyring)

Open Keychain Access and click the '+' at the top to create a new password, enter vscode-insiders-github.login as item name, account as account name, and [{"id":"738dcd4d-3a51-4d79-9eab-04cfc9e7531d","accessToken”:”garbage”,”account":{"label":"RMacfarlane","id":3672607},"scopes":["read:user","repo","user:email"]}] as the value
Delete vscode-insidersvscode.github-authentication from the keychain if you have it
(The reason for these steps is that the key for auth was migrated this iteration. The new value has additional encryption, so to be able to test bad data in the keychain, we need to use the old key value and let VS Code migrate it over to the new one)
Open VS Code. You'll be prompted that VS Code wants access to the keychain for this value, since VS Code was not the process that created it
Check that the "GitHub Authentication" output channel has a message that says "Getting account info failed: Unauthorized" and that you are not signed into the GitHub Pull Requests and Issues extension, meaning that the bad credential was removed


Developers / Projects / Achievements section styles refined with Flexbox

Resolves #563 - Bad Spacing on Homepage
The Developers / Projects / Achievements section of the homepage has been styled in such a way that all the elements align correctly regardless of height differences.

Sweet thanks!


Exposing environment variables to allow optional configuration of log retention

New request to provide squashed commit as requested in PR #8.
This closes #7

Thanks!


Fix IndexOutOfBoundsException when FixedCompositeByteBuf is construct…

…ed with an empty array.
Motivation:
When FixedCompositeByteBuf was constructed with new ByteBuf[0] and IndexOutOfboundsException was thrown.
Modifications:
Fix constructor
Result:
No more exception

@trustin PTAL

LGTM

Cherry-picked into 4.1(7d3ca7f) and 4.0(e3aced6)


Auto Create Vector Store Resources

[x] I have read and agree to the contributing guidelines.

Is your feature request related to a problem? Please describe.
When using Vector Store Drivers, users need to manually create the appropriate resources (indexes/collections/etc) before using the Driver. This can be time consuming and error prone.
Describe the solution you'd like
Griptape should have the option to automatically create these resources when first using the Vector Store Driver.
Describe alternatives you've considered
Call use the underlying SDK directly.
Additional context
https://github.com/griptape-ai/griptape/pull/1022#discussion_r1695524314

It was already implemented and removed https://github.com/griptape-ai/griptape/issues/607

Great point, @michal-repo! I had completely forgotten that we intentionally decided to remove that functionality to reduce Griptape's maintenance surface area. Griptape's focus should remain solely on data-plane operations.


items not always populated with latest results

Hi,
first of all thanks for your component. It's easy to use and works well.
Howeve, I sometimes come across an issue where the latest query completes before previous ones.
For instance, if I type "something" it will trigger the following queries:
"som"
"some"
"somet"
"someth"
"somethi"
"somethin"
"something"
Sometimes query for "some" will complete after "something" so final items are not the ones that match "something" but those of "some". Bummer.
I've implemented a solution and will submit for review.
Thanks.

Thanks!


Add proper argument parsing via argparse

Currently only env vars are possible

Fixed with https://github.com/bbusse/webdriver-login/commit/c340de24fd169a51013539d681f7d61dcfe99064


Many-to-many relation on same table

Hey there,
I'm trying to implement a many-to-many relation on the same table. Wondering how I would do that in pothos?
This is my Prisma model:
model Product {
  id            String             @id @default(dbgenerated("gen_random_uuid()")) @db.Uuid
  ...
  name          String?
  parentProducts   ProductsOnProducts[] @relation("ProductsOnProducts_parentProduct")
  childProducts    ProductsOnProducts[] @relation("ProductsOnProducts_childProduct")
}
model ProductsOnProducts {
  id              String  @id @default(dbgenerated("gen_random_uuid()")) @db.Uuid
  parentProductId String  @db.Uuid
  parentProduct   Product @relation("ProductsOnProducts_parentProduct", fields: [parentProductId], references: [id])
  childProductId  String  @db.Uuid
  childProduct    Product @relation("ProductsOnProducts_childProduct", fields: [childProductId], references: [id])
}

and my pothos schema:
builder.prismaNode('ProductsOnProducts', {
  id: { field: 'id' },
  findUnique: (id) => ({  }),
  fields: (t) => ({
    parentProduct: t.relation('parentProduct'),
    parentProductId: t.field({
      type: 'ID',
      resolve: (root) => encodeGlobalID('Product', root.parentProductId),
    }),
    childProduct: t.relation('childProduct'),
    childProductId: t.field({
      type: 'ID',
      resolve: (root) => encodeGlobalID('Product', root.childProductId),
    }),
  }),
});
builder.prismaNode('Product', {
  id: { field: 'id' },
  findUnique: (id) => ({ id }),
  fields: (t) => ({
    name: t.exposeString('name', { nullable: true }),
    childProducts: t.relation('childProducts'),
    parentProducts: t.relation('parentProducts'),
  }),
});

but right now when I try to query it looks like this:

Curious how I could change this so instead I can just do
product {
  childProducts {
    id
    name
  }
}

This is probably what you are looking for: https://pothos-graphql.dev/docs/plugins/prisma#indirect-relations-eg-join-tables
Basically, you need to use the normal t.field along with a custom select to select the data you want.   This is a little more complex than the simple t.relation pattern, but hopefully won't be too hard to get working.
Circular references can also cause some issues, so breaking things up will help
const ProductRef = builder.prismaNode('Product', {
  id: { field: 'id' },
  findUnique: (id) => ({ id }),
  fields: (t) => ({
    name: t.exposeString('name', { nullable: true }),
    parentProducts: t.relation('parentProducts'),
  }),
});

builder.prismaField(ProductRef, 'childProducts', t => t.field({
  type: [ProductRef],
  select: (args, ctx, nestedSelection) => ({
        childProducts: {
          select: {
            childProduct: nestedSelection(true)
          },
       },
  }),
  resolve: product => product.childProducts.map(child => child.childProduct),
})

Awesome! thanks for the response.
I figured it out by digging a little more:
builder.prismaObjectField(Product, 'childProducts', (t) =>
  t.field({
    select: (args, ctx, nestedSelection) => ({
      parentProducts: {
        select: {
          childProduct: nestedSelection(true),
        },
      },
    }),
    type: [Product],
    resolve: (parent) => parent.parentProducts.map((child) => child.childProduct),
  })
);

builder.prismaObjectField(Product, 'parentProducts', (t) =>
  t.field({
    select: (args, ctx, nestedSelection) => ({
      childProducts: {
        select: {
          parentProduct: nestedSelection(true),
        },
        distinct: 'id',
      },
    }),
    type: [Product],
    resolve: (parent) => parent.childProducts.map((child) => child.parentProduct),
  })
);

Now I'm trying to add a distinct but doesn't look like it's working.
I may have a childProduct multiple times with the same parent.
This is what I'm getting

sorry I meant it's not working... :) the distinct part.

ha! nvm figured it out. I need to distinct by distinct: ['parentProductId'],
Thanks so much.


Allow to specify metaWriter in loadUsing

Otherwise I have to copy and paste that function for this rather simple change

Sorry it took me so long to get around to merging this! 🙈
Hopefully you copy-pasted in the meantime and it didn't slow you down too much.
Thanks for the patch 🙌🏼

Uploaded to version <IP_ADDRESS>


need to print raw sql of beego orm into docker_mysql.log for debug reason

If you are reporting a problem, please make sure the following information are provided:
1)Version of docker engine and docker-compose
2)Config files of harbor, you can get them by packaging "Deploy/config" directory
3)Log files, you can get them by package the /var/log/harbor/
If we use orm, it is better to output orm logs into files to make it convenient to check orm errors. Hope to realize this feature.

I will handle this.

If you use raw sql you need to log it manually, if you use orm's exposed method there's a flag in orm to print the sql commands.
Meanwhile need to consider protecting sensitive data from leaked in the log.

use the following code as a solution when debug
func main() {
    orm.Debug = true
...

don't apply it to the product which may cause performance problem.
Without any problem, you may close this issue. @wemeya


Commit activity is invalid

Commit activity inside README.md is shown as invalid.

I know but it's not a problem. Those badges are broken one day and the other too but they fix themself automatically with an amount of time. Thanks anyway for your report @archetipico
I'm closing.


A few questions and some minor language/formatting issues

Interoperability

Any references to terms and JSON Schemas defined in BCP-004-01 relate to version 1.0.0.

Probably reasonable to mention the version number, but I suggest v1.0 rather than v1.0.0.

http://hostname/x-nmos/...

I suggest e.g. http://api.example.com/x-nmos/... like other parts of the NMOS specs. (example.com is reserved for examples by IANA.)

The Flow Compatibility Management API should be

There are at least two instances of lowercase "should". If they are intended to be conformance key words they SHOULD be in uppercase. :-)
I suggest some more articles in a few places, e.g.
> when set of Inputs/Outputs associated with the Sender/Receiver is changed, or when Status of Sender/Receiver is updated
---
< when the set of Inputs/Outputs associated with the Sender/Receiver is changed, or when the Status of a Sender/Receiver is updated

Data Model: Identity Mapping

Output ID
[...]
SHOULD change if:
[...]
A different upstream counterpart is connected to the same interface

If I've understood correctly, that should be downstream?
Behaviour: Server Side

Non-empty constraint_sets array MUST be treated as unordered unless at least

"Treated" by which actor? The Sender? Or a Controller?
Also, I suggest "of equal preference" rather than "unordered".

Once a Sender accepts proposed Active Constraints, this Sender, the Flow and the Connection API /transportfile resource (if used) associated with this Sender and the Source associated with this Flow MUST satisfy the Active Constraints when the Sender is active.

I'm just checking... does "satisfy the Active Constraints" mean "satisfy at least one of the Constraint Sets in the Active Constraints"? If so, why does it help for the Sender to have multiple Constraint Sets associated with it?
In the section on different states, does the relative precedence of Active Constraints Violation and No Signal need to be indicated?
I think the use of capitalized space-separated strings for enum values may be inconsistent with other NMOS specs. If a human-readable string is required that should be a separate property, such as description.
I suggest lowercase, underscore separated, for example:
"enum": {
  "unconstrained",
  "constrained",
  "active_constraints_violation",
  "no_signal",
  "awaiting_signal"
}

Behaviour: Client Side

If multiple Receivers have equal Constraint Sets, NMOS Controller MUST take only one of them. If they differ only in urn:x-nmos:cap:meta:preference, then one with the highest priority MUST be taken.

What does it mean for the Controller to "take" a constraint set?

The most of the questions/suggestions makes sense to me so I tried to apply them in #89.

Non-empty constraint_sets array MUST be treated as unordered unless at least

I checked https://specs.amwa.tv/bcp-004-01/releases/v1.0.0/APIs/schemas/with-refs/constraint_set.html and recalled that preference has default value set to zero. Do we need this paragraph at all?

I'm just checking... does "satisfy the Active Constraints" mean "satisfy at least one of the Constraint Sets in the Active Constraints"? If so, why does it help for the Sender to have multiple Constraint Sets associated with it?

Right. It helps to configure an Input and determine whether Source, Flow and transport file, which properties may change at any moment, satisfy Active Constraints.

In the section on different states, does the relative precedence of Active Constraints Violation and No Signal need to be indicated?

What do you mean? active_constraints_violation means there is signal from an Input, but it can't be streamed due to the violation.

What do you mean? active_constraints_violation means there is signal from an Input, but it can't be streamed due to the violation.

Ah, I see, OK.


SCRUM-1082 Filter name fixes

This would work fine... but you might want to put the word "Filter" on the end of each of those as thats the "name" of the filter thats getting sent to the backend. The only requirement is that they are unique so they can be named whatever.

@oblodgett - "Filter" suffixes added


message format in user configuration is undocumented

See original issue on GitLab
In GitLab by [Gitlab user @tristanvb] on Jul 25, 2018, 13:32
Summary
While inspecting #509, we noticed that the message-format configuration is not properly documented.
We need to explain how this works in the user configuration docs, and ensure that we've listed and explained all of the possible identifiers which are valid for the message format.
The identifiers can be gleaned from the source currently at: https://gitlab.com/BuildStream/buildstream/blob/master/buildstream/_frontend/widget.py#L328

In GitLab by [Gitlab user @cs-shadow] on Aug 3, 2020, 22:51
mentioned in commit aaab87da53f4e08c00122c20201b39f7c68309ba

In GitLab by [Gitlab user @cs-shadow] on Aug 3, 2020, 22:52
mentioned in merge request !2013

In GitLab by [Gitlab user @cs-shadow] on Aug 4, 2020, 11:35
mentioned in commit c7f9214c8e097f27fffa38b4b7c0b0042c9d9034

In GitLab by [Gitlab user @marge-bot123] on Aug 4, 2020, 13:14
mentioned in commit 06f406473e67a92ce5b2c8505d6f73fddb1f2726


Proposal to improve Purview documentation about using REST-api

There is a wrong (or at least very confusing) comment  in the documentation. See section "Set up authentication using service principal", step 4 " Select the root collection...". So it states that you need to select the root collection. I tested whether it is possible to select a lower level collection. This test was successful. So in my opinion step 4 shoud be: "4. Select teh root collection or one of its subcollections in the collections menu. Teh root collection will be teh top collection in the list, and will have the same name as your Azure Purview account".
Note: it is important to be able to set up rolses at teh right collection level, so taht also via API security can be managed via collections.

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

ID: 6c30758f-e4a0-9673-b531-5447c0cfee0e
Version Independent ID: 935b5097-6ca8-4a3d-5fca-4b4ac4c27a41
Content: How to use REST APIs for Azure Purview Data Planes - Azure Purview
Content Source: articles/purview/tutorial-using-rest-apis.md
Service: purview
Sub-service: purview-data-catalog
GitHub Login: @nayenama
Microsoft Alias: nayenama

@HenriHeijltjes - Thank you for the feedback! I see where you are coming from.
This particular article is a Tutorial, which typically shows a set of steps to perform a single, specific operation, rather than showing all possibilities.
That said, I understand how this language is limiting/confusing here, and I will update it today. Thank you!

@HenriHeijltjes  - I have spoken with the developers to confirm behavior and will be updating the documentation to include this note:
You can also assign your service principal permission to any sub-collections, instead of the root collection. However, all APIs will be scoped to that collection (and sub-collections that inherit permissions), and users trying to call the API for another collection will encounter errors.
Thank you for your feedback! These updates will be available by the end of the day.

#please-close


Convenience methods

Is it possible to include convenience APIs from dart:html? For example, the web template starts with this code:
import 'dart:html';

void main() {
  querySelector('#output')?.text = 'Your Dart app is running.';
}

The package:web version looks like this:
import 'dart:js_interop';
import 'package:web/web.dart';

void main() {
  document
      .querySelector('#output'.toJS)
      ?.setHTML('Your Dart app is running.'.toJS);
}

Here, it would be convenient if querySelector was a top-level method like before, elements had a text setter, and you didn't need to call toJS on strings.

FYI - the latest version now accepts directly all primitives, so no need for .toJS on numbers, booleans, and Strings.

FYI - the latest version now accepts directly all primitives, so no need for .toJS on numbers, booleans, and Strings.

Is this true now, seems it's not?
import 'dart:js_interop';

import 'package:web/web.dart';

Future<void> main() async {
  final response = await window.fetch('blah.json').toDart;
  print(response.ok);
}

$ dart analyze
Analyzing web_test...                  0.5s

  error • web/main.dart:6:39 • The argument type 'String' can't be assigned to the parameter type 'JSAny'.  • argument_type_not_assignable


[FLINK-15018][docs] Add event time page to Concepts

What is the purpose of the change
This PR separates the conceptual overview of event time from the DataStream documentation so that it can be used by DataStream and SQL users. This has the added benefit of making the page more stable as it will not have to be rewritten or touched every time an API is changed.
Brief changelog

Add event time page to concepts and glossary
Update links through-out the docs

@fhueske & @alpinegizmo would either of you like to take a look at this?


Is there any way to have LinkedIn post to Twitter via linkedin-cli?

Hi! Thanks for a great useful tool.
When I go to LinkedIn in the Web browser, it lets me post simultaneously to LinkedIn and Twitter.
Is there any way to do that using linkedin-cli?

Linkedin api has a feature for it called distribution but it's not supported with linkedin-cli yet.
https://docs.microsoft.com/en-us/linkedin/marketing/integrations/community-management/shares/ugc-post-api?tabs=http#distribution

Understood, thank you @maaydin !


[libpqxx] fix linking errors by enabling c++20

people suffer from this upstream issue https://github.com/jtv/libpqxx/issues/732
while upstream issue is non-windows, we're able to reproduce this. on x64-windows-static as well

adding cxx standard as upstream issue comments suggest fixes the error

This is the thing that should be set in a toolchain/triplet file. Otherwise you'll get inconsistent results and also break users with C++ <= 17.

@Osyotr so this is bad? https://github.com/microsoft/vcpkg/blob/83444c57754d01d584a4a371a070c5147e8a5aae/ports/libunifex/portfile.cmake#L23
I added this line because found lots search results https://github.com/search?q=repo%3Amicrosoft%2Fvcpkg DCMAKE_CXX_STANDARD&type=code

I'd say this is https://learn.microsoft.com/en-us/vcpkg/contributing/maintainer-guide#do-not-use-features-to-control-alternatives-in-published-interfaces and https://learn.microsoft.com/en-us/vcpkg/contributing/maintainer-guide#a-feature-may-replace-polyfills-with-aliases-provided-that-replacement-is-baked-into-the-installed-tree because the option replaces API, though some CMAKE_CXX_STANDARD usages you linked were added by vcpkg maintainers.
@BillyONeal

According to the upstream discussion around https://github.com/jtv/libpqxx/issues/732#issuecomment-1727703919, the issue occurs when the package is consumed with a different C++ standard than it is built with. So this PR wouldn't fix downstream linking errors, but only move them to other users. At the same time, it would block the mitigation: Setting per-port options in custom triplet files.

@Pospelove, I'm not convinced this is the right solution for your problem. You can always fix this for yourself through your toolchain/triplet file. I'm going to close the PR for now. Feel free to repopen the PR if you feel I am mistaken or have identified the real bug/fix.

I'd say this is https://learn.microsoft.com/en-us/vcpkg/contributing/maintainer-guide#do-not-use-features-to-control-alternatives-in-published-interfaces and https://learn.microsoft.com/en-us/vcpkg/contributing/maintainer-guide#a-feature-may-replace-polyfills-with-aliases-provided-that-replacement-is-baked-into-the-installed-tree because the option replaces API, though some CMAKE_CXX_STANDARD usages you linked were added by vcpkg maintainers.

Some ports do horrible things like bake which C++ standard is used into the built bits. But in general assuming everyone in the program has the same C or C++ standard setting are wrong. Effectively, we shouldn't do this unless upstream demands a specific cxx version. The thing we really want to ban is trying to control this with a feature on the grounds that it's an alternative. But if a port needs a later version to build, it's totally normal to set that.


Add contract improvements to readme

Closes https://github.com/dapperlabs/nba-smart-contracts/issues/133

Thank you for submitting this! I should have been more clear in the issue, but I mostly meant that I wanted to make sure that the information that was already in the README was still correct after the upgrade, and that there were instructions for QuerySetData were included.
I think it is still good to mention that there was an upgrade and link to the blog post, but I don't think there needs to be details about each part of the upgrade except the for set metadata part.
Does that make sense?

@joshuahannan how about now?


Update to Minecraft 1.19

Update to Minecraft 1.19
Yarn 1.19+build.4
Fabric Loader 0.14.8
Fabric API 0.56.3+1.19
Update code to reflect API changes.

can we get a new action artifact? current one is expired


legacy ACL v1 no longer working  starting from Consul version 1.11

since the day I upgraded consul I get the following error:
Warning: Cannot retrieve ACLs: invalid return code 410 uri: /v1/acl/list?token=XXXXXXXXXXXXXXXXXX body:
Error: /Stage[main]/Geant_consul::Consul/Consul_acl[neteng_token]: Could not evaluate: Session neteng_token create: invalid return code 410 uri: /v1/acl/create?token=

And if I query consul I get the following unequivocal output message:
Endpoint /v1/acl/list for the legacy ACL system was removed in Consul 1.11.

it has to do with my settings: I'm using legacy and now I need to upgrade.

I am re-opening this same issue because it's a real one, and the reason is explained at this link:
Legacy ACL removal
Starting from versions 1.11, after migrating the tokens, I get this error:
# curl -X PUT http://<IP_ADDRESS>:8500/v1/acl/create?token=xxxxxxx-xxxxxxxx-xxxxxx
Endpoint /v1/acl/create for the legacy ACL system was removed in Consul 1.11

I ran into this issue as well. The new list url is /v1/acl/policies. The output is also different and breaks the current consul_acl provider.

I was able to get things working with the current state of this module on Consul 1.11.4. I switched from using the acl provider to using the policy provider. The policy provider and the token provider worked and have the correct urls. @maxadamo maybe this will work for you too.

Can one of you make a PR to the readme in the version matrix?

ty @jonesbrennan . Is that all that users need to do then, upgrade the puppet module >6.0.0?

The user needs to use the policies and tokens parameters instead of the acls parameters.

I should read the documentation more carefully before raising an issue! :facepalm:
This is not an issue and the change in the README will suffice.