How to set QuickFilter default value (older version of Servenity 2.1.9, pre .tt)

Hi Volkan,
I have an application built using an older version of Serenity. Trying to set the quick filter default value as such but it's not working:
        protected override List<QuickFilter<Widget, object>> GetQuickFilters()
        {
            List <QuickFilter<Widget, object>> filters = base.GetQuickFilters();

            QuickFilter<Widget, object> qf = filters[0];
            qf.Field = "2017";

            return filters;
        }

Many thanks in advance for a hint.

That would change the field, not value, you should do similar to initializing quick filter sample


how are the 5k test images in  coco train_val picked? randomly?

yeah,I also found the same problem,quite confused!

the picked images json file could download in http://dl.yf.io/fs-det/datasets/cocosplit/. hope can help you.


Add phplogin_menu.php as fallback menu

I realized a flaw in my last menu-commit, the custom menu will not be displayed when you're at a settings page. This patch always tries to include a file named "phplogin_menu.php". I'm not super satisfied with the solution - any other ideas?
Allow either override by defining $barmenu as array or try to include the phplogin_menu.php file which should define the menu array. If no such file don't show anything in the menu bar.

Hey Daniel, sorry I've been busy the past week or so and didn't have time to get to your previous commit, thanks for this. I didn't notice the issue myself from the earlier merge that I pulled in, but there were a few people saying they were running into bugs related to this, so this is great!

Hey @daniel-falk sorry about running behind on these, it's been a really busy couple of weeks. I think it's a pretty decent solution. We are running into an a small error though for some reason on some page loads with an undefined $barmenu variable in pagehead.php. I'm tweaking that as well as doing some other small tweaks (I like the idea of moving that phplogin_menu.php file into the same partials folder as where nav.php is. I'm also just renaming it barmenu.php to make it more recognizable). The partials folder I was thinking would have files that are meant to be more user-editable (though now I realize there are two files in there that shouldn't be in there, so I'll have to move them to a more logical location too.
Also, I just noticed that the config page has an error in it (it looks like maybe the token_validity setting was just inserted wrong in the installer), so I'll patch that up too.

Ok, I got those sorted out @daniel-falk , I think it should be good now


Problem: merkel-sha3 not published to crates.io, hence build broke

Solution: publish merkel-sha3 to crates.io and bump the rustRegistry
This ensures we don't introduce impurities into the build env

:+1:


traceback issue

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
No
Source
source
TensorFlow version
2.17.0
Custom code
No
OS platform and distribution
windows 10
Mobile device
No response
Python version
3.11.7
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
when trying to run the code shows
Standalone code to reproduce the issue
.

Relevant log output
Traceback (most recent call last):
  File "C:\Users\PC\AppData\Roaming\Python\Python311\site-packages\tensorflow\python\pywrap_tensorflow.py", line 70, in <module>
    from tensorflow.python._pywrap_tensorflow_internal import *
ImportError: DLL load failed while importing _pywrap_tensorflow_internal: Error en una rutina de inicialización de biblioteca de vínculos dinámicos (DLL).

@kevo200,
Could you please provide the tensorflow and the compatible version which you are trying to install. Also there are at least 3 possible scenarios:
- You need to install the MSVC 2019 redistributable
- Your CPU does not support AVX2 instructions
- Your CPU/Python is on 32 bits
- There is a library that is in a different location/not installed on your system that cannot be loaded.

https://github.com/tensorflow/tensorflow/issues/61887
Thank you!


label MANE select transcript in list of transcripts in small variant filter

Is your feature request related to a problem? Please describe.
For reporting purposes the investigator needs to report a variant based on the reference transcript. While varfish already shows a list of transcripts, the MANE select transcript is not explicitly labeled.
Adding this label would greatly help in collecting information on variants of interest.
Describe the solution you'd like
Label the MANE-select transcript in the variant window.
Additional context
The MANE select information is only available for hg38, but liftovers to hg37 exist.

Partially done, rest subsumed by #1300.


Don't include upcoming features in main README

Currently the main README includes details of unreleased features. We should only show the README from the current release.

Thanks for the head's up @G-Rath.
I've temporarily created a last-stable branch for now and made that the main branch to avoid confusion from new users.
This is not ideal (new PRs will be opened against this by default instead), but for now I can't think of another clean/easy fix.
@another-rex WDYT?

Longer term we probably want to move the docs into a dedicated rendered site, and reduce the amount of detailed documentation in the main README.

This is probably the best option for now, in the slightly longer term @hayleycd's suggestion to move the actual usage instructions from the README over to it's own static site / github page should resolve this fully.

Well, since moving instructions to it's own page is the plan, maybe for now it'll be less disruptive if we set the default back to main, and add a note to check out the last-stable branch readme for usage instructions, and that main has unreleased changes (e.g. #195)

Good point: we can have a lighterweight hybrid approach for now. I think we can:

Reduce the root README.md to be the bare minimum, and move the bulk of the documentation to a USAGE.md or something similar.
Point to the last-stable USAGE.md in the main README.md. There is a chicken and egg in that USAGE.md doesn't exist for last-stable currently. For now we can just point it to the main README.md, and fix things up next release.

@another-rex, if this sounds good to you, I'll assign this to you :)

I've actually done a fair amount of the work to move the docs to a github page, we could have it up soon. We can build the github page from whatever branch you like. We can add to the docs on main, moving over changes to the stable branch when ready.

I'm fine with the approach. I'm just letting you know we could have a page ready in a couple of days.

SG! Let's get this quick fix in now, and then move over to the proper documentation site. Thanks a ton @hayleycd !!

Fixed by moving documentation to github page in #201


Fix mWeb Compose Focus bug

Details

Buckle up this is a doozy.
Bug exists because React Native handles UX differently between web/mWeb/Desktop and native platform - for iOS and Android when a TextInput has focus tapping/clicking away from that input won't "blur" the input and focus will remain unless another TextInput is given focus. Web/mWeb/Desktop allows focus to blur how you would expect by clicking/tapping outside of an input.
With how can canFocusInputOnScreenFocus to detect between platform it is wrongly assuming b/c mWeb has touch events that the Native UX for focus will be there and that we don't need to trigger the code to manually give focus back to the compose component.
We are mainly focusing on fixing the bug here for now. We'll spin up a new issue to address the UX inconsistencies around the focusing of the compose component in a new issue.
Fixed Issues

$ https://github.com/Expensify/App/issues/13443
Tests

[X] Verify that no errors appear in the JS console

Offline tests

N/A same as QA steps since this is just some UX that doesn't involve specific offline behavior
QA Steps

[X] Verify that no errors appear in the JS console

Click into any chat.
Focus (click/tap) into the compose input at the bottom
Click the + icon to reveal the add attachment or split/request money modal
click/tap away from the modal to dismiss it
confirm focus is returned to the compose input
Click the + icon to reveal the add attachment or split/request money modal
Click to send money, on the modal that appears click the x to dismiss it.
(Existing Behavior) On Web/mWeb/Desktop focus should be given back to the compose input. On Android / iOS focus should not be given back to the compose input. One could argue that these behaviors should be the same and they would be right but this functionality was previously coded this way. We are going to fix it in a future issue.

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[X] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that comments were added to code that is not self explanatory
[X] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is correct English and approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://user-images.githubusercontent.com/4073354/212431264-518e3239-caef-42e0-90a5-0fc7fa2a35dd.mp4

Mobile Web - Chrome

https://user-images.githubusercontent.com/4073354/212431292-0b0a06e0-5f77-4896-8b4f-efc0c211c22a.mp4

Mobile Web - Safari
https://user-images.githubusercontent.com/4073354/212431445-48f6c611-1a12-4642-9960-97dbc969f42f.mp4

Desktop
https://user-images.githubusercontent.com/4073354/212431465-f9bec562-ea7c-472e-8179-3297c324c7ec.mp4

iOS
https://user-images.githubusercontent.com/4073354/212431489-b0d21bf6-f147-4ba7-8051-fcfb25ef856f.mp4

Android
https://user-images.githubusercontent.com/4073354/212431317-898710f7-864b-4bc6-83d5-667a689ceb6a.mp4

The code looks good!
I've found an issue when choosing "Add attachment" option: when you dismiss attachment modal, the input is correctly focused, but it's hidden by the keyboard because the screen didn't scroll up.
https://user-images.githubusercontent.com/9059945/214034816-b9a486a3-9b0d-4e66-b40b-750b24c0c72c.mp4
I've tested this on Pixel 4 emulator and it doesn't have this problem, so this is most certainly depends on the flavor of Android.

😭 oh bother... I think I saw some other discussions or maybe code comments about this, @marcaaron do you remember anything about this?
I've definitely noticed that same thing happening on other apps too (not saying we shouldn't fix it) so that might be a somewhat common problem people end up needing to fix, going to see if I can see what the internet has to say about it.

I think I saw some other discussions or maybe code comments about this, @marcaaron do you remember anything about this?

I don't sorry 😞

okay sorry I haven't had a chance to dig back in on this one.
@eVoloshchak which emulator is your video from? I'm going to see about testing this on my phone (pixel 6) to see if it's reproducible there and will report back.

@bondydaa, this video is shot on a physical device, Samsung Note 10 running Android 12
While on Pixel 4 emulator running Android 13 the issue doesn't exist

ah awesome.
My pixel is on Android 13 so I'll test again today with it and see what happens.

okay after getting this branch built on my phone (pixel 6 android 13 I wasn't able to reproduce either.
Wonder if maybe it's an android 12 thing or maybe specific to Samsungs even?

I have a samsung tablet with android 13 that I'm going to see if I can build the app on 🙏

Yep I can't reproduce on my samsung tablet with android 13 so I think it's probably something with android 12 then.
I'm not sure what Android versions we fully support? The hope is we end up cleaning up the behavior on a follow up issue with focus going back and forth and hiding/showing the keyboard so I think we could probably merge this and clean up anything else on a follow up.

Reviewer Checklist

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I included screenshots or videos for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

[ ] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] I verified any copy / text that was added to the app is correct English and approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If any new file was added I verified that:

[ ] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

Mobile Web - Chrome

Mobile Web - Safari

Desktop

iOS

Android

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

🚀 Deployed to staging by https://github.com/bondydaa in version: 1.2.63-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/thienlnam in version: 1.2.63-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


fix static notifiers and static notifier families

I was unable to reproduce the specific errors that were pointed out, but this fixes related errors.

Awesome! Will merge when the CI completes

This should be good to go. It fixes most of the issues observed in #434 with migrating static notifiers, though I haven't been able to reproduce the issue where it doesn't add .notifier. Possibly these changes already fix that, but let's leave the issue open to see if anyone else runs into similar edge cases so we can reproduce.

Thanks!


Add Universal Collector to collect metrics from files or HTTP endpoint

Is your feature request related to a problem? Please describe.
Harvest 1.6 supported running extensions (scripts in any language) to easily collect custom metrics. One example is collecting NFS mount information from ONTAP CLI. But extensions were able to collect any metric from any source. This functionality is missing in Harvest 2.0. Many users already have some scripts to collect custom metrics which they can not easily integrate to Harvest. Currently only solution is to rewrite those scripts in Go an create a Harvest collector.
Describe the solution you'd like
Idea was suggested by @georgmey. Create a collector that will collect metrics in open-metric-format from files or a remote HTTP endpoint. This would be very similar to Prometheus' node exporter.
Describe alternatives you've considered
Implementing the same Extension framework that we had in Harvest 1.6 would require too much work. Moreover, running custom scripts as part of Harvest runtime would raise considerable security concerns (one of the reasons we discontinued 1.6).
Additional context
The general workflow of the Universal collector:

User has a custom script or program that generates metrics and writes them either in a file on exposes on an HTTP endpoint
Universal Collector collects those metrics and parses them into the Matrix.
User can choose to which databases to export these metrics (as per usual).

The Universal collector will be only good for small number of metrics. It would not be a good solution for large chunks of metrics where high-performance is critical.

Seems like this might be a better fit for Telegraf.

Covered by existing plugins


refactor: create TaskFieldRenderer.ts

Description
Extract FieldLayoutDetail and FieldLayouts classes from TaskLineRenderer.ts to a separate file.
Motivation and Context
Separate concerns is TaskLineRenderer.ts.
How has this been tested?
Unit tests.
Types of changes

[x] Refactor (prefix: refactor - non-breaking change which only improves the design or structure of existing code, and making no changes to its external behaviour)

Checklist

[x] My code follows the code style of this project and passes yarn run lint.
[x] My change has adequate Unit Test coverage.

Terms

[x] My contribution follow this project's contributing guide
[x] I agree to follow this project's Code of Conduct

Thanks!


Send templates based on Mailable class

Detailed description
It would be nice to be able to send mails with a template based on the Mailable class. Now it is only possible to send mails based on templates via Notifications.
Context
I don't want to use the more advanced Notifications-stuff, but I would like to use the Mail functionality with Postmark templates.
Possible implementation
It would be nice if a template and data could be provided in build() of a Mailable.
Your environment
Include as many relevant details about the environment you experienced the bug in and how to reproduce it.

Laravel Postmark Version: 2.10.0
Laravel Version: 8.45.0
PHP Version: 8.0.7

I have implemented this way now, works quite nice:
<?php

namespace App\Mail;

use Illuminate\Bus\Queueable;
use Illuminate\Contracts\Queue\ShouldQueue;
use Illuminate\Mail\Mailable;
use Illuminate\Queue\SerializesModels;

class PostmarkTemplate extends Mailable
{
    use Queueable, SerializesModels;

    public $alias;
    public $model;

    /**
     * Create a new message instance.
     *
     * @return void
     */
    public function __construct($alias, $data)
    {
        $this->alias = $alias;
        $this->model = $data;
    }

    /**
     * Build the message.
     *
     * @return $this
     */
    public function build()
    {
        return $this
            ->view('postmark::template');
    }
}

Fixed via #130


Update _index.md

We are proposing a release document change as we are releasing new version of control center tomorrow(17th Aug 2023) at 9:00 CET.

 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.


Error with Workload Identity Federation in when service_account parameter is not specified and using gcloud run deploy

TL;DR
The google-github-actions/auth@v2 action does not correctly authenticate using Workload Identity Federation if the service_account parameter is omitted, even though the documentation suggests it is optional.
Expected behavior
When configuring Workload Identity Federation in google-github-actions/auth@v2, the action should authenticate with the default Service Account in the Workload Identity Pool if service_account is not explicitly provided.
Observed behavior
When the service_account parameter is omitted, the action gcloud run deploy fails with PERMISSION_DENIED errors, specifically missing permissions such as run.services.get, despite the Service Account having the correct roles assigned. Deployment worked only when the service_account parameter was explicitly specified.
Action YAML
name: Deploy ExampleApp to Cloud Run

on:
  workflow_dispatch:
  push:
    branches:
      - development-pipeline

jobs:
  build:
    permissions:
      contents: 'read'
      id-token: 'write'

    runs-on: ubuntu-latest
    environment: ${{ github.ref == 'refs/heads/development' && 'development' || github.ref == 'refs/heads/testing' && 'testing' || github.ref == 'refs/heads/staging' && 'staging' || github.ref == 'refs/heads/main' && 'production' }}

    steps:
      - name: Checkout
        uses: actions/checkout@v4

      - name: Authenticate to GCP
        uses: google-github-actions/auth@v2
        with:
          project_id: 'example-project-id'
          workload_identity_provider: 'projects/PROJECT_NUMBER/locations/global/workloadIdentityPools/runner-pool/providers/github'
          service_account<EMAIL_ADDRESS>
      - name: Set up Cloud SDK
        uses: google-github-actions/setup-gcloud@v2
        with:
          project_id: 'example-project-id'

      - name: Configure Docker for Google Artifact Registry
        run: gcloud auth configure-docker example-region-docker.pkg.dev

      - name: Build and Push Container
        run: |
          docker build -t "example-region-docker.pkg.dev/example-project-id/api-services-test/example-app:${{ github.run_id }}" -f Dockerfile_exampleapp .
          docker push "example-region-docker.pkg.dev/example-project-id/api-services-test/example-app:${{ github.run_id }}"

      - name: 'Deploy to Cloud Run'
        id: 'deploy'
        run: |
          gcloud run deploy example-app \
          --image example-region-docker.pkg.dev/example-project-id/api-services-test/example-app:${{ github.run_id }} \
          --update-secrets 'EXAMPLE_SECRET_ONE=example_secret_one:latest,EXAMPLE_SECRET_TWO=example_secret_two:latest,EXAMPLE_SECRET_THREE=example_secret_three:latest,EXAMPLE_SECRET_FOUR=example_secret_four:latest,/path/to/certificates/example-cert-path=example_cert_secret:latest,/path/to/creds/example-cred-path=example_cred_secret:latest' \
          --update-labels 'managed-by=github-actions,commit-sha=${{ github.run_id }}' \
          --format json \
          --region example-region \
          --ingress internal \
          --min-instances 1 \
          --max-instances 1 \
          --service-account<EMAIL_ADDRESS>\
          --vpc-connector example-connector \
          --vpc-egress private-ranges-only

Log output
ERROR: (gcloud.run.deploy) PERMISSION_DENIED: Permission 'run.services.get' denied on resource 'namespaces/example-project-id/services/example-app' (or resource may not exist). This command is authenticated as principal://iam.googleapis.com/projects/PROJECT_NUMBER/locations/global/workloadIdentityPools/runner-pool/subject/repo:ORG/REPO:environment:false using the credentials in /home/runner/work/REPO/REPO/gha-creds-xyz123.json, specified by the [auth/credential_file_override] property.

Additional information
No response

As documented, if service_account is omitted in the auth, then the GitHub Action uses Direct Workload Identity Federation, meaning you need to grant the federated identity (principal:// or principalSet://) permissions to deploy Cloud Run services.
The --service-account flag specifies the identity the service should run as, it does not specify the identity to use when making the API call to deploy to Cloud Run.

Yes, that's what I was using - just Direct WIF with principalSet, but maybe I'm setting my principalSet wrong. Also I meant specification of service_account in Authenticate to GCP step.

As documented, if service_account is omitted in the auth, then the GitHub Action uses Direct Workload Identity Federation, meaning you need to grant the federated identity (principal:// or principalSet://) permissions to deploy Cloud Run services.
The --service-account flag specifies the identity the service should run as, it does not specify the identity to use when making the API call to deploy to Cloud Run.

In your original issue, you said:

When the service_account parameter is omitted, the action gcloud run deploy fails with PERMISSION_DENIED errors, specifically missing permissions such as run.services.get, despite the Service Account having the correct roles assigned.

When you omit the service_account parameter from the auth step, auth uses Direct WIF, which means the the principalSet:// must have IAM permissions to deploy to Cloud Run (roles/cloudrun.developer).

Here's an example using Direct WIF:
- uses: google-github-actions/auth@v2
  with:
    workload_identity_provider: 'projects/PROJECT_NUMBER/locations/global/workloadIdentityPools/runner-pool/providers/github'

and you'd grant the WIF identity permission to deploy doing something like:
$ gcloud projects add-iam-policy-binding "my-project" \
    --role "roles/cloudrun.developer" \
    --member "principalSet://projects/PROJECT_NUMBER/locations/global/..."

Here's an example using service account impersonation:
- uses: google-github-actions/auth@v2
  with:
    workload_identity_provider: 'projects/PROJECT_NUMBER/locations/global/workloadIdentityPools/runner-pool/providers/github'
    service_account<EMAIL_ADDRESS>
and you'd grant the WIF identity permission to deploy doing something like:
$ gcloud projects add-iam-policy-binding "my-project" \
    --role "roles/cloudrun.developer" \
    --member<EMAIL_ADDRESS>

Both of these are entirely independent of what you specify for the service_account on the Cloud Run service itself, which is the runtime service agent.

I wasn't able to get the Direct WIF example above to work for my project, but I was able to get the following permissions-setting approach to work. Maybe the result of a recent update to gcloud? I'd be happy to submit a PR to the README for this if useful (seems more broadly useful in terms of service-permissions-setting than the secrets example currently in README):
# Uses BigQuery User role as example and grants any member of the github pool ability to interact with that service as a BQ User
gcloud projects add-iam-policy-binding "${PROJECT_ID}" \
    --role "roles/bigquery.user" \
    --member "principalSet://iam.googleapis.com/projects/${PROJECT_NUMBER}/locations/global/workloadIdentityPools/github/*"

@emigre459 is the primary different the ian.googleapis.com?

Yes I think that + the guidance I found elsewhere (specific to my project/risk profile to somd extent) to do /* at the end too. Sounds trivial but took me a lot of trial and error before I got it right.


WKE - High CPU usage

Hello.
wke version 1.02
blink build 98096
build time Sat Aug 5 14:09:29 2023
CPU AMD Rizen 7 1700X
In headless mode, even after opening a web page (quite simple Json), the application continues to use the CPU at the level of 2.5-3%
How to solve this problem? Because your WKE browser from the examples does not consume as much CPU.
I provide an approximate code:

int main(int argc, char **argv) {

  char cCurrentPath[FILENAME_MAX];
  GetCurrentDir(cCurrentPath, sizeof(cCurrentPath));
  char dllpath[MAX_PATH];
  strcpy(dllpath, (const char *) cCurrentPath);
  strcat(dllpath, "\\node.dll");

  size_t cSize = strlen(dllpath) + 1;
  wchar_t *wdllpath = (wchar_t *) malloc(sizeof(wchar_t) * cSize);
  mbstowcs(wdllpath, dllpath, cSize);

  HMODULE hMod = LoadLibraryW(wdllpath);
  FN_wkeInitializeEx wkeInitializeExFunc = (FN_wkeInitializeEx) GetProcAddress(hMod, "wkeInitializeEx");
  wkeInitializeExFunc((wkeSettings *) 0);
    WKE_FOR_EACH_DEFINE_FUNCTION(WKE_GET_PTR_ITERATOR0, WKE_GET_PTR_ITERATOR1, WKE_GET_PTR_ITERATOR2,
                                 WKE_GET_PTR_ITERATOR3,
                                 WKE_GET_PTR_ITERATOR4, WKE_GET_PTR_ITERATOR5, WKE_GET_PTR_ITERATOR6,
                                 WKE_GET_PTR_ITERATOR11);
  free(wdllpath);

  // set proxy
  wkeSetProxy(&proxy)

  wkeWebView webView = wkeCreateWebView();
  wkeSetHeadlessEnabled(webView, true);

  wkeOnLoadUrlBegin(webView, LoadUrlBeginCallback, 0);
  wkeOnLoadUrlEnd(webView, LoadUrlEndCallback, 0);

  wkeSetUserAgent(webView, Task_ua);
  wkeSetLanguage(webView, "en-US");

  wkePostURL(webView, "https://httptest.pp.ua/post?=2562228&t=1728567681378", Task_postData, (size_t) strlen(Task_postData));

  wkeRunMessageLoop();
}

bool LoadUrlBeginCallback(wkeWebView webView, void *param, const utf8 *url, wkeNetJob job) {
    wkeNetHookRequest(job);
    wkeNetSetHTTPHeaderField(job, (const wchar_t *) L"accept", (const wchar_t *) L"application/json, text/javascript, /; q=0.01", false);
    wkeNetSetHTTPHeaderField(job, (const wchar_t *) L"x-requested-with", (const wchar_t *) L"XMLHttpRequest", false);
    return false;
}

void LoadUrlEndCallback(wkeWebView webView, void *param, const utf8 *url, wkeNetJob job, void *buf, int len) {
    printf("%s\n", buf);
}

I'm in the process of upgrading the kernel and the performance will get a big boost then. I'll post a version next week that you can try.

I'm in the process of upgrading the kernel and the performance will get a big boost then. I'll post a version next week that you can try.

Thank you. I will wait.

I'm in the process of upgrading the kernel and the performance will get a big boost then. I'll post a version next week that you can try.

Hello.
Maybe the problem is not in the kernel? Maybe something wrong with the code?


search: index project tags

We have everything in place to index tags but we are not doing it. Do that and make them searchable from the ui. If the change is sound i'll add proper tests.

I'm linking this PR here since it could be interesting for you: https://github.com/rtfd/readthedocs.org/pull/3373
Eric has been working on an upgrade of elasticsearch some time ago but he ran out of time and wasn't able to finish it yet.

Yea, there is a lot of background around ES with why we're not searching tags. Did you confirm that this works properly in a production instance @xrmx?

@ericholscher i've tested this on my local installation with elastic search 1.5.2 from their docker image. I'll add some tests next week.

So i've taken a look at adding tests but I don't find any elastic search related tests :) I'll take a look at https://github.com/vrcmarcos/elasticmock and see if i can come up with something

Looks like I'm not able to mock Elasticsearch client :| Anyway updated the PR to index by tag slugs and not names.

Going to close this. We can address the issue of supporting tags or not in #4267 -- but if we want to support it, this work should go into our GSOC search upgrades.


query topN with join the result not rollup

i use topN query with join，result not rollup，only i use groupBy the result will correct. how can i use top with join?
{
"queryType": "topN",
"dataSource": {
"type": "join",
"left": "xxx",
"right": {
"type": "query",
"query": {
"queryType": "scan",
"dataSource":{
"type":"table",
"name":"yyy"
},
"context": {
"timeout": 180000
},
"intervals": "2020-07-02T00Z/2020-07-03T00Z",
"granularity": "all",
"columns": [
"uid",
"channel"
],
"limit": 100000,
"lenientAggregatorMerge": false,
"descending": false
}
},
"rightPrefix": "r.",
"condition": "uid== "r.uid"",
"joinType": "INNER"
},
"context": {
"timeout": 180000,
"priority":<PHONE_NUMBER>,
"queryId": "wwh-test",
"skipJwoodCache": false,
"weight": 0
},
"intervals": "2020-07-02T00Z/2020-07-03T00Z",
"filter": {
"type": "in",
"dimension": "r.channel",
"values": [
"googleplay",
"appstore"
],
"lowerStrict": false,
"upperStrict": false,
"alphaNumeric": false
},
"aggregations": [
{
"type": "thetaSketch",
"name": "!Theta_counted_uid",
"fieldName": "counted_uid",
"byRow": false,
"inputThetaSketch": false
},
{
"type": "doubleSum",
"name": "item_golds",
"fieldName": "item_golds",
"byRow": false,
"inputThetaSketch": false
}
],
"postAggregations": [
{
"type": "thetaSketchEstimate",
"name": "counted_uid",
"field": {
"type": "fieldAccess",
"fieldName": "!Theta_counted_uid"
}
}
],
"granularity": "all",
"lenientAggregatorMerge": false,
"merge": true,
"dimension": {
"type": "default",
"dimension": "r.channel",
"outputName": "channel"
},
"threshold": 500,
"metric": {
"type": "lexicographic"
},
"descending": false,
"virtualColumns": []
}

Hi @wengwh, based on the description in your issue, it seems possible this will be resolved by #10053, which fixed a bug I ran into when using top-n queries as a part of a join query caused by a flaw in the query engine algorithm selection. This fix will be part of 0.19, which should be released soon.

@clintropolis thanks i will wait for the  new version,.


Is Ryven still in development?

I'm thinking about porting some of the tools I use at work into ryven. It'd take some time to do, but only seems well worth it assuming that development on Ryven continues. What does the next year of development looks like for Ryven? Is this a solo project? Will it be bigger? I've been looking for something flow-based in python for quite some time now.

I have been working a lot on the project over the past 2 years, and I'm slowing down a bit the development from my side to focus on some other things, but I will not leave this project anytime soon.
I think the project is now in a state clean enough to be developed further mainly by the community through contributions. Ryven has become an OS-independent general-purpose platform, cleanly distributed into different python packages, with headless deployment support, and as a Qt-based application is easily integrable into many existing desktop applications (Blender and UE4 plugins as PoCs).
Whether it will receive frequent contributions and improvements I absolutely can't say. The past few months have had a few large contributions, but not so many. But I think a very big part of building this community is developing large thought-out node packages to showcase applications. Therefore, I am always happy to see new people developing nodes for their use cases. Just always remember that not everything that can be written in code has a neat representation as a flow. But when it does, it can be a game changer.

I thought it could be used for simple computations, like those college courses. Just connect the formulae and the work's done. Could be great for teaching.


fixed typo in lane record

In my last PR I used wrong variable name.  I fix it here.
Kind of important because this data is serialized.

sorry, bad branch name!


Cheats Overhaul (2 proposals)

Utilizing cheats in multiplayer could use a bit of a rework. Turn on Classic Sight? Sure, but you can't gain medals and your stats will be frozen, so you can't make any progress... just for using the GoldenEye sight lol, a 'cheat' awarded by completing the first mission of the game.
Proposal 1: The following cheats be exempt from freezing your stats and medals (do not enable 'Cheat Combat Simulator')
under "Fun":
Small Jo (rename to: "Small Player"?)
Small Characters
under "Gameplay":
Jo Shield (rename to: "Spawn With Shield"?)
Super Shield
Enemy Shields
Perfect Darkness
under "Weapons":
Classic Sight
Proposal 2: In CS, include a Cheats page which displays all multiplayer-friendly cheats, with scrolling text at the bottom of the screen denoting whether a respective cheat enables "Cheat Combat Simulator". I may be missing some, but cheats that I have tested to be multiplayer-friendly include:
DK Mode
Small Jo ("Small Player")
Small Characters
Invincible
Jo Shield ("Spawn With Shield")
Super Shield
Enemy Shields
Perfect Darkness
Classic Sight
Unlimited Ammo, No Reloads
I believe fleshing out these key areas will help lay the groundwork for developing additional cheats in the future and welcoming their use in Combat Simulator!

Additionally, I would like to brainstorm a name for "(Two-Handed Weapons)". The description that scolls can just be "Allows you to dual wield any weapons, including two requiring two hands to handle!"
Also, I agree that the "cheat" menu should be accessible from the MP menu. So annoying to have to get out of the MP section, into the options area and back to MP. I also agree with Atari-Dude that some cheats shouldn't impede your progress.

I recall that Meowsy used a better name for "(Two-Handed Weapons)" in his PCDX mod


circleci: Resolve node installation error in CircleCI.

The Git clones in 'install-node' were breaking because
we use CircleCI's special step, checkout, to clone source
code. It configures Git to use SSH while cloning.
More info:
https://circleci.com/docs/2.0/configuration-reference/#checkout
The error no longer occurs.
Codecov has released the new version which fixes the find error.

Testing plan:
GIFs or screenshots:

@ganpa3 why did you squash the 2 commits that were doing different thing into 1? The codecov commit should be separate.

@ganpa3 why did you squash the 2 commits that were doing different thing into 1? The codecov commit should be separate.

@rht I didn't squash them. I thought it's a simple change so I just made 1 commit.

@rht I didn't squash them. I thought it's a simple change so I just made 1 commit.

@ganpa3  it's simple to you because you know the context around the codecov changes at this moment. People browsing the commits in the future will get confused.

@ganpa3  it's simple to you because you know the context around the codecov changes at this moment. People browsing the commits in the future will get confused.

Ohk. Will change that. However, what should I do with the change regarding node since the error doesn't seem to occur now? Should I just remove the git config hack?

Ohk. Will change that. However, what should I do with the change regarding node since the error doesn't seem to occur now? Should I just remove the git config hack?

Yes, remove the .gitconfig hack in a separate commit, but make sure to explain below the commit message that we no longer observe the nvm git clone error, etc, etc.

Yes, remove the .gitconfig hack in a separate commit, but make sure to explain below the commit message that we no longer observe the nvm git clone error, etc, etc.

@rht Updated the PR!

@rht Updated the PR!

Small nit: the prefix of the commit message already says "circleci" so it is redundant to say 'in CircleCI' at the end.

Small nit: the prefix of the commit message already says "circleci" so it is redundant to say 'in CircleCI' at the end.

@rht Thanks for the help. Done.

@rht Thanks for the help. Done.

It appears CircleCI didn't run on this PR for some reason.  I've rebased to retrigger CI, which should help us see if this is indeed correct.
Also, can you update the commit message for the first commit to explain clearly whether this is just a code cleanup, or whether that rm operation was causing other problems with current CircleCI?

It appears CircleCI didn't run on this PR for some reason.  I've rebased to retrigger CI, which should help us see if this is indeed correct.
Also, can you update the commit message for the first commit to explain clearly whether this is just a code cleanup, or whether that rm operation was causing other problems with current CircleCI?

@timabbott Done.

@timabbott Done.

I have a hypothesis: what if CircleCI no longer forces git clone to use ssh and that's why it works now. This can be tested by cat ~/.gitconfig. If CircleCI decides to force ssh again, we might have to rm .gitconfig again in the future.

I have a hypothesis: what if CircleCI no longer forces git clone to use ssh and that's why it works now. This can be tested by cat ~/.gitconfig. If CircleCI decides to force ssh again, we might have to rm .gitconfig again in the future.

CircleCI still didn't run -- only GitHub actions did.  So I'm concerned that this branch may break CircleCI.

CircleCI still didn't run -- only GitHub actions did.  So I'm concerned that this branch may break CircleCI.

@rht CircleCI forces git to use ssh and it still runs successfully.
Commit: https://github.com/ganpa3/zulip/commit/46b8bec5380bcb473acc9c56bdea69e17c35cf0d

@rht CircleCI forces git to use ssh and it still runs successfully.
Commit: https://github.com/ganpa3/zulip/commit/46b8bec5380bcb473acc9c56bdea69e17c35cf0d

@rht Any ideas on why CircleCI isn't running? It is running on my setup. I pushed again but it still didn't work. I also made a new branch (and PR), but it still didn't run.

@rht Any ideas on why CircleCI isn't running? It is running on my setup. I pushed again but it still didn't work. I also made a new branch (and PR), but it still didn't run.

CircleCI wasn't running on https://github.com/zulip/zulip/pull/17112, and this PR was made an hour ago. Probably just it being down or something.

CircleCI wasn't running on https://github.com/zulip/zulip/pull/17112, and this PR was made an hour ago. Probably just it being down or something.

@timabbott Ping for a review!

@timabbott Ping for a review!

This is ultimately made obsolete by https://github.com/zulip/zulip/pull/17611, please check that we don't need similar work in the GitHub Actions copy.  But merging in case we ever end up resurrecting the CircleCI stuff.

@amanagr FYI -- this will cause some minor merge conflicts in deleted files.


Time Zone not taken from secrets.yaml in Docker

Describe the bug
The time zone settintgs do not seem to be taken from the secrets.yaml when using the docker image from docker hub. If I manually add it to the docker parameters, it is working.
To Reproduce
Try it with only setting it in the yaml and then when also placing it into the docker parameters.
Expected behavior
Take timezone from yaml.
Home Assistant installation type

Home Assistant Core on Synology Docker

Your hardware

OS: Synology

EMHASS installation type

Docker from hub

Thanks for reporting this. Could you please tell me what version of emhass you are using?

And just to be sure, are you passing the secrets data yaml file when launching the container like this?:
docker run -it --restart always -p 5000:5000 -e "LOCAL_COSTFUN=profit" -v $(pwd)/config_emhass.yaml:/app/config_emhass.yaml -v $(pwd)/secrets_emhass.yaml:/app/secrets_emhass.yaml --name DockerEMHASS <REPOSITORY:TAG>

I'm using 0.3.25.
The containers are started via the Synology GUI but the files are mapped, yes. Also the deferrable sensor values from EMHASS to HA are correctly transferred so I gues this tells us that it reads the yaml files.

I cannot see in the code how if present in the yaml file the time zone would not be passed later to the computations.
The only thing I can see for now is the presence of time_zone: Europe/Paris respecting this format. Did you double check that?

time_zone: Europe/Vienna is my setting. I just installed the newest version of EMHASS and now do not have the problem, also after removing the TZ value from the parameters. So I guess something went wrong while running the previous one.

Ups sorry, just reloaded the page and time is wrong again. So the issue is still there.

Ok, there has been some modifications lately. Please upgrade to the latest v0.3.34 and see if there any improvements. You should be able to pull the latest image very soon. Just wait for v0.3.34 to be availabe on dockerhub

Hi, did you had the time to test this with the latest version?

Feel free to reopen if this wasn't solved with latest version.


Fixes Comment Deletion by Moderators

Moderators should now be able to delete comments using the button below:

Closes #531
Closes #442

Which roles are allowed to remove comments?
When testing, admins could remove comments, but they could not flag (I assume that's ok).
Editors/Moderators cannot remove comments, but can report. (Mods should be allowed to remove).
No-one can see how many users have reported a given comment (at least admins used to be able to see that?)

Can we get on a call to discuss this?

I created a user which I added to the "Moderators" group, and in that group I checked "Can moderate comments" and "Object permissions - comments - delete". The latter enabled that user to delete comments.
However, even though I checked "Can moderate comments", the moderator tag doesn't appear next to that user's posts, and the "X users reported this comment" doesn't show either. I see the logic in comments/templates/django_comments_xtd/comment_tree.html  that should enable this, do you know why this isn't working?


Installing

I'm running the command to add the plugin into my project
cordova plugin add cordova-instagram-plugin --save
The plugin installation fails when it's trying to pull in the depend project: Visigo/CordovaClipboard
with error:
Error: Failed to fetch plugin https://github.com/Visigo/CordovaClipboard.git via registry.
I have tried this a number of times with the same result.
I'm using Cordova 7.0.1.
Would it be possible to address the CordovaClipboard repository so it can be loaded?
Regards,
Vince

@vinceharmon are you still facing this problem? If not please close the issue

Seems to be related to the clipboard plugin which I just changed to cordova-clipboard, see #104
This should be fixed. Closing.


add rollover logic for default incremeter in optimistic_lock

As discussed in the previous issue, this PR is for avoiding an exception when the lock version exceed the PostgresSQL and MySQL signed integer size limit.

Beautiful. Can you please add a test?

I added the test. Do you think it's sufficient?
I was thinking of adding like an integration test for actual DB operation to assert not exception raise but then realize there should be no DB adapter library required in Ecto repo itself. So I didn't add the integration test.

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


Testing github actions - do not review this PR

…ted unit and integration tests for LocalUriInput. Refactored various other classes and components to support LocalUriInput.
Signed-off-by: AWSHurneyt<EMAIL_ADDRESS>Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #8 (e05e1b9) into main (5b7542b) will increase coverage by 6.08%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main       #8      +/-   ##
==========================================
+ Coverage   50.81%   56.89%   +6.08%
==========================================
  Files         200      197       -3
  Lines        4668     4598      -70
  Branches      594      645      +51
==========================================
+ Hits         2372     2616     +244
+ Misses       2294     1980     -314
  Partials        2        2

Impacted Files
Coverage Δ

...eMonitor/containers/DefineMonitor/DefineMonitor.js
70.61% <0.00%> (-16.72%)
:arrow_down:

...omponents/TriggerEmptyPrompt/TriggerEmptyPrompt.js
50.00% <0.00%> (-16.67%)
:arrow_down:

...mponents/MonitorOverview/utils/getOverviewStats.js
71.42% <0.00%> (-3.58%)
:arrow_down:

...eTrigger/containers/DefineTrigger/DefineTrigger.js
14.92% <0.00%> (-1.75%)
:arrow_down:

...gin/public/pages/Dashboard/containers/Dashboard.js
61.01% <0.00%> (-1.49%)
:arrow_down:

...erting-dashboards-plugin/public/utils/constants.js
100.00% <0.00%> (ø)

...-plugin/public/pages/Dashboard/utils/tableUtils.js
52.17% <0.00%> (ø)

...ateTrigger/components/TriggerQuery/TriggerQuery.js
72.00% <0.00%> (ø)

...onitor/containers/CreateMonitor/utils/constants.js
100.00% <0.00%> (ø)

.../components/MonitorDefinition/MonitorDefinition.js
61.53% <0.00%> (ø)

... and 31 more

Continue to review full report at Codecov.

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


build: export ROOK_IMAGE env variable

Update script to export ROOK_IMAGE env variable so that it can be updated with custom values during the build
Checklist:

[ ] Commit Message Formatting: Commit titles and messages follow guidelines in the developer guide.
[ ] Reviewed the developer guide on Submitting a Pull Request
[ ] Pending release notes updated with breaking and/or notable changes for the next minor release.
[ ] Documentation has been updated, if necessary.
[ ] Unit tests have been added, if necessary.
[ ] Integration tests have been added, if necessary.

/retest

/hold

/unhold

Based on this comment https://github.com/red-hat-storage/rook/pull/599#issuecomment-2014842480 seems like we are good with the PR

@sp98 can you please review this PR?
TIA :)


istio-ingressgateway xdsproxy always timeout

(NOTE: This is used to report product bugs:
To report a security vulnerability, please visit https://istio.io/about/security-vulnerabilities
To ask questions about how to use Istio, please visit https://discuss.istio.io)
Bug description
There are more than two thousand services in my cluster.
the istio-ingressgateway log always print like:
2021-03-01T06:20:17.354852Z	info	xdsproxy	Envoy ADS stream established
2021-03-01T06:20:17.354991Z	info	xdsproxy	connecting to upstream XDS server: istiod-1-8-3.istio-system.svc:15012
2021-03-01T06:20:40.597597Z	error	xdsproxy	upstream send error for type url type.googleapis.com/envoy.config.listener.v3.Listener: context deadline exceeded
2021-03-01T06:20:40.597693Z	info	xdsproxy	disconnected from XDS server: istiod-1-8-3.istio-system.svc:15012
2021-03-01T06:20:40.598036Z	warning	envoy config	StreamAggregatedResources gRPC config stream closed: 2, context deadline exceeded
2021-03-01T06:20:40.956026Z	info	xdsproxy	Envoy ADS stream established
2021-03-01T06:20:40.956181Z	info	xdsproxy	connecting to upstream XDS server: istiod-1-8-3.istio-system.svc:15012
2021-03-01T06:24:52.747024Z	error	xdsproxy	upstream send error for type url type.googleapis.com/envoy.config.endpoint.v3.ClusterLoadAssignment: context deadline exceeded
2021-03-01T06:24:52.747086Z	info	xdsproxy	disconnected from XDS server: istiod-1-8-3.istio-system.svc:15012
2021-03-01T06:24:52.747474Z	warning	envoy config	StreamAggregatedResources gRPC config stream closed: 2, context deadline exceeded
2021-03-01T06:24:53.148384Z	info	xdsproxy	Envoy ADS stream established
2021-03-01T06:24:53.148563Z	info	xdsproxy	connecting to upstream XDS server: istiod-1-8-3.istio-system.svc:15012

And all gateways proxy-status not Synced.
istio-ingressgateway-j92dr.istio-system                                       STALE (Never Acknowledged)     STALE (Never Acknowledged)     STALE (Never Acknowledged)     STALE (Never Acknowledged)     istiod-1-8-3-96b4c8dc-7txlx       1.8.3
istio-ingressgateway-jl4zh.istio-system                                       STALE (Never Acknowledged)     STALE (Never Acknowledged)     STALE (Never Acknowledged)     STALE (Never Acknowledged)     istiod-1-8-3-78575d6cc5-7pxgd     1.8.3
istio-ingressgateway-n6v64.istio-system                                       STALE (Never Acknowledged)     STALE                          STALE (Never Acknowledged)     STALE (Never Acknowledged)     istiod-1-8-3-96b4c8dc-7txlx       1.8.3

Affected product area (please put an X in all that apply)
[ ] Docs
[ ] Installation
[X] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
[ ] Upgrade
Affected features (please put an X in all that apply)
[ ] Multi Cluster
[ ] Virtual Machine
[ ] Multi Control Plane
Expected behavior
Not timeout
Steps to reproduce the bug
Version (include the output of istioctl version --remote and kubectl version --short and helm version --short if you used Helm)
1.8.3
How was Istio installed?
istio-operator
Environment where the bug was observed (cloud vendor, OS, etc)
Additionally, please consider running istioctl bug-report and attach the generated cluster-state tarball to this issue.
Refer cluster state archive for more details.

https://github.com/istio/istio/blob/d994fcbf5495ddbedb42db732a34796bd0722c5b/pkg/istio-agent/xds_proxy.go#L66
Maybe we can provide a way to overrite the default timeout value.
Such as setting a environment value named XDS_SEND_TIMEOUT ?


Codec Issue

Peer connect is established between 2 any native or browser but when 1 Android device sent streaming to 2 android device then getting "Unable to create an H.264 decoder fallback. Decoding of this stream will be broken." error. and 1 android device to browser is working perfectly.
As per my knowledge i think there is something codec issue with android device.
Peerjs Version : 0.3.14 & 0.3.15

i don't understand your problem, peerjs have nothing to H.264


Remove double init() call CompositeUrlRule.php

Q
A

Is bugfix?
yes

New feature?
no

Breaks BC?
no

Tests pass?
yes

Fixed issues
-

Thank you for spotting this!


resolves #30 set empty status text

resolves #30

Actually, both response and responseText are empty when an error occurred in asynchronous mode...

Since we are not running in the browser, if we return an empty response and responseText the caller won't know why the request failed. I think it's a fair trade-off to set the stringify error in responseText.
Alternatively, we could add a non-standard field with the error. For instance, onerror is called with a ProgressEvent. We could add an error field on this object to pass the error.

I've decided to pass the error as a parameter of the onerror callback function, see #34.


Scoped CSS

How do you think about Scoped CSS?
Most popular reason why the programmers hate CSS is that it doesn't have scope. Only Firefox supports this feature, and other browsers will unlikely support it. Web Components may be an answer to this problem in the future. (In the React context, React Style is hot, but I can't beleive that web-designers prefer such a way)
I'm not sure this is a right way or not, but it seems not so hard to simulate "Scoped CSS" in Riot.js. I think this will be a potential killer feature for designers. But I don't want to bring more complexity to Riot.js, too.
<my-tag>

{ opts.title }

</my-tag>

Style would be compiled like below.
my-tag { display: block; border: 2px }
my-tag h3 { color: blue }

Any feedback is welcome. Thanks.

Ractive.js does it, if you need the feature right now:
https://github.com/ractivejs/ractive/issues/452
..and we already touched the surface a bit here:
https://github.com/muut/riotjs/issues/176

@pakastin Cool. Thanks for letting me know. They discussed about it 1-year ago!
I've found them use GUID to specify the element but it may not be necessary in Riot.js case. Custom-tags would play the same role, wouldn't it?

Reactive.js way

You could just write my-tag {display: block; border: 2px} like you wrote yourself..
With using CSS preprocessor you can nest it hierarchially:
my-tag {
  h3 {
    color: blue;
  }
}

In Ractive.js the dynamic CSS is the thing which I think makes the difference why you couldn't use CSS.

Exactly :)
...but my-tag in css seems little bit redundant. I'm getting fed up with writing my-tag on each row in my tag file. (I agree that a preprocessor is one of the solutions)
Anyway, I like the minimalistic approach of Riot.js.

Actually with preprocessors (at least with stylus) you could even:
index.styl:
my-tag
  @import 'my-tag'

my-tag.styl:
&
  display: block
  border: 2px

h3
  color: blue

Thanks. I personally don't use preprocessors.
I'd rather focus on the advantage of Riot.js that everything can be put in .tag file together, as you said.

there is also convenience in putting the CSS inside components

Here're some examples for comparison. I'm guessing which does seem cleanest...?

I think scoped CSS makes perfect sense inside custom tags, especially since it's an emerging standard.
But I feel it is completely up to the CSS pre-processor/autoprefixer to deal with the contents of the style tags and there should be no Riot specific trickery in between. So the pre-processor of choice would need to support scope in some ways. Riot would only pass the scope name for it.

@tipiirai Thanks. I like your idea.

How about that this (above) means just shorthand of this (below) ?

I think the scoped or type="scoped-css" are unnecessary. The contents of style tag is always extracted and given for the optional pre-processor with a tag name param and the pre-processor can do what it pleases. The type param should be reserved for the processor name: "less", "stylus", "sass" etc..

Oh, I see.
Like AltJS, I think it'll be easy to understand the concept if Riot.js has official supports for some (ex. LESS, Sass and scoped-css). At least I'd like use scoped-css in browser-compilation. How do you think?
Below is just an example spec what I was guessing yesterday.
 equal to
 equal to

css: do nothing like now
scoped-css: add tag name as prefix for each class
less,sass: official support like CoffeeScript and LiveScript
other-custom-something: like custom parser function

which compolend would do the pre-processing in case of scoped-css?

Maybe, we need no component. Just one liner like below.
css.replace(/(?:^|\})?\s*([^\{\},]+)\s*(\{|,)/g, tagName + ' $1 $2')

Yeah... could work. I'm just bouncing between making this a plugin or including to the core.
The plugin system isn't ready yet, but I'm going to make one. Something like React mixins but better.

Sounds nice!
I like your original idea that Riot.js should keep itself simple and small. So I hope all compilers will take away from the Riot.js and be pluggable, except riotjs compiler and hopefully scoped-css (but I don't stick it).

scoped css looks really nice! As a user, I really hope I can use it.
Looking forward to the plugin system.

I'm actually beginning to like this approach more and more. CSS is a core part of HTML components and would be nice to have all the relevant stuff wrapped together. This seems like a core feature.
Btw. I quickly tried your regexp and it failed with following CSS
h5 { color: blue } #id { font-style: normal }

So happy to hear that!
Here's a little refined rexexp and the test code for it. Pls try it ;)
var CSS_SELECTOR = /(^|\}|,)\s*([^\{\},]+)\s*(?=\{|,)/g

function addScope (tag, src) {
  return src.replace(CSS_SELECTOR, function (m, p1, p2) {
    if ('@'      == p2.substr(0, 1)) return p1 + '\n' + p2
    if (':scope' == p2.substr(0, 6)) return p1 + '\n' + p2.replace(':scope', tag)
                                     return p1 + '\n' + tag + ' ' + p2
  }).trim()
}

var src = "h5 { color: blue } #id { font-style: normal }\n"
        + "section p.test:hover { display: block }\n"
        + "h3, a[href=#] {\n"
        + "  color: red;\n"
        + "  background: white;\n"
        + "}\n"
        + ":scope { font-size: 10px }\n"
        + ":scope > ul { padding:0 }\n"
        + "@font-face { font-family: 'FontAwesome' }\n"

console.log(addScope('sample', src))

Expected output:
sample h5 { color: blue }
sample #id { font-style: normal }
sample section p.test:hover { display: block }
sample h3,
sample a[href=#] {
  color: red;
  background: white;
}
sample { font-size: 10px }
sample > ul { padding:0 }
@font-face { font-family: 'FontAwesome' }

How about sel[0] == '@' and can /^:scope/.test(sel) test be removed?

I'm happy to merge a pull request regarding to this functionality. Please include tests as well.
Awesome job!

Sure! I've sent the PR right now ;)

@tipiirai I chose the  pattern in my PR.
I agree that just

Hello world!

</my-tag>

@cognitom obviously you are working alone and not in a team :)

Not so match. It seems a preference between vertical specialization or horizontal division of labor, kind of :D

@cognitom great to hear about your positive experiences around scoped CSS. Also like your standard based proposal about the script / css inclusion. Perhaps an external tool (for gulp / grunt etc..) rather than part of the core compiler?
Also think that this decision should be left for the user. Seems inclusion is useful on bigger tags and inlined style is nicer on smaller tags.

Does this patch add sass support?
Would the api be:
<custom-tag>

</custom-tag>

@btakita , this is supported in the current version of the compiler (sure not for the browser), is the indentation of the CSS in your example ok?

This is great!
I love scoped tag styles, but prefer to have my .css in separate files next to the tag and separately compiled via postCSS and webpack.
The gulp example @cognitom posted is very helpful, but I'm wondering if anyone has a good example of riotjs scoped external CSS via webpack? I can't seem to find a way to extract CSS within the


[Feature]: Change order of label / value in tooltips

What problem does this feature solve?

The current tooltip can be difficult to read. When reading from left to right, your eyes see the value before you know what the value is referring to. So your brain goes "heres a value, what is it for? It's for active subscriptions. Ok, so wait, what was the value again?" Repeat. vs "active subscribers is x"
What does the proposed API look like?
Introduce a prop called tooltip to allow customisation, and use labelFirst: boolean to switch the ordering.
<LineChart
        className="mt-6"
        autoMinValue={true}
        data={tremorData}
        index="date"
        categories={[
          'Active subscribers',
          'Active subscriptions',
          'Active products (subscription lines)',
        ]}
        curveType="linear"
        tooltip={{ labelFirst: true }}
      />

Hi @njbmartin , I would like to work on this issue.

Heya! Thanks for the thorough description. This sound like a reasonable change to me, will take this to the team.

@MuhammadAakash feel free to take this. We think its better to not add a new prop but just to reverse the values as suggested by @njbmartin.

Sure @mitrotasios . Thanks, will start working on it.

@mitrotasios Can you please confirm whether I should move the dot, which is currently positioned on the left side of the number, to the right side after reversing the position of the label and value? Or should I leave it as it is right now?


New package: Combina v0.0.1

Registering package: Combina
Repository: https://github.com/aldma/Combina.jl
Created by: @aldma
Version: v0.0.1
Commit: c2dfcae7ace17035e554a37468c48691ae2cb9cd
Reviewed by: @aldma
Reference: https://github.com/aldma/Combina.jl/issues/1#issuecomment-2376003141
Description: Solving binary approximation problems in Julia

Hi, congrats on the new package!
While I understand your wish to stay close to the Python name, the Python package itself is not widely popular, so I wouls suggest going with something like BinaryApproximationProblems.jl.

Hi, thanks @gdalle for the quick feedback!
The name CombinatorialIntegralApproximations.jl should be different enough from other packages and clear to the intended users.

superseded by #116034


Newsboat failed to wrap lines in the content

Newsboat version (copy the output of newsboat -v or the first line of git show):
Newsboat r2.35
System: Linux 6.8.9-zen1-2-zen (x86_64)
Compiler: g++ 13.2.1 20240417
ncurses: ncurses 6.4.20230520 (compiled with 6.4)
libcurl: libcurl/8.7.1 OpenSSL/3.3.0 zlib/1.3.1 brotli/1.1.0 zstd/1.5.5 libidn2/2.3.7 libpsl/0.21.5 libssh2/1.11.0 nghttp2/1.61.0 nghttp3/1.2.0 (compiled with 8.7.1)
SQLite: 3.45.3 (compiled with 3.45.3)
libxml2: compiled with 2.12.6
Config file (copy from ~/.newsboat/config or ~/.config/newsboat/config):
# themes
#include ~/.config/newsboat/dark

auto-reload		yes
reload-time		30
text-width		0

browser			firefox

urls-source		"ttrss"

ttrss-url		""
ttrss-login		""
ttrss-passwordeval	""

ttrss-flag-star 	"s"
ttrss-flag-publish 	"p"

# the macros
macro v set browser "mpv %u &"; open-in-browser-noninteractively; set browser firefox

Steps to reproduce the issue:

set text-width to any value

Tiny Tiny RSS subscribes to this newsfeed: https://us21.campaign-archive.com/feed?u=80c714e7abe5eab0ae57c8693&id=eda3a2b209

Newsboat opens any article in the subscribed feed

Other info you think is relevant:
Newsboat failed to wrap lines in the content (lines spilled outside of termimal width), but the header portion wraps just fine
One of the articles' HTML: https://mailchi.mp/67025a37335c/apple-is-sipping-pia-coladas-while-others-race-12672819

Thanks, I can reproduce this issue locally, even without using Tiny Tiny RSS.
I didn't really look into it yet, but my guess is that it has something to do with the many nested tables:

Possibly relevant part of the code:
https://github.com/newsboat/newsboat/blob/e98b1ad53816377e3a64499a94294a9dd09f39ad/src/htmlrenderer.cpp#L1210-L1296


[WIP] Add README

Rendered preview: https://github.com/cloudfour/test-mule/blob/readme/README.md
Still TODO in README:

Device Emulation
Debug/leave browser open
Switching to headed browser

@emersonthis feel free to take a look / review


add morf_id to all status emails

This should include the morf_id attribute of the MorfJobConfig object, ideally in the subject and body of the email.

resolved in most recent commit/merge of doi branch


Print more information when terraform fails

Run terraform output when terraform apply fails
fix some other fmt issues

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: mkumatag
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [mkumatag]

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

/cc @Rajalakshmi-Girish


BUG: FILES Preview <IP_ADDRESS> Spanish translation.

The translation and its function are incorrect. Tags can be assigned to both files and folders. The literal translation "Editar etiquetas" (Edit tags) is the correct and functional translation.
In Crowdin, it cannot be modified since its translation has been approved.

Thanks for the report.

In Crowdin, it cannot be modified since its translation has been approved.

Can you add a comment a Crowdin? We're tracking translation issues there.

In Crowdin, it cannot be modified since its translation has been approved.

Can you add a comment on Crowdin? We're tracking translation issues there.

Do you mean an overall assessment of Crowdin? If you are referring to a comment on this particular translation, I already made a comment in Crowdin two months ago.

@dracp can you send a link to the comment and I'll un-approve the current translation?

@yaira2 https://crowdin.com/translate/files-app/all/en-es?filter=basic&value=0#q=editar etiquetas

@dracp can you send a link to the comment and I'll un-approve the current translation?

https://crowdin.com/translate/files-app/all/en-es?filter=basic&value=0#q=editar etiquetas

I also made a request to correct a wrong French translation of the word "Formater..." because it is a verb and it is translated as "Format..." in English.
https://crowdin.com/translate/files-app/34/en-fr?filter=basic&value=0#7767

FILES <IP_ADDRESS>
Edit Tags (Editar etiquetas - FOLDER/FILE). Correct


Login Use case (For-12)

Half complete, I have implemented the log in use case. Not yet finish the login View, but close to finish

Hey @LL333111 I see that u made some changes to the Entity folder to make ur code work (which is good!), but izabelle is also going to make changes to that so there will be merge conflicts. So for this PR can u unstage the entity files u changed (keep ur other code the same, if no changes need to be made from the reviews). I'm going to leave it to u to figure out how to unstage files from git, some google search would help :)

This PR is technically closed, it's kept here for reference


Remove batching functionality from cell and pixel mask generation

What is the purpose of this PR?
Addresses part 2 of #718.
How did you implement your changes
Change the iteration in generate_and_save_{pixel/cell}_cluster_masks to a per-FOV basis, which requires deprecating the batch_size parameter.
generate_{pixel/cell}_cluster_mask will also support just a single fov string identifying which FOV to process (as opposed to a batch of FOVs).
Because save_fov_images and label_cells_by_cluster are used elsewhere in the repo, it's best to continue requiring a fovs list for these, and instead pass in a list with a single FOV from generate_{pixel/cell}_cluster_mask.

Which other functions use save_fov_images and label_cells_by_cluster? Are we going to end up removing the batching functionality from them in the future? Or is it really worth keeping them as is?

@ngreenwald label_cells_by_cluster is called in both the post-clustering notebook (via call to create_mantis_project) and the neighborhood analysis notebook, both of which pass in a list of FOVs. save_fov_images seems just limited to create_mantis_project, which also passes in a list of FOVs.
I thought about looping over each FOV in the create_mantis_project function, but this doesn't resolve the issue in the neighborhood analysis notebook. I have a few ideas about how to condense this to allow save_fov_images and create_mantis_project to work on just single FOVs, however it requires some refactoring to be done in the neighborhood analysis notebook itself. Not sure if this is within the scope of this PR, we can address that in a separate PR if need be.


x86 recover_cfg.py fails with TypeError: unsupported operand type(s)

I have just finished installing the latest version of barf and was trying the example scripts.  When I ran the recover_cfg.py script (all other x86 scripts worked) barf would error out with:
$> cat barf.log
2018-04-12 10:26:45,874: barf.barf:INFO: [+] BARF: Initializing...
2018-04-12 10:26:45,879: smtlibv2:DEBUG: >(set-option :global-decls false)
2018-04-12 10:26:45,880: smtlibv2:DEBUG: >(set-logic QF_AUFBV)
2018-04-12 10:26:45,880: smtlibv2:DEBUG: >(declare-fun MEM_0 () (Array (_ BitVec 64) (_ BitVec 8)))
2018-04-12 10:26:45,884: smtlibv2:DEBUG: >(set-option :global-decls false)
2018-04-12 10:26:45,884: smtlibv2:DEBUG: >(set-logic QF_AUFBV)
2018-04-12 10:26:45,884: smtlibv2:DEBUG: >(declare-fun MEM_0 () (Array (_ BitVec 64) (_ BitVec 8)))
2018-04-12 10:26:45,923: barf.analysis.basicblock.basicblock:ERROR: Failed to save basic block graph: ~/barf-project/examples/bin/x86/branch4_cfg (dot)
Traceback (most recent call last):
  File "~/barf/local/lib/python2.7/site-packages/barf-0.2.1-py2.7.egg/barf/analysis/basicblock/basicblock.py", line 415, in save
dot_graph.write("{}.{}".format(filename, format), format=format)
  File "build/bdist.linux-x86_64/egg/pydot.py", line 1756, in write
s = self.create(prog, format, encoding=encoding)
  File "build/bdist.linux-x86_64/egg/pydot.py", line 1836, in create
self.write(tmp_name, encoding=encoding)
  File "build/bdist.linux-x86_64/egg/pydot.py", line 1750, in write
s = self.to_string()
  File "build/bdist.linux-x86_64/egg/pydot.py", line 1492, in to_string
graph.append( node.to_string()+'\n' )
  File "build/bdist.linux-x86_64/egg/pydot.py", line 623, in to_string
node += ' [' + node_attr + ']'
TypeError: unsupported operand type(s) for +=: 'long' and 'str'

I am trying to run this on Ubuntu 16.04:
Linux devlinux 4.4.0-116-generic #140-Ubuntu SMP Mon Feb 12 21:23:04 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Hi! I've tried to reproduce the error you found but I couldn't. Did you install BARF via pip or download the latest release and install it through python setup.py?

Since i dont remeber right now and i am not at home to double check, i will get back to you later tonight
Sent via the Samsung Galaxy S8, an AT&T 4G LTE smartphone
-------- Original message --------From: cnheitman<EMAIL_ADDRESS>Date: 4/13/18  15:57  (GMT-05:00) To: programa-stic/barf-project<EMAIL_ADDRESS>Cc: OhioDuckFarmer<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [programa-stic/barf-project] x86 recover_cfg.py fails with
  TypeError: unsupported operand type(s) (#48)
Hi! I've tried to reproduce the error you found but I couldn't. Did you install BARF via pip or download the latest release and install it through python setup.py?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/programa-stic/barf-project","title":"programa-stic/barf-project","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/programa-stic/barf-project"}},"updates":{"snippets":[{"icon":"PERSON","message":"@cnheitman in #48: Hi! I've tried to reproduce the error you found but I couldn't. Did you install BARF via pip or download the latest release and install it through python setup.py?"}],"action":{"name":"View Issue","url":"https://github.com/programa-stic/barf-project/issues/48#issuecomment-381245277"}}}

Looks like i used pip.  I will uninstall and will try the latest build via setup.py
Sent via the Samsung Galaxy S8, an AT&T 4G LTE smartphone
-------- Original message --------From: cnheitman<EMAIL_ADDRESS>Date: 4/13/18  15:57  (GMT-05:00) To: programa-stic/barf-project<EMAIL_ADDRESS>Cc: OhioDuckFarmer<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [programa-stic/barf-project] x86 recover_cfg.py fails with
  TypeError: unsupported operand type(s) (#48)
Hi! I've tried to reproduce the error you found but I couldn't. Did you install BARF via pip or download the latest release and install it through python setup.py?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/programa-stic/barf-project","title":"programa-stic/barf-project","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/programa-stic/barf-project"}},"updates":{"snippets":[{"icon":"PERSON","message":"@cnheitman in #48: Hi! I've tried to reproduce the error you found but I couldn't. Did you install BARF via pip or download the latest release and install it through python setup.py?"}],"action":{"name":"View Issue","url":"https://github.com/programa-stic/barf-project/issues/48#issuecomment-381245277"}}}


fix: alert mobile close should works

PR
PR Checklist
Please check if your PR fulfills the following requirements:

[ ] The commit message follows our Commit Message Guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

@all-contributors please add @LinboLen for code

@LinboLen 恭喜你成为OpenTiny 2023年度开源新星，请添加下OpenTiny小助手微信：opentiny-official，如已添加，请发一条信息给小助手，谢谢！


Terraform logo

I was about to send a pull request, but the repo's readme instructs users to submit logos through the issue tracker instead, so... uh, okay.
<svg version="1.1" xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" x="0px" y="0px" width="456px" height="512px" viewBox="0 0 456 512">
	<path d="M155.4604187,88.826889l146.8236389,83.8918304v164.7937927l-146.8236389-83.8920593V88.826889z M310.076416,336.7729492l145.546051-83.642334V89.4021683l-145.546051,83.0672989V336.7729492z M0,164.7935638l146.8236237,83.8920746V83.8920593L0,0V164.7935638z M155.4604187,428.1079407L302.2840576,512V346.6309204l-146.8236389-83.8918457V428.1079407z"/>
</svg>

That's already optimised, unless you'd rather reduce the floating point length. In which case, hey, go for it.

Added 081a89a93b6af2bba5cf4eeace7a5c430785c003
Thanks

> tabs to spaces in an SVG file
> lol

Uh, okay.

#zuera


Mendix and IBM Watson integration

Hi,
I am having a tough time integrating the IBM Watson chatbot on my Mendix landing page. Is there any way I can get some help with this?. Thanks.

Hi Andrew, the best thing you can do to get help outside of referring to the documentation is to post a question on the Mendix Forum: https://forum.mendixcloud.com/index4.html. The Mendix community is very active there and is sure to help you if you explain your scenario and what you are trying to accomplish.
Cheers,
Adam

Just to verify - have you checked this documentation?: https://docs.mendix.com/refguide/ibm/ibm-watson-connector#WatsonAssistant


JSON parse of git checkout https://github.com/puppetlabs/puppet/pull/390 fails

I get a syntax error in the eval str part of hub when running git checkout https://github.com/puppetlabs/puppet/pull/390.
It looks like there is not enough escaping of Ruby-style code in there:

⚡ git checkout https://github.com/puppetlabs/puppet/pull/390
/Users/daniel/bin/hub:1435:in `string': (eval):1:in `string': compile error (SyntaxError)
(eval):1: syntax error, unexpected '}', expecting ')'
...:Face.colorize(:red, 'string'}\"\r\n\r\nRelated spec tests a...
                              ^
(eval):1: syntax error, unexpected $end, expecting '}'
...ated spec tests are included."
                              ^
	from /Users/daniel/bin/hub:1405:in `eval'
	from /Users/daniel/bin/hub:1435:in `string'
	from /Users/daniel/bin/hub:1405:in `value'
	from /Users/daniel/bin/hub:1416:in `hash'
	from /Users/daniel/bin/hub:1399:in `object'
	from /Users/daniel/bin/hub:1405:in `value'
	from /Users/daniel/bin/hub:1416:in `hash'
	from /Users/daniel/bin/hub:1399:in `object'
	from /Users/daniel/bin/hub:1394:in `parse'
	from /Users/daniel/bin/hub:1381:in `parse'
	from /Users/daniel/bin/hub:806:in `checkout'
	from /Users/daniel/bin/hub:603:in `send'
	from /Users/daniel/bin/hub:603:in `run'
	from /Users/daniel/bin/hub:1445:in `initialize'
	from /Users/daniel/bin/hub:1449:in `new'
	from /Users/daniel/bin/hub:1449:in `execute'
	from /Users/daniel/bin/hub:1498

Bug with my JSON parser, obviously. Thanks for reporting. Will look into it

Fixed, thanks! brew upgrade hub --HEAD


Can't uninstall or update

OS
Window 11
VERSION
Windows 11 Enterprise, 22H2
ISSUE DESCRIPTION
I am unable to uninstall or update this app. I have tried uninstalling or installing it with and without admin rights. I also do not see any AnotherRedisDesktopManager application running in the Task Manager. What is going wrong?

uninstall APP if you can
win+r input 'regedit'
ctrl + f to find any related to 'Another'
delete all of them
install without admin

Thank you.
I have the same issue when I install/uninstall without admin rights.
I will try to delete it from appData & regedit.


Incorrect error when having different amount of parameters for singular/plural messages

See the screenshot below. I have a translation with a different amount of parameters for the singular and plural message. I believe my translation is in the correct format, but Poedit says there is an error. The .mo file is however compiled correctly.

I believe my translation is in the correct format, but Poedit says there is an error.

Did you consider the possibility that your belief is incorrect and Poedit's (msgfmt, actually) check is right? Because that's exactly the case: msgfmt validates per target language's requirements and if it reports an error, you have to fix it for the translation to work correctly.
It even tells you exactly what is wrong with the translation: you didn't use %d in the first one and you simply can't do it like that (if you insist on using "Een", try positional format specifiers, that may be more permissive). In any case, there's nothing Poedit itself can do about it.
P.S. Please only use the issue tracker when you're sure it is a bug; there are better support channels listed at http://poedit.net/support for other situations.


Differing resolutions on Dual Monitors will trigger engine crashes when Project Settings & Script Docs are closed on different screen as engine (DisplayServer regression)

Godot version
4.0 Stable
System information
Windows 10 , Nvidia GeForce RTX 2080 , V 528.49 , Vulkan
Issue description
When using a dual monitor setup, with different resolutions ( I have a 4k and 1k setup ) , if you open project settings or script docs on the large window, move it to the smaller window, and then close it there, the next time you try to open it Godot will lock up and prevent you from doing anything. You will often ( though not always ) need to close it through task manager.
As a workaround - you can set your resolution on your larger screen to match the smaller one, and functionality is restored.
This issue does not occur if the past position of the project-settings or windows page when closed was on same side as the fully maximized Godot instance.
I believe it is related to this already closed issue, but I do not believe it is the same problem.
https://github.com/godotengine/godot/issues/58591
Steps to reproduce

Open Godot and open any project.
Maximize Godot to fill the largest of two windows.
Open Project Settings
Move project settings to second window
Close project settings
Try to open project settings again
Observe Godot locking up - menu bar is no longer accessible.
--- That's the bug. Now do the following to help isolate it.
Set your resolution on your larger resolution screen to match your smaller resolution screen.
Open Godot and open the same project.
Open Project Settings
Observe Godot functioning normally

No repro project included. This is reproducible with a fresh project with nothing in it.
Minimal reproduction project
N/A

As a workaround, enable Single Window Mode in the Editor Settings.

Thanks for the quick response.
You can also downgrade your second monitor to a matching resolution, which is what I'm doing.

I've also discovered this bug. Primary monitor is 4K on the left, secondary 1K on right. Main Godot window on 4K, open Project Settings and move that window to the 1K display, close Project Settings. Reopen Project Settings (sometimes you see a flash on the 1k monitor). Godot is now locked as the invisible Project Settings window is modal, to unlock Godot, press Escape this will close the hidden window (see below if you've already clicked out of the window though).
To get the window back I found the following workaround:

Open .godot/editor/project_metadata.cfg
Edit the 'project_settings' value within the [dialog_bounds] section
Set the first number in the Rect2 values to zero (first value is the x position, you might need to do this differently depending on your monitor layout)
project_settings=Rect2(XXXX, 588, 1350, 1050)
Save that file

If you've clicked away from the hidden 'Project Settings' window, then pressing Escape won't get you out of it. I found this way to get Godot to close the window itself:

Open 'project.godot' at the root of your project
Add a blank line somewhere, or a space on a blank line, we just want the filesystem to see that the file has changed.
Save the file, Godot should now response that the 'project.godot' file has changed and offer to reload it. This kills the hidden modal window giving access to Godot again.

See #94315


Having a clear documentation for SSH download strategy (#3891)

I am used to create a new GitHub API Token everytime I have to provision a new device. However, I also have to setup SSH keys so that I can use Git over SSH.
I figured maybe brew supports using my SSH key instead of having to generate a token. I found #3891 which I believe is the feature I'm looking for, but I see no documentation regarding on how to set it up. Since I already setup my SSH keys on GitHub, and brew is still giving me Warning: Error searching on GitHub: GitHub Bad credentials:HOMEBREW_GITHUB_API_TOKEN may be invalid or expired I assume it's not an automatic setup, but I probably have to do some manual steps.
Documentation on this would be perfect!

We don't officially support using SSH keys for Homebrew clones, sorry. You shouldn't need a token when cloning public repositories. HOMEBREW_GITHUB_API_TOKEN is used for searches rather than cloning.


sick_scan: 1.6.0-2 in 'kinetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository sick_scan to 1.6.0-2:

upstream repository: https://github.com/SICKAG/sick_scan.git
release repository: https://github.com/SICKAG/sick_scan-release.git
distro file: kinetic/distribution.yaml
bloom version: 0.9.3
previous version for package: 1.6.0-1

sick_scan
* NAV 210+NAV245 support added code reformated
* NAV310 added
* Contributors: Michael Lehning

Holding for sync

Holding for Kinetic sync.

I just looked a little close at this and noticed that it only has a debian increment release which is what you want to use if you're just patching something in the release repository but not in the source code. However I looked at the repository and see several recent commits on master. Were those intended to be released? If so you need to tag the new release in the source branch before running bloom-release to get it to release that content correctly. Can you confirm that you wanted to do a debian increment and not release the new content from the upstream repository?

Hi Tully,
Thanks for you reminder. We plan to start a new bloom release today, which supports also noetic, and integrate the mentioned issue in this process.
Best regards,
Michael
Viele Grüße
Michael Lehning
Ing.-Büro Dr. Michael Lehning
Dr.-Ing. Michael Lehning
Schulstr. 21B
D-31137 Hildesheim
T: +49 5121 645 70
M: +49 173 37 69 242
E<EMAIL_ADDRESS><EMAIL_ADDRESS>http://www.lehning.de/ www.lehning.de
Ihr Spezialist für Digitale Bild- und Signalverarbeitung
Von: Tully Foote<EMAIL_ADDRESS>Gesendet: Donnerstag, 2. Juli 2020 01:14
An: ros/rosdistro<EMAIL_ADDRESS>Cc: Michael Lehning<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Betreff: Re: [ros/rosdistro] sick_scan: 1.6.0-2 in 'kinetic/distribution.yaml' [bloom] (#25422)
@michael1309 https://github.com/michael1309  we're waiting for your response to the above questions. With no response we'll close this issue in a little bit.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/ros/rosdistro/pull/25422#issuecomment-652689983 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AH6A5MJU746DYVTOOCGFO23RZO7KJANCNFSM4N3OVUKA .  https://github.com/notifications/beacon/AH6A5MKSHGRX5TDDCT75KQ3RZO7KJA5CNFSM4N3OVUKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOE3TUEPY.gif

@tfoote
After setting up noetic and further code adaptations, we have now released the new version 1.7.6-1, which builds error-free in the Bloom process, and are waiting for the integration into the distribution. The version 1.6.0-2 is therefore obsolete. Thanks for your support.

replaced by #25661


Node shapes

Having 12 different node shapes would be useful for visualization.

Thanks for filing this issue! We've already got an issue for this (#7), so I'm going to close this ticket, but I'm working on it!


bootlint.com is offline

Should we remove the link to it from README?

Yes, we should. I have no idea who controlled this.
I'm gonna remove it now.


Added important note #3077

I've tried to reproduce the issue that the person is reporting but I was unable to do so.
Added an important note related to E4 Licenses.
Copy review @nenonix @JohanFreelancer9 is required on this one guys.
Thank you in advance!

@tonysmit PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


TypeError: unsupported operand type(s) for |: '_GenericAlias' and '_GenericAlias'

please delete this issue

@vm-bioinfo Hi! I saw you had issues with some sort of web app. Did it turn out to be elsewhere in your project?