Update ng submodule to latest version

Task/Issue URL: https://app.asana.com/0/488551667048375/1203126514402198/f
Description
Update NG submodule. See asana task to see changes
Steps to test this PR
AppTP smoke tests

[ ] install from this branch
[ ] launch app and enable AppTP
[ ] open speed test app
[ ] verify AppTP blocks tracking attempts

Current dependencies on/for this PR:

develop

PR #2387   👈

This comment was auto-generated by Graphite.


Fix wrong index marking

The index was wrong set in docs, so let's fix it
Signed-off-by: Kai Qiang Wu(Kennan<EMAIL_ADDRESS>

ping @thaJeztah

built locally and LGTM, thanks!


customize username and avatar

Describe the feature
I hope we can customize username and change avatar
Environment

Postcat Version [0.5.0]
Windows 10

@ritchie2000 Thanks for the suggestion


Snippets section is added to package.json

Fixes #54.
contributes/snippets section is added to package.json to enable included snippets. Also, variables in the snippets are replaced with placeholder to avoid VS Code warning. See: https://code.visualstudio.com/docs/editor/userdefinedsnippets#_creating-your-own-snippets

Coverage remained the same at 38.462% when pulling 70de680c30c7ffac3693318e4898121f8526e83f on nikitozz:issue-54-snippets into 98d4a7ef96f245d23e5108f3995e0260a39093ed on k--kato:master.


Add a clean target

A phony "clean" which removes everything would be very useful to test issues where the build system is likely reporting an old issue and needs a wipe.

Just linking #32 here. Ha. Almost 100 issues in-between.

100 issues or 15 days - you guys are moving fast!

For info, if you delete shake-files/.db then Shake considers everything dirty and tries rebuilding it, which is a reasonably good way to test the build system.

100 issues or 15 days - you guys are moving fast!
:100:

For info, if you delete shake-files/.db then Shake considers everything dirty and tries rebuilding it, which is a reasonably good way to test the build system.

I'm running
$ make clean && make distclean && make maintainer-clean && git clean -df && ./boot && ./configure && rm -fR shake-build/.db && rm -fR shake-build/.shake

to be on the safe side :)

Can we add that to the readme, or as a target "superclean" - agreed it's probably too paranoid, but useful to know.

As soon as we handle #113 this issue should be straightforward to implement.

@ndmitchell are you up to the phony "clean" Pull Request challenge? :-)

I'm not totally sure what the phony should do. I guess rm .build, and shake-build/.shake and the include files that were floating around?

@ndmitchell And all stuff we put into inplace.
It's going to be rather tedious to implement and hard to maintain. An ideal solution would be to eventually move everything into .build.

Shouldn't it be about 4 or 5 directory names which don't change that often?

With includes it's not that simple: there are some source files which we'd like to keep.
However, directories inplace/bin and inplace/lib seem to contain only built artefacts.

:( - that does seem a bit painful. But I don't think having clean is optional, sounds like it might be one for @snowleopard though.

@ndmitchell Agreed, I'll do this after #113.

As a note, you generally only need to use removeFilesAfter if you are specifically removing the .shake.database file, which is otherwise locked open. For everything else it's better to use liftIO $ removeFiles.

@ndmitchell Thanks, I will fix this. Do we want to remove the Shake database too when doing clean?

I see no reason not to remove the shake database - it gives you the best chance of reproducing issues from scratch, which is a major goal of doing clean.

Done, I agree.

Thanks for this, I was waiting for a clean target before I conducted any more testing, so I'll give my Windows/Stack build a try tonight.

I hope I haven't missed anything. If you spot any left-over files while testing please let me know.

@snowleopard - why not just test it properly on the CI machines? At the end of the build do clean, and then check something like git diff --exit-code, perhaps ignoring certain files, perhaps not applying the .gitignore. If that list is "longer than you expect", then error out. For info, I do this on all my builds, especially to check that those libraries with generators have had the generators applied and checked in (I have that pattern in derive, filepath and extra).

@ndmitchell Good idea! I'm a bit concerned that this might increase CI time; we are already close to AppVeyor time limit, where we are at the very edge even when building GHC stage1 only. I don't want to lower the bar any further (below GHC stage1). Can you spot any optimisation opportunities?
Of course, we don't have to run this check on all CI instances.
Another idea: could we ingrate the proposed check right into the clean rule? Then it will become self-checking.

I wouldn't put the check into clean - users might (probably will) have locally modified files, or other stuff they have put somewhere else in the directory, and we don't want to report that as an error. Adding a cleancheck target that does it (just so the code can be properly integrated) which itself depends on clean, isn't a bad idea.
I suspect the clean and check will take ~10s at most - it should not be an expensive thing to do (if it is, let me know). I think the best thing is to find optimisation opportunities in the rest of the build, configure in parallel being the most obvious one I saw. Plus you can only run this on Travis, not Appveyor.

So, let me reopen this issue as it requires further work.

@ndmitchell You seem to be very familiar with the solution you are proposing, perhaps, you could take this over? Here are the tasks we have:

[ ] Add a clean check on Travis
[ ] Implement a cleancheck rule (optional)

Certainly, with the caveat that it will be about 1-2 weeks before I do it (assuming I don't get time tonight). The remaining part is unimportant though so can remilestone til later.

Thanks @ndmitchell! Take your time. I don't think this is on the critical path.


Should CiceroMark allow import of Clauses

From a syntax perspective, focused on the template grammar, here is a proposition for how to import a  Clause in a template, inspired by https://handlebars-draft.knappi.org/guide/#partials
Payment
-------
{{>payment<EMAIL_ADDRESS>

Issue inherited from #392

Also related to Ergo issue https://github.com/accordproject/ergo/issues/202


access instance in attributes blocks

not a bug it's more of a question.
Sorry if dup I can't find anything on SO nor here
short
Is there a reason we can't access the instance itself into an "attribute block" ?

long
lets have a parent model and a child model. Parent.has_one(:child)
In a callback (such as after create), we have access to the created instance as following :
FactoryBot.define do
  factory :childs do
    # ...
    after(:create) do |newly_created_child_instance, _params|
      # ...
    end
  end
end

But when declaring an attribute such as a_parent_model, the block yields the current factory object (with a class class ...). Which expose the instance in write_only.
So I ended up using factory_object.instance_variable_get(:instance) to get it ... :
FactoryBot.define do
  factory :childs do
    parent_model do |factory_object|
      association(
        :parent_model,
        child: factory_object.instance_variable_get(:@instance)
      )
    end
  end
end

I have to do this because :

my parent model has_one child
my parent model have an after_create callback where he performs a  Child.find_or_create_by(parent: self),
so I can't just declare  parent { association :parent } in my factory, because I ended up with 2 child childs (the one created in the parent callback, and the one created by the factory)
I don't want to simply do a

# my spec
let(:child_instance) { parent_instance.child)
let(:parent_instance) { create(:parent) }

because it gets funkyer when using some custom params or trait on child. I have to perform some update instead of just
# spec
let(:child_instance) { create(:child, :some_traits_and_attributes, ...) }
let(:parent_instance) { child_instance.parent }

Hope it's clear enought ... 😞

hy @composerinteralia
The question here was more about "why".
ATM, I found a way through with a simple monkey patch, but If it ment to be a feature, I can give a try and make a PR

I'm having trouble understanding your precise use case here. Would you be able to provide a runnable example, preferably in this format: https://github.com/thoughtbot/factory_bot/blob/master/.github/REPRODUCTION_SCRIPT.rb?

I noticed that https://github.com/thoughtbot/factory_bot/issues/1309 also mentions making a public reader for @instance. Is your use case similar to the one mentioned there?

Yes it seems the same kind of issue.


Update twitcasting extractor

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[x] Bug fix
[x] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
Fixes the twitcasting extractor (see #24868) and adds support for treating the "Live History" tab (all the videos from a user) as a playlist. Note that this change doesn't fix live videos as described in the issue.

And it broke #24868

The ffmpeg issues related to this PR appear to be fixed so it's alive again. The native HLS can't download the files correctly. I fixed that but I assume that should be a different PR. Is there some way to prevent the native downloader from being used by default for this extractor since it produces an unplayable file?

The issues with the extractor appear to have been addressed with in https://github.com/ytdl-org/youtube-dl/commit/5ff881aee6532a2f588ab4b3424c72159f1d8e63 .
I'll pull those changes in and re-open this with just the history extractor.


Memory leak in confuse.c when parse error

System info
Ubuntu X64, gcc (Ubuntu 5.5.0-12ubuntu1), cfgtest (latest master a42aeb)
Configure
CFLAGS="-g -fsanitize=address" LDFLAGS="-fsanitize=address" ./configure
Command line
./examples/cfgtest ./memory-leak-confuse-1820
AddressSanitizer output
=================================================================
==73357==ERROR: LeakSanitizer: detected memory leaks

Direct leak of 80 byte(s) in 1 object(s) allocated from:
    #0 0x4da688 in calloc (/home/seviezhou/libconfuse/examples/cfgtest+0x4da688)
    #1 0x52329a in cfg_init /home/seviezhou/libconfuse/src/confuse.c:1820:8

Indirect leak of 3168 byte(s) in 3 object(s) allocated from:
    #0 0x4da688 in calloc (/home/seviezhou/libconfuse/examples/cfgtest+0x4da688)
    #1 0x51c8ae in cfg_dupopt_array /home/seviezhou/libconfuse/src/confuse.c:670:12

Indirect leak of 154 byte(s) in 19 object(s) allocated from:
    #0 0x436c80 in strdup (/home/seviezhou/libconfuse/examples/cfgtest+0x436c80)
    #1 0x51ca80 in cfg_dupopt_array /home/seviezhou/libconfuse/src/confuse.c:686:21

Indirect leak of 72 byte(s) in 9 object(s) allocated from:
    #0 0x4da688 in calloc (/home/seviezhou/libconfuse/examples/cfgtest+0x4da688)
    #1 0x51c121 in cfg_addval /home/seviezhou/libconfuse/src/confuse.c:613:30

Indirect leak of 72 byte(s) in 5 object(s) allocated from:
    #0 0x4da8e0 in realloc (/home/seviezhou/libconfuse/examples/cfgtest+0x4da8e0)
    #1 0x51c099 in cfg_addval /home/seviezhou/libconfuse/src/confuse.c:608:8

Indirect leak of 63 byte(s) in 3 object(s) allocated from:
    #0 0x436c80 in strdup (/home/seviezhou/libconfuse/examples/cfgtest+0x436c80)
    #1 0x51cb92 in cfg_dupopt_array /home/seviezhou/libconfuse/src/confuse.c:697:28

Indirect leak of 14 byte(s) in 1 object(s) allocated from:
    #0 0x436c80 in strdup (/home/seviezhou/libconfuse/examples/cfgtest+0x436c80)
    #1 0x51ed9e in cfg_tilde_expand /home/seviezhou/libconfuse/src/confuse.c:1893:14
    #2 0x7f4c1305db96 in __libc_start_main /build/glibc-OTsEL5/glibc-2.27/csu/../csu/libc-start.c:310

Indirect leak of 5 byte(s) in 1 object(s) allocated from:
    #0 0x436c80 in strdup (/home/seviezhou/libconfuse/examples/cfgtest+0x436c80)
    #1 0x524fd4 in cfg_opt_setnstr /home/seviezhou/libconfuse/src/confuse.c:2185:12

Indirect leak of 5 byte(s) in 1 object(s) allocated from:
    #0 0x436c80 in strdup (/home/seviezhou/libconfuse/examples/cfgtest+0x436c80)
    #1 0x51cc1d in cfg_dupopt_array /home/seviezhou/libconfuse/src/confuse.c:703:28

Indirect leak of 5 byte(s) in 1 object(s) allocated from:
    #0 0x436c80 in strdup (/home/seviezhou/libconfuse/examples/cfgtest+0x436c80)
    #1 0x5232db in cfg_init /home/seviezhou/libconfuse/src/confuse.c:1824:14

SUMMARY: AddressSanitizer: 3638 byte(s) leaked in 44 allocation(s).

##POC
memory-leak-confuse-1820.zip

Thank you for the report.  This is a bug in an example program that comes with libConfuse, not a bug in libConfuse itself.  It stems from cfgtest not calling cfg_free() on error conditions, e.g. an invalid config file in this case.


fix: Remove forSale default filter for gene artworks

This PR resolves ONYX-1436
Description
Remove forSale default filter for gene artworks.
PR Checklist

[ ] I have tested my changes on iOS and Android.
[ ] I hid my changes behind a feature flag, or they don't need one.
[ ] I have included screenshots or videos, or I have not changed the UI.
[ ] I have added tests, or my changes don't require any.
[ ] I added an app state migration, or my changes do not require one.
[ ] I have documented any follow-up work that this PR will require, or it does not require any.
[ ] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

Warnings

:warning:

❌ An error occurred while validating your changelog, please make sure you provided a valid changelog.

  Generated by :no_entry_sign: dangerJS against b6e54ad7520ca68a325048fe3594bb86dde578ae


Support debugging flutter drive integration tests from the editors

Hi,
I can't find any way debugging integration tests run with flutter drive.
in flutter site it's only say to use Observatory for debug, but not explain how to do this.
https://flutter.io/docs/testing/debugging
i also tried to connect to link of listening to Observatory like show in screenshot i attache

, but i couldn't see my code and debut it.
please help me.
thanks

Please add the output of flutter doctor -v.
What IDE are you using?

i am sorry for my late respond. i thought i suppose to get an email when you respond and i didn't check here.
i am using vscode IDE.
output of flutter doctor -v
PS C:\Users\p0026329\git\flutter\digihaler-flutter6\packages\digihaler> flutter doctor -v
[√] Flutter (Channel master, v0.11.4-pre.6, on Microsoft Windows [Version 10.0.17134.228], locale en-US)
• Flutter version 0.11.4-pre.6 at C:\flutter
• Framework revision df57dc9da9 (3 weeks ago), 2018-11-11 22:36:47 -0800
• Engine revision 5646e86a6f
• Dart version 2.1.0 (build 2.1.0-dev.9.3 9c07fb64c4)
[√] Android toolchain - develop for Android devices (Android SDK 28.0.3)
• Android SDK at C:\Users\p0026329\AppData\Local\Android\sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-28, build-tools 28.0.3
• Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b02)
• All Android licenses accepted.
[√] Android Studio (version 3.1)
• Android Studio at C:\Program Files\Android\Android Studio
• Flutter plugin version 29.0.1
• Dart plugin version 173.4700
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b02)
[√] VS Code (version 1.29.1)
• VS Code at C:\Users\p0026329\AppData\Local\Programs\Microsoft VS Code
• Flutter extension version 2.21.0
[√] Connected device (1 available)
• ONEPLUS A5010 • 94cbebe4 • android-arm64 • Android 7.1.1 (API 25)
• No issues found!
PS C:\Users\p0026329\git\flutter\digihaler-flutter6\packages\digihaler>
thanks

Right now, there's no built-in support in the editors for debugging flutter drive (though the post linked by @mmcc007 seems like a reasonable workaround).
I think it's a valid request, so I'll rename this for people to out 👍's on. I think we'd need to extend flutter drive to have a --machine flag that talks JSON like flutter run/flutter test that provides us with the Observatory port so we can attach directly.

I want to debug integration tests, too.

@xuexin The following article should show how to debug integration tests:
https://medium.com/flutter-community/hot-reload-for-flutter-integration-tests-e0478b63bd54

@mmcc007 Great job!
But that way seems too cumbersome for debug at later version of Flutter.
(Flutter 1.5.4-hotfix.2 • channel beta)

Now working in VSCode: https://github.com/Dart-Code/Dart-Code/issues/2609#issuecomment-654830800

@mmcc007 Hello, I am trying follow your article; however, it is nor working at all for me. Could you please give same tip/clue. Taht the erro I am getting:
"C:\flutter\bin\cache\dart-sdk\bin\dart.exe --enable-asserts --enable-vm-service:4627 C:\Users\SERVIDOR\Projects\flutter_reversi-master\test_driver\integr_ui_view_test.dart
Observatory listening on http://<IP_ADDRESS>:4627/QN_QqTMZHEI=/
00:00 +0: Group CI-CD-01: Integration UI/VIEW Test - View.Dart (setUpAll)
VMServiceFlutterDriver: Connecting to Flutter application at http://<IP_ADDRESS>:8888/
VMServiceFlutterDriver: Isolate found with number:<PHONE_NUMBER>978919
VMServiceFlutterDriver: Isolate is not paused. Assuming application is ready.
VMServiceFlutterDriver: Flutter Driver extension is taking a long time to become available. Ensure your test app (often "lib/main.dart") imports "package:flutter_driver/driver_extension.dart" and calls enableFlutterDriverExtension() as the first call in main().
"
AS you can see, I am using 'enableFlutterDriverExtension', but it is not working at all for me.
void main() {
  enableFlutterDriverExtension();
  mainApp.main();
}

Thx a lot

Hello @Hixie
The answer to this issue is here https://github.com/Dart-Code/Dart-Code/issues/2609#issuecomment-654830800
can you please a look here?
Thanks

FWIW there have been improvements to how integration tests work in Flutter (see https://github.com/flutter/flutter/pull/76200) so the functionality mentioned in the Dart-Code issue linked above will soon be somewhat superseded with https://github.com/Dart-Code/Dart-Code/pull/3133 (which will make integration tests work much more like standard tests without special handling in the editor).


Support WebGL without stencil

Expected Behavior
Browser/device configurations that support WebGL but don’t support stencil should use fallbacks for features such as masking, that the framework is currently relying on stencil to perform.
Current Behavior
Configurations without stencil support will fail with a runtime error saying that WebGL isn’t supported and to use the legacy branch instead.
Possible Solution
Masking can be performed in other ways in WebGL. Some explanations of two alternate methods can be found on this stackoverflow thread
Environment

**pixi.js @latest

I agree, and I think I know fallbacks. I'm gonna prepare big stencil-related PR for 5.2.0.

I also agree, using fallbacks for when we don't have stencil rather than just giving up on WebGL entirely would be awesome.

@ivanpopelyshev addressed this in his two recent PRs for stencils. closing.

@bigtimebuddy @ivanpopelyshev when this feature will be available?
I Still get "WebGL renderer unsupported in this browser" with Chrome  on PixiJS 5.3.0

I also have this problem with Chrome Version 85.0.4183.83 (Official Build) (64-bit) on MacOS 10.12.6.
What fails is the call to gl.getContextAttributes() returning null inside isWebGLSupported


Getting this Issue on IOS

Unhandled Exception: MissingPluginException(No implementation found for method scan_bytes on channel qr_scan)

Seems like a duplicate of #19

scanner doesn't work on iOS. pls check....
No implementation found for method scan on channel qr_scan

same problem here, flutter clean didnt solve the issue
(ios 14.1 and flutter 1.22. )


In patterns, file:file_name should be file:name

yeah - I'm closing #70 as a duplicate

part of pull request #68


Deleting columns in sqlite... causes sql syntax error

I just installed dbeaver and created an sqlite database to have a play with it. I created some columns and wanted to delete one, but it fails:

Hi @katsar0v ,
SQLite does not support ALTER TABLE DROP COLUMN statement. You could find how to do it here
BR

Then why did dbeaver allow me to delete columns, and if possible, why did the wrong syntax was written?

Feature request to solve this is already created #5447

Ok, closed an issue because of "solved" or "won't solve"? This is definitely a bug and unwanted behavior in dbeaver

@katsar0v issue is closed as a duplicate of already existing feature request. Drop Column behaviour will be implemented in scope of #5447


OpenSSL3 compatibility ?

Can anybody help to take a look at this???
https://github.com/Azure/azure-c-shared-utility/issues/601
Thank you ...

Either don't use deprecated functions or don't error when using them.

@jiapei100, that was a very snarky answer. Please be helpful in your future posts.
Define OPENSSL_NO_DEPRECATED instead. The macro OPENSSL_NO_DEPRECATED_3_0 is for internal use.


Delete cloudformation directory

Per conversation with @shubham22, we are deprecating this directory.

@shubham22 granted me privileges.


Not working

Maybe, I'm just too stupid, but I can't make it work.
Installed the plugin via
yarn add strapi-plugin-slugify

Added it to the configuration:
    slugify: {
      enabled: true,
      config: {
        contentTypes: {
          article: {
            field: 'slug',
            references: 'title',
          },
        },
      },
    },

and rebuilt it via
yarn build

In Strapi (4.1.1, node v16.14.0), I can see that the plugin is enabled, but I can't query anything:
{
  "errors": [
    {
      "message": "Cannot query field \"findSlug\" on type \"Query\".",
      "locations": [
        {
          "line": 2,
          "column": 3
        }
      ],
      "extensions": {
        "code": "GRAPHQL_VALIDATION_FAILED",
        "exception": {
          "stacktrace": [
            "GraphQLError: Cannot query field \"findSlug\" on type \"Query\".",
            "    at Object.Field (/node_modules/graphql/validation/rules/FieldsOnCorrectTypeRule.js:48:31)",
            "    at Object.enter (/node_modules/graphql/language/visitor.js:323:29)",
            "    at Object.enter (/node_modules/graphql/utilities/TypeInfo.js:370:25)",
            "    at visit (/node_modules/graphql/language/visitor.js:243:26)",
            "    at validate (/node_modules/graphql/validation/validate.js:69:24)",
            "    at validate (/node_modules/apollo-server-koa/node_modules/apollo-server-core/dist/requestPipeline.js:186:39)",
            "    at processGraphQLRequest (/node_modules/apollo-server-koa/node_modules/apollo-server-core/dist/requestPipeline.js:98:34)",
            "    at processTicksAndRejections (node:internal/process/task_queues:96:5)",
            "    at async processHTTPRequest (/node_modules/apollo-server-koa/node_modules/apollo-server-core/dist/runHttpQuery.js:187:30)",
            "    at async /node_modules/apollo-server-koa/dist/ApolloServer.js:82:59",
            "    at async bodyParser (/node_modules/koa-bodyparser/index.js:95:5)",
            "    at async cors (/node_modules/@koa/cors/index.js:56:32)",
            "    at async returnBodyMiddleware (/node_modules/@strapi/strapi/lib/services/server/compose-endpoint.js:52:18)",
            "    at async policiesMiddleware (/node_modules/@strapi/strapi/lib/services/server/policy.js:24:5)",
            "    at async /node_modules/@strapi/strapi/lib/middlewares/logger.js:22:5",
            "    at async /node_modules/@strapi/strapi/lib/middlewares/powered-by.js:16:5"
          ]
        }
      }
    }
  ]
}

What am I doing wrong?
Oh and yes, a (collection) type named article (resp. articles in singular) exists, and it has a title attribute.

Hello @SoftCreatR,
At this time the field must be created in the admin for it to work as you have discovered. Their is a note about this in the configuration section of the docs.
Regarding setting the field to read only or hiding it, I have not found a way to do this via a plugin. The only current way to limit is is via the Role settings in Settings -> Administration Panel -> Roles.

Their is a note about this in the configuration section of the docs.

I know. But I misunderstood that part. But it's all good now :)

Regarding setting the field to read only or hiding it, I have not found a way to do this via a plugin. The only current way to limit is is via the Role settings in Settings -> Administration Panel -> Roles.

What about a custom field (shipped by the plugin, or a separate plugin) that is read-only?

At the moment custom fields are not officially supported.
We could add a readonly field provided by the plugin but users would need to add it manually to the schema which is not ideal.
As the role settings are available for fields I would say we leave it for now and it can be discussed further once custom field support is released.
Once its released the contentTypes part of the settings can probably be removed altogether.


Slicer updates preparation

These patches allow building this toolkit with C++14, VTK 9, & ITK 5.3.

Are these tools still worth maintaining as a whole, or are some of the tools outdated?
FYI:  I am also updated https://github.com/NIRALUser/DTIProcessToolkit/pull/50 which will depend on this set of changes.

Thanks for updating these. We are indeed still using these regularly and though some are outdated and do not need to be maintained (from our end): 3D4DImageConversion, DTITensorNormalization, DWI_NIFTINRRDCONVERSION, ImageNoise, anything in Unsupported and one of the curve comparison (not sure which is right one of the 2). Thanks also for updating the dtiprocesstoolkit

@styner No problem.  Great tools need a little love once in a while :).   I have also updated all the other packages that this one depends upon and verified that they build.  After those other packages are merged, the hashes of those packages need to be updated for this repo as well.

@styner  NOTE:  This PR should work for all the internally contained source code.

@styner, ok to merge?

Yes, ok to merge

@bpaniagua A follow-up PR will be needed once the external code repos that this depends upon are also updated.


Board Meeting: Missing Agenda Packet

The 6/27/19 Regular Board Meeting page is missing the "Download Agenda and Attachments" link:

@shrayshray this has been resolved! https://boardagendas.metro.net/event/regular-board-meeting-ba05088b6d26/

for posterity: this was caused because of an error in the pdf merger for which we should have received an error notification, but didn't. we upgraded the sentry sdk to resolve the missing errors, and re-ran the merge for events to resolve the issue.

https://github.com/datamade/metro-pdf-merger/pull/20


The token '&&' is not a valid statement separator in this version.

Issue Type: Bug
Attempt to debug a python module containing "print("hello, world")" in with python 3.6 throws this error ( although it happens on anything and everything):
PS K:\new-\video\VideoRedo\vprj rename tool\vrprename> cd "k:\new-\video\VideoRedo\vprj rename tool\vrprename" && cmd /C "set "PYTHONIOENCODING=UTF-8" && set "PYTHONUNBUFFERED=1" && "k:\new-\video\VideoRedo\vprj rename tool\vrprename\venv\Scripts\python.exe" c:\Users\evana.vscode-insiders\extensions\ms-python.python-2019.11.50794\pythonFiles\ptvsd_launcher.py --default --client --host localhost --port 31327 "k:\new-\video\VideoRedo\vprj rename tool\vrprename\helloworld.py" "
At line:1 char:58

cd "k:\new-\video\VideoRedo\vprj rename tool\vrprename" && cmd /C "s ...

                                                     ~~

The token '&&' is not a valid statement separator in this version.
+ CategoryInfo          : ParserError: (:) [], ParentContainsErrorRecordException
+ FullyQualifiedErrorId : InvalidEndOfLine
Neither VSC 1.41.1 nor  VSCodium 1.41.1 do this, I have removed the path fragments for all three from the system path. Only VSC Inisder is running. There are no other relevant envars set,  It is probably worth mentioning that VSC Insider worked just fine  until it was updated. This happened several weeks ago and I failed to capture the relevant data. But having said that it occurrs to me that another approach would be to reinstall earlier versions until the problem goes away... or doesn't. I will try that tomorrow and let you know how it goes.
<EMAIL_ADDRESS>VS Code version: Code - Insiders 1.42.0-insider (7c0095ee2d064033fc13184127a9adc603454729, 2019-12-20T07:57:35.999Z)
OS version: Windows_NT x64 10.0.19536

System Info

Item
Value

CPUs
AMD FX(tm)-8120 Eight-Core Processor            (8 x 3110)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_offsurface_control: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.95GB (12.14GB free)

Process Argv

Screen Reader
no

VM
0%

Please report this issue against the python extension.

According to the VSC-Python crew:
@microsoft/pvsc-team this is an upstream bug in VS Code. We do not control the &&. The command is build by VS Code and sent to the terminal. We merely provide parts of the command such as cwd, env variables, and the like. Issue #84793
This is consistent with my evaluation and is the reason I didn't submit it as an extension bug, The fact that the same extension version works in both VSC 1.42.1 and VSCodium 1.41 would seem to argue against it being an extension bug... unless, of course, some documented change in VSC Insider could be employed to show that the extension has not been properly adjusted to work in VSC Insider.
<EMAIL_ADDRESS>

It seems you are using PowerShell but VS Code doesn't detect this and builds the command line for a regular command prompt ("cmd.exe").
Does it help if you set the terminal setting terminal.integrated.shell.windows to C:\\Windows\\System32\\WindowsPowerShell\\v1.0\\powershell.exe?

Yup. I have both Powershell and Powershell 6 installed and things run normally when they are selected as the default shell. The problem only occurs with cmd. I don't normally select Powershell as the default but I probably should have thought to try that.

Thanks, I'm currently investigating why VS Code doesn't see you default terminal.
But the workaround should work for you: just configure the terminal.integrated.shell.windows to reflect the default.

/duplicate #87700

@eascott since #87088 is fixed now it would be great if you could verify that your issue is now resolved.


multiple cpu cores

Hi,
Thank you very much for your guide and I finally got customised trained yolov3 tiny version working on my laptop. Also the yolov3 version is working at 4~5FPS. My pc shows 1 cpu is utilised at 100%, it is possible to allocate the computation to multiple cpus? I can tell the batch size is forced to 1, would it work in async with higher batch size and potentially increase over all inference speed when running a larger model like yolov3?

The performance of OpenVINO depends on the total number of CPU cores. Please see the benchmark article below. I can not try because I do not have the following high-performance CPU.
https://www.intel.ai/cpu-inference-performance-boost-openvino/#gs.c1wkmh

If you quantize a model, it may work even faster.
https://www.intel.ai/introducing-int8-quantization-for-fast-cpu-inference-using-openvino/#gs.c21087


Add Tabbing to Consumer Resources + Education Navs

This PR adds a "#" link as a way to remove the blank href on the first
two subnav items. Anchor links without a href are not keyboard
navigable.
This is the method being used by 18F
https://github.com/18F/web-design-standards/issues/743
Additions

Adds "#" and a check to make sure links to "#" don't link to non-existent overview pages.

Testing

Using your keyboard, navigate to the first an item in the Consumer Resources section. Quick guide here.

Review

@anselmbradford
@sebworks
@jimmynotjim

Checklist

[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the front end playbook
[ ] Passes all existing automated tests
[ ] New functions include new tests
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)

LGTM :+1:  After this goes in we can add overview links to Consumer Tools and Educational Resources too but disable link style them if the URL is #. That's something Jess asked about.

:+1:


Fix typo

Motivation

Solution

PR Checklist

[ ] Added Tests
[ ] Added Documentation
[ ] Updated the changelog

Codecov Report

Merging #29 (3895419) into master (dc8a3cd) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

@@           Coverage Diff           @@
##           master      #29   +/-   ##
=======================================
  Coverage   24.27%   24.27%
=======================================
  Files          26       26
  Lines        5553     5553
=======================================
  Hits         1348     1348
  Misses       4205     4205

Flag
Coverage Δ

test
24.27% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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


Proxies are editable under read-only mode (deployment source set)

Environment URL: functions-next.azure.com
Repro steps:

Create a function app and a proxy.
Set a deployment source.
Try create a function, verify page alert "read-only mode".
Try edit or create a proxy, verify the operations all succeeded.

Similar to #1163


How to migrate the artifacts registered with GTO v0.2.*

Assume we have a repo with artifacts registered with GTO v0.2.x (with file artifacts.yaml).
After upgrading GTO to v0.3.x one should change the artifacts.yaml to dvc.yaml and start using dvc.api (in Python) in order to get gto describe or gto annotate functionalities.
The question is, how to use new dvc.api with old artifacts, registered with GTO v0.2.x.
The problem is that dvc.api expects the file dvc.yaml in the repo.Index.
How to migrate all registered artifacts?
Shall we rebase all commits and re-register the artifacts again?
Any script for this process would be helpful.

Hi @rnoxy. Do you use your artifacts from CLI or from Studio? We made Studio support both formats (old and new), but for CLI I was assuming you either use old GTO CLI or new DVC API, not trying to do both at once.
There is this script that moves annotations from artifacts.yaml to dvc.yaml, but to move annotations for existing commits you'll need to rewrite git repo history, which will be a complex task.
If you need to work with both old and new formats, I guess the easiest option would be to use some kind of try...except construction, trying to use new format, and if annotation doesn't exist, fall back to the old annotation.
Hope this is helpful. You can also ask other GTO users that participated in https://github.com/iterative/gto/issues/337, they may have better ideas how to handle that.

Hi @aguschin, I am using pure GTO with DVC by CLI and recently with Python API, only.
We use DVC only for data version control, not for experiments, pipelines, ...
We do not have Studio, as well.
I really liked the approach with gto describe and gto annotate.
I do not understand why such commands were removed.
I think I can implement them by my own with Python git.Repo.
For example, here is some first approach, which searches for artifacts.yaml and dvc.yaml
def _get_dvc_artifact_path(fs: DVCFileSystem, artifact_name: str) -> Optional[str]:
    """
    Load the artifacts YAML file from the DVC repository and return the path to the artifact.

    This method is compatible with GTO v0.2.x (artifacts.yaml) and v0.3.x (dvc.yaml)
    format of the artifacts YAML file.

    In case of any error (e.g. the file is not found, the artifact is not found, etc.)
    None is returned.

    Args:
        fs: The DVCFileSystem object
        artifact_name: The name of the artifact to load.
    """
    import yaml
    from dvc.scm import RevError
    for artifacts_filename in ["artifacts.yaml", "dvc.yaml"]:
        try:
            with fs.open(artifacts_filename) as f:
                artifacts = yaml.safe_load(f)
                # In `artifacts.yaml` the artifacts are at the root level
                # In `dvc.yaml` the artifacts are under the "artifacts" key
                if artifacts_filename == "dvc.yaml":
                    artifacts = artifacts["artifacts"]
                return artifacts[artifact_name].get("path")
        except (KeyError, FileNotFoundError):
            pass
        except RevError:
            break
    return None


Lost the build on server restart

After successful installation of nominatim, we restarted over server to downgrade aws ec2 instance. We lost all the tables and index. How do we preserve the table and index

OS & Version: Ubuntu 20.04
Docker Version: 20.10.7
Nominatim Version: 4.0

https://github.com/mediagis/nominatim-docker/tree/master/4.0#persistent-container-data


[vm] Usage of dart_native_api.h requires linking dlls to either dart.exe or dart_precompiled_runtime.exe

On Windows, using dart_native_api.h requires statically linking dlls to dart.exe or dart_precompiled_runtime.exe.
  if (is_win) {
    libs = [ "dart.lib" ]
    abs_root_out_dir = rebase_path(root_out_dir)
    ldflags = [ "/LIBPATH:$abs_root_out_dir" ]
  }

https://github.com/dart-lang/sdk/blob/76b18c455e2caa71215c4c63c41d0abbef6d67ff/sdk/BUILD.gn#L38
All occurrences in our code base:
https://github.com/dart-lang/sdk/blob/76b18c455e2caa71215c4c63c41d0abbef6d67ff/runtime/bin/BUILD.gn#L1061-L1183
A workaround is making different DLLs for using in JIT in AOT, but this requires exposing whether Dart is running in JIT or AOT in dart:ffi to load the right DLL.
A better solution would be to have the Dart runtime as a separate DLL which would be shared between JIT and AOT.
Issue discovered via https://github.com/dart-lang/sdk/issues/40564.
Error without statically linking an executable:
LINK(DLL) ffi_test_functions.dll ffi_test_functions.dll.lib
FAILED: ffi_test_functions.dll ffi_test_functions.dll.lib
C:/src/depot_tools/bootstrap-3_8_0_chromium_8_bin/python/bin/python.exe ../../build/toolchain/win/tool_wrapper.py link-wrapper environment.x64 False link.exe /nologo /IMPLIB:./ffi_test_functions.dll.lib /DLL /OUT:./ffi_test_functions.dll /PDB:./ffi_test_functions.dll.pdb @./ffi_test_functions.dll.rsp
ffi_test_functions.ffi_test_functions_vmspecific.obj : error LNK2019: unresolved external symbol Dart_PostCObject referenced in function "void __cdecl dart::NotifyDart(__int64,class std::function<void __cdecl(void)> const *)" (?NotifyDart@dart@@YAX_JPEBV?$function@$$A6AXXZ@std@@@Z)
ffi_test_functions.ffi_test_functions_vmspecific.obj : error LNK2019: unresolved external symbol Dart_ExecuteInternalCommand referenced in function CollectOnNthAllocation
./ffi_test_functions.dll : fatal error LNK1120: 2 unresolved externals

cc @mraleph @mkustermann @askeksa-google

@askeksa-google idea:

Make a dart_api.lib which contains the symbols, but non exported.
These symbols are implemented with GetModuleHandle(nullptr) and look up the real symbol in the executable.
Both when using dart.exe and dart_precompiled_runtime.exe, dlls can statically link against dart_api.lib.

Approved crashes of ffi/ffi_native_test/none on windows. Quite positive it's due to this linking issue (like other tests as well that use dart_native_api.h.

What are the downsides of this mechanism?

Use Pointer.fromFunction() to get pointer of a Dart function.
Register a callback on the native code using the function pointer from the previous step.
Listen for a spesific event and invoke the callback with the event data.
On Dart side, add the event data into a Dart Stream.

This solution does not use Dart_PostCObject from dart_native_api.h and I would like to know if this has any disadvantages over Dart_PostCObject.

@easeccy Please don't post off-topic questions on random unrelated issues and definitely don't post the same question multiple times. There are other channels for such inquiries (e.g. mailing lists, discord, etc). To answer you question: Dart_PostCObject allows (asynchronous) communication between different threads, while callbacks can only be used on a thread which has a Dart isolate associated with it (in other words they are only suitable for synchronous communication).


Enable interoperability of _filter with matchesSearchRequest

Currently matchesSearchRequest doesn't work with queries using _filter.
It would be a desirable feature enhancement to enable interoperability of _filter with matchesSearchRequest.
See:
https://www.medplum.com/docs/search/filter-search-parameter
https://www.medplum.com/docs/sdk/core.matchessearchrequest

Related: https://github.com/medplum/medplum/issues/3627


Exception "division by 0" error with NaNs in DataDrift if stattest jensenshannon is used

If we have more than 1000 values in a reference dataset and NaN values in a current dataset and try to get DataDrift report, an error ZeroDivisionError: division by zero occurs.
Code for reproducing:
import pandas as pd
from evidently import ColumnMapping

from evidently.dashboard import Dashboard
from evidently.dashboard.tabs import DataDriftTab

ref_test_data = pd.DataFrame(
    {
        "target": [1, 2, 3, 1, 1] * 1000,
        "num": [1, 2, 3, 4, 5] * 1000,
        "cat": [1, 0, 1, 0, 0] * 1000,
    }
)

curr_test_data = pd.DataFrame(
    {
        "target": [1, 2],
        "num": [None, None],
        "cat": [14, None],
    }
)

data_columns = ColumnMapping(
    categorical_features=["cat"], target="target",
)
report = Dashboard(tabs=[DataDriftTab()])
report.calculate(
    current_data=curr_test_data, reference_data=ref_test_data, column_mapping=data_columns
)

Traceback
Traceback (most recent call last):
  File "/Users/tapot/prg/evi/evidently_dev/test.py", line 540, in <module>
    main()
  File "/Users/tapot/prg/evi/evidently_dev/test.py", line 536, in main
    test_bug_with_jensenshannon_stattest()
  File "/Users/tapot/prg/evi/evidently_dev/test.py", line 516, in test_bug_with_jensenshannon_stattest
    current_data=curr_test_data, reference_data=ref_test_data, column_mapping=data_columns
  File "/Users/tapot/prg/evi/evidently_dev/evidently_repo/src/evidently/dashboard/dashboard.py", line 186, in calculate
    self.execute(reference_data, current_data, column_mapping)
  File "/Users/tapot/prg/evi/evidently_dev/evidently_repo/src/evidently/pipeline/pipeline.py", line 50, in execute
    self.analyzers_results[analyzer] = instance.calculate(rdata, cdata, column_mapping)
  File "/Users/tapot/prg/evi/evidently_dev/evidently_repo/src/evidently/analyzers/data_drift_analyzer.py", line 41, in calculate
    data_drift_options=data_drift_options,
  File "/Users/tapot/prg/evi/evidently_dev/evidently_repo/src/evidently/calculations/data_drift.py", line 229, in get_overall_data_drift
    drift_result = test(ref_feature, curr_feature, feature_type, threshold)
  File "/Users/tapot/prg/evi/evidently_dev/evidently_repo/src/evidently/calculations/stattests/registry.py", line 35, in __call__
    drift_score, drifted = self.func(reference_data, current_data, feature_type, actual_threshold)
  File "/Users/tapot/prg/evi/evidently_dev/evidently_repo/src/evidently/calculations/stattests/jensenshannon.py", line 25, in _jensenshannon
    reference_percents, current_percents = get_binned_data(reference_data, current_data, feature_type, n_bins, False)
  File "/Users/tapot/prg/evi/evidently_dev/evidently_repo/src/evidently/calculations/stattests/utils.py", line 31, in get_binned_data
    current_percents = np.array([current_feature_dict[key] / len(current) for key in keys])
  File "/Users/tapot/prg/evi/evidently_dev/evidently_repo/src/evidently/calculations/stattests/utils.py", line 31, in <listcomp>
    current_percents = np.array([current_feature_dict[key] / len(current) for key in keys])
ZeroDivisionError: division by zero

Discord discussion with real-data dataset https://discord.com/channels/815991896916361276/1020337385202012183

Out stat tests are not defined when current or reference data are empty.
so here we need to add correct error information:
Value Error: an empty column provided for drift calculation in the current dataset
or
Value Error: an empty column provided for drift calculation in the reference dataset
We should add it in all stattests: chisquare, jensenshannon, kl_div, ks, psi, wasserstein  and z-test

@0lgaF it also makes sense for me to raise an error for this case.
Btw, when I run the given code I get:
     26 report = Dashboard(tabs=[DataDriftTab()])
     27 report.calculate(
---> 28     current_data=curr_test_data, reference_data=ref_test_data, column_mapping=data_columns
     29 )

c:\Users\AX-St\hackoctober\evidently\evidently\src\evidently\dashboard\dashboard.py in calculate(self, reference_data, current_data, column_mapping)
    187     ):
    188         column_mapping = column_mapping or ColumnMapping()
--> 189         self.execute(reference_data, current_data, column_mapping)
    190
    191     def __dashboard_data(self) -> Tuple[str, DashboardInfo, Dict]:

c:\Users\AX-St\hackoctober\evidently\evidently\src\evidently\pipeline\pipeline.py in execute(self, reference_data, current_data, column_mapping)
     48             instance = analyzer()
     49             instance.options_provider = self.options_provider
---> 50             self.analyzers_results[analyzer] = instance.calculate(rdata, cdata, column_mapping)
     51         for stage in self.stages:
     52             stage.options_provider = self.options_provider

c:\Users\AX-St\hackoctober\evidently\evidently\src\evidently\analyzers\data_drift_analyzer.py in calculate(self, reference_data, current_data, column_mapping)
     39             reference_data=reference_data,
     40             dataset_columns=columns,
...
---> 94             raise ValueError(f"Column {column_name} in current dataset should contain numerical values only.")
     95
     96     reference_column = reference_column.replace([-np.inf, np.inf], np.nan).dropna()

ValueError: Column num in current dataset should contain numerical values only.

Hey @AntonisCSt ,
thank you for your input. We added additional ValueErrors for described case:
An empty column '{column_name}' was provided for drift calculation in the reference/current dataset

Fix is merged, it will be available in the next package.


System.InvalidOperationException while testing my bot

While testing my bot for the 1000 games I got following error.
Unhandled exception. System.InvalidOperationException: Operations that change non-concurrent collections must have exclusive access. A concurrent update was performed on this collection and corrupted its state. The collection's state is no longer correct.
   at System.Collections.Generic.Dictionary`2.FindValue(TKey key)
   at System.Collections.Generic.Dictionary`2.TryGetValue(TKey key, TValue& value)
   at ChessChallenge.Application.BoardUI.DrawSquare(Int32 file, Int32 rank) in c:\Users\TBMPFW\Chess-Challenge\Chess-Challenge\src\Framework\Application\UI\BoardUI.cs:line 327
   at ChessChallenge.Application.BoardUI.ForEachSquare(Action`2 action) in c:\Users\TBMPFW\Chess-Challenge\Chess-Challenge\src\Framework\Application\UI\BoardUI.cs:line 233
   at ChessChallenge.Application.BoardUI.Draw() in c:\Users\TBMPFW\Chess-Challenge\Chess-Challenge\src\Framework\Application\UI\BoardUI.cs:line 205
   at ChessChallenge.Application.ChallengeController.Draw() in c:\Users\TBMPFW\Chess-Challenge\Chess-Challenge\src\Framework\Application\Core\ChallengeController.cs:line 382
   at ChessChallenge.Application.Program.Main() in c:\Users\TBMPFW\Chess-Challenge\Chess-Challenge\src\Framework\Application\Core\Program.cs:line 41

Thanks,
TBMPFW

It only happend once. So if anyone's wondering why they've got this error, just restart the bot.


First bits of documentation for django-modeltranslation integration

Got some time to write a quick overview for the django-modeltranslation integration. However, I need native english speakers to correct and improve the text.

Hi Mathias
I am happy to do that.
Can you just tell me the process to follow.
Cheers
Graham
On 16 March 2015 at 04:24, Mathias Ettinger<EMAIL_ADDRESS>wrote:

Got some time to write a quick overview for the django-modeltranslation
integration. However, I need native english speakers to correct and improve
the text.
You can view, comment on, or merge this pull request online at:
https://github.com/stephenmcd/mezzanine/pull/1231
Commit Summary

Added first bits of documentation for django-modeltranslation
integration

File Changes

M docs/index.rst
https://github.com/stephenmcd/mezzanine/pull/1231/files#diff-0 (1)
A docs/multi-lingual.rst
https://github.com/stephenmcd/mezzanine/pull/1231/files#diff-1 (183)

Patch Links:

https://github.com/stephenmcd/mezzanine/pull/1231.patch
https://github.com/stephenmcd/mezzanine/pull/1231.diff

—
Reply to this email directly or view it on GitHub
https://github.com/stephenmcd/mezzanine/pull/1231.

Thank @orotau
You can fork my repo and work from my modeltranslation-doc branch. If you make a pull request, it will show up here when I merge it.
Otherwise, you can send me a message with your modification and I’ll integrate them in my repo.

I'm sending over a PR - I did a pass on the spelling/grammar.  Feel free to use it and thanks for your work on translations.

ok I guess I don't need to do anything then. Very new to collaborating using github so treading carefully :-)

Hi guys,
I'm merging this in now - I made a ton of tiny style tweaks, but generally left all the content intact as it is incredibly well done. Just wanted to say a huge thanks for going the extra mile and adding such great documentation, so thank you!


[QUESTION] Dependency in _global folder

Hi,
I´m working on implementing this repository structure for my own project.
The documentation states that there can be a _global folder for resources like IAM.
My question would be, how can I implement a dependency of a resource in the _global folder to a resource in an environment folder
My terragrunt.hcl in the environment folder has the following config
.
.
.
include {
  path = find_in_parent_folders()
}

# Set Module dependencies
dependencies {
  paths =  [
    "../../../_global/tf-iam"
    ]
}

When running the plan-all command in the environment folder i get the following error message.
Module /home/ec2-user/aws-account-name/eu-central-1/dev/tf-ec2 ../../../_global/tf-iam as a dependency, but that dependency was not one of the ones found while scanning subfolders:

Any help would be appreciated.
Thanks!

You can pass in the --terragrunt-include-external-dependencies flag.

@yorinasub17 , thank you for your fast response!
This works like a charm.


Update to .NET Aspire 9.0

I think we can also update the global.json file to point to the .NET 9 RC 2 SDK since it was released today?

This should be ready to merge once CI passes.


FieldBuilder phase 2 improvement

Give field builder unsupported type a default value instead of throw error.
https://github.com/Azure/azure-sdk-for-java/pull/10500#discussion_r416946182

Use collection base class. + Fix the array type.
https://github.com/Azure/azure-sdk-for-java/pull/10500#discussion_r418753877

Name property override.
https://github.com/Azure/azure-sdk-for-java/pull/10500#discussion_r418773038

Fixed the problem of not taking inner class.

Is a whitelist easier to maintain than a blacklist?

Regarding the name override, instead to align with mapping actual fields from a document payload (from a query) to index names, we should instead support our respective frameworks' attributes that would already be used. For example, in .NET we use (in track 2) JsonPropertyName from the built-in System.Text.Json. That way, we're not duplicating (or forcing devs to duplicate, rather) property<-> field renames. Since, IIRC, you're using Jackson maybe it's a BeanProperty given a different name that you use.

Give field builder unsupported type a default value
#10500 (comment)

@conniey Regarding this question, FieldBuilder will convert property without annotation to complex field. This is recursive call.
If customer use the following code:
private char something;

Our logic will fall into parsing all properties in char.class. Whitelist is not enough as we do not know if this is a real complex type or the type we do not parse.
Solutions:

Map all blacklist to search DataType.
@heaths what's your proposal on this?

Regarding the name override, instead to align with mapping actual fields from a document payload (from a query) to index names, we should instead support our respective frameworks' attributes that would already be used. For example, in .NET we use (in track 2) JsonPropertyName from the built-in System.Text.Json. That way, we're not duplicating (or forcing devs to duplicate, rather) property<-> field renames. Since, IIRC, you're using Jackson maybe it's a BeanProperty given a different name that you use.

This might introduce complexity in near future.  Java is working on adding plugin layer for json library. Customer will have the choice of which library to use.
In this case, we can either only allow Jackson library user to override the name. Or use some other complex helper to get the overridden property name.

Maybe it's a progression, then: we support name overrides in our attributes, then check common ones, then fall back to the name as-is. It's important to remember that Search is case-sensitive. Probably not an issue for Java or JavaScript if people use Java/Script-like field names (examples show this, so probably most common), but an issue for .NET (CamelCase) and Python (snake_case).
As for the DataType question, I'm not sure what you mean.

Fixed bugs for preview 4.
https://github.com/Azure/azure-sdk-for-java/pull/11532

Move to preview 5 for additions.

Will continue working on @JsonProperty (or Gson annotation) to rename the field name

Closed by PR: https://github.com/Azure/azure-sdk-for-java/pull/13496


Flag Issue

When I select United States it displays Canada's Flag

@collinsmarra I have fixed the issue in the new update, Please update the package to see the changes.


Convert templates to Nunjucks

Description

Configures Pattern Lab to use Nunjucks instead of Mustache
Converts all Mustache templates to Nunjucks
Refactors data to work better with Nunjucks pattern engine (which, for example, doesn't support listitems.json properly)

Motivation
Right now:

Pattern Lab uses Mustache

The Mustache templates use features unique to Pattern Lab such as pattern params, styleModifiers and list items, which means they cannot easily be re-used outside of Pattern Lab

The Buildit website has its own set of templates, written in Handlebars, which largely recreate the same HTML structures we do within Pattern Lab

Furthermore, a number of Handlebars helpers have been configured because the site needed features that Handlebars didn't provide out of the box

We need to get to a state where both Pattern Lab and the Buildit website (and potentially more sites in future) share the same templates. This will cut down the maintenance burden and make it easier to roll out changes. Therefore, both need to align on the same template language.
Mustache is too basic for the site. Also, as mentioned above, our current Mustache templates are using proprietary Pattern Lab features anyway.
We could go with Handlebars, but it just so happens that Nunjucks can do out-of-the-box most of what the website needs but currently needs Handlebars helpers to achieve.
So, the decision was made to migrate to Nunjucks.
This PR is the first step. Once approved and merged, the next step will be to make builds export and publish the templates as a new NPM package, which the website and others can then consume.
Notes
All templates have been converted and our pattern library builds fine. The rendered HTML is the same as before.
But:

In anticipation of the template files eventually being used outside of Pattern Lab, I have updated many of them to use Nunjucks' macro feature. While it does add some complexity to the template, it enables other templates to import the macro (as opposed to include the template) and then pass values into the macro.

With includes, the included template references the same context data the parent template has. Therefore, when there are multiple instances of a template being included, it becomes difficult to provide different data to each one.
The macro mechanism on the other hand is akin to passing props to React components - you explicitly pass data into the macro and thus have more control.

I have used macros where I needed to, or where I felt it made sense. However, I've worked on this PR in drips and drabs so it's been done somewhat ad hoc. I think it's sufficient for now, but we may need to revisit this later and make things more consistent. Perhaps everthing should be a macro? Or, perhaps, we should have some way (tagging in meta-data? naming convention?) to make it clear which templates are intended to be imported and which included.
While I have preserved the template names, organisation and HTML output for atoms - templates, I have taken the liberty to tweak things a little for the pages. Since these are just examples, it won't affect consumers of Gravity at all.
Going through all our templates with fresh eyes has highlighted a number of questionable choices. Some things are poorly organised (e.g. we have different kinds of lists scattered all over the place), naming is a bit confusing and the whole section organism thing is a massive clusterfuck. Once this gets merged we really should spend some time spring cleaning. :-D
I've rejigged the JSON data a fair bit. There is now much less shared stuff in data.json and more per-pattern JSON data. While this does introduce some duplication, I think it makes it easier for developers to understand where data for a particular template is coming from. PL's a bit opaque in this respect unfortunately, so debugging missing or unexpected values is a royal pain in the arse.

A beneficial side-effect is that this side-steps some of PL's data merging quirks and makes our page examples tidier. For instance, we no longer have weird Lorem ipsum nav links appearing there!

For me, the html view option of pattern lab is presenting poorly formatted html and I can't see all of it in the view. e.g. http://<IP_ADDRESS>:49689/?p=templates-homepage shows a large whitespace before the first tag:

Good spot @dw-buildit . I'll look into that. Nunjucks does have some whitespace control features which should let us tidy this a bit.
Things like indentation probably won't be perfect though - just as they aren't when using Mustache. So, perhaps longer term it's worth investigating if we can make a Pattern Lab plugin the beautifies the mark-up before it is output.

@dw-buildit I have now tweaked all the templates to eliminate redundant whitespace in the rendered HTML output. The final output is not necessarily indented nicely, but it should be no worse than it was when we were using Mustache templates.
Please review when you next get a chance and approve - or not :-P. Thanks!

Looks good now :thumbsup:


[WIP]: mc_pos_control: split control_position into two function; clean up of…

… auto takeoff

a better solution is coming

Closing because I took over for a new approach. Will pr as soon as I have first successful tests.


Reopens recently closed folders and apps

Description of the new feature / enhancement
Remembers all recently closed folders and applications, so that we can restore them in the same sequence in which they were closed using some hotkeys like Windows + Shift + Z. Just like the way Firefox, Google Chrome and other browsers let we open the recently closed tab(s). When all internet browsers allow this, so why can't Windows allow this?
Scenario when this would be used?
Sometimes we close a File Explorer window by mistake, and we want to reopen it, but it is deeply buried in a list of folders. It is a pain to reopen it. Also, when we accidentally close an app like notepad or calculator.
Supporting information
https://www.addictivetips.com/windows-tips/undo-close-for-windows-addictivetips-apps/
https://superuser.com/questions/1373524/reopen-recently-closed-file-explorer-window-in-windows-10

Timeline (under Win+Tab) shows a history or recently opened files, but that's not quite what you're looking for.

i don't think we could make this to be minimal lovable.  Why: I think we'd have to have to restore the application state and i don't think we'd be able to do that well.
I feel the pain but i think the app itself would have to handle that.  Without this feature, i feel this would just cause frustration and be only like 10% done due to that.

I do think it is a good idea, i just don't think PT could execute this well at end of day


Add CHANGELOG.md for tracking changes

Current dependencies on/for this PR:

develop

PR #48   👈

This comment was auto-generated by Graphite.


Tables are not part of the output

Describe the bug
When using this sample document it doesn't get the contents of the table. I also tested with other tables in word, and no luck. Only pure text is supported?
To Reproduce
whatever.docx

Filetype: DOCX
Any additional API parameters:

req = shared.PartitionParameters(
    # Note that this currently only supports a single file
    files=shared.Files(
        content=stream.read(),
        file_name=file_name,
    ),
    # Other partition params
    strategy="auto",
    include_page_breaks="true",
    # chunking_strategy="by_title",
    # max_characters=5000,
    # combine_under_n_chars=1500
    # xml_keep_tags=True,
    # skip_infer_table_types=False
)

Environment:

Using the hosted API or self hosting? Hosted API
How are you calling the API? (Langchain, SDKs, cUrl, etc.) Python SDK

@leolorenzoluis our docx partitioner definitely supports tables, those are a very important element-type.
When I partition this file I get three elements: [Table, PageBreak, Title] containing the text:
'Whatever Daga tangina |  | |  | |  | |  |',
'',
'Ewan',

respectively.
Inspecting the docx contents, I see that much of the content is contained in "structured data tags" more commonly called something like "form fields". Text inside form-fields is currently not captured, which explains this behavior for this particular document.
Form fields are relatively uncommon in the broad corpus of Word documents, but are perhaps common in certain collections, like if you use Word to capture information from respondents for some reason. If you try this with other .docx files I expect you'll see the behavior you're expecting.
If you work with a lot of Word documents with text in form fields, let us know more about your situation and use cases and we can consider adding support for this.

I'm going to close this issue for now @leolorenzoluis just because it is not immediately actionable. But don't hesitate to reopen it if you are willing to describe your use-cases and advocate for us adding this support :)


ci: add cd and performance improvements

Closes https://github.com/hirosystems/devops/issues/1341

Enables a pipeline to deploy into a dev, staging, and production environment much like the Explorer.

Example run

Each deployment to an environment is guarded by a manual approval gate. When approval is received, the pipeline will continue with the deployment to the targeted environment, then wait at the next approval gate.

Currently only @MicaiahReid and @lgalabru have permissions to approve deployments into an environment, but anyone with admin permissions can configure this here.

Significantly reduces build time to <10 minutes at best by leveraging GH Actions cache and cutting out arm64 builds.

GH Actions runners use an amd64 CPU arch, so any arm64 Docker images must be built in an emulator which is notoriously slow.

Adds concurrency definition to auto-cancel GH action runs on the same branch or PR.
Updates many GH actions to their latest version

@MicaiahReid I forgot to mention, deployment to the dev and staging environments will auto-approve when a new release is publish -- so most commits/merges to main. And deployments to prod are restricted to pipelines where a new version is released. There is another GH actions workflow I need to add for rollback functionality, but that can come in another PR.

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


Do not force the pure Go name resolver

Revert to Go's default mechanism that will decide between the netgo pure-Go implementation and the netcgo cgo-based implementation depending on OS and environment variables.
This allows, among other things, to use Prometheus to scrape mDNS targets, thereby addressing #2537.
Context: I had a look at past commits and discussions, and kind of understand why it was chosen to use netgo by default (essentially performance concerns and compatibility). On the other hand, there are cases where the netcgo backend is really preferable, as with mDNS names. I think letting end users override the default via an environment variable is the most flexible solution, and it is already implemented as of Go 1.7:

The resolver decision can be overridden by setting the netdns value of the GODEBUG environment variable (see package runtime) to go or cgo, as in:
export GODEBUG=netdns=go    # force pure Go resolver
export GODEBUG=netdns=cgo   # force cgo resolver

https://golang.org/pkg/net/#hdr-Name_Resolution

Discussion welcome!

Wouldn't it be better if the pure Go name resolver received the functionality its missing from netcgo instead?
That way it will not matter which one is available on the target system.

I am strongly against this, for several reasons.
Support complexity
One reason to force netgo was that there is not one "the" system resolver.
What that is depends a lot on the OS, the libc, and in some cases,
configuration and .so file availability. Combined with Go picking one or
the other, we'll have a myriad of combinations of "why doesn't this work".
Release binaries
Which libc would we link them against? If we link them against glibc, they
will segfault on non-glibc systems and systems with incompatible versions –
even if we link "statically"" glibcs to dynamically load resolver libraries
including the one for mDNS.
If we don't, you most likely won't get mDNS from the release binaries
anyway.
Containers
Somewhat an extension of "release binaries", we would have to ensure that
every possible library dependency is included. Which OS are our images
built on at the moment?
On Mon, Apr 10, 2017, 09:01 Patrik<EMAIL_ADDRESS>wrote:

Wouldn't it be better if the pure Go name resolver received the
functionality its missing from netcgo instead?
That way it will not matter which one is available on the target system.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/prometheus/pull/2538#issuecomment-292865800,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAICBn-zNi7jwvA_6HtOfgFE8Duxk7odks5rudOvgaJpZM4Mri2A
.

I agree with @matthiasr, this seems unlikely to change.
If you want to improve this, I'd suggest doing so in Go itself.


nns:  support SRV records

Support SRV record in NNS plugin after https://github.com/nspcc-dev/neofs-contract/issues/254

We don't have to support this
https://github.com/nspcc-dev/neofs-contract/issues/254#issuecomment-1218184423


CopyExternalsPlugin does not copy external modules marked /* webpack: external */

CopyExternalsPlugin does not copy external modules marked /* webpack: external */

Is there anything else missing that I need to do in order to get this merged?

@mergifyio backport release/3.7.x


fix(deploy): fix git ref verify

This problem is encountered when deploying with pm2 deploy ref ref_hash.
git rev-parse --quiet --verify ${ref_hash} does not check if ref_hash really exists, it only checks if the format is valid.
For example, the command can be executed under the current repo:
# true ref
git rev-parse --quiet --verify 4e9655c042fa43ea82506cf5f4fad583a880568d

# fake ref
git rev-parse --quiet --verify 4e9655c042fa43ea82506cf5f4fad583a8805680

+1. Just ran into this same issue. It looks like a branch can be supplied as the next param, but the documentation doesn't say anything about that. It'd be great to see this fix applied, or the docs updated with the proper way to specify a ref hash.
In order to work around this, we ended up adding a pre-deploy hook which runs a git fetch --all in order to be able to refer to the ref hash passed in the deploy command.


fix: fix element cannot re-create when type changed

修复当元素类型变化时，没有按照预期重新绘制的问题
调整了 Dimond 节点

Fix to not redrawing as expected when element type changes
Adjusted Dimond nodes

元素类型变更不会执行动画逻辑


Patch 1

Hi @ninimsft, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.azure.com.

TTYL, AZPRBOT;

@ninimsft, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, AZPRBOT;

@ninimsft - Please resubmit via the private repo. See email for more info.


Adding repositories from gitlab.com

How can I help to get this repository:
https://gitlab.com/geertjanklaps/abap-openapi-ui
from gitlab.com added to the page?

currently not possible, see readme "Hosted on GitHub(more git based hosts to be added later)"
but pull request welcome
its the code in https://github.com/dotabap/dotabap-validator that runs daily to keep the list updated

@gregorwolf:
I just moved the original project to GitHub and created a pull request to get ABAP OpenAPI in the list.
I think this can be closed after merging :)


wire up drop CQ statements

fixes #2141
i can't say for sure if what i've wired up is doing the right thing but it appears to be working

ping @toddboom

@neonstalwart ha, that looks about right. nice catch! if you can sign the CLA, i'll get this merged in!
http://influxdb.com/community/cla.html

@toddboom i signed the CLA this morning

i signed again just in case it didn't work - let me know if there are any problems.

@neonstalwart ah, sorry about that - i may have missed you on the list. merging now!


⚠ Bofrost.si has degraded performance

In afe8439, Bofrost.si (https://www.bofrost.si) experienced degraded performance:

HTTP code: 200
Response time: 2901 ms

Resolved: Bofrost.si performance has improved in 8ffdc13.


Fonts

This requires some level of terminal support for fonts. It'd be nice if these were listed somewhere

Terminal support for unicode characters is required; now updated in docs.


Node.js remote debug: Using remote and local paths problem...

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

Version: 1.67.1
Commit: da15b6fd3ef856477bf6f4fb29ba1b7af717770d
Date: 2022-05-06T12:39:22.915Z
Electron: 17.4.1
Chromium: 98.0.4758.141
Node.js: 16.13.0
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 5.10.0-14-amd64
Steps to Reproduce:
In a docker container running a Nodejs app started with --inspect flag, attach a debugger to it and put a breakpoint somewhere in the code. Then when execution hits the breakpoint file cant be edited.  Reading in the documentation said that using localroot and remoteroot solves the problem but it doesn't. I tried different configurations an nothing seems to work.  My launch.json looks like this:
{
  "version": "0.2.0",
  "configurations": [
    {
      "name": "Attach to container",
      "type": "pwa-node",
      "request": "attach",
      "port": 9229,
      "sourceMaps": true,
      "restart": true,
      "address": "<IP_ADDRESS>",
      "localRoot": "/home/fciliberti/Trabajos/gekkoloco/app",
      "remoteRoot": "/usr/src/app",
     }
  ]
}

/jsDebugLogs

Hey @connor4312, this issue might need further attention.
@ellocofray, you can help us out by closing this issue if the problem no longer exists, or adding more information.

Hm, the bot should have closed the issue after 7 days of the info-needed tag. Doing so manually... if you are able to get logs, please comment on this issue or create a new one.


Set email options in all envs

seeds.rb is breaking in prod bc the email settings (triggered by devise) weren't set

https://stackoverflow.com/questions/7219732/missing-host-to-link-to-please-provide-host-parameter-or-set-default-url-optio

I was going to fix this by copying into config/environments/production.rb the line from my other PR, at https://github.com/maebeale/mutual-aid/blob/f09c45ef0d49fe6ae0036128f1544174fb05c0b9/config/environments/test.rb#L50
But then I realized I don't know what the host in the URL for prod should look like. Do you want to set it to localhost for now so the links deliberately don't work right yet, or do you have a hostname for prod yet?

hey there, @h-m-m ! i invited you to two heroku apps: staging and prod. my vote would be to set it to an ENV var and we can store it using dotenv that you set up. what do you think?

That sounds smart!

yay! lmk if you want/need me to add the env vars to the heroku apps themselves.

completed


purchase order maintenance item qty not updating

The item quantity in purchase order maintenance is not updated when clicking on the update button in the detail panel...once a line has already been created.   This has been corrected with patch 6.5.1.

patch 6.5.1 has been released and available for download

patch in 6.5.2...closing


Hello! Please bring back the top left and top right buttons, it's inconvenient to use without these two buttons in the theme. I hope it can be restored to allow users to freely decide whether to show or hide

Hello! Please bring back the top left and top right buttons, it's inconvenient to use without these two buttons in the theme. I hope it can be restored to allow users to freely decide whether to show or hide
Version: 3.470
OS: Windows 10

+1, much needed. I was hoping to find it in the Style Settings.

+1, much needed. I was hoping to find it in the Style Settings.

added in latest update


Convolution operation vectorization

Most of the performance IREE achieves on GEMM is due to the use of vector dialect. A similar approach is needed for Convolutions. So far convolution performance has only been improved in a straw-man fashion.

The SPIR-V backend has vectorization support for linalg::Conv2DNhwcHwcfOp and linalg::DepthwiseConv2DNhwOp.
Nod lab has looked at ways of vectorization 1d Conv operations here.

A more general story needs to be flushed out by

[ ] Expand the work done on Conv1D above to add batch dimensions, channel dimensions and expand to 2D. These might have different approaches that work the best.
[ ] If the convolution operations vectorize, then these operations can be tiled and fused the same way GEMM operations are supposed to be (see #6973). This would then naturally generalize to fusing convolutions with elementwise operations.
[ ] It would also allow for fusing pad operations with convolutions at vector level. There are some patterns in core (here) that vectorize the linalg.pad_tensor operations. So with tile and fuse + vectorization this should generate pretty efficient code for convolution operations.

With PR #8460 , #8517 and #8527 IREE CPU backends have a path for vectorizing convolutions with fusion at vector level.
PR #8049 pad operations are fused with convolution consumers. The addresses the scope of this issue. More work is needed on this that is out of scope of this bug, and will be tackled subsequently

The CPU side vectorization is just being turned on. There is definitely more scope for improvement there, but the inital metrics look great.
The fusion of padding with convolutions results in very branchy code. It does help performance but an alternative approach of fusing padding with producers is also an interesting approach to evaluate that might result in overall better code generation.


Add error handling for add_ouput in Python SDK

Summary:
This pull request addresses a critical flaw in the add_output method within the Python SDK. Currently, the method incorrectly adds None to the output list when the Output parameter is None, which contradicts the intended behavior. The expected functionality is for the method to reject None as an invalid output or, alternatively, raise an exception.
Changes Made:

Modified the add_output method to raise an exception when the Output parameter is None.

Context:
This change ensures that the add_output method behaves as expected, providing more robust and predictable behavior when handling output parameters.
Related Issues:
Closes #529
Test Plan:
New test similar to test_add_output_existing_prompt_no_overwrite but with overwrite needs to be added.
Things left to complete:

[x] Add test for add_output with overwrite enabled

Approaching this with curiosity. What's the best approach to managing PR comments?
Should the author address the comment and mark it as resolved, or is it the responsibility of the reviewer to close out the comments after an update or commit?

Approaching this with curiosity. What's the best approach to managing PR comments?
Should the author address the comment and mark it as resolved, or is it the responsibility of the reviewer to close out the comments after an update or commit?

Personally I would say the PR author addresses comments and resolves as they're fixed. Some people prefer that reviewer be more strict and review everything, but I feel that reviewer should just give another final look to confirm everything is cool anyways

This looks great! Thanks for being so diligent with testing and fixes from comments! Looking forward to continue working with you :)


Missing RuboCop::Cop::PackwerkLite::Base when rails environment is loaded

After installing pack_stats and rubocop-packs I am getting a NameError when it is loaded in my rails environment.  I went looking through the codebase for the Base class to see if I could fix it with a PR, but could not find one anywhere, am I missing something?
# I trimmed out extraneous bits from the paths
rake aborted!
NameError: uninitialized constant RuboCop::Cop::PackwerkLite::Base
Did you mean?  Base64
gems/rubocop-packs-0.0.30/lib/rubocop/cop/packwerk_lite/privacy_checker.rb:40:in `<module:PackwerkLite>'
gems/rubocop-packs-0.0.30/lib/rubocop/cop/packwerk_lite/privacy_checker.rb:5:in `<module:Cop>'
gems/rubocop-packs-0.0.30/lib/rubocop/cop/packwerk_lite/privacy_checker.rb:4:in `<module:RuboCop>'
gems/rubocop-packs-0.0.30/lib/rubocop/cop/packwerk_lite/privacy_checker.rb:3:in `<main>'
gems/bootsnap-1.13.0/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:32:in `require'
gems/bootsnap-1.13.0/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:32:in `require'
gems/rubocop-packs-0.0.30/lib/rubocop/packwerk_lite.rb:6:in `<main>'
gems/rubocop-packs-0.0.30/lib/rubocop-packs.rb:9:in `require_relative'
gems/rubocop-packs-0.0.30/lib/rubocop-packs.rb:9:in `<main>'
gems/bootsnap-1.13.0/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:32:in `require'
gems/bootsnap-1.13.0/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:32:in `require'
gems/pack_stats-0.0.1/lib/pack_stats/private/metrics/packwerk_checker_usage.rb:4:in `<main>'
gems/bootsnap-1.13.0/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:32:in `require'
gems/bootsnap-1.13.0/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:32:in `require'
gems/pack_stats-0.0.1/lib/pack_stats/private/datadog_reporter.rb:8:in `<main>'
gems/bootsnap-1.13.0/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:32:in `require'
gems/bootsnap-1.13.0/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:32:in `require'
gems/pack_stats-0.0.1/lib/pack_stats.rb:12:in `<main>'
gems/bootsnap-1.13.0/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:32:in `require'
gems/bootsnap-1.13.0/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:32:in `require'
gems/bundler-2.3.22/lib/bundler/runtime.rb:60:in `block (2 levels) in require'
gems/bundler-2.3.22/lib/bundler/runtime.rb:55:in `each'
gems/bundler-2.3.22/lib/bundler/runtime.rb:55:in `block in require'
gems/bundler-2.3.22/lib/bundler/runtime.rb:44:in `each'
gems/bundler-2.3.22/lib/bundler/runtime.rb:44:in `require'
gems/bundler-2.3.22/lib/bundler.rb:187:in `require'
rails/config/application.rb:19:in `<top (required)>'
rails/Rakefile:1:in `require'

A co-worker pointed out that it is probably trying to extend RuboCop::Cop::Base and I threw a debugger in when running specs and verified that is why specs are passing for me and failing in my environment.
=> 41:       class Privacy < Base
   42:         extend T::Sig
   43:
   44:         sig { returns(T::Boolean) }
   45:         def support_autocorrect?
(byebug) Base
RuboCop::Cop::Base

And that is because we are an ancient version of rubocop (0.81.0) and it was not added till 0.87.0, will try upgrading.

Thanks for the issue reporting @ransombriggs ! Let me know if upgrading doesn't resolve your issue!

Upgrading to 0.90.0 worked for me (needed to go past 0.87.0 to get an unrelated fix)


Error

Permasigner | Version main_6f5687d
Program created by Nebula | Original scripts created by zhuowei | CoreTrust bypass by Linus Henze
[?] Would you like to use an external or a local IPA? [E, L] L
[?] Paste in the path to an IPA in your file system: /var/mobile/CloneApp/Appcake_7.2.2-Fix.ipa
[*] Unzipping IPA...
Found app directory!
[*] Reading plist...
Traceback (most recent call last):
File "main.py", line 4, in
main.main(False)
File "/private/var/mobile/permasigner/permasigner/main.py", line 72, in main
ps.start()
File "/private/var/mobile/permasigner/permasigner/main.py", line 245, in start
out_dir = self.run(tmpfolder, ldid_in_path, dpkg_in_path, data_dir)
File "/private/var/mobile/permasigner/permasigner/main.py", line 372, in run
app_author = app_bundle.split(".")[1]
IndexError: list index out of range
xXx:~/permasigner root#

Could you please share the IPA?

Here I'm going to assume that the bundle id is 1 word without dots, which is not normal convention for bundle id's.
You can pass the --bundleid any.text.here flag to make it build correctly.

Here I'm going to assume that the bundle id is 1 word without dots, which is not normal convention for bundle id's.
You can pass the --bundleid any.text.here flag to make it build correctly.

Thanks

Could you please share the IPA?
It's not a bug in permasigner, rather a mistake by whoever modified the Info.plist

Of course :)
https://github.com/Oct4Pie/appcake-patch

Of course :)
https://github.com/Oct4Pie/appcake-patch

Plist looks fine to me. Can't reproduce it with the latest release.


Build & Run Issue

Hello Team,
I am facing some trouble while building freeTensor. Here are the logs for reference,
[ 26%] Building CXX object 3rd-party/antlr/antlr4/runtime/Cpp/runtime/CMakeFiles/antlr4_static.dir/src/tree/xpath/XPathWildcardElement.cpp.o
[ 27%] Building CXX object 3rd-party/antlr/antlr4/runtime/Cpp/runtime/CMakeFiles/antlr4_static.dir/src/tree/xpath/XPathRuleElement.cpp.o
[ 27%] Building CXX object 3rd-party/antlr/antlr4/runtime/Cpp/runtime/CMakeFiles/antlr4_static.dir/src/tree/xpath/XPathWildcardAnywhereElement.cpp.o
[ 27%] Building CXX object 3rd-party/antlr/antlr4/runtime/Cpp/runtime/CMakeFiles/antlr4_static.dir/src/tree/xpath/XPathLexer.cpp.o
[ 27%] Building CXX object 3rd-party/antlr/antlr4/runtime/Cpp/runtime/CMakeFiles/antlr4_static.dir/src/tree/xpath/XPathTokenElement.cpp.o
c++: fatal error: Killed signal terminated program cc1plus
compilation terminated.
make[2]: *** [3rd-party/antlr/antlr4/runtime/Cpp/runtime/CMakeFiles/antlr4_static.dir/build.make:90: 3rd-party/antlr/antlr4/runtime/Cpp/runtime/CMakeFiles/antlr4_static.dir/src/ANTLRErrorStrategy.cpp.o] Error 1
make[2]: *** Waiting for unfinished jobs....
c++: fatal error: Killed signal terminated program cc1plus
compilation terminated.
make[2]: *** [3rd-party/antlr/antlr4/runtime/Cpp/runtime/CMakeFiles/antlr4_static.dir/build.make:160: 3rd-party/antlr/antlr4/runtime/Cpp/runtime/CMakeFiles/antlr4_static.dir/src/BufferedTokenStream.cpp.o] Error 1
c++: fatal error: Killed signal terminated program cc1plus
c++: fatal error: Killed signal terminated program cc1plus
compilation terminated.
c++: fatal error: Killed signal terminated program cc1plus
compilation terminated.
make[2]: *** [3rd-party/z3/src/util/CMakeFiles/util.dir/build.make:188: 3rd-party/z3/src/util/CMakeFiles/util.dir/fixed_bit_vector.cpp.o] Error 1
c++: fatal error: Killed signal terminated program cc1plus
compilation terminated.
make[2]: *** Waiting for unfinished jobs....
make[2]: *** [3rd-party/antlr/antlr4/runtime/Cpp/runtime/CMakeFiles/antlr4_shared.dir/build.make:370: 3rd-party/antlr/antlr4/runtime/Cpp/runtime/CMakeFiles/antlr4_shared.dir/src/LexerNoViableAltException.cpp.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[2]: *** [3rd-party/antlr/antlr4/runtime/Cpp/runtime/CMakeFiles/antlr4_shared.dir/build.make:230: 3rd-party/antlr/antlr4/runtime/Cpp/runtime/CMakeFiles/antlr4_shared.dir/src/ConsoleErrorListener.cpp.o] Error 1
compilation terminated.
make[2]: *** [3rd-party/antlr/antlr4/runtime/Cpp/runtime/CMakeFiles/antlr4_shared.dir/build.make:300: 3rd-party/antlr/antlr4/runtime/Cpp/runtime/CMakeFiles/antlr4_shared.dir/src/InputMismatchException.cpp.o] Error 1
c++: fatal error: Killed signal terminated program cc1plus
compilation terminated.
make[2]: *** [3rd-party/antlr/antlr4/runtime/Cpp/runtime/CMakeFiles/antlr4_shared.dir/build.make:258: 3rd-party/antlr/antlr4/runtime/Cpp/runtime/CMakeFiles/antlr4_shared.dir/src/DiagnosticErrorListener.cpp.o] Error 1
c++: fatal error: Killed signal terminated program cc1plus
compilation terminated.
make[2]: *** [3rd-party/antlr/antlr4/runtime/Cpp/runtime/CMakeFiles/antlr4_static.dir/build.make:216: 3rd-party/antlr/antlr4/runtime/Cpp/runtime/CMakeFiles/antlr4_static.dir/src/CommonTokenStream.cpp.o] Error 1
c++: fatal error: Killed signal terminated program cc1plus
compilation terminated.
make[2]: *** [3rd-party/antlr/antlr4/runtime/Cpp/runtime/CMakeFiles/antlr4_shared.dir/build.make:188: 3rd-party/antlr/antlr4/runtime/Cpp/runtime/CMakeFiles/antlr4_shared.dir/src/CommonToken.cpp.o] Error 1
c++: fatal error: Killed signal terminated program cc1plus
compilation terminated.
It looks like the termination is because of out of memory I suppose. But It would be great if you could provide some insights.
Current Memory situation [WSL2]:
12GB + 4GB Swap
Thank you in advance!

You can try make instead of make -j, then the files would be compiled in serial, which saves the memory.

My bad. Thank you!


Fix typo and duplicate line of code

LGTM.  I can't tell if this exists upstream (could be a merge artifact) but we should check.

@rachel-fenichel Duplicate code isn't in upstream (though the thw typo is).

@fsih reminder to merge your PRs once they are LG'd!

Oooops thanks!


Pager should have getTotalResults method function

I am requesting this because it is easy to make.
The problem is when i am using Paginate on model and it makes this query that counts all results from the database.
But i have no option to retrieve the Total Count and have to make another query to get countAllResults again.
So i am duplicating queries because of this.

Code for getTotalResults from Pager
/**
 * Returns the total count of the results.
 *
 * @param string|null $group
 *
 * @return integer
 */

public function getTotalResults(string $group = 'default'): int
{
	$this->ensureGroup($group);

	return $this->groups[$group]['total'];
}

This works great Surely as i see the pattern from all other functions

We already have the getDetails() method which will return a handful of data as an array - including total number of results.


:sparkles: run ci via bundle

This needs https://github.com/konveyor/operator/pull/278

Verified the changes work in https://github.com/djzager/ci/actions/runs/6786233781/job/18446292708

Now also relies on https://github.com/konveyor/operator/pull/279


Adds DataCalculator Unit Tests and new edge-cases to DataCalculator

I'm not sure why it's showing a list of all merged commits. I tried rebasing but that didn't work.
For now, you may take a look at the "Files Changed" or the last two commits after the branch was merged.

I would suggest doing the following and redo the PR:

git remote -v
from this get the remote name for datacommons.org, assuming it is "datacommons"

git status
Make sure you don't have pending changes.

git checkout -b calculation
git reset --hard HEAD^3
git pull origin datacommons --rebase
Edit the two files in this PR.
git add *
git commit -m "commit message"
git push origin calculation -u
in github, create a new PR


Window glitch on flicking image

This is a relatively small bug, but I was wondering why the status bar appears to fall down the screen after an image is flicked away. I seems to be part of the overlay that dims the background but I'm not sure. This only happens with images and gifs, but not videos using the reddit player.

I'd guess that this is default activity close animation where it slides down out of view. And that dark part might be the shadow which is visible under status bar.


[BUG] no response when call getMediaUrl on some video.

Hi All,
We face an issue on iOS. we have a video picker.
We found we cannot getMediaUrl on some device. (no response)
This issue easy reproduce on iOS "slo-mo" video.
Is there any solution ?
Tommy.

Is this still reproducible in the latest version (1.2.1)?

Also happened here. Unable to call  getMediaUrl of some videos, when apple account is signed-in in the device.
Every video works fine, if no account is logged in. Tried with latest sdk of Flutter i.e. 2.2.1 and dart 2.13.1. Also, the latest version(1.2.2) of photo manager was used.

Is this issue occurred on every slow motion videos, or is it related to some iCloud videos?

@AlexV525
For me it happens on icloud videos. Sometimes I get media url and sometimes I don't.
I also tried to filter the videos using try catch, but for some cases it does throw exception (comes in catch block) and at times it doesn't.
So its been some days I am totally lost. Now thinking to do something native in IOS to bypass this problem. This has become a deal breaker for me.
I have tried on latest flutter sdk versions as well. not sure its flutter vs IOS problem or something else.

My requirement is not that tough, all I want is to show all the photos and videos from user's phone (with or without iCloud logged in, synchronised or not synchronise) whatever user is seeing in photos app should appear in my app. And user should be able to play it within the app.
Any other solution is highly welcomed to achieve the above.

Moreover, I have also tried this library https://pub.dev/packages/photo_gallery and its method to achieve file in order to play video shows same problem (thread dies forever). Which is making me think it's not related to Photo Manager library. It is something Flutter vs IOS... 😞

@AlexV525 I'm seeing the same issue on a physical device with SlowMo being the easiest to reproduce.

Tested with some live photos and this issue seems not reproducible on my device.
Test with:
Stopwatch watch = Stopwatch()..start();
final String? url = await entity.getMediaUrl();
watch.stop();
print('Media URL: $url');
print(watch.elapsed);

Result:
flutter: Media URL: null
flutter: 0:00:00.000175


[Request] add caption words attention/emphasys for training

Is it possible to implement emphasys changes on words from the captions during training?
This is the explanation of this feature in auto.
Example: "an astronaut riding a horse on the (moon:1.5)"
Thanks in advance!
Edit: I'm seeing that there is a regex related to this in here, but would someone know if this is applied during training? Thanks!

The code from lpw_stable_diffusion.py doesn't look too hard to implement, and the negative prompts bits can be dropped.
I'll give it a shot when I have time, and ping @swfsql in the PR if it works to help do some testing for effectiveness.

Thanks, please let me know if I can be of help with the testing.
I'm somewhat new to both python, AI and SD, and new to LoRa and training and so on, but I was trying to train a LoRa on gif images, and I had no luck so far.. and I also only got 6GB of vram.
But my original thought was that I could try separating the gif files in different "duration" concepts, where I would try to correlate the actual gif duration with the attention of the "duration" token(s) according to some formula. For example, saying (gif duration:0.5) could mean the gif tagets 0.5s of duration, and saying (gif duration:1.5) could mean the gif targets 5s of duration..
I'm pretty sure this would not work, as I only had failures so far, but anyway I couldn't help but try asking if this feature existed XD But coming back to you, I can try coming with other ways to test it in case you implement it, even if it's not related to the case I was thinking originally!

gif images

Unfortunately the only format that will load are IMAGE_EXTENSIONS = [".png", ".jpg", ".jpeg", ".webp", ".bmp", ".PNG", ".JPG", ".JPEG", ".WEBP", ".BMP"]
The way I see this feature working is to emphasize foreground tokens and de emphasize background tokens.
ie
(SKS woman:1.1), (wearing a black dress:0.9), (sitting in a restaurant:0.8)
In this way, we can hopefully learn the important features more strongly than the backgrounds, without ignoring the backgrounds completely.
Best practices for captioning will have to come from experimentation, but this will hopefully give us a new tool to experiment with.

Feel free to test out the PR. Will add Dreambooth and finetuning as soon as I'm able.

Thanks @AI-Casanova , I'm looking into testing this change. I'm using the linux docker version of the code, and it seems that it's behind from some file refactors that has happened. I'll try applying the changes locally and see then try to see what differences it gives!

re: loss graphs
Loss is nearly completely determined by the number of training timesteps, which under set seed (deterministic training) track exactly. The loss graph on the PR is actually orange and yellow (thanks TensorBoard!) Tracking exactly, but producing these different results, all else held equal.

I'll be doing more testing myself once work calms down.


add support for ccp only upgrade and default to upgrade all

With the new 0801 api version, we now supports control plane only cluster upgrade. Added the control-plane-only argument for this purpose and change the default to upgrade all (by setting kubernetes version of all agentpool profiles explicitly).

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

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

If this PR is for a new extension or change to an existing extension, use the following to try out the changes in this PR:
docker run -it microsoft/azure-cli:latest
export EXT=<NAME>
pip install --upgrade --target ~/.azure/cliextensions/$EXT "git+https://github.com/xizhamsft/azure-cli-extensions.git@xizha-apclifix#subdirectory=src/$EXT&egg=$EXT"

check https://github.com/Azure/azure-cli/pull/10513/files#diff-fe2d1dd3c9fc1d841b3437f1a9aac146 for main CLI sync

ok, decided to merge it:
actually I was wondering if we could handle following scenarios, when user specified:
--control-plane-only
and
--control-plane-only False
specifically to avoid displaying prompt.
But this can be considered / checked later.


Passing params like --generate-hashes to pip-compile

My default pip-compile command is something like pip-compile --generate-hashes --allow-unsafe .... --generate-hashes guarantees that hosted versions don't change unexpectedly. --allow-unsafe, despite its name, seems to actually be the preferred way: https://github.com/jazzband/pip-tools/issues/989

Whoops, I missed https://github.com/AndydeCleyre/zpy/issues/15
Sorry for the noise

It's ok, let's iron this out here. Some notes:

I should probably add --allow-unsafe to the default flags anyway
For functions like pipc, pipacs, and pipcs, you can use the double-dash syntax to pass through flags to pip-compile, like pipc -- --allow-unsafe
For those functions, you can also pass -h to generate hashes
For ALL internal use of pip-compile, you can replace the main flags passed using zstyle:% zstyle ':zpy:*' pip-compile-args --generate-hashes --allow-unsafe --no-header --annotation-style=line --strip-extras

You can do that before running any function, or put it in your shell profile (it does not need to be before the plugin is loaded, just before you run any relevant function).
The current defaults are: --no-header --annotation-style=line --strip-extras

The other issue is specifically about adding the double dash syntax to pipz, but I don't know that it's worthwhile since the above covers a lot of cases already.
With this info, does the current state cover your needs?

Thanks for the detailed explanation!
Yes, the current state covers my needs perfectly- I always use --generate-hashes --allow-unsafe, so the zstyle option is great.


Allpad.json and most keys added

How we go about loading different files?

Merged in commit 2b55bd778bdca3417b8f126fa32838a3d867b7bd