After migration from V1 to V2 Groups failing  in search resolvers.

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
14.7.0
Amplify CLI Version
8.2.0
What operating system are you using?
Mac
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
no manual changes made
Amplify Categories
api
Amplify Commands
Not applicable
Describe the bug
After a migration from v1 to v2 and changing our schema to meet the requirements from the official document GraphQL Transformer v1 to v2 migration.
Then while testing we noticed that we were not able to search all records a user had rights to, users could only see records that they own/ created.
After a long day of debugging and try to find what went wrong. We noticed that the auto generated VTLs for the auth.req (Query.searchReceivers.auth.1.req.vt) was missing the .keyword when setting our groups.
e.g: Before:
  "terms_set": {
      "groupsCanRead": {
          "terms":       $util.defaultIfNull($ctx.identity.claims.get("cognito:groups"), ["___xamznone____"]),
          "minimum_should_match_script": {
              "source": "1"
      }
    }

Then by creating a custom resolver and adding the .keyword like this. it started working.
  "terms_set": {
      "groupsCanRead.keyword": { <---- added the missing keyword
          "terms":       $util.defaultIfNull($ctx.identity.claims.get("cognito:groups"), ["___xamznone____"]),
          "minimum_should_match_script": {
              "source": "1"
      }
    }

Expected behavior
To be able to search and retrieve all the records that the user owns and has rights to.
Reproduction steps

Model with owner and group auth rules
Amplify push api
Search for records using the rules

GraphQL schema(s)

# Put schemas below this line
type Receiver
  @model
  @auth(
    rules: [
      { allow: owner, ownerField: "creatorId" }
      { allow: groups, groupsField: "groupsCanRead", operations: [read] }
      { allow: groups, groupsField: "groupsCanUpdate", operations: [update] }
      { allow: groups, groupsField: "groupsCanDelete", operations: [delete] }
      { allow: groups, groupsField: "admins" }
    ]
  )
  @searchable {
  id: ID!
.......
}

Log output

# Put your logs below this line

Additional information
No response

Hey @n-francky, Thank you for reaching out. On replicating the schema in my application I observed the "groupsCanRead.keyword" did get generated locally in Query.searchReceivers.auth.1.req.vtl.
In order to further dive deeper into the issue, could you please run amplify api gql-compile and the check the Query.searchReceivers.auth.1.req.vtl file to see if the keyword has been appended. If in case it did not could you please provide us the cli.json file.

Hello @n-francky, following up on this to see if you are experiencing any issues.

@ykethan Thanks for your response.
no still does not work on my side.
here is the cli.jso
{
  "features": {
    "graphqltransformer": {
      "addmissingownerfields": false,
      "improvepluralization": false,
      "validatetypenamereservedwords": true,
      "useexperimentalpipelinedtransformer": true,
      "enableiterativegsiupdates": false,
      "secondarykeyasgsi": false,
      "skipoverridemutationinputtypes": false,
      "transformerversion": 2,
      "suppressschemamigrationprompt": true,
      "securityEnhancementNotification": false,
      "showfieldauthnotification": false
    },
    "frontend-ios": {
      "enablexcodeintegration": false
    },
    "auth": {
      "enablecaseinsensitivity": false,
      "useinclusiveterminology": false,
      "breakcirculardependency": false,
      "forcealiasattributes": false,
      "useenabledmfas": false
    },
    "codegen": {
      "useappsyncmodelgenplugin": false,
      "usedocsgeneratorplugin": false,
      "usetypesgeneratorplugin": false,
      "cleangeneratedmodelsdirectory": false,
      "retaincasestyle": false,
      "addtimestampfields": false,
      "handlelistnullabilitytransparently": false,
      "emitauthprovider": false,
      "generateindexrules": false,
      "enabledartnullsafety": false
    },
    "appsync": {
      "generategraphqlpermissions": false
    },
    "latestregionsupport": {
      "pinpoint": 0,
      "translate": 0,
      "transcribe": 0,
      "rekognition": 0,
      "textract": 0,
      "comprehend": 0
    },
    "project": {
      "overrides": true
    }
  }
}

Hello @n-francky, Thank you for providing me the cli.json file. I tied to replicate the behaviour with the provided cli.json file but did not observe the missing keyword behaviour.
Could you please try to delete the build folder under the api ->  and run amplify api gql-compile. Then check the build Query.searchReceivers.auth.1.req.vtl resolver file.
I did observed a bit of difference in the cli.json from my amplify project but did not run into the missing keyword behaviour. Please find my cli.json as follows.
{
  "features": {
    "graphqltransformer": {
      "addmissingownerfields": true,
      "improvepluralization": false,
      "validatetypenamereservedwords": true,
      "useexperimentalpipelinedtransformer": true,
      "enableiterativegsiupdates": true,
      "secondarykeyasgsi": true,
      "skipoverridemutationinputtypes": true,
      "transformerversion": 2,
      "suppressschemamigrationprompt": true,
      "securityenhancementnotification": false,
      "showfieldauthnotification": false,
      "usesubusernamefordefaultidentityclaim": true,
      "usefieldnameforprimarykeyconnectionfield": false
    },
    "frontend-ios": {
      "enablexcodeintegration": true
    },
    "auth": {
      "enablecaseinsensitivity": true,
      "useinclusiveterminology": true,
      "breakcirculardependency": true,
      "forcealiasattributes": false,
      "useenabledmfas": true
    },
    "codegen": {
      "useappsyncmodelgenplugin": true,
      "usedocsgeneratorplugin": true,
      "usetypesgeneratorplugin": true,
      "cleangeneratedmodelsdirectory": true,
      "retaincasestyle": true,
      "addtimestampfields": true,
      "handlelistnullabilitytransparently": true,
      "emitauthprovider": true,
      "generateindexrules": true,
      "enabledartnullsafety": true
    },
    "appsync": {
      "generategraphqlpermissions": true
    },
    "latestregionsupport": {
      "pinpoint": 1,
      "translate": 1,
      "transcribe": 1,
      "rekognition": 1,
      "textract": 1,
      "comprehend": 1
    },
    "project": {
      "overrides": true
    }
  }
}

Hello @ykethan, I did delete the build but still the same. And I compare with your cli.json indeed there is some difference.
Are you using the same version cli that I am using? mine is 8.2.0.

Hey @n-francky, yes I tested the gql compile on the CLI version 8.2.0. In order to further dive deeper into the issue could you please send us your Amplify folder at<EMAIL_ADDRESS>

Hey @n-francky, I understand that the folder has sensitive information. On bring up the issue with the team, in order to further dive deeper into the issue could you please provide us the full schema or the model types being used. If the schema contains business logic that you would not like to share in the github issue, please feel free in sending the schema at<EMAIL_ADDRESS>

Hey @ykethan, I just send the email and referenced this issue.

Hey @n-francky, Thank you for sending us the email but it appears the attachments have been striped from the email. Could you please try re-sending email or send me the attachments on discord. My handle is ykethan.

@ykethan I re-send it, if this does not work then I will send it via the discord.

Hey @n-francky, Thank you for sending us the schema. I was able to reproduce the missing keyword in my application using CLI 8.2.0.
I will be transferring this issue to amplify-category-api repository for visibility.


Test formal verification regressions

Check formal verification results.
For tests that fail, check if they were not on a list of passing tests.
For test that pass, make sure they are added to the list of passing tests.

CI is broken intentionally to showcase results, see https://github.com/antmicro/yosys-systemverilog/actions/runs/3546772471 for a passing build.


[infra] Use bash shell when running the build-test

Fixes #12.

@ferbetanzo can this be merged (once the conflict is resolved)?


Add support for python3.8

Is your feature request related to a problem? Please describe.
As python 3.8 is the current "stable" version, would it make sense to add CI tests for it? I've run the test suite locally on it, and it all passes, so I think it's just a matter of adding the version to .travis.yml.
Describe the solution you'd like
Just have the CI tests run for python 3.8 to give confidence that this version is supported
Additional context
I'm happy to send a PR for it :)

Good spot, I was talking with Robin the other day that we should in test python 38 in Travis, I will add it in this week.

It is now added in commit 86955c378db3d216d1675392370a69239e942cf8


fix(world): SnapSync incorrect index for records

The function getRecords() in SnapSyncSystem populates the records array with elements from the specified table, starting from a specified offset.
The for loop's index (i) is for the table's records (ie with an offset), but the index to populate records should be from 0 (ie without an offset). Currently, it's including the offset. This causes an out of bounds error.
Found the bug when running getSnapSyncRecords from the client which uses an offset after a table has more than 100 keys.
Also, added a test that uses an offset.

small request: could you add a changeset describing this change? (run pnpm changeset locally)

small request: could you add a changeset describing this change? (run pnpm changeset locally)

Patch or minor bump to world?

small request: could you add a changeset describing this change? (run pnpm changeset locally)

Patch or minor bump to world?

patch is probably fine!

Patch or minor bump to world?

patch is probably fine!

added!

we removed snap sync in #1311 since it wasn't compatible with the v2 networking stack and the recommended way to load a MUD app is via store-indexer (the snap sync module added a lot of gas to each store operation on-chain, and syncing via the RPC node is computationally heavy, can't be cached, and is expensive in terms of RPC cost, so not suitable for production use cases)


NTR: L-lactate dehydrogenase inhibitor activity

Please provide as much information as you can:

Suggested term label:

L-lactate dehydrogenase inhibitor activity

Definition (free text)

Binds to and stops, prevents or reduces the activity of L-lactate dehydrogenase

Reference, in format PMID:#######

33406399

Gene product name and ID to be annotated to this term

C0HLV8 MLDHR

Parent term(s)
neg reg of GO:0004459
is_a GO:0004857

+[Term]
+id: GO:0160193
+name: L-lactate dehydrogenase inhibitor activity
+namespace: molecular_function
+def: "Binds to and stops, prevents or reduces the activity of L-lactate dehydrogenase." [PMID:33406399]
+intersection_of: GO:0140678 ! molecular function inhibitor activity
+intersection_of: directly_negatively_regulates GO:0004459 ! L-lactate dehydrogenase activity
+property_value: term_tracker_item "https://github.com/geneontology/go-ontology/issues/28309" xsd:anyURI
+created_by: rynl
+creation_date: 2024-07-08T17:25:44Z


Add bol

Description
https://play.google.com/store/apps/details?id=com.bol.shop
:x: Bug fix (non-breaking change which fixes an issue)
✅ Icon addition (non-breaking change that adds/modifies Lawnicons's icons)
:x: General change (non-breaking change that doesn't fit the above categories like copyediting)
Icons addition information
Icons added

App Name (com.bol.shop)

I will make icon new


Text align ignored

Trying out typography with textAlign: center and it isn't reflected on the canvas.

Thanks for looking into this.
Do you think textAlign should be an attribute of text styles, or a parameter of the Text component?
At least at Airbnb, text alignment isn't part of the typography guidelines. It'd be part of the individual component that needs aligned text. Basically I've been imagining it going here somewhere:

However, Sketch includes alignment as part of the text style, so that could make sense too. Or potentially it can be part of the text style, but also overriden by the Text component.
cc @ryngonzalez

I agree that it really should be part of the component, however I also envisioned people using the studio outside of a pure component system, say just for generating sketch files from unified DLS.
Having it on the text style is convenient but actually causes more issues I think as you'd probably end up overriding it more often than not in the course of designing full featured projects.
Probably makes more sense to leave it solely up to the component how the text should be aligned, also considering that by default the text component is set to fit content which means textAlign: center isn't immediately effective.

Having it on the text style is convenient but actually causes more issues I think as you'd probably end up overriding it more often than not in the course of designing full featured projects.

I agree with the sentiment here; I think alignment is going to generally be use-specific, so it would make more sense to keep it out of the text styles. If alignment needs to be reused and shared, it can be encoded into a component, with a use case (and appropriate name) in mind.

Closing this for now! Please reopen if needed.


[zh-cn] sync /docs/concepts/security/cloud-native-security.md

Description

Issue

Closes: #

/lgtm
/approve


License Problem during Installation

Hi, I am very interested in using TensorFlow with large model support. When I tried to install the tensorflow-gpu from WML CE, I always get stuck with reading the IBM license, which requires me to:
Press Enter to continue viewing the license agreement, or, Enter "1" to accept the agreement, "2" to decline it or "99" to go back to the previous screen, "3" Print, "4" Read non-IBM terms.
I press "1" many times but the Executing transaction process seems running forever so that I can't install the packages.
Since I try to install this package on my school server, I am wondering if I should have a root permission to accept the license when using conda to install the tensorflow-gpu from WML CE.
Thank you very much!

Hi,
Can you post some more information? Does your school have WML-CE or conda itself centrally installed?
The license accept should save a file in your home that marks your install has accepted the license. If your home dir, or conda, or conda package locations are on various remote file locations that could complicate things. Post a bit more info and we can try to help.

Hi,
Can you post some more information? Does your school have WML-CE or conda itself centrally installed?
The license accept should save a file in your home that marks your install has accepted the license. If your home dir, or conda, or conda package locations are on various remote file locations that could complicate things. Post a bit more info and we can try to help.

I don't think our school uses WML-CE before. I try to run the License Acceptance Process directly by command lapc. But when I type "1" and enter. It said "Exited with 15. The license is not accepted." What is the exit code "15" meaning?

I don't think our school uses WML-CE before. I try to run the License Acceptance Process directly by command lapc. But when I type "1" and enter. It said "Exited with 15. The license is not accepted." What is the exit code "15" meaning?

I think its not able to write out the accept file. Where do you have conda installed?  What is your home dir like?

I think its not able to write out the accept file. Where do you have conda installed?  What is your home dir like?

So, is that a permission error. The anaconda is installed by the administrator of our school server. I just export the installation path to my home dir and "/home/myusername/". I create .conda directory under my home dir and create Python virtual  environment there.

So, is that a permission error. The anaconda is installed by the administrator of our school server. I just export the installation path to my home dir and "/home/myusername/". I create .conda directory under my home dir and create Python virtual  environment there.

Here is the full error output
Exited with: 15
License NOT accepted. Installation will not proceed.
b'License NOT accepted. Installation will not proceed.\n'                                                                                done
ERROR conda.core.link:_execute(700): An error occurred while installing package 'https://public.dhe.ibm.com/ibmdl/export/pub/software/server/ibm-ai/conda::powerai-license-1.6.0-536.10b2576'.
Rolling back transaction: done
LinkError: post-link script failed for package https://public.dhe.ibm.com/ibmdl/export/pub/software/server/ibm-ai/conda::powerai-license-1.6.0-536.10b2576
location of failed script: $environment_path$/bin/.powerai-license-post-link.sh
==> script messages <==
License NOT accepted. Installation will not proceed.
==> script output <==
stdout:
stderr:
return code: 1
()

Here is the full error output
Exited with: 15
License NOT accepted. Installation will not proceed.
b'License NOT accepted. Installation will not proceed.\n'                                                                                done
ERROR conda.core.link:_execute(700): An error occurred while installing package 'https://public.dhe.ibm.com/ibmdl/export/pub/software/server/ibm-ai/conda::powerai-license-1.6.0-536.10b2576'.
Rolling back transaction: done
LinkError: post-link script failed for package https://public.dhe.ibm.com/ibmdl/export/pub/software/server/ibm-ai/conda::powerai-license-1.6.0-536.10b2576
location of failed script: $environment_path$/bin/.powerai-license-post-link.sh
==> script messages <==
License NOT accepted. Installation will not proceed.
==> script output <==
stdout:
stderr:
return code: 1
()

By the way, can I use Tensorflow 2.1 with CUDA 10.1 provided by powerai, since our school server didn't provide CUDA10.2 driver?

By the way, can I use Tensorflow 2.1 with CUDA 10.1 provided by powerai, since our school server didn't provide CUDA10.2 driver?


Allow user to select from a drop down menu of pre-filled settings

Create a settings directory
Use drop down menu to call afqb.global.readsettings() function.

@richford I think the default behavior should be to load the settings file in the directory. I noticed that when I save the settings.json file (or replace lines in the file) that these changes aren't reflected until I click the "load settings" button. For example, if the "title" attribute is edited in the settings, we want that to show up as the default for the afq-browser instance

Yeah, if you have the browser open and are editing the settings.json file at the same time, you'll have to use the "load settings" button for those changes to be reflected in the browser. Alternatively, if you refresh the browser, it should see the changes that you made.
I don't know of a way to make the browser continuously observe the settings.json file for changes.

My issue was that when I refresh the browser it doesn't load the settings (such as the title) unless I click load settings

Thanks for pointing that out. I'll submit a PR that fixes this (and the same issue with mouseover highlight).

PR submitted. We should keep this issue open even after accepting PR #157 since it doesn't add the dropdown menu. Thanks for pointing out that issue @jyeatman.

This is not for the neuroimaging paper milestone, correct? We may very well fix it before the paper is accepted, but is it important to fix this before the paper is initially submitted?

I dont' think so
Jason D. Yeatman, PhD
Assistant Professor, Institute for Learning & Brain Sciences (I-LABS)
Department of Speech & Hearing Sciences
University of Washington
http://brainandeducation.com
On Thu, Aug 17, 2017 at 1:17 PM, Adam Richie-Halford <
<EMAIL_ADDRESS>wrote:

This is not for the neuroimaging paper milestone, correct? We may very
well fix it before the paper is accepted, but is it important to fix this
before the paper is initially submitted?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/yeatmanlab/AFQ-Browser/issues/155#issuecomment-323182504,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABsYyoWrXnFSoPjU441a-f0RwdFfhDCJks5sZJ_ugaJpZM4OtCIr
.

After further conversation, wait until the resolution of #167.

Obviated by #185.


Source?

Hey, thanks for publishing this project.
I would like to make some changes (specifically to get browser support), but am unsure where, since the src-folder is almost empty and the lib folder appears to mostly contain compiled code (the commit messages seem to support this theory). Can you help me out here?
Thanks!

Well that's dumb - Looks like I simply overlooked it! Oops, sorry (to be fair to myself, when asking for the definition vscode jumps into the lib folder ;-)!

Hi @solarkraft, I tried to get it to work in the browser myself before but ran into issues since you have to set the user agent headers to a specific value, which the browser usually doesn't allow or at least overwrite with something else, making Picnic block the requests. Let me know if do find a way, then we can implement that in this library as well 👍

I noticed that they also block requests via CORS headers, so I'd at least need a server for proxying the requests - then I can also build one that does some more logic (it doesn't actually seem that easy to get an md5 hash in the browser, for instance) and transforms the data nicely.


Fix getting config like ignore_checks from the template

Signed-off-by: Irving Popovetsky<EMAIL_ADDRESS>Description of changes:
I noticed that setting configuration in my templates (such as ignore_checks) wasn't working.  digging around, I noticed this if statement was never evaluating to True: https://github.com/awslabs/cfn-python-lint/blob/master/src/cfnlint/main.py#L77
because:
>>> template is dict
False
>>> template.__class__
<class 'cfnlint.parser.dict_node'>

this is a more reliable check:
>>> isinstance(template, dict)
True

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Makes sense to me. We test using isinstance a lot not sure how we missed this one. Thanks.


Addition of PGGD collection

Changes
Addition of collections
pggd-01: drilling-of-borehole-on-notice ;
pggd-02: completion, bhse:alteration, bhse:abandonment ;
pggd-03: hydraulic-fracturing-on-notice;
pggd-04: bhse:hydraulic-fracturing-abandoned, bhse:hydraulic-fracturing-completed;
Reviewers
SMEs @LizDerrington @BlakePaul Is the above correct? Not sure if drilling-abandoned or fraccing-abandoned belong in with the PGGD forms at all. Please let me know.
Tech @LukeHauck and @DavidCrosswellGSQ
FYI - @lisa-woody

Undefined prefix "gsqdt:" on line 130.

Undefined prefix "gsqdt:" on line 130.

fixed! thanks.


bug: missing file is being imported in Deno using ESM

Bug report
Describe the bug
A seemingly missing file is imported when importing this package from esm in Deno. This behaviour causes Deno to attempt to download the file on each run, slowing down development.
To Reproduce
Steps to reproduce the behaviour while Deno is installed:

Run deno info https://esm.sh/<EMAIL_ADDRESS>Observe Deno attempting to download the non-existent https://esm.sh/v110/<EMAIL_ADDRESS>file and reporting it as missing from the printed file list.

Expected behaviour
Imports don't point to non-existent files.
Screenshots

System information

OS: macOS
Version of @supabase/auth-helpers-shared: 0.3.0
Version of Deno: 1.31.2

Additional context
CC @thorwebdev, this issue comes from my findings originating in https://github.com/iuioiua/fresh-saas-toolkit/issues/8.

Hello, I also faced this problem.
Here is a temporary solution which I use the flag: deno run --cached-only

Hello, I also faced this problem.
Here is a temporary solution which I use the flag: deno run --cached-only

The workaround works! Thank you.

Gently pinging @david-plugge. Have you had a chance to look into this?

I didn´t yet try to replicate this. Does this error not appear when using just supabase-js?

This issue has been fixed in the latest release of the auth-helpers.

This issue was fixed in esm-dev/esm.sh#572. Thanks for looking into this either way, guys 🙏🏾


Adding a new retrieval task

Hi big fan of this package/leaderboard. I wanted to extend it to include Scandinavian languages (let me know if you want a PR or prefer that I keep it as a fork).
I however have a few issues with understanding the "relevant_docs" for the retriever. Currently, I am having issues with this specific dataset:
import datasets
from mteb.abstasks.AbsTaskRetrieval import AbsTaskRetrieval

from sentence_transformers import SentenceTransformer

from mteb import MTEB

class SweFAQRetrieval(AbsTaskRetrieval):
    """ """

    @property
    def description(self):
        return {
            "name": "SweFAQ",
            "hf_hub_name": "AI-Sweden/SuperLim",
            "description": "A Swedish FAQ dataset. Available as a part of Superlim",
            "reference": "https://spraakbanken.gu.se/en/resources/superlim",
            "type": "Retrieval",
            "category": "s2p",
            "eval_splits": ["test"],
            "eval_langs": ["sv"],
            "main_score": "ndcg_at_10",
            "revision": "7ebf0b4caa7b2ae39698a889de782c09e6f5ee56",
            "beir_name": "NA",
        }

    def load_data(self, **kwargs):
        """
        Load dataset from HuggingFace hub and convert to MTEB format
        """
        if self.data_loaded:
            return

        self.dataset = datasets.load_dataset(
            self.description["hf_hub_name"], "swefaq", revision=self.description.get("revision")
        )
        self.data_loaded = True

        self.corpus, self.queries, self.relevant_docs = {}, {}, {}
        for split in self.description["eval_splits"]:
            dataset = self.dataset[split]  # type: ignore
            # answers = dataset["candidate_answer"]
            answers = dataset["correct_answer"]
            self.corpus[split] = {
                str(idx): {"_id": str(idx), "title": "", "text": answer} for idx, answer in enumerate(answers)
            }

            questions = dataset["question"]
            self.queries[split] = {str(idx): question for idx, question in enumerate(questions)}

            # Not sure what to do here:
            # first guess: Relevant documents is a mapping between each query and the desired responses (most relevant documents) for that query
            # result: Performance is 0.0 for all metrics
            self.relevant_docs[split] = {str(idx): {str(idx): 1} for idx in range(len(answers))}

            # second guess: Relevant documents is a mapping between each query and the relevant documents a pre-search would return
            # in which case. There is no pre-search, so we assume all documents are relevant.
            # result: Performance is perfect for all metrics

            # self.relevant_docs[split] = {}
            # # assume all are relevant
            # scores = {str(idx): 1 for idx in range(len(answers))}
            # for idx in range(len(answers)):
            #     self.relevant_docs[split][str(idx)] = scores

The code runs as is and can be run using:
model = SentenceTransformer("sentence-transformers/all-MiniLM-L6-v2")
evaluation = MTEB(tasks=[SweFAQRetrieval()])
evaluation.run(model)

It'd be amazing to have those in MTEB as a PR! I can then add a leaderboard tab for them (similar to the German Clustering tab).
Also this PR adding Polish retrieval may be helpful for reference.
Your first guess is correct, relevant docs should map to all document ids relevant to that query.
I'm surprised self.relevant_docs[split] = {str(idx): {str(idx): 1} for idx in range(len(answers))} doesn't work. Maybe the model you tested is just not very good at the task?

It'd be amazing to have those in MTEB as a PR! I can then add a leaderboard tab for them (similar to the German Clustering tab).

Perfect. I will make a PR once I have them ready.

Also https://github.com/embeddings-benchmark/mteb/pull/121 adding Polish retrieval may be helpful for reference.
Yea did take a look at it, but they seem to use the BEIR format

Your first guess is correct
Good to hear. I will keep using it and try it out with a few other models as well.

I have added it in a PR. Along with the remainder of the Scandinavian tasks. I still can't get it to work (as mentioned in the PR).

I have resolved this issue over at the Scandinavian embedding benchmark. I will port the datasets to MTEB once I have a reasonable set of tasks.


Migrating emulator CI setup to Jenkins pipeline

Signed-off-by: peusterm<EMAIL_ADDRESS>

test this please

test this please


How are you getting the router attached to the context in your components, like the SignupRoute?

I am trying to follow some of the patterns that you have in your app, but I keep getting undefined for my router in my handleSubmit?  How is the router getting attached to the context in this example?

You should add these lines into your component's class:
static contextTypes = {
  router: React.PropTypes.object
}

then you can get router like this:
render {
  const router = this.context.router;
  router.transitionTo(...);
}

But it really depends on what version of react-router you use.
This example uses 1.0.0-beta3.
In 1.0.0-rc1 there's no router in context, but there are history and location.
Anyway you could use redux-router now.
It's just plug'n'play solution that glues together react-router and redux nicely :-)

So in the newest version, if you wanted to change routes based on a state
change (e.g you publish a blog post), in the action creator you would use
history.pushState instead of router.transitionTo?
On Mon, Sep 28, 2015 at 7:24 AM Sergey Slipchenko<EMAIL_ADDRESS>wrote:

You should add these lines into your component's class:
static contextTypes = {
router: React.PropTypes.object
}
then you can get router like this:
render {
const router = this.context.router;
router.transitionTo(...);
}
But it really depends on what version of react-router you use.
This example uses 1.0.0-beta3.
In 1.0.0-rc1 there's no router in context, but there are history and
location.
Anyway you could use redux-router https://github.com/rackt/redux-router
now.
It's just plug'n'play solution that glues together react-router and redux
nicely :-)
—
Reply to this email directly or view it on GitHub
https://github.com/GetExpert/redux-blog-example/issues/12#issuecomment-143758323
.

Yes, I guess react-router should listen for changes on that object.
I've personally just decided to just use redux-router after 1.0.0-rc1 of react-router :-)

Do you have a code snippet for more context? Trying to get my bearings
straight with routing in Redux.
On Mon, Sep 28, 2015 at 8:12 AM Sergey Slipchenko<EMAIL_ADDRESS>wrote:

Yes, I guess react-router should listen for changes on that object.
I've personally just decided to just use redux-router after 1.0.0-rc1 of
react-router :-)
—
Reply to this email directly or view it on GitHub
https://github.com/GetExpert/redux-blog-example/issues/12#issuecomment-143771438
.

To get history you should define needed context in your class:
static contextTypes = {
  history: React.PropTypes.object
}

And get it where needed and call methods:
componentDidUpdate() {
  const history = this.context.history;
  history.pushState({ some: 'state' }, '/some/path');
}

I do have the contextTypes set.  Here is the whole file:
class LoginContainer extends React.Component {
	static contextTypes = {
		router: React.PropTypes.object
	}

	handleLogin() {
		this.props.login();
		console.log("Router info:");
		console.log(this.context.router);
	}

	render() {
		return <Login handleLogin={this.handleLogin.bind(this)} />
	}
}

function mapPropsFromState(state) {
	return {
		login
	}
}

export default connect(mapPropsFromState)(LoginContainer);

In the console.log(this.context.router), I get undefined.  I will check out redux-router`, but I am just trying to understand your code.

@mjhamm75 Can you tell which react-router version you have?

1.0.0-rc1

As I said there's no router in context for that version.
So you should follow that path
https://github.com/GetExpert/redux-blog-example/issues/12#issuecomment-143775485


[stable/datadog] Set dnsPolicy for using hostNetwork

What this PR does / why we need it:
When using hostNetwork, dnsPolicy needs to be set ClusterFirstWithHostNet.
https://kubernetes.io/docs/concepts/services-networking/dns-pod-service/#pod-s-dns-policy

/ok-to-test

/lgtm


Honor CXXFLAGS in configure.py

Prior request for this: https://github.com/ninja-build/ninja/issues/673
Prior attempted implementation: https://github.com/ninja-build/ninja/pull/676
As with the prior implementation, this tries to ensure existing workflows that depend on using CFLAGS are not affected. Supplied CFLAGS and CXXFLAGS are also now added to LDFLAGS, since some C++ flags like -stdlib are needed by the linker too.

Thanks!


Placing legend in custom location

Hey There,
Awesome plugin and thank you for making this available.
I saw a previous closed issue about moving the legend using custom css.
Probably a noob question but how would I with css place the legend below the chart?
Thanks again.

I don't know about CSS - there probably is some fancy way to switch element positions. But it may be simpler and more maintainable to do it in JavaScript.
Currently the plugin appends to the chart container before the chart itself is drawn, so the legend is above the chart. I see two options..

Switch the positions after the fact at some point. Something like $('.ct-legend').insertAfter($('.ct-legend').next()) will work to put the legend on the bottom.

Fork the plugin (recommended if you anticipate needing to make more changes to legends, this plugin is very small and rudimentary). Replace the line chart.container.appendChild(legendElement); with            chart.on('draw', function () { chart.container.appendChild(legendElement); }); to append the legend after the chart draws (put the legend below).

Adding a option for side positioning would be quite sweet IMHO, until now i personally solved this with a CSS hack:
transform: translateY(yourChartsHeightInPx)

A PR is welcome, I don't have time to fix this myself atm.

Gonna make a PR soon in this case.
On 02 Sep 2016, at 11:16, Kees Kluskens <notifications<EMAIL_ADDRESS>wrote:
A PR is welcome, I don't have time to fix this myself atm.

You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/CodeYellowBV/chartist-plugin-legend/issues/22#issuecomment-244325221, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AP0yvQdhDEEwy5fYu6cVgMhH3R20UvS9ks5ql-lkgaJpZM4JQh0f.


Failure to reproduce MLA > MHA

I tried out MLA and it was a good amount worse than MHA and wanted to try to find out why. Firstly, I am using a hybrid model therefore I am not using any Rope in either MLA or MHA, and therefore use the basic version of MLA. I suspect the issue could be due to the part saying:
"In addition, the low-rank compression and fine-grained expert segmentation
will impact the output scale of a layer. Therefore, in practice, we employ additional RMS Norm
layers after the compressed latent vectors, and multiply additional scaling factors at the width
bottlenecks (i.e., the compressed latent vectors and the intermediate hidden states of routed experts) to ensure stable training."
It is unclear if the additional scaling factor is done before or after the RMSNorm, also what this factor would be. Another reason could be that the rope version of MLA gives it a performance boost.
Any clarification on this scaling factor and its placement would be great, thanks!

The following are factors that affect the final result:

Rope positional embedding
Scale factors (you can check the open-source checkpoint)

@faresobeid

Thank you for replying, I’m still a bit unclear on how to check the scaling factors and if it is done before or after the RMSNorm.

Thank you!

Sorry to reopen this issue but I have been having some issues with stability at scale with MLA. Like I said before I am using a hybrid model and therefore the MLA that I am using is the basic version with no Rope.
kv = W_b(rms_norm(W_a(x)))
I have also tried having
kv = W_b(rms_norm(W_a(ln(x))))
but that also has some issues with performance and stability. To be more specific, this is a 24 layer model with model dimension 2048 with the last third of layers using MLA (~300M params). Is there any recommended scaling factors or ways to mitigate this issue? Thank you

24-layer dense model? @faresobeid

Yes, although stability has been fine without the inner rms norm but still any recommendations would be helpful


Use master branch of platy-browser-data repository

The platy-browser is currently set to the dev branch of the data repository. This branch was already merged into master and is not up-to-date
anymore, so does not contain the latest data versions.
@tischi please use the master instead.

done. the default is now the master.

@tischi my fiji does not update platy browser yet.
Did you forget to push the version or does it just take some time to take effect?

I did not push yet, because I need to implement automated detection of versions first.


mgmt: mcumgr: update bt smp initialization

Small oneline fix to smp_bt.c smp transport register.
smp_client_transport_register() is a void function and has no return value, so it should be ignored.

@Alex-T-Stark can you fix the compliance issues?

Amended commit message to fix compliance issue.
Rebased to current upstream main.

@Alex-T-Stark still there:
 4: UC4 Commit message body line exceeds max length (100>75): "smp_client_transport_register() is a void function and has no return value, so it should be ignored."

@fabiobaltieri looks ready to merge


Add CI tasks that can be run in a Docker container

Enables a Docker based build server to create logsearch dev releases using a command like:
docker build -t $JOB_NAME $(pwd)

docker run -e BOSH_TARGET=https://xx.yy.zz.bb:25555 -e BOSH_USER=sudobot -e BOSH_PASSWORD=$SUDOBOT_BOSH_PASSWORD -v $(pwd):/workspace -t $JOB_NAME rake dev_release:create_and_upload_and_deploy

@mrdavidlaing, this is a (very) long running PR with no recent activity. I'm not using the existing Dockerfile or Rakefile myself - is this something we currently need and should still be merged, or shall we close it?

We should close it without merging in favour of investigating a consol.io
approach
On Wednesday, May 6, 2015, Danny Berger<EMAIL_ADDRESS>wrote:

@mrdavidlaing https://github.com/mrdavidlaing, this is a (very) long
running PR with no recent activity. I'm not using the existing Dockerfile
or Rakefile myself - is this something we currently need and should still
be merged, or shall we close it?
—
Reply to this email directly or view it on GitHub
https://github.com/logsearch/logsearch-boshrelease/pull/61#issuecomment-99566368
.

--
David Laing
logsearch.io - build your own open source cloud logging cluster
http://davidlaing.com


Add animation when deleting an item from a list

What

Add animation when deleting an item from a list

Why

Avoid surprise from users

Part of

Lists

Screenshot/Mockup/Before-After

Can i work on this issue?

sure thing, just assigned you 👍

I have resolved this issue
https://user-images.githubusercontent.com/46370525/160696467-69e76a3b-6f6b-4cd3-8d97-8abdf5a54444.mp4
https://github.com/openfoodfacts/smooth-app/pull/1411

I have resolved this issue, you can now merge my PR @teolemon


getting error(evaluating 'webRTCModule.mediaStreamTrackGetSources')

Hai, i cloned your repo and run as per the document,facing problem when using the code in separate project, getting an error like this undefined is not an object(evaluating 'WebRTCModule.mediaStreamTrackGetSources') below is the error

please give me suggestions that is there any thing need to add to my project.

Here i tried some part(added the frameWorks) above error is not showing but not running in xcode facing error like arm64 below is the error

Please respond for it

please respond

Please respond

I also have same bug when get local stream via RTCView to my source code. I tried with same react, react-native, react-native-webrtc libs version, but no luck. Can someone suggest me about this issue?

please try latest demo which use master branch of react-native-webrtc now

any support on this, guys? still not working

what is your error exactly?

the demo itself, or your own project.
the version of webrtc or demo
use case
how to reproduce it
what have you tried
a bit of code snippets would help.

Howdy!
I've also encountered this error. Let me see if I can provide a little more info that could help.

Running RCTWebRTCDemo master branch, pulled today.
React Native v 0.40
React Native CLI v 2.0.1
MacOS 10.11.6
Xcode 8.2.1

I haven't really tried anything to fix it yet, but I got the same-ish error message as shaikhussian.

I followed the error messages to the following code snippets:
getSources
MediaStreamTrack.js:30
export default class MediaStreamTrack extends EventTarget(MEDIA_STREAM_TRACK_EVENTS) {
  static getSources(success: (sources: Array<SourceInfo>) => void) {
    WebRTCModule.mediaStreamTrackGetSources(success);
  }

getLocalStream
main.js:205
  MediaStreamTrack.getSources(sourceInfos => {
    console.log(sourceInfos);
    let videoSourceId;
    for (const i = 0; i < sourceInfos.length; i++) {
      const sourceInfo = sourceInfos[i];
      if(sourceInfo.kind == "video" && sourceInfo.facing == (isFront ? "front" : "back")) {
        videoSourceId = sourceInfo.id;
      }
    }

unknown
main.js:205
socket.on('connect', function(data) {
  console.log('connect');
  getLocalStream(true, function(stream) {
    localStream = stream;
    container.setState({selfViewSrc: stream.toURL()});
    container.setState({status: 'ready', info: 'Please enter or create room ID'});
  });
});

Also, I had run into the same problem as reported in this issue: https://github.com/oney/RCTWebRTCDemo/issues/41
I was able to silence the error by making the change listed there, but I dont know if that some affected socket.io down the line.

Also, here's the console log for when I tried to run the code in the Simulator.
2017-01-16 12:15:14.833410 RCTWebRTCDemo[17641:913618] [] nw_host_stats_add_src recv too small, received 24, expected 28
2017-01-16 12:15:14.833928 RCTWebRTCDemo[17641:913618] [] nw_host_stats_add_src recv too small, received 24, expected 28
2017-01-16 12:15:14.836077 RCTWebRTCDemo[17641:913617] [] nw_host_stats_add_src recv too small, received 24, expected 28
2017-01-16 12:15:14.854 [info][tid:main][RCTBatchedBridge.m:72] Initializing <RCTBatchedBridge: 0x60000018b120> (parent: <RCTBridge: 0x6080000b2660>, executor: RCTJSCExecutor)
2017-01-16 12:15:14.961632 RCTWebRTCDemo[17641:913618] [] nw_host_stats_add_src recv too small, received 24, expected 28
2017-01-16 12:15:15.046773 RCTWebRTCDemo[17641:913632] [] nw_host_stats_add_src recv too small, received 24, expected 28
2017-01-16 12:15:15.177308 RCTWebRTCDemo[17641:913632] [] nw_host_stats_add_src recv too small, received 24, expected 28
2017-01-16 12:15:15.182019 RCTWebRTCDemo[17641:913632] [] nw_host_stats_add_src recv too small, received 24, expected 28
2017-01-16 12:15:15.185040 RCTWebRTCDemo[17641:913632] [] ____nwlog_simulate_crash_inner_block_invoke dlopen CrashReporterSupport failed
2017-01-16 12:15:15.185331 RCTWebRTCDemo[17641:913632] [] __nwlog_err_simulate_crash simulate crash failed "nw_socket_set_common_sockopts setsockopt SO_NOAPNFALLBK failed: [42] Protocol not available"
2017-01-16 12:15:15.186367 RCTWebRTCDemo[17641:913632] [] nw_socket_set_common_sockopts setsockopt SO_NOAPNFALLBK failed: [42] Protocol not available, dumping backtrace:
        [x86_64] libnetcore-856.30.16
    0   libsystem_network.dylib             0x0000000109443666 __nw_create_backtrace_string + 123
    1   libnetwork.dylib                    0x0000000109cb1006 nw_socket_add_input_handler + 3164
    2   libnetwork.dylib                    0x0000000109c8e555 nw_endpoint_flow_attach_protocols + 3768
    3   libnetwork.dylib                    0x0000000109c8d572 nw_endpoint_flow_setup_socket + 563
    4   libnetwork.dylib                    0x0000000109c8c298 -[NWConcrete_nw_endpoint_flow startWithHandler:] + 2612
    5   libnetwork.dylib                    0x0000000109ca7ae1 nw_endpoint_handler_path_change + 1261
    6   libnetwork.dylib                    0x0000000109ca7510 nw_endpoint_handler_start + 570
    7   libnetwork.dylib                    0x0000000109cbf1f9 nw_endpoint_resolver_start_next_child + 2240
    8   libdispatch.dylib                   0x00000001091c0978 _dispatch_call_block_and_release + 12
    9   libdispatch.dylib                   0x00000001091ea0cd _dispatch_client_callout + 8
    10  libdispatch.dylib                   0x00000001091c7e17 _dispatch_queue_serial_drain + 236
    11  libdispatch.dylib                   0x00000001091c8b4b _dispatch_queue_invoke + 1073
    12  libdispatch.dylib                   0x00000001091cb385 _dispatch_root_queue_drain + 720
    13  libdispatch.dylib                   0x00000001091cb059 _dispatch_worker_thread3 + 123
    14  libsystem_pthread.dylib             0x00000001095934de _pthread_wqthread + 1129
    15  libsystem_pthread.dylib             0x0000000109591341 start_wqthread + 13
2017-01-16 12:15:15.189735 RCTWebRTCDemo[17641:913632] [] __nw_connection_get_connected_socket_block_invoke 4 Connection has no connected handler
2017-01-16 12:15:15.242 [warn][tid:com.facebook.react.JavaScript] Warning: Native component for "RTCVideoView" does not exist
2017-01-16 12:15:15.247 [info][tid:com.facebook.react.WebSocketModuleQueue][RCTSRWebSocket.m:555] SocketRocket: In debug mode.  Allowing connection to any root cert
2017-01-16 12:15:15.314889 RCTWebRTCDemo[17641:913632] [] nw_host_stats_add_src recv too small, received 24, expected 28
2017-01-16 12:15:15.317257 RCTWebRTCDemo[17641:913632] [] __nwlog_err_simulate_crash simulate crash already simulated "nw_socket_set_common_sockopts setsockopt SO_NOAPNFALLBK failed: [42] Protocol not available"

Okay I got to the bottom of this (kinda). I just didn't setup the iOS project correctly.
I followed the directions from here and here and here:

http://stackoverflow.com/questions/40595830/simple-example-for-accessing-the-camera-via-webrtc-in-react-native-android
https://github.com/oney/react-native-webrtc/blob/master/Documentation/iOSInstallation.md
https://github.com/facebook/react-native/issues/4968

From the error message, here are the steps I followed to get the app working correctly on my phone.

remove npm module: rm -rf $YourProject/node_modules/react-native-webrtc
watchman watch-del-all
clean npm cache: npm cache clean
npm install react-native-webrtc
npm start -- --reset-cache
In xocde project, click Product -> clean
Build/Run on my test iPhone, NOT in simulator.

thanks for sharing, mate!

So on simulator demo won't work?

Try this (iOS):
Targets(RCTWebRCTDemo) => Build Phrases => Link Binary With Libraries
Add libRCTWebRTC.a, rebuild

Be sure to do a react-native link
This will install the native code. I had this problem until I did this

@jam1401 It works like charm! Thanks ;)

You should update your version: "react-native-webrtc": "^1.67.1"
It's work for me !!!

I've tried all solutions here... still doesn't work.
I even tried starting a new project
I have the libraries linked, and I followed the IOS installation
I ran react-native link
My react-native-webrtc version: "react-native-webrtc": "^1.69.1"
I also followed the suggestion by @rfelixallen
Nothing is fixing it... any other ideas?


recommend watching: typed redux store

I love redux... but felt it needed typing.
I posted this video:
https://www.youtube.com/watch?v=bEkPEnudm7s&feature=youtu.be
it's the best of both worlds, typed redux
regards
Sean

Probably not something worth opening an issue over, but that is a useful video.  Nice.
FYI, Redux does ship with type definitions for both Flow and Typescript.

tx, got it!
regards


Add dependencies to pip

Fixes #2574

Times have changed :-) Now pip is a decent package manager, so it's time for us to demand all modules we really depend on (like rope, pyflakes and sphinx) as real dependencies.
This PR also wants to fix:

[ ] Tests for pip (now they only work for conda)
[ ] The Optional dependencies dialog, which will become just the Dependencies one.

Remember to remove then pep8 from external utils as well ? to avoid duplicating stuff
Looking good :+1: !

@goanpeca, that will come in another PR, once we move to a better way to create Windows installers.
For now this PR helps us to create wheels that pull all our real deps :-)

Ok, this is (finally!) ready :-)


Patch/make es5 compatible

Using non-transpiled dependencies causes problems for users of create-react-app (https://github.com/facebookincubator/create-react-app/issues/1125).

thanks for the contribution! merged!


ISPN-12220: expression defaults not working correctly

Defaults for infinispan.cluster.name and infinispan.cluster.stack are set hardcoded, the defaults within the expression ${:XXX} are not working.
https://issues.redhat.com/browse/ISPN-12220

Pushed to 11.0.x too


feat(explorer): allow element explorer to start as a server

If element explorer is run with a port (i.e. --debuggerServerPort 1234),
it will start up a server that listens to command from the port instead of
a repl that listens to process.stdin.
Note instead of starting either repl or server, another approach is to
feed the server socket into the repl as the input/output streams. The
advantage is that the process becomes much more realistic because now we're
using the normal repl. However, it was not possible to test autocomplete
this way since we cannot immitate the TAB key over the wire.

all comments addressed. PTAL

LG after a couple small comments. I'm very excited to have these tests!!

PTAL

LGTM

merged via https://github.com/angular/protractor/commit/45341c944bf60537849776c7f362ee0442b219e6


matplotlib chart generation?

sounds interesting, no idea about what you trying to achieve ;-; (and how)


Get All doesn't work

Bug description
Clicking the button Get All after having done a search, doesn't start any downloads.
Steps to reproduce

Set source to derpibooru
Search for pony AND -safe AND -monochrome
Click on Get All

Expected behavior
It should start downloading.
Context
As for the log, these are the last few entries after having tried to above several times:
[19:25:52.196][Info] [derpibooru.org][Json] Receiving page https://derpibooru.org/api/v1/json/search/images?per_page=20&page=1&q=pony AND -safe AND -monochrome
[19:25:52.208][Info] [derpibooru.org][Json] Parsed page https://derpibooru.org/api/v1/json/search/images?per_page=20&page=1&q=pony AND -safe AND -monochrome: 20 images (20), 0 tags (296), 34845 total (34845), 1743 pages (1743)

(anything before that is just looking around, nothing special)
In other words, clicking the button adds nothing to the log.
It may also help to know, after clicking the button I see 0% cpu utilization and 0% network utilization for Grabber.
As for settings, I left them default. I haven't touched any setting since I started using this tool, which is about a week ago.
System information

OS: Windows 10
Grabber version: 7.3.2

Additional context
A few days ago, downloading a large amount of images, similar to this amount, worked fine for danbooru.

The "get all" button does not start downloads.
It only adds them to the "Downloads" tab where you can start them.
If you go there, can you see your added download and start it with the buttons at the bottom / right clicking it?

Hm, you're right. Thanks!
I don't know how I managed to do it last time then. Just messing around until something happened in a tool I'm not familiar with 🤷♂
Can you maybe add some ind of feedback that makes it clear what actually happens when clicking the button? Because as you might know, clicking the button appears to do nothing, as nothing happens visually. Do you think you can come up with a UX-friendly thing to provide a visual cue?

I agree that the UX is unintuitive and I also tripped on it the first few times I used Grabber, but I don't have an immediate suggestion on how to improve it.

Some ideas:

Flash the Downloads tab several short times, or highlight it until opened.
Add job counter in its title, or several counters like "(1/2/3)" meaning running/finished/total, without being long and requiring more translation. Probably complicated to consistently update from various other code places, but I think it's not useless.
Rename button captions from "Get all/this page" to "Add all/this page to Downloads tab". Simple, but needs translation.

Yeah, renaming "Get this" to "Enqueue download" or similar sounds like a quick fix, along with an option to immediately start downloads maybe, as I personally have never needed to "edit" my download list. When I submit something for download, I have to go to the tab, find the right button to press, click that, wait for the downloads to complete, close the popup and then delete them from the download list, so they aren't downloaded again (or attempted to). That's probably not the only use case of this, since I realize you can also keep a download item to get everything with a certain tag, but that's what monitors are for now, I presume.

Agreed. Changing the label of the "Get" options would make sense as they're misleading at the moment. That, or changing "Get " to "Added to Download tab" after you click it.


App Icons

Create App Icons and other assets needed

First stab at the app icon here:
dat-desktop-icons.zip
Let me know if we need other assets or formats.

awesome! i'll put them into the build process and see how it goes


Add tags to UDS resources during terraform deployment

Tagging of catalog (shared services)

Cumulus tf has been updated to include tags for remaining UDS resources in shared services dev, test, prod.


Contrast on bottom bar in light mode

Pretty much all variants do black text on dark grey which is pretty frustrating to try to read

Compare to Light (Visual Studio) theme

Hi @pattrickrice
The status bar should have the theme's colours, please see screenshot for mine

Could you please confirm that you're using the correct theme as there's a few gruvbox ones in the marketplace. The theme from this repo is https://marketplace.visualstudio.com/items?itemName=jdinhlife.gruvbox
Perhaps some more information in case we need to debug, such as your vscode version (stable/insiders), your OS (windows/macos/linux/...)

Well, I don't know how it got set, but I found this setting in my user settings that turned out to be the fault. Apologies for the false report!
    "workbench.colorCustomizations": {
        "statusBar.background": "#767676"
    },


Isort Compatiblity Black [Pre-Commit]

Description
I think we should have set the profile mode of isort to Black: Ref
Rationale
I had a lot of problem with github actions and git hook every time i have to check details. Fixed it with that config hope that help for others

How did you integrate this into your project, if you don't mind me asking?

How did you integrate this into your project, if you don't mind me asking?

you just have to put profile = black in last line of [isort] in setup.cfg
[isort]
line_length = 88
known_first_party = muslim_love,config
multi_line_output = 3
default_section = THIRDPARTY
skip = venv/
skip_glob = **/migrations/*.py
include_trailing_comma = true
force_grid_wrap = 0
use_parentheses = true
profile = black

Thanks for your reply, much appreciated.

Hi, I am just starting to contribute to open source, and I believe this issue would be a good fit for me. Can I take it up?

@priyank-panchal asking permission to fix an issue isn't necessary. Looking forward to your pull request.

Actually, that should be fixed since #4276


Please provide clearer instructions

Please, in this documentation page, stress more the fact that @triggerBody().folderPath @triggerBody().fileName must be used in the configuration page of the trigger (click Next in the triggering event details page to see the trigger params page), and NOT as Default Values or values for the Pipeline parameters, otherwise they won't work. The trigger won't even fire with expressions like @trigger().body.fileName. Errors reported by Datafactory are very cryptic and confusing (full of messed up HTML characters like "'" or "&#44") they should help solve the error with meaningful error code. It's very difficult to troubleshoot anomalies using the explanations in the error notifications of the "Monitor" view.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: b7ee5166-ff40-f4ed-ce07-5312f935e388
Version Independent ID: 68f01b43-e3b2-003b-24cb-446a08aa0c1b
Content: Create event-based triggers in Azure Data Factory
Content Source: articles/data-factory/how-to-create-event-trigger.md
Service: data-factory
GitHub Login: @sharonlo101
Microsoft Alias: shlo

@stefano-detoni Thanks for the feedback. We are investigating into the issue and will update as appropriate.

@stefano-detoni I am assigning the issue to the content owner to have a look and update.
@sharonlo101 Can you please take a look at this and update as appropriate?

I agree. This documentation is very incomplete

Thanks stefano. I was stuck when the monitor logs mentioned fileName and folderPath are not available in triggerBody until I saw your comment.

@djpmsft Hi Daniel, this doc needs to be updated. Could you please take a look.
Thanks.

I am updating the Event Trigger Documentation today

The article has been updated and includes a section on pipeline parameters. Apologies that the earlier guide was confusing

#please-close

The article has been updated and includes a section on pipeline parameters. Apologies that the earlier guide was confusing

Thank you @djpmsft


[APIReview] Rename UpdatedAt => UpdatedOn

We've been using On as the suffix for dates.  However, if a customer is upgrading from ASP.NET to ASP.NET Core, they are already used to UpdatedAt (and it's in their models), so this may not be a good thing.

For historical reasons (the old Zumo code uses ITableData), this is not going to be done.


Make swiping backwards on a list of forms take you to the previous screen

http://manage.dimagi.com/default.asp?157945#878277
Slightly concerned that I'm opening up a can of worms with this, but I don't have a good argument against swiping being equivalent to "back" in as many places as possible.

I like it a lot


wd_demo.au3 - SetupGecko - preset browser location

Pull request
Proposed changes
fixing issue _WD_CreateSession and WD_Demo extension
Checklist

[x] I have read and noticed the CODE OF CONDUCT document
[x] I have read and noticed the CONTRIBUTING document
[x] I have added necessary documentation or screenshots (if appropriate)

Types of changes

[ ] Bugfix (change which fixes an issue)
[ ] Feature (change which adds functionality)
[ ] Code style update (formatting, renaming)
[ ] Refactoring (functional, structural)
[ ] Documentation content changes
[x] Other

What is the current behavior?
https://github.com/Danp2/au3WebDriver/issues/424
What is the new behavior?
Fixed.
Additional context
https://github.com/Danp2/au3WebDriver/issues/424
System under test
Firefox

@mlipok Perhaps you misunderstood when I stated I would recommend calling _WD_GetBrowserPath, checking for errors, and then only perform the _WD_CapabilitiesAdd('binary' in the event that a path was found.
I was expecting you to check the result from _WD_GetBrowserPath before using it blindly.

@mlipok Perhaps you misunderstood when I stated I would recommend calling _WD_GetBrowserPath, checking for errors, and then only perform the _WD_CapabilitiesAdd('binary' in the event that a path was found.
I was expecting you to check the result from _WD_GetBrowserPath before using it blindly.

I thought checking ERROR was sufficient
https://github.com/Danp2/au3WebDriver/blob/4dded0ab3476e3498fb5f35553eb8a0db7d895a3/wd_demo.au3#L1144-L1146
https://github.com/Danp2/au3WebDriver/blob/4dded0ab3476e3498fb5f35553eb8a0db7d895a3/wd_demo.au3#L1197-L1199

currently:
https://github.com/Danp2/au3WebDriver/blob/de50ec3a16fe1d7bc03ea9eaff835abaf2408ea9/wd_demo.au3#L1144-L1148
https://github.com/Danp2/au3WebDriver/blob/de50ec3a16fe1d7bc03ea9eaff835abaf2408ea9/wd_demo.au3#L1201-L1205

I thought checking ERROR was sufficient

It is... but that wasn't how you originally did it here. 😉


Add Fluid Slider

Project URL
https://github.com/Ramotion/fluid-slider
Description
A slider widget with a popup bubble displaying the precise value selected.
Why it should be included to awesome-ios (optional)
Checklist

[x] Only one project/change is in this pull request
[ ] Addition in chronological order (bottom of category)
[x] Supports iOS 9 / tvOS 10 or later
[x] Supports Swift 4 or later
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

          3 Warnings

:warning:
Please rebase to get rid of the merge commits in this Pull Request

:warning:
Too many changes (when adding, please keep it to one project per Pull Request)

:warning:
Found 12 link issues, a project collaborator will take care of these, thanks :)

Link issues by awesome_bot

Line
Status
Link

205
Dupe
https://github.com/Wasappli/WAAppRouting

206
Dupe
https://github.com/button/DeepLinkKit

207
Dupe
https://github.com/intentkit/IntentKit

208
Dupe
https://github.com/joeldev/JLRoutes

209
Dupe
https://github.com/IanKeen/IKRouter

210
Dupe
https://github.com/hyperoslo/Compass

211
Dupe
https://github.com/SwiftKitz/Appz

212
Dupe
https://github.com/devxoul/URLNavigator

213
Dupe
https://github.com/avito-tech/Marshroute

214
Dupe
https://github.com/skyline75489/SwiftRouter

215
Dupe
https://github.com/freshOS/Router

1270
301
https://creativesdk.adobe.com/ redirects tohttps://www.adobe.io/apis/creativecloud/creativesdk.html

  Generated by :no_entry_sign: Danger


Releases are not being generated properly

🐛 Bug Report
For some reason, the CI/CD is not generating the releases properly. It is not a big problem but it would be cool to keep it working. Guix build system is not being affected by this. The sources of the tag are automatically published so it's not a major inconvenient.

Solved.


After loading game, new cities' default names restart from first name

I loaded a saved game from Civilization which already had "Moscow" and tons of other cities, built a new city, and it suggested "Moscow" again.  You can test it (and some other things probably) locally by loading this save.
http://lex.clansfx.co.uk/dump/civil0.map
http://lex.clansfx.co.uk/dump/civil0.sve

I wasn't aware of this yet, but it is linked to #153
Load/Save games are not fully implemented and extremely buggy at the moment.


Dialog/BottomSheet background color breaks on Impeller

Is there an existing issue for this?
This might be related to https://github.com/flutter/flutter/issues/135659 but it's closed now.

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

Steps to reproduce

Run the project using Impeller rendering engine
Press Show Dialog or showModalBottomSheet button.

Expected results
I except the background color to be white.
Actual results
The background color is not white.
Alternative workaround:
Setting elevation to 0 should fix this.
Code sample
Code sample
import 'package:flutter/material.dart';

void main() => runApp(const MyApp());

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      debugShowCheckedModeBanner: false,
      theme: ThemeData(
        colorScheme: ColorScheme.fromSeed(seedColor: Colors.deepPurple),
        useMaterial3: true,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  final String title;

  const MyHomePage({
    Key? key,
    required this.title,
  }) : super(key: key);

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: [
            TextButton(
              onPressed: () => showDialog<String>(
                context: context,
                builder: (BuildContext context) => AlertDialog(
                  backgroundColor: Colors.white,
                  title: const Text('AlertDialog Title'),
                  content: const Text('AlertDialog description'),
                  actions: <Widget>[
                    TextButton(
                      onPressed: () => Navigator.pop(context, 'Cancel'),
                      child: const Text('Cancel'),
                    ),
                    TextButton(
                      onPressed: () => Navigator.pop(context, 'OK'),
                      child: const Text('OK'),
                    ),
                  ],
                ),
              ),
              child: const Text('Show Dialog'),
            ),
            TextButton(
              child: const Text('showModalBottomSheet'),
              onPressed: () {
                showModalBottomSheet<void>(
                  context: context,
                  backgroundColor: Colors.white,
                  builder: (BuildContext context) {
                    return SizedBox(
                      height: 200,
                      child: Center(
                        child: Column(
                          mainAxisAlignment: MainAxisAlignment.center,
                          mainAxisSize: MainAxisSize.min,
                          children: <Widget>[
                            const Text('Modal BottomSheet'),
                            ElevatedButton(
                              child: const Text('Close BottomSheet'),
                              onPressed: () => Navigator.pop(context),
                            ),
                          ],
                        ),
                      ),
                    );
                  },
                );
              },
            ),
          ],
        ),
      ),
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration

Logs
No response
Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.13.6, on macOS 13.5.2 22G91 darwin-arm64 (Rosetta), locale en-EG)
    • Flutter version 3.13.6 on channel stable at /Users/bx777/.puro/envs/stable/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ead455963c (7 weeks ago), 2023-09-26 18:28:17 -0700
    • Engine revision a794cf2681
    • Dart version 3.1.3
    • DevTools version 2.25.0

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/bx777/Library/Android/sdk
    • Platform android-33, build-tools 33.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15A240d
    • CocoaPods version 1.13.0

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

[✓] Android Studio (version 2022.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)

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

[✓] Connected device (3 available)
    • iPhone 14 Pro Max (mobile) • 8F0DE058-66D7-4970-8C9D-DD01B3953F4B • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-17-0 (simulator)
    • macOS (desktop)            • macos                                • darwin-arm64   • macOS
      13.5.2 22G91 darwin-arm64 (Rosetta)
    • Chrome (web)               • chrome                               • web-javascript • Google
      Chrome 119.0.6045.123

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

• No issues found!

Thanks for filing this.
This doesn't seem to be an issue with flutter itself but the expected behaviour of the colour of Dialogs with Material 3, you can see https://m3.material.io/components/dialogs/overview#4ed577f5-fb0b-4e9f-9e41-cb0dbd4e9265 for more details.

Closing this as this seems to be the expected behaviour, if you disagree or if I misunderstood your issue, please comment and I'll reopen this.
Thank you!

You're right. The change in the background color comes from surfaceTintColor, setting it to Colors.transparent fixes it.
Thanks!


Website example paragraph gets shorter and shorter on multiple clicks.

Hi there, thanks for a great plugin! I noticed a bug on the website example. If you keep clicking on Truncate Text / Reset button, then the paragraph will get shorter and shorter. In the end you end up with Shave is a zero dependency javascriptthethe originaltact!. It's pretty easy to replicate.

@KrisOlszewski Thanks for pointing that out - it came with the optimization (~2x faster) & will be fixed as soon as possible. 🍻

The irony though :)

Fixed @KrisOlszewski . Thanks for pointing that out!


Make Ethernet cable for AKD service port to DAQ cube switch

This will allow adjustment of AKD parameters via Kollmorgen Workbench without attaching a cable each time.

Done yesterday.


MySQL table updater

Working on #7 resulted in a new table.
In this case, the user only needs to create a new table.
However, in the future, name and data types of columns may change, which would be highly my fault for having a poor design to start with.
Anyways, account for this and create a table updater script

Spent way too much time on making updater because I'm new to bash and things are so confusing. I need help or I need to learn. It's not worth this much time in this.
Also, because it's an updater, I expect the user to have mysql and Muniverse table in the user's mysql database if he/she needs updater.
Using mysqldiff or mysqldbcompare can fix many things as well.
So, I'll commit for now, and this updater will be done later.


Fix csp for wasm

This fixes the issue of not being able to generate a keyring with sr25519 type.
This PR also makes our CSP more strict. To my testing, making it stricter did not create any issue.
I followed Fabian's suggestions and made a small change on it: https://github.com/tauri-apps/tauri/issues/4160#issuecomment-1130363459
The small change: style-src 'unsafe-inline' 'self'; -> this means: for css, we are enabling inline scripts execution.
Actually, our app works without this as well, but becomes more ugly. Here is the difference
with this change:

without this change:

It is not recommended to enable this, I don't know how risky it will be for CSS files. It is up for discussion

We probably don't need unsafe-inline for styling. The color styling is coming from appData.ts as:
    Dialog.create({
      title: `Confirm selected directory.`,
      message: `

      Seleted plot directory.

      ${plotDirectory}

`,

Although we have self as the source for style-src, it seems appData.ts is not covered in the scope of self. I will research more on this and get rid of unsafe-inline.


Advertise use by Discord

I learned from Hacker News that Discord is using Rustler. This is just a suggestion, but if you can get a quote from Discord and put it on README, it would be a great marketing for production-readiness of Rustler.

@sanxiyn I think this would be interesting. I will reach out to someone and see if someone would be willing to provide such a quote.

I am sure you have seen it, but Discord made an official blog post: https://blog.discordapp.com/using-rust-to-scale-elixir-for-11-million-concurrent-users-c6f19fc029d3.

@sanxiyn - yep. Pretty neat. Thanks for pinging this issue again.


Invalidate importlib caches after dynamically installing module

Description:
As specified in the importlib documentation, the importlib caches must be invalidated after dynamically installing a module.
Checklist:

[x] My PR is ready for code review
[ ] I have added some tests, if applicable, and run the whole test suite, including linting tests
[ ] I have updated the documentation, if applicable

Codecov Report

Merging #838 into master will decrease coverage by 0.06%.
The diff coverage is 44.44%.

@@            Coverage Diff            @@
##           master    #838      +/-   ##
=========================================
- Coverage   88.96%   88.9%   -0.07%
=========================================
  Files          79      79
  Lines        5095    5103       +8
  Branches      968     968
=========================================
+ Hits         4533    4537       +4
- Misses        417     421       +4
  Partials      145     145


Add pipeline support to Get-DbaModule

Type of Change

[ ] Bug fix (non-breaking change, fixes #)
[X] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[X] Pester test is included
[ ] If new file reference added for test, has is been added to github.com/sqlcollaborative/appveyor-lab ?
[ ] Nunit test is included
[X] Documentation
[ ] Build system

Purpose
Add Pipeline support to the Get-DbaModule function allowing piping from Get-DbaDatabase.
Approach
Added InputObject that takes a Microsoft.SqlServer.Management.Smo.Database collection
Commands to test
Get-DbaModule

Love it! Great idea. Which reminds me, we should write a blog post about how useful get-dbamodule is. i use it all the time!


[WIP] Fill in the default bridge port options if missing

When adding port into bridge without stp options, the nmstate will fail the
validation as stp-hairpin-mode and other stp options are missing.
To fix that, when stp is enabled, fill in the default value of stp
options if missing.

Coverage decreased (-0.06%) to 69.327% when pulling 521182cabf97b0aec694065a59cb90ebef61336a on cathay4t:br_fix into aea3286f8d1cb18efa2b3ebdc8d0eb6d8ba2c0c8 on nmstate:master.

Coverage decreased (-0.06%) to 69.327% when pulling 521182cabf97b0aec694065a59cb90ebef61336a on cathay4t:br_fix into aea3286f8d1cb18efa2b3ebdc8d0eb6d8ba2c0c8 on nmstate:master.

@tyll @EdDev Even this PR is WIP, but I need your input forehead before I spend time on creating test case for it.

Does this work with stp enabled: false too ?

Does this work with stp enabled: false too ?

That's will be another PR. Currently other nmstate guys want a PR as small as possible.
Once this PR approved, it should be very simple to handle when stp enabled: false.
Thanks for your remind.

Unfortunately, I do not understand from the commit message what is the exact scenario you are trying to fix here.
For my feedback below, I am guessing that it has to do with specifying a port with only the name and nothing else:
If the problem is with the verification (desire vs current), the fix should be in the partial check and not in the canonicalization of the input. Defaults are dangerous, they may change between versions and we may not notice. It is also not built to last for long (maintenance), as new properties may be added in the future.
As we do with other special properties, we could dedicate for the ports a dedicated assertion which will check inclusion and not full state.

If the problem is with the verification (desire vs current), the fix should be in the partial check and not in the canonicalization of the input. Defaults are dangerous, they may change between versions and we may not notice. It is also not built to last for long (maintenance), as new properties may be added in the future.
As we do with other special properties, we could dedicate for the ports a dedicated assertion which will check inclusion and not full state.

This makes sense to me.

If the problem is with the verification (desire vs current), the fix should be in the partial check and not in the canonicalization of the input. Defaults are dangerous, they may change between versions and we may not notice. It is also not built to last for long (maintenance), as new properties may be added in the future.

I actually think enforcing default value explicitly is better:

We give clear expectation/documentation to user other than let NM decide the default values is. This matters more when we support multiple backends.
As you mentioned, default values could be changing between versions, we need to enforcing the default values to the setting profiles.

@EdDev New patch uploaded.
Changes:

Rewrite everything, please review again.

The workflow is copy from the object based PR #425 .

I don't have the unit test case create yet. Will do if you approve this workflow.

We give clear expectation/documentation to user other than let NM decide the default values is. This matters more when we support multiple backends.
As you mentioned, default values could be changing between versions, we need to enforcing the default values to the setting profiles.

The strategy you propose here contradicts the existing one.
In practice, you would like Nmstate to enforce defaults when the user is not specifying properties.
Taking this path will require Nmstate to cover all other configurations this way to be consistent and adhere to this strategy. This will impose on Nmstate the ownership of setting these defaults, instead of passing it to the providers and implementation (kernel).
I do not think we are interested in this burden or ownership.
From the user perspective, the user specifies what it is interested in, anything else is in the "I do not care" area. Therefore, the user is fine with the defaults not being hard defined, otherwise the user would have been specified those explicitly.
History has shown that defaults change, especially between kernel versions. Also, new properties are frequently added (with defaults). All hint towards a strategy of not controlling them, letting the logic be taken down the stack, avoiding the maintenance burden on Nmstate.
An example of an extreme are the Linux bonds, those have a complicated list of properties, dependent on the bond mode.
If I look at the bridge port settings, what would those defaults be in case STP is enabled on the birdge and what  would they be if it is enabled? Are the current default correct? Who are owning them, NM or the kernel? Is Nmstate the 3rd layer who manages such defaults? I prefer to avoid the need to answer all these for all possible configurations of all devices.
Exceptions may exists, but those would probably be focused on Nmstate related abstractions, not on lower level details.

OK. I will create new patch for ignore undefined property at verification stage.

@EdDev New patch uploaded.
Changes:

Change the verification equal to use new function _state_is_match().

@EdDev I don't get your expectation on this PR. Are you suggesting me to do a simple fix for linux bridge
or you want me to add more test to ensure current fix does not break things.

@cathay4t , I am fine with your approach here, but it needs unit testing that state_match functionality.
There are a lot of scenarios, so it may be a lot of tests.
For example:
assert state_match({}, {})
assert state_match({}, {'foo': 'boo'})
assert not state_match({'foo': 'boo'}, {})
assert not state_match({'foo': 'boo'}, {'boo': 'foo'})
assert not state_match({'foo': []}, {'boo': ['foo']})
...

Doing something only for the bridge ports (using the update) is the alternative, there you need no unit tests because it is supposed to be covered already (I hope).
But I am not favouring this option over the general one, I'm just pointing out what is the alternative and costs.

New patch uploaded.
Changes:

Renamed function to state.state_match() using all() to simplify the code.
Unit test case added.
New integration testlib assertlib.assert_state_match() for matching check.
This allows other CI tests still using __eq__ for verification which could also
testing whether state.state_match().


Consider adding the ability to get/set a signal's value without being tracked

It can be somewhat easy to end up in an infinite reactivity loop when accessing/setting signals. Often times, it is desirable to access/set a value silently, without triggering that signal's dependents. This should be possible on all signal types, such as ReadSignal, Memo, etc. If ReadSignal::derive mentioned here were to be implemented, it would make this even more useful, as I could imagine it not being possible to silently get a value if using just a plain impl Fn() -> _.

I think this is a great idea and the inner workings are already there
https://github.com/gbj/leptos/blob/96238c553e414664e0a688cf6e5804af4fc9dfe8/leptos_reactive/src/signal.rs#L149-L151
Would you like to offer a PR that implements something like with_untracked on ReadSignal/RwSignal/Memo and set_untracked on WriteSignal/RwSignal?

Sure! How about an UntrackedSignal trait?

As opposed to similar associated functions? I think that probably makes sense.


can we show these notifications in the uProxy app ?

can we show these notifications in the uProxy app ?

Agreed, these notifications need to be cleaned up
On Fri, Jan 30, 2015 at 3:45 PM, klazizpro<EMAIL_ADDRESS>wrote:

can we show these notifications in the uProxy app ?
[image: capture]
https://cloud.githubusercontent.com/assets/1900042/5982973/befcd210-a896-11e4-8cf9-2274af80617e.PNG
—
Reply to this email directly or view it on GitHub
https://github.com/uProxy/uproxy/issues/829.

Do these automatically dismiss after [x] seconds?

No, but we can do that.
On Fri, Jan 30, 2015 at 6:45 PM, Izzie Zahorian<EMAIL_ADDRESS>wrote:

Do these automatically dismiss after [x] seconds?
—
Reply to this email directly or view it on GitHub
https://github.com/uProxy/uproxy/issues/829#issuecomment-72290438.

Wonderful. That will help clean up a lot.

Add a checkbox in the advanced settings to show them or not?

Moved UI/UX for showing/not showing notifications to a separate issue. https://github.com/uProxy/uproxy/issues/948


class_weights dictionary for one-hot vectors?

What is the best practice in Keras for class balancing with one-hot vectors? I can't make a dictionary with {vectors:weights} because vectors are not hash-able.
Or is this a non-issue? Does Keras map class labels and weights to the appropriate one-hot vectors in my batches?

Maybe, we can use sample_weight parameter?

sample_weight: optional array of the same length as x, containing
weights to apply to the model's loss for each sample.
In the case of temporal data, you can pass a 2D array
with shape (samples, sequence_length),
to apply a different weight to every timestep of every sample.
In this case you should make sure to specify
sample_weight_mode="temporal" in compile().

https://github.com/fchollet/keras/blob/d89afdfd82e6e27b850d910890f4a4059ddea331/keras/engine/training.py#L1392

There's sample_weights for generator, too.
https://github.com/fchollet/keras/blob/73bf06fb023a8b37ddf2e2a168bbf920c7a6c766/keras/models.py#L1030


Write documentation guidelines

[WIP]
This is an issue to keep track of the items we want to address in our updated contribution guidelines for the new devhub launch.
Components
List all different components: Tip, Advice, Note, Warning, Important
Convention and examples of which instances to use them in
Docs, Tutorials, How-to guides
Full-stops after bullet-points. Convention tbd
TODO: Add link to the different guidelines (htgs, kb docs) here.

Add in stuff on Prettier here: https://github.com/substrate-developer-hub/substrate-docs/pull/210

I like the idea of customizable titles, but I would like to revisit the use
of these components (when, where, and how to use them). Is this something a
Ui / UC person recommended? If so, I’d like to involve him or her in the
discussion post launch.
On Thu, Oct 7, 2021 at 10:43 AM Dan Shields @.***>
wrote:

<Message
type={gray}
title={Note}
text={"Experienced developers may 'prefer' i've to skip that tutorial and
install the Node Template according to the instructions in > its readme."}
/>
So we want to enforce that the exact titles are preserved as we have in
v3/docs/Examples/Components/index.mdx ?
I actually prefer that we can edit this to really make that title stand
out. Here is an example:
[image: image]
https://user-images.githubusercontent.com/35669742/136435768-23932a88-5317-428b-b4bb-7d3efe7293de.png
If we don't want people to change (or have the ability to remove) the
title, I would propose we make a new issue (phase-2 or latter) to remove
this property and draw that from the type where that was not color, but
of the selection note,info,warn,adv,future-reading,... and not have a
user defined color.
Happy to go with whatever you think is best right now @sacha-l
https://github.com/sacha-l - just indicate yes/no here for custom
titles for us 🙏
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/substrate-developer-hub/substrate-docs/issues/4#issuecomment-938015127,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKRISG6YI5J5DV24XBCZS2LUFXL4FANCNFSM5BCSUV6A
.

💡 A note to include Substrate's contributing guidelines somehow. Maybe have a separate section referring to this doc for developers wanting to contribute to the codebase.

From Imad's recent PR: https://github.com/substrate-developer-hub/substrate-docs/pull/449#issuecomment-954503200
Steps to Contribute
Docs/HTGs (Adding new .mdx files)

Create a New .mdx file(s) in the same folder tree
Update DevNavMenu to reflect the addition of pages.

Tutorials (Adding new ones)

Create a folder under /v3/tutorials/
Create or place index.mdx inside the folder.
Make sure to have ALL the following frontmatter defined

---
title: Create your first Substrate blockchain
slug: /tutorials/v3/create-your-first-substrate-chain
hideNav: true
sideNav: firstChain
version: 3.0
section: tutorials
category: substrate chain
keywords: node template, basics
difficulty: 1
duration: 1 Hour
relevantSkills:
  - Rust
  - Blockchain basics
---

Update DevNavMenu.tsx and add the menu items for tutorial. NOTE the name MUST match sideNav field in frontmatter.
Gatsby will do the rest. No need to go into gatsby-config or gatsby-node file.
🥳

With https://github.com/substrate-developer-hub/substrate-docs/pull/482 - we want to run yarn fmt:docs locally before making any final PR merge, and #500

With Main-md branch launching, we will need to update and add this to the new content

Closing this issue as obsolete (MDX-specific) and buried in the queue.
We do have a writing style guide:
https://www.notion.so/paritytechnologies/Using-this-guide-for-writing-and-style-581ffe4e80ea498d9fa0ffe4c00a753d
https://github.com/substrate-developer-hub/substrate-docs/tree/archive/main-mdx/v3/docs/00-style-and-contributor-guidelines/b-main-style-guide
We do have contributor's guidelines:
https://github.com/substrate-developer-hub/substrate-docs/tree/archive/main-mdx/v3/docs/00-style-and-contributor-guidelines/a-contributor-guidelines
They haven't been moved over and updated.


The "originally published" prefix is getting added multiple times

LoadRewoundFeed method was modifying the original feed when rescheduling and updating feed items, which got persisted in the cache.


disallow blocking event loop, add Catnip#catnipAsync, only print codec warning once

also first PR of the year

First merge of 2019 wheee


[DEVOPS] Add docker config. validation step to test workflow

Tickets addressed: None
Review: By commit
Merge strategy: Merge (no squash)

Description
Adds docker config. validation step to test GH Action.
Verification
Tested with an intentionally erroneous config.:
ERROR: The Compose file '././docker-compose.yml' is invalid because:
services.aerie_merlin.build.args contains an invalid type, it should be an object, or an array

Documentation
None.

Does this fail the test workflow when it finds an error in a compose file?

@camargo I'll test this out in this PR, good idea. I tested this with a ... && echo passed locally and the echo statement was not hit so a non-zero exit status should propagate to the test workflow.

@camargo https://github.com/NASA-AMMOS/aerie/runs/6012834575?check_suite_focus=true The error propagates :) I'll drop the erroneous commit now.


Inconvenient Transition Back to the Query Editor

When you open up ClickHouse Query Editor you see this

You can click on the Go to Query To see something else

However this screen does not have any links to go back to the previous settings.
Suggestion: It would be better if link is added (as exists in MySQL Data Source for Example)

For me, pressing the pencil button in the raw query view, brings me back to the query builder. Is this what you're after?

I think this can be closed as it works like @olafbuitelaar pointed out.

@negbie ok, let's close issue


Istio with MTLS strict reporting certificate expired errors

Bug Description
Using istio 1.12.4 with MTLS set to strict on GKE cluster v1.21.9-gke.300, we’re currently facing an issue with tcp connections failing to get established to destination pods after they’ve been running for quite some time (span of days and weeks). By enabling debug logs through "sidecar.istio.io/logLevel": debug we were able to spot the following TLS error:
debug	envoy connection	[C8] TLS error: 268436501:SSL routines:OPENSSL_internal:SSLV3_ALERT_CERTIFICATE_EXPIRED

Inspecting the config dump reported by both client and destination pods we saw the entry for the certificate chain and by decoding it we saw that the certificate is still valid. So we don’t know exactly where this error is coming from.
Restarting the pod or setting mtls to disable helps to mitigate this issue and connections start working again, however, this is far from ideal.
Other issues that seem to be related:
https://github.com/istio/istio/issues/38081
https://github.com/istio/istio/issues/14942
Version
Istio version: 1.12.4
K8s version: v1.21.9-gke.300

Once it starts breaking does it break 100% of requests, or only some? Sounds like it never recovers without intervention, right? Any chance you waited >12hrs without intervention before and witnessed if it recovered?
Any chance you could look at the time as reported by the cluster nodes to verify it is accurate?

Can you paste the logs of sidedcar, you can strip the accesslog. There are some logs about cert create/rotate

To give more context about our environment, this is affecting various databases that have been deployed for a few weeks already (right now 23 days+, but it could have started earlier without us noticing).
We’ve done more thorough tests about the amount of requests that fail and also trying other mechanisms to get around this issue with the databases that we use, in this case postgres, mysql and mongodb.
To answer specifically your question, once this issue starts, for postgres and mongodb the initial connection attempt from the client pod to the database will fail. We rewrote our pod command in a way that when the container starts, it doesn’t try to connect right away but sleeps for 60s instead and then attempts to connect. This worked well and connection was successfully established.
Another experiment we did was to wrap the connection command from the client pod in a for loop with 5 iterations. In this case the first iteration always failed, but the remaining ones succeeded without issues.
For postgres failed connections the error reported is:
psql: server closed the connection unexpectedly
This probably means the server terminated abnormally

And for mongodb:
Failed: can't create session: could not connect to server: server selection error: server selection timeout, current topology: { Type: ReplicaSetNoPrimary, Servers: [{ Addr: mongo-0.mongodb-headless.namespace.svc.cluster.local:27017, Type: Unknown, Last error: connection() error occured during connection handshake: read tcp <IP_ADDRESS>:47328-><IP_ADDRESS>:27017: read: connection reset by peer }, ] }

However, we also experimented the same way with various mysql databases that were showing this same symptoms (also 23+ days old). Adding the sleep or the for loop didn’t help to mitigate this issue. We also spawned a mysql-client pod, exec into it and tried to connect multiple times to the database. All of the connection attempts failed with the following error: ERROR 2013 (HY000): Lost connection to MySQL server at 'reading initial communication packet', system error: 0 .
We then verified the istio debug logs, and all of databases reported the same error for failed connections TLS error: 268436501:SSL routines:OPENSSL_internal:SSLV3_ALERT_CERTIFICATE_EXPIRED. We also spotted the access logs for connection error with UF, URX codes: [2022-04-08T18:28:14.492Z] "- - -" 0 UF,URX - - "-" 0 0 29 - "-" "-" "-" "-" "<IP_ADDRESS>:3306" outbound|3306||postgresql-primary.namespace.svc.cluster.local - <IP_ADDRESS>:3306 <IP_ADDRESS>:54712 - -
For all of the databases, either restarting it or disabling mtls helped to mitigate the situation. After that, client pods were able to connect to the database without the need of a sleep/retry mechanism.
The issue doesn’t go away on its own, looks like when it starts happening it’s there until either we disable mtls or restart the database.
Here I attach the logs for one of the postgres databases.
Explore-logs-2022-04-08 14_08_36 (1).txt

were you able to figure out why this is happening ? We are seeing this as well in istio 1.14 with EKS 1.22


Unable to download the Health Alpaca model checkpoint

Hi,
When clicking on the Google Drive link for downloading the HealthAlpaca checkpoint, it is asking to “Request Access”. Kindly provide access to the model

I have the same issue accessing the checkpoint. It's kinda hard to reproduce the result without his permission. Some important files and dataset is missing in his project.


Update netty-all, netty-codec-http to 4.1.60.Final

Updates

io.netty:netty-all
io.netty:netty-codec-http

from 4.1.56.Final to 4.1.60.Final.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.netty" } ]

labels: library-update

Superseded by #79.


Fixed a bug where mk_dateWithoutTime would return nil

Sometimes with no apparent reason, mk_dateWithoutTime  would return nil
This PR fixes that problem, plus, it might be more efficient this way (compared to using reverse formatting).

Cheers. Indeed, it seems to be more efficient way. I really wonder why it would return nil though. I will merge it shortly.

I am afraid I cannot merge it as it fails unit tests.


Find strategy for DHT to always find peers

Currently we have 2 bootstrap nodes deployed in our test network.
Whenever a new peer comes in, uses those bootstrap nodes to find other peers and propagate themselves.
In the scenario where 2 peers start up ~ at the same time (current deployment) it is possible that the local DHT for each peer doesnt have the latest routing info of the peers connected to the network.
As we are using kubernetes deployment, the internal IP of one of the interfaces changes (virtual networking) and it updates the bootstraps but just after the other nodes have finished bootstrapping so they do not get the latest.
Sounds like a pretty basic problem that the DHT should handle, but it doesnt seem to.
Find a solution to this.

Fixed by using external IPs https://github.com/centrifuge/go-centrifuge/pull/373