Nested resource cannot be json-ized

Nested resources which contain only one sub resource cannot be rendered as json.
Testing with only ONE line uncommented:
$hal = new \Nocarrier\Hal();
// following options do NOT work
// $hal->setResource(
//     'foo',
//     (new \Nocarrier\Hal())
//         ->setResource('bar', new \Nocarrier\Hal())
// );
// $hal->setResource(
//     'foo',
//     (new \Nocarrier\Hal())
//         ->addResource('bar', new \Nocarrier\Hal(), false)
// );
// $hal
//     ->setResource('foo1', (new \Nocarrier\Hal())->setResource('bar', new \Nocarrier\Hal()))
//     ->setResource('foo2', new \Nocarrier\Hal());
// $hal->setResource(
//     'foo',
//     (new \Nocarrier\Hal())
//         ->setResource('bar1', new \Nocarrier\Hal())
//         ->setResource('bar2', new \Nocarrier\Hal())
// );

// following options DO work
// $hal->setResource('foo', new \Nocarrier\Hal());
// $hal
//     ->setResource('foo1', new \Nocarrier\Hal())
//     ->setResource('foo2', new \Nocarrier\Hal());
// $hal->addResource(
//     'foo',
//     (new \Nocarrier\Hal())->setResource('bar', new \Nocarrier\Hal()),
//     false
// );
// $hal->addResource(
//     'foo',
//     (new \Nocarrier\Hal())->addResource('bar', new \Nocarrier\Hal(), false),
//     false
// );

echo ($hal->asJson(true));

Non-working options throw:
ErrorException in HalJsonRenderer.php line 164: Undefined offset: 0
Seems to relate to the test count($embedded) === 1 in https://github.com/blongden/hal/blob/master/src/Nocarrier/HalJsonRenderer.php#L159
I am using current version 0.9.11

:+1: confirmed - will have a look. Thanks for the report.

Ok that was harder to track down than I was anticipating - but I think I got it. Should now be fixed on master and will be incorporated into the next release assuming nobody reports any issues.
Thanks for the clear report :)

Thanks a lot for fixing :+1:

Well then, if it was fixed, why not close the issue :)

Because I was sunning myself in Fuerteventura for a couple of weeks ;)


[analyzer_plugin]: setContextRoots event never received

Hello!
TL;DR, in certain situation, the setContextRoots event is never received by plugins.
I'm not sure what exactly triggers it, so I made a reproduction repository. It is composed of:

the analyzer_plugin.
It is the bare minimum for a plugin, with no logic in it besides extending ServerPlugin.
many packages (some using analyzer_plugin, some not).
They are composed if nothing but pubspec.yamls and maybe a pubspec_overrides.yaml
The packages that uses the analyzer_plugin do so through a path dependency

The reproduction repository is here: https://github.com/rrousselGit/analyzer_plugin_bug_repro
Steps to reproduce:

Clone this repository
Do a global search of path: /Users/remirousselet/dev/rrousselGit/fake_riverpod/packages/custom_lint and replace it with an absolute path that points to packages/custom_lint
Run dart pub get everywhere (the IDE should suggest it)
Enable analyzer logs
Restart the analyzer server

In the logs, you will see some requests made to the analyzer plugin. Such as:
1723714208683:PluginReq:{"id"::"14","method"::"plugin.versionCheck","params"::{"byteStorePath"::"/Users/remirousselet/.dartServer/.analysis-driver","sdkPath"::"/Users/remirousselet/dev/flutter/bin/cache/dart-sdk","version"::"1.0.0-alpha.0"}}:file::///Users/remirousselet/.dartServer/.plugin_manager/12c93ced09bb72bc715613b8b9aeae4c/analyzer_plugin/bin/plugin.dart::
1723714208690:PluginRes:{"id"::"14","requestTime"::1723714208686,"result"::{"isCompatible"::true,"name"::"Custom Lint","version"::"1.0.0-alpha.0","interestingFiles"::["*.dart"]}}:file::///Users/remirousselet/.dartServer/.plugin_manager/12c93ced09bb72bc715613b8b9aeae4c/analyzer_plugin/bin/plugin.dart::

But there's never a analysis.setContextRoots request sent by default.
Note:
If you restart the analyzer server while the plugin is running, and override
ServerPlugin.handleAnalysisSetContextRoots to log somewhere in a while,
you will see that the method is invoked right before shutdown is invoked.
It is hard to see in the analyzer logs, because the request appears in the log right
before the log file is cleared.

Dart SDK version: 3.5.0 (stable) (Tue Jul 30 02:17:59 2024 -0700) on "macos_arm64"

Note that this is a regression, and it completely breaks analyzer_plugins in this scenario.
No setContextRoots = no analysis at all.
So this is fairly important IMO

I can repro - the plugin does get requests when I open Dart files, but it never seems to get context roots (I don't even see this during shutdown).

Will debug and post back.

Doesn't seem to happen when I use latest Flutter master:

But does happen on 3.5. Not sure if it's a race (and coincidence), or if the issue is fixed since the release. I'll try to narrow it down to confirm.

Debugging this in 3.5, I think there is a race condition. Haven't compared to master yet, but documenting it here while I have it in my head.
There's code here for addPluginToContextRoot which fires for each context root for the plugin:
https://github.com/dart-lang/sdk/blob/3.5.0/pkg/analysis_server/lib/src/plugin/plugin_manager.dart#L320
It starts the plugin if it's not already started, and then it calls plugin.addContextRoot(contextRoot). For a single plugin and single context root, this code works fine - the plugin is started, the SendPort is received (during the await for plugin.start) and then the root is added.
However, in the bad case, it appears we run this code twice for the same plugin/root at approximately the same time:

In this case, the first call starts setting up the plugin and waits for it (so it can then add the root), but the second request comes in and skips most of the method (because plugin != null at this point) and then adds the root itself. Because the plugin isn't initialized yet, it triggers setContextRoots which gets dropped because there is no SendPort:
https://github.com/dart-lang/sdk/blob/3.5.0/pkg/analyzer_plugin/lib/src/channel/isolate_channel.dart#L246
Then when the first call finishes starting the plugin and tries to add the root, it finds the root has already been added (even though the plugin wasn't notified because it wasn't fully started at the time) so it skips adding the root (and therefore notifying the plugin).
The reason we're calling this multiple times is because driver.enabledPluginNames has dupes in this loop:
https://github.com/dart-lang/sdk/blob/3.5.0/pkg/analysis_server/lib/src/plugin/plugin_watcher.dart#L44-L45

1724078222205:Info:adding plugins custom_lint, custom_lint, custom_lint, custom_lint, custom_lint, custom_lint to C::\Dev\Temp Projects\analyzer_plugin_bug_repro

I haven't verified yet, but I'm wondering if this could be related to https://github.com/dart-lang/sdk/issues/56047 because the child analysis options were being recording against the root context instead of their own. If so, then the cherry-pick for that fix should resolve this in the next 3.5 hotfix. I'll patch this in and verify shortly.
But, I also wonder whether this code here should use a Set because I feel like you could still trigger this by having the plugin names in multiple analysis_options files within a single context even with the fix? @srawlins @bwilkerson thoughts?
https://github.com/dart-lang/sdk/blob/3.5.0/pkg/analyzer/lib/src/dart/analysis/driver.dart#L349-L353

I suspect that nobody considered the impact of the multiple-analysis-options-per-context work on plugins. I know that I didn't.
I suspect that even if setContextRoots were being sent like it should be that we'd be passing the root of the analysis context(s). What we really need to be passing in are the directories containing the analysis options file in which the plugin is enabled, otherwise the plugin might start reporting against code for which it wasn't enabled.

The cherry-pick above didn't solve the issue - I think it was coincidence I initially didn't see it on master, it certainly reproduces when I'm running from source with my additional debug logging.
Changing the List to a set reduces the issue, but does not solve it - because the same race still appears to occur for a single plugin and multiple context roots, because PluginWatcher.addedDriver does not await manager.addPluginToContextRoot, and so when multiple drivers are created at the same time, they can all still enter plugin.addContextRoot(contextRoot) while the plugin is still starting.
We can perhaps fix this by having plugin.start() wait on the plugin to be initialized if it's already in progress  (instead of just skipping to adding the root), however I can't help but think something else isn't quite right here. For example in the log I see:
1724079379402:Info:adding plugins custom_lint to C::\Dev\Temp Projects\analyzer_plugin_bug_repro
However as far as I can tell, custom_lint is not enabled in the root of this repository, so I wonder if this is a difference that's caused this to show up. Specifically, I'm wondering whether looking at the whole of analysisOptionsMap here is valid (or if it's the entire workspace). I need to better understand the changes made relating to combining the contexts to properly understand what this should do.
https://github.com/dart-lang/sdk/blob/3.5.0/pkg/analyzer/lib/src/dart/analysis/driver.dart#L349-L353

@bwilkerson

I suspect that nobody considered the impact of the multiple-analysis-options-per-context work on plugins. I know that I didn't.
I suspect that even if setContextRoots were being sent like it should be that we'd be passing the root of the analysis context(s).

Yeah, I think you're right that this wouldn't work and probably we should only look at the analysis_options at the root of the package instead.
But that aside, there's still something wrong here based on above. This also looks suspicious to me - the context for a nested project is shown here as using the analysis options for the root, so if I fixed it to only look at the root, I feel like it'd still get the wrong results.

I'll try to get some tests to repro this as it'll be easier to debug and understand when I can use the debugger instead of the instrumentation log!

Yeah, I think you're right that this wouldn't work and probably we should only look at the analysis_options at the root of the package instead.

I don't think "package"s have anything to do with it because the algorithm that determines where the analysis contexts are doesn't take package boundaries into consideration.

So I think this issue is explained by looking at the entire tree of analysis_options instead of only the one for the context (eg. this project shouldn't have the plugin at all, but it's getting it from the flattened analysis options of the whole project).

That doesn't match my understanding of how things are suppose to work now, so if that's how they're actually working that does sound like a bug.
For one thing, there isn't suppose to be any concept of "the flattened analysis options of the whole project". Every library (technically file, but we never access the options for a part) knows the options that should be used when analyzing it.
The most consistent semantics would be for any library for which a plugin P is enabled it would be analyzed by P, and any library that hasn't enabled P wouldn't be analyzed by P. That means that we do need to look in every analysis options file to see whether P is enabled, and build the included and excluded lists appropriately.

@bwilkerson

For one thing, there isn't suppose to be any concept of "the flattened analysis options of the whole project".

Right, so I think this code here is wrong?
https://github.com/dart-lang/sdk/blob/3.5.0/pkg/analyzer/lib/src/dart/analysis/driver.dart#L349-L353
It seems to read all enable plugins from every project, so for the root folder in your workspace, it tries to enable every plugin from every nested project (which is both incorrect, and includes dupes).
If I change that code like this:

Then it appears to solve the problem - we only look at the analysis options for the root of that context (although it seems odd to use the analysis options file path to get the appropriate file path). I don't know if this is a good fix or whether it has other mismatches though. The additional race I noted above when a plugin exists for multiple drivers still occurs, but turns out to not actually be a problem because when the first context finishes initializing the plugin, it re-sends all roots (not only the one it was adding), so this "race" has probably existed forever and isn't actually impacting anything.
However, I'm less sure that this "fix" above works for Pub workspaces, because if I understand correctly, that's when we may have a single context/driver that spans multiple projects and/or analysis_options, in which case I think the nested projects would not be able to have their own plugins enabled - although I'm also unsure if that is a problem or an expectation (I think it's not clear to me what the definition of "context root" in the plugin API really means in a Pub workspace).

Also:

I don't think "package"s have anything to do with it because the algorithm that determines where the analysis contexts are doesn't take package boundaries into consideration.

I'm a little confused by this. I thought contexts were where pubspec.yaml or analysis_options.yaml are (so not a perfect match, but close) - however looking at the code it seems like _createContextRoots looks for package_config.jsons (or BUILD.gn). However, if I open the SDK pkg folder, I do see individual contexts for each package like analyzer even though it doesn't have a package_config.json.
I think I know less about this than I assumed. In particular it's not clear to me:

what are context roots in the server (if they're not package boundaries and they're based on package_config, why do I get them for each pkg/* in the SDK?)
does the definition of context roots sent to plugins match the servers definition
where is it valid to enable a server plugin (only at a context root, or at any that may be inside one but is not a context root)?

(if the answers to these already exist somewhere I should have read, I'm happy to be pointed there instead of just given the answers!)

^ These are questions I've asked myself too.
I think ContextRoot would benefit from dartdoc that explains what those are. The current doc is fairly unhelpful about it:

/// Information about the root directory associated with an analysis context.
///
/// Clients may not extend, implement or mix-in this class.

It seems to read all enable plugins from every project, so for the root folder in your workspace, it tries to enable every plugin from every nested project (which is both incorrect, and includes dupes).

(I'm not sure what you mean by "project", but I'm assuming you mean "the directory that was opened in the IDE".)
Yes, that's wrong.

analysisContext?.contextRoot.optionsFile

As far as I'm aware, ContextRoot.optionsFile shouldn't exist; it's meaningless. @pq In case I'm missing something.

I don't know if this is a good fix or whether it has other mismatches though.

It doesn't match the semantics I think we want. 🙂

However, I'm less sure that this "fix" above works for Pub workspaces ...

Right. The semantics I described earlier we intended for that situation (though I think they also nicely handle the non-workspace case as well).
Although, in all fairness, the semantics I gave were ignoring the fact that we will only load a single plugin.
Given that restriction, I think the right algorithm is to iterate over all of the analysis options files (as it's doing now), and look for enabled plugins. When it finds the first enabled plugin it needs to remember the path of the plugin and the set of files in the directory containing the analysis options file that are to be analyzed. After that it can ignore analysis options files that don't enable the one chosen plugin, and add the included and excluded files if the options file does enable the plugin.

I'm a little confused by this. I thought contexts were where pubspec.yaml or analysis_options.yaml are (so not a perfect match, but close) - however looking at the code it seems like _createContextRoots looks for package_config.jsons (or BUILD.gn).

The first part used to be true, but we should now only be creating a context when we find a package_config.json file.

However, if I open the SDK pkg folder, I do see individual contexts for each package like analyzer even though it doesn't have a package_config.json.

That isn't what I would have expected. @keertip

what are context roots in the server

They are the information about where to create an analysis context that is gathered before we create any actual contexts. There's a 1-to-1 correspondence between a context root and an analysis context.

does the definition of context roots sent to plugins match the servers definition

Good question. It used to, but I don't know whether that was preserved in the workspace support because I don't think anyone considered the impact on plugins. It probably shouldn't match anymore because of the way I'd like to see plugin enablement work (see the next paragraph), but I'm open to other designs than the one I proposed.

where is it valid to enable a server plugin (only at a context root, or at any that may be inside one but is not a context root)?

I think that the UX would be best if users could enable plugins for any subset of an analysis context. I'd do that by allowing them to enable plugins in any analysis options file and use the location of the options file to build the included and excluded lists sent to the plugin. That seems consistent with the way the rest of the analysis options work.
That definition would also minimize the number of changes that users have to make to their options files in order to use a workspace.

(I'm not sure what you mean by "project", but I'm assuming you mean "the directory that was opened in the IDE".)

Sorry, I meant "package" (eg. a think with a pubspec that has some name). In C#, these were called "Projects" (and a workspace of multiple projects was a "Solution") and I often use package and project interchangeably (but try to use package!).

However, if I open the SDK pkg folder, I do see individual contexts for each package like analyzer even though it doesn't have a package_config.json.

That isn't what I would have expected. @keertip

Ok, let me debug why that's happening and I can open a separate issue if it seems like something is wrong there.

It probably shouldn't match anymore because of the way I'd like to see plugin enablement work (see the next paragraph), but I'm open to other designs than the one I proposed.
...
I think that the UX would be best if users could enable plugins for any subset of an analysis context. I'd do that by allowing them to enable plugins in any analysis options file and use the location of the options file to build the included and excluded lists sent to the plugin.

Currently it appears that we can't pass an include list to a plugin. When we call setContextRoots, we  pass a single root folder and a set of exclude, but no include. So is your suggestion that we pass the folder that contained the analysis_options that enabled the plugin as the "context root" folder (so that context roots for plugins are actually "the roots for where the plugin is enabled" rather than what the server now considers roots, and the package_config.jsons could be further up the tree)?
I think this means we'll need to track "plugin roots" specifically so that if a plugin is added to an analysis_options file, the plugin is notified of that new root even though it didn't change any server context roots?
I'm not up-to-speed with the plugin changes that were in progress - is this something that makes sense to try and handle now, or when that work is further along (@srawlins?).
And since this is a much larger change than could be cherry-picked, do we need to do anything about this issue for 3.5? (it's not currently clear to me if there is an easy fix that restores the previous behaviour and I'm less sure now that the change I made above matches the previous behaviour (or whether doing something like that is sound anyway).

However, if I open the SDK pkg folder, I do see individual contexts for each package like analyzer even though it doesn't have a package_config.json.

That isn't what I would have expected. @keertip

Sorry, this was user error. I see many contexts when I open my "analysis server workspace" because it's a multi-root workspace. So when I tested, I closed that and opened the pkg folder directly, but since that doesn't have my VS Code setting to point at a bleeding-edge SDK, it was loading with a stable version 3.4 (because I hadn't updated my stable version since the release).
I just verified with bleeding-edge opening the pkg folder and I only had 4 contexts (one for pkg, and others because of nested projects I'd triggered pub get in).

So is your suggestion that we pass ...

No, my suggestion was based on a faulty memory of what information we could pass to the plugin.

I think this means we'll need to track "plugin roots" specifically so that if a plugin is added to an analysis_options file, the plugin is notified of that new root even though it didn't change any server context roots?

That is probably the right long term solution. I just don't know whether it makes sense to try to do that in the short term.

And since this is a much larger change than could be cherry-picked, do we need to do anything about this issue for 3.5?

I'm not ready to answer that question because I don't understand either the problem or the solution well enough at this point. We're also taking a look at it.

The fix for this issue was included in Dart 3.5.2 (released last week) and Flutter 3.24.2 (release today).


Add new LENGTH digester & more fixes

This PR contains an important bugfix, and required a new release:

Families-check was modifying plugin-digesters while iterating.

Detailed changes:

Length of bytes-hashed are provided for each file as just another digester.
Send to stdout explanatory msgs about what is being hashed only when
hashing multiple items.  The point is for script using omnihash to
easily parse its output in the case of a single file, but still contain
all necessary info in case it's input is being redirected in a log file
and multiple files are hashed.
Retrofit all TCs as TestCase.methods, to easily view what's was the failing condition.
Added TCs for hash-length.
Now ALL TCs OK.

An excellent solution!


Update Clash for windows to 0.11.1(latest)

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

Microsoft Reviewers: Open in CodeFlow

Ребят где найти плагины на него


Error: "You have requested a synthetic service ("request")." when using excluded_404s

Whey I configure the bundle to use excluded_404s for a fingers_crossed handler I get the following error:
RuntimeException in ContainerBuilder.php line 915:
You have requested a synthetic service ("request"). The DIC does not know how to construct this service.

When looking in the code, I think this causes the error (MonologExtension.php line 293-298)
} elseif (!empty($handler['excluded_404s'])) {
    $activationDef = new Definition('%monolog.activation_strategy.not_found.class%', array($handler['excluded_404s'], $handler['action_level']));
    $activationDef->addMethodCall('setRequest', array(new Reference('request', ContainerInterface::NULL_ON_INVALID_REFERENCE, false)));
    $container->setDefinition($handlerId.'.not_found_strategy', $activationDef);
    $activation = new Reference($handlerId.'.not_found_strategy');
} else {

I'm using symfony 2.6.7 with monolog-bundle 2.7.1
My guess is that the MonologExtension should use the request stack, but that might not be compatible with older versions of symfony.

Yeah and in my experience this isn't an issue so I am not sure why you experience that.


Compare tool removes common part of filename

This behavior may be intended (if so, ignore...), but I was pretty confused by it when trying out the tools, so I wanted to bring it up anyway.
I ran the following command to compare two of the example documents:
$ java -jar tools-java-1.1.0-jar-with-dependencies.jar CompareDocs compareOutput.xlsx ../testResources/SPDXJSONExample-v2.2.spdx.json ../testResources/SPDXJSONExample-v2.3.spdx.json

In the resulting spreadsheet, the two names were shown as 2.spdx.json and 3.spdx.json; the common part in the beginning was most likely trimmed by the normalizeDocNames function.

@nicoweidner - It's been a while, but I don't think this is intentional.  It would be better to just remove the common portions of the path before the filename.


Automate release note generation using release-drafter

PR's in all the different repos:
https://github.com/NVIDIA-Merlin/models/pull/439
https://github.com/NVIDIA-Merlin/core/pull/88
https://github.com/NVIDIA-Merlin/systems/pull/103
https://github.com/NVIDIA-Merlin/systems/pull/107
https://github.com/NVIDIA-Merlin/Transformers4Rec/pull/415
https://github.com/NVIDIA-Merlin/Merlin/pull/308

Each of the repos has a request in the CONTRIBUTING document to use labels.  Each repo has at least one label for each of the categories.  Closing.


Fix explanation of how env is set

refer: https://github.com/metalbear-co/mirrord/pull/189, closes https://github.com/metalbear-co/mirrord/issues/338

Thanks!


W-10945032: Changed "variables" to "vars"

Thanks for your contribution. We incorporated it into a separate pull request: https://github.com/mulesoft/docs-dataweave/pull/140


Remove the flag to opt out of SkParagraph.

The flag was added to work around regressions in the switch to SkParagraph. This bug tracks the remaining issues to fix before the flag can be removed and the backend deleted.

I believe this is done. cc @jason-simmons


[bug]: Error at compile-time: The argument type 'Object?' can't be assigned to the parameter type 'T?'

Describe the bug
Haven't changed my stacked dependency in a long while today after doing pub upgrade I guess I got this error when trying to run my flutter app:
Error (Xcode): ../../.pub-cache/hosted/pub.dev/stacked_core-1.2.4/lib/src/code_generation/locator/stacked_locator.dart:405:19: Error: The argument type 'Object?' can't be assigned to the parameter type 'T?'.
First got that for stacked_shared package, and now for stacked_core.
I checked stacked_shared on pub.dev (https://pub.dev/packages/stacked_shared/score) and I guess something's wrong because even there I can see they're getting the same error:
(I even tried to use an older version for this transitive dependency and then the same error appeared on stacked_core-1.2.4)
To reproduce

Just doing flutter run (for iOS in my case)
"Failed to build iOS app"

Expected behavior
No response
Screenshots

Additional Context
No response

Hello,
Yes, I've got the same issue.  I updated to the latest Flutter and did a re-install of Stacked cli and still getting the same error as you on iOS.
Cheers,
Alex

same

I have the same error en FLutter web

temp solution:
dependency_overrides:
    stacked_shared:
        git:
            url: https://github.com/ihadabs/core

(need to be tested)

Temporary solution works. But it should be fixed immediately on repo

Wil check into this now.

@ihadabs what was the fix that you made?

This was due to a get_it update. It's fixed in latest stacked_shared.
If this happens again we'll fix our versions and do manual updates in the future. Thanks for the patience and reporting the issue so quickly.

Update to stacked_shared: ^1.3.2

../../../.pub-cache/hosted/pub.dev/stacked_core-1.2.4/lib/src/code_generation/locator/stacked_locator.dart:405:19: Error: The argument type 'Object?' can't be assigned to the parameter type 'T?'.

'Object' is from 'dart:core'.
instance: instance,
^
Target kernel_snapshot failed: Exception

got error from stacked_core as well

Provided solution doesn't seems to work. I added stacked_shared: ^1.3.2 to the pubspec but I keep getting the same error

Provided solution doesn't seems to work. I added stacked_shared: ^1.3.2 to the pubspec but I keep getting the same error

Have cleaned Flutter ?
flutter clean

Yes, I agree, there also seems to be an issue with stacked_core.  pub.dev is showing a 'static analysis' failure on the 'score' tab and is the same error I'm getting run trying to run my project.  Downgraded get_it to 7.2.0 as a temporary fix.


Fixed #8660 (Calender Icon Compatible with Multiple Browsers)

Issue Reference
#8660
Description
Set the display Property in parent div of calender icon
How To Test This?
Go to 'Admin panel'
See the stats calendar icon OR
Click on 'edit/create product' inside the catalog/products
See customer group price date & time

@JunaidJaved248 Getting a few issues.

Special price date input field shrink. Here
Getting the same above issue on on create event page. Here
Date-only attribute input field size shrink. Here
On the date and time input field calendar icon goes out on the left. Here

@Amitk-webkul Let me check

@JunaidJaved248 Getting a few issues.

Special price date input field shrink. Here
Getting the same above issue on on create event page. Here
Date-only attribute input field size shrink. Here
On the date and time input field calendar icon goes out on the left. Here

@Amitk-webkul All Four Pointed Issue Fixed. You can test it now
Thanks

@JunaidJaved248 All the issues are fixed and now it is working fine.

@JunaidJaved248 All the issues are fixed and now it is working fine.

Great


Can't Open Text Files NOT Created By App

I know I can import the files via the app but not via a file manager directly.

Another user already expressed this issue


Adds new stream input into modal

Solves this issue #455 , by adding the stream name input into the modal.

@disissid thanks for working on this!  Can you post screenshots of what the new UI looks like so folks interested in the design can review without needing to check this out in the dev VM?
Also can you fix the lint error indicated by the Travis CI "backend" test suite?   It's easy for you to see it locally with tools/lint-all.
It looks like the frontend tests also failed with some sort of infinite loop issue; I suspect that this actually broke the tests, though -- try running ./tools/test-all and in particular ./tools/test-js-with-casper locally to investigate.

@timabbott  okay I'll fix the errors, here are the screenshots

Hey @timabbott I'm not quiet sure  but I think tests needs to be updated for this, because earlier stream name was asked before the modal so in tests it is waiting for the stream name input to load first, that's where it is breaking.
https://github.com/zulip/zulip/blob/master/frontend_tests/casper_tests/04-subscriptions.js
casper.waitForText('Subscribed', function () {     casper.test.assertTextExists('Subscribed', 'Initial subscriptions loaded');     casper.fill('form#add_new_subscription', {stream_name: 'Waseemio'});     casper.click('form#add_new_subscription input.btn'); }); casper.waitForText('Waseemio', function () {     casper.test.assertTextExists('Create stream Waseemio', 'Modal for specifying new stream users');     casper.click('form#stream_creation_form button.btn.btn-primary'); });
Is that right?

Yeah that was it. Updated the test file.
casper.waitForText('Subscribed', function () {     casper.test.assertTextExists('Subscribed', 'Initial subscriptions loaded');     casper.click('form#add_new_subscription input.btn');     casper.fill('form#stream_creation_form', {stream_name: 'Waseemio'}); }); casper.waitForText('Create stream', function () {     casper.test.assertTextExists('Create stream', 'Modal for specifying new stream users');     casper.click('form#stream_creation_form button.btn.btn-primary'); });
worked like a charm :)

Ah, should I commit the test file in this PR only or should I make another one for it?

You should put the test file in this PR -- usually it makes sense to update and/or add tests in the same PR that introduces the behavior change or feature, since then the related bundle of stuff is all together.

Hey @timabbott I've updated the casper tests, you can review it now.

Looks like it is still failing the Casper tests in Travis CI?

Are you saying even after updating the tests, they are failing?

@timabbott  I've updated the tests, there was a line which was supposed to be in the next block.

@disissid I just tested this out and it's looking pretty great!  There's a few things you'll need to do though:

Fix the !should_list_all_streams case to still have the input outside of the modal
Rebase the branch on top of master and squash it into a single commit

@disissid just checking in -- I'd love to get this finished and merged; let me know if you're stuck on anything!

@timabbott I'll do this by tomorrow, was busy with college assignments and
labs. Sorry for the delay.
On Sat, Apr 9, 2016 at 9:57 AM, Tim Abbott<EMAIL_ADDRESS>wrote:

@disissid https://github.com/disissid just checking in -- I'd love to
get this finished and merged; let me know if you're stuck on anything!
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/zulip/zulip/pull/514#issuecomment-207701693

@timabbott I've updated the PR, squashed the commits and cleaned up the commit message. Please review.

Thanks for all the work on this @disissid!  I think this is nearly done.

@disissid just wanted to check in  -- are you going to have time to finish this PR or should we find someone else with time to finish it?

@disissid I just want to check in about whether you think you'll be finishing this up in the next few weeks -- if not, I would like to refer some other contributors to this pull request so they could help finish it up. Thanks!

This PR has been integrated into #711, so I think we should close this and just work on the #711 copy of it.


Short solution needed: "How to set index file" (nginx)

Please help us write most modern and shortest code solution for this issue:
How to set index file (technology: nginx)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments to make solution explained.
Link to this issue in comments of pull request.

How to set index file


Fix the question 21

The output is undefined because it is an assignment, then we have to ask for the value of the sum const.

Looks good.


Consider to include Human Nephrogenesis Atlas expression values in manual curation of high-evidence genes

Instead of or in addition to using Descartes fetal gene expression, consider using "Human Nephrogenesis Atlas".
The "Human Nephrogenesis Atlas" provides ssRNA seq data of two 14-weeks human kidneys (after QC: 8,316 cells (6,667 nephrogenic)).
They identified 18 cell clusters in the developing kidney. For each gene, you can get expression values of each cell cluster or of the total "Developmental Kidney". I suppose the "Developmental Kidney" expression value could be used if you only want 1 value per gene for the fetal kidney. This values is accessible via an API (script in progress).
So far, I did not find out, how exactly they calculate this expression value (which normalization technique etc.). I screened the original paper, but they didn't specify it there. I have not read the paper in detail, yet.
https://sckidney.flatironinstitute.org/
To get an example expression value: => Tab 'Gene Network' => select 'Developmental Kidney' => select gene of interest

`# Nephrogenesis Atlas - API
library(httr)
library(jsonlite)
library(tidyverse)
library(progress)
download HGNC gene table from github repository "kidney-genetics"
hgnc_gt_version <- "2023-06-21"
gene_table_url <- paste0("https://github.com/halbritter-lab/kidney-genetics/raw/main/analyses/A_AnnotationHGNC/results/non_alt_loci_set_coordinates.", hgnc_gt_version, ".csv.gz")
download.file(url = gene_table_url, destfile = paste0("gene_score/raw/HGNC_", hgnc_gt_version, ".csv.gz"))
gunzip(filename = paste0("gene_score/raw/HGNC_", hgnc_gt_version, ".csv.gz"),
destname = paste0("gene_score/raw/HGNC_", hgnc_gt_version, ".csv"))
load HGNC gene table and filter for protein-coding genes # TODO: change paths
HGNC_table <- read.csv("gene_score/raw/HGNC_2023-06-21.csv") %>%
filter(locus_group == "protein-coding gene")
function for getting expression values via the Human nephrogenesis atlas API
get_nga_expression <- function(cell_cluster, entrez_id_vec){
set query url
entrez_id_collapse <- paste(paste0("entrez=", entrez_id_vec), collapse = "&")
url <- "https://sckidney.flatironinstitute.org/integrations/query/"
query_url <- paste0(url, cell_cluster, "?", entrez_id_collapse)
set the header
headers <- c("Content-Type" = "application/json")
get the response
response <- GET(query_url, add_headers(headers))
get the response content
content <- content(response, as="text", encoding = "UTF-8")
content_list <- fromJSON(content)
select queried genes
res <- data.frame(content_list$genes) %>%
filter(query == TRUE) %>%
dplyr::select(standard_name, systematic_name, entrez_id = entrez, expression_value) %>%
mutate(cell_cluster = cell_cluster)
return(res)
}
get expression values chunk wise (all genes at once not possible)
entrez_table <- HGNC_table %>% dplyr::select(entrez_id) %>% unique()
entrez_split_list <- split(entrez_table$entrez_id, ceiling(seq_along(entrez_table$entrez_id) / 250))
expr_vals_df <- data.frame()
cell_clusters <- c(57) # 57 = Developmental Kidney
pb <- progress_bar$new(total = length(entrez_split_list)*length(cell_clusters))
for (cell_cluster in cell_clusters){
for (i in entrez_split_list){
pb$tick()  # Increment progress bar
cat("\n")
sub_df <- get_nga_expression(cell_cluster, i)
expr_vals_df <- dplyr::bind_rows(expr_vals_df, sub_df)

}
}
View(expr_vals_df)
write csv ...`

a <- b

library(httr)
library(jsonlite)
library(tidyverse)
library(progress)

# download HGNC gene table from github repository "kidney-genetics"
hgnc_gt_version <- "2023-06-21"
gene_table_url <- paste0("https://github.com/halbritter-lab/kidney-genetics/raw/main/analyses/A_AnnotationHGNC/results/non_alt_loci_set_coordinates.", hgnc_gt_version, ".csv.gz")
download.file(url = gene_table_url, destfile = paste0("gene_score/raw/HGNC_", hgnc_gt_version, ".csv.gz"))
gunzip(filename = paste0("gene_score/raw/HGNC_", hgnc_gt_version, ".csv.gz"),
       destname = paste0("gene_score/raw/HGNC_", hgnc_gt_version, ".csv"))


trivial typo in ci_build README

bellow -> below

Can one of the admins verify this patch?

LGTM

@tensorflow-jenkins test this please.
(for auto-merge happiness even though it's just a doc change)

Jenkins, this this PLEASE!

Jenkins? test this please?

It's a Friday - he probably left early to go to Yosemite.  In fairness, the falls are absolutely gorgeous this time of year.

@martinwicke, you mistyped the first "test this".  Maybe Jenkins needs some
NLP autocorrect.  :P
On Fri, Apr 15, 2016 at 4:11 PM, David Andersen<EMAIL_ADDRESS>wrote:

It's a Friday - he probably left early to go to Yosemite. In fairness, the
falls are absolutely gorgeous this time of year.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/tensorflow/tensorflow/pull/1972#issuecomment-210678466


bro, this doesn't work after the 1.6.7 patch, even u say u updated it.

it doesn't work in 1.6.7. It says "INVALID VERSION" then stops working.

it's a trojan
pls, trust me!!!!
I made one no virus, 609+ views, hand-coded with 4K horizon and me dad for the latest update.
Direct video link: https://cutit.org/krunkerhackvid
My channel: https://cutit.org/GoBayMax
(wuz at 35 subs, posted this video, now 59 subs)


Favoriting GraphQL Endpoints

It would be nice to be able to favorite GraphQL endpoints to allow users to easily switch between endpoints (for example dev / prod).
If this get's approved, I would take a chance at implementing the feature.

Hi

@romarioraffington It sounds like you are referring to the ability to set and use different values for different environments (that is, environment variables), similar to Postman environments?

I was thinking of something simpler @imolorhe. Maybe a dropdown to switch between different GraphQL endpoints or the ability to favorite endpoints that you can select.

@romarioraffington You should make it and just use graphiql as a dependency! 👍 I mean, if you can make it that would be awesome 💯

@acao or @benjie, can you expound on what exactly is a plugin (in the graphiql context)?
Initially, I was thinking of just adding a new component. Thanks

That will be determined soon. I have a proposal for the plugin API I hope to get set up and show to the GraphQL IDE working group (agendas are listed here in the repo), but we still have probably hundreds of people hours of work to get to the point where we are ready to implement a plugin API.


No JSON response on failures

The error codes seem to be fine, but there is no json output for some api errors:
$ curl -sL http://<IP_ADDRESS>:7000/api/brokers/add?id=0

<meta http-equiv="Content-Type" content="text/html;charset=ISO-8859-1"/>
<title>Error 400 </title>

HTTP ERROR: 400

Problem accessing /api/brokers/add. Reason:

    Broker 0 already exists

Powered by Jetty://

$ curl -sL http://<IP_ADDRESS>:7000/api/brokers/stop?id=999

<meta http-equiv="Content-Type" content="text/html;charset=ISO-8859-1"/>
<title>Error 400 </title>

HTTP ERROR: 400

Problem accessing /api/brokers/stop. Reason:

    broker 999 not found

Powered by Jetty://

it might be good to have a field for status so it can be parsed on failure (or success) and on failure parse the error, is that what you were thinking?

@joestein yea that would be great. just something that is json parse-able.
my team is currently building some tools around the api and if I can get some free time, I can try to get a PR for you. but if you have time and the need, feel free to take it on :)

Merged PR with small changes.


[FR]: Hotkey to toggle back from CSV view to database view

Contact Details
<EMAIL_ADDRESS>Present your request
I want to see Hotkey to toggle between CSV view and database view back and forth.
For which platform do you request this request??
Desktop

Hello @Ardalan-Saberi
with CSV you mean the markdown? or a readOnly table as the preview mode?

Shoot, yup! meant to say markdown view.


Open nodes when no childs

Problem is when we get information from other API and want to load categories and send request after click click on expand . And we want to show categories that can expand and after someone click to expend then send API request and respond add to tree. When i put
tmp_array["children"] = true;
i get plus at categories but it can't expend like its lock and documentation is:
"When using AJAX set children to boolean true and jsTree will render the node as closed and make an additional request for that node when the user opens it."
But there is not response of any kind when i try to open it. Thank you.

Sorry, but I can not help you without a demo or some code. You have either misconfigured jstree or you are not returning the data properly. I can assure you lazy loading works fine as can be seen in all the demos and jsfiddles I have provided.
For future how-to questions please use the discussion group. Of course feel free to provide a demo or some code here in this issue.
Best regards,
Ivan

Of course  i am sorry for mistake, here is code:
first json
[
{
"status": "success",
"request":
{
"mid":1515
}
,
"categories":[
{
"categoryId":123,
"categoryName":"Drinks",
"productCount":160,
"childCount":5
},
{
"categoryId":124,
"categoryName":"Dessert",
"productCount":25,
"childCount":5
},
{
"categoryId":125,
"categoryName":"Lunch",
"productCount":246,
"childCount":5
}
]
}
]
and code:
$(function () {
var array_json_tree = null;
$.getJSON("data/tree.json", function (data) {
array_json_tree = data[0].categories;
var tmp_set_tree = [];
if (array_json_tree !== null)
tmp_set_tree = replace_json_for_tree(array_json_tree);
    $('.js-tree').jstree({
        'plugins': ["wholerow", "checkbox"],
        'core': {
            'data': tmp_set_tree,
            'themes': {
                'name': 'proton',
                'responsive': true
            }
        }});
});

function replace_json_for_tree(json) {
    var retVal = [];

//        console.info(json);
$.each(json, function (index, value) {
var tmp_array = {};
tmp_array["id"] = value.categoryId;
tmp_array["text"] = value.categoryName;
if (value.childCount)
tmp_array["children"] = "A";
//ptmp_array["children"] = true; when set to true don't expend
retVal.push(tmp_array);
    });
    return retVal;
}

$('.js-tree').on("before_open.jstree", function (e, data) {
    console.log(data.node.id);
    var url = "data/Drinks.json";
    var jqxhr = $.getJSON(url, function (data1) {
        var tmp_set_tree = replace_json_for_tree(data1[0].categories);
        console.info(data.node);
        $('.js-tree').jstree("delete_node", data.node.id, "inside", tmp_set_tree, false, true);
        $('.js-tree').jstree("refresh.jstree");
    });
});

$(document).on("click", "#test-tree", function () {
    var checked_ids = $('.js-tree').jstree('get_checked');
    alert('You have selected: ' + checked_ids);
});

});

I figure what is problem but not solution for it. Lazy loading working great, but only when i have children is know variable, but i don't know what children are until someone click on plus mark. On that point i get loading icon and that it is. no before open is called and then i can not add children's dynamically. By the way great  library and great work, thank you. You don't have donating on web site or something like that? Regards.

I do have a donate button on the site - jstree.com :)
Anyway - you are not using lazy loading, so you can't expect anything to happen - you are just loading a static object as data. Read the docs on lazy loading - in your case - you should use core.data as a function. Something like this:
    $('.js-tree').jstree({
        'plugins': ["wholerow", "checkbox"],
        'core': {
            'data': function (node, cb) {
               if(node.id === "#") {
                 cb(tmp_set_tree);
               }
               else {
                 alert(node.id);
                 /* make a separate AJAX as needed, `node.id` is the node whose children you need to return,
                    once loaded - call cb with the result:
                    $.post('/some/endpoint', {'id':node.id}, function (data) { cb(replace_json_for_tree(data)); });
                   */
               }
            },
            'themes': {
                'name': 'proton',
                'responsive': true
            }
        }});

This is far from optimal code, but you get the idea of using core.data as a function - read the docs on it if this is not enough - there are examples on the front page of this repo.
Best regards,
Ivan

Its work perfect, thank you vary much.

I do have a donate button on the site - jstree.com :)
Anyway - you are not using lazy loading, so you can't expect anything to happen - you are just loading a static object as data. Read the docs on lazy loading - in your case - you should use core.data as a function. Something like this:
    $('.js-tree').jstree({
        'plugins': ["wholerow", "checkbox"],
        'core': {
            'data': function (node, cb) {
               if(node.id === "#") {
                 cb(tmp_set_tree);
               }
               else {
                 alert(node.id);
                 /* make a separate AJAX as needed,
                    `node.id` is the node whose children you need to return,
                    once loaded - call cb with the result:
                    $.post('/some/endpoint', {'id':node.id}, function (data) {
                      cb(replace_json_for_tree(data));
                    });
                  */
               }
            },
            'themes': {
                'name': 'proton',
                'responsive': true
            }
        }});

This is far from optimal code, but you get the idea of using core.data as a function - read the docs on it if this is not enough - there are examples on the front page of this repo.
Best regards,
Ivan

How do I call just a function that returns some json data on demand. I don't want to make ajax call as i have data and also I don't want to load data before creating the tree. I just want load some static data only when a node is opened? How do I handle this case?

I do have a donate button on the site - jstree.com :)
Anyway - you are not using lazy loading, so you can't expect anything to happen - you are just loading a static object as data. Read the docs on lazy loading - in your case - you should use core.data as a function. Something like this:
    $('.js-tree').jstree({
        'plugins': ["wholerow", "checkbox"],
        'core': {
            'data': function (node, cb) {
               if(node.id === "#") {
                 cb(tmp_set_tree);
               }
               else {
                 alert(node.id);
                 /* make a separate AJAX as needed,
                    `node.id` is the node whose children you need to return,
                    once loaded - call cb with the result:
                    $.post('/some/endpoint', {'id':node.id}, function (data) {
                      cb(replace_json_for_tree(data));
                    });
                  */
               }
            },
            'themes': {
                'name': 'proton',
                'responsive': true
            }
        }});

This is far from optimal code, but you get the idea of using core.data as a function - read the docs on it if this is not enough - there are examples on the front page of this repo.
Best regards,
Ivan

But these nodes that have "plus"/childs on demand, i want to sort them to the bottom of the list. how can I do that?

Please add this kind of stuff in the documentation. I researched for 4 days just for this. Tried many hacks. Thank God and vakata, I finally found it


HTML Button Install

3 buttons needed

Alphabet Button
Numbers Button
Definitions Button
(Give classes and ID's)

Done


[Question] Why io_ring_submit is doing the actual socket IO?

I am using io_uring to write a blocking mode socket. I expect the socket write will be executed asynchronously. But from the stack the io_ring_submit is actually doing the sendmsg syscall.
   - 2.80% User::Called::asyncWrite<Message>                                                                                ▒
      - 2.80% io_uring_submit                                                                                ▒
         - 2.79% syscall                                                                                ▒
            - 2.78% el0_svc                                                                                ▒
               - el0_svc_handler                                                                                ▒
                  - 2.77% el0_svc_common.constprop.2                                                                                ▒
                     - 2.70% __arm64_sys_io_uring_enter                                                                                ▒
                        - 2.61% io_ring_submit                                                                                ▒
                           - 2.59% io_submit_sqe                                                                                ▒
                              - 2.52% io_queue_sqe                                                                                ▒
                                 - 2.51% __io_queue_sqe                                                                                ▒
                                    - 2.48% __io_submit_sqe                                                                                ▒
                                       - 2.47% io_send_recvmsg                                                                                ▒
                                          - 2.06% __sys_sendmsg_sock                                                                                ▒
                                             - 1.94% ____sys_sendmsg                                                                                ▒
                                                + 1.92% sock_sendmsg                                                                                ▒

I am using kernel 5.4. How will blocking/non-blocking mode affect the io_uring? Why io_ring_submit is doing the actual socket IO?

I checked the code. For blocking sockets, io_submit_sqe will always try to async send/recv if possible. Only when sendmsg/recvmsg returns EAGAIN, will it then issue the async IO.

@zayg If you haven't checked out https://unixism.net/loti/index.html info provide there might help answer some of your questions.

I am using kernel 5.4. How will blocking/non-blocking mode affect the io_uring? Why io_ring_submit is doing the actual socket IO?

Because it's faster and of lower latency if the data is already there. There were some problems with that approach, where there is a recommendation to use IOSQE_ASYNC in that case. Unfortunately, there is no true async for socket I/O, so there are nonblock/poll/blocking-offload inside  io_uring to make it good enough.

@zayg, did that clear the question?


Why does it Override my Laravels default dd() method?

Why does it Override my Laravels default dd() method?
(I don't want it and I didn't use the how to to do it, like in the informations described: require DIR.'/../bootstrap/autoload.php';)

Now its okay. Unfortunately I don't know why...


Tasks

Download release binary base on arch when run npm install

https://github.com/etclabscore/dl-github-releases
https://github.com/jgm/pandoc/releases/
Extracted to node_modules/pandoc.js/.pandoc-local
https://github.com/GoogleChrome/puppeteer/blob/faa452718e/index.js (puppeteer)
https://github.com/GoogleChrome/puppeteer/blob/master/lib/BrowserFetcher.js

Maintain all support input/output format
Print missing runtime dependencies after install
Implement everything what pdc did
Emit dependency error when convert unsupported format

puppeteer 用 extract-zip
https://github.com/GoogleChrome/puppeteer/blob/faa452718e5b257ffe262f410e52f676b6587e80/lib/BrowserFetcher.js#L260

弄一下 https://github.com/visionmedia/node-progress


Nabu prov load and release graph incorporated into the templates

At present I need to review the "ops templates" and review the loading of triples.  I think currently just the summoned triples are loaded.
Need to review and, if needed, add the prov triples as a new step in the loading done by Nabu.

Need to add in the Nabu graph release building as a stage here too.


[DO NOT MERGE] SEO edits for Migrate to SharePoint

Edited description to add the SPMT keyword and focus on the other tool options.
Edited metadata title to focus more on the tool options, since the article doesn't actually take you through the steps of migrating to SharePoint.
Added several instances of keywords "online migration" and "migration tool".
@DaniHalfin

@danihelfin remove online where it says online migration to the cloud.  migration to the cloud is sufficient.  Also the product name is no longer SharePoint Online, but simply SharePoint.

@jdmartinez36 - can you follow up on the feedback above?

@DaniHalfin
Reverted 2 edits based on the author's request.


Reintroduce missing functions needed for the language server

Pull request
Description
Adding missing functions for tremor-language server (They fell prey to my unused code deletion rampage)
Checklist

[x] The RFC, if required, has been submitted and approved
[x] Any user-facing impact of the changes is reflected in docs.tremor.rs
[x] The code is tested
[x] Use of unsafe code is reasoned about in a comment
[x] Update CHANGELOG.md appropriately, recording any changes, bug fixes, or other observable changes in behavior
[x] The performance impact of the change is measured (see below)

manually chcked the coverage since coveralls isn't reporting back


Add info on Kube secrets

Proposed changes

https://docs.docker.com/ee/ucp/ucp-architecture/
Add info for how we handle secrets in Kubernetes
Formatting changes

Related issues (optional)

https://docker.atlassian.net/browse/ENGDOCS-244

Deploy preview for docsdocker ready!
Built with commit 7e27b53428124c2999966305123855bfa846836d
https://deploy-preview-9915--docsdocker.netlify.com

@david-yu @jlhawn @usha-mandya PTAL.

Hi @traci-morrison , do you know if there is also place to call out Secrets in a first class location in docs? Perhaps somewhere in a security section under Kube?

Hi @traci-morrison , do you know if there is also place to call out Secrets in a first class location in docs? Perhaps somewhere in a security section under Kube?

Hi @david-yu We talk about Kube network encryption, but according to @jlhawn, what we have now would probably best belong in the architecture docs as it’s not configurable.


IAM controller GA release

Determine exit criteria for IAM controller GA release and implement them.

Hi,
Just wondering what's the approximative timeline for the GA release?
Thanks

Hi, Just wondering what's the approximative timeline for the GA release? Thanks

I'm looking for a Q4 2022 timeframe for the IAM controller GA.

I believe this is now GA and can be closed?


ListMetadata for ChildXML Names for Custom Object Returning undefined

I am looking to use jsforce to list all metadata components for a specific type. It works for all metadata types returned from the DescribeMetadata Call, but not for the children components (ie: CustomField).
For example, the console.log returns null even when I have many custom fields in my org:
conn.metadata.list(types, '39.0', function(err, metadata) {
  if (err) { return console.error('err', err); }
    console.log(metadata);
});

Am I doing something wrong?

@dmgerow - can you try to omit the folder property completely?
conn.metadata.list([{ type: 'CustomField' }], '42.0', (err, metadata) => {
  if (err) {
    return reject(err);
  }
  resolve(metadata);
});

Also, for any types that are in a folder (e.x. reports), it is really dumb, but you must remove "Folder" from the name, except for "EmailFolder" which has a full name of "EmailTemplate".
Here is what I did when I needed to get parent and child types:
Note: this code has been slightly modified, but I am successfully able to get ALL types from describe including all child items and items in folders

call listMetadata for the parent type [{ type: 'CustomField' }] - no folder property
Make sure the response is actually something, because sometimes undefined is returned and if there is only one return item it is an object instead of an array.

response = _.isUndefined(response) ? [] : response;
response = !Array.isArray(response) ? [response] : response;

Then I loop through the responses to get the child items - BUT special processing is needed.

If Folder is in the type name, you need to remove Folder
If the type is EmailFolder then the fullName is EmailTemplate
The FullName from the child item is the folder name

const type = existingParentType
    .replace('EmailFolder', 'EmailTemplate') // special case with salesforce inconsistent naming
    .replace('Folder', '');
const folder = res.fullName;
conn.metadata.list([{type, folder}], '42.0', (err, childResponse) => { /* handle me */ });

Then I used lodash to flatten everything for me if needed because the return types were not always consistent.
const output = [];
if (Array.isArray(childResponse)) {
  const output = [];
  output.push(..._.flattenDeep(childResponse));
} else if (_.isObject(childResponse)) {
  output.push(childResponse);
}

This is very helpful - thank you @paustint !


Fix attractor size when plotting different orbits

Related to #111. Postponing.

I was about to open an issue when I saw this one.
As title says, attractor size when plotting distant orbits seems too big.

It can be fixed changing line 126 in plotting.py from:
radius = max(orbit.attractor.R.to(u.km).value, orbit.r_p.to(u.km).value / 6)

to
radius = orbit.attractor.R.to(u.km).value

but I suppose that without that max statement, attractor would appear really small in most cases, so I don't know if there is any solution.

but I suppose that without that max statement, attractor would appear really small in most cases, so I don't know if there is any solution.

Yep, that's the problem. There is also another thing:
https://github.com/poliastro/poliastro/blob/286e524/src/poliastro/plotting.py#L125
The attractor is being plotted one time for each planet. We could always plot the "real" radius, and although it would be too small in most cases, it is the easiest way to fix a sensible value without ruining the plot. Perhaps we should plot the attractor only once in the first place?


Duplicate Feature behavior

Selecting a feature and duplicating it:

Null % criteria number is not duplicated, it's set to default 10%, should copy whatever is being duplicated.
Name should duplicate similar to windows: Name_1, or Name_2 instead of a random string of letters.

kind of tricky to perfectly add a _#, so I add _$$$ where $$$ are random


adding action and filter to route53 for query logging configuration

Our team needed to enable GuardDuty in all our accounts and DNS query logs are required by GuardDuty. I've added a filter to determine the state of query logging for a hosted zone and an action to enable/disable it.
Please let me know what you think.

curiosity are you using tools/c7n-guardian on the guard duty enablement?

yes, we are using c7n-guardian

Thanks guys

Thanks again for the pr on this, there's definitely some subtlety to getting these setup correctly.  I've written up a lot of review comments on this, and i wanted to be clear that this pr is pretty good in terms of implementing the functionality. One guiding principle for custodian is to do minimal number of api calls for any given policy, and the
I've pushed a refactored version of the branch to https://github.com/capitalone/cloud-custodian/compare/master...kapilt:eisjcormier-master?expand=1
One non compatible change to this pr is removing the trailing . suffix on the log group name.
One thing i've investigated is also handling the creation of the log resource policy, which is also implemented but not wired in.

Hi Kapil,
Thanks for all the feedback and suggestions.
I think your changes look good.
Did you want to change this:
@HostedZone.action_registry.register('enable-query-logging')
to this:
@HostedZone.action_registry.register('set-query-logging')
Same question for the type-schema()
Did you need me to make any additional changes?
It is no longer in scope with your changes but the log rentention validation I was doing was based on these docs:
https://docs.aws.amazon.com/cli/latest/reference/logs/put-retention-policy.html
http://boto3.readthedocs.io/en/latest/reference/services/logs.html#CloudWatchLogs.Client.put_retention_policy
-Jon

i can take it from here re getting this pr merged, the biggest question is do you think we should be setting the log resource policy as part of this, we'll first try and find one that already exists? Definitely re change on action name. There's an issue with the azure dependency set on ci atm, i've got a pr in the works that resolves.

Yes, I think we should be setting the log policy. In your policy statement, I think you also need to add this action: logs:CreateLogStream
Thanks again


Adding more unit tests to the Owner class methods

@SunXP could you please use your own fork to test Github's pull request features?


Add a margin option.

It allows to add a margin (in pixels) on the left or right of the sidebar depending on its orientation.

closing due to inactivity


Remove deprecated OptionBuilder

This removes the deprecated OptionBuilder and replaces it with
Option.builder().

Closing, as this was fixed by #2957


Version 2.0.10

Fix form using yes/no instead True/False
Fix form for value tag that is using ; as list (out of bound)
Add limit parameter for pagination
Simplified the donation form and added Donate link to the main menu
Update mapping Key:electricity -> electricity
Unset the tag that is empty value

Coverage decreased (-0.03%) to 39.505% when pulling<PHONE_NUMBER>734ee0cd754facfb600fbc480b77f8 on develop into 2313dae3b2a661ce6abb97f9373f8ad43655db4f on master.


Registry correction

[Enter feedback here]
The following Registry Key is for GCCHigh and DoD. So, please .com for commercial, Charity and EDU O365.
Registry key | Value
SasUrl | https://adnotifications.windowsazure.us/StrongAuthenticationService.svc/Connector
StsUrl | https://login.microsoftonline.us
ResourceUri | https://adnotifications.windowsazure.us/StrongAuthenticationService.svc/Connector

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

ID: f333e8cb-c046-ee06-d33d-d7dc8e8af695
Version Independent ID: 10838695-f60e-42a7-b0e8-eed7820fd333
Content: Configure AD FS 2016 and Azure MFA
Content Source: WindowsServerDocs/identity/ad-fs/operations/configure-ad-fs-and-azure-mfa.md
Product: windows-server
Technology: ad-fs
GitHub Login: @billmath
Microsoft Alias: billmathRegistry key	Value
SasUrl	https://adnotifications.windowsazure.us/StrongAuthenticationService.svc/Connector
StsUrl	https://login.microsoftonline.us/
ResourceUri	https://adnotifications.windowsazure.us/StrongAuthenticationService.svc/Connector

#assign:xelu86

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


NDRS-468: Depth Since last finalized block

Related to: https://casperlabs.atlassian.net/browse/NDRS-468

I started a branch off an WIP branch causing merge conflicts, closing this PR to work the branch issues


milvus vectordb integration

Solves #146 .
In this PR, I implemented the same interface as the llm-chain-qdrant with some caveats :

All collection manipulations (ie creation, partitioning, index creation, loading etc...) is left to the user.
The add_text and add_documents methods return Vec, where string are UUIDs generated by the client in qrant. Milvus has an auto_id FieldSchema, we produce the id generated by Milvus. Arguably we should test that an id field exists :thinking: ?
qdrant offers a direct payload field, whereas Milvus does not, I implemented the metadata in insert + document content as a VARCHAR field column. The content and metadata are put in a HashMap and sent to Milvus as a string. VARCHAR field have a max_length limit of ~65000. I kept this not to break the API
search_similarity has the same issue with retrieving and parsing the metadata.

I think that rethinking the VectorStore trait would probably cleanup this implementation (and others ?) a bunch.

Hello  @AmineDiro i think we need to be able to support new_primary_varchar where we can insert our own primary in varchar instead of auto generated


Remove "name" property tracking from page view events

It seems that although we've added title values per https://github.com/blockstack/app/issues/149, we're still reporting name values for these events and inaccurately at that:

Name values are unneeded now in general so we can remove them completely.

I thought we talked about this, and agreed that keeping (and using name) was good? Because the "title" may change, as a display property, but we can always keep "name" consistent.
Also, in this case, it was the title that was inaccurate, not the name.

I think I created this issue before our chat about keeping a more canonical attribute.
The value for name here is getting set as the same non-canonical page title value as the title attribute, and as such, we need to update it to use the screen state values found here: https://github.com/blockstack/app/blob/5934829a40338ac269b80783912c8dad17af1962/packages/app/src/store/onboarding/types.ts#L9
e.g. screens/SECRET_KEY instead of "Copy Secret Key"
I'd also recommend simplifying those screen name values so they're not so verbose and uppercase (e.g. "secret_key" instead of "screens/SECRET_KEY") unless we're relying on the current format for something.


feat: adds sourcekit lsp

Adds sourcekit language server

@williambotman /fixup

Hello! Thanks but sourcekit is not a Mason package?

I'm sorry. I should have looked at it first. I will close this and reopen it when sourcekit is added (will try to come up with a PR for Mason =).
Best regards!


Updated goscaleio client

Description
Updated goscaleio client
GitHub Issues
List the GitHub issues impacted by this PR:

GitHub Issue #

https://github.com/dell/csm/issues/117

Checklist:

[x] I have performed a self-review of my own code to ensure there are no formatting, vetting, linting, or security issues
[x] I have verified that new and existing unit tests pass locally with my changes
[x] I have not allowed coverage numbers to degenerate
[x] I have maintained at least 90% code coverage
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have maintained backward compatibility

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration

[x] Executed unit tests

run e2e

run e2e test

run e2e test

run e2e test


implment Deep coral: Correlation alignment for deep domain adaptation

domainbed implment coral as MMD loss
https://github.com/facebookresearch/DomainBed/blob/dad3ca34803aa6dc62dfebe9ccfb57452f0bb821/domainbed/algorithms.py#L866
https://arxiv.org/pdf/1607.01719
MMD between two domains:
https://github.com/facebookresearch/DomainBed/blob/dad3ca34803aa6dc62dfebe9ccfb57452f0bb821/domainbed/algorithms.py#L878C33-L879C1

Merge pull request https://github.com/marrlab/DomainLab/pull/875 from marrlab/coral
smilesun
smilesun
committed
1 minute ago
·
Verified


[BUG] follows not registering as events

Describe the bug
When a channel receives a follow, there is no event being received in TAU.
To Reproduce
Steps to reproduce the behavior:

Auth TAU for channel.follow permission
Click the follow button on the Twitch profile (using another account that isn't following currently)
See nothing appear in TAU

Expected behavior
A channel-follow event should be registered in TAU.
Screenshots
n/a
Deployment:
Deployed on Railway. Forked repository is fully up to date with TAU/main.
Additional context
Test follows work fine. I have a channel goal for follows and that one is updating too. So it's literally the channel.follow endpoint causing some kind of issue. Too deep in there for me to figure out, but i see that some stuff is required for the v2 that might not have been for v1.
Was updated in September with this PR: https://github.com/Team-TAU/tau/commit/2d1c3cb0eecca531ba42664c751e1903fd6f8cf6

@cgsdev0 do you have any idea why your PR might not be working? I tried debugging more but am lost :/

i had to like, reset all the webhooks and re-register them in the TAU dashboard i think

@cgsdev0 oh i see! in the TAU webhooks monitor I don't even know how to reset these, as clicking these 3 buttons on the bottom doesn't change anything. is there a guide somewhere on this?

ok i thiiink i just toggled the channel.follow one on and off a bunch of times and clicked the update button a bunch
i really don't remember :S

@cgsdev0 you were right. I forcefully click around in my local TAU instance and then pushed it to my server-hosted instance on Railway and it magically started working again 🎉 I'll go ahead and close out this ticket, as it just seems to be an initialisation problem...


Support Postgis?

Hello, I tried to run the project with the Postgis backend but this error occurs
django.core.exceptions.ImproperlyConfigured: 'django_postgrespool2.postgis' isn't an available database backend.        | Try using 'django.db.backends.XXX', where XXX is one of:        |     'mysql', 'oracle', 'postgresql', 'sqlite3'        | Error was: No module named postgis.base
any solution?

@lzambrano18 postgis backend will be available in release 0.2.0. I think I will make it in 2-3 days. Right now you can try to use master branch, it should works:
$ pip install git+git://github.com/lcd1232/django-postgrespool2.git@master

Hello @lcd1232, when do you think the release with postgis support will be available?

@matthewh it is available since version 0.2.0
$ pip install "django_postgrespool2>=0.2.0"

the package you pushed to pypi as 0.2.0 does not contain the postgis code.

@lcd1232 is this issue still open?

No, PostGIS is available at pypi


Migrate pattern library to Fractal

Is your feature request related to a problem? Please describe.
So far we have been using Pattern Lab to generate our pattern library website. While it has served us well, we are facing some issues and limitations:

The UI of the current 3.x alphas/betas is buggy ("view all" links don't always work, preview iframe doesn't resize its height when you navigate to other patterns, etc.). While the PL community is very active, it's not clear how soon these issues will be fixed. Also, we don't want to downgrade back to 2.x as that's not getting updates any more
The latest version of the Nunjucks pattern engine no longer seems to produce lineage links for {% import ... %} and {% extend ... %}. Since we use those widely, a lot of our lineage links have therefore disappeared :-(

Fortunately, an alternative exists: Fractal. It offers pretty much all the same features PL has, but doesn't suffer from the 2 issues described above. While Fractal's continued development was uncertain a few months ago, a community has since rallied behind it and work on it is active once more.
Furthermore, Fractal has some features that PL does not which are useful to us:

Ability to use different preview layouts per component. E.g.: We could use this to have a preview that overrides the usual flex on body stuff for individual components, and another which does not for templates and pages
Grouping component files into folders. It's nice to have the data, docs and template altogether as a unit. This also makes it easier if we want to reorder or reorganise components, since we only need to change the directory name
We can use Fractals API to make a customised component template export function (they have an example and I have done something very similar in a personal project, so can confirm it works). This will help pave the way to publishing the Nunjucks templates as an NPM package for other projects to consume
Ability to use JS files to provide context data. We could use this to auto-generate dummy content or fetch realistic content from some APIs. Saves writing everything out as JSON manually!

Based on some discussions and experiments, the Gravity maintainers have therefore decided to migrate to Fractal.
Describe the solution you'd like

Remove Pattern Lab dependencies
Add Fractal dependencies
Rename and update templates and data files to work with Fractal

Related to #106

Closing this as it's been done by PR #262


InvalidArgumentError (see above for traceback): No OpKernel was registered to support Op 'RoiPoolGrad' with these attrs.  Registered devices: [CPU], Registered kernels:

/home/chen/anaconda3/envs/tensorflow/lib/python3.5/site-packages/tensorflow/python/ops/gradients_impl.py:100: UserWarning: Converting sparse IndexedSlices to a dense Tensor of unknown shape. This may consume a large amount of memory.
"Converting sparse IndexedSlices to a dense Tensor of unknown shape. "
Traceback (most recent call last):
File "/home/chen/anaconda3/envs/tensorflow/lib/python3.5/site-packages/tensorflow/python/client/session.py", line 1322, in _do_call
return fn(*args)
File "/home/chen/anaconda3/envs/tensorflow/lib/python3.5/site-packages/tensorflow/python/client/session.py", line 1305, in _run_fn
self._extend_graph()
File "/home/chen/anaconda3/envs/tensorflow/lib/python3.5/site-packages/tensorflow/python/client/session.py", line 1340, in _extend_graph
tf_session.ExtendSession(self._session)
tensorflow.python.framework.errors_impl.InvalidArgumentError: No OpKernel was registered to support Op 'RoiPoolGrad' with these attrs.  Registered devices: [CPU], Registered kernels:
device='GPU'; T in [DT_FLOAT]
 [[Node: gradients/pool_5_grad/RoiPoolGrad = RoiPoolGrad[T=DT_FLOAT, pooled_height=7, pooled_width=7, spatial_scale=0.0625](conv5_3/Relu, roi-data/rois, pool_5:1, gradients/fc6/transpose_grad/transpose)]]

During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "./faster_rcnn/train_net.py", line 109, in
restore=bool(int(args.restore)))
File "./faster_rcnn/../lib/fast_rcnn/train.py", line 401, in train_net
sw.train_model(sess, max_iters, restore=restore)
File "./faster_rcnn/../lib/fast_rcnn/train.py", line 149, in train_model
sess.run(tf.global_variables_initializer())
File "/home/chen/anaconda3/envs/tensorflow/lib/python3.5/site-packages/tensorflow/python/client/session.py", line 900, in run
run_metadata_ptr)
File "/home/chen/anaconda3/envs/tensorflow/lib/python3.5/site-packages/tensorflow/python/client/session.py", line 1135, in _run
feed_dict_tensor, options, run_metadata)
File "/home/chen/anaconda3/envs/tensorflow/lib/python3.5/site-packages/tensorflow/python/client/session.py", line 1316, in _do_run
run_metadata)
File "/home/chen/anaconda3/envs/tensorflow/lib/python3.5/site-packages/tensorflow/python/client/session.py", line 1335, in _do_call
raise type(e)(node_def, op, message)
tensorflow.python.framework.errors_impl.InvalidArgumentError: No OpKernel was registered to support Op 'RoiPoolGrad' with these attrs.  Registered devices: [CPU], Registered kernels:
device='GPU'; T in [DT_FLOAT]
 [[Node: gradients/pool_5_grad/RoiPoolGrad = RoiPoolGrad[T=DT_FLOAT, pooled_height=7, pooled_width=7, spatial_scale=0.0625](conv5_3/Relu, roi-data/rois, pool_5:1, gradients/fc6/transpose_grad/transpose)]]

Caused by op 'gradients/pool_5_grad/RoiPoolGrad', defined at:
File "./faster_rcnn/train_net.py", line 109, in
restore=bool(int(args.restore)))
File "./faster_rcnn/../lib/fast_rcnn/train.py", line 401, in train_net
sw.train_model(sess, max_iters, restore=restore)
File "./faster_rcnn/../lib/fast_rcnn/train.py", line 143, in train_model
grads, norm = tf.clip_by_global_norm(tf.gradients(loss, tvars), 10.0)
File "/home/chen/anaconda3/envs/tensorflow/lib/python3.5/site-packages/tensorflow/python/ops/gradients_impl.py", line 532, in gradients
gate_gradients, aggregation_method, stop_gradients)
File "/home/chen/anaconda3/envs/tensorflow/lib/python3.5/site-packages/tensorflow/python/ops/gradients_impl.py", line 701, in _GradientsHelper
lambda: grad_fn(op, *out_grads))
File "/home/chen/anaconda3/envs/tensorflow/lib/python3.5/site-packages/tensorflow/python/ops/gradients_impl.py", line 396, in _MaybeCompile
return grad_fn()  # Exit early
File "/home/chen/anaconda3/envs/tensorflow/lib/python3.5/site-packages/tensorflow/python/ops/gradients_impl.py", line 701, in
lambda: grad_fn(op, *out_grads))
File "./faster_rcnn/../lib/roi_pooling_layer/roi_pooling_op_grad.py", line 23, in _roi_pool_grad
data_grad = roi_pooling_op.roi_pool_grad(data, rois, argmax, grad, pooled_height, pooled_width, spatial_scale)
File "", line 139, in roi_pool_grad
File "/home/chen/anaconda3/envs/tensorflow/lib/python3.5/site-packages/tensorflow/python/framework/op_def_library.py", line 787, in _apply_op_helper
op_def=op_def)
File "/home/chen/anaconda3/envs/tensorflow/lib/python3.5/site-packages/tensorflow/python/framework/ops.py", line 3414, in create_op
op_def=op_def)
File "/home/chen/anaconda3/envs/tensorflow/lib/python3.5/site-packages/tensorflow/python/framework/ops.py", line 1740, in init
self._traceback = self._graph._extract_stack()  # pylint: disable=protected-access
...which was originally created as op 'pool_5', defined at:
File "./faster_rcnn/train_net.py", line 101, in
network = get_network(args.network_name)
File "./faster_rcnn/../lib/networks/factory.py", line 29, in get_network
return VGGnet_train()
File "./faster_rcnn/../lib/networks/VGGnet_train.py", line 17, in init
self.setup()
File "./faster_rcnn/../lib/networks/VGGnet_train.py", line 84, in setup
.roi_pool(7, 7, 1.0/16, name='pool_5')
File "./faster_rcnn/../lib/networks/network.py", line 36, in layer_decorated
layer_output = op(self, layer_input, *args, **kwargs)
File "./faster_rcnn/../lib/networks/network.py", line 235, in roi_pool
name=name)[0]
File "", line 57, in roi_pool
File "/home/chen/anaconda3/envs/tensorflow/lib/python3.5/site-packages/tensorflow/python/framework/op_def_library.py", line 787, in _apply_op_helper
op_def=op_def)
File "/home/chen/anaconda3/envs/tensorflow/lib/python3.5/site-packages/tensorflow/python/framework/ops.py", line 3414, in create_op
op_def=op_def)
File "/home/chen/anaconda3/envs/tensorflow/lib/python3.5/site-packages/tensorflow/python/framework/ops.py", line 1740, in init
self._traceback = self._graph._extract_stack()  # pylint: disable=protected-access
InvalidArgumentError (see above for traceback): No OpKernel was registered to support Op 'RoiPoolGrad' with these attrs.  Registered devices: [CPU], Registered kernels:
device='GPU'; T in [DT_FLOAT]
 [[Node: gradients/pool_5_grad/RoiPoolGrad = RoiPoolGrad[T=DT_FLOAT, pooled_height=7, pooled_width=7, spatial_scale=0.0625](conv5_3/Relu, roi-data/rois, pool_5:1, gradients/fc6/transpose_grad/transpose)]]

when i train the TFFRCNN ,there have a bug.
what should i do now?
thanks!

Please fill the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Along with the template, please provide as many details as possible to find the root cause of the issue. It would be great if you can provide a small code to reproduce the error. Thanks!

Closing due to lack of recent activity. Please update the issue when new information becomes available, and we will open a new issue. Thanks!


GPU Support

Hello @wozeparrot , thanks for this amazing repo. This is a great learning experience.
How would I port this to GPU ? I understand that GPU is supported in TinyGrad, either through Cuda or OpenCL. What should I change to get this code working on GPU ?

To get it working on GPU all you should need to do is to run it with the environment variable GPU set to 1.
Like, GPU=1 python main.py gen
I do find that I don't get the best performance that way, probably need some more optimization.
For me, its faster if you run with llvm instead: LLVM=1 LLVMOPT=1 python main.py gen

Thanks ! Closing this issue.


Inconsistent behaviour with validate_assignment

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
Not sure if this is a bug as such, or more something that should be documented as an anti-pattern.
I'm setting up some nested models and am using model validators to ensure that the contents of one field is a subset of another. (+1 for model_validators, they're incredible!)
An issue I ran in to when using this is that using a += operator on a list type field will cause the revalidate_assignment to kick in, whilst using .append() or .extend() doesn't. I'm not sure if there is a good way to capture the change with the latter methods, but if not then highlighting that this is a pattern to avoid is worth doing!
Example Code
import pydantic
class TestModel(pydantic.BaseModel, validate_assignment = True):
    list_of_strings : list[str]

TestModelInstance = TestModel(list_of_strings=['a', 'b', 'c'])

TestModelInstance.list_of_strings.append(1)
TestModelInstance.list_of_strings.extend([2]) # Makes it here fine
TestModelInstance.list_of_strings += [3] # Here, not so much...

Python, Pydantic & OS Version
pydantic version: 2.1.1
   pydantic-core version: 2.4.0
     pydantic-core build: profile=release pgo=false mimalloc=true
            install path: /Users/matt/dev/elysia/gen3/.venv/lib/python3.10/site-packages/pydantic
          python version: 3.10.12 (main, Jul 28 2023, 18:34:01) [Clang 14.0.3 (clang-14<IP_ADDRESS>.1)]
                platform: macOS-13.5-arm64-arm-64bit
optional deps. installed: ['devtools', 'typing-extensions']

Selected Assignee: @dmontagu

This is (unfortunately) a known issue. I'll close this as a duplicate of https://github.com/pydantic/pydantic/issues/6597 please subscribe there for updates. But yes for now you'll either (1) have to avoid assigning or (2) use object.__setattr__.


Request offline map support for Canada

Is it possible to add Canada to the list of available offline maps. It should be relatively small compared to us-west for example.

will look into it.
thanks for the suggestions,
cheers

should be there


Removed aka.ms links

Docs Build status updates of commit 0e4d7e0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

biztalk/core/dynamic-send-port-handler-is-configurable.md
:white_check_mark:Succeeded
View

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


fix changed namespace SamplingPercentageEstimatorSettings

Fix Issue #727.
SamplingPercentageEstimatorSettings's namespace was changed

from: Microsoft.ApplicationInsights.WindowsServer.Channel.Implementation.
to: Microsoft.ApplicationInsights.WindowsServer.TelemetryChannel.Implementation

(Channel -> TelemetryChannel)
Fixing this by adding a proxy class to support former public api, but refer to new class.

[ ] I ran Unit Tests locally.

For significant contributions please make sure you have completed the following items:

[ ] Design discussion issue #
[ ] Changes in public surface reviewed
[ ] CHANGELOG.md updated with one line description of the fix, and a link to the original issue.
The PR will trigger build, unit test, and functional tests automatically. If your PR was submitted from fork - mention one of committers to initiate the build for you.

Thank you!


[BUG] Inconsistency in online_shapelet_distance

Describe the bug
raised in the old place, this seems like a genuine bug that will alter results. Thank you @margauxgeuzaine by proxy
https://github.com/sktime/sktime/issues/5480
"There is an inconsistency in sktime/transformations/panel/_shapelet_transform_numba.py, between line 24 where "std = (sum2 - mean * mean * length) / length" and line 56 where "std = math.sqrt((sums2[n] - mean * mean * length) / length)".
Because of that, some time series are treated differently than others regarding normalization. The former are actually divided by the variance, and the latter are divided by the standard deviation. The second way of doing is supposed to be the right one."
Steps/Code to reproduce the bug
No response
Expected results
add sqrt to stdv calculation, will alter reference results and will need to assess if it makes any sig difference overall *may make it better :)
Actual results
doesnt take sqrt
Versions
No response

It is a real problem indeed. I ended up with a mix of shapelets transforms which were actually computed quite differently from one another, therefore affecting the classification. Not sure it will improve the accuracy though cause your transforms will be less distinct afterwards… Anyways needs to be fixed.

welp, thats just been there forever apparently 🙂
If we go by the original, having the sqrt in the loop seems to be the error?
Can test both I guess.


Update the binary ground truth segmentation to make it soft

See: https://github.com/sct-pipeline/contrast-agnostic-softseg-spinalcord/issues/84#issuecomment-1790130848
Before doing this, I would like to go over the existing ground truths: https://github.com/sct-pipeline/fmri-segmentation/issues/22 #13 #25

For now, we will train a model using hard segmentations, and we will go back to this issue at a later stage.


Bug: TypeError cannot pickle

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
Hello! I would like to create a class with one private attribute, that is a Signal from PySide6's library. The issue is, because this type cannot be pickled, Pydantic will just raise an error.
I could not find any help on the internet to fix this issue. To me, it is not a problem if some fields cannot be pickled, as long as they are not part of the model. I did a dummy fix of re-implementing the Signal class, but I'd like to use PySide6's class for performance reasons.
Seems related to https://stackoverflow.com/questions/72759846/pydantic-cannot-pickle-private-attribute, but the proposed fix does not work. Also, it seems that they use v1, not v2.
See https://github.com/jeertmans/manim-slides/pull/250, especially the files diff.

Error traceback
ImportError while loading conftest '/export/home/eertmans/repositories/manim-slides/tests/conftest.py'.
tests/conftest.py:8: in
    from manim_slides.config import PresentationConfig
manim_slides/config.py:58: in
    class Keys(BaseModel):  # type: ignore[misc]
/home/eertmans/.cache/pypoetry/virtualenvs/manim-slides-M-PeeMP--py3.10/lib/python3.10/site-packages/pydantic/_internal/_model_construction.py:178: in __new__
    complete_model_class(
/home/eertmans/.cache/pypoetry/virtualenvs/manim-slides-M-PeeMP--py3.10/lib/python3.10/site-packages/pydantic/_internal/_model_construction.py:492: in complete_model_class
    '__signature__', generate_model_signature(cls.__init__, cls.model_fields, config_wrapper)
/home/eertmans/.cache/pypoetry/virtualenvs/manim-slides-M-PeeMP--py3.10/lib/python3.10/site-packages/pydantic/_internal/_model_construction.py:547: in generate_model_signature
    kwargs = {} if field.is_required() else {'default': field.get_default(call_default_factory=False)}
/home/eertmans/.cache/pypoetry/virtualenvs/manim-slides-M-PeeMP--py3.10/lib/python3.10/site-packages/pydantic/fields.py:501: in get_default
    return _utils.smart_deepcopy(self.default)
/home/eertmans/.cache/pypoetry/virtualenvs/manim-slides-M-PeeMP--py3.10/lib/python3.10/site-packages/pydantic/_internal/_utils.py:317: in smart_deepcopy
    return deepcopy(obj)  # slowest way when we actually might need a deepcopy
/usr/lib/python3.10/copy.py:153: in deepcopy
    y = copier(memo)
/home/eertmans/.cache/pypoetry/virtualenvs/manim-slides-M-PeeMP--py3.10/lib/python3.10/site-packages/pydantic/main.py:694: in __deepcopy__
    deepcopy({k: v for k, v in self.__pydantic_private__.items() if v is not PydanticUndefined}, memo=memo),
/usr/lib/python3.10/copy.py:146: in deepcopy
    y = copier(x, memo)
/usr/lib/python3.10/copy.py:231: in _deepcopy_dict
    y[deepcopy(key, memo)] = deepcopy(value, memo)
/usr/lib/python3.10/copy.py:161: in deepcopy
    rv = reductor(4)
E   TypeError: cannot pickle 'PySide6.QtCore.Signal' object

Example Code
from typing import Optional

from pydantic import (
    BaseModel,
    Field,
    PrivateAttr,
    PositiveInt,
)
from PySide6.QtCore import Signal

class Key(BaseModel):
    """Represents a list of key codes, with optionally a name."""

    ids: List[PositiveInt] = Field(unique=True)
    name: Optional[str] = None

    __signal: Signal = PrivateAttr(default_factory=Signal)

    @property
    def signal(self) -> Signal:
        return self.__signal

Python, Pydantic & OS Version
pydantic version: 2.2.1
        pydantic-core version: 2.6.1
          pydantic-core build: profile=release pgo=true
                 install path: /export/home/eertmans/.cache/pypoetry/virtualenvs/manim-slides-M-PeeMP--py3.10/lib/python3.10/site-packages/pydantic
               python version: 3.10.12 (main, Jun 11 2023, 05:26:28) [GCC 11.4.0]
                     platform: Linux-6.2.0-26-generic-x86_64-with-glibc2.35
     optional deps. installed: ['typing-extensions']

Selected Assignee: @davidhewitt

Well, maybe the code provided is not a MWE :'-)
That why I was pointing to the PR as well as the logs.
The validation seems to fail as soon as we nest Key inside another model:
from typing import Any, Callable, List, Optional

from pydantic import (
    BaseModel,
    Field,
    PositiveInt,
    PrivateAttr,
)
from PySide6.QtCore import Qt, Signal

Receiver = Callable[..., Any]

class Key(BaseModel):

    ids: List[PositiveInt] = Field(unique=True)
    name: Optional[str] = None

    __signal: Signal = PrivateAttr(default_factory=Signal)

    @property
    def signal(self) -> Signal:
        return self.__signal

class Keys(BaseModel):
    QUIT: Key = Key(ids=[Qt.Key_Q], name="QUIT")

@samuelcolvin I am sorry to ping back, but you closed the issue too soon...


Update install.sh.example

Install script was failing in ubuntu when PATH+=:$prefix/bin

@isserrano what version of ubuntu?  Also, how are you running the script?

14.04 Server
I ran this:
sudo ./install.sh
./install.sh: 16: ./install.sh: PATH+=:/usr/local/bin: not found
On Fri, 14 Aug 2015 at 17:47 ssgelm<EMAIL_ADDRESS>wrote:

@isserrano https://github.com/isserrano what version of ubuntu? Also,
how are you running the script?
—
Reply to this email directly or view it on GitHub
https://github.com/github/git-lfs/pull/586#issuecomment-131176479.

And you are running the install script unmodified?  It looks like the script is being executed with dash instead of bash, however the shebang specifies bash so I'm not sure why that would be happening.

I am using the script as it comes from the download.
And yes, with bash works, but not with dot.
On Fri, 14 Aug 2015 at 17:51 ssgelm<EMAIL_ADDRESS>wrote:

And you are running the install script unmodified? It looks like the
script is being executed with dash instead of bash, however the shebang
specifies bash so I'm not sure why that would be happening.
—
Reply to this email directly or view it on GitHub
https://github.com/github/git-lfs/pull/586#issuecomment-131178377.

If you run sudo env bash do you end up at a bash prompt?

Ah, I see the problem.  This is a bug in the currently released script that does not exist in master.  The currently shipping shebang is #!/bin/sh -eu which will indeed fail on ubuntu/debian.  This PR is not necessary for the current version in master.

ok, good to know , thanks!
On Fri, 14 Aug 2015 at 18:26 ssgelm<EMAIL_ADDRESS>wrote:

Ah, I see the problem. This is a bug in the currently released script that
does not exist in master. The currently shipping shebang is #!/bin/sh -eu
which will indeed fail on ubuntu/debian. This PR is not necessary for the
current version in master.
—
Reply to this email directly or view it on GitHub
https://github.com/github/git-lfs/pull/586#issuecomment-131187907.


My code is bad...HELP !

Issue Type: Feature Request
What happened? help with code
VS Code version: Code 1.65.2 (c722ca6c7eed3d7987c0d5c3df5c45f6b15e77d1, 2022-03-10T14:33:55.248Z)
OS version: Windows_NT x64 10.0.22000
Restricted Mode: No

from pixellib.instance import instance_segmentation #ImportError: numpy.core.multiarray failed to import ,numpy.core.multiarray failed to import
File "C:\Users\User\Desktop\My Programming\object_detection_in_an_image\my.py", line 1, in
from pixellib.instance import instance_segmentation
def object_detection_on_an_image():
segment_image = instance_segmentation()
segment_image.load_model(r"C:\Users\User\Desktop\My Programming\object_detection_in_an_image\mask_rcnn_coco.h5")
segment_image.segmentImage(
    image_path="istockphoto.jpg",
    output_image_name="output.jpg"
)

def main():
object_detection_on_an_image()

/question
This is issue tracker for VS Code editor itself that is not suitable to general questions.


9:16 videos in full screen are shown as 16:9, should be 9:16

See examples on this page: https://www.equinor.com/energy/techstars

Hi @millianapia, when working on the dependencies upgrade of hls.js  #1854 I had to do a small tweak in the video code. So I also ended up looking into this issue and am pushing a fix :) hope you don't mind.

@meols @NickHaggerty1, this is ready to be tested on global-dev

Nice. Approved @fernandolucchesi


Mistake with URL found in document: Update todoTask

In the document: Update todoTask
There is a mistake in the HTTP request section (Line 2):
PATCH /me/todo/lists/{todoTaskListId}
PATCH /users/{id|userPrincipalName}/todo/lists/{todoTaskListId}/tasks

It should have been:
PATCH /me/todo/lists/{todoTaskListId}
PATCH /users/{id|userPrincipalName}/todo/lists/{todoTaskListId}

There should not be /tasks as a postfix, since there is not in any other sections on this page and the URL with this postfix brings an invalidRequest error.

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

ID: 8a138d12-c87e-1bb5-111d-55a403b9a1fa
Version Independent ID: 25f74f9a-572e-6acb-21b1-ac6259a9d914
Content: Update todoTaskList - Microsoft Graph v1.0
Content Source: api-reference/v1.0/api/todotasklist-update.md
Product: outlook
Technology: microsoft-graph
GitHub Login: @avijityadav
Microsoft Alias: MSGraphDocsVteam

PR https://github.com/microsoftgraph/microsoft-graph-docs/pull/23893 fixes this issue.

Alright, please close this issue once the document webpage is updated.