Safari  devtools + Exponent-in-iOS-simulator hangs on auto-reload.

If I open the iOS simulator via the Exponent Mac App and then use the Safari devtools to access the Simulator -> JsContext, both Safari and the Exponent app (inside the simulator) hangs on code reload, forcing me to reload the app in the simulator as well as start Safari again (and go into the dev tools). Not the biggest deal, but still annoying.
Safari Version 10.0 (116<IP_ADDRESS>.10)
Mac os 10.11.6 (15G1004
Exponent Mac App Version 2.8.0 (2.8.0
Simulator Version 10.0 (SimulatorApp-700.5.3

Do you know if this happens in raw React Native?

I don't know I'm afraid, and I don't have a raw React Native project around to test :(

So I tried this with a project created from react-native init and in an Expo project and they behave the same -- this was maybe a bug with an old version of react-native.
That said, there appears to be no workaround for the debugger closing each time the app reloads. The JSContext is torn down and re-created each time, with an Expo app or a plain react-native app. You could probably write some AppleScript to handle this for you though!


Build timeout

https://grpc-testing.appspot.com/job/gRPC_master/15876/config=dbg,language=c,platform=macos/console
START: bins/dbg/h2_Build timed out (after 120 minutes). Marking the build as failed.
Build was aborted
[htmlpublisher] Archiving HTML reports...
[htmlpublisher] Archiving at BUILD level /jenkins/workspace/gRPC_master/config/dbg/language/c/platform/macos/reports to /home/jenkins/jenkins_home/jobs/gRPC_master/configurations/axis-config/dbg/axis-language/c/axis-platform/macos/builds/15876/htmlreports/HTML_Report
ERROR: Specified HTML directory '/jenkins/workspace/gRPC_master/config/dbg/language/c/platform/macos/reports' does not exist.
[copy-to-slave] Copying 'report.xml', excluding nothing, from 'file:/jenkins/workspace/gRPC_master/config/dbg/language/c/platform/macos/' on 'hudson.slaves.DumbSlave[grpc-mac-node1]' to 'file:/home/jenkins/jenkins_home/jobs/gRPC_master/workspace/config/dbg/language/c/platform/macos/' on the master.
Recording test results
ERROR: Step ‘Publish JUnit test result report’ failed: No test report files were found. Configuration error?
Finished: FAILURE

I believe this is a transient Jenkins failure and should be closed.


intervention.io alternative but for video

Hi, i'm integrating intervention into my project, i would like if you can recommend to me an alternative similar to intervention but for videos. Thanks!

Sorry, I don't know such a package. Maybe this is an opportunity to create it?


Rss fix

2019: Mni6RssOid - requires 16 CPUs to test 16 queues
2022:
NICStrictPropertyValidation - no logs (would be good to rerun to see the failure reason)
Hardware-enforced Stack Protection Compatibility Test - no logs, but this test is manual, please check why it runs

2019: Mni6RssOid - requires 16 CPUs to test 16 queues
2022:
NICStrictPropertyValidation - no logs (would be good to rerun to see the failure reason)
The test should be separately installed
Hardware-enforced Stack Protection Compatibility Test - no logs, but this test is manual, please check why it runs
@kostyanf14 please check what happen.

NICStrictPropertyValidation - no logs (would be good to rerun to see the failure reason)

We have an issue in PlanIO about this test. Some tool is missing.


Try and accelerate the spammer triage process

Reformatted a bit to make it more readable + added requirement to
explain why people need an account, so that it's easier and quicker for
infra team to decide it's likely a real user or an effin' spammer.

As requested @larrys :-) here's already a proposal

This conflicts with https://github.com/jenkins-infra/account-app/pull/31 (which has some other changes in it)

No problem, I'll rebase onto it when merged. Btw, that makes another argument in favor of splitting the test in many lines: that it would make history, conflict resolution and diff doable per (sub)sentence.

My main change was to remove the link to the google groups, which you could easily incorporate. Or maybe we could link to a wiki page instead, so it does not require code changes to change our message when this gets triggered.

OK, got it for your first sentence, just force pushed with #31 text change.
For the wiki page, you mean linking to some page like 'Requesting an account under special conditions' or something like that?
Replacing the whole text w/ just :
Due to the spam problem, we need additional verification for your sign-up request.
Please refer to the associated documentation to get your account created.

WDYT?

Including @rtyler @kohsuke @daniel-beck @abayer  to provide feedback.
I think referring them to the wiki where we have the information is a good start, so we don't require code changes to change the verbiage.
However, it might be helpful to instead have that page (that rejects account creation) with a form for them to fill out asking them for further information to help us determine if they will spam or not. Then the content of that form (plus other information we collected along the way) is included in the email that normally is emailed to a mailing list (not the dev list, the current list should be fine, provided we stop getting mails hit by a spam filter)
Then the user does not have to send an email at all. We just need volunteers to monitor this mailing list, and act on real humans. We could still link them to a wiki page so if they wanted to follow up on their request, then they can ping the dev list. (I've tried to avoid having it create something like a JIRA ticket, since that would complicate things even more).

:+1: and to make it easier for admins, there could even be a unique generated link in the email (is there already? I didn't check the code) to directly validate a request that clearly seems legit (?)

It provides a link to create the account, but that is a GET request with all the values pre-filled out.
The current problem is that the subject is the same, so Gmail buckets them all together, so I think the subject should be a bit more unique (my pull request referenced above addresses this by including their first and last name in the email subject)

Rebased onto just merged #31 just in case.


Remove sudo and package dependencies from travis file (so we can have…

… travis containers)
Our unit tests here shouldn't require sudo or apt updates, so remove them in order to support new travis containerized infrastructure.
Putting up a PR to test this, but I don't think it will work with on-core yet.

Yep, the gc-stats dependency prevents us from testing without g++ :(


Saving predicted outputs in MXNetR

Hello, I have a question regarding the usage of MXNetR. After each training epoch, I want to save the predicted values using training inputs and testing inputs into a file so that I can plot them later on. I tried to customize my callback function but found no way to get access to the predicted outputs generated during the training process. I am quite new to MXNetR so it would be great if you can give me some general ideas of how I should do this.
P/S: I am aware that MXNetR allows the user to save the model after each training epoch. I can save all models generated during training and then predict the outputs with these models. However, if the number of models is large, this method would consume a lot of space.

Hello, I have a question regarding the usage of MXNetR. After each training epoch, I want to save the predicted values using training inputs and testing inputs into a file so that I can plot them later on. I tried to customize my callback function but found no way to get access to the predicted outputs generated during the training process. I am quite new to MXNetR so it would be great if you can give me some general ideas of how I should do this.
P/S: I am aware that MXNetR allows the user to save the model after each training epoch. I can save all models generated during training and then predict the outputs with these models. However, if the number of models is large, this method would consume a lot of space.

I think this can be done in a callback function.
In function(iteration, nbatch, env, verbose), the env is an environment, where you can store your results.

You can modify the metric function to save the predicted outputs. For example in the accuracy function:
mx.metric.accuracy <- mx.metric.custom("accuracy", function(label, pred) {   ypred = max.col(t(as.array(pred)), tie="first")   return(sum((as.array(label) + 1) == ypred) / length(label)) })
the label and predicted data are passed and you can implement the saving feature here.

@kimtang : Thank you, it does help.


Cross-chain transfer interface

Enabling asset transfer between inkchain and another public blockchain.
Ethereum and Qtum would be the start.

Please set more sub-tasks, including both public chain side (contract) and consortium chain side (chain and contract). Detailed documentations are needed as well.

public chain side need a smart contract provides these features:

lock the number of INK coins transferred from user
transfer the number of INK coins from contract to user
validate the signature strings from consortium chain side

consortium chain side need a chaincode provides these features:

issue the number of INK coins from INK issue account
request public chain node to validate the turn out transaction has confirmed
endoser sign the transaction‘s args


Does it support Elastic Search Open Distro

Does ES Open Distro (https://www.elastic.co/downloads/past-releases/elasticsearch-oss-7-10-2) supports this exporter ??

Yes, this exporter supports Elasticsearch 7.10.2.

Please note that I did mention ES OSS 7.10.2 , not ES 7.10.2

Whether it the OSS version or otherwise has no bearing on the operation of exporter, so long as the cluster is functional and the exporter can access it.
If you have any concerns, I suggest you try it.


Get-Process -Id <root_pid> fails when not running sudo

Steps to reproduce
Find a process running as root and try to use Get-Process on it when PowerShell was not started w/ sudo, e.g.
Get-Process -Id ((ps -ef | grep ^root | select -first 1 | % { $_ -split ' +' })[1])

Expected behavior
The process object is returned
Actual behavior
PS /etc> Get-Process -Id ((ps -ef | grep ^root | select -first 1 | % { $_ -split ' +' })[1])
Get-Process : Cannot find a process with the process identifier 1.
At line:1 char:1
+ Get-Process -Id ((ps -ef | grep ^root | select -first 1 | % { $_ -spl ...
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (1:Int32) [Get-Process], ProcessCommandException
    + FullyQualifiedErrorId : NoProcessFoundForGivenId,Microsoft.PowerShell.Commands.GetProcessCommand

Environment data

PS /home/jasonsh/src/PowerShell/tools> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      6.0.0-alpha
PSEdition                      Core
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   <IP_ADDRESS>
GitCommitId                    v6.0.0-alpha.9
CLRVersion
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

This seems to be a corefx issue. Filed dotnet/corefx#13135.

Awesome investigation, thanks @nzubair.

This appears to be fixed.


TypeError: F is not a function when using @emotion/styled

Describe the bug
My team has built a library  (called ui-react) of React components styled with Emotion. Both react and emotion are peer dependencies.
Since introducing these components to our main project, most of our tests are now failing (coverage down from 90 to 20%), with just this error message:
TypeError: F is not a function
 ❯ node_modules/@mycompany/ui-react/dist/@mycompany/ui-react.js:18201:12

The issue is the same whether we are using the styled('div')({}) or the styled.div({}) syntax. The error message changes slightly:
TypeError: F.div is not a function
 ❯ node_modules/@mycompany/ui-react/dist/@mycompany/ui-react.js:18201:14

If I look at the ui-react bundle:
import b1 from "react";
import F from "@emotion/styled";

We are indeed importing the default "styled" function from emotion. Its type is CreateStyle, which extends BaseCreateStyle, which is a function:
/**
 * @desc
 * This function accepts a React component or tag ('div', 'a' etc).
 *
 * @example styled(MyComponent)({ width: 100 })
 * @example styled(MyComponent)(myComponentProps => ({ width: myComponentProps.width })
 * @example styled('div')({ width: 100 })
 * @example styled('div')<Props>(props => ({ width: props.width })
 */
export interface CreateStyled {
    ...
    <several function signatures are defined here>
    ...
}

So I'm not sure what to do here, the rest of the project if working fine, only vitest is complaining about the typing.
Could this be solved with the right vitest configuration or is it a bug?
Reproduction
See instructions in the README file
https://github.com/fthebaud/repro-vitest-styled-issue
System Info
System:
    OS: Linux 4.19 Debian GNU/Linux 10 (buster) 10 (buster)
    CPU: (12) x64 Intel(R) Core(TM) i7-9850H CPU @ 2.60GHz
    Memory: 2.98 GB / 15.30 GB
    Container: Yes
    Shell: 5.7.1 - /usr/bin/zsh
  Binaries:
    Node: 18.14.1 - ~/.nvm/versions/node/v18.14.1/bin/node
    Yarn: 1.22.5 - /usr/bin/yarn
    npm: 9.3.1 - ~/.nvm/versions/node/v18.14.1/bin/npm
  Browsers:
    Chrome: 113.0.5672.92
    Firefox: 102.10.0esr
  npmPackages:
    @vitejs/plugin-react: 4.0.0 => 4.0.0
    @vitest/coverage-istanbul: 0.30.1 => 0.30.1
    vite: 4.3.1 => 4.3.1
    vitest: 0.30.1 => 0.30.1

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Here's a repo that reproduce the issue. Instructions are in the README.
https://github.com/fthebaud/repro-vitest-styled-issue

Some libraries bundle their code for bundlers to consume so they don't follow Node.js ESM semantics. In this case emotion has a default export that is not actually a function when imported inside Node.js, but an object with another "default" property. This was fixed in<EMAIL_ADDRESS>When updating the package I get a different error:
TypeError: Cannot read properties of null (reading 'useContext')

Why this happens I do not know, it's probably imported somewhere from ESM and somewhere from CJS, so it creates two different react instances.

I've upgraded emotion in my original project and it fixed the issue, I didn't have to add anything to deps.inline.
Thanks for your help @sheremet-va, much appreciated.


explanation of parameters

Could you please a short explanation of parameters of modules. For instance what does it mean n_filter, s_filter, offset and to, height, depth, width in both convolution and pool operator

It's very useful to me if any document is proposed


remove files that emit unsupported custom metrics

We don't support Stackdriver-style custom metrics in Cloud Monitoring.  These "expanded" files rely on that feature, so we should remove these files.

:shipit:

@kele heads-up.


Support VerificationStatement in the naming plugin

Previously, verification statements (assert, assume, cover, and printf)
were only named via reflection.
Contributor Checklist

[NA] Did you add Scaladoc to every public function/method?
[x] Did you add at least one test demonstrating the PR?
[x] Did you delete any extraneous printlns/debugging code?
[x] Did you specify the type of improvement?
[ ] Did you add appropriate documentation in docs/src?
[x] Did you state the API impact?
[x] Did you specify the code generation impact?
[x] Did you request a desired merge strategy?
[x] Did you add text to be included in the Release Notes for this change?

Type of Improvement

bug fix

API Impact
Verification statements are now named in the same way as normal hardware components.
Backend Code Generation Impact
In rare cases, some signals may get a new prefix based on the name of a verification statement, but this should only occur in cases where the statement is returned from a function/block and assigned to a val, eg.
val foo = {
  val wire = Wire(UInt(8.W))
  wire := ...
  assert(wire =/= 123.U)
}

In this case, wire will now be named foo_wire.
Desired Merge Strategy

Squash

Release Notes
Handle naming and prefixing of verification statements in the same way as Data and Mems. This primarily affects the names used by annotations but also will now cause prefixing when a verification statement is returned from a function or block and assigned to a val.
Reviewer Checklist (only modified by reviewer)

[ ] Did you add the appropriate labels?
[ ] Did you mark the proper milestone (Bug fix: 3.4.x, [small] API extension: 3.5.x, API modification or big change: 3.6.0)?
[ ] Did you review?
[ ] Did you check whether all relevant Contributor checkboxes have been checked?
[ ] Did you do one of the following when ready to merge:

[ ] Squash: You/ the contributor Enable auto-merge (squash), clean up the commit message, and label with Please Merge.
[ ] Merge: Ensure that contributor has cleaned up their commit history, then merge with Create a merge commit.

Lgtm. I didn't realize asserts returned a node to assign to, that's pretty nonintuitive/ maybe we should get rid of that?

Lgtm. I didn't realize asserts returned a node to assign to, that's pretty nonintuitive/ maybe we should get rid of that?

@mwachs5
It was only fairly recently added. While I agree it's somewhat unintuitive, it's needed for annotating them.

Sure, but not sure why that impacts the name of some wire that is an input to the assertion...? Or maybe I am just misinterpreting your example, the name foo_wire has nothing to do with the wire being the thing returned, as it would always be called foo_wire inside a prefix block assigned to foo

Yeah this is a fair question and we can debate the behavior. This PR does 2 things (illustrated by the 2 tests):

Makes the plugin name verification statements so they do not rely on reflective naming
Makes verification statements assigned to vals provide a prefix

The combination of these 2 makes the behavior uniform with that of Data and Mem, but we could do (1) without (2). I decided to make the behavior uniform for 2 reasons.
A) Consistency tends to be more predictable and thus understandable by users
B) There is a reported regression in moving from @chiselName to the compiler plugin where all vals used to cause prefixes and now only vals of type Data and Mem do.
I was a bit motivated by the problem/complaint in (B) to at least make the verification statement vals included in the prefixing behavior, but I grant it's only a tiny tiny piece of the overall feature regression there.


[feat] Include useApi in this package

🙏Thanks for this simple and straight forward package @sandrinodimattia! It is now almost no work to setup client side auth in react apps ⚛🚀!
Suggested feature
Because SPA's usually need data from an API, I suggest to include the useApi hook  in this package. This is the same useApi as you use in the example. If you agree I would be happy to create a PR for it 🔜.

Thanks for your feedback @ties-v. While it would be useful, I think providing a useApi hook is beyond the scope of this library. There's people using swr, GraphQL, websockets, ... that might have a variety of needs when it comes to interacting with other systems.
The only thing I think would make sense is a hook which can wrap a library, eg: a hook where you can provide axios, a GraphQL client, ...

On a second thought, you are right @sandrinodimattia !
Also a wrapper is probably not that useful as it also differs per use case how often data needs to be fetched etc., so I will close this one.
PS: I was wondering if you have any plans for the future with this repo or is it mostly a proof of concept?


set use_single_app_instance config through env var

Modified the shared-functions to use a given password and not credhub
Changes to be less diego specific

Hey @vitreuz, we've updated the PR based on your feedback. Let us know if this looks good.


Empty persist value, still persist the file with filename ~

Empty persist value in config, still persist the file with filename ~.
To Reproduce
Steps to reproduce the behavior:

Change a valid config.yml, and set the persist to nothing

sources:
  - input:
      url: 'file.m3u'
      persist:

Run the command in verbose

working dir: "/srv/IPTV/m3u"
persist file: Some("/srv/IPTV/m3u/~")

check directory content

$ ls -l /srv/IPTV/m3u/
drwxrwxr-x 2 xxx xxx     4096 Jan 12 22:28  ./
drwxr-xr-x 4 xxx xxx     4096 Jan 12 22:28  ../
-rw-rw-r-- 1 xxx xxx    27011 Jan 12 22:28 '~'

Expected behavior
No file should be created at all.
Screenshots
None
Desktop (please complete the following information):

OS: Linux Ubuntu
Browser N/A
Version 22.04

Additional context
with "empty string" the behavior is the expected one
sources:
  - input:
      url: "file.m3u"
      persist: ""

working dir: "/srv/IPTV/m3u"
persist file: None

no file is created

This is a bug iin the serde_yaml library v0.8 https://github.com/dtolnay/serde-yaml/issues/87.
I will update the lib.


Create an SAP Fiori Elements-Based UI

Tutorials: https://developers.sap.com/tutorials/btp-app-create-ui-fiori-elements.html
Step 3 Modify the UI with OData annotations
Section 3, On the launch page that now comes up, choose the Risks tile:
Expected according to tutorial: A launch page with a Risks tile inside

What really shows up: The webapp application index.html directly

Thank you so much for taking the time to contribute to this tutorial. We will get back to you asap.


[Feature Request] Phone push notifications from Desktop to Phone

Having notifications from VRCX on phone would be amazing. I sometimes am not on computer but I always have my phone with me. I never close VRCX cause I like to see what my friends are doing and maybe join them if they do something fun. Having phone notifications would def be nice for that.
I saw feature Request for Android app but that def is really hard..

I could forward notifications to mobile using something like Pushbullet.

there is a mobile "VRCX Like" for android called VRC Tracker https://play.google.com/store/apps/details?id=com.tekiro.vrctracker&hl=en_US&gl=US until this feature can be added

there is a mobile "VRCX Like" for android called VRC Tracker https://play.google.com/store/apps/details?id=com.tekiro.vrctracker&hl=en_US&gl=US until this feature can be added

Also if you have an android phone, Squid is working on a mobile version of the site essentially.
https://twitter.com/VRSquid/status/1379987861160755207?s=20

🤔 Interesting will try VRCTracker for now but it looks interesting 🤔

there is a mobile "VRCX Like" for android called VRC Tracker https://play.google.com/store/apps/details?id=com.tekiro.vrctracker&hl=en_US&gl=US until this feature can be added

Also if you have an android phone, Squid is working on a mobile app version of the site essentially.
https://twitter.com/VRSquid/status/1379987861160755207?s=20

Oh wow that app from Squid is great. Thanks for mentioning it!


Vehicle code requirements

Consolidated #24172, #19692, #14864

Rework buckle.
Remove the sprite layering + entity offset hacks it does
Vehicle should itself should be very lightweight (I imagine the ideal vehicle code is maybe a couple hundred lines) with other components to enhance its functionality.

Vehicle code should handle:

Relaying damage
Insertion
Inputs
Movement prediction

The Air for contained entity stuff needs moving to a new component. It should be more generic for any container. #19692

"Contained entity needs air" doesn't need 1 morbillion different implementations, just make it one.

MechAirComponent.Air
EntityStorageComponent.Air
DisposalUnitComponent.Air
DisposalHolderComponent.Air
CryoPodComponent.Air (Not sure about this one)

Keys needs splitting to their own component.

Remove the hard-requirement of keys from vehicles + the buckle hacks it does.

Make existing systems vehicles

Mechs
Cardboard boxes
Wheelchairs

It should support both in-container + parented versions.

additionally gas tank should share air storage with vechicles/mechs/pods, the gas tank component would remain for inserting into canisters + internals + exploding/leaking


Protect react-native branch

Could you please protect the react-native branch from being pushed to (like a master branch)?
Thanks!
https://help.github.com/articles/configuring-protected-branches/

Done, PR and one approval needed.


Adding tokenURL by default into the generated client

Description:
The requirement is to add the tokenURL given in the securitySchemes (like given below) of the OpenAPI, into the generated client code so that connector consumer will not have to specify it when initializing the client object.
"securitySchemes": {
        "oAuth": {
          "type": "oauth2",
          "flows": {
            "clientCredentials": {
              "tokenUrl": "https://apim.preview-dv.choreo.dev:443/oauth2/token",
              "scopes": {}
            }
          }
        }
      }

Describe your problem(s)
Currently connector consumer have to initalize the client like below,
samples13_1:Client c = check new ({
        auth: {
            clientId: "fINUrUtSuNVc6Wrw1AsGacByrooa",
            clientSecret: "xxxx",
            tokenUrl : "https://apim.preview-dv.choreo.dev/oauth2/token"
        }
    });

The expectation is to make the tokenURL default and let user to initialize the client like below;
samples13_1:Client c = check new ({
        auth: {
            clientId: "fINUrUtSuNVc6Wrw1AsGacByrooa",
            clientSecret: "xxxx"
        }
    });

The objective is to reduce the number of configs user have to give since the token URL is less changeable value.
Can we have this kind of functionality in the Ballerina OpenAPI tool?
Suggested Assignees (optional):
@aneeshafedo

@IsuruBoyagane15
auth: {
}

can accept many types of records
Ex: http:OAuth2ClientCredentialsGrantConfig|http:BearerTokenConfig|http:OAuth2RefreshTokenGrantConfig
We give freedom for the user to use any of these options.
But the tokenURL is needed only for http:OAuth2RefreshTokenGrantConfig

For the custom connectors generated by choreo-connector-builder service, we currently generate clients that supports only the http:OAuth2RefreshTokenGrantConfig authentication config.
Therefore, the users who initialize the client for custom connector that is generated from choreo-connector-builder, has to always give the tokenURL when initializing the client. (which is redundant work when using custom connectors generated by the choreo-connector-builder).
This issue intended to have the tokenURL added to the client for this specific case. Is there a possibility for that?

Please find my design for the solution.

We have to create our own record for OAuth2RefreshTokenGrantConfig instead of usinghttp:OAuth2RefreshTokenGrantConfig and add the default value for refreshUrl which is taken from the openapi.yaml

public type OAuth2RefreshTokenGrantConfig record {|
    string refreshUrl = "https://zoom.us/oauth/token";
    string refreshToken;
    string clientId;
    string clientSecret;
    string[] scopes?;
    decimal defaultTokenExpTime = 3600;
    decimal clockSkew = 0;
    map<string> optionalParams?;
    oauth2:CredentialBearer credentialBearer = oauth2:AUTH_HEADER_BEARER;
    oauth2:ClientConfiguration clientConfig = {};
|};

Since some records are from ballerina oauth module, we need to import ballerina/oauth

import ballerina/oauth2;

Wherever we use http:OAuth2RefreshTokenGrantConfig, we have to use OAuth2RefreshTokenGrantConfig

Case 1: (When we expect ClientConfig in init function)

Case2: (When we expect AuthConfig in init function)

@LakshanSS shall we use record inclusion for OAuth2RefreshTokenGrantConfig record ?
public type OAuth2RefreshTokenGrantConfig record {|
    *http:OAuth2RefreshTokenGrantConfig;
    string refreshUrl = "https://zoom.us/oauth/token";
|};

For the custom connectors generated by choreo-connector-builder service, we currently generate clients that supports only the http:OAuth2RefreshTokenGrantConfig authentication config.
Therefore, the users who initialize the client for custom connector that is generated from choreo-connector-builder, has to always give the tokenURL when initializing the client. (which is redundant work when using custom connectors generated by the choreo-connector-builder).
This issue intended to have the tokenURL added to the client for this specific case. Is there a possibility for that?

@IsuruBoyagane15
Is it OAuth2ClientCredentialsGrantConfig or OAuth2RefreshTokenGrantConfig you use for your requirement?
I guess it is OAuth2ClientCredentialsGrantConfig right?

@IsuruBoyagane15
If OAuth2ClientCredentialsGrantConfig is your requirement, we will add the following record
public type OAuth2ClientCredentialsGrantConfig record {|
    *http:OAuth2ClientCredentialsGrantConfig;
    string tokenUrl = "https://zoom.us/oauth/token";
|};

Other than the tokenUrl in the  AuthorizationCode flow, there are few other fixed values such as authorizationUrl and scopes that can be found in Open API definition's security schema section.
ClientCredentials flow
clientCredentials:
          tokenUrl: https://accounts.brex.com/oauth2/v1/token
          scopes:
            openid: openid
            offline_access: offline access
            https://onboarding.brexapis.com/referrals: referrals

Password flow
password:
          tokenUrl: https://api.getbase.com/oauth2/token
          scopes:
            read: Grant read-only access to all your data except for the account and user info
            write: Grant write-only access to all your data except for the account and user info
            profile: Grant read-only access to the account and user info only

AuthorizationCode flow
authorizationCode:
          authorizationUrl: https://slack.com/oauth/authorize
          tokenUrl: https://slack.com/api/oauth.access
          scopes:
            users:read: Read user information
            users:write: Modify user information
            im:read: Read messages
            im:write: Write messages
            im:history: Access the message archive
            search:read: Search messages, files, and so on

Implicit flow
implicit:
          authorizationUrl: https://api.getbase.com/oauth2/authorize
          scopes:
            read: Grant read-only access to all your data except for the account and user info
            write: Grant write-only access to all your data except for the account and user info
            profile: Grant read-only access to the account and user info only

Shall we provide default values for tokenUrl, scopes, authorizationUrl when found in any of the oauth2 flows ?
@indikasampath2000 @abeykoon @LakshanSS

For the custom connectors generated by choreo-connector-builder service, we currently generate clients that supports only the http:OAuth2RefreshTokenGrantConfig authentication config.
Therefore, the users who initialize the client for custom connector that is generated from choreo-connector-builder, has to always give the tokenURL when initializing the client. (which is redundant work when using custom connectors generated by the choreo-connector-builder).
This issue intended to have the tokenURL added to the client for this specific case. Is there a possibility for that?

@IsuruBoyagane15 Is it OAuth2ClientCredentialsGrantConfig or OAuth2RefreshTokenGrantConfig you use for your requirement?
I guess it is OAuth2ClientCredentialsGrantConfig right?

Yes it is OAuth2ClientCredentialsGrantConfig.

For the custom connectors generated by choreo-connector-builder service, we currently generate clients that supports only the http:OAuth2RefreshTokenGrantConfig authentication config.
Therefore, the users who initialize the client for custom connector that is generated from choreo-connector-builder, has to always give the tokenURL when initializing the client. (which is redundant work when using custom connectors generated by the choreo-connector-builder).
This issue intended to have the tokenURL added to the client for this specific case. Is there a possibility for that?

@IsuruBoyagane15 Is it OAuth2ClientCredentialsGrantConfig or OAuth2RefreshTokenGrantConfig you use for your requirement?
I guess it is OAuth2ClientCredentialsGrantConfig right?

Yes it is OAuth2ClientCredentialsGrantConfig.

Thanks @IsuruBoyagane15

Other than the tokenUrl in the AuthorizationCode flow, there are few other fixed values such as authorizationUrl and scopes that can be found in Open API definition's security schema section.
ClientCredentials flow
clientCredentials:
          tokenUrl: https://accounts.brex.com/oauth2/v1/token
          scopes:
            openid: openid
            offline_access: offline access
            https://onboarding.brexapis.com/referrals: referrals

Password flow
password:
          tokenUrl: https://api.getbase.com/oauth2/token
          scopes:
            read: Grant read-only access to all your data except for the account and user info
            write: Grant write-only access to all your data except for the account and user info
            profile: Grant read-only access to the account and user info only

AuthorizationCode flow
authorizationCode:
          authorizationUrl: https://slack.com/oauth/authorize
          tokenUrl: https://slack.com/api/oauth.access
          scopes:
            users:read: Read user information
            users:write: Modify user information
            im:read: Read messages
            im:write: Write messages
            im:history: Access the message archive
            search:read: Search messages, files, and so on

Implicit flow
implicit:
          authorizationUrl: https://api.getbase.com/oauth2/authorize
          scopes:
            read: Grant read-only access to all your data except for the account and user info
            write: Grant write-only access to all your data except for the account and user info
            profile: Grant read-only access to the account and user info only

Shall we provide default values for tokenUrl, scopes, authorizationUrl when found in any of the oauth2 flows ? @indikasampath2000 @abeykoon @LakshanSS

Adding scope might help to restrict the scopes in the tokens. The tokens will only have access for the scopes given in the configuration if those scopes have already been authorized with some other scopes.
I guess we don't need to focus on authorizationUrl since authorization cannot be processed through ballerina.


CB-10029: rename edge to dev

@dblotsky merge at your own peril

LGTM. Rebasing first actually made this surprisingly easy,


feat: add limits to container

Limit resource usage of container.
CPU and memory can be limited but this is not possible for disk usage yet.

Please rebase against the project-demo-202208 branch.


Strong Name

Previous versions V3 etc where built and signed with a strong name (framework versions)
Can we get a V5 built with a strong name

Sorry yes been meaning to re-add strong named DLLs. I'll try and set aside
some time to do this over the weekend.
On Thu, 25 Jan 2024, 2:19 am SteveA74, @.***> wrote:

Previous versions V3 etc where built and signed with a strong name
(framework versions)
Can we get a V5 built with a strong name
—
Reply to this email directly, view it on GitHub
https://github.com/charlesw/tesseract-samples/issues/16, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB7HSH44JYZ3LJONWUGAADYQEQ65AVCNFSM6AAAAABCI6DANGVHI2DSMVQWIX3LMV43ASLTON2WKOZSGA4TQNJQGE2TCNY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


about ERB band filter bank

Hi! Iwonder if we can reduce nerb which is currently using 256. I think if it has less number of bands it can reduce IOPS.

based on Baidu paper, I guess they are using ERB Bands applied in [10].

As I noticed in PercepNet paper(A Perceptually-Motivated Approach for Low-Complexity, Real-Time
Enhancement of Fullband Speech) they are using 34 bands with 20khz highest frequency.

This seems to be a valid question. If i may know the solution for this please ?

I Found that they used 256 number of erb bands in implementation section. my fault.

Hi jzi040941,
Thanks for your attention, the spafe toolkit provides such ERB filter banks which I didn't use in the repo. In my opinion, feature compression processing such as ERB-scale or Bark-scale cannot simply be inversed by th.pinverse, this may not work. The inverse processing (from compressed feature to raw) like percep-net you given is a better way to deal with it.
In conclusion, please note some details of the feature inverse process.
Any suggestions are welcome!


fix(anchor): ignore anchor hash when parsing internal link syntax

Fix RM-2485

There's a bug with our internal link feature where if you pass a url hash of non-unicode characters as an anchor link, the href doesn't get parsed in the Anchor component.
Here's an example: https://busted-links.readme.io/docs/intercom-3
🧰 Changes

Ignore anchor hash when parsing internal link syntax that generates a link's path, and append the hash to the path string separately.

🧬 QA & Testing

Broken on production.
Working in this PR app.

This PR was released!
🚀 Changes included in v6.52.0


Add script to tag repos for Open edX releases

This tag_release.py script will tag repositories for Open edX releases. It accepts one argument, the name of the tag to create. It determines which commits need to be tagged in which repos, and asks for confirmation. Then it uses the GitHub API to create tags on all the repos. There are also automated tests to cover the most important functionality of the script.
Relies on https://github.com/edx/repo-tools-data/pull/5

Will this help us with tagging point releases?  The repos.yaml file doesn't seem to have a way to distinguish between cypress and dogwood, and what would have to be tagged in each?  Or maybe we don't need tags in the point releases?

I'm going to merge this now, since it works. The script still needs some improvements -- most notably, the ability to create a RC branch, and create tags based on that branch -- but this is ready to be demo'ed at the engineering demos this morning.


APK from app bundle does not contain the dynamic module requested

Chatted with Ibrahim, and was asked to file an issue.
Repro: https://github.com/wisechengyi/android-dynamic-features/tree/yicdn

Click 'show assets' once app launches, and it will show 'error loading assets', whereas it should a textbox titled 'Asset Content'. When stepping through https://github.com/wisechengyi/android-dynamic-features/blob/yicdn/app/src/main/java/com/google/android/samples/dynamicfeatures/MainActivity.kt#L56, manager.installedModules is empty.
If switching from 'APK from app bundle' to 'Default APK', it works.

FWIW, dynamic delivery works by deploying the aab to play console, so likely a local toolchain issue.

I've filed a bug internally and I'm waiting for a reply to update this issue.
To test locally from the App Bundle, you can use bundletool.
In particular, once you've generated the App Bundle, you can use the following commands to install the right minimum set for your app:
bundletool build-apks --bundle app.aab --output app.apks
features/app/build/outputs/bundle/debug$ bundletool install-apks --apks app.apks

Keep in mind that this is not going to be able to download the additional modules. To test the download, at this moment, you've to go through Play.
Or you can install the complete application (universal APK) using the commands:
bundletool build-apks --mode=universal --bundle app.aab --output app.apks
features/app/build/outputs/bundle/debug$ bundletool install-apks --apks app.apks

More information on how to use Bundletool can be find on the documentation site.

I was supposing that this was now fixed in the newer AS, but seems that it's still a problem.
Closing this and follow up on the public issue.

Encountering a similar error. When using the above build configuration (APK from app bundle), my build fails with the error message:
The APK Set archive does not contain the following modules: [list of my dynamic feature modules]
Any indea on how to fix this?
I'm getting an "acces denied" message when trying to view your "public issue" link.


Cached GTF or temp files?

Hi Job,
I'm currently using sci-rocket for our non-model planarian species, utilizing a GTF file with 81k genes categorized into various types, including protein-coding, lncRNA, and rRNA. Among these, we have approximately 24k protein-coding genes.
Initially, I ran sci-rocket on the unedited GTF file, resulting in a CDS object with 81k features. To address this, I filtered the GTF file to include only the protein-coding genes. However, despite this adjustment, the resulting CDS object consistently contains 81k features instead of the expected 24k.
I'm wondering whether the pipeline might be caching the GTF file or utilizing temporary files, potentially causing it to revert to the original unfiltered version. What do you think?
Thanks,
Gaurav

Hi Gaurav,
How does your config.yaml look like? The GTF is used during the STAR-index generation and those annotations are used during the alignment/counting. Unless a previous star_index path is given as in that case it will always symlink and use the pre-generated index.
It should be re-generating the STAR-index based on the supplied genome / GTF if you replace it. Make sure the GTF is differently named so the STAR-index command is different compared to the previous run (as that's how Snakemake determines whether to re-run).
Alternatively, if it keeps using the old generated index even when changing the GTF, you can just delete resources/index_star/{species}/ and it will re-generate it based on the genomes/GTF supplied in the config.yaml.
Let me know if this helps!
Best,
Job

Thanks a lot, I will try it out!


Fix building for iOS

To fix the following error in Release build for simulator:
ld: in /Users/fangjun/open-source/sherpa-ncnn/build-ios/sherpa-ncnn.framework/sherpa-ncnn(allocator.cpp.o), building for iOS Simulator, but linking in object file built for iOS, file '/Users/fangjun/open-source/sherpa-ncnn/build-ios/sherpa-ncnn.framework/sherpa-ncnn' for architecture arm64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

Change No to Yes in the following screenshot:

See also
https://stackoverflow.com/questions/63607158/xcode-building-for-ios-simulator-but-linking-in-an-object-file-built-for-ios-f/64139830#64139830


Support url form encoded bodies

Motivation
Opening this as a preliminary approach to adding support for url encoded forms #182.
I notice that we're probably going to want to take a different approach to this implementation by first building out a custom URLFormEncoder (#192), so this might be mostly redundant at this point, but wanted to at least document this here as my first contribution to the project.
It may be that work on #192 can be merged into the Converter methods in the associated runtime changes
Modifications
Add a new urlEncodedForm content type to support type safe x-www-form-urlencoded requests.
Result
Url form requests will take a Codable object type, rather than a raw Data input
Test Plan

Updated and passed reference tests with example url form request
Not yet added additional unit tests as this PR is only intended as a draft for feedback

Hi @benfrearson – thanks for getting started on this! Let me read it all and drop some comments, then we can figure out how we approach staging this support in.


Make geometry classes

Simple 2D geometry such as vectors.
Estimated time: 25 m.

Fixed by 4509dc2.
No more geometry is needed for sprint 1.


Combination of Belongs-To and Has-Many unexpectedly generate duped constraints

GORM Playground Link

https://github.com/go-gorm/playground/pull/281
Description

Bidirectional Belongs-To and Has-Many associations unexpectedly generate duped FOREIGN KEYconstraints that can be unified.
type User struct {
	gorm.Model
	Pets      []*Pet
}

type Pet struct {
	gorm.Model
	UserID *uint
	User User
}

$ ./test.sh
$ docker-compose exec mysql mysql -ugorm -pgorm gorm -e "SHOW CREATE TABLE pets"
...
CREATE TABLE `pets` (
  CONSTRAINT `fk_pets_user` FOREIGN KEY (`user_id`) REFERENCES `users` (`id`),
  CONSTRAINT `fk_users_pets` FOREIGN KEY (`user_id`) REFERENCES `users` (`id`)

The same thing happens with sqlite and postgres too.

Fixed, thank you for your report.


Operation objects

Created transaction classes for the operations:

reserve
pay
cancel.

Moved payment method specific data to entity classes.
A transaction class can reference such an entity if necessary.

Coverage remained the same at 100.0% when pulling 51cb941bd55a11fd136178d1cfddf6ff97c88995 on reka:operation-objects into 94e091add9f4cadc50763f367dc6cd2d728109e3 on wirecard:42-redesign.


Doomfist Uppercut lockout

In GitLab by @jonjon435 on Feb 22, 2024, 16:50
In this code doomfist is able to use his abilities, primary fire and ult immediately after using his Shift (uppercut)
compared to overwatch 1 where he had a 0.35 second lockout leaving him vulnerable and susceptible to being killed or stunned
Source:
https://web.archive.org/web/20220717010007/https://overwatch.fandom.com/wiki/Doomfist

mentioned in commit 28f9e154dd462f746ca8228cf83795b21bcad234

likely implementable, but not too worried for now

mentioned in commit 5fe1d62c14c3382488b2af1934f4372c877af866


renderer-comparison.html uses getQueryParameter

Related to https://github.com/phetsims/chipper/issues/516.
renderer-comparison.html contains usages of phet.chipper.getQueryParameter, which is now deprecated.  @jonathanolson said that @samreid owns this file, and options are to:
(1) convert it, if it still has value (@ jonathanolson volunteered to do this)
(2) delete it (I volunteered to do this :)

(2) delete it (I volunteered to do this :)

The file should remain, it's more a question of the query-parameter handling in the file.

I'm going to take care of this.

Checking...

            if ( phet.chipper.getQueryParameter( queryParameterKey ) ) {
              var test = phet.chipper.getQueryParameter( queryParameterKey );

should be something like this:
            if ( QueryStringMachine.containsKey( queryParameterKey ) ) {
              var test = QueryStringmachine.get( queryParameterKey, { type: 'string' } );

I'll assign it to @pixelzoom since he said:

I'm going to take care of this.

I converted as @samreid suggested.  But...
(1) It seems like we should be using this schema for test (similar to screens):
{
      type: 'array',
      elementSchema: {
        type: 'number'
      },
      defaultValue: null
    },

(2) filterSubset is applied to both renderer and test values, but its internal variables appear to be related to test.  Probably something that was repurposed and not generalized. Confusing. And maybe buggy in the case of renderer, see below.
(3) Recommended to rename renderer to renderers, so as not to be confused with renderer in initialize-globals.js.
(4) I expected ?renderer=canvas.svg to perform only the tests for canvas and svg.  But instead this is what I see:

In the above commit, I fixed the schemas.  Testing looks good:

My only complaint is that the test indices get renumbered when you select a subset.  But that doesn't seem worth solving now.


fix: Make remove channel works when collection meta gone

List collection with lastest logic
If all collections are gone, make remove channel workable

/lgtm
/approve


Introduce the ability to download Stata code corresponding to the checks selected in the UI

I have added this in code-export branch.


awkwardness around committing auto-generated files

We generate documentation files and commit them to the repository so that they are visible/browseable in the github web interface. However, this causes some problems

there's no guarantee (without additional scripting) that the current state of the templates is represented by the current state of the documentation
someone might modify an auto-generated file by mistake (risk mitigated by a warning in the files)
a change to the templates might break the doc generation, but not be noticed
merges, rebases, and various VCS diff activities become unwieldy with large changes in the auto generated files

This problem might get worse if/when we move to generating more stuff.
I was considering ways this might be improved and one possible solution would be to use a separate git repository for all the auto-generate content. The two repositories could be linked together via git submodule. Here is an example:

https://github.com/jmtd/application-templates/tree/docs-submodule-test is a fork of the repo with ./docs deleted and a submodule registered in its place, pointing at ...
...a new test repository https://github.com/jmtd/test-app-templates-doc-output which has committed the result of running the docs generator in the other repository.

There are likely caveats with this approach, but on the face of it the integration is pretty good on the website front. "Docs" is a clickable hyperlink in the main repository, just as before, and all the generated stuff is browseable right in the github UI.
Github also has the notion of "GitHub pages" which might be a better fit for the ancillary repository. I haven't looked at that, yet.

Some usage instructions for this demo
$ git clone -b docs-submodule-test git@github.com:jmtd/application-templates.git submodule-test
$ cd submodule-test
$ git submodule update --init

This gives you the main repo checkout and the sub-module checked out inside ./docs.
Modifying the contents of ./docs looks like this from the POV of the parent repository:
$ ./gen_template_docs.py
$ git status
On branch docs-submodule-test
Your branch is up-to-date with 'origin/docs-submodule-test'.
Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git checkout -- <file>..." to discard changes in working directory)
  (commit or discard the untracked or modified content in submodules)

	modified:   docs

no changes added to commit (use "git add" and/or "git commit -a")


angular2.ts was removed but is still referenced in index.html

https://github.com/auth0/angular2-electron/blob/22801161b83397bff68f515bfa79b784ccee4ebd/app/index.html

Do you mean the angular2.js build? That should be there :) I just merged a PR that removes angular2/angular2 from the Webpack build, was that causing an error perhaps?


Uncaught Snapshot missing on Livewire component with id

Package
filament/filament
Package Version
v3.1.18
Laravel Version
v10.35.0
Livewire Version
v3.3.0
PHP Version
php:8.2-fpm
Problem description
When multiple (1+) Livewire components are on a page, clicking a header action will remove all but one livewire components. The  error "Uncaught Snapshot missing on Livewire component with id" is thrown when this occurs. In the recording below I put 3 test livewire component. Using different components will give the same result.
https://github.com/filamentphp/filament/assets/106535819/ba354f8e-b9ee-42df-85fb-d93f13ea6931
Expected behavior
I expect that that the, in this case, action form modal is opened and that the livewire components stay on the page.
Steps to reproduce
The simplest way to reproduce the error is to add two or more Livewire components to a page. Add a header section and
    public static function infolist(Infolist $infolist): Infolist
    {
        return $infolist
            ->schema([
                Livewire::make(Interference::class),
                Livewire::make(Interference::class),
                Livewire::make(Interference::class),
                Section::make('break')
                    ->headerActions([
                        Action::make('throw error')
                            ->form([
                                TextInput::make('TestInput')
                            ])
                    ])
            ]);
    }

I added an reproduction repository with a full project with a livewire component
Reproduction repository
https://github.com/Lars-icepay/filament-bug-test
Relevant log output
Uncaught Snapshot missing on Livewire component with id: C2gixlm2VfB448Pfvq0f

Fixed by #10946. You will need to give each component a unique key(), so we can pass it to Livewire for tracking.


Export to GLB of unmodified model with animations is having deformations on bones

I am using a glb with animation and trying to export it as such without any alteration.

But, the export to GLB is having deformations on bones.
It is reprocible in Babylon Sandbox.

I think this issue is open.
Originally posted by @tskarthikus in https://github.com/BabylonJS/Babylon.js/issues/7244#issuecomment-924657790

Please, as the template is suggesting, could you start the discussion on the forum https://forum.babylonjs.com/
Github is exclusively used for validated bugs/feature request which have been discussed on the forum.
When you post there, attach the model and so on so that we could repro.

Thanks indeed.
I've posted it in the forum.
https://forum.babylonjs.com/t/export-to-glb-of-unmodified-model-with-animations-is-having-deformations-on-bones/24010
On Wed, Sep 22, 2021 at 5:30 PM sebavan @.***> wrote:

Please, as the template is suggesting, could you start the discussion on
the forum https://forum.babylonjs.com/
Github is exclusively used for validated bugs/feature request which have
been discussed on the forum.
When you post there, attach the model and so on so that we could repro.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/BabylonJS/Babylon.js/issues/11142#issuecomment-924862057,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKTBFZTIJVCI2EO5P7NVHDUDHANXANCNFSM5EQTYQZA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

--
k A r T h I k__________


Delete the cache and relaunch the game causes crash

Crash reason: Access Violation
Unknown thread
ragefiledriverdirecthelpers.cpp:230 F:/Etterna/Songs/Knit's Favorite Collection/02/*
ragefiledriverdirecthelpers.cpp:230 F:/Etterna/Songs/Suomi 100/downfall (Wh1teh)/*
ragefiledriverdirecthelpers.cpp:230 F:/Etterna/Songs/Knit's Favorite Collection/16warz/*
ragefiledriverdirecthelpers.cpp:230 F:/Etterna/Songs/877xe/.357 Magnum/*
ragefiledriverdirecthelpers.cpp:230 F:/Etterna/Songs/Dingles Finale/Smoke Bomb (Dingles)/*
Thread: Main thread
ragefiledriverdirecthelpers.cpp:230 F:/Etterna/Songs/You're Streaming Again/*
ragefiledriverdirecthelpers.cpp:230 F:/Etterna/Songs/Yume's 1st Restart Pack/*
ragefiledriverdirecthelpers.cpp:230 F:/Etterna/Songs/Yume's 2nd A Half Pack/*
ragefiledriverdirecthelpers.cpp:230 F:/Etterna/Songs/Zero kara Hajimeru Stepmania/*
ragefiledriverdirecthelpers.cpp:230 F:/Etterna/Songs/Zinowo/*
Thread: Decode thread
Thread: Mixer thread
Thread: Music thread
d1a85bf2: Etterna!d16d0000
Static log:
Etterna0.64-git-4c0339daf4
Compiled 20181205 @ 03:26:12 (build 4c0339daf4)
Log starting 2018-12-08 20:58:12
Verbosity: 1
Windows 6.2 (Win8) build 9200 []
WARNING: Unmatched 3 in "0003"
WARNING: Unmatched 3 in "2123"
Partial log:
00:16.419: Song file "/Songs/4k osumania dan/4K 9th Dan v2 (9th Dan) [Shoegazer]/4K 9th Dan v2.sm" has an unexpected value named "ATTACKS".
00:16.437: Song '/Songs/osu ranked/152065 Ras - STAGER [no video]/' points to a music file that doesn't exist, found music file 'bass01 (1).wav'
00:16.437: Format wav failed: Not a WAV file
00:16.437: Format mp3 failed: Failed to read any data at all
00:16.444: Sound file "" is really oga.
00:16.535: Song file "/Songs/4k osumania dan/4K Luminal v2 (Luminal) [Shoegazer]/4K Luminal v2.sm" has an unexpected value named "ATTACKS".
00:16.655: Song file "/Songs/4k osumania dan/4K Tachyon v3 (Tachyon v3) [snoverpk]/4K Tachyon v3.sm" has an unexpected value named "ATTACKS".
00:16.693: Song file "" has no BPM segments, default provided.
00:16.693: Song '/Songs/osu ranked/153316 DJ Genericname - Dear You [no video]/' points to a music file that doesn't exist, found music file 'Dear You.mp3'
00:16.791: Loaded 12 songs from "Songs/4k osumania dan"
0.60 can still relaunch the game, crash started from 0.61

it's almost certainly a file or pack with some unusual structure or format, possibly the osu loader? Clear cache again and move the packs around the 4k osumania dan out of your songs folder and see if it loads, if not, move about the first half of them out
i don't really remember offhand what changed with song load from 0.6 to 0.61 but if you could pinpoint the cause of the crash and send it we can fix it

it's almost certainly a file or pack with some unusual structure or format, possibly the osu loader? Clear cache again and move the packs around the 4k osumania dan out of your songs folder and see if it loads, if not, move about the first half of them out
i don't really remember offhand what changed with song load from 0.6 to 0.61 but if you could pinpoint the cause of the crash and send it we can fix it

I think the "osu ranked" pack caused the problem, but  i really have no idea what the cause was. Anyway, it's not a so big problem i think.

in that case it's most likely fixed by 887407008e24adc85038766a21da5f2dc28841a3


iOS crash when file name has empty space.

I have attached the code and log of exceptions thrown when I tried to initialize a player with an empty name in iOS. Is there a workaround for this issue if anyone has encountered it before?

Please reach out to the Flutter team regarding this video_player issue. Thanks.


Transfer to FIRSTMap organization

This is up to you but I would prefer it.

I can do this but I believe I will need permission to create repositories in the FIRSTMap organization to transfer it.


Toggle stuck

Sometimes , the toggle is stuck.
Example:

I had this issue, but then I surrounded my input with the label tags and it worked. Something like this.
<label>                                 Active:<br />                                 <input type="checkbox" data-on="success" data-off="danger" data-off-label="No" data-on-label="Yes" data-indeterminate="true" data-bind="bootstrapSwitchOn: status" />                             </label>

Interesting. @ghost Did you figure out why over the past months?


Will not work in Safari until browser window is resized

Hi Saabi,
Thank you for the time and effort you've put into this polyfill.
I'm experiencing one issue while using this with Safari in a test webpage I'm building, wherein the vw, vh, vmin etc untils are not being converted to pixels until after the browser is resized.
If you know anything about this issue, any tips you have will be appreciated.
Thanks,

Were you able to resolve this? I'm having an issue with Safari as well.


PyPi package?

Any chance this project could be reformatted into a PyPi project I could install with
pip install gui-maker

and then run it with
gui-maker

or something like that?
I can help out with this if you would like.

Honestly I don't know how to do that, I'll read about it. But yeah it'd be great! Any help is greatly appreciated

I'll take a look at doing this tm, and I'll make a pull request. I'll explain what changes I made to make it work as well. The main codebase shouldnt need to change.

Thank you! I've read about it and I should be able to prepare it myself, if you prefer. Though I don't know how to run it through terminal with "gui-maker" command

Of course if you wanted to do it yourself that would be great. For the gui-maker, in your setup.py file for the packaging, just add an entry-point as a console script, pointing at the main function to start your program, and you can name it gui-maker or whatever you would like.

I tried, is this correct? I wasn't able to add the entry point though, I'll wait for you before publishing to PyPi

You can check if it works by running
pip install .

in the project directory. For entry points, it should be this:
    entry_points={
        'console_scripts': [
            'GUICoder= GUICoder.app:main',
        ],
    },

You will just have to move the contents of if __name__ == '__main__': to a main() function, and then call that in if __name__ == '__main__':

I tried but it gives me this error:
Traceback (most recent call last):   File "/Library/Frameworks/Python.framework/Versions/3.7/bin/GUIcoder", line 6, in <module>     from GUICoder.app import main ModuleNotFoundError: No module named 'GUICoder'
Problem is I don't get where it takes GUICoder (with the capital C) from..
Can you help?

Hi, not sure if you figured it out yourself already, but it all worked for me!
(PyEnv) E:\ProgrammingWorkspace>git clone https://github.com/max-dotpy/GUI_maker
Cloning into 'GUI_maker'...
remote: Enumerating objects: 104, done.
remote: Counting objects: 100% (104/104), done.
remote: Compressing objects: 100% (74/74), done.
remote: Total 104 (delta 49), reused 64 (delta 22), pack-reused 0
Receiving objects: 100% (104/104), 24.17 MiB | 5.26 MiB/s, done.
Resolving deltas: 100% (49/49), done.

(PyEnv) E:\ProgrammingWorkspace>cd GUI_maker

(PyEnv) E:\ProgrammingWorkspace\GUI_maker>ls
GUIcoder  LICENSE.txt  README.md  README.txt  requirements.txt  setup.cfg.txt  setup.py  simple_demonstration.mov

(PyEnv) E:\ProgrammingWorkspace\GUI_maker>pip install .
Processing e:\programmingworkspace\gui_maker
Collecting Pillow
  Downloading Pillow-7.0.0-cp37-cp37m-win_amd64.whl (2.0 MB)
     |████████████████████████████████| 2.0 MB 3.3 MB/s
Collecting pyperclip
  Downloading pyperclip-1.7.0.tar.gz (15 kB)
Installing collected packages: Pillow, pyperclip, GUIcoder
    Running setup.py install for pyperclip ... done
    Running setup.py install for GUIcoder ... done
Successfully installed GUIcoder-0.1 Pillow-7.0.0 pyperclip-1.7.0

(PyEnv) E:\ProgrammingWorkspace\GUI_maker>py
Python 3.7.0 (v3.7.0:1bf9cc5093, Jun 27 2018, 04:59:51) [MSC v.1914 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> import GUIcoder
>>> exit()

(PyEnv) E:\ProgrammingWorkspace\GUI_maker>GUIcoder

(PyEnv) E:\ProgrammingWorkspace\GUI_maker>

And the program opened up like normal

I needed to update the version number and delete the previous "entrypoint exec file". Now it seems to be working fine. Thank you !!

No problem! Will use this for sure, much easier to work with if I can just do pip install now


Move source code into sub-directory /src

Closes #4

Well, I'm not sure here. I revisited this by checking the log via TortoiseGit. TortoiseGit is correctly listing all files as renames, which you can see from the attached screenshot. When I create the merge request on GitHub, GitHub seems to disagree with some of the files just being renamed.
I tried reverting back to the master branch, then repeating the operation in a new branch. I thought this should have restored the local copies of the source code to how they exist in the repository. Then I moved the files back into the src directory and committed. It appears to have done the exact same thing. Is it possible that somehow my local repository is smart enough to track whitespace changes locally as not being changes that are being reported as changes on the remote? This doesn't seem right though.

I prefer the files marked as rename in the repository since it improves tracking the file changes. However, I really don't have an appetite for slogging through the bowels of Git right now to get everything in agreement.
-Brett

I'm seeing the Lines Added and Lines Removed are not 0 on a lot of the renames. GitHub is not counting them as renames if there are any line changes, which is different behavior from TortoiseGit.
I just created a fresh clone of the repository from GitHub on my computer. Then I moved the files using Windows Explorer and committed using TortoiseGit. I did not open Visual Studio at all during the process. The results were exactly the same. Whatever is happening doesn't appear to be related to Visual Studio. Not sure if Windows Explorer or TortoiseGit is applying the whitespace changes.
Also, not sure what makes TortoiseGit call them renames even though it reports the entire file changes and GitHub to not call these renames.
 Many are setup with automatic line ending conversion settings by default.
I'll check in more detail a little later.

Ok, I looked at a diff that highlighted whitespace changes. It seems there are trailing spaces or tabs on some lines that were removed. In some cases, it appears to be the auto indent tab on blank lines. In others it's whitespace after the semicolon or some trailing punctuation.
Perhaps you can run a regex search and replace on the code to find all whitespace followed by the end of line character (\s+$), and replace it with an empty string. You can run that conversion as the first commit on the branch, and then try moving the code in a second commit. If the whitespace changes are pre-done, the files will hopefully be properly marked as moved at that point.
I checked both branches, and they both have identical looking whitespace changes.

Oh, and on a side note. I believe Git does not actually track renames in the repository with metadata. Renames are detected using heuristics. Hence the discrepancy when there are subtle changes to the file.

I'm fairly certain the actual problem is Git changing line endings on commit. There is a setting to standardize line endings in Git. It is currently set to auto on my local machine and has decided to change the line endings on the files.
I don't know why it has decided it needs to change the line endings at this time. I don't know if the auto is seeing the new makefile in the repo?
There is a way to standardize line endings across the repository using a .gitattributes file if we want to pick one and go with it. https://help.github.com/articles/dealing-with-line-endings/
I did try deleting random 'hanging' spaces and tabs occuring at the end of the line, but this did not change rename status. Changing the line endings to 'do not change' in Git did fix it regardless of any of the hanging spaces or tabs existing.
Will try to upload soon.

This has been replaced by Pull Request #14.

I have a global Git setting to ensure my client doesn't make any unexpected changes:
core.autocrlf=false

My take on it:
I like the global setting better. You don't need to litter every repository with a settings file, which will no doubt be forgotten for some repo when you least expect it. Plus, the repo specific settings don't enforce a policy, they just set a default behaviour. Someone can override it if they want to. Hence, a repo setting isn't really protecting that repo from anything. Besides, these changes affect your own commits, not ones you download, so it makes sense to use settings based on how you work.
My personal taste, is to not have unexpected and unrelated changes to code in my commits. If I need a change, I can explicitly make a change. I want the power in my hands.
Anyway, thanks for cleaning this up and getting this done.

I'm having major line ending problems now in OP2Utility that don't seem to be solved by setting the clrf settings in the repository itself to false. I think I'm setting the core/global as well with no help but I need to spend some time on it to verify. In OP2Utility, entire files are listed as changed for just changing 3 lines since every line ending is getting updated.
The nice part about littering everyone's local repository with a settings file is they know what the repository expects by examining it, possibly before jumping in and commiting code with different line endings than the owner wants.
Not really sure what I want here, except that I don't want to be worried about line endings on top of all the other concerns attached to coding. I just want it to work and not muddy up changes. I'll try to stop grumping about it though. Clearly setting Git to Auto for line endings isn't working anymore.

Ahh yes, I think that's what got me originally to turn off the autocrlf setting. I was trying to checkout a Windows project on Linux, and every file appeared modified in git status. I believe git status detects file changes by comparing the file hash, hence if the line endings change, the hash changes, and the files appear modified. To make it even more frustrating, git diff showed nothing, as simple whitespace changes like that were hidden.
It seemed like the auto settings would always produce a problem for someone. Either Windows developers would see a ton of ghost changes, or Linux developers would see a ton of ghost changes. I think the setting of input was a little better, but still made magical changes. As it was my first time using Git, I wasn't exactly pleased I had to spend time researching this issue right off the bat.
At any rate, my personal preference to for the version control system to get out of the way, and not make any magical changes to the files. Besides, there is no way it can get it right 100% of the time. It can't know if a file is text or binary, with certainty, without being told, and changing a binary file would corrupt it. At best, it can use heuristics to get it right most of the time. Seems like a broken solution to the wrong problem to me. The real problem is text editors and tools that can only handle one of the platform specific LF or CRLF.
The .gitattributes file does nothing to enforce repository policy. Hence using one is also not a solution in that regards. It merely sets default settings, which most programmers won't even be aware of. When they are set badly, I've found myself making temporary changes to .gitattributes while I work, effectively disabling the settings, and then just not committing my local changes to that file. Related, most people don't really know what the settings in that file do, or how they work, and the settings are named in a not very descriptive and possibly misleading way, so I wouldn't count on them being correct.
There's a bit of info on the options over at Git Configuration.
In terms of policy enforcement, I think that's what Git Hooks are able to do, as they can actually stop a commit if it fails a certain check script.

Ranting aside, sorry to hear you're having trouble with this. Does it appear to be doing EOL conversions even though the setting is false? Does this affect Git status, Git diff, or both? Does it differ between Git clients? I believe use of the .gitattributes file is dependent on the Git client you use. In particular, I think I once read a bug report suggesting Visual Studio's Git plugin didn't respect the autocrlf setting in a .gitattributes file.


Missing 22621 ARM64 FOD iso link

Hi folks!
In the page I can get one FOD iso for AMD64, however what I need is ARM64:
https://github.com/MicrosoftDocs/azure-docs/blob/main/articles/virtual-desktop/windows-11-language-packs.md
What I can get is 22621.1.220506-1250.ni_release_amd64fre_CLIENT_LOF_PACKAGES_OEM.iso
What I need is 22621.1.220506-1250.ni_release_arm64fre_CLIENT_LOF_PACKAGES_OEM.iso
May I kindly request a link for ARM64? I would greatly appreciate it.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 36da14cf-4a3d-c509-9b18-9c7bc398a6a2
Version Independent ID: 1b2a078c-ed85-34fc-a206-b9579f91691c
Content: Install language packs on Windows 11 Enterprise VMs in Azure Virtual Desktop - Azure
Content Source: articles/virtual-desktop/windows-11-language-packs.md
Service: virtual-desktop
GitHub Login: @Heidilohr
Microsoft Alias: helohr

@alastorid
Thanks for your feedback! We will investigate and update as appropriate.


Documentation for directory structure and working directory for complex locust projects

The documentation provided is quite lacking in locustfiles that do any imports of helper modules. It seems to imply that you will always have a completely flat directory structure of locustfiles like:
cwd/
    locustfile1.py
    locustfile2.py
    locustfile3.py

But one of the big advantages of locust is creating complex user behaviors that may require additional logic and data to be shared across locustfiles, such as configs or auth. Similar to how most applications are run, we should think about whether we want to continue to have the locustfile be the "root" from which to import or if we want to have a root dir that contains multiple locustfiles. e.g.:
locustproject/
    locustfiles/
        locustfile1.py
        locustfile2.py
        locustfile3.py
    helpers/
        auth.py
        config.py

See https://github.com/locustio/locust/pull/484, which proposes including the cwd in the path, but I think this might have undesirable side effects for current integrations. I think we can go that route, but official docs should recommend only running from the project root (locustproject) in the example above.

I gave it a crack (#501).  I might be missing some spots, but first let me know if you think this is better or worse.  I think my change sufficiently exposes alternatives to flat-directory projects.

This is in the docs now and the original PR is being reviewed.


export tokenizer parameter

Add back the tokenizer parameter, since it's hard to set tokenizer now.

Can one of the admins verify this patch?

Assigning to @ludimagister for general API

Yes, this should be fine. Thanks for adding.

Can you squash the commits?  Then we'll test and merge

@vrv sure, done.

Thanks!
@tensorflow-jenkins: test this please

@vrv I just checked the log, looks weird, what should I do now?

//tensorflow/user_ops:ackermann_test                                     PASSED in 1.9s
//tensorflow/python:sparse_add_op_test                                  TIMEOUT in 65.0s
/var/lib/jenkins/workspace/tensorflow-pull-requests-cpu/bazel-ci_build-cache/.cache/bazel/_bazel_jenkins/eab0d61a99b6696edb3d2aff87b585e8/workspace/bazel-out/local_linux-fastbuild/testlogs/tensorflow/python/sparse_add_op_test/test.log
Executed 222 out of 387 tests: 386 tests pass and 1 fails locally.
Parameterized build ends with FAILURE at: Wed Mar 30 01:29:11 UTC 2016 (Elapsed time: 378 s)
Build step 'Execute shell' marked build as failure
Unable to get pull request builder trigger!!
Setting status of 19083e1e2f97bb46ecfff9fad35c7f4d10cdc2cb to FAILURE with url http://ci.tensorflow.org/job/tensorflow-pull-requests-cpu/525/ and message: 'FAILURE
'
Using context: Linux CPU Tests
Finished: FAILURE

I think it's a 'flaky test' due to timeouts -- I think it's fine, so I'm going to merge.


1euro filter in pozyx.py

Instead of a floating average on the language side

actually pozyx offers a smoothing option


Minor update to default rescaling params in resizerawtrafo

I'll go ahead and merge this once the tests pass.


Play pause not working in 0.4.5 version

void  PlayClicked(object sender, System.EventArgs e)
{
await CrossMediaManager.Current.Play();
CrossMediaManager.Current.Play();
}
void PauseClicked(object sender, System.EventArgs e)
{
await CrossMediaManager.Current.Pause();
CrossMediaManager.Current.Pause();
}
using this method for loading from url
await CrossMediaManager.Current.Play("");
its loading and playing, but once we pause its not playing again.

@ranjithkumaraa Update the issue by using an issue template, or it will be closed.
Also only the latest version is supported.


ANN typing

For most ANN fields, the type is known in advance, so we can do type conversion. For unknown fields, try auto typing those (the user should be able to opt-out or opt-in, here [not yet implemented]).

If we merge this, we should also mention/explain this in the README


fix mk complete to not use hardcoded client and cache connections

Fixes #360

Codecov Report

Merging #362 (2b6ae0d) into main (35d96db) will decrease coverage by 0.12%.
The diff coverage is 12.50%.

@@            Coverage Diff             @@
##             main     #362      +/-   ##
==========================================
- Coverage   66.92%   66.81%   -0.12%
==========================================
  Files         246      246
  Lines       13286    13255      -31
  Branches     1926     1874      -52
==========================================
- Hits         8892     8856      -36
- Misses       3967     3975       +8
+ Partials      427      424       -3

Flag
Coverage Δ

unittests
66.81% <12.50%> (-0.12%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

meerkat/ops/complete.py
30.43% <12.50%> (-9.57%)
:arrow_down:

... and 10 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


How do we smoothen User Experience of Github/Gitlab issue/commits collection for both NewUser/Advanced User

Description
Problems:

Consider the pipeline user flow alone: Github/Gitlab both have api rate limit mechanism, so we developed gitextractor to collect commits and ref by cloning repo instead of calling api. However, this approach introduce learning curve to our user, they have to know about why/what/how we have to collect data from a single repo by 2 separate plugin.
Consider the big picture: we have beening alway lacking a wizard feature to guide user through out setup process, maybe, we could solve the problem above by a step by step guide. and Wizard is a must, no matter what, the detail would be discussed in another issue [to put wizard issue link here]

Describe the solution you'd like
Solution 1: gitextractor as lib by @hezyin
gitextractor to provide a set of functions for Github/Gitlab utilization
Questions to be answered:

Do we save commits/refs into tool layer tables?
Do we seperate gitextractor subtasks into commits/branch/tag?
Do these subtasks present as a set of subtasks in Github/Gitlab plugin
Now that Github/Gitlab depend on gitextractor, they have to be updated if gitextractor got updated
Subtasks can only be executed in sequetial order, so issues/commits can not be collected in parallel

Solution 2: wizard would create a optmized pipeline for user by @klesh
all plugins stay separated, and wizard ask user sth like Do u have a github repo,is it public sth like that, and create a optimized pipeline to collect issues , commits and other in parallel, and prompt user to create a blueprint for recurring data collection if pipeline was carried out successfully.
Advantages:

No need to refactor gitextractor, and do anything specifically for github/gitlab
Github/Gitlab issues collection and commits collection can run in parallel

Has the Feature been Requested Before?
No

Related to #1700

Addressed by #1725


Use extra request body search fields

Hi is there any option to use script_fields for rawSearch() described here
https://www.elastic.co/guide/en/elasticsearch/reference/7.0/search-request-script-fields.html

@AndreyLanko did you find a solution for this?

@codana-user, you can create a feature request if you need script fields support 🙂


Document %sqlcmd profile

We need to document the profile command. You can use the %sqlplot documentation as an example.
The idea is to look at the source code and document the arguments that each command has along with some short examples.
The easiest way to write the examples is to use DuckDB with the penguins as shown here: https://jupysql.ploomber.io/en/latest/integrations/duckdb.html
The tutorial can also be helpful for this: https://jupysql.ploomber.io/en/latest/user-guide/data-profiling.html

hi @TegveerG please work on this issue. there's a PR that already contains some work, please use the  doc/api/magic-profile.md file in this PR https://github.com/ploomber/jupysql/pull/395/files and start a new one. please write the acceptance criteria


Difficulty in scrolling of Content in Safe Paths News of Latest News screen- Android app v1.0.1

Version
Type of phone: Realme
OS Version: Android 9
Model: U1
Steps to reproduce
Open the App
Open Hamburger menu
Tap "Latest News" to navigate to SafePath News screen
Tap on Hamburger Menu against COVID SafePaths
Tap on FAQ and scroll through the contents
Expected Behavior
Content scrolling should be smooth and user friendly
Actual Behavior
Difficulty in scrolling of the content

Can you give more details about the difficulty? Is it lagging, skipping or not moving when you drag the content?

It is not moving when I drag the content.

Hello! I would like to tackle this issue.

Thanks @Namit-Agrawal ! BTW- are you on our Slack? (I couldn't find you using your github name....) We have an #app_dev channel that would be good to get you connected with as well as a lot of documentation. If you fill out our volunteer onboarding form I'll send you all the details.
https://docs.google.com/forms/d/e/1FAIpQLSdzqAxrlrxb_HqLh1KXnfPu1rse4aByS2krL1OYlN3qKChyqA/viewform

Hi Jay,
I do have a slack account! Would love to get connected. I have filled out
the form below.
Best Regards,
Namit Agrawal
On Mon, May 11, 2020 at 5:23 AM Jay S<EMAIL_ADDRESS>wrote:

Thanks @Namit-Agrawal https://github.com/Namit-Agrawal ! BTW- are you
on our Slack? (I couldn't find you using your github name....) We have an
#app_dev channel that would be good to get you connected with as well as a
lot of documentation. If you fill out our volunteer onboarding form I'll
send you all the details.
https://docs.google.com/forms/d/e/1FAIpQLSdzqAxrlrxb_HqLh1KXnfPu1rse4aByS2krL1OYlN3qKChyqA/viewform
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Path-Check/covid-safe-paths/issues/737#issuecomment-626614904,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKIPMQDYSLQ5JPZ35S543PLRQ7GZBANCNFSM4MVOEDBA
.

Hi @Namit-Agrawal I've added you to our slack space and the #app_dev channel.

Closing as this is being addressed in a larger UI/UX refactoring.


Package Visualizer into wheel

Given that visualizer is currently a separate repo, let's look into creating a separate wheel for the visualizer, which depends on the main tt-metal wheel.


Expose useEnvData with no overloads

Is your feature request related to a problem?
Initially, envData (formerly userContext), only had a couple of properties. Now the properties have grown, and they can really be very useful.
The problem is that currently useEnvData doesn't follow the pattern of the other use methods, and only one property can be accessed at a time:
const url = useEnvData('url');
const requestHeaders = useEnvData('requestHeaders');
[...]

Describe the solution you'd like
As a developer, I'd like to have a function even without overloads that returns the envData object, so that destructuring is also possible:
const envData = useEnvData():
const { url, requestHeaders } = useEnvData();

Describe alternatives you've considered
I don't know other alternatives.
Additional context
@manucorporat If you like the proposal I can do a pr. Thanks

i am a little worried of developers using this API haha cuz it's very low level and it will not serialize data, ie it's server only...
I am not against this change, even thinking if we should rename this API to:
useServerEnvData()
and deprecate the current API.
The value will not be persisted in client side...

cc @mhevery @adamdbradley

@manucorporat This API is important precisely because it is available only server side.
It provides information such as: current url, headers, cookies, locale, Qwik City env data, and custom data from onGet. This data is strategic in several cases:

If Qwik City is not used
Or before Qwik City is initialized
Or before any initialization or rendering: such as the url origin to build absolute urls server-side, accept-languages for localization, technical cookies to comply with the cookie consent, multitenant apps etc.

The only alternative to this API is to get the data in adaptors (for example express) and then pass it to the root component. But this requires extra work and is different for each adaptor.
I could do a pr that renames it and documents it too.

@manucorporat agree that adding "server" is better than a more generic name
thanks @robisim74 ! I say, unless anyone else thinks of a better name in the next few days - go for what Manu suggested - userServerEnvData and open that PR

yes, under my point of view it's still a low level api, of course, very useful, but easily wrappable by higher level APIs, qwikcity developers should never have to use this API ever in their app, library authors sure!

@manucorporat is there another way to access the current url, headers, cookies in root component or in external libraries that don't depend on Qwik City?


Delivery Service Request "originals"

What does this PR (Pull Request) do?

[x] This PR is not related to any Issue.

TThis PR overhauls how DSRs work (in APIv4). Instead of a single deliveryservice property, DSRs will now have requested and original.
requested is the Delivery Service as it is requested to exist. For delete-type DSRs, this property will not exist in responses and will be ignored in requests. In most senses, it's pretty analogous to the old deliveryservice property.
original is the Delivery Service as it exists/existed before the DSR is/was/would have been applied. This property does not exist on create-type DSRs.  For non-create DSRs that are "open", this is either given in the request (for delete DSRs) or generated by Traffic Ops for responses from the current DS that matches the DSR's requested's XMLID (for update DSRs). When a DSR is "closed", the original is saved from the current DS so that even if that DS changes in the future, the actual change enacted by the now-closed DSR is preserved.
Also, a couple more changes: Assignee ID, Author ID, and Last Edited By ID are no longer present on objects returned by the API.
For a (hopefully) complete explanation of the new model and interface, refer to the PR's docs changes.
Finally, this makes TP use the stable API 3.1 endpoints for DSR-related operations, to avoid shoving TP work into this PR.
Which Traffic Control components are affected by this PR?

Documentation
Traffic Ops
Traffic Portal

What is the best way to verify this PR?
Make sure the tests pass, make sure working with DSRs in Traffic Portal is unaffected.
The following criteria are ALL met by this PR

[x] This PR includes tests
[x] This PR includes documentation
[x] This PR includes an update to CHANGELOG.md
[x] This PR includes any and all required license headers
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY

getting error on migration @ocket8888
goose: migrating db environment 'development', current version:<PHONE_NUMBER>000000, target:<PHONE_NUMBER>000000
2021/03/26 13:06:34 FAIL<PHONE_NUMBER>000000_dsrs_originals.sql (pq: check constraint "appropriate_requested_and_original_for_change_type" is violated by some row), quitting migration.
Can't run goose: exit status 1

For update requests that change the XMLID, to which XMLID does the "top-level" XMLID property refer?

well, you can't ever change xmlId

Oh yeah, you can't do that I forgot

This no longer works:

GET /deliveryservice_requests?xmlId=foo&status=submitted

It was a pretty handy way to see if a ds had "open" dsrs... and TP leverages that query param to "auto-complete" DSRs.

Which part of that no longer works? I can filter by both xmlId and status. I tested this just now by:

created an "update" DSR for demo1 in "draft" status
requested deliveryservice_requests?xmlId=demo1 - returns that DSR
requested deliveryservice_requests?xmlId=demo2 - returns an empty array
requested deliveryservice_requests?xmlId=demo1&status=submitted - returns an empty array
update the status to "submitted" (which succeeds but returns a 500, for some reason I still have to look into)
requested deliveryservice_requests?xmlId=demo1 - returns that DSR
requested deliveryservice_requests?xmlId=demo2 - returns an empty array
requested deliveryservice_requests?xmlId=demo1&status=submitted - returns that DSR

So it's all working fine from what I can see. Plus I'm pretty confident that the API tests exercise at least the XMLID param.

updating an update DSR - fails

I believe I have fixed this, testing updating a "delete" DSR now

Yeah, updating "delete" DSRs is working now, too.

looking pretty good so far but i hit this potential bug:
i fulfilled a 'delete' request which
1. set the DSR to pending and

2. deletes the DS

but then i tried to set the DSR to 'complete' and got this:
ERROR: api.go:240: 2021-04-09T16:32:32.924133Z: <IP_ADDRESS>:56897 expected exactly one DS with XMLID 'dns-345346546', found: 0


Link reference definitions indented by spaces not recognized

If link reference definitions are indented by spaces, then only the first link reference definition is recognized (without leading spaces, it works).


Ensuring all struct fields have sizeof > 0.

Fixes #6735.

Two lit test failures on the Linux CI: https://source.cloud.google.com/results/invocations/09221f62-4379-43b2-8cfe-1e1850e034ca/targets/iree%2Fgcp_ubuntu%2Fcmake%2Flinux%2Fx86-swiftshader%2Fpresubmit/log
/tmpfs/src/github/iree/iree/compiler/Dialect/VM/Target/C/test/constant_ops.mlir:12:17: error: CHECK-NEXT: expected string not found in input
 // CHECK-NEXT: uint8_t rwdata[0];
                ^

/tmpfs/src/github/iree/iree/compiler/Dialect/VM/Target/C/test/global_ops.mlir:8:17: error: CHECK-NEXT: expected string not found in input
 // CHECK-NEXT: iree_vm_ref_t refs[0];
                ^


How to use bazel to build a module?

Such as : I want to modify the code in planning model, then I want to build this planning model to debug?
How to build this planning model only?
or how to debug?
Any steps or specs links? thank you very much!

Such as : I want to modify the code in planning model, then I want to build this planning model to debug?
How to build this planning model only?
or how to debug?
Any steps or specs links? thank you very much!
Just run

bazel build -c gdb modules/planning/...

where -c dbg means debug mode, see bazel compilation mode.


Object model

We need a chapter that outlines the object model the VM will use, what the trade-offs are, etc, etc.

I'll try to sum up a few different models. Please correct me where i'm wrong ;)

Prototype based: Used by Self, Javascript etc. An object is a hashtable with Strings as keys. In the common case, objects are created by cloning other objects. Code reuse is mostly done by runtime modification and methods are stored as values.
Pro: It's easy to monkeypatch, easy to implement, easy to understand, flexible
Contra: It's easy to monkeypatch ;), memory overhead (objects of the same kind don't share information about their structure), runtime overhead (every method call is a hashtable lookup), no type safety of any kind, since all objects have the same type (the flipside of the mentioned flexibility)
The python approach: Basically the same, but with classes on top. (??? I'm not really sure how this actually works and what is specified and what is just CPython implementation). From what i understand: Objects behave like dictionaries, but have special attributes for type information and there are mechanisms for inheritance etc.
Honorable mention: Lua. Dictionaries (named tables) are the only data structure in the language and there are different utilities for OO programming build on top as libs. A unique property is that every table can have a meta-table. If a lookup fails, the language tries to find the key in the metatable and so on. This lends itself to inheritance like mechanisms. Also the language dispatches operations like +, *, etc to special keys.
Records, used by closure and i think in the common lisp object model CLOS (don't know that on very well): Structural information between objects are shared and methods are defined outside of the record.
Pro: Allows multiple dispatch (if thats desired), decoupling of methods and data can be a solution for the expression problem.

All of these approaches are not big on information hiding.
Also:

The Classic, a struct that contains a pointer to a Virtual Function Table. Used by C++, Java. Maybe not that interesting for a dynamic language.
to be continued...

In general i'd like to see inheritance de-emphasised or completely left out. Another aspect that we can discuss is the addition of a trait system. There is a little bit of overlap with the described record approach that allows open method definition.

@madmalik

no type safety of any kind, since all objects have the same type (the flipside of the mentioned flexibility)

I disagree, type safety has little to do with prototype OO. See TypeScript for example, and Inko uses prototype OO and the language offers type safety (through gradual typing).
One of the big benefits of prototype OO is that it's super easy to model other object systems (class based OO for example) on top of it, while the inverse is not necessarily true. I personally also found it a bit easier to wrap my head around.

@YorickPeterse

I disagree, type safety has little to do with prototype OO

Your are correct, i worded that wrong. What i meant is that the structure is more... amorphous? But I'll take a look into how typescript handles that (and of course Inko ;) ).

@madmalik Yeah, the underlying data structures are definitely less specialised. For example, for Inko it's (more or less) just this:
struct Object {
  prototype: *mut Object,
  attributes: Box<HashMap<*mut Object, *mut Object>>,
  value: ObjectValue
}

enum ObjectValue {
  Integer(i64),
  Float(f64),
  String(Box<String>),
  ...
}


Add Doxygen CI job and enforce Doxygen warnings

Subject of the issue
Doxygen has some helpful warnings that we’re currently ignoring. We ought to elevator those warnings to errors then add a CI job to build the docs to prevent regressions. For example these warnings caught a docs bug for an sf::Context constructor that remained unnoticed for a year or more.

Here is where I generate docs for the sake of catching warnings: https://github.com/ChrisThrasher/argon/blob/fa7b4b75d49eefb1cbeb7bc236dbdcc14e80e407/.github/workflows/ci.yml#L132-L147
Here is where I generate docs for the sake of deploying to GitHub Pages: https://github.com/ChrisThrasher/argon/blob/fa7b4b75d49eefb1cbeb7bc236dbdcc14e80e407/.github/workflows/cd.yml#L15-L44
In our case we don't care about GitHub pages per se but between these two jobs we should be able to figure out the best way to generate docs and create a release artifact from them so we can easily deploy them to the website.


fix: always respect file importance order

Solves the following issue:
Steps to reproduce
$ cat Gemfile
# frozen_string_literal: true

source "https://rubygems.org"

ruby File.read("#{File.dirname(__FILE__)}/.ruby-version")

gem "pry"

group :development, :test do
  gem "dotenv"
end
$ cat Gemfile.lock
GEM
  remote: https://rubygems.org/
  specs:
    coderay (1.1.3)
    dotenv (2.8.1)
    method_source (1.0.0)
    pry (0.13.1)
      coderay (~> 1.1)
      method_source (~> 1.0)

PLATFORMS
  ruby

DEPENDENCIES
  dotenv
  pry

RUBY VERSION
   ruby 3.1.2p20

BUNDLED WITH
   2.3.16

$ cat bin/scripts/console
#!/usr/bin/env ruby
# frozen_string_literal: true

require "pry"
Pry.start
$ cat .env.important
EXAMPLE_VAR=5678
$ cat .env.most.important
EXAMPLE_VAR=1234
$ bundle exec dotenv -o -f ".env.most.important,.env.important" ./bin/scripts/console
[1] pry(main)> ENV["EXAMPLE_VAR"]
=> "5678"
[2] pry(main)> exit
$ bundle exec dotenv -o -f ".env.important,.env.most.important" ./bin/scripts/console
[1] pry(main)> ENV["EXAMPLE_VAR"]
=> "1234"

Expected behavior
The environment variables in .env.most.important take precedence over those in .env.important.
$ bundle exec dotenv -o -f ".env.most.important,.env.important" ./bin/scripts/console
[1] pry(main)> ENV["EXAMPLE_VAR"]
=> "1234"

Quoting from the README.md file:

The dotenv executable also accepts a single flag, -f. Its value should be a comma-separated list of configuration files, in the order of most important to least. All of the files must exist. There must be a space between the flag and its value.
$ dotenv -f ".env.local,.env" ./script.rb

Actual behavior
The environment variables in .env.most.important are overloaded by those in .env.important, i.e. the file priority policy is violated.
$ bundle exec dotenv -o -f ".env.most.important,.env.important" ./bin/scripts/console
[1] pry(main)> ENV["EXAMPLE_VAR"]
=> "5678"

System configuration
dotenv version: 2.8.1
Ruby version: 3.1.2
Additional info
As thoroughly described in issue https://github.com/bkeepers/dotenv/issues/424, the overload option violates what seems to be the official, documented file priority policy. A patch for Rails was offered in PR https://github.com/bkeepers/dotenv/pull/453, while these changes attempt to target the root cause, including updating the behavior of the CLI.
Some notes:

This is a breaking change.
The documentation and some of the tests appear to contradict each other. It is possible that me (and other affected parties) have misunderstood how the overload option is meant to function. Hence, an alternative solution is to update the documentation where applicable so that it is less ambiguous. Specifically, it would be good to address how the -f and -o flags work in tandem.
Suggestions for changes/other solutions are very welcome.

Would love to see this get merged soon as we spent several hours the last couple days trying to figure out why things weren't loading as we expected with the overload flag on.

would like to see this merged too, seeing the same issue locally: I have two files .env and .env.development.local and I'm getting the .env values loaded locally (they are intended to be used in production).
took me a couple of hours to find this 😞

@eriklovmo thanks for the pull request, and sorry it took so long to review and merge.
I'm going work on a few other breaking changes and will release this in 3.0 soon.


Develop -> Main

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