Proactive protection against GovAction deposit losses

Current situation:
We have a guard present that disallows you to deregister a stakeaddress if there is still a rewards-balance > 0  in the ledger-state.
We also had a change in the way stakeaddress-registration must be signed, needs the signing with the stake.skey now. Because the stakeaddress deposit fee can change over time, and we wanna make sure that we refund the correct deposit amount to the owner of the address.
So, it comes as a bit of a surprise that there are currently no safety guards when it comes to stakeaddresses used in governance action proposals.

We have the scenario that someone uses a wrong stake address as the deposit return address. There is currently no need to sign a governance transaction also with the stake secret key. Once submitted, there is no way to correct the wrong address anymore afterwards. Deposit funds would be lost.

Another scenario is, that someone is doing a stakeaddress deregistration while that same stakeaddress is involved in an active governance proposal. Once the proposal times out or gets enacted, the deposit fee (currently 100kAda) would be lost.

There is also the scenario with a wrong or deregistered stakeaddress in a treasury withdrawal action. For the treasury payout stakeaddress, that would not be a big issue, the funds would simply go back into the treasury. So we may not have to cover that.

We should introduce some safety mechanism into the ledger, to proactively restrict users from submitting deregistration certificates while there are active governance actions are present involving the same stakeaddress.
To make sure, that noone can "block" a strangers stakeaddress, or that users submit an action with a typo in the stakeaddress, we might wanna think about to add the stakekey signing as a required signature for a action submit transaction too.
There is also a request in the cardano-cli repo to add the currently locked up actionDepositFee to the stake-address-info query (https://github.com/IntersectMBO/cardano-cli/issues/592)
Best regards,
Martin

I've mentioned this before that all of those guards can be implemented on he tooling side.
However, point #1 is certainly something we should try and protect against, just because deposit for proposals is quite significant. Loosing it for the proposer would be devastating.
So, I suggest we add a predicate failure for the next intra-era hard fork that checks that reward accounts specified for deposit in all of the proposals and in TreasuryWithdrawals are indeed registered.
With respect to preventing deregistration of staking credentials if they are used in other places in the system is not something that we are going to do for two reasons:

It would be an expensive operation
Anyone could prevent staking credential from being deregistered, by using it for their own (proposal, pool or treasury withdrawal). It is highly unlikely that someone would use a proposal for that today :smile:, but nevertheless we never know if the deposit will always be that high for proposals.

Isn't it somehow possible to store the action deposit fee also in the rewards-account-balance? Not directly for a withdrawal, but the amount will be there once the proposal is over. So after a action deposit, this amount will come back to that rewards-account no matter what. So in the case of a deregistration attempt, couldn't the node simply lookup in that additional rewards-acount-balance storage and refuse to continue as long as there is a value present?

Of course, this is technically doable, but:

first of all, it is not as simple as it sounds. We'd have to track not only which reward accounts are being used, but also by which entities: which proposals, stake pools, treasury withdrawals. This is because when any of them are retired we'd have to remove those exact instances from that Map. So, it would have to be a BiMap for efficient lookups.
secondly, this whole complexity comes at a computational cost and memory overhead.

We want to avoid making Cardano slower and overly complex, especially when a problem can be solved in tooling.
Most importantly, however, despite that it is technically possible to implement such a guard, there is a fundamental issue with this.
Let's say you have a reward account r1. And it so happens that I really don't like you for some reason, so much so that I am willing to pay money to mess with you. Then I can register a stake pool and specify your account r1 as an account where my deposit should be returned. Until I deregister my pool, which may never happen, you would not be able to deregister you r1 account and get your 2 ADA deposit back.
I know that it doesn't sound very plausible, but in our job we have to look at all possibilities. And this one, despite being not very probable, would be theoretically possible.

We should introduce some safety mechanism into the ledger, to proactively restrict users from submitting deregistration certificates while there are active governance actions are present involving the same stakeaddress.
To make sure, that noone can "block" a strangers stakeaddress, or that users submit an action with a typo in the stakeaddress, we might wanna think about to add the stakekey signing as a required signature for a action submit transaction too.

In other words, none of this is really worth the complexity it entails. It is much simpler and cheaper for a wallet to implement this sort of guard to protect their users from making mistakes.

so, why can't we than add the stakekey as a required signing to a stakepool registration for the rewads account? we had this in the documentation/tutorials for a long time as a mistake. also making the signing via stakekey mandatory for a transaction containing an action proposal? that would help and also make it highly unlikely that you use the wrong stakeaddress for the deposit return.

This nearly happened to us during the very first mainnet governance action. We were about to submit an info action with an unregistered stake address but fortunately remembered at the last minute that we needed to register it first. That was a close call.
We definitely need a mechanism to prevent such mistakes. Specifically, if an address isn’t registered when submitting a governance action, the system should block the transaction from being submited.
Regarding de-registration during the governance action lifetime, would it be possible to keep the 100k ADA in the same pot even after the govActionLifetime expires? The only way to retrieve it would be by signing a specific transaction with the owner's stake.skey. This way, the governance deposit pot could serve as a temporary emergency "reward address" for cases where a stake address is mistakenly unregistered.
Does that make sense, or am I off track?

would it be possible to keep the 100k ADA in the same pot even after the govActionLifetime expires? The only way to retrieve it would be by signing a specific transaction with the owner's stake.skey.

This is also technically possible, but that is a huge change that requires a whole new era, changes to plutus context, etc. Moreover then the same question arises, why only governance actions? Why not create this protection for the pool deposits?
My answer is again: ledger does not need to solve this problem if it can be solved in the tooling!
Wallets and cli tools that build transactions can easily check if reward account is being used and prevent it from being deregistered.
This feature that is being requested is not needed for safety of Cardano, it is needed to protect users from mistakes and wallets are a great place for this.
The only reason why I am willing to accept deviation from the spec and adding a check for presence of reward account for the proposal deposit, is because it is very easy to do that has almost no overhead on the Ledger.

We were about to submit an info action with an unregistered stake address but fortunately remembered at the last minute that we needed to register it first. That was a close call.

You could have registered the stake address after you submitted the proposal and you would have been fine. As long as the stake address is registered when proposal is expired you don't loose the deposit.

we care about 2 Ada, so we def. should care about such a way higher amount.

You totally missed my point, but that's ok. It's not my job to convince others.

but it should be also done on the ledger level imo.

I appreciate your opinion. But none of this is necessary for the safety of the system.
When you send ADA to a wrong address that does not exist, you also loose that money.
I gave you my expert explanation on why we are not going to do this. I'd appreciate if you stop pushing your opinion. If you don't agree with what I said, feel free to submit a CIP and if there is support form the community on changing the system in the way you are suggesting, it will get implemented. Until then this is a wallets' issue, not ledger's.

You could have registered the stake address after you submitted the proposal and you would have been fine. As long as the stake address is registered when proposal is expired you don't loose the deposit.

Oh! Awesome I wasn't sure about that TBH. Thank you. 🤩


changed event type to be more clear

@banisadr,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@djrosanova #sign-off

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

Microsoft authors of articles need to make updates through the private repository so the updates can be staged and validated by the current validation rules. After you review staging and fix all validation issues, you can add the sign-off comment to let the PR reviewers know the updates are ready to be merged.
It's easy to access the private repo version of your articles in GitHub. Just add -pr to the URL, as follows:
Public repo URL: https://github.com/Microsoft/azure-docs/blob/master/articles/azure-portal-overview.md
Private repo URL: https://github.com/Microsoft/azure-docs-pr/blob/master/articles/azure-portal-overview.md
We've closed this pull request. Please resubmit the updates through the private repo. If you need perms to the private repo, see the contributor guide instructions


[Feature Request]: how can I structure of chromadb collection from [embeddings, documents, metadatas, ids] to something else?

Describe the problem
I am currently using Chroma to store a collection of documents. The current structure of the collection is as follows:
[   {     "embeddings": [       [1.0, 2.0, 3.0],       [4.0, 5.0, 6.0]     ],     "documents": [       "This is a document.",       "This is another document."     ],     "metadatas": [       {         "key": "author",         "value": "chromadb"       },       {         "key": "title",         "value": "A Collection of Documents"       }     ],     "ids": [       1,       2     ]   } ]
I would like to be able to change the structure of the collection to something like the following:
[   {     "id": 1,     "document": "This is a document.",     "metadata": {       "author": "Anant",       "title": "A Collection of Documents"     },     "other_information": "This is an extra information"     "extra": ["extra1", "extra2"]   },   {     "id": 2,     "document": "This is another document.",     "metadata": {       "author": "Bard",       "title": "Another Collection of Documents"     }     "other_information": "This is an extra information"     "extra": ["extra1", "extra2"]   } ]
Is there a way to do this?
Describe the proposed solution
Allow users to specify the structure of the collection when it is created.
Provide a way to convert an existing collection to a different structure.
Alternatives considered
I am using Chroma in a production environment and I need to be able to change the structure of the collection without disrupting my users.
Importance
i cannot use Chroma without it
Additional Information
No response

Can't you just transform the input/output from chroma yourself?
results = collection.query(query_texts=["this is my query"])

formatted_results = [{
    "id": id,
    "document": results[0].document[index],
    "metadata": results[0].metadatas[index]
  } for id, index in enumerate(results[0]["ids"])]

Note: did not test this

Can't you just transform the input/output from chroma yourself?
results = collection.query(query_texts=["this is my query"])

formatted_results = [{
    "id": id,
    "document": results[0].document[index],
    "metadata": results[0].metadatas[index]
  } for id, index in enumerate(results[0]["ids"])]

Note: did not test this

Yes, I tried this. But, I want to use the custom structure of Chromadb.Collection. Currently chroma db only has structure as [embeddings, documents, metadatas, ids]. Can I do it like, [ids, embeddings, document, metadatas, feedback]
Means I want add more columns in chromadb collection?

@anant-patankar You could just add additional fields like feedback to metadata

@anant-patankar currently we don't have the idea of additional "columns" as that would introduce things like schema, types, migrations, etc - complexity which we don't want to add (yet). adding additional fields to metadata is the happy path currently.
We will likely move from column based to row based input and output in the not too distant future. Thanks @Russell-Pollari for sharing this little example code! I actually think some small util for row->column (add) and column->row (query) would be a nice thing to add to the project.

let's not make it more complex. I added whatever data I want to add in a new column to the metadata itself by setting up an extra key.
Thank you @jeffchuber, @Russell-Pollari  for your prompt response. I sincerely appreciate the time you took to address my issue.

@anant-patankar sounds great :)


Update the title

It looks like i can't update the title without destroying the selectpicker and changing the html, like this.
$('.selectpicker').selectpicker('destroy');
$('select.selectpicker').html(someOptions);
$('.selectpicker').selectpicker({title = 'New Title'});

Did I miss something along the way? Can this be added as a new feature?

This change will be a part of the v1.14.0 release. This issue will be updated after it's made available.

Released in v1.14.0-beta!


Component functions for REST API and GraphQL

Transitioning to Laravel Auth is great. However, implementing functions directly into components is unfortunate. These functions can't be used when creating REST API and GraphQL interfaces, which leads to unnecessary duplication of logic.
Wouldn't it be better to separate the logic common to components and APIs into a separate class? For example, password reset requests use PasswordBroker. The User class implements a trait with the function sendPasswordResetNotification(). Although there is a setUrlForPasswordReset() function available, I don't see a way to call it if I want to use REST API and don't have a resetPassword component in the CMS. I would also prefer to send the email through the queue. But how can I override the sendPasswordResetNotification function in this trait?

Dood points, @xmelic8. Thank you!

Is there a plan to implement these changes in the near future versions? Or should I create a fork, make the changes, and submit pull requests for testing?

Yes, please. Then we can make a new release for it alongside #565


Remove Primitive Types example

Removing the file because it was in a directory that never displayed in the web page and also some parts of the example were covered by other files.

I think we should keep somewhere the part about the different ways numeric literals can be expressed in (hex, decimal, etc.).

Good point, I just added the files to the correct folder and fixed a few issues the example had (it wasn't going to compile). We are having someone from the community doing a review/report on this project so he'll let us know if this example is useful. The more examples to evaluate the better.


Dynamic example fails on OpenShift instance

I am using<EMAIL_ADDRESS>on an OpenShift instance. I am trying the examples on the README page. The static example works, but the dynamic one fails with a strange error. Any hints?
I would expect it might be due to the limited environment available on OpenShift instances and I wonder what is the cause so I can try to fix it.
> cat > static.js
var scraperjs = require('scraperjs');
scraperjs.StaticScraper.create('https://news.ycombinator.com/')
    .scrape(function($) {
        return $(".title a").map(function() {
            return $(this).text();
        }).get();
    }, function(news) {
        console.log(news);
    })
> node static.js
[ 'Show HN: My SSH server knows who you are',
  'Show HN: JAWS – A JavaScript and AWS Stack',
  'Federal Judge Strikes Down Idaho ‘Ag-Gag Law’',
...

> cat > dynamic.js
var scraperjs = require('scraperjs');
scraperjs.DynamicScraper.create('https://news.ycombinator.com/')
    .scrape(function() {
        return $(".title a").map(function() {
            return $(this).text();
        }).get();
    }, function(news) {
        console.log(news);
    })
> node dynamic.js

events.js:72
        throw er; // Unhandled 'error' event
              ^
Error: listen EACCES
    at errnoException (net.js:905:11)
    at Server._listen2 (net.js:1024:19)
    at listen (net.js:1065:10)
    at net.js:1147:9
    at asyncCallback (dns.js:68:16)
    at Object.onanswer [as oncomplete] (dns.js:121:9)

It seems that scraperjs tries to open a port on the host. When I run the dynamic example on a different host I capture this with netstat:
tcp        0      0 <IP_ADDRESS>:45873         <IP_ADDRESS>:*               LISTEN      355/node

The port is a random port which an user account should be able to open but I guess this is not allowed on OpenShift instances. Could the library achieve the required functionality without opening a port?

Actually it seems that ports can be open but only within this range [1]:

It is possible to bind to the internal IP with port range: 15000 - 35530.

I think I fixed this. DynamicScraper uses Phantom. Phantom allows for an options argument where a port can be specified. I added an options argument to DynamicScraper where a port can be specified. For example:
scraperjs.DynamicScraper.create(url, {port: 29999})

See #42 for a pull request.

For OpenShift, since it only allow for opening ports on the internal IPs [1], a complete example is:
scraperjs.DynamicScraper.create(url, {
    port: 29999,
    hostname: process.env.OPENSHIFT_NODEJS_IP || '<IP_ADDRESS>'
})

I'm glad that you were able to solve the issue. It's related with node-phantom. They need a network connection to communicate, hence the necessity for a port definition.


Handling cell deletion

Hi @j-fu,
Great work with this meshing package! Particularly appreciate the partitioning feature! 🚀
Quick question - have you ever considered the problem of cell deletion? Let's just imagine a heat conduction problem and a melting material. If one cell reached the melting temperature, we ought to remove it because it "melted" away.
I have not found documentation or discussion about this in ExtendableGrids, would you happen to have any idea?

No, we don't have this and didn't think about this so far. It seems to be possible to implement something like this based on a marker vector.  Essentially this would amount to creating a new grid because removing cells also would lead to removing vertices.  One would have to figure out what to do with the boundary. However, something like this is  already in the subgrid code. May be tweaking cellregions and extracting subgrids could work.


Full test coverage for sysinfo.software

As the title says, this PR is improving on the coverage for sysinfo.software.  Original coverage is captured here: https://coveralls.io/builds/3655258/source?filename=pyfarm%2Fagent%2Fsysinfo%2Fsoftware.py

@guidow, PTAL

Looks okay to me.


module fletcher does not contain draw-diagram

Hi, thanks for your work! I am facing this error -
error: module `fletcher` does not contain `draw-diagram`
    ┌─ main.typ:337:17
    │
337 │         fletcher.draw-diagram(grid, nodes, edges, options)
    │

I am using the same test diagram you are testing for (but you use a different way of importing). A minimal reproduction can be found at - https://typst.app/project/wlQtX5yQCf0v80si_SPvPo
#import "@preview/fletcher:0.2.0" as fletcher: node, edge
#import "@preview/cetz:0.1.2"

#figure(
  fletcher.diagram(
    node((0,0), $A$, stroke: 1pt),
    node((2,1), [Bézier], stroke: 1pt),
    render: (grid, nodes, edges, options) => {
      cetz.canvas({
        fletcher.draw-diagram(grid, nodes, edges, options)

        let n1 = fletcher.find-node-at(nodes, (0,0))
        let p1 = fletcher.get-node-anchor(n1, 0deg)

        let n2 = fletcher.find-node-at(nodes, (2,1))
        let p2 = fletcher.get-node-anchor(n2, -90deg)

        let c1 = cetz.vector.add(p1, vector-polar(20pt, 0deg))
        let c2 = cetz.vector.add(p2, vector-polar(70pt, -90deg))

        fletcher.draw-arrow-cap(p1, 180deg, (thickness: 1pt, paint: black), "head")

        cetz.draw.bezier(p1, p2, c1, c2)
      })
    }
),
  caption: [
    A minimal LLM system that includes a context store.
  ],
)

Hi! Thanks for finding this. I made a mistake with how I organised the package’s internal namespace — the module fletcher actually contains a sub-module called fletcher, which included all the internal functions. This is probably too confusing!
Here’s working example:
#import "@preview/fletcher:0.2.0": fletcher, node, edge
#import "@preview/cetz:0.1.2"

#figure(
  fletcher.diagram(
    node((0,0), $A$, stroke: 1pt),
    node((2,1), [Bézier], stroke: 1pt),
    render: (grid, nodes, edges, options) => {
      cetz.canvas({
        fletcher.draw-diagram(grid, nodes, edges, options)

        let n1 = fletcher.find-node-at(nodes, (0,0))
        let p1 = fletcher.get-node-anchor(n1, 0deg)

        let n2 = fletcher.find-node-at(nodes, (2,1))
        let p2 = fletcher.get-node-anchor(n2, -90deg)

        let c1 = cetz.vector.add(p1, fletcher.vector-polar(20pt, 0deg))
        let c2 = cetz.vector.add(p2, fletcher.vector-polar(70pt, -90deg))

        fletcher.draw-arrow-cap(p1, 180deg, (thickness: 1pt, paint: black), "head")

        cetz.draw.bezier(p1, p2, c1, c2)
      })
    }
),
  caption: [
    A minimal LLM system that includes a context store.
  ],
)

Oh, I see. Thanks and Merry Christmas to you as well!


[DO NOT MERGE] Tracking patch orm5.4: tracking master, hibernate-orm-master-h2-main

Pull Request Test Coverage Report for Build 1

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 86.706%

Totals

Change from base Build 16:
0.0%

Covered Lines:
10129

Relevant Lines:
11682

💛  - Coveralls

Closing. I was trying to see if it's possible


feat: Remove title input field from HighlightInputForm

Description
This PR removes the title input field from the HighlightInputForm.

What type of PR is this? (check all applicable)

[x] 🍕 Feature
[ ] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[ ] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Related Tickets & Documents

Fixes #1657
Mobile & Desktop Screenshots/Recordings

Before:

After:

Added tests?

[ ] 👍 yes
[x] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

Added to documentation?

[ ] 📜 README.md
[ ] 📓 docs.opensauced.pizza
[ ] 🍕 dev.to/opensauced
[ ] 📕 storybook
[x] 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

Hi @RitaDee,
Is this PR ready for review?

Hi @RitaDee,
Is this PR ready for review?

It’s not ready.

@brandonroberts, I have done the fix.

Ideally this PR merges after https://github.com/open-sauced/app/pull/1646
cc @diivi for awareness.

Yup

@OgDev-01  @RitaDee  lgtm! one minor comment is that I'd remoe the "Post a highlight to show your work" span and make it just a placeholder for the form, but I think Divii has done that on the PR that is going to be merged with this


use dateStyle and timeStyle instead of custom impl

Try to replace this : https://github.com/onigoetz/i18n/blob/master/packages/intl-formatters/src/date.ts#L7-L47
with this : https://www.intl-explorer.com/DateTimeFormat?locale=fr
It should be locale independent and even smaller.

currently keeping the original, browser support is quite recent: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Intl/DateTimeFormat


Improvements for v3 branch

enforced standalone ability of each package (~except babel-plugin-glamor~)
fixed typings problems in most packages
added package.jsons where necassary

I tested some of the packages by linking them in one of our production SPAs and it works seamlessly :)

All packages have now a package.json and can be published as standalone modules. I removed the new/ dir to get a cleaner repo, but this can be reverted and its still available in git history. Examples are now in a standalone package named glamor-examples which is private. glamor server is now a standalone package.
Commands available in root:
yarn: install devDeps, bootstrap and build all packages (maybe the latter is not necessary)
yarn run bootstrap: bootstraps all packages
yarn run test: executes the test script in all packages (if present)
yarn run build: builds all packages via build script (if present)
yarn run examples: will start a webpack-dev-server for all examples
Commands available per package:
yarn run build
yarn run test
yarn run test:watch

Awesome change!


FR: Allow searching settings

Is Your Feature Request Related to a Problem? Please Describe.
It's so hard to find settings, please let us search
Describe the Solution You'd Like
Search settings. This search should also include searching through the aliases given in rules.md

I am not sure I would be able to implement a search as the UI is not my strong suite. But I would be open to PRs to add this.

@Explosion-Scratch , you might be able to use obsidan setting search to allow you to search settings until the linter is able to get something setup for either searching the rules or making it easier to find rules.

I think I have thought up a way to add the search feature, but I will have to test it out after custom lint commands is added.
I am thinking you will be able to search all rules that are regular rules. This means that general settings and custom lint commands (a pending feature) would not be included in the search.
Programmatically I am thinking the following should work nicely:

Add rule alias as an attribute for each rule's element
Add a search box above the generated rule options, but below the heading. When a search is made, hide any that do not match the search.
Rules would be displayed as all or nothing.

I wanted to give an update on this. I am not the best when it comes to UI, but I have converted the  settings layout into a tab layout. From there, I am going to add a search settings tab.
While I would love for this to be visually pretty, it may not be at first. But I will do my best. This looks like I may be able to add it by the end of the week. However it would not be available until the next release unless you pulled from master.

I have not forgotten this. A search is in the works, but I decided to redesign the settings as well to help make things clearer on what rules exist and where to find them. Here is what I have so far:

I still need to iron out how search will work and get it and a new layout for mobile. But once that is done, I can go ahead and merge the changes.

I just finished up search for desktop. I am working on the mobile design and searching. I will hopefully get to that today.

This should now be added for both mobile and desktop. Mobile has a slightly different navigation, but it follows the same principles. It should be easy enough to pickup how it works with search as well. Please let us know if there are any issues with this in the next release or on master.


Replace the pagination from the project page.

Describe the task

The pagination should be removed from the CRM->Projects and the lazy loading should be implemented in place of that.

Expected behaviour

The pagination should be removed from the CRM->Project page.

Not needed. Closing.


Changing Awesome Android Alternatives to Awesome Android Apps/Applications

I think that would be less confusing. What do you guys think?

Awesome Android Apps

So I think the winner is Awesome Android Apps. I'm going to change the logo soon. However I am a average user of Inkscape so if someone wants to help me - please do so.


[Home Loan Page] Update content modeling for Product Loan section/block

Completed via https://github.com/aemsites/piramal/pull/77


JUnit XML for Unit Tests and Bug Fix

The bug was that when a Prow job fails, no xml results are uploaded because the test script exit on nonzero status immediately and the xml file was move to artifacts / upload directory as the last step. By creating the files in _artifacts dir directly, it is always uploaded by Prow.
Also git ignores the junit results so developers running tests locally will not pollute his/her workspace. Passing junit xml file names through environment variables provides more flexibility and prevents overwriting results if executing multiple commands that all generate junit xmls.

/hold

@chxchx: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
2af765e0390fd3cb7508ff20d1a41bfb642ad889
link
/test istio-unit-tests

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.

/hold cancel

@chxchx: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
e0b19e1636baac132c9d7ec242db48a849bc4133
link
/test istio-unit-tests

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.

@chxchx: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
e0b19e1636baac132c9d7ec242db48a849bc4133
link
/test istio-unit-tests

prow/istio-presubmit.sh
e0b19e1636baac132c9d7ec242db48a849bc4133
link
/test istio-presubmit

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.

Agree with proposal from @mattdelco to use $ISTIO_OUT as default location and call mkdir from make recipes.

@chxchx: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
e0b19e1636baac132c9d7ec242db48a849bc4133
link
/test istio-unit-tests

prow/istio-presubmit.sh
72e5ce835afdb8df6ed94a2db1ceba38aa76855f
link
/test istio-presubmit

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.

@chxchx: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
72e5ce835afdb8df6ed94a2db1ceba38aa76855f
link
/test istio-presubmit

prow/istio-unit-tests.sh
72e5ce835afdb8df6ed94a2db1ceba38aa76855f
link
/test istio-unit-tests

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.

/lgtm
but my PR says expect the tests to fail for unrelated reasons (something wrong with updating vendor/)

/lgtm

@chxchx: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests.sh
2b9e0754f9ee13745099a36d73455a4b7cd643aa
link
/test e2e-bookInfo

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.

/test e2e-bookInfo
should pass now

but my PR says expect the tests to fail for unrelated reasons (something wrong with updating vendor/)

that should be fixed now, sorry about that

@chxchx: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests.sh
2b9e0754f9ee13745099a36d73455a4b7cd643aa
link
/test e2e-bookInfo

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.

PTAL
It is very cool that Prow UI also changes given the junit results. Once this PR is in, testgrid will render per test case pass/fail info from each run.

I think the approach for go get would only work if someone has their PATH set up.  A scheme I've been using elsewhere for other tools is to first have a variable that says where to find the parser, e.g.:
JUNIT_REPORT:=$(shell which go-junit-report 2> /dev/null || echo "${ISTIO_BIN}/go-junit-report")
and then have a rule that tells Make how to download the tool if necessary:

${ISTIO_BIN}/go-junit-report:
	@echo "go-junit-report not found. Installing it now..."
	unset GOOS && CGO_ENABLED=1 go get -u github.com/jstemmer/go-junit-report

and then have an order-only dependency on $(JUNIT_REPORT) (so that Make doesn't throw a fit about not knowing how to build the instance that might already exist elsewhere on the system) and then use "$(JUNIT_REPORT)" instead of "go-junit-report" in the build recipe.

/lgtm
A possible refinement is to also use "&& unset GOARCH" with the
unset GOOS && CGO_ENABLED=1 go get -u github.com/jstemmer/go-junit-report
though it's not a big deal yet.

/assign @sebastienvas @costinm


add san-domains

Let's Encrypt supports SAN domains. concert should give the user the ability to generate certs with multiple domains that are not subdomains.

https://gist.github.com/secured/ee82f214e4d061958ed627887b50c20d

Please send a proper PR @secured Thanks for the PR.

#22

Fixed by #22


User can create namespaces without permission

Rancher versions:
rancher/rancher: 2.1.0
kubernetes (if applicable): 1.11.3
Docker version: 17.3.2
Type/provider of hosts: baremetal
Setup details: singel-node
Environment Template: Kubernetes
Steps to Reproduce:

setup Rancher2 + auth via LDAP

make "User Base" default role

create a project project1

add a testuser from LDAP only to that project with custom permissions ALL without 'create namespace', 'manage-members'

Results:
User is still able to create namespaces, which shouldn't be..

Is the testuser added in any other project?
There is a caveat about project roles:
https://rancher.com/docs/rancher/v2.x/en/admin-settings/rbac/cluster-project-roles/#project-roles

@deniseschannon: No, this was a clean setup and the testuser is only a member in that project.

Can you provide a little more info by showing us the role(s) you used that did not have the create namespace permission?

either a screenshot or  view it in the api and copy pasting the output

I just tested this scenario with OpenLDAP with these steps and could not reproduce:

spin up rancher
turn on OpenLDAP integration
Add a cluster
Open the default project in the cluster, go to members tab
Click Add member and then select the custom option and check all the boxes except Create Namespaces and Manage Project Members.
In a different browser (ie firefox instead of chrome bc my admin session was still active in chrome), log in as the user added in step 5.
Attempt to create namespaces in the cluster and project that I was granted access in.

Result:
Request rejected with this error message:
namespaces is forbidden: User "u-6wxyvvf3ra" cannot create namespaces at the cluster scope

Note that the enforcement of this rule is done by kubernetes itself. Does the cluster that you are trying this in have RBAC turned on? How did you deploy that cluster?

Let us know if you can still reproduce.

Hi,
this feels  a little awkward, but i am running into the same problem again with 2.1.3 on another cluster...
> k create ns test
namespace "test" created
> k delete ns test
Error from server (Forbidden): namespaces "test" is forbidden: User "u-zbw65mdtej" cannot delete resource "namespaces" in API group "" in the namespace "test"

global->users->ci:
x  | Create Clusters | n/a |
x  | Manage Authentication | n/a |
x  | Manage Catalogs | n/a |
x  | Manage Node Drivers | n/a |
x  | Manage PodSecurityPolicy Templates | n/a |
x  | Manage Roles | n/a |
x  | Manage Settings | n/a |
x  | Manage Users | n/a |
x  | Use Catalog Templates | n/a |

- no cluster roles
- no project roles

cluster: prod->members:
Default Admin (admin)

cluster: prod->Projects->testperm:
ci dev ( ci ) User  View Config Maps  6:13 PM
ci dev ( ci ) User  View Ingress  6:13 PM
ci dev ( ci ) User  View Volumes  6:13 PM
ci dev ( ci ) User  View Secrets  6:13 PM
ci dev ( ci ) User  View Service Accounts   6:13 PM
ci dev ( ci ) User  View Services   6:13 PM
ci dev ( ci ) User  View Workloads  6:13 PM

OK, some more investigating showed a lot of (probably leftover) clusterrolebindings "clusterrolebinding-hash".
After removing ALL of them, and re-adding the group via rancher-ui it seems to work again.


new BAS screen shots and minor text changes

That's weird, the image is right here: https://github.com/sap-tutorials/Tutorials/pull/20821/files#diff-f19e3cae143a16336bd4df4d44ba2d27eef2b29cb0e555d3fd0ea6116f26d9f2


Services

{package} {version} {:snowflake:}
Destination channel: {Snowflake | defaults}
Links

{ticket_number}
Upstream repository
Upstream changelog/diff
Relevant dependency PRs:

...

Explanation of changes:

...

Linter check found the following problems:
ERROR conda.cli.main_run:execute(124): `conda run conda-lint /tmp/abs_4epo3zutqx/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:17: avoid_noarch: noarch: python packages should be avoided
===== Final Report: =====
0 Errors and 1 Warning were found


Fix global links generated by url() function

Urls and links, generated by url function with appear to be broken:

I've caught it not properly apply the path alias on nodes (on global / translated titles)

It often makes duplicate prefixes (like what the referenced bug here is describing)
I've also had problems with it applying the correct language to a URL

This is why for many days now I've been manually building the URL with path_alias and get prefix, because it always works fine without problem. Whereas url() consistently gives me problems and does not behave well with global functionality.
@deadlybutter, https://github.com/DoSomething/phoenix/pull/5808#discussion_r45375334

Examples: #5796, #5474, #5423, #5808
cc @angaither

@angaither is this one we should add to 2.1 milestone?


conda-forge issue

The recipe has been merged, but the feedstock for gliderpy is  empty
https://github.com/conda-forge/gliderpy-feedstock
Not sure what's going on here :/

GitHub had an outage yesterday. We are still "catching up."


Question about the design of pulsar schema

Producer producer = client.newProducer(Schema.AVRO(User.class)).create();

Producer producer = client.newProducer(Schema.generic(schemaInfo)).create();

When a pulsar producer is created, it is binding with a specific schema. May I ask what is the advantage of this design？
In kafka, one Producer is not binding with schema, so it can support message sending with multiple schemas. Is it possible for Pulsar to do that？

Producer producer = client.newProducer(Schema.BYTES).create();

You can use BYTES schema as above. I think the schema mostly designed for pulsar-sql

@MatrixHB Please check the schema documentation https://pulsar.apache.org/docs/en/schema-get-started/#why-use-schema which shows why to use Pulsar Schema.

You can use BYTES schema as above. I think the schema mostly designed for pulsar-sql

It's not only for pulsar-sql, it allows users to send type messages, not only binary stream.

Closed as answered and no response.
Feel free to open a Q&A discussion at https://github.com/apache/pulsar/discussions if it's still relevant for maintained versions.


feat: add max Speed column to drives table

In addition to the max Power column, I would also find it helpful to see the max Speed of past drives.

approved, added this commit via cherry-pick to #4284 so you can merge both changes via #4284 @JakobLichterfeld

Thanks for your suggestion! Merged via #4284


ROCm restructuring

Comit message preview:
Flattened out page structure for improved navigability.

Change Table of Contents
Update the install guides for windows and linux
Removed extraneous index pages
GPU architecture pages duplicate entries removed
spack page cleanup


TokenExpiredError: Who's Responsible for Refreshing the Token

azure-sdk-for-python: v2.0.0rc3
Python v3.5.1 on Windows 10
I got the following exception in long running code.
2016-06-01T16:03:27.482 Token expired or is invalid. Attempting to refresh.
Traceback (most recent call last):
  File "c:\Python35\lib\site-packages\msrest\service_client.py", line 196, in send
    **kwargs)
  File "c:\Python35\lib\site-packages\requests_oauthlib\oauth2_session.py", line 321, in request
    http_method=method, body=data, headers=headers)
  File "c:\Python35\lib\site-packages\oauthlib\oauth2\rfc6749\clients\base.py", line 194, in add_token
    raise TokenExpiredError()
oauthlib.oauth2.rfc6749.errors.TokenExpiredError: (token_expired)

During handling of the above exception, another exception

  File "c:\Python35\lib\site-packages\azure\mgmt\compute\operations\virtual_machine_extension_images_operations.py", line 235, in list_versions
    response = self._client.send(request, header_parameters, **operation_config)
  File "c:\Python35\lib\site-packages\msrest\service_client.py", line 201, in send
    raise_with_traceback(TokenExpiredError, msg, err)
  File "c:\Python35\lib\site-packages\msrest\exceptions.py", line 47, in raise_with_traceback
    raise error.with_traceback(exc_traceback)
  File "c:\Python35\lib\site-packages\msrest\service_client.py", line 196, in send
    **kwargs)
  File "c:\Python35\lib\site-packages\requests_oauthlib\oauth2_session.py", line 321, in request
    http_method=method, body=data, headers=headers)
  File "c:\Python35\lib\site-packages\oauthlib\oauth2\rfc6749\clients\base.py", line 194, in add_token
    raise TokenExpiredError()

It isn't clear to me who owns refreshing the token.  I would have thought the SDK does this automatically and transparently.
Is this the correct way to refresh the token?
creds = ServicePrincipalCredentials(...)
creds.refresh_session()

You should not have to refresh the token yourself, should be automatic. We got some problem with it and there is a fix in the RC4 for the refresh. Could you try the RC4? I know this does not solve the whole problem, but it will be interested to see if it solves yours.
Thank you

@annatisch

Will do.  I did not realize RC4 was out.

Hi @boumenot, did you find upgrading to RC4 resolved this issue? Let me know if you're still getting any errors with regards to token refresh :)

Sorry, I've been OOF since I first reported the issue.  I've pulled RC4, and I'm trying it now.  I'll report back.

I tried to upgrade to 2.0.0rc4 on Windows, and I keep getting the following error.
  File "C:\dev\Compute-Runtime-Tux-Pipeline\dep\azuresdk.py", line 40, in <genexpr>
    storage = next(x for x in self._client.storage_accounts.list() if x.name == name)  # type: StorageAccount
  File "c:\Python35\lib\site-packages\msrest\paging.py", line 64, in __iter__
    for i in self.next():
  File "c:\Python35\lib\site-packages\msrest\paging.py", line 104, in next
    self._response = self._get_next(self.next_link)
  File "c:\Python35\lib\site-packages\azure\mgmt\storage\operations\storage_accounts_operations.py", line 533, in internal_paging
    request, header_parameters, **operation_config)
  File "c:\Python35\lib\site-packages\msrest\service_client.py", line 167, in send
    session = self.creds.signed_session()
AttributeError: 'StorageManagementClientConfiguration' object has no attribute 'signed_session'

I upgraded to 2.0.0rc4 with the following command.
c:\Python35\Scripts\pip.exe install -U azure==2.0.0rc4

When this failed, I tried deleting all pip packages, and reinstalling but the error is the same.
I think the issue is with me, but I am not able to determine what or where it is.

Hi @boumenot
This specific error is because StorageManagementClientConfiguration does not exist anymore in RC4. See the ChangeLog of RC4 for details:
https://github.com/Azure/azure-sdk-for-python/releases/tag/v2.0.0rc4

Thank you!  I like the changes.

I finally have evidence that everything is working well.  I setup up a job to run over the weekend that was continually interacting with Azure.  I can see from the logs that the token was refreshed by the SDK, and no exceptions were raised.  I'm closing the issue.
Thanks!


Lionel Fotso Kamga - DS3- Unit 2- Regression

Great work on regression! This tool will be very valuable in many prediction models. Keep working with regression as well as model diagnosis to gain an intuition for the terminology. We will get more practice feature engineering throughout the course. Keep up the great work !


Unify lattices in C++ under a single template

This has proven difficult/impossible. As a result, adding Anna lattice types requires recompilation, and potentially code changes.

The gist of the problem is that our KVS has an abstract superclass, Lattice<T>. There are many subclasses of Lattice, such as BoolLattice, SetLattice<S>, etc. (which are Lattice<bool>, Lattice<std::unordered_set<S>>, and so on). To write a KVS get method that returns a Lattice*, it seems like you can't have a return type without the type template uninstantiated, which functionally means that you need to have a separate type signature for get for every lattice type, which is potentially large because we want to take advantage of lattice composition.
The only solution I can come up with for this is removing the type template and simply relying on class hierarchies for type checking, but we lose typechecking of compositions.


Failed GAMESS test

Test 41 and 44 fail. Unreasonable SCF energies (does not converge)

Removed Gamess from repo


Error page flashing before redirecting user to /action/sign-up

We're seeing cases on this error occurring due (probably) to this error message.
Even though Sentry suggests otherwise, all of the events are happening in Safari, and the only reproducible way to get to this issue is when going to /profile page when logged out. In those cases the error page flashes before I get redirected to the signup screen.

This issue is related to server side redirects inside pages with suspense boundaries on nextjs, so both /profile and /action/become-member pages are affected. The automatic refresh when the error page is displayed is caused by a bug on next where server side redirects makes 2 redirects instead of only one. Updating Next to the latest 14 release did not fix neither of these issues.
It is possible that upgrading to Next 15 fix the issue. Next team merged a fix to canary related to double redirect. There is also this opened issue about the same problem we are facing.
For now the solution was to make this redirect client side instead of server side.


clean up after closing session.

default cleanup involves stopping the client, wich makes closing the
session redundant. (see #36)

In #36 you wrote:

technically, the same bug exists in the vanilla close function - do you recall a specific reason why it was implemented in that order back in the day?

I've just looked at old patches that introduced it I don't see any specific reason behind that ordering.
Thanks for patching it! I'll push a new release.


BEAM-1310 Add integration tests for JdbcIO

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[X] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[X] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[X] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[X] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

Hi @tgroh  can you please take a look?
This adds an integration test & data loading script for Jdbc IO
You'll note there are TODOs for verifying the contents of the rows in the tests - I will get to those, but wanted to get a first, useful test in so that we can continue working on the end to end integration test infrastructure.
This IT demonstrates:

Having a separate load script for read tests
Using TestPipelineOptions that are not test based, but rather data source based.
Writing kubernetes scripts for instantiating an instance of the data source.

per discussion - cancelling this particular PR since it contains merges


Adding docs related to release process

/assign @deepak-vij
Adding docs to explain the coordinated release management between Poseidon and Firmament.

/lgtm

/ok-to-test


Update background color of the top area of the timetable screen.

Idea Description
Please update background color of timetable screen so that we can align we figma.
Reference images and links

🙋 I'd like to try this issue!

Thanks! Assigned you!

@upon0426
Hi!
Please check my PR.


IbcReceiver semantic issue

I think there's potential for confusion when using the IbcReceiver etc. naming, which might indicate that it's related only to recvPacket part of the packet lifecycle
This is especially problematic in the case of an 'asymmetric' dApp, i.e. on 1 chain the IBC app is only meant to send (and ack/timeout) packets, not receive them. Yet it would still implement the IbcReceiver interface...
I propose instead using something like IbcCallbacks / IbcCallbacksContract / IbcCbContract  with sub-interfaces for packet and channel callbacks where required, e.g. IbcPacketCallbacksContract

New version solves this


Output of marts vs staging

Hey there,
I just found your package and it is awesome.
I want to materialize the output of the models on different schemas / datasets, I have this configuration but when I ran the model in my PROD deployment, everything gets written in PROD instead of PROD & STAGING. What am I doing wrong?
models:
  my_dbt_project:
    reports:
      +materialized: view
    ga4:
      +tags: "googleanalytics"
      marts:
        +schema: prod
      staging:
        +schema: staging

vars:
  ga4:
    project: "my_bq_project"
    dataset: "my_bq_dataset"
    start_date: "20230701"
    static_incremental_days: 3

You've got GA4 nested under your project. It should be equal with the project.
models:
  my_dbt_project:
    reports:
      +materialized: view
  ga4:
    +tags: "googleanalytics"
    marts:
      +schema: prod
    staging:
      +schema: staging

That doesn't explain why the prod schema got applied to everything. It should be that nothing gets applied, but start with that.

Thank you so much @dgitis, that made the trick!


wildcard permissions in the permissions.yml down't work

Description
With using of LuckPerms I can't use wildcards in the permissions.yml. Normal permissions are usable.

The LuckPerms config option apply-wildcards: is activated.
Tested with storage type H2 and mysql.
I tested the LuckPerms config option enable-ops:, but the options haven't an impact of this behavior.

It this as intended?
In difference, with PEX for example I can use wildcards in the permissions.yml.
Reproduction Steps
Permissions.yml:
demo:
       description: test
       default: false
       children:
          fawe.*: true
          worldedit.*: true
          plots.*: true

-> /lp user <user> permission set demo
Expected Behaviour
The expected behavior in this case is, that the user with the "demo" permission have the permissions fawe.*, worldedit.* and plots.* too.
Server Details
git-Paper-259 (MC: 1.17.1)
LuckPerms Version
tested with 5.3.74 and 5.3.61
Logs and Configs
No response
Extra Details
I need the permissions.yml to get new, often used permission groups. For example, FAWE have hundreds of permissions, and a part of it I would like to collect before I define the LuckPerms permissions.

I have a quick question - why do you rely on the permissions.yml rather than LuckPerms' groups feature to set groups of permissions? Surely they do the same thing and the bonus of using LuckPerms is that it's already integrated into your existing permissions setup?

Yea, use luckperms to set permissions, not in the permissions.yml file.


Suggestion on additional configuration

May I suggest some additional config options? And in case there is an existing workaround, please do let me know.

The option to not actually straight up install said recently built package, just move it to the repository and update the database, as its not always you want that specific package installed in the current build system.
I particularly fail to see the point of showing us the listed files, as opposed to for example preview the make file, in case you want change a setting or so, in any case I'd suggest making that preview optional, since its a unneeded interaction.

Hi there 👋

Are you sure this actually happens? In any case, build in chroot and aurutils will not touch your local pacman at all (aur sync -c). Besides, you'll get all the benefits of building in chroot!

It is very important to review PKGBUILD (or make a conscious decision not to review it this time), so there won't be an option to disable this. But if you want an actionable review, install vifm — if vifm is present, you will be able to preview and edit all the files on the spot.

Honestly I've been using chroot version since the beginning, so I don't remember how the non-chroot one behaves. You don't need to worry about additional setup, aurutils will prepare everything — in fact, with chroot builds you can solve the root permission timeouts completely, see this section in man aur-build.

What you get is the contents of repo on AUR, which includes PKGBUILD + all extra files (.install files, maybe patches, some people even put some scripts there). When you don't have vifm, aurutils indeed shows you just the list of files so in a new terminal you can check them all for malicious contents. If I remember correctly, you can use other tools instead of vifm such as ranger, check this issue for details: https://github.com/AladW/aurutils/issues/51

While those seem reasonable, what I'm suggesting is slightly different, with regards with sudoers work around, since what I was looking is for the package not to be installed, as opposed to being installed automatically.
With regards for the vifm, I still think its overkill for most use cases, may I suggest that in case a file manager is not available (AUR_PAGER as it was mentioned in that thread), it could use just a plain regular EDITOR solely for PKGBUILD. Because from what I understand everything branches off that build script and usually that's the most important thing users should be checking anyway (and making alterations if they need to). That is of course only if the other methods are not available, thus why I'm suggesting as an optional configuration.

Packages are only installed when they are required as (make)dependencies for the build process. By default, they are removed after the build finishes (both for aur-build and aur-sync.)

Regarding EDITOR, that implies annoying the user with potentially hundreds of prompts (one per PKGBUILD or other important files like .install). Pacman wrappers may be satisfied with such an approach but it's not something I want to do.

Some closing remarks: vim works as AUR_PAGER too, using netrw. Same for emacs using dired. Other editors are a waste of time so that covers everything. :slightly_smiling_face:
aur-build also has --nosync (implied by --chroot) if you want to perform no pacman operations.


Feat/ubuntu to centos7

I have migrated the Docker to Centos7
Added a Makefile
Added a .dockerignore
Please review and let me know if there any issues. It should be compatible with the mesos system.
I´m pushing this image to the tag centos7-to-ocp on quay.io (https://quay.io/repository/3scale/echoapi?tab=tags)
P.s: I decided to recreate the branch with the changes to create a cleaner PR.
The comments of the original PR can be found here: https://github.com/3scale/echo-api/pull/15

All the corrections are done on the commits: d8e763e, f69e565,  437d8a1, 088f43f.
Please review it when you can

Carrections done on 	7b34c5d

All the corrections are done on 7b34c5d

please rebase with master and squash before merge.
Good job! 👍

Everything should be good to merge 😃

@maneta go for it.


M> Research Group Reviews

I've been contacted by CHAINROCK which is a digital asset research and investment group, focused on blockchain technology and cryptoassets. They've released an analysis of our project cited here : https://chainrock.io/rchain/
I believe this is our first research group review and I find the analysis fair and valid with criticisms similar to those that have been conveyed to me in other social channels.
I also believe there is opportunity for us to clear the air with regards to some of the concerns addressed in the report. This could also help instill further confidence in our project within the larger Crypto community which could drive membership and future platform adoption.
With another ICO planned after the Mercury platform release, I believe we could set up an issue with rewards for Research groups to give a fair and objective review to build the RChain brand and get our name out there.
Chainrock is interested in setting up an interview with a member of the team to address some concerns. Would anyone be interested in taking this on?

@Mervyn853 this is great!
It would need to be someone who could address the token supply terms directly.

@Mervyn853 hi, please direct these to me. I'll find someone or answer it myself. Let's connect offline to follow up. Thank you!

I think it deserves to be open.

@Mervyn853 Has anyone been able to talk to chainrock? When I last read the review I couldn't find any definitive concern about the technology except a subjective one. If there are concerns concerning the tech we can get to set up an interview with Nash foster.

Yes. Me and Lawrence are in contact with Don. Will work to schedule something since Nash has volunteered to take on interviews.

Is Don from chainrock?

Yes @Ojimadu . Also, I might have more reviews coming in, will update again.

@Mervyn853 I'll close this issue for the sake of trimming our marketing issue list, and because it seems like the time has come and past for this (right?).
Feel free to reopen if you still deem it relevant!


CONTRIBUTING.md: Doc references non-existent CONTRIBUTORS file

Hello,
CONTRIBUTING.md has the Copyright headers section which references a CONTRIBUTORS document that does not exist. The section also describes that the AUTHORS file will populate commit authors to the file periodically which does not seem to be the case.
https://github.com/cue-lang/cue/blob/4d2ba88849b660e30df921aa1d6f44d4c75ed8d8/CONTRIBUTING.md#copyright-headers
We should clarify what we want to do with these files and update CONTRIBUTING.md to reflect this.

Thanks for spotting and reporting! Sending a patch shortly to clean this up. It took us a while as we weren't exactly sure how we wanted to track contributors going forward.


Omit Empty translations CSV2Any

Hi is there any way to omit empty translations during import?
Example:
labels.example,Example in Default lang, Other Lang,,,,,,,,,,,
And to prevent for non translated languages yet:
in iOS:
"labels.example" = "";
and Android:
<string name="labels.example"></string>

Hi,
You cannot omit, but you can replace empty values by a translated languages value, see option: default_lang
https://github.com/netbe/Babelish/blob/137a0f9886d994ce2578a2c7d7c243b609b3f0b4/.babelish.sample#L26

Hi,
You cannot omit, but you can replace empty values by a translated languages value, see option: default_lang
https://github.com/netbe/Babelish/blob/137a0f9886d994ce2578a2c7d7c243b609b3f0b4/.babelish.sample#L26

Thanks for response, I'll check it. Seems like good enough for me.

Thanks for response, I'll check it. Seems like good enough for me.


Bearer only configuration

Hi, Is there any example about how to config a bearer only wcf rest api, please help me.

Hi, i'am also interested in Bearer only configuration with WebAPI.
Can someone give an example?


FORTECH-102: Fixed focus on dropdown activator after selection

The Issue
After making a selection from a dropdown menu, focus would be lost and unrecoverable.
The Solution
Upon closing (hiding) the menu, the focus would remain inside the dropdown list menu popup, and since the list container was set to "self-only", spotlight wasn't allowed to leave that container. However, the container was only hidden, not destroyed, so the normal flow couldn't happen, causing Spotlight to just be stuck in an invisible box, like a real-life mime.
From @germboy:

I added a Spotlight.focus()  bit in the dropdown transition component’s onHide method that will set focus back to the dropdown button/item when it closes.

I added several other aria props and ID from Sandstone to help with accessibility.

I added several other aria props and ID from Sandstone to help with accessibility.

Thanks for supporting the A11y features in Dropdown.
But we might not need the ariaLabelledBy prop.
If we have to support it, then the prop name could be aria-labelledby.
But there is no title in the Dropdown, so I think, we could remove it.


Add Canada Calgary 2024 Sustainability weekly meetup to the event Ref Issue 535

Reference Issue

Hi @Marvelrex you edited the file for sustainability week 2023! The correct one is this one website/content/en/events/2024-cloud-native-sustainability-week.md, you can cut and paste your changes there and we should be good to go!
Also, I just noticed one of the checks failed and is about the commit signature. Can you please sign your next commit? Here's the GH guide, if you're not familiar with signed commits.


ARROW-8270 : [Python][Flight] Updated Python's flight server example to support TLS

Updated server.py to add support for TLS security

I couldn't push changes to your fork so I opened #6787 instead.

Closing in favour of new PR.


[4.0 RC1 & RC2] TouchScreenButton BUG

Godot version
4.0 RC1 & RC2
System information
Linux Ubuntu
Issue description
TouchScreenButtons have stopped working in version RC1 and RC2.
The same single project working in Beta 17, fails in RC1 and RC2.
Steps to reproduce
We just run the project and clicking on the TouchScreenButton doesn't work.
Minimal reproduction project
Attached very simple project where the failure occurs.
touch-test.zip

Duplicate of https://github.com/godotengine/godot/issues/73457, fixed by https://github.com/godotengine/godot/pull/73470.


readme updated, BridgeCrew compliance fixes

what

BridgeCrew compliance checks fix
readme updated
default behaviour changed: Public IP for EC2 instance disabled by default
default behaviour changed: Instance Metadata Service Version 2 forced by default
default behaviour changed: Encryption of the root block device enabled by default
default behaviour changed: Encrypt EBS Volume enabled by default

why

To be able to position our modules as standards compliant
stay in sync with code
To comply BridgeCrew check

references

https://docs.bridgecrew.io/docs/public_12
https://docs.bridgecrew.io/docs/general_13
https://docs.bridgecrew.io/docs/bc_aws_general_31
https://docs.bridgecrew.io/docs/general_3-encrypt-eps-volume

/test all

/test all

/test all

/test all


contrib/pyopenssl should be wrapping SysCallError

https://github.com/kennethreitz/requests/issues/2852
Traceback (most recent call last):
  File "..\requests-2.8.1-py2.7.egg\requests\api.py", line 122, in put
    return request('put', url, data=data, **kwargs)
  File "..\requests-2.8.1-py2.7.egg\requests\api.py", line 50, in request
    response = session.request(method=method, url=url, **kwargs)
  File "..\requests-2.8.1-py2.7.egg\requests\sessions.py", line 468, in request
    resp = self.send(prep, **send_kwargs)
  File "..\requests-2.8.1-py2.7.egg\requests\sessions.py", line 576, in send
    r = adapter.send(request, **kwargs)
  File "..\requests-2.8.1-py2.7.egg\requests\adapters.py", line 370, in send
    timeout=timeout
  File "..\requests-2.8.1-py2.7.egg\requests\packages\urllib3\connectionpool.py", line 559, in urlopen
    body=body, headers=headers)
  File "..\requests-2.8.1-py2.7.egg\requests\packages\urllib3\connectionpool.py", line 376, in _make_request
    httplib_response = conn.getresponse(buffering=True)
  File "C:\Python27\ArcGIS10.3\lib\httplib.py", line 1067, in getresponse
    response.begin()
  File "C:\Python27\ArcGIS10.3\lib\httplib.py", line 409, in begin
    version, status, reason = self._read_status()
  File "C:\Python27\ArcGIS10.3\lib\httplib.py", line 365, in _read_status
    line = self.fp.readline(_MAXLINE + 1)
  File "C:\Python27\ArcGIS10.3\lib\socket.py", line 476, in readline
    data = self._sock.recv(self._rbufsize)
  File "..\requests-2.8.1-py2.7.egg\requests\packages\urllib3\contrib\pyopenssl.py", line 179, in recv
    data = self.connection.recv(*args, **kwargs)
  File "..\OpenSSL\SSL.py", line 1320, in recv
    self._raise_ssl_error(self._ssl, result)
  File "..\OpenSSL\SSL.py", line 1178, in _raise_ssl_error
    raise SysCallError(errno, errorcode.get(errno))
SysCallError: (10054, 'WSAECONNRESET')

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/27786406-contrib-pyopenssl-should-be-wrapping-syscallerror?utm_campaign=plugin&utm_content=tracker%2F192525&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F192525&utm_medium=issues&utm_source=github).

So, specifically the problem is SysCallError in our recv wrapper.

Mmm, what should it be wrapped with?

Good question. I think socket.error?

@steveoh Would you be interested in making this change?

I would need some hand holding. I was trying to make an https request with requests and have no knowledge of this lib.

Here's the line in question: https://github.com/shazow/urllib3/blob/master/urllib3/contrib/pyopenssl.py#L178
Give it a shot, send a PR and we'll give you feedback. :)
@Lukasa Worth wrapping all the raises while we're here?

@shazow Maybe? Might be worth considering that a slightly lower priority task than this one, and then put it on a backlog of things for me to deal with at some point.


NCCL graph and topology incompatible with A100

I'm using the ubuntu-hpc 2204 x64 Gen 2 image on a Standard NC24ads A100 v4 VM.
I train a vLLM model that uses NCCL and observe the following error:

Error
::16674:16674 [0] NCCL INFO Bootstrap : Using eth0:<IP_ADDRESS><0>
::16674:16674 [0] NCCL INFO NET/Plugin : Plugin load (libnccl-net.so) returned 2 : libnccl-net.so: cannot open shared object file: No such file or directory
::16674:16674 [0] NCCL INFO NET/Plugin : No plugin found, using internal implementation
::16674:16674 [0] NCCL INFO cudaDriverVersion 12020
NCCL version 2.18.6+cuda11.8
::16674:17023 [0] NCCL INFO NET/IB : Using [0]mlx5_an0:1/RoCE [RO]; OOB eth0:<IP_ADDRESS><0>
::16674:17023 [0] NCCL INFO Using network IB
::16674:17023 [0] NCCL INFO comm 0x5640f24752d0 rank 0 nranks 1 cudaDev 0 nvmlDev 0 busId 100000 commId 0x2064c00fc2f91516 - Init START
::16674:17023 [0] NCCL INFO NCCL_TOPO_FILE set by environment to /opt/microsoft/ncv4/topo.xml
::16674:17023 [0] NCCL INFO Setting affinity for GPU 0 to ffffff
::16674:17023 [0] NCCL INFO NCCL_GRAPH_FILE set by environment to /opt/microsoft/ncv4/graph.xml
::16674:17023 [0] graph/search.cc:703 NCCL WARN XML Import Channel : dev 1 not found.
::16674:17023 [0] NCCL INFO graph/search.cc:733 -> 2
::16674:17023 [0] NCCL INFO graph/search.cc:740 -> 2
::16674:17023 [0] NCCL INFO graph/search.cc:840 -> 2
::16674:17023 [0] NCCL INFO init.cc:880 -> 2
::16674:17023 [0] NCCL INFO init.cc:1358 -> 2
::16674:17023 [0] NCCL INFO group.cc:65 -> 2 [Async thread]
::16674:16674 [0] NCCL INFO group.cc:406 -> 2
::16674:16674 [0] NCCL INFO group.cc:96 -> 2
Traceback (most recent call last):
...
self.llm_engine = LLMEngine.from_engine_args(engine_args)

File "/home/azureuser/miniconda3/envs/snomedct/lib/python3.10/site-packages/vllm/engine/llm_engine.py", line 356, in from_engine_args
engine = cls(*engine_configs,
File "/home/azureuser/miniconda3/envs/snomedct/lib/python3.10/site-packages/vllm/engine/llm_engine.py", line 111, in init
self._init_workers()
File "/home/azureuser/miniconda3/envs/snomedct/lib/python3.10/site-packages/vllm/engine/llm_engine.py", line 151, in _init_workers
self._run_workers("init_model")
File "/home/azureuser/miniconda3/envs/snomedct/lib/python3.10/site-packages/vllm/engine/llm_engine.py", line 983, in _run_workers
driver_worker_output = getattr(self.driver_worker,
File "/home/azureuser/miniconda3/envs/snomedct/lib/python3.10/site-packages/vllm/worker/worker.py", line 84, in init_model
init_distributed_environment(self.parallel_config, self.rank,
File "/home/azureuser/miniconda3/envs/snomedct/lib/python3.10/site-packages/vllm/worker/worker.py", line 253, in init_distributed_environment
torch.distributed.all_reduce(torch.zeros(1).cuda())
File "/home/azureuser/miniconda3/envs/snomedct/lib/python3.10/site-packages/torch/distributed/c10d_logger.py", line 47, in wrapper
return func(*args, **kwargs)
File "/home/azureuser/miniconda3/envs/snomedct/lib/python3.10/site-packages/torch/distributed/distributed_c10d.py", line 2050, in all_reduce
work = group.allreduce([tensor], opts)
torch.distributed.DistBackendError: NCCL error in: /opt/conda/conda-bld/pytorch_1702400366987/work/torch/csrc/distributed/c10d/ProcessGroupNCCL.cpp:1333, unhandled system error (run with NCCL_DEBUG=INFO for details), NCCL version 2.18.6
ncclSystemError: System call (e.g. socket, malloc) or external library call failed or device error.
Last error:
XML Import Channel : dev 1 not found.
::16674:16674 [0] NCCL INFO comm 0x5640f24752d0 rank 0 nranks 1 cudaDev 0 busId 100000 - Abort COMPLETE

This is a single GPU machine, but /opt/microsoft/ncv4/graph.xml and topology.xml reference 4 GPUs. If I update them to refer to a single GPU, everything works.

graph.xml
<graphs version="1">
  <graph id="0" pattern="4" crossnic="0" nchannels="2" speedintra="12" speedinter="12" latencyinter="0" typeintra="SYS" typeinter="PIX" samechannels="0">
    <channel>
      <gpu dev="0"/>
    </channel>
    <channel>
      <gpu dev="0"/>
    </channel>
  </graph>
  <graph id="1" pattern="1" crossnic="0" nchannels="4" speedintra="12" speedinter="12" latencyinter="0" typeintra="SYS" typeinter="PIX" samechannels="0">
    <channel>
      <gpu dev="0"/>
    </channel>
    <channel>
      <gpu dev="0"/>
    </channel>
    <channel>
      <gpu dev="0"/>
    </channel>
    <channel>
      <gpu dev="0"/>
    </channel>
  </graph>
  <graph id="2" pattern="3" crossnic="0" nchannels="4" speedintra="12" speedinter="12" latencyinter="0" typeintra="SYS" typeinter="PIX" samechannels="0">
    <channel>
      <gpu dev="0"/>
    </channel>
    <channel>
      <gpu dev="0"/>
    </channel>
    <channel>
      <gpu dev="0"/>
    </channel>
    <channel>
      <gpu dev="0"/>
    </channel>
  </graph>
</graphs>

topology.xml
<system version="1">
  <cpu numaid="0" affinity="00000000,00000000,00ffffff" arch="x86_64" vendor="AuthenticAMD" familyid="175" modelid="1">
    <pci busid="0001:00:00.0" class="0x030200" vendor="0x10de" device="0x20b5" subsystem_vendor="0x10de" subsystem_device="0x1533" link_speed="" link_width="0">
      <gpu dev="0" sm="80" rank="0" gdr="1">
        <nvlink target="0002:00:00.0" count="12" tclass="0x030200"/>
      </gpu>
    </pci>
    <nic>
      <net name="eth0" dev="0" speed="100000" port="0" latency="0.000000" guid="0x0" maxconn="65536" gdr="0"/>
    </nic>
  </cpu>
</system>

Looking into it.

I found the same issue and fix with Standard_NC48ads_A100_v4 with these images.

Yes, the topo/graph files are not needed for the smaller NCv4 VM sizes. Next VM image release will have this not loaded automatically (in /etc/nccl.conf) for these vm sizes. Until then, please delete the reference to topo/graph files and it should be good.


Add com.snowplowanalytics.snowplow.badrows/bq_loader_runtime_error/jsonschema/1-0-0

Closed in favor of https://github.com/snowplow/iglu-central/issues/985


Implement CheckChecker method class

need piece.movegenerate to test checkchecker

I have rebased CheckChecker. It is best git pull --force the checkchecker branch, if you've yet to write code. If you have already written some code, it may be very challenging to resolve the conflict.

Changes incorporated in #58


Re-adding the ability to read splunk config from hubble.d

Somewhere along the way to 3.0.1 build, hubble lost its ability to pickup splunk configuration dynmaically.
As a result, hubble needs to be restarted if we intend to update the splunk config.
Some of our functionalities are dependent of this feature.
Making some changes in this PR to add that functionality back to hubble.
In this PR, I am recreating the file which was deleted in https://github.com/hubblestack/hubble/pull/580
Moreover, I am changing the hec/opt.py file so that first it looks in grains,get for splunk config ( and then fallback to config.get ),

Opening this PR in 3.0. Closing here.


[FEATURE] Test Students Guide

Universities Monday 2021/05/31 / 2021-05-31
3.6 Misho - Illinois Client App (UIUC)
3.6.1 Add a button to debug that let's us paste the JSON as we do in Admin
3.6.2 If there is a structure, add a button to browse for "Info Content" and then a new panel when clicked that shows the JSON as per the UI design. (I think all panels except detail would be Flutter).

Wrong repo - closing.


Add support for OpenAPI v3

Not Ready For Review
This PR will add an OpenAPI v3 Parser to API-Flow.
@philsturgeon It's being worked on!

Awesome! Can I help at all? I'll of course test things when it's considered readyish.

@philsturgeon If you have shareable OpenAPI v3 files lying around, I'd love to have them as test-case for the e2e testing. Otherwise, if you're up for it, I'm looking to massively upgrade dependencies while trying to not break too many things.
As for your install issue, I believe the issue you're having with the installation stems from having used the npm package. It's outdated iirc, and I don't have the rights to publish on it anymore.
Here's the updated install step:
git clone git@github.com:JonathanMontane/API-Flow.git
cd API-Flow
nvm use
yarn install

then you can make your runners. for instance, creating the node runner would be done like so:
make runners TARGET="node"
# you can create multiple targets in the same command
# make runners TARGET="node web webworker"

once you've created the node lib, you can use it like this:
const ApiFlow = require('./dist/node/api-flow.js').default; // the .default is due to es6 modules structure.

const options = {
    source: {
        format: 'swagger',
        version: 'v2.0'
    },
    target: {
        format: 'raml',
        version: 'v1.0'
    }
}

const promise = ApiFlow.transform({
  options,
  // this also works with file uri like: file:///some/path, or simply /some/path
  // check out https://apis.guru/browse-apis/ -- it's amazing
  uri: 'https://api.apis.guru/v2/specs/6-dot-authentiqio.appspot.com/6/swagger.json'
})

promise.then((data) => {
  console.log(data)
})

I just did all the steps, and it worked, so it should work for you too.
Note For some reason I had an Error: define cannot be used indirect bug when I did an npm install instead of a yarn install (with make runners generating warnings about /~/lib/ajv or something), so I'd recommend using yarn, although I'm not too sure how that got resolved.

@JonathanMontane thanks for the mention of APIs.guru (I'm the current maintainer). You may not know that there is a static branch full of OpenAPI v3 conversions at https://github.com/APIs-guru/openapi-directory/tree/openapi3.0.0/APIs

Would be amazing to have this support!

Moving to Postman until this is done


Requirements: Need a story for optional and repeated syntax pieces

https://code.world/#PnQkjrztp45GAom-7ardWUA fails to check, despite having two declarations that appear, at first glance, to match the template.  At first, I thought this was a failure to consider non-toplevel declarations.  But it's quite the opposite!  The non-toplevel declaration of b matches.  It's the declaration of a that fails to match!  The reason: the template has no where clause, but the definition of a does have a where clause.  Therefore, it's not a match.
This is surprising to me, but I'm not sure how to specify exactly the behavior that I do want.

Broadening the scope of this bug.
In the matcher, there are plenty of places where optional or repeated parts of syntax behave in counterintuitive ways.  These include:

Multiple equations in a function definition.
where clauses on functions
arguments to a function, where we may want to allow any number.
Optional clauses in an import

One option is to default to matching when the template is any subset of these optional items.  This relies on a negated form, so that it remains possible to say "and not with that extra thing..."


add EOL for 1.26

What this PR does / why we need it:
This adds the EOL date and also updates our scripting for yq4 (hooray, we arrived in 2021!)
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:

/hold
@xmudrii says the date is wrong.

/hold cancel
just to unblock the periodic job


Improve tests by reducing EVAL blocks, refactoring & merging test files

This commit:

Merges contents of file '11-non-breaking-space.t' into file '12-non-breaking-space.t'

Removes file '11-non-breaking-space.t' from version control

Changes the EVALed code in test 12-non-breaking-space.t to a normal test

Removes subroutines used by:
S26-documentation/12-non-breaking-space.t
S32-str/space-chars.t
to a new module:
packages/Test-Helpers/lib/Test/Misc.pm6

Files affected:
S26-documentation/11-non-breaking-space.t
S26-documentation/12-non-breaking-space.t
S32-str/space-chars.t
packages/Test-Helpers/lib/Test/Misc.pm6

@ugexe just said on IRC #raku-dev he was happy with last changes.


chore: ⏫ upgrade tun2socks library to v3.1.0

Codecov Report
Base: 45.91% // Head: 45.91% // No change to project coverage :thumbsup:

Coverage data is based on head (7a0e816) compared to base (2950ed3).
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1576   +/-   ##
=======================================
  Coverage   45.91%   45.91%
=======================================
  Files          40       40
  Lines        1934     1934
  Branches      308      308
=======================================
  Hits          888      888
  Misses       1046     1046

Flag
Coverage Δ

unittests
45.91% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Restore docs/reference-1.5.1.html

docs/reference-1.5.1.html was a redirect to docs/reference-1.5.0.html
The latter war removed in a6b5ca192acb47ae0c8dfd8e1d7e6e4cde78f841.
This PR restores the deleted docs/reference-1.5.0.html as docs/reference-1.5.1.html

Oops, nicely spotted — thanks!


Bug: Last 30 Days report on Guardium Dashboard should only show Guardium generated items

Summary

Configure and run integrations
If vulnerable items are created, they will appear in IBM Guardium > Test Result Dashboard
Currently VIT created by other integrations are appearing in the list.  It is expected that only Guardium VIT will be in the list.

I changed all Guardium reports to "Last 7 Days" and changed the IBM Guardium > Preferences > Test Result Import > Number of days to 7 days to match the reports.
I also changed the Vulnerable Item report to show only Guardium entries created within the last 7 days.

Fix is in Update Set v1.2.17


Escape pagination

Review @gchtr's changes on pagination

Coverage decreased (-1.4%) to 92.941% when pulling ca231f5c6d2649e8582f6770cf28219f048652b3 on escape-pagination into 795f78b7e2004308c0d30978d204149b97eaaa56 on master.

Coverage decreased (-1.4%) to 92.941% when pulling ca231f5c6d2649e8582f6770cf28219f048652b3 on escape-pagination into 795f78b7e2004308c0d30978d204149b97eaaa56 on master.

Coverage decreased (-1.4%) to 92.941% when pulling ca231f5c6d2649e8582f6770cf28219f048652b3 on escape-pagination into 795f78b7e2004308c0d30978d204149b97eaaa56 on master.

Lookin' good @gchtr. I ran your tests against master and confirmed that this resolves the issues. This should close out #1642. I'm going to update the readme, etc. so we can push to Packagist and WP.org ASAP


My Database password has been changed, how to update it in your script?

I changed my database password and want to know how can i change it for your script.

Have your looked inside your configuration file (config.ini) ?

May I ask you to please "star" the repository if you like the software ?


feat: hide head sort buttons/head cell menu buttons when they are not needed

should hide the head sort button/head menu button when constraints.active is true in edit mode
should hide the head sort button/head menu button when clicking a header cell and cells are selected

Not what we want to do. We want to hide the sort icon and hide the ... menu button. We still want to keep the label button, disabling it somehow


pianobar-windows: Add version 2022.10.09

Closes #11022

[x] I have read the Contributing Guide.

/verify


[release-2.5] Updated go to 1.18

Related issue: https://github.com/stolostron/backlog/issues/24211
Signed-off-by: dislbenn<EMAIL_ADDRESS>

Based upon earlier discussions, we will close this PR and keep the go version to go1.17. If there are problems with any go packages, we will upgrade to go1.18; however, the downstream builds are already using go1.18. The next step will be to prioritize upgrading the current release to go1.19.


Flutter Doctor advises invalid dart pub cache clear command on first run

Is there an existing issue for this?

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

Steps to reproduce

Perform fresh Flutter SDK installation (issue only happens upon first run)
Run flutter doctor

Expected results
When pre-existing pub cache is found, flutter tool should advise to run dart pub cache clean
Actual results
Running dart pub cache clear is advised instead (clear instead of clean).
Code sample
Code sample
Not applicable

Screenshots or Video
No response
Logs
Logs
> flutter --verbose doctor
Found an existing Pub cache at C:\Users\<username>\AppData\Local\Pub\Cache.
It can be repaired by running `dart pub cache repair`.
It can be reset by running `dart pub cache clear`.
[✓] Flutter (Channel stable, 3.10.4, on Microsoft Windows [Version 10.0.19045.2965], locale cs-CZ)
    • Flutter version 3.10.4 on channel stable at C:\tools\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 682aa387cf (6 days ago), 2023-06-05 18:04:56 -0500
    • Engine revision 2a3401c9bb
    • Dart version 3.0.3
    • DevTools version 2.23.1

[✓] Windows Version (Installed version of Windows is version 10 or higher)

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0-rc4)
    • Android SDK at C:\Users\<username>\AppData\Local\Android\sdk
    • Platform android-33, build-tools 34.0.0-rc4
    • Java binary at: C:\Users\<username>\AppData\Local\JetBrains\Toolbox\apps\AndroidStudio\ch-0\222.4459.24.2221.10121639\jbr\bin\java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-9586694)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[✓] Visual Studio - develop for Windows (Visual Studio Community 2019 16.11.23)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.11.33214.272
    • Windows 10 SDK version 10.0.19041.0

[✓] Android Studio (version 2022.2)
    • Android Studio at C:\Users\<username>\AppData\Local\JetBrains\Toolbox\apps\AndroidStudio\ch-0\222.4459.24.2221.10121639
    • Flutter plugin version 74.0.2
    • Dart plugin version 222.4582
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-9586694)

[✓] VS Code, 64-bit edition (version 1.79.0)
    • VS Code at C:\Program Files\Microsoft VS Code
    • Flutter extension version 3.40.0

[✓] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.19045.2965]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 114.0.5735.110
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 114.0.1823.43

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

• No issues found!
> dart pub cache clear
Could not find a subcommand named "clear" for "dart pub cache".

Did you mean one of these?
  clean

Usage: dart pub cache [arguments...]
-h, --help    Print this usage information.

Available subcommands:
  add      Install a package.
  clean    Clears the global PUB_CACHE.
  repair   Reinstall cached packages.

Run "dart help" to see global options.
See https://dart.dev/tools/pub/cmd/pub-cache for detailed documentation.
>

Flutter Doctor output
Doctor output
See above

Looks like this was later filed in https://github.com/flutter/flutter/issues/128663, and that issue has active PRs associated with it, so I will close this issue in favor of that one.


Allow DBStorageClass to be set from Storage Cluster CR for Noobaa

Signed-off-by: Kaustav Majumder<EMAIL_ADDRESS>

/assign @umangachapagain

/bugzilla refresh

/retitle Allow DBStorageClass to be set from Storage Cluster CR for Noobaa

For PRs in master, we don't need the BZ number

/ok-to-test


App crashed while permanently deleting a protected folder

Describe the bug
A clear and concise description of what the bug is.
App crashed while permanently deleting a protected folder
To Reproduce
Steps to reproduce the behavior:

Go to Documents
Right-click on the feedback hub folder and hit Shift key + D
App will crash

Expected behavior
A clear and concise description of what you expected to happen.
The app should show a message on why it failed and why you can't delete it.
Desktop (please complete the following information):

OS Version: 20H2 Release Preview
App version: v<IP_ADDRESS>

Log file
Please post the log file here so that we can understand your problem better. You can access it from Settings->About->Open log location.
2020-09-30 14:38:40.2681|ERROR|Files.App|The system cannot find the file specified. (Exception from HRESULT: 0x80070002)|System.IO.FileNotFoundException: The system cannot find the file specified. (Exception from HRESULT: 0x80070002)
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() + 0x21
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task) + 0x70
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task) + 0x38
   at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task) + 0x17
   at Files.Filesystem.ItemViewModel.<UpdateFileOrFolder>d__83.MoveNext() + 0x2ac
   at System.Runtime.InteropServices.McgMarshal.ThrowOnExternalCallFailed(Int32, RuntimeTypeHandle) + 0x21
   at __Interop.ComCallHelpers.Call(__ComObject, RuntimeTypeHandle, Int32) + 0xb8
   at __Interop.ForwardComStubs.Stub_11[TThis](__ComObject, Int32) + 0x24
   at Microsoft.AppCenter.Utils.ApplicationLifecycleHelper.<.ctor>b__17_1(Object, UnhandledErrorDetectedEventArgs) + 0x3d
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() + 0x21
   at Microsoft.AppCenter.Utils.ApplicationLifecycleHelper.<.ctor>b__17_1(Object, UnhandledErrorDetectedEventArgs) + 0x75
   at System.EventHandler`1.Invoke(Object, TEventArgs) + 0x2e
   at __Interop.Intrinsics.HasThisCall__23[TArg0](Object, IntPtr, Object, TArg0) + 0x36
   at Files!<BaseAddress>+0x134bc10

2020-09-30 14:38:40.5581|ERROR|Files.App|The system cannot find the file specified. (Exception from HRESULT: 0x80070002)|System.IO.FileNotFoundException: The system cannot find the file specified. (Exception from HRESULT: 0x80070002)
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() + 0x21
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task) + 0x70
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task) + 0x38
   at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task) + 0x17
   at Files.Filesystem.ItemViewModel.<UpdateFileOrFolder>d__83.MoveNext() + 0x2ac
   at System.Runtime.InteropServices.McgMarshal.ThrowOnExternalCallFailed(Int32, RuntimeTypeHandle) + 0x21
   at __Interop.ComCallHelpers.Call(__ComObject, RuntimeTypeHandle, Int32) + 0xb8
   at __Interop.ForwardComStubs.Stub_11[TThis](__ComObject, Int32) + 0x24
   at Microsoft.AppCenter.Utils.ApplicationLifecycleHelper.<.ctor>b__17_1(Object, UnhandledErrorDetectedEventArgs) + 0x3d
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() + 0x21
   at Microsoft.AppCenter.Utils.ApplicationLifecycleHelper.<.ctor>b__17_1(Object, UnhandledErrorDetectedEventArgs) + 0x75
   at System.EventHandler`1.Invoke(Object, TEventArgs) + 0x2e
   at __Interop.Intrinsics.HasThisCall__23[TArg0](Object, IntPtr, Object, TArg0) + 0x36
   at Files!<BaseAddress>+0x134bc10

2020-09-30 14:38:40.6541|ERROR|Files.App|The system cannot find the file specified. (Exception from HRESULT: 0x80070002)|System.IO.FileNotFoundException: The system cannot find the file specified. (Exception from HRESULT: 0x80070002)
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() + 0x21
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task) + 0x70
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task) + 0x38
   at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task) + 0x17
   at Files.Filesystem.ItemViewModel.<UpdateFileOrFolder>d__83.MoveNext() + 0x2ac
   at System.Runtime.InteropServices.McgMarshal.ThrowOnExternalCallFailed(Int32, RuntimeTypeHandle) + 0x21
   at __Interop.ComCallHelpers.Call(__ComObject, RuntimeTypeHandle, Int32) + 0xb8
   at __Interop.ForwardComStubs.Stub_11[TThis](__ComObject, Int32) + 0x24
   at Microsoft.AppCenter.Utils.ApplicationLifecycleHelper.<.ctor>b__17_1(Object, UnhandledErrorDetectedEventArgs) + 0x3d
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() + 0x21
   at Microsoft.AppCenter.Utils.ApplicationLifecycleHelper.<.ctor>b__17_1(Object, UnhandledErrorDetectedEventArgs) + 0x75
   at System.EventHandler`1.Invoke(Object, TEventArgs) + 0x2e
   at __Interop.Intrinsics.HasThisCall__23[TArg0](Object, IntPtr, Object, TArg0) + 0x36
   at Files!<BaseAddress>+0x134bc10

It seems like it tries to delete a file again and again. In this case, it should simply ignore and wait for deletion to complete.

@Jaiganeshkumaran Is this still an issue in v1.5?


Failure to find io.confluent:kafka-avro-serializer:jar

It's good to remove this dependency and move this to test repo along with Code/Test.
Because this is not inside maven realm. Let's have a thought !

Failure to find io.confluent:kafka-avro-serializer:jar:5.1.0

:::Note::: There is a work around to this. But the point it how we can we keep it straight!

Can't we keep it optional to the users not-using AVRO. Avro users can add this explicitly
$
...

[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 0.994 s
[INFO] Finished at: 2019-02-04T14:07:20+00:00
[INFO] Final Memory: 9M/245M
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal on project test-sandbox: Could not resolve dependencies for
project org.jsmart:test-sandbox:jar:0.0.1-SNAPSHOT:

Failure to find io.confluent:kafka-avro-serializer:jar:5.1.0 in
https://artefactrepository.<mybank>.co.uk/repository/public/ was cached in the local repository,
 resolution will not be
reattempted until the update interval of nexus has elapsed or updates are forced -> [Help 1]
[ERROR]
http://cwiki.apache.org/confluence/display/MAVEN/DependencyResolutionException
$

Linking to simillar Issue https://github.com/authorjapps/zerocode/issues/196.

Can't we keep it optional to the users not-using AVRO. Avro users can add this explicitly

Yes, it will be bit painful from Framework-Development point of view.

Version 1.3.1 has been released with the fix.

Confluent Avro has been removed from the POMs.
Sill allows to add explicitly and test (explained in Hello Kafka Testing repo)


Fix Tutorial

Fixed by adding a revision date. This makes it load noticeably slower, but is probably the best option for longevity. Maybe wikipedia will start caching it for us, or we should build our own caching layer.

I tried this out and it was definitely much slower to load.


added ability to add preloaded fonts, added example of preloaded font

Awesome !
It would be even better if you could update your example's text to something explaining the purpose of the example. I mean the 3D text in the generated panel, which is currently "This library supports line-break-friendly-characters...".
Something like "This example shows how to use pre-loaded font files.", or anything else ?
It's really useful, nice from you to add this feature.

related issue : #42

@felixmariotto updated panel text to:
This example shows how to use pre-loaded font files
You can preload font or font and texture, and add it to FontLibrary!
After that, all added text of this font will be displayed with no loading delays!

OK, it's merged and published to NPM ( version 4.4.0 ).
Thanks again !


Review use of mark_safe when building HTML code in template tags

          After reading https://docs.djangoproject.com/en/3.2/ref/utils/#django.utils.html.format_html I think we should review our use of `mark_safe`, as `format_html` may be more appropriate.

Originally posted by @pbanaszkiewicz in https://github.com/carpentries/amy/pull/2553#discussion_r1389697360
In the wake of https://github.com/carpentries/amy/pull/2567 I agree that this is a good idea.

High priority to investigate; can be re-assessed and re-prioritised once we know if/how much work there is to do & if there are any security problems that need to be addressed urgently.


Is there a way to delete orphan when update my model ?

Your Question
Hello,
I looked at the doc and issues on github but couldn't find an answer.
I've got a recipe model :
type Recipe struct {
	gorm.Model
	Title       string        `json:"title"`
	Quantity    []Quantity    `json:"quantity" gorm:"constraint:OnDelete:CASCADE,OnUpdate:CASCADE;"`
}

That has many Quantity children:
type Quantity struct {
	gorm.Model
	RecipeID     uint
	IngredientID uint
	Ingredient   Ingredient `json:"ingredient"`
	Weight       int     `json:"weight"`
}

that itself has many-to-many relationships:
type Ingredient struct {
	gorm.Model
	Name string `json:"name"`
}

To update it, I'm doing
rp.db.Session(&gorm.Session{FullSaveAssociations: true}).Model(&recipe).Where("id = ?", recipe.ID).Updates(&recipe)
It's working well if I'm changing the weight of Quantity, if I add a Quantity element to the array, but not if I'm removing one of them
Example before update:
recipe: {
    ...
    quantity: [
        {
            recipe_id: 1,
           ingredient_id: 1,
           weight: 250
        },
        {
            recipe_id: 1,
            ingredient_id: 2,
            weight: 250
        },
    ]
}

After update, I have the exactly same array, whereas I should have:
recipe: {
    ...
    quantity: [
        {
            recipe_id: 1,
            ingredient_id: 1,
            weight: 250
        }
    ]
}

Does anyone know how I can delete orphan ?

@NazariiMed Unfortunately no, but I'm open to any suggestions or ideas you might have !


Sdk 36 upgrade

changes:

upgrade to sdk 36
use react-native-fast-image to render GIFs and images
upgrade apollo to v2

things left to do:

test GIF performance on old iOS devices
consider loading spinner


Use NixOS for compile server(s)

Closes #53

OS config based on this guide https://github.com/nix-community/nixos-anywhere/blob/main/docs/quickstart.md