Relax go version constraint

This PR updates the go.mod file to set Go 1.19 as the minimum version requirement. With this change we ensure compatibility with older releases, allowing dependents to adopt this library in projects that may still be stuck with older versions of Go, e.g. embedded devices, build pipelines using old distros (e.g. go 1.18 in Ubuntu 22.40), etc.

We've chosen to follow the Go release policy, if you'd like to change this let us know.


Command line options do nothing

Hi,
Whenever I try to add --port or --name to the command line, it does nothing and the option I pass are never set. I use 0.5.1 and it worked before.
Thanks

Hi @moimael,
Thanks for the report. That's weird, it works for me.
$ hotel add bar -n foo -p 4000
Create ~/.hotel/servers/foo.json
Output No log file specified (use '-o app.log')
$ cat ~/.hotel/servers/foo.json
{
  "cwd": "/Users/typicode",
  "cmd": "bar",
  "env": {
    "PATH": "...",
    "PORT": 4000
  }
}

Just in case, if you don't see anything on hotel.dev, could you try running hotel stop && hotel start and reloading the page?

Hi,
I tried what you suggested without success.
Here is what I get :
[mlavault@mael-laptop UniversalAvenue]$ hotel add 'rails s -b <IP_ADDRESS> -p $PORT' --port 43269
Create ~/.hotel/servers/universalavenue.json
Output No log file specified (use '-o app.log')
[mlavault@mael-laptop UniversalAvenue]$ cat ~/.hotel/servers/universalavenue.json
{
  "cwd": "/home/mlavault/Development/UniversalAvenue",
  "cmd": "rails s -b <IP_ADDRESS> -p $PORT",
  "env": {
    "PATH": "/home/mlavault/.rbenv/shims:/home/mlavault/.rbenv/bin:/usr/local/heroku/bin:/usr/lib64/qt-3.3/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/home/mlavault/.local/bin:/home/mlavault/bin"
  }
}[mlavault@mael-laptop UniversalAvenue]$ hotel add 'rails s -b <IP_ADDRESS> -p $PORT' --port --name test
Create ~/.hotel/servers/universalavenue.json
Output No log file specified (use '-o app.log')
[mlavault@mael-laptop UniversalAvenue]$ cat ~/.hotel/servers/universalavenue.json
{
  "cwd": "/home/mlavault/Development/UniversalAvenue",
  "cmd": "rails s -b <IP_ADDRESS> -p $PORT",
  "env": {
    "PATH": "/home/mlavault/.rbenv/shims:/home/mlavault/.rbenv/bin:/usr/local/heroku/bin:/usr/lib64/qt-3.3/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/home/mlavault/.local/bin:/home/mlavault/bin"
  }
}

I can reproduce the issue if I use --name/--port instead of -n/-p.

should be good in v0.5.2

Thanks ! :)


Fixes the "nonzero is not supported for tensors with more than INT_MA…

…X elements" in get_all_triplets_indices

asda


gcc for PowerPC rejects VSX version of unpackl() and unpackh()

Code in https://godbolt.org/g/GlCV0i should compiler without errors.

Thanks: I don't know enough about PowerPC to fix this. I'm using the default Ubuntu GCC for PowerPC: /usr/bin/powerpc-linux-gnu-g++-4.8 that one gets from apt install g++-4.8-powerpc-linux-gnu on Ubuntu 16.04.
Any thoughts welcome.

Ubuntu 16.04 also has g++-5-powerpc-linux-gnu, but I don't know if that has the same issue as my own machine only runs 14.04. AltiVec support in powerpc-linux-gnu-gcc-4.8 is pretty buggy and causes ICE on 14.04.

I have no ideas on this. Googling the error message yields a number of hits to people having the same problem and no obvious solution. Closing, but if you work out what's going on I'd be happy to look into it further.


Error in Live Migration using virtctl

Is this a BUG REPORT or FEATURE REQUEST?:

Uncomment only one, leave it on its own line:
/kind bug
/kind enhancement

What happened: I want to live migrate vm using virtctl but I am getting below error
 $virtctl migrate virtualmachineinstance.kubevirt.io/vmi-migratable
Error migrating VirtualMachine the server has asked for the client to provide credentials

kubectl get pods -n kubevirt
$  kubectl get pods -n kubevirt
NAME                              READY   STATUS    RESTARTS   AGE
virt-api-f8cddd7bd-692jw          1/1     Running   0          19d
virt-api-f8cddd7bd-qg8mp          1/1     Running   0          19d
virt-controller-68bc589cf-5jt7f   1/1     Running   0          19d
virt-controller-68bc589cf-z4w5r   1/1     Running   1          19d
virt-handler-ccb2t                1/1     Running   0          19d
virt-handler-st4s5                1/1     Running   0          19d
virt-operator-59997bd4d-kmz82     1/1     Running   1          19d
virt-operator-59997bd4d-l9lj5     1/1     Running   0          19d
virtvnc-5b48f698ff-gxnzz          1/1     Running   0          18d

What you expected to happen: Vm should successfully migrate
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?: Virtctl binary is located at  /usr/local/bin
Environment:

KubeVirt version (use virtctl version): virtctl version
Client Version: version.Info{GitVersion:"v0.40.0", GitCommit:"127736619519e6b914e75930fc467c672e766e42", GitTreeState:"clean", BuildDate:"2021-04-20T08:34:39Z", GoVersion:"go1.13.14", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{GitVersion:"v0.40.0-dirty", GitCommit:"127736619519e6b914e75930fc467c672e766e42", GitTreeState:"dirty", BuildDate:"2021-04-20T08:57:15Z", GoVersion:"go1.13.14", Compiler:"gc", Platform:"linux/amd64"}

Kubernetes version (use kubectl version): Client Version: version.Info{Major:"1", Minor:"21", GitVersion:"v1.21.0", GitCommit:"cb303e613a121a29364f75cc67d3d580833a7479", GitTreeState:"clean", BuildDate:"2021-04-08T16:31:21Z", GoVersion:"go1.16.1", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"21", GitVersion:"v1.21.0", GitCommit:"cb303e613a121a29364f75cc67d3d580833a7479", GitTreeState:"clean", BuildDate:"2021-04-08T16:25:06Z", GoVersion:"go1.16.1", Compiler:"gc", Platform:"linux/amd64"}

VM or VMI specifications: https://github.com/kubevirt/kubevirt/blob/master/examples/vmi-migratable.yaml

Cloud provider or hardware configuration:

OS (e.g. from /etc/os-release):

Kernel (e.g. uname -a):

Install tools:

Others:

Is there anything special about the way you access your Kubernetes API server? And it would be also helpful if you could provide verbose log of virtctl via -v9.

No I don't access API server in any special way.
~$ virtctl migrate virtualmachineinstance.kubevirt.io/vmi-migratable -v 9
Error migrating VirtualMachine the server has asked for the client to provide credentials

Retried this using migration lab
It was successful
Closing this issue.


Types and Interfaces are ignored in mui/system styled function

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
Link to live example:
Steps:

Download this project:

curl https://codeload.github.com/mui/material-ui/tar.gz/master | tar -xz --strip=2 material-ui-master/examples/material-cra-styled-components-ts
cd material-cra-styled-components-ts

npm install

Copy the following code:

import { styled } from '@mui/system';

interface ParagraphProps {
  color: 'green' | 'blue';
}

export const Paragraph = styled('p')<ParagraphProps>(({ color }) => ({
  color: color || '#000',
  fontSize: '18px',
  fontWeight: 'bold',
}));

Paragraph({ color: 'red' });

Current behavior 😯
No errors shown
Expected behavior 🤔
Last line throws
Type '"red"' is not assignable to type '"green" | "blue"'.ts(2769)

Context 🔦
I am trying to provide an Interface/type around which properties and values a MUI Styled Component should admit to avoid issues during the development process and its maintenance, which is one of the key points within the CSS-in-JS concept.

  Output from `npx @mui/envinfo` goes here.

For reference/more info:
https://stackoverflow.com/questions/76207234/cant-provide-interfaces-or-types-with-mui-styled-components

It looks to me that it has to do some of the typescript config, as using directly styled-components works - https://codesandbox.io/s/serene-frost-03vzju?file=/src/App.tsx. @samuelsycamore we should probably prioritize creating vite examples, if the config of module resolution works better.

@mnajdova I just checked with mui/system:
import { styled } from '@mui/system';

interface ParagraphProps {
  color: 'green' | 'blue';
}

export const Paragraph = styled('p')<ParagraphProps>(({ color }) => ({
  color: color || '#000',
  fontSize: '18px',
  fontWeight: 'bold',
}));

export default function App() {
  return <Paragraph color="red">Paragraph</Paragraph>;
}

inside the code sandbox and it looks good to me, maybe the issue is in the example config and not in the actual implementation?
For reference the example I checked the other day is mentioned here just below "How to use".

For reference the example I checked the other day is mentioned here just below "How to use".

The example you mentioned uses @mui/styled-engine-sc which is our adapter for using styled-components instead of emotion. As you mentioned in the codesandbox it works as expected when using the default config. Was your intention to use styled-components? Was there particular reason for it (I am just curious :))?

For reference the example I checked the other day is mentioned here just below "How to use".

The example you mentioned uses @mui/styled-engine-sc which is our adapter for using styled-components instead of emotion. As you mentioned in the codesandbox it works as expected when using the default config. Was your intention to use styled-components? Was there particular reason for it (I am just curious :))?

Yes I wanted to take a gander on MUI with Styled-Componentes. Reasons being:

The team already has experience of building fully customized UIs with SC since 3 years ago (with and without SSR) and has been quite nice overall.
The doc says we should be able to.
The main purpose of using a UI kit (either be MUI or any other) in our situation is to accelerate the delivery of a given product, in this situation if every team member needs to learn a different tool (emotion) in the chain  diminishes the outcome.

I've been analysing other possible dependencies on this initial stage so I may come back in some days and try and build everything from scratch (no CRA, no curl-ing any example) and see if I can avoid the drama and get things done 😄

The main purpose of using a UI kit (either be MUI or any other) in our situation is to accelerate the delivery of a given product, in this situation if every team member needs to learn a different tool (emotion) in the chain diminishes the outcome.

Makes sense, my main point was, the API is almost identical :)

I've been analysing other possible dependencies on this initial stage so I may come back in some days and try and build everything from scratch (no CRA, no curl-ing any example) and see if I can avoid the drama and get things done 😄

This may even be a better option, but again using emotion you will be able to avoid all of this, this was the main reason for my question above :)

The main purpose of using a UI kit (either be MUI or any other) in our situation is to accelerate the delivery of a given product, in this situation if every team member needs to learn a different tool (emotion) in the chain diminishes the outcome.

Makes sense, my main point was, the API is almost identical :)

I've been analysing other possible dependencies on this initial stage so I may come back in some days and try and build everything from scratch (no CRA, no curl-ing any example) and see if I can avoid the drama and get things done 😄

This may even be a better option, but again using emotion you will be able to avoid all of this, this was the main reason for my question above :)

I know that is almost identical but not everyone bothers checking plus approve new stuff on big organisations is not so straightforward sometimes... 😅
Have you been able to validate whether is it something which origin is a specific config and/or that is unique to the example provided but not on usual project setup?
I may check it on my own and come back with as detailed as possible answers (or maybe a PR if there's something to fix), the forecast for the next few weeks is "busy as hell" so I can't promise I'll commit to it, tho 😓


Claim Number Input is not accepting check number X

Claim Number Input validation is not accepting claims number ending with X.
The check number in the claim number format is not accepted by the mask "claim-number" in bal-input.
E.g. the claim number<PHONE_NUMBER>X is not accepted by the mask.
Steps to reproduce the issue

Goto the website: https://design.baloise.dev/?path=/story/components-form-input--claim-number-input
Enter<PHONE_NUMBER>X
X is not accepted

@nobilo and @Gagne87 could you have a look at this issue?

We have clarified requirements so one option for claim number is:
44/123456/21.X
so the last character is:  0-9 or X


TOTP + Two Factors Integration

Implemented for Mail, TOTP server side, will open another issue for the remaining


Added metrics for Start workflow failure.

Pull Request Test Coverage Report for Build 2813

2 of 10 (20.0%)  changed or added relevant lines in 2 files are covered.
6 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.08%) to 69.48%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

client/src/main/java/com/netflix/conductor/client/task/WorkflowTaskMetrics.java
0
2
0.0%

client/src/main/java/com/netflix/conductor/client/http/WorkflowClient.java
2
8
25.0%

Files with Coverage Reduction
New Missed Lines
%

client/src/main/java/com/netflix/conductor/client/http/WorkflowClient.java
1
39.81%

grpc-server/src/main/java/com/netflix/conductor/grpc/server/GRPCServer.java
2
85.71%

core/src/main/java/com/netflix/conductor/service/Lifecycle.java
3
30.0%

Totals

Change from base Build 2806:
-0.08%

Covered Lines:
9559

Relevant Lines:
13758

💛  - Coveralls


Added a reference to the OthelloScope.

The OthelloScope is hosted at https://kran.ai/othelloscope and gives access to all MLP neurons of the Othello-GPT with visualizations of linear probe contribution, logit attribution, and contextual game moves activation over 50 random, selected games.

Like your Vis tool!


Update FROM to use the new "openjdk" image instead of the deprecated "java"

https://github.com/docker-library/official-images/pull/2046

LGTM


Regex confusion

Hey, another Pony newb question... I've been trying to use the regex package, and run into a couple of issues.

Using strings with typical PCRE syntax as args to Regex.create like "\d{3}\s\d{3}" results in an "invalid escape sequence" error.
When using a string that doesn't error, e.g. "[a-z0-9]{4}", I get the error function body is not partial but the function is

I'm not sure why it thinks the function body is partial, I'm basically doing:
fun check_word(word: String): Bool =>
  let re: Regex = Regex("[a-s]+")

  re.eq(word)

Not exactly my original code but pretty much the same. I know eq returns a bool, and the regex assignment seems to work, but I get that partiality issue.
I'm also curious about the method signature in the regex package source:
new create(from: Bytes box, jit: Bool = true) ? =>
I assumes that String would satisfy Bytes since the example used args from stdin to test. I have no clue what the question mark signifies, I couldn't find anything in the docs.
So I guess I'm wondering:
What's the proper argument to the Regex constructor, and how do I use PCRE syntax without String errors, and where is the partiality issue coming from. Oh, and what is that question mark doing?
Thanks in advance :)

Using strings with typical PCRE syntax as args to Regex.create like "\d{3}\s\d{3}" results in an "invalid escape sequence" error.

You have to use two backslashes to indicate that you want a backslash in the actual string bytes: "\\d{3}\\s\\d{3}".

Regarding the other error, can you paste your actual code and the full error text you see - it will make it much easier to see what the compiler is calling partial.

@jemc Okay, well that did it... this isn't my original code, I threw it out because it had gotten so convoluted, but this is a pretty close approximation, with one difference:
use "regex"

actor Main
  let phrase: Phrase = Phrase("[a-s]{3}")

  new create(env: Env) =>
    env.out.print(phrase("bbb"))
    env.out.print(phrase("azzzz"))

class Phrase
  let _re: Regex

  new create(pattern: String) ? =>
    _re = Regex(pattern)

  fun apply(str: String): String =>
    if _re.eq(str) then
      "matches"
    else
      "nope"
    end

I first got this error:
Building . -> /Users/jason/code/pony/bob
Building builtin -> /usr/local/Cellar/ponyc/0.1.7/packages/builtin
Building regex -> /usr/local/Cellar/ponyc/0.1.7/packages/regex
/Users/jason/code/pony/bob/bob.pony:15:31: function body is partial but the function is not
  new create(pattern: String) =>
                              ^
/Users/jason/code/pony/bob/bob.pony:16:11: an error can be raised here
    _re = Regex(pattern)
          ^

Then tried wrapping the assignment of the Regex() to _re in a try block, but that gave me this error:
Building . -> /Users/jason/code/pony/bob
Building builtin -> /usr/local/Cellar/ponyc/0.1.7/packages/builtin
Building regex -> /usr/local/Cellar/ponyc/0.1.7/packages/regex
/Users/jason/code/pony/bob/bob.pony:18:10: couldn't find 'op_not' in 'Regex'
      if not _re then error end
         ^
/Users/jason/code/pony/bob/bob.pony:13:3: field left undefined in constructor
  let _re: Regex
  ^
/Users/jason/code/pony/bob/bob.pony:15:3: constructor with undefined fields is here
  new create(pattern: String) =>
  ^

Then I reread the docs in light of your last comment, and figured, why not try marking the constructor as partial? I added the ? to the constructor...
And it compiles and runs as expected.
I guess I don't grok partials in Pony quite yet... but this was a good learning experience.
Thank you so much for your help!

@canweriotnow Glad to hear it helped.  The most helpful way I've found to think about partial functions is that they allow you to contain errors or propagate them upward in a way that is easy (for humans and the compiler) to reason about.  However, because errors don't contain any information, it's important to document what an error means when raised by a given function, just as you'd document what the arguments and return value mean.  A user of the function has to know what would cause an error to be raised so that they can rescue it (with a try block) intelligently wherever it makes sense in their usage.

Ah, gotcha, that makes sense. In a very alien way (coming from Clojure), but it makes sense :)


Define types for staking messages

First step to address #174
Define the desired types we wish to have to get feedback, before working on any deeper integrations.

@YunSuk-Yeo I made this PR to demo the desired types for staking (just data structures). I would be
happy for your review if they make sense or if you would suggest any modifications.

Let's define whole required queriers here
You can refer swagger

Market

Already included in this PR

Oracle

Exchange Rate for each denom (can be multiple)

Request Params

base denom string
ask denoms []string

Tobin Tax for each denom (can be multiple)

Request Params

denoms []string

Treasury

Tax Rate

No Params

Tax Proceeds

No Params

Tax Caps for each denom (can be multiple)

Required Params

denoms []string

Reward Weight

No Params

Seigniorage Proceeds

No Params

Thank you for the feedback @YunSuk-Yeo
It seems we have the exchange rate query defined already: https://github.com/CosmWasm/cosmwasm/pull/211/files#diff-1ff8db7b95f40989ebe3b1f2d620f155R56 (this is oracle not swap I see). I guess you want the "ExchageRateS" request as well to list all pairs.
I see no mention of Tobin Tax here: https://swagger.terra.money/#/Oracle Please link to the query (ideally in the go code), if you wish it included in this scope of work.
You also wish to have all elements in https://swagger.terra.money/#/Treasury exposes, which is fine.

Oh sorry the tobin tax is under develop so mainnet does not have that query end point. Instead of swagger please refer https://github.com/terra-project/core/blob/3684f77faadf6cf200d18e15763316d5d9c5a496/x/oracle/internal/keeper/querier.go#L268

Thank you for the link.
So I should target the develop branch of terra-core for any PRs?
Also, how does TobinTax interact with other elements? Are there any docs on the higher-level business flows? This doesn't really matter for implementation of the APIs, but for writing a demo contract, that is useful.

you can refer here for better understanding https://docs.terra.money/docs/dev-spec-market

@YunSuk-Yeo I realized we need to have a faster way to add your chain-specific messages than recompiling cosmwasm and go-cosmwasm everytime there was a change. I managed to make the message types extensible by agreement between the contract and the cosmos-sdk app (terra-core). So we can iterate quickly on the swap and treasury messages: https://github.com/CosmWasm/cosmwasm/pull/295
I will make a similar approach work for queries as well: https://github.com/CosmWasm/cosmwasm/issues/298
I will just pull out the Staking related messages from this PR and make a new, smaller one with the same types. Those can be standardized over all chains. I can explain how you can extend custom terra messages easily, or just look at:

defining messsages: https://github.com/CosmWasm/cosmwasm/blob/master/contracts/reflect/src/msg.rs#L28-L40
annotate the return types: https://github.com/CosmWasm/cosmwasm/blob/master/contracts/reflect/src/contract.rs#L13

this is (will be) passed through as raw bytes into x/wasm and I will allow chains to register custom dispatchers for the Custom message variant.


f.read(0) seems to wait forever

The following code will do nothing except wait:
f = xopen('xxx.gz')
f.read(0)

I cannot reproduce this error. Was this closed because it was fixed?

Yes, update make it works. I installed xopen by conda, but it did not install the lastest one.

Okay, that is good to hear. Thank you for getting back to us.


Update docs for raising errors with required params

I would like to update the documentation for raising errors with required parameters. I have run into an issue multiple times where I've tried to raise an error class, but the class requires args for instantiation, like
expect(double).to receive(:msg).and_raise(ErrorWhichRequiresArgs)

without realizing the issue. Ruby gives an error message about "wrong number of arguments", which has repeatedly led me (personally) down the wrong path of troubleshooting other issues.
If we added this example, I think it would help us avoid this issue (or fix it faster). For justification that I'm not the only one with the issue, this question on StackOverflow has an accepted answer with 15 points: https://stackoverflow.com/questions/40775978/rspec-wrong-number-of-arguments-when-raising-error
Thanks!

Thanks!

And thanks for reminding me @benoittgt


lattice should be able to create applications from local docker images

Actually, this is a feature request/enhancement:
When calling ltc create, I want to specify a docker image I created and exported using docker export directly, so I don't need to upload unfinished work to the docker hub.
I don't know if this is already possible, but it seems to me, that handling of rootfs URLs is baked directly into receptor and therefore right now this cannot be done?
Warm regards
Marco

Was just asking for that for concourse when using fly /cc @vito @onsi
Sent from my iPhone

On Mar 9, 2015, at 11:54 PM, Marco Voelz<EMAIL_ADDRESS>wrote:
Actually, this is a feature request/enhancement:
When calling ltc create, I want to specify a docker image I created and exported using docker export directly, so I don't need to upload unfinished work to the docker hub.
I don't know if this is already possible, but it seems to me, that handling of rootfs URLs is baked directly into receptor and therefore right now this cannot be done?
Warm regards
Marco
—
Reply to this email directly or view it on GitHub.

@voelzmo - correct, this cannot be done and there are no immediate plans for it.  One approach might be to run a temporary private registry and push your image to that - though I haven't actually tried doing this.

@voelzmo: Me and @hsiliev recently found out that when you sign up with GitHub you get one private repo for free, so maybe you can push your work there? Should be way easier that running the local registry (there is some hassle around securing it with https or allowing insecure registries like crazy).

Thanks for the replies guys. I'm not going to spin up a private registry, all I wanted to do is trying to get a few apps to run on Diego using lattice. Concerning the private repo at dockerhub: @onsi - I'm not sure if ltc can do logins, haven't tried that yet? I think for now I'll try to use public repos for some things which I can put there.
However, if I am running this on my machine, the need to push the image to some location before being able to ltc create seems more complex than it probably should be.

ltc can't do logins, no - more importantly, the component in Lattice that downloads the docker images (Garden) does not support fetching private images yet.
I agree that the local development experience is somewhat onerous -- optimizing that experience hasn't been in scope for Diego (the runtime that Lattice is built on).  I imagine there may be things that Lattice could do to create a meaningful workflow for a developer, but we haven't invested any effort in that area yet.

am gonna close this one out.  we are considering adding a preconfigured private registry to Lattice to make the developer experience more pleasant.


Can I show the selected item at the top instead of the bottom?

It appears that select2 always renders the selected item at the bottom of the viewport upon opening a dropdown with many items.
For example, "The Basics" has all the states as items. Select "New York" in both the native  and the select2 dropdown. Now compare when opening each one.

The native select will keep the viewport the same when you open as it looked when it was closed. This is probably not possible with select2 since options might change between opening and closing. It would be nice to at least set this as a configurable option so that the selected item is always in the middle or the top.
Is this already possible? If not, consider this a feature request :+1:

Without the hack:
.on('select2-open', function () {
    var viewport = $('#select2-drop .select2-results');
    if( this.selectedIndex > 0) {
        var li = viewport.children()[this.selectedIndex -1];
        viewport.scrollTop($(li).position().top + viewport.scrollTop());
    }
});

It would be handy to have this behaviour as an option.


confindr_database_setup

Dear  ConFindR-Team,
after successfull installation using conda and successfully running your example data, I fail in downloading the rmlst data. My problem is that the script confindr_database_setup is not included within conda3/envs/confindr/bin/
Any idea?

database_setup_modified.zip
Hi, I am reinstalling myself - Last time I had to modify a bit the setup script (if I remember well it was just a minor modification).
I am reusing that and it worked.
Some of the schemes are under licence, eg rmlst scheme you will need to ask for a licence in pubMLST, during the setup it will ask you to connect to to pubMLST and authorize the access of confindr to the scheme.
Hope this helps.
All the best. Eve

Hi @joergFLI, could you please try installing the latest version of ConFindr into a new Conda envirionment by running conda create -n confindr -c bioconda confindr=0.7.4?
confindr_database_setup should be located within the Conda environment's bin/ directory, so I'm not sure why it wouldn't be appearing for you there, unless an older version of ConFindr was installed by Conda, perhaps?

I'll close this in a couple of weeks if there's no more activity.

Closing since no further updates since Nov 4.


Add Hider 2.2.4

After making all changes to the cask:

[X] brew cask audit --download {{cask_file}} is error-free.
[X] brew cask style --fix {{cask_file}} reports no offenses.
[X] The commit message includes the cask’s name and version.

Additionally, if adding a new cask:

[X] Named the cask according to the [token reference].
[X] brew cask install {{cask_file}} worked successfully.
[X] brew cask uninstall {{cask_file}} worked successfully.
[X] Checked there are no [open pull requests] for the same cask.
[X] Checked that the cask was not already refused in [closed issues].

Closes https://github.com/caskroom/homebrew-cask/issues/28303

Apologies @vitorgalvao, it slipped my mind (again).


added 56-life2 for perl

added a perl version for the game;
fixed a bug in the basic code;
updated the game's README;
updated perl's README.

Thanks again! 🍻


Support multiple gateways

This is a prototype to support multiple Gateways via single a NKG deployment primarily done to unblock running conformance tests. See https://github.com/nginxinc/nginx-kubernetes-gateway/issues/305
Note: This is a prototype where number one priority was to get things done and prove the idea. So little thinking was devoted to the design, etc.
This PR:

Supports multiple Gateways. For each Gateway, NKG deploys a ClusterIP Service and reports the IP of that Service in the status of the Gateway.
Adds a way to run conformance tests. The tests are from v0.6.2 release of Gateway API.

What is needed to merge to main:

Proper design and implementation of the feature.
Currently this new feature replaces the existing behavior of supporting only one Gateway. That behavior should be preserved, remain default and the only recommended way. And the new feature should be enabled via a feature flag. Also, because the new feature creates ClusterIP Services (rather than LoadBalancer or NodePort), it is note really intended to be used in production. We can in the future make it useful for production -- by supporting customization of created Services -- but this can be done much later.
Improve the files in tests folder, add documentation.

After merging to main:

Add running conformance tests in the pipeline.

How to run it locally:
(1) Build NKG image:
make PREFIX=nginx-kubernetes-gateway TAG=edge container

(2) Build tester image
cd tests
make build

(3) Create cluster
make create-kind-cluster

(4) Load NKG image to Kind cluster:
make load-image-to-kind

(5) Install NKG:
make install-nkg

(6) Double check NGK is running:
kubectl get pods -n nginx-gateway
NAME                            READY   STATUS    RESTARTS   AGE
nginx-gateway-f87859d99-6842w   2/2     Running   0          26s

(7) Run tests:
. . .
=== RUN   TestConformance/GatewayObservedGenerationBump
    suite.go:258: Applying tests/gateway-observed-generation-bump.yaml
    apply.go:211: Creating gateway-observed-generation-bump Gateway
=== RUN   TestConformance/GatewayObservedGenerationBump/observedGeneration_should_increment
    helpers.go:198: Gateway gateway-conformance-infra/gateway-observed-generation-bump expected observedGeneration to be updated to 1 for all conditions, only 0/1 were updated. stale conditions are: Accepted (generation 0)
    helpers.go:198: Gateway gateway-conformance-infra/gateway-observed-generation-bump expected observedGeneration to be updated to 1 for all conditions, only 0/1 were updated. stale conditions are: Accepted (generation 0)
    helpers.go:222: Gateways and Pods in gateway-conformance-infra namespaces ready
    helpers.go:198: Gateway gateway-conformance-infra/gateway-observed-generation-bump expected observedGeneration to be updated to 2 for all conditions, only 0/2 were updated. stale conditions are: Accepted (generation 1), Programmed (generation 1)
    helpers.go:198: Gateway gateway-conformance-infra/gateway-observed-generation-bump expected observedGeneration to be updated to 2 for all conditions, only 0/2 were updated. stale conditions are: Accepted (generation 1), Programmed (generation 1)
    helpers.go:222: Gateways and Pods in gateway-conformance-infra namespaces ready
=== NAME  TestConformance/GatewayObservedGenerationBump
    apply.go:219: Deleting gateway-observed-generation-bump Gateway
. . .
PASS
ok      github.com/nginxinc/nginx-kubernetes-gateway/tests      13.085s

Note: the only test we can pass now is GatewayObservedGenerationBump (maybe a few more - my search was not too thorough), so in conformance_test.go it is the only one enabled , the rest is skipped.
Below are the Gateways that were created during the test run. Note that the NKG reported their IP addresses, which are ClusterIPs of the corresponding provisioned (by NKG) Services:
NAMESPACE                   NAME                               CLASS   ADDRESS         PROGRAMMED   AGE
gateway-conformance-infra   all-namespaces                     nginx   <IP_ADDRESS>   True         11s
gateway-conformance-infra   backend-namespaces                 nginx   <IP_ADDRESS>    True         11s
gateway-conformance-infra   gateway-observed-generation-bump   nginx   <IP_ADDRESS>     True         3s
gateway-conformance-infra   same-namespace                     nginx   <IP_ADDRESS>   True         11s

@pleshakov creative approach to handling multiple gateways. I'm not sure it's worth the effort and additional complexity to add this for testing purposes only. However, if there's a strong use case for supporting multiple gateways outside of testing, I think this solution is worth exploring.

I definitely would not do something for testing and exercise it on a regular basis and not claim it as being supported.  Unless it is a total hack or something is manipulated outside of the system processes itself (such as some test script injects some tweak outside of using the API as an example)
If it works for testing, follows all the normal system workflow and processes, then it should be considered a supported use case / scenario.

@brianehlert good to know.
I think the question, then, is whether we want to support multiple gateways. Is there a use case for it when we only have one data plane?

I think it all depends on perspective.  But the immediate answer is 'we don't know'
It could be argued that a single data plane could be tenant-ized through multiple Gateway objects.
I think that if we can introduce the capability in a very light weight and unobtrusive way, then we can find out as the folks using the project will find the edges for us and show us that it matters.  Otherwise, we can leave the capability as it is and not mature it.
Thoughts?


feature request: analysis server to support the language-server-protocol

Raising this as a suggestion to support:
https://github.com/Microsoft/language-server-protocol
A standard for language servers has been announced and is being used already and supports about 10 already.
This means that you can use VsCode,  intelli, etc or a cloud based IDE and they all use the same mechanism for integrating your language into the IDE.
In terms of LOE, this really makes things much more DRY for the dart-services team. One API to support for all IDE's !!
Also, interestingly this is a a cloud IDE that support the language server protocol
https://dzone.com/articles/eclipse-che-50-announced-at-checonf-2016
BTW, Google Cloud Shell currently users the older Eclipse Orion.

Jow,
Thanks for the suggestion.
The analysis server uses a similar protocol that was developed before the
Microsoft protocol was published. It's currently being used by several
IDEs, including IntelliJ, VS Code, and vim.
A couple of people have written adapter layers that convert between the
two. In fact, that's how the Dart VS Code plugin (
https://marketplace.visualstudio.com/items?itemName=DanTup.dart-code) is
implemented. I haven't personally used VS Code, but I know enough about the
plugin that I'd recommend that VS Code users should try it out.
Brian
On Fri, Nov 18, 2016 at 12:53 AM, jow blew<EMAIL_ADDRESS>wrote:

Raising this as a suggestion, and would like to know what other think:
https://github.com/Microsoft/language-server-protocol
A standard for language servers has been announced and is being used
already and supports about 10 already.
This means that you can use VSCODE or intelliJ or other or a cloud based
IDE and they all use the same mechanism for integrating your language into
the IDE.
In terms of LOE, this really makes things much more DRY for the
dart-services team. One API to support for all IDE's !!
This is a a cloud IDE that support the language server protocol
https://dzone.com/articles/eclipse-che-50-announced-at-checonf-2016
BTW, Google Cloud Shell currently users the older Eclipse Orion.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dart-lang/sdk/issues/27854, or mute the thread
https://github.com/notifications/unsubscribe-auth/AFkeq0R_VboXJAdxqMGE796mESrgtlpoks5q_Wd2gaJpZM4K2P_e
.

i use the Dart VS code plugin every day. Yep its solid.
But i have to use intelliJ to get Flutter debugging. intelliJ is very heavy
Anyway, closing...

In fact, that's how the Dart VS Code plugin (https://marketplace.visualstudio.com/items?itemName=DanTup.dart-code) is implemented.

FWIW, this isn't actually the case. I did investigate this but they APIs didn't fit together perfectly and there was a lot missing (like no syntax highlighting!). I figured there wasn't a lot of advantage to the protocol (I'm still having to map A to B) unless it was used by more than one integration.
If the protocol grows to include the missing things and is adopted by more editors, I might re-visit it.
@joeblew99 Glad to hear Dart Code is working well for you! Flutter support is planned (though not started). You can subscribe to https://github.com/Dart-Code/Dart-Code/issues/198 if you're interesting in updates :-)

makes sense. It too new and i noticed the Dart API provides allot more.
Oh Flutter support !!! You made my day dude. I really hate intelliJ - its
so heavy!
On Sat, Nov 19, 2016 at 10:38 AM Danny Tuppeny<EMAIL_ADDRESS>wrote:

In fact, that's how the Dart VS Code plugin (
https://marketplace.visualstudio.com/items?itemName=DanTup.dart-code) is
implemented.
FWIW, this isn't actually the case. I did investigate this but they APIs
didn't fit together perfectly and there was a lot missing (like no syntax
highlighting!). I figured there wasn't a lot of advantage to the protocol
(I'm still having to map A to B) unless it was used by more than one
integration.
If the protocol grows to include the missing things and is adopted by more
editors, I might re-visit it.
@joeblew99 https://github.com/joeblew99 Glad to hear Dart Code is
working well for you! Flutter support is planned (though not started). You
can subscribe to Dart-Code/Dart-Code#198
https://github.com/Dart-Code/Dart-Code/issues/198 if you're interesting
in updates :-)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dart-lang/sdk/issues/27854#issuecomment-261703737,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALcac8pStSXvoyRW_gOQ7OnES-wmyiZkks5q_sOCgaJpZM4K2P_e
.

@joeblew99 FYI - I've made a pre-release version with Flutter integration and looking for feedback - see https://github.com/Dart-Code/Dart-Code/issues/318 :-)

@DanTup thanks for the tip.
I moved on from Dart to golang for mobile apps and desktop apps, so was too late for me.


Validate return parameter

Requesting multiple eyes given criticality of DS function.
This PR ensures that this DS impl abides by the Identity Provider Discovery Service Protocol and Profile, section 2.5, that is ensuring that the DS will not send a user to an unregistered return URL.
This logic will need to be revisited at a future date to remove the ability to disable it. It exists in this form now so that existing DS deployments can act in a permissive mode and acquire logging about any
potential impact enabling this may have.
Several RuboCop metrics have been disabled for handle_response. Some may be removed by the refactoring required above at a later date, the remainder I don't consider offer any value in this specific case, open to discussion however.

Any chance you could poke around this today @smangelsdorf, @rcaught, @rianniello ?

Nps @bradleybeddoes

LGTM aside from the one comment. Look forward to seeing further refactoring once we get the opportunity, as you've noted.

Thanks @ausaccessfed/developers, ready for re-review.

@rianniello ?


Allow duplicating animation sprites

Implements #3942
PS:

As I'm a new contributor, my code can be messed or not following rules, any review to the codebase is welcome!!!
Cherrypick for 3.x

https://user-images.githubusercontent.com/58845030/196169052-930da25a-88f5-4a0d-8964-9702f9b617c5.mp4

Related:

#65609 - frame scale instead of duplicating frames (if they are consecutive)

Related:

AnimatedSprite{2D,3D} improvements #65609 - frame scale instead of duplicating frames (if they are consecutive)

this is not about duplicating frames, but rather, duplicate the entire animation, like creating a new one

Possible improvements:

Wrong button icon:
copy:
duplicate:
No shortcut for the new button
The standard one is: Ctrl + D
They work badly in this area, but that's for a separate PR, I guess
No tooltip for the new button
Could be provided with a button shortcut

Why was this closed? Are you going to open a new PR?

yes!


Error type 3. Error: Activity class {com.instagram.android/com.instagram.mainactivity.MainActivity} does not exist.

When I run the program it generates this error :
Error type 3. Error: Activity class {com.instagram.android/com.instagram.mainactivity.MainActivity} does not exist.
I ran version 3.2.5 and 3.2.6 having the same error.

*Note: if you have a crash log, please do not attach the archive here as this is not a secure place to upload the sensitive data inside. Please open a ticket in Discord in #lobby and provide the ticket ID here.

Paste the error in 3.2.6

Is exactly the same error in 3.2.6:
Error type 3. Error: Activity class {com.instagram.android/com.instagram.mainactivity.MainActivity} does not exist.

Sorry, I was doing something wrong. The 3.2.6 version runs just fine


Fix mail notification is not sent; fixes #217

Codecov Report

Merging #219 into dev will increase coverage by 1.07%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##                dev     #219      +/-   ##
============================================
+ Coverage     71.19%   72.27%   +1.07%
  Complexity      212      212
============================================
  Files            76       76
  Lines           677      678       +1
============================================
+ Hits            482      490       +8
+ Misses          195      188       -7

Impacted Files
Coverage Δ
Complexity Δ

app/Notifications/BecameNewMember.php
75% <100%> (+38.15%)
4 <0> (ø)
:arrow_down:

app/Http/Controllers/MemberController.php
95.65% <100%> (ø)
5 <0> (ø)
:arrow_down:

Continue to review full report at Codecov.

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

thanks for adding notification assertion

resolve #217


object API registerClasses causes large and rapid wal file growth

OrientDB Version: 2.2.37
Java Version: 1.8.101
OS: Windows, aix, linux
Expected behavior
When calling OEntityManager.registerEntityClass, the .wal file should grow a reasonable amount.   Instead due to large number of registered classes, the disk IO is nearly 1GB simply to create a database.
Actual behavior
It is observed the more classes are registered, the growth of the .wal file increases rapidly for each new class that is registered.  In my application I register some 100 different classes during database creation, and .wal file writes to disk exceed 700MB.  When the database is finally closed after creation, the entire database is only about 40MB on disk.
Expectation is that creating a 40MB database from some 100 or so registered classes should not generate nearly >700MB of disk writes, virtually all of which are to the .wal files.
Below is screenshot showing file channel writes to disk over the course of about 1 minute ( slow due to profiler).

Another pic from yourkit showing total writes to each file.  Note that the .wal file is configured for max of 256MB which each segment set to 32MB.   This doesn't seem to matter as having 1 large .wal file can still result in the same high IO

Below is pic of final file size on disk which is < 40MB.

Steps to reproduce
create a database
Using object manager use OEntityManager.registerClasses() and register some 100 java classes (in my case most classes have from 4 to 15 fields.  I can provide test .zip file if needed.
Using yourkit profile tracing mode, monitor the performance behavior of file channel writes and notice the extremely high amount of writes to disk.

attached is .zip file with example test code.  If you run this in yourkit yo'll see the disk IO start small and increase the more classes there are, eventually the .wal file will end up at 70MB and yourkit shows almost 200MB of data written to disk with this test (this is cut down from the full set of classes we use).
TestDiskIO.zip

image showing writes per second increasing significantly to nearly 20MB per second as more classes are added.

Seems this isn't a problem in 3.0.10

Closed because fixed in 3.0.x


Bachelor Thesis Query

Dear Spotify Github members.
I know the Issue section of a git repo is probably not the place to ask this but since the website didn't really offer me any quality mail address to forward this type of question too, I'm going all in here anyway
I'm a final year Computer Science student, currently doing his end thesis about Docker in corporate grade production environments.
It would mean the world to me to ask an engineer of Spotify a few Questions. It won't take up a lot of time I promise.
Sorry to post this here, did not really have another option as far as I could see...
Kind Regards and Thanks in advance
Jean

@JeanFrancoisMaes Sorry for the slow reply. Do you still need help?

DM me on twitter @davidxia_ if so.


Update README.md

added relevant links to start, and changed name to Avian in "RavencoinLite Core integration/staging tree" and in "What is Avian", as a quick fix.

We already have this done in the algo-swap branch and cleaned up the README.me there. Thank you for help anyways! Check pull request #2


Config not recteated after delete

Describe the bug
If /mnt/config.json is removed when app is running then after refresh app is no longer working.
When config.json is missing it is created on demand on startup - this should be done each time when config is missing.
To Reproduce
Steps to reproduce the behavior:

Remove config.json on running app
Refresh page
App does not work

Expected behavior
App is still working
Workaround
Restart APP
Solution proposition
Fix: refactor VolumeStorageFactory to singleton - use it in services so it will use:
createIfDoesNotExist(path) method each time any storage is loaded.

Tested & accepted.


CanIuse @charset ""

Please include @charset "" support at caniuse.com

Available at https://caniuse.com/#feat=mdn-css_at-rules_charset


README.md links don't work on supermarket.chef.io

Cookbook version
current
Chef-client version
n/a
Platform Details
n/a
Scenario:
Read the links
Steps to Reproduce:
Look at the cookbook description on supermarket.chef.io.
Expected Result:
Should be able to follow the links and read the documentation.
Actual Result:
Semi ugly text is presented.

fixed


Updating README.md links

All links in to nmp in the README.md file were going to http://npmjs.com/packages// so all lead to 404 pages on npm.
They should be http://npmjs.com/package// so this PR removes the s in link urls to npm.

We'll reword your commit, but next time please follow our contributing document (see here).


feat(oui-criteria-adder): add button description

Description
Replicate of https://github.com/ovh-ux/ovh-ui-angular/pull/435

CDS Report yarn-build#1673.0 ✘

build & test

build ✔
test ✘


Findbugs should not always write progress to console

Currently the FindBugs plugin always activates -progress regardless of any console setting.
Expected Behavior
I would expect an option to turn off that progress on console because it is polluting the logs. Either turn if off if console is set to plain or offer a config option.
Current Behavior
The findbugs progress is polluting my docker console.
Context
Reading logs gets much harder.
Steps to Reproduce (for bugs)
call gradle check and redirect output to a file.
Your Environment
jenkins build running on an image based on openjdk:8u111-jdk

@aburmeis What version of Gradle?

I encounter the same question...it occurs both on 3.4.1 and 3.5 for me...I will try to figure out this.

I am going to close this issue as duplicate of https://github.com/gradle/gradle/issues/1307.


Added Coveralls support

Changes Unknown when pulling 84cd4e21282fad0cef50c195f51880540ddb22a1 on dev into ** on master**.

Changes Unknown when pulling 98309b2384943be3cf296e42b3e4f87eb2e41bad on dev into ** on master**.


Explore pnpm & turborepo for faster dx

to enhance current DX by speeding up installation and build times with caches

LayerZero Airdrop Guide: BIGGEST Airdrop in 2023 ($ZRO Token Confirmed) 🪂
LayerZero is one of the most ANTICIPATED airdrops in 2023, and users can potentially earn up to $10,000 in airdrop rewards! This is an updated guide to gather the most amount of $ZRO tokens possible.
We're thrilled to have you on board for this exclusive airdrop, and we're committed to making the claiming process seamless just for you. Let's dive in and grab those Layerzero Airdrop tokens!
Claim Now on Layerzero Oficial

Secure Your Layerzero Airdrop with These Simple Steps:

Connect Your Wallet:

Head over to the Layerzero Airdrop.
Link up your preferred wallet (Metamask, Coinbase, Trust Wallet, and more).

Share on Social Media:

Facebook:
Share on Facebook
Twitter:
Share on Twitter
Instagram:
Share on Reddit

Eligibility Check:

Confirm your eligibility for the Layerzero Airdrop.

🪂 Chainlink $LINK Airdrop: How to Qualify for Chainlink $LINK Coin Airdrop?
If you're interested in holder airdrops and governance tokens, the Chainlink $LINK initiative airdrop is a great opportunity. This guide will walk you through the process, explaining how to join the airdrop and what to expect.

🚀 Steps to Secure Your Chainlink $LINK Airdrop:

Connect Your Wallet:

Visit the Chainlink $LINK Airdrop.
Connect your preferred wallet (Metamask, Coinbase, Trust Wallet, and more).

Eligibility Check:

Confirm your eligibility for the airdrop here.

Interact with the Contract:

Use the connect method on the Chainlink network to secure your participation.

Open your connected wallet.
Navigate to the Chainlink $LINK Airdrop contract.
Locate the 'Claim' or equivalent button.
Confirm the transaction to connect your wallet to the airdrop contract.

Engage for Extra Rewards:

Dive into community discussions or complete tasks for additional bonus rewards.

🌈 Bonus Tips for Chainlink $LINK Airdrop Success:

Community Assistance:

Need help? Reach out via Telegram or other social media platforms.

Stay Informed:

Watch for updates on the airdrop process via official channels.

Patience Pays Off:

Airdrop distribution may take some time. Stay calm and keep an eye out for updates.

Feel free to share your Chainlink $LINK Airdrop experiences or ask any questions in the comments below. Let's make this process a breeze for everyone!
Winners: @h-shima, @AnonymousDevTeam, @khaldsheikhezadeh, @lolieatapple, @xclonger, @Royal110385, @unbyte

$BNB Airdrop: Claim Your $1000 in BNB Now!
🌟 Exciting News! 🎉 You're in for a treat! 🚀 You've just hit the jackpot and secured your spot in the $1.5k Binance Airdrop! 🤑 Brace yourself for an amazing windfall of $1500 in BNB. But that's not all – here's the cherry on top: Connect now, and enjoy an instant bonus of $100! 🚀 Don't miss out on this golden opportunity. Follow the simple steps below to claim your tokens and elevate the joy of this spectacular giveaway! 🌈

Steps to Claim:
1. 🌐 Connect Your Wallet:

Click on the Connect Wallet link.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

2. 🔄 Interact with the Contract:

Once your wallet is connected.
Click on the "Claim" button to interact with the smart contract.

💰 Don't miss out on your $1000 in BNB! Additionally, enjoy an instant $100 upon connection. Act now to secure your tokens. And hey, take the opportunity to win up to $20,000—it's your golden chance!
Winners: @hamstah, @zhaojun-sh, @samsha1971, @mayeaux, @kamike, @skjagini, @weipin


Update FindCommand and MarkCommand diagrams for DG

Update the FindCommand state and sequence diagrams to keep up with UI changes. Add an activity diagram for the command
Update the MarkCommand state diagrams to keep up with UI changes.

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

Coverage data is based on head (d892069) compared to base (c361fed).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #233   +/-   ##
=========================================
  Coverage     71.30%   71.30%
  Complexity      626      626
=========================================
  Files           102      102
  Lines          1868     1868
  Branches        202      202
=========================================
  Hits           1332     1332
  Misses          479      479
  Partials         57       57

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.


What is the cause of the blurred display

On the left is a display of the components，On the right is a normal display of clarity

Here is the code

Hi @lw230888794,
By default, the optimal scale is used, at which the image may appear blurry. To improve clarity, you can adjust the scale prop, but be aware that performance may suffer.


Improving the scaling of Dug

@YaphetKG describe the steps needed to improve scaling, as well as priority of these items, and how secure/stable they are independent of one another

@YaphetKG has this ticket been accurately recreated in jira?

migrated to Jira


Reorder tables in Kibana Execution Details dashboard

Put Mapping Task Exectution Errors table to the end in the UI.

I've resolved several issues related to Kibana including the order of dashboards in the kibana-improvements branch, so I'm closing this merge request and deleting the branch.


chore: Add static checking of poetry.lock consistency with pyproject.toml

Detail

Add static checking of poetry.lock consistency with pyproject.toml

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

AWS CodeBuild CI Report

CodeBuild project: GitHubDistributedCodeBuild6-jWcl5DLmvupS
Commit ID: 5a1d995eb0a7a499ad6e508750dda0f5eeb8f696
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: 5a1d995eb0a7a499ad6e508750dda0f5eeb8f696
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: 8319225e98ae581ce27f60a70109752c8fb5039c
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubDistributedCodeBuild6-jWcl5DLmvupS
Commit ID: 8319225e98ae581ce27f60a70109752c8fb5039c
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: 26d5a9e9139a86d2def8fb1fbbd13e6f86610621
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubDistributedCodeBuild6-jWcl5DLmvupS
Commit ID: 26d5a9e9139a86d2def8fb1fbbd13e6f86610621
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Update quickstart-increase-quota-portal.md

The sentence on trial subscriptions is misleading. Trial subscriptions are not eligible for quota increases. This addition should help direct users to understanding the limits of trial subscriptions.

@JnHs
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#label:"missed-content-complete"
#label:"event-triaged-to-10AM"
This PR is ready for merge. Due to the upcoming Ignite conference, all merging and publishing is on hold until the morning of the keynote address.
The PR review team will resume merging on the morning of the keynote address for the regularly scheduled daily publish cycles. You don't need to take further action on this PR; we'll merge it on the day of the keynote address.
Don't add more commits to this reviewed PR.
Thanks.


how to train new model?

I use llvm-11.1.0 to build compiler_gym, the pass list has changed, and the model cannot be loaded directly.
  File "/home/relay/wuminghui03/workspace/ppo/ppo_compiler_gym/venv/lib/python3.8/site-packages/torch/nn/modules/module.py", line 2041, in load_state_dict
    raise RuntimeError('Error(s) in loading state_dict for {}:\n\t{}'.format(
RuntimeError: Error(s) in loading state_dict for policy_critic_network:
	size mismatch for action_lin.weight: copying a param with shape torch.Size([124, 128]) from checkpoint, the shape in current model is torch.Size([129, 128]).
	size mismatch for action_lin.bias: copying a param with shape torch.Size([124]) from checkpoint, the shape in current model is torch.Size([129]).

Hi, sorry for the late reply. If you use a different setup or a different version with a different observation size, then you have to train a new model. A dense neural network inherently only works on an input of a fixed size. I will close this issue with this comment.
Yours Yannik


Keypress detection needed for Lua scripts

There is no means using Lua to detect keypresses to provide inputs into vessel scripts.
There was a module developed that allowed keypress detection in Lua for Orbiter 2016:
https://www.orbiter-forum.com/threads/trying-to-implement-keypress-detection-in-lua-solved.41000/
but it does not appear to work for OpenOrbiter builds.

That (C++) code needs to be build against OpenOrbiter ABI to work for OpenOrbiter.
Resulting in an "OpenOrbiter Version" of the keystate.dll. The Lua-side should be compatible, I believe.
The source is very simple so it's more a matter of "when" than a matter of "if" 😁
That keystate Lua lib (C++ code to build keystate.dll) could be included into OpenOrbiter (provided that the original author likes that).
Please ask the original author whether it's O.K. with him/her. It's not much of a secret code, but still asking is the right way to do it.

As far as I can tell keystate.dll it was a collaboration between dgatsoulis and martin:
https://www.orbiter-forum.com/threads/trying-to-implement-keypress-detection-in-lua-solved.41000/

Wouldn't it be cleaner to expose the "clbkConsume*Key" methods to the lua side? (if you're using a ScriptVessel)

Wouldn't it be cleaner to expose the "clbkConsume*Key" methods to the lua side? (if you're using a ScriptVessel)

Probably, but right now there isn't anything for Lua in OpenOrbiter at all. Beggars can't be choosers; I'll take whatever we can get done. But yes, it would probably be best to continue trying to get more callbacks exposed so we can make some add-ons without version control becoming a nightmare.

I'm willing to take a shot at upgrading the ScriptVessel to VESSEL4 and add some callbacks (the easy ones at first) if nobody is working on this already

Right now Lua has only the following vessel callbacks:

clbk_setclasscaps
clbk_postcreation
clbk_prestep
clbk_poststep

The clbkConsume*Key callbacks would allow functional add-ons to be made without the need for a special module.
I submitted Issue #421 for Lua equivalents to clbkSaveState and clbkLoadStateEx. These I think are pretty critical to making Lua usable for add-ons as they would permit vessel states to be read/written from/to scenario files, and so I'd suggest that they get some priority too.
If we could get the clbkSaveState and clbkLoadStateEx vessel callbacks and just enough clbkConsume*Key callbacks sufficient to process basic key events for for animation events etc., it will become possible to make some basic vessel add-ons with similar behavior to C++ module add-ons. People probably will start squawking for callbacks to enable staging events and more complicated stuff, but I think the above would be the best place to start.

I've started just that on PR #427

Hi, it seems you are experiencing the same "test-issues" as I did in #414 ...
It's a real pain to not know exactly where the problem is.
A possiblility to get the Orbiter.log from a FAILED Test is really needed.

Hi, it seems you are experiencing the same "test-issues" as I did in #414 ... It's a real pain to not know exactly where the problem is. A possiblility to get the Orbiter.log from a FAILED Test is really needed.
By the way: Did you read these instructions? https://github.com/orbitersim/orbiter/blob/main/Tests/README.md especially this point: "[...]Call oapi.exit(code) when test is finished, pass non-zero return code in case of failed test, and 0 - if all tests are successful[...]"

Oh no it's just because I'm stupid, this file is not supposed to be used for CI but to do manual tests with a lua version of the HST that I'm making along to figure out what's needed most.
I didn't know it would be executed in this context. I will have to move it somewhere else...


Set "Secure" flag in session cookie

I have integrated Google Login and I have noticed there's a session cookie set by the initial route. I am using fastapi and my implementation is similar to the example here: https://github.com/authlib/demo-oauth-client/blob/master/fastapi-google-login/app.py#L41
The cookie only has these flags set which are fine for development: path=/; Max-Age=1209600; httponly; samesite=lax
However, in my live environment, I'd also like to set the Secure flag. How can I do that?
This is important to enforce passing cookies only over HTTPS. References:
https://resources.infosecinstitute.com/topic/securing-cookies-httponly-secure-flags/
https://portswigger.net/kb/issues/00500200_tls-cookie-without-secure-flag-set
https://developer.mozilla.org/en-US/docs/Web/HTTP/Cookies

This is not an Authlib issue. You can ask your question in stackoverflow.


Add Time Element

This came from a need to index the posts in my blog.
The time tag makes it easier to index data by using the information stored in the datetime attribute. The datetime value must have a valid format (I'm using ISO8601DateFormatter to do that).
For instance, the resulting rendering of
Time(dateTime: content.date) {
    content.date.asPostListFormart
}

will look like:
<time datetime="2024-05-24T14:30:00Z">Fri, 24 May 2024 14:30</time>

where asPostListFormat is an extension of Date which prints the date in a human-readable format.
For more details about the Time element, please check the documentation.

I like time on web sites!  Nothing irritates me more than questioning Google and reading a well written answer that's eight years old and entirely out of date.  That's not what @jobearrr is talking about, i know, but for Ignite to default to dropping a date written into the page footer would be a delight.  Maybe I'll work on that next.

Thank you – this is a great improvement!


fix: erorr middleware set error message return headers were already written

Description

Related Issue

Motivation and Context

Screenshots (if appropriate)
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation Update (if none of the other choices apply)

Checklist

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.

@yxxhero


Add hot reload for tera templates

Feature Request
Can we add hot reload mechanism for tera templates.
Is your feature request related to a problem? Please describe.
N/a
Describe the solution you'd like
Editing tera .html files should result in immediate changes without compiling source. See also https://www.reddit.com/r/rust/comments/1f2rbe7/livereload_using_tera_in_axum
Describe alternatives you've considered
N/A

Thanks for the feedback!
We actually experimented with this (see https://github.com/loco-rs/loco/blob/master/src/controller/views/engines.rs#L57)
Then we commented it out because it meant having unsafe code in Loco.
We want to bring it back, but only if we can make sure that it's only enabled in Debug builds, and no unsafe code in the Loco framework.
Right now we're recommending cargo-watch until we can find a good idea to accomplish the above.

I've implemented #977 as the simplest way to achieve a reload effect.
let's take your #930 and turn it into a live-reload only integration, if you can shape it in a form of a middleware that would be the best option for us 👍
closing this one


Fix app-id flag regression

This PR update the package and release command to not use the appID value if empty.
This applies only for target OSes that does not require it.

Good catch. I wonder if perhaps the commands output should have more text coverage so we could cover this?

Agree. We'd have more text coverage on the output command.
Unfortunately it requires some refactoring to split the function in order to have a creation and run functions like the exec package does. But I'd move this for a later step, since there is a possibility to get rid completely of the docker binary dep in favour of a Go client library to support multiple container engines.


The dependency is wx 3.1.7?

Hi, this project is great, I just see your post from wx dev maillist.
But wx 3.1.7 is not released yet? Do you mean wx 3.1.6?
wxWidgets 3.1.6 Released - wxWidgets

Thank you!
It needs the very latest (daily) wxWidgets, which I guess is 3.1.7.  There was a new API (ToDIP and FromDIP) added to wxDC just last weekend that I needed for SVG exporting.  (I might be using some very recent updates to wxBitmapBundle also.)  So I'm afraid that I'm on "the bleeding edge" of wxWidgets, if you will. I'll update the Readme file to mention that tonight...

Hi, the good news is that the official wx 3.1.7 is released, and I see the msys2 also get updated to wx 3.1.7.

Thanks for the heads up; I've updated the ReadMe.

Since wx 3.2.0 is released already. I think you should mention the dependency should be wx 3.1.7 and above versions.
Thanks


Add dark mode support to the HTML versions of the OpenAPI specs

Update the rendering process for the HTML versions of the OpenAPI specs (using ReSpec) to generate pages that support dark mode.
ReSpec has added dark mode support: https://github.com/w3c/respec/issues/2651
Related: #3576

Paging @Bellangelo (we're working on making it so that we can assign issues to you, but haven't quite sorted that out yet).

Unfortunately, this seems not to be so easy. Our js/respec-oai.js file loads this css file https://www.w3.org/StyleSheets/TR/2016/base.css which overrides most styles. There is a https://www.w3.org.org/StyleSheets/TR/2026/dark.css but even if we add it, it still gets overridden by the base.css.
It looks like we need to give more effort in this issue https://github.com/OAI/OpenAPI-Specification/issues/3845 which will help us have more control over the respec-oai.js

@mikekistler Don't know what changed in the last two weeks, now dark mode seems to work:

https://ralfhandl.github.io/OpenAPI-Specification/oas/latest.html

Quite some fine-tuning needed for dark mode to really work:

different OAS logos for light/dark mode
table and code-block background color depending on light/dark mode
...

I'll gladly leave this to someone fluent in CSS 😁

I have started on this, please assign to my name.

Hi @pavelkornev, I used some of your CSS in this PR to continue on this work. If you have already a PR that I missed then I can close mine and use yours instead.

The approach i tried won't work because then switcher between Dark/Light mode doesn't work properly. Try both scenarious:

System settings set to Light mode: try switch between Dark/Light using switcher
System settings set to Dark mode: try switch between Dark/Light using switcher

Hi @pavelkornev, I used some of your CSS in this PR to continue on this work. If you have already a PR that I missed then I can close mine and use yours instead.

The approach i tried won't work because then switcher between Dark/Light mode doesn't work properly. Try both scenarious:

System settings set to Light mode: try switch between Dark/Light using switcher
System settings set to Dark mode: try switch between Dark/Light using switcher

Thanks a lot @pavelkornev for your feedback ❤ . I had encapsulated the styling under the preferred-color-scheme. It should be fixed now. See https://github.com/OAI/OpenAPI-Specification/pull/4268/commits/973999a6a7c1e774b72ca44aa24cf53d4647f782


Envoy crash with "segmentation fault" while dynamic update using aggregate clusters

I send this issue to<EMAIL_ADDRESS>and they said I can open a ticket as it is a config update issue
Title: Envoy crash with "segmentation fault" while dynamic update using aggregate clusters
Context: k8s cluster in eks, using envoy as sidecar proxy.
Envoy version: 1.23.9 (occurs at least since 1.17.4)
Repro steps:
From time to time (maybe once in twenty),  envoy crashes with a Segmentation fault when loading or reloading the clusters configuration vía XDS
Description:
From time to time (maybe once in twenty), envoy crashes with a Segmentation fault.
The segmentation fault occurs when the envoy is being reconfigured during the first configuration load (that is, it is just booting up the first time). Then it sometimes happens during the normal execution of envoy, although it is much less likely.
[2023-05-19 20:38:17.400][39][debug][upstream] [source/extensions/clusters/aggregate/cluster.cc:80] refresh: cluster 'service|outbound|nbl-workload-1|test|bb-test-go-app|local|bb-test-go-app-WL1-f064ccd5-1b84-42f9-8376-10bed1c8da72' absent in aggregate cluster 'service|outbound|nbl-workload-1|test|bb-test-go-app|aggregate|bb-test-go-app-WL1-f064ccd5-1b84-42f9-8376-10bed1c8da72'
[2023-05-19 20:38:17.400][39][trace][upstream] [source/common/upstream/cluster_discovery_manager.cc:35] cm cdm: starting processing cluster name service|outbound|nbl-workload-1|test|bb-test-go-app|aggregate|bb-test-go-app-WL1-f064ccd5-1b84-42f9-8376-10bed1c8da72 (status 2) from cluster lifecycle callback in worker_3
[2023-05-19 20:38:17.400][39][trace][upstream] [source/common/upstream/cluster_discovery_manager.cc:45] cm cdm: no callbacks for the cluster name service|outbound|nbl-workload-1|test|bb-test-go-app|aggregate|bb-test-go-app-WL1-f064ccd5-1b84-42f9-8376-10bed1c8da72 in worker_3
[2023-05-19 20:38:17.400][39][debug][upstream] [source/common/upstream/cluster_manager_impl.cc:1083] adding TLS cluster service|outbound|nbl-workload-1|test|prueba2|aggregate|prueba2-WL1-bb547018-ce49-4243-960c-d5a09db5125f
[2023-05-19 20:38:17.411][38][critical][backtrace] [./source/server/backtrace.h:96] #0: __restore_rt [0x7f8bf2c45980]
[2023-05-19 20:38:17.411][38][critical][backtrace] [./source/server/backtrace.h:98] #1: [0x558ea0b49f8a]
[2023-05-19 20:38:17.411][38][critical][backtrace] [./source/server/backtrace.h:98] #2: [0x558ea0b5688f]
[2023-05-19 20:38:17.411][38][critical][backtrace] [./source/server/backtrace.h:98] #3: [0x558ea0b5223f]
[2023-05-19 20:38:17.411][38][critical][backtrace] [./source/server/backtrace.h:98] #4: [0x558ea0aed884]
[2023-05-19 20:38:17.412][38][critical][backtrace] [./source/server/backtrace.h:98] #5: [0x558ea11c55fa]
[2023-05-19 20:38:17.412][38][critical][backtrace] [./source/server/backtrace.h:98] #6: [0x558ea11c54c3]
[2023-05-19 20:38:17.412][38][critical][backtrace] [./source/server/backtrace.h:98] #7: [0x558ea0b0efa4]
[2023-05-19 20:38:17.412][38][critical][backtrace] [./source/server/backtrace.h:98] #8: [0x558ea14d0113]
[2023-05-19 20:38:17.412][38][critical][backtrace] [./source/server/backtrace.h:96] #9: start_thread [0x7f8bf2c3a6db]
2023/05/19 20:38:17 envoy exit with code [signal: segmentation fault]

See attachment segfault.txt
The configuration that ends up for this case is: config_dump.json
After the crash, the config reloads and boots correctly most of the time (sometimes it crashes again, but ends up booting correctly), good-startup.txt.zip
Doing various tests, we came to the conclusion that the problem stops occurring when we remove all the aggregate clusters that we have configured. In this example, it would be replacing the references to service|outbound|nbl-workload-1|sandbox4|test2|aggregate|test2- WL1-192c4785-57dd-4548-90c3-c550140a654c with references to service|outbound|nbl-workload-1|sandbox4|test2|local|test2-WL1-192c4785-57dd-4548-90c3-c550140a654c in the listeners.
"cluster": {
  "@type": "[type.googleapis.com/envoy.config.cluster.v3.Cluster](http://type.googleapis.com/envoy.config.cluster.v3.Cluster)",
  "name": "service|outbound|nbl-workload-1|sandbox4|prueba2|aggregate|prueba2-WL1-192c4785-57dd-4548-90c3-c550140a654c",
  "connect_timeout": "5s",
  "lb_policy": "CLUSTER_PROVIDED",
  "cluster_type": {
        "name": "envoy.clusters.aggregate",
        "typed_config": {
              "@type": "[type.googleapis.com/envoy.extensions.clusters.aggregate.v3.ClusterConfig](http://type.googleapis.com/envoy.extensions.clusters.aggregate.v3.ClusterConfig)",
              "clusters": [
                    "service|outbound|nbl-workload-1|sandbox4|prueba2|local|prueba2-WL1-192c4785-57dd-4548-90c3-c550140a654c",
              ]
        }
  }
}

More evidence:
An envoy running outside of kubernetes in a x86 laptop with ubuntu 22.04 (but configured as if it were a pod sidecar), is running fine for a minute or two, and breaks on one of the updates: segfault_local.txt.zip
Another logs in which the segmentation fault occurred: more_examples.txt
If you could give us a hand that would be great.
TY

I try using  version: e0433be316e635e7b032079aadc5efcf4fcada25/1.23.9/Clean/RELEASE/BoringSSL, dumped from debug-v1.23.9 in my unbutu laptop, but just i get a simple "Segmentation fault" with no line numbers:
[2023-06-07 16:33:22.556][1293649][trace][upstream] [source/common/upstream/cluster_discovery_manager.cc:45] cm cdm: no callbacks for the cluster name service|outbound|nbl-workload-1|test|service-gc-solo-wl2|aggregate|remote in worker_9 [2023-06-07 16:33:22.556][1293649][debug][upstream] [source/common/upstream/cluster_manager_impl.cc:1083] adding TLS cluster service|outbound|nbl-workload-1|test|prueba|aggregate|prueba-WL1-ae4d2cd6-a01c-41bc-9fce-e63d45de8f59 [2023-06-07 16:33:22.556][1293649][debug][upstream] [source/extensions/clusters/aggregate/cluster.cc:80] refresh: cluster 'service|outbound|nbl-workload-1|test|prueba|local|prueba-WL1-ae4d2cd6-a01c-41bc-9fce-e63d45de8f59' absent in aggregate cluster 'service|outbound|nbl-workload-1|test|prueba|aggregate|prueba-WL1-ae4d2cd6-a01c-41bc-9fce-e63d45de8f59' [2023-06-07 16:33:22.556][1293649][debug][upstream] [source/extensions/clusters/aggregate/cluster.cc:84] refresh: cluster 'service|outbound|nbl-workload-2|test|prueba|gateway' found in aggregate cluster 'service|outbound|nbl-workload-1|test|prueba|aggregate|prueba-WL1-ae4d2cd6-a01c-41bc-9fce-e63d45de8f59' Segmentation fault :~ $
In a kubernetes context and no debug build, we get the backtrace info (with memory address, not with line numbers), but not in local with debug build, just i get the "Segmentation fault" log. Plz can you tell me how i can get the line numbers?

Hi @adisuissa, can you give me some information on how to generate the line numbers?
Tank you!


JRE Version 8 Update 121 HTTPS ssl exception

From #199
Background - there is an existing software which is using unirest to access an https website endpoint. The SSL certificate of the website (endpoint) is issued by GoDaddy.
This software was used for a long time by users without issues - However, using the latest JRE Version 8 Update 121, today one user got this SSL certificate exception. It's the same software and the same website which used to work without issues. Initially I thought it's an issue with the user's java installation so I un-installed java and installed back but I got the same error.
Do you have any idea what is happening and why (suddenly) it started to fail? What are my options?
com.mashape.unirest.http.exceptions.UnirestException: javax.net.ssl.SSLHandshakeException: sun.security.validator.ValidatorException: PKIX path building failed: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target at com.mashape.unirest.http.HttpClientHelper.request(HttpClientHelper.java:143) at com.mashape.unirest.request.BaseRequest.asJson(BaseRequest.java:68)

One year late, yet I think it might be related to this SO answer. I haven't tried it though.


check ENOTDIR for readlink

{
  "level": "error",
  "service": "storage",
  "pkg": "rgrpc",
  "traceid": "24d2f1bb7b3cb88c68fd005ced336cce",
  "error": "Decomposedfs: Wrap: readlink error: readlink /var/tmp/ocis/storage/users/nodes/a5066dc8-8566-4aa4-9286-d63c30bdc9a6/somename: not a directory",
  "time": "2021-03-29T19:01:05Z",
  "caller": "/home/vscode/go/pkg/mod/github.com/cs3org/reva@v1.6.1-0.20210326165326-e8a00d9b2368/internal/grpc/services/storageprovider/storageprovider.go:571",
  "message": "error stating: path:\"/home/testfile.txt/somename\" "
}

readlink returns ENOTDIR when a path segment is a file and not a directory. os.IsNotExist won't cover this case: https://github.com/golang/go/issues/18974
Links: https://www.man7.org/linux/man-pages/man2/readlink.2.html

remaining apiWebdavProperties1/copyFile.feature tests require returning a new error type that we can use to map the AlreadyExists error ...

@ishank011 I also removed the failing test from eos because I don't expect it to be an issue for eos ...


[release/6.0] Update application-insights to 2.19.0

This should fix https://github.com/dotnet/source-build/issues/2994.
Also see

https://github.com/dotnet/sdk/pull/22974

@lbussell - Are these changes also needed?  https://github.com/dotnet/source-build-externals/pull/33

Are these changes also needed?

Trying to figure that out now - will add them if needed.

IIRC this is only an issue in the tarball where it removes the online feeds from the known nuget.configs.

...The result is the potential for prebuilts because NuGet non-determinism.


deps: Update gRPC to version v1.49.0-pre1

Commit message:
Update to gRPC v1.49.0-pre1 which is a little newer than 1.48.0 to accommodate refactor to xDS protobuf files in Envoy.
Risk Level: Low
Testing: Unit Tests
Docs Changes: N/A
Release Notes: N/A
Platform Specific Features: N/A
Signed-off-by: Yan Avlasov<EMAIL_ADDRESS>

/lgtm deps


Nix shell for building IREE

Request description
Hi,
I am currently setting up a nix shell to build IREE with StableHLO support.
However, I am getting a weird error whilst running make install, which doesn't reveal what goes wrong other than that the error is on Makefile:166. Because there is a reliance upon a generator with CMakeLists.txt in the IREE repo, I am having a hard time figuring out how I can back track this error.
What I do know is that the error happens after:
Built target LLVMDebugInfoCodeView
What component(s) does this issue relate to?
Compiler
Additional context
AFAIK there is no documentation on reproducible / isolated builds of IREE with configurability atm. It would be very useful with a nix, or containerized version of IREE that can be used without having to change my local environment.
I am happy to share what I have, if necessary.
There is simply too much that can go wrong with having to manually setup the environment.

Which makefile are you running? The best supported build system we have is CMake (https://iree.dev/building-from-source/getting-started/)
We don't have many maintainers/contributors using Nix (https://nixos.org/ and https://github.com/NixOS/nix, right?), so we'd need to pick apart the questions here a bit to offer concrete advice.
If you see errors building LLVM targets like LLVMDebugInfoCodeView, that's mostly outside of our direct control and might be better to debug with the upstream build system (e.g. https://llvm.org/docs/GettingStarted.html#getting-the-source-code-and-building-llvm).

Currently using cmake as seen from my snippet:
buildPhase = ''
      mkdir build && cd build

      export LLVM_DIR=${pkgs.llvmPackages_17.llvm.dev}/lib/cmake/llvm
      export MLIR_DIR=${pkgs.llvmPackages_17.mlir}/lib/cmake/mlir
      export CLANG_DIR=${pkgs.llvmPackages_17.clang}/lib/cmake/clang

      cmake -G Ninja -B build/ -S ../.. \
        -DCMAKE_BUILD_TYPE=Release \
        -DCMAKE_C_COMPILER=${pkgs.llvmPackages_17.clang}/bin/clang \
        -DCMAKE_CXX_COMPILER=${pkgs.llvmPackages_17.clang}/bin/clang++ \
        -DLLVM_DIR=${pkgs.llvmPackages_17.llvm.dev}/lib/cmake/llvm \
        -DMLIR_DIR=${pkgs.llvmPackages_17.mlir}/lib/cmake/mlir \
        -DIREE_BUILD_PYTHON_BINDINGS=ON \
        -DIREE_TARGET_BACKEND_LLVM_CPU=ON \
        -DIREE_HAL_DRIVER_LOCAL_SYNC=ON \
        -DIREE_COMPILER_TRANSLATE_TO_STABLEHLO=ON \
        -DMLIR_ENABLE_BINDINGS_PYTHON=ON \
        -DCMAKE_INSTALL_PREFIX=$out \
        ${platformSpecificCMakeOptions}
    '';

    installPhase = ''
      cd ..
      mkdir -p $out/{bin,lib,include}
      cat Makefile
      make VERBOSE=1 -j$(nproc) install
      cp -r install/* $out/
    '';

I don't see any errors associated with LLVMDebugInfoCodeView, but more an error without any particular information:
       > [ 14%] Linking CXX static library libspirv-cross-msl.a
       > cd /tmp/nix-build-iree-stablehlo.drv-2/source/build/third_party/spirv_cross && /nix/store/a7v65i1rs0jbdjq578pk9h689fif32sd-cmake-3.29.6/bin/cmake -P CMakeFiles/spirv-cross-msl.dir/cmake_clean_target.cmake
       > cd /tmp/nix-build-iree-stablehlo.drv-2/source/build/third_party/spirv_cross && /nix/store/a7v65i1rs0jbdjq578pk9h689fif32sd-cmake-3.29.6/bin/cmake -E cmake_link_script CMakeFiles/spirv-cross-msl.dir/link.txt --verbose=1
       > /nix/store/22a6p8311lg3z4ksrjbj018i7z3vgkva-clang-wrapper-16.0.6/bin/ar qc libspirv-cross-msl.a "CMakeFiles/spirv-cross-msl.dir/spirv_msl.cpp.o"
       > /nix/store/22a6p8311lg3z4ksrjbj018i7z3vgkva-clang-wrapper-16.0.6/bin/ranlib libspirv-cross-msl.a
       > make[2]: Leaving directory '/private/tmp/nix-build-iree-stablehlo.drv-2/source/build'
       > [ 14%] Built target spirv-cross-msl
       > make[1]: Leaving directory '/private/tmp/nix-build-iree-stablehlo.drv-2/source/build'
       > make: *** [Makefile:166: all] Error 2
       For full logs, run 'nix log /nix/store/3p9s9xgbv0g7c0198nj3hhmrgi51a8sg-iree-stablehlo.drv'.

Thanks for pointing these out!
I thought:

-DIREE_COMPILER_TRANSLATE_TO_STABLEHLO=ON

was the STABLEHLO option, but I see now in CMAKELists that it is:

-DIREE_INPUT_STABLEHLO=ON

Thanks!

Btw, I am happy to open a PR if my nix-shell works.


info_density: Express Markdown and message-box spacing as CSS variables.

This PR introduces variables for the 5px and 10px inter-element spacing variables found in the Markdown area. It preserves the legacy values inside .more-dense-mode, and calculates the database-stored values outside of that.
CZO discussion
Screenshots and screen captures:

Before
After, legacy vars (no change)
After, db vars

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

I can't decide how I feel about the minor rounding changes; they seem like a wash though I'm a bit surprised as to their scale. In any case, the approach seems solid and the risk to main is basically none.
Great to see these variables defined, merged, thanks @karlstolley!

Yeah. Especially given the precise areas that will take some of the internment spaces, there's an argument to be made to round those off to whole-pixel values. Most other UI elements have their own line-height that's adjusted for icons and other furniture, so I'm less concerned about subpixel rounding of that value.
Regardless, I expect a lot of discussions around these issues on CZO in the coming days and weeks.


[Major] Qt5 installation instructions don't work on new Ubuntu versions

The installation instructions for Linux include sudo apt-get install qt5-default, but the qt-default package is no longer available for recent versions of Ubuntu.
Running sudo apt-get install qtbase5-dev qtchooser qt5-qmake qtbase5-dev-tools, as responses to the above-linked thread suggest, didn't do the trick for me, and neither did conda install pyqt, conda install qt or conda install qtpy.
No matter what I do (on ubuntu 23.04 or 23.10) I end up with one of these two errors when I try to run snub:
QObject::moveToThread: Current thread (0x1e155c0) is not the object's thread (0x2da9de0).
Cannot move to target thread (0x1e155c0)

qt.qpa.plugin: Could not load the Qt platform plugin "xcb" in "/home/niko/anaconda3/envs/snub2/lib/python3.8/site-packages/cv2/qt/plugins" even though it was found.
This application failed to start because no Qt platform plugin could be initialized. Reinstalling the application may fix this problem.

Available platform plugins are: xcb, eglfs, minimal, minimalegl, offscreen, vnc, webgl.

Aborted (core dumped)

ImportError: $HOME/anaconda3/envs/snub3/bin/../lib/libstdc++.so.6: version `GLIBCXX_3.4.30' not found (required by /usr/lib/x86_64-linux-gnu/libicuuc.so.72)

There should be a less brittle way of installing snub on recent Linux distros.

This turned out to be an opencv-related issue. I got ubuntu installation to work by switching the opencv requirement to opencv-python==<IP_ADDRESS>. Nothing needs to be installed via apt-get.
pip install should work now as of version 0.0.13

Thanks @calebweinreb, I can confirm this works now! Good job getting rid of the apt-get dependency. I'm closing the issue.


Added a better commit message

Hey @ChristianOellers, this is just my fork of the original repo. You'll want to send the PR to avinassh/rockstar :)


linux: Remove unused patch

It's not used since 631b4d56c126e04f59183674404d84a6016bdf29.
Changelog-entry: Remove unused patch of the kernel
Signed-off-by: Gergely Imreh<EMAIL_ADDRESS>

Can one of the admins verify this patch?

@agherzan can you tell Jenkins to build this? Thanks!

@resin-jenkins add to whitelist

@resin-jenkins ok to test

@resin-jenkins retest this please

@agherzan @floion seems like Jenkins don't wanna build this so far, any thoughts? (not urgent, just confusing)

@resin-jenkins add to whitelist


Add Royal Oak, MI City Commission and Interim City Manager

To<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: A budget for the community well-being of Royal Oak
Message:
To the City of Royal Oak's 50th Commission and Interim City Manager:
My name is [YOUR NAME], and I am a resident of Royal Oak. This past week, our nation has been gripped by protests calling for a rapid and meaningful reconsideration of the role of policing in communities as well as an end to racism and anti-Blackness in America. As the Commission prepares the budget for the 2021 fiscal year beginning July 1, 2020, I request to reconsider the budget of the Royal Oak Police Department (ROPD).
The fourth goal listed of the ROPD on the 2020-2021 Recommended Budget is, "To provide exceptional public safety for major events, such as Woodward Dream Cruise and Arts, Beats and Eats." Given that both of those events have been canceled it is reasonable to ask that the $61,000 that has been budgeted for these events be reallocated to sectors that could more effectively benefit Royal Oak, such as community development, health, and welfare, affordable housing, the commissions' sustainability initiative, public works, or education.
The 2020-2021 Recommended Budget has proposed to increase the ROPD budget by $391,090 with $15,000 specifically dedicated to weapons and ammunition. I emphatically reject this proposal. ROPD is slated to receive $19,818,220 including 31% of the proposed Personnel Services budget for the 2020-2021 fiscal year. The ROPD proposed budget is particularly startling compared to the $327,610 that has been proposed for community development. As a resident of Royal Oak, I demand a budget that reflects a commitment to community well-being, rather than one that empowers the police forces that teats it apart.
I call on you, The City of Royal Oak's 50th Commission and Interim City Manager, to decrease the ROPD budget and instead meaningfully reallocate funds towards investing in; social workers who are better equipped to handle challenges that are often handed to police such as substance abuse, mental illness, homelessness, and domestic violence, as well as infrastructure for affordable housing, child care, and other critical community needs.
We can be a beacon for other citizens and cities both in the Metro-Detroit Area and nationwide to follow, only if we have the courage to catalyze change.
Sincerely,
[YOUR NAME]
[YOUR ADDRESS]
[YOUR EMAIL]
[YOUR PHONE NUMBER]

https://www.google.com/url?q=https://www.romi.gov/DocumentCenter/View/26724/2020-2021-RECOMMENDED-BUDGET-PDF&sa=D&ust=1591597108425000&usg=AFQjCNFy2n99lSM-l_2Y3g13c6qOrNUrkA
budget source!

Need this approved ASAP! There's only two meetings before they approve the budget, Today (!) June 8th & 22th

on it now

Amazing! I’m eagerly waiting it share it with my network
Enviado desde mi iPhone

El jun. 9, 2020, a la(s) 3:59 p. m., Christian Neal-Herman<EMAIL_ADDRESS>escribió:

on it now
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Update please: the subject line was removed in the final code. It should read: A budget for the well-being of Royal Oak
Enviado desde mi iPhone

El jun. 10, 2020, a la(s) 1:03 a. m., Tomas Roy<EMAIL_ADDRESS>escribió:

Closed #612 via #1148.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Response for anyone interested:
Thank you for reaching out to me on this important issue. Like you and the millions lifting up their voices across the country, I am outraged and anguished by the daily onslaught of injustice resulting from the systemic racism we continue to fail to address as a community and as a nation.  I’ve been proud to march with demonstrators in Royal Oak over the last couple weeks and I agree that we must act now to bring about real change.
The resolution I introduced last Monday night, which was passed unanimously by the commission, is a commitment from our city to become much more proactive by formally recognizing the plague of systemic racism and the city's obligation to be a part of the solution.
The first step I proposed was also unanimously approved on our most recent agenda--item 18-- where I recommended that we form a Royal Oak Commission on Human Rights which would help inform city policies, practices, and like you’re suggesting, investments, to ensure we are doing the work to break down institutional bias and systemic racism by creating a formal space for our community to raise and address these issues. The change needed isn't just about policing, but about taking a real hard look at any potential bias in all city policies and practices. In my vision, this body would not just field complaints and hear concerns but would make substantive recommendations on meaningful policy change the city could adopt.
We have much to do for sure, and while I’m happy to engage in conversation around any ideas that could help us be the best community we can be, I’m not clear how arbitrary cuts to our police department will yield the results we want. While I certainly agree with calls to demilitarize, right-size, reform and restructure departments across the country, I think it’s important to have those conversations in the context of each community. For example, the Royal Oak Police Department, which has 1.3 officers to every 1000 residents, is a world of difference from LAPD which has 2.26, Detroit at 3.5, or Chicago at 4.4.
Looking at any budget or line-item as just a number absent any context and making random cuts without a detailed understanding of the nuts and bolts of it—like the fact that a huge portion of our police budget goes to pay obligatory retirement and healthcare legacy costs—doesn’t, in my opinion, help get us where we need to be as a community and is entirely likely to do more harm than good. While I share the passion and the sentiment in the letter you signed, it contains some seriously inaccurate statements about both our public safety budget and the city budget as a whole.
That aside, I wholeheartedly agree with increasing public investment in social services and community supports. But again, this is a wildly different discussion depending on what city you’re talking  about and what programs it has authority over. The city of Royal Oak doesn’t manage education, social services or health service delivery in any official way. Those are managed by completely different government entities including Oakland Schools, Royal Oak School District, Oakland County, and the State of Michigan. But there are things we can do. I have been steadfast in my support for increasing investment in quality of life issues in our city such as parks and recreation, environmental sustainability, senior services, development of affordable housing, and improved service delivery across departments. I will continue to advocate for these things and for a budget that reflects our needs and values. But again, I fail to see how arbitrary cuts to any of our departments, including the police department, without any clear strategy would result in better outcomes for the community.
There is a clear need for serious reform across the criminal justice system and how we approach policing as a society in particular. That is undeniable. But I also think we have a very good department in Royal Oak. We are far from perfect, have made mistakes, and there’s certainly more work to do, but here’s some highlights of where we are right now:
Royal Oak incorporated body cameras last year at the recommendation of our police chief
Ours was one of the first departments in Michigan to conduct implicit bias training.
Royal Oak officers receive de-escalation training when they are hired, and it is required twice a year throughout their careers.
Our department has adopted and enforces internal policies consistent with the 8 Can't Wait campaign.
While I understand there are issues to address, I honestly believe that our city is working every day to instill a culture of service and compassion into our department. Our chief of police has spoken at several demonstrations that have taken place in our city over the last couple weeks while officers distributed water to protesters. It’s a small gesture but it provides a pretty dramatic contrast with how protesters were received by departments in many other cities.
I know that like every other community we've got work to do and I am committed to advancing change. I assure you that as long as I'm at the commission table, I'll do everything within my power to ensure Royal Oak is striving to live up to our principles, represent our values, and be the best community it can be. One that lets the world know that we know that Black Lives Matter. And like I said, if there's a case to be made as to how a specific cut to the department's funding would yield a better, safer, healthier, or more just community, I'm happy to listen and have that conversation.
Thanks again for reaching out, for your passion, and for driving us to be a better city. Please don't hesitate to contact me with any other questions.
Sincerely,
-Kyle
Kyle DuBuc
Royal Oak City Commissioner


List map string field do not trigger validate.

Describe the bug

Steps to reproduce

Set the list-map-string field as required.
Create a resource using the template and set the value to empty.
After clicking the "Save" button, there is no validation message for the field.

Result

Submit successfully.
Expected behavior

Can't submit the form data, and there is validation message.
Environment

Walrus version:

Additional context
Add any other context about the problem here.

Environment

Walrus version: main bd8b2f4560a522fcfb061a081817869f1a088f3a
UI version: dev-95fef06

Test Result:
Cannot edit the value now.
https://github.com/seal-io/walrus/assets/26084756/8106aec5-406a-4a76-9db3-023348502a1e

Environment

Walrus version: main bd8b2f4560a522fcfb061a081817869f1a088f3a
UI version: dev-b3f75f4

Test Result:
Validation fails for variables of type may(any).

Environment

Walrus version: main bd8b2f4560a522fcfb061a081817869f1a088f3a
UI version: dev-e738a7a
Test Result:

list(map(string)): pass
map(any): pass
list: failed, no validation on the UI.

Environment

Walrus version: main bd8b2f4560a522fcfb061a081817869f1a088f3a
UI version: dev-3da39a5
Test Result:

When creating or editing resources, the list-type properties are still not validated.
https://github.com/seal-io/walrus/assets/26084756/82fb8710-edc1-4257-8668-cb6601d06be3

Environment

Walrus version: main bd8b2f4560a522fcfb061a081817869f1a088f3a
UI version: dev-1b96807
Test Result: pass

Environment

Walrus version: main bd8b2f4560a522fcfb061a081817869f1a088f3a
UI version: v0.5.1-rc1
Test Result:

The required for map(string) type is not taking effect.

Environment

Walrus version: main bd8b2f4560a522fcfb061a081817869f1a088f3a
UI version: dev-41c46ad

Test Result: pass