Issue in passing tenant domain in Federation and SAML SSO with office 365

Description

Follow [1] and configure Azure AD to trust IS
Follow [2] and configure office 365 federation with IS for a tenant user. - Eg : User name<EMAIL_ADDRESS>Try to access the url "https://login.microsoftonline.com/" once the configurations are done.
Enter the user name with in the ofice 365 login page - Eg<EMAIL_ADDRESS>
redirected to IS login page
5. Enter the complete user name with domain Eg : kaushie6<EMAIL_ADDRESS>The user domain is not recognized. (screenshot attached)
The same issue can be observed during SAML SSO with office 365.
How do we pass the tenant domain here.
In the attempt to add a user on Azure AD with along with the tenant domain for User principle name , an Invalid Syntax error is thrown.
What is the correct way to handle this?
Note that this works in super tenant mode.
https://localhost:9444/authenticationendpoint/login.do?client-request-id=f893af43-dfb4-4b22-a12a-8e480ddfba1b&commonAuthCallerPath=%2Fpassivests&forceAuth=false&passiveAuth=false&popupui=&wa=wsignin1.0&wctx=estsredirect%3D2%26estsrequest%3DrQIIAeNisNLJKCkpKLbS1y_ILypJzNHLzUwuyi_OTyvJz8vJzEvVS87P1csvSs9MAbGKhLgEnNdP_rHlvpL3Qq0-D977u6RXMSoTNkL_AiPjC0bGW0yC_kXpninhxW6pKalFiSWZ-XmPmHhDi1OL_PNyKkPys1PzJjHz5eSnZ-bFFxelxafl5JcDBYAmFCQml8SXZCZnp5bsYlZJTTW3NDJJTdNNtTQy0jUxSk7WTTQ1NtE1MUk2MTc1SDazTLO4wCLwg4VxESvQzec2xEndFnru0iHjWVnmKyp8ilW_IiTY3Du8LCUvxcLFIynP1dXU0MfP0a0828PTNSitwLcy16UgM8LJL0-_3NbAyvAAJyMA0&wtrealm=urn%3Afederation%3AMicrosoftOnline&sessionDataKey=a3a8f2b3-ef82-46db-8775-51c488459eb0&relyingParty=urn%3Afederation%3AMicrosoftOnline&type=passivests&sp=Office365&isSaaSApp=false&authenticators=BasicAuthenticator:LOCAL&authFailure=true&authFailureMsg=user.tenant.domain.mismatch.message

[1] https://docs.wso2.com/display/IS520/Configuring+Azure+Active+Directory+to+Trust+WSO2+Identity+Server
[2] https://docs.wso2.com/display/IS520/Configuring+Office+365+WS-Federation+with+Identity+Server
Note
There are two important points to note.

When configuring the IDP URL in Azure AD we have to give the correct tenants IDP URL. Which means if the SP is registered in a tenant, the IDP UTL must have the query param tenantDomain=wso2.com. Else if the SP is registered in super tenant space SaaS application configuration must be enabled. The screen shot attached indicates probably this kind of issue.

The subject value that is sent back to Azure AD must be understood by Azure AD as a unique AD user. I.e. if the subject value ends with "wso2.com", which is the tenant domain in IS, it doesn't mean anything to AD and it can't understand that subject. You may have to remove this part by unchecking "Use tenant domain in local subject identifier".

Following Results were observed after applying the above two points

Passed the tenant domain as a query parameter in the $passiveLogonUri.
"Use tenant domain in local subject identifier" was not checked.
The SP was registered in a tenant domain and the user was registered within the same tenant domain.

redirected to IS login page
Enter the complete user name with domain Eg : akk666<EMAIL_ADDRESS>redirected to'https://login.microsoftonline.com/login.srf' (Screenshot attached)
User does not get logged in.

Type
Bug
Priority
Normal
Affected version
5.2.0, 5.8.0
Environment
30th Aug (pack) beta2
PostgreSQL 9.5.3
Ubuntu 14.04
JDK 1.7
Active Directory primary user store
Component/s
saml2-sso
Severity
Major
Estimated Complexity
Moderate
Label/s
Type/Improvement, Component/Identity Governance, Component/Identity Mgt, Affected/5.2.0, Affected/5.8.0, Severity/Major, Complexity/Medium, Priority/Normal
Moved from https://wso2.org/jira/browse/IDENTITY-5516

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


add a quic.Config option to request connection ID truncation

Fixes #524, fixes #434.
I don't really like the name of the quic.Config option though, maybe @lucas-clemente has a better idea (RequestConnectionIDTruncation or so, but that's too long).
I added a handshake.TransportParameters struct. The name comes from the QUIC WG draft (https://quicwg.github.io/base-drafts/draft-ietf-quic-transport.html#transport-parameter-definitions). Currently it only has one field, but it will need some more in the future. I hope this will make contributing to the other want-help issues easier.
The option is currently only valid for the client. In principle, we could also allow servers to allow connection ID truncation, but we'll need to add some logic to the server to assign packets to the right sessions then.

Codecov Report

Merging #606 into master will increase coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #606      +/-   ##
==========================================
+ Coverage   90.85%   90.87%   +0.02%
==========================================
  Files          79       78       -1
  Lines        5453     5464      +11
==========================================
+ Hits         4954     4965      +11
  Misses        319      319
  Partials      180      180

Impacted Files
Coverage Δ

h2quic/client.go
88.48% <100%> (+0.07%)
:arrow_up:

session.go
87.44% <100%> (+0.35%)
:arrow_up:

server.go
90.73% <100%> (ø)
:arrow_up:

handshake/crypto_setup_client.go
89.12% <100%> (+0.12%)
:arrow_up:

client.go
84.42% <100%> (+0.1%)
:arrow_up:

crypto/config_for_client_1.8.go

Continue to review full report at Codecov.

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


PIO SM refactor

DMA allows you to get the DMA channel number from the claim API. PIO however makes you suggest and test the PIO state machine.
Would it not be better to make the PIO SM API use a pio_hw_t reference? Technically DMA could work this way also.

pio_claim_unused_sm works.

@kilograham Why didn't you name it pio_sm_claim_unused? Do you have any plans to create GPIO claim API?

Do you have any plans to create GPIO claim API?

A claim_unused_gpio doesn't seem very useful, given that you need to actually have stuff wired up to the GPIO pins to actually do something? :electric_plug:

Even though it is static some people do dynamic things with it. This can be chaotic. Ideally they belong to a manager which passes or releases the IO resource. This would prevent hazards via dependency/sanity checks. Overall this requires good style to matter, but does the SDK need to forbid this?
Ideally the modules are decoupled. We claim the modules with minimal impact to other modules. Shared resources are singletons while non-shared are factories. (PIO has a strange singleton in the instruction memory.) Most of the GPIO multiplexer and clock tree are singletons which create dependencies.
Would have been interesting, if the RISC-V used 128-bit SIMD in RP235x.

see #1490


Extended Onboarding Experiment

Task/Issue URL:
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

#4417  👈
develop

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @nalcalag and the rest of your teammates on  Graphite


Implement GetPropertyChangedSignal

We need for this a feature of Roact!

Linked to #1?

Nah, this one is a pretty simple change; it'd work just like how Changed works right now in Lemur.

Whoops, forgot lemur had that.

This should be closed now

Thanks!


feat: add besu el

Signed-off-by: salaheldinsoliman<EMAIL_ADDRESS>Feature
Changes:

Added Besu required template

Types of changes
What types of changes does your code introduce?

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Other (please describe):

Testing
Requires testing

[x] Yes
[ ] No

In case you checked yes, did you write tests?

[ ] Yes
[x] No

Need to run smoke tests

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

Coverage data is based on head (61ff1b8) compared to base (815e75d).
 Consider uploading reports for the commit ad83e80 to get more accurate results

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #123   +/-   ##
========================================
  Coverage    85.15%   85.15%
========================================
  Files           34       34
  Lines         2008     2008
========================================
  Hits          1710     1710
  Misses         239      239
  Partials        59       59

Flag
Coverage Δ

unittests
85.15% <ø> (ø)

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

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


Indicator lines don't seem to consistently zoom with candles

Spawned from #20. One very very very minor observation is the zoom seems slightly inconsistent. You can see here, the indicator seems shifted-slightly to the right of the last candle:

If I zoom in and move it to the left, it fits correctly:

If I move the same view to the right, you see the offset again:

But this is very very minor, something about how the zooming works.

I don't think it's very minor though. Could you post your data-set (without trades) to reproduce the problem? Github gist will be perfect.

Absolutely, trades are lazily included if that's okay:
https://gist.github.com/tiagosiebler/2356a4f4f291f45a2fb5d1b6f8e3c5d3

It turned out that  it's a duplicate of #17
Do you have missing candles? That's an exact issue I'm working on today.

Missing candles are possible, as my dataset is limited to what the exchange has available, and the exchange may have time gaps (e.g downtime). In this example, data's based on candles from the crypto exchange, binance.

The candle count and indicator value count should be a match though, should be an equal number of rows.

Yeah, it's all about my candle-rendering method.

You can update the repo and check - it should be ok now.

Looks great! Thank you, nice work!


I think the traversal of the refqueue of the memory leak detector should be implemented using a daemon thread similar to ObjectCleaner

private void reportLeak() {
//如果不支持记录错误级别日志 清理掉引用queue
if (!logger.isErrorEnabled()) {
//清理掉refQueue中所有的引用对象
clearRefQueue();
return;
}
    // 当对象T被gc回收后，追踪器添加到refQueue中
    for (;;) {
        //获取追踪器
        @SuppressWarnings("unchecked")
        DefaultResourceLeak ref = (DefaultResourceLeak) refQueue.poll();
        if (ref == null) {
            break;
        }

        //删除追踪器在allLInk map中
        if (!ref.dispose()) {
            continue;
        }

        //获取追踪器的记录（遍历Record链表）
        String records = ref.toString();
        //添加到报告过的的追踪器map
        if (reportedLeaks.putIfAbsent(records, Boolean.TRUE) == null) {
            //没有没有record信息的几率
            if (records.isEmpty()) {
                reportUntracedLeak(resourceType);
            } else {
                //记录有信息的泄露
                reportTracedLeak(resourceType, records);
            }
        }
    }
}

This string of code does not guarantee that the trace record will be written to the log

We don't want to introduce another Thread here. Why you think the current implementation is not sufficient ?

We don't want to introduce another Thread here. Why you think the current implementation is not sufficient ?

Zhège fāngfǎ de for xúnhuán zhǐ néng chǔlǐ zhīqián de zhuīzōng qì zuìhòu yīgè tiānjiā dào duìliè de zhuīzōng qì bù huì bèi chǔlǐ
39/5000
The for loop of this method can only process the previous tracker. The last tracker added to the queue will not be processed.

sure but it will be processed "at some point". So this should be good enough.

Closing this as I think it works as expected


USelectMenu current active not work if using fetch data from API

const QUERY = gql`
  query prfSelect {
    items: prf_check {
      id
      lo: prf_lo
      en: prf_en
    }
  }
`

const items = ref([])

const { data } = await useAsyncQuery<any>(QUERY)

if (data.value) {
  items.value = data.value.items
  console.log(items.value)
}

Version
@nuxt/ui:  ex: v2.8.1
nuxt: ex: v3.7.3
Reproduction Link
https://stackblitz.com/edit/nuxt-starter-rpvejz?file=pages%2Findex.vue

Steps to reproduce
What is Expected?
What is actually happening?

Would you mind providing a StackBlitz reproduction? Did you set a v-model?

Would you mind providing a StackBlitz reproduction? Did you set a v-model?

https://stackblitz.com/edit/nuxt-starter-rpvejz?file=pages%2Findex.vue
<template>

    Welcome To NUXT3

    userID: {{ selected }}
    <USelectMenu
      v-model="selected"
      value-attribute="id"
      option-attribute="email"
      :options="users"
    />

</template>

This has already been fixed in https://github.com/nuxt/ui/commit/0544a01c5b7ae534a595e6c91d2884a601ae3185 but not released yet as you can see when using the edge version: https://stackblitz.com/edit/nuxt-starter-s1xxgn?file=pages%2Findex.vue


Add username choice for sign buffer requests

This will allow for websites to allow users to select the user from a dropdown instead of specifying the user ahead of time.

The response's data field will have the chosen username.
Example page is hosted https://eonwarped.github.io/steem/sk/main.html  as well.

Now that I did this, I realize that I would prefer to do something more generic:
Just something like  "requestUser"

User selects user from dropdown
Username gets returned with available auths.

For now, just want to simplify the scope to just sign buffer. It will then be used as the source for getting an account and then requesting any extra steps afterwards.
Changed behavior so that there is no 'enforce username' toggle. Instead, the API is such that it will only 'enforce' if the username is provided. Otherwise it will show a dropdown.


Redshift: Patch Redshift DDL for the com.callrail/call_complete event

Thanks! Putting this into the Blocked schemas milestone, as we cannot unfortunately merge this into master without breaking pipelines in the wild...

I believe this should address everything. I can issue PR if you would like - or wait until the milestone completion gets closer.

You are very welcome to open the PR @digitaltouch - thanks...

Migrated to: https://github.com/snowplow/iglu-central/issues/534


Development

What's the purpose of this PR?

Codecov Report

Merging #67 into master will increase coverage by 1.61%.
The diff coverage is 88.88%.

@@             Coverage Diff             @@
##             master     #67      +/-   ##
===========================================
+ Coverage     82.59%   84.2%   +1.61%
- Complexity      331     511     +180
===========================================
  Files            29      58      +29
  Lines          1235    1564     +329
  Branches        211     250      +39
===========================================
+ Hits           1020    1317     +297
- Misses           85      96      +11
- Partials        130     151      +21

Impacted Files
Coverage Δ
Complexity Δ

...ents/internal/models/v1/SocialMediaHelperImpl.java
75.97% <ø> (ø)
24 <0> (?)

...rnal/servlets/FormActionTypeDataSourceServlet.java
90.47% <ø> (ø)
4 <0> (ø)
:arrow_down:

...e/components/internal/models/v1/form/TextImpl.java
84.61% <ø> (ø)
16 <0> (?)

...onents/internal/models/v1/form/OptionItemImpl.java
100% <ø> (ø)
5 <0> (?)

...re/components/internal/jackson/PageSerializer.java
100% <ø> (ø)
5 <0> (?)

...internal/servlets/TextValueDataResourceSource.java
72.72% <ø> (ø)
3 <0> (ø)
:arrow_down:

...components/internal/models/v1/form/ButtonImpl.java
71.42% <ø> (ø)
8 <0> (?)

...ponents/internal/models/v1/NavigationItemImpl.java
100% <ø> (ø)
3 <0> (?)

...onents/internal/servlets/AdaptiveImageServlet.java
80.69% <ø> (ø)
55 <0> (ø)
:arrow_down:

...ponents/internal/models/v1/form/ContainerImpl.java
84.37% <ø> (ø)
15 <0> (?)

... and 79 more

Continue to review full report at Codecov.

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


target conformance to MultipartFormUploadable

I merged in v2/api-client to fix conflicts and yanked the rfv. Slap that bad boy back up there when you get the request functions changed to take multipart parts.


[Bug]

Describe the bug
The BOt When Opens the chrome it automaticly detects that it has an bot behindf and twitch wont allow to log in

Screenshots

Logs
[2022-11-17 16:00:44] [error] Uncaught exception: Error: Page closed!
at onPageClosed (file:///C:/Users/fuher/Twitch-Drops-Bot/dist/pages/login.js:36:23)
at C:\Users\fuher\Twitch-Drops-Bot\node_modules\puppeteer-core\lib\cjs\third_party\mitt\index.js:3:232
at Array.map ()
at Object.emit (C:\Users\fuher\Twitch-Drops-Bot\node_modules\puppeteer-core\lib\cjs\third_party\mitt\index.js:3:216)
at CDPPage.emit (C:\Users\fuher\Twitch-Drops-Bot\node_modules\puppeteer-core\lib\cjs\puppeteer\common\EventEmitter.js:87:22)
at C:\Users\fuher\Twitch-Drops-Bot\node_modules\puppeteer-core\lib\cjs\puppeteer\common\Page.js:187:18
at processTicksAndRejections (node:internal/process/task_queues:96:5)
[2022-11-17 16:00:47] [error] Error: Protocol error (Network.getCookies): Session closed. Most likely the page has been closed.
at CDPSessionImpl.send (C:\Users\fuher\Twitch-Drops-Bot\node_modules\puppeteer-core\lib\cjs\puppeteer\common\Connection.js:316:35)
at next (C:\Users\fuher\Twitch-Drops-Bot\node_modules\puppeteer-extra-plugin-stealth\evasions\sourceurl\index.js:34:41)
at CDPSessionImpl.send (C:\Users\fuher\Twitch-Drops-Bot\node_modules\puppeteer-extra-plugin-stealth\evasions\sourceurl\index.js:67:18)
at CDPPage.cookies (C:\Users\fuher\Twitch-Drops-Bot\node_modules\puppeteer-core\lib\cjs\puppeteer\common\Page.js:724:91)
at LoginPage. (file:///C:/Users/fuher/Twitch-Drops-Bot/dist/pages/login.js:65:53)
at Generator.next ()
at fulfilled (file:///C:/Users/fuher/Twitch-Drops-Bot/dist/pages/login.js:4:58)
C:\Users\fuher\Twitch-Drops-Bot>npm run start

<EMAIL_ADDRESS>start
node dist/index.js

[2022-11-17 16:03:19] [info] Loading config file: config.json
[2022-11-17 16:03:19] [info] Logging in...
[2022-11-17 16:03:30] [info] Waiting for cookies to be created...
[2022-11-17 16:03:33] [info] Waiting for cookies to be created...
[2022-11-17 16:03:36] [info] Waiting for cookies to be created...
[2022-11-17 16:03:39] [info] Waiting for cookies to be created...
[2022-11-17 16:03:42] [info] Waiting for cookies to be created...
[2022-11-17 16:03:45] [info] Waiting for cookies to be created...
[2022-11-17 16:03:48] [info] Waiting for cookies to be created...
[2022-11-17 16:03:51] [info] Waiting for cookies to be created...
[2022-11-17 16:03:54] [info] Waiting for cookies to be created...
[2022-11-17 16:03:57] [info] Waiting for cookies to be created...
[2022-11-17 16:04:00] [info] Waiting for cookies to be created...
[2022-11-17 16:04:03] [info] Waiting for cookies to be created...
[2022-11-17 16:04:06] [info] Waiting for cookies to be created...
[2022-11-17 16:04:09] [info] Waiting for cookies to be created...
[2022-11-17 16:04:12] [info] Waiting for cookies to be created...
[2022-11-17 16:04:15] [info] Waiting for cookies to be created...
[2022-11-17 16:04:18] [info] Waiting for cookies to be created...
[2022-11-17 16:04:21] [info] Waiting for cookies to be created...
[2022-11-17 16:04:24] [info] Waiting for cookies to be created...
[2022-11-17 16:04:27] [info] Waiting for cookies to be created...
[2022-11-17 16:04:30] [info] Waiting for cookies to be created...
[2022-11-17 16:04:33] [info] Waiting for cookies to be created...
[2022-11-17 16:04:36] [info] Waiting for cookies to be created...
[2022-11-17 16:04:39] [info] Waiting for cookies to be created...
[2022-11-17 16:04:42] [info] Waiting for cookies to be created...
[2022-11-17 16:04:45] [info] Waiting for cookies to be created...
[2022-11-17 16:04:48] [info] Waiting for cookies to be created...
Terminate batch job (Y/N)? y
C:\Users\fuher\Twitch-Drops-Bot>

See if the fix from this comment works for you.
https://github.com/TychoTheTaco/Twitch-Drops-Bot/issues/238#issuecomment-1284550648

no, ive done this before and did not resolve

Hi @zarigata, since you are getting this message [2022-11-17 16:03:33] [info] Waiting for cookies to be created..., it means you don't have the cookies.json.
If you have the cookies.json then try running this:
C:\Users\fuher\Twitch-Drops-Bot>node dist/index.js --cookies-path C:/Users/fuher/Twitch-Drops-Bot/cookies.json

instead of npm run start.


Document how to use system prefs to manage wallpapers

Document how to use OS X to do the wallpaper management via the System prefs.
I'm opting for this solution vs. programmatically setting desktops. I think this gives the user more control anyway.
https://github.com/sindresorhus/macos-wallpaper/issues/4

Just now realizing that the OS solution will not update the desktop with the latest pano image.
I'm thinking I can run a JSX (formerly AppleScript) solution to set desktops from node.
https://github.com/JXA-Cookbook/JXA-Cookbook/wiki

SMH at apple. I've tried numerous solutions and they all have the same limitation. You have to be on the desktop in order for it to change.
Seriously lame.
Maybe the rule will be, run the script on desktop you want it to appear on; one could set random image on one desktop and a specific image on another desktop? This is turning into a waste of a Sunday.

Wiki page added; will be improved over time.


Account, personal data: spelling mistakes

Reproduce:

Log in
Go to Account > Personal data

2 issues:

The spelling of the button "Change Name or Email" doesn't conform to the overall spelling of links of that category across the site. Only the first word should be capitalized, anything else should be written in lowercase. Expected: "Change name or email".
Do we want to use the personal pronoun in the links showing here or not? The current solution is inconsistent: "Change Name or Email", "Change your password", "Delete your account".

Issues still present for poster version 2.1.


math: Two display-style equations in a quote env don't render

Description
Another weird little math bug: This
> \[
> a
> \]
> \[
> b
> \]

isn't shown as two subsequent equations, but as

For comparison, this is what
> ```math
> a
> ```
> ```math
> b
> ```

renders as on GitHub:

($$-math in quotes in broken.)
Gitea Version
1.22.0+dev-233-g796ff26e0
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
No response
How are you running Gitea?
https://try.gitea.io/
Database
None

Still an issue on 1.23.0+dev-753-gc5422fae9a (Dec 2024).

Will fix it in 1.23

1.23 nightly is ready and I think it should have been fixed, feel free to take a try

Confirmed working, thanks!


BatchLabs and Azure portal does not show any data

Hi,
I`ve just install appinsight as it said in docs and set env variables but I do not see any data/graphs in BatchLabs and neither in the Azure portal. Is there any way to check/debug that?

Yep, its working!


npm build warnings

How can we (and should we) suppress these warnings?
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS> (node_modules\chokidar\node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS> wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"})

This is a normal warning regarding optional dependencies not used in windows.


SPA framework on top of the MVC

HI,
Are you thinking of creating SPA framework on top of the MVC? Let’s say there are tag helpers or special attributes to the existing tags which automatically generates SPA framework.  The goal is to turn parts of the app into SPA.

At moment the main contribution of Asp.net core to spa are JavaScriptServices, that are tools to execute JavaScript code and to render SPA views on the server side by exploiting Node.js. This way, initial pages of SPA may be rendered on the server side with the following advantages:

SPA might be indexed by search engines
Rendering of first page of SPA might be fast also on slow portable devices.

However, in the Asp.net core Mvc Controls Toolkit we are trying to implement something very similar to what you described. Namely, all toolkit tag helpers (that include also complex templated grids) are rendered by providers. There may be providers for both server side technologies and for client side technologies. Thus, the same tag helper may generate either standard Mvc code or code for some famous SPA framework (angular2 for instance).
At moment we have implemented just a server provider, but after the next release that should be delivered next week, we will start the implementation of an angular2 provider.

Are you thinking of creating SPA framework on top of the MVC?

We have current plans to implement a javascript framework.

Let’s say there are tag helpers or special attributes to the existing tags which automatically generates SPA framework. The goal is to turn parts of the app into SPA.

A big part of tag helpers is that you're writing a lot of the HTML yourself so you can just write the HTML and we get out of the way.
If you've got a great idea for how something like this would work with your framework of choice give a try and let us know if you hit any blockers.


SSR developing with redux modal - window is not defined

I'm developing server side rendered react App.I'm using redux modal for sign up.
i'm getting below error.

/home/xx/Documents/Projects/..../node_modules/redux-reactstrap-modal/build/redux.reactstrap.modal.js:10 })(window, function(WEBPACK_EXTERNAL_MODULE__0, WEBPACK_EXTERNAL_MODULE__1, WEBPACK_EXTERNAL_MODULE__2, WEBPACK_EXTERNAL_MODULE__3, WEBPACK_EXTERNAL_MODULE__4, WEBPACK_EXTERNAL_MODULE__5) { ^
ReferenceError: window is not defined at Object. (/home/home/xx/Documents/Projects/..../node_modules/redux-reactstrap-modal/build/redux.reactstrap.modal.js:10:4)

https://stackoverflow.com/questions/54086516/redux-reactstrap-modal-window-is-not-defined-in-ssr

Fixed by #8

Fix is deployed on v2.0.0


Bring back documentation on Link DOM props

In v3 the docs clearly indicated that <Link> takes DOM props to be put on <a>.
In v4 docs this information is missing.
Nevertheless, the functionality is still here.
The contributors refer to it as expected, public functionality.
The activeClassName prop on NavLink clearly suggests that a regular className is also accepted.

Thanks!


[Feature]: Move utils used by profile and organization

Contact Details
No response
What do you like to be improved?
Some utils functions are at profile.server.ts but used at organization, too.

Idea: Some utility functions are only used in specific routes. Why not add utils.server.ts for each route?


[Feature]:

Description
Hi,
I don't find nothing clear about logging new tables (or logging whatever in general) to an existing and finished run. I would like to be able to run my trianing in a remote machine and then in my local machine log new tables to the run created in the remote machine which is supposed to be finished.
Suggested Solution
I want to be able to compute metrics (for example) and log them in a table to an existing and finished run. I don't know how to implement this in low-level because I am just a user.
Alternatives
Be able to create subproject and link runs between subprojects. For example, given project "foo" I would spawn two subprojects one for training results and one for test/val/whatever results: "foo-train", "foo-test". Then, link runs between these two subprojects in order to visualize them in a single panel.
Additional Context
No response

Hi @MBakirWB,
Definitely I should get more familiar with Weaves, thanks for pointing that out ;).


release-notes/docker-engine: add missing closing curly dude

Proposed changes
I fixed the syntax of a {% raw %} tag so the site would build.
Without this, you get Syntax Error in 'raw' - Valid syntax: raw in release-notes/docker-engine.md
Unreleased project version (optional)
I think this applies to any ref which has this line!
Related issues (optional)
Just passing by... 😄

Thanks, @parkr - nice to see you again. :) Hopefully our GH Pages builds will work again!

😢 🎺 wah wahhh still broken

I'll take a closer look tonight! Sorry about this, y'all.

I'm sure it's our fault, it's just pretty opaque to troubleshoot.

Unfortunately so. The build logs are sent to an internal system. Our Support team is incredibly helpful and will help folks debug by getting the error message & troubleshooting it for you if you write in and ask, but that takes time. I'll try to get a reproducible development build going here which can help anyone debug an issue. As a bonus, I'll use our docker images!

Got it! Another one of these raw tag issues (https://github.com/docker/docker.github.io/pull/4256). It looks like y'all have a Gemfile setup. Do you want a Dockerfile or a script that runs Jekyll in a container? If so, are there any best practices you'd like me to follow (e.g. at GitHub, this would be put in script/build to follow our convention of Scripts To Rule Them All in script/)?


[Request deployment] New Hub: UW ISEA

This hub will serve the ISEA Hackweek hosted by the University of Washington during July 2024.
https://www.amplifylearn.ai/isea/
Hub important dates

Target Start Data: 2024-06-22
Required Start Date: 2024-07-01
Shutdown Date: 2024-07-31

Phase 2: Cluster
Deploy on 2i2c-aws-us
Phase 3 : Hub setup
Phase 3.1: Initial setup (READY)

Question
Answer
Notes

Name of the hub
isea

Dask gateway
no

Splash image
AmplifyLearn.AI logo
see comment below

URL
https://www.amplifylearn.ai/isea/

funded_by
Institute of Education Sciences (IES)

Authentication Mechanism
GitHub (via GitHubOAuthenticator)

Admin Users
@lovenoor

At the end of this phase, both 2i2c engineers and the admin users mentioned can login to the hub.
Phase 3.2: Object storage access  (READY)

Question
Answer
Notes

Scratch bucket enabled?
no

Persistent bucket enabled?
no

Requestor pays requests to external buckets allowed?
no

Phase 3.3: Profile List (READY)
These are the standard profile list options we want to start with. Can be customized later

Question
Answer
Notes

profileList used?
no

Image?
jupyter/scipy-notebook
Use latest available date tag for jupyter/scipy-notebook

RStudio (with Rocker)?
no

Allow users to specify any image they want to use?
no
Enable unlisted_choice

Max RAM option allowed
guarantee: 384M   limit: 1.5G
(this is the same as the cloudbank hubs)

GPU enabled?
no

Default Interface
JupyterLab

Allow multiple concurrent servers per user?
no
Translates to allowNamedServers

Phase 3.4: Authentication tuning (READY)

Question
Answer

Authentication Mechanism
GitHub (via GitHubOAuthenticator)

Org based access?
no

List of GitHub orgs / teams to be granted access

Initial hub admin user
@lovenoor

Note:
Other users can be added through the JupyterHub interface  (expected ~30 users total)

Phase 3.5: Profile List finetuning

Question
Answer
Notes

Custom image to be specified?
N/A

FYI @lovenoor. This issue will track 2i2c's deployment of the ISEA hub.

The anchor comment in this issue will be refactored by @jmunroe to follow 2i2c's recent improvements on the hub turn up process.

Hi @lovenoor .  Is this a suitable logo to use a 'Splash Image' for this hub? If not, please let us what you would prefer.

I'll recommend that you go with a jupyter/scipy-notebook image which has a number of useful Python packages for data science (including pandas, numpy, seaborn, sklearn, statsmodels, and random) that it looks like you have introduced in the slides.
If your hackweek projects end up wanting to investigate frameworks like pytorch or tensorflow, it is very easy to also deploy a different prebuilt image. See https://jupyter-docker-stacks.readthedocs.io for additional information.
The RAM per user is set to max out at just 1.5G but we can increase this if you anticipate more memory per user.
Your participants can either be dropped into a single predefined image that you control or you could give them a list of different image choices (and amounts of available memory).  This tradeoff is more of a pedagogical choice for you: what do you want to make available for your participants.  It also can be changed after the hub is deployed.

Hi James,
Yes, the splash image is suitable.
For now, the jupyter image seems sufficient. Will let you know if we need
to change things up.
Lavi
Lovenoor (Lavi) Aulck
Data Scientist - Finance, Planning & Budgeting
Managing Director - Innovation Science for Education Analytics
University of Washington
On Fri, May 17, 2024 at 11:59 AM James Munroe @.***>
wrote:

Hi @lovenoor https://github.com/lovenoor . Is this a suitable logo to
use a 'Splash Image' for this hub? If not, please let us what you would
prefer.
download.png (view on web)
https://github.com/2i2c-org/infrastructure/assets/6181563/df7f288d-0ca3-4bbb-9fef-2a98892ac0f6
I'll recommend that you go with a jupyter/scipy-notebook
https://jupyter-docker-stacks.readthedocs.io/en/latest/using/selecting.html#jupyter-scipy-notebook
image which has a number of useful Python packages for data science
(including pandas, numpy, seaborn, sklearn, statsmodels, and random) that
it looks like you have introduced in the slides.
If your hackweek projects end up wanting to investigate frameworks like
pytorch or tensorflow, it is very easy to also deploy a different prebuilt
image. See https://jupyter-docker-stacks.readthedocs.io for additional
information.
The RAM per user is set to max out at just 1.5G but we can increase this
if you anticipate more memory per user.
Your participants can either be dropped into a single predefined image
that you control or you could give them a list of different image choices
(and amounts of available memory). This tradeoff is more of a pedagogical
choice for you: what do you want to make available for your participants.
It also can be changed after the hub is deployed.
—
Reply to this email directly, view it on GitHub
https://github.com/2i2c-org/infrastructure/issues/4090#issuecomment-2118212157,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC2LQVEETNJ5COHY6GQOUMTZCZHSJAVCNFSM6AAAAABH2VN7C6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMJYGIYTEMJVG4
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi @lovenoor, the hub was deployed here: https://isea.aws.2i2c.cloud/. Can you log in?

Hi Alejandro,
It looks like I'm able to log in.
Lavi
Lovenoor (Lavi) Aulck
Data Scientist - Finance, Planning & Budgeting
Managing Director - Innovation Science for Education Analytics
University of Washington
On Tue, Jun 11, 2024 at 9:21 PM Alejandro Ismael Silva <
@.***> wrote:

Hi @lovenoor https://github.com/lovenoor, the hub was deployed here:
https://isea.aws.2i2c.cloud/. Can you log in?
—
Reply to this email directly, view it on GitHub
https://github.com/2i2c-org/infrastructure/issues/4090#issuecomment-2162076543,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC2LQVEM52IV5F4CSP4KHR3ZG7EERAVCNFSM6AAAAABH2VN7C6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRSGA3TMNJUGM
.
You are receiving this because you were mentioned.Message ID:
@.***>


Update UG

Updates UG with:

screenshot of client -a command
screenshot of client -e command
screenshot of client -d command
screenshot of client -f command
screenshot of issue -f command
screenshot of project -f command
Explanation of the mobile phone in the context of international codes

Closes #167 , #174

Codecov Report

Merging #210 (555d0c9) into master (2cf2949) will increase coverage by 0.01%.
The diff coverage is 25.00%.

@@             Coverage Diff              @@
##             master     #210      +/-   ##
============================================
+ Coverage     38.75%   38.77%   +0.01%
  Complexity      505      505
============================================
  Files           124      124
  Lines          3042     3041       -1
  Branches        381      382       +1
============================================
  Hits           1179     1179
+ Misses         1696     1695       -1
  Partials        167      167

Impacted Files
Coverage Δ

...ddress/logic/commands/client/PinClientCommand.java
100.00% <ø> (ø)

.../address/logic/commands/issue/PinIssueCommand.java
100.00% <ø> (ø)

...ress/logic/commands/project/PinProjectCommand.java
100.00% <ø> (ø)

...ess/logic/commands/project/SortProjectCommand.java
0.00% <ø> (ø)

...in/java/seedu/address/logic/parser/ParserUtil.java
27.95% <25.00%> (+0.17%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Fixes #181


[Enhancement] Nullsafety

Other than the given nullsafety breaking change, I've changed the parameters for the the client to take the port number as an optional parameter.

Thank you, I'll find some time to review this soon.

@isoos

First check looks really good, I have only a few small worries and requests.
Please also write a CHANGELOG entry for this change.

Should I bump the version up too? Seems like a major release.

It does break for apps that use earlier versions of the flutter sdk though, right? Maybe a prerelease would work? Or a 2.3.0-nullsafety?

On Feb 28, 2021, at 2:14 AM, István Soós<EMAIL_ADDRESS>wrote:
@isoos commented on this pull request.
Should I bump the version up too? Seems like a major release.
There is no functional or otherwise breaking change, so this could be 2.3.0. The SDK constraint makes it clear for clients on 2.10 that they won't be able to use it. We can then do a 3.0.0 with the proposed change of the port property and maybe other parts can have adjustments...
In lib/src/connection.dart https://github.com/stablekernel/postgresql-dart/pull/153#discussion_r584271276:

 String fmtString, {

Map<String, dynamic> substitutionValues,
bool allowReuse,
int timeoutInSeconds,

Map<String, dynamic>? substitutionValues,
bool allowReuse = true,
Please remove all of these default values. It makes it much easier to develop a tool like package:postgres_pool

In lib/src/connection.dart https://github.com/stablekernel/postgresql-dart/pull/153#discussion_r584271330:

 String fmtString, {

Map<String, dynamic> substitutionValues,
bool allowReuse,
int timeoutInSeconds,
bool resolveOids,

Map<String, dynamic>? substitutionValues,
required bool allowReuse,
int? timeoutInSeconds,
bool resolveOids = true,
it could be explicitly required too.

In lib/src/connection.dart https://github.com/stablekernel/postgresql-dart/pull/153#discussion_r584271383:

   String fmtString,

 {Map<String, dynamic> substitutionValues,

 bool allowReuse,

 int timeoutInSeconds}) async {

 {Map<String, dynamic> substitutionValues = const {},

 bool allowReuse = false,

same here: please remove defaults
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/stablekernel/postgresql-dart/pull/153#pullrequestreview-600241773, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA6S7P3CWRE56LVNHZLFFELTBIJQTANCNFSM4YGHD6JQ.

It does break for apps that use earlier versions of the flutter sdk though, right?

I do not see how it would break them. As far as I know Flutter respects the Dart SDK constraints too.

Maybe a prerelease would work? Or a 2.3.0-nullsafety?

I don't see it making much difference, it can work too.

I’m still a bit new to nullsafety migration. My experiences have involved being yelled at pub get for conflicting dependencies. Just realized it’s the responsibility of the end user to upgrade the required sdk version to use the null safety features anyway. I need to check the naming standards for nullsafety. Now that I think about it -nullsafety might just be a sort of buzzword trick to get people to use the library. Does everything look ok outside of the versioning?

On Feb 28, 2021, at 2:20 AM, István Soós<EMAIL_ADDRESS>wrote:
It does break for apps that use earlier versions of the flutter sdk though, right?
I do not see how it would break them. As far as I know Flutter respects the Dart SDK constraints too.
Maybe a prerelease would work? Or a 2.3.0-nullsafety?
I don't see it making much difference, it can work too.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/stablekernel/postgresql-dart/pull/153#issuecomment-787428961, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA6S7PYXCPYND65LWUXMQGTTBIKHNANCNFSM4YGHD6JQ.

Does everything look ok outside of the versioning?

There are still pending default values in the public API that should be rather reverted, like int timeoutInSeconds = 30,. Otherwise I think it looks good.

Regarding version bump, this night be of interest: https://github.com/dart-lang/sdk/issues/41398#issuecomment-786217515
(The -nullsafety prerelease tag was only recommended while pre-beta, nowadays there so no need for that any more)

Yoops, I got it. Pushed.

On Feb 28, 2021, at 2:34 AM, István Soós<EMAIL_ADDRESS>wrote:
Does everything look ok outside of the versioning?
There are still pending default values in the public API that should be rather reverted, like int timeoutInSeconds = 30,. Otherwise I think it looks good.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/stablekernel/postgresql-dart/pull/153#issuecomment-787430750, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA6S7PYYWYUERQLNED5SV5TTBIL3HANCNFSM4YGHD6JQ.

This has been adopted in my soft-fork repository and published as 2.3.0-null-safety.0.


rustup install fails when ~/.cargo is a symlink

I would like to put ~/.cargo on another device, but rustup update produced this:
info: checking for self-updates
info: downloading self-update
error: failed to set permissions for '/home/mbp/.cargo/bin/rustup-init'
info: caused by: No such file or directory (os error 2)

For context:
% readlink ~/.cargo
/vol/ssd-2/mbp-home/.cargo

Curiously, re-running rustup update succeeded:
rustup update
info: syncing channel updates for 'stable-x86_64-unknown-linux-gnu'
info: syncing channel updates for 'nightly-x86_64-unknown-linux-gnu'
info: checking for self-updates
info: downloading self-update

   stable-x86_64-unknown-linux-gnu unchanged - rustc 1.31.0 (abe02cefd 2018-12-04)
  nightly-x86_64-unknown-linux-gnu unchanged - rustc 1.33.0-nightly (93c2f055b 2018-12-15)

Hi, marshallpierce
I tried to reproduce this issue on my Linux system, but rustup performed self update and installed beta channel succesfully. I tried a few times but every of them was successful. Are you still able to reproduce the issue?

Nope, it seems okay now. :man_shrugging:


UI hangs for indefinite time

Greetings!
So the issue happens when we are trying to load Prometheus UI and it becomes unavailable (it's not possible to click on anything, to enter the queries or change tabs).
At the same time, the Prometheus is working very well, only UI is unavailable. I've tried to launch the performance check, and it shows that some jQuery loads infinitely - screenshot is attached below.
On the screenshot, you can see that XHR Load took 49 seconds, but it actually loads much longer, I just stopped the performance check after 60 seconds.
I was using an older version of Prometheus initially (2.7 or 2.8) and then I've tried to update to the latest version, but it didn't help.
Can you please recommend something on this?
Thank you!
What did you do?
Open Prometheus UI
What did you expect to see?
Interactive, accessible UI
What did you see instead? Under which circumstances?
Frozen, not accessible UI
Environment

System information:
Linux 4.15.0-1045-aws x86_64

Prometheus version:

prometheus, version 2.13.0 (branch: HEAD, revision: 6ea4252299f542669aca11860abc2192bdc7bede)
build user:       root@f30bdad2c3fd
build date:       20191004-11:25:34
go version:       go1.13.1

Hi,

I tried on Safari, Firefox, and Chrome - results are the same. I'm able to reproduce this issue not only on my local machine - all my colleagues working on Mac/Linux laptops are experiencing the same. That makes me think that this is not browser-related. But just for the info:

Chrome: 77.0.3865.120 (Official Build) (64-bit)
Firefox: 69.0.3 (64-bit)

Same issue with the incognito mode being enabled (and plugins disabled)

This demo page works just fine for me.

Ohhhh, I see. So the issue is specific to that Prometheus server, not to the browser. The initTypeahead in your trace suggests this is about initializing the typeahead autocompletion from the metric names that your Prometheus server has. This in turn suggests that you might have a huge number of metric names and the UI cannot cope with it well.
How many metric names do you see under /api/v1/label/__name__/values on your server?

Oh wow,
I see there ~ 602233 metric names. Can you please suggest what could we do about this? Apart from reducing the number of the metric names :)

Ah yeah, Prometheus is not made for hundreds of thousands of metric names - normally you'd have a couple thousand at most :)
But this came up in these issues too: https://github.com/prometheus/prometheus/issues/5919 and https://github.com/prometheus/prometheus/issues/5421
It seems there's a PR out attempting to fix this: https://github.com/prometheus/prometheus/pull/6100

@Loopy88 As for what you can do in the short term: you might be able to use a browser plugin to block XHR requests going to /api/v1/label/__name__/values?

Thank you for the help and information @juliusv, I hope this PR will fix this situation soon!
In the meantime, I'll try to figure out why we have so many metric names.
And yes, blocking the requests coming to *://*/api/v1/label/__name__/values?_=* is helping as a temporary workaround.

We went through this issue in our bug scrub and are closing this as a dupe of #5421


Refactor: Use loader to fetch contributors data.

fixes #745

Ye to major refactor lag rha tha Sir 🥹😭

@priyankarpal I've resolved it. Please Review.

Increased @priyankarpal

Looks Good 🎉

By the way I think, We can fetch only 100 contributors at max using this endpoint? Because it is still fetching only 100 contributors.


Removing .setInner() from public access; changing CreatedResources into Map; sample enhancements

users should not be able to call .setInner(), and we already documented it as "internal use only". It is not needed on the interface
CreatedResources is now a map for better discoverability of how to get a created resource based on the key of the creatable definition
updated (and enhanced) the samples affected by this

Flagging as compat-breaks relative to beta4.1 since:

we're removing the .setInner() method, but in reality users should not have written code against it because it was documented as "internal use only"
CreatedResources returned by the parallel create() is now a Map<String, T> indexed by the key of the definition ("creatable")

Can one of the admins verify this patch?

Looks good to me with 2 minor comments.

@anuchandy, since i wound up piling more changes onto the previous PR (due to issues with our CI), it'd be great if you could also take a look - in particular the CreatedResources changes


Step 55 of OOP game should allow for multiple valid solutions

Describe the Issue
A camper was trying to pass with this valid code here
window.addEventListener("keydown", (event) => {
  const {key} = event;
});

but couldn't because the test is expecting this solution
window.addEventListener("keydown", ({ key }) => {

});

we should update the tests to allow for multiple valid solutions
Affected Page
https://www.freecodecamp.org/learn/javascript-algorithms-and-data-structures-v8/learn-intermediate-oop-by-building-a-platformer-game/step-55
Your code
see explanation above
Expected behavior
see explanation above
Screenshots
No response
System

Device: [e.g. iPhone 6, Laptop]
OS: [e.g. iOS 14, Windows 10, Ubuntu 20.04]
Browser: [e.g. Chrome, Safari]
Version: [e.g. 22]

Additional context
No response

Step 57 has similar pattern https://www.freecodecamp.org/learn/javascript-algorithms-and-data-structures-v8/learn-intermediate-oop-by-building-a-platformer-game/step-57.
Personally I'd reword instructions to indicate destructuring is expected in the parameter.


PDOException: Cannot assign requested address

[2023-05-18 12-45-12] PDOException: SQLSTATE[HY000] [2002] Cannot assign requested address #2002 in .../src/libs/Database.php:32
[2023-05-19 00-32-05] PDOException: SQLSTATE[HY000] [2002] Cannot assign requested address #2002 in .../src/libs/Database.php:32
[2023-05-19 00-35-06] PDOException: SQLSTATE[HY000] [2002] Cannot assign requested address #2002 in .../src/libs/Database.php:32
[2023-05-19 02-49-05] PDOException: SQLSTATE[HY000] [2002] Cannot assign requested address #2002 in .../src/libs/Database.php:32

This did not happen for few months, closing.


[SQL] az sql dw update: fix update to not accept backup-storage-redundancy; az sql db update: commit storage_account_type updates

Description
A previous PR from earlier this year added backup-storage-redundancy to all SQL create/update methods. This inadvertently added the functionality to SQL DW as well. However, this parameter should not be available during dw update (not supported in the backend), so this PR removes the customer's ability to update a DW's backup storage redundancy.
Additionally, this fixes an issue for az sql db update; where the update command was not updating backup storage redundancy values on existing databases.
Testing Guide

History Notes

[SQL] az sql dw update: do not accept backup-storage-redundancy argument
[SQL] az sql db update: update backup storage redundancy as requested from command

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

@Juliehzl please help with this PR.


Please support none-IDE mode

alter layout, and fix styling issues. All without leaving VS Code.

That made me cry a little. Many users of VS Code like it to be an editor, not an IDE. I personally don't see a reason why VS Code should replicate the contents of a website in a tiny embedded widget and from my experience I am not the only user that thinks like this.
So, please consider a none-IDE integration along the full IDE mode. In my ideal world it would work like this:

Show browser-things in the browser, e.g the elements tab, the performance tools etc
Show source code in VS Code, e.g when I select styles.scss inside dev tools, then open that file in VS Code. If picking a color updates a file, do that with VS Code (there is API for all of that)

I hope all of that makes sense, feel free to get in touch with me, and in case a none-IDE integration already exists please ignore this. Anyways please don't forget those VS Code users that like pressing Alt/Cmd-Tab and that like OS window managers, they do exist.

@jrieken Thanks for the feedback!
This extension is only the Elements tool. Since there is already a Debugger/Console extension: Debugger for Edge we wanted to experiment with having the most commonly used tools (Elements, Console, Debugger) within VS Code and get customer feedback. So again, thanks for the feedback 😊.
The Elements tool is our current focus and currently do not have plans to move all the tools within VS Code. So to your point - we also feel some tools belong in the browser and some tools could live in Editors like VS code, but where it makes sense.
As a principle, we want to have as similar experiences as we can across the browser DevTools and any tool integrations we may build.
I am super interested in your second bullet point 😊. So a couple questions:

In your ideal experience, when you click on a link in the Styling Pane, would you prefer to have it open in VS Code vs. the Sources tab in the browser? By default? Right-click context menu option? A setting you enable in the Settings Panel?
If the source mapping is incorrect would you prefer to have it fall back on the Sources tab or error out?
Would you prefer to open JavaScript files and set breakpoints in VS Code vs. Sources tab? Basically, have the same "Open files in my Editor" experience for all files?

Again thanks for the feedback! We're super curious about how folks are using their Editors and the Edge Browser together to build out their web experiences.

In your ideal experience, when you click on a link in the Styling Pane, would you prefer to have it open in VS Code vs

Always VS Code! At least when I have started in VS Code, e.g when I launch Edge from VS Code I would expect to get back to it for editing source files. Actually, I want that for all source file links, e.g. links in the performance or memory tab. Having more options in the context menu is always good, esp. when I have started in Edge itself.

If the source mapping is incorrect would you prefer to have it fall back on the Sources tab or error out?

I am not a fan of source maps and try to avoid them at all costs, e.g I turn them off whenever possible. Tho, assuming I would use them and assuming the mapping is defunct, I would still expect the underlying (JS/css) file to open in VS Code, e.g. you can register a file system for the "real" (unmapped) files and show them in VS Code.

Would you prefer to open JavaScript files and set breakpoints in VS Code vs. Sources tab? Basically, have the same "Open files in my Editor" experience for all files?

That's good a question. Today, I use the source tab and set breakpoints in JS files (no source maps, see above) but that actually feels clunky. Ideally, I can do both, e.g set breakpoints in TS files in VS Code (and they make their way down to the JS files) and set breakpoints in JS files, also in VS Code. In short, debugging is (should be) part of my inner loop and for that I should be in VS Code

Love it! This is great feedback.

Always VS Code! At least when I have started in VS Code, e.g when I launch Edge from VS Code I would expect to get back to it for editing source files. Actually, I want that for all source file links, e.g. links in the performance or memory tab. Having more options in the context menu is always good, esp. when I have started in Edge itself.

I've added a feedback item via the Edge Send-a-Smile with a link to this issue to collect this feedback on in browser tools.

I am not a fan of source maps and try to avoid them at all costs, e.g I turn them off whenever possible. Tho, assuming I would use them and assuming the mapping is defunct, I would still expect the underlying (JS/css) file to open in VS Code, e.g. you can register a file system for the "real" (unmapped) files and show them in VS Code.

I'd like to follow up to understand your workflow here. So I'll reach out for that.
Thanks again!

Seems follow-up happened as this was internal


Automate updates of trusted tools

Run make update-trusted somewhere
Close any existing automated update PRs that haven't been merged
Generate PR (one per toolset, the install automation doesn't work on changes to multiple toolsets in one batch)
A usegalaxy.* admin merges the PR if the test build was successful

Or maybe we should only automate the common toolset and each usegalaxy.* admin can set up their own fork to automate their updates.

A specific set of tasks that I need to complete so we can at least do this for the usegalaxy.org toolset:

Currently I have an allow-list of tools that run in Singularity as I work through what tools work and don't work. This needs to be flipped to a block-list so that all new installs of tools run in Singularity by default. For this to work I need to extract all installed versions and explicitly map by version instead of a versionless tool, since doing so would exclude both old and new versions from Singularity, when we only want old versions.
Fix automatic toolbox reloading (currently requires a full restart of Galaxy to ensure tools are properly loaded on all workers).
Probably need to adapt mvdbeek/usegalaxy-tests to run on PRs in this repo - once a tool install is published and visible on usegalaxy.org, run tests against it. This will require extracting the installed tools IDs, since all we have here are repos and revisions (and even then we don't pull out which repos were updated in the PR, we just run the entire section YAML for any YAMLs that contained any modifications).
If the tests succeed, merge the PR. If the tests do not succeed, disable/hide this tool version (@mvdbeek was working on an API for this if memory serves?) and flag admins to fix.

Then, complete the stuff in the issue above.
I wonder if we would be better off, from an ease of use standpoint, instead of people editing the YAML/lock files directly, people can create an issue with a list of tool IDs to install/update, and those get transformed into repos and a PR is automatically generated updating the lock files?

The allow/block list flip is done in galaxyproject/usegalaxy-playbook@12ba644507fcb9d1090a002da6d16d2fd8823a76.


CDO Constructor Error 0.9.8

Hello! After updating to 0.9.8 in can't run CarlaUE4.sh without getting the following error:
Error: CDO Constructor (TrafficLightManager): Failed to find Blueprint'/Game/Carla/Blueprints/TrafficLight/BP_TLOpenDrive.BP_TLOpenDrive'
I'm Working on the latest Content with two own custom Vehicles.
If i run make launch-only everthing works properly.

The error seem not to effect the Carla server. Maybe it is just some kind of warning.


Spaces are allowed in email registration field

Came across a user who registered an email with a space in it. We should really block that since you can't have spaces in an email address.
Screenshot in https://github.com/coolpick/sidekick/issues/81

Closing as duplicate of https://github.com/SitekickRemastered/Game/issues/37


Autofill In Browser layout improvements for landscape / iPad

Task/Issue URL: https://app.asana.com/0/72649045549333/1202809065048067/f
Tech Design URL:
CC:
Description:
Landscape and iPad layout handling for all prompts including:

Save login
Save username / password
Save additional login
Update username / password
Choose from saved login (single)
Choose from saved login (multiple)

Steps to test this PR:

Run through the different view states confirming the layouts match designs for iPhone landscape and iPad portrait & landscape
Confirm iPhone portrait layouts are still correct

Note:

iOS 14 has not been throughly tested (will do a full iOS 14 review of all Autofill screens separately)
Custom detents not implemented for iOS 16 (will tackle in an iOS 16 improvements PR)

Copy Testing:

[ ] Use of correct apostrophes in new copy, ie ’ rather than '

Orientation Testing:

[ ] Portrait
[ ] Landscape

Device Testing:

[ ] iPhone SE (1st Gen)
[ ] iPhone 8
[ ] iPhone X
[ ] iPhone 14 Pro
[ ] iPad

OS Testing:

[ ] iOS 14
[ ] iOS 15
[ ] iOS 16

Theme Testing:

[ ] Light theme
[ ] Dark theme

Internal references:
Software Engineering Expectations
Technical Design Template
When ready for review, remember to post the PR in MM

Not sure what's going on, but the diff looks slightly weird
Testing looks good though, although I think the spacing below the buttons here in portrait is a bit off


ffpyplayer does not play nice with Clock

Versions

Python: 3.4
OS: Arch Linux, Mac OSX
Kivy: 1.11.1
Kivy installation method: pip

Description
I have a program which, after some time has passed, needs to play a video. Then after some more time has passed, another video is played. This is simple enough and worked quite well on my Windows machine using GStreamer.
Issues started arising when I tried running my program on a Linux or OSX machine and was told by the install guide to use ffpyplayer. From then on, whenever a video would be played, the program would freeze. However, if I sent a Ctrl+C to the program, it would tell me it was cancelling the Clock, and would then continue working properly again.
Somewhere ffpyplayer seems to allow the Clock to take the main thread and not let go, GStreamer, again, does not have this issue.

Can you please provide a minimal reproducible example?

I've done more research and discovered that my tracing of the problem to Clock wasn't exactly right. The issue here lies with trying to use video.unload() when using ffpyplayer. There is a warning about this in the source code of Kivy's ffpyplayer backend, but this is not mentioned anywhere in the public documentation.
Basically, load a video, then unload it when it ends, then go to a different screen, this is what cause the actual issue. I'll see if I can whip up a minimal reproducible example.

I hit the issue today:

Python 3.7.0
OS: Arch linux / miniconda
Kivy: 1.11.1
Kivy installation method: pip

I have a screen that when it leaves, i call self.video.unload() (self.video is the Video widget, not the core provider), then remove it from the tree.
On the next Clock tick, everything is stuck. When i control+c, this is the backtrace:
^C--Call--
> /home/tito/.conda/envs/pebblo3/lib/python3.7/pdb.py(188)sigint_handler()
-> def sigint_handler(self, signum, frame):
(Pdb) bt
  /home/tito/code/pebblo_mobile_app/main.py(33)<module>()
-> run(options)
  /home/tito/code/pebblo_mobile_app/pebblo/app.py(775)run()
-> app.run()
  /home/tito/code/kivy/kivy/app.py(950)run()
-> runTouchApp()
  /home/tito/code/kivy/kivy/base.py(560)runTouchApp()
-> EventLoop.mainloop()
  /home/tito/code/kivy/kivy/base.py(334)mainloop()
-> self.idle()
  /home/tito/code/kivy/kivy/base.py(375)idle()
-> Clock.tick()
  /home/tito/code/kivy/kivy/clock.py(646)tick()
-> self.post_idle(ts, self.idle())
  /home/tito/code/kivy/kivy/clock.py(689)post_idle()
-> self._process_events()
  /home/tito/code/kivy/kivy/core/video/video_ffpyplayer.py(117)__del__()
-> self.unload()
  /home/tito/code/kivy/kivy/core/video/video_ffpyplayer.py(365)unload()
-> self._trigger.cancel()
> /home/tito/.conda/envs/pebblo3/lib/python3.7/pdb.py(188)sigint_handler()
-> def sigint_handler(self, signum, frame):
(Pdb) up
> /home/tito/code/kivy/kivy/core/video/video_ffpyplayer.py(365)unload()
-> self._trigger.cancel()
(Pdb) self._trigger
<ClockEvent (0.0) callback=<bound method VideoFFPy._redraw of <kivy.core.video.video_ffpyplayer.VideoFFPy object at 0x7f0fd1dc6bc8>>>
(Pdb) self._trigger.cancel()
^C^C

^Z
[1]+  Stoppé                 PEBBLO_SCREEN=iphone5,portrait ./run.py montauban --offline```

What you can see is:
1. everything is stuck on trigger cancel
2. control+c doesn't even work when i try to cancel it manually again.

So strange.

So it's just a lock contention issue.
It is hard to reproduce, because it depend of the __del__ beeing called at anytime by the GC. And in the backtrace, it is beeing called during a tick.

Clock.tick already acquire the clock Lock.
__del__ is called, trying to cancel the trigger
cancel method also acquire the clock lock => stuck.

I guess scheduling the destruction is better than using __del__  (this is also sort-of the same reason why graphics object are scheduled to be released rather using python method for object deletion, because you don't control when it will be collected.)

I have trouble to make it crash with Python 3.7, but it consistently crash with Android / Python 3.8.

Hey,
I just stumbled upon this same exact problem and found this issue ..
I did not test it yet but I suppose PR #7418 should have also fixed this, shouldn't it?
It seems to have removed that cancel call altogether.
This issue could than be closed I guess..

Anyone want to confirm whether the problem is solved with Kivy 2.2.1?


Output type "book" should put a page break before a new part

I believe the pdf output should add a page break before a new part if the book style is used. It does already add a page break after the part and I do not see how it can be useful to have the header of the part on the page before.
Currently, I have to put the page break in manually. Example:
[abstract]
This script give an introduction into basic Android development tasks.

<<<
= Introduction to Android development

:imagesdir: ../com.vogella.tutorials.android/Android/img

include::../com.vogella.tutorials.android/Android/010_overview.adoc[]
include::../com.vogella.tutorials.android/Android/140_manifestmf.adoc[]

<<<
= Android Studio installation

include::../com.vogella.tutorials.android/Android/016_androidstudioinstallation.adoc[]
include::../com.vogella.tutorials.android/Android/024_tutorial_firstproject_androidstudio.adoc[]

This is a duplicate of #74.

Marking as closed.

The more specific issue is #329, which is resolved for the upcoming 1.5.0.alpha.13 release.


Compute 2019-07-01 version

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Automation for azure-sdk-for-python

Automation for azure-sdk-for-go

Can one of the admins verify this patch?

@zikalino Could you check why model validation fails in this PR?  Thanks

@zikalino  the validation error says "Model definition 'DiskEncryptionSet' must have the properties 'name', 'id' and 'type' in its hierarchy and these properties must be marked as readonly."  But 'DiskEncryptionSet' has only id property which is set by a client in request body.

These validation failures are expected.
Potential new ARM errors
========================
R2020 - RequiredPropertiesMissingInResourceModel
Model definition 'DiskEncryptionSetParameters' must have the properties 'name', 'id' and 'type' in its hierarchy and these properties must be marked as readonly.
at specification/compute/resource-manager/Microsoft.Compute/stable/2019-07-01/compute.json:6879
R3016 - DefinitionsPropertiesNamesCamelCase
Property named: 'diskIOPSReadWrite', for definition: 'DataDisk' must follow camelCase style. Example: 'diskIopsReadWrite'.
at specification/compute/resource-manager/Microsoft.Compute/stable/2019-07-01/compute.json:7113
R3016 - DefinitionsPropertiesNamesCamelCase
Property named: 'diskIOPSReadWrite', for definition: 'VirtualMachineScaleSetDataDisk' must follow camelCase style. Example: 'diskIopsReadWrite'.
at specification/compute/resource-manager/Microsoft.Compute/stable/2019-07-01/compute.json:8420

@zikalino Hi, we finalized our Swagger spec from our side.  Could you review this PR?  The failed validation errors are expected.

@hyonholee so we need to disable these validation errors

@hyonholee   For errors in LintDiff, if you'd like to suppress these validation errors, you may refer to specification/compute/resource-manager/readme.md of this repo and add them to the suppress part.

@KrisBash   I added default responses to the new APIs.


Arch user repository outdated.

The AUR download for stationhub is very outdated, last updated 2020-4-29.
https://aur.archlinux.org/packages/stationhub

## GNU_Linux ##
[Ctrl] + [Alt] + [T]
wget https://raw.githubusercontent.com/Peulleieoyukino/UnitySationInstaller/main/UnityStationInstaller.sh -O ~/UnityStationInstaller.sh; sudo chmod 750 ~/UnityStationInstaller.sh; sudo ~/UnityStationInstaller.sh

the shortcut will be in your Menu/Game

we don't maintain that repo, feel free to do it yourself if you want. I think the person who used to maintain it was a player and we haven't seen them in a while.


agent_interval and agent_flush_interval

There was an option for agent_interval, but the template file hard coded it at 10s anyway.  There was no corresponding option for flush interval.

fine for me, merged.


Running self.compute_metastable_states modifies schur decomposition

Say I compute my schur decomposition using self.comptue_schur(), which by default gives me a decomposition of size 10. If I now compute metastable states using self.comptute_metastable_states(n_states=3), then this also restricts my schur decomposition to be of size 3, which is bad if afterwards, I want to compute n_states=4 metastable states (because we need to recompute). Therefore, computing metastable states should not restrict the saved schur decomposition, but only enlarge it if needed (in case we have to recompute).

Implemented via #178

Thanks!


Revert "Generate concourse pipelines on website creation (#366)"

This reverts commit 52dd75fdce957cedea31d00acb5e914d373183aa.
What are the relevant tickets?
N/A
What's this PR do?
Reverts the pipeline commit because the fly executable is not being installed as expected.
How should this be manually tested?
Tests should pass

Codecov Report

Merging #389 (d47cfcf) into master (c35b442) will increase coverage by 4.72%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #389      +/-   ##
==========================================
+ Coverage   90.03%   94.76%   +4.72%
==========================================
  Files         135       69      -66
  Lines        4397     1737    -2660
  Branches      681      382     -299
==========================================
- Hits         3959     1646    -2313
+ Misses        395       89     -306
+ Partials       43        2      -41

Impacted Files
Coverage Δ

ocw_import/api.py

content_sync/models.py

...dev/management/commands/populate_local_starters.py

content_sync/factories.py

content_sync/serializers.py

content_sync/backends/github.py

main/utils.py

websites/config_schema/api.py

main/apps.py

websites/config_schema/validators.py

... and 47 more

Continue to review full report at Codecov.

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


Computed property support in interfaces

Currently, interfaces only support methods.
Would it be possible to declare an interface for computed properties?
Hashable: interface {
  hash: Int { get }
}

This is old 👴🏻😄


Default an empty constraint as any, not latest

When a package is specified without any restrictions, default the constraint to gps.Any(), not latest from the default branch.
Work in pending pull requests for the importers has made it clear that this change is universally needed and shouldn't be done one-off in each importer.

oh, yes, this definitely makes a lot of sense. i think i'd assumed we were doing it this way already 😛

I'm pretty sure that I misread a TODO and flipped the behavior a while back... 😝

basic impl looks good, seems like we just need test fixes

I've removed the offending the testdata, it really belonged in PR #898.


Need to update API specs and examples in JSON_RPC_API.md file

There are outdated or missing entries.

Resolved.


add vocabulary-theme header, footer, and specific overrides, while preserving original functionality

Description

fixes #72
applies the header and footer areas from current "vocabulary-lite"
adjusts inner-content to be a little more responsive
re-organizes files structure to better label legacy stuff
removes legacy pre-compile sass files and config file
encapsulates styling within css cascade layers

Technical details
This does not change the styling of the content, since Vocabulary still lacks much of the functionality that would be required to do so.
Screenshots
Small viewport

Default Viewport Top

Default Viewport Bottom

Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the default branch of the repository (main or master).
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no
visible errors.

Developer Certificate of Origin

For the purposes of this DCO, "license" is equivalent to "license or public domain dedication," and "open source license" is equivalent to "open content license or public domain dedication."

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

deployed to production Tue 17 Oct 2023 04:27:39 PM UTC (may take 30 minutes for caches to expire)


Using selectRow causes the page to crash on Internet Explorer 11

Description
If use selectRow in BootstrapTable 4, the table does not render on Internet Explorer.
Version
Internet Explorer 11 - 11.0.135
react-bootstrap-table-next 3.2.0
To Reproduce
Add SelectRow property and look in IE11. The code something like below
<BootstrapTable         keyField="id"         data={products}         columns={columns}         selectRow={selectRow}       />
Reference Code
The Entire code on sandbox. Unfortunately, sandbox will not run on IE to reproduce
Sanbox Link

This issue is addressed by adding polyfill for find method in Array. The issue got Fixed. Thanks.


Django Web Server is not opening

I want to use your pharmacy web-application but when I run this command
python3 manage.py runserver the blank command is showing instead of using this command I used this one python manage.py runserver I got these errors.

Now I got This error

Change the database configuration in settings.py. For easier access, use dbsqlite3, it is been commented out in settings.py.
Comment current database configuration.

You have to migrate by
python manage.py runserver
Man you need to have some basic knowledge of django and python before running the project.


[ISSUE #3940]Optimize AllocateMessageQueueStrategy

Make sure set the target branch to develop
What is the purpose of the change
#3940
Brief changelog
XX
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage increased (+0.07%) to 51.704% when pulling a7fe863e6958e93aef04e69eba6f6cc0e008059f on zhangjidi2016:optimize_code into 7a5d937c76c2b99e65a18388f9c3acb3d80afd16 on apache:develop.

Codecov Report

Merging #3941 (a7fe863) into develop (7a5d937) will increase coverage by 0.03%.
The diff coverage is 76.00%.

@@              Coverage Diff              @@
##             develop    #3941      +/-   ##
=============================================
+ Coverage      47.64%   47.67%   +0.03%
- Complexity      4957     4961       +4
=============================================
  Files            633      634       +1
  Lines          42577    42556      -21
  Branches        5589     5584       -5
=============================================
+ Hits           20285    20288       +3
+ Misses         19786    19762      -24
  Partials        2506     2506

Impacted Files
Coverage Δ

.../consumer/rebalance/AllocateMachineRoomNearby.java
70.73% <0.00%> (+9.50%)
:arrow_up:

...r/rebalance/AllocateMessageQueueByMachineRoom.java
73.07% <33.33%> (+73.07%)
:arrow_up:

...sumer/rebalance/AllocateMessageQueueAveragely.java
86.66% <50.00%> (+25.79%)
:arrow_up:

...nsumer/rebalance/AllocateMessageQueueByConfig.java
55.55% <50.00%> (+55.55%)
:arrow_up:

...ebalance/AbstractAllocateMessageQueueStrategy.java
100.00% <100.00%> (ø)

...balance/AllocateMessageQueueAveragelyByCircle.java
90.00% <100.00%> (+28.88%)
:arrow_up:

.../rebalance/AllocateMessageQueueConsistentHash.java
81.25% <100.00%> (+3.75%)
:arrow_up:

...va/org/apache/rocketmq/store/FlushDiskWatcher.java
81.25% <0.00%> (-9.38%)
:arrow_down:

...org/apache/rocketmq/store/ha/WaitNotifyObject.java
66.07% <0.00%> (-5.36%)
:arrow_down:

...apache/rocketmq/remoting/netty/ResponseFuture.java
85.00% <0.00%> (-5.00%)
:arrow_down:

... and 28 more

Continue to review full report at Codecov.

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


Failed: Update autorest version to prerelease 3.0.0-alpha.20240517.10

Triggered from pull request: https://github.com/Azure/autorest.csharp/pull/4707

API change check
API changes are not detected in this pull request.


perf: increase packer vm sizes

What type of PR is this?
/kind performance
What this PR does / why we need it:
Merging this PR will decrease the amount of time it takes ADO pipelines / packer to create our custom VHDs. Additionally, increasing the VM size will serve as the bedrock for future optimizations that make use of parallel processing and increased network bandwith.
Which issue(s) this PR fixes:
Unsatisfactory custom VHD build times.
Requirements:

[ ] uses conventional commit messages
[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2348

Relevant Lines:
2926

💛  - Coveralls

For your presentation/documentation, I would recommend keeping track of the build time improvements with each change merged (depends on the order). It'll be easier to show the contribution of each change.


LabelledData.map drops None values returned by map function

Discussion about why this fix is needed in https://github.com/ioam/holoviews/issues/1681

Tests have passed and I am happy to merge. That said, if map is discussed anywhere in the user guide atm this particular behavior should be mentioned...

I think we meant to mention it in the nesting data user guide but haven't done so far.

Please open a docs issue as a reminder to do that.


feat: Add workflow to generate DBT documentation and deploy to GitHub Pages

This PR adds automatic dbt docs generation for the dbt-synthea project.
Why bother?
The docs are a nice place to start for those not used to either dbt / OMOP to explore the models/DAG/etc (IMHO - feel free to disagree!); is easier for us to point people to our project!
This approach requires little/no effort of maintainers once setup.
How does it work

runs when a relevant file in github has been modified in the main branch (as per paths filter)
installs python + requirements (caches as possible)
uses duckdb rendering of the project (lighterweight/friendly flavour of sql compared to others)
uploads to GitHub pages (this needs to be enabled by an admin)

Next steps

Enable pages (if approved/agreed as useful)
Add other docs to make prettier/complete - e.g. an overview page, etc.

I do agree with adding this!  Thanks for the initiative!  I think I enabled Pages:

If this looks right, go ahead and merge to test it out!

It works!!! https://ohdsi.github.io/dbt-synthea

Excellent!! 🥳

For some reason, the data types are not showing up in the GH Pages site, though they do show up when I build it locally. Quick Google says this might be a fix? dbt-labs/dbt-spark#988

Interesting... I have can reproduce the same thing. The documentation on the docs is ironically thin on how you should do this.
I can confirm that if you use type then the docs works. Buttt - the reason we don't have this issue locally is that if you have dbt run / dbt build your project and have your warehouse available then dbt will grab the data type metadata from the database. I.e. if we update the workflow to do dbt build then it will work fine.

Going down the rabbit hole, I am torn as to what to do:

The dbt yaml schema declares that data_type is correct (type is not valid)
data_type exists so that if you enable/enforce contracts then dbt will check the we have matched our expected schema on project build.

I think it would be nice to be able to enforce checking of this if we want in the future - help to prove our code is CDM compliant etc.
I tried enabling this for care_site and failed as the ID uses bigint:

Interestingly, this should be handled by the adapter:

https://github.com/duckdb/dbt-duckdb/blob/41794e3edc2dfb4237bee0167e4df07f581e5ba6/dbt/adapters/duckdb/column.py#L18-L24
But does not seem to work!
What do you think @katy-sadowski ? I think doing the dbt run step before building the docs would be a way around this, and allow us to enforce schema checking in the future - but just using type is another valid solution

I've gone too deep into the weeds, but essentially if we want contract enforcement to work then the dbt-duckdb adapter needs to be updated to overload the DuckDBColumn class here with an updated TYPE_LABELS dict which would include mappings for 'BIGINT': 'INTEGER'.
I do think it would be cool to have contractually bound type checking using the CDM docs as the source but this would require upstream dbt-duckdb changes, and I think BIGINT is thought to be a distinctly different type to INT (although I am not sure why, the contract feature allows type aliasing to be enabled or disabled at will.)

@katy-sadowski I will pull this out into a new issue and we can discuss there! 🧠


Static code analysis _018 (Negative Character Value)

Short summary (details in attached file):
Negative Character Value help at ini.c:27
isspace() is invoked here with an argument of signed type char, but only has defined behavior for int arguments that are either representable as unsigned char or equal to the value of macro EOF(-1).

gptp_018.txt

Fix was merged.


Example KCL to pass resource data between created cloud resources

I'm writing KCL code to create Azure resources using the KCL crossplane function as part of a composition.  I need to supply some of one resource's "atProvider" data to another resource's "forProvider" data.
In the past this would be achieved by using a patch with the patch and transform function but I would prefer to keep this encapsulated within the KCL code if possible.
Should I use the option("params").ocds some how to access the named resource and obtain its atProvider data?  If so, can you provide an example of accessing a a resource this way.  Please also let me know if this is not the preferred way of doing this.

Do you mean this?
apiVersion: apiextensions.crossplane.io/v1
kind: Composition
metadata:
  name: function-template-go
spec:
  compositeTypeRef:
    apiVersion: example.crossplane.io/v1
    kind: XR
  mode: Pipeline
  pipeline:
  - step: normal1
    functionRef:
      name: kcl-function
    input:
      apiVersion: krm.kcl.dev/v1alpha1
      kind: KCLRun
      metadata:
        name: basic
      spec:
        target: Resources
        source: |
          {
              metadata.name = "resource1"
              spec.forProvider.network: "somenetwork"
          }
  - step: normal2
    functionRef:
      name: kcl-function
    input:
      apiVersion: krm.kcl.dev/v1alpha1
      kind: KCLRun
      metadata:
        name: basic
      spec:
        target: Resources
        source: |
          {
              metadata.name = "resource2"
              spec.atProvider = option("params").dcds["resource1"].Resource.spec.forProvider
          }

Thanks @Peefy .  Other way around, i.e. atProvider holds the data created by the remote provider (Azure in my case), and forProvider has data properties I need to set.
Is dcds the correct collection to use for resources that have are "ready" as in synced with crossplane and also created at the provider?

I think dcds it is a suitable field for any synchronized resources. Or the function-auto-ready function is your need in the document https://github.com/crossplane-contrib/function-go-templating

Thanks @Peefy.  I think I'll be fine by just copying the data with dcds rather than waiting for ready. reconcile loops should eventually resolve the data.

Make sense. Did you mean this fields? I think there is an opportunity to improve it

I'm assuming the dcds dictionary does the same too I would imagine.

I've opened a PR to remove the component resource prefix from the name field of KCLRun. https://github.com/crossplane-contrib/function-kcl/pull/57


German Youtube with new processors

Pipeline for YouTube subset of German language which includes following steps:

CreateInitialManifest - Create initial manifests based on pairs of .opus audio + .srt related transcript (with ground-truth timestamps)
AggregateSegments - Aggregate ground-truth segments to longer one based on duration threshold
Identification of the text language and audio language
ASRInferenceParallel - added processor for the parallel ASR Inference (and following MergeManifests)
preprocessing to audio-based TN
running of audio-based TN
postprocessing of audio-based TN
filtration with metrics thresholds
data saving

Each processor should have docstring. Please see other processors from the Main branch


After scoop install failing it needs to uninstall the package

Hi!
I tried the example from the doc:
scoop install apache
but got an error (404). Then I tried to install it again, but got another error: ... try scoop uninstall apache. It will be better no need to uninstall the package after failed installing.
Also, scoop can't install apache :(
The screenshot:
https://yadi.sk/i/PGut8znRmp2Vw

@fremail We've all had this happen before, bad ref, checksum, etc. To keep I/O to a minimum Scoop uses a cache folder. From the Scoop wiki:

scoop help cache -> overview of the cache syntax

scoop cache show ->cache disk usage

scoop cache rm <app> -> delete a single app

scoop cache rm * -> clear the cache (Arguably you wouldn't use a glob by itself - safer to just use Explorer and delete just in case the ps1 script ever changes. I don't think the glob/wildcard is documented in Scoop, @lukesampson feel correct me if it is officially supported or if I've just been lucky all this time :stuck_out_tongue_winking_eye:

@saildata So I removed the apache from scoop's cache and tried to install the apache again, but got an 404 error.

  1.6 MB 7zip (15.10)                                                               http_www.7-zip.org_a_7z1510-x64.msi
     0 B apache (2.4.17)              ...w.apachelounge.com_download_VC11_binaries_httpd-2.4.17-win32-VC11.zip.download
  3.9 MB curl (7.45.0)                                           http_www.confusedbycode.com_curl_curl-7.45.0-win64.zip
441.2 KB grep (2.5.4)                       http_ufpr.dl.sourceforge.net_project_gnuwin32_grep_2.5.4_grep-2.5.4-bin.zip
877.2 KB grep (2.5.4)                       http_ufpr.dl.sourceforge.net_project_gnuwin32_grep_2.5.4_grep-2.5.4-dep.zip
610.1 KB openssh (5.4p1-1)            ...w_MSYS_Base_msys-core_msys-1.0.13-2_msysCORE-1.0.13-2-msys-1.0.13-bin.tar.lzma
 11.4 KB openssh (5.4p1-1)            ...xtension_minires_minires-1.02_1-2_libminires-1.02_1-2-msys-1.0.13-dll.tar.lzma
274.7 KB openssh (5.4p1-1)            ...SYS_Extension_openssh_openssh-5.4p1-1_openssh-5.4p1-1-msys-1.0.13-bin.tar.lzma
  4.8 KB openssh (5.4p1-1)            ...SYS_Extension_openssh_openssh-5.4p1-1_openssh-5.4p1-1-msys-1.0.13-lic.tar.lzma
654.9 KB openssh (5.4p1-1)            ...ension_openssl_openssl-1.0.0-1_libopenssl-1.0.0-1-msys-1.0.13-dll-100.tar.lzma
 36.7 KB openssh (5.4p1-1)            ...t_mingw_MSYS_Extension_zlib_zlib-1.2.3-2_zlib-1.2.3-2-msys-1.0.13-dll.tar.lzma
 22.7 MB php (7.0.1)                               http_windows.php.net_downloads_releases_php-7.0.1-Win32-VC14-x64.zip
 16.4 MB php53 (5.3.29)                   http_windows.php.net_downloads_releases_archives_php-5.3.29-Win32-VC9-x86.zip
 18.9 MB php55 (5.5.30)                           http_windows.php.net_downloads_releases_php-5.5.30-Win32-VC11-x86.zip
 21.7 MB php56 (5.6.16)                           http_windows.php.net_downloads_releases_php-5.6.16-Win32-VC11-x64.zip

total: 15 files, 88.0 MB
PS C:\Users\antond> scoop cache rm apache
PS C:\Users\antond> scoop install apache
installing apache (2.4.17)
downloading https://www.apachelounge.com/download/VC11/binaries/httpd-2.4.17-win32-VC11.zip...The remote server returned an error: (404) Not Found.```

Did I do something wrong?

Oh! I needed to update the scoop before. So, the apache was installed.
@saildata, thank you.

@fremail - glad it worked out. Out of (sheer) laziness I have scoop aliased to s so when I boot up I s update ;su would have felt wrong.. This project is just about the only thing that makes working in Windows bearable for a Linux user :trophy: @lukesampson

@lukesampson scoop is a cool app, but I think it should store data about packages on github for example. Then it would be faster to update it (don't need to update scoop on each computer).
Though it would be more better to parse download links automatically, but it can cause other bugs.


Exclude specific messages per test

Hey there
I've stumbled upon the limitation that you cannot exclude specific messages on a "per test" basis.
Sometimes you want the application to log an error and you want to ignore that error.
After trying out different things with the plugin I came up with this solution:
support/index.ts
if (Cypress.env('FAIL_ON_CONSOLE_ERROR')) {

  const excludeMessagesDefault = [
    /Invalid Host\/Origin header/,
    /\[WDS] Disconnected!/
  ];

  Cypress.on('window:before:load', (win) => {
    cy.spy(win.console, 'error').as('consoleError');
  });

  afterEach(() => {
    const exclusions: RegExp[] = Cypress.env('excludeMessages')
      ? [...excludeMessagesDefault, ...Cypress.env('excludeMessages')]
      : excludeMessagesDefault;
    cy.log(`analyzing console errors with exclusions:\r\n${exclusions.join('\r\n')}`);
    cy.get('@consoleError', { timeout: 4000 }).should((sinonSpy) => {
      const spy = sinonSpy as unknown as Agent<sinon.SinonSpy>;
      const consoleErrors: string[] = [];
      spy.getCalls().forEach((spyCall) => {
        const consoleErrorText = spyCall.lastArg.toString();
        const ignoreError = exclusions.some((excludeMessage) => excludeMessage.test(consoleErrorText));
        if (!ignoreError) {
          consoleErrors.push(consoleErrorText);
        }
      });
      expect(consoleErrors.length).to.eq(0, `verify that no console errors occurred\r\n${consoleErrors.join('\r\n')}`);
    });
  });
}

test example
it('my test', {
  env: {
   excludeMessages: [/my message to exclude/]
  }
}, () => {
 // ...
});

I didn't address the configuration of log levels, but this seems easy to configure as well.
What issues do you see in this solution compared to yours?
What I know of:

The test doesn't end on the first console error; it collects them all (which MAY be better, so you can collect all errors in one go. But it could also lead to follow up errors being reported)
Once a test failed due to the assertion in afterEach, all remaining tests in the spec will be skipped (works as designed by Cypress). Not ideal, but not much I can do about it. Don't think the current behaviour is going to be changed.
Every test needs to start with cy.visit() or the window:before:load hook will be meaningless; This means you cannot create a test which starts with a single visit and then run multiple tests on the same page. I know it shouldn't be done this way, but imagine having a page that loads very slow and you just take the risk of having side effects in later tests (from previous tests). One workaround would be to just include all actions/assertions in the same test but then you loose the structure of the tests (if you have very long tests you might want to split them up)

Hi @mirobo ,
looks like a solid solution with its own pros and cons. I am glad that I did inspire you with my plugin :-)


Register updated application version before attempting to update stream

@sabbyanandan i'm working through the tutorial and coming across a few issues like this... what woudl be the best way for me to open PRs for them? One big PR with all the small changes - or a PR for each line item?

@madorb Thank you for the PR.
If you already have the changes then please create a single PR that captures all of them.

LGTM, merging.

@madorb I suggest you create PRs based out of separate github branches in your fork instead of creating them from master branch.

Merged as 47537ea


Modify schema validation for additional property domain

The latest helm chart for istio-csr v0.9.0 results in the following schema error:
Helm install failed for release cert-manager/csr with chart<EMAIL_ADDRESS>values don't meet the specifications of the schema(s) in the following chart(s):
      cert-manager-istio-csr:
      - (root): Additional property domain is not allowed

Please allow for domain property or additional properties in the schema validation.
values.yml example use-case:
values:
      domain: {{ domainName }}
      app:
        certmanager:
          namespace: istio-system
          issuer:
            name: vault-issuer
            kind: Issuer
            group: cert-manager.io
        tls:
          trustDomain: "cluster.local"
          istiodCertificateEnable: true
          rootCAFile: /var/run/secrets/istio-csr/ca.cert.pem
          certificateDNSNames:
            - cert-manager-istio-csr.cert-manager.svc
          certificateDuration: 24h
          istiodCertificateDuration: 24h
          istiodCertificateRenewBefore: 30m
          istiodCertificateEnable: true
          istiodPrivateKeySize: 2048

This error does not occur in v0.8.1 as there was no schema validation check in that release or at least values.schema.json did not exist in the helm chart.

Same problem here but for nameOverride, which is actually in use: https://github.com/cert-manager/istio-csr/blob/main/deploy/charts/istio-csr/templates/_helpers.tpl#L6

@volvera10 domain does not seem to be a value that is used in any of the Helm templates? Could you link to where that value is used in the Helm chart?
@jgascon-nx I created a PR to add support for nameOverride: https://github.com/cert-manager/istio-csr/pull/349

@inteon the issue seems to be due to the additionalProperties  being set to false in the values.json.schema
https://github.com/cert-manager/istio-csr/blob/main/deploy/charts/istio-csr/values.schema.json#L4C8-L4C28
This results in not allowing additional properties being set in the values.
https://json-schema.org/understanding-json-schema/reference/object#additionalproperties


Swin model gs demo Benchmarking [To Create PR]

GS e2e Demo
test uses batched real inputs and produces batched outputs
this test is looped (takes several batches of inputs, produces several batches of outputs)
test evaluates results of TT model against validation data in selected data set
test is documented
how to run test (commands to execute)
what does the model do (high-level explanation in a couple of sentences)
description of what is input and what is output
this demo should load inputs and expected outputs from a data set on Weka (TBD which one)

The Benchmarking of GS Swin Classification model is completed.
Data-set: IMAGENET
Metric: Accuracy
commit link

This PR contains benchmarking of Swin gs_demo for IMAGENET test dataset.
model is tested with 20 datapoints extracted from the dataset and the accuracy is listed below:
Metric: Accuracy
Accuracy: 0.0
Note: Swin model getting low PCC, thereby affecting its accuracy during benchmarking.
PR - #4774

PR merged, closing the ticket.