Question: How to dynamically configure options of a command?

Is it possible to configure the option list dynamically instead of statically chaining the option() to vorpal.command()?
Background is I want to be able to create my vorpal commands from some kind of metadata objects, i.e. I want to save some work for commands which work similarly by generating those commands based on a definition.

Well, you can always go back and edit a command, or remove it and re-create it, so you could re-build commands dynamically with some sort of meta data:
const options = ['toppings', 'foo', 'bar'];

for (let i = 0; i < options.length; ++i) {
  vorpal.find('metacmd').option(options[i]);
}

This is a super simple example, but you can add as much metadata as you want to it.
Another pertinent command would be vorpal.find('metacmd').remove();

Would you mind posting this on Stack Overflow? In this way, others with the same question can benefit easier!

Thanks a lot for the fast answer. I'll post the question to SO later today.


update readme file

improve the readme file
correct linter

fixed linters config file issue*
set the right version of linters
closes #4


Use Julia 1.10 for tests

This PR changes CI to use Julia 1.10 for tests

@albertpod There is an issue with the continuous transition node, not sure what is causing it.


Is it intended BC break in unreleased version? Parameter annotations from method no more auto-bounded to parameters

In the master branch there's an unreleased (yet) commit that dropped support for Doctrine annotations - https://github.com/thecodingmachine/graphqlite/commit/870b4489f5bf5d789005c04e838a87dbbe8ea53f.
This commit also drops [1] a feature about allowing to use "method"-bound annotations (attributes) with explicit "target" (example: [2]).
The question is - is this dropping a feature intended? Are you open to reintroducing it - to simplify an upgrade path?
[1] https://github.com/thecodingmachine/graphqlite/commit/870b4489f5bf5d789005c04e838a87dbbe8ea53f#diff-b4cb271cb57cae8e3c9c55d3590bab4ea0781066f699d94ac970f89f8b7e06b4L356-L364
[2] https://github.com/thecodingmachine/graphqlite-symfony-validator-bridge/blob/2f677f6dc5c81660505e6c73952661d4d98f1199/tests/Fixtures/Controllers/UserController.php#L37

If I add this snippet locally, the logic about auto-bound annotations (attributes) from method to parameters will work as before.
// \TheCodingMachine\GraphQLite\AnnotationReader::getParameterAnnotationsPerParameter
        $parameterAnnotationsPerParameter = [];

        $attributes = $method->getAttributes();
        foreach ($attributes as $attribute) {
            if (false === \is_a($attribute->getName(), ParameterAnnotationInterface::class, true)) {
                continue;
            }

            $parameterAnnotation = $attribute->newInstance();
            $parameterAnnotationsPerParameter[$parameterAnnotation->getTarget()][] = $parameterAnnotation;
        }

I think this is an unnecessary BC break for v7.
Here's a PR that reintroduces the feature https://github.com/thecodingmachine/graphqlite/pull/709

@andrew-demb dropping annotations wasn't a mistake and is the path forward.  Is there any reason why someone cannot switch to attributes?  There is even a rector that'll do it all for you...
https://getrector.com/blog/how-to-upgrade-annotations-to-attributes
Aside from that I'm not entirely sure what the issue is here.  Isn't this bundle just a wrapper around the Symfony Validator lib?  Maybe you can clarify the issue here.  I'm not a fan of this parameter targeting when the proper way to "annotate" is to put the attribute with the param, directly.  I'm also not a fan of GraphQLite jumping through hoops to accommodate bundles, especially when encouraging incorrect design practices.

@oojacoboo this library provides a binding for symfony/validator .
I made a PR just because a stable release of thecodingmachine/graphqlite (7.0.0) already supported doctrine/annotations. And there's a stable release of thecodingmachine/graphqlite-symfony-validator-bridge (7.0.0) with test, that works fine with 7.0.0 [1].
But this test fails with the current dev-master of thecodingmachine/graphqlite (because of dropped support for "target method" annotations).
So I try simplifying the upgrade path and minimizing the number of BC breaks for developers that already use thecodingmachine/graphqlite-symfony-validator-bridge 7.0.0.
[1] https://github.com/thecodingmachine/graphqlite-symfony-validator-bridge/blob/v7.0.0/tests/Fixtures/Controllers/UserController.php#L37

To be clear , my question was about "target method" parameter annotation, not "doctrine/annotation".
Thanks in advance.

@andrew-demb I'm really not a huge fan of this.  But I'll allow it this time.  The bundle needs to make clear that this is deprecated and to be removed in 8.0.  Can you please add phpdoc deprecation tags to your PR here as well, so it's searchable for deprecation removal.  Thanks.

Agreed; imho target was only ever intended for annotations and not attributes, so I wouldn't consider it as a different breaking change from annotations.

@andrew-demb I'm really not a huge fan of this. But I'll allow it this time. The bundle needs to make clear that this is deprecated and to be removed in 8.0. Can you please add phpdoc deprecation tags to your PR here as well, so it's searchable for deprecation removal? Thanks.

Here's a PR for thecodingmachine/graphqlite-symfony-validator-bridge for providing a way to use the attribute as a target to parameter and a soft deprecation of using it as a target to method.
https://github.com/thecodingmachine/graphqlite-symfony-validator-bridge/pull/75

Thanks to @andrew-demb for the work done! :+1: @oojacoboo , @oprypkhantc I assume nobody here wants to maintain useless code. The reason why @andrew-demb has raised this issue is that he was working on adding support for Symfomy 7 to the graphqlite adapter. I believe the intention of everybody is for good but what I can see here is just some miscommunication and misunderstanding. I don't think @andrew-demb likes to pitch committing deprecated code to repo. So it looks like the whole discussion would be much shorter or even would not exist if there was a clear BC breaking/versioning strategy explained on README/contribution guide. Based on the review of code @andrew-demb, he assumes the server strategy applied, but most likely the adapter versioning is tried to be aligned with symfony versions or so. Let's discuss and develop versioning strategy and it can heavily influence the whole purpose of https://github.com/thecodingmachine/graphqlite-symfony-validator-bridge/pull/75

@oojacoboo What do you think of setting up an automatic release action using https://github.com/semantic-release/semantic-release?
It automatically releases new versions, taking into account types of commits. Fixes and performance commits are patch versions, features are minor versions and anything with a BREAKING CHANGE message in the commit description are major versions. It also keeps the CHANGELOG file up-to-date, and creates a GitHub release. So aside from merging PRs, you wouldn't have to do anything and versions would me released immediately after merging a PR.

@oprypkhantc I think this is a good idea.  Releases for GraphQLite are a bit tedious with the website versioning and the release changelog.  At the moment, that's all required to be done manually.  If you (or someone else) want to set that up, that'd be awesome.  We could give it a shot on this release.  It sounds like everyone wants to go with 8.0 on this release due to the removal of Doctrine annotations.  We'd need to update the README with PR/commit message requirements.

@oojacoboo what target branch should be used to prepare the 7.1.0 (added deprecations and upgrade path)?
Is this branch (aka 7.x) should be created from v7.0.0 (https://github.com/thecodingmachine/graphqlite/commit/19458fabf2113ac75f67f157a8d6c73d47a02ecc)?

@andrew-demb we're pushing to master and targeting 8.0.0.  There won't be a 7.1 release.

PR about reverting BC break for parameter annotations merged - this issue is resolved.


[master] Add insecure command to Windows cluster registration

This adds an insecure command to Windows cluster registration by adding a checkbox that updates the code snippet when checked. This is made possible by recent changes on the backend that introduces an insecure Windows command by updating from iwr to curl.exe.
#3580

Here are some screenshots to go along with the change:

cluster ready:

insecure checked:


chore: update repository template to 781e8fb2

Updated repository templates to https://github.com/ory/meta/commit/781e8fb26afe90cedb50a939ad3fb6fa3798eefd.

Coverage remained the same at 43.429% when pulling 31d8ea939bdbb0c252152f53a72d41c4dccfd2a9 on meta-11-20-20-16-38-25 into 6ca73fce478c23047feb6c63b377512a3d1bb9d4 on master.


Privacy deprecation warnings on CLI

PR description

Add deprecation messages to help descriptions
Add warning log if privacy-enabled option is specified

Note I did not add any @Deprecation annotations to types or methods - feedback welcome on whether you think this is needed.
Fixed Issue(s)

Thanks for sending a pull request! Have you done the following?

[ ] Checked out our contribution guidelines?
[ ] Considered documentation and added the doc-change-required label to this PR if updates are required.
[ ] Considered the changelog and included an update if required.
[ ] For database changes (e.g. KeyValueSegmentIdentifier) considered compatibility and performed forwards and backwards compatibility tests

Locally, you can run these tests to catch failures early:

[ ] unit tests: ./gradlew build
[ ] acceptance tests: ./gradlew acceptanceTest
[ ] integration tests: ./gradlew integrationTest
[ ] reference tests: ./gradlew ethereum:referenceTests:referenceTests

After a conversation with @jflo I added Deprecation annotations to the privacy plugin classes since there is active ongoing development with plugins generally


Chutes incorrectly remove music discs from jukeboxes

Description
If a Chute is placed under a Jukebox, and a Music Disc is placed in the Jukebox (manually) then the Chute will remove the Music Disk, but it bypasses Lnet/minecraft/block/entity/JukeboxBlockEntity;removeStack(II)Lnet/minecraft/item/ItemStack; which results in the Jukebox continuing to play and continuing to have the has_record property be set to true.
Game Log
N/A
Debug Information
No response

fabric API implements transfer support for the jukebox, report there

are you on the latest version though? support should've been fixed ages ago

Fabric api: 0.92.2
Create Fabric: 0.5.1-f-build.1417

Was supposedly fixed in FAPI 0.92.1


Please invite me to the Recode-Hive GitHub Community Organization

Name
manishika
Discord Username (if applicable)
No response
Additional Context
No response

Invitation sent to join the GitHub Organisation. Welcome to the community 🎉
Don't forget after accepting to make it public so it appears on your GitHub profile for everyone else to see. You can do this by finding your name in the GitHub organisation list and change the dropdown to public https://github.com/orgs/recodehive/people


[FLINK-27376][sql] Support current_database function

What is the purpose of the change
To support current_database() function in Flink
Brief change log

Add CURRENT_DATABASE definition in BuiltInFunctionDefinitions.java  and FlinkSqlOperatorTable.java
Add method currentDatabase to call CURRENT_DATABASE in Expressions.java to make current_database can be used in TableAPI.
Before translation, set a configuration with value current database in PlannerBase. And  read the configuration value put to get the current in codegen phase for current_database() function.
Disable current_database in HiveMoudle, so that it'll always use Flink's built-in current_database function.

Verifying this change
Added test in testCurrentDatabase, testCurrentDatabase, testCurrentDatabase, testCurrentDatabase for batch/stream, sql/table.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature?  Yes
If yes, how is the feature documented? Yes

@fsk119 Thanks for review. Now, I add function current_database  for flink itselft. So that both Flink sql and Hive dialect can use it.

@flinkbot run azure

@fsk119 The ci is passed now. Could you please help merge when you're free?


fix travis tests broken due to 3.5, pypy issues due to dist: trusty switch

Also changed mongodb as service as dist:trusty was not able to handle it right.
3.5 broken:
https://github.com/pytest-dev/pytest/issues/2779
pypy in trusty:
https://github.com/travis-ci/travis-ci/issues/6865
Changes to collection.py in pymongo, currently stubbed.
https://github.com/mongodb/mongo-python-driver/commit/8fdb581c6a6ba16678a67ec192972ad430bb6e67

@vmalloc @mdomke  could you please review and let me know any comments.

LGTM.


torch 1.7.1 supported?

System:
OS: Ubuntu 18.04
Python version: 3.7.6
PyTorch version: 1.7.1
Torchvision version: 0.8.2
CUDA/cuDNN version: 11.1/8.0.4
GPU: RTX3080
I tried running python setup.py install , but I experienced the following: Searching for torch<=1.7.0,>=1.5.0
so it does not support torch 1.7.1?

Solved by #389


Support skipping unsupported SASL mechanisms

Fixes #2, allows controllers to skip tests for unsupported SASL mechanisms.
Used by adding a line like this to the controller's __init__():
self.supported_sasl_mechanisms = ['PLAIN']

Haven't looked into the table stuff yet since I'm not as familiar with unit testing, this just does the test skipping.

By the way, if you want me to squash this just let me know.

No, that's fine. Thanks!


Cleanup login state request loop shutdown

This patch simplifies the shutdown of the login state request loop.
There were some CI failures with sends to closed channels.  Those channels were closed and then set to nil (the nil being to block sends), but some operations were sneaking in between those two statements.
Instead of that, this just adds a shutdown channel that will stop the loop.  The request channels are still there, but unbuffered, so they will block like before when they were closed and set to nil.
Tried repeatedly in CI and it passed.

nevermind, still an issue

Current issue is that signup is taking longer than 5s, causing login state request to timeout waiting for it to finish.

Ok, PTAL


Touch event of Tabbar

Touch event of TabBar is smaller when we have 2 or 3 items. Only able to tap on home icon or search. Touch event needs to add with title also.

hi, I can't reproduce that issue :/ It also has touch event the entire BarItemView, not just the icon. If barItem is not selected, its title is hidden.

Pl check this video
Thank you
Sanjay Mali
Mob: +917506630983
http://thecodeacademy.in/, www.theappmaker.in
Create your own email signature
https://chrome.google.com/webstore/detail/free-email-signature-gene/adabkammndnejioekmmdhgdnlfenpkci
On Sat, Dec 25, 2021 at 6:09 PM Yuşa Doğru @.***> wrote:

hi, I can't reproduce that issue :/ It also has touch event the entire
BarItemView, not just the icon. If barItem is not selected, its title is
hidden.
[image: Screen Shot 2021-12-25 at 15 38 32]
https://user-images.githubusercontent.com/19792753/147384949-7148b9d1-0cc2-4645-88d2-454425f658b3.png
—
Reply to this email directly, view it on GitHub
https://github.com/yusadogru/CardTabBar/issues/1#issuecomment-1001012643,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACD75WUOIWDLJ2Y3ARU37GDUSW3RFANCNFSM5KRWXKZA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>


Classifier-Free Guidance, cond_drop_prob=1.0, attn_mask=False: Error!!!

When using text conditions, even if the parameter text_condition_cond_drop_prob is set to 0.25 when initializing the MeshTransformer, it is easy to overlook the cond_drop_prob parameter in MeshTransformer.forward_on_codes(cond_drop_prob = 0.).
_, maybe_dropped_text_embeds = self.conditioner(             text_embeds = text_embeds,             cond_drop_prob = cond_drop_prob         )
Accidentally retaining the default parameter cond_drop_prob = 0. means that text conditions are not properly dropped out during training, which is not conducive to the subsequent use of Classifier Free Guidance.
It is recommended that the author sets the default parameter of cond_drop_prob to None.  @lucidrains
However, most importantly, when I set MeshTransformer.forward_on_codes(cond_drop_prob = 1.0), the mask for text_embedding is all false, meaning that during the calculation of cross-attention, attn_mask = False, which causes the output to be NaN.
How can the above issues be resolved ???
import torch.nn.functional as F  out = F.scaled_dot_product_attention(               q, k, v,               attn_mask = mask,               dropout_p = self.dropout if self.training else 0.,                is_causal = causal           )
It seems that attn_mask cannot be False at all positions !!!

@fighting-Zhang thanks for catching the first point, made the fix
as for the second, i think the latest x-transformers should handle all rows being masked out, but i added my favorite extra precautionary measure (appending a few memory key/values to cross attend to)

best with your research


enable ingress controller and disable adservice monitoring

Changes
Required modifications:
  Change the host to point your IP or public dns: otel-demo.<IP_ADDRESS>.nip.io

Required commands:
// install the nginx ingress controller
$ helm install --namespace kube-system nginx ingress-nginx --repo https://kubernetes.github.io/ingress-nginx
// install the demo helm chart with the ingress extension
$ helm install -f deployment.yaml -f deployment_ingress.yaml my-otel-demo open-telemetry/opentelemetry-demo

Merge Requirements
For new features contributions, please make sure you have completed the following
essential items:

[ ] CHANGELOG.md updated to document new feature additions
[ ] Appropriate documentation updates in the docs
[ ] Appropriate Helm chart updates in the helm-charts

Maintainers will not merge until the above have been completed. If you're unsure
which docs need to be changed ping the
@open-telemetry/demo-approvers.

Can we add commands listed in the PR description to the README?

Definitely, but I think we should first decide what is the scope of these changes:

Do we want to enable the ingress route + disable some service with the same configuration file?  What should be the name of this file? Which description we should provide in the README? I reckon it would be the best if we explain the reasoning behind explicitly disabling the corresponding services.

Maybe documented under a README section named "RCA workshop"?

Do we know which services we want to disable for the RCA workshop?

@andrewvc

Thanks, @rogercoll -- @LucaWintergerst has put together a gist of the workshop modifications we'd like (it's the entire deployment.yaml with changes made, not sure if that's best or just having a separate file with changes only), but without the change above re: the ingress controller.
And yes, we should either move this to a branch or explicitly mark it as RCA workshop or similar.

not sure if that's best or just having a separate file with changes only

I would suggest better using two files if possible, the second file that we provide for the RCA workshop should only contain the changes for this specific environment. That way we benefit from the main file maintained configuration too (in case we add new envs, extended collector config, etc.).

@rogercoll I'm also not super familiar with nip.io—the idea is that each engineer needs to modify the yaml file and change that hostname to include their own public IP?

Exactly, basically by using nip.io (I guess there are other public similar services) you avoid having to modify your hosts file, it should work for private ip addresses too (I think we can remove it and just rely on mofing the hosts file). This PR was filled just to showcase the enablement of the demo's ingress controller, but please feel free to open a new one if it's easier for you to contribute these changes.

If it works for you, I would close this PR in favor of this branch: https://github.com/elastic/opentelemetry-demo/tree/rca_ingress_obs


Add Chinese-Simplified Translation

Translation added but I failed to test (App crashed for missing libetopan.so)

The Android app requires libetopan. For debugging/testing you can setup Rust and Android NDK and execute ./run android, which compiles the required library. In the app build.gradle resourceConfigurations was missing 'zh-rCN'. I added and tested it now.


Use ReactDOM webpack external

Summary

Ticket Link
Fixes https://mattermost.atlassian.net/browse/MM-47046

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.


Bugfix/enable index metrics

Packages impacted by this PR
@azure/cosmos
Issues associated with this PR
This PR tends to enable index metrics for aggregate queries in node/JS sdk
Describe the problem that is addressed by this PR
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Are there test cases added in this PR? (If not, why?)
Yes
Provide a list of related PRs (if any)
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[ ] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[ ] Added a changelog (if necessary)

API change check
API changes are not detected in this pull request.


An error occurs when I execute npm run dev

npm version: 6.11.3
node version: v10.17.0
Failed to compile.

./node_modules/_material-survey<EMAIL_ADDRESS>Module not found: Can't resolve 'styled-components' in '/home/owen/git/my-app/node_modules/_material-survey@1.0.40@material-survey/components/APIAutocompleteQuestion'
/home/owen/git/my-app/node_modules/react-scripts/scripts/start.js:19
  throw err;
  ^

Error: ENOENT: no such file or directory, stat '/initrd.img'
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start: `react-scripts start`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/owen/.npm/_logs/2021-01-18T08_24_44_436Z-debug.log

My operation is as follows：
npx create-react-app my-app
cd my-app
npm start
npm install react-image-annotate --save

src/app.js  :
import ReactImageAnnotate from "react-image-annotate";

Node 10 isn't supported (it's being deprecated soon)

Node 10 isn't supported (it's being deprecated soon)

When I use NVM to upgrade node to 12.20.1 and 14.15.4, the error still exists

When I use NVM to upgrade node to 12.20.1 and 14.15.4, the error still exists


feat: update vite.config.ts

fix: modify right base url
What is it?

[ ] Feature / enhancement
[ ] Bug
[ ] Docs / tests
[ ] Other

Why is it needed?

Checklist:

[ ] My code follows the developer guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have ran pnpm change and documented my changes
[ ] I have add necessary docs (if needed)
[ ] Added new tests to cover the fix / functionality

@gioboa Who can merge this PR?
Because my search doc PR needs this PR's code

Nice 👏 @JerryWu1234

✅ Done

Nice 👏 @JerryWu1234

✅ Done

when will we release a new version?

when will we release a new version?

Let's talk about this new version in the Qwik UI Discord server. Thanks


Failed to Resolve with index.tsx files

With the following structure:
  src/common/components/SafeContainer
├── constants.ts
├── elements.tsx
├── index.test.tsx
└── index.tsx

Then in another file I have:
import { SafeContainer } from "@/common/components/SafeContainer";

This is my tsconfig for the aliasing:
{
  "compilerOptions": {
    "baseUrl": ".",
    "jsx": "react",
    "esModuleInterop": true,
    "paths": {
      "@/*": ["./src/*"],
      "@assets/*": ["./assets/*"]
    },
    "strict": true
  },
  "exclude": ["node_modules", "dist"]
}

After running skott index.js --verbose I see
Start resolution for src/common/components/SafeContainer
 ✖ Failed to resolve src/common/components/SafeContainer/index.js

 ✖ Failed to resolve src/common/components/SafeContainer.ts

 ✖ Failed to resolve src/common/components/SafeContainer/index.ts

 ✖ Failed to resolve s.ts

 ✖ Failed to resolve src/common/components/SafeContainer.js

 ✖ Failed to resolve src/common/components/SafeContainer.tsx

 ✖ Failed to resolve src/common/components/SafeContainer.jsx

 ✖ Failed to resolve src/common/components/SafeContainer/index.js

Looks like library is not checking index.tsx/index.jsx paths?
Thanks for your help.

@robertoyoc thanks for opening that issue! It's indeed correct, skott is not checking index.jsx/index.tsx modules yet. TSX/JSX support was added incrementally, and this is something I completely forgot.

Fixed in #121 by @robertoyoc, will be released in 0.31.3 🎉
@all-contributors please add @robertoyoc for code


Apply various Clang-Tidy rules

The issues were identified Clang-Tidy, but where addressed individually.
The fixes are mainly manual, partly because Clang-Tidy cannot  fix the issues automatically but also because some fixes do not apply to qpdf and therefore require suppression of warnings rather than fixes.

I'm looking forward to studying this. I am going to learn things. I am not aware of emplace, for example.

I find the modernize-use-emplace rule quite interesting. Theoretically using emplace should improve performance as it avoids unnecessary copying/moving. The reality is that for both clang-15  and gcc-12 it doesn't make a difference performance wise. At some level this makes sense: if clang-tidy can detect that emplace should be used, why would clang not optimize the unnecessary copy away? I guess this explains why the rule is called "modernize-use-emplace" rather than "performance-use-emplace".
I still feel that it is useful to apply the modernize-use-emplace rule for three reason:

it expresses in code more clearly the cost of the operation
it is likely to improve code quality metrics for qpdf
it may well have a significant impact for users of other/older compilers

just I ran ./abi-perf-test at various stages, and reran it now to make sure.


Update site

Docs Build status updates of commit 4050a8b:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/MicrosoftDocs/AdaptiveCards, working directory: W:\gc3e-s, target branch: live, target branch commit: 99c7b64d6fc66da336c454951406fb42cd2a7427, source commit: 4050a8bbba7ee8683d0355ae0bb04f3c998cf150.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 21ac9bb:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

AdaptiveCards/authoring-cards/card-schema.md
:white_check_mark:Succeeded

AdaptiveCards/index.md
:white_check_mark:Succeeded

AdaptiveCards/rendering-cards/extensibility.md
:white_check_mark:Succeeded

AdaptiveCards/rendering-cards/getting-started.md
:white_check_mark:Succeeded

AdaptiveCards/rendering-cards/implement-a-renderer.md
:white_check_mark:Succeeded

AdaptiveCards/rendering-cards/renderer-status.md
:white_check_mark:Succeeded

AdaptiveCards/rendering-cards/speech.md
:white_check_mark:Succeeded

AdaptiveCards/resources/future.md
:white_check_mark:Succeeded

AdaptiveCards/resources/partners.md
:white_check_mark:Succeeded

AdaptiveCards/resources/tools.md
:white_check_mark:Succeeded

AdaptiveCards/sdk/rendering-cards/android/actions.md
:white_check_mark:Succeeded

AdaptiveCards/sdk/rendering-cards/android/extensibility.md
:white_check_mark:Succeeded

AdaptiveCards/sdk/rendering-cards/android/getting-started.md
:white_check_mark:Succeeded

AdaptiveCards/sdk/rendering-cards/android/native-styling.md
:white_check_mark:Succeeded

AdaptiveCards/sdk/rendering-cards/android/render-a-card.md
:white_check_mark:Succeeded

AdaptiveCards/sdk/rendering-cards/ios/render-a-card.md
:white_check_mark:Succeeded

AdaptiveCards/sdk/rendering-cards/net-html/render-a-card.md
:white_check_mark:Succeeded

AdaptiveCards/sdk/rendering-cards/net-wpf/actions.md
:white_check_mark:Succeeded

AdaptiveCards/sdk/rendering-cards/net-wpf/extensibility.md
:white_check_mark:Succeeded

AdaptiveCards/sdk/rendering-cards/net-wpf/host-config.md
:white_check_mark:Succeeded

AdaptiveCards/sdk/rendering-cards/net-wpf/render-a-card.md
:white_check_mark:Succeeded

AdaptiveCards/toc.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Create PDF export function

Description
When the user does a harmonisation, we currently only have an Excel export. Can we make a PDF export using libraries such as pdfkit and airium so that there is a human readable PDF report of how many items were harmonised, etc?
Rationale
This is easier to understand for some users than the matrix in the Excel and allows us to include graphics

My peers and I will work on this issue.

Thanks Evelyn! If you want to discuss any of this feel free to reach out to
me on Harmony's Discord, my name is Thomas Wood
On Sat, 23 Nov 2024, 18:49 Evelynn Chen, @.***> wrote:

My peers and I will work on this issue.
—
Reply to this email directly, view it on GitHub
https://github.com/harmonydata/harmony/issues/53#issuecomment-2495619372,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADUBTVLHSWYFHXJF3K2NWJ32CDE5JAVCNFSM6AAAAABQIUEXQCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOJVGYYTSMZXGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

My peers and I will collaborate to resolve this issue.

Thanks @narendrakumar9867 ! Happy new year!


Navbar not displaying properly in all pages

navbar is not displaying in all pages, in some pages it is acting as mobile view navbar
https://github.com/akshitagupta15june/PetMe/assets/169196836/d8d2afd2-6ebf-4ee6-a904-eba68ddcae87

i would like to fixed this issue

someone is working on this


Added Simple check for gunicorn package

Hi,
Thanks for the great work. During deployment, it turned out that gunicorn package is needed to be installed or service  would fail on start. Added a simple check to notify user if the package is not installed during the deployment.
Regards,
Rafi

Thanks for updating the readme. This commit was merged.
But the second one is not needed because the deploy script installs the gunicorn itself.


Add frontend karma jasmine tests

This adds karma jasmine tests.
To test, run npm run test.
The packages had to be updated to get tests to work.
New packages had to be added to get tests to work.
Rename the default log file from vireo.log to sage.log.

Coverage remained the same at 24.205% when pulling f7eaab5c1f9d6334de9e979cdcc7bd3c023a43ae on sprint-sep18-test_improvements into 820822bc13f8b9a16b0e35988ec745d6daae38dd on sept-dame-sprint-staging.


Numeric extensions don't supported with vanilla DependencyLib

Example: https://jsfiddle.net/kxrvzcxy/
Each 'keydown' event cause an error 'An invalid or illegal string was specified' at line #92 of inputmask.dependencyLib.js. It caused by illegal calling at line #392 of inputmask.js. After the commit 15c9202 there is no more this binding, but inputmask.numeric.extensions.js relies on that.

@artptr ,
Thx for reporting.  This was also a problem with other dependencyLibs, but they didn't bring it forward.
I will add the vanilla dependencyLib to the unittests.


[FT-Auth] Request Password Reset using Registered Email

Is your feature request related to a problem? Please describe.

User can reset password if they have forgot it or just feel like resetting

Describe the solution you'd like

Write code for resetting password
Provide reset password endpoint with JWT tokens

Describe alternatives you've considered

N/A

Additional context

N/A

This is already done!


MAINT: local Python.h lint failures

As mentioned by Tyler somewhere recently, I can also see
Header before Python.h in file scipy/ndimage/src/nd_image.c
Python.h on line 37, other header(s) on line(s) [36]
Python-including header not used before python constructs in file scipy/odr/__odrpack.c
Construct on line 55
Python-including header not used before python constructs in file scipy/special/_gufuncs.cpp
Construct on line 90
Python-including header not used before python constructs in file scipy/special/_special_ufuncs.cpp
Construct on line 825

locally. We should investigate why these are occurring locally but not in CI.

I now can't reproduce these.

Those files all look fine as far as I can tell. I suggest we close this since it's not actionable. In case it happens again and you can reproduce locally, it'd be good to get more details.


Bandage using

I tried your new released and when we use a bandage with ox_inventory, that’s healing but we got infinite bandage, if we have 1 bandage we can use it infinitely.

And we have an error with registerhook in ox_inventory it’s said that he does not exist


parser: cherry pick #975

What problem does this PR solve?
This PR cherry picks #975
Related PR: https://github.com/pingcap/tidb/pull/20837/
What is changed and how it works?
Check List
Tests

Unit test

Code changes

None

Side effects

None

Related changes

None

@wjhuang2016 @tangenta PTAL


add support for stable lm/ stable coder models

would love to see support for this model, its really good for its size
https://huggingface.co/stabilityai/stable-code-instruct-3b

Working on an automatic model optimizer :)

Nice, new stable lm 12b looks promising


Refactor buffer implementations and add ByteBuffer backed implementations

This PR is a significant improvement to available Bytes and Buffer implementations, providing alternatives to Unsafe usage via Java's ByteBuffers. It also ensures that byte order is consistent across buffers.

Coverage increased (+2.7%) to 50.925% when pulling ce9bcafce050081343698576a9c41948485445d5 on byte-buffer-buffers into 1f12eb1964ee1569a0bae1ca2bd7ebbe07c1281e on master.


[BUG] Keep default options consistent between Spring and Java SDK

Context
In Spring configuration properties, it's common that there is explicit default-value assignment, e.g., the load-balancing strategy. However, this can cause the default values of Spring and Java SDK become inconsistent when SDK changed their default options. Referring to https://github.com/Azure/azure-sdk-for-java/issues/25039
Solution
Remove those default options in Spring properties and hand over to SDK.

All the configuration component provided by Spring Cloud Azure can be devided into 3 types as below:
Configuration with Azure SDK:

[x] app configuration
[ ] cosmos & data cosmos
[ ] service bus
[ ] event hubs
[ ] kv cert
[ ] kv secret
[ ] storage blob
[ ] storage file share
[ ] storage queue
[ ] resource manager
[ ] Common properties

[ ] authN
[ ] client
[ ] profile
[x] proxy
[ ] retry

Configuration without Azure SDK

[ ] aad & b2c --> omit? What if the default value of global(common) property changes
jdbc  --> covered by global properties
event hubs for kafka  --> covered by global properties
[ ] service bus for jms  --> need to be aligned with Spring JMS or SB SDK?
redis -> covered by resource manager

Other modules

cloud foundry
compatibility


Feature Request: Exit button or similar

I'd really like to see some sort of button or menu option to exit Nightcode. I ask for this as I use the program in a crouton integration window on my chrome desktop. If i close the crouton integration window using the X button on the chrome window (rather than a button in the program within that window) then the program doesn't actually close and becomes unreachable. To make matters worse, I cannot reopen the program without force terminating my chroot.

what about Ctrl-q ?

That does the trick. Thanks.
I still think there ought to be something visible that shows this shortcut.

I'll try to think of a place to show the shortcut, but since it corresponds to the system's window close button I can't display a hint over it.


Add shared encryption controllers logic

Signed-off-by: Monis Khan<EMAIL_ADDRESS>@sttts

/retest
Unrelated flakes.

Fixes as follow-ups.
/lgtm
/approve


Feature/sort imports

Configure the sort imports plugin to sort based on the codestyle and checkstyle.
Turned on the checkstyle import sort check rule to enforce the order
You can now feel free to let any IDE sort imports based on the codestyle

Just to confirm

the impsort's default search just happens to be the google recommended sort order?
Expected the re-sort to maintain the same number of lines, taking a quick look, are we just losing 300 lines of whitespace or is there something more significant going on?

yep
yep, losing whitespace. Google just has two chunks of imports: static and non-static (all sort alphabetically). We had javax, java, org, static


Update CODEOWNERS so that we don't accidentally deploy a new version of react-native-onyx

Problem
react-native-onyx has had this PR merged to main which adds SQLite support, but we are not ready to deploy it to App yet.
Solution
I will review all PRs with changes to package.json to ensure it doesn't get deployed.
Tests
None
Offline tests
None
QA Steps
Internal QA:

Create a PR that modifies package.json
Verify that I am assigned to it for review

PR Author Checklist
Checklist doesn't apply to any of the changes in this PR
Screenshots/Videos
Screenshots don't apply to this PR

I'll go ahead and merge this since it's a pretty fair concern. The failing tests are OK since the checklists don't apply to the changes in this PR.

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by @tgolen in version: 1.2.48-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by @luacmartins in version: 1.2.48-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by @luacmartins in version: 1.2.49-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


[#9] SSR requests fail for core components 2.11.1

Fix for #9

Can you also add tests for new code

Hi @SushantBhasin, I've added tests to cover this and other methods already delegated to the core components Page.


Compatibility with sealed_value feature

When using sealed_value, the compiler fails with the following error:
[error] examplepb/example/Cat.scala: Tried to insert into file that doesn't exist.
[error] examplepb/example/Dog.scala: Tried to insert into file that doesn't exist.

Presumably, because the generated type is no longer in its own top-level file due to the behaviour of sealed_value.
NOTE: this is only adding a failing test case to demonstrate the issue. I can take a stab at fixing this with some additional guidance.

Thanks for reporting and providing an example case. The fix for this issue is in ScalaPB's compilerplugin and will become available on the next release. I pushed it to both ScalaPB 0.10.x branch and master.


CMake add ability to use WINSSL

When setting USE_OPENSSL=OFF you have to set also CMAKE_USE_OPENSSL=OFF to tell CURL to use alternative SSL implementations like winSSL.

https://github.com/whoshuu/cpr/blob/c8d33915dbd88ad6c92b258869b03aba06587ff9/test/CMakeLists.txt#L54-L61
Shouldn't this be changed to:
 if (USE_OPENSSL)

or
 if (SSL_ENABLED)


Issue with Annotation Key Points in CVAT

Actions before raising this issue

[X] I searched the existing issues and did not find anything similar.
[X] I read/searched the docs

Steps to Reproduce
No response
Expected Behavior
Hello CVAT team,
We are experiencing a problem with annotation key points. Once we finish annotating, the key points change position when we navigate to the next or previous image. Could you please investigate this issue and provide a solution.
Thank you.
Regards,
Sandesh Pavan
Possible Solution
No response
Context
No response
Environment
No response

Hello CVAT Team,
I Haven't received any updated from your end, can you please provide a quick action regarding this issue.
Hoping for a positive response.
Thank you.

Hello,
It is not clear from your explanation what is the problem.
Please, provide more details: steps to reproduce, screenshots, screencast, task id, organization id, username on app.cvat.ai.
Thank you.

Hello,
Please find the attachment of key point annotation screencast video and screenshot, which is being scattered while working on each task and tried with different Id's still we are facing the same issue.
Screencast from 02-07-24 05_07_20 PM IST.webm
Screencast from 02-07-24 05_11_09 PM IST.webm

Steps to reproduce:

Login ID:
Job ID:
Clonk on Skeleton keypoints

User Id's and Task Id's are mentioned below :
user Id 1: neha.anjum
Mail Id1<EMAIL_ADDRESS>Job Id: 976523
User name 2: kusuma.cb
mail Id 2<EMAIL_ADDRESS>Job Id: 976498
Thank you.

Hello,
It is not clear from your explanation what is the problem. Please, provide more details: steps to reproduce, screenshots, screencast, task id, organization id, username on app.cvat.ai.
Thank you.

Hello,
We have shared some data samples, screencasts and screenshots, kindly check and do the needful or do we have any possibility to sync up for call to explain this issue briefly ?

On frame 6 you have skeleton shape with id 592
On frame 7 you have skeleton shape with id 734
They are two different and independent objects because they are shapes.
Adjusting one of them will not affect another one.
From your description, it looks like you need to annotate skeletons using tracks, not shapes.
Please, read in CVAT documentation the difference between shapes and tracks.
https://docs.cvat.ai/docs/manual/basics/track-mode-basics/
https://docs.cvat.ai/docs/manual/advanced/track-mode-advanced/
https://docs.cvat.ai/docs/manual/basics/shape-mode-basics/
https://docs.cvat.ai/docs/manual/advanced/shape-mode-advanced/


[BUG] Error installing certificate on Linux

This fails with the following error:
Error installing certificate:  No polkit authentication agent found.
Error occurred in handler for 'CERTIFICATE_SERVICE_INSTALL_CERTIFICATE_AUTOMATICALLY': Error: No polkit authentication agent found.
    at Object.throw (/tmp/.mount_ProxymPqXNXx/resources/app.asar/dist/main/webpack:/proxyman/src/main/services/certificate-service.ts:79:17)
    at Generator.throw (<anonymous>)
    at o (/tmp/.mount_ProxymPqXNXx/resources/app.asar/dist/main/webpack:/proxyman/src/main/services/certificate-service.ts:13:24)

Screenshot:

Log file:
install_certificate_error.log
Also, when "Open Document" in the error dialog box one is referred to "https://docs.proxyman.io/proxyman-windows/install-certificate", i.e. instructions to manually install a certificate on Windows, rather than on Linux. Unfortunately I didn't find an equivalent version for Linux. I even tried guessing a url, i.e. "https://docs.proxyman.io/proxyman-linux/install-certificate" but this doesn't seem to exist.

Hi @seidnerj what Linux distro (and version) you're using?
Currently, Proxyman Linux works well with the Ubuntu (20 or 22). For others, it depends

@NghiaTranUIT, I am using Ubuntu 22 running on Azure and connecting via X11:
Linux  6.5.0-1016-azure #16~22.04.1-Ubuntu SMP Fri Feb 16 15:42:02 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
It would be great if you could at least tell me how to generate/install a certificate manually on Linux in the meanwhile.
I also couldn't find the "Auto Start Recording Traffic at Launch" feature in he UI, could I edit config.json manually and switch this off?

@seidnerj  You can turn this off and the application will not record the traffic at launch.
This is in the Preferences dialog, click Ctrl+, or go to Proxyman -> Preferences (at the top left) to open this dialog.

@seidnerj You can turn this off and the application will not record the traffic at launch. This is in the Preferences dialog, click Ctrl+, or go to Proxyman -> Preferences (at the top left) to open this dialog.

That's the the override for the proxy settings, not the start recording...

Having same issue. Is there any update on this one?

@runetech0 What Linux distro you're using?
I will add a Manual Tab to help you manually install the certificate your Linux Trust Store

@runetech0 can you install lxpolkit to your system?
I tested on Ubuntu 22 and it's already installed in the system. Not sure why you don't have it.

@runetech0 What Linux distro you're using?
I will add a Manual Tab to help you manually install the certificate your Linux Trust Store

I'm using Ubuntu 24 with i3wm.
I have already tried installing lxpolkit but no luck.

What is the workaround? I have a Pixelbook, which uses Debian 12.


Bug or crash

After press play (demo or full version) appears black screen with control...but disappear after 1 second and return on boardwalk without any error report. My device is: Huawei Mate 8 (NXT-L29) 1920x1080, 366dpi, ART2.1.0, Android 6.0, opengl es 3.1 and 3GB ddr [ram]

1.2b relase fixed bug


Don't use muvaffak's personal resource group for SQL instances

Description of your changes

Checklist

I have:

[x] Run make reviewable to ensure this PR is ready for review.
[x] Ensured this PR contains a neat, self documenting set of commits.
[ ] Updated any relevant documentation, examples, or release notes.
[ ] Updated the RBAC permissions in clusterrole.yaml to include any new types.

side note: this makes me wonder how many of these examples still work and are correct with up to date paradigms 🤔

this makes me wonder how many of these examples still work and are correct with up to date paradigms

We've been updating and checking them as we go (for example this group name was introduced when Azure was bumped to v1beta1). That said, I think trying to maintain this big bundle of examples all here in crossplaneio/crossplane is a losing game - we need a better strategy.


Support recording rules without alerts

It doesn't seem there is a straight forward way to use recording rules with this playbook.  You can specify them as part of the alerting rules but there is a conditional on task that manages alert rules which stops it if no prometheus_alertmanager_config is specified.
https://github.com/cloudalchemy/ansible-prometheus/blob/ff7830d06ba57be1177f2b6fca33a4dd2d97dc20/tasks/configure.yml#L2-L14
I was able to work around this by adding a stub alerting rule
prometheus_alertmanager_config:
  - timeout: 10s

https://prometheus.io/docs/prometheus/latest/configuration/recording_rules/

Sorry for delay. Could you create a PR with a fix? It seems pretty easy to do.


stages/yum.config: add a new stage to configure YUM langpacks plugin

The new stage enables users to configure the langpacks plugin of YUM. Its
schema is derived from the rhsm stage and is designed to be flexible if
we need to add ability to configure more YUM plugins, or even YUM itself.

I'm not 100% sure how to test this. Maybe with CentOS 7? The issue with Fedora is that there's no YUM anymore.
A sample config:
{
  "type": "org.osbuild.yum.config",
  "options": {
    "plugins": {
      "langpacks": {
        "locales": [
          "en_US.UTF-8"
        ]
      }
    }
  }
},

I rebased this PR on top of #872 and added a test. The schema remains the same, it was just merged with the prior work made by @thozza.

@ondrejbudai for some reason there are merge conflicts which can not be resolved by GitHub. Could you please rebase the PR? I just merger the #872

Done :)

@ondrejbudai raised a valid point about where plugins should be configured, now that we have a PR where rhsm configures the yum (and dnf) plugin. Maybe we should move the plugin configuration code to its own stage, per-plugin, and then share the code in utils so it can also be shared with the rhsm stage?

@ondrejbudai raised a valid point about where plugins should be configured, now that we have a PR where rhsm configures the yum (and dnf) plugin. Maybe we should move the plugin configuration code to its own stage, per-plugin, and then share the code in utils so it can also be shared with the rhsm stage?

It definitely makes sense to share to code among stages which configure YUM/DNF plugins.
I still think that it makes sense to keep the configuration of RHSM-specific YUM/DNF plugins in the RHSM stage, because IMO it configures RHSM as such and the plugins are part of it. However It would also make sense to have a separate stage for configuring these plugins or let it be general and part of the {dnf,yum}.conf stages. Since we need the later anyway, it probably makes more sense to not scatter this functionality in too many places (meaning e.g. RHSM stage).

I still think that it makes sense to keep the configuration of RHSM-specific YUM/DNF plugins in the RHSM stage, because IMO it configures RHSM as such and the plugins are part of it.

Absolutely! I totally wanna keep it there. I think 3rd party yum plugins should be configured in the third party stage.

However It would also make sense to have a separate stage for configuring these plugins or let it be general and part of the {dnf,yum}.conf stages. Since we need the later anyway, it probably makes more sense to not scatter this functionality in too many places (meaning e.g. RHSM stage).

I think the main trade-off is: too many stages, if we do one per plugin, vs harder to maintain yum.config stage if we support too many and maybe some of the plugins option change in an incompatible way. Although for yum that is a theoretical argument since it not developed anymore. Have a separate stage per plugin makes the rhsm case also less confusing from an UX pov. To sum up: I am torn.

Ok, I think I have come around to this; let's not overthink it. Let's keep it as is. I am not merging it to give @ondrejbudai a chance to react to the nits that @thozza identified.

I force-pushed a new version to implement Tomáš's suggestions.
Configuring yum plugins in separate stages vs. configuring them in one bug yum configuration monostage is a hard decision. I tend to agree with @gicmo though, let's not overthink it for now, I hope that we don't need to implement more for yum as it's obsolete technology nowadays.


LeaveOneOutEncoder - Inconsistent behaviour and bug

Expected Behavior
perform fit_transform(X, y=None)
Actual Behavior
In comparison to OneHotEncoder or BinaryEncoder, LeaveOneOutEncoder produces the error:
fit_transform() missing argument: y
This is incosistent with the other encoders, which do not produce this error. The problem is the encoders
are part of a preprocessing pipeline which is applied not only to my estimators. For example:
I calculate gamma myself, which is only based on X. When I write pipe.fit(X) however, this is not possible anymore
with this encoder.
Steps to Reproduce the Problem

Put the encoder in a pipeline
Call pipe.fit only on X
Error

Specifications

Version:
Platform:
Subsystem:

Hi @nilslacroix
This is actually the expected behaviour.
Some encoders in the category encoders package also require target information like the LeaveOneOut encoder whereas others don't like OneHotEncoding. Obviously encoders that use the target for encoding will raise an error if called without a target.
Unfortunately the link given in the documentation of LeaveOneOutEncoder, that point to how the algorithm works, is broken.


Basic Usage - Variables: incorrect variable name

The state variable is named 'var', but it is referred as 'state' afterwards.
Changing this:
new_value = tf.add(state, one)
update = tf.assign(state, new_value)
To this:
new_value = tf.add(var, one)
update = tf.assign(var, new_value)
Solves the problem.

Duplicate of https://github.com/tensorflow/tensorflow/issues/236, which is itself a duplicate of an earlier bug.


Chef 16+ compatibility

We are in the process of upgrading our Chef clients to us Chef 17. While setup up the kitchen test for this, we noticed the following issue.
NoMethodError
       -------------
       undefined method `logstash_config' for cookbook: logstash, recipe: broker :Chef::Recipe

System Info:
------------
chef_version=17.9.18
platform=ubuntu
platform_version=18.04
ruby=ruby 3.0.3p157 (2021-11-24 revision 3fb7d2cadc) [x86_64-linux]
program_name=/opt/chef/bin/chef-client
executable=/opt/chef/bin/chef-client

After doing some research, it appears there was a breaking change in Chef 16 on how Chef calls custom resources.
https://github.com/chef/chef/issues/10016
The fix seems pretty straightforward as it just needs a provides method for the custom resource. I can work on a PR for this, but wanted to bring to this your attention as I'm not sure how this will work for Chef versions that are not < 16+. I'm3 curious if we should make separate version of this cookbook that works specifically for Chef versions < 16+?
In the meantime, I will create a branch for testing to see if it this will fix the current issue for Chef 16+ clients. I would prefer not fork this cookbook for many reasons. There's other logstash cookbook that resolves this issue, but it would require a complete cookbook rewrite to use the custom resources. We have been using this cookbook for a long time now and it central to our logging infrastructure. We would prefer not to do a complete cookbook rewrite if don't have to.

Hi.
I don't actively use Chef nowadays, but I'm ok with PR and with major version bump and with the chef_version >= 16 restriction.


Please release a new version

Hi!
We need the fix from commit https://github.com/NorthPage/telegraf-cookbook/commit/9c6c3a21096de1e255969edae5251316990f46f8.
Please release a new version.

+1 to this

Sorry about that - just released 0.4.0


Type out Layout Options Dictionary Keys

Restructure the layout options dictionaries to avoid using the ElkTypes directly as keys. Instead keep the dictionaries json serializable.
https://github.com/jupyrdf/ipyelk/pull/24#discussion_r512815584

Explore:

react-json-schema-form

https://github.com/deathbeds/wxyz/tree/master/src/ts/wxyz-json-schema-form


OADP-732: Enable ability to use private registry for custom plugins (add image pull secrets)

Is your feature request related to a problem? Please describe.
I want to use a custom plugins, the image of it is located on a private container registry.
It is not possible to add an imagePullSecret to the Velero deployment.
Describe the solution you'd like
Allow to set the imagePullSecrets property.
Describe alternatives you've considered
If the configuration of the imagePullSecrets property is not possible, it could be nice to indicate to the controller to ignore the modifications of this property, this one could be modified by a mutation at admission.
This is probably not the best idea, I'm not very inspired.

@kaovilai, yes I can create a secret that contains the information needed to authenticate to the private registry but how do I tell to OADP Operator to use this secret (and thus add the secret to imagePullSecret of the Velero deployment)?
If I modify the deployment of Velero, the operator undo my modifications, which I understand.
Without this possibility, I don't see how to use a custom plugin which is located on a private registry (https://github.com/openshift/oadp-operator/blob/master/docs/config/plugins.md).

Ok thanks. Reading it again this is a feature we have to implement.

Tracked in https://issues.redhat.com/browse/OADP-732

/lifecycle frozen


Wrong default webpack cache behaviour in monorepos

Link to reproduction
https://github.com/davelsan/payload-webpack-cache-bug
Describe the Bug
Monorepo details
package manager: yarn 3.4.1
nodeLinker: node-modules

Bug description
On first run (yarn dev), the payload webpack server creates a .cache folder inside the workspace node_modules folder. This causes an error whereby following run leads to a "Module not found" error, because it cannot resolve that path to node_modules/payload/dist/admin correctly.
ERROR in main
Module not found: Error: Can't resolve '/Users/davelsan/Develop/external/payload-webpack-cache-bug/apps/payload/node_modules/payload/dist/admin' in '/Users/davelsan/Develop/external/payload-webpack-cache-bug/apps/payload'

Payload exposes the webpack config and respects user settings. Therefore, setting the cacheDirectory field in the webpack config solves the error.
export default buildConfig({
  admin: {
    webpack: (config) => ({
      ...config,
      cache: {
        type: 'filesystem',
        cacheDirectory: path.resolve('../../node_modules/.cache/webpack/payload')
      },
    })
  },

I'm not sure whether this should be considered a bug or a feature request. I understand if you move it to a discussion. However, given the popularity of monorepos, perhaps this could be in the docs somewhere?
Related

#3661

To Reproduce

Clone the repro and set up the .env files appropriately.
Run yarn dev and wait for the .cache folder to be created. Usually after the webpack compiled successfully message.
Run yarn dev again.

Payload Version
^2.0.0
Adapters and Plugins
"@payloadcms/bundler-webpack": "^1.0.0", "@payloadcms/db-mongodb": "^1.0.0", "@payloadcms/plugin-cloud": "^2.0.0", "@payloadcms/richtext-slate": "^1.0.0"

I am encountering this issue even with the latest version, and the workaround above did not solve it for me.


Crash since updating

Old app worked fine with NS6 and NS7. Had to mod project so updated to Nativescript 7.1.2
Application refuses to load any page with a mapView defined. Upgraded to and nativescript-google-maps-sdk version 3.0.2 and now get this crash report when running ns debug iOS: -
NativeScript encountered a fatal error: Uncaught ReferenceError: NativeClass is not defined
at
(file: node_modules/nativescript-google-maps-sdk/map-view.ios.js:49:0)
at ../node_modules/nativescript-google-maps-sdk/map-view.js(file:///app/vendor.js:65443:30)
at webpack_require(file: app/webpack/bootstrap:816:0)
at fn(file: app/webpack/bootstrap:120:0)
at (file: app/app.js:41:18)
at ./app.js(file:///app/bundle.js:221:30)
at webpack_require(file: app/webpack/bootstrap:816:0)
at checkDeferredModules(file: app/webpack/bootstrap:43:0)
at webpackJsonpCallback(file: app/webpack/bootstrap:30:0)
at (file:///app/bundle.js:2:57)
at require(:1:137)
(CoreFoundation) *** Terminating app due to uncaught exception 'NativeScript encountered a fatal error: Uncaught ReferenceError: NativeClass is not defined
at
(file: node_modules/nativescript-google-maps-sdk/map-view.ios.js:49:0)
at ../node_modules/nativescript-google-maps-sdk/map-view.js(file:///app/vendor.js:65443:30)
at webpack_require(file: app/webpack/bootstrap:816:0)
at fn(file: app/webpack/bootstrap:120:0)
at (file: app/app.js:41:18)
at ./app.js(file:///app/bundle.js:221:30)
at webpack_require(file: app/webpack/bootstrap:816:0)
at checkDeferredModules(file: app/webpack/bootstrap:43:0)
at webpackJsonpCallback(file: app/webpack/bootstrap:30:0)
at (file:///app/bundle.js:2:57)
at require(:1:137)

Ok this is a repeat of https://github.com/dapriett/nativescript-google-maps-sdk/issues/444
Not fixed in 3.0.2

Ok this is a repeat of https://github.com/dapriett/nativescript-google-maps-sdk/issues/444
Not fixed in 3.0.2

Any solution other than the workaround?


[RELEASE] Hot-fix drop_last to cls task

Instance-segmentation:

Semantic-segmentation: WIP

Multi-class:

Multi-label:

H-label: WIP
Detection:


Adds 'composer install' command

This command is required by every project, think it makes sense to add it into docker-compose.

Why would you want to run a compose install whenever the project is started? It only needs to be run once

Why would you want to run a compose install whenever the project is started? It only needs to be run once

Running this command at each start-up will not cause any issues, even if packages already installed. Anyway this will automate some stuff, so I think it's better than manually running same mandatory command from the container per each new project.


Few clarifications

Most of eco dev effort is missing from this excerpt, probably because tracked in notion.
Not sure how you want this @corpetty @chair28980

It looks like we need to strike a balance with EcoDev items existing between Github and Notion. The Pipeline and related coordination items should stay in Notion, while the items we need for efficient tracking and updates remain in Github.


Reproducible search history

This is similar to #23099 an others but there is an important difference.
I often need to go back to previous search results.
To do that precisely I need to restore the old values in

search box
files to include
files to exclude

Just keeping history of the search box is not enough. In a larger code base I often need to limit the search using files to include/exclude boxes.
It would be great if each search remembers the values of all 3 boxes. Then there should be a way to see the list of searches in chronological order. When selecting an entry in the list all 3 boxes are restored and the search is repeated. To improve performance it would be nice if the search results are remembered too.

I think this would be best handled by https://github.com/Microsoft/vscode/issues/16488


Breakpad on OSX always fail to initialise

I've never been able to have the crash dump working, and I always have the following log
[60832:1027/110731:ERROR:breakpad_mac.mm(238)] Breakpad initializaiton failed
Which is emitted around here https://github.com/rogerwang/node-webkit/blob/0afb5476f5cbd29d7efa299acedc28d86ae7cdd9/src/breakpad_mac.mm#L236-L240
Right now I have a very odd crash with 0.11.0-rc1 which I ain't able to isolate, having a crash dump might help.

Want to back this issue? Place a bounty on it! We accept bounties via Bountysource.

Hi.
May I kindly ask what is the resolution status of this rather rusty issue?

this was an upstream issue and will try to merge the wordaround in the next version:
https://code.google.com/p/chromium/issues/detail?id=414239
https://code.google.com/p/google-breakpad/issues/detail?id=612

@rogerwang Thank you very much for your quick response.

@rogerwang I see you released a new version two days ago, but I'm still getting the breakpad init error. I'm guessing the workaround you mentioned will be for 0.11.7 then?

I'm getting the same error

I have the same problem, but when i run it with "sudo", it work.

Same problem with me, and sudo don't work.

Having the same issue (I haven't tried sudo yet). The "hello world" application exhibits this error.

fixed in #3226

Hi!  New to NWJS.
My experience:  npm install nodewebkit -g :+1:
nodewebkit
[16747:1111/080657:ERROR:breakpad_mac.mm(238)] Breakpad initializaiton failed
[16748:1111/080657:ERROR:breakpad_mac.mm(238)] Breakpad initializaiton failed
[16749:1111/080719:ERROR:breakpad_mac.mm(238)] Breakpad initializaiton failed
[16750:1111/080724:ERROR:breakpad_mac.mm(238)] Breakpad initializaiton failed

:-1:


Update README.md

Comment the directory_source attribute to redirect pp user to tho directory + remote_directory.
Plus add an example for this.

Thanks for the PR. I think it's better to write under directory the intended usage, no?

@xenlo I've re-read it and I don't understand exactly what's going on. Your use-case is having a compiled module and wanting to deploy it?

Closing due to inactivity. Please reopen if this is still relevant


Issue 7 - Navbar

Closes #7
Feature
Criar Navbar do Site Petdex
Solution

Checklist

[x]  Issue linked
[x]  Build working correctly

Additional info

atualiza a branch!


For threadsafe implementations, waiting should only require &self

The SlidingWindowRateLimiter is designed in a threadsafe manner, such that its implementation of wait_until_ready and wait_with_cost could take &self instead of &mut self. However, since the RateLimiter and VariableCostRateLimiter traits use &mut self, it's not possible for users to take advantage of this.
I suggest either creating a "shared" version of the traits, or just making the wait... methods inherent to the struct itself and making impl <trait> call those inherent methods.
The "shared" version of RateLimiter could be implemented somewhat like this:
trait SharedRateLimiter {
    fn wait_until_ready(&self) -> impl std::future::Future<Output = ()> + Send;
}

impl<L: SharedRateLimiter> RateLimiter for L {
    fn wait_until_ready(&mut self) -> impl std::future::Future<Output = ()> + Send {
        <Self as SharedRateLimiter>::wait_until_ready(self)
    }
}

...and VariableCostRateLimiter would be similar.

Hey, thanks for raising the issue! Looks like a nice non-breaking change that adds some utility I haven't had a use case for yet.
Can you confirm that https://github.com/Brendan-Blanchard/async-rate-limit/pull/2 is what you had in mind? I'll add the variable one and get a version out this weekend if so.

Yeah, that's exactly what I need! Thanks!

Perfect! This and the variable version are now released as part of v0.1.1 - please open another issue if something else comes up!


fix: add missing modprobe command to datadog-agent container

What does this PR do?
Add modprobe command that is missing inside agent container  more info in #6215
Motivation
Fix broken system-probe functionality on Kubernetes nodes.

Regarding the install of kernel-headers, see also this comment. Since we're running standard amazon linux 2 images for EKS (https://github.com/awslabs/amazon-eks-ami), there appear to be other packages needed as well as some other caveats.

Regarding the install of kernel-headers, see also this comment. Since we're running standard amazon linux 2 images for EKS (https://github.com/awslabs/amazon-eks-ami), there appear to be other packages needed as well as some other caveats.


How to export train result as ply

I want export the train result as a ply file.But the ns-export commend did not work.

Did you experience some error?

On the dn-splatter environment,the ns-export gaussian-splat commend didnot do anything.There was nothing happened after I use the commend,just passed,nothing output.
So I switched to gsplat environment. And I used commendns-export gaussian-splat --load-config,but it seemed that the config.yaml was not right.Then I modified the config.yaml from the gsplat training config.yaml.And then it worked.

Do you consider to build a standard export process like gsplat?

What do you mean that the ns-export gaussian-splat did not do anything? Can you please provide more information. The output is generated into a separate file. There is not much output in the actual terminal log, but if you inspect the output folder you will find the splat.ply file there.
e.g. the command ns-export gaussian-splat --load-config path_to_/config.yml --output-dir ./my_output
will generate a new file called ./my_output/splat.ply which is the ply file you want.

Do you consider to build a standard export process like gsplat?

ns-export gaussian-splat already does gaussian splatting export into a .ply file, which is viewable in other applications of viewers.

Here are steps to create a splat.ply file:

train a model using ns-train dn-splatter [options]
make a note of where the config.yml file is saved after training.
run ns-export gaussian-splat --load-config path_to_/config.yml --output-dir ./my_output and it will create a new file called /my_output/splat.ply

just prinnted nothing

just prinnted nothing

Have you checked what is inside the exportPly/20 folder?

just prinnted nothing

Have you checked what is inside the exportPly/20 folder?

yeah,there was an empty directory

This is the expected behaviour:

What does the command ns-export --help or ns-export gaussian-splat --help say? I am wondering if there are some problems with the nerfstudio installation

Can you run the command ns-install-cli and then try again. Also does training with dn-splatter work i.e. does ns-train dn-splatter work?

train process worked

the command ns-install-cli did not work because i use windows?

I find it strange that ns-export is not working for you. This is expected output when running the command. I am not sure what is wrong, but I suspect somehow nerfstudio is not completely installed correctly, since the cli scripts are not working on your machine. But you also mentioned that you switched to some other conda environment (gsplat?) and you were able to run the ns-export command? Is my understanding correct?

I find it strange that ns-export is not working for you. This is expected output when running the command. I am not sure what is wrong, but I suspect somehow nerfstudio is not completely installed correctly, since the cli scripts are not working on your machine. But you also mentioned that you switched to some other conda environment (gsplat?) and you were able to run the ns-export command? Is my understanding correct?

I fixed it by https://github.com/nerfstudio-project/nerfstudio/issues/3483#issuecomment-2430423841


Rework App Manager

The goal is to restore lost functionality and move as much as possible into the dedicated feature modules

[x] Move mobile UI to feature module
[x] Move wear components to feature module
[x] Migrate mobile UI to use navigation graphs
[x] Restore icon display functionality
[x] Allow partial list updates
[ ] Create tests for logic
[ ] Create tests for UI

There will also be various UI/UX improvements throughout.

Once merged, this will resolve #230


[5pt] Change depth to WSE in rating curves

This is simply changing the text of y axis label from "Average Depth" into "Average Water Surface Elevation".  (The numbers are all correct).


New season!

Are there any plans to keep this project going? Either by Patrick or another gifted developer?
🙏🙏🙏🙏🙏

I don't have a THUD license and am not really willing to pay $18 a month for it, so it would have to be someone else

I don't have a THUD license and am not really willing to pay $18 a month for it, so it would have to be someone else

If alls you need is a license my man, i got you! I would love to see this being supported again! Let me know if you are interested!
Addy#1518 is my discord tag!


[Enhancement] Change title

Motivation / Problem
Current title ("MVG Portal Demonstrator") is not sufficient for the MVP version
Solution
Change to "Minimal Viable Gaia-X"

addressed in feat/mvp


Lifecycle callbacks on embeddables, possible or not?

Encountered behaviour where embeddables seem not to respect lifecycle callbacks, but do respect when the configuration, as also described here.
The documentation does not mention embeddables, so I suppose it is not supported. If so: might be good to document this more explicitly?
The strange thing: when defining a lifecycle callback that points to a non-existing function: an exception is thrown, so the definition is respected.

Not possible: an embeddable has no lifecycle on its own. The lifecycle is
the one of the entity.
On 28 Nov 2017 18:15, "Menno Holtkamp"<EMAIL_ADDRESS>wrote:

Encountered behaviour where embeddables seem not to respect lifecycle
callbacks, but do respect when the configuration, as also described here
https://groups.google.com/forum/#!topic/doctrine-user/g2_vurFHEMM.
The documentation
http://docs.doctrine-project.org/projects/doctrine-orm/en/latest/reference/events.html#lifecycle-callbacks
does not mention embeddables, so I suppose it is not supported. If so:
might be good to document this more explicitly?
The strange thing: when defining a lifecycle callback that points to a
non-existing function: an exception is thrown, so the definition is
respected.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/doctrine/doctrine2/issues/6855, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJakFYKpEWWtDx-Kax0ZuK9kBFhBxaZks5s7D-3gaJpZM4QtmI3
.

Wow, < 1 minute response time, raspact!
Should I submit a PR for the documentation on the Entity LifeCycle to explicitly indicated this will not work for embeddables? Or... is that too much ;). Maybe others have the same "use case".

Sending a PR to the docs would be fab!
Marco Pivetta
http://twitter.com/Ocramius
http://ocramius.github.com/
On Tue, Nov 28, 2017 at 6:18 PM, Menno Holtkamp<EMAIL_ADDRESS>wrote:

Wow, < 1 minute response time, raspact!
Should I submit a PR for the documentation on the Entity LifeCycle to
explicitly indicated this will not work for embeddables? Or... is that too
much ;). Maybe others have the same "use case".
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/doctrine/doctrine2/issues/6855#issuecomment-347596420,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJakM8rG7gIuc_pY6EsI_Ti8igNA0etks5s7EBugaJpZM4QtmI3
.

This behaviour is also discussed in https://github.com/doctrine/doctrine2/issues/4568, and can maybe be covered using https://packagist.org/packages/tarifhaus/doctrine-nullable-embeddable


Fix Issues With Avatars

explore avatars to use the project/fallback to dao
DAO avatars are currently not consistently displaying. Sometimes they work. At others they do no.

Can this be tested again? I have it using the project avatar and falling back the dao. There are times when it might take a few seconds to load and in that case it'll show the blockie until we find an avatar.


Solve day 10 part 2

https://adventofcode.com/2022/day/10#part2

This works, but it requires manually identifying the characters in the output. I'd like to come up with a solution that can do that automatically.


ctrl+enter only executes the first command if there are mutliple commands on a single line separated by semicolon

System information:

Operating system (distribution) and version: Ubuntu 19.10
DBeaver version: 6.3.5
Additional extensions: Vrapper

Connection specification:

Database name and version: PostgreSQL 11
Driver name: PostgreSQL Standard Driver
Do you use tunnels or proxies (SSH, SOCKS, etc)?  no

Describe the problem you're observing:
I have an editor with multiple commands on a single line, separated by a semicolon.
I'd expect ctrl+enter to execute all of them, not just the first one.
Steps to reproduce, if exist:
open an editor, type
create table foo (id serial); create table foo2 (id serial);

press ctrl+enter
only foo gets created

Hi @abolognino ,
this is expected behavior. To execute all the commands you should use "Execute SQL Script" (Alt+X)

@uslss I'll explain my usecase: I have a list of commands to execute provided by someone else, I paste them and execute them one by one with ctrl+enter. If there are multiple commands on a single line only the first one is executed and I'll often miss that.
I understand that alt+x executes the whole script, but there are cases where I want to check each command first and execute them one by one.
Selecting the whole line and pressing ctrl+enter does what I expect.
This is a design decision more than anything else though. Feel free to close it.

@abolognino alt+x doesn't always execute the whole script: if you select some statements, it will execute only the selected portion of the script.


Markdown Tables don't render in snap

I'm trying to use a Markdown (MD) table. If I do nothing it's too large for the page:
---

Folder|Re-Runnable|Description
--- | ---
`/build`|@fa[ban]|Scripts to build release
`/data`|@fa[sync]|Re-runnable data scripts
`/packages`|@fa[sync]|Ex: `pkg_emp.pks` and `pkg_emp.pkb`
@css[text-red text-bold](`/release`)|@fa[times-circle]|_More Later_
@css[text-grey](`/scripts`)|@fa[ban text-grey]|@css[text-grey](Common scripts / helpers for developers)
`/synonyms`|@fa[sync]|Synonyms
`/views`|@fa[sync]|Views
`/www`|@fa[ban]|Web files (for APEX)

---

If I try to wrap the image in a snap to set the font size to a smaller font it does not render properly:
---

@snap[midpoint text-07]

Folder|Re-Runnable|Description
--- | ---
`/build`|@fa[ban]|Scripts to build release
`/data`|@fa[sync]|Re-runnable data scripts
`/packages`|@fa[sync]|Ex: `pkg_emp.pks` and `pkg_emp.pkb`
@css[text-red text-bold](`/release`)|@fa[times-circle]|_More Later_
@css[text-grey](`/scripts`)|@fa[ban text-grey]|@css[text-grey](Common scripts / helpers for developers)
`/synonyms`|@fa[sync]|Synonyms
`/views`|@fa[sync]|Views
`/www`|@fa[ban]|Web files (for APEX)

@snapend

---

Looks like:

From the second screenshot, the MD table is not rendered properly

@martindsouza thanks for reporting this issue. Looks like a bug in the new markdown rendering engine used for rendering content within snap-layouts. I will update this thread once I have had a chance to investigate.

Btw, if you need a temporary workaround you could use regular HTML syntax to create your table. Just make sure that you do not indent your HTML tags as doing so may cause your table to be treated as raw code not a table.
For example, you could do something like this:

@snap[west span-50 text-04]

Company
Contact
Country

Alfreds Futterkiste
Maria Anders
Germany

Centro comercial Moctezuma
Francisco Chang
Mexico

Ernst Handel
Roland Mendel
Austria

@snapend

@martindsouza a new patch to fix the snap-rendering of markdown tables has just gone live on gitpitch.com. This same patch has also been integrated into GitPitch Desktop. Pull the latest version of the desktop from Docker Hub.
When your time permits can you please test and verify the fix. Thanks.

@gitpitch Still getting the same issue. The image I'm using is 924c8ba14a51

@martindsouza it looks like the public build did not complete earlier. Apologies, you were testing an old image. The new image is now available. Please docker pull and try again.

@gitpitch still the same issue. Image is 725ba03a7bed

@martindsouza I believe the issue is with your current table syntax. The snap-layouts markdown renderer requires you to use a pipes (|) when opening and closing a row in any table. Add those pipes to your markdown and your table should render as expected.

Works now. For others this is my new table structure:
---

@snap[midpoint text-06 span-100]

| Folder | Re-Runnable | Description |
| --- | --- | --- |
| `/build` | @fa[ban] | Scripts to build release |
| `/data` |@fa[sync] | Re-runnable data scripts |
| `/packages` | @fa[sync] |Ex: `pkg_emp.pks` and `pkg_emp.pkb` |
| @css[text-red text-bold](`/release`) | @fa[times-circle] | _More Later_ |
| @css[text-grey](`/scripts`) | @fa[ban text-grey] | @css[text-grey](Common scripts  | / helpers for developers)  |
| `/synonyms` | @fa[sync] | Synonyms |
| `/views` | @fa[sync] | Views |
| `/www` | @fa[ban] | Web files (for APEX) |

@snapend

---

Results in:

![image](https://user-images.githubusercontent.com/2925726/64256162-1a416400-cee0-11e9-8bd2-834495a31dea.png)


Token Error

@MomenSherif I'm getting this error while using useGoogleLogin() hook, but still, I'm getting an access token.
Can you pls help me how to fix that error?
FYI, I am using custom button

Are you using gapi to talk to google services?
Regardless using the package to login

No, @MomenSherif
I've just added one custom button, useGoogleLogin() hook & GoogleOAuthProvider as wrapper to the App.
I'm able to log in successfully but still, I'm getting this warning

Really don't know,
didn't face this warning before


BLE devices disconnecting sporadically on Native

Logging shows:
API MISUSE: Cancelling connection for unused peripheral <CBPeripheral: 0x2816bc4e0, identifier = 23584B66-2C3C-09F2-CF3F-BC60BDA79D4B, name = JUUL2 QDA0N0U4, mtu = 23, state = connecting>, Did you forget to keep a reference to it?
This error comes from CoreBluetooth.
Haven't found much rhyme or reason for when it happens yet, but when it does the device will not reconnect--even on subsequent relaunches of the consuming app.

Just got more misuse logs. Now one for powered on state:
`
API MISUSE: Cancelling connection for unused peripheral <CBPeripheral: 0x2827fd5f0, identifier = 23584B66-2C3C-09F2-CF3F-BC60BDA79D4B, name = JUUL2 QDA0N0U4, mtu = 23, state = connecting>, Did you forget to keep a reference to it?
API MISUSE: <CBCentralManager: 0x282dff480> can only accept this command while in the powered on state
`


esc key close dialog, enter key commit dialog

I like the idea of closing the modal with the esc key. However, the enter key for submit seems tricky, I would prefer the user to press the confirmation for deletion.
Also, I suppose that we want to clear the listener after the modal is closed.
I'm working on a complete refactor for the modal component, and I will include the feature for closing on esc.
Thank you!


[SPARK-26285][CORE] accumulator metrics sources for LongAccumulator and Doub…

…leAccumulator
What changes were proposed in this pull request?
This PR implements metric sources for LongAccumulator and DoubleAccumulator, such that a user can register these accumulators easily and have their values be reported by the driver's metric namespace.
How was this patch tested?
Unit tests, and manual tests.
Please review http://spark.apache.org/contributing.html before opening a pull request.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

ok to test

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5856/
Test PASSed.

Test build #99831 has started for PR 23242 at commit 45cfada.

Test build #99831 has finished for PR 23242 at commit 45cfada.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class AccumulatorSource extends Source

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99831/
Test FAILed.

Test build #99877 has started for PR 23242 at commit de531c8.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5893/
Test PASSed.

Test build #99877 has finished for PR 23242 at commit de531c8.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99877/
Test PASSed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/6261/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/6262/
Test PASSed.

Test build #100293 has finished for PR 23242 at commit 7f60730.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/100293/
Test PASSed.

Test build #100373 has started for PR 23242 at commit 682cfd7.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/6337/
Test PASSed.

Test build #100373 has finished for PR 23242 at commit 682cfd7.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/100373/
Test PASSed.

merged to master, thanks @abellina


Control visibility of the input

The editing variable is private in the component, but in case I want to control validation of the input there is no way to do so atm.
Adding a public setEditMod(mode:boolean) variable would help transfer control of the visibility of the editor to the calling component.

Thanks for your feedback!

This feature will be added with #57 using the method edit( { editing: boolean, doFocus: boolean } ) 👍

This should be fixed in the last version (0.2.0-alpha.0). With the change of name, you should do an npm i @qontu/ngx-inline-editor and update your imports 👍


The detail CONTRIBUTING.md

Feature Request
Your document is excellent.
It will be sufficient for experienced individuals.
However, it is anticipated that there will be significant difficulties for newbies in understanding the structure.
Especially in the area of transformations and coding while grasping the Abstract Syntax Tree (AST), it can be time-consuming.
If there are explanations about the structure or information on where specific functionalities are located, it would reduce the learning curve and make it easier to overcome trial and error.
Thank you for your attention.

Welcome your PR

@samchon As I am working with Typia, I am noticing some general quality of life improvements that could be made (ie. comments in source code, more developer/maintainer documentation). I'll make a PR eventually with additional docs.
On a side note, should the test/generated/output directory be tracked in git? I ask because I ran the test generation and git status is showing me about 807 files modified, which all seem to be brand new output files! Because it's generated during the test procedure, I have a feeling this should be added to .gitignore...

Also, as an improvement, I would suggest creating a wrapper interface for the ts package to reduce the size and complexity of the code, and create functions with multiple arguments rather than ()=>()=>()=>()=>()=>{}()()()()().


DOC: Adding more documentation on how to run rhapsody

This also provides some details on how to interpret the results
CC @ElDeveloper  - this may help with running / interpreting the model

Ok - I think the comments on this have been addressed.  Merging this in.  Thanks @ElDeveloper !

👍


Add a CODE_REVIEW_CHECKLIST.md

The Chef Server 12.15 release had a number of regressions. As part of
the post-mortem for that release, we agreed that a PR checklist may be
helpful to have more fruitful discussions during code review.
This is a first attempt at such a checklist.
Signed-off-by: Steven Danna<EMAIL_ADDRESS>

I do wonder if we have a set of first- and second-class deployment options? (I'm thinking about HA, tiered, external PG, external RMQ, and the like)

Yup, we have another task to define the list of our top-tier deployment scenarios.


Input issue

Describe your bug here.
If you miss a note and press another direction (with ghost tapping enabled) it will count as 2 misses for some reason.
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
I didn't edit anything. I was using the 0.5 prerelease build.

If you press 2 note at the same time?

If you miss a note and press another button right after.

So say you miss an up note and press left right after. It'll count as two misses even though you only missed one note.

???
Isn't that how Psych's ghost tapping is supposed to work? You are able to miss ONLY when there's notes near the strum arrows.

Wait really? I didn't know it worked like that. I thought it only allowed you to miss if you didn't hit a note.


Support pretty-printing declarations

What do you want to use? Do you want to reconstructure it or is there a lib you prefer?

I think we probably want to walk the tree and reconstruct it.


Multi-processor NeighborSampler feature request

🚀 Feature
Multi-processor support for NeighborSampler.
Motivation
In my application, I use the NeighborSampler in order to train on large graphs. I'm finding, however, that it is very slow. On smaller graphs, when I can load the full graph into GPU memory and don't use NeighborSampler I have a significant reduction (order of magnitude or more) in run-time. If it's possible to parallelize NeighborSampler, I think the speedup would be substantial.
Thanks for the consideration.

Yes, a num_workers argument is in the pipeline, although I personally find the NeighborSampler already quite fast :) Are you sure that this is the bottleneck? Would be interesting to see some benchmarks on this.

Interesting. It could be something I'm doing, I'll profile my code and get back to you!

You're right, it's not the NeighborSampler after all.
I'm using a graph autoencoder to get unsupervised node embeddings. I try to reconstruct the input graph structure in the output. Since I'm batching, the loss function sees a subgraph each time its called. Obtaining this subgraph is the slow part (my own function, nothing to do with pytorch-geometric).
This brings up a good point though: say for example I have a DataFlow object that looks like (128<-543<-831). My loss function requires the subgraph corresponding to the batch (i.e. a 128x128 graph). How can I efficiently obtain this subgraph from the DataFlow object?
Thanks again!

So you are trying to reconstruct the subgraph from the DataFlow object? Then the graph you are trying to reconstruct does not have shape 128 nodes, but rather 128+543+831 nodes, given that there are no duplicated nodes in the bipartite graphs (which is highly unrealistic). This is a tricky one, and I assume it does need another API which we discussed here.

The output of my model is always a batch_size x batch_size tensor, and the objective is to minimize the loss between output and a batch_size x batch_size tensor target that corresponds to the subgraph given by the nodes in the batch.
The way I'm thinking about this is that if NeighborSampler yields a DataFlow like (4<-10<-12) then my model should only compute features for the 4 nodes at the end of the bipartite graph (and give a 4x4 tensor post dot-product), not the 12 at the beginning. So my problem is: how do I subset data_flow.edge_index to only include edges from the 4 nodes? If I can get these edges, then I can create a sparse tensor and make it dense to get the corresponding 4x4 subgraph.

Ok, I understand. You can use this function to filter your original edge_index.

Nice, thanks for the suggestion. I ended up using the subgraph function from torch_geometric.utils since I need to keep track of edge attributes as well. This runs significantly faster now - thanks for the help!

Ah, makes sense as well :)


Fix validation of notifications

Now validation table is local and has correct count of data.
Relates to APPLINK-23704
Please review @dtrunov, @OHerasym, @Kozoriz, @VProdanov  @LuxoftAKutsan

Reviewed.

Reviewed.

Reviewed

Reviewed

@AGaliuzov please merge


flow: Nullable properties in query and mutation variables should be output as optional

The generated flow type
export type SomeQueryVariables = {
  fieldName: ?string,
};

should preferably be
export type SomeQueryVariables = {
  fieldName?: ?string,
};

since fieldName is not required.
I could fix this by monkey patching https://github.com/apollographql/apollo-codegen/blob/master/src/flow/language.js#L63 as
const isNullableVariableProperty =
  generator.scopeStack[0].typeName.endsWith('Variables') &&
  typeName.startsWith('?');

generator.printOnNewline(
  `${propertyName}${isNullableVariableProperty ? '?' : ''}: ${typeName}`,
);

but I don't think that's a good solution.

@yenbekbay Thanks for the report. I think we can do something simpler. Can you try to put your monkey patch into a PR so we can check it out and define what to do.


Implicit grant type for Bitbucket

Hello! Now that Netlify supports Bitbucket, I'm wondering if it also supports the implicit oauth flow for Bitbucket?
According to Bitbucket the implicit flow is supported by them: https://confluence.atlassian.com/bitbucket/oauth-on-bitbucket-cloud-238027431.html
It would be great to use Netlify CMS and Bitbucket without a server like the GitLab support.
Thanks for the great project!

The reason implicit authentication is not supported for BitBucket yet is because BitBucket requires that the user re-authenticates every hour. When using a server-side flow it provides a "refresh" feature so that the update can occur in the background. When using the implicit flow, however, the user would have to be asked to re-authenticate using some sort of popup. I had worked on this before in PR #560, but there are still issues to deal with so that users do not lose what they are working on.

does implicit grant allow refresh tokens?
https://confluence.atlassian.com/bitbucket/oauth-on-bitbucket-cloud-238027431.html#OAuthonBitbucketCloud-Refreshtokens

This should have been closed with https://github.com/netlify/netlify-cms/pull/2247.
@satyapramodh refresh tokens aren't supported for implicit grant, check the "Refresh Tokens" section: https://developer.atlassian.com/cloud/bitbucket/oauth-2/


Documentation

Documentation needs writing for the platform.

This will be dealt with by carbontwelve/tapestry-cloud-src


Fix insertion order when adding a list of models to a collection and specifying at

Currently when adding a list of models to a collection at a specified position using add, the list is added in reverse order.
var a = new Backbone.Model({id: 1});
var b = new Backbone.Model({id: 2});
var coll = new Backbone.Collection([a, b]);
var c = new Backbone.Model({id: 3});
var d = new Backbone.Model({id: 4});
coll.add([c, d], {at: 1});
// coll.models is [a, d, c, b], when it should be [a, c, d, b]

This change fixes the issue by adding the models in reverse order if a position is specified.

interesting patch... but what happens when there's a comparator on the collection?

I don’t see a problem. The reversed adding happens only when options.at is set, in which case the comparator is never used (see the ternary operator chain in _add).

@knuton thanks, I was just curious about which would take precedence. Seems like a reasonable addition :)
+1

Fixed for options.at === 0 thanks to a hint from @gruber76.

This should be fixed now on master.

Looks like it, thanks.


EXCEPTION_ACCESS_VIOLATION_READ: Attempted to dereference null pointer

Hi, first of all thank you for this very useful project :)
I think I have found a bug in the source code which causes a null pointer dereference:
    device->Activate(__uuidof(IAudioEndpointVolume), CLSCTX_ALL, NULL,
        (LPVOID *)&endpointVolume);

    if (device_cb == NULL)
        device_cb = new SoundMixerAudioEndpointVolumeCallback(this);
    endpointVolume->RegisterControlChangeNotify(device_cb);

    return true;

From: https://github.com/m1dugh/native-sound-mixer/blob/master/cppsrc/win/win-sound-mixer.cpp#L308-L315
According to https://learn.microsoft.com/en-us/windows/win32/api/mmdeviceapi/nf-mmdeviceapi-immdevice-activate the endpointVolume could be NULL when the Activate call fails. However, the return value is not checked and later endpointVolume->RegisterControlChangeNotify(device_cb); is done. When endpointVolume is NULL, this causes a null pointer dereference.
I have no test case to reproduce this behavior because I am only seeing this from production users by collecting crashes. In case it might be useful for you, here is a snippet of the native crash dump:
EXCEPTION_ACCESS_VIOLATION_READ: Attempted to dereference null pointer.

0  win-sound-mixer.node +0xeaab  0xeaab
   r10 = 0x5555555555555555   rbp = 0x1b5a0af79f0
   r11 = 0xaaaa88a8c00aa2aa   rbx = 0x1b5a0a54d70
   r12 = 0x1b5a0ade5e0        rcx = 0x0
   r13 = 0x497fffc3b8         rdi = 0x1b5a0a54d00
   r14 = 0x1b55d6b4250        rdx = 0x1b55d681b70
   r15 = 0x0                  rip = 0x7ffb654aeaab
    r8 = 0x0                  rsi = 0x1b5a0a54d18
    r9 = 0x555577573ff55d55   rsp = 0x497fffc110
   rax = 0x7ffb654cdbe0
1  win-sound-mixer.node +0xc95b  0xc95c
2  ntdll.dll +0x3dca4            0x3dca5
3  win-sound-mixer.node +0x10beb 0x10bec
4  win-sound-mixer.node +0xe056  0xe057
5  win-sound-mixer.node +0x728b  0x728c
6  win-sound-mixer.node +0x31fd  0x31fe
7  win-sound-mixer.node +0x2767  0x2768

This is the instruction on where it fails: mov     rax, [rcx]. You can clearly see that rcx is set to 0x0 which being dereferenced.
I don't know the code well enough to know how to handle this case. Do you maybe now why Activate could fail?

Hello,
I don't know how activation could fail, however, I will fix the bug and deploy the new package soon, unless you want to make a PR with the fix that I will be happy to accept.
Thanks

Hi @m1dugh, thank you for the quick response! It would be great if you could fix this issue since then I don't have to setup the development environment :)

Hi @m1dugh, thank you for the quick response! It would be great if you could fix this issue since then I don't have to setup the development environment :)


serve locally over https

https://www.npmjs.com/package/devcert

Def makes this easier, but still requires user to jump through hoops (insecure browser warnings and root pw on the CLI) and it's only needed in specific cases. Nextjs defaults to http, for instance.
Closing for now, should revisit if we can make it optional.