skip hog tests in build environments where hog is not built or not available

Thanks to @thmzlt, I think that this should unblock our test run during nix builds (so we can make hobbes available on nixOS).

I noticed the nixpkgs tarball in default.nix is set to track the latest version of the unstable branch, and gcc5 is no longer available there. We should either pin the nixpkgs version to a commit or use a lower/higher version of gcc:

It makes sense to pin the nixpkgs version at least to make the change history make more sense (so when upgrading a gcc compiler version it'd be in a PR for that purpose).
Ultimately we will probably want to update gcc too.  I'm not sure what most people are using, we're probably a little behind the times.
Do you know if there's a way I can try the nixOS build process on this PR?  Since the tests make some references to local files, and the nixOS build process seems to play some funny games with build directories, I might need to iterate on this PR a few times to shake out those issues.
Alternately, I guess we could disable tests for the nixOS build since we already do them here, but that probably doesn't look kosher.

I was just able to build hobbes with the following default.nix:
{ pkgs ? import (fetchTarball https://github.com/nixos/nixpkgs-channels/archive/cc1ae9f21b9e0ce998e706a3de1bad0b5259f22d.tar.gz) {} }:

pkgs.stdenv.mkDerivation {
  name = "hobbes";
  src = ./.;

  nativeBuildInputs = with pkgs; [
    ccls
    cmake
    gcc8
    llvm_6
    ncurses
    python27
    readline
    zlib
  ];

  doCheck = false;
  checkTarget = "test";
}

The commit is what nixpkgs-unstable was pointing to at the time. LLVM 6 and GCC 8.3 are the highest versions that worked (LLVM 7 and GCC 9 break the build). I set doCheck to false in order to complete the build without running tests.

Do you know if there's a way I can try the nixOS build process on this PR?

Do you mean locally? I'm using nix-build at the project root.

Looks like now the non-nix build is getting stuck running tests.

Yeah, it looks very strange considering that this PR now has no code changes (so is the same as all of the builds that have tested fine for months).  I think it's probably an issue on the CI host, will have to dig further tonight.

I started another PR just to test it: https://github.com/Morgan-Stanley/hobbes/pull/331
It looks like we're not getting an inotify signal for a file update that we expect (and which was always delivered before).  It might be interesting.  :)

Can you trigger a new build of this PR (via the Travis UI)?

Yeah, it looks like something has changed on the ci Linux hosts, very strange, at least it's repeatable between restarts.  I will work on getting to the bottom of it this weekend.  This is the first time that this has happened.  :T

It looks like something like this (https://william-yeh.net/post/2019/06/inotify-in-containers/) has started making the inotify-based tests unreliable.  I've temporarily disabled them to get these two pull requests passing (nothing in either PR changes the inotify logic in any way).  I have a separate conversation going out of band with folks at travis-ci to sort out what changed.
Hopefully this will be enough to resume progress on the nixOS PR as well.

Already merged through vtrunc, which was a superset of this PR.


Access blocked: DVC’s request is invalid

I tried running the following scripts :
Pull in data already available
!dvc pull -q
After running the above scripts i got the following output
Go to the following link in your browser:
https://accounts.google.com/o/oauth2/auth?client_id=710796635688-iivsgbgsb6uv1fap6635dhvuei09o66c.apps.googleusercontent.com&redirect_uri=urn%3Aietf%3Awg%3Aoauth%3A2.0%3Aoob&scope=https%3A%2F%2Fwww.googleapis.com%2Fauth%2Fdrive+https%3A%2F%2Fwww.googleapis.com%2Fauth%2Fdrive.appdata&access_type=offline&response_type=code&approval_prompt=force
Enter verification code:
When I click the above link i got the following message:
Access blocked: DVC’s request is invalid
<EMAIL_ADDRESS>You can’t sign in because DVC sent an invalid request. You can try again later, or contact the developer about this issue. Learn more about this error
If you are a developer of DVC, see error details.
Error 400: invalid_request
Please assist

Thanks for creating this issue @eoproject2022
This seems to be tied to the Google Colab oauth upgrade. It looks like a fix: https://github.com/iterative/dvc/issues/7867 may be available in a newer version of dvc. However, this version is not possible to install because Google Colab has an older python version (3.7). I am actively investigating a fix.

I am stuck on this error as well

Current options being investigated:

Move dvc remote storage from Google Drive to a publicly readable Google Cloud Storage Bucket (this will hopefully get around the authentication issue but will require additional permissions for pushing the model to dvc)
Change python version inside Colab to be able to install a newer version of dvc

This will be fixed when is merged #117


Automated cherry pick of #11805

Cherry pick of #11805 on release-7.7.
/cc  @AGMETEOR
NONE

/update-branch

Trying to auto merge this PR.

Pull Request successfully merged
SHA: 884cefb94ed64653550929047323fbc2a999e09e


Change with_cte representation to {map, query}

The first element of the tuple now is a map storing the name of the cte and whether it's materialized.

I've named the map opts in places, not sure if there's a better name.
Not really happy with the merge_queries function, it's certainly less elegant than the List.keystore. Open to suggestions

I'll follow up with a PR in ecto_sql
Fixes https://github.com/elixir-ecto/ecto/issues/4067#issuecomment-1360056485

:green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:


Shortcuts should work in all keyboard layouts.

Both IDEA specific (like Ctrl-Alt-s) and general (like Ctrl-c, Ctrl-z) don't work in Russian keyboard layout.
Ubuntu 14.04
IntelliJ IDEA 2016.1.1
Build #IC-145.597, built on March 29, 2016
JRE: 1.8.0_76-release-b18 amd64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o

@Chushuhuch How is it connected to the LaTeX plugin ?

I'm sorry, I was sure it's working everywhere else. Please ignore.


[Event Request] codeunit 99 "Posting Selection Management" - procedure "GetShipInvoiceSelectionForWhseActivity"

Dear ALAppExtensions Team,
We would like to request the following change
    local procedure GetShipInvoiceSelectionForWhseActivity(DefaultOption: Integer; var Selection: Integer): Boolean
    var
        UserSetupManagement: Codeunit "User Setup Management";
        ConfirmManagement: Codeunit "Confirm Management";
        Ship: Boolean;
        Invoice: Boolean;
+        IsHandled: Boolean;
    begin
+        OnBeforeGetShipInvoiceSelectionForWhseActivity(DefaultOption, Selection, IsHandled)
+        If IsHandled then
+          exit(true);

        UserSetupManagement.GetSalesInvoicePostingPolicy(Ship, Invoice);
        case true of
            not Ship and not Invoice:
                begin
                    Selection := StrMenu(ShipInvoiceFromWhseQst, DefaultOption);
                    if Selection = 0 then
                        exit(false);
                end;
            Ship and not Invoice:
                begin
                    if not ConfirmManagement.GetResponseOrDefault(GetShipConfirmationMessage(), true) then
                        exit(false);
                    Selection := 1;
                end;
            Ship and Invoice:
                begin
                    if not ConfirmManagement.GetResponseOrDefault(GetShipInvoiceConfirmationMessage(), true) then
                        exit(false);
                    Selection := 2;
                end;
        end;
        exit(true);
    end;

+    [IntegrationEvent(false, false)]
+    local procedure OnBeforeGetShipInvoiceSelectionForWhseActivity(DefaultOption: Integer; var Selection: Integer, var IsHandled: Boolean)
+    begin
+    end;

Availability update: We will publish a fix for this issue in the next update for release 22.
Build ID to track: 58441.


[Question] session.userData loses an Object type on waterfall step

System Information

SDK Language: Node.js
SDK Version: 3.9
Development Environment: localhost

Issue Description
Assign Date object to my custom variable session.userData.start, however on next waterfall step or parent dialog this variable become string.
Waterfall Code Example
[
function (session, results) {
        session.userData.city = results.response;
        builder.Prompts.time(session, "What are Your travel dates?");
    },
    function (session, results) {
        // Get booking start time
        session.userData.start = builder.EntityRecognizer.resolveTime([results.response]);
        // On this step session.userData.start is Date

        // Prompt for rooms
        builder.Prompts.text(session, "How many rooms?");
    },
    function (session, results) {
        // On this step session.userData.start become string
    }
]

Expected Behavior
Hope variable remains as Date type with Date() constructor and Date methods.

Because the data is saved in JSON, complex objects (like Date) cannot be stored in the session.userData object. What you can do instead is something like calling Date.toISOString() and storing that inside of session.userData.
When you need to use the Date again, you would simply take the string from session.userData and convert it back to a Date object.
var start = new Date(session.userData.start);

Does this help?

Yeah, this helped: https://stackoverflow.com/q/45724975/630169 of course.
However documentation mentioned:

Since the Bot Builder SDK for Node.js implements each storage container as a JavaScript object, you can store any type of data that a JavaScript object supports.

So thought any type is allowed. Need to fix either code or documentation then. :)

Thank you for providing the link! I was hoping that our documentation was correct, but it obviously needs to be updated. I'm going to leave this open until a PR is filed for the docs.

Probably it is good session saved in JSON - it is possible to leave conversation, jump to some other resource (web site for ex.) and back to same session via RESTful request.

Closing this issue as I've forwarded the issue to the docs team for a fix.

This was fixed in the documentation today. Cheers!

Cool! Thanks!
Good if they extend other documentation and examples, for example for buttons, cards, their actions, etc: to understand and SEE how they work need to go to the doc, examples, github issues, stackowerflow, blogs, etc. Good if doc will be enough.


Why do root objects keep reference counts?

I've been looking through the code, and I can't figure out why Runtime::ObjectImpl has reference count? From what I see this is only used to determine if root objects are reachable, because non-root objects will always have their counts zero.
Would it not be better to maintain a list of rootObjects along with allObjects and instead of adding an overhead on every object created in the program? Are there any cases where root will have more than one reference, and it still wouldn't be possible to tell if it is alive or not?

Correct: the reference count is just used to find roots. Reference counting can't be used in general, as there may be cycles: for example a ModuleInstance -> TableInstance -> FunctionInstance -> ModuleInstance.
The reason I use reference counting is that it's very simple even when you have multiple threads that are creating and destroying root references. It also doesn't require global knowledge of other possible root references to an object to correctly maintain the root set.
The memory used by the reference count is really not worth worrying about. For example, GlobalInstance is the smallest object that has a reference count, and in that case the reference count would be 8/56 bytes, not counting memory allocated for the global in each Context in the same Compartment.

Yeah, you may have a point about the memory overhead, as it's usually only a problem when small structs in arrays (int32?) start having references to them. I'm not suggesting using the refcount, you'd still need to sweep periodically to clear cycles anyway.


Refactor to set qubit properties when constructing target

Summary
Also sub class QubitProperties to add anharmonicity.
TODO:

[ ] Add tests

Details and comments
Fixes #308
Fixes #309

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

-16 of 27 (40.74%)  changed or added relevant lines in 3 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.05%) to 67.519%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit_ibm_runtime/ibm_qubit_properties.py
7
10
70.0%

qiskit_ibm_runtime/utils/backend_converter.py
3
16
18.75%

Files with Coverage Reduction
New Missed Lines
%

qiskit_ibm_runtime/utils/backend_converter.py
1
14.61%

Totals

Change from base Build<PHONE_NUMBER>:
0.05%

Covered Lines:
2642

Relevant Lines:
3913

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

11 of 27 (40.74%)  changed or added relevant lines in 3 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.05%) to 67.519%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit_ibm_runtime/ibm_qubit_properties.py
7
10
70.0%

qiskit_ibm_runtime/utils/backend_converter.py
3
16
18.75%

Files with Coverage Reduction
New Missed Lines
%

qiskit_ibm_runtime/utils/backend_converter.py
1
14.61%

Totals

Change from base Build<PHONE_NUMBER>:
0.05%

Covered Lines:
2642

Relevant Lines:
3913

💛  - Coveralls


Main to Live Sync

Main to Live Sync

Docs Build status updates of commit 1b7ba3d:
:warning: Validation status: warnings

File
Status
Preview URL
Details

microsoft-community-training/frequently-asked-questions/faqs-general.md
:warning:Warning
View
Details

microsoft-community-training/settings/configurations-on-the-training-platform.md
:warning:Warning
View
Details

microsoft-community-training/frequently-asked-questions/faqs-user-management.md
:bulb:Suggestion
View
Details

microsoft-community-training/infrastructure-management/install-your-platform-instance/configure-login-social-work-school-account.md
:bulb:Suggestion
View
Details

microsoft-community-training/analytics/custom-reports/power-bi-integration.md
:white_check_mark:Succeeded
View

microsoft-community-training/index.yml
:white_check_mark:Succeeded
View

microsoft-community-training/infrastructure-management/configure-your-platform-infrastructure/set-up-custom-homepage-for-your-mct-instance.md
:white_check_mark:Succeeded
View

microsoft-community-training/media/featuredisableautoenrollcourse.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/Steps to retrieve learner certificates for a course.docx
:white_check_mark:Succeeded

microsoft-community-training/media/Steps to retrieve learner certificates for a course_LP.docx
:white_check_mark:Succeeded

microsoft-community-training/frequently-asked-questions/faqs-general.md

Line 196, Column 1: [Warning: hard-coded-locale - See documentation] Link 'https://docs.microsoft.com/en-us/azure/active-directory-b2c/language-customization?pivots=b2c-user-flow#support-requested-languages-for-ui_locales' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 196, Column 1: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/en-us/azure/active-directory-b2c/language-customization?pivots=b2c-user-flow#support-requested-languages-for-ui_locales' will be broken in isolated environments. Replace with a relative link.

microsoft-community-training/settings/configurations-on-the-training-platform.md

Line 195, Column 79: [Warning: hard-coded-locale - See documentation] Link 'https://docs.microsoft.com/en-us/azure/industry/training-services/microsoft-community-training/content-management/create-content/create-course-category/create-a-new-course' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 195, Column 79: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/en-us/azure/industry/training-services/microsoft-community-training/content-management/create-content/create-course-category/create-a-new-course' will be broken in isolated environments. Replace with a relative link.

microsoft-community-training/frequently-asked-questions/faqs-user-management.md

Line 24, Column 239: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/azure/active-directory-b2c/overview' will be broken in isolated environments. Replace with a relative link.
Line 186, Column 211: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/azure/active-directory-b2c/custom-policy-overview' will be broken in isolated environments. Replace with a relative link.
Line 190, Column 34: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/azure/active-directory-b2c/tutorial-create-tenant' will be broken in isolated environments. Replace with a relative link.
Line 191, Column 3: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/azure/active-directory-b2c/tutorial-register-applications?tabs=app-reg-ga' will be broken in isolated environments. Replace with a relative link.
Line 191, Column 139: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/azure/active-directory-b2c/tutorial-register-applications?tabs=app-reg-ga#enable-id-token-implicit-grant' will be broken in isolated environments. Replace with a relative link.
Line 257, Column 4: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/azure/active-directory-b2c/tutorial-create-user-flows?pivots=b2c-custom-policy' will be broken in isolated environments. Replace with a relative link.

microsoft-community-training/infrastructure-management/install-your-platform-instance/configure-login-social-work-school-account.md

Line 85, Column 51: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/azure/active-directory/develop/quickstart-register-app' will be broken in isolated environments. Replace with a relative link.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


ci/papr: Disable and remove docker on FAH27

For now, per discussion e.g. iptables rules can interfere.

Looks like something is trying to connect to docker after you tried to kill it in the tests.

Looks like something is trying to connect to docker after you tried to kill it in the tests.

OK, I took a stab at reworking this :arrow_up:  - let's see.

@cgwalters I can make this work for us but I am already doing something similar with the cloud image.  That's why I was asking about having a customized predeployed image for FA27 that ran podman and not docker.

Looks like this was obsoleted by https://github.com/projectatomic/libpod/pull/524


[Feature] Velaux implement Component/Trait Definition management page

Is your feature request related to a problem? Please describe.
We can only use vela CLI to manage the Definitions, maybe we should to implement the Definitions manage page in Velaux.
Describe the solution you'd like

@Ghostbaby Could you provide some helps for this feature? Such as contributing to the API implementation.

If we implement this feature, we can have two steps:

just provide a way to copy-paste existing definition with UI operations, users should edit in their local IDE and debug to make it work.
We should provide a convenient way to edit the definition files, provide validation, auto-completion in some cases.


Windows-style slashes in portfile fix suggestinons

Describe the bug
When port install fails, vcpkg suggests changes to the failed portfile.cmake which includes incorrect Windows-style slashes:
There should be no empty directories in C:\projects\vcpkg\packages\clipboardxx_x86-windows
The following empty directories were found:

    C:/projects/vcpkg/packages/clipboardxx_x86-windows/debug

If a directory should be populated but is not, this might indicate an error in the portfile.
If the directories are not needed and their creation cannot be disabled, use something like this in the portfile to remove them:

    file(REMOVE_RECURSE "${CURRENT_PACKAGES_DIR}\debug")

It's a bug because if you copy-paste a provided piece of code (containing \), it results in a CMake error.
Environment

OS: Any
Compiler: Any

To Reproduce
Steps to reproduce the behavior:

Take a new port from my PR https://github.com/microsoft/vcpkg/pull/24023/files
Change /debug to /debug/include in portfile.cmake:17
vcpkg install clipboardxx
Congrats! You've got a wrong tip

Expected behavior
Expected / character instead:
    file(REMOVE_RECURSE "${CURRENT_PACKAGES_DIR}/debug")

Failure logs
CMake Error at ports/clipboardxx/portfile.cmake:17 (file):
  Syntax error in cmake code at

    C:/projects/vcpkg/ports/clipboardxx/portfile.cmake:17

  when parsing string

    ${CURRENT_PACKAGES_DIR}\debug

  Invalid character escape '\d'.
Call Stack (most recent call first):
  scripts/ports.cmake:145 (include)

Related line https://github.com/microsoft/vcpkg-tool/blob/main/src/vcpkg/postbuildlint.cpp#L744

Seems to be fixed in upstream.

This issue has not been fixed. If you add the --binarysource=clear option in the install command, it will occurs again.


Inconsistent crop window size

Describe the bug
Given a constant parent element size, the size of the crop window (be it circle or rectangular) changes based on whether the images fits the parent container
To Reproduce
Steps to reproduce the behavior:

Go to the example: https://codesandbox.io/s/react-easy-crop-demo-with-cropped-output-forked-3r9whb?file=/src/index.js:453-550

Change the link address to a photo that's of a different aspect ratio.
Eg. https://upload.wikimedia.org/wikipedia/commons/thumb/d/da/Яйца_всмятку_или_денатурация_альбумина.jpg/500px-Яйца_всмятку_или_денатурация_альбумина.jpg

You can now see that the crop box isn't the right size (no longer fills vertical height) because it's adjusting to the image
Expected behavior
A clear and concise description of what you expected to happen.

I believe this is caused by the image initially having something like the CSS property object-fit:contain (just as an analogy)

It seems that the image you shared in 2. is the same than the one used in the 1. sandbox. Did you overwrite the sandbox?

It seems that the image you shared in 2. is the same than the one used in the 1. sandbox. Did you overwrite the sandbox?

My apologies, I meant to use the link from the original example, just updated the issue

Oh I think it is! Thank you 👍


#143239263 : Enable org level service access in acceptance tests

Given that we don't need  enable service access before acceptance test. you may want to revise readme.MD accordingly, say  1) remove enable service access to all orgs in section 'register service'  2) add enable service access to all or specific orgs in section 'use service'

@boyang9527  updated the README.md.

LGTM

LGTM


Problems with browser cache

Problem
I have a problem with the brwosercaching of js files, that are included into die view php files of my extension. In the deveoloper console i can see that pagekit appends something like "?v=3326" to the js files. How can i count this version up to force a refresh of the browser cache?
Technical Details

Pagekit version: 1.0.8
Webserver: apache
Database: mysql
PHP Version: 7

Either updating the config secret or the version number seems to trigger it. I'm not sure if you should do that without knowing the consequences (breaking stuff that's being used by the secret of disabling the update method)
https://github.com/pagekit/pagekit/blob/6d881162c0f62e482aeea176cfeb58bc859a54f8/app/system/src/SystemModule.php#L26-L27

because of this i created the issue :-) What is the best way to clear the java cache when i have done some changes in the js files of an extension.

You are trying to solve this problem in the wrong place. This cache is designed to break all caches after system updates by changing the filenames. If you use this mechanism during development, you have to change the name after each change, which is not an elegant solution.
Best practise for this is to simply disable your browsers cache. All major browsers provide settings for this (e.g. Chrome even has a setting for just disabling caching if the dev tools are open).

Ok.. but what happens if i have created an update-package for my extension? Will the version number also change in that case? I can't tell all the visitors of my page to clear the cache manually, you know?

You can provide the version number of your extension when adding new scripts. By doing so Pagekit will use this version for calculating the cache breaker instead of the Pagekit version.

If you include your scripts like this, the version of your module will be in the header, breaking the users cache on update.
$view->script('m-script', 'namespace/module:app/bundle/script.js', ['vue','uikit-tooltip'], ['version' => App::config('system')->get('packages.namespace/module')]);

While developing, disabling the browsers cache should work.


Change caddy path to github.com/caddyserver/caddy

1. What does this change do, exactly?
Changes caddy's import path from github.com/mholt/caddy to github.com/caddyserver/caddy
2. Please link to the relevant issues.
Closes #62
3. Which documentation changes (if any) need to be made because of this PR?
None

Thanks Sergey. Sorry I didn't have time to get around to it myself


Introduce MHLO_BUILD_EMBEDDED build option

This option allows to skip calling find_package(MLIR), enabling to
embed MHLO into other project, e.g. IREE.

@GMNGeoffrey: FYI


Add messageEdited and messagedDeleted events to ChatAdapter. Update the unread message badge when a message is deleted.

What

Add messageEdited and messagedDeleted events to the ChatAdapter.
Use these events to update the unread message badge in the callwithchat composite when a message is deleted.

Why
Unread messages badge should update when an unread chat message is deleted.
How Tested
Verified locally deleting a chat messages decrements the unread message count to other participants in the callwithchat

Added breaking-change label as the breaking change is adding new string literals to the on function which is acceptable level


Added examples

Added one example, corrected one example and added few notes to help in parameter sections

Docs Build status updates of commit b678a00:
:clock10: Full build: the average full build time is 14 min(s) 42 sec(s), this is based on the last 1000 build(s) for this repository.

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

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/federation-and-hybrid/Set-FederationTrust.md
:x:Error

Details

exchange/exchange-ps/exchange/federation-and-hybrid/Set-FederationTrust.md

[Error]

Failed to parse this MAML markdown file due to :93:(1183) '>[NOTE] For Office 365 or Exchange online tenants,...'
 Expect Heading, please use PlatyPS to do content validation

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.


Make generated YAML more readable and better formatted

Currently the output YAML has some weird empty lines, puts certain keys in weird order, and so on. This is because the library uses kaml library which has its quirks and doesn't give full control over e.g. order of emitted YAML keys.
A proposed solution is to generate the YAML manually. It would also make development easier because on the code level, kaml imposes some limitations as well (e.g. sealed hierarchy has to be used for actions which makes it impossible to divide them into packages).

Work in progress: https://github.com/krzema12/github-actions-kotlin-dsl/tree/rework-backend
Trying dead-simple approach, by just gluing strings. So far it's manageable and simple. At first I wanted to write my own library to emit YAML having great control over how it's done, but for now I decided to Keep It Simple :tm:  :) Let's see how far we get with this approach.


:goal_net: Feat: Good error messages

Description
Concise description of proposed changes
Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

#1186  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @roninjin10 and the rest of your teammates on  Graphite


Python extension can't find any interpreter

Environment data

VS Code version: 1.45.1
Extension version (available under the Extensions sidebar): 2020.5.80290
OS and version: Linux x64 5.6.15-1-MANJARO
Python version (& distribution if applicable, e.g. Anaconda): XXX
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): virtualenv
Relevant/affected Python packages and their versions: /
Relevant/affected Python-related VS Code extensions and their versions: /
Jedi or Language Server? Language Server
Value of the python.languageServer setting: Microsoft

Expected behaviour
VSCode should detect that python is installed
Actual behaviour
VS Code will ask you to install Python to your OS, even if it is installed.
Steps to reproduce:

Launch VS Code
Wait for the extension to load
Will ask you to choose an interpreter no matter what

Logs
Fail to run interpreterInfo.py :
Error Python Extension: 2020-06-05 12:27:50: Failed to get interpreter information for '/home/arbaes/.virtualenvs/sc-devtracker/bin/python3'
[Error: Command failed: "/home/arbaes/.virtualenvs/sc-devtracker/bin/python3" "/home/arbaes/.vscode-oss/extensions/ms-python.python-2020.5.80290/pythonFiles/pyvsc-run-isolated.py" "/home/arbaes/.vscode-oss/extensions/ms-python.python-2020.5.80290/pythonFiles/interpreterInfo.py"/home/arbaes/.virtualenvs/sc-devtracker/bin/python3: can't open file '/home/arbaes/.vscode-oss/extensions/ms-python.python-2020.5.80290/pythonFiles/pyvsc-run-isolated.py': [Errno 2] No such file or directory
	at ChildProcess.exithandler (child_process.js:308:12)
	at ChildProcess.emit (events.js:203:13)
	at maybeClose (internal/child_process.js:1021:16)
	at Process.ChildProcess._handle.onexit (internal/child_process.js:283:5)]
{  killed: false,  code: 2,  signal: null,  cmd: '"/home/arbaes/.virtualenvs/sc-devtracker/bin/python3" "/home/arbaes/.vscode-oss/extensions/ms-python.python-2020.5.80290/pythonFiles/pyvsc-run-isolated.py" "/home/arbaes/.vscode-oss/extensions/ms-python.python-2020.5.80290/pythonFiles/interpreterInfo.py"'}

it seems that pyvsc-run-isolated.py indeed does not exist anymore, even after the extension reinstallation.

Fixed it by removing the extension manually:
rm -rf ~/.vscode-oss/extensions/ms-python.python-2020.5.80290

Then reinstall it. I should have tried that first, sorry.


错字

Description
https://learn.microsoft.com/zh-cn/aspnet/core/blazor/tutorials/build-a-blazor-app?view=aspnetcore-9.0#build-a-todo-list-blazor-app
下的“向 Todo 组件添加 AddTodo 方法，并使用 @onclick 属性来为按钮注册方发。 选择按钮时，会调用 AddTodo C# 方法：”的“方法”写成了“方发”
Page URL
https://learn.microsoft.com/zh-cn/aspnet/core/blazor/tutorials/build-a-blazor-app?view=aspnetcore-9.0
Content source URL
https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/blazor/tutorials/build-a-blazor-app.md
Document ID
23f2cad2-31f9-d5d1-671c-5613fb812b19
Article author
@guardrex
Related Issues

Under "Add the AddTodo method to the Todo component and use the @OnClick attribute to register the button for the button. When the button is selected, the AddTodo C# method is called:" the "method" is written as "party"

Hello @aoxixixp ...
When looking at the non-English version of the article, please use the Feedback form to report it (the word "Feedback" will be in a non-English language) ...

When you select the button for "No" (translated), there's an entry for the type of feedback: 👉 "Quality of the translation" 👈 (also translated).
When you select that entry and describe the translation problem, the feedback goes to the localization team.


about eth2.0 staking

Hello
I have a question.
I know people are staking eth to become a validator in eth 2.0.
Is it possible to participate in eth staking even with a contract address that is not a general eth address?
If possible, ask how to delegate eth in the contract code.

Yes, you can use Ethereum mainnet contract address to make a deposit (staking) transaction to the deposit contract or to generate the withdrawal credentials.
This is a repository for the specification discussion. Please visit the launchpad FAQ and ethstaker community to get further supports. 🙂


Add Speech to Text Support (STT)

STT currently doesn't work due to lack of HTTPS support
Why doesn't STT/TTS work in my deployment?
https://docs.openwebui.com/faq/#q-why-doesnt-stttts-work-in-my-deployment
Is Open WebUI HTTPS Capable?
https://github.com/open-webui/open-webui/discussions/2127
One option is to host a nginx-proxy to wrap the open-webui docker service
https://github.com/nginx-proxy/nginx-proxy/tree/main/docs#ssl-support

Workaround described here:
https://tinyllama.insightreactions.com/guides/open-webui/allow-microphone.html


Fix error when displaying a graph with id properties

In graph.js, this.options.data.properties.id doesn't exist. It's this.options.data.properties.nodes.id.
In config.js, id must be false or the id property name, not true.

It seems it's breaking the rendering even on the ad1.json sample given, isn't it ?


[SPARK-3162][MLlib] Local Tree Training Pt 1: Refactor RandomForest.scala into utility classes

What changes were proposed in this pull request?
Breaks up #19433 to help unblock #19666; after this PR is merged, #19666 can be merged.
This PR contains the changes made to migrate functionality from RandomForest.scala into the following utility classes:

AggUpdateUtils
ImpurityUtils
SplitUtils

The PR also adds tests for split selection logic in TreeSplitUtilsSuite.
A follow-up PR will include the other changes from #19433:

Local decision tree data structures & tests
Local tree training logic & tests

How was this patch tested?
Adds unit tests for split selection logic in TreeSplitUtilsSuite
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)
Please review http://spark.apache.org/contributing.html before opening a pull request.

Test build #83910 has started for PR 19758 at commit b93f9f3.

Test build #83911 has started for PR 19758 at commit b4a5f3b.

Test build #83910 has finished for PR 19758 at commit b93f9f3.

This patch fails to generate documentation.
This patch merges cleanly.
This patch adds no public classes.

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/83910/
Test FAILed.

Test build #83911 has finished for PR 19758 at commit b4a5f3b.

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/83911/
Test PASSed.

Test build #83914 has started for PR 19758 at commit b6291e1.

Test build #83914 has finished for PR 19758 at commit b6291e1.

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/83914/
Test PASSed.

Test build #84379 has started for PR 19758 at commit 5bcccda.

Test build #84379 has finished for PR 19758 at commit 5bcccda.

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/84379/
Test PASSed.

ping?
I'm mostly interested in SPARK-3162

Any updates? this PR seems to address critical issue: https://issues.apache.org/jira/browse/SPARK-3162


external_redirects definition can now contains full URL

Hello.
I created this pull request, because it need it in my project.
My use case is:
I'm working on site A, that is a oAuth "client" of site B.
So some HTTP redirections occur from A to B.
We white-listed site B in our configuration, and that is working fine.
But now, I also run a (dev) copy of site B. And so the URL is different.
I don't want to pollute your config.yml with my local endpoint.
Therefore, I already have a parameter in my application to customize
the endpoint of B and I would like to re-use it. But my endpoint
parameter also constain the scheme.
That's why, if in the white-list, an url is provided, we extract
and use only the host part.
What do you think ?

Sounds fine to me. Thanks


Notifications with pinia

Adds pinia for #47

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[pigeon] KotlinOptions errorClassName not used in Kotlin to Dart communication

Is there an existing issue for this?

[X] I have searched the existing package issues
[X] I have read the guide to filing a bug

What package does this bug report belong to?
pigeon
What target platforms are you seeing this bug on?
Android
Have you already upgraded your packages?
Yes
Dependency versions
pubspec.lock
# Generated by pub
# See https://dart.dev/tools/pub/glossary#lockfile
packages:
...
  analyzer:
    dependency: transitive
    description:
      name: analyzer
      sha256: ea3d8652bda62982addfd92fdc2d0214e5f82e43325104990d4f4c4a2a313562
      url: "https://pub.dev"
    source: hosted
    version: "5.13.0"
  args:
    dependency: transitive
    description:
      name: args
      sha256: eef6c46b622e0494a36c5a12d10d77fb4e855501a91c1b9ef9339326e58f0596
      url: "https://pub.dev"
    source: hosted
    version: "2.4.2"
  collection:
    dependency: transitive
    description:
      name: collection
      sha256: f092b211a4319e98e5ff58223576de6c2803db36221657b46c82574721240687
      url: "https://pub.dev"
    source: hosted
    version: "1.17.2"
  meta:
    dependency: transitive
    description:
      name: meta
      sha256: "3c74dbf8763d36539f114c799d8a2d87343b5067e9d796ca22b5eb8437090ee3"
      url: "https://pub.dev"
    source: hosted
    version: "1.9.1"
  path:
    dependency: transitive
    description:
      name: path
      sha256: "8829d8a55c13fc0e37127c29fedf290c102f4e40ae94ada574091fe0ff96c917"
      url: "https://pub.dev"
    source: hosted
    version: "1.8.3"
  pigeon:
    dependency: "direct dev"
    description:
      name: pigeon
      sha256: c897b1abb2fa1d8fdb61aca34216796160ca7d9e34fef360a733e2608191acd4
      url: "https://pub.dev"
    source: hosted
    version: "14.0.0"
  yaml:
    dependency: transitive
    description:
      name: yaml
      sha256: "75769501ea3489fca56601ff33454fe45507ea3bfb014161abc3b43ae25989d5"
      url: "https://pub.dev"
    source: hosted
    version: "3.1.2"
...
sdks:
  dart: ">=3.1.4 <4.0.0"
  flutter: ">=3.7.0"

Steps to reproduce

Create a pigeon that uses Kotlin to Dart communication (i.e. FlutterApi). Here is some example code:

@ConfigurePigeon(PigeonOptions(
  dartOut: 'lib/pigeon/host_calls_dart_pigeon.g.dart',
  kotlinOut: 'android/app/src/main/kotlin/.../pigeon/HostCallsDartPigeon.g.kt',
    kotlinOptions: KotlinOptions(
        errorClassName: 'HostCallsDartPigeonFlutterError'
    )
))
@FlutterApi()
abstract class HostCallsDartPigeon {
  bool example();
}

Run the generator e.g. dart run pigeon --input pigeons/host_calls_dart_pigeon.dart
Output does not use HostCallsDartPigeonFlutterError:

// Autogenerated from Pigeon (v14.0.0), do not edit directly.
// See also: https://pub.dev/packages/pigeon

import android.util.Log
import io.flutter.plugin.common.BasicMessageChannel
import io.flutter.plugin.common.BinaryMessenger
import io.flutter.plugin.common.MessageCodec
import io.flutter.plugin.common.StandardMessageCodec
import java.io.ByteArrayOutputStream
import java.nio.ByteBuffer

private fun createConnectionError(channelName: String): FlutterError {
  return FlutterError("channel-error",  "Unable to establish connection on channel: '$channelName'.", "")}

/**
 * Error class for passing custom error details to Flutter via a thrown PlatformException.
 * @property code The error code.
 * @property message The error message.
 * @property details The error details. Must be a datatype supported by the api codec.
 */
class HostCallsDartPigeonFlutterError (
  val code: String,
  override val message: String? = null,
  val details: Any? = null
) : Throwable()
/** Generated class from Pigeon that represents Flutter messages that can be called from Kotlin. */
@Suppress("UNCHECKED_CAST")
class HostCallsDartPigeon(private val binaryMessenger: BinaryMessenger) {
  companion object {
    /** The codec used by HostCallsDartPigeon. */
    val codec: MessageCodec<Any?> by lazy {
      StandardMessageCodec()
    }
  }
  fun example(callback: (Result<Boolean>) -> Unit) {
    val channelName = "....HostCallsDartPigeon.isHomeScreenVisible"
    val channel = BasicMessageChannel<Any?>(binaryMessenger, channelName, codec)
    channel.send(null) {
      if (it is List<*>) {
        if (it.size > 1) {
          callback(Result.failure(FlutterError(it[0] as String, it[1] as String, it[2] as String?)))
        } else if (it[0] == null) {
          callback(Result.failure(FlutterError("null-error", "Flutter api returned null value for non-null return value.", "")))
        } else {
          val output = it[0] as Boolean
          callback(Result.success(output))
        }
      } else {
        callback(Result.failure(createConnectionError(channelName)))
      }
    }
  }
}

Expected results
Kotlin's example method should use HostCallsDartPigeonFlutterError instead of FlutterError when specified.
This all started with separating my pigeons into multiple files. Since each file uses FlutterError I would get Redeclaration: FlutterError. To resolve this, I set the errorClassName in the kotlinOptions of the @ConfigurePigeon.
Is there a better way to approach this or something that could be improved in pigeon to better handle multiple files of pigeons? I also noticed I can't seem to supply multiple files to --input.
Actual results
Kotlin's example method uses FlutterError.
Code sample
Code sample
@ConfigurePigeon(PigeonOptions(
  dartOut: 'lib/pigeon/host_calls_dart_pigeon.g.dart',
  kotlinOut: 'android/app/src/main/kotlin/.../pigeon/HostCallsDartPigeon.g.kt',
    kotlinOptions: KotlinOptions(
        errorClassName: 'HostCallsDartPigeonFlutterError'
    )
))
@FlutterApi()
abstract class HostCallsDartPigeon {
  bool example();
}

Screenshots or Videos

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.13.8, on Microsoft Windows [Version 10.0.19045.3693], locale en-US)
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
[√] Chrome - develop for the web
[√] Visual Studio - develop Windows apps (Visual Studio Community 2019 16.11.22)
[√] Android Studio (version 2022.3)
[√] VS Code (version 1.83.1)
[√] Connected device (3 available)
[√] Network resources

• No issues found!

Thanks for the detailed report and potential fix. Was able to replicate using given steps.

 stable, master flutter doctor -v
[!] Flutter (Channel stable, 3.16.0, on macOS 12.2.1 21D62 darwin-x64, locale
    en-GB)
    • Flutter version 3.16.0 on channel stable at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision db7ef5bf9f (12 days ago), 2023-11-15 11:25:44 -0800
    • Engine revision 74d16627b9
    • Dart version 3.2.0
    • DevTools version 2.28.2
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

[!] Flutter (Channel master, 3.17.0-15.0.pre.29, on macOS 12.2.1 21D62
    darwin-x64, locale en-GB)
    • Flutter version 3.17.0-15.0.pre.29 on channel master at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision c45f583799 (19 minutes ago), 2023-11-27 00:04:28 -0500
    • Engine revision 292a92166a
    • Dart version 3.3.0 (build 3.3.0-152.0.dev)
    • DevTools version 2.30.0-dev.4
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/dhs/Library/Android/sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for
      more details.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 13C100
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] IntelliJ IDEA Ultimate Edition (version 2021.3.2)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 65.1.4
    • Dart plugin version 213.7228

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.29.0

[✓] Connected device (3 available)
    • Darshan's iphone (mobile) • 21150b119064aecc249dfcfe05e259197461ce23 • ios
      • iOS 15.3.1 19D52
    • macOS (desktop)           • macos                                    •
      darwin-x64     • macOS 12.2.1 21D62 darwin-x64
    • Chrome (web)              • chrome                                   •
      web-javascript • Google Chrome 109.0.5414.119

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.


[MI-2343][server+webapp]: Added support to show pipeline release link preview.

Added support to show pipeline release link preview
Made changes to show the pipeline icon completely in MM post

Codecov Report
Base: 52.91% // Head: 51.65% // Decreases project coverage by -1.26% :warning:

Coverage data is based on head (a0802a3) compared to base (e8ff0c2).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@               Coverage Diff                @@
##           epic_phase_3     #105      +/-   ##
================================================
- Coverage         52.91%   51.65%   -1.27%
================================================
  Files                15       15
  Lines              2126     2178      +52
================================================
  Hits               1125     1125
- Misses              916      967      +51
- Partials             85       86       +1

Impacted Files
Coverage Δ

server/plugin/api.go
39.63% <0.00%> (-0.44%)
:arrow_down:

server/plugin/client.go
52.61% <0.00%> (-1.98%)
:arrow_down:

server/plugin/plugin.go
63.82% <0.00%> (-4.36%)
:arrow_down:

server/plugin/taskLinkPreview.go
43.97% <0.00%> (-12.40%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


ExcludeFromSourceBuild not working in aspnetcore

Similar to https://github.com/dotnet/source-build/issues/1825, ExcludeFromSourceBuild is not fully working in aspnetcore.  Recently when trying to debug an SDK failure I saw that src/Mvc/Mvc.Razor/test/Microsoft.AspNetCore.Mvc.Razor.Test.csproj was being compiled - this should be excluded because it's a test project.

There are at least two distinct causes that we know of:

in #1825, the suspicion is that we statically restore runtime without the DotNetBuildFromSource variable being available.
building a solution that explicitly includes a project with ExcludeFromSourceBuild=true will still build it.

The case under I found this was a local build issue (that wasn't reproducible for others) when I was testing the bootstrapping process.  The stage1 build did not get a Razor SDK binplaced into it.  This caused it to fail to compile the above Razor test project when trying to build the stage2 SDK.  The Razor smoke-tests also caught this missing SDK issue once I tried running them.


Request : Icon Location

Hello, I'm using Minimal UI Module for my convenience.
I think it would be more convenient if the BC Dice Icon moves under the sidebar icon group of character token, so that it does not breaks other UI modules.
It is a tiny, small request but please consider this.
Thank you for your nice work.

possibly related to #23

Hmmm, this is interesting. I'll have to play around with it to see how UI modules apply styles. But yes, I'll definitely try to resolve this soon.

@jbblily I've modified the way the control is loaded in so it shouldn't stick out so much anymore. It's not under the character token subgroup because I don't think that necessarily makes sense grouping wise. I'm looking at ways to make its location configurable but that will be something I work on later.


Added go-contribution-guidelines.md

This solves an issue brought up in https://github.com/ipfs/go-ipfs/pull/1734; how do we maintain similar standards over many repositories using the same language without duplicating language guidelines? All go repositories should now point to this file, and we can update it accordingly.
Most of this was pulled from https://github.com/ipfs/go-ipfs/blob/master/contribute.md. I have a branch ready to strip that file and point here if this is merged.

thanks @RichardLitt


Revert "Store banner change"

Reverts blueedgetechno/win11React#358

❌ DEPLOY PREVIEW BUILD HAS FAILED

Shut


jnitrace blank response

c:\adb>jnitrace -l libsc.so com.exampleapp.android
Tracing. Press any key to quit...
Traced library "libsc.so" loaded from path "/data/app/com.exampleapp.android-1/lib/arm".
Why i can't grab any information?
Thanks.

+1

Hi,
There are lots of reasons why this might be the case. There should be a bug or the library might not be being called for your steps.
Can you share more information on the application you are testing and the versions of Frida and jnitrace you are running.
Thanks


Include additional folder for the container volume mounts in dind-rootless variant

I have a k8s pod consisted of an app container and a dind container and want to share the dind volumes to the app container. It would be nice to create a folder in the dind-rootless image for that (/home/rootless/data perhaps? /home/rootless/volumes?).
As of now, I need to build another image from dind-rootless to make permissions work across images since there is no way to set up permissions during the volume mount (I'm using a non-root user on the app container as well):
FROM docker:19.03.11-dind-rootless

RUN mkdir -p /home/rootless/volumes \
  && chown -R rootless /home/rootless/volumes

Not a huge deal, however, it seems like a common use-case and perhaps it's reasonable to include in the dind-rootless image itself.

Isn't this what fsGroup and fsGroupChangePolicy are for?
(https://kubernetes.io/docs/tasks/configure-pod-container/security-context/#set-the-security-context-for-a-pod and https://kubernetes.io/docs/tasks/configure-pod-container/security-context/#configure-volume-permission-and-ownership-change-policy-for-pods)

@tianon thanks, yeah, that would be a proper solution in the context of k8s, however, it would be still useful to have this sorted using only Docker.
For example, we are also using the same images with docker-compose for testing, development containers, etc.


How can I let the browser follow the link?

If route(function() {}) callback is triggered, but I still want to let the browser follow the link, how can I do that?

what do you mean by let the browser follow the link?

To just load the page the link is pointing to.

Closing this issue because it's related to an old router version. Please update to the latest @riotjs/route version if you can.


Output image ID at end of build/push

Also -- keep a handle on a unique image name during the build process to
avoid race conditions.
Closes #802

@zeke you might want test build and push still work as expected... I haven't fully tested this...

Works for me! Here's how I tested it:
~/git/replicate/cog output-image-id-at-end-of-build
$ make install
...

~/git/zeke/cog-haiku main
$ cog --version
cog version v0.4.4+dev (built 2022-11-15T15:11:30-0800)

~/git/zeke/cog-haiku main
$ cog push
Building Docker image from environment in cog.yaml as r8.im/zeke/haiku-standard...
...
4ed121b04368: Layer already exists
d9d07d703dd5: Layer already exists
latest: digest: sha256:8ab3c4ed267b3badd9ff4b638693a000a19800987a7d863007d5b44df89444c4 size: 3269
Image 'r8.im/zeke/haiku-standard' pushed

Run your model on Replicate:
    https://replicate.com/zeke/haiku-standard
sha256:795173aa5b619f9cdb69d32cd344986327cda5da2d295597ef33b16419128bdc

$ STDOUT=$(cog push)
...

$ echo $STDOUT | tail -n1 |  sed -e 's/sha256://'
91467606ad24771b3a9555f8a30028a6986334b09e0bc2134d70909e6e4550bf

This is not the only thing output on stdout -- we should fix that.

I don't think this does work -- wrong IDs.

I guess we're abandoning this for now?


imageParams on Image react component don't play nice with allowedMediaParams

Description
The adding the same values to imageParams in an Image component and allowedMediaParams in a patch config does not result in an image getting sized correct. The original media URL coming from the layout service includes original h/w/iar/hash parameters that aren't getting cleared when mediaApi.updateImageUrl gets called. This appears to have changed as part of issue #172 and PR #275 .
Expected behavior
Passing in the exact same parameters to imageParams and allowedMediaParams should result in an image getting sized correctly.
Steps To Reproduce

Create an Image instance with imageParams.
Add the exact same imageParams to the allowedMediaParams patch config.
Look at the rendered image to see if it's sized correctly based on the params.

Possible Fix
Revert the change to the mediaApi.

@ssartell Fixed in #449. Will be released in JSS 15


activities and pois show as vector tiles

Overview
This PR includes the following proposed change(s):

activities and pois show as vector tiles

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Refactoring / Documentation
[ ] Version change

if your change is a breaking change, please add breaking change label to this PR
How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Screenshots
Please add any relevant UI screenshots if applicable.
Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] New and existing unit tests pass locally with my changes

/rebase


Use Minimum Login Timeout as 1 sec in .NET Core and enable behavior by default

This PR brings over app context switch "UseOneSecFloorInTimeoutCalculationDuringLogin" to use 1 second instead of 0 seconds as minimum login timeout to prevent indefinite timeouts and potential application hangs.
Also enables the app context switch by default, in both NetFx and NetCore.
NOTE: Public documentation for this App Context switch needs to enable support in .NET Core and .NET Standard when this change gets released.

LGTM, but why are we adding an app context switch to address a bug? Whoever is facing this edge case situation must have suffered from it. Is this just to avoid breaking changes?

LGTM, but why are we adding an app context switch to address a bug? Whoever is facing this edge case situation must have suffered from it. Is this just to avoid breaking changes?

The switch was already there. This just brings it to netcore and turns it on, by default. Keeping it around will let customers turn it off, just in case there was a good reason someone added it to netfx in the past. When we were talking about it, we couldn't think of a good reason the change was put under a context switch.
It seemed like there were two possible side effects of the change under the switch.

It could slightly extended a connect timeout time. (But who would actually notice this?)
Connection timeout errors could occur when slow connections right on the edge of your connect timeout might have previously succeeded beyond the timeout. (Fix: Extend your timeout if you were expecting this behavior.)


feat: add shape factory & shape

Checklist

[ ] npm test passes
[ ] tests and/or benchmarks are included
[ ] geometry 调用 element + shape 的逻辑

Description of change

Pull Request Test Coverage Report for Build 892926704

3 of 228 (1.32%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-14.6%) to 53.063%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/types/geometry.ts
0
1
0.0%

src/visual/coordinate/index.ts
0
1
0.0%

src/util/coordinate.ts
0
3
0.0%

src/geometry/factory.ts
0
52
0.0%

src/util/path.ts
0
168
0.0%

Totals

Change from base Build 892293122:
-14.6%

Covered Lines:
639

Relevant Lines:
1147

💛  - Coveralls

shape 这一块的核心改动是什么，科普下 @hustcc


Pipette definition not compliant with Opentrons API v2.20

Need to change how the pipettes are defined so that they can be removed from the tsv file.

Fixed in v3


Engine upload should a̶c̶c̶e̶p̶t̶ handle invalid metadata

Currently, a gear's output - files and metadata - are dropped if there is a metadata validation error. This isn't visible to the end user, and risks losing a lot of work.
My proposal would be that:

The metadata (invalid or otherwise) is saved on the job object, as it normally would be.
The metadata is not applied to the destination container. Duh.
The files are accepted. It's too confusing for a non-author to do otherwise: "where's my files?!"
The job gains a new field, saving the validation error for later use.

Show that error, along with original metadata, in the UI when viewing job logs.

Consider having the UI highlight the files when their origin is a job that has this issue.
~Maybe the job gains a new state as well, invalid? Not sure I like that.~
Ideally, files produced in this error condition would not trigger other jobs.

Example follows, ref scitran-apps/dicom-mr-classifier#4:
16:27:29 base.py:427 from None None POST /api/engine {u'job': u'5923108aa56881001eb8409b', u'id': u'5923108aa56881001eb84099', u'level': u'acquisition'} request_id=bbc36a4a-1495470449
16:27:29 base.py:332 788400000.0 is not of type u'integer', u'null'

Failed validating u'type' in schema[u'properties'][u'session'][u'properties'][u'subject'][u'allOf'][0][u'properties'][u'age']:
    {u'type': [u'integer', u'null']}

On instance[u'session'][u'subject'][u'age']:
    788400000.0 request_id=bbc36a4a-1495470449
16:27:29 POST /api/engine?id=5923108aa56881001eb84099&job=5923108aa56881001eb8409b&level=acquisition => generated 314 bytes in 9 msecs (400)

I'm going to let this sit for perhaps a week, then assume we're in agreement.

I have gone back and forth on this over the course of the day... whether bad metadata should render the run a failure and thus discard the outputs altogether. The bottom line for me is that the gear was meant to do something and, and in this case it didn't do it. I agree that it's confusing to not have files show up because the metadata were invalid, however, if those things go together, or the metadata is required as part of an analysis pipeline (that's getting ahead I know) then I would want to know that happened and I would consider that a gear execution a failure. What makes me uneasy is that this is a second-order failure, in that the gear actually executed without failure (exit code = 0), but the api will reject the result. We would have to make that very clear in the UI, be it by displaying another state or something else.
Other thoughts?

I agree with you in principle. But, I think two things are very clear to me:

We IMO don't have any option other than showing the result files where they would display normally. There is just no way Joe Non Author can be expected to understand his gear didn't work, and instead the results are hiding three tabs to the left and a modal down, or something.

Gear metadata failure has the potential to be --> therefore will be unpredictable. Edge cases that aren't frequently / ever activated, buggy code, divergence. I saw the classifier change ("our" fault, we got more strict about a key) because I was watching the engine logs, but in a bug bash we probably wouldn't have seen it unless checking the long provenance tab.

For these reasons I'm preeeetty convinced we should put the files where they'd normally go, and use the origin field to highlight them in the UI. Maybe these files can be red, and clicking them brings up a modal explaining the job failure, or something. Visible in the same place you'd go for successful results.
I agree that data could be invalid if the metadata is missing - what if we offered an option to delete all produced data when they're viewing the "why this job failed" UI?
I also like your comment about pipelines. Files produced by a "semi"-failed job should probably not trigger other jobs! That could definitely be part of this change.
Finally, long ago I offered the idea that gears could choose this behavior. Basically:
{
   // ...
   "on-metadata-failure": "continue || reject || warn"
}

In this case the default would be continue (the proposed behavior above), but you could be more strict as desired.

I continue to agree with @kofalt. Outright rejection is not a option, and neither is calling the job run a success. Prominent indication of "metadata failure" seems prudent.

Sounds good. Proposal accepted; the gear-spec field can be a future enhancement.

Proposal looks reasonable to me given the current state of affairs. I presume that this will be re-visited when pipeline support becomes more refined.
Will the same proposal be applied for non-validation related metadata errors, such as Json can't be parsed, metadata too large for mongo storage, other?
Related to offering flexibility with this behavior in the future. If that becomes a reality, users should have the ultimate authority, not the gear authors.
Regarding pure classifier gears, where output is metadata for the input file, and not any output files: I presume that the job would still go into a failed status?

I presume that this will be re-visited when pipeline support becomes more refined.

I hope not: the job still goes into a failed state, this is just a more specific interpretation of the failure. But maybe there's more nuances there, if a pipeline discussion ticket is filed I'll definitely read with interest.

Will the same proposal be applied for non-validation related metadata errors, such as Json can't be parsed, metadata too large for mongo storage, other?

No, because I don't think it's realistic to handle those cases. This is very specifically for metadata that we don't accept; the gear is still responsible for acting reasonably.

users should have the ultimate authority, not the gear authors.

Mmm, I don't agree, but we can worry about it another time :)

Regarding pure classifier gears, where output is metadata for the input file, and not any output files: I presume that the job would still go into a failed status?

Correct. This only changes the job object, not the file objects (or lack thereof).

Will the same proposal be applied for non-validation related metadata errors, such as Json can't be parsed, metadata too large for mongo storage, other?

No, because I don't think it's realistic to handle other cases. This is very specifically for metadata that we don't accept; the gear is still responsible for acting reasonably.

@kofalt I don't think your response is wrong. It does move the needle further away from simple and predicable API behavior. Is it not realistic due to implementation complexity, or would it lead to negative user satisfaction?

Implementation complexity. In general anything touching I/O is pretty complex, with a lot of edge cases and legacy platform issues


add ctrl+(p/n)=up/down for dmenu

hi @sminez,
I don't know if you are accepting contributions, I like the idea of this editor, I'm trying to use it
what I noticed immediately was lack of some standard vim keys that I'm used to
let me know if you're interested, I will add more e.g. $, ^, 0, ctrl+(left/right) - bprevious/bnext and so on

Hi @arekmaz, thank you for raising this but I'm afraid that I'm not really accepting contributions at the moment as the project is still very much in its early stages. I don't have any contributor guidelines in place or the overall design written up for people to reference.
Importantly for this PR: adding all of the "missing" vim bindings is actually something I'm deliberately not doing. Especially for the main editor as the goal is very much not to clone the full behaviour of vim, even in terms of motions and key bindings. There are definitely more key bindings that I will want / need to add but I'm using the editor daily at the moment in an effort to work out where things are missing and how I want to address them.
In the case of moving between lines in the minibuffer you can see from the lines that you've edited in the PR that alt-j/k will handle that at the moment and I find that significantly easier to type and more in keeping with hjkl motions elsewhere in the editor. Depending on how far I get with customising key bindings it may be possible to provide custom keymaps for the different editor modes and the minibuffer at some stage, but doing so now would add a significant amount of complexity to the code base so it's not something I'm planning on tackling soon I don't think.


install-chef-suse: verify tables exist before deleting on failed install

If initial installation fails before production.sqlite3 has been created
or populated, subsequent runs of the install script will bail out early
with "Error: no such table: proposals" when it tries to do some cleanup.
This change checks to make sure those tables actually exist before
attempting to delete them.
Signed-off-by: Tim Serong<EMAIL_ADDRESS>

Hrm, I think the intent was to replace this code with a call to what @tboerger added in https://github.com/crowbar/crowbar-core/pull/141
@tboerger what's the status on that?

@tboerger what's the status on that?

I have no idea, that was a task for @rsalevsky

@rsalevsky feel free to integrate https://github.com/tboerger/crowbar-core/commit/2a03287e0dbc7773e42a9d46dc45232bebf5c771

Closing in favor of https://github.com/crowbar/crowbar/pull/2204


Bump matplotlib from 3.3.4 to 3.5.2

Bumps matplotlib from 3.3.4 to 3.5.2.

Release notes
Commits

updated-dependencies:

dependency-name: matplotlib
dependency-type: direct:production
update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot]<EMAIL_ADDRESS>Proposed changes:

...

Status (please check what you already did):

[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog (please check changelog for instructions)
[ ] reformat files using black (please check Readme for instructions)

Closing this as it is a duplicate of #11121 .


Add TerminalRenderer API

@dgriffen, this is for  #56 TerminalRenderer API
This PR adds support for ITerminalRenderer. Terminal renderer encapsulates terminal UI with xterm control on VS tool window, and delegates handling of terminal stdio streams and resizing to the consumer.
There are two kinds of renderers: auto-fit and fixed size.
Auto-fit renderer behaves the same way as the normal terminal, always fitting its tool window. Whenever it is resized, it fires the TerminalResized event.
Fixed size renderer starts with predefined size and always keeps this size, regardless of the tool window. This size later can be changed with ITerminalRenderer.ResizeAsync(). If the tool window is smaller than the fixed size renderer, the renderer is clipped. If the tool window is larger, empty space is filled with background color. Vertical scroll bar always appears on the right size of the tool window, unless the tool window is smaller and it clips the render. There are no double scroll bars.

Added window resize event and dotted border for fixed size terminal.
Window resize event will fire when terminal window size changes. It may be used to adjust the terminal size. The idea is to always fit the terminal into the window, and for that we need to know when the window resizes. Then the client can resize the terminal to fit the window, and to notify others.
Here is how it will look when the terminal size smaller than the window size. The dotted border is on the right and bottom sides and uses themed color.

There are no changes in the appearance for the auto-fit mode.
@dgriffen this is the final iteration for VSLS, please take a look.

@dgriffen Updated PR as per your feedback.

@dgriffen Updated again per your feedback. Fixed the window transient state for embedded terminal window.

The sooner the better. I already have VSLS changes ready.

I'll try and get something out this weekend then.


[NOT YET READY FOR REVIEW] Add new evaluation tasks for JP models

Closing in favor of https://github.com/Stability-AI/lm-evaluation-harness/pull/67


Refactor and add getting the raw whois data also for unsupported tld's

use the same type of file reading on Popen and the testFile
rename variables in init query: d -> dl, pd -> data, q -> whois_str, so the function args are the same as are all others already
split of the beginning of query into clearly named functions to increase readability of the steps done

add return_raw_text_for_unsupported_tld to return only the raw text for unsupported domains
updated the readme

thanks
On Sun, 13 Nov 2022 at 13:34, MooCow @.***> wrote:

Merged #238 https://github.com/DannyCork/python-whois/pull/238 into
master.
—
Reply to this email directly, view it on GitHub
https://github.com/DannyCork/python-whois/pull/238#event-7798334595, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACU73CXTVDWWYS3COOOXYRDWIDN6PANCNFSM6AAAAAAR4NSJKA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Figure out the process of issue management for the OVN

This issue is being assigned to Oita, Jim, and Jon, to resolve in their meeting of 2021.01.05.


fix save algo with similar file

This change is

/deploy

/deploy

/deploy

/deploy

/deploy


fix warning in docs generation

Fixes warnings in docs generations:

Two links in docs in functions in the Curve trait were not correctly referenced, so I added references to the functions they were trying to reference.
Two links in portfolio.rs had redundant information when referencing other parts of the code causing errors in docs generation so they were removed.

After this I don't see any warning in docs generation when I build locally, and I the links go to the functions.

Awesome, thank you very much !


--access checkout does ignore rollups

when working with --access checkout, it seems like cbr still sends api calls to github in order to figure out the commits contained in a rollup instead of using the cloned repository.

I don't think it is possible to get the information from the repository. It has to access the GitHub comment posted by the perf bot to know which artifacts to download.


The leshan jetty UI has wrong number of connected client.

When I added the 201 clients to leshan server,
initially, the UI shows 201 connected clients and, then,
it drops to 14 connected clients and, then,
it back to 201 again and, then,
it drops to 1 ...
However, when I use redis-cli to issue "keys REG:EP:*",
this command always return 201 records.
The Jedis implementation from Leshan has some issues.

I suppose you are using Leshan-server-demo with redis option (-r) ?
Did you faced the same kind of behavior without redis ?

The Jedis implementation from Leshan has some issues.

What do you mean exactly, could you be more specific ?

Hi,
I always run it with redis.
Here is my code to get the count of registrations.
Iterators.size(lwServer.getRegistrationService().getAllRegistrations());
However, after registering 200+ devices, after a while, the number became to drop.
It might go back to the original number and, then,
it will be dropping again ...
it will give some wrong small number eventually ...
However, if I get the number of registration directly through redis-cli,
the number of the connected devices is correct.
It means something wrong with the piece of code.
Iterators.size(lwServer.getRegistrationService().getAllRegistrations());
I also try to use hasNext(), next() instead of Iterators.size.
It gives me the same result.
Hence, I strongly suspect the iterator implementation of leshan has some issue ...
Maybe, it needs to reset the iterator or something.
Myself, I implement a lua script to get the total count.
Maybe, leshan should adopt that.
Thanks :)

One more thing, I try to send many requests when the device updates and I have another process accessing the redis, but they are not touching each other's data in redis. Does leshan server need to have a dedicated redis instance ?

I am thinking, perhaps, the iterator provided by leshan server's registration server will be affected by other services if the redis instance is shared ... just my guess ...

I found a bug which could match to your description.
I provide a fix #393.
Could you test this and see if it behaves better now ?

thanks, sbernard31.
i am working on the fixed version.
i don't think my version will be allowed to change anytime soon ...
will this be merged to the master branch soon ?

I just tried it. It seems to work now. The numbers are right :)

As it seems to work I merged it into master. We can close this issue.
Thx for reporting that @chuchiliusandc

thanks :)


LUCENE-10334: Introduce a BlockReader based on ForUtil and use it for NumericDocValues

see https://issues.apache.org/jira/browse/LUCENE-10334
#11370
Changes:

Make ForUtil fit for 64bit.
Introduce the new BlockWriter and BlockReader.
Replaces DirectReader with BlockReader and DirectReader with BlockWriter.

Hi @gf2121 -- did we ever performance test this change?  It sounds compelling!


[BUG] User is not able to use expanded volume if reverted to a snapshot before volume expansion

Describe the bug
User is not able to write in expanded part of volume if reverted to snapshot which was created before volume got expanded.
To Reproduce
Steps to reproduce the behavior:

Create a volume(3 replicas) - 3 GB, Attach to a pod
Write 2 GB data into the volume. file1
Create a Snapshot 1 and backup 1.
Expand the volume to 5 GB.
Write 2 GB data again into the volume. - file2
Create a snapshot 2 and backup 2.
Remove the file2.
Revert to snapshot 1, Try to write 2 GB into the volume.
It throws error, no space left in device

Note
If user write without reverting to snapshot, system allows to write till device is not full.
Expected behavior
If volume is expanded, it should allow user to use the expanded part of volume even if we revert to a snapshot where volume was not expanded.
Environment:

Longhorn version: v1.0.0-rc1
Kubernetes version: v1.17.5
Node OS type and version: Ubuntu 18.04

This is similar to https://github.com/longhorn/longhorn/issues/17
By following https://github.com/longhorn/longhorn/blob/v0.8.0/docs/expansion.md#handling-volume-revert, I was able to resize the volume and use it.


Avoid geometry copy error

Some meshes can have some weird vertexBuffer with no vertexData associated to, like the one imported from the obj file format : world0, world1...
This MR avoid to get an error copying the geometry and just keep copying vertexBuffers with actual vertex data

Thanks!


Add onboardings to vega lite treemap

Closes #107
Summary:

Added the onboarding to show the chart title.

screenshot:

@dvdanielamoitzi In vega treemap, only title is added to the onboarding message. From the runtime object I can only get the information related to title.

Fixed in #283


Update Portainer to v 2.5.1

Updating Portainer to the current version.
Proposed Changes
updating from v 1.24.0 to v 2.5.1
Related Issues

(Github link to related issues or pull requests)

Can you detail the testing that was performed for the upgrade?

I was already using 2.5.1 via portainer/portainer-ce from docker hub on another machine.
I decided to move it on the HA host for convenience, and I noticed it was an ancient version.
Since the Dockerfile only does hass-io ceremonial stuff other than untarring the tar-ball, I thought to bump up the tar's version for "contemporarity".
Other than that I don't write portainer, so not sure what testing I should to besides what everybody using it is already "doing" 😆

There are major changes between 1.x and 2.x for Portainer, particularly around some of the flags used on startup and features they relate to.
Please at least test the functionality locally before submitting a PR.

no idea how 🤷♂
cheers


Introduce support for NF4 data type for OV weight compression

What does this PR do?

Added a new possible value for --weight-format CLI argument: nf4.
Added a couple of tests for compression with the  nf4 data type.

Usage examples:
Optimum CLI:
optimum-cli export openvino -m facebook/opt-125m --task text-generation-with-past --weight-format nf4 ./tmp

Optimum Python API:
model = OVModelForCausalLM.from_pretrained(
    model_id="gpt2",
    export=True,
    quantization_config=OVWeightQuantizationConfig(bits=4, weight_format="nf4"),
)

Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

@AlexKoff88 please take a look

@l-bat, can you fix the style, please?


Promises

Proof of concept
enables the following syntax.
const image = new Jimp('./exampleImages/dice.png')
  .scale(0.5)
  .grayscale()
  .write('image-small.png')
  .blur(50)
  .write('image-blur.png')
  .then(res => {
    console.log('all done', res);
  });

all prototype methods are turned into promises
then must come at the end of the chained methods
can pass 'sync' as the last parameter to new Jimp(...options, 'sync') and the constructor and methods act synchronously.

still doesn't work for filePaths or custom constructors

Here is where all the magic happens:
	jimpPromise = cb => {
        const p = new Promise(cb);

		// Wrap each jimp method in a jimpPromise. This enables the above syntax
        Object.entries(this)
            .filter(([name]) => name[0] !== '_')
            .forEach(([method, func]) => {
                p[method] = (...args) =>
                    this.jimpPromise(resolve =>
                        p.then(image => resolve(this[method](...args)))
                    );
            });

		// Expose the normal EventEmiiter `on` function
        p.on = (...args) => this.on(...args);

        return p;
    };

[ ] change get buffer back to still return this

this would be a huge breaking change though. And it leads to weird states weird you want a sync jimp and would have to return promise in new places. I'm gonna tinker with this one a bit more.


Pin isort version to 4.x due to big changes in 5.x

isort 5.0 both makes backwards-incompatible changes to the Python API and stops sorting and de-duplicating imports across blocks. I created an isort-5 branch to just update to the new API, but then I realized the codegen output didn't look as nice. Maybe a minor issue, but you can decide which approach you prefer!

Codecov Report

Merging #6 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master       #6   +/-   ##
=======================================
  Coverage   95.28%   95.28%
=======================================
  Files           1        1
  Lines         382      382
=======================================
  Hits          364      364
  Misses         18       18

Continue to review full report at Codecov.

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

thanks for catching!


bug: handle and log throttled FetchResponses

Kafka 2.2 returns a throttled FetchResponse with an empty array of topics.
Previously Sarama treated this as an ErrIncompleteResponse and would
abandon and re-create the subscription instead of retrying the
FetchRequest.

@bai Since 2.0, Kafka can return an empty FetchResponse in case Quotas are violated (KIP-219).
Without this fix, when this happens, we get an error, for example:
kafka: error while consuming hub-crusher-6partition/0: kafka: response did not contain all the expected topic/partition blocks

Many thanks!

Should this in some way slow down FetchResponse requests? When we have a process that is behind in messages we've started to see a ton of FetchResponse entries in the log and eventually processing for a single consumer comes to a complete halt.

@gaffneyc yes a well-behaved client should backoff its request rate when throttled, but recent broker versions do also enforce this at the backend by sleeping on the socket

Okay, to make sure I understand you're saying the throttling is happening on the server side?
What we're seeing is a consumer coming to a complete deadlock and only a consumer rebalance will fix it. I'm guessing it would be better to open a new ticket.


Draft cleanup assets.

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
✔

Tests pass?
✔

Fixed issues
#34

Thanks!


[AutoGen Studio] Add instructions for running AutoGen Studio from docker

          What about removing this part for now? @victordibia or others can add it in another PR and solve the env var issue.

Originally posted by @sonichi in https://github.com/microsoft/autogen/pull/1111#discussion_r1444079371

Related #1241

I just submitted a pull. Updated the dockerfiles for base and full to have autogenstudio autostart.


Create build functionality

Should be a new page on which you have to fill in the following:

platforms (ios/android)
generate screenshots
submit to playstore

Should not allow multiple builds at the same time for the same app
Receive webhooks from jenkins to set the build status/build nr/download url etc
Move 'prepare iOS app' to 'ios-1-app'

Blocked until the jenkins server is updated


Feature Request: Overlay Text

Hi,
I'm not sure how easy it would be to implement, but it would be pretty cool to have the stats overlaid by specifying the position, size etc.
Much like how motion does with the time / date / text field.
Would this be something that is doable?
Great work by the way, saving my bandwidth!

Luke

What kind of overlay are you talking about? Overlay on the MJPEG stream? Like a JPEG overlay?

Yes, overlay text onto the MJPEG stream to display for example number of viewers or perhaps custom text?
If you see the current stream at  http://media.aiopi.com:8081/stream http://media.aiopi.com:8081/stream you will already see that motion overlays some text but it would be nice if this could be controlled by mjpeg-relay.

On 18 Apr 2015, at 13:41, Philippe Vaucher<EMAIL_ADDRESS>wrote:
What kind of overlay are you talking about? Overlay on the MJPEG stream? Like a JPEG overlay?
—
Reply to this email directly or view it on GitHub https://github.com/OliverF/mjpeg-relay/issues/7#issuecomment-94162988.

In my opinion this is a bad idea... this information is better shared as plain text next to the widget that displays the MJPEG stream. Creating an overlay directly on the JPEGs would also use lots of CPU, I'm not even sure it's feasible with more than 5 frames per seconds.
Anyway, we'll see what @OliverF will say about it.

No worries. It was just a thought ;)
I could perhaps generate a separate graphic in mjpeg-relay, i’m sure my python is capable enough to do this based on the existing status information.
Only reason for wanting it integrated was to be able to have the info in a standard  tag for example on forums etc.

On 18 Apr 2015, at 15:08, Philippe Vaucher<EMAIL_ADDRESS>wrote:
In my opinion this is a bad idea... this information is better shared as plain text next to the widget that displays the MJPEG stream. Creating an overlay directly on the JPEGs would also use lots of CPU, I'm not even sure it's feasible with more than 5 frames per seconds.
Anyway, we'll see what @OliverF https://github.com/OliverF will say about it.
—
Reply to this email directly or view it on GitHub https://github.com/OliverF/mjpeg-relay/issues/7#issuecomment-94167094.

Just display another div on top with the desired information.

Hi Luke,
I've considered this in the past but I believe the low latency of mjpeg-relay is achieved through minimal processing of the stream. As such, I think adding such a feature would increase the latency and, as Silex mentioned, may significantly increase the CPU usage. Perhaps if you were able to demonstrate this feature working without too much of a performance hit then I could incorporate it? If you do implement your own, drop me a message and I'll check it out for sure!
Thanks


Prevent Tab Press from popping to Initial Scene

Version
Tell us which versions you are using:

react-native-router-flux v3.38.0
react-native v0.42.0

I'm trying to familiarize myself with this library, and I have a sample app with two tabs: Tab1 and Tab2. There are two scenes nested in the Tab1 group: scene1 and scene2, where scene1 is the initial view. If I navigate to scene2, then switch tabs by pressing Tab2, then go back to Tab1, it automatically pops scene2 from Tab1 and goes back to scene1 instead of staying on scene2, which is not the behavior I want.
So my question: is it possible to can prevent the views popping to the root when I press the tab bar item? Ideally it would only pop to the root view when I'm already viewing a view within the same tab bar item. So in the example above, when switching from Tab2 to Tab1, when I press Tab1 the first time it stays on scene2, but if I press it again then it pops back to scene1.
Any help/suggestions greatly appreciated!!!
`
<Scene key="tabbar" tabs initial tabBarStyle={{ backgroundColor: '#f6f6f6' }}>

    <Scene key="dolphins_tab" title="Dolphins" icon={TabIcon}>
      <Scene key="aqua" component={AquaScene} initial title='Dolphins Aqua' />
      <Scene key="dolphins_orange" component={DolOrangeScene} title='Dolphines Orange' />
    </Scene>
  </Scene>

`

If anyone else is experiencing this issue, just tested it in on the previous version 3.37.0 and that fixed the issue (I was using 3.38.0. However, 3.37.0 has the issue mentioned on #717 where tapping a tab while you're viewing it doesn't pop back to default scene. So it looks like by attempting to resolve #717 in version 3.38.0 it caused the above issue. Would be sweet if next version could address both cases.

@adriandiaz Thank you!
Rolling back to 3.37.0 fixed problem with switching tabs popping back to the starting tab scene!
3.38.1 caused many duplicate dependency errors in the packager
3.38.0 Fixed duplicate dependency errors in the packager
    "react": "~15.4.1",
    "react-native": "0.42.3",
    "react-native-router-flux": "3.37.0",


Add tabular KY service data

Perhaps this will kick off our efforts to stand up the Ohana API!

+1


Updating filenames cxx

@ajpmaclean :
I have added the example filenames that needs to passed as arguments.
Is this fine to do like this where ever necessary in Cxx examples?

That's a really good idea. I have always mentioned the test files in the code, it gives the user something to try really quickly instead of hunting through the CMakeLists.txt file in the folder to find the parameters. I do a similar thing in Python. Generally if I convert an example to Python I'll update the C++ example, if it exists, at the same time. For Java and Python it is more important as there is no CMakeLists.txt file so the user needs hints as to what files to use, simple in Python because of argparse.

Sure. I'll update C++ examples wherever appropriate.
Thanks for your inputs.


Channels Not working at all

LaserIO version
1.3.1
Minecraft Version
1.18
Forge Version
No response
Modpack & Version
Direwolf20 - 1.7.0
Do you have optifine installed?
No
Describe the issue
Channels don't show color or work at all. Everything is treated as channel 0 at all times.
Steps to reproduce

Place two chests next to each other with 1 space between them.
Place a laser node in the middle of the 2 chests.
Use an item card on extract facing one chest.
Use an item card on insert BUT use a different channel on the other chest.
The items go from one chest to the other even though they are on different channels.

Expected behaviour
I expect that the item won't extract at all.
Screenshots
No response
Log files
No response
Additional information
When I shift mouse over the cards they say channel 0 even though I have changed them (by right clicking them from within the node interface). The icon itself don't change colors either.
The visual lasers connecting the chest to the node don't change colors as I would expect.
This was noticed when I tried to upgrade my auto-crafting from one furnace into 24. My setup ended up double smelting iron since one of the mods allows you to smelt iron ingots.
https://youtu.be/AMp0v8at3ns  video link

I'm stupid, the channel is the other button in the bottom left. Please add a title to differentiate the two since both just have the number display. Maybe "Channel #" instead for the bottom and "Redstone Channel #" for the top.


External configuration of what default context propagation means

Based on discussions around issue #27, e.g. what contexts should be propagated by default, there was a discussion about the proc/cons of having this "baked" into spec versus leaving it up to implementation.
While both sides had good arguments, the way to go could be to hard-code default into spec (whatever the default will be) and allow for external configuration via some file - yaml, xml - which would be able to override the defaults. Basically implementations would look for this will and follow what's in it, if there is no setup, spec defaults will be used.
During the mtg, someone also mentioned that we could use MP Config - but I am not really sure what was exactly meant by that in practice, feel free to expand on that bit.

I suggested that. I seem to recall that all MP specs have to use MP Config for configuration, instead of doing their own thing.

Just to respond to *could be to hard-code default into spec"
+1, the default should be in the spec

first off, I agree that the spec should define a default set of contexts.
As for configuration, MP Config is mainly useful when you want to control configuration that varies based on where the application is deployed (i.e. dev/test/staging/prod).  If configuration remains constant through all environments, annotation-based configuration is superior because it is type-safe and enforced by the compiler.
For example, it doesn't make sense to only want Security context in prod but not dev environments.
However, for some other configuration attributes in this spec, such as @ManagedExecutorConfig(maxAsync=5), I expect it will be common to want to reconfigure these values based on the environment.
Ultimately, I think it would be good to allow configuration through MP Config for all default instance values for consistency.  So something like:
org.eclipse.microprofile.concurrency.DefaultThreadContext.value=SECURITY,TRANSACTION
org.eclipse.microprofile.concurrency.DefaultManagedExecutor.maxAsync=5

The pattern being:
org.eclipse.microprofile.concurrency.Default<IFC_NAME>.<ATTR_NAME>=<VALUE>

Under #85, I wrote up the spec doc for integration with MP Config based on what we decided on the prior MP Concurrency call.  I tried to cover all of the scenarios I could think of.  @manovotn could you review?  I will also locate an MP Config expert to review as well, to be certain that we are following conventions appropriately.


should return success - Fail

exit code was 1.

Created by TestQuality - Linked to TestQuality.com defect D3. Do not remove or edit. D3


function - cite all packages

make a vector containing the name of all package needed to be loaded.
Use it to load them in a for loop
use it with write.bib(packages_vec, "assets/packages.bibtex"
concat "assets/bibliography.bibtex" with it into "references.bibtex"
make a pachages_ref.Rmd whith
---
nocite: |
  @package1, @package2
...

and child it in the last chunk
http://rmarkdown.rstudio.com/authoring_bibliographies_and_citations.html

done


Background tasks refreshing issues

Tasks progress in Background Tasks tab is not refreshed correctly. For example, when copying table data between databases I see task description like: Transfer data from TABLE1 into TABLE2 and Transfer table data: 393000 rows fetched. This progress information should refresh itself (it sometimes works), but often just stays the same until I do something like refresh table list, refresh table data or just run query. In other words - when task is added to Background Tasks list.
This seems to be issue with interface, because exporting works correctly (but with some minor issues).
I'm using version 5.3.0 CE.

can't reproduce it against the recent version.
The ticket will be closed. if the issue is still actual for you - feel free to reopen the ticket