feat: Enable CI for tag v* and prepare for tag v2.0.0

Signed-off-by: imjoey<EMAIL_ADDRESS>This PR is going to do the preparations for the release v2.0.0, which contains enabling CI for tags and adding changelog. If this PR is got merged, then I will push the tag v2.0.0 from this commit.
Fixes https://github.com/api7/apisix-build-tools/issues/75.

ping @spacewander @tzssangglass @nic-chen for review. Many thanks.


Fix Generator

We need to get the generator code working again.
My cursory investigation also leads me to wonder if we shouldn't be basing our code off of W3C HTML 5.1 instead of the HTML Living Standard like we're currently doing. If we are going to use the latter, we probably should be running the generator regularly (before releasing). There appears to be considerable disdain toward W3C from WHATWG members, but I also kind of like the idea of saying we are "version X" compatible. 🤷♂
Living Standard also has a raw file on github we can parse instead of working with a much larger html file.
This might also mean updating Jari's IDL code? Or forking it? Or rewriting something similar but different?

For giggles I tried to implement something with some complicated nokogiri parsing, completely avoiding the webidl gem. It.. mostly works, but obviously I needed to use hard coded variables and things that aren't desirable, and I'm stuck on getting the GlobalEventHandlers and WindowEventHandlers to work, since they aren't at all delineated in the xml. And multiple elements implement them; right now now everything is getting put in HTMLElement where only GlobalEventHandlers belong. :-/
Anyway, this is it. Not sure how worthwhile it is to keep going with it. I think it'd get us 90% of what we want, but then there's a chance that it's going to miss things. https://github.com/watir/watir/commit/6de374
There have been a bunch of changes to the spec in the past 2 years, though it's hard to say how much because the order of the old and new spec files is also very different.

I'm not sure when I'll have time to take a look at it, but generally usage of WebIDL gem is simple, we just need to extract IDL interfaces correctly and pass them in.
In past, most of the issues were coming from the fact that HTML spec had bugs and/or invalid IDLs.

Fixed now.


Fix a few more regressions from #4235.

Explanations in the commit messages.
For the second (doubled slashes in Apple auth endpoint), following Greg's comment at https://github.com/zulip/zulip-mobile/pull/4265#issuecomment-697040991 — chat.zulip.org seems perfectly happy to accept the double-slash version, like it did before cfebe3f53752968235e33ee428e2239025b5c11c 😆. At first I was confused about why it seemed to work both before and after my commit here. Anyway, that's probably why.

chat.zulip.org seems perfectly happy to accept the double-slash version, like it did before cfebe3f . At first I was confused about why it seemed to work both before and after my commit here. Anyway, that's probably why.

Hmm! Now that you mention that, I do seem to recall finding that some classes of server API routes tolerated duplicate slashes, while others didn't, just because they went through different pieces of infrastructure code that handled that case differently. The web auth flows certainly use some distinctive pieces of infrastructure code, so I guess it fits that they might be in the group that tolerates such variations in the URL while our main API endpoints aren't.


How to use --stylesheet

He always looks for the weasyprinter\python folder, how to find other folders?
Because css is a separate file.

We could add an option to specify the -s parameter as mentioned in the weasyprint docs.


Correct "mdk" typo to "msk" for SDP solver

I installed SCIP-SDP this week and noticed this important typo in the install directions. It's the first time the "msk" option is mentioned so it's especially confusing.
Thank you for your hard work on SCIP-SDP. It's a great project and one of a kind.

Duplicate! Just realized there's an open pull request on this issue already.

Thanks! It is also good to know that someone reads the install files.


Remove Self: Type bounds in Encode / Decode implementations

Notes on the first commit:

This changes a few panic!s behaviors. I'm assuming not using a format string there was an oversight, rather than those panics really being meant to be catch-able as std::io::Errors.
It looks like enter_runtime for tokio was actually wrong before, due to the runtime guard being dropped before calling the function argument.

About the second ("main") commit: I've only tested that the change of the slice implementation doesn't result in compilation errors, nothing more.

This allows one to work around #298 like this:
#[derive(sqlx::Encode)]
struct Foos<'a>(&'a [Foo]);

impl sqlx::Type<sqlx::Postgres> for Foos<'_> {
    fn type_info() -> PgTypeInfo {
        PgTypeInfo::with_name("_foo")
    }
}

query_as!(
    Whatever,
    "<QUERY with $1 of type foo[]>",
    Foos(&foo_vec) as _,
)

Very cool. Nice stuff @jplatte.

Thank you very much for this. At least it eases some hardships

Does this work for Decode implementations? I'm getting an error :
thread 'main' panicked at 'internal error: entered unreachable code: (bug) use of unresolved type declaration [kind]', /home/pintert3/.cargo/registry/src/github.com-1ecc6299db9ec823/sqlx-core-0.5.5/src/postgres/type_info.rs:744:17
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Sounds like you probably used the wrong type name. Is the postgres type a custom one and did you prefix it with an underscore in PgTypeInfo::with_name()?

Ya, I did. Well to give more context, I made an adhoc type from this query:
        select people.*, (select array(select c from cars c where c.id = $1)) "cars_owned:_" from people where people.id = $1

$1 was taken from a variable id:i32.
So I created a an output type only for getting the result of the query


Don't make a symbol for nil

Alternative for #377 since a symbol for nil doesn't have much value now.
Don't make a symbol for nil and enable semantic tokens for SymbolBasicValue

Currently prefer nil not having a type.


apk install fail caused by incorrect config

android-minSdkVersion should be moved outside the platform tag

This can be closed, https://github.com/phonegap/phonegap-template-framework7/pull/10 removed the android-minSdkVersion
cc @macdonst @devgeeks


make properties pub

i wanted to use some of the property impls in a ui crate, but they were pub(crate).
i've forked but if you're happy to make them pub that would be great.

I don't see a problem exposing those.


Amend Text On Upgrade Screen To Show Actual Organisation Name

After having clicked the 'upgrade availble' button the screen displayed showed the message 'This will upgrade all data to version 2.02 compatible data for organization 297.'
Please could the message be amended so that it uses the actual name of the organisation rather than 'organization 297'

This issue has already been fixed.


Add a test to ensure public core headers are C89

Built on #4562 which should be merged first.
Fixes #4565

Fixes #4565

Fixed


Update DAGs

Refactored DAGs to import from includes.
New version of weaviate provider had upsert and branch operator for check schema.
Add cohere rerank to schema
Moved env vars to .env
Added openlineage data source
Fixes to enable astro deploy
Added ASK_ASTRO_ENV var to enable the same code to run in all environments. Specify 'prod', 'dev', 'test', or 'local' in each astro deployment or local CLI.

@mpgreg I have rebased your PR on main branch (that contains pre-commit hooks) and resolved merged conflicts & fixed issues.

Why are tests not running on this PR 🤔

Had to change a CircleCI setting


How to set header title with value from redux state

Hi, i connected react-navigation to redux and when I try to add values from redux as part of the header title, I get an error. What is the correct code to accomplish this?
my code:
    //define navigation option - show header
    static navigationOptions = {

        title: `Total: K${this.props.totalCharge}`,
        header: {
            visible: true,
        }
    };

the error:

AFAIK you can't hook directly to a redux store but you could do something like this:
class SuperManComponent extends React.Component {
    {...}
    static navigationOptions = {
        title: ({ state }) => `Total K:${state.params && state.params.totalCharge ? state.params.totalCharge : ''}`,
        {...}
    }
    {...}
    componentWillReceiveProps(nextProps) {
        if (nextProps.totalCharge) {
            this.props.navigation.setParams({ totalCharge });
        }
    }
}

const connectedSuperMan = connect(state => ({ totalCharge: state.ReducerYo.totalCharge }))(SumerManComponent);

@rmevans9 is right, that is one way to do it, the other way is to have a pre-connected title component:
const MyTitle = ({ navigation, text }) => <TitleView text={text} navigation={navigation} />;
const MyConnectedTitle = connect(storeState => ({ text: storeState.title }))(MyTitle);

class SuperManComponent extends React.Component {
    {...}
    static navigationOptions = {
        title: (navigation) => <MyConnectedTitle navigation={navigation} />

@rmevans9 thanks, my problem was solved

@rmevans9 Your solution worked for me too, cheers!

@rmevans9
I did the same however componentWillReceiveProps gets called infinitely. Any ideas? Thanks

@yingdongzhang
Check to make sure only when the nextProp is chaging
componentWillReceiveProps(nextProps) {
    if (this.totalCharge !== nextProps.totalCharge
        && nextProps.totalCharge) {
        this.props.navigation.setParams({ totalCharge });
    }
}

@cmendes0101
Thank you that makes sense. Cheers

The componentWillReceiveProps method is deprecated from React 16.3, so this is no more the good place to set the header title. Any idea where we could do that ?

its really simple you can make stateless component and connect it with redux like this :
const view = ({ state, color }) => {
{state.something}
}
export default connect(
(state) => ({
data: state.data,
}),
null
)(view);
and then use it in static options :
static navigationOptions = ({ navigation }) => {
    return {
        tabBarIcon: ({ tintColor }) => <view color={tintColor}/>
    }
};

its really simple you can make stateless component and connect it with redux like this :
const view = ({ state, color }) => {
{state.something}
}
export default connect(
(state) => ({
data: state.data,
}),
null
)(view);
and then use it in static options :
static navigationOptions = ({ navigation }) => {
    return {
        tabBarIcon: ({ tintColor }) => <view color={tintColor}/>
    }
};

Please see this answer for an easier way to do this: https://stackoverflow.com/a/51883306/6942666


⚠ Beulah Revival Network has degraded performance

In a769d34, Beulah Revival Network (https://beulah.clicki.digital) experienced degraded performance:

HTTP code: 200
Response time: 14449 ms

Resolved: Beulah Revival Network performance has improved in 3078bc4.


Replace internal links in documents that link to a migrated document

Whitehall allows publishers to link to other Whitehall documents using an internal URL (rather than the public one).  On migration of a document out of Whitehall, links to this document will no resolve.  Therefore, we need to replace internal links to this document in other documents with the public URL.
E.g.
a [link](/government/admin/policies/101010)

in a document would be changed to (when edition 101010 is migrated):
a [link](https://www.gov.uk/news/test)

Trello card: https://trello.com/c/jSKNSjtA

I've refactored this out of the controller, removed the nested loop and stopped the translation's updated_at from being changed.

Good point.  I've made a small change to work through all editions for a document.


Adjust pkg/netns go formatting to make Go1.19 happy

Fix a go 1.19 format warning (which breaks many tests)

Thanks @aspsk!
Go related linter checks passed and this PR only changes Go comments. No need to run e2e tests, merging.


Non-imported GPG key leads to hanging 'migrate2rocky.sh -r'

[root@tux ~]# ./migrate2rocky.sh -r

Removing dnf cache
Preparing to migrate CentOS Linux 8 to Rocky Linux 8.

Determining repository names for CentOS Linux 8.Importing GPG key 0x24098CB6:
 Userid     : "Grafana<EMAIL_ADDRESS> Fingerprint: 4E40 DDF6 D76E 284A 4A67 80E4 8C8C 34C5 2409 8CB6
 From       : https://packages.grafana.com/gpg.key
^C
[root@tux ~]#
Failed to fetch info for package apr-util-ldap.x86_64.

An error occurred while we were attempting to convert your system to Rocky Linux. Your system may be unstable. Script will now exit to prevent possible damage.

A log of this installation can be found at /var/log/migrate2rocky.log
[root@tux ~]#

Thus my conclusion was:
[root@tux ~]# dnf info apr-util-ldap.x86_64
grafana                                                                                2.5 kB/s | 454  B     00:00
grafana                                                                                19 kB/s | 1.7 kB     00:00
GPG-Schlüssel 0x24098CB6 wird importiert:
 Benutzer-ID  : »Grafana<EMAIL_ADDRESS> Fingerabdruck: 4E40 DDF6 D76E 284A 4A67 80E4 8C8C 34C5 2409 8CB6
 Von          : https://packages.grafana.com/gpg.key
Ist dies in Ordnung? [j/N]: y
grafana                                                                                7.0 MB/s | 5.4 MB     00:00
InfluxDB Repository                                                                                235 kB/s |  33 kB     00:00
Verfügbare Pakete
Name         : apr-util-ldap
Version      : 1.6.1
Release      : 6.el8
Architecture : x86_64
Size         : 25 k
Quelle       : apr-util-1.6.1-6.el8.src.rpm
Repository   : appstream
Summary      : APR utility library LDAP support
URL          : http://apr.apache.org/
Lizenz       : ASL 2.0
Description  : This package provides the LDAP support for the apr-util.

[root@tux ~]#

Unfortunately, this didn't solve anything but lead to exactly the same issue again (and again). The only thing that really helped was to set repo_gpgcheck=0 and gpgcheck=0 in /etc/yum.repos.d/grafana.repo temporarily during the migration.

Might be a duplicate of #85

Closing ... Duplicate of #85


Threshold-Lines in line charts are rendered outside the chart and raise javascript error

When using threshold lines in line-chart, they are rendered outside the chart. The corresponding tooltip is shown in the right place.

t)
Additionally, I get this javascript error message:

Here is the corresponding axis configuration:
axes: {
left: { mapsTo: "value",
thresholds: [
{
value: 20,
label: "My theshold"
},
],
},
bottom: {
mapsTo: "date",
scaleType: "time",
  },
},

@usefool Hi there. A better place for this issue would be the carbon-charts.

Ok, opened it there: https://github.com/carbon-design-system/carbon-charts/issues/1214


Support multiple configuration sources

What should we add or change to make your life better?
The current implementation of IProxyConfigProvider is registered as a singleton service, which prevents the use of multiple configuration sources.
Registering a custom programatic source prevents the use of the configuration file.
Why is this important to you?
Having multiple configuration sources could allow us to have a set of default Routes / Clusters on the appsettings.json while we perform programatic configuration on a second implementation of IProxyConfigProvider.
It may also allow for overriding some Routes/Clusters with less destinations for Debugging/Developer workflow.

I am facing a similar issue.
At first I said to myself: no big deal, I'll just throw in a composite, like
services.AddSingleton<IProxyConfigProvider>(sp =>
            {
                return new CompositeProxyConfigProvider(new [] {
                    new ConfigurationConfigProvider(sp.GetService<ILogger<ConfigurationConfigProvider>>(), config, sp.GetService<ICertificateConfigLoader>()),
                    new ImmutableProxyConfigProvider(GetRoutes(), GetClusters())
                });
            });

The idea was for the composite to expose an IProxyConfig with routes and clusters that are a concatenation of the two (optionally reporting errors for duplicates etc.)
It's not a general solution (no idea how to deal with the change token, but in my case is always the null one), but nevertheless it's not possible because ConfigurationConfigProvider is internal.
Is there a reason why is it so? Wouln't it be better to have it public to allow more scenarios (like this one?)

ConfigurationConfigProvider is internal.
Is there a reason why is it so?

There hadn't been a need for it yet. We try to keep infrastructure types internal so we can update their API as needed without breaking people.
The composite config provider is an interesting idea. Feel free to download the repo and do some more experimenting. If you can show the composite idea works then that will be a good reason to make ConfigurationConfigProvider public.

How hard is this to do by having a single config provider that then manages and calls out to further child providers? It would then be responsible for managing collisions, and coordinating updates when multiple sub-providers are in flux.

ConfigurationConfigProvider is internal.
Is there a reason why is it so?

There hadn't been a need for it yet. We try to keep infrastructure types internal so we can update their API as needed without breaking people.
The composite config provider is an interesting idea. Feel free to download the repo and do some more experimenting. If you can show the composite idea works then that will be a good reason to make ConfigurationConfigProvider public.

It totally makes sense, I do the same on my code, trying to expose just interfaces and not implementation (if not needed).
I will check give it a try as soon as possible and be back here to report my findings. Thanks!

should we do a sample for this?

For me that would also be an important feature since that opens up a lot of possibilities. Since I was asked in #1455 to copy my notes here is how i solved it  for the time being.
For now i am doing LoadFromConfig to get the routes, letting this parse and then doing this:
var oldConfig = services.BuildServiceProvider().GetRequiredService<IProxyConfigProvider>();

builder.Services.AddSingleton<IProxyConfigProvider>(new InMemoryConfigProvider(oldConfig.GetConfig().Routes));

This approach is bad because then I need to build the ServiceProvider in the ConfigureServices method.
However, looking at ConfigurationConfigProvider there is a lot of logic for the parsing from a config file so i don't want to recreate/copy that.
I have a few suggestions how it could be done better

The entire ConfigurationConfigProvider could become public
If that exposes too much logic, extracting the parsing logic into a new interface so that this can be used in other IProxyConfigProvider implementations would also work.
Maybe it should be allowed to use multiple IProxyConfigProviders and having their configurations combined when the proxy is used.
Another option would be to separate the routes from the cluster configuration so that different providers can contribute these. But that seems very limited in general utility.

To me 1 or 2 would be enough because then you can manually combine the providers together. It would require ever consumer that needs this functionality to recreate the merging/conflict handling logic but it is probably acceptable for now.
In addition, I believe the InMemoryConfigProvider should also go into the library since its also a very common scenario. Now it is in the samples and will be copied in a lot of projects.
Otherwise, I didn't encounter any issues with the library, its great for a first version!

Option 3 seems likely/useful, if we can decide on how to combine the data sets and resolve conflicts. The infrastructure is almost ready for this now, and we wouldn't have to expose new surface area.

Triage:

Scoping to merging routes and clusters nodes is reasonable. If there is overlap, based on policy we should act (one wins vs. error).

This would definitely be appreciated. Glad there's a PR for it.
I actually implemented this morning a solution that removes the config-config provider from DI and then uses the factory from the descriptor to inject it into my own class. That class returns the original config plus a dynamically generated set of routes based on a list (that follows the same "template") allowing us to keep the config.json short without relying on a massive regex route constraint.
This would definitely ease that a bit as we'd no longer need to track/wrap the initial config, nor hack the DI.. and can just add on the bits what we need seperately


Our layout for tab 1 and 2 are slightly larger than screen size, forcing users to scroll.

The user experience would be smoother if we could adjust the layout/size of our plots and tabs so that everything was visible without forcing the user to scroll.

presently resolved the issue for tab1 by making layout changes to layout and the plot. Will further resolve it for tab2 once the tab2 layout is finalized.


Merge a part of reference templates to related resource templates

Replace a set of reference including statements in main templates with resource templates.
Add several new resource templates, which contained in mapping spreadsheets while not revealed before.
Add comments in the front of template resource templates introducing reference varaibles used in this template.
Remove condensed reference templates (approximately 31 files).

Let's make sure we have the same changes made in the generation tool.


Support other sources as input images

If other input images are specified, they appear as a browse button to load a static image files.
It would be nice if other source could also be selected (for dynamic masks, for instance)

Could be useful. Complexity depends on how easily OBS exposes the output texture for specific sources. The properties would also need some way to identify textures that should be populated from a source rather than a texture. (Shader annotation syntax?)

afaik from skimming the source, this repo is mostly a passthrough for OBS's existing filter plugin API, exposed to allow dynamic loading.
imo the shaderfilter's usecase is as a simple "filter", more complex effects should use external programs to construct their own pipelines, to be captured by OBS


fix: fixing typo on guide

Description

small typo

Testing required outside of automated testing?

[ ] Not Applicable

Screenshots (if appropriate):

[ ] Not Applicable

Rollback / Rollforward Procedure

[ ] Roll Forward
[ ] Roll Back

Reviewer Checklist

[ ] Description of Change
[ ] Description of outside testing if applicable.
[ ] Description of Roll Forward / Backward Procedure
[ ] Documentation updated for Change

:tada: This PR is included in version 1.153.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Travis builds post coverage info to coveralls.io and add README badge.

Seems vscode already has istanbul instrumentation enabled in tests. It would be nice to see the coverage info as a badge in the REPO readme.
You'll need to add the following environment vars in travis.
COVERALLS_REPO_TOKEN = XXXX (Generate from https://coveralls.io/repos/118314/regenerate_token)
COVERALLS_GIT_COMMIT = HEAD

Hi @nojvek, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Manoj Patel). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

Changes Unknown when pulling db9a0c5e4ebc4b1de44852b6313b2b8ccdf0dc50 on nojvek:master into ** on Microsoft:master**.

Wow that's pretty cool.

Anyone from VSCode team ?

That would be me. Let's do it in August, though.

Sounds good. Will ping you in August.
I'm guessing, the delay because you're all flat out on a release?
On Thursday, July 28, 2016, João Moreno<EMAIL_ADDRESS>wrote:

That would be me. Let's do it in August, though.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/vscode/pull/9791#issuecomment-235821492,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA-JVKw92dYcAkebcCbvSN2r3M4UhN1kks5qaFvBgaJpZM4JVR6c
.

Thanks, pretty cool!


feat: add support for collpased header height

Hey @PedroBern, this allows to configure a height for the header when collapsed, in other words, the header will be considered collapsed once we reach, header height - collapsed height, I tested all the examples and they seem to be working properly, so I believe I got all the right variables in their place, but if you can give a new look, since you know the project/behavior better than me
Closes #5

@PedroBern Related to the snapThreshold, do you think it should be applied to the headerHeight or the headerScrollDistance(headerHeight - minHeaderHeight)? at the moment it's applied to the headerScrollDistance

@PedroBern Related to the snapThreshold, do you think it should be applied to the headerHeight or the headerScrollDistance(headerHeight - minHeaderHeight)? at the moment it's applied to the headerScrollDistance

headerScrollDistance is correct 👍


Restaurant filters

Also made the changes requested in @medmouine 's PR for the restaurants endpoints - directed the PR into med's.

Also a small point, we should support pagination. See the get all users call:
    const { page, q } = req.query
    const limit = req.query.limit ? Number(req.query.limit) : 10
    let query = {}

    if (q) {
      query = {
        name: new RegExp(q, 'i')
      }
    }

    const docs = await User.find(query).limit(limit).skip(limit * page)
    const count = await User.count(query)

    const users = docs.map(d => d.toDTO())

    res.status(200).send({
      items: users,
      total: count
    })

Let's support the page query param + adjust to follow the same structure of return (items and total)

All comments addressed - good for another review!


[fix] Execute SHA2-256 KAT before executing the ROM integrity check

This changes re-arranges the KAT and ROM integrity check execution such that the SHA2-256 KAT is run before running the ROM integrity check. This is needed per FIPS requirement AS10.20.

@JohnTraverAmd Can you review this?


Merge the new-ui branch into main

The new UI is ready to be integrated into konflux-ci.

Login is not possible using the username (user1/user2)
We need to use the email to login (user2@konflux.dev).
Is it expected?

| Login is not possible using the username (user1/user2)
We need to use the email to login (user2@konflux.dev).
Is it expected?
wasn't it always like that?

wasn't it always like that?
No, we used to login using the username (user1/user2)

@avi-biton I fixed the README.


Running costs

This tech looks so cool! How does it compare with OpenAI api in terms of cost? How much does it cost on average to run each request?

Thank you :)
The costs are exactly what the OpenAI api charges.
For the initial messages it typically uses 500 ish tokens, so that's around 1 cent per message.
If a conversation grows, the cost will start to rise, as the message history builds up to use more tokens.
Maximum is around 4 cents per message (2k tokens) when the conversations get long enough.
You can configure the max tokens to be lower, so that will be cheaper, but as a result Shell will remember worse.
You can configure the max tokens to be higher so that Shell can remember more, up to around 3.5k tokens per message (+ 500 tokens per response). That will bring the cost to be higher, of course.

Thank you for clarifying. If I understood the project well, it runs on top of the  Wolfram Alpha infrastructure. Thus, the costs you mentioned must come directly from them.
Did I get it right? 😊

The project runs by using OpenAI's GPT-3 Large Language Model. Wolfram Alpha integration is just one of the features. The main costs come from using OpenAI's API.
Wolfram Alpha integration will also be costly (not as much as OpenAI I think) if we use something other than the simple version, but currently we get 1000 "simple" Wolfram Alpha requests for free :)


Fixing #211

Fixing https://github.com/mzabriskie/axios/issues/211

@dublx Could you please update README.md?

Hi, I'm not too experienced with PR's.. do I update README and then submit new one?
Anyway, reason I change max code was because then the promise would be rejected.
I will change it then:  check if maxRedirects === 0, the idea being that there should be no redirects, but also no promise rejection. Agree?

You can just push a new commit to the same branch. It will be automatically added to your PR.
As for max code, my suggestion is to not change the status code error range at all. If you don't want axios to reject the promise for 3XX responses, you will be able to do that via validateStatus config option when #308 is merged.

@dublx Please let me know if you need any help

@nickuraltsev Hi, I've really enjoyed using axios in the chatbot I'm creating for my company. I need this PR to support some needed functionality and currently plan to fork my own copy until this is available. If it is ok with @dublx I would be comfortable with cleaning this branch up to not change the status code error range, update the README and fix any conflicts. Alternatively I could simply create a new PR.
In short, I'd like to see the work I'm about to do be put back into axios to share with others and so I don't have to maintain my own fork. Please let me know if I can be assistance.

@cspray Yes, I think the right way to deal with this is to merge @dublx commits into your own branch, make necessary changes, and submit a new PR. If you could do that, it would be great. Thank you!

@nickuraltsev  Will do!

Hi guys @dublx @nickuraltsev @cspray @mzabriskie, is there a chance that this PR would be merged soon? I'm really looking forward to it :) Can I help with it somehow?
Cheers :)

Merged


Synchronization of pools of observational data between different compute platforms

Is your feature request related to a problem? Please describe.
There has been a growing demand to synchronize the pool of obs data supported in ESMValTool to other platforms than DKRZ and CEDA-Jasmin.
Current situation: when a cmorizer is added or updated, I happen to add the data (raw and cmorized) to DKRZ and then run some rsync commands to ship these data to CEDA-Jasmin (see doc). Due to lack of space on Jasmin, only OBS/Tier1 and OBS/Tier2 datasets are synchronized.
Drawbacks: this mechanism requires an account on both platforms and cannot be triggered with a cronjob since those are not liked by HPC system admins.
This situation would make it difficult but not impossible to replicate parts of the DKRZ data pools to other platforms. We should discuss how to design a scalable system that would ensure data availability on platforms where ESMValTool is used and develop.
Points to consider:

Central repository: should DKRZ remain the reference place where data are stored and from which maintainers of obs data pools on other platforms would grab the data from? Or should we think of moving data to a more accessible place, e.g. ESGF (long-term project)?
Transfer mechanism: Shall we aim at replacing the rsync solution by something like Globus (this was not really possible on DKRZ-Mistral)?
others?

Disclaimer: I assume that we are talking here about transferring data for which there would be not license issue or restriction (e.g. Tier3) and that transfer of Tier3 would be done with great care...

Hi Claire @hot007

and deal with ESM core probably failing because of vagaries of it being a different system

I can assist with this (if you are referring to ESMValCore, that is) - let me know if you want a fix and if so, please provide me with a bit more details on specific issues, platform etc :beers:

Thanks @valeriupredoi , I've been meaning to reach out to you folk to help get us up and running properly. Next time I come up for air I'll get back to you - it may turn out to be a non-issue, it's just getting ESMValTool set up from scratch is a bit of work :) Really appreciate you reaching out, I will be in touch soon-ish.

Hi @hot007, nice to hear about your interest in ESMValTool! Obs4MIPs is actually supported in ESMValTool, so for data that is published in Obs4MIPs no extra cmorization is needed. For an example, have a look here.
As for your local trouble, while we have tried to make the installation of ESMValTool as painless as possible, different machines can trigger different problems. Perhaps it would be easier to just hop on a video call and sort things out?

sounds like a plan @hot007 :beer:

Hi @hot007, thanks for your interest in using ESMValTool! Don't hesitate to contact us on github for issues related to the installation of ESMValTool.

Anyway, in the short-ish term (months), would you recommend fighting the EVT cmoriser for obs data on our system (not including the Australian data for which a CMORiser doesn't already exist), or look to replicating from DKRZ in a structure that's already known to work with EVT? (ie can it be made public at DKRZ or would an account there be required?)

I doubt it would be feasible to set up replica of the DKRZ pools soonish for various reasons. Nevertheless, we have recently upgraded the handling of obs data in ESMValTool to make it more user-friendly (since version 2.5.0). Once ESMValTool is installed on your shared analysis platform, the maintainer could use a few ESMValTool command to download and cmorize data (see this doc) to create a local pool of obs data. Some more explanations are provided in https://github.com/ESMValGroup/ESMValTool/discussions/2560 to explain each command.


Find out maximum supported integer for encoding/decoding

Perform some research to find out what the maximum number is for encoding and decoding. Document these limits and create a new issue to increase the limit.

$obfuscator = new IdObfuscator('');

for ($id = PHP_INT_MAX; $id > 0; $id--) {

    $encoded = $obfuscator->encode($id);

    try {
        $decoded = $obfuscator->decode($encoded);
    } catch(TypeError $exception) {
        continue;
    }

    if ($decoded === $id) {
        echo 'Max number is: ' . $id;
        break;
    }
}

I've run this script to calculate the biggest number and found out that it's affected by the character set.
The salt seems to be irrelevant.

So the encode works fine, but decoding again is where the problem is?

Also what does the exception/error actually say?

Encode have problems too when diving the number by the character set length..
I think we need a way to provide optional support for big numbers libraries.
I say optional because on my machine I get<PHONE_NUMBER>854775232 as max number and I think it would be big enough in most cases.

It's a TypeError, decode() returns a float instead of an integer. Something like 1.4411518807586E+17

You could probably get away with casting the decode value to an int. Yeah maybe we could use a strategy pattern to use bcmath or something. I think for v1 though<PHONE_NUMBER>854775232 is plenty


Added description and example of each component and enhancer

:memo: Documentation
I separated the documentation with a section for every enhancer and component.
Enhancers' description have been improved by describing which props will the component get.
Added description and usage example of ParentSize previously missing.

LGTM thanks!


live demo does not work

Forwards to an old url

and the page shows this message

Thanks for your bug report; this is a known issue you can track here:
https://github.com/graphql/swapi-graphql/issues/143
In the mean time, try:
https://graphql.github.io/swapi-graphql/

Closing as duplicate graphql/swapi-graphql#143


Potential regression between 1.80 and 1.81 when adding tests with TestItemCollection.add

Setup

Does this issue occur when all extensions are disabled?: N/A -- this is an "extension API" bug
VS Code Version: (works) 1.80; (doesn't work) 1.81
OS Version: Windows and Linux

Steps to Reproduce (short version)

Add a test to a test hierarchy using TestItemCollection.add
Observe that the item does not appear if the item does not previously have siblings

Steps to Reproduce (long version)
We are using the following API in our extension:

https://github.com/microsoft/vscode/blob/f19d12309511a2053a0a928dbbd0e05365b52b6a/src/vscode-dts/vscode.d.ts#L16518-L16519

which states:

Adds the test item to the children. If an item with the same ID already exists, it'll be replaced.

However, it seems that between 1.80 and 1.81, this API got broken, and we have to delete the nodes to get them to appear -- that is, the replacement does not work.
Here's a complete reproducer:
import * as vscode from "vscode";

let globalController: vscode.TestController;
let flag = false;

/* hard-coded to false to demonstrate the bug */
const alwaysChildren = false;

function loadAllTests(controller: vscode.TestController) {
  const envNode = controller.createTestItem("ENV", "ENV");

  const unitNode = controller.createTestItem("UNIT", "UNIT");

  const subNode = controller.createTestItem("SUB", "SUB");

  /*
   * If you have this, things works.
   * That is: if we have one sibling, the update works.
   * If you have no siblings, the update **does not** work
   */
  if (alwaysChildren) {
    const testNode0 = controller.createTestItem("TEST0", "TEST0");
    subNode.children.add(testNode0);
  }

  /* if flag is true, add tests */
  if (flag) {
    const testNode1 = controller.createTestItem("TEST1", "TEST1");
    subNode.children.add(testNode1);
    console.log("Added one test");
  } else {
    console.log("Added no tests");
  }
  unitNode.children.add(subNode);
  envNode.children.add(unitNode);
  controller.items.add(envNode);

  /* flip flag */
  flag = flag ? false : true;
}

export function activate(context: vscode.ExtensionContext) {
  const disposable = vscode.commands.registerCommand(
    "extension.addTests",
    () => {
      loadAllTests(globalController);
    },
  );
  context.subscriptions.push(disposable);

  globalController = vscode.tests.createTestController(
    "test-controller",
    "Tests",
  );
  context.subscriptions.push(globalController);
}

What this code does is:

The first time you run extension.addTests, it creates test nodes of ENV, UNIT and SUB (where each is a child of the previous)
The second time you run extension.addTests, it creates test nodes of ENV, UNIT and SUB (where each is a child of the previous), and creates TEST1 under SUB
Each subsequent time you run extension.addTests, it tries to add or remove TEST1 (depending on the value of flag) -- there are logging prints so you can see what the intended behaviour is.

However, in 1.81, the second time the code runs, the TEST1 node does not appear. If you run this code under 1.80, the node TEST1 does appear.
Interestingly, if you create a TEST0 node the first time you create ENV, UNIT and SUB (see the conditional code under alwaysChildren), then when you come to add/remove TEST1 later on, this code works.
You can only trigger the bug (it seems) if you previously did not have a sibling, and now you are trying to add a new child to a node that previously did not have any children.
I've created a full end-to-end demo extension of this here: https://github.com/aytey/vscode_bug

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.81.1. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

It looks like you may be using an old version of VS Code, the latest stable release is 1.81.1

Yes, confirmed this bug exists in 1.81.1:
Version: 1.81.1
Commit: 6c3e3dba23e8fadc360aed75ce363ba185c49794
Date: 2023-08-09T22:18:39.991Z
Electron: 22.3.18
ElectronBuildId: 22689846
Chromium: 108.0.5359.215
Node.js: 16.17.1
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 6.4.6-1-default

Does not appear in:
Version: 1.80.2
Commit: 2ccd690cbff1569e4a83d7c43d45101f817401dc
Date: 2023-07-27T20:41:52.615Z
Electron: 22.3.14
ElectronBuildId: 22695494
Chromium: 108.0.5359.215
Node.js: 16.17.1
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 6.4.6-1-default


Add setter equivalent to Color constants:

Color.ORANGE is to mycolor.setToOrange()
Add setter equivalents to using the mutable static fields. It's one of many attempt to workaround the mutable static final fields Of the Color class.
// Example Simple Scenario

// a/b/c/MyInit.java
static Color c1 = Color.GREEN;
// Some code somewhere assigned a field with the
// value reference to Color.GREEN instead of a copy

// z/y/x/SomeHelper.java
Color c2 = new Color().set(Color.GREEN);
assert c1 != c2;
assert c1.equals(c2);

Color c3 = new Color().setToGreen();
assert c1 != c3 && c2 != c3;
// May fail
assert c1.equals(c3) && c2.equals(c3);

Other Approaches
Immutability

Add a static instance to Vector2 (libgdx - issue - #194)
#3424

TODO
Naming
The naming may be adjusted now, it's the only good initial format I thought it could have: setTo<ColorInCamelCase>.
Possible Issues
Adding, Updating, or Removing Color Definitions: missing the other equivalent
This could be mostly addressed by having constant definitions depend on their method counterparts: Color ORANGE = new Color(...) turns into Color ORANGE = new Color().setToOrange(). Though may not happen often, it may be more likely to miss a constant than method with this solution.
Formatting

Initially, the method and its body are formatted on one line

I'm not clear on the advantage of adding all these new methods over using myColor.set(Color.ORANGE).

Oh, I forgot to mention the why I made this!
I'll add it later. So, the methods should be

All these assertions pass for me (with assertions enabled - verified using assert false).

(moved from last comment edit)
Okay. setToOrange prevents the possibility of Color.ORANGE being mutated somewhere else.
(addition)
Yeah, describing the scenario instead of adding it to my example scenario wasn't helpful at all I notced. Woops! I'll go add c1.set(1.0, 2.0, 3.0, 4.0).

I'd love to see a proper encapsulation PR. Are method calls even expensive still?

I am not sure if method calls are not as expensive on the systems where they are. Though I am unsure if these kind of calls (set(Color.RED) and setToRed()) are always done each frame (probably not?).

Sorry for not being clear, I meant encapsulation method calls. For example making r private and require using getR() and setR() instead of directly changing the Color.
Doing so would allow us to create immutable colors.

I believe getters and setters will get inlined? During runtime, or possibly even earlier if ProGuard optimisations are enabled. But I can't speak with certainty, especially when it comes to iOS.

It's up to you, but //@off and //@on can be used to disable Spotless checks for a section of code (i.e. these methods that used to take a quarter of the lines that they do now) then re-enable them after.

All existing code that uses the r, g, b, and a properties would break if we forced those to change to getters. So much would break, inside libGDX and outside it, including code that wasn't mutating any Color instances, and so wasn't wrong. There's no automated way to fix it when people update. We would absolutely need to deprecate for at least a full minor version to give people a chance to prepare.
What I think is the better way, if you do decide to break everything, would be to use int or float primitives to represent RGBA colors, since those can be made final and thus fully immutable. You would use static methods on a class like Color to access the red, green, blue, or alpha segment of an int or float primitive representing an RGBA color. This is what I have been doing in colorful-gdx and SquidSquad for years now.

Maybe there is another solution that minimize breaking. Something like that :
public class Color {
    public static class ConstantColor {
        public final float r,g,b,a;
        public ConstantColor(float r, float g, float b, float a){ ... }
    }
    public static final ConstantColor RED = new ConstantColor(1,0,0,1);
    ...
    public Color(ConstantColor color){ /* simply copy rgba fields */ }
    public void set(ConstantColor color){ /* simply copy rgba fields */ }
}

With this solution, a common code adaptation required would be things like : Color color = Color.RED which is already prone of issues. And the fix is easy : Color color = new Color(Color.RED).
We could duplicate other Color methods like add, sub, lerp, equals, mul but i'm not sure it worth it.
For calls like : spriteBatch.setColor, ScreenUtils.clear, shapeRenderer.setColor, and maybe few others we could add methods supporting ContantColor. I don't think there are a lot (i didn't deeply checked).
Note that i found this issue in libgdx code recently : https://github.com/libgdx/libgdx/blob/master/extensions/gdx-freetype/src/com/badlogic/gdx/graphics/g2d/freetype/FreeTypeFontGenerator.java#L759
and this one : https://github.com/libgdx/libgdx/blob/054b25b8ced6fa54ba0c8d959b6a08a8d54de9d9/gdx/src/com/badlogic/gdx/graphics/g2d/RepeatablePolygonSprite.java#L45
I'm not sure @PokeMMO solution (setters) is good : my concern is what setters do for these immutable colors ?

Do nothing ? that would be hard to spot why a color doesn't change when you change it.
Throw exception ? You'd need good test coverage to spot them before production and spend time to find the source of the issue.

What do you think guys?

Subclass seems good. I'm not too concerned with your examples from the libGDX codebase since changing FreeTypeFontParameter's color is a rare thing to do (tinting a font is effectively free; generating a new one is slow on mobile especially) and in RepeatablePolygonSprite it is private with a setter but no getter, though it wouldn't kill us to err on the side of caution and new Color() the FontParam.
Setters could log a non-fatal error with stacktrace if immutable. But to be honest I'm not that keen, the more I think about it. Oversimplistic example without delta, clamping, or Interpolation class: col.a += .1f would become col.setA(col.getA() + .1f)?

The more "correct" way to do immutability is to make all Colors immutable and return a copy for the set methods.
The middle ground fix, retaining some performance, but requiring an api change is setX() which modifies and returns this for mutable, and a copy for immutable.
Both of those are very breaking, and would require everyone to change the way they use Colors.
The yolo performance focused way, is to throw an exception/log a warning when trying to modify an immutable color, as @Frosty-J mentioned. This is what I would suggest, as if you do this, you're already shooting yourself in the foot. I'm sure a lot of us have accidentally modified a static Color and then had to figure out why something was inexplicably broken.

@PokeMMO don't take it badly but i'm not convinced by the encapsulation solution. Any logging/throw exception/do nothing/copy behaviors are not good imho.
To clarify my concerns above : Depending on game code complexity of branching and possible states you could have some rare cases where a Color object arrive into some mutating code. Could happen in dev (best case scenario), could happen in production (worse case scenario). Making game crashing for that reason would be worse than current state. (i guess some lucky games can live with the mistakes in their code).
The auto copy approach is maybe the better in the 4 possible behavior with still possible mistakes of generating garbage : eg.
Color myColor = Color.WHITE; // at initialisation time
myColor.setR(time % 1f) // at every frames.

I prefer a solution that prevent user to make the mistake at compilation time like the one i suggested.
Also, in term of code readability, public fields are way better. As you noticed we'd need to do that for Vector2 and Vector3 as well.
(TBH if we make these fields private someday, i'll surely monkey patch these classes to keep the readability and avoid huge code migration in my stuff).
I'll stop here and read other team members thoughts on all that.


Window background is broken after resizing

Environment
Windows build number: Microsoft Windows [Version 10.0.19041.388]
Windows Terminal version (if applicable): 1.1.2021.0

Steps to reproduce

Open Windows Terminal.
Open About window.
Enlarge Windows Terminal window.

Expected behavior
Properly resized background.
Actual behavior
Broken background.

hmm even i have the same issue

Thanks for the report! This is a /duplicate of #2380 and it has its roots in the UI toolkit we are using ☹


Tmuxinator doesn't run commands in tmux windows

I'm running tmux 2.2. I know this isn't the recommended version with this package but I'd love it if I could use this to script my tmux session. I'm looking to open a few windows and panes and have them run some commands. However, no matter what I change, the commands aren't executed and I just end up getting the different windows and panes.
Here's my yml file:
# ~/.tmuxinator/docker.yml
name: docker
root: ~/docker
windows:
  - dev:
      layout: main-vertical
      panes:
        - vim
        - docker:
          - cd /work/dev/
          - git status
  - logs: tail -f /logs/dev.log

The commands just seem to get printed on the window screen and not executed.

Hi, @yatishb.
In order to help you debug this, it'd be helpful to know a few things:

Could you share the output of tmuxinator debug docker?
Which version of Tmuxinator are you using?
Which OS/version are you using?

It's possible that you're seeing this issue, which has been fixed but has not yet made it into a Tmuxinator release.

@yatishb A new version of the gem has been released. You should try upgrading to see if the fix for #414 solves your problem.

:+1: I'm getting the same:

[anthony ~/$/dia/diaco]$ tmuxinator debug dia
#!/bin/bash

# Clear rbenv variables before starting tmux
unset RBENV_VERSION
unset RBENV_DIR

tmux start-server;

  cd /Users/anthony/\$/dia/diaco

  # Run pre command.

  # Create the session and the first window. Manually switch to root
  # directory if required to support tmux < 1.9
  TMUX= tmux new-session -d -s dia-test -n dev

  # Create other windows.
  tmux new-window  -t dia-test:1 -n console
  tmux new-window  -t dia-test:2 -n server

  # Window "dev"
  tmux send-keys -t dia-test:0 ls C-m

  # Window "console"
  tmux send-keys -t dia-test:1 bundle\ exec\ rails\ console C-m

  # Window "server"
  tmux send-keys -t dia-test:2 bundle\ exec\ rails\ server C-m

  tmux select-window -t 0
  tmux select-pane -t 0

  if [ -z "$TMUX" ]; then
    tmux -u attach-session -t dia-test
  else
    tmux -u switch-client -t dia-test
  fi

looks to bet eh TMUX= empty assignment?

I had the same issue which was caused by C-m (or C-j) remapped to something else in my ~/.inputrc. Removing this mapping the issue solved. Hope it helps you.

@ptpt worked for me! <3

@ptpt I don't have any remap of C-m, but it's true that changing or disabling my ~/.inputrc makes it work or not... but in a random way?
To me it looks like some times the two consecutive send-keys are sent too fast and the actual app doesn't receive the keys?
Using the debug script I've been able to add a long sleep 5between the two consecutive send-keys and it worked with no more randomness... it seems.
Is there any way to add some sleep between the send-keys?
pre_window is not useful, as it actually creates an extra send-keys... and only before the first one for each pane.
Thanks!

Timing to the send-keys-values would be awesome. Some script just are not fast enough...

Hello, I'm experiencing the same issue with tmuxinator 1.1.1, tmux 2.9a and zsh. I don't have a ~/.inputrc file and it happens even doing
mux new sample
mux start sample

output of mux debug sample:
#!/usr/bin/zsh

# Clear rbenv variables before starting tmux
unset RBENV_VERSION
unset RBENV_DIR

tmux start-server;

cd /home/jhon

# Run on_project_start command.

  # Run pre command.

  # Run on_project_first_start command.

  # Create the session and the first window. Manually switch to root
  # directory if required to support tmux < 1.9
  TMUX= tmux new-session -d -s sample -n editor
  tmux send-keys -t sample:1 cd\ /home/jhon C-m

  # Create other windows.
  tmux new-window -c /home/jhon -t sample:2 -n server
  tmux new-window -c /home/jhon -t sample:3 -n logs

  # Window "editor"
  tmux send-keys -t sample:1.1 vim C-m

  tmux splitw -c /home/jhon -t sample:1
  tmux select-layout -t sample:1 tiled
  tmux send-keys -t sample:1.2 guard C-m

  tmux select-layout -t sample:1 tiled

  tmux select-layout -t sample:1 main-vertical
  tmux select-pane -t sample:1.1

  # Window "server"
  tmux send-keys -t sample:2 bundle\ exec\ rails\ s C-m

  # Window "logs"
  tmux send-keys -t sample:3 tail\ -f\ log/development.log C-m

  tmux select-window -t sample:1
  tmux select-pane -t sample:1.1

  if [ -z "$TMUX" ]; then
    tmux -u attach-session -t sample
  else
    tmux -u switch-client -t sample
  fi

# Run on_project_exit command.

Same issue here, it looks like send-keys happens before the shell is ready? When I start up tmux using the same commands that debug gives me it all works.

I have the same issue. If I put the debug output into a script and then add a sleep 1 after the last new-window command and the first send-keys command then that gives the new shells in the new windows time to start and the commands are processed properly.
I think I'm also impacted by using fish which handles "pending" shell commands differently to bash & zsh. If you run sleep 20 and then type some other command whilst sleep is running then fish won't run that other command when sleep stops. It just puts it on the next prompt ready to be run. Same for multiple commands:
$ sleep 20
echo bob
echo bill
echo amy
$ echo bobecho billecho amy

Whereas I believe zsh and bash will attempt to runs those commands. Apparently fish is the correct behaviour by some definition so they aren't looking to change. By contrast zsh & bash use slightly hacky approaches to overcome the situation. This is according to a kind person on the fish issues tracker who dug into it all a bit.

I am using Arch Linux with tmuxinator version 3.0.1
I also have same issue.

I also, have the same issue. However, like beatrichartz mentions, it seems to be a timing issue.
If I export the tmux script from tmuxinator, by piping the output of $ tmux debug my.yml > my.bash
and then edit that script, so that all of the send-keys are grouped together. And put a delay above the send-keys, then I get a successful start up. This seems to work because the send-keys commands are all targetted to specific panes.
#!/bin/bash

# Clear rbenv variables before starting tmux
unset RBENV_VERSION
unset RBENV_DIR

tmux -f ~/.tmux.conf start-server;

cd /Users/jschank

# Run on_project_start command.
# code ~/Mobius/mobius-repo

  # Run pre command.

  # Run on_project_first_start command.

  # Create the session and the first window. Manually switch to root
  # directory if required to support tmux < 1.9
  TMUX= tmux -2 -f ~/.tmux.conf new-session -d -s mobius -n mobius

  # Create other windows.

My original yaml file:
---
name: mobius
root: ~/

# Optional tmux socket
# socket_name: foo

# Note that the pre and post options have been deprecated and will be replaced by
# project hooks.

# Project hooks
# Runs on project start, always
on_project_start: code ~/Mobius/mobius-repo
# Run on project start, the first time
# on_project_first_start: command
# Run on project start, after the first time
# on_project_restart: command
# Run on project exit ( detaching from tmux session )
# on_project_exit: command
# Run on project stop
# on_project_stop: command

# Runs in each window and pane before window/pane specific commands. Useful for setting up interpreter versions.
# pre_window: rbenv shell 2.0.0-p247

# Pass command line options to tmux. Useful for specifying a different tmux.conf.
tmux_options: -f ~/.tmux.conf

# Change the command to call tmux.  This can be used by derivatives/wrappers like byobu.
# tmux_command: byobu

# Specifies (by name or index) which window will be selected on project startup. If not set, the first window is used.
# startup_window: editor

# Specifies (by index) which pane of the specified window will be selected on project startup. If not set, the first pane is used.
# startup_pane: 2

# Controls whether the tmux session should be attached to automatically. Defaults to true.
# attach: false

windows:
  - mobius:
      root: /Users/jschank/Mobius/mobius-repo/
      layout: tiled
      panes:
        - nest-app:
          - cd workspace
          - npm run nest
        - mobius-app:
          - cd workspace
          - npm run mobius
        - elastic:
          - docker start elasticsearch
          - docker attach elasticsearch
        - flask:
          - cd ../dcat-tool
          - git pull
          - cd src
          - make -f ./Makefile clean
          - make -f ./Makefile template
          - make -f ./Makefile run
        - daemon:
          - cd ../bare-repos
          - ./serve

My tmux.config...
set -g prefix C-a
unbind C-b

set -s escape-time 1
set -g base-index 1
setw -g pane-base-index 1

bind r source-file ~/.tmux.conf \; display "Reloaded!"

bind C-a send-prefix

bind | split-window -h
bind - split-window -v

bind h select-pane -L
bind j select-pane -D
bind k select-pane -U
bind l select-pane -R

bind -r C-h select-window -t :-
bind -r C-l select-window -t :+

bind -r H resize-pane -L 5
bind -r J resize-pane -D 5
bind -r K resize-pane -U 5
bind -r L resize-pane -R 5

set -g mouse on

# set -g default-terminal "screen-256color"
set -g default-terminal 'xterm-256color'
set -ga terminal-overrides ',xterm-256color:Tc'
# set -g status-left-length 40
# set -g status-left "#[fg=green]Session:#S #[fg=yellow]#I #[fg=cyan]#P"
# set -g status-right "#[fg=cyan]%d %b %R"

set -g status-justify centre
setw -g monitor-activity on
set -g visual-activity on

# make scrolling with wheels work
bind -n WheelUpPane if-shell -F -t = "#{mouse_any_flag}" "send-keys -M" "if -Ft= '#{pane_in_mode}' 'send-keys -M' 'select-pane -t=; copy-mode -e; send-keys -M'"
bind -n WheelDownPane select-pane -t= \; send-keys -M

set-option -g history-limit 50000
set-option -g default-shell /usr/local/bin/fish

I found this issue happens to me if I have a customized shell environment, for example, installing .oh-my-zsh or .oh-my-bash


fix linting in query and variable editor

Linting in the query and variable editor broke with the latest upgrade of the codemirror version: https://github.com/graphql/graphiql/commit/eab257ba3e65e47625216fc3545b56426e0c4e61
The reason are internal changes to the lint addon implementation, in particular the options object is not split up between options and linterOptions where our custom properties (e.g. schema) are added to the latter: https://github.com/codemirror/CodeMirror/commit/70c615c5ff7d25e91dd50190945ef295b9ce7f09
To fix this we also modify the linterOptions object in the editor state when we want to update one of our custom options.
Props @benjie who noticed the broken linting 👏

Good catch! This warrants an end to end test perhaps?

Good call @acao! Added a test suite that asserts that the linter is working for query and variables 👍

innnnnteresting...

@thomasheyenbrock perhaps it's the specificity and a change in error formatting between versions, if i recall. perhaps try just two words that are consistent like Unexpected character?


!important suffix for styles

from @monolithed (#26)

!important
When an important rule is used on a style declaration, this declaration overrides any other declarations.

I added some missing tests to avoid regressions in the future.

@zinserjan, thanks!


MML and GTK use cookies in their logo image. Can we host them to avoid this?

See also issue #90

https://www.gtk.fi/wp-content/uploads/2020/01/GTK-logo.pdf (needs conversion)
https://www.maanmittauslaitos.fi/themes/custom/mml/images/suomi_logo_rgb.svg
We already host some balloons. Perhaps an /icon folder would be in order?


Add filters to "Watched Projects" feed

Imported from CenterforOpenScience/osf#85
Additional comments are present but were not imported.

By @sloria:

Currently, every logged event for every watched project is shown in the feed. This leads to active projects dominating the feed.

This raises questions:

Should we set up a filtering system for the feed?

If so, which event types should be shown?
Should the filtering be configurable? If so, how will the user configure it?
How granular should configuration be? (e.g. "I want to hide all wiki changes from the feed" vs. "I want to hide all wiki changes for project Foo but still want to see wiki changes for project Bar")

Addressed in dashboard redesign.


♻ consolidate zpmod

Type of changes

[ ] CI
[ ] Bugfix
[ ] Feature
[ ] Generic maintenance tasks
[ ] Documentation content changes
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no URL changes)
[x] Improving the performance of the project (not introducing new features)
[ ] Other (please describe):

Issue Number: N/A
What is the new behavior?

Merged and improved function, and updated subcommand options.
Check the module's compile timestamps and recompile if needed, if no action is required, no message will be printed.

Does this introduce a breaking change?

[ ] Yes
[x] No

Additional information

Wiki update not required: wiki/zpmod
Repository update not required: z-shell/zpmod
The install page can be updated with new options, however current not deprecated: install/zpmod

https://github.com/z-shell/wiki/pull/350

Closed in favor of https://github.com/z-shell/zi/pull/267


Dropdown mutiple selection with API not working well

I have a <select> dropdown list filled with an API, this works fine showing results and selecting one result, but when I try to add another option to it, is not adding the option into the dropdown, but the option disappears from the result options. This is my code:
<select class="ui multiple search selection dropdown products" multiple ng-model="ctrl.brand.products" ng-optionss="product.id as product.name for product in ctrl.products">
    <option value="">Search for options</option>
</select>

$('.ui.dropdown.products').dropdown({
  saveRemoteData: false,
  apiSettings: {
    url: 'http://localhost:5000/products?name={query}',
    onResponse: function( data ) {
      var response = {};
      response.success = true;
      for (var i = 0; i < data.length; i++) {
        data[i].value = data[i].id;
      }
      response.results = data;
      return response;
    }
  }
});

I found what the problem is.
My API is returning an Array of results, but the 'onResponse' function is giving me back an object with the index as key instead the array of data.
My API result
[
  { id: 1, name: "some name" },
  { id: 2, name: "another name" }
]
```

Te response function data
```javascript
{
  "0": { id: 1, name: "some name" },
  "1": { id: 1, name: "another name" }
}
```

Why is this happening? It suppose to have to work like that?

Hi @nestorvanz,
That’s most certainly a back-end issue. What technology stack do you use there, and do you send your API response with a Content-Type: application/json header? I’ll close the issue as a usage question, but feel free to keep this thread going; you might also want to copy the issue to our Gitter chatroom in case you’re using a back-end technology I’m not familiar with. The following is a standard reply.
Thank you for posting, but although it’s a valid usage question, we’ve limited GitHub Issues to bug reports and feature requests, keeping the board more manageable for maintainers; see the contributing guidelines for more information on what kind of posts should find themselves into the GitHub Issues board.
To get answers or feedback that might allow you to repost this issue, please use one of our other support resources:

Gitter chatroom
Official forums
StackOverflow


Procedural Background Texture

Howdy! I found your template on a Command & Conquer modding forums and its been a blast to use! There's something missing or at least I missed in the v1.00-alpha1 version of the template mainly "preview 2" script you mentioned here  that allows for procedural background texture.

How do I do this in the most recent build of the template?

Hi @vibrantrida thanks for the feedback! I'll be including that preview bg texture in a near future


find() sporadically raises KeyError from cache

What happened:
The code which failed in this case has worked thousands of times before, but just randomly failed once for us recently. We are just listing all files (with no max-depth) beneath a certain path with fs.find(prefix), and it raises a KeyError on some path beneath that prefix:
[INFO]	2021-04-29T15:55:02.30Z 	Listing versions beneath 'BUCKET_NAME/prediction/refs/tags'
'BUCKET_NAME/prediction/refs/tags/v6.0.0.dev23': KeyError
Traceback (most recent call last):
  File "/var/task/lambda_handler.py", line 37, in main
    payload = handler(event)
  File "/opt/python/api/handlers.py", line 124, in post_model
    config, user, tags, description, agent_run=runid.agentrun
  File "/opt/python/api/components/abc/abc.py", line 55, in submit_job
    version = cls._validate_version(config)
  File "/opt/python/api/components/abc/abc.py", line 265, in _validate_version
    include_prereleases=True
  File "/opt/python/api/components/abc/abc.py", line 225, in list_releases
    versions = cls.list_versions(tags_only=True)
  File "/opt/python/api/components/abc/abc.py", line 196, in list_versions
    files = fs.find(str(prefix))
  File "/opt/python/fsspec/spec.py", line 434, in find
    for _, dirs, files in self.walk(path, maxdepth, detail=True, **kwargs):
  File "/opt/python/fsspec/spec.py", line 413, in walk
    yield from self.walk(d, maxdepth=maxdepth, detail=detail, **kwargs)
  File "/opt/python/fsspec/spec.py", line 383, in walk
    listing = self.ls(path, detail=True, **kwargs)
  File "/opt/python/s3fs/core.py", line 619, in ls
    files = self._ls(path, refresh=refresh)
  File "/opt/python/s3fs/core.py", line 487, in _ls
    return self._lsdir(path, refresh)
  File "/opt/python/s3fs/core.py", line 413, in _lsdir
    return self.dircache[path]
  File "/opt/python/fsspec/dircache.py", line 62, in __getitem__
    return self._cache[item]  # maybe raises KeyError
KeyError: 'BUCKET_NAME/prediction/refs/tags/v6.0.0.dev23'

What you expected to happen:
I don't expect find() (or ls(), glob(), etc) to ever raise KeyError.
Minimal Complete Verifiable Example:
I can't say that this is verifiable, since this happened once amongst a very large number of successful runs. I've verified that nothing was occurring on the target S3 bucket around the time of the failure. But this is the best I can provide in terms of what was being executed
import s3fs

BUCKET_NAME = "mybucket"  # Replace with an actual bucket name

botoconfig_kwargs = dict(
    region_name="ap-southeast-2",
    retries=dict(mode="standard", max_attempts=20),
)
fs = s3fs.S3FileSystem(
    listings_expiry_time=10,  # seconds
    default_fill_cache=False,
    default_cache_type="none",
    config_kwargs=botoconfig_kwargs,
)

fs.find(f"{BUCKET_NAME}/subfolder")

Anything else we need to know?:
In this particular case, BUCKET_NAME/prediction/refs/tags/v6.0.0.dev23 (the path which the KeyError was raised from) is an S3 prefix which has had files living beneath it for months.
I would love to upgrade to the new version of s3fs, but since it's moved to using asyncio in the background, it is rather unstable on Python 3.6. However since this issue seems to come from the cache, I don't know if this is something which has actually changed since s3fs version 0.4.2 / fsspec 0.8.7?
Any help would be greatly appreciated, thanks.
Environment:

Dask version: Not using Dask
s3fs version: 0.4.2
fsspec version: 0.8.7
Python version: 3.6 (from AWS Lambda python3.6 runtime, unsure of specific version)
Operating System: Linux (on AWS Lambda, unsure of specifics)
Install method (conda, pip, source): pip installed into local directory, then zipped up for deployment to AWS Lambda.

This was probably fixed by https://github.com/dask/s3fs/pull/471

I think you're right, I haven't seen this since upgrading. Cheers!


:sponge:  Open JDK is an implementation (not a language) :coffee:

:grey_question: About
Actually, OpenJDK is mentionned as a language, while it's not : only java is a langugae... while OpenJDK is "just" an implementation :

:bookmark_tabs: Related contents

https://github.com/endoflife-date/endoflife.date/issues/2350
https://github.com/endoflife-date/endoflife.date/issues/2349
https://github.com/endoflife-date/endoflife.date/issues/2348
https://github.com/endoflife-date/endoflife.date/issues/2347
https://github.com/endoflife-date/endoflife.date/issues/2346
https://github.com/endoflife-date/endoflife.date/issues/2345
https://github.com/endoflife-date/endoflife.date/issues/2344
https://github.com/endoflife-date/endoflife.date/issues/2343
https://github.com/endoflife-date/endoflife.date/issues/2342
https://github.com/endoflife-date/endoflife.date/issues/2341
https://github.com/endoflife-date/endoflife.date/issues/2334
https://github.com/endoflife-date/endoflife.date/pull/2311
https://github.com/endoflife-date/endoflife.date/pull/2259
https://github.com/endoflife-date/endoflife.date/pull/2259
https://github.com/endoflife-date/endoflife.date/pull/2259
https://github.com/endoflife-date/endoflife.date/pull/2259
https://github.com/endoflife-date/endoflife.date/issues/2249
https://github.com/endoflife-date/endoflife.date/pull/2231
https://github.com/endoflife-date/endoflife.date/issues/2225
https://github.com/endoflife-date/endoflife.date/pull/1655
https://github.com/endoflife-date/endoflife.date/pull/1631
https://github.com/endoflife-date/endoflife.date/pull/1394
https://github.com/endoflife-date/endoflife.date/pull/1370
https://github.com/endoflife-date/endoflife.date/pull/1337

We use lang for other similar runtimes as well. For eg, the Ruby page tracks the Ruby MRI implementation, and the Python page tracks the CPython runtime.
We can solve this better with tags runtime:openjdk or jdk-implementation for eg.

We use lang for other similar runtimes as well. For eg, the Ruby page tracks the Ruby MRI implementation, and the Python page tracks the CPython runtime.

:ok:

We can solve this better with tags runtime:openjdk or jdk-implementation for eg.

Looks like an interesting option. What do you think about that @marcwrobel :grey_question:

:zap:

I agree with the statement, but that's just a name issue IMHO (this page is in fact tracking Oracle JDK). And this page will be updated with #2350 anyway.
And do not forget Java / OpenJDK are often used interchangeably, even if it is not correct per se.
Tags would be useful too.


Column Rows do not show up, until select all

System information:

MacOS Big Sur 11.0 Beta
DBeaver: Version <IP_ADDRESS>010191702

Connection specification:

MySQL database

Describe the problem you're observing:
The first time I open a table, the Columns text do not show up. Then, select all Column rows, all the rows will stay.

Could you please send log files?

Could you please send log files?

Where is the log file? There is no error, it's just the rows are not showing up until I hit "command + A" or use the mouse to select them all.

Maybe we manage to find something usefull here. Please see where logs are: https://github.com/dbeaver/dbeaver/wiki/Log-files

Maybe we manage to find something usefull here. Please see where logs are: https://github.com/dbeaver/dbeaver/wiki/Log-files

ok. I will take a look later today. and give you a screenshot.

Thank you! You can send files here or<EMAIL_ADDRESS>

Thank you! You can send files here or<EMAIL_ADDRESS>
I opened the error log, however, I don't see anything related to this. here is the screenshot.

Then I select all, all the rows under "Columns" will stay there.

When I try to create a new column, on the left panel, it contains the new column, however, I cannot see it in the properties.
when I click on save, the sql does not show up properly as well: maybe needs to padding-top? because I only see the third new column (see screenshot 2). when I clicked on open editor on the second screenshot: it contains the sql data (see screenshot 3)

I did a test on another mac machine, not with the Big Sur system. I think maybe the Big Sur is the reason why it got displayed improperly.

Thank you for the additional details. Could you please also send log files https://github.com/dbeaver/dbeaver/wiki/Log-files?

Thank you for the additional details. Could you please also send log files https://github.com/dbeaver/dbeaver/wiki/Log-files?

I have downgraded my system from Big Sur to Catalina. I don't have the issue in the Catalina system anymore.

Here is the log file. But nothing is related to view properties error
dbeaver-debug.log

https://github.com/forcedotcom/dataloader/issues/217
This is an SWT related issue, that has become "incompatible" with macOS Big Sur. For the sake of all eclipse developers, i hope Apple fixes this, because there are THOUSANDS of old eclipse apps that cannot be upgraded to the more recent SWT VERSIONS (when eclipse fix them, anyway...)
If you guys find a workaround for this that does not involve updating Eclipse SWT plugins, please post it!

ok. I see. thanks for letting me know.

I have the same question in macOS big sur, even if this OS version has been already released.
My table rows does not displayed until mouse moveover and click one row or select all.

I have the same question in macOS big sur, even if this OS version has been already released.
My table rows does not displayed until mouse moveover and click one row or select all.

yeah. wait for the fix now.

Possibly it will be fixed when updating Eclipse (DBeaver release 7.3.1).

Fixed the issue with invisible rows. All other UI bugs related to SWT will be fixed in one of our future releases when we upgrade our Eclipse version.

Hi @marcolopes. At least in this situation, the workaround turns out to be quite simple.

thanks.

Could you please test the fix on EA version https://dbeaver.io/files/ea/?

Hi @marcolopes. At least in this situation, the workaround turns out to be quite simple.

Thanks! I got to that conclusion also... i had problems with the SWT TABLES, and a forced REDRAW resolves the issue!
Algo had other problems, and i had to "patch" the SWT LIB (Display class) to avoid a core runtime error (this was fixed on more recent versions of SWT that i can't use right now)

Could you please test the fix on EA version https://dbeaver.io/files/ea/?

I've downgraded my system to catalina. maybe @lechie can help. thanks

The fix was made and will be included to the next releaseю Please try early access https://dbeaver.io/files/ea/

Fixed in 7.3.0. Could you please try EA https://dbeaver.io/files/ea/?

Hello @kseniiaguzeeva I'm using 7.3.0 but have same problem in "Execution Plan" Result table.
Only when I clicked it renders result.
Editor before 7.3.0 also have a lot of rendering issues, but 7.3.0 solves.

Please tell your OS and send the screenshot

Sorry @kseniiaguzeeva, it is the same: macOS Big Sur 11.0.1 (20B29).
Before click:

After click:

Thank you for the report. It is fixed in 7.3.1 EA version. You can try it https://dbeaver.io/files/ea/

Thank you for the report. It is fixed in 7.3.1 EA version. You can try it https://dbeaver.io/files/ea/

thanks.


Add FFmpegSlice postprocessor

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[X] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[X] Searched the bugtracker for similar pull requests

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[X] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Bug fix
[ ] Improvement
[ ] New extractor
[X] New feature

Description of your pull request and other information
Explanation of your pull request in arbitrary form goes here. Please make sure the description explains the purpose and effect of your pull request and is worded well enough to be understood. Provide as much context and examples as possible.
Gives the possibility of slicing downloaded media. The command below will download video and cut a slice from second 3 to second 8. When finished processng It will generate a 5 seconds long clip as a result.
youtube-dl https://www.youtube.com/watch?v=gwWzuHPoq7E --slice-start 3 --slice-end 8

Hello @dstftw It seems that the build is breaking due to extractors, couldn't see any failing tests related to my code in Travis. How should I proceed here?

Any news on this pull request?

isn't it better add support for this feature at the downloader level instead of a post prossesor.
for the other protocols not supported by ffmpeg(dash, f4m), it won't be too difficult to add support for this options.
using the method in this PR, i see that it's a waste to download the full video while you're interested in a small part of the video.

It's been a year since the last update. What should we do with this PR?


Negative indentshift values aren't handled properly

It looks like negative indentshift is supposed to indent from the opposite side?  See this example from the MathML3 test suite.

==> In Testsuite
MathMLToDisplay/Topics/LineBreak/issue769-1a.html
MathMLToDisplay/Topics/LineBreak/issue769-1b.html
MathMLToDisplay/Topics/LineBreak/issue769-2a.html
MathMLToDisplay/Topics/LineBreak/issue769-2b.html
MathMLToDisplay/Topics/LineBreak/issue769-3a.html
MathMLToDisplay/Topics/LineBreak/issue769-3b.html
MathMLToDisplay/Topics/LineBreak/issue769-4a.html
MathMLToDisplay/Topics/LineBreak/issue769-4b.html
MathMLToDisplay/Topics/LineBreak/issue769-5a.html
MathMLToDisplay/Topics/LineBreak/issue769-5b.html
MathMLToDisplay/Topics/LineBreak/issue769-6a.html
MathMLToDisplay/Topics/LineBreak/issue769-6b.html


Update subscriptions.md - Add link to CD Services HDH Page

Add link to CD Services HDH Page
Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the CODEOWNERS.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screenshot.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65f2357179bf5d45a04777e9--harness-developer.netlify.app


Feature/better docs

Coverage decreased (-1.1%) to 98.863% when pulling 6b3e9d5a6751026c721bcebeb397b52c6d7a74fc on feature/better_docs into 8e3d9c22ef5a0969256e48bd9e550e3b571f106a on develop.

Coverage remained the same at 100.0% when pulling c4e519f9ff7509c01b4041e03b5c6215baed6e55 on feature/better_docs into 8e3d9c22ef5a0969256e48bd9e550e3b571f106a on develop.

Coverage remained the same at 100.0% when pulling 56ba508efbeda84152aebfdf7652579c6cd0148c on feature/better_docs into 8e3d9c22ef5a0969256e48bd9e550e3b571f106a on develop.

Coverage remained the same at 100.0% when pulling 39db5c161e15a5a12d564ec0123075d810861779 on feature/better_docs into 8e3d9c22ef5a0969256e48bd9e550e3b571f106a on develop.

Coverage remained the same at 100.0% when pulling f90730d921f8a687e94601a579837c8a360435d9 on feature/better_docs into 8e3d9c22ef5a0969256e48bd9e550e3b571f106a on develop.

Coverage decreased (-0.05%) to 99.951% when pulling 3ff61efb22749b85306f2e3bc1abb942308ae281 on feature/better_docs into 8e3d9c22ef5a0969256e48bd9e550e3b571f106a on develop.

Coverage decreased (-0.1%) to 99.852% when pulling f744906a442b76d459159e883b6db7d07e1a8ad0 on feature/better_docs into 8e3d9c22ef5a0969256e48bd9e550e3b571f106a on develop.

Coverage decreased (-0.1%) to 99.852% when pulling f744906a442b76d459159e883b6db7d07e1a8ad0 on feature/better_docs into 8e3d9c22ef5a0969256e48bd9e550e3b571f106a on develop.

Coverage decreased (-1.4%) to 98.622% when pulling a481598c3f47452c1169c0aeeff5c12ee4c0f3b6 on feature/better_docs into 8e3d9c22ef5a0969256e48bd9e550e3b571f106a on develop.

Coverage decreased (-1.4%) to 98.622% when pulling a481598c3f47452c1169c0aeeff5c12ee4c0f3b6 on feature/better_docs into 8e3d9c22ef5a0969256e48bd9e550e3b571f106a on develop.

Coverage decreased (-0.3%) to 99.661% when pulling d2538856e1347e505eee9c37af8b49429073b13e on feature/better_docs into 8e3d9c22ef5a0969256e48bd9e550e3b571f106a on develop.

Coverage decreased (-0.1%) to 99.903% when pulling 29940e3ab01401184e4219349cd5e3f196c37376 on feature/better_docs into 8e3d9c22ef5a0969256e48bd9e550e3b571f106a on develop.

Coverage decreased (-0.1%) to 99.903% when pulling 29940e3ab01401184e4219349cd5e3f196c37376 on feature/better_docs into 8e3d9c22ef5a0969256e48bd9e550e3b571f106a on develop.

Coverage decreased (-0.1%) to 99.903% when pulling 29940e3ab01401184e4219349cd5e3f196c37376 on feature/better_docs into 8e3d9c22ef5a0969256e48bd9e550e3b571f106a on develop.

Coverage decreased (-0.1%) to 99.903% when pulling 29940e3ab01401184e4219349cd5e3f196c37376 on feature/better_docs into 8e3d9c22ef5a0969256e48bd9e550e3b571f106a on develop.

Coverage decreased (-0.1%) to 99.903% when pulling 29940e3ab01401184e4219349cd5e3f196c37376 on feature/better_docs into 8e3d9c22ef5a0969256e48bd9e550e3b571f106a on develop.

Coverage decreased (-0.1%) to 99.903% when pulling 29940e3ab01401184e4219349cd5e3f196c37376 on feature/better_docs into 8e3d9c22ef5a0969256e48bd9e550e3b571f106a on develop.

Coverage decreased (-0.1%) to 99.903% when pulling 29940e3ab01401184e4219349cd5e3f196c37376 on feature/better_docs into 8e3d9c22ef5a0969256e48bd9e550e3b571f106a on develop.

Coverage remained the same at 100.0% when pulling 31496bb049149a5dc351272f289c0f202722d0a4 on feature/better_docs into 8e3d9c22ef5a0969256e48bd9e550e3b571f106a on develop.

Coverage remained the same at 100.0% when pulling 31496bb049149a5dc351272f289c0f202722d0a4 on feature/better_docs into 8e3d9c22ef5a0969256e48bd9e550e3b571f106a on develop.

Coverage remained the same at 100.0% when pulling 31496bb049149a5dc351272f289c0f202722d0a4 on feature/better_docs into 8e3d9c22ef5a0969256e48bd9e550e3b571f106a on develop.


Widgets not rendering after reloading vscode

Environment data

VS Code Insiders: 1.56.0-insider<PHONE_NUMBER>07c15252e581e8abe1b820b9d311ee x64
Jupyter Extension version (available under the Extensions sidebar): 2021.6.718745073
Python Extension version (available under the Extensions sidebar): 2021.3.680753044
OS (Windows | Mac | Linux distro) and version: Windows10
Python and/or Anaconda version: 3.9.2
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): conda
Jupyter server running: Remote
IpyWidgets==7.6.3

Behavior
When I'm initializing the jupyter server, connecting the notebook for the first time and runing the following cell
from tqdm.notebook import tqdm
[k for k in tqdm(range(10))]

The ipywidget is displaying correctly. However, if I exit the notebook and comeback or if I reload the notebook, the Ipywidget does not render anymore.

When I toggle the developper tools, on reload I have the following warning

This is a side effect of us closing kernels when closing notebooks. Jupyter has the same problem but at least tells you why:

Ok thanks for the information,
But, when I rerun the cell with a new runing kernel, the widget is  still not rendering. In jupyter I think the ipywidgets is rerendering if you rerun the cell with a new kernel after shuting down the other.
In vscode if I close a notebook/reload it and then rerun the cell it doesn't work.

I'm reproing that too. Very weird. Although closing and reopening the notebook makes it work for me.
My guess is that reloading a kernel does not remove the preload scripts that the first kernel added. This interferes with ipywidgets working.

 If I do that, the IpyWidgets works again. Just closing the notebook an reopening it doesn't work for me

In general IPyWidgets will not be re-loaded whether we re-load or close/re-open a notebook, unless we have https://github.com/microsoft/vscode-jupyter/issues/4404

Validated (I think). In Insiders now if I execute a notebook with ipyvolume widgets, then reload, the widget doesn't initially show, but I can reexecute the cell and see the widget appear in the cell output.


Issue when Configuring superset (config.py) with LDAP

A clear and concise description of what the bug is.
I am trying to do authentication:LDAP in superset using the guide at:
https://flask-appbuilder.readthedocs.io/en/latest/security.html
Please note that I am attempting it on Ubuntu 20.04 VMs behind a proxy. And I am running superset on http://localhost:9000/(in dev mode)
After following all the steps successfully when I run it on 9000 port
I got this error: Invalid login. Please try again
How to reproduce the bug
1 Clone Superset repository
2 cd superset
3 docker-compose up
4 in another terminal(cd superset/superset-frontend)
5 npm run dev-server
Expected results
Login Successfully.
Actual results
Invalid login. Please try again
modify config.py
from flask_appbuilder.security.manager import AUTH_LDAP
AUTH_ROLE_ADMIN = 'Admin'
AUTH_USER_REGISTRATION = True
AUTH_USER_REGISTRATION_ROLE = "Public"
AUTH_TYPE = 2
AUTH_LDAP_SERVER = "ldap://ldapserver.new"
AUTH_LDAP_USE_TLS = False
AUTH_LDAP_SEARCH = "cn=rrscw,ou=users,dc=ldapserver,dc=new"
AUTH_LDAP_UID_FIELD = "uid"
AUTH_LDAP_BIND_USER = "uid=rrscw,ou=users,dc=ldapserver,dc=new"
AUTH_LDAP_BIND_PASSWORD = "rrscw"
Screenshots
If applicable, add screenshots to help explain your problem.
Environment
OS: Ubuntu Server 20.04
Python on VM: 3.8.10
Checklist
Make sure to follow these steps before submitting your issue - thank you!

[ ] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[ ] I have reproduced the issue with at least the latest released version of superset.
[ ] I have checked the issue tracker for the same issue and I haven't found one similar.

I am moving this to Discussions, as this isn't specifically reporting a bug, but requesting configuration help.


After using command "I signed it!" in a PR, the googlebot will respond to every (edited) comment when the CLA is not OK.

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
See PR #11096 where at this comment I wrote the command: I signed it!
googlebot responded with an issue, which is fine:

We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for the commit author(s). If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google.
In order to pass this check, please resolve this problem and have the pull request author add another comment and the bot will run again.

Here is the issue:
Instead of what it says in the last sentence, the googlebot responds to EVERY new comment, or an edit of an comment.
While it should stay silent, until I signed it! is written again.
Complete output when running fastlane, including the stack trace and command used

[INSERT OUTPUT HERE]
Environment

[INSERT OUTPUT HERE]

Although I am also irritated by its behaviour, isn't what you describe exactly what it tells you it will be doing?

... have the pull request author add another comment and the bot will run again

It runs again for each additional comment interaction on the thread of the PR author (probably can't or intentionally does not differentiate between new comment and edited comment).

You're right.
I read "the comment" (referring to I signed it!) instead of "another comment", because I assumed that someone was able to make a sensible requirement and the implementation was just erroneous.
However, reality seems worse as the requirement is the problem.
So my suggestion would be to change that to:

In order to pass this check, please resolve this problem and add a comment with "I signed it!" and the bot will run again.

Personally, I agree that @googlebot is quite chatty and annoying in this case, see your PR :/
I will close this issue, as this is only indirectly related to fastlane itself.
But @alexsinger, can you make sure this feedback gets forwarded to the correct team? There doesn't seem to be a repo with issues for @googlebot where this could get reported. Thanks.

@janpio and @alexsinger Thanks for transferring this to the correct team.


Fixing Issues with CrossPlane Integrations and Adding RDS Compositions

This PR Does the following :

Fixing Issues with CrossPlane Integrations
Adding RDS Compositions to CrossPlane Integrations

thanks @elamaran11 for this.
I agree with Manabu on the compositions. Looks like we would want to take the crossplane compositions under https://github.com/cnoe-io/backstage-crossplane-integrations and couple them with the corresponding Backstage templates, similar to what we have done for Terraform. wdyt?

@nimakaviani @nabuskey  As part of Manabu's concern i already have this PR in backstage-crossplane-integrations repo to add Backstage support for RDS. His second concern was duplicating from crossplane-on-eks either we have to duplicate it somewhere for seamsless delivery of these patterns. I have no issues in removing RDS compositions and moving it to https://github.com/cnoe-io/backstage-crossplane-integrations but as part of consistency then the S3 compositions should also be removed and the crossplane-compositons.yaml will make no sense because currently it points to composition folder in stacks (in this repo). So i recommend adding RDS compositions to this repo or  If you move compositions to the other repo, then you need another Argo Application for compositions in backstage-crossplane-integrations repo. Basically we have to work on rearchitecture to

First remove crossplane-compositions.yaml and compositions folder from this repo.
Create a new Argo Application in backstage-crossplane-integrations
Move the S3 compositions and add RDS and all new compositions to backstage-crossplane-integrations repo and maintain it there.

Overall the end result is same but we arent really achieving a lot with this approach vs doing lot of moving of stuff here and there. Please let me know WY both think, i can find time next week to work on this.

Thanks @elamaran11 ... I agree with your point about also moving the s3 compositions and the related backstage templates over to the backstage-crossplane-integrations repo. We initially didn't do it because that repo did not exist and also we dont have proper instructions on the website on how to get the stuff to work together.
Lets work on consolidating everything under backstage-crossplane-integrations

@nimakaviani Thankyou for agreeing to my approach. I will work on another PR to stacks repo and crossplane integrations repo to consolidate crossplane compositions. @nimakaviani @nabuskey Please merge this PR which has bug fix to fix a show stopper with crossplane integrations because crossplane currently fails without this fix.


reporter endpoint backward compat

This change is needed to assure backward compatibility. In previous implementation, all of segment will be reported through /skywalking.v3.TraceSegmentReportService/collect. But if we have ~8.3.0, it won't work anymore. To resolve this, use compat endpoint.

Doesn't namespace make the agent having less chance to conflict with other libs?

Doesn't namespace make the agent having less chance to conflict with other libs?

@wu-sheng Yes. But this change don't have namespace pollution because it is only to specify gRPC endpoint here. https://github.com/SkyAPM/cpp2sky/pull/70/files#diff-b7b38cc1608732e0b2c7157563c8f06e6dadd97c29717f4c6d089f1c6aa7d966R120
TraceCompat endpoint allows both "with namespace" segment and "without namespace" one simultaneously. https://github.com/Shikugawa/skywalking-data-collect-protocol/blob/master/language-agent/TracingCompat.proto#L36-L37

Got your point. Very special case.

I can see 2 issues existing in released 0.2.0, could you recheck the milestone? https://github.com/SkyAPM/cpp2sky/milestone/2

I can see 2 issues existing in released 0.2.0, could you recheck the milestone? https://github.com/SkyAPM/cpp2sky/milestone/2

These issues are not high priority. Let's delay.

@wu-sheng Could you merge this?

I can see 2 issues existing in released 0.2.0, could you recheck the milestone? https://github.com/SkyAPM/cpp2sky/milestone/2

These issues are not high priority. Let's delay.

Then we should move those into 0.3, and close the 0.2


circuit method aliases?

This was raised in the review of #4035 by @jaygambetta
We have a bunch of aliases right now:
circuit.toffoli : circuit.ccx
circuit.id - circuit.i - (wire)
circuit.cx - circuit.cnot
circuit.fredkin- circuit.cswap

There might be a danger of proliferating circuit methods (which are already a lot). Can we decide on a canonical Qiskit name for each gate and then document somewhere other aliases in the literature? Otherwise there is no clear policy and someone might feel there should be an alias like circuit.x <-> circuit.not or other things.

my vote is not to aliases and just teach circuits to be made with a set we choose. I dont see any real reason other than people like them and its very subjective.
If we are going to then we should go all the way.
circuit.p - circuit.s
circuit.not - circuit.x

hmm maybe it is only 6 that I can think off so maybe it is not as bad as I was thinking.

There can be more like uniformly-controlled (uc) vs. multiplexer..
And when we start adding square-root(X) and square-root(CX) I think we will call them sx and scx but some people call these v and cv.

Also circuit.iden is still a alias for circuit.i, and circuit.mct - circuit.mcx.
About
circuit.p - circuit.s

didn't we want to change u1 to p for "phase gate"?
Not sure if s is a good prefix for sqrt, since the s gate already exists. Couldn't we spell it out? circuit.sqrtx? That would be pretty clear and is also not too long. Also, why are the sqrt gates referred to as v sometimes?

Not sure if s is a good prefix for sqrt, since the s gate already exists. Couldn't we spell it out? circuit.sqrtx? That would be pretty clear and is also not too long.

I'm not sure, I see your point. But we now use single-letter prefixes:
r.. -> rotation
c.. -> control
So I figured we can use the s.. prefix to mean square-root.
sx is also used here: https://arxiv.org/pdf/2003.09382.pdf
But I can see confusion with S gate.

Also, why are the sqrt gates referred to as v sometimes?

Nielsen and Chuang (probably based on earlier papers) have decompositions of U in terms of its square-root, which they call V.

What are the current aliases?

Currently defined in quantumcircuit.py:

id -> i
cnot -> cx
toffoli -> ccx
fredkin -> cswap
mct -> mcx

I didn't look at extensions, or anything like that.

Update for Santa- QGoGP
✅ qauntumcircuit.py has all the aliases mentioned by santa @jakelishman above!
Extensions has the following alias as far as I checked out!
iso -> isometry
Will update comment/add more comments if cause is found/fixed!

Python version 3.9.7
qiskit-terra version: 0.19.1

I'm going to close this issue as stale now.  There's probably more user pain in us deprecating and removing aliases than there is actual cost to us to maintain the very small set we still have.
Feel free to re-open if there's more to discuss.


expose some fields on session state

Which issue does this PR close?

Closes https://github.com/apache/datafusion/issues/11715
Rationale for this change

As stated in #11715, this patch implements the second approach for discussion.
What changes are included in this PR?

Three new APIs in SessionState for planning and optimizing.
Are these changes tested?

Are there any user-facing changes?

Sounds good 👍 I'll add one in this patch.

I add a new one planner_api.rs as the example. I didn't find an appropriate one from existing examples, please let me know if there is.


[SPARK-38907][PYTHON] Implement DataFrame.corrwith

What changes were proposed in this pull request?
implement DataFrame.corrwith

parameters axis,numeric_only are not supported,
only the default pearson correlation is supported.

Why are the changes needed?
Increase pandas API coverage in PySpark
Does this PR introduce any user-facing change?
yes, new function added:
In [4]: ps.set_option("compute.ops_on_diff_frames", True)

In [5]: df1 = ps.DataFrame({"A":[1, 5, 7, 8],  "X":[5, 8, 4, 3],  "C":[10, 4, 9, 3]})

In [6]: df2 = ps.DataFrame({"A":[5, 3, 6, 4],  "B":[11, 2, 4, 3], "C":[4, 3, 8, 5]})

In [7]: df1.corrwith(df2)
Out[7]:
A   -0.041703
C    0.395437
X         NaN
B         NaN
dtype: float64

In [8]: df1.corrwith(df2.B)
Out[8]:
A   -0.844007
X   -0.151186
C    0.767234
dtype: float64

How was this patch tested?
added UT

Thanks @zhengruifeng for working on this. @xinrong-databricks @itholic FYI

build link: https://github.com/zhengruifeng/spark/actions/runs/2171318537

all tests passed

build link: https://github.com/zhengruifeng/spark/actions/runs/2181710517  all tests passed

cc @HyukjinKwon

Also same as https://github.com/apache/spark/pull/36246#issuecomment-1105830519.

I took a cursory look, and LGTM. Would be great to have a second look from @itholic and @xinrong-databricks

Merged to master.