Added some information about how multiple pull secrets are handled.
Issue
Closes: n/a

Shout out to https://medium.com/pareture/how-using-multiple-k8s-imagepullsecrets-works-77d2a2ca7a47

/approve


assetic:dump does not scan all themes

so i have configured my config :
liip_theme:
themes: ['AAA', 'BBB']
active_theme : 'BBB'
and i executed :
./app/console cache:clear --env=local
./app/console assetic:dump --env=local
but the dump just scan for BBB theme dir only
And AAA not scanned
i want the theme is dynamic, so the assetic:dump must scan all themes dir

are you using the latest version of the bundle ? there should be an assetic integration now

yes i did pull git the assetic bundle and liipTheme bundle

did you configure the bundle in question in the assetic configuration?

i did...
assetic
bundle:["MYBUNDLE"]

can you search for the getAssetic_AssetManagerService() method in your cached container in the cache dir and see what is generated? should help you debug the issue. thx.

ping?

ok. sorry....
in debug it's show all my bundle but the assetic:dump just "dump" my active_theme only

hmm i cannot reproduce the issue locally. the relevant class is TemplateResourcesPass:
https://github.com/liip/LiipThemeBundle/blob/master/DependencyInjection/Compiler/TemplateResourcesPass.php
as you can see i am using the parameter to determine the list of themes to iterate over.

i can confirm this problem in the generated cache file all paths are loaded as Resource.
You can test this when you create a layout file with a asset and a theme with the same file with another asset. Now it generate only routes or assets (assetic:dump) for the default theme (Resource/view), but not for all themes.
After some debugging i think the problem is that the TemplateLocator does not know where the resource template file is it gets lost in the FileResource.
/path/to/vendor/bundles/Symfony/Bundle/AsseticBundle/Factory/Resource/FileResource.php
has two properties path and bundle
string(116) "/path/to/vendor/bundles/Gf/MyBundle/Resources/themes/phone/layout_mo.html.twig"
string(16) "GfMyBundle"

getTamplate() gives createTemplateReference()
string(16) "GfMyBundle"
string(19) "layout_mo.html.twig"

so createTemplateReference() create the following TemplateReference
object(Symfony\Bundle\FrameworkBundle\Templating\TemplateReference)#389 (1) {
  ["parameters":protected]=>
  array(5) {
    ["bundle"]=>
    string(16) "GfMyBundle"
    ["controller"]=>
    string(0) ""
    ["name"]=>
    string(9) "layout_mo"
    ["format"]=>
    string(4) "html"
    ["engine"]=>
    string(4) "twig"
  }
}

so this is a limitation in assetic bundle?

after having this problem i found a workaround. i had to move my themes to the "views"-folder and changed the configuration to use these folders. now assetic is loading all theme-styles.
# src/Acme/LayoutBundle/Resources/views/themes/default/layout.html.twig
# src/Acme/LayoutBundle/Resources/views/themes/mobile/layout.html.twig
# src/Acme/LayoutBundle/Resources/views/themes/tablet/layout.html.twig

#
liip_theme:
    themes: ['default', 'tablet', 'mobile']
    active_theme: 'default'
    load_controllers: false
    path_patterns:
        app_resource:
            - %%app_path%%/views/themes/%%current_theme%%/%%template%%
            - %%app_path%%/views/%%template%%
        bundle_resource:
            - %%bundle_path%%/Resources/views/themes/%%current_theme%%/%%template%%
        bundle_resource_dir:
            - %%dir%%/views/themes/%%current_theme%%/%%bundle_name%%/%%template%%
            - %%dir%%/views/%%bundle_name%%/%%override_path%%

before these changes only the last theme "mobile" was rendered - the others were ignored.

Created a PR which seems to fix this: https://github.com/liip/LiipThemeBundle/pull/118

#118 is merged. can we close this issue now?

I did test the assetic:dump on master branch and it seems not to be fixed, I quickly added two new themes with assets but only the active theme was generated also I needed to remove the cache manually(rm app/cache/*) with app/console cache:clear the active theme was not deleted from cache.
the config:
liip_theme:
    themes: ["default", "testtheme", "testtheme2"]
    active_theme: "default"
    load_controllers: false
assetic:
    use_controller: false

@alexander-schranz you need to explicitly enable assetic integration, check the README file.

sorry forget the
    assetic_integration: true

it works awesome @dantleech

Great! Good work dan!


chore: import repository https://github.com/Sathwik-Anumala/jenkinsx-http-gradle.git

this commit will trigger a pipeline to generate the CI/CD configuration which will create a second commit on this Pull Request before it auto merges

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign
You can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@Sathwik-Anumala: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

verify
11a804a3c6893da98877249badb854e8e7ced7e6
link
/test verify

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.


feat(side-drawer): add end and start positions

@yinonov
Modal and dismissible side-drawers only work on full pages, could you clarify which example you are looking at?

check modal and dismissible

@yinonov
Modal and dismissible side-drawers only work on full pages, could you clarify which example you are looking at?

check modal and dismissible

@yinonov
Modal and dismissible side-drawers only work on full pages, could you clarify which example you are looking at?

Align to end should also work on modal / dismissible


Update version on README

Small changes to the README, update to the latest version of the client.

Stale. Closing.


[SDP-1224] Improve how we handle erros on kafka event producers

What
Improve how we handle erros on kafka event producers
Why
Address https://stellarorg.atlassian.net/browse/SDP-1224
Checklist
PR Structure

[ ] This PR has a reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR title and description are clear enough for anyone to review it.
[ ] This PR does not mix refactoring changes with feature changes (split into two PRs otherwise).

Thoroughness

[ ] This PR adds tests for the new functionality or fixes.
[ ] This PR contains the link to the Jira ticket it addresses.

Configs and Secrets

[ ] No new CONFIG variables are required -OR- the new required ones were added to the helmchart's values.yaml file.
[ ] No new CONFIG variables are required -OR- the new required ones were added to the deployments (pr-preview, dev, demo, prd).
[ ] No new SECRETS variables are required -OR- the new required ones were mentioned in the helmchart's values.yaml file.
[ ] No new SECRETS variables are required -OR- the new required ones were added to the deployments (pr-preview secrets, dev secrets, demo secrets, prd secrets).

Release

[ ] This is not a breaking change.
[ ] This is ready for production.. If your PR is not ready for production, please consider opening additional complementary PRs using this one as the base. Only merge this into develop or main after it's ready for production!

Deployment

[ ] Does the deployment work after merging?

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr304.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr304.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr304.previews.kube001.services.stellar-ops.com

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr304.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr304.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr304.previews.kube001.services.stellar-ops.com

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr304.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr304.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr304.previews.kube001.services.stellar-ops.com

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr304.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr304.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr304.previews.kube001.services.stellar-ops.com

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr304.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr304.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr304.previews.kube001.services.stellar-ops.com


Unhandled errors and excessive usage of "console" causes performance issues

These errors and logs are thrown on every save or file edit:
In both modes:

These happen in Nodejs mode only:

Thank you for letting me aware of this problem.
About the first:
I would like to fix it soon.
About the second:
"Node.js mode" was introduced experimentally for compatibility with atom-typescript. However, since the support of the workspace folder has been added in Deno1.10, it does not have this feature is no longer needed.
In the next release, this package will correspond to the workspace folder and will remove this feature. I'm working on it.


Fixed BATCH checkpoint mode issue and updated default vlaue

…s BATCH; set the default value of the checkpoint mode to RECORD
Description
A few sentences describing the overall goals of the pull request's commits.
Todos

[ ] Tests
[ ] Documentation

Codecov Report

Merging #728 into 1.1.x will decrease coverage by 0.02%.
The diff coverage is 50.00%.

@@             Coverage Diff              @@
##              1.1.x     #728      +/-   ##
============================================
- Coverage     56.19%   56.17%   -0.03%
  Complexity      754      754
============================================
  Files           181      181
  Lines          3189     3192       +3
  Branches        233      234       +1
============================================
+ Hits           1792     1793       +1
- Misses         1262     1264       +2
  Partials        135      135

Impacted Files
Coverage Δ
Complexity Δ

...g/integration/eventhub/impl/EventHubProcessor.java
35.48% <0.00%> (-2.45%)
3.00 <0.00> (ø)

...ration/eventhub/impl/AbstractEventHubTemplate.java
86.53% <100.00%> (+0.26%)
20.00 <0.00> (ø)

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 284ef83...9924903. Read the comment docs.


Question about Import the MIGraphX module in Python script for a custom install location

When the AMDMIGraphX library is not installed in the default location (e.g /opt/rocm), do users need to create .deb packages first to import the MIGraphX module in Python script ?

Hi @jinz2014 Thanks for reaching out, and sorry for the lack of responses. To answer your question, no a .deb package is not required.  Even if AMDMIGraphX is not installed in the default location, as long as the shared libraries are included in your LD_LIBRARY_PATH and PYTHONPATH points to the directory containing cpython bindings correctly, it should work.
To elaborate, let's say migraphx is installed through apt, then, under /opt/rocm/lib , you should find a cpython binding library migraphx.cpython-38-x86_64-linux-gnu.so and a directory called migraphx that contains the include and the lib subdirectories.
If you install migraphx to a custom directory /my/path, then you should find still find the cpython bindings at /my/path/migraphx.cpython-38-x86_64-linux-gnu.so.  You want to set the your PYTHONPATH to include this binding with
export PYTHONPATH=$PYTHONPATH:/my/path

Next, let's say you have the libraries that that would normally be under /opt/rocm/lib/migraphx/lib installed at /my/path/libs, then you would want to update your LD_LIBARARY_PATH accrodingly with
export LD_LIBRARY_PATH=$LD_LIBRARY_PATH:/my/path/lib

Finally, make sure you have the correct version of cpython bindings for your python verison. For example, migraphx.cpython-38-x86_64-linux-gnu.so will only work with python 3.8.
Hope this helps!

Thank you for your answers!


Fix adui 4535 connect react vapor input to selector

Proposed Changes
then again trying to do just right by using the components selectors
Potential Breaking Changes
none
Acceptance Criteria

[ ] The proposed changes are covered by unit tests
[ ] The potential breaking changes are clearly identified
[ ] README.md is adjusted to reflect the proposed changes (if relevant)

Codecov Report

Merging #1280 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1280      +/-   ##
==========================================
- Coverage   99.45%   99.45%   -0.01%
==========================================
  Files         358      358
  Lines        8978     8975       -3
  Branches     2130     2128       -2
==========================================
- Hits         8929     8926       -3
  Misses         49       49

Impacted Files
Coverage Δ

...react-vapor/src/components/input/InputSelectors.ts
100% <ø> (ø)
:arrow_up:

...apor/src/components/radio/RadioSelectConnected.tsx
100% <100%> (ø)
:arrow_up:

...vapor/src/components/radio/RadioSelectSelectors.ts
100% <100%> (ø)
:arrow_up:

...eact-vapor/src/components/input/InputConnected.tsx
100% <100%> (ø)
:arrow_up:

...or/src/components/filterBox/FilterBoxConnected.tsx
100% <100%> (ø)
:arrow_up:

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 efd3032...5e75c21. Read the comment docs.


PostOrder Traversal C++

Thanks for contributing to NeoAlgo!
Please remove this before submitting your PR.
Don't forget to visit our website!
All Submissions:
Before starting:
PLEASE PLEASE PLEASE READ AND CHECK THE BOXES BELOW

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked the issues to see if your request isn't assigned to someone else?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[ ] If your submission is to improve an existing code, have you created an issue to specify that?
[x] Don't tag anyone unless specifically requested to do so
[x] Don't create any subfolders unless specifically requested to do so.

New Code Submissions:

[x] Does your code work locally?
[x] Have you cleaned your code for better readability? (no extra unnecessary lines, etc.)
[x] Have you updated the README.md file to add the link to your code?

Hey bro, you could've submitted both (#197 and #198) under the same PR :)


MIFARE ULTRALIGHT CARD NOT DETECTED

Hello,
I musing MFRC522 with Arduino Mega, I m able to detect MIFARE classic cards and tags but unable to detect the MIFARE ULTRALIGHT CARD and TAGs. Will you please guide me on this. ????? I m able to detect the Ultralight Card from my cell phone

Good evening.
I also had this problem. I found one way, instead 3.3volts I connected to 5 volts. its work for me. you can check also. As I remember, the module is tolerant to 5 volts. anyway my module works and reads all cards like Mifare: 1k, mini, ultralight etc..

but it doesn't support ULTRALIGHT C 3DES Authentication

Only unencrypted communication is supported by this library.
DES is not supported by hardware, it has to be implemented in software but nobody done yet.
It would be nice if you implement and make a pull request :)


with add more v8 metrics

Describe what enhancement you'd like to have
we can add a function like, and post in the checkHeapLimit with a new poco logger "V8" print as info level log is considered.
std::string getHeapStatisticsString(v8::HeapStatistics& heap_statistics) {
    return fmt::format(
        "Total Heap Size: {}\t"
        "Total Heap Size Executable: {}\t"
        "Total Physical Size: {}\t"
        "Total Available Size: {}\t"
        "Used Heap Size: {}\t"
        "Heap Size Limit: {}\t"
        "Malloced Memory: {}\t"
        "External Memory: {}\t"
        "Peak Malloced Memory: {}\t"
        "Does Zap Garbage: {}\t"
        "Number Of Native Contexts: {}\t"
        "Number Of Detached Contexts: {}\t"
        "Total Global Handles Size: {}\t"
        "Used Global Handles Size: {}",
        heap_statistics.total_heap_size(),
        heap_statistics.total_heap_size_executable(),
        heap_statistics.total_physical_size(),
        heap_statistics.total_available_size(),
        heap_statistics.used_heap_size(),
        heap_statistics.heap_size_limit(),
        heap_statistics.malloced_memory(),
        heap_statistics.external_memory(),
        heap_statistics.peak_malloced_memory(),
        heap_statistics.does_zap_garbage(),
        heap_statistics.number_of_native_contexts(),
        heap_statistics.number_of_detached_contexts(),
        heap_statistics.total_global_handles_size(),
        heap_statistics.used_global_handles_size()
    );
}

this check happens in the new group, if use info level log is flooded, we can move to :
if hits the soft limit, then throw an exception, in the exception, we check the v8::heapstat as the previous function demo. in this way , we no need consider poco::logger


[Feature]: Able to search Inventory

Describe the feature
With #453 Implemented, it should be possible to search the Inventory or Inventories.
This would make it much easier to search for specific items over different Characters
Describe the implementation

grey out everything in the current Bag, which not has the current search as Substring inside them (question is: only Name or also description?)
pop up another little window which contains a TextField on the Upper part and a List of all Items on the lower one, Then start listing all items with the substring inside them with the third character (so not all items of all characters needed to be shown)

Duplicate of #26


Kucerar/3153 abstract role request

https://redmine.czechidm.com/issues/3153

Issues were fixed. Some unresolved FE things were fixed in other branches, LGTM


Bump cmake minimum version. Support pre-3.28 in FindVulkan.

Description
Bumps the minimum cmake version for CMakeLists.txt and app/CMakeLists.txt consistent with the CMake features used. In the case of CMakeLists.txt the version is bumped to 3.20 for consistency with app and to support use of cmake_path which is used in the accompanying fix to FindVulkan.cmake to support versions earlier than 3.28.
Fixes #1100
General Checklist:
Please ensure the following points are checked:

[x] My code follows the coding style
[x] I have reviewed file licenses
[ ] I have commented any added functions (in line with Doxygen)
[x] I have commented any code that could be hard to understand
[ ] My changes do not add any new compiler warnings
[ ] My changes do not add any new validation layer errors or warnings
[ ] I have used existing framework/helper functions where possible
[x] My changes do not add any regressions
[ ] I have tested every sample to ensure everything runs correctly
[x] This PR describes the scope and expected impact of the changes I am making

I have left unchecked items which are not relevant to this change.
Note: The Samples CI runs a number of checks including:

[x] I have updated the header Copyright to reflect the current year (CI build will fail if Copyright is out of date)
[ ] My changes build on Windows, Linux, macOS and Android. Otherwise I have documented any exceptions

If this PR contains framework changes:

[ ] I did a full batch run using the batch command line argument to make sure all samples still work properly

Sample Checklist
If your PR contains a new or modified sample, these further checks must be carried out in addition to the General Checklist:

[ ] I have tested the sample on at least one compliant Vulkan implementation
[ ] If the sample is vendor-specific, I have tagged it appropriately
[ ] I have stated on what implementation the sample has been tested so that others can test on different implementations and platforms
[ ] Any dependent assets have been merged and published in downstream modules
[ ] For new samples, I have added a paragraph with a summary to the appropriate chapter in the readme of the folder that the sample belongs to e.g. api samples readme
[ ] For new samples, I have added a tutorial README.md file to guide users through what they need to know to implement code using this feature. For example, see conditional_rendering
[ ] For new samples, I have added a link to the Antora navigation so that the sample will be listed at the Vulkan documentation site

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

Just spotted a completely unnecessary unset in my changes which I have removed in https://github.com/KhronosGroup/Vulkan-Samples/pull/1101/commits/11f46534cf4cf8e8f18017e62e4861c09e18c09d. Very sorry this is post your review @SaschaWillems thus invalidating it.

Hi @MarkCallow , we discussed this MR on the Samples call today and have reservations about merging this w/o some changes.  There appears to be 2 main items covered here: 1) bumping the cmake version number; 2) changes related to FindVulkan.  The first item seems to only be required for iOS builds - but it is likely to cause problems for other platforms (especially Android) where folks may have good reasons to stay with the older version.  We suggest splitting this change out into a separate PR and scoping it to only iOS platforms.  We aren't quite sure about the FindVulkan changes - but we think it makes sense to sync up with LunarG on this - they have plans for future versions that might impact our directions here.  @gpx1000 is the most familiar with this on the team and has offered to help - Steve, please chime in as needed.  Thanks!

How to scope to only iOS platforms? As far as I know cmake_minimum_required must appear before the project command and I know that the IOS variable is only set after the project() command. Perhaps an explicit check of the version after project, if(IOS)with a message(FATAL_ERROR ...). Suggestions welcome.
What is the minimum CMake version for which support is desired? There is no question that the XCODE_EMBED items in app/CMakeLists.txtrequire CMake 3.20. My question above relates to this file. For CMakeLists.txt, the workaround in FindVulkan.cmake for versions earlier than 3.28 can be modified to use the deprecated get_filename_component instead of cmake_path thus removing the need to up its minimum version to 3.20.

I think you're right about the idea of if(IOS and version less than 3.20 in findVulkan.cmake with an assert saying minimum cmake of 3.20 is required is best.  That way we're keeping the check to the file we're recommending people copy from.  It also keeps the project wide changes to where they currently are for all other platforms.  This reduces the amount of testing we need to do.

I intend to sync with LunarG but currently FindVulkan is not part of the Vulkan SDK. @gpx1000's blog on developing an iOS app with the Vulkan SDK says to copy the file from here. Thus the problems need to be fixed here now not at some future time. The fixes I have provided do not break any other usage. The code to ensure CMAKE_FIND_FRAMEWORK is FIRST within FindVulkan does not affect the value outside FindVulkan. The other fixes are vanilla bug fixes.
I see no reason to split this into 2 PRs.

FindVulkan.cmake should hopefully never be included in the SDK.  FindVulkan.cmake is included with cmake and that is woefully out of date; and cmake is trying to remove it.  Ideally, we'd transition to vulkanconfig.cmake and include that with the SDK so it's not a project only solution.  However, I think FindVulkan.cmake in the current iteration should be thought of as a template.
I think if you do the path I mentioned above of leaving the version bump off and inlcude an assert in FindVulkan.cmake when it's being used by cmake less than 3.20, then one PR makes sense.  Otherwise, the version bump would benefit from being in a separate PR so we can make sure it works in all platforms by itself as that is a fairly big change.


This adds support for tenancy aware Storage::url() method

This fixes #78 and #201 and enables using Storage::url('file.jpg') with local driver in tenants (see Laravel docs).
With this, you can use for example Spatie media library and local disk without any modifications.
You only need to create symlinks in the public directory which points to the tenants' disks root.
For creating symlinks you can use the new command tenants:link

Thanks a lot for the PR! The code looks very high quality. I'll review & merge this in a few weeks when I start work on v4 and will be making all changes at once. Thanks again!

Thanks :-) Tomorrow I'll try to write a better description of all changes.

@stancl any plans of merging this? idk but seems like tests failed

@stancl any plans of merging this? IDK but seems like tests failed

Hi, tests failed not because of code, but because of an error in building one of the docker containers in workflow. Otherwise, all tests should pass.
You can test it yourself by cloning this pull request (e.g. to Gitpod) and run provided docker (see CONTRIBUTING.md)
If you want to use this in v3 you can extract my changes to custom TenancyBootstrapper.
I would help you more, but right now I'm at the hospital recovering from an accident and I'm limited to my phone screen.

@dontfreakout first of all thank you for commenting and thank you for the pr as well. Yes I'm thinking to use your pr code to my custom tenancy boostrapper And I hope @stancl will merge this to currentl v3 or upcoming v4 🤗

@aneeskhan47 you are very welcome. It should be pretty straightforward. This PR is extracted (with some modifications) from the bootstrapper I'm using in v3.
If all goes well, next week I'll be back home and can create a gist, if it helps.

@dontfreakout i have extracted your pr code to mine. i have added my own FilesystemTenancyBootstrapper (with your code) + the link command to my App\Console\Commands directory and modified the tenancy config to use my FilesystemTenancyBootstrapper plus added the url_override option. i also ran the tenant:link command and it succesfully linked, but now the global_asset helper got kinda broken:

@dontfreakout also the link command didn't add the folder,files of spatie media library from to app/storage to public/storage (tenant symlink)

@dontfreakout actually the reason of the global asset broke is that in tenancy original TenancyServiceProvider.php in /src the globalUrl was bounding to the Stancl\Tenancy\Bootstrappers\FilesystemTenancyBootstrapper::class i have copied that and added to my TenancyServiceProvider in  app/providers boot method and now it works :).
for the time being i have copied your PR code and i hope this PR to get merged @stancl so then i can use the package normally :)
Thanks @dontfreakout and @stancl

@aneeskhan47 did you disabled original FilesystemBootrapper in the config?
I can't do much right now. The phone screen is not very comfortable for anything about code 😒
About the symlinks... Can you check if created tenant dir in the public dir is a symlink or just a plain directory?

@aneeskhan47 you are faster than me 🙂
Glad you got it working.

Couldn't find the button to re-run the build, so I pushed a small change. The Docker issue was probably a random thing that'll be fixed by re-running the build.
Thanks for the help here @dontfreakout

Ok, so @aneeskhan47 were right about the bug in the link command (at least for L6).
For the link command, I'm extending Laravel StorageLinkCommand, but L6 can create links only from "public/storage" to "storage/app/public".
I'll fix this once I get to my PC (next week, I hope).

Ok, so @aneeskhan47 were right about the bug in the link command (at least for L6).
For the link command, I'm extending Laravel StorageLinkCommand, but L6 can create links only from "public/storage" to "storage/app/public".
I'll fix this once I get to my PC (next week, I hope).

@dontfreakout Hi, i hope you are doing well. when you will make the fix ? if you are unable for some reason tell me the changes and i'll add them 😄

@aneeskhan47  Hi, I'm back home after almost three weeks in hospital, so I'm doing definitely better. Thanks for asking :-)
The fix is committed. I've also added a few more features: Events, RemoveSymlink Job. And added Jobs to TenancyServiceProvider stub (to create and remove symlink when created/deleted Tenancy).
@stancl to manually trigger a workflow, the workflow must be configured to run on the workflow_dispatch, see the docs.

This adds a lot of new stuff, so I'll add it in v4, but I'll also have to make some changes when I'll be merging this. As I mentioned in #654, Laravel 9 uses flysystem 2, which forces me to rewrite how the FilesystemTenancyBootstrapper works.
So once I have the v4 filesystem bootstrapper done, I'll update the code of this and merge 👍🏻

Sorry, if i upgrade package to v3.5 can i use php artisan tenants:link ?

It will be available in v4.

FWIW the filesystem logic (that uses adapters) will need to be adjusted after #802 is merged

Add this code to your AppServiceProvider.php and add storage/tenant-xxx directory to the public/public-xxx
\Illuminate\Support\Facades\URL::macro('tenantFile', function($file) {
    $prefix =str_replace(
        '%tenant_id%',
        tenant()->getKey(),
        config('tenancy.filesystem.url_override.public', 'public-%tenant_id%')
    );

    return url($prefix.'/'.$file);
});

Seems like I didn't resolve the conflicts correctly. Adding a note to add back some of the removed logic from here https://github.com/archtechx/tenancy/pull/689/commits/accbf5b3afacd48619f8b497716bf60026901c1e#diff-a7131a9582938c1b642f3d517a9064cbacb7b83d75fb52fe2c6e52dbca26a584

The conflict resolution is a bit of a mess now. I'm thinking about forcibly reverting to the point before I merged master into this branch, and you'd try to resolve the conflicts there from scratch. Now it's a mix of my changes with your changes, and your changes still need to be modified (eg the review above). And there are more conflicts to resolve in other files.
Perhaps you could create a branch based on 5ed5aea6d602c84f14d646f242948ac5dcad7574 and try to resolve the conflicts there.
I'd like to avoid having excessive back & forth about the conflict resolution in this specific thread.

I'll close this now in favor of #909


Use Path:: methods instead of Filesystem instance

Filesystem->isAbsolutePath does not check if the file exists and just make quick checks on the string.
So here the Path utility methods seem a better choice as they are platform-agnostic and allow safe operations like ::join  handle ../../ in paths and handle full path canonicalization.
And the tests can be done on any strings, independently on the Fixtures files here.
I think this method is used before calling new File(), so maybe the whole thing could be replaced by some factory/provider returning an Asset or something like that?
I've not touched the constructor (BC) or the DI for now, depending on your feedback & vision on this.

I've not touched the constructor (BC) or the DI for now, depending on your feedback & vision on this.

It's a BC break but the only way to be affected is to redefine the 1st argument in a CompilerPass. I'm not sure that someone already did that. And this bundle is experimental :angel: .
BTW, this service should be marked as internal to encourage decoration.
For me, we can update this service but we can wait for other maintainers opinion.

I think this package should... not be kept like this.
Right now it's not much more than a static helper.


WandB callback changes the train step's unique ID, but does not change the results

Actually, callbacks can change the result. So we'll close this.


Looks like Mint cookies have changed

I just re-logged into Mint to reset my cookies, and I do not see accounts.intuit.com or pf.intuit.com in Cookies at all.
And no ius_session or thx_guid cookies anywhere. (The only 'session' type cookie I see is MINTJSESSIONID from mint.intuit.com)
Is it just me? Or does this API need rewriting now?
I hope that @mrooney has time to look into this!

Unfortunately I don't have time to look into this :/ There are other issues for the recent changes, see #118 among other recent ones.


Error on Seeders

runing php artisan migrate:refresh --seed
I get
SQLSTATE[22001]: String data, right truncated: 1406 Data too long for column 'description' at row 1 (SQL: insert into example (name, description, status_id) values (Velit architecto voluptates quam et., Qui voluptatibus possimus tenetur et sapiente esse distinctio. Alias est est ipsam dicta praesentium. Ducimus perspiciatis aut enim nobis suscipit enim ducimus. Consequatur ipsa sit nulla exercitationem reiciendis qui placeat. Voluptatum consequatur molestias ex voluptatem., 4))
I've fixed it changing
\database\seeds\ExampleSeeder.php
'description'       => $faker->paragraph(3,true),
to
'description'       => $faker->paragraph(2,true),
Thanks!

1406 Data too long for column 'description'

That error is telling you that you're trying to put too much data into a column.
For example, if the data type for the description column is a varchar (which has max length of 255) and you are trying to put 300 characters in that column, it will throw that error.
What is the column/data type for the description column?

Yes!
Seeder is Generating 3 paragraph. The resoult is bigger than descriptin sice (I don´t remember your field definition) You should modify Seeder to fill the field with a smaller faker, like 2 paragraph ;-)
Greets!

Make it to paragraph(2,true) for testing


libcec: 4.0.7 -> 6.0.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/Pulse-Eight/libcec/releases.
meta.description for libcec is: "Allows you (with the right hardware) to control your device with your TV remote control using existing HDMI cabling"
meta.homepage for libcec is: "http://libcec.pulse-eight.com"
meta.changelog for libcec is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 6.0.2 with grep in /nix/store/k1r340n9wxgr9cxgfvxxgj73vf7qka84-libcec-6.0.2

directory tree listing: https://gist.github.com/d7838add5712a7021e22215e5af3ecf8

du listing: https://gist.github.com/bccb17559d1e77dd8a36219f94368d32

Rebuild report (if merged into master) (click to expand)

60 total rebuild path(s)

18 package rebuild(s)

18 x86_64-linux rebuild(s)
17 i686-linux rebuild(s)
8 x86_64-darwin rebuild(s)
17 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
kodi
kodi-gbm
kodi-wayland
kodiGBM
kodiPlain
kodiPlainWayland
kodiPlugins.inputstream-adaptive
kodiPlugins.joystick
kodiPlugins.kodi
kodiPlugins.kodi-platform
kodiPlugins.pvr-hdhomerun
kodiPlugins.pvr-hts
kodiPlugins.pvr-iptvsimple
kodiPlugins.steam-controller
kodiPlugins.vfs-libarchive
kodiPlugins.vfs-sftp
libcec
moonlight-embedded

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/k1r340n9wxgr9cxgfvxxgj73vf7qka84-libcec-6.0.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A libcec https://github.com/r-ryantm/nixpkgs/archive/966cc8e8ab744a2b9dfd0a0931c86f74a77c7740.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/k1r340n9wxgr9cxgfvxxgj73vf7qka84-libcec-6.0.2
ls -la /nix/store/k1r340n9wxgr9cxgfvxxgj73vf7qka84-libcec-6.0.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review 1

17 packages built:

kodi (xbmc)
kodi-gbm
kodi-wayland
kodiGBM
kodiPlain (xbmcPlain)
kodiPlainWayland
kodiPlugins.inputstream-adaptive
kodiPlugins.joystick
kodiPlugins.kodi-platform
kodiPlugins.pvr-hdhomerun
kodiPlugins.pvr-hts
kodiPlugins.pvr-iptvsimple
kodiPlugins.steam-controller
kodiPlugins.vfs-libarchive
kodiPlugins.vfs-sftp
libcec
moonlight-embedded

Maintainer pings
cc @titanous for testing.

kodi 19.x has been merged into unstable so in theory this PR is good to go now...


Create Whitelist contract inside access

Branch new/whitelist
This is a stripped down version of RBAC.sol, with just one Role.
An Ownable contract that keeps a mapping of address => bool
Methods:
isMember: returns mapping(address)
addMember: sets mapping(address) = true
removeMember: delete mapping(address)
There is no listing of members, that will be done in a separate issue and contract.

https://github.com/HQ20/contracts/pull/58

I've noticed this is a copy of OpenZeppelin's Roles.sol. I need to work on it a little bit to see how we reference their work.


multi-plugin support

This branch adds support for the multi-plugin stuff in https://github.com/google/xi-editor/pull/255.
Plugins are listed in a dropdown in the Debug menu:

And running multiple plugins is now fairly easy (here is syntect + the very simple bracket closing sample plugin)

note: to run the python plugins, you will need to install the plugin lib: from xi-editor/python/xi_plugins run pip3 install ..
also: you may run into problems with Xcode resetting environment variables. To set environment variables in Xcode, option-click on the Run button to open the scheme editor, and set variables there:

Not related to this PR but the gif you posted:
Something is terribly wrong. In the gif there are times when the character you type isn't highlighted instantly (when you are typing the comment). There's no way that should ever happen. Anyone have any idea why highlighting isn't completing properly within the space of one frame? Because that's like worse-than-Atom level slowness.

A couple of observations from playing with it. First, it doesn't work with master xi-mac, it looks like the protocol has changed incompatibly.
Second, the syntect plugin looks like it needs an update to track the plugin protocol changes:
plugin response json err: ErrorImpl { code: Message("data did not match any variant of untagged enum UpdateResponse"), line: 0, column: 0 }

This may actually have been previous plugin changes (to add an edit response), not this PR. Also, it works, just prints these errors. In any case, I definitely want to track it.

Above comment was meant to be posted to xi-editor, and is now https://github.com/google/xi-editor/pull/255#issuecomment-299745751 , as these are more core than FE concerns. However, leaving it, as there is some overlap.

Really cool to see this.


Feature: Can we create forms using Lexical Plugins?

Is it possible to create forms similar to https://tally.so/ using lexical custom plugins?

Not a relevant place to ask this type of question..

Not a relevant place to ask this type of question..


Delete snsKeyPublic.snk

Hi @Tersembunyi1990, if you believe this change is appropriate, we encourage you to file an issue in the repo describing the rationale. We aren't able to accept this contribution as an external PR at this time.


Support sending raw transactions via the GSN provider

Support sending raw transactions via the GSN provider. We probably need to catch eth_sendRawTransaction besides eth_sendTransaction.
Requested by the CryptoManiacs team. Make sure to reach out to them after this issue is closed.

We're still waiting for their reply after Dror's comment. It just doesn't make sense and we're trying to figure out what they are trying to do with eth_sendRawTransaction.

We may however want to catch eth_sendRawTransaction and error out if _withGSN is true, since this is probably a developer error.

error flagging seems the right way to go. Using sendSigned/sendRaw probably means a user that didn't understand the system..

@nventuro yep we just deserialize tx from eth_sendRawtransaction to put it into eth_sentTrasnsaction flow, which is working. But it seems you could handle signed transaction more gently.

@nventuro yep we just deserialize tx from eth_sendRawtransaction to put it into eth_sentTrasnsaction flow, which is working. But it seems you could handle signed transaction more gently.

it is much better to reject this transaction (so the user knows he signed the wrong thing, and has to figure out how to do sendTransaction with the correct credentials and "from" field,)
Silently trying to "fix" the mistake and handle this transaction can lead to unexpected results: wrong from field, strange gas usage, value ignored, etc

@drortirosh what do you mean "he signed the wrong thing"? You not got what I am talking about. Clients of your library - developers can decide to sign transaction on their own. You can get this signed transaction and forward to your nodes only. Whom from what are you going to protect?

Your usual flow:

Web3 submits unsigned Tx
GSN provider gets Tx and forwards it to original provider for signing
Users signs Tx with MetaMask or other wallet
GSN provider forwards signed Tx to GSN nodes

@k06a I believe what @drortirosh is trying to convey is that what the user signs when using the GSN is not an Ethereum transaction, but a relay request, which are slightly different. Among those differences, the address of the relayer is one of the signed values, hence the need for the provider to receive the raw values, and only ask the user for a signature once it has found a working relayer.
The required information (to, data, gasPrice, gasLimit`) can of course be extracted from a signed transaction, discarding value and the signature itself, but this poses two issues:

the user had two sign two values: the discarded Ethereum transaction and the GSN relay request
the value field of the transaction will be discarded

It's because of these reasons that we think it's better to raise an error when an attempt to perform a relay request with already signed data is made.

@nventuro got it, that's makes sense.


Provide a dedicated service for server request error response generation

Provide a dedicated service for the server request error response generator
Instead of re-using the existing ErrorResponseGenerator, this patch does the following:

Separates functionality for generating the error response into a trait, Zend\Expressive\Response\ErrorResponseGeneratorTrait. The method prepareTemplatedResponse() now expects the error, the response, and an array of data to provide to the template.
Updates Zend\Expressive\Middleware\ErrorResponseGenerator to use the new trait.
Creates a new class, Zend\Expressive\Response\ServerRequestErrorResponseGenerator, which expects a response factory, and optionally the debug flag, renderer, and template to render to. It composes the trait, and generates a response on-the-fly to pass to the trait methods.

The RequestHandlerRunnerFactory has been updated to use this new service, and the ServerRequestErrorResponseGeneratorFactory was updated to generate an instance of the new class.
These changes mean the constant SERVER_REQUEST_ERROR_RESPONSE_GENERATOR is no longer needed, and was thus removed.

@webimpress Please re-review; this removes the constant, and provides a dedicated service.


Support another status item background color to indicate "warning" state

We currently allow extensions to set a background color for status bar items limited to statusBarItem.errorBackground. This is a very strong red indicator that should be used for error cases.
To give extensions a way of indicating warning situations, we should introduce another color that is less strong. Examples include a indication that some feature is not enabled and thus the experience may be less than ideal (though not an error case).
Some ideas:

The former would be a new color for warning, the latter would be exposing our existing prominent color to extensions.
//cc @egamma

@luabud do you think it makes sense to use this for when no Python interpreter is selected?

@brettcannon Great question. I'd love to get @misolori's opinion on it, as I'm not sure how that could look like when it comes to different themes.
The idea would be to change the background color for when the following is displayed on the status bar:


Let Composer load the Test\Fhp namespace too

This lets us get rid of TestInit.php and also fixes the unit tests in IDEs which assume pure Composer loading.

Verified on Travis that it still executes the 69 tests and doesn't just skip.


Dump database schema to file for sqlc

sqlc needs a schema to reference in order to produce the correct types during codegen.
After the initial goose migration, dump the database schema and load it into the sql folder to pass to the sqlc config.

Blocks #5.


chore: add NCT commitments for smart contracts team

@3esmit @gravityblast I'd like you to review these as well, just to make sure we don't over commit.
Do you guys have any doubts on whether we'll be able to get these things shipped by EOY? Also considering that in November part of the team is at Devcon


Deleting a (last?) profile in the SUI sends the Terminal to a farm upstate

Make a new blank profile
Save the settings
Head over to	that "delete profile" button and hit it
ZOOP, no Terminal

Stack was useless - something above a DispatcherLoop with a ReportUnhandledError about a "The application attempted to access data outside the valid range" but I apparently don't have !xamlstowed installed anymore.

I cannot repro this on current main nor on preview (1.14.891.0). Do you consistently hit this or only sometimes?

Okay, got it to hit the second time I tried this. First time, no repro. Second time, I made the Terminal wide enough that the nav view was in its expanded state, and then the terminal died on the delete.

Gosh I can't find it now, but we had some crash in the past about changing the elements of a nav view. Something about the vector of items.... @carlos-zamora  do you remember what I'm thinking of?

        // Add an event handler for when the user wants to delete a profile.
        profile.DeleteProfile({ this, &MainPage::_DeleteProfile });

I'd bet we're adding the delete handler TWICE. Stepping through MainPage::_DeleteProfile, the first time it succeeds, but then there's a SECOND call to that handler. Why is there a second call? That's not right.
It might mean that the repro is

Make a new blank profile
Save the settings
Click on the profile ABOVE the new profile
Click back to the new profile
Head over to	that "delete profile" button and hit it
ZOOP, no Terminal


Silk worm bug

So i downloaded Dawn of time into a custom modpack with some friends and I looked to find silk worms, I checked a dozen biomes to find none, So we spawned me the plant and the egg to get started because of I couldnt find them, i then was able to get silk but then recently I placed the hatchery on the drying plate and when it finished it was an empty plate and I didnt get the silk worms.

Silkmoths start to spawn, if some grown bushes from mod nearby. Also don't make big farm with these bushes. Silkmoths doesn't despawn normally.

thanks it just started working, I dont know what did it but the mod just now started working again


More wrong head vote in v1.15.1

Describe the bug
this is stable 1k node, v1.15.1 was deployed in Feb 08 and there are more wrong head vote since then (pre Feb 08 is v1.15.0, post is v1.15.1)

Same to Sync Committee performance

on stable mainnet node, import head late (>4s) is also higher in v1.15.1

Expected behavior
Need to make sure "time to become head" metric is the same to v1.15.0
Steps to reproduce
No response
Additional context
No response
Operating system
Linux
Lodestar version or commit hash
v1.15.1

block processing time was increased in holesky

this may not be related to our release, there are more validators deposited in holesky

I don't see the same metric in mainnet node

Wrong Head Vote has been stable in the last 30 days on mainnet, closing the issue


v1.11 backports 2023-06-05

[ ] #25724 -- ipsec: Fix XfrmInNoStates drops on ESK & AKS upgrades (@pchaigno)

Minor conflicts in node.go due to https://github.com/cilium/cilium/pull/24208 not being in v1.11.
Trivial conflicts in config.go and daemon_main.go due to viper->vp change.

[x] #25744 -- ipsec: Reinitialize IPsec for new devices in ENI mode (@joamaki)

Trivial conflicts in includes and due to minor code movements in loader.go.

[ ] #25784 -- ipsec: Fix leak of XFRM OUT policies (@pchaigno)

Trivial conflict in node_ids.go due to a new function added in main.

[ ] #25735 -- ipsec: Fix XfrmOutPolBlock drops on upgrades (@pchaigno)

Minor conflict in includes of state.go.

Once this PR is merged, you can update the PR labels via:
$ for pr in 25724 25744 25784 25735; do contrib/backporting/set-labels.py $pr done 1.11; done

/test-backport-1.11

Closing in favour of https://github.com/cilium/cilium/pull/26021 that includes #25936.


strace: 6.3 -> 6.4

Description of changes
ChangeLog: https://github.com/strace/strace/releases/tag/v6.4

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 240205 run on x86_64-linux 1

1 package blacklisted:

tests.nixos-functions.nixos-test

8 packages built:

libabigail
libabigail.bin
libabigail.dev
strace
strace-analyzer
tests.testers.nixosTest-example
tests.testers.runNixOSTest-example
tests.trivial-builders.references


HLS issue

The fluid-player is great, but HLS gives a 404 after some change around 18. april 2022:

Adding the referenced file in code like this gives a CORS error:

Some change in hls.js that needs to be resolved in Fluid?

Hi! Same problems here. I am getting
Uncaught (in promise) ChunkLoadError: Loading chunk 3 failed.
(timeout: https://cdn.fluidplayer.com/3.0.0/vendors~hlsjs.min.js)
error. Please check this problem! Thanks!

@Minervatra could you check for this issue again? I suppose it should be solved in version 3.1.0 or in the newer version post that.

moving the ticket to done since there is no update from the reporter

same problem here


How to Add Slide from Other Presentation

how to add slide from one pptx presentation to another one?

Closing as duplicate of #175.


Allow token validation without signature

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[x] New feature or enhancement
[x] UI change (please include screenshot!)
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Internationalization and localization
[ ] Other (please describe):

What is the current behavior?

It is impossible to "validate" various parts of a JWT without providing the signature / signing key.
What is the new behavior?
This PR separates signing key validation as a separate step from overall token validation. As a result, I can choose to validate only the expiration, for example, without providing a key, which is much more convenient.
https://github.com/veler/DevToys/assets/7417301/e3fd3a9d-a04f-4c53-ad62-4532e02853ec
Other information
If we were starting from scratch, I think it would make sense for the overall validation operation to be called something like ValidateToken, and then it could have an option like ValidateSignature. However, the latter term is already used for the whole operation (and refers to a persistent setting), so I didn't want to change that. Instead I introduded a new option, ValidateIssuerSigningKey (which does align with the TokenValidationParameters), and as a result, ValidateSignature now has a slightly different meaning. I did update the UI strings to reflect this. Hopefully it's ok!
Quality check
Before creating this PR:

[x] Did you follow the code style guideline as described in CONTRIBUTING.md
[x] Did you build the app and test your changes?
[ ] Did you check for accessibility? On Windows, you can use Accessibility Insights for this.
[ ] Did you verify that the change work in Release build configuration
[x] Did you verify that all unit tests pass
[x] If necessary and if possible, did you verify your changes on:

[x] Windows
[ ] macOS (DevToys 2.0)
[ ] Linux (DevToys 2.0)

@veler Please tell me if I'm creating too much burden with my recnet PRs! I know you're pushing towards 2.0 and a lot of this will have to be rewritten anyway.

Would it be possible to add unit tests for the changes in this PR? :)

Would it be possible to add unit tests for the changes in this PR? :)

Of course! d1740e4ddfcbc5598c3cedfdd8c190780cbebcbd

Awesome! Thank you so much ! :D
Looks like we have a small merge conflict now 😅 Should be good to go once resolved.

Looks like we have a small merge conflict now 😅 Should be good to go once resolved.

Should be all set now! Thanks!

Thank you so much! :D


Add support for filtering server via labels

This pull request adds support for filtering servers by their labels, as per https://rpm.newrelic.com/api/explore/servers/list

@mwarkentin Thank you for opening this! This would be a great feature. Do you mind making the filter_labels param a dictionary? The key/values could be joined like:
['{}:{}'.format(label, value) for label, value in filter_labels.items()]

@micahhausler Sure, so the param would be a dict, where key is the type, and value is the value?
response = self.server.list(filter_labels={'Type1': 'Value1', 'Type2': 'Value2'})

Yep! And if you could add that note in the method docstring that'd be great!

@micahhausler Sorry for the commit spam, but this should be working now.

Looks great! Thanks! :+1:

Awesome, thank you!


Peer ID doesn't match libp2p public key

Describe the bug
Running in denver gnosis testnet, Lodestar is unable to discover peers because all discovered peers throw
Aug-01 23:17:38.392[NETWORK]         debug: Error dialing discovered peer peer=16...zfqTuk Error: Error occurred during XX handshake: Error occurred while verifying signed payload: Peer ID doesn't match libp2p public key.
AggregateError: Error: Error occurred during XX handshake: Error occurred while verifying signed payload: Peer ID doesn't match libp2p public key.
    at maybeSettle (/usr/app/node_modules/p-some/index.js:31:11)
    at /usr/app/node_modules/p-some/index.js:69:23
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
Aug-01 23:17:38.393[NETWORK]         debug: Error dialing discovered peer peer=16...zfqTuk Error: Error occurred during XX handshake: Error occurred while verifying signed payload: Peer ID doesn't match libp2p public key
AggregateError: Error: Error occurred during XX handshake: Error occurred while verifying signed payload: Peer ID doesn't match libp2p public key.
    at maybeSettle (/usr/app/node_modules/p-some/index.js:31:11)
    at /usr/app/node_modules/p-some/index.js:69:23
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
Aug-01 23:17:38.393[NETWORK]         debug: Error dialing discovered peer peer=16...JgcSVS Error: Error occurred during XX handshake: Error occurred while verifying signed payload: Peer ID doesn't match libp2p public key.
AggregateError: Error: Error occurred during XX handshake: Error occurred while verifying signed payload: Peer ID doesn't match libp2p public key.
    at maybeSettle (/usr/app/node_modules/p-some/index.js:31:11)
    at /usr/app/node_modules/p-some/index.js:69:23
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
Aug-01 23:17:38.393[NETWORK]         debug: Error dialing discovered peer peer=16...JgcSVS Error: Error occurred during XX handshake: Error occurred while verifying signed payload: Peer ID doesn't match libp2p public key
AggregateError: Error: Error occurred during XX handshake: Error occurred while verifying signed payload: Peer ID doesn't match libp2p public key.
    at maybeSettle (/usr/app/node_modules/p-some/index.js:31:11)
    at /usr/app/node_modules/p-some/index.js:69:23
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
Aug-01 23:17:38.394[NETWORK]         debug: Error dialing discovered peer peer=16...yteSaC Error: Error occurred during XX handshake: Error occurred while verifying signed payload: Peer ID doesn't match libp2p public key.

Expected behavior
Not reject all peers

Related comment https://github.com/ChainSafe/lodestar/issues/4243#issuecomment-1183531691
Got a better stack trace there.
Seems we initialize a noise handshake w a peer id (optionally), and the peer id is also exchanged during the handshake.

Closing for now since this issue is not actionable. To collect more data on future cases:

This PR will tell us the mismatched key https://github.com/ChainSafe/js-libp2p-noise/pull/273
By enabling metrics we can track the frequency of this type of errors


Install-WindowsFeature NET-WCF-HTTP-Activation45 failing on microsoft/iis container

I'm new to this and experimenting; attempting to add NET-WCF-HTTP-Activation45 to a microsoft/iis based container.  What am I missing?  Thanks in advance.
Starting container like this:
docker run -v d:\:c:\ddrive -it --entrypoint powershell microsoft/iis
From container attempting to install feature:
Install-WindowsFeature NET-WCF-HTTP-Activation45
Result is
Install-WindowsFeature : The request to add or remove features on the specified server failed.
Installation of one or more roles, role services, or features failed.
The service did not respond to the start or control request in a timely fashion.  Can you try "Install-WindowsFeature NET-WCF-HTTP-Activation45" on the container host machine?

Sure.  Here is the output from the VM container host, along with some version info:
C:\Windows\system32> Get-WmiObject -Class Win32_ComputerSystem | Select-Object -Property model

model
-----
Virtual Machine

C:\Windows\system32> hostname
GCARRDEV01
C:\Windows\system32> Get-WindowsFeature NET-WCF-HTTP-Activation45

Display Name                                            Name                       Install State
------------                                            ----                       -------------
        [X] HTTP Activation                             NET-WCF-HTTP-Activat...        Installed

C:\Windows\system32> Remove-WindowsFeature NET-WCF-HTTP-Activation45 -Verbose
VERBOSE: Uninstallation started...
VERBOSE: Continue with removal?
VERBOSE: Prerequisite processing started...
VERBOSE: Prerequisite processing succeeded.

Success Restart Needed Exit Code      Feature Result
------- -------------- ---------      --------------
True    No             Success        {HTTP Activation}
VERBOSE: Uninstallation succeeded.

C:\Windows\system32> Get-WindowsFeature NET-WCF-HTTP-Activation45

Display Name                                            Name                       Install State
------------                                            ----                       -------------
        [ ] HTTP Activation                             NET-WCF-HTTP-Activat...        Available

C:\Windows\system32> Install-WindowsFeature NET-WCF-HTTP-Activation45 -Verbose
VERBOSE: Installation started...
VERBOSE: Continue with installation?
VERBOSE: Prerequisite processing started...
VERBOSE: Prerequisite processing succeeded.

Success Restart Needed Exit Code      Feature Result
------- -------------- ---------      --------------
True    No             Success        {HTTP Activation}
VERBOSE: Installation succeeded.

DigitalProductId4         : {248, 4, 0, 0...}
InstallTime               :<PHONE_NUMBER>09694018
RegisteredOwner           : Windows User
RegisteredOrganization    :
PSPath                    : Microsoft.PowerShell.Core\Registry::HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Windows NT\CurrentVersion
PSParentPath              : Microsoft.PowerShell.Core\Registry::HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Windows NT
PSChildName               : CurrentVersion
PSDrive                   : HKLM
PSProvider                : Microsoft.PowerShell.Core\Registry

C:\Windows\system32> docker version
Client:
 Version:      17.10.0-ce
 API version:  1.30 (downgraded from 1.33)
 Go version:   go1.8.3
 Git commit:   f4ffd25
 Built:        Tue Oct 17 19:00:02 2017
 OS/Arch:      windows/amd64

Server:
 Version:      17.06.2-ee-10
 API version:  1.30 (minimum version 1.24)
 Go version:   go1.8.7
 Git commit:   66261a0
 Built:        Fri Apr 27 00:54:58 2018
 OS/Arch:      windows/amd64
 Experimental: false
C:\Windows\system32>

@glenncarr Could you please zip and attach all the logs from the %windir%\logs\cbs and %windir%\logs\dism folders? Thanks.

@sujitnayak here ya go
container-logs.zip
host-logs.zip

@sujitnayak Here they are...
container-logs.zip
host-logs.zip
Thanks,
Glenn

@glenncarr in the meantime, would starting from the official WCF container image make more sense?
@sujitnayak do you have any updates on this?

https://hub.docker.com/_/microsoft-dotnet-framework-wcf wcf has its own image.


getInputStream(_:outputStream:)

This seems to be a method available on Apple's NetService:
https://developer.apple.com/documentation/foundation/netservice/1418325-getinputstream

It's not available in this library. There is a limitation imposed by Swift-Foundation that prevents us from creating arbitrary Streams based off CFStreams. Other than that, I'd welcome a PR adding such functionality.

I'm sorry but I'm a little rusty in my NetService knowledge.  Does stream creation exceed the service-discovery goals of Bonjour / Zeroconf / mDNS?  I assume based on what you have implemented, I'll be able to get an address and initiate my own connection on that basis?

That's correct; after resolving a NetService, it provides you with addresses: [Data]. However the API from Foundation is not so nice to use. This library provides addresses: [Socket.Address] instead.
It might be nice to have something similar to getInputStream(_:outputStream:), but as mentioned it can't return the same types as Apple's implementation.

Cool.  Thanks.


Create index.html Vue App

Vue app index.html which shows index-latest.json in a table.
Similar to https://github.com/fair-software/fairtally/blob/main/fairtally/data/index.html.template, instead of inlining data fetch json file.

working off of the data structure in https://github.com/tortellini-tools/action/issues/35#issuecomment-856541067 for the moment, but easy enough to change

PR #122

done in #122.


Vue-like directives

I've grown to enjoy Vue's v-if, v-show, and v-for directives.  Maybe it's because I've gotten used to Vue, but it feels cleaner than the @if and @foreach blocks used by Razor and Blazor.  Specifically, it's easier for me to visualize the document structure when the code expressions are inlined with the elements versus external to them.  I also like how well it jells with declarative and reactive client-side development.
Could Razor/Blazor add a similar syntax?
I realize this is a personal preference and that there are folks that dislike this declarative style, so I'm not asking to replace the current syntax but rather to add to it in a Blazor-compatible way.
For example:
Razor/Blazor (today)
@if (bar != null)
{

        @foreach (var foo in bar)
        {
<EMAIL_ADDRESS>        }

}

Vue-like (suggested, or something similar)

    <li @foreach="var foo in<EMAIL_ADDRESS>

This isn't solving a problem. It will just confuse any developer/maintainer opening your code for the first time.

Would this be a good fit for tag helpers? https://docs.microsoft.com/en-us/aspnet/core/mvc/views/tag-helpers/intro?view=aspnetcore-2.1

@rynowak, @SteveSandersonMS thoughts regarding Blazor support for this?

Only that this is not a decision we'd make lightly. This changes the core design principles of the language.

Totally understand it's not a simple decision.  But it feels like we're almost there conceptually with tag-helpers allowing us to create a condition tag helper (albeit server-side) and the discussion of item templates for Blazor which would almost enable a foreach directive.

We have no plans to handle this work in the near future, as it doesn't align with our current priorities. Closing this for now. May the priorities change and we'd like to consider this for the future, we may reopen it then.


BSDFValidator starter files

First steps towards #382
Currently, there is only a single button in GUI which loads .vert and .frag files - which would be used by the user to upload glsl function definitions.
These loaded function definitions would then be appended to the shader source code, which already have the function signatures. The function signature of bsdf_cos_sample should be present in the vertex shader because it outputs positions, but it is currently in the fragment shader.
I've also encountered a bug where, as soon as I take control of the GUI cursor, the yellow quad on the screen, which was visible earlier, disappears.
It seems like there is a *.layout entry in the .gitignore file which prevents the XML layout (which also has the extension .layout) from being tracked.

Will review tomorrow

It seems like there is a *.layout entry in the .gitignore file which prevents the XML layout (which also has the extension .layout) from being tracked.

this is a remnant of a time before cmake, we used to have codeblocks project files in the repo. Now we can allow .layout to be tracked.

Thanks for the review.
Do you have an specific GUI in mind?

Do you have a specific GUI in mind?

512x512 window.
Drop down list of BSDFs to use:

Diffuse
Custom (Load File)

Diffuse BSDF and sample function shouldn't be too hard for you to put together.
When "Custom" gets chosen, then file open dialog pops up asking for a .glsl file

How about instead of a drop down let's just default to Diffuse BSDF, then whenever the user presses the Load Function Definitions button they'll be prompted to open a .glsl file?
If going back to Diffuse BSDF is important we can add a Reset button as well.
I feel like a drop down will introduce unnecessary complexity.

I feel like a drop down will introduce unnecessary complexity.

Fine.

Function signatures in the Fragment Shader maybe wrong, it is just to build without errors. They'll be updated accordingly.
Next is the reset button.

I need to spec out the signatures of the functions for the bsdf for you

Are the function signatures finalized?

On it

You can use this as a draft
// do not use this struct in SSBO or UBO, its wasteful on memory
struct BSDFBasicParams
{
   vec3 L;
   vec3 N;
   // optional can have undef value (specular only)
   vec3 V;
   // for differentials
   vec3 dPosdx;
   vec3 dPosdy;
};

// do not use this struct in SSBO or UBO, its wasteful on memory
struct BSDFSurfaceParams
{
   BSDFBasicParams basic;
   float NdotL;
   float NdotL_squared;
   float NdotV;
   float NdotV_squared;
   float NdotH;
   float VdotL; // same as LdotV
   float VdotH; // same as LdotH
   float LplusV_rcpLen; // left over for BSDF that want to implement bump mapping
};

// do not use this struct in SSBO or UBO, its wasteful on memory
struct BSDFAnisotropicParams
{
   BSDFIsotropicParams isotropic;
   vec3 T;
   float TdotH;
   vec3 B;
   float BdotH;
   vec3 TdotL;
   vec3 BdotL;
   vec3 TdotV;
   vec3 BdotV;
};

BSDF function signatures to be used in your program
// Spectrum can be exchanged to a float for monochrome
#define Spectrum vec3

//! This is the function that evaluates the BSDF for specific view and observer direction
Spectrum bsdf_cos_eval(in BSDFIsotropicParams);

//! generates a incoming light sample position given a random number in [0,1]^2 + returns a probability of the sample being generated (ideally distributed exactly according to bsdf_cos_eval)
BSDFSample bsdf_cos_gen_sample(in vec2 _sample);

//! returns bsdf_cos_eval/Probability_of(bsdf_cos_gen_sample) but without numerical issues
Spectrum bsdf_cos_sample_eval(out DirAndDifferential L, in ViewSurfaceInteraction V, in vec2 _sample)

Ok @achalpandeyy I stopped editing around the BSDF struct definitions

The user is going to fill in the struct details in the loaded function definitions, right?
So that means, things like VdotL, NdotL etc are not necessary, because different BSDFs will have different formulae and they might not require VdotL (for example) specifically for their evaluation.

There is no BSDFSample struct anymore !

The user is going to fill in the struct details in the loaded function definitions, right?

No, but I'm making those functions for you.

There is no BSDFSample struct anymore !

Its there
// do not use this struct in SSBO or UBO, its wasteful on memory
struct BSDFSample
{
   DirAndDifferential L;  // incoming direction, normalized
   float probability; // for a single sample (don't care about number drawn)
};

So that means, things like VdotL, NdotL etc are not necessary, because different BSDFs will have different formulae and they might not require VdotL (for example) specifically for their evaluation.

Then the compiler will optimize them away ;)

No, but I'm making those functions for you.

So, they are not meant to be included in the fragment shader source code?

So, they are not meant to be included in the fragment shader source code?

Not the user's code, you will prepend all that.

Right.
So, the user is expected to write function definitions in terms of these structs?

Yep.

@achalpandeyy getter functions
https://github.com/buildaworldnet/IrrlichtBAW/issues/384#issuecomment-546903906
You can either:
A
Map each pixel in a cubemap to an incoming (light) direction.
Then produce a number of different cubemaps (cubemap array) for varying outgoing (view) direction.
B
Assign one axis of a 2D image to the angle between the view and normal (NdotV) the other to the light and normal (NdotL).
Then produce a number of different textures (2D texture array) for varying VdotL values.
I believe Disney calls it BRDF slices, (you'd just have to extend the axes from [0,1] to [-1,1] for the NdotB and NdotL)
For anisotropic BSDF you can map your texture array layers to varying VdotL AND TdotV.
It will take you a bit of trigonometry to derive TdotL from the 4 other angles.
Notes
If you have anisotropic BSDF then you need to come up with a way to turn a [0,array_count] sequence into different samples of 2D values.

The official BSDF signatures are now here
https://github.com/buildaworldnet/IrrlichtBAW/issues/384

Unfortunately, I ran out of time to work on this. I will not be able to make any significant progress for at least a month. If its urgent, please feel free to take it from here.

Unfortunately, I ran out of time to work on this. I will not be able to make any significant progress for at least a month. If its urgent, please feel free to take it from here.

No problem.

You should move this work onto the shader_pipeline branch from my repository, since you're like 10-15% done, it would make sense to use the new API.


Does git-tfs branch support the --export | -x option

So you can clone a TFS repository with the --export | -x option to preserve work item tracking. You can also provide that option to git-tfs pull (which I only knew from the interactive help--it's not documented here on GitHub in the /docs folder, unlike for git-tfs fetuch, where it is documented in the online help), but the documentation for git-tfs branch (either online here at GitHub, nor in the interactive help) does not mention whether or not this command accepts a --export | -x option when --init-ing a TFS branch (e.g. a branch that was created in TFS after your initial clone).
So the command I want to run is:
$ git tfs branch -x --init My/TFS/Branch

Is this possible to do today? If not, then this is a feature request to bring the branch command more inline with the fetch and pull, and clone commands.

Accidentally opened a dup--keeping the "new" one because it contains a spelling correction.


Unimplemented type for cast (INTERVAL -> TIME)

In the ODBC spec (see sqlext.h), the data type INTERVAL has the same data type identifier as TIME. When casting a DuckDB INTERVAL to an ODBC INTERVAL, I get the following error:
ODBC: ERROR [22007] ODBC_DuckDB->GetDataStmtResult Not implemented Error: Unimplemented type for cast (INTERVAL -> TIME)

Repro steps:

Create a new DuckDB database with an INTERVAL variable:

create or replace table test AS (
    select
        x * (INTERVAL 5 HOUR) as customer_interval_field
    from generate_series(1, 10) g(x)
);

Load the data into Power Query with the following script:

let
    Database = "path\to\test.db",
    OdbcDatasource = DuckDB.Contents(Database)
in
    OdbcDatasource{[Item="test"]}[Data]

This throws the error:

ODBC: ERROR [22007] ODBC_DuckDB->GetDataStmtResult Not implemented Error: Unimplemented type for cast (INTERVAL -> TIME)

This may need to be addressed in the DuckDB ODBC driver.
Related to #5 .

A similar error when using a TIME field:
ODBC: ERROR [22007] ODBC_DuckDB->GetDataStmtResult Not implemented Error: Unimplemented type for cast (TIME -> DATE)

Similar issue when using UUID field:
Unimplemented type for cast (UUID -> DOUBLE)

There was a related workaround needed for the Tableau connector: https://github.com/MotherDuck-Open-Source/duckdb-tableau-connector/blob/main/tableau_connectors/duckdb_jdbc/dialect.tdd

Here is a related issue with TIMESTAMP that I was able to reproduce: https://github.com/duckdb/duckdb/issues/9963

Relevant issue: https://community.fabric.microsoft.com/t5/Desktop/Duration-type-columns-are-not-recognized-from-an-ODBC-connection/td-p/2156211

Most recent screenshot post-fix (also casting UUID to VARCHAR):


Refactor Image SBOM creation and image pulling to use goroutines

Description
Whenever I generate a package if I forget to use --skip-sbom I usually end up cancelling and modifying the command because it can take a relatively long time. SBOM generation is easily made concurrent though and has massive speedups on systems with enough cores. This PR rewrites the image SBOM creation to spawn a goroutine for each image and then uses a select statement to wait for the generation to be done.
While I was at it I added concurrency for image pulling as well!
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Other (security config, docs update, etc)

Checklist before merging

[ ] Test, docs, adr added or updated as needed
[ ] Contributor Guide Steps followed

bigbang example package generation
modded run times:
real    2m35.223s
user    6m15.280s
sys     0m59.717s
---
real    2m36.357s
user    6m20.161s
sys     0m59.530s
---
real    2m35.577s
user    6m24.044s
sys     0m56.640s

0.28.0 run times:
real    3m35.299s
user    9m21.381s
sys     0m47.760s
---
real    3m34.414s
user    9m33.166s
sys     0m46.836s
---
real    3m41.898s
user    8m47.642s
sys     0m44.218s

0.28.0 skip sbom run times:
real    2m28.487s
user    2m38.983s
sys     0m12.519s
---
real    2m22.387s
user    2m37.232s
sys     0m13.272s
---
real    2m22.329s
user    2m36.669s
sys     0m12.810s

tldr:

goroutine    avg 2m35.719s
v28 normal avg 3m37.204s
v28 skip      avg 2m24.401s
sbom creation time is now 11s vs 1m13s on my machine with the bigbang example package

hijacked the PR to include image pull concurrency too @Racer159 Can move it to another PR if it's not wanted here.

Don't know why ci is failing, same commands work fine on my local machine


Add Config Connector solution service-account

Adding a new solution for Config Connector service-account resource.
This solution comes from the TF CFT service_account solution.

Hi @AlexBulankou @maqiuyujoyce @trodge,
Could you take a look at this solution for service-account please?
Thanks!

Hi @AlexBulankou @maqiuyujoyce @trodge @BrianPi
PR has been updated and ready to review. Please take a look at the latest changes.
Thanks!


Githubdam h64 updates test iOS VoiceOver

Hi Marc, Please review the branch if you find any incorrect please advise.

Completed - ok to delete


Fixed definition file for newer TS versions

In typescript 4.2 definition file causes error:
Error: node_modules/resize-observer-polyfill/src/index.d.ts:19:18 - error TS2717: Subsequent property declarations must have the same type.  Property 'contentRect' must be of type 'DOMRectReadOnly', but here has type 'DOMRectReadOnly'.
Declaring DOMRectReadOnly as global fixes that issue.

We also have this issue. Can we get this merged?

the same for us. having TS error using Angular 12. please review the issue @que-etc
thank you!

This would solve the issue described in #83

Hi, this branch is now open for quite some time - could this be merged somewhere near future?  :)

I have the same issue, could you please merge the fix and release a new version?


Dynamic exposure adjustment to make sure we can visualise the reconstructions. E.g:

We might have to do some dynamic exposure adjustment to make sure we can visualise the reconstructions. E.g:
Default:

Exposure corrected:

Originally posted by @crangelsmith in https://github.com/alan-turing-institute/affinity-vae/pull/127#discussion_r1217751170

Fixed by min-max normalisation


Is (Storing events + publish events) Atomic in UoW?How?

Hi.
Based on Sending Events to Event Bus post in Google Groups.
I think the best way to control the atomicity is having an Outbox table or transaction log tailing in DB , EventBus can simply poll the table for new events and remove events from outbox when published.
what is your opinion?
Thanx.
Transaction log tailing
microservices-aggregates-events-cqrs
Using sagas to maintain data consistency in a microservice video '43:00'

Excuse me, I did not really understand how Axon control atomicity with attached Transaction(SpringTransactionManager,...) in UoW.
What happens if the system fails or eventbus fails before events are released?

Hi @mehdichitforoosh,
please do not use the issue tracker for questions. You can post questions on the mailing list<EMAIL_ADDRESS>To briefly answer the question: appending an event to the log is one operation. Handling it is another. If handling fails, you either retry or skip... Furthermore, each handler has its own transaction on the consuming side. Also, "consuming" doesn't remove the message from the queue. Consumers are "free" to revisit streams if they wish.

Hi @abuijze
I agree with you.
Let me say another way:
One solution to this problem is to add an extra column to the DOMAIN_EVENTS table(DomainEventEntry) that tracks whether an event has been published. The event publisher(EventStore)  would then use the following process:
1 Find unpublished events by executing this SELECT statement:
SELECT * FROM DOMAIN_EVENTS where PUBLISHED = 0 ORDER BY event_id ASC
2 Publish events to the message broker or an event bus.
3.Add a callback for example onSuccessPublish() to EventBus.
4 Mark the events as having been published:
UPDATE DOMAIN_EVENTS SET PUBLISHED = 1 WHERE EVENT_ID in(?,?,...)

I guess this question reflects what I recently discussed in the forum.
The issue arises whenever the event store and the component which emits the event cannot take part of the same database transaction. The outbox pattern closes this gap as it publishes the event to the event bus asynchronously.
The corresponding thread: https://groups.google.com/forum/m/#!topic/axonframework/_VNvbIATXj4

Hi.
I solved this problem by seprating Event store from Event Bus.
1.first transaction persist events or entity (Without  publish events)
2.a scheduler calls rePublish() method in EventBus and then polls event store or Outbox table.
3.publish events to (RabbitMq,Kafka,...) and then calls markAsPublished() in EventBus when second transaction successfully commits.
I think Axon Framework ultimately needs to use this method to be reliable.
Of course this is my opinion.
Thanks.

What @OLibutzki signals with the Outbox Pattern is quite identical to having a separate process poll for new events being published.
Reading that, this seems to be exactly what the TrackingEventProcessor does.
The thing you're pointing too, @mehdichitforoosh, is thus already implemented by the framework if you'd ask me. You should just refrain from utilizing the SubscribingEventProcessor for guaranteed message delivery, is that simply doesn't provide it.
Note that the Kafka extension is a StreamableMessageSource solution, thus suitable for a `TrackingEventProcessor.
The nature of RabbitMQ in the AMQP extension is however such that it would require quite some custom code to be a polling mechanism. Hence, it is implemented as a push solution covered by the SubscribableMessageSource, suitable for SubscribableMessageSource.
As already discussed, the SubscribableMessageSource does not give you the security you're looking for, @mehdichitforoosh. I'd thus suggest you move away from that pattern if you require ensured delivery.
So, to reference your comment just now:

I think Axon Framework ultimately needs to use this method to be reliable.
Of course this is my opinion.
I believe the framework already does this, this conversation just started off in the wrong direction.

If any of my description above does not fit the explanation you, @OLibutzki, or you, @mehdichitforoosh, have provided, then please correct me.

Hi.
Thank you @smcvb
Ok.I'll check TrackingEventProcessor and we will discuss about it.
Good luck.

No problem @mehdichitforoosh, we're here to help!
Hope it covers what you're looking for.
And, as pointed out, if it doesn't, please do not hesitate to reach out.
I do want to ask though that questions like that will be posed either on Stack Overflow with the axon tag or on our User group.
We'd like to keep Github a place for features, enhancements and bugs mainly; hence the question.

Hi @smcvb,
thanks for answering.

What @OLibutzki signals with the Outbox Pattern is quite identical to having a separate process poll for new events being published.
Reading that, this seems to be exactly what the TrackingEventProcessor does.

Exactly, it's similar to what the TrackingEventProcessor does. But there are slight differences:

It's possible that events are processed by multiple TrackingEventProcessors.
In an outbox a message/event is processed exactly once. There is a single tracker which is responsible for:

detecting new outbox entries
publishing them to the event bus
and deleting deleting the outbox entries or marking them as read

In my opinion you need both: The TrackingEventProcessor and an OutboxTracker (or whatever it might be called).

I do want to ask though that questions like that will be posed either on Stack Overflow with the axon tag or on our User group.

Sure, that's what I did in the first place. I just stumbled upon this issue which seemed to be similar to my request.

Thanks for your added feedback @OLibutzki, I'll see to this suggestion will be discussed at on point or another.


Add cloud logging support

We've had a few customers ask about logging to google cloud from Ruby applications.  It would be great if we could support cloud logging.
https://cloud.google.com/logging/docs/api/
We should make sure to work with the cloud logging team (they will have feedback on which API to use).

@JustinBeckwith Great to hear you can report interest in this, because @blowmage has been pushing for us to support it, and afaik it is the next new service that we will tackle.

Sweet.  You'll want to chat with @andreweames on the logging team - let me know how I can help!

@JustinBeckwith We are starting work today on support for Cloud Logging (v2beta1) in gcloud-ruby!
I'm closing this original issue, as it has been replaced by finer-grained ones in the 0.7.0 milestone.
/cc @andreweames

Great news!

@JustinBeckwith We are really excited to get after this one. We have issues for Ruby Logger compatibility (#505) and for some docs demonstrating Rails integration (#506). If there are any additional issues you can suggest, please do so!


Bug: NotSupportedColumn UUID

Describe the bug
Startletter-admin does not support uuid column
To Reproduce

import from sqlalchemy.dialects.postgresql import UUID
add uuid = Column(UUID(as_uuid=True), default=uuid.uuid4, unique=True) to table

Environment:

Starlette-Admin version: 0.5.0
ORM/ODMs: SQLAlchemy
Framework: FastAPI
Database: PostgreSQL

Additional context
When I take out the UUID column from the database table, the admin interface works. And when I add it back, it gives me this error:
raise NotSupportedColumn(  # pragma: no cover starlette_admin.contrib.sqla.exceptions.NotSupportedColumn: Column UUID is not supported
However, when I accessed this file starlette_admin/contrib/sqla/helpers.py, line 149, in the convert_to_field function, I see that it supports "dialects.postgresql.base.UUID": StringField

Thank for reporting this


Add support for resetting tab stops (DECST8C)

Description of the new feature/enhancement
DECST8C is an escape sequence which resets the tab stops to their default values, i.e. every 8 columns. It's especially useful for a modern terminal because it's the only practical way to restore the default values over an "infinite" width. Without this you'd have to set the positions manually, which limits you to the current screen width (technically you could also perform a hard reset with RIS, but that's not something an application should be doing).
Proposed technical implementation details (optional)
We actually already have a method in AdaptDispatch that performs a tab reset (it's used in the hard reset implementation). We just need to make it public, and hook it up to the relevant escape sequence in the state machine.
It's a little bit more complicated than I'm making out, because the escape sequence takes a parameter value, and DECST8C is only applicable for one specific value, but that's easy enough to handle.

I have my own internal notation i use, but i just used tab stops today for the first time and they worked....
I basically cleared all tab stops, moved to column 7, set one there, and then the colortool.exe char aligned properly (column 1 waqs the only one with variable-width values in it to screw up tabbing)

@ClaireCJS If you wanted to reset all the tab stops to the default 8 column spacing (which is what colortool is really expecting), you could also just use echo %ESCAPE%[?W (that's the DECST8C sequence that we added here).


PyJWT 2.0 not compatible

The current version is not compatible with version 2.0 of PyJWT, since jwt.ExpiredSignature has been renamed to jwt.ExpiredSignatureError (used in line 17 in backend.py).
Suggested suggestions:

Either realign with PyJWT 2.0
Cahnge install_requires in setup.py from 'pyjwt >= 1.0.1', to 'pyjwt >= 1.0.1, < 2.0',

A temporary workaround for people facing this issue is to pin the PyJWT package to v1 in your own dependency files.
E.g. if you use requirements.txt, adding PyJWT==1.7.1 should solve the problem.

A temporary workaround for people facing this issue is to pin the PyJWT package to v1 in your own dependency files.
E.g. if you use requirements.txt, adding PyJWT==1.7.1 should solve the problem.

PyPI version released.

PyPI version released.


Save with given node as root

I often have a large SGF file corresponding to a whole game with many variations, or a joseki tree, and would like to export a portion of it, just everything below a given node, so that the exported SGF file starts with the given node's position. This would require removing everything above that node in the tree and doing any other initialization in that node that was previously implicit (e.g., place all the stones on the board, update number of captures).
I'm sufficiently interested in this feature that I might try to work on it myself.

Sabaki has the ability to 'flatten' a node that does what you described, I think. Use 'Edit' > 'Flatten' or 'Flatten' in the context menu in the game tree.

Ah, so it does! For some reason I thought that Flatten removed variations as well, but I should have actually tried it. Great! Now I just have to remember to not Save and blow away my file with the full tree. :)


Cray-mpich compatibility

Cray-mpich does not conform to MPI standard and has a lower limit for maximum tag than other MPI implementations.
This means that tile tag system needs to be redesigned. Probably by splitting main communicator...
Current implementation is a hack to make it work for now.

Note: actually mpich-cray does conform to standard but the maximum tag value is ridiculously small (2^15-1 = 32,767). Cray has 2^21-1 ~3e6 after which tags start to overflow.

Tried:

[x] dropping tag cycling values and adding mpi barriers to code

loop evaluation hangs for some weird reason

[x] new tag system where instead of tile id (that can be very large; e.g., 1024^2 = ~1e6) create tag based on cid ordering per each mpi origin/destination rank.

Fixed also corgi tile initialization. It no longer sends info message to everybody about number of incoming tiles but instead uses the deterministic nature of the algorithm and each rank computes incoming tile numbers themselves.

Closing this - performance improvements are moved to another branch....


Bug: ECMPS UI Error Handling

The ECMPS UI is not handling errors from the api's properly.
The issue
Currently there are occasions where an error has happened in the app but no error message is displayed in the UI and the screen goes blank
Expected Results
The app should always render something even if there is an error. An error at the top of the content section and display the actual error to the user.
Steps to Reproduce

Revert #50 widows creek
export
import


AdSecGH-63 Fix unflatten Test

@DominikaLos there is a failure on the test due to the pre-commit module. Can you make sure you setup this locally? I have added instruction how to do that on the README.
Once you do this you will be able to do pre-commit run and fix the file, there recommit.


sekai de ichiban ohime sama [¶(⁄•̑˅̮•̑∖)⁋]

i had to do these changes on master branch to check how it works
yml version is completely draft (someone please complete it)

good enough


Add more Game category icons

Icon name
Bow and Arrow, Dagger, Staff, Melee, Spell, Sphere, Cube, Cone, Potion, Bludgeoning, Piercing, Slashing, Poison, Immune, Resistant, Vulnerable
Use cases
I am building an app that makes playing Tabletop Role-play games easy and we have been getting feedback requests to be more visual. An easy way to do this is through iconography through our chosen lib (Lucide!) but there are quite a few use cases missing in the Game category.
Design ideas
Our app is high-end (hence why we are using Lucide), so the icons could resemble the following:

Bow: Like the swords but a bow with an arrow
Dagger: Mini sword
Staff: Magical staff - think Lord of the Rings (wooden with a gem or something)
Melee: Action that resembles close range attacks
Bludgeoning - Same as Elden Ring attack with bludgeoning type weapon
Piercing - same a Elden Ring - sharp pointy weapon
Slashing - same as Elden Ring - sword type weapon
Spell: Magical essence? 🤷♀
Sphere/Cube/Cone - 3D transparent visuals
Potion - potion bottle
Poison - exactly what it means
Immune - visual that will represent if the character is Immune to X thing
Resistant - visual that will represent if the character is resistant to X thing
Vulnerable - visual that will represent if the character is vulernable to X thing

Checklist

[X] I have searched if someone has submitted a similar issue before and there weren't any. (Please make sure to also search closed issues, as this issue might already have been resolved.)
[X] I have searched existing icons to make sure it does not already exist and I didn't find any.
[X] I am not requesting a brand logo and the art is not protected by copyright.
[X] I am not requesting an icon that includes religious, political imagery or hate symbols.
[X] I have provided appropriate use cases for the icon(s) requested.

Can you come up with even more use cases?
The more use cases, the quicker it usually makes it into the lib.

Can you come up with even more use cases? The more use cases, the quicker it usually makes it into the lib.

Addendum: use cases per icon, not as a group.


Make sure current item is never larger than total number

RELEASE_NOTES=[BUGFIX] Fix fsck progress bar.
Fixes #911
Signed-off-by: Dominik Schulz<EMAIL_ADDRESS>

Codecov Report

Merging #1384 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #1384   +/-   ##
=======================================
  Coverage   57.78%   57.78%
=======================================
  Files         193      193
  Lines       10997    10998    +1
=======================================
+ Hits         6354     6355    +1
  Misses       3831     3831
  Partials      812      812

Impacted Files
Coverage Δ

internal/action/fsck.go
54.29% <100.00%> (+1.34%)
:arrow_up:

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 d41ebd0...7f22c8a. Read the comment docs.


Pattern proposal: Provision databases and their bindings on demand from any provider

Why?
Developers want to deploy typical database systems like PostgresQL to one of several providers without having to select the provider in advance. Developers also need credentials and URLs for provisioned databases to be injected for app code to use at runtime, rather than at build time or earlier.
This pattern will show how to delay selection of a database provider till runtime and inject required binding info in a uniform way at runtime for apps to use.
How?
Following are some implementation proposals I intend to start with; tips and other recommendations are welcome!

Use Crossplane to provision a PostgresQL database system from any of at least four providers: Azure, AWS, GCP and CrunchyData.
Research binding providers and use one that can provide the exact same experience regardless of which provider provisions the actual database. Potential candidates include:

service-binding-operator: https://github.com/redhat-developer/service-binding-operator
Vault agent and secret engines: https://github.com/hashicorp/vault/tree/main/command/agent
Secrets Store CSI driver: https://github.com/kubernetes-sigs/secrets-store-csi-driver

Run an API service in a container which simply allows CRUD of database records. Several frameworks should be able to consume the same bindings published in the previous step.

Initial examples will be in Java (Spring Boot or Quarkus) and Node.js.

Finally, demonstrate how to deploy these components reliably and in order using TBD - will start with shell scripts.

Closing this issue as there are no further action items associated with it.


feat(rest/server): support TLS protocol

Can you add a link to resources about TLS connections in JSDoc comment block?


Provide SDK and Cli of multi-cloud

@BaiHuoYu Cli should support authentication with keystone and ak/sk.

Please add a release note.


Loop interval timing

In the documentation, it says that the loop timing must be >2 seconds while the code only waits 1 seconds. I set my pause to 2.5 seconds and it helped somewhat. Still throws quite a few errors (I am testing 7 of them simultaneously).
Thanks a million for this work! It really helped me.

What kind of errors do you receive? At runtime, at compile time?

I wish I remembered. I caved and bought wifi sensors instead. My best guess/memory is that during runtime, the sensors would return bad data if the poll time was too short. It would compile fine.


Invalid api key

Hello @fugkco ,
I have installed jackett with elementum. But I getting error as invalid api key. How I can get that or how to fix this. Many thanks for your support..

did you even configure the api key in plugin settings?

this goes without saying because it is expected that if you know about Jackett then you know that you need to install it
https://github.com/Jackett/Jackett (see installation steps)
or find a public Jackett in internet.
script.elementum.jackett is just a "broker" between elementum and jackett.

Yes, you need to set up Jackett and input the API key in this addon's settings. Closing as there's no response from OP. Feel free to comment, I'll be happy to re-open.


Added support for optional param documentation with using

Without these changes the using does only work for requires and not with optional parameters. This restriction is currently not documented.
The feature / fix is required to provide a more complete DRY approach when using grape-swagger and grape-entity. With these changes, one can document the possible fields and values, e.g. when using PATCH update, without requiring them to be present.

Odd that we only did it for requires. This looks good, thank you.

:+1:


Add huskyci-client into main project

Closes #160
This PR will add huskyci-client into this repository.
Instructions to build and run this client have been added to Makefile.

🥇 Champion!

Looks great! Thanks @rafaveira3!