yarn build is broken

I'm using the PRO Version. Running yarn install and yarn build results in multiple errors which leads to a undeployable product.
1. Linting Issues
$ next build
 ⚠ Invalid next.config.js options detected:
 ⚠     Unrecognized key(s) in object: 'appDir' at "experimental"
 ⚠ See more info here: https://nextjs.org/docs/messages/invalid-next-config
 ⚠ App router is available by default now, `experimental.appDir` option can be safely removed.
 ✓ Creating an optimized production build
 ✓ Compiled successfully

Failed to compile.

./src/components/card/CardMenu.tsx
37:7  Error: Do not pass children as props. Instead, nest children between the opening and closing tags.  react/no-children-prop

info  - Need to disable some ESLint rules? Learn more here: https://nextjs.org/docs/basic-features/eslint#disabling-rules
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Can be hotfixed by excluding ./src/components/card/CardMenu.tsx from the linting process. Real fix is mentioned in the Error message.
2. Naming Issues
$ next build
 ⚠ Invalid next.config.js options detected:
 ⚠     Unrecognized key(s) in object: 'appDir' at "experimental"
 ⚠ See more info here: https://nextjs.org/docs/messages/invalid-next-config
 ⚠ App router is available by default now, `experimental.appDir` option can be safely removed.
 ✓ Creating an optimized production build
 ✓ Compiled successfully
   Linting and checking validity of types  .Failed to compile.

./src/components/admin/dashboards/default/YourCard.tsx:2:24
Type error: Already included file name '/myproject/src/components/card/MasterCard.tsx' differs from file name '/myproject/src/components/card/Mastercard.tsx' only in casing.
  The file is in the program because:
    Imported via "components/card/MasterCard" from file '/myproject/src/components/admin/dashboards/default/YourCard.tsx'
    Imported via 'components/card/MasterCard' from file '/myproject/src/components/admin/main/account/billing/YourCard.tsx'
    Root file specified for compilation

  1 | import Card from "components/card";
> 2 | import MasterCard from "components/card/MasterCard";
    |                        ^
  3 | import Transaction from "components/dataDisplay/Transaction";
  4 | import React from "react";
  5 | import { AiOutlinePlus } from "react-icons/ai";
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Can be fixed by renaming components/card/Mastercard.tsx to components/card/MasterCard.tsx
3. Missing Dependencies
$ next build
 ⚠ Invalid next.config.js options detected:
 ⚠     Unrecognized key(s) in object: 'appDir' at "experimental"
 ⚠ See more info here: https://nextjs.org/docs/messages/invalid-next-config
 ⚠ App router is available by default now, `experimental.appDir` option can be safely removed.
 ✓ Creating an optimized production build
 ✓ Compiled successfully
   Linting and checking validity of types  .Failed to compile.

./src/components/admin/main/profile/settings/Delete.tsx:9:8
Type error: Cannot find module '@chakra-ui/react' or its corresponding type declarations.

   7 |   Text,
   8 |   useColorModeValue,
>  9 | } from '@chakra-ui/react';
     |        ^
  10 | import Card from 'components/card/Card';
  11 |
  12 | export default function Settings() {
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Can be fixed by running yarn add @chakra-ui/react. Why is chakra part of this repo when we are using Tailwind?
4. Broken References
$ next build
 ⚠ Invalid next.config.js options detected:
 ⚠     Unrecognized key(s) in object: 'appDir' at "experimental"
 ⚠ See more info here: https://nextjs.org/docs/messages/invalid-next-config
 ⚠ App router is available by default now, `experimental.appDir` option can be safely removed.
 ✓ Creating an optimized production build
 ✓ Compiled successfully
   Linting and checking validity of types  .Failed to compile.

./src/components/admin/main/profile/settings/Delete.tsx:10:18
Type error: Cannot find module 'components/card/Card' or its corresponding type declarations.

   8 |   useColorModeValue,
   9 | } from '@chakra-ui/react';
> 10 | import Card from 'components/card/Card';
     |                  ^
  11 |
  12 | export default function Settings() {
  13 |   // Chakra Color Mode
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Changing import Card from 'components/card/Card'; to 'components/card'; inside ./src/components/admin/main/profile/settings/Delete.tsx fixed the issue because Card component is defined in components/card/index.tsx.
5. ReferenceError: window is not defined
$ next build
 ⚠ Invalid next.config.js options detected:
 ⚠     Unrecognized key(s) in object: 'appDir' at "experimental"
 ⚠ See more info here: https://nextjs.org/docs/messages/invalid-next-config
 ⚠ App router is available by default now, `experimental.appDir` option can be safely removed.
 ✓ Creating an optimized production build
 ✓ Compiled successfully
 ✓ Linting and checking validity of types
 ✓ Collecting page data
   Generating static pages (0/49)  [    ]ReferenceError: window is not defined
...
 ✓ Generating static pages (49/49)

> Export encountered errors on following paths:
        /admin/dashboards/car-interface/page: /admin/dashboards/car-interface
        /admin/dashboards/default/page: /admin/dashboards/default
        /admin/dashboards/smart-home/page: /admin/dashboards/smart-home
        /admin/main/account/all-courses/page: /admin/main/account/all-courses
        /admin/main/account/billing/page: /admin/main/account/billing
        /admin/main/applications/calendar/page: /admin/main/applications/calendar
        /admin/main/users/users-reports/page: /admin/main/users/users-reports
        /auth/forgot-password/centered/page: /auth/forgot-password/centered
        /auth/lock/centered/page: /auth/lock/centered
        /auth/sign-in/centered/page: /auth/sign-in/centered
        /auth/sign-up/centered/page: /auth/sign-up/centered
        /auth/verification/centered/page: /auth/verification/centered
        /rtl/dashboards/rtl/page: /rtl/dashboards/rtl

Fixing this seems to be a bigger rewrite.

Hello, @JanOwiesniak ! The Chakra component got into the archive by mistake 😅
As for the other issues :
-Linting issue fixed
-ApexCharts uses window for their charts, the problem was solved by dynamic importing the base chart components
Regards, Vlad.

Thanks for fixing those issues. You forgot to rename components/card/Mastercard.tsx to components/card/MasterCard.tsx as mentioned above.


Fix dist runs

Motivation
Fixes #6534
Modification
Use torchrun instead of the deprecated torch.distributed.launch.
BC-breaking (Optional)
I'm not sure when torchrun was introduced, so if people are using older pytorch versions then it might cause issues.
Checklist

[x] Pre-commit or other linting tools are used to fix the potential lint issues.
[x] The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness. <- I don't think there's a test for this
[x] If the modification has potential influence on downstream projects, this PR should be tested with downstream projects, like MMDet or MMCls. <- I don't think it has a downstream effect (it might be worth changing this in the other repos though)
[x] The documentation has been modified accordingly, like docstring or example tutorials.

 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.

Hi @vakker ,
The version of pytorch is indeed an issue. For now, MMDetection needs to ensure the usage with PyTorch 1.5. Therefore, can we keep the original command but provide new examples about torchrun in the documentation and the scripts?

@ZwwWayne what about checking if torchrun is available and if not then falling back to the previous version?
I.e. something like this:
if command -v torchrun &> /dev/null
then
    torchrun .....
else
    python -m torch.distributed.launch ....
fi

@ZwwWayne what about checking if torchrun is available and if not then falling back to the previous version? I.e. something like this:
if command -v torchrun &> /dev/null
then
    torchrun .....
else
    python -m torch.distributed.launch ....
fi

I agree, we can do that. And we should also keep both commands in the documentation for now.

@ZwwWayne I'm just waiting on some input in the corresponding issue (#6534), I'll wrap this PR up once the root cause is concluded.


New Feature: Adding watsonx.ai LLM Platform support

Pull Request Type

[x] ✨ feat
[ ] 🐛 fix
[ ] ♻ refactor
[ ] 💄 style
[ ] 🔨 chore
[ ] 📝 docs

What is in this change?
Adding watsonx.ai support as LLM Platform
Additional Information
New LLM backend supporing various LLMs is added. Running watsonx.ai will require:

WATSONX_AI_ENDPOINT
WATSONX_AI_APIKEY
WATSONX_AI_PROJECT_ID
WATSONX_AI_MODEL (i.e meta-llama/llama-2-70b-chat)
WATSONX_EMBEDDING_MODEL_PREF (i.e. baai/bge-large-en-v1)
In addition, we are intorducing AI Guardrails for input and LLM output

Developer Validations

[ x] I ran yarn lint from the root of the repo & committed changes
[ x] Relevant documentation has been updated
[ x] I have tested my code functionality
[x ] Docker build succeeds locally

@timothycarambat here is the link: https://cloud.ibm.com/docs/account?topic=account-userapikey&interface=ui
API is obtained via https://cloud.ibm.com/ and the ProjectID is obtained via https://dataplatform.cloud.ibm.com  after project creation.


envmodules configuration sections should follow conda environment files

Currently, there is a separate envmodules key for every rule, but a given conda environment file is shared between rules. Since both keywords solve the same problem, envmodules should be shared between rules, following the conda environment sharing. For instance, envs/malt.yaml is shared between four rules, each of which has a separate envmodules config.

Agree that this would be nice, I am assuming the conda environment organization reflects some kind of functional modularity (i.e. they are grouped depending on the analyses that are being run?)

The conda environment is meant to provide the minimum set of (conda) dependencies to run a rule. For some rules, not everything in a environment file is needed, but the differences are so small, and conda envs so costly to install, that the fewer conda env files, the better. Loading an extra environment module or two has little overhead, so we might as well provide the same grouping.

Right, do we also want to move envmodules.yaml someplace else? I was just looking at the docs and it might make sense to move the section to the runtime config in .profile

though I don't know if I understood correctly how profiles are meant to be used...

Yes, I think I had misunderstood how the profiles should be used..

For now I would suggest sticking with envmodules in config, and maybe add support for an environment variable such that it could be placed in e.g. .config/snakemake/envmodules.yaml or similar?

Do you mean a .config/ folder in the installation directory? That or any other "global" location that would be suitable (maybe workflow/envs/?)

No, I meant the "regular" config directory would be the default location, if no other parameters have been set. What do we think will be the user-case scenario? You have an analysis folder (separate from the repo) in which there is a config directory with config.yaml and samples.tsv, and the envmodules.yaml file residing in some directory accessible to all analyses. This doesn't necessarily have to be in the repo. I'm open to any suggestions, but hopefully it will become clearer when we test multiple projects.
Following up on the environment variable discussion, this is what the help for snakemake --profile flag says:
 --profile PROFILE     Name of profile to use for configuring Snakemake.
                        Snakemake will search for a corresponding folder in
                        /etc/xdg/xdg-lxqt/snakemake and
                        /home/peru/.config/snakemake. Alternatively, this can
                        be an absolute or relative path. The profile folder
                        has to contain a file 'config.yaml'. This file can be
                        used to set default values for command line options in
                        YAML format. For example, '--cluster qsub' becomes
                        'cluster: qsub' in the YAML file. Profiles can be
                        obtained from https://github.com/snakemake-profiles.
                        The profile can also be set via the environment
                        variable $SNAKEMAKE_PROFILE. [env var:
                        SNAKEMAKE_PROFILE] (default: None)

So there is already a use case where one puts config in a .config directory. I guess this could also be the current working directory, but would it be confusing to have both a .config and a config directory to keep track of?

Closed via #50


🗺👯👯 Discussion about Project 🔔 Language Translations ✅☑✋ translators/languages

Project URL: https://github.com/cevap/ion/projects/10

I'll repost it just for overview (current version):
🔔 Language Translations ✅☑✋ translators/languages - Bounties
We need ASAP translations to some languages. Let's discuss about it, this is mainly a project where community of ion should come to express/help/suggest.
ℹ Information

Goals status (legends) page will be updated on the fly if any changes happen
Project will continuously grow until everything is translated, then it will be closed and new maintenance project will be created.
New languages might be added, as well as some removed if this project results in such way
If and after bounties are set we will publish them on current project overview page

Goals

Translate changes of user interface
Translate documentation and all readme files
Translation to languages required by priority

✔  🇨🇳 Chinese (Simplified)
✅ 🇷🇺 Russian
☑ 🇪🇸 Spanish
✅ 🇩🇪 German
✋  🇮🇹 Italian
✋  🇮🇩 Indonesian

Legend to be used/explanation:
✅ - We have active Translator

User is member/contributor of CEVAP
User is already translating
User is native speaker
User has GitHub account
Everybody is welcome to join, we need more native speakers

✔ - We know a translator, in work

User being reviewed
Unknown if language skills are real
Everybody is welcome to join, we need more native speakers

☑ - We have application from at least 1 user

User has applied to be reviewed
Unknown if language skills are real
Everybody is welcome to join, we need more native speakers

✋ - We do not have anybody

No application
No review possibility

Source code language/inline translations
if we use something like command line translation over google, then it is possible also to automate translations of the source code itself. We need to discuss about it and about maintenance efforts to achieve that and what do we get with that.

Discuss about possibilities beside Transifex

we want to rely strongly on services for CE, which have to be free
this will be probably one big difference from bitcoins core even if we are on latest version

Discuss about possible usage of Transifex

We could order Transifex only for releases.

Can we cooperate with ionomy, we need to ask them to give us at least 2 accounts?
Should we buy it regardless of ionomy?

Discuss Source Code translations

What should that be good for?

What is the effort to maintenance it

Do we have enough time/contributors to deal with that
What does it mean if we offer other languages (ie. translation of all documents)
Can we rely on services for languages which are not spoken by anybody of contributors

Transifex registered and will be used from now on. There is no better alternative for our project and Transifex is great, closed with that.
ℹ - We still need people who can verify translations


Update schema to allow for github-pages in job-environment-name

Just trying to make #74 more actionable.
I'm not familiar with this schema, so it might make sense to add the value in other places as well. Or maybe this is the wrong approach altogether.

Turns out it is a wrong approach to address the problem we were experiencing. 🤦🏻♂
See https://github.com/actions/deploy-pages/issues/344#issuecomment-2158468047.
Please have no hesitation to correct me if I'm wrong again.


Fix failing unit tests

The following unit tests are failing (for me). Version is master as on feb 24 2021:
1) "before each" hook for "has correct default props"
     helpwin/HelpWin.vue props
     TypeError: Cannot read property 'windowTitle' of undefined
    at VueComponent.data (webpack:///src/components/helpwin/HelpWin.vue:40:1 <- index.js:114854:86)
    at getData (webpack:///~/vue/dist/vue.esm.js:4757:1 <- index.js:7847:17)
    at initData (webpack:///~/vue/dist/vue.esm.js:4714:1 <- index.js:7804:7)
    at initState (webpack:///~/vue/dist/vue.esm.js:4651:1 <- index.js:7741:5)
    at VueComponent.Vue._init (webpack:///~/vue/dist/vue.esm.js:5015:1 <- index.js:8105:5)
    at new VueComponent (webpack:///~/vue/dist/vue.esm.js:5163:1 <- index.js:8253:12)
    at createComponentInstanceForVnode (webpack:///~/vue/dist/vue.esm.js:3292:1 <- index.js:6382:10)
    at init (webpack:///~/vue/dist/vue.esm.js:3123:1 <- index.js:6213:45)
    at createComponent (webpack:///~/vue/dist/vue.esm.js:5989:1 <- index.js:9079:9)
    at createElm (webpack:///~/vue/dist/vue.esm.js:5936:1 <- index.js:9026:9)

2) "before each" hook for "has correct default data"
     helpwin/HelpWin.vue data
     TypeError: Cannot read property 'windowTitle' of undefined
    at VueComponent.data (webpack:///src/components/helpwin/HelpWin.vue:40:1 <- index.js:114854:86)
    at getData (webpack:///~/vue/dist/vue.esm.js:4757:1 <- index.js:7847:17)
    at initData (webpack:///~/vue/dist/vue.esm.js:4714:1 <- index.js:7804:7)
    at initState (webpack:///~/vue/dist/vue.esm.js:4651:1 <- index.js:7741:5)
    at VueComponent.Vue._init (webpack:///~/vue/dist/vue.esm.js:5015:1 <- index.js:8105:5)
    at new VueComponent (webpack:///~/vue/dist/vue.esm.js:5163:1 <- index.js:8253:12)
    at createComponentInstanceForVnode (webpack:///~/vue/dist/vue.esm.js:3292:1 <- index.js:6382:10)
    at init (webpack:///~/vue/dist/vue.esm.js:3123:1 <- index.js:6213:45)
    at createComponent (webpack:///~/vue/dist/vue.esm.js:5989:1 <- index.js:9079:9)
    at createElm (webpack:///~/vue/dist/vue.esm.js:5936:1 <- index.js:9026:9)

3) "before each" hook for "onWinXClose switches show"
     helpwin/HelpWin.vue methods
     TypeError: Cannot read property 'windowTitle' of undefined
    at VueComponent.data (webpack:///src/components/helpwin/HelpWin.vue:40:1 <- index.js:114854:86)
    at getData (webpack:///~/vue/dist/vue.esm.js:4757:1 <- index.js:7847:17)
    at initData (webpack:///~/vue/dist/vue.esm.js:4714:1 <- index.js:7804:7)
    at initState (webpack:///~/vue/dist/vue.esm.js:4651:1 <- index.js:7741:5)
    at VueComponent.Vue._init (webpack:///~/vue/dist/vue.esm.js:5015:1 <- index.js:8105:5)
    at new VueComponent (webpack:///~/vue/dist/vue.esm.js:5163:1 <- index.js:8253:12)
    at createComponentInstanceForVnode (webpack:///~/vue/dist/vue.esm.js:3292:1 <- index.js:6382:10)
    at init (webpack:///~/vue/dist/vue.esm.js:3123:1 <- index.js:6213:45)
    at createComponent (webpack:///~/vue/dist/vue.esm.js:5989:1 <- index.js:9079:9)
    at createElm (webpack:///~/vue/dist/vue.esm.js:5936:1 <- index.js:9026:9)

4) "before each" hook for "has correct default props"
     helpwin/ToggleButton.vue props
     TypeError: Cannot read property 'on' of undefined
    at Object.fn [as activator] (webpack:///src/components/helpwin/ToggleButton.vue?6e9e:13:1 <- index.js:174273:26)
    at webpack:///~/@vue/test-utils/dist/vue-test-utils.js:2395:68 <- index.js:17485:125
    at Array.map (<anonymous>)
    at Proxy.render (webpack:///~/@vue/test-utils/dist/vue-test-utils.js:2395:1 <- index.js:17485:50)
    at VueComponent.Vue._render (webpack:///~/vue/dist/vue.esm.js:3557:1 <- index.js:6647:22)
    at VueComponent.updateComponent (webpack:///~/vue/dist/vue.esm.js:4075:1 <- index.js:7165:21)
    at Watcher.get (webpack:///~/vue/dist/vue.esm.js:4488:1 <- index.js:7578:25)
    at new Watcher (webpack:///~/vue/dist/vue.esm.js:4477:1 <- index.js:7567:12)
    at mountComponent (webpack:///~/vue/dist/vue.esm.js:4082:1 <- index.js:7172:3)
    at VueComponent.Vue.$mount (webpack:///~/vue/dist/vue.esm.js:9063:1 <- index.js:12153:10)

5) "before each" hook for "has correct default data"
     helpwin/ToggleButton.vue data
     TypeError: Cannot read property 'on' of undefined
    at Object.fn [as activator] (webpack:///src/components/helpwin/ToggleButton.vue?6e9e:13:1 <- index.js:174273:26)
    at webpack:///~/@vue/test-utils/dist/vue-test-utils.js:2395:68 <- index.js:17485:125
    at Array.map (<anonymous>)
    at Proxy.render (webpack:///~/@vue/test-utils/dist/vue-test-utils.js:2395:1 <- index.js:17485:50)
    at VueComponent.Vue._render (webpack:///~/vue/dist/vue.esm.js:3557:1 <- index.js:6647:22)
    at VueComponent.updateComponent (webpack:///~/vue/dist/vue.esm.js:4075:1 <- index.js:7165:21)
    at Watcher.get (webpack:///~/vue/dist/vue.esm.js:4488:1 <- index.js:7578:25)
    at new Watcher (webpack:///~/vue/dist/vue.esm.js:4477:1 <- index.js:7567:12)
    at mountComponent (webpack:///~/vue/dist/vue.esm.js:4082:1 <- index.js:7172:3)
    at VueComponent.Vue.$mount (webpack:///~/vue/dist/vue.esm.js:9063:1 <- index.js:12153:10)

6) has correct default data
     ol/Map.vue data
     AssertionError: expected undefined to deeply equal { code: 'EPSG:3857', units: 'm' }
    at Context.<anonymous> (webpack:///test/unit/specs/components/ol/Map.spec.js:65:37 <- index.js:129371:37)

7) Setup permalinkController
     ol/Map.vue data - PermalinkController successfully setup
     AssertionError: expected 4 to equal 2
    at Context.<anonymous> (webpack:///test/unit/specs/components/ol/PermalinkController.spec.js:83:63 <- index.js:129741:63)

8) Layer Listeners are (re)created when the layer stack changes
     ol/Map.vue data - PermalinkController successfully setup
     AssertionError: expected 4 to equal 2
    at Context.<anonymous> (webpack:///test/unit/specs/components/ol/PermalinkController.spec.js:91:63 <- index.js:129749:63)

9) Setup and apply permalinkController - apply from document.location.hash
     ol/Map.vue data - PermalinkController applied from document.location.hash/search
     AssertionError: expected 4 to equal 2
    at Context.<anonymous> (webpack:///test/unit/specs/components/ol/PermalinkController.spec.js:153:58 <- index.js:129811:58)

Several errors deal with HelpWin: mismatch between expected and provided config, can/should be fixed in the test spec file.
Others may be caused by config changes like Layers. Need to dive into.

Closed by #182


choose uncertainties from data channel

I'd like the option to be able to use the uncertainties values in my obs file instead of assigning global % + floor.  This would be an additional option to use file values OR the global % + floor.

GEOPY-213


Experimental: disable color textures, lighting toggle, force y-up

Motivation and Context
Do not merge.
I'm just documenting this experimental lighting_viewer branch in case anyone wants to use it. It's useful when running the viewer to evaluate lighting. These changes probably aren't appropriate to merge to master.
It has three features:

It forces y-up convention for glb files because that's commonly what I need (our Viewer app should get a command-line option to choose up convention; that would be a much better solution).
It adds a lighting toggle, L key. It cycles between 5 camera-relative light directions, then back to default lighting. Viewing lighting from camera-left then camera-right is useful for evaluating normal maps.
It adds a toggle to disable color textures, ';' key. With color textures disabled, most things look white but still lit. It's much easier to evaluate lighting and normal maps with the color textures removed.

How Has This Been Tested
Nope
Types of changes

[ ] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have completed my CLA (see CONTRIBUTING)
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

I am planning on full scene_dataset_config support for viewer, so the limitiations that you have experienced will soon (hopefully) be a thing of the past.

@bigbike

Here for reference, not merging. I'm closing this.

Here for reference, not merging. I'm closing this.


test-branch/execute-job/wrong-commit-sha-on-pr/755801a7-db62-4b7d-836d-5acdb210bc6d

PR for testing 55b56a999b98e19e726f216ff642d44198aeded4

/canonical/self-hosted-runners/run-workflows 931ad21dd9ace68aaa0a7d341e94de96ad93fa6b


Public GitHub webhooks re-broadcaster / message queue

Infrastructure and tooling ran by contributors can't easily use the GitHub webhooks service since it requires being explicitly configured for each relevant project to point to recipient endpoints.
It would be nice and helpful if we had a public waterhose of webhook events that anyone could subscribe to without having to go find an org admin to add their project to the repo's configuration (something for which we don't really have a process, so it's likely all ad-hoc...). This would take the form of an officially maintained webhook receiver service running on non-critical-infra which accepts ephemeral subscribers and delivers webhook events. TBD: filtering, backlog, etc.
TODO: is there already existing open source infra to do this?

Maybe we can open up Ofborg's message queue, but I don't know what the implications are there.

Something like https://github.com/vigour-io/webhook-forwarder ?


Autotype for selfhosted version

Is it possible to Connect to a selfhosted instance of Bitwarden/Vaultwarden just like the Desktop App? Actually i do not see the option. Maybe this cann be added to the initial Configuration.

This is basically the same as https://github.com/anonymous1184/bitwarden-autotype/issues/4, https://github.com/anonymous1184/bitwarden-autotype/issues/29, https://github.com/anonymous1184/bitwarden-autotype/issues/33, https://github.com/anonymous1184/bitwarden-autotype/issues/42, and https://github.com/anonymous1184/bitwarden-autotype/issues/58
See https://github.com/anonymous1184/bitwarden-autotype/issues/42 or https://github.com/anonymous1184/bitwarden-autotype/issues/29 for instructions on how to change the server URL.


Incorrect decomposition for 周?

The current entry shows
周:st(田,口)

Which seems wrong to me.

Looks like it's correct in https://raw.githubusercontent.com/grolang/vy/master/data/charData.txt.
@gavingroovygrover is there a newer revision of this data than what's in this repo?


Plugin isn't picking up the right commit start time on drone 0.5

Used config:
pipeline:
  email_notify:
    image: drillster/drone-email
    from<EMAIL_ADDRESS>    host: mail.email.com
    username<EMAIL_ADDRESS>    password: "123"
    recipients:
      -<EMAIL_ADDRESS>      -<EMAIL_ADDRESS>    subject: Resultado do build da branch ${DRONE_COMMIT_BRANCH}
    when:
      branch:
        - branchname
        - otherbranchname

Date displayed on all received emails:

Thanks for reporting this!
It appears to be a known Drone issue; for some reason DRONE_BUILD_STARTED is always 0.
I will build a temporary workaround that will use DRONE_BUILD_CREATED instead.

@mjwwit Thanks for fixing it! :)


Show docker run command as you create container via Rancher Add Container

As you fill out the form to adding a container via the Rancher UI, we should automatically fill out and show an equivalent docker run command so people can copy and paste that and run it locally.

I think it's fair to say this hasn't received much interest, and does not mix well with the emphasis on services over standalone containers...


Demo on tour page skips section.

Tour on demo page skips feature Consistent behavior across all major browsers in list where every other feature is highlighted.

Hey, that is intentional. If you look closely, it also skips the second one. For browser compatibility I didn't try it yet on every other browser so didn't add that :)


Unable to run script on arbitrum with python SDK

I'm getting an invalid network error when I run the script for arbitrum network using python sdk
> npm run start:dev

><EMAIL_ADDRESS>start:dev
> nodemon --watch src --watch forta.config.json -e py --exec "forta-agent run"

[nodemon] 2.0.15
[nodemon] to restart at any time, enter `rs`
[nodemon] watching path(s): src/**/* forta.config.json
[nodemon] watching extensions: py
[nodemon] starting `forta-agent run`
listening for blockchain data...
fetching block 8046206...
42161 is not a valid Network
ERROR: Error: python: 42161 is not a valid Network
[nodemon] clean exit - waiting for changes before restart

updated network.py to allow the other networks . the issue has also been update from what I see on the commit

yes, the commit has been made to fix this and will be published in the next 24 hours


Interaction not exercised: GET

Trying to implement the Contract test using Pactum, here is my sample code,
const { reporter, flow, mock, handler } = require("pactum");
const pf = require("pactum-flow-plugin");
const jwtTokenHandler = require("./jwtTokenHandler");
let bearerTOKEN;

function addFlowReporter() {
  pf.config.url = "http://localhost:8080";
  pf.config.projectId = "Order-Service";
  pf.config.projectName = "[T01] Order-Service";
  pf.config.version = "1.0.0";
  pf.config.username = "*****";
  pf.config.password = "*****";
  reporter.add(pf.reporter);
}

// global before
before(async () => {
  addFlowReporter();
  await mock.start(4000);
  bearerAccessToken = await jwtTokenHandler.generateBearerToken();
});

// global after
after(async () => {
  await mock.stop();
  await reporter.end();
});

handler.addInteractionHandler("Fetch the orders for a given lead", () => {
  return {
    provider: "Node API", // same as provider project
    flow: "fetch the order for lead id", // same as provider flow name
    request: {
      method: "GET",
      path: "https://<test_server>/api/order?leadId=434",
      withBearerToken: `${bearerAccessToken}`,
    },
    response: {
      status: 200,
      body: {
        responseCode: 200,
        message: "Success!",
        data: [[]],
      },
    },
  };
});

it("Post Orders to Order_Service", async () => {
  await flow("Post an order for a lead")
    .useInteraction("Fetch the orders for the Lead_Id")
    .post("https://<test_server>/api/order")
    .withHeaders("Content-Type", "application/json")
    .withBearerToken(bearerAccessToken)
    .withJson({
      orderDate: "2024-08-02 00:00:00",
      maturityDate: "2024-08-29 00:00:00",
      transactionAmount: 1000,
      leadID: 434,
      orderStatusTypeID: 10,
      loanTypeID: 2,
      applicationNumber: "string",
      approvedAmount: 500,
      isFileSubmissionDone:
      studentProfileId: 1,
    });
});

I am getting the following error,
[I] Mock server is listening on http://<IP_ADDRESS>:4000
[W] Interaction Not Exercised
{
  "method": "GET",
  "path": "https://<test_server>/api/order?leadId=434"
}
[W] Request
{
  "url": "https://<test_server>/api/order",
  "method": "POST",
  "headers": {
    "Content-Type": "application/json",
    "Authorization": "Bearer <bearerAccessToken>"
  },
  "path": "/api/order",
  "body": {
    orderDate: "2024-08-02 00:00:00",
      maturityDate: "2024-08-29 00:00:00",
      transactionAmount: 1000,
      leadID: 434,
      orderStatusTypeID: 10,
      loanTypeID: 2,
      applicationNumber: "string",
      approvedAmount: 500,
      isFileSubmissionDone:
      studentProfileId: 1
  },
  "timeout": 3000
}
[W] Response
{
  "statusCode": 200,
  "headers": {
    "content-length": "53",
    "content-type": "application/json; charset=utf-8",
    "date": "Thu, 04 Jul 2024 06:11:10 GMT",
    "access-control-allow-origin": "*",
    "etag": "W/\"35-nYRf1SCdO02PSh/SneMK1h340bA\"",
    "x-powered-by": "Express"
  },
  "body": {
    "responseCode": 200,
    "message": "Success!",
    "data": true
  }
}
  1) Post Orders to B2B Order Service
[I] Mock server stopped on port 4000

Publishing "Order-Service" with version "1.0.0"

Creating Project
Creating Analysis
Uploading Interactions
Uploading Flows
Processing Analysis

  0 passing (1s)
  1 failing

  1) Post Orders to B2B Order Service:
     AssertionError [ERR_ASSERTION]: Interaction not exercised: GET - https://<test_server>/api/order?leadId=434
      at Expect.fail (node_modules\pactum\src\models\expect.js:461:12)
      at Expect.validateInteractions (node_modules\pactum\src\models\expect.js:88:14)
      at Tosser.validateNonBackgroundInteractions (node_modules\pactum\src\models\Tosser.js:234:17)
      at Tosser.validate (node_modules\pactum\src\models\Tosser.js:209:12)
      at Tosser.toss (node_modules\pactum\src\models\Tosser.js:46:18)
      at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

In Interaction handler path should be relative..
handler.addInteractionHandler("Fetch the orders for a given lead", () => {
  return {
    provider: "Node API", // same as provider project
    flow: "fetch the order for lead id", // same as provider flow name
    request: {
      method: "GET",
      path: "/api/order?leadId=434",
      withBearerToken: `${bearerAccessToken}`,
    },
    response: {
      status: 200,
      body: {
        responseCode: 200,
        message: "Success!",
        data: [[]],
      },
    },
  };
});

@ASaiAnudeep thank you for prompt answer, but when I changed and rerun I am getting
[W] Interaction Not Exercised
{
  "method": "GET",
  "path": "/api/order?leadId=434"
}
[W] Request
{
  "url": "https://<test_server>/api/order",
  "method": "POST",
  "headers": {
    "Content-Type": "application/json",
    "Authorization": "Bearer <bearerAccessToken>"
  },
  "path": "/api/order",
  "body": {
    orderDate: "2024-08-02 00:00:00",
      maturityDate: "2024-08-29 00:00:00",
      transactionAmount: 1000,
      leadID: 434,
      orderStatusTypeID: 10,
      loanTypeID: 2,
      applicationNumber: "string",
      approvedAmount: 500,
      isFileSubmissionDone:
      studentProfileId: 1
  },
  "timeout": 3000
}
[W] Response
{
  "statusCode": 200,
  "headers": {
    "content-length": "53",
    "content-type": "application/json; charset=utf-8",
    "date": "Thu, 04 Jul 2024 08:25:42 GMT",
    "access-control-allow-origin": "*",
    "etag": "W/\"35-nYRf1SCdO02PSh/SneMK1h340bA\"",
    "x-powered-by": "Express"
  },
  "body": {
    "responseCode": 200,
    "message": "Success!",
    "data": true
  }
}
  1) Post Orders to B2B Order Service
[I] Mock server stopped on port 4000

Publishing "Order-Service" with version "1.0.1"

Creating Project
Creating Analysis
  2) "after all" hook for "Post Orders to B2B Order Service"

  0 passing (3s)
  2 failing

  1) Post Orders to B2B Order Service:
     AssertionError [ERR_ASSERTION]: Interaction not exercised: GET - /api/order?leadId=434
      at Expect.fail (node_modules\pactum\src\models\expect.js:461:12)
      at Expect.validateInteractions (node_modules\pactum\src\models\expect.js:88:14)
      at Tosser.validateNonBackgroundInteractions (node_modules\pactum\src\models\Tosser.js:234:17)
      at Tosser.validate (node_modules\pactum\src\models\Tosser.js:209:12)
      at Tosser.toss (node_modules\pactum\src\models\Tosser.js:46:18)
      at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

  2) "after all" hook for "Post Orders to B2B Order Service":
     StatusCodeError: 400 - {"error":"Analysis already exists"}
      at Object.reject (node_modules\phin-retry\src\index.js:184:11)
      at Object.__fetch (node_modules\phin-retry\src\index.js:268:21)
      at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
      at async createAnalysis (node_modules\pactum-flow-plugin\src\publisher.js:29:15)
      at async publish (node_modules\pactum-flow-plugin\src\publisher.js:116:16)
      at async run (node_modules\pactum-flow-plugin\src\runner.js:12:3)
      at async Object.end (node_modules\pactum-flow-plugin\src\reporter.js:13:3)
      at async Object.end (node_modules\pactum\src\exports\reporter.js:32:23)
      at async Context.<anonymous> (test\contract\order.contract.spec.js:31:3)


Fix typo steep->step

Shall it read step instead of steep?

"steep" (steil) is the intention, not "step" (Sprung). So the original text is correct.

Thanks for clarifaction.


Feature: Add Dalli::Client memcache metrics for web_collector

Adds memcache metrics to HTTP requests

Hey @SamSaffron , would you please take a look at this?

@SamSaffron could you please check this?

@SamSaffron Could you please have a look at this?

Thanks and sorry for the delay here, looks good!


[Ansible] broken links in Ansible galaxy

The Homepage and Repo links in https://galaxy.ansible.com/jfrog/platform do not work correctly.
See

https://github.com/jfrog/JFrog-Cloud-Installers/blob/0c323eaccf30fad675e45786335903e5267f55bb/Ansible/ansible_collections/jfrog/platform/galaxy.yml#L56

https://github.com/jfrog/JFrog-Cloud-Installers/blob/0c323eaccf30fad675e45786335903e5267f55bb/Ansible/ansible_collections/jfrog/platform/galaxy.yml#L62

@weakcamel Thanks for reporting, will be fixing in next release


Enable IDE0048: AddRequiredParentheses

https://docs.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/ide0047-ide0048

Default is always_for_clarity for arithmetic, relational, and other binary operators.

This PR did not fix all IDE0048 violations, see #13936. Since the severity was set to warning, the build should not have completed!

Is it a bug in the Roslyn analyzer?

Is it a bug in the Roslyn analyzer?

No EnforceCodeStyleInBuild was not enabled.


Watchdog timer triggered on both ESP32 Capacitive Touch examples

Both ESP32 examples fail with WDT.
p.s. The examples need the BTN_ prefix change :)
Capacitive Touch Demo
E (10329) task_wdt: Task watchdog got triggered. The following tasks did not reset the watchdog in time:
E (10329) task_wdt:  - IDLE (CPU 0)
E (10329) task_wdt: Tasks currently running:
E (10329) task_wdt: CPU 0: loopTask
E (10329) task_wdt: CPU 1: IDLE
E (10329) task_wdt: Aborting.
abort() was called at PC 0x400d9e9d on core 0
Thanks for a great package - couldn't live without it.

Thx. will check it out...


Rate limiting plugin - limit per consumer and IP.

It would be great to rate-limit requests per (consumer, IP) pair. Right now it can only limit per consumer or IP.
Thanks

I think it will be great to rate-limit requests by setting a sum.

@ShinEDL can you please elaborate on what you mean by "sum"

Thank you for your response.
I was really mean whether we can limit requests with a specific number, no matter what IP or consumer.

@Tieske Thank you for your response.
I was really mean whether we can limit requests with a specific number, no matter what IP or consumer？

We have this own our radar, thanks for the suggestion!


MATCH wildcard in MultiplexerTransform

Would it be possible to support MATCH wildcards where multiple callbacks have the same output?
Currently it errors and gives an error similar to this:
TypeError: dict id values must be strings, numbers or bools,
found {'name': 'div', 'id': <MATCH>} in id {'id': { 'name': 'div', 'id': <MATCH> } 'prop': 'children', 'idx': 0}

MWE:
from dash_extensions.enrich import DashProxy, MultiplexerTransform, Input, Output, MATCH
import dash_html_components as html

app = DashProxy(transforms=[MultiplexerTransform()])

app.layout = html.Div([
  html.Button("Button 1", id={'type': 'button', 'id': 0}),
  html.Button("Button 1", id={'type': 'button', 'id': 1}),
  html.Div(id={'type': 'div', 'id': 0}),
  html.Div(id={'type': 'div', 'id': 1}),
])

@app.callback(
  Output({'type': 'div', 'id': MATCH}, 'children'),
  Input({'type': 'button', 'id': MATCH}, 'n_clicks'),
)
def cb1(n):
  return "Hello from callback 1"

@app.callback(
  Output({'type': 'div', 'id': MATCH}, 'children'),
  Input({'type': 'button', 'id': MATCH}, 'n_clicks'),
)
def cb2(n):
  return "Hello from callback 2"

if __name__ == '__main__':
  app.run_server(port=7777)

I am trying to wrap my head around if this is possible or not. What if you make multiple callbacks, each with MATCH and/or ALL. Then the resulting callback would need to have multiple MATCH and ALL in both input and output. I don't think this is supported in Dash? If you have any conceptual ideas for the implementation and/or any resources suggesting otherwise, please let me know :)

Currently, I don't have any ideas on how to do this...
As a side note, would it be possible to add a note on this limitation in the README until this is resolved or if we see that it can't be solved?

Yes, I plan to update the readme with the next update, which brings clientside callback support and customizable proxy component wrapping.

After looking a bit more into dash pattern matching callbacks, my (preliminary?) conclusion is that the MATCH and ALLSMALLER wildcard would require the presence of one multiplexer proxy component for each MATCH output. And since the latter is dynamic (that's kind of the point), I don't think it is possible to handle this case in the multiplexer. Instead, I have added a more clear error message and a disclaimer in the documentation.
For the ALL wildcard on the other hand, I have made an implementation. You can see my demo example here,
from dash_extensions.enrich import DashProxy, MultiplexerTransform, Input, Output, ALL
import dash_html_components as html

def make_callback(i):
    @app.callback(Output({'type': 'div', 'id': ALL}, 'children'), Input({'type': f'button{i}', 'id': ALL}, 'n_clicks'))
    def func(n):
        return [f"Hello from group {i}"] * len(n)

def make_block(i):
    layout = [
        html.Button(f"Button 0 in group {i}", id={'type': f'button{i}', 'id': 0}),
        html.Button(f"Button 1 in group {i}", id={'type': f'button{i}', 'id': 1}),
        html.Div(f"Div {i}", id={'type': f'div', 'id': i}),
    ]
    make_callback(i)
    return layout

app = DashProxy(transforms=[MultiplexerTransform()], prevent_initial_callbacks=True)
app.layout = html.Div(make_block(0) + make_block(1))

if __name__ == '__main__':
    app.run_server(port=7778, debug=True)

It should work with the latest (0.0.52) release. If it doesn't please reopen this issue :)

I would love to have this available for my current situation after a bit of thinking I might have a solution, but I also might not. I just want to state my idea incase it works or even churns out a different way to solve this issue.
IDEA:
Could you not create a store for each match component with an id={type="multiplexer_match_id", index="index of store"} and then utilize the ALL and type in id to say if any of those change then output to the output we want?
I could be way over my head on this one. Please let me know!

I have linked an example of how I build this idea and it works as intended for my project, however I would much rather use the Multiplexer Transform with Match. I generate stores which then are all listened to for a change, if a change is made then it will write to the one output we need. I just change the index of the store to act as our proxy. Then the last store in the list is used as a previous state holder so we can tell which store has changed since we are listening to all of them for a change.
Websocket MULTIPLEXER Example.txt

Is it correctly understood that in your IDEA, you would want to maintain a dynamic number of Store components during the app lifetime, i.e. add/remove Store components while the app is running?

Correct. Currently it is manually adjustable however, as more output proxies (in my case, websocket sends outputs) need to be made, then the list would increase allowing for more proxies. In a more dynamic settings it would be better to have a specific previous state specific store, and then stores that expand as more outputs are added.


Set NoCredentialsProvider on tests configuration

Previously, if an app used Pub/Sub and another service that requires
credentials, using the emulator would prevent credentials to be normally
passed to that service.

Travised failed with the:
/home/travis/build/spring-cloud/spring-cloud-gcp/spring-cloud-gcp-autoconfigure/src/main/java/org/springframework/cloud/gcp/autoconfigure/pubsub/GcpPubSubEmulatorConfiguration.java:19:8: error: Unused import - com.google.api.gax.core.CredentialsProvider.
/home/travis/build/spring-cloud/spring-cloud-gcp/spring-cloud-gcp-autoconfigure/src/main/java/org/springframework/cloud/gcp/autoconfigure/pubsub/GcpPubSubEmulatorConfiguration.java:20:8: error: Unused import -

Oops.. ran mvn clean install but forgot the -Dcheckstyle.skip=true from previous runs :sweat_smile:


cannot set header-font-color to inherit

Hi,
I would like to not style headers (h1..h6) with a specific color ($header-font-color in _settings.scss) but instead to inherit it. I think this makes sense since sometimes the headers should not have different colors at all (but e.g. just differ in size, ...).
If I set the variable to inherit, there is an error since e.g. $subheader-font-color depends on a color value.
Is there a way to only use $header-font-color in other calculations if it is a color?

+1

if you don't set it at all it already will inherit it from body

@r3wt you sure about that?
https://github.com/zurb/foundation/blob/master/scss/foundation/components/_type.scss#L253

This should be reopened. Setting $header-font-color: $body-font-color; is not the same as $header-font-color: inherit;. The h1-h6 font colors should not have to be explicit. They should inherit the color of their container's content, not the color of body.

Agree that this should be reopened.


Fix modifier state not being updated in raw_input

Hi!
This adds a line to the response to ModifiersChanged event that also updates the state of modifiers in raw_input.modifiers.
Fixes the issue that some egui widgets did not work properly. For example, DragValue uses slower speed if shift is pressed while dragging.

LGTM


release configuration for the repo

Hello!
I always check the latest updates of vue-tsc and it is very hard to understand the changes using git tags without any information. Could you configure semantic-release or another approach?

Hi @Oleksii14, is Changelog hard to read the changes of vue-tsc?

My bad 🤦, didn't notice the change log. I was always looking at the tags section and there was no description at all. Thanks!


Docker-compose validator service issue - workaround

When using the docker-compose setup the validator service crashes.
I think this issue located in the docker-compose.yml file:
 volumes:
      - ./bridge.yml:/src/hedera-eth-bridge-validator/config/bridge.yml
      - ./node.yml:/src/hedera-eth-bridge-validator/config/node.yml

Volumes gets the bridge.yml and then adds it to the root directory. Since there is no bridge.yml or node.yml file in the root directory docker instead creates a empty directory.
This is known behavior make sure that files exist in both side (inside host machine and inside container).
If one of them does not exist then docker will create a directory instead of a file.
When this happens the validator will crash.
So a work around is to remove the empty directories node.yml and bridge.yml created by docker in the root directory, and manually create the node.yml and bridge.yml file in the root directory. Then the validator service will work.
I'm not sure what is the best solution to fix it.
If helpfull here is the crash log:
ERROR: for hedera-evm-bridge-validator_validator_1  Cannot start service validator: OCI runtime create failed: container_linux.go:380: starting container process caused: process_linux.go:545: container init caused: rootfs_linux.go:76: mounting "/home/noctilux/bcw/hedera-evm-bridge-validator/node.yml" to rootfs at "/src/hedera-eth-bridge-validator/config/node.yml" caused: mount through procfd: not a directory: unknown: Are you trying to mount a directory onto a file (or vice-versa)? Check if the specified host path exists and is the expected type

ERROR: for validator  Cannot start service validator: OCI runtime create failed: container_linux.go:380: starting container process caused: process_linux.go:545: container init caused: rootfs_linux.go:76: mounting "/home/noctilux/bcw/hedera-evm-bridge-validator/node.yml" to rootfs at "/src/hedera-eth-bridge-validator/config/node.yml" caused: mount through procfd: not a directory: unknown: Are you trying to mount a directory onto a file (or vice-versa)? Check if the specified host path exists and is the expected type
ERROR: Encountered errors while bringing up the project.

Thank you for opening this issue. It has been resolved.


Tracking/managing 2023 annual assessment

This is an issue for our "Ongoing" column for Dec 2022 through March 2023. Since much of the actual work is on the 3PAO's SmartSheet, we'll not detail this exhaustively. However, we may create subtasks so our "Doing" column is correct.

Reviewing strategy for ARL as of 12/6

Reference:
cloud-gov/cg-compliance#211

Annual assessment is wrapped up, so closing this ticket


move from asynk to aflowt; add auction_timeout to instance data

Signed-off-by: stevelr<EMAIL_ADDRESS>

Looks good other than the build failure @stevelr


Loading with Redgate article

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

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

TTYL, AZPRBOT;

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

#sign-off

@TwoUnder : Thanks for your contribution to the Azure documentation! The author, @mausher, has been notified to review your proposed change.

#sign-off

Only the author of the article can use the sign off comment to add the ready-to-merge label.

@TwoUnder - Hi Matt, please resubmit this through the private repo. Thanks, Katie


Update 02-toolsurvey.md

nicer look & feelsy
smaller headlines

thanks! I renamed your commit for clarity


fix(security): can_access with None crashes on builtin roles

SUMMARY
Calling can_access with None values causes an exception when using builtin roles. Will make this more resilient on FAB, but this should be fixed here also.
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report

Merging #10039 into master will decrease coverage by 6.11%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #10039      +/-   ##
==========================================
- Coverage   65.59%   59.48%   -6.12%
==========================================
  Files         585      400     -185
  Lines       31046    12856   -18190
  Branches     3277     3180      -97
==========================================
- Hits        20364     7647   -12717
+ Misses      10501     5031    -5470
+ Partials      181      178       -3

Flag
Coverage Δ

#javascript
59.48% <ø> (+0.11%)
:arrow_up:

#python
?

Impacted Files
Coverage Δ

superset-frontend/src/chart/chartAction.js
49.72% <0.00%> (-0.55%)
:arrow_down:

...rc/explore/components/controls/AnnotationLayer.jsx
2.34% <0.00%> (-0.03%)
:arrow_down:

superset-frontend/src/setup/setupPlugins.ts
0.00% <0.00%> (ø)

...uperset-frontend/src/views/chartList/ChartList.tsx
61.22% <0.00%> (ø)

...rontend/src/explore/components/PropertiesModal.tsx
13.63% <0.00%> (ø)

.../src/components/Select/WindowedSelect/windowed.tsx
64.70% <0.00%> (ø)

.../src/explore/components/controls/SelectControl.jsx
81.11% <0.00%> (ø)

superset/examples/__init__.py

superset/models/sql_types/presto_sql_types.py

superset/utils/dashboard_import_export.py

... and 185 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 91517a5...e81167e. Read the comment docs.

True, I'm incrementally adding them on FAB also


Sign Out option in Navbar for events page

Issue

User unable to sign out.

Description
Currently, When the user visits the rds website, they sign in with git hub but are unable to sign out. This feature will enable them to sign out by providing them with an option to sign out in a dropdown from the navbar.
Solution being implemented

An API for signout in the backend

PR for backend: https://github.com/Real-Dev-Squad/website-backend/pull/679
Backend ticket: https://github.com/Real-Dev-Squad/website-backend/issues/683

We require An option to call for signout API in the front end

In navbar's user greeting we would add a dropdown that would contain links to sign out and visit your profile.

This PR might help you in getting the task done faster

https://github.com/Real-Dev-Squad/website-www/pull/268


[SPARK-35493][K8S] make spark.blockManager.port fallback for spark.driver.blockManager.port as same as other cluster managers

What changes were proposed in this pull request?

spark.blockManager.port does not work for k8s driver pods now, we should make it work as other cluster managers.
Why are the changes needed?

spark.blockManager.port should be able to work for spark driver pod
Does this PR introduce any user-facing change?

yes, spark.blockManager.port will be respect iff it is present  && sparkdriver.blockManager.port is absent
How was this patch tested?

new tests

Test build #138843 has started for PR 32639 at commit cd433ea.

cc @dongjoon-hyun thanks

Test build #138843 has finished for PR 32639 at commit cd433ea.

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

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43365/

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

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43365/

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

Merged to master/3.1/3.0.


Fixes #174 - Set "text/html" Response Content Type for EmailPreviewPlug

Change: Added Pipe for |> Plug.Conn.put_resp_content_type("text/html")
Rationale: When getting "/:id/html" set the response content type to "text/html"
Fixes issue where the iframe in index.html.eex interprets the source as 'text' and embeds in a set of

 tags, such that HTML tags aren't rendered.
Issued occurred for me on:
Chrome - Version 51.0.2704.103 (64-bit)
Safari - Version 9.1.2 (11601.7.6)

Thank you so much! I'm excited to get this in :D

Thanks @davidkuhta!


Treesitter parser for 'html' wasn't found! It is required for basic html tag support.

Describe the bug
After a recent update, markview is refusing to run due to a missing treesitter parser. I am running the latest NeoVIM 0.10 build, built from the release-0.10 tag in the neovim git repository. I also tried the latest master, but have the same issue. I have not looked into building a newer version of treesitter.
Are you using a distro?
No
To Reproduce
Steps to reproduce the behavior:

Install markview.nvim.
Open a markdown document.

Expected behavior
Document opens and is formatted
Actual behavior
I get the following error multiple times:
[ markview.nvim ] : Treesitter parser for 'html' wasn't found! It is required for basic html tag support.

And the document is in its non-formatted form.
Neovim versions tried
NVIM v0.10.2-dev-14+gee57bb5a8
NVIM v0.11.0-dev-582+g93347a67b
NVIM v0.10.1

Bah.. I guess I should read the treesitter documentation. I didn't realize that they have a command to install additional parsers: :TSInstall html did the trick.


Add codecov.io support

Depends on #185.

Awesome. LGTM. It works with codecov Chrome plugin also:


Spoiler Blocks

Describe the feature you'd like
Add a formatting style for "spoilers" or anything that should be hidden by default but can be toggled between hidden and visible by clicking on it.
Describe the benefits this feature would bring to BookStack users
Would allow users to put mildly sensitive information in BookStack so that anyone looking over your shoulder doesn't see it. For instance it could be for common shared passwords for WiFi or any other password where encryption and security are not as important but just ease of letting those with permissions to the page to view the content. This would allow a page to have instructions on how to do a task and include the password but require them to click a toggle of some sort to view the password without it being openly visible by default.
Additional context
I think this could be done with simple CSS and/or JS but would like it to be a simple format from the dropdown menu so anyone could do it easily.

Thanks for the request @mflagler.
Although for a different use case, I think this would be covered by #78. Therefore, I'll likely close this off unless:

This can be done via an alternate semantic HTML element.
There's a strong case why collapsible content blocks would not be suitable for the requested purpose.

Will therefore close this off.


ci: don't run tests for doc only changes

What this PR does / why we need it: To prevent running tests for doc only changes
Which issue this PR fixes: #16

 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.


fix:fix the tail of 0.

Fix the tail of 0.
When a is "aV" and b is "aV0V", the mid return is aV0, which is not expected.
The result should be aV0G

Thanks
I was curious how this worked in the js implementation which was the source of truth. Updated the code to better match that and now the test passes.


MD5 Hash Decrypt

Dear Concern,
I have check to your python script; it is easy to use and find output smoothly. It is great software; but unable to my MD5 hash could not find my password: 2048bfd2430726c9b83a7555d47ac94a. I have check with hashcat.exe 7 all mix character & 10 lowercase uppercase 12 digit; but could no be found it. Could you please checked MD5 Hash to cracking password; if you have found password it will help to me. Also i will donate & contribute accordingly.
Regards,
Mohammad Ashraful Alam

Hey, sorry for delay.
this code based on users input and web sources.
It gathers a couple sites as it prints the results without the need for searching. I believe there might be some hashes that won't be found as clearText, but again - it all lays on existing sources.
Thanks for using, hope to make it even better :)


Do you have any kind of Null iterator/discard iterator

Suppose I need to sort pairs but I don't care about sorted keys.
Is it possible to discard sorted key result?
It will reduce memory footprint that might be very useful sometimes.

We do have a discard iterator:
https://github.com/NVlabs/cub/blob/1.6.4/cub/iterator/discard_output_iterator.cuh
But I don't think it will do you any favors here.  Radix sorting works in
passes, typically 5-7 passes for 32-bit keys.  You would only be able to
discard them on the very last pass, so any performance improvement from
removing only a quarter of the last pass' I/O overhead would be marginal at
best, I would say.
On Mon, Dec 26, 2016 at 2:46 PM, Vladimir<EMAIL_ADDRESS>wrote:

Suppose I need to sort pairs but I don't care about sorted keys.
Is it possible to discard sorted key result?
It will reduce memory footprint that might be very useful sometimes.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/NVlabs/cub/issues/79, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABaFwEMUK30PpIWgYMqGfiSCvekMv9aqks5rMBl9gaJpZM4LV5UC
.

Look like RadixSort only supports pointer as an input. Could you give an example how to use it with iterator?

Could you give an example how to use it with iterator?

There isn't a way to use iterators.  Radix sorting works in passes; need to
read in the input several times.  So we can't really take a custom input
iterator that might transform the input -- it's not useful for any passes
other than the first.  (And you'd still have to provide two regular
pointers for the other passes.)


Revert "Add setuptools to integration test app"

See https://python-poetry.org/blog/announcing-poetry-1.7.0/#official-support-for-python-312 With 1.7, we shouldn't need setuptools.
Also needs https://github.com/paketo-buildpacks/poetry/pull/340
This reverts commit f5db36663c770e508b33ae1942c9bae31d35244c.
Checklist

[x] I have viewed, signed, and submitted the Contributor License Agreement.
[ ] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[ ] I have added an integration test, if necessary.
[x] I have reviewed the styleguide for guidance on my code quality.
[x] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

Official docs (linked above) says  If you rely on setuptools or wheel, please make it an explicit dependency in your pyproject.toml. - So I think it's the right choice to keep it in dependencies


Error with the cors-proxy endpoint

The https://cors-proxy-ccg.herokuapp.com service seems to be down and it's causing that the library doesn't works

+1


[REVIEW comments] for the paper and code submitted to JOSS

Hello, @RayLei-TRI and @JosephMontoya-TRI  sorry for my late review report of the manuscript submitted to Journal of Open Source Software https://github.com/openjournals/joss-reviews/issues/5476
Here are my minor comments which you may consider during revision

[x] I personally think the requirements.txt is not complete. To test the code, I built a new virtual environment and installed the code following the instructions. When testing the examples, the code reminded me that I needed to install ase, numpy, and so on. I realized some of these libraries were included in the requirements.txt of earlier versions, could authors explain why did they remove it? To make the code ready completely, I would suggest including all the necessary and major libraries in the requirements.txt

[x] In the manuscript, when referring to the "cffi" library, it may be more appropriate to use "Cffi" or "CFFI" to distinguish it as a specific terminology. Upon reading the summary at the beginning, I mistakenly assumed it was a typographical error.

[x] The reference of ray in the manuscript was not formatted properly (e.g. "Ray?" appears in the current summary).

[ ] I noticed that CIF files taken from ICSD couldn't been read by ase_read. However, if I convert the CIF from ICSD to a CIF from VESTA software. It works. I know this issue doesn't from GMP-featurizer itself, but you may consider raising a warning if read of CIF fails. Note that I only test one CIF from ICSD, and you may cross-test it for confirmation.

[x] In the section Overview of manuscript, "Please refer to section ??" should be fixed.

[x] Authors are invited to create a webpage to document the tutorials and manuals.

Hi @yw-fang , Thank you very much for your great and detailed suggestions! We have updated the package accordingly, and here are our responses:

We took out ASE and pymatgen from the requirement list, because they are not required to run the featurizer itself, and adding them as requirements might make the package cumbersome. However, since they are popular packages for the likely users of GMP-Featurizer, we provided converters to interface GMP-featurizer with them (e.g. for IO purposes).  However, it is a little strange that numpy was not installed since it was in the requirement.txt file, and I was not able to reproduce the problem. Would you mind sharing more detail about the problem? It would be great if you could include information about the OS, python version, error details, and we will try to reproduce and fix the bug.

Thanks for the suggestion! we have updated it to CFFI.

Thanks for catching it!  We fixed it earlier on the main branch but did not merge the change into the JOSS branch until now. Now it should be fixed.

Thanks for the suggestion! Would you mind sharing more information about the error? Did ase_read() fail silently, and GMP-featurizer errored out because of it? It would be great if you could share with us the example file you used, and we will try to reproduce the error and update the package accordingly.

Same as 3), it should be fixed now, thanks a lot for catching it!

Thanks for the suggestion! We have created a minimal working website for hosting the docs and manuals: https://tri-amdd.github.io/GMP-featurizer/html/index.html

Thanks again for your review comments!

Hello @RayLei-TRI thanks for your quick response.
My comments are as follows

Hi @yw-fang , Thank you very much for your great and detailed suggestions! We have updated the package accordingly, and here are our responses:

We took out ASE and pymatgen from the requirement list, because they are not required to run the featurizer itself, and adding them as requirements might make the package cumbersome. However, since they are popular packages for the likely users of GMP-Featurizer, we provided converters to interface GMP-featurizer with them (e.g. for IO purposes).  However, it is a little strange that numpy was not installed since it was in the requirement.txt file, and I was not able to reproduce the problem. Would you mind sharing more detail about the problem? It would be great if you could include information about the OS, python version, error details, and we will try to reproduce and fix the bug.

It makes sense, since you'd like to make the code lightweight by excluding the heavy libraries. As for numpy, it's my fault. The command 'pip install requirements.txt' can install numpy as expected.

Thanks for the suggestion! we have updated it to CFFI.
Thanks for catching it!  We fixed it earlier on the main branch but did not merge the change into the JOSS branch until now. Now it should be fixed.
Thanks for the suggestion! Would you mind sharing more information about the error? Did ase_read() fail silently, and GMP-featurizer errored out because of it? It would be great if you could share with us the example file you used, and we will try to reproduce the error and update the package accordingly.

Sure. Here is the error information:
--------------------------------------------------------------------------
RuntimeError                              Traceback (most recent call last)
Cell In[3], line 6
      3 from ase.io import read as aseread
      5 # Loading cif file as a ase atoms object
----> 6 image = aseread("./SmFeO3.cif")
      7 # The input to the featurizer should be a non-empty list
      8 images = [image]

File ~/miniconda3/envs/py38/lib/python3.8/site-packages/ase/io/formats.py:736, in read(filename, index, format, parallel, do_not_split_by_at_sign, **kwargs)
    733     return list(_iread(filename, index, format, io, parallel=parallel,
    734                        **kwargs))
    735 else:
--> 736     return next(_iread(filename, slice(index, None), format, io,
    737                        parallel=parallel, **kwargs))

File ~/miniconda3/envs/py38/lib/python3.8/site-packages/ase/parallel.py:275, in parallel_generator.<locals>.new_generator(*args, **kwargs)
    269 @functools.wraps(generator)
    270 def new_generator(*args, **kwargs):
    271     if (world.size == 1 or
    272         args and getattr(args[0], 'serial', False) or
    273         not kwargs.pop('parallel', True)):
    274         # Disable:
--> 275         for result in generator(*args, **kwargs):
    276             yield result
    277         return

File ~/miniconda3/envs/py38/lib/python3.8/site-packages/ase/io/formats.py:803, in _iread(filename, index, format, io, parallel, full_output, **kwargs)
    801 # Make sure fd is closed in case loop doesn't finish:
    802 try:
--> 803     for dct in io.read(fd, *args, **kwargs):
    804         if not isinstance(dct, dict):
    805             dct = {'atoms': dct}

File ~/miniconda3/envs/py38/lib/python3.8/site-packages/ase/io/cif.py:604, in read_cif(fileobj, index, store_tags, primitive_cell, subtrans_included, fractional_occupancies, reader)
    602 # Find all CIF blocks with valid crystal data
    603 images = []
--> 604 for block in parse_cif(fileobj, reader):
    605     if not block.has_structure():
    606         continue

File ~/miniconda3/envs/py38/lib/python3.8/site-packages/ase/io/cif.py:537, in parse_cif_ase(fileobj)
    534 if not line or line.startswith('#'):
    535     continue
--> 537 yield parse_block(lines, line)

File ~/miniconda3/envs/py38/lib/python3.8/site-packages/ase/io/cif.py:500, in parse_block(lines, line)
    498 assert line.lower().startswith('data_')
    499 blockname = line.split('_', 1)[1].rstrip()
--> 500 tags = parse_items(lines, line)
    501 return CIFBlock(blockname, tags)

File ~/miniconda3/envs/py38/lib/python3.8/site-packages/ase/io/cif.py:189, in parse_items(lines, line)
    187     tags[key.lower()] = value
    188 elif lowerline.startswith('loop_'):
--> 189     tags.update(parse_loop(lines))
    190 elif lowerline.startswith('data_'):
    191     if line:

File ~/miniconda3/envs/py38/lib/python3.8/site-packages/ase/io/cif.py:161, in parse_loop(lines)
    158 headers = list(parse_cif_loop_headers(lines))
    159 # Dict would be better.  But there can be repeated headers.
--> 161 columns = parse_cif_loop_data(lines, len(headers))
    163 columns_dict = {}
    164 for i, header in enumerate(headers):

File ~/miniconda3/envs/py38/lib/python3.8/site-packages/ase/io/cif.py:149, in parse_cif_loop_data(lines, ncolumns)
    147 if tokens:
    148     assert len(tokens) < ncolumns
--> 149     raise RuntimeError('CIF loop ended unexpectedly with incomplete row')
    151 return columns

RuntimeError: CIF loop ended unexpectedly with incomplete row

You may reproduce the error using this CIF from ICSD database:  https://ehubox.ehu.eus/s/QnrJX25JGA435Gt  (the link expires on June 24 Spain time)

Same as 3), it should be fixed now, thanks a lot for catching it!
Thanks for the suggestion! We have created a minimal working website for hosting the docs and manuals: https://tri-amdd.github.io/GMP-featurizer/html/index.html

It appears to be a promising start! I'll keep an eye on it! Thanks for your contribution to the community.

Thanks again for your review comments!

Hi @yw-fang, Thank you for the comment and sorry for the late reply.
It seems the IO error was cost by the ase io module, and it printed out the details of the error as well, so I think it might not make sense to address it in this package. However, I did add a sample cif file in the examples folder so that users don't have to find their own to just try the package, hope this address your concern
Thanks again, and please let me know if you have any additional comments :)

Hi @yw-fang, Thank you for the comment and sorry for the late reply. It seems the IO error was cost by the ase io module, and it printed out the details of the error as well, so I think it might not make sense to address it in this package. However, I did add a sample cif file in the examples folder so that users don't have to find their own to just try the package, hope this address your concern Thanks again, and please let me know if you have any additional comments :)

I agree. As I mentioned initially, this issue doesn't come from your code itself. I'm fine with your response.
I'll tick all the tasks, and recommend the present version to be published.
Of course, feel free to close this GIthub issue.


os x linker supports -rpath in xcode 3 (os x 10.5)

This issue was originally created at: 2008-07-03 17:04:31.
This issue was reported by: lukasnellen.

lukasnellen said at 2008-07-03 17:04:32

One of the changes Apple introduced in Leopard (os x 10.5) is the support for RPATH (see http://developer.apple.com/releasenotes/DeveloperTools/RN-dyld/index.html). This means that issue #1892 and its solution don't hold anymore.
Unfortunately, the implementation of rpath differs from that under linux, so that simply switching back to gnulink might not be the right way to make portable builds easy.

gregnoel said at 2008-07-15 20:27:25

Bug party triage.  Gary to look at the OS X versions and see if rpath variations can be categorized.  If they can, assign the issue to an actual milestone based on the complexity.
This is part of a larger issue concerning changes by OS X with newer versions; at least one other place is the keywords needed for shared libraries and bundles.  Perhaps some secondary identifier should be extracted when it's determined that the platform is OS X; other code could key off the identifier as such changes are identified.

garyo said at 2008-10-29 19:06:35

Here's what I learned.

gcc on 10.5 supports -rpath, earlier versions didn't.  But it is a little odd, in that it only sets the path for special libs/frameworks linked in a certain way.
We can check the Mac OS version with /usr/bin/sw_version:

% /usr/bin/sw_vers
ProductName:    Mac OS X
ProductVersion: 10.4.11
BuildVersion:   8S2167

dyld (MacOS X version of ld) supports executables with RPATH, but only searches for libs/frameworks with install names beginning with @rpath.  See the tech note above.  Not totally relevant to this bug, but will be needed to make a test case.

Simplest way forward is to use gnulink again, but only allow RPATH if the linker supports it, either by parsing sw_vers or doing a test link a la autotools.  Otherwise ignore RPATH setting.
Since I've researched this now, I'm resetting it back to unspecified.  I don't think it's a huge deal to fix now, so I recommend 2.x/p4 unless someone has a need for it earlier.  And of course users can always just add the RPATH setting to LINKCOM anyway, that's why I suggest p4.

garyo said at 2008-10-29 19:07:08

... and changing owner back to nobody.

gregnoel said at 2008-11-21 16:48:34

Bug party triage.  Getting OS-specific version info can easily be factored into the toolchain revamp, which will also restructure how platform configuration is done, so include it with that work.

gregnoel said at 2008-12-26 13:20:23

Adjust triage of issues.

semi-related Stack Overflow:
https://stackoverflow.com/questions/54591196/py-initialize-throws-modulenotfounderror-no-module-named-encodings/54604729#54604729
Rpath had to passed in the LINKFLAGS instead of RPATH because it's not supported correctly on OSX.

@bdbaddog any thoughts on this old issue?


First time not load PDF File

I am trying to open pdf file after download first time it's through the below error.second time PDF it's showing
/storage/emulated/0/Android/data/ae.gov.mrhe/files/ISKANAAPP/Files/sample(22).pdf not found as a file or resource.

I would guess you are trying to open the file before its actually downloaded so the file is not found, are you checking the file has been downloaded before trying to open?
When you try a second time i would imagine the download has already completed.
Hard to say without any code but that would be the first place i would be looking.

I would guess you are trying to open the file before its actually downloaded so the file is not found, are you checking the file has been downloaded before trying to open?
When you try a second time i would imagine the download has already completed.
Hard to say without any code but that would be the first place i would be looking.

I also facing same issue like above, I already download and open it on next page (passing path).


[BUG] links page in not responsive on mobile view

Has this bug been raised before?

[X] I have checked "open" AND "closed" issues and this is not a duplicate

Where did you find this bug?
Local Development
Version of BioDrop (for example "v1.2.3")
2.96.0
Description
While testing the project locally I noticed that the account/links page is not responsive.
Steps to Reproduce
If you know the steps, follow the below format and provide steps to reproduce
For example:

Go to http://localhost:3000/account/manage/links page in mobile view.

Screenshots

Do you want to work on this issue?
No
If "yes" to above, please explain how you would technically implement this
No response

hi @sital002 can you confirm if this happens on production and on a mobile device?  I have tested on my device and cannot replicate. (Android pixel running chrome)

It does exists on production as well if we pinned emabled and goruped the link here is the screenshot in production

@amandamartin-dev @sital002 I didn't check it on my phone but leaving the links section aside, even the buttons seem out of proportion for smaller screen sizes.
Coming to the links, we will have to come up with a different design on how to show the links data.

Instead of text for Pinned and Enabled we will use Icons in mobile view, does it make sense?

It does exists on production as well if we pinned enabled and grouped the link here is the screenshot in production

Thank you for clarifying this is seen when there is a group.  I was able to confirm. I saw you do not want to work on this but if you havre a suggestion on the best way to resolve feel free to offer a solution.

Instead of text for "Pinned" and "Enabled" we will use Icons in mobile view, does it make sense?

Thank you for offering this idea.  How will mobile users know what the icons each mean if we switch to icons?

@amandamartin-dev @sital002 I didn't check it on my phone but leaving the links section aside, even the buttons seem out of proportion for smaller screen sizes.
Coming to the links, we will have to come up with a different design on how to show the links data.

Do you have a suggestion for improving this area?

It does exists on production as well if we pinned enabled and grouped the link here is the screenshot in production

Otherwise, we can do like this

It does exists on production as well if we pinned enabled and grouped the link here is the screenshot in production

Otherwise, we can do like this

Oh I like the idea of stacking the tags underneath! I would like some other folks to weigh in though on the design change.  WDYT @eddiejaoude ?

Glad you liked it! Please assign me if approved

yep stacking underneath seems like a good solution 👍

I have assigned to you @sohail-313

I am unable login because of missing "client_id"

I am unable login because of missing "client_id"
screenshot of error

You need to add the environement variable for Github Oath to work. here is the link on how to setup your environement variable https://www.biodrop.io/docs/environments/environment-variables

@amandamartin-dev @sital002 I didn't check it on my phone but leaving the links section aside, even the buttons seem out of proportion for smaller screen sizes.
Coming to the links, we will have to come up with a different design on how to show the links data.

Do you have a suggestion for improving this area?

@amandamartin-dev I would suggest doing few things.

Adding some space between different elements. It will give clear design for smaller screens.
To address to the buttons issue above, we could use fluid design techniques to makes the buttons fluid depending upon the width of the screen/device.

This issue was assigned one month back and is not yet closed. I want to work on this issue can you assign this to me?

This issue was assigned one month back and is not yet closed. I want to work on this issue can you assign this to me? eddiejaoude

@sid10on10 as you will be able to see from the PR the assignee and the maintainers are still working on this so at this time we will not be assigning this issue to anyone else. Please also do not tag specific maintainers - all maintainers receive notifications and can deal with these.  I suggest you take a look at this project's Contributing Guide, to familiarise yourself as to how this project is run.


[DOC] Update ingest pipelines content and page navigation

Description
Updates ingest pipelines content and page navigation
Issues Resolved
#4193
#4283
Checklist

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

See PR #4299. Revised pipeline documentation included in that PR.


How can we fix "Invalid Bundle Structure" error when submitting to app store

Hi, We have our project.yml file configuration as below:
dependencies:
- sdk: libABC.a
link: true
When we submit the build to app store, it gives us the following error:
Invalid Bund Structure - The binary file libABC.a is not permitted. Your app can't contain standalone executables or libraries, other than a valid CFBundleExecutable of supported bundles. Refer to the Bundle Programming Guide for information.
What kind of changes should we make in project.yml to fix this?
We found a similar issue on stackoverflow, just not sure what are the exact changes needed.
https://stackoverflow.com/questions/20251252/invalid-bundle-structure-the-app-may-contain-only-one-executable-file#:~:text=This error occurs when a,Copy Bundle Resources build phase.
Thanks!

This is described here: https://github.com/yonaskolb/XcodeGen/issues/800#issuecomment-597148421
Simply remove these files from Sources step


Update macOS cache test to use actions/setup-python

No real reason for this test to use Brew's Python. It's probably faster to just install from the GitHub official setup-python action.
Originally posted by @zanieb in https://github.com/astral-sh/uv/pull/2729#discussion_r1545394084

This should be a simple change to the ci.yaml file and is a great way to make a first contribution to the project.

I thought it was intended to literally test Brew Python installs.

system-test-macos-aarch64 is intended to cover a Brew Python install cache-test-macos-aarch64 should be Python interpreter agnostic.

Sorry I did this in #2735 because CI was broken.


Encoding problem?

store works very fine but I wanted to investigave it with Redis Insight and saw this wierd encoding. Is this intentional or is there a way to fix it? thank you.

It's encoded with Rust MsgPack.


make some binaries more readable

I am not too certain about this PR - it tries to make hex strings more inspectable - so that people do not freak out when they see a big hex blob in the middle of the code, and start thinking about the XZ compromise.

Very nice, thanks @nyurik for your time!


Working Docker setup -- patch

These changes will allow docker to be setup and

@allwyn-pradip please make sure all the circleci tests are passing.
Looks like you have some flake8 errors
src/iris/webhooks/freshping.py:8:1: F401 'iris.webhooks.webhook.webhook' imported but unused
src/iris/webhooks/freshping.py:41:34: F821 undefined name 'basestring'
src/iris/webhooks/freshping.py:71:1: W293 blank line contains whitespace
Makefile:29: recipe for target 'flake8' failed

update the conflicts


Puddle code is re-used alongside kudzu: great.
Spreader updates per second: trash fire.
Touch base on this after mirror makes her changes.

Was all up in spreader system for #18218.
Before considering getting elbow deep in Node Group and Spreader algorithm optimisation,
"SpreaderCooldown" seems to directly control the spreader update rate.
This could be changed dynamically to improve performance?
Is this already sorted out, or should I try some stuff?


Module rethink

R: @jeffposnick @addyosmani @philipwalton
Fixes #943
This is just for workbox-routing at the moment, but wanted to get early feedback before proceeding to all modules.

LGTM, thanks for doing all this!


bump for release

No changes needing a change description found.

You can try these changes here

🛝 Playground
🌐 Website
📚 Next docs


Refactor bullet chart vars

Need to refactor bullet chart vars. The existing vars were created from a mock, but they need to be updated to match the implementation in react-charts.
For example, we need a red color scale to show negative primary measures. Additional vars are needed, while some vars are no longer used at all. A few var names have changed slightly as well.
Bullet chart implementation:
https://github.com/patternfly/patternfly-react/pull/2708

:tada: This issue has been resolved in version 2.26.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Use a loader while generating addresses

I added nice logs and a spinning loader. Useful to see that the program is running and not stuck.
It doesn't affect the file log at all ( -l option )

Hi,
I'm not sure who is supposed to review this PR, @kvhnuke is the author of 100% of the code, but has not committed for months.
Is it @tayvano ?

Thanks @bokub that's a really cool addition :)

You're welcome!
Don't forget to NPM publish :+1:

Hi @kvhnuke , I see that the last version of vanity-eth is still 1.0.4  on NPM.
Is there a particular reason you don't want to publish, or did you just forget to do it?

Ooops yes completely forgot I was working on another project. My bad! Will do it now

Published! Thanks again!


Create configOnly flag for android

#93407
Inspired by the work the iOS team has done to create a similar flag.
Additionally this is pre work for helping the flutter/packages work move towards more offline CI.

Looks like you have compilation errors in your test

Looks like you have compilation errors in your test
https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket/8787336610580064177/+/u/Analyze/analyze/test_stdout

Thank you, For my own notes I was able to find and fix these by useing. flutter analyze --flutter-repo


Fix typo

a -> an

@changeworld : Thanks for your contribution! The author, @mumian, has been notified to review your proposed change.

@changeworld - Thank you for the feedback.
#sign-off


Alert Detail Activity and related changes

-> A detail screen for alert have been introduced from where you can change the state of alert read important detail about it. Read and send comments on the alert.
-> Previously Android client showed only Open Alerts which have been changed to something similar server. By it shows {OPEN & ACK} when you press '+' sign it includes resolved alerts.
Video of current work
https://drive.google.com/file/d/0B8tYKC1-UERHSDVSa1hGQ3VEbzQ/view?usp=sharing

@danielpassos  @pilhuhn Can you please review and merge the PR. A lot of new changes are being collected for next PR
Thanks :)

👍

landed at https://github.com/hawkular/hawkular-android-client/commit/6a8ec47302c48cb7bea8ef638e17aa6ac2e19820


Location must not be Hardcoded Regression

In the very latest release of ARM TTK for The Location must not be hardcoded rule.
TTK Release - https://github.com/Azure/arm-ttk/releases/tag/20220311
Repo Template - https://github.com/drewkg/Azure/blob/main/ARM/Resource/AutomationAccountWebHook/azureDeploy.json
I have confirmed that the previous version of ARM TTK did fail this template, but the latest version the template fails the Location rule.

I'm also experiencing the same issue with latest version.
I got:
[-] Location Should Not Be Hardcoded (944 ms)
mainTemplate.json must use the location parameter, not resourceGroup().location or deployment().location        (except when used as a default value in the main template)
On mainTemplate.json where resourceGroup().location is only used as default value.

Also seeing this test fail this morning
  [-] Location Should Not Be Hardcoded 49ms
    WriteErrorException: azuredeploy.json must use the location parameter, not resourceGroup().location or deployment().location (except when used as a default value in the main template)
    at <ScriptBlock>, D:\a\1\a\arm-ttk\arm-ttk\testcases\deploymentTemplate\Location-Should-Not-Be-Hardcoded.test.ps1: line 94
    at Test-Case, D:\a\1\a\arm-ttk\arm-ttk\Test-AzTemplate.ps1: line 254
    at <ScriptBlock>, D:\a\1\a\arm-ttk\arm-ttk\Test-AzTemplate.ps1: line 471

Hey, Even we are facing same issue since 2 days and found the fix with the recent code changes.
The variable is named as paramSection instead of paramsSection.

Why wait and just create a PR ;)

fixed in #611 - the unit tests for this particular test where not running, so that was fixed in #612
@v-rucdu - thanks for the debug
@spoelly - thanks for the PR

oh - and this is in 0.5


Tutorial: JSON Table Schema

Already have basics here: http://frictionlessdata.io/guides/json-table-schema
May want to add there or have separate mini-tutorials (if that is focused just on what JTS is ...)
Things to add:

[x] Inferring the schema - tools
[x] Use for validation
[ ] Profiles / registry (if we have it)

This issue was moved to frictionlessdata/frictionlessdata.io#40


[RELEASE] 2024 April 29 - Tangle dApp

Summary of changes
Provide a detailed description of proposed changes.

Updated CHANGELOG.md file.
Bumped Tangle dApp version

Proposed area of change
Put an x in the boxes that apply.

[ ] apps/bridge-dapp
[ ] apps/hubble-stats
[ ] apps/stats-dapp
[x] apps/tangle-dapp
[ ] apps/testnet-leaderboard
[ ] apps/faucet
[ ] apps/zk-explorer
[ ] libs/webb-ui-components

I will proceed with the merge without requiring approval, as this solely pertains to release updates, rather than alterations in the dApps code.


Fixes for styling bugs

Should fix the reported styling issues. The choice of placement of the Pending Posts button is totally arbitrary. I thought about restyling it a little to fit in with the common buttons in the nav bar instead of the purple background.

Oops, there's an issue outstanding with single-thread view that prevents the LOL script from working. I'll fix it before the end of tonight.

Should be good to go. Let me know if I missed anything.


TensorFlow Lite C++ minimal example build

Click to expand!
Issue Type
Build/Install
Have you reproduced the bug with TF nightly?
No
Source
binary
Tensorflow Version
Build error
Custom Code
Yes
OS Platform and Distribution
Raspbian GNU/Linux 11 (bullseye) armv
Mobile device
Raspberry Pi 3 Model B Plus Rev 1.3
Python version
Python 3.9.2 (default, Mar 12 2021, 04:06:34)
Bazel version
No response
GCC/Compiler version
[GCC 10.2.1 20210110] on linux
CUDA/cuDNN version
n/a
GPU model and memory
n/a
Current Behaviour?
[ 28%] Building ASM object _deps/xnnpack-build/CMakeFiles/microkernels-prod.dir/src/cs16-bfly4/cs16-bfly4-samples1-asm-aarch32-neon-x4.S.o
[ 35%] Building CXX object _deps/abseil-cpp-build/absl/strings/CMakeFiles/absl_strings.dir/numbers.cc.o
/tmp/ccWtfVcs.s: Assembler messages:
/tmp/ccWtfVcs.s:65: Error: selected processor does not support vsdot.s8 q8,q12,d7[0]' in ARM mode /tmp/ccWtfVcs.s:67: Error: selected processor does not support vsdot.s8 q9,q10,d7[0]' in ARM mode
Standalone code to reproduce the issue
# https://github.com/tensorflow/tensorflow/tree/master/tensorflow/lite/examples/minimal
git clone https://github.com/tensorflow/tensorflow.git tensorflow_src
mkdir minimal_build
cd minimal_build
cmake ../tensorflow_src/tensorflow/lite/examples/minimal
cmake --build . -j

Relevant log output
gmake[1]: *** Waiting for unfinished jobs....
[ 35%] Building CXX object _deps/abseil-cpp-build/absl/strings/CMakeFiles/absl_strings.dir/str_replace.cc.o
[ 35%] Building CXX object _deps/abseil-cpp-build/absl/strings/CMakeFiles/absl_strings.dir/str_split.cc.o
[ 35%] Building CXX object _deps/abseil-cpp-build/absl/strings/CMakeFiles/absl_strings.dir/string_view.cc.o
[ 35%] Building CXX object _deps/abseil-cpp-build/absl/strings/CMakeFiles/absl_strings.dir/substitute.cc.o
[ 35%] Linking CXX static library libabsl_strings.a
[ 35%] Built target absl_strings
gmake: *** [Makefile:149: all] Error 2

Hi @baqwas
Can you try with latest stable version r2.13?
I was able to build with r2.13 without any error. Please find the screenshot.

Thanks.

Thanks, @pjpratik. I had limited myself to r2.12 owing to an incorrect understanding of the prerequisites.
Regards.


use django-guardian to set up investigation permissions

Each user should have their permissions on objects determined at the Investigation level. If a user is added to an Investigation as a collaborator, they should have access to the ContextMap, ConnectionsMap, and Pins in that investigation.
However, Dataset objects should have their own permissions. These should be public unless otherwise specified, and in the case that they are private, these should have their own lists of users who can read them. If a Dataset is public, it can be added to any Investigation's ContextMap.

main now uses django-guardian to determine per-object permissions on both the Investigation model and the Dataset model.
The Investigation model can be shared with more investigators, who have write access, and observers, who have read access. The Dataset model can be either public or private to an explicit set of users.


”Sfântu Gheorghe” is listed in the drop-down menu

Description:
"Sfântu Gheorghe" is listed in the drop-down menu alongside its corresponding county and commune, according to the law.
Precondition:
The website is up an running.

Step 1
Type "Sfântu Gheorghe" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Sfântu Gheorghe, TULCEA (Sfântu Gheorghe)".

Environment: Run Tests - 6207
Browser: Chrome 123
OS: Windows 10
Steps to reproduce:
1 Open the @Adelina Guliman Test case ui.orase
2. Check TC: ”Sfântu Gheorghe” is listed in the drop-down menu](https://github.com/peviitor-ro/ui.orase/issues/6207)
#6207
Expected results:
Precondition (request):
The website link (https://peviitor-ro.github.io/ui.orase/) is up and running
The link to the law is displayed :"Sat Sfântu Gheorghe, TULCEA (Sfântu Gheorghe)"

Actual results:
In "https://peviitor-ro.github.io/ui.orase/" is displayed : "Oras Sfântu Gheorghe, TULCEA (Sfântu Gheorghe)"

Fixed! Poti sa retestetzi


LabelOverlapMeasuresImageFilter, get measures for specific label values

Please add a feature, so that it is possible to get overlap measures for specific label values as it is in the original ITK code of the filter. Thank you!

ref: https://discourse.itk.org/t/how-to-pass-a-specific-label-to-labeloverlapmeasuresimagefilter/2300/2


add initial git pre commit hook options

Resolve #15
This PR adds some pre commit hooks options:

mypy
flake8
black
isort

pre-commit-hooks are very useful to check code before commit.
let me know if these options are useful to pyopensci :)

I think this is cool! However I think we should definitely document all of this before exposing configuration options to folks. Could we add documentation for these to the dev guide so people know what they mean?

thanks for the feedback @choldgraf ! that makes a lot of sense. I will work on that!
I also was thinking that maybe would be good to test the pre-commit hook itself. to avoid to add a recipe if some bug. but I didn't find yet a good approach. any thought?

@choldgraf I opened a PR to add some information about git pre-commit hook into dev_guide: https://github.com/pyOpenSci/dev_guide/pull/57

re: tests, I think we could add a test that includes running pre-commit and checks whether there is a diff. I've used something like this in other projects (with github-actions):
  pre-commit:

    runs-on: ubuntu-latest

    steps:
    - uses: actions/checkout@v2
    - name: Set up Python 3.7
      uses: actions/setup-python@v1
      with:
        python-version: 3.7
    - name: Install dependencies
      run: |
        python -m pip install --upgrade pip
        pip install -e .[code_style]
    - name: Run pre-commit
      run: |
        pre-commit run --all-files || ( git status --short ; git diff ; exit 1 )

@choldgraf sounds good! I will work on that! thanks!

@choldgraf I added the test for github actions, but as I am not a collaborator for this repo, it was not triggered.
should I move it to travis-ci?

it seems that the github action was triggered .. but not listed here

Huh - that's weird, I'd have thought you could do this since you're a member of the POS organization. I just added you as a maintainer to this repo, can you try again?

thanks @choldgraf !
I cannot see the new actions here or in the previous list I receive the error: https://github.com/OpenScienceLabs/cookiecutter-pyopensci/actions
not sure what I am missing here .. should I create a workflow first direct in the actions tab?

hmmm, maybe it needs to be merged first before it'll work. Is this ready to merge otherwise? Then we can spot-check github actions in another PR?

I tested it locally ... so I think it is ready to merge! thanks! I will check in another PR the github actions.

sounds good! thanks for making this contribution @xmnlab :-)

thank you @choldgraf for all the recommendations and tips!

it seems the github actions worked: https://github.com/pyOpenSci/cookiecutter-pyopensci/runs/696767222?check_suite_focus=true


https://docs.neo.org/docs/en-us/tooldev/concept/wallets.html is in Chinese.

As described in the title.

This issue is fixed.


should init p.sandbox earlier

otherwise it may lead to a panic
Signed-off-by: Wang Xu<EMAIL_ADDRESS>

LGTM


Fix 'Redundant blank line at the start of a code block should be deleted.' issue in Stack\Maximum Rectangular Area in a Histogram\Maximum_Rectangular_Area_in_Histogram.cpp

CodeFactor found an issue: Redundant blank line at the start of a code block should be deleted.
It's currently on:
Stack\Maximum Rectangular Area in a Histogram\Maximum_Rectangular_Area_in_Histogram.cpp:45

@skully-coder please assign me this issue

@skully-coder this issue and am I allowed to fix all blank line  issues ?

@Mr-vabs you can fix all blank lines issues.