Determine if the hand-written well-known types are necessary

SwiftProtobuf has three hand-written implementations of the types in three of Google's well-known type files: Any, Struct, and Wrappers. These pose difficulties for future maintenance if they must be kept in sync by hand instead of generated.
We need to determine if there are advantages to keeping these hand-written, and if the advantages are significant enough to keep doing it compared to generating them. For example, can any custom behavior just be added via extensions instead?

Wrappers could easily be implemented as a generated base + extensions for the custom JSON behaviors.  Someone recently asked me about "easy starter projects" -- I think that would be one.
I suspect Struct and even Any could also be done that way, though Any might require some adjustments to the rest of the library to expose the right customization points.

The NullValue enum in struct.proto is going to make this a little trickier. Since it's an enum and the json property is generated directly on the type, there's no way to have an extension replace it. (Aside: Even if we extract JSON support into separate extensions, we can't introduce our own hand-written extension on the same type without it causing an ambiguity.)
I haven't thought too deeply about how to solve this yet, but the options that come to mind feel like hacks:

Add a method to ProtoEnum like remapJSONName(_ value: Self, _ name: String) and have the generated code call it, passing it the JSON string for the case. The default implementation in the protocol extension would just return the string verbatim. A hand-written extension on concrete enum types could then override it and return something else (in this case, "null").
Just add a special case for Google_Protobuf_NullValue in the JSON serialization code.
Other options...?

Any option will pick will likely have a small cost in terms of runtime library size, generated code size, and execution speed. Hopefully they're negligible; in terms of execution speed at least, surely the rest of the string processing will dominate whatever we add.

Since it's an enum and the json property is generated directly on the type, there's no way to have an extension replace it.

The technique I used to allow overriding generated methods on messages could work here.  In the library, define:
extension ProtobufEnum {
   var json: String {return _protoc_generated_json}
}

The generator then emits a definition for _protoc_generated_json instead of defining json directly.  Then we can override json in an extension (struct/enum extensions can override protocol extensions).

@allevato is working on this for Struct.  Once that lands, I've got an idea for how we might be able to do this for Any.

#387 removed the Any version.  This is now done.

🎉 🎉 🎉

Good work!


DOM Node Count Vs. Actual Nodes

Hi,
In Chrome Develper Tools at timeline, i saw below counters:
1.Document Count
2.DOM Node Count
3.Event Listener Count
I was able decrease Document Count & Event Listener Count by using below ractive methods:
.teardown()
.off()
.cancel()
But, i could not do so for DOM Node Counter. How to reduce this count?
Huge variation between actual nodes and DOM node counter. In spite of this, my application is woking smoothly. But would like to know the reason.
Does it have anything to do with the way dom is updated by ractive?
Thanks
koti

I don't see any bonus nodes floating around, so I'm going to close this. Feel free to reopen if it crops back up.


Fix panic in iptables when there is an error

This occurs when err is nil but there is stderr output
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any user-facing changes. This may include CLI changes, API changes, behavior changes, performance improvements, etc.

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
51bc6a850ebb31bb37ebf12b50de7ed85b459aab
link
/test unit-tests_istio

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

/retest

/cherry-pick release-1.11

@bianpengyuan: new pull request created: #34281

In response to this:

/cherry-pick release-1.11

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


MH-12797 Explain UI actions (added missing tooltips)

Hi @lkiesow This is the follow-up pull request of PR #164 I've promised.


Support for SxS .Net Core SDK's

Being able to automatically get them

We don't know what this means yet without out a design, so it's XLarge until then.

Since the SDK comes through NuGet, it's possible to have SxS SDKs.


[Task]: Deduplicate components & Grayscale/theming support in template page

Issues with templates:

Grey background seems to be missing on the template images
Width of each template card has decreased when compared to prod
Icon is missing in the Fork Template dialog box
Components layout are broken on the Reconnect datasources screen

Loom for all the above mentioned issues: https://www.loom.com/share/2f54b110eb6e4aa4b06babb7eb9822e7


Add my 3.0 Preview 1-3 breaking changes

@bricelam @divega Merging this so I can re-base my PR on it. Content can be reviewed when I send out my PR.


Question about API key

Hello, sorry i can find yours email. I have a question about option "SteamApiKey". Can I use one API key (e.g. from master account) or i must spend $5 for each bot and get its own?

Every account should have unique API key.
Cards are locked and you can't trade them on limited accounts anyway.

Tell me, how the exchange is going on? need some commandos to send to send or exchange it passes automatically to the specified main steamid

If you could stop stealing other users threads I'd appreciate.

really can not understand the principle of your program, the description is not told about sending trades. I did not understand you or not yet realized.

Damn, thanks, i fogot about it...


"Cleanup" cast list after manually uninstalled an app

When I uninstall an app manually without brew cask uninstall, the brew cask list still shows this app. How to tidy up the app list or check out which app is not in /Applications directory?

Use brew cask uninstall --force. We can’t guess when you install things manually.


docs(samples): add usage samples to show handling of LRO response Operation

Fixes #415
Description

For a detailed description read internal ref b/189985582
The Kubernetes engine API protos were written before the API Improvement Proposal (AIP-151)
AIP-151 suggests to annotate API calls that takes a long time to complete with a certain annotation. This will enable the auto-generator (that generates our client libraries) to wrap the response an object of type google.longrunning.Operation in a language native (Python Future or Nodejs Promise) async construct.
However, since  AIP-151 came long after the container.v1 API protos were written, the container API protos don't have the necessary annotation as guided in AIP-151
As a result the auto generated container client libraries do not return the very useful google.longrunning.Operation object, instead they return google.container.v1.Operation
This object does not have the necessary built-ins to easily verify if the operation has completed.
This this PR adds a few samples to show to customers how they can use this Operation object to validate if their operation has successfully completed using polling with backoff on the getOperation() API

Changes

Added 2 samples (create_cluster & delete_cluster) to showcase usage of the library and to show how to handle responses of type google.container.v1.Operation
Upon creation we use a fibonacci backoff to retry until the operation is complete
I have also added test cases for the added samples

Testing

Create a project
Enable the container (GKE) API
Create a service account with permissions to create/delete clusters (can use Owner)
Download a key for the created service account

export GOOGLE_APPLICATION_CREDENTIALS="<PATH_TO_KEY>"
gcloud config set project <GCP_PROJECT>

cd <repo_path>
git checkout lro-fix

node samples/create_cluster.js --zone=us-central1-b --name=gke10
node samples/delete_cluster.js --zone=us-central1-b --name=gke10

npm run samples-test

@sofisl - Please TAL and let me know if it's good!

Thanks for the very detailed pull-request description, Shabir. 👏
I have yet to provide more rigorous feedback and test the two scripts.
I'll likely be making some tweaks to this pull-request some time this week.
In the meantime, I realized that one of the tests is failing on Kokoro.
I believe it's because Node.js 12 doesn't support Crypto.randomUUID().
Judging from this article and this StackOverflow post, we may need to:
npm install uuid

If we could somehow use Node.js 14.17 or higher, we could use crypto.randomUUID natively. But I assume a Node upgrade is outside the scope of this PR.

Beside LRO, looks good to me. Gax LRO cover the retry logic to polling the operation status, once it finish, it return the response object. See details at HOWTO. Please feel free to reach out if you have further question.

Thank you for the review @summer-ji-eng. Please see the response to your comment about the LRO operation. Let me know if I had mis-understood it!

Thanks for the very detailed pull-request description, Shabir. 👏
I have yet to provide more rigorous feedback and test the two scripts. I'll likely be making some tweaks to this pull-request some time this week.
. ...............

Thank you very much @NimJay for the review. I have updated the sample to use your suggestion and tested in with node12.

@summer-ji-eng , @sofisl -
Seems like the test is failing because the default network on the project used for the tests is of type Legacy. The project used for the tests seems to be long-door-651 as per the kokoro configs. And this project's default network is indeed of type Legacy. Is it possible that we migrate the tests to run a different project with a non-legacy default network maintained by the client-libraries-nodejs team?


Validate user-returned data from resolveInput hook

The resolveInput hook allows the app developer to return an arbitrary object which then gets passed into internal workings of KS5. If this object is incorrectly structured, it will throw confusing errors such as:
Task validation failed: goal: Cast to ObjectID failed for value "{ connect: { id: '5cf8c1dfc69e5402f905811c' } }" at path "goal"

In this case, the mistake is returning originalInput from the resolveInput() method instead of the resolvedData:
resolveInput({ resolvedData, originalInput }) {
  return originalInput; // <-- WRONG
}

(where:
const resolvedData = {
  goal: "5dslkjdsflkjwer",
};
const originalInput = {
  goal: {
    connect: { id: "5gdfgiousdfjkhasdlkj" }
  }
};

)
Ideally we'd catch this mistake and throw an error such as:

resolveInput() hook returned data of incorrect shape, expected "String" at path "goal", received "Object". Did you mean to return resolvedData?

I had some trouble last night using hooks too. In the afterChange hook, I was comparing updatedItem.id with the array of ids of related items.
The array of ids of Relationship items is an array of Mongo ObjectIds while updatedItem.id is a string.
Is that a related issue? If not I will create a new one.
keystone.createList('SkillNode', {
  fields: {
    id: MongoId.primaryKeyDefaults.mongoose.getConfig(),
    name: { type: Text, schemaDoc: "The name of this skill" },
    slug: { type: Slug, from: 'name', isUnique: true, isRequired: true },
    children: {
      type: Relationship,
      ref: 'SkillNode.children,
      many: true,
    },
    children: {
      type: Relationship,
      ref: 'SkillNode.parents',
    },
  },
  hooks: {
    afterChange: ({ updatedItem }) => {

      const { children, existingItem, id } = updatedItem

      console.log(children.findIndex(el => {
        return el === id                         // false
      }))

      console.log(children.findIndex(el => {
        console.log(typeof el, typeof id)        // object string
        console.log(Object.keys(el))             // [ '_bsontype', 'id' ]
        return el.toString() === id.toString()   // true
      }))
      return updatedItem
    }
  }
});

This sounds like a seperate issue which we need to address! Can you open a new GitHub issue? Good catch 👍

I've created a new issue: https://github.com/keystonejs/keystone-5/issues/1499

Seems like I've landed here again! This looks like a related issue this time, what do you think?
bundle.esm.js:63 Uncaught (in promise) Error: GraphQL error: Cast to ObjectId failed for value "{ connect: { id: '5d57836667a5b04d9b5bf577' } }" at path "_id"
    at new ApolloError (bundle.esm.js:63)


Huge master nodes cpu/memory consumption

After upgrading to 4.11 we noticed some issue with master nodes. All of sudden master nodes start using a lot of cpu/memory which causes nodes crash.
Pods stuck in ContainerCreating and Terminating status.
Please see the attached gather logs.
must-gather.tar.gz

Hi again @dkoci! Have you looked over https://github.com/okd-project/okd/issues/1310?
It's a pretty epic read but there's some graphs and narrative in there and lots of folks have chimed in as well. Worth going through some of that to see if it matches your observations.

Thanks for sharing. It looks the same as our issue. But There is no solution on the horizon.

Dupe of #1310


RHCLOUD-36959 | fix: integration tests not passing

When reducing the noise in the logs by the commit
03488aa83e4479f5fce9bb7bd3e5e7f1140caa16, I forgot to fix the integration tests for Sources.
This new commit fixes those, and leverages the use of "errors.As" instead of having to compare that to a custom created object for the tests.
Jira ticket
[RHCLOUD-36959]

/konflux


virtualAddresses with mask on Service_HTTPS creates port redirect 80 with /32

Environment

Application Services Version: 3.20
BIG-IP Version: 15.1

Summary
Create a declaration with virtualAddresses as a network mask in Service_HTTPS, and you will notice that the port 80 redirect does not have the mask. It's a /32.
Steps To Reproduce
Steps to reproduce the behavior:

Submit the following declaration:

{
  "class": "AS3",
  "action": "deploy",
  "persist": true,
  "declaration": {
    "class": "ADC",
    "schemaVersion": "3.20.0",
    "id": "123abc",
    "label": "Sample 1",
    "Sample_01": {
      "class": "Tenant",
      "A1": {
        "class": "Application",
        "template": "https",
        "serviceMain": {
          "class": "Service_HTTPS",
          "virtualAddresses": [
            "<IP_ADDRESS>/24"
          ]
        }
      }
    }
  }
}

Check BIG-IP virtual servers and look for port 80 and 443 new objects

Expected Behavior
A port 80 redirect with address <IP_ADDRESS>/24 going to 443 <IP_ADDRESS>/24.
Actual Behavior
A port 80 redirect with address <IP_ADDRESS>/32 going to 443 <IP_ADDRESS>/24.

Workaround...create serviceAddress and reference that in the virtualAddresses object. Now the 80 redirect uses the /24 mask.
    "Sample_01": {
      "class": "Tenant",
      "A1": {
        "class": "Application",
        "template": "https",
        "serviceAddress":{
          "class":"Service_Address",
          "virtualAddress":"<IP_ADDRESS>/24"
        },
        "serviceMain": {
          "class": "Service_HTTPS",
          "virtualAddresses": [
            {
              "use": "serviceAddress"
            }
          ],

Your example declaration is invalid:
{
    "code": 422,
    "errors": [
        "/Sample_01/A1/serviceMain: should have required property 'serverTLS'"
    ],
    "declarationFullId": "",
    "message": "declaration is invalid"
}

Reproduced with this declaration:
{
    "class": "ADC",
    "schemaVersion": "3.0.0",
    "tenant": {
        "class": "Tenant",
        "application": {
            "class": "Application",
            "service": {
                "class": "Service_HTTPS",
                "virtualAddresses": [
                    "<IP_ADDRESS>/24"
                ],
                "serverTLS": "webtls"
            },
            "webtls": {
                "class": "TLS_Server",
                "certificates": [
                    {
                        "certificate": "webcert"
                    }
                ]
            },
            "webcert": {
                "class": "Certificate",
                "remark": "in practice we recommend using a passphrase",
                "certificate": "-----BEGIN CERTIFICATE-----\nMIICnDCCAgWgAwIBAgIJAJ5n2b0OCEjwMA0GCSqGSIb3DQEBCwUAMGcxCzAJBgNVBAYTAlVTMRMwEQYDVQQIDApXYXNoaW5ndG9uMRAwDgYDVQQHDAdTZWF0dGxlMRQwEgYDVQQKDAtmNV9OZXR3b3JrczEbMBkGA1UEAwwSc2FtcGxlLmV4YW1wbGUubmV0MB4XDTE3MTEyNjE5NTAyNFoXDTE4MDIyNTE5NTAyNFowZzELMAkGA1UEBhMCVVMxEzARBgNVBAgMCldhc2hpbmd0b24xEDAOBgNVBAcMB1NlYXR0bGUxFDASBgNVBAoMC2Y1X05ldHdvcmtzMRswGQYDVQQDDBJzYW1wbGUuZXhhbXBsZS5uZXQwgZ8wDQYJKoZIhvcNAQEBBQADgY0AMIGJAoGBALEsuXmSXVQpYjrZPW+WiTBjn491mwZYT7Q92V1HlSBtM6WdWlK1aZN5sovfKtOX7Yrm8xa+e4o/zJ2QYLyyv5O+t2EGN/4qUEjEAPY9mwJdfzRQy6Hyzm84J0QkTuUJ/EjNuPji3D0QJRALUTzu1UqqDCEtiN9OGyXEkh7uvb7BAgMBAAGjUDBOMB0GA1UdDgQWBBSVHPNrGWrjWyZvckQxFYWO59FRFjAfBgNVHSMEGDAWgBSVHPNrGWrjWyZvckQxFYWO59FRFjAMBgNVHRMEBTADAQH/MA0GCSqGSIb3DQEBCwUAA4GBAJeJ9SEckEwPhkXOm+IuqfbUS/RcziifBCTmVyE+Fa/j9pKSYTgiEBNdbJeBEa+gPMlQtbV7Y2dy8TKx/8axVBHiXC5geDML7caxOrAyHYBpnx690xJTh5OIORBBM/a/NvaR+P3CoVebr/NPRh9oRNxnntnqvqD7SW0U3ZPe3tJc\n-----END CERTIFICATE-----",
                "privateKey": "-----BEGIN RSA PRIVATE KEY-----\nProc-Type: 4,ENCRYPTED\nDEK-Info: AES-256-CBC,D8FFCE6B255601587CB54EC29B737D31\n\nkv4Fc3Jn0Ujkj0yRjt+gQQfBLSNF2aRLUENXnlr7Xpzqu0Ahr3jS1bAAnd8IWnsR\nyILqVmKsYF2DoHh0tWiEAQ7/y/fe5DTFhK7N4Wml6kp2yVMkP6KC4ssyYPw27kjK\nDBwBZ5O8Ioej08A5sgsLCmglbmtSPHJUn14pQnMTmLOpEtOsu6S+2ibPgSNpdg0b\nCAJNG/KHe+Vkx59qNDyDeKb7FZOlsX30+y67zUq9GQqJEDuysPJ2BUNP0IJXAjst\nFIt1qNoZew+5KDYs7u/lPxcMGTirUhgI84Jy4WcDvSOsP/tKlxj04TbIE3epmSKy\n+TihHkwY7ngIGtcm3Sfqk5jz2RXoj1/Ac3SW8kVTYaOUogBhn7zAq4Wju6Et4hQG\nRGapsJp1aCeZ/a4RCDTxspcKoMaRa97/URQb0hBRGx3DGUhzpmX9zl7JI2Xa5D3R\nmdBXtjLKYJTdIMdd27prBEKhMUpae2rz5Mw4J907wZeBq/wu+zp8LAnecfTe2nGY\nE32x1U7gSEdYOGqnwxsOexb1jKgCa67Nw9TmcMPV8zmH7R9qdvgxAbAtwBl1F9OS\nfcGaC7epf1AjJLtaX7krWmzgASHl28Ynh9lmGMdv+5QYMZvKG0LOg/n3m8uJ6sKy\nIzzvaJswwn0j5P5+czyoV5CvvdCfKnNb+3jUEN8I0PPwjBGKr4B1ojwhogTM248V\nHR69D6TxFVMfGpyJhCPkbGEGbpEpcffpgKuC/mEtMqyDQXJNaV5HO6HgAJ9F1P6v\n5ehHHTMRvzCCFiwndHdlMXUjqSNjww6me6dr6LiAPbejdzhL2vWx1YqebOcwQx3G\n-----END RSA PRIVATE KEY-----",
                "passphrase": {
                    "ciphertext": "ZjVmNQ==",
                    "protected": "eyJhbGciOiJkaXIiLCJlbmMiOiJub25lIn0"
                }
            }
        }
    }
}

Added to our internal product backlog as AUTOTOOL-1662.

Your example declaration is invalid:
{
    "code": 422,
    "errors": [
        "/Sample_01/A1/serviceMain: should have required property 'serverTLS'"
    ],
    "declarationFullId": "",
    "message": "declaration is invalid"
}

It wasn't the full declaration...just a snippet. Sorry

I am no longer able to reproduce this issue, so I believe it has been resolved. Can you confirm if the latest version of AS3 resolves this issue for you?

validated that the proper redirect is setup with correct mask instead of /32. Thx!


Show product information on the product select page.

On the product select page:
User should be able to search products like our other apps.
Results should be grouped by parent product
Products should be selectable

products do not need to be selectable, if everything else is handled, we can close.

Product search and grouping of products on the basis of parent product is working as expected.

Closing as done


Pull local copies of supported versions of Angular & Office UI Fabric?

Right now our demos are pulling CDN references for Angular & Office UI Fabric. While it works, it can also be problematic. For instance, the Office UI Fabric guys shipped 1.2 late last week, but it won't be in the CDN for another 1-2 weeks. As such @jjczopek had to self-host it and reference that link.
I propose we add two NPM dependencies on Angular 1.4.8 & Office UI Fabric 1.2.0 (both of which are live and update the demos to point to the local copies).
Thoughts? I'm in favor and can retrofit everything before our 0.1.0 release.
Picking NPM over bower to eliminate the extra step people would need to try the demos out.

Seeking input from @ngOfficeUIFabric/ngofficeuifabric-members

NM... exec decision to do this. I've got a seperate release-0.1.0 branch going with some minor updates to the repo, including this, that will be pushed out when I cut the release on JAN-13. Went ahead and updated all demos


Excellent work

Just had to drop in and say that you did a very nice job. this is one cool plugin!

Thanks a lot ❤


add source of ensure_repository command

Description of changes:

source public-ecr.sh to make ensure_repository command available

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

merging manually to unblock controller release. Please leave feedback and i will address it.


An input redirection issue.

Well, i write a shell script on REAL Ubuntu16.04,  It help me to git push something without inputing the Username and Password every time.
The question is, it works on Ubuntu, but it still need I to input Username and Password On Bash On Windows.
The structure of  it:
.
├── pushit
├── README.md
└── setup.sh
setup.sh:
#!/bin/bash

read -p "Username:" username
echo $username > username
read -p "Password:" -s password
echo password > password
echo

cd /etc
sudo rm -r pushit_Info

sudo mkdir pushit_Info
cd -
sudo mv username password /etc/pushit_Info

sudo cp pushit /usr/local/bin
echo "You can Use pushit command Now!"
echo "You can modify the Username And Password by run setup.sh again"

pushit
#!/bin/bash
git add .
echo "Input the Commit Message:"
read CommitMessage
git commit -m ${CommitMessage}
#echo "Input the branch(default is master):"
git push </etc/pushit_Info/username </etc/pushit_Info/password

And I think the problem happens in the process of redirection(I am not sure,I am a novice..but the problem is indeed exist).

git push </etc/pushit_Info/username </etc/pushit_Info/password

I am thinking you must have something unintentionally different on your Real Linux™ box that is causing it too "succeed". That construct isn't going to pass the username and password into stdin AFAICT. Simplifying what you are doing above:
ken@zesty-vm:~$ echo hello > hello
ken@zesty-vm:~$ echo world > world
ken@zesty-vm:~$ cat < ./hello < ./world
world

Note only 'world' is echoed. This behaves the same on WSL and Real Linux. Or maybe I am missing something obv and someone else will jump in.
I can't run your test as is because I've got my git passwords set up (and I'm not going to undo that). You probably should get them set up too. You can check out the git password cache documentation here. Better still, use ssh instead of https as your protocol, and set up ssh public keys per here. That link is geared toward github but it's a friendly explanation to start. More detailed info about git protocols here.
Bonne chance.

@Humbertzhang: Check the output of git remote -v on your linux box, my gut tells me that you have a username@ part in there, probably from an earlier attempt to solve this issue another way...


self documenting routes hopefully without all the networkx stufff

Could somebody check if this works at the /spec endpoint? I get 'web_1  | ModuleNotFoundError: No module named 'owlready2'' when I try to build.

@biotom I dont see Apispec included in the requirements.txt file

@biotom I dont see Apispec included in the requirements.txt file

yep. causing the docker build to fail for this branch at the minute

Yeah, apologies- I've fixed this, I'll push it when I'm not at work

@NickCallaghan @rodriguesk this is pretty simple so far- I wanted to check I was on the right track before trying to get the formatted comments working

Should not be requirements.txt in knowledge_graph

sorry, must have accidentally created that. thanks for catching!

I think this should be a lot nicer to use- the docs aren't quite as comprehensive, but to add a new endpoint you just use the @auto.doc decorator, and it presents a nice output with the docstring as a description

Nice @biotom! I think we at lease have a chance of keeping that up to date. Maybe combing that with a postman collection should give us a decent amount of documentations.
I've just done a push to remove the swagger stuff. doesn't look like we'll be using it.
As far as I know we don't use /ontology or /get_actions maybe we should remove these from the project altogether

@biotom Recon we should move the docs to the / route? Nothing there anyway, will be easier for folks to find.

@biotom Recon we should move the docs to the / route? Nothing there anyway, will be easier for folks to find.

Could do- could we redirect / to /documentation? It feels nicer to have it as a resource to me.


self signed certificate in certificate chain

Unable to withdraw due to error above.  Anyone seen this before?  Thanks  Just trying to withdrawal from wallet to an exchange.

Hello, I am not sure if this error could be from Arizen. Are you sure? Do you have any screenshot or additional information?

No reaction.


Send doesn't hide recaptcha

Send should be hiding recaptcha on submission.
If the page is left open, then the recaptcha expires.
See attached screen shot.

Just so you know, it's not a priority for the core team, but we would appreciate a contribution from the community for this.
Just a note, the behavior would need to be more complex:

Hide on submit
Re-show if failed submit
Keep hidden if submit successful
If multiple forms are shown on the page, this show/hide should only affect the current form

Since Mobius 5 uses the new, hidden recaptcha, this issue isn't relevant any more.


Refactor UnsavedController to use event.preventDefault() to trigger browser confirmation dialog

Fixes #12132, supersedes #12139.
Per https://developer.mozilla.org/en-US/docs/Web/API/Window/beforeunload_event

best practice is to trigger the dialog by invoking preventDefault() on the event object, while also setting returnValue to support legacy cases.

We don't need to support legacy cases, as our supported browsers all support the preventDefault() approach.
See also:
https://caniuse.com/mdn-api_window_beforeunload_event_preventdefault_activation

Asking @lb- for review in case you'd like to see this. No pressure at all if you don't get a chance to – I can ask Thibaud later 😄


Fix sizing of program builder so that long/short questions don't cause components to resize

Description
This also adjusts the content to take up the majority of the screen since this is a case where more screen real estate is more useful. See https://github.com/civiform/civiform/issues/2808#issuecomment-1234660045 for more details.
While here, I also moved a function out of AdminLayout that really was shared between two views and could have been better shared through inheritance.
Release notes:
Adapt the CiviForm admin program builder to not shrink or grow based on the length of the contained questions.
Checklist

[X] Added the correct label: < feature | bug | dependencies | infrastructure | ignore-for-release >
[ ] Created tests which fail without the change (if possible)
[ ] Extended the README / documentation, if necessary

Issue(s)
Fixes #2808

Tried the civiform developers queue, but it assigned to @kmcnellis, who is out. Reassigned to someone else manually.

Codecov Report

Merging #3331 (3ddd56d) into main (be55897) will increase coverage by 0.10%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main    #3331      +/-   ##
============================================
+ Coverage     79.89%   80.00%   +0.10%
- Complexity     2341     2343       +2
============================================
  Files           287      287
  Lines          9227     9242      +15
  Branches        616      617       +1
============================================
+ Hits           7372     7394      +22
+ Misses         1616     1608       -8
- Partials        239      240       +1

Impacted Files
Coverage Δ

server/app/models/ApplicationEvent.java
70.00% <0.00%> (ø)

...erver/app/services/applicant/ApplicantService.java
85.64% <0.00%> (ø)

...s/applications/ProgramAdminApplicationService.java
100.00% <0.00%> (ø)

.../controllers/admin/AdminApplicationController.java
46.22% <0.00%> (+5.54%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


router.navigate does not work in mozilla firefox

[ X ] Bug Report
[ ] Feature Request
[ ] General Query

My version of UI-Router is:
"@angular/router": "^5.0.0",
"@angular/core": "^5.0.0",
Bug Report
Current Behavior:
Here is my code:
Current link is /app/masterdashboard
this is component.ts method
public onSelectedObject(object: any): void {
StorageUtils.setObjectId(object.id);
this.selectedObject = this.currentObject();
this.router.navigate(['/app/dashboard']);
this.broadcaster.broadcast(events.ObjectIdChanged);
}
I'm in Object id = -1 where the name of dashboard is masterdashboard
When click on some Object id !== -1 from lists of Ids in DOM, for example ObjectId = 2, the link will be changed from '/app/masterdashboard' to '/app/dashboard' but this is not happen in firefox, the link stay '/app/masterdashboard'.
This work correctly with chrome.

I do have the same problem.
@DimitarDimitrov123 can you please describe how you solved it with settimeout?
Thanks!

myFunction() {
this.router.navigate(['/somewhere']);
setTimeout(function () {
// your code
}, 300);
}

This seems to work. Thank you!

I've had to route dynamically from a constructor. Somehow the component to route to never got initialized in FF, even though the code was reached and the promise got fully & successfully executed. The command to navigate was already placed at the end of the constructor; so there was no additional code to delay.
Setting a symbolic timeout around the routing did solve it for me:
setTimeout(() => {
    this.router.navigateByUrl("/route_to_component")
}, 1);

Dynamically re-routing is sometimes necessary e.g. when you can only determine the index component at runtime. Not the best framework-based routing I've dealt with so far.

@gekkedev thank you for the help. We tried to add some delay as you did but unfortunately it doesn't work, neither with ngzone. We have some third party sdks that we call at this time (google analytics and others), when we remove these sdk everything works fine without the async workaround. I think that is related to zone changes detection and it still an issue for us.
Any updates from your side ?

@houssemFat since this is almost a year ago so I don't quite remember the context and don't have access anymore to that code where it was used because it is corporate property. Your guess on a possible ngZone bug sounds like a step in the right direction. Browser-related problems can be really troublesome every once in a while.


Split AKTuningTable into extensions, move into Microtonality subdirectory

Sorry for the ugly pull request...accidentally included some UI files and it took some reverts to redo correctly.
Note that this will break the playground because I renamed the indian scales.  I need some help setting up the playgrounds so I can easily test while modifying AudioKit (so I don't have to build_frameworks all the time).

This looks good.  I think I'll accept the pull request and then make Travis-CI happy.  Need to add three slash comments eventually too.


Update Logback Dependency

Fix for https://nvd.nist.gov/vuln/detail/CVE-2017-5929
See #27

Codecov Report

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

@@            Coverage Diff            @@
##             master      #33   +/-   ##
=========================================
  Coverage     78.64%   78.64%
  Complexity       74       74
=========================================
  Files            16       16
  Lines           281      281
  Branches         43       43
=========================================
  Hits            221      221
  Misses           39       39
  Partials         21       21

Continue to review full report at Codecov.

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


Production environment prompting for a password to access the website even after setting it as "publicly visible" in the Amplify console

Before opening, please confirm:

[X] I have checked to see if my question is addressed in the FAQ.
[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.

App Id
dvqke3jj21la8
AWS Region
us-east-1
Amplify Hosting feature
Access control, Deployments
Describe the bug
I'm facing an issue with my production environment where, even after setting it as "publicly visible" in the Amplify console, my main branch is still prompting for a password to access the website. I need help to solve this problem as I have tried several solutions, including multiple redeployments of the application, but I have not been able to get rid of the password prompt.
I would like to point out that I did not set any password for my main branch, so I don't understand why it is being prompted. I believe there might be some configuration or compatibility issue with my application.
I would be very grateful if you could help me solve this problem as soon as possible, as my application is facing availability issues due to the password prompt.
Expected behavior
The correct behavior is not to request username and password to access the production environment, as it must be publicly visible
Reproduction steps

Go into the Amplify App
Goto App settings -> Access Control
Hit the Manage access button
Uncheck the option to restrict access
Hit Save

Build Settings
version: 1
frontend:
  phases:
    preBuild:
      commands:
        - nvm install 16
        - nvm use 16
        - yarn install --silent --frozen-lockfile
    build:
      commands:
        - yarn build
  artifacts:
    baseDirectory: .next
    files:
      - '**/*'
  cache:
    # paths:
    #   - node_modules/**/*

Log output
These are my project's current settings:
{
  "app": {
      "appId": "dvqke3jj21la8",
      ...
      "platform": "WEB_COMPUTE",
      "environmentVariables": {
          ...
          "_LIVE_UPDATES": "[{\"pkg\":\"@aws-amplify/cli\",\"type\":\"npm\",\"version\":\"latest\"},{\"pkg\":\"yarn\",\"type\":\"npm\",\"version\":\"latest\"},{\"pkg\":\"next-version\",\"type\":\"internal\",\"version\":\"latest\"}]"
      },
      ...
      "enableBranchAutoBuild": false,
      "enableBranchAutoDeletion": true,
      "enableBasicAuth": false,
      "basicAuthCredentials": "...",
      ...
      "enableAutoBranchCreation": false,
      "autoBranchCreationPatterns": [
          "feature/*"
      ],
      "autoBranchCreationConfig": {
          "stage": "NONE",
          "enableAutoBuild": false,
          "environmentVariables": {
              ...
          },
          "basicAuthCredentials": "...",
          "enableBasicAuth": false,
          "enablePullRequestPreview": false
      },
      "repositoryCloneMethod": "SSH"
  }
}

### Additional information

_No response_

Hi @MarcusVRdoN 👋🏽 thanks for raising this issue. It looks like your website is accessible and doesn't prompt for Access Control settings for any of your branches. Your project settings shows the access control setting as being disabled as well. Can you please provide a screenshot of what you're seeing on your end?


Board Management for User

Description
I think it would be useful if I can manage the boards for regular users.
For example, you can create an administration board and then make it available to the admin accounts.
Normal users can get a normal board, which they cannot change and therefore cannot gain access to the administration board.
And in the settings where you can edit the user, I think it would be cool if there was a switch that you could use to say whether the user can change their board or not
Priority
Medium (Would be very useful)

Duplicate of #1668


Use of TinyCoreLinux 6.0 and x64 userspace for boot2docker.iso

This pull request aims at proposing to use TinyCoreLinux 6.0 in boot2docker.iso
What is supported:

The VM can be initiated and launched via boot2docker-cli.
Containers can be loaded and used properly
SSH using boot2docker is working properly

What is not yet there:

ACPID and NTPD are not yet there, as 64bit flavours of the tcz extensions are not yet available. As an alternative to stop the boot2docker vm, "boot2docker poweroff" can be used.
Shared folder using Virtualbox is not functional. vboxsf seems to compile properly, yet /sbin/mountfs.vboxsf can't be used.

@afressancourt here's my working 64bit branch, only contains changes to VMware Tools but it's rebased to the top of the b2d master branch https://github.com/frapposelli/boot2docker/tree/vmware-64bit
I've opened an issue #749 to open a branch here for 6.x development so we can send PRs against it.

So, ntpclient isn't there, but ntpd is, right?  We don't use ntpclient anymore (#661), so it could be removed entirely.
Do you guys think we'll be able to reasonably switch to 6.x before the next release? (ie, acpid seems like the only thing we're really missing -- can we fix that problem in time, or is that outside our control?)

I am struggling with acpid, with no success for now. Beyond that, the vbossf part is not functional, for a reason I ignore.

@afressancourt we'll talk about that these days at work, but i think i'll manage to help on the vboxsf part, as i've managed it in the past with my dduportal/boot2docker vagrant box.

Are we making any progress with this PR? It would be nice to move to 64 bits for Docker 1.6.0 so I can add shared folders support for VMware.

@frapposelli : sorry, i forgot to comment here, i think i'm not skilled enough for recompiling vbox (it seems related to dbus).
Anyway, the PR does not seems to be in-sync with master : we shoudl give another try (for me, around this friday)

Hello,
On the TCL side, I have contacted the community and they compiled missing extensions in order that acpid can work properly. I will synchronize with @dduportal on Friday to put the PR in sync and work on VBox compilation.

Awesome @dduportal @afressancourt I will send a PR against your repo once you have rebased.
cc: @tianon @SvenDowideit

this would be a 1.7 thing :)

Sorry for the lack of progress, I am fine if we can make it a 1.7 thing

Sorry also on my side, didn't have enough time due to personnal stuff :(
But i think we'll need help around the vbox compilation on TCL x64...

I've rebased this in #854


⚠ Bofrost.at has degraded performance

In d5ca11c, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1352 ms

Resolved: Bofrost.at performance has improved in d7c2e0f.


Uncaught TypeError in repomap.py line 200

Aider version: 0.62.2.dev2+g49fc1b40
Python version: 3.11.4
Platform: Windows-10-10.0.19045-SP0
Python implementation: CPython
Virtual environment: No
OS: Windows 10 (64bit)
Git version: git version 2.45.2.windows.1
An uncaught exception occurred:
Traceback (most recent call last):
  File "main.py", line 827, in main
    coder.run()
  File "base_coder.py", line 738, in run
    self.run_one(user_message, preproc)
  File "base_coder.py", line 775, in run_one
    message = self.preproc_user_input(user_message)
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "base_coder.py", line 764, in preproc_user_input
    return self.commands.run(inp)
           ^^^^^^^^^^^^^^^^^^^^^^
  File "commands.py", line 236, in run
    return self.do_run(command, rest_inp)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "commands.py", line 208, in do_run
    return cmd_method(args)
           ^^^^^^^^^^^^^^^^
  File "commands.py", line 123, in cmd_chat_mode
    raise SwitchCoder(
aider.commands.SwitchCoder

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "repomap.py", line 213, in load_tags_cache
    self.TAGS_CACHE = Cache(path)
                      ^^^^^^^^^^^
  File "core.py", line 499, in __init__
    sql(query, (key, value))
  File "core.py", line 666, in _execute_with_retry
    return sql(statement, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
sqlite3.OperationalError: disk I/O error

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "repomap.py", line 185, in tags_cache_error
    shutil.rmtree(path)
  File "shutil.py", line 759, in rmtree
    return _rmtree_unsafe(path, onerror)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "shutil.py", line 622, in _rmtree_unsafe
    onerror(os.unlink, fullname, sys.exc_info())
  File "shutil.py", line 620, in _rmtree_unsafe
    os.unlink(fullname)
PermissionError: [WinError 32] The process cannot access the file because it is being used by another process: '\\\\<IP_ADDRESS>\\YonaVmDataShare\\Projects\\posexport\\.aider.tags.cache.v3\\cache.db'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "__main__.py", line 7, in <module>
    sys.exit(main())
             ^^^^^^
  File "main.py", line 835, in main
    coder = Coder.create(**kwargs)
            ^^^^^^^^^^^^^^^^^^^^^^
  File "base_coder.py", line 155, in create
    res = coder(main_model, io, **kwargs)
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "base_coder.py", line 401, in __init__
    self.repo_map = RepoMap(
                    ^^^^^^^^
  File "repomap.py", line 60, in __init__
    self.load_tags_cache()
  File "repomap.py", line 215, in load_tags_cache
    self.tags_cache_error(e)
  File "repomap.py", line 200, in tags_cache_error
    except (SQLITE_ERRORS, OSError) as e:
TypeError: catching classes that do not inherit from BaseException is not allowed

Thanks for trying aider and filing this issue.
This looks like a duplicate of #2279. Please see the comments there for more information, and feel free to continue the discussion within that issue.
I'm going to close this issue for now. But please let me know if you think this is actually a distinct issue and I will reopen this issue.


Update Cosmos (testing with snapshot build)

High Level Description
What features does this change enable? What bugs does this change fix? High-Level description of how things changed.
Related Issues

DCOS_OSS- Foo the Bar so it stops Bazzing.
DCOS- Foo the Bar so it stops Bazzing.

Checklist for all PR's

[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[ ] Read the DC/OS contributing guidelines
[ ] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[ ] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[ ] Test Results: [link to CI job test results for component]
[ ] Code Coverage (if available): [link to code coverage report]

PLEASE FILL IN THE TEMPLATE ABOVE / DO NOT REMOVE ANY SECTIONS ABOVE THIS LINE
Instructions and Review Process
What is the review process and when will my changes land?
All PRs require 2 approvals using github's pull request reviews.
Reviewers should be:

Developers who understand the code being modified.
Developers responsible for code that interacts with or depends on the code being modified.

It is best to proactively ask for 2 reviews by @mentioning the candidate reviewers in the PR comments area. The responsibility is on the developer submitting the PR to follow-up with reviewers and make sure a PR is reviewed in a timely manner. Once a PR has 2 ship-it's, no red reviews, and all tests are green it will be included in the next train.

Do not merge

@mesosphere-mergebot label Work In Progress

@mesosphere-mergebot bump-ee

@mesosphere-mergebot bump-ee

@mesosphere-mergebot bump-ee

@mesosphere-mergebot label Ready For Review

@jsancio @cruhland please review

@lingmann please review


New job for Cloud7-8 disruptive upgrade

If no database migration is needed (mariadb already in SOC7),
normal upgrade path should work without significant changes...

I don't know yet; I've tested manually and maybe forgot to transform some manual changes into PR's...


Native Typeface Error.

Hi, I'm trying to use this library on my smartphone running Android 4.0 but I'm having this error:
06-05 15:53:54.036: E/AndroidRuntime(19147): FATAL EXCEPTION: main
06-05 15:53:54.036: E/AndroidRuntime(19147): java.lang.RuntimeException: native typeface cannot be made
06-05 15:53:54.036: E/AndroidRuntime(19147): 	at android.graphics.Typeface.<init>(Typeface.java:175)
06-05 15:53:54.036: E/AndroidRuntime(19147): 	at android.graphics.Typeface.createFromAsset(Typeface.java:149)
06-05 15:53:54.036: E/AndroidRuntime(19147): 	at com.wdullaer.materialdatetimepicker.TypefaceHelper.get(TypefaceHelper.java:31)
06-05 15:53:54.036: E/AndroidRuntime(19147): 	at com.wdullaer.materialdatetimepicker.date.DatePickerDialog.onCreateView(DatePickerDialog.java:262)
06-05 15:53:54.036: E/AndroidRuntime(19147): 	at android.app.Fragment.performCreateView(Fragment.java:1699)
06-05 15:53:54.036: E/AndroidRuntime(19147): 	at android.app.FragmentManagerImpl.moveToState(FragmentManager.java:885)
06-05 15:53:54.036: E/AndroidRuntime(19147): 	at android.app.FragmentManagerImpl.moveToState(FragmentManager.java:1057)
06-05 15:53:54.036: E/AndroidRuntime(19147): 	at android.app.BackStackRecord.run(BackStackRecord.java:694)
06-05 15:53:54.036: E/AndroidRuntime(19147): 	at android.app.FragmentManagerImpl.execPendingActions(FragmentManager.java:1435)
06-05 15:53:54.036: E/AndroidRuntime(19147): 	at android.app.FragmentManagerImpl$1.run(FragmentManager.java:441)
06-05 15:53:54.036: E/AndroidRuntime(19147): 	at android.os.Handler.handleCallback(Handler.java:800)
06-05 15:53:54.036: E/AndroidRuntime(19147): 	at android.os.Handler.dispatchMessage(Handler.java:100)
06-05 15:53:54.036: E/AndroidRuntime(19147): 	at android.os.Looper.loop(Looper.java:194)
06-05 15:53:54.036: E/AndroidRuntime(19147): 	at android.app.ActivityThread.main(ActivityThread.java:5391)
06-05 15:53:54.036: E/AndroidRuntime(19147): 	at java.lang.reflect.Method.invokeNative(Native Method)
06-05 15:53:54.036: E/AndroidRuntime(19147): 	at java.lang.reflect.Method.invoke(Method.java:525)
06-05 15:53:54.036: E/AndroidRuntime(19147): 	at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:833)
06-05 15:53:54.036: E/AndroidRuntime(19147): 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:600)
06-05 15:53:54.036: E/AndroidRuntime(19147): 	at dalvik.system.NativeStart.main(Native Method)

It's related to the Typeface loading as I can read.
Is somebody having this same issue?
How did you solve it?

Ok, I just solved it by modifying the TypefaceHelper.java file.
Changed the SimpleArrayMap structure by a Hastable structure.
Here is the new TypefaceHelper.java:
public class TypefaceHelper{
	private static final Hashtable<String, Typeface> cache = new Hashtable<String, Typeface>();

	public static Typeface get(Context c, String assetPath){
		synchronized (cache){
			if(!cache.containsKey(assetPath)){
				try{
					Typeface t = Typeface.createFromAsset(c.getAssets(), String.format("fonts/%.ttf", assetPath));
					cache.put(assetPath, t);
				} catch(Exception e){
					return null;
				}
			}
			return cache.get(assetPath);
		}
	}

}

SimpleArrayMap is a more lightweight drop in replacement for HashMap.
It's part of the v4 support library, so it shouldn't be causing any issues on Android 4.0
Were you using the version from Maven central or did you build it yourself?
In case you've build it yourself, a clean build might help to resolve all the dependencies properly.
Also the exception you're showing is related to creating the typeface, rather than an operation on the SimpleArrayMap.
Rather than changing the container, I think the rebuild that it triggered fixed your issue.
Can you roll back your change and verify this?

Hi, yes I can take a look to it in a couple days.
I'll post the result of changing Hashtable to SimpleArrayMap again :-).

Hi,
I changed the structure from Hashtable to SimpleArrayMap and it didn't crashed but when I changed my modified code for yours it crashed again with the same error. So, I just re-wrote the one you code and it worked as you said (kind of weird if you ask me).
Thanks!

Modern IDEs aggressively cache build results.
In any case, I'm happy it's resolved :-)


Feature/marathon constraint model improvements

Adding tests to MarathonConstraint methods
Adding MarathonConstraint.from_string classmethod

Can you address the lint failures?

Fixed

Is there something else you want me to do?

Nope! Thank you.


Add functions supporting the routing shapefile

This PR does the following:

Places the imported libraries for several functions at the top of gis.py for visibility.
Updates requirements in setup.py and environment.yml (synchronizing and adding missing libraries).
Modifies several docstrings to be a bit clearer in gis.py
Refactors existing HyrdoBASINS function to create  get_location_wfs and get_feature_attributes_wfs and creates two functions using them to be used by routing-shapefile WPS processes in RAVEN.
Renames a test class to follow CamelCase convention.

This is all in satisfying parts of https://github.com/Ouranosinc/raven/issues/244.

@huard Thanks for the comments. Please feel free to keep them coming. One thing I'd love to do is organize the functions in gis.py and utils.py. GIS-related functions are peppered in both, but fixing them will need to be done alongside raven processes at the same time. Once we have this PR done, I can move on to performing this in another PR with RAVEN changes synonymously.

@huard Thanks for the comments. Please feel free to keep them coming. One thing I'd love to do is organize the functions in gis.py and utils.py. GIS-related functions are peppered in both, but fixing them will need to be done alongside raven processes at the same time. Once we have this PR done, I can move on to performing this in another PR with RAVEN changes synonymously.

@huard I'm adding the final tests for gis.py tomorrow where it will be good to be merged. Many of the functions in utils.py have been fixed in https://github.com/Ouranosinc/raven/pull/335, and I believe it makes sense to port those to RavenPy once both these PRs are merged. With some work, we can probably hammer all this out by next week.

@huard @cjauvin This is good to be merged. You'll notice a lot of FIXME and TODO notes; These will be dealt with when both this PR and https://github.com/Ouranosinc/raven/pull/335 are merged. I didn't want to touch code that has been touched in raven, hence why there's a lot of them. Thanks for your patience.

I will have a deeper look later but does it fix the problems we had with RTD?

No, the docs are still broken. Given that the changes here touch the requirements, causing Sphinx to complain, I can start adding those changes in here.

For some reason, mock doesn't seem to register. I'm getting errors for geospatial libraries not being found despite listing them to be mocked. @tlvu do you have any advice?

Figured out my issues (dataclasses is not in the Python3.6 builtins, I was forgetting to mock osgeo).
I have a theory as to why the doc builds are giving me 849 warnings. I'm seeing some docstrings that start with a :, such as:
class SubBasinsCommand(NamedTuple):
    """:SubBasins command (RVH)."""

I think this is causing sphinx to freak out. Additionally, classes that are both published in user_api.rst and in the auto-generated ravenpy.models.rst are causing TONS of section header warnings. That type of fix might need another PR as that requires restructuring the docs layout, I imagine.

It looks as though the last hiccup I have is that the values expected for certain GIS operations are dependent on versions of GDAL. Very frustrating.


Add formula for Dasht

All Submissions:

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

New Formulae Submissions:

[x] Does your submission pass
brew audit --strict --online <formula> (where <formula> is the name of the formula you're submitting)?
[x] Have you built your formula locally prior to submission with brew install <formula>?

@BrewTestBot test this please

Test is failing. Another missing dependency?
/usr/local/Cellar/dasht/2.0.0/bin/dasht: line 72: w3m: command not found

Full failure:
failed: brew test --verbose dasht

Stacktrace

        Testing dasht
==> Using the sandbox
/usr/bin/sandbox-exec -f /tmp/homebrew20160404-498-7pc2ce.sb /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/bin/ruby -W0 -I /usr/local/Library/Homebrew -- /usr/local/Library/Homebrew/test.rb /usr/local/Library/Taps/homebrew/homebrew-core/Formula/dasht.rb --verbose
==> /usr/local/Cellar/dasht/2.0.0/bin/dasht
/usr/local/Cellar/dasht/2.0.0/bin/dasht: line 72: w3m: command not found
ls: /private/tmp/dasht20160404-499-lqwaki/.local/share/dasht/docsets: No such file or directory
dasht: no results found
/usr/local/Cellar/dasht/2.0.0/bin/dasht: line 69: kill: (-508) - No such process
Error: dasht: failed
Failed executing: /usr/local/Cellar/dasht/2.0.0/bin/dasht
/usr/local/Library/Homebrew/formula.rb:1481:in `block in system'
/usr/local/Library/Homebrew/formula.rb:1418:in `open'
/usr/local/Library/Homebrew/formula.rb:1418:in `system'
/usr/local/Library/Taps/homebrew/homebrew-core/Formula/dasht.rb:18:in `block in <class:Dasht>'
/usr/local/Library/Homebrew/formula.rb:1327:in `block in run_test'
/usr/local/Library/Homebrew/extend/fileutils.rb:37:in `mktemp'
/usr/local/Library/Homebrew/formula.rb:1323:in `run_test'
/usr/local/Library/Homebrew/test.rb:28:in `block in <main>'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/timeout.rb:66:in `timeout'
/usr/local/Library/Homebrew/test.rb:27:in `<main>'

@BrewTestBot test this please

Merged in https://github.com/Homebrew/homebrew-core/commit/590cc64bd41d628b4cd9afae001432db42b8507b. Thank you for your contribution to Homebrew @rjcoelho; we appreciate it! :smiley_cat:


[Bug] The org page of CASP in OC redirects to 404

Is there an existing issue for this?

[X] I have searched the existing issues

What product(s) are you seeing the problem on?
OpenChallenges
Current behavior
The org search page does not show the org nor does the org page works. I got the org login form OC Data sheet.
https://openchallenges.io/org/casp
Expected behavior
No response
Anything else?
No response
Commit ID
No response
Are you developing inside the dev container?

[X] I am using the dev container

Code of Conduct

[X] I agree to follow this project's Code of Conduct

CASP org page is listed when deploying from main. The issue may have been fixed since last time we deployed to production.


[Docs] Refactor Getting Started Intermediate

Tasks:

[ ] Update https://docs.getindexify.ai/getting_started_intermediate/
[ ] Be more verbose
[ ] Make sure input and output is shown
[ ] Add diagrams
[ ] Explain core extraction in detail

assigned: @ritwikraha

Completed by: https://github.com/tensorlakeai/indexify/pull/737


Saving/restoring network state?

This question is going to reveal my Torch ignorance, but I'm hoping there's a simple/quick solution.
I've implemented a simple beam search for torch-rnn but, because I don't know how to "save" and return to a previous network state, for each "branch" (or "beam" I guess?) I'm forced to reset the whole thing, then warm it up with my current working sequence plus a candidate character, which seems… not ideal. It currently looks something like this (code simplified to get the gist across):
local T0 = working_sequence:size(2) -- length of working sequence
self:resetStates()
local scores = self:forward(working_sequence)[{{}, {T0, T0}}]
-- now that the net is where i want it to be...
self:forward(one_char)
-- do stuff

self:resetStates() -- :(
scores = self:forward(working_sequence)[{{}, {T0, T0}}]
-- now that the net is where i want it to be AGAIN...
self:forward(a_different_char)
-- do different stuff

What I'd like to do inside LanguageModel.lua (but haven't been able to figure out, either by looking at the torch-rnn code or Torch documentation elsewhere) is something like this:
local working_state = self:saveState()

self:forward(one_char)
-- do stuff

self:setState(working_state)
self:forward(a_different_char)
-- do different stuff

self:setState(working_state)
self:forward(the_winning_char)
-- proceed as normal

Is anything like that possible?
Thanks for your consideration!

There's no clean thing built in to either Torch or torch-rnn that would let you do that, but I agree that it would be a very good addition especially for beam search.
Right now you can reset states by calling the resetStates method of the LanguageModel, which in turn calls the resetStates method of the underlying LSTM or VanillaRNN layers. A simple extension would be to implement a stack for saved states by adding pushState and popState methods to LSTM, VanillaRNN, and LanguageModel. That should be relatively easy to implement, and I think it would be sufficiently powerful to let you cleanly implement beam search.

Thanks for the reply! I suspect the state stack implementation might be a bit beyond my skill level but if I was going to try it… in LSTM.lua I see the reset function:
function layer:resetStates()
  self.h0 = self.h0.new()
  self.c0 = self.c0.new()
end

Is stashing and retrieving copies of self.h0 and self.c0 the core of what I want to do? Or is it subtler than that?
Thanks again.

I think it should be as simple as stashing self.h0 and self.c0 and setting them to new tensors on push, and setting them to the last stashed version on pop.


Change iOS Snackbar/Toast default background color

Description of Change
Change the default background color for iOS Toast and Snackbar
Bugs Fixed

Fixes #

Behavioral Changes
Dark theme

Light theme

PR Checklist

[ ] Has a linked Issue, and the Issue has been approved
[ ] Has tests (if omitted, state reason in description)
[x] Has samples (if omitted, state reason in description)
[x] Rebased on top of main at time of PR
[x] Changes adhere to coding standard

[ ] Updated documentation

Hello @radical Firstly thank you for you effort but same problem still continues for v1.2.0 and v1.3.0-pre2 on iOS

 var options = new SnackBarOptions
                {

                    MessageOptions = new MessageOptions
                    {
                        Foreground = textColor,
                        Font = Font.OfSize("RobotoRegular", 14),
                        Padding = new Thickness(4),
                        Message = message

                    },
                    CornerRadius = new Thickness(0, 0, 0, 0),
                    Duration = TimeSpan.FromMilliseconds(3500),
                    BackgroundColor = bgColor,

                    Actions = new List<SnackBarActionOptions>
                    {
                        new SnackBarActionOptions
                        {
                            ForegroundColor = textColor,
                            BackgroundColor = Color.Transparent,
                            Font = Font.OfSize("FontAwesomeSolid", 14),
                            Text = FontAwesome.FontAwesomeIcons.Times,
                            Action = () =>
                            {
                                return Task.CompletedTask;
                            }
                        }
                    }

                };

Hello again also CornerRadius does not work on IOS

var options = new ToastOptions
                {

                    MessageOptions = new MessageOptions
                    {
                        Foreground = textColor,
                        Font = Font.OfSize("RobotoRegular", 14),
                        Padding = new Thickness(16, 8),
                        Message = message,
                    },
                    CornerRadius = new Thickness(50,50,50,50),
                    Duration = TimeSpan.FromMilliseconds(3500),
                    BackgroundColor = bgColor
                };

as for the 1st code. you set background color Transparent.
as for the 2nd code, will need to check

as for the 1st code. you set background color Transparent.
as for the 2nd code, will need to check

Okay first options as fallowing and result as below

  var options = new SnackBarOptions
                {

                    MessageOptions = new MessageOptions
                    {
                        Foreground = textColor,
                        Font = Font.OfSize("RobotoRegular", 14),
                        Padding = new Thickness(4),
                        Message = message

                    },
                    CornerRadius = new Thickness(0, 0, 0, 0),
                    Duration = TimeSpan.FromMilliseconds(3500),
                    BackgroundColor = bgColor,

                    Actions = new List<SnackBarActionOptions>
                    {
                        new SnackBarActionOptions
                        {
                            ForegroundColor = textColor,
                            Font = Font.OfSize("FontAwesomeSolid", 14),
                            Text = FontAwesome.FontAwesomeIcons.Times,
                            Action = () =>
                            {
                                return Task.CompletedTask;
                            }
                        }
                    }

                };

Second version of option result as below

var options = new SnackBarOptions
              {

                  MessageOptions = new MessageOptions
                  {
                      Foreground = textColor,
                      Font = Font.OfSize("RobotoRegular", 14),
                      Padding = new Thickness(4),
                      Message = message

                  },
                  CornerRadius = new Thickness(0, 0, 0, 0),
                  Duration = TimeSpan.FromMilliseconds(3500),
                  BackgroundColor = bgColor,

                  Actions = new List<SnackBarActionOptions>
                  {
                      new SnackBarActionOptions
                      {
                          ForegroundColor = textColor,
                          BackgroundColor = bgColor,
                          Font = Font.OfSize("FontAwesomeSolid", 14),
                          Text = FontAwesome.FontAwesomeIcons.Times,
                          Action = () =>
                          {
                              return Task.CompletedTask;
                          }
                      }
                  }

              };

ok. will take a look


[Add] Responsiveness for all Screens

Details
This website is awesome but it is not responsive to mobile screens. I check this at 360px on a mobile device but it looks like the "screenshot I shared below"
Screenshots

Type of Contribution

[ ] Update to an existing Animated Button
[ ] Adding a new Button
[X] Resolving a bug
[ ] Proposal to the Repository
[ ] Changes related to documentation or README.md
[ ] Other Changes

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue
[X] I am a GSSoC'23 contributor

Assign me. I can do it.

@avinasdube Actually I want to work on this bro
@Spyware007 can you assign it to me under GSSoc'23 i want to work on this

@avinasdube Actually I want to work on this bro @Spyware007 can you assign it to me under GSSoc'23 i want to work on this

okay. i misunderstood you as maintainer/owner of this repo.


fix: create a minimal config for sampleRUM

Fix #628

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

GitHub release
v6.38.3

Your semantic-release bot :package::rocket:


Meta tag not included when using index.ejs template

Description
Hello,
I am using HtmlWebpackPlugin to define an index.ejs file as template. I am also passing some templateParameters through the HtmlWebpackPlugin (below given random names for security but you get the picture). If I leave the csp meta tag in the index.ejs, it doesn't get overwritten by your csp plugin, and if I remove it completely then it's not even created. I must add that I am using webpack-dev-server with hot reloading to develop and test it locally. I am giving you an excerpt of both plugin configurations up till now, could you maybe shed some light as to how to use them properly?
// ... rest of code

const cspOptions = {
      'script-src': ['"self"', '"unsafe-eval"', '"resource:"'],
      'style-src': ['"self"', '"unsafe-inline"'],
      'connect-src': ['"blob:"', '"http://localthost:*"', '"ws://localhost:*"'],
      'font-src': '"self"',
      'object-src': '"blob:"',
      'img-src': ['"data:"', '"self"'],
      'media-src': '"self"',
      'form-action': '"self"',
      'child-src': ['"blob:"'],
      'worker-src': ['"blob:"', '"resource:"'],
      'frame-src': ['"blob:"', '"self"'],
      'base-uri': '"resource"',
      'block-all-mixed-content': true // Is this even working like that?
};
new HtmlWebpackPlugin({
       cspPlugin: {
           enabled: true,
           policy: cspOptions,
        }
        inject: false,
        template: './public/index.ejs',
        templateParameters: {
            platform: 'browser',
            mode: 'development',
            pathPrefix: './',
            backend: 'http://localhost:3001',
            metaDescription: 'Testing CSP',
            title: 'Test App',
        },
        minify: false
      }),
new CspHtmlWebpackPlugin(cspOptions)
// ... rest of code

What type of issue is this? (place an x in one of the [ ])

[ ] bug
[ ] enhancement (feature request)
[x] question
[ ] documentation related
[ ] testing related
[ ] discussion

Requirements (place an x in each of the [ ])

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

Bug Report
Filling out the following details about bugs will help us solve your issue sooner.
Reproducible in:
slackhq/csp-html-webpack-plugin version: v3.0.4
node version: v8.11.1
OS version(s): Windows 10
Expected result:
Have an overwritten csp meta tag within the built index.html or if it wasn't existing have a new csp meta tag created with this plugin's configuration
Actual result:
No meta tag is overwritten or created based on this plugin's configuration

If you're able to provide a sample repo, please open a new issue, and I can look into this
Thanks!


Windows Defender says there's a trojan associated with MicMute

It's just a false positive, it almost always detects this with autohotkey executables
I will report this to microsoft though, they usually remove these in a couple of days

now I understand why this issue stays here, it did indeed disappear once microsoft resolved this but it will probably reappear with every release.
This message to clarify for others : if you pick a very fresh release that might happen.
(And good luck finding the proper buttons in windows defender to whitelist or allow it anyway)

I submit new releases to Microsoft anyway, and they have already removed the detection for 1.2.4


some config uses 1.xxxx others user xxxx.1

From an email thread on openssl-users
Argh.  Thanks for the detailed explanation.
It seems to me that if I add suffixes ".nnn" support to auto_info() than we can move xxx.field to deprecated at some point?
On 3/20/20, 10:45 PM, "Richard Levitte"<EMAIL_ADDRESS>wrote:
The correct answer is, it depends.  This is an unfortunate
evolutionary artefact, and is governed by very different pieces of
code.

The config.pod example revolves around subject names and the config
for 'openssl req'.  The code that uses this is the function
auto_info(), found in apps/req.c.

The x509v3_config.pod example revolves around X.509 v3 extensions, and
the config for those is used by diverse functions in crypto/x509v3/
(1.1.1) or crypto/x509/ (masterand upcoming 3.0), and ultimately, the
key name comparison is done by name_cmp(), found in v3_utl.c.

So both manuals are correct.  Unfortunately...

Cheers,
Richard

On Fri, 20 Mar 2020 22:12:08 +0100,
Salz, Rich via openssl-users wrote:
>
>
> The doc/man5/config.pod file says to use
>
>                 1.OU = “My first OU”
>
>                 2.OU = “My second OU”
>
> But doc/man5/x509v3_config.pod says to append the numeric, as in
>
>                 email.1 = steve@here
>
>                 email.2 = steve@there
>
> I believe the second form is correct.  Can anyone confirm?
>
>

Marking as inactive, to be closed at the end of 3.4  dev, barring further input

Not going to rile up the installed base with a gratuitious change.


add hipchat_room_id to service

goes along with https://github.com/Affirm/cabot-alert-hipchat/pull/1

just checking if we are planning this for tomorrow

@mahendra not planning this one for tomorrow

@mahendra made the changes we discussed (add HipchatInstance model and select one in service also). Tested successfully on stage :)


Image request randomly fails nightwatch test

Description of Issue
Image http request in here randomly cause nightwatch test failure. It doesn't get expected image in 3000 ms.

[ ] Plan to store images in our source repo to fix this.

For this issue, we will also need to add a loader for images: https://webpack.js.org/guides/asset-management/#loading-images

For this issue to be resolved, we should be able to turn wifi off on our computer and the tests should all still pass.

When running nightwatch tests offline, I receive this errror for every package:
 FAIL  packages/terra-list/tests/jest/List.test.jsx
  ● Test suite failed to run

    proxies not supported on this platform. On v8/node/iojs, make sure to pass the --harmony_proxies flag

      at new global.Proxy (node_modules/harmony-reflect/reflect.js:2158:13)
      at Object.<anonymous> (node_modules/identity-obj-proxy/src/index.js:17:9)
      at Object.<anonymous> (packages/terra-list/node_modules/terra-base/lib/baseStyles.js:3:1)
      at Object.<anonymous> (packages/terra-list/src/List.jsx:4:1)
      at Object.<anonymous> (packages/terra-list/tests/jest/List.test.jsx:2:13)
      at handle (node_modules/worker-farm/lib/child/index.js:41:8)
      at process.<anonymous> (node_modules/worker-farm/lib/child/index.js:47:3)
      at emitTwo (events.js:87:13)
      at process.emit (events.js:172:7)
      at handleMessage (internal/child_process.js:686:10)
      at Pipe.channel.onread (internal/child_process.js:440:11)

PR: https://github.com/cerner/terra-core/pull/359

This issue should be resolved in #359.


Pagination?

Can you please show in the README how the pagination can be done? For example, api.get_books_list() returns 100 elements in "data" bu also tells me there are 174 elements. I see you let me pass "id" as parameter, but how about pagination parameters?

For the record: count, offset etc need to be passed as query params in the API: https://demo.bookstackapp.com/api/docs#listing-endpoints

I installed the bleeding edge like this: pip install https://github.com/coffeepenbit/bookstack/archive/refs/heads/master.zip
And I called it like this:  all_books = bookstack_api.get_books_list(params={"count": 500})
This worked.

Can we get a new oficial release with this params support enabled? Right now I have to:
pip install https://github.com/coffeepenbit/bookstack/archive/refs/heads/master.zip


How to load pretrained model?

I want to finetune t2v model on my own dataset.
However, when I used the pretained model '65x512x512/diffusion_pytorch_model.safetensors' as the pretrained model, the error '_pickle.UnpicklingError: invalid load key, '\xbb'.' occured.
Any suggestions?

Sorry, we fixed the training script. In fact each stage is a fine-tuning of the previous stage, and you can observe the difference between stage 2 scripts and stage 1 to customize your fine-tuning scripts.
See https://github.com/PKU-YuanGroup/Open-Sora-Plan/blob/main/scripts/text_condition/train_videoae_65x512x512.sh#L31

In face, I can not get the model you mentioned here, so I use this pretrained model to finetune(just like --pretrained='LanguageBind/Open-Sora-Plan-v1.0.0/65x512x512/diffusion_pytorch_model.safetensors' in my training script), but still reported this error  '_pickle.UnpicklingError: invalid load key, '\xbb'.

Sorry again. We fixed that.


Visa immigration block

Added back the right to work/study and immigration status pages to this prototype.
Improved right to work/study question page, completely redone immigration status page to include preset answers in radio button form.
Added a blocking pattern for courses that do not sponsor visas to disallow users from applying if they don't have the right to work/study and require a visa sponsorship.
Deleted a couple of unused pages.
Updated content on Find a course page to show the newest version.

Looks ace!!
Is there a specific training provider I need to select to see the visa blocking pattern?

For now if you select 'No' to the right to work/study question you will be able to see the visa blocking pattern for any course/provider you select. It's because we don't have an API connected to this prototype

Looks ace!!
Is there a specific training provider I need to select to see the visa blocking pattern?

For now if you select 'No' to the right to work/study question you will be able to see the visa blocking pattern for any course/provider you select. It's because we don't have an API connected to this prototype

Oh DOH! I'm being dumb. Ignore me


Export isNotEmpty types as it is now part of ramda 0.30.0

Can we please export the new types required for isNotEmpty? I see that a PR is open for it but not yet merged (https://github.com/ramda/types/pull/101). Unfortunately ramda-adjunct has upgraded their release to now import isNotEmpty from ramda which has the unfortunate consequence of breaking builds.
ramda changelog: https://github.com/ramda/ramda/issues/3453

<EMAIL_ADDRESS>and<EMAIL_ADDRESS>have been released


MatchMaking - Shuffle bug

When reaching the last 4 cards and hitting Play again, 2 cards will be dealt out but freezes and gets a ArgumentOutOfRangeException Error.

Confirmed fixed.

Can also confirm it has been fixed. @tvledesign


Remove mention of the JG newsletter

I intend to eventually sunset the newsletter. It's better if they just go to the web and check the job postings there.

@befeleme I changed the wording (four times 😅). What about now?

I like it! Let's merge & squash it.


WARNING: Client API version = 0.9.15  WARNING: Simulator API version = 0.9.13

Hi! I'm dealing with a problem with version mismatch while running PythonAPI on Carla server. I have carla lib installed with pip install carla, but it always installs 0.9.15 and i cant make it to get 0.9.13.
CARLA version: 0.9.13
Platform/OS: Ubuntu 20.04
Problem you have experienced: WARNING: Client API version = 0.9.15 WARNING: Simulator API version = 0.9.13 after running PythonAPI example
What you expected to happen: Generate traffic
Steps to reproduce: run ./CarlaUE4.sh and then in another terminal cd PythonAPI/examples && python3 generate_traffic.py
Other information (documentation you consulted, workarounds you tried): /CARLA_0.9.13/PythonAPI/carla/dist has only 0.9.13 version files, 2x .whl and 2x .egg.
-->

Solved.
just go into
/CARLA_0.9.13/PythonAPI/examples
and then run
pip uninstall carla
pip install carla==0.9.13


RFC: Allow the Blade view to be set using front matter for Markdown pages

Abstract
Summary
This proposal adds a front matter option for Markdown pages to allow a custom Blade template to be used.
Usage
---
extends: my-custom-layout
---

# Markdown content here

Lorem Ipsum Dolor Sit Amet.

Implementation
Implementation is simple by adding the following method override to the MarkdownPage class.
This makes the change fully backwards compatible as it null coalesces to the default behaviour.
public function getBladeView(): string
{
    return $this->matter->get('extends') ?? parent::getBladeView();
}

Alternate implementations
I initially went with extends as the front matter key, as that matches the Blade syntax. I also see that Jigsaw uses extends. However Jekyll uses layout. We could also support both, same as we support both group and category for navigation data.
My thoughts on the matter
Something like this was considered when originally creating HydePHP as I did not find it necessary, and it risks adding complexity without value. Additionally, for most sites this is not needed as the various page types have different views that get automatically configured, and an override could make things less obvious.
Furthermore, the general use of HydePHP means that it's okay that most Markdown-based pages look the same in terms of layout. However, I realize now that I think it could be useful for Markdown pages for sites with a lot of Markdown page content to have this option.
And considering how surprisingly light this implementation is codewise it comes down to the following: Is this a design pattern we want to encourage? And if the answer is no, is that reason enough not to implement this? Even if something is discouraged we can still have the option for users do things the way they want. For some people this might fit their workflow. And since it's opt-in and doesn't add any unnecessary boilerplate for those who are happy with the defaults I think this feature could be useful.
I always like to get user feedback, hence why I drafted this as an RFC, paired with a Twitter poll. In addition to knowing if users are for or against this, I would also love to get feedback on the front matter property name issue mentioned above: if we should use extends or layout.
Voting
Please vote with an emoji on this post:

Yes: 👍
No: 👎
Undecided: 😕

Or vote via the Twitter poll.
Please also leave feedback in the comments!

Some other static site generators make all front matter properties available in the view. So adding foo: bar in the front matter means that $foo can be used in the view. Hyde does not support this, at least not for now. (The reason I chose not to support this at the time of first thinking of this is because it can cause confusion as we need to filter out some variables form being added as they can cause conflicts)

Noting this is easily resolved, but it could change existing sites so might want it for v2. Anyways, a simple array merge does the trick (it's important that the matter is first in the call)
// Page compile method

$data = array_merge($this->page->matter->toArray(), [
    'title' => $this->page->title,
    'content' => $this->page->markdown->toHtml(static::class),
]);

This is now implemented in the experimental Bladedown extension. We can try it out there, and consider merging it here later on. https://github.com/hydephp/bladedown


Check for app version of WireGuard

With #125 we are pinning the WireGuard version, but if WireGuard is already installed we do not update or downgrade the app. We should check for the version of WireGuard installed to ensure a known state. To implement this we need a function to read out the details of apps installed on the device.
Users experiencing issues with mismatching WireGuard version should simply remove the currently installed version from their test device and rerun the analysis as a workaround.

To implement this we need a function to read out the details of apps installed on the device.

Tracked in https://github.com/tweaselORG/appstraction/issues/127


Failed to compile sass file with sass-loader and ibm-products

Hi guys,
when I try to build the sass file with sass-loader it always get in stuck in a loop and then crash.
Now with these packages:
"carbon-components": "^10.50.0",
"carbon-components-react": "^7.50.0",
"carbon-icons": "^7.0.7",
"@carbon/ibm-products": "^1.2.3",
"@carbon/icons-react": "^10.44.0",
"@carbon/elements": "^10.49.0",
and this webpack:
const sassLoader = {
loader: 'sass-loader',
options: {
prependData() {
return           $feature-flags: (             ui-shell: true,             enable-css-custom-properties: ${CARBON_REACT_STORYBOOK_USE_CUSTOM_PROPERTIES},           );        ;
},
sassOptions: {
//includePaths: [path.resolve(__dirname, '.', 'node_modules')],
},
sourceMap: true,
},
};
and pointing to index_full_carbon I get a missing import.
Can you please provide an example of webpack configuration?

I kind of solved importing in my index.js<EMAIL_ADDRESS>Then my app.scss starts with
`
@import '@carbon/type/scss/type';
@import '@carbon/colors/scss/colors';
@import '@carbon/themes/scss/themes';
// Use the gray 10 theme
$carbon--theme: $carbon--theme--g10;
@include carbon--theme();
$css--font-face: true;
$css--plex: true;
$feature-flags: (
grid-columns-16: true,
ui-shell: true,
enable-css-custom-properties: true,
);
`
But the theme colors do not seem to be loaded properly.
Also if I add the scss in my index webpack is not able to compile with error: STRING_TOO_LONG
@carbon/ibm-cloud-cognitive/scss/index-full-carbon.scss

solved


Implement React Router

Implement React Router with the following pages

[x] Home
[x] Login
[x] Accounts
[x] Balances
[x] Transactions

Fixed with commit #17


Infra: Support for dual-stack IPv4/IPv6

We first need to investigate how a dual-stacked AS looks internally after https://github.com/scionproto/scion/issues/1710 is done.

To give some context on the use case:
To run a dual-stacked AS, all infra (BS, CS, PS) and BR internal addresses need to be dual-stacked.
Hosts can be single stacked, either IPv4 or IPv6.
This is a scenario for ASes that want to support both types of hosts.

Is this still an issue? What is exactly the functionality that is still missing?


WIP: Captcha api protection

Adding a captcha check to the requestPhoneCode endpoint. It is currently based on Google's reCaptcha, but the provider might change.

@jrbuhl93 @nicolasburtey

We need to change all the Chinese comments to English.

Maybe only focus on some specific comments to change to english? I think is better to try to make as little modifications as possible to this external library (i.e. if it needs to be replaced because it was updated).

@dolcalmi The responsibilities point you bring is a good one. I see it in a similar way, the ip and captcha checks purpose is to protect the api, not doing galoy functionality. But maybe this is the right place for these checks? And then the calls to galoy logic be abstracted elsewhere?...

Converted to a local branch, now #458


Translatable does a lot of queries even with HINT_CUSTOM_OUTPUT_WALKER

I have the following strange behavior:
In my entity repository, I have a function of the form
public function getEntities($property) {
  return $this
    ->createQueryBuilder('e')
    ->where('e.property = :property')
    ->setParameter('property', $property)
    ->getQuery()
    ->setHint(\Doctrine\ORM\Query::HINT_CUSTOM_OUTPUT_WALKER, 'Gedmo\\Translatable\\Query\\TreeWalker\\TranslationWalker')
    ->getResult();
}

This function is called several times in the process. I clearly see the effect of the setHint in the doctrine debug panel, in the sense that each call to that function makes a "big" query fetching all the needed translations.
However, only the first call to that function is correctly used, that is, all the subsequent calls are making the "big" query, but each use of an entity then remakes translation queries to the database, as if the setHint were absent.
I also changed the order of the calls, and it's always the first one that "works" correctly.
Did I miss anything in the configuration? The fact that the first call is correctly used seems to mean that I configured it correctly, but maybe I missed something else?
Please ask for more details if I forgot something.

Forgot to mention: I use v2.4.13

I finally found the origin of the problem:
The "walkSelectStatement" function in lib/Gedmo/Translatable/Query/TreeWalker/TranslationWalker.php doesn't get called when the Doctrine query is cached, which it is by default. Then, the query is reused (hence the "big" query), but the Translatable hydrator doesn't get called, hence the subsequent calls to fill the hole.
Maybe the cache should be disabled by the walker to avoid that kind of behavior? (more generally it seems preferrable to me not to cache that kind of select statement)
As far as I can see, there is no way to ask for the query not to go to the cache (we can only ask for it not to get fetched), so I guess it's too late when Translatable comes to the scene. I may have missed something, so I let you decide whether to keep the issue open or not

add a hint for locale, because it will cache only the first locale you call it with. cache key must be unique

hmm the thing is, I'm calling for different values of a property that is not the locale, so the cached query is correct. The problem here is that having a cache "removes" the Translatable hydration process

it should not require hydration as far as I remember, maybe you can try to replicate that test in unit tests? Because I think there is a cached query test.

is other caching working, like simple key value cache you are using, try to test it. So many people would complain if that would not work. Most problems in php are cache related.

There is a hydration done after the query. I went through the code to find the origin of the problem given above. And I confirm that if I disable the "query cache" (with ->useQueryCache(null) just after  the getQuery() in the given example), everything starts working as expected (only one query per "function call" is done instead of one per entity). Note that here the problem is specific to the case where you do twice the same "prepared query" with different parameters, otherwise we have a cache miss and the problem doesn't show up. That would explain the little number of people seeing that issue.

I'll try to create a unit test that fails for that specific case.

Ok I give up I don't have the required level to write that test. Here is what I was trying to do:
If you look at the file tests/Gedmo/Translatable/TranslationQueryWalkerTest.php, function "shouldHandleQueryCache" (which I think is incorrect by the way, the last $q->getArrayResult() should probably be $q2->getArrayResult(), but it's the same for my claim).
Then you expect to do exactly two "select" (with left joins), one for each query. Here, you will actually do 3, unless you put $q2->useQueryCache(null): two with joins and one that queries only the translation table.
I have no proof for that except that it corresponds exactly to what I see in the symfony profiler, but I'm not able to count or extract those queries to prove it.

The problem is caused by hydration mode change inside translation walker https://github.com/Atlantic18/DoctrineExtensions/blob/113a5c0494254d2e29d8b4ebc622dba0c8abe922/lib/Gedmo/Translatable/Query/TreeWalker/TranslationWalker.php#L118-L143
The translation walker changes hydration mode inside the walkSelectStatement method, which is actually called by ORM query parser. As parser is used only for parsing DQL to AST and the cached query is already parsed the walker is not used, hydration mode remains unchanged, therefore the postLoad event on all hydrated entities will result in additional database queries for translations.
Disabling query cache works as a workaround as the parser is always used.

I was having the same issue. Setting the locale hint, and everything as documented in the docs. But it was still making a lots of queries during hydration.
$query->getSQL() was giving me the right query but the results were all wrong. Turns out when you are using custom sql walker it doesnt set the skip bit to false in the listener, which goes ahead and does all the queries anyhow.
This is how I am getting around it.
$listener = $this->get('gedmo.listener.translatable');
$skip = $listener->isSkipOnLoad();

$listener->setSkipOnLoad(true);
$foods = $repo->getAll($filters);
$listener->setSkipOnLoad($skip);

I had the same problem. Language hint is present, main query correct (with joins to translations). Still one extra query for each record. This workaround works perfectly but maybe it's possible to fix it?


Tag values not mutating

The following test fails (on the systems branch):
#[derive(Debug, Clone, PartialEq, Eq)]
struct SomeTag(i32);

#[test]
fn tag_test() {
  let mut world = Universe::new().create_world();
  let entity = *world.insert((), vec![(42,)]).first().unwrap();

  // Set the tag to 1
  world.add_tag(entity, SomeTag(1));

  // All is well in the world.
  assert_eq!(*world.get_tag::<SomeTag>(entity).unwrap(), SomeTag(1));

  // Set the same tag to 2.
  // Even if you do this first: world.remove_tag::<SomeTag>(entity);
  world.add_tag(entity, SomeTag(2));

  // Panic:
  // ... assertion failed: `(left == right)`
  //   left: `SomeTag(1)`,
  //  right: `SomeTag(2)`',
  assert_eq!(*world.get_tag::<SomeTag>(entity).unwrap(), SomeTag(2));

  // The world is broken. Brutal.
}

I didn't test master because hopefully systems merges soon. Also tagging @jaynus in hopes of a pointer on what might be happening.

After digging into this, its a disparity between get_tag and querying tags:
There was a test, mutate_change_tag that tests using a query to retrieve the tags, but not using get_tag. get_tag specifically fails while the query succeeds.

Tigging in more, it seems get_Tag is not retrieving the correct archetype as the ID changes on set.
get_tag: EntityLocation { archetype_index: 0, set_index: 0, chunk_index: 0, component_index: 1 }, ArchetypeId(WorldId(0), 0)
get_tag: EntityLocation { archetype_index: 0, set_index: 0, chunk_index: 0, component_index: 1 }, ArchetypeId(WorldId(0), 0)
Set location! 1, EntityLocation { archetype_index: 1, set_index: 0, chunk_index: 0, component_index: 0 }Set location! 1, EntityLocation { archetype_index: 0, set_index: 1, chunk_index: 0, component_index: 0 }CHANGED

get_tag: EntityLocation { archetype_index: 0, set_index: 1, chunk_index: 0, component_index: 0 }, ArchetypeId(WorldId(0), 0)

This was fixed by this change here:
https://github.com/jaynus/legion/commit/439b9584dd79f580a8653de6d42cb4d59e6fdd34#diff-9de97664aabc3e1e7269e1e0fbec6613L498

@jaynus thank you! I'll leave this open as a tracking issue for the main Legion repo.


AusweisApp2: 1.26.7 -> 2.0.0

Description of changes
Update AusweisApp2.
https://github.com/Governikus/AusweisApp2/releases/tag/2.0.0

Rename package to AusweisApp
Add an alias to the old name
Enable Wayland support
Update metadata (description and homepage)

Things done

Tested "Smartphone as Card Reader" function

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage

[x] Tested basic functionality of all binary files (usually in ./result/bin/)

23.11 Release Notes (or backporting 23.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

The name change from AusweisApp2 to AusweisApp should probably also be done in the module (nixos/modules/programs/ausweisapp.nix).

Since the pull request already introduces a substantial number of changes to the package, this presents a great opportunity to align with best practices. Consider replacing rec with the finalAttrs pattern and eliminating the need for with lib; by utilizing lib.* directly.

By the way, I just checked to see if there are any conventions regarding the use of capital letters in package names.
https://github.com/NixOS/nixpkgs/blob/eb2f8aa686fe22faf939f3ef158f9062f908e6e2/pkgs/README.md?plain=1#L330

Thank you for the feedback. I was not aware of the "no uppercase letters" rule, but I agree that it looked wrong.
I have updated all names and removed the rec and with lib; uses.

2.0.1 has just been released, which includes a minor fix for MacOS.

Pushed a new update with version 2.0.1.
I also added a version test, to ensure the binary version matches the expected one (I forgot to change the src hash and used the old one for some time).

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

2 packages blacklisted:

nixos-install-tools
tests.nixos-functions.nixos-test

1 package built:

ausweisapp


Change channel id to native-channel-name in intan

From the discussion in #1402.
I think this will be distracting over there and be solved as an small PR here.
From a discussion with Sam, the purpose of an id is to be unique whereas name is meant to be user determined or more descriptive.
Right now, we are using an index as channel id. The present PR uses the native_channel_name which accomplishes the design purpose of channel id while at the same time is more meaningful.
The relevant section in the specification:

@zm711 I just realized this as I was working on the PR.
The memmap field should be named as the channel ids as they are unique. Right now we are using names which is wrong as field names in structured dtypes have to be unique.

@h-mayorquin , I already made that change in the other PR. Upon your recommendation. :)

I see what you changed though!

@zm711 Yeah, I am aware, I just think that it will simplify the PR if this small issues is addressed separatley. Plus, I think we should change the way we are naming the memmap fields.

I agree with the memmap field name. You are right. The names can change the ids can't so we should do the ids to be safe.


History improvements

Following up https://github.com/excalidraw/excalidraw/issues/307.
Covers:

Simplify redoOnce function to match undoOnce shape.
Move clearRedoStack call to pushEntry (we might be erasing the redo stack when we haven't created any new entry in the history if the generated current entry is equal to last history entry).
Remove skipRecording call from restoreEntry because we already make sure to don't loop by checking that the history entry we want to push is not equal to the last one. If we want to optimize the number of calls to pushEntry in componentDidUpdate, we should do it via external mechanism (same way we only call pushEntry if isRecording). The history itself should not be aware of this recording thing.

cc @vjeux as he was in the context 👍

Thanks!


Incorrect XML documentation comment for 'OpenXmlBasePart.OpenXmlPackage'

The following source code XML comment (which likely propagates to the developer documentation, though I didn't verify that) is incorrect (and somewhat illogical):
https://github.com/OfficeDev/Open-XML-SDK/blob/601492385e0130f63b39ac1735d58d3a394e3a69/src/DocumentFormat.OpenXml/Packaging/OpenXmlBasePart.cs#L265-L268
This should instead read: "Gets the OpenXmlPackage which contains the current part."
The suggested text is taken from the (correct) XML documentation for the OpenXmlPackage property in the OpenXml.Packaging.DataPart class:
https://github.com/OfficeDev/Open-XML-SDK/blob/601492385e0130f63b39ac1735d58d3a394e3a69/src/DocumentFormat.OpenXml/Packaging/DataPart.cs#L42

Can you submit a PR for this? Thanks!

@twsouthwick  I could try, but I'm not yet fully comfortable with my git competency yet and have never gone through a contribution process...

That's fine - we can help you through the process. Changes like this are the best to start with since they're small :) For this change, you could do it all online by clicking the 'Edit' button in the file you found has the doc issue. If you need any help, feel free to ask on here


Why label is called 'instance' and in config it is called 'target' ?

This issue track is for bugs and feature requests, https://groups.google.com/forum/#!aboutgroup/prometheus-users is the best place to ask questions of this nature.


Test fixes

SetDirectory* tests got broken since commands how check if host is running. Updated and added cases for remote.
PeekInternalFunction01 was failing b/c startupInfo.WorkingDirectory was empty and not null which caused setwd to fail at the host startup.
Editor test needs to be UI test. Was failing occasionally depending on the test execution thread,

Will merge since this basically just text fixes.


TableControllers appear on both the /api and the /tables routes

TableControllers should only appear on the /tables routes.

Yes please!


CVODES tests are segfaulting on v0.7

On v0.6 they were fine, showing the the wrapper is fine. The issue is the test setup. Something about how we're wrapping one of the cfunctions is not v0.7-compatible. I tried a bit and couldn't figure it out.
pinging @axsk since it was your code I was trying to update.

Thanks for the attempt, I'm looking at it now.

Maybe reopen if still an issue?