add travis badge, should trigger travis builds

travis.yml → .travis.yml

nice catch :D

There was a test failure on 1.7.27. Used Numeric#negative? which is not available on that Ruby version.


Add Scratch Dir

This add functionality for Scratch Folder.
Hope this time i made a merge and pull correctly.

might be a formatting tool or option you are doing, but it shows TONS of line changes and I can't verify what you are doing with this PR. I can't accept this.


Add swedish language

I have added the translations for the Swedish language, which should automatically enable if user has enabled SwedishSE as language in NamelessMC

Thank you!


Bring in 4D hybrid capability

This is a place holder for any discussions for how to go about hooking the 4D hybrid capability.
There is some undoing inside the code to be able to handle 4d from the saber level. The implementation should simply look like what is presently done for 3d but with an index (or time stamp) passed down from saber for GSI to know which time slot is to be handled.

Closing this issue since this has been resolved in JEDI.


Adds GPT-4 Turbo preview model

@qianc62 This may have broken something.
I'm not quite sure, but I'm unable to run it successfully with the incremental mode yet.


Add building instructions to README.md

Add pre-requisites and also instructions on how to use Arduino IDE to build and upload the sketch.

Thank you!


firebase cloud messaging Error

FAILURE: Build failed with an exception.

What went wrong:
Could not resolve all files for configuration ':firebase_messaging:lintClassPath'.

Could not download intellij-core.jar (com.android.tools.external.com-intellij:intellij-core:26.3.0)
Could not get resource 'https://dl.google.com/dl/android/maven2/com/android/tools/external/com-intellij/intellij-core/26.3.0/intellij-core-26.3.0.jar'.
Premature end of Content-Length delimited message body (expected: 14623712; received: 4456416
Could not download kotlin-compiler.jar (com.android.tools.external.com-intellij:kotlin-compiler:26.3.0)
Could not get resource 'https://dl.google.com/dl/android/maven2/com/android/tools/external/com-intellij/kotlin-compiler/26.3.0/kotlin-compiler-26.3.0.jar'.
Premature end of Content-Length delimited message body (expected: 21121529; received: 5504992

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 17m 19s
Finished with error: Gradle task assembleDebug failed with exit code 1
firebase_messaging: ^4.0.0+2

how do you fix this ?


crypto/tls: incompatibility with Trust Settings in CA certificate

What version of Go are you using (go version)?

$ go version
go version go1.12.4 linux/amd64

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
golang:latest @ docker
go env Output
$ go env
GOARCH="amd64"
GOBIN=""
GOCACHE="/root/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/go"
GOPROXY=""
GORACE=""
GOROOT="/usr/local/go"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build114266566=/tmp/go-build -gno-record-gcc-switches"

What did you do?
Standard crypto/tls library is incompatible with CA certificates (either root or intermediate one) when they contain Trust Settings (https://www.openssl.org/docs/manmaster/man1/x509.html#TRUST_SETTINGS):

$ openssl -in rootCA.crt -text -noout
...
No Trusted Uses.
No Rejected Uses.
Alias: XXX
...

Since widely used MS AD-based CAs DO have such Settings in their certs, it is hard to use encrypted connections in golang programms in such environments.
What did you expect to see?
The successfull TLS-handshake.
What did you see instead?
In our case server's cert is issued by intermediate CA which is issued by root CA and server.crt contains server's cert followed by the intermediate CA's one.
In the case when the Trust Settings are present in the intermediate's cert, tls-negotiation fails with the following errors:
client.go:40: tls: failed to parse certificate from server: asn1: syntax error: trailing data
server.go:55: remote error: tls: bad certificate
If only the root's cert contains the Settings, the errors are slightly different:
client.go:40: x509: certificate signed by unknown authority
server.go:55: remote error: tls: bad certificate
(Example client&server apps are available at https://github.com/kruftik/go-tls-vs-windows-ad-ca, build.sh prepares binaries)
Once the Settings are being stripped from both root and intermedia CA certs with the openssl x509 -in rootCA.crt -out rootCA.clean.crt command, the problem is going away.

@FiloSottile

@andybons @dmitshur hi! Any news? :)

@katiehockman

Experiencing similar issue


Refactor IndexPipelineId and MergePipelineId

My hunch...
We  don't need have the pipeline_ord on split metadata. It has become meaningless for merged pipelines, and the
debugging value is limited.
We can probably ditch the indexpipelineid / merge pipeline id distinction, and just sparkle pipeline_ord on the rare place where it is needed.

Fixed in #2214


There are some bad links in doc.

File: /docs/en_US/rules/overview.md

@CrazyWisdom Could you help to take a look? Thanks.

@CrazyWisdom Could you help to take a look? Thanks.

Ok.

@CheolhoJeon Thanks, It will be fixed in the next version update.


Change retrieve function in production locations API

Changed the endpoint /v1/production-locations/{os_id} to return a single object instead of list of items with the count.

@protsack-stephan Good to me, but do we need to update RELEASE NOTES?


Not consistent method parameter names

Context / Scenario
All extension methods for Text-/TextEmbeddingGeneration are using the parameter ITextTokenizer? textTokenizer. The WithSemanticKernelTextGenerationService and the WithSemanticKernelTextEmbeddingGenerationService methods are using instead the ITextTokenizer? tokenizer parameter. For consistency it should be considered to rename the parameter.
If wished I could provide a PR for this.
What happened?
nothing
Importance
edge case
Platform, Language, Versions
all
Relevant log output
No response

PR is available #707


losing last column of nodes table

@anotherjoshsmith @richford Whatever is the final column in the nodes.csv table is not showing up in the "Plot Type" drop down menu. In other words, we are somehow losing one of the values (in the case of my current data it is md, but it seems to be whatever is the last column in nodes.csv)

Oops, falling back on my python instincts for slicing! In javascript array.slice(3,-1) slices to the second to last element whereas array.slice(3) slices through the end. I'll fix it now!

I'll take a look at the PR once you submit it
Jason D. Yeatman, PhD
Assistant Professor, Institute for Learning & Brain Sciences (I-LABS)
Department of Speech & Hearing Sciences
University of Washington
http://brainandeducation.com
On Wed, Feb 15, 2017 at 11:42 AM, Josh Smith<EMAIL_ADDRESS>wrote:

Oops, falling back on my python instincts for slicing! In javascript
array.slice(3,-1) slices to the second to last element whereas
array.slice(3) slices through the end. I'll fix it now!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/yeatmanlab/AFQ-Browser/issues/119#issuecomment-280116941,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABsYytBUieiK19yCYgioXInCAAyTRxECks5rc1U6gaJpZM4MCEXM
.

Closed through #121


How to readQuery writeQuery without this.$apollo.mutate({})

I'd like to read/write directly to client cache.(store)
Can this be done with vue-apollo without using this.$apollo.mutate ?
https://www.apollographql.com/docs/react/features/caching.html

You can try this.$apolloProvider property like this.$apolloProvider.defaultClient.readQuery({})


Increase the base of mute shot from 8h to 24h

Now:      8-16-24-32-40-48
Proposal: 24-48-72-96-120-144

So, @getjump just added cleanup feature, and at the same time, I have increase base time for mute. Now it is 16h


Sovrn accepts sizes param.

@nanek
[#120596721]

Let's merge this into a different branch besides master. master we can keep in sync with upstream repo without worrying about conflicts. dev, prod, custom, sd or something similar. Otherwise LGTM.


Any Request or Response Time View

Any option for detail timing, such as request sent, watting, content download?

There is a feature for this in mitmproxy 0.12:

As an aside, we need to improve the way that timing is displayed there - timestamp for connection established, and then time elapsed from there for everything else.

Yes, having something similar to what mitmweb has would be a good contribution.

Looking forward to the new version, but I have modified the code and display the request time.


separate CSS?

Love the simplicity of single-file, but feel that the CSS should be separate SASS/SCSS file. Are you open to a PR?

Of course I am!
My current problem is that I've to define a contribution guidelines but feel free to open it whenever you want 👍


react-redux: option to require that all props are injected

Take this code (try flow link):
// @flow
import React, { type ComponentType, type ElementConfig } from 'react';

declare type MapStateToProps<RS: Object, SP: Object, RSP: Object> = (state: RS, props: SP) => RSP;
declare export function connect<Com: ComponentType<*>, RS: Object, DP: Object, RSP: Object>(
  mapStateToProps: MapStateToProps<RS, DP, RSP>,
  mapDispatchToProps?: null
): (component: Com) => ComponentType<$Diff<ElementConfig<Com>, RSP> & DP>;

function Post(props: { title: string, body: string }) {
  return (

{props.title}

  );
}

type Posts = {
  [number]: {
    title: string,
    body: string
  }
}

type OwnProps = {
  postId: number
}

function mapStateToProps(state: Posts, props: OwnProps) {
  const post = state[props.postId];
  return {
    title: post.title
  };
}

const ConnectedPost = connect(mapStateToProps)(Post);

// $ExpectError
<ConnectedPost postId={1} />;

The resulting ConnectedPost is a component that accepts/requires body as a prop. But what if you wanted to make sure that all the required props in Post were provided by the connect call? This is a common pattern at the top-level of the component tree, such as components created by react-router. I would think that if I declared OwnProps to be an exact object, it would require every prop to be injected (try flow link)
// @flow
import React, { type ComponentType, type ElementConfig } from 'react';

declare type MapStateToProps<RS: Object, SP: Object, RSP: Object> = (state: RS, props: SP) => RSP;
declare export function connect<Com: ComponentType<*>, RS: Object, DP: Object, RSP: Object>(
  mapStateToProps: MapStateToProps<RS, DP, RSP>,
  mapDispatchToProps?: null
): (component: Com) => ComponentType<$Diff<ElementConfig<Com>, RSP> & DP>;

function Post(props: { title: string, body: string }) {
  return (

{props.title}

  );
}

type Posts = {
  [number]: {
    title: string,
    body: string
  }
}

type OwnProps = {| postId: number |}

function mapStateToProps(state: Posts, props: OwnProps) {
  const post = state[props.postId];
  return {
    title: post.title
  };
}

// $ExpectError since the only prop that `ConnectedPost` should accept is `postId`,
// every other prop should be expected to be provided by `mapStateToProps` or
// `mapDispatchToProps`.
const ConnectedPost = connect(mapStateToProps)(Post);

Is something like this even possible?

@villesau

@rickharris You might want to check out how mergeProps works. That's more like react-redux-way of defining what the component will get from props, mapStateToProps and mapDispatchToProps. see the documentation. Would that work for you? Other than that I don't think libdefs should have way of enforcing there is no such functionality in react-redux either. That would be a job for some sort of linter which I don't think exist.

Thanks for the response! I don't think mergeProps would help here, but I could be wrong. I don't need to actually use mergeProps for any functionality, so unless it unlocked some ability in flow I don't think it would help.
Have you come across this situation with react-router before, where you have <Route component={ConnectedComponent} />, and the react-redux libdef is unable to verify that every required prop is being passed in, since the call site of the connected component is inside of react-router? This is a use case that I would think is pretty common (people using redux, react-router, and flow together) that would be really useful if there was a way to support it. I'm fairly competent in flow, although I'm not sure if I'm competent enough to take this on. Maybe I'll give it a shot sometime.
I think I have an alternate solution though - instead of providing a connected component directly to a route in react-redux, I could use an intermediate component that translates the props passed to by the router and creates a connected component so that the call-site of the connected component is outside of react-router. Example here.

If you had a test that used the component without passing in extra props, wouldn't that validate it?


[WelcomeUX] Clicking relative link doesn't update scrollbar

Testing #18091
Scrollbar is still at the top when it should be in the middle

Tracked with #18864


Add Pusher Notifications for Upload Service

@jwagenaar These changes are not working for me locally - I get the following error

Previously had merged these changes and ran into issues in dev environment and had to revert

What might be going wrong here? Thanks!


release: v1.5.8

[1.5.8] (May 17 2024)
Fix:

Fixed a bug where styles were broken on some websites

/bot create ticket

[Creating Ticket] 🔖 Done https://sendbird.atlassian.net/browse/SDKRLSD-1284


Child page not properly updating when path updates

yew_router_broken.zip
Describe the Bug
When part of the path is passed to a different module and handled there, it seems that path updates are not properly updating the DOM tree.
To Reproduce

Match part of the path in a parent object and part of it in an object that is passed to the child. Also pass a callback to the child which enables it to change the path using  tags or buttons etc.

Change the path from the child element in a way that should cause the child to render differently or update, ie. send the callback to the parent to change the path and propagate the changes down to the child.

The child page should not render differently

PLEASE REFER TO ATTACHED MINIMAL EXAMPLE, THIS SHOULD RUN STRAIGHT FROM THE FOLDER USING THE INCLUDED PYTHON SERVER.

To demonstrate the bug here, unzip the project
use wasm-pack build --target web from the root of the unpacked folder
serve with python3 bad_server.py
navigate a browser to localhost:8000 - you should see a very basic website saying:

Blog
Back to Blog

Clicking Blog should take you to the fake "blog" section (notice the URL changing to /Blog) expected. This should then display:

Blog
GOTO A BLOG PAGE

Clicking GOTO A BLOG PAGE should take you to the URL /Blog/post/TEST_A
This should display different text on the page, however it doesnt.
Reloading fixes the lack of updating and will show the correct text:

Blog
Back to Blog

Once again, the Back to Blog button takes you back to /Blog, however the rendering doesn't update and the old page remains until you reload.

Expected Behavior
The child should render again based on the updated path information
Related Info

rustc version:  rustc 1.43.0-nightly (6d69caba1 2020-02-27)
Browsers versions: Firefox 74.0
yew version: yew = { version = "0.13", features = ["web_sys"] }
yew-router version: yew-router = { version = "0.10", features = ["web_sys"]}
cargo-web version, or other build toolchain: wasm-pack 0.9.1

You are not updating self.route
match &self.route {
  BlogRoute::Root => self.render_root(),
  BlogRoute::Post(post) => self.render_post(),
}

You want to render render_post but I cannot find any self.route =  you are not changing self.route, you are sending messages to parent which will set route property correctly.
But you do not update self.route on property changes.
fn change(&mut self, props: Self::Properties) -> ShouldRender {
    self.route = props.route;
    true
}

@fbucek Thank you for your advice. I am trying to modify the provided example to make this work but I cant seem to get it working. Also I'm not sure what you mean by fn change - when should this be called?

Thanks so much, this helped a lot


Android Speech to Text Error - (SPXERR_START_RECOGNIZING_INVALID_STATE_TRANSITION)

Describe the bug
I am using the SDK on a Xamarin forms application.
On most devices, the SDK works without issue.
However, on some devices, the following error occurs when trying to start the recognition service.
I was hoping to get more information about the below error code and exception.
To Reproduce
Steps to reproduce the behaviour:

Setup project as per sample project provided by Xamarin Here
The exception is triggered when StartContinuousRecognitionAsync is invoked

Version of the Cognitive Services Speech SDK

v1.12.1

Platform, Operating System, and Programming Language

OS: Android 9+
Hardware - ARM, example device - Galaxy xcover s4
Programming language: C#

Additional context
Message
Exception with an error code: 0x13 (SPXERR_START_RECOGNIZING_INVALID_STATE_TRANSITION)
Stacktrace
at Microsoft.CognitiveServices.Speech.Internal.SpxExceptionThrower.ThrowIfFail (System.IntPtr hr) [0x0005d] in :0 \n  at Microsoft.CognitiveServices.Speech.Recognizer.StartContinuousRecognition () [0x0005c] in :0 \n
at Microsoft.CognitiveServices.Speech.Recognizer.DoAsyncRecognitionAction (System.Action recoImplAction) [0x0005d] in :0 \n  at Microsoft.CognitiveServices.Speech.SpeechRecognizer.b__44_0 () [0x00001] in :0 \n
at System.Threading.Tasks.Task.InnerInvoke () [0x0000f] in /Users/builder/jenkins/workspace/archive-mono/2019-10/android/release/external/corert/src/System.Private.CoreLib/src/System/Threading/Tasks/Task.cs:2476 \n  at System.Threading.Tasks.Task.Execute () [0x00000] in /Users/builder/jenkins/workspace/archive-mono/2019-10/android/release/external/corert/src/System.Private.CoreLib/src/System/Threading/Tasks/Task.cs:2319

@Jhanbury Thanks for the feedback, this problem may occur e.g. if microphone/input stream gets closed before stopping the recognition, we will address that issue in future releases.
Are you using the sample as you referred directly which use the default configuration from microphone? If so, we need to investigate the specific devices which repro the issue, are you able to reproduce the issue with Galaxy xcover s4 consistently ? Could you provide logs from the speech SDK using instructions below ?
https://docs.microsoft.com/en-us/azure/cognitive-services/speech-service/how-to-use-logging

@jhakulin I will send on the SDK logs you mentioned once I get access to the device.
I have been able to run the code without issue on a similar device (Galaxy xcover 4).
This would lead me to believe that there is an issue with the device rather than an SDK issue for those devices.
I will know more once I have the SDK logs, thanks again for your help

Thank you @Jhanbury!
I am going to close this issue for now. When you attach additional information, please reopen this.
We've added this bug to our backlog, and as we make changes to the underlying functionality that will address this, we will update this issue with a reference to the release that will contain the fix.
Thank you for using the Speech SDK!


[BUG] New update broke icon sizes

Currently using 1.0.6 on Obsidian 0.13.14.
Some icons won't load (see image below) while the rest are at their native size.
Bug persists with all themes.

Is it related to issue #30 ?
In addition, is there any error in the console?

I confirm it's the same issue.
I solved it by uninstalling the pluging and reinslling again, yet I lost all icon settings as expected.

I will close it for now.


Block user script

Coverage decreased (-0.4%) to 85.858% when pulling d159cffd0dece678b641b0b3829b2eee0069942c on block-user-script into 8f0d1dde60afc432a1d43baef9ed21c1dfa58c2c on master.


Create repository with deployment variables

Background
Enable easy deployment of the service
Feature
Create repository with deployment variables

JFYI the repository has been created here: https://github.com/absa-group/atum-service-deployment
And we have another task for service deployment: #46. One more thing is remaining though before I close this ticket, and that is a self-hosted GH Runner for CICD. I asked Harry to do this and provided some basic instructions.

JFYI the self hosted runner is implemented here: https://github.com/absa-group/atum-service-deployment/pull/3


Customer feedback | az afd endpoint | need JSON example for --domains parameter

Describe the bug
Feedback category
Need more information
Reference command
az afd endpoint purge --domains
Summary
From Learn Customer Feedback, a customer is requesting more information on the --domains parameter. The sentence Support shorthand-syntax, json-file and yaml-file. Try "??" to show more. appears over 5,337 times in our autogenerated content.  The lack of a JSON or shorthand-syntax example generates recurring customer complaints.  See GitHub issue 29479.
Request

Add an example that includes multiple domain values for the --domains parameter

Doc link
https://learn.microsoft.com/en-us/cli/azure/afd/endpoint
Customer feedback
"
Please add information on how to add multiple domains to the '--domains' parameter.
"
Related command
az afd endpoint purge
Errors
n/a
Issue script & Debug output
n/a
Expected behavior
n/a
Environment Summary
Azure CLI 2.63.0
Additional context
No response

Thank you for opening this issue, we will look into it.


strange schema.org/location error output

Every time the linter runs into markup containing statements about a schema.org/Person's homeLocation or workLocation it generates an error message:
property schema:location: Subject <http://example.org/#WileECoyote> not compatible with domainIncludes (schema:Action,schema:Event,schema:InteractionCounter,schema:Organization)

Somehow the location property (parent property of the others) gets shown in the output, which rightfully triggers the error message, though I am clueless as to why this property gets added. Is this a bug in the schema.org vocabulary or in the Linter?
What's extra confusing is that I only have seen it happen in regards to Person statements.
Markup example:
<script type="application/ld+json">
{
  "@context": "https://schema.org",
  "@id": "#WileECoyote",
  "@type": "Person",
  "name": "Wile E. Coyote",
  "workLocation": {
    "@id": "#FantasyLand",
    "@type": "Country",
    "name": "Fantasy land"
  }
}
</script>

Thanks for bringing up the issue. I think it is, indeed, a bug in the linter, but a challenging one to fix. Person is a valid range of workLocation, even though it is not of location. But, because of that way that subPropertyOf works, :a :workLocation :b. implies :a :location :b..
The definition of workLocation is:
:workLocation a rdf:Property ;
    rdfs:label "workLocation" ;
    :domainIncludes :Person ;
    :rangeIncludes :ContactPoint,
        :Place ;
    rdfs:comment "A contact location for a person's place of work." ;
    rdfs:subPropertyOf :location .

From RDFS, we can see via rdfs5:
```xxx rdfs:subPropertyOf yyy . yyy rdfs:subPropertyOf zzz . => xxx rdfs:subPropertyOf zzz .`
The fact that _location` does not have Person in its range is not an error, as RDFS doesn't have error conditions. The linter is inferring a problem given the inferred triples.
The fix is probably to filter out errors of this kind, but that's going to be hard to do.

Actually, the way to address it is probably to add a rule such as the following:
aaa rdfs:subPropertyOf bbb .
aaa schema:domainIncludes ddd .

=>

bbb schema:domainIncludes ddd .

Same thing for schema:rangeIncludes.

Thanks for the reaction @gkellogg.
I can imagine you have a ton of other things to do before tackling this. I'm already happy to know I wasn't seeing ghosts for once.
If you'll ever get around to fixing it it'd be great, if not, at least it's documented in this issue.

Just for the sakes of documenting the issue in case people are searching for it, here's another example of the same issue:
<script type="application/ld+json">
{
  "@context": "https://schema.org",
  "@id": "#ShortStory",
  "@type": ["DefinedTerm", "http://www.wikidata.org/entity/Q49084"],
  "termCode": "ShortStory",
  "url": "https://schema.org/ShortStory",
  "inDefinedTermSet": "https://schema.org",
  "sameAs": "https://en.wikipedia.org/wiki/Short_story"
}
</script>


GT-191: Enhance PR Labeler

GT-191

Fixed the issue where the Changes in staging label does not show up in all the affected PRs
Cleaned up the code
Add support for adding labels to multiple target branches

Closing due to inactivity


Parse time values in tables

@adammertel : can you answer above?

This probably relates to matching the still textual (non-Conceptified) temporal relations in coding sheets. One option is to Conceptify them, another is to devise a parsing method. Probably there is a lot of data here => automation better.


Fix Azure_IoT_Central_ESP32.ino

Weirdly in my IDE I could compile the sketch even with the wrong function signature. It must have resolved to some other function, given the common name "mqtt_event_handler".

Fixes https://github.com/Azure/azure-sdk-for-c/issues/2847

Ports and replaces https://github.com/Azure/azure-sdk-for-c-arduino/pull/60


lit-element doesn t allow return of only a slot element inside the render()

When only returning html <slot></slot> element in the render() function
the output in html gives only a empty shadow dom and all the elements are placed inside the light dom.
When adding a div element and a property to the returning html the shadow dom is filled as expected.
is this a bug or expected behaviour ?

Will you post this on CodePen or jsbin?

how can i use lit-element inside codepen i never used it before so if you could assist me that would be great

There are several hosted code sandboxes listed in the readme that you can use as a starting spot.
https://github.com/Polymer/lit-element#getting-started

Seems to work fine to me.
https://glitch.com/edit/#!/lit-element-base-slot?path=index.html

The  example provided by @jsilvermis is working fine. I don t get it I am doing the same thing but with an external file, could it be web pack and typescript? Another thing I would like to discuss your documentation is up to date but the application on npm isn't the version on npm needs _ before the render function, and the GitHub version without the _. this is the same inside the jsbin and codepen examples.

For now, you can install the latest version using:
npm install @polymer/lit-element@dev

which currently resolves to<EMAIL_ADDRESS>

Not exactly sure what you mean there, but if you mean toggling a boolean attribute, you can use the following syntax:
<some-el ?attr=${someBooleanOrExpression}></some-el>

or in the case of something like checked on a checkbox, you should set the boolean as a property to ensure proper updates, as in:
<input type="checkbox" .checked=${someBooleanOrExpression}>

If these aren't correct, please add an example of what you're trying to achieve.

<input ?readonly=${read_only}></input>

<input ?readonly=${read_only}></input>
this doesn't t work because of it not being a boolean. When the readonly property is set inside the input element it will always be readonly.
So if the boolean changes it will not change between readonly or writeable
I found the error with the question I asked
this was my code:
connectedCallback(){          console.log(${this.id})     };
 render(){         console.log("render called on Hbox");         return html<slot></slot>;          }
when I used the connectedCallback the render wasn t working properly. When I removed the connectedCallback everything worked fine. Is this expected behavior or not?

@MakleSoft I did already figure it out after I tried some more things but thank you anyway.
I will close this issue now.


add an expiration to the space member permissions

This PR changes the structure of the space permissions and adds an optional expiration date.
Now the permissions array holds one object per grant instead of holding one object per role like before.

The UI needs some changes now that the permissions structure is different. Also the datepicker for the expiration date is shown but the value is not sent.

I am fine with merging backend and frontend updates independently.

:boom: Acceptance test localApiTests-apiSpacesShares-ocis failed. Further test are cancelled...

I am fine with merging backend and frontend updates independently.

This would break the spaces members since listing won't work properly without the proper changes.
Required for this PR are:

https://github.com/cs3org/reva/pull/3628
https://github.com/owncloud/web/pull/8320

https://github.com/owncloud/owncloud-sdk/pull/1204

My point was, that it would be ok if the ocis master / web combination is broken for some hours.

@C0rby Web bump was successful #5507


Add dynamic reconfiguration implementation to the examples.

Thank's to this PR, there is now possible to setup dynamic reconfiguration!
There is an examples folder with fully functional code on how to use this library. But the new feature added is not there yet.
Create an example using the style of the other modules in there and call the function from the entry point, or somewhere else where appropriate.

Can I take this up ?

@hionnode Please do, much appreciated!


some improvements

Summary
I noticed some issues while doing perf test.
Documents # Lambda extension perf test doc
Testing
Tested with eu poc org.

I noticed in pusher.go file you are nilling out doneC, and before that you are writing a value to it. I am not sure about the behavior of the consumer of that channel since channels are reference types. According to docs, receiving on nil channel is blocks forever. I mean in pusher:
                        if doneC != nil {
				doneC <- struct{}{}
			}
			doneC = nil

and in main:
       select {
	case <-ctx.Done():
		log.Print("Invocation context is done")
		return
	case <-doneC:
		log.Print("Pusher is done")
		return
	}

doneC is a local variable defined on line 126


CFE error in type inference for ArgumentError.checkNotNull

See this dartpad example (reproduces at head) https://nullsafety.dartpad.dev/a4772f72bd4d0f58a04356efd0f43355.
It infers the T type to be the nullable variant of the arg instead of the non-nullable variant.
This is likely a result of the T? type specified for the first argument not playing nicely with an already nullable T.
cc @johnniwinther can you have somebody take a look at this?

@stefantsov is working on it. See #41447

Sounds good, closing this as a dupe then


[SPARK-49153] Increase Gradle JVM memory to 4g like Spark repo

What changes were proposed in this pull request?
This PR aims to increase Gradle JVM memory to 4g like Spark repo.

https://docs.gradle.org/current/userguide/performance.html#increase_the_heap_size

Why are the changes needed?
Unlike CI, in the local build, Gradle Daemon is reused. The compilation is unstable like the following because the default Gradle JVM memory is 512MB.
> Task :spark-operator:compileTestJava
Note: /Users/dongjoon/APACHE/spark-kubernetes-operator/spark-operator/src/test/java/org/apache/spark/k8s/operator/metrics/healthcheck/SentinelManagerTest.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
OpenJDK 64-Bit Server VM warning: CodeCache is full. Compiler has been disabled.
[1.372s][warning][codecache] CodeCache is full. Compiler has been disabled.
OpenJDK 64-Bit Server VM warning: Try increasing the code cache size using -XX:ReservedCodeCacheSize=
[1.372s][warning][codecache] Try increasing the code cache size using -XX:ReservedCodeCacheSize=
CodeCache: size=2944Kb used=2943Kb max_used=2943Kb free=0Kb
 bounds [0x0000000105004000, 0x00000001052e4000, 0x00000001052e4000]
 total_blobs=1102 nmethods=464 adapters=554
 compilation: disabled (not enough contiguous free space left)
              stopped_count=1, restarted_count=0
 full_count=1

Does this PR introduce any user-facing change?
No. This is a dev-only change.
How was this patch tested?
Check the JVM setting via jps locally.
Was this patch authored or co-authored using generative AI tooling?
No.

cc @viirya and @jiangzho

Looks good to me.

Thank you, @viirya !
Merged to main.


Add supported Timezone values to the schema in NTP class

Is your feature request related to a problem? Please describe.
BIGIP supports list of timezone values and those were populated for user in UI to select from that list. Whereas DO schema allows any value(string) user can enter. Because of this there is a chance of user making mistake by wrongly entering the timezone value.
Describe the solution you'd like
It would be great if supported(allowed) timezone values are defined in the schema and BIGIQ UI can show them as dropdown list. With this enhancement, can avoid user making mistakes.
Additional context

These are the supported timezone values in BIGIP 16
Supported Time zone values in BIGIP 16.docx

Thank you for your input and the list of allowed time zone values. I have added this to our internal product backlog as AUTOTOOL-1956.


Auto binding doesn't work with ES6-style components

When creating a component 'ES6-style' (e.g. class Card extends React.Component), event callbacks will not automatically bind scope.
In the following example, I had to replace
onPress={this.onPress}
with
onPress={this.onPress.bind(this)}.
Using React Native 0.3.11
class Card extends React.Component {
    constructor(properties) {
        super(properties);
        this.state = {
            paired: false,
            visible: false
        };
    }

   onPress() {
        var state = this.state;
        console.log("I am no longer null", state);
    }
    render() {
        return (
          <TouchableHighlight
            onPress={this.onPress.bind(this)}
            underlayColor="blue"
            activeOpacity={0.5}>
            <Text>Dummy Text</Text>
          </TouchableHighlight>
        );
    }
};

This is expected behavior. In ES7 you'll likely be able to use a property initializer to write
onPress = () => {
  // ...
}

which will autobind. This syntax is supported by Babel already in an experimental transform and hopefully will be supported in react-native sometime. You can continue to use React.createClass if you prefer its semantics.


Found possible security issue. How should I report it?

I found a possible security issue in this repository.
How would you like me to share it with you?
Best regards
Max

Thanks for opening this issue.
Let me discuss with @soumith first and then I'll get back to you.

Can you send me an e-mail to<EMAIL_ADDRESS>where you replace [my-github-username] with my user name?

Sure. I just sent you the email.

Discussed this on e-mail.


Update tutorials

Codecov Report

Merging #76 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #76   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           16        16
  Lines          239       239
=========================================
  Hits           239       239

Continue to review full report at Codecov.

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


Add DSL support for backreferences.

Allow capture and tryCapture to assign the captured value to a Reference, which can be used as a regex later in the scope.
Reference initialization creates a unique identifier. The compiler converts the identifier to an absolute backreference offset.

Example:
let regex = Regex {
  let a = Reference()
  let b = Reference()
  capture("abc", as: a)
  capture("def", as: b)
  a
  capture(b)
}

Idea for an alternative syntax:
let regex = Regex {
  let a = Reference<String>()
  let b = Reference<String>()
  capture("abc", into: a)
  capture("def", into: b)
  a
  capture(b)
}

@tevelee Good point.  We are going with using let statements.

@swift-ci please test Linux

@swift-ci please test Linux

@swift-ci please test Linux@swift-ci please test Linux

@swift-ci please test Linux

@swift-ci please test Linux

PTAL 58c6428. This adds support for subscripting a match result on a Reference to retrieve a capture. This is akin to named captures.
let a = Reference()
let result = regex.match {
  tryCapture(as: a) {
    oneOrMore(.digit)
  } transform: { Int($0) }
}
result[a] // => Any
result[a, as: Int.self] // => Int

@swift-ci please test Linux

To simplify the call site, I chose to precondition instead of returning optionals.

This is in direct contradiction to stdlib style for non-indexing subscripts. Is there any chance you can split the more controversial part into its own PR?

@swift-ci please test Linux

This is in direct contradiction to stdlib style for non-indexing subscripts. Is there any chance you can split the more controversial part into its own PR?

This is no longer the case as I've switched to typed references. PTAL.

@swift-ci please test Linux

Nonetheless, I don't want to delay longer to merge backreferences, so I will be merging my first commit to add a non-generic Reference without subscript.

https://github.com/apple/swift-experimental-string-processing/pull/148#issuecomment-1048288748
I reverted my second commit as the first one (basic backreference functionality) was approved. I will open a new PR to add string types and subscript, to avoid further delays on the basic functionality.


Update injecting-parameters.md

Purpose

There's no proper documentation that tells how database connection parameters can be configured in WSO2 EI. This document tells how parameters can be injected dynamically but regarding database the info is not in detail. Also, I found out that database parameters can be injected from configuration files as well, whereas this document says it can be done only using environment variables.

Goals

To give a detailed overview of injecting parameters to a database dynamically using configuration files and environment variables.

Approach

Did few POC's and tried injecting parameters dynamically for a database in WSO2 EI, using both configuration files and environment variables, in the same manner as given for other EI artifacts like Endpoint, Message Store etc. in this document. And it worked successfully.

Test environment

OS - Windows 64-bit
JDK- openjdk version "11.0.5"
WSO2 EI - 7.1.0
WSO2 Integration Studio - 8.0
MySQL 8.0

Changes done in the document

Removed lines at 143, 144 as I did a POC and tested injecting parameters to data service using a configuration file. It worked fine and database connectivity was established.
Made minor changes in lines 154 and 172.
Added additional content from line 187 to 218 to show the configuration using file in Data Service.
Added additional content from line 220 to 283 to show injecting parameters in DB Report and DB Lookup mediators. I did a POC to test this - configuring DB connection parameters from both configuration files and environment variables in DB Report and DB Lookup mediators and they worked successfully.

@DinithiDiaz Please update the other doc repo (including apim) branches too.


Question about fix for issue 2181

Recently we got below error after Redis M/S switchOver at Redisson 3.10.1:
org.redisson.client.RedisTimeoutException: Subscribe timeout: (9500ms). Increase 'subscriptionsPerConnection' and/or 'subscriptionConnectionPoolSize' parameters.
And i saw it was fixed at 3.11.2 by: https://github.com/redisson/redisson/issues/2181
I have one question about the fix: https://github.com/redisson/redisson/commit/12e81de984698d06b3dbc236bac00479ec7e3d61
reattachPubSub method was only called in MasterSlaveEntry.slaveDown. What will happen in Redis M/S switchover
in case we configure  "subscriptionMode: "MASTER""? To my understanding:
slaveDown should not impact subscription as we configured Mode to master.

reattachPubSub method was only called in MasterSlaveEntry.slaveDown

I'm afraid you're mistaken. Have a look closer. It's invoked in both cases: slave down and master down (failover)

@mrniko : Thanks, i was checking 3.12.2 code ,that was called only from "slave Down", now in latest it's from both.

@mrniko : I just wonder to confirm if this BUG still existing in 3.12.2 when configure "subscriptionMode: "MASTER""? From code i see fix was applied after 3.12.2(call reattachPubSub  when master down). If my understand is correct, could you help me to know this fix was from which release? Thanks a lot.

Well , i found one issue: https://github.com/redisson/redisson/issues/2825
Seems that fix was from 3.13.1.

I think yes. I would recommend you to update to 3.13.1 version or the latest version


[WIP] Add playbooks to set up the nat64 net and VM

TODOs:

README
Tests

As a pull request owner and reviewers, I checked that:

[ ] Appropriate testing is done and actually running
[ ] Appropriate documentation exists and/or is up-to-date:

[ ] README in the role
[ ] Content of the docs/source is reflecting the changes

recheck


Multi-GPU training for detector

Can we train the CNN detector using multiple GPUs? The available arguments cause the program to hang after loading the data and model. I'm using the following line to initiate the process.
python -m torch.distributed.launch --nproc_per_node=2  --use_env train_detector.py  --config-file config/e2e_faster_rcnn_R_50_FPN_1x_multiscale_noClasses.yaml DATASETS.TRAIN [\"grozi-train\"] DATASETS.TEST [\"grozi-val-all\"] INPUT.MIN_SIZE_TRAIN [480,600,768,960,1152,1344,1536] INPUT.MAX_SIZE_TRAIN 2048 INPUT.MIN_SIZE_TEST 960 INPUT.MAX_SIZE_TEST 1280 OUTPUT_DIR output/exp0000-R-50-noCl-grozi

Hi,this probably can done but will likely require adjusting the code. We've never tried training on multiple GPUs.

Okay. In my case, I have a 48GB GPU, but the training occupies only 8GB, thereby taking three days to train the detector. How were you able to do this in a shorter time?

But what is the processing load of your GPU? if it is low you can try increasing batch size.
Another idea: it might be fine to train for significantly fewer iterations, you just need to monitor the behaviour of the validation loss to stop the process early.

GPU runs at 100% capacity, but most memory is left idle. Increasing the batch size in the config file isn't reflected in the final parameters. Did you face this issue while experimenting?
For the second approach, the current code doesn't have a tfboard support. Should I monitor the logs instead?

GPU runs at 100% capacity, but most memory is left idle. Increasing the batch size in the config file isn't reflected in the final parameters. Did you face this issue while experimenting?

It sounds weird, changing train.batch_size and train.class_batch_size should  definitely change the training process - at least the GPU memory usage should go up.
However, if the GPU is already at 100% simply changing batch size is not likely to increase training speed.

For the second approach, the current code doesn't have a tfboard support. Should I monitor the logs instead?

Yes, the code does not have tensorboard but it includes another visualization tool: os2d/utils/plot_visdom.py

Got it, thanks!


Need a way to selectively disable component size budgets

🚀 Feature request
Command (mark with an x)

[ ] new
[X] build
[ ] serve
[ ] test
[ ] e2e
[ ] generate
[ ] add
[ ] update
[ ] lint
[ ] xi18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

Description
Currently the anyComponentStyle budget applies to all components.  However we have a couple of components that significantly exceed the budget.  We'd like to make an exception for those components, but still get warnings/errors if other components go over budget.
Describe the solution you'd like
A way to either specify an exception list to the anyComponentStyle budget, or a way to set a per component budget that would override the anyComponentStyle budget for the specified component.
Describe alternatives you've considered
We've upped our budget, but the components in question are considerably larger than our average, so the budget is now pretty much useless.

I agree. Our component styles are usually very small. However, we have one component that wraps a third-party component which comes along with its own CSS. I could of course include those styles in my styles.css oder via the styles property in my angular.json. However, with those solutions, the integration of the third-party component would spread across the project instead of being encapsulated by the one component.

I'm facing the same issue, and also I'm using this because angular universal because I can''t found a way to defer styles on load with preload, prefetch or preconnect...

Is there already a discussion on how we can implement this feature? My idea is to add a property like ignorePattern in the budgets section of angular.json. This property accepts an array of regex patterns which components should be ignored. What do you think about this?

@markuscodes1 , I do not think the pattern is a suitable approach because, patterns are usually applicable for a group of things and people usually want to name the component styles that should be excluded. These component styles are usually do not belong to a group. So, a property ignoreComponents  which takes an array of paths referring to the component that has to be excluded would make more sense.

@markuscodes1 , I do not think the pattern is a suitable approach because, patterns are usually applicable for a group of things and people usually want to name the component styles that should be excluded. These component styles are usually do not belong to a group. So, a property ignoreComponents  which takes an array of paths referring to the component that has to be excluded would make more sense.

HI all,
Let's continue tracking this issue https://github.com/angular/angular-cli/issues/13393
Thanks.


Schedule nightly checks in CI

We would benefit from running some long-running checks nightly in CI.
In particular, the CI jobs triggered on PRs only build for selected boards; it would be useful to build for every board every night using laze --verbose build --global.
Additionally, build failures should be reported on our Matrix channel.
Ideally, in case of failure, that scheduled CI job would also bisect the commit that introduced breakage and open an issue about it, assigning the original committer.

Ideally, in case of failure, that scheduled CI job would also bisect the commit that introduced breakage and open an issue about it, assigning the original committer.

ah, should we keep this issue open for the bisecting part? TBH, I'd rather move this into a generic "nice to have" tracking issue.


In-place transform fix

🚀 🚀 Pull Request
Checklist:

[ ]  My code follows the style guidelines of this project and the Contributing document
[ ]  I have commented my code, particularly in hard-to-understand areas
[ ]  I have kept the coverage-rate up
[ ]  I have performed a self-review of my own code and resolved any problems
[ ]  I have checked to ensure there aren't any other open Pull Requests for the same change
[ ]  I have described and made corresponding changes to the relevant documentation
[ ]  New and existing unit tests pass locally with my changes

Changes

Could you elaborate on what the problem with transforms was and how this fixes things?

There were issues with encoder ndim being wrong when it is empty (which was merged in fast merge pr) and transforms ignoring enabled tensor attribute.


How to set custom template URL

Hey there,
By default the service looks for the notification template at '/angular-ui-notification.html'.
How do we go about setting this value in our app with a custom template URL?
Thanks,
Luke

You can pass custom template
for example

and use it
Notification.primary({message: "Just message", template: "custom_template.html"});

Also you can pass any template url to template option

Ok thanks.
It would be awesome to be able to pass a 'templateUrl' param to the config options. That means you don't have to repeat yourself by adding in the 'template' value every time you write a Notification when using a custom template location.
I've copied the script for now and manually changed the 'uiNotificationTemplates' value but this stops me using the module from bower directly which is a shame.
What do you think?

I agree. It would be useful.
I will implement it.

Awesome! :)

+1 for this request.

+1

+1

Added option 'templateUrl' in v0.0.12 (f843c43).
Option can be passed to the configuration provider and also to Notification's methods.
See examples in ./demo directory.
Attention
Value-service "uiNotificationTemplates" has been deleted!


Remove setting root element from README

This would not be a typical use case and should be discouraged.

Done


plasma5: Fix setup hook some more

Motivation for this change

[[...]] is more robust wrt empty expansions than [..]

${foo-} is simpler and won't accidentally be a url compared to
${foo:-}

+= works even when the variable is undefined.

Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Notify maintainers
cc @

@ofborg build kwin

Thanks @Ericson2314 for fixing this.


Merge passes in Vulkan mobile renderer

After various discussions with @darksylinc, did the work to merge passes that didn't need to be separate subpasses in the mobile renderer. This greatly simplifies things.
There are now 3 scenarios:

The most optimal scenario is where Opaque, sky and transparent all run in a single subpass, and tonemapping runs in a second subpass. In this approach our 3D buffer is never written out to texture memory and the only output is our render target
The slightly less optimal scenario is where Opaque, sky and transparent all run in a single pass. Tonemapping runs in a separate pass. This will happen if: MSAA, glow or DOF are enabled.
The least optimal scenario is where Opaque and sky run in a single pass, transparent runs in a separate pass and finally tonemapping runs in a separate pass. This will happen if reading from screen texture or depth texture is enabled.

This PR also has an optimalisation that when the background is copied from canvas, this no longer happens in a separate pass, but becomes the first step in the opaque pass.
Fixes #82175
Fixes #81910

I'll be adding some before/after timing info as soon as I have recharged some phones :)
I suspect this will have no impact on desktop GPUs, but in theory there should be a measurable difference on phones and standalone VR hardware.

Test project used:
https://www.dropbox.com/scl/fi/pcm5xo42zi4xt0ksf62cx/TestXR.zip?rlkey=cpogyt2w70frgkadvg0ofottp&dl=0
(requires installation of appropriate android templates but otherwise configured for Quest)

I'm wondering if https://github.com/godotengine/godot/issues/82175 was really a bug, so might try and "unfix" that and see if that actually works. In theory we should be able to render MSAA in a pass, resolve it, and then do tonemapping in a subpass, which should result in no data being written our to MSAA or 3D buffers, only to our render target.

After doing some more testing with this, found out the conclusion in 82175 was indeed wrong, the problems were due to the transparent subpass getting skipped on Adreno hardware and MSAA thus not getting resolved, not because MSAA doesn't work with the subpass approach.
Merging the passes already solves the Adreno issue so I've undone the exception and we now use the 2-subpass approach with MSAA enabled.
@darksylinc this could use a review from you. I know that there is more to MSAA especially on Forward+, but for Mobile all scenarios are now working and I think this would be a good thing to merge ASAP once our merge window for 4.3 is open (really wish this had ended up in 4.2 as it solves a number of issues people are currently running into in VR).
@clayjohn could use your stamp of approval too :)

Testing a modified version of the TPS demo with this PR. I am seeing a slight GPU time regression (~16mspd -> ~18mspf) using intel integrated graphics.
I'll investigate a bit more as it is not obvious where that regression would come from, especially on desktop hardware

Testing a modified version of the TPS demo with this PR. I am seeing a slight GPU time regression (~16mspd -> ~18mspf) using intel integrated graphics.
I'll investigate a bit more as it is not obvious where that regression would come from, especially on desktop hardware

Indeed, weird, seeing it should do less, not more ;)

Fixes #83481
tested only my 'minimal reproduction project' (comment)

Any chance of merging this one in soon before conflicts are introduced?

@warent There are conflicts already, as pointed out 5 days ago :) So it can't be merged.
But be assured that we keep track of approved and ready to merge rendering PRs, there are just many of them with interdependencies which we need to merge in a specific order. Clay is managing that.

Sorry, missed the last bit of feedback, its been a bit of a weird week. Should all be ok now.

Great! Should be good to merge now

Thanks!

Are there some updates on this issue? I am still able to reproduce it with v4.3.stable.official [77dcf97d8] :(

Are there some updates on this issue? I am still able to reproduce it with v4.3.stable.official [77dcf97] :(

Did you mean to post on this PR? This isn't a bug report, this is a PR that was merged a year ago. There won't be any updates since it was already merged


Add copy number to germline exome workflow

As far as germline CNV, we will definitely need something other than Varscan. xhmm is what Ira's group has used. We should look into that and ask around for other recommendations. Haley Abel would be a good person to talk to. There is also the GATK machinery. V4 has copy number (https://software.broadinstitute.org/gatk/documentation/article?id=11682)

It sounds like the xhmm approach would be a separate joint-calling workflow.
Direct from the docs: XHMM was explicitly designed to be used with targeted exome sequencing at high coverage (at least 60x - 100x) using Illumina HiSeq (or similar) sequencing of at least ~50 samples.

For now, this means CNVkit. We can address alternative callers as a separate issue.


Changed the code of ns.html from XHTML+RDFa (XHTML 4) to HTML+RDFa (HTML 5)

I have changed the whole code restructured it and made to be written in html5, also I have organized the code with proper indentation and spacing.
Fixes
#396
Description
As the ns.html file is written in HTML 4 to convert it to HTML 5 I have made some changes which are I have change:

Doctype Declaration

Changed the doctype declaration from:

Use code with caution.
To:

Use code with caution.
This specifies that the document is written in HTML5.

Changed the xml:lang attribute on the html element to lang.
Removed unnecessary namespaces declared on the html element.
Added the vocab attribute to the html element to define the base vocabulary used for RDFa annotations.

Checklist

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

Developer Certificate of Origin

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

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

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

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

Developer's Certificate of Origin 1.1

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

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

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

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

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

Please test that the RDF of the current pages matches exact to the RDF to the new HTML. I haven't tested this myself.
There are plenty of pages that refer to this page, any change here would mean changing the meaning of other pages. I know for example that millions of pages on europeana.eu include references to the namespace.

It was already RDFa, the main issue was to convert HTML 4 to HTML 5 @mzeinstra


Crashes when DLSS is added to the plugin file

REFramework works fine until I add the upscaler and dlss file to the plug in folder. It crashes as soon as reframework initializes.
This is happening on every game except re4 remake.
amd7745hx and 4060 mobile gpu (legion 5 pro)
re2_framework_log.txt

Same here. Don't know why this is even happening, as it was working a couple of months ago, then i try it today and it no longer works even though all the files are exactly the same.
So i updated everything to the latest files and now it doesn't crash, but REFramework fails to initialize. Looks like loads of people are having crashing issues.

same issue, and based on our logs (same as yours), its not loading DLSS eventhough we have latest drivers. not sure why or how to fix

I've been trying to diagnose an odd stutter issue with some of the RE games when run from a HDD, and I nabbed the cracked version to test against. The upscaler crashes when run with a cracked copy.
For the actual Steam version though, it loads fine. So if you're not running the actual release, that's perhaps the issue.

I can confirm the DLSS plugin crashes the Upscaler/REFramework when running any RE games other than legit versions, on the contrary FSR2 & XeSS run fine

Yeah, it's only the cracked/DRM-free version that crashes. But why did DLSS work perfectly fine before, and then it started crashing? Before the crashing started i had played through almost all of Village with DLSS without any issues.
I'd expect this problem to affect a lot of people because Village had severe performance issues on release, due to Denuvo DRM. So even though i have a legit copy i had never touched it until i tested it a moment ago.

Yeah, it's only the cracked/DRM-free version that crashes. But why did DLSS work perfectly fine before, and then it started crashing? Before the crashing started i had played through almost all of Village with DLSS without any issues.
I'd expect this problem to affect a lot of people because Village had severe performance issues on release, due to Denuvo DRM. So even though i have a legit copy i had never touched it until i tested it a moment ago.

To be somewhat pedantic, the performance issues weren't actually related to Denuvo, it was Capcom's own DRM. I get the hatred for Denuvo, but Capcom's DRM actually being the one at fault was spelled out in the initial cracks readme.
While that initial release was truly ridiculous in that it seemingly took all that media fervor for Capcom to finally take action, it was indeed fixed over 2 years ago (and that fix also got plenty of media attention). As for why it just started happening, who knows - changes in RE framework or the upscaler plugin could have triggered it, but you can't expect the developer to test against a cracked .exe.
It's fine for this thread to let people know what the reason is, but it's not really an 'issue' when it's working fine with the legit copy.

I had this problem before. Latest drivers is not always the best. Just Use a different driver, I used the one from my laptop manufacturer, use dlss dll 2.5.1. Try 1080p in graphic settings, and lowest all other settings. Once it works, start to increase ur graphic settings from there. Dont forget to turn off fidelity fx in the graphic settings.

add SelfProtect=0 to steam_emu.ini
Should look like this:
Offline=0
SelfProtect=0
If you want to thank me, please take the time to tell others this fix when you encounter someone posting a similar issue. Cheers.


Can only activate one/two predetermined effect

Hello ! First of all, cool plugin and nice work so far!
Current behavior :

Nvidia Broadcast Camera effect selected : Background replacement
Stream deck buttons set :

One for Background Replacement
One for Background Removal
One for Background Blur

Nvidia Broadcast Secondary Camera effect selected : Vignette
Stream deck buttons set :

One for Auto-Frame
One for Vignette

When I push on Streamdeck on the button for Background replacement, I can turn it on / off . Same goes for the Vignette.
When I push on Streamdeck the button for Background Removal / Blur or Auto-Frame ( for secondary effect ) , they do not do anything as they are not selected within the Nvidia Broadcast.
Expected behavior :
When I push on Streamdeck the Background Removal button, to change the option in Nvidia Broadcast and actually take effect.
I understand that I cannot have multiple effects active at the same time from the same group ( Primary effect - Background replacement/removal/blur), Secondary effect ( Autoframe,Vignette,Eye Contact, Video Noise removal ) , so when activating Background removal, it should stop the background replacement and activate brackground removal.
The buttons on Streamdeck also do not do anything if no effects are selected whatsoever in Nvidia Broadcast ( if they were removed )
The buttons on Streamdeck should add and enable / turn on the specific option even when Nvidia Broadcast doesn't have them selected.
Thank you very much for listening and hope this can be done!

Hi,
Thanks for the detailed description, I'll see what I can do. Main issue is NVIDIA doesn't have control identifiers on the toggles so I have to dig around to figure out which effect is on which toggle. I haven't tried to switch the effects themselves, but I'm thinking that if I give you a way to say:
When I push the button on StreamDeck, and no effect matches, then override toggle 1 or toggle 2 and swap in the desired effect.
Then, you can specify which toggle you would like to set/override on button push.
I have a new version submitted to the marketplace for review that changes up how everything works, so keep an eye out for it, it should arrive in the next few days.

Try this one out:
https://github.com/BackTrak/StreamDeckNvidiaBroadcastPlugin/releases/download/v<IP_ADDRESS>/com.zaphop.nvidiabroadcast.sdPlugin.zip
There might still be some bugs... let me know if you like it!

Hello again ! Thank you for the update !
Steps I took : Downloaded the zip, opened it, double clicked on "com.zaphop.nvidiabroadcast.exe" , it said created some new files and if I want to archive them, I first said yes, and after testing a bit I reinstalled choosing not to archive them.
I then realised that's not how it's supposed to install, so I don t know what that did, and I went to "AppData\Roaming\Elgato\StreamDeck\Plugins" and removed the old folder to put in the new one. ( typing this cause maybe it will help others )
After playing around for quite a bit, I can say that I crashed Nvidia quite a lot of times. I simply tested the possibilities around.
Main reason for those crashes I believe is the "Slot" option, which is not really defined.. what is "Slot 1/ Slot 2" ?
In my head they meant as checking the option for Performance / Quality. Buut it doesn't really change them, and not only that it doesn't change them, but it crashes the app only when some specific "Slot" is set up in the plugin. And it is not always the same one . Some effects seem to crash on Slot 1, while others on Slot 2 ( both having the option of Performance and Quality. Because there is for example AutoFrame / Vignette that don't have Performance / Quality , Video Noise removal has Strong / Weak instead, I believe these should be arranged in a data that contains the specific informations that will then be updated.
I didn't get to play with changing devices / resolutions these times, If Nvidia crashed, I have to End the task of it like 4-5 times and fast in Task manager in order to be able to run it again and make another test.
As another idea ( you can tell me if you want me to make another "issue / ticket" ) , maybe you can add another functionality, to choose where would the silder be on the specific filter ?
I mean, to be able to set if I want 3 buttons for the background blur, at 1%, at 34% and at 80% . ( basically any option I want, but to be able to edit it inside Streamdeck app / plugin via either a slider, a text or why not , both that will update with each other, and not to depend on what was already selected inside Nvidia Broadcast )
All in all, if I do get to know exactly what slots not to touch, and which to pick, it seems to run fine so far. So nice job !
Question, I believe I won't get "updated" automatically when Marketplace will have the new version , right ? Or.. ?
I will try to provide more help whenever a new version comes out again, and even outside github, if you're willing to try and work something together ^o^ 🍻 Cheers !

Hi there,
Was the feature above released? I found the same issue and it'd be perfect if it is running right now.

Hi there,
Was the feature above released? I found the same issue and it'd be perfect if it is running right now.

Hello !
As of right now, on marketplace, the latest official update is <IP_ADDRESS>
In this thread , in the link above that the developer posted, the version is ( unofficial ) <IP_ADDRESS> which is an upgrade.
It still has the flaws that I mentioned in the previous comment but it's still better than <IP_ADDRESS>
And if you work with it a bit and find the right associations, it will work pretty fine. There would be a few upgrades that could be done afterwards. Hope the developer will find some time for us to make them happen <3 and hope this info helps you.


Add get_logger and logging level

Add tests for update logger level

should we add a test to ensure this scenario is also tested.

Agree, would probably be good to cover this to prevent regression.

should we add a test to ensure this scenario is also tested.

Agree, would probably be good to cover this to prevent regression.

@KeithMnemonic , if you are referring to test that the config file includes the
logging section for testing the level, that test is included in test_adapter
def test_update_logger_from_config():
if the test should be general, I did not include that as IMHO that should be a different PR

Sean and I are suggesting "a test that the config file DOES NOT include the
logging section" as that is how the files are currently distributed, without a section.

Sean and I are suggesting "a test that the config file DOES NOT include the logging section" as that is how the files are currently distributed, without a section.

a test (that the config file DOES NOT include the logging section) for the update logger level or a different test ?

Sean and I are suggesting "a test that the config file DOES NOT include the logging section" as that is how the files are currently distributed, without a section.

a test (that the config file DOES NOT include the logging section) for the update logger level or a different test ? I understand you mean a different one to check the section does not exist

To be explicit, there is a test on this PR for a config file without logging section here
so, I was confused and I understand you meant/mean a different test

Sean and I are suggesting "a test that the config file DOES NOT include the logging section" as that is how the files are currently distributed, without a section.

a test (that the config file DOES NOT include the logging section) for the update logger level or a different test ? I understand you mean a different one to check the section does not exist

To be explicit, there is a test on this PR for a config file without logging section here so, I was confused and I understand you meant/mean a different test

I see. My confusion is you call  update_logger_from_config two times in this test function once with a config and once without.  I think the confusion is the function itself (and corresponding test) is called  'update_logger_from_config' but really there may not be an entry in config.  Maybe a better option is just "update_logger_settings" or ""update_logger_config""something that does not imply it is only getting the values from config. As when that section is missing in the config, it gets default values.
Thankyou  for clearing up my confusion.


Add console command

The package would be much easier to use if there was a console command.
I'm thinking something in the lines of:
7to5 convert <sourceDirectory> <destinationDirectory>

:+1: seems about right

Console command contributed by @JolitaGrazyte
Will probably do some small refactorings on it soon.


starter_build.sh file URL 404 not found

I tried to download the starter_build.sh with "wget -O starter_build.sh https://raw.githubusercontent.com/deepgenomics/GenomeKit/main/starter/build.sh" but got:
--2024-11-19 15:55:40--  https://raw.githubusercontent.com/deepgenomics/GenomeKit/main/starter/build.sh
Resolving raw.githubusercontent.com (raw.githubusercontent.com)... <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, ...
Connecting to raw.githubusercontent.com (raw.githubusercontent.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 404 Not Found
2024-11-19 15:55:40 ERROR 404: Not Found.

Looks like something must have changed on the end of genome_kit team. This link should still work!
https://raw.githubusercontent.com/deepgenomics/GenomeKit/v6.0.3/starter/build.sh


Removed animation block if transition block is set to None

According to Apple docs there is a chance that animation block with
zero duration finishes with delay:
“If the duration of the animation is 0, this block is performed at the
beginning of the next run loop cycle.”

It works as expected. Could you push this fix to cocoapods? Thanks!


Question about Data Generation

Hi, zongyi, thank you for your excellent work! I'm trying to examine FNO in a more complicated setting so I have to regenerate some data for my experiments. I use data_generation/navier_stokes/ns_2d.py but I notice the output solution is in the shape of [20, 64, 64, 200], where the last dimension is supposed to be 30 for the T=30 setting. Could you please tell me which parameter should I change?

You may want to change record_steps=30 which sets how many time steps to record.


x/pkgsite: show "Added in go.1.x" for each std package as a whole

For std packages, pkgsite shows "Added in go1.x" for each symbol that was not present when the package was created, but there is no indication of which version the Go package itself was created.
See also:

https://github.com/golang/go/issues/58853

Similar Issues

x/pkgsite: show the stdlib/module version when a symbol was added #42933
x/pkgsite: show the version when a symbol was added to a package #37102
x/pkgsite: "added in version" does not account for new packages #51013

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

Dup of https://github.com/golang/go/issues/51013


podman package run-time requires catatonit, but isn't including it

Describe the bug
The (latest?) podman package now requires catatonit at run-time, but isn't including it in what's installed when the podman package installed.
Steps To Reproduce
Steps to reproduce the behavior:

nix profile install 'nixpkgs#podman' (or equivalent nix shell, nix develop, nix-shell, or nix-env)
From within the environment, podman run ... any container
See Error

Expected behavior
Installing podman gives a runnable podman program, excluding things that can't possibly be handled by nix (e.g. subuid, subgid, global container storage and registry config, etc).
It shouldn't need running an extra nix profile install 'nixpkgs#catatonit' to make it runnable.
Screenshots
N/A
Additional context
Example error:
Error: container-init binary not found on the host: stat /usr/libexec/podman/catatonit: no such file or directory

Notify maintainers
@zowoq
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
...snip...
- system: `"x86_64-linux"`
- host os: `Linux 5.15.0-76-generic, Ubuntu, 20.04.6 LTS (Focal Fossa), nobuild`
- multi-user?: `yes`
- sandbox: `yes`
- version: `nix-env (Nix) 2.15.0`
- nixpkgs: `/nix/store/rspism4p5xvwms84p5wfsnj93c87avmf-source`

@mtalexan is this still an issue? I've tried to reproduce it on NixOS/unstable and Fedora 38/nixpkgs23.05 but it worked on both systems as expected.
catatonit was added as a dependency to the podman package end of February this year: https://github.com/NixOS/nixpkgs/pull/218355
Maybe your system didn't include that change at the time of opening this issue?

It doesn't seem to be a problem anymore. It might have been something with the default config in Fedora 38 at some prior time (since Nix podman doesn't include a config). I just installed it a few different times on different systems and I'm not getting the problem anymore.

The Ubuntu 20.04 shipped with a very old and unmaintained podman, so installing the host version to get the config may have been configuring for catatonit as the default init when --init was used, and then requiring that to be present regardless of whether or not you used the init flag.
Multiple installs to Ubuntu 22.04 have worked after massaging the /usr/share/containers config files the distro-provided version includes. Notably Ubuntu's podman config files no longer default to catatonit and don't install it as a dependency themselves anymore (possibly only in 22.04 though?).
Regardless, I'm blaming this on the config files I had to come up with.


add image mimetypes

Describe the PR
add mime type for comsumes and aproduces

image/png
image/jpeg
image/gif

Relation issue
https://github.com/swaggo/swaggo.io/issues/15

@pei0804 Any idea why ci build failed?

I will fix it later
There is no problem with this commit.

@tanan thanks for your contributions 💯


Changing terrain provider moves billboard to incorrect location

This came up on the forum. Here's a Sandcastle example. To reproduce:

Zoom in close enough to the billboards and tilt the camera until you see them curve

Select "delete billboards"
Select "change terrain provider"
Select "add billboards"
Select "change terrain provider"
Observe that they're now in a different position:

This may be just another manifestation of https://github.com/AnalyticalGraphicsInc/cesium/issues/4776 but I couldn't dig deep enough to figure out what the underlying issue was.

One detail that may be relevant.  If there weren't different pixel offsets for each billboard in this sandcastle, the billboards would render on top of other billboards in the set.
So they end up rendering on top of each other when this issue happens.

So many billboard tickets out there, but I think this is the most relevant to what I'm seeing and I wanted to add my observations. So here it goes...
Stationary billboards do not clamp correctly when entities are removed just before terrain is reloaded and entities are added while terrain is still loading.
Note that in the example code(see link below), all entities are removed, but even if one or more entities are removed and new ones are added you can see the issue.
Observations

calling removeAll seems to work better vs individually removing entities

we don't have the option to call removeAll in our use case

This does not happen when disabling the reuse of "unused" billboards in EntityCluster

I modified code so that new billboards are always created and deleted on removal
Though this makes the problem go away, I do not think this is the exact underlying cause since it just prevents updateHeight callbacks on reused billboard objects

It looks like the QuadtreePrimitive customData list keeps growing and some stale height callbacks are being made with old coordinates.  This seems to be in agreement with @lilleyse comment in #8650
For my case I can delay loading of entities (by timer or wait until all tiles are loaded), but it really isn't optimal.
Note that we have been seeing this issue since at least 1.57, even possibly as far back as 1.51, but have not had time to debug.
Middle click will reload terrain and demonstrate issue
https://sandcastle.cesium.com/#c=jVVtb+M2DP4rRIABys6nuC06oE1SrGuKtUDuerhkuy8BAsWmE6GyFEhyguyQ/z4qfkmal2EfbFkU+ZB8RNIrYWElcY0W+qBxDU/oZJHzv3cyNmklu/2T0V5IjXbSancnekVWiZLJ+4vQqTq2HSUWUY+WIsHnFWpfKbHSD3cJauSJIBQXwA6BuEP/qpeFf0y8NJqxpXEyfLWh/wA/JxrAIumMAgYL1tvoktvxZon8y+tgMHyeDt5+fK0D92gt5fLNmpVM0ZJ0orNC7xwSujIiHZcqjzp91p78o2Pt2vvB+S4AAPIrPO41u9DpwIyE7y4ch4cEKSqxAW9Aoa9jgKxQagMBstRaS52adWBhLHM0hWcfwSO4juN4l/flsOtYjxKlS6pu4JQBuHBENhW7ZRw/jFUfsv8YxjETIY4dQ0Y7D84L64fCE+jNb91juZ6T/PNVXJ5kxgILTMl+3AUJPbim5dOnKjXYsShzMUeymrQ47+w2rpOJRCrpN3wus0mrWyrLDJiEX+Aa+v0+xA0InIUoM54OzdxMzQot3Rtf6nmDti2XijGssuUiTdkeN70Hyb0ZeSv1nLWj+qAu6Pua2CdKnr6EvuGZNfkA51TGjtWkRA1tn5n8NeZxfNXeo82kUjMjLHlrXFdJ3ZdLtBcvUM4X/jtmaFEn2ETw8lHOn4aPX75Nx2/TP7+//fV10CBsq69tWfbb0yLcd2YZDpW9dK5ASA060MaDSZLCwlr6BannxO6jUrBy5YaYAqMRZpuwlADHLDdWVfOFJ9RCrbCv8sZkJVRBi6M5g5VVc0Z19iySBWMk2c+Y08st3f6xeU2DJpdpxUEdRHifNu+laRLB7UEXj2j2JcJ5hXxHIaXQcFozWcWTiByt4JnavJgcWdw+7N0LHAWN0ADVbaeY0SBP2VGnt9tHqZ9OgnNj49yUKXTloy4T+I+x2j2b3diwnyk6L7X4P80SxkYEN7f87u7ulmZkYDeCtLCVdbwtidjSuxW1es5vFD7Udf27zJfGeiisYjQFPOZLRWPMdWZF8o6eJ861q2R7nUPTXipX1On9M39J+j0K5+gkTPiR/AcnrYdeh/RPTAMxVPhv5aAJaourh2Ep5Jz3OrQ9b+mNUTNhj5D/BQ

Note that the delay in loading entities does not always help and also I noticed that the customData length continues to grow each terrain reload so that is a bit worrisome.


"undefined is not a function error" in ember-cli Production environment

Ember-forms is working great, but when I run the ember-cli server in production mode I get an Uncaught TypeError: undefined is not a function error. Since this only happens in production, I'm not even sure how to start trying to find the source of this. If I remove ember-forms from my app, it works as normal (except the pages using the forms of course).
I suppose I could turn off minification in the production script to try and find the issue.
I am simply including the globals version in my Brocfile: app.import('vendor/ember-forms/dist/globals/main.js');
Here is a screenshot of the error message which might help: http://inft.ly/fvPryUr

Running into this as well

@yangbai1991 Sorry I have no time to maintain this project any longer, seeking for maintainers though


Font scaling

Summary | Résumé
Super-sized text caused UI issues. Text would take up most of the screen making it difficult to effectively use the app.
This branch fixes that issue by limiting how big the text can get and adjusts the layout and size of the components.

Checked both iphones & Androids using both Large/Largest font text sizing. Android (MotoG6) was the only capture that looked the worst when in the Largest Font - but is fine in large text.
MotoG6 - Largest Font:

MotoG6 - Large Font:

All other iPhones (both small & big screens) looked good in the Largest Font.
iPhone5 (small screen) - Largest Font:

Android - Galaxy S7 (small screen) - Largest Font:

Given the above, all screen sizes are optimized well when using the Large Font. Largest still doesn't optimize 100% with bigger screen Androids. All screens for iPhones are optimized as expected in the largest font.

Merging the latest - will adjust further as need be in the next set of test builds.

Retested with build version 1.2.54 (259). Still seeing an issue with largest font on MotoG6.
MotoG6 - Largest Font:

However, my LGg4 has a setting for Maximum Font that I only used this time and noted the same issue as Moto.
LGg4 - Maximum Font:


Use antlr4 mysql workbench grammar to generate the MySQL parser

Using  https://github.com/mysql/mysql-workbench.git and antlr-4.11.1-complete.jar to generate the MySQL parser.
https://github.com/mysql/mysql-workbench/tree/8.0/library/parsers/grammars
The parser is used with a tree walker and a listener to translate MySQL create tables to CH SQL.
Please note that the generated python was amended because the grammar is designed for C++, not python.
To test the parser separately you can run this command
python db_load/mysql_parser/mysql_parser.py test1.sql

echo "alter table t add c1 int after c0, add column c2 int not null, add c10 Date null, add c11 int, drop column c3, modify column c4 DateTime(6) not null default '' first,  add key k(c2), rename to t1, rename column c3 to c4, change column c1 c5 int not null, change c5 c6 bigint not null;" >test9.sql

python db_load/mysql_parser/mysql_parser.py test9.sql

The support for alter needs more work :

the field definitions that are supported in create table in CH are not supported in the alter.
MySQL semantics of multiple alters is different from CH and needs special attention

MySQL can support this : adding a column that already exists if it is renamed, CH does not.
drop table if exists t;
drop table if exists t1;
create table t(c0 int, c4 DateTime, c5 int, c3 int, c2 int, c1 int) engine=InnoDB;
alter table t add c1 int after c0, add column c2 int not null, add c10 Date null, add c11 int, drop column c3, modify column c4 DateTime(6) not null default '2022-01-01' first,
 add key k(c2), rename to t1, rename column c2 to c7, change column c1 c5 int not null, change c5 c6 bigint not null;


Add option for specifying the capture attr on the file input

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[x] Feature
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[x] All tests are passing (there aren't any, but I adapted the mock-data file)

If adding a new feature, the PR's description includes:

[x] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Reasoning
On mobile chat apps, it is very common nowadays to send photos which are captured directly in the app. In HTML5, we fortunately have the capture attribute on file inputs, which helps us acheive mostly the same user experience. When activating a file input with the capture attribute set to either true, "user" or "environment", the user gets the option to directly capture an image or video to populate the file input.
Other information:

@antoine92190 any input on this? Ideally I'd need this before the beginning of February, for a yearly event I'm writing the software for. Otherwise I'll have to keep my fork of this project as a git submodule in my project and keep transpiling it myself...

Look good, thanks

Done


Implement column level filter to hide / show columns based on selection.

This feature is available in multiple grid libraries. Provide filters on Table. One of the filter feature could be column hide / show options. When developer want to hide few columns by default to end user this feature helps.

HI @Srikanth4 what do you mean for based on selection ?

Sorry guys I did'n have time for this implementation.

@yuri-sakharov never mind :)

We have new release for fixing this issue: please check RELEASE NOTE and upgrade related packages
this is docs for column toggle and this is online demo
Please open another issue if you find any bugs, thanks


[WIP] Various improvements for DQN and the replay memory

My dqn model that works in master is failing on this branch. I will try and do some investigation on this but I wanted to report it first...
ryan@Redman:~/workspace/DeepSF$ python dqn.py -d ~/Dropbox/cmu-sf/deepsf-data2/ --visualize -r rawpnts
Using Theano backend.
Using gpu device 0: GeForce GTX 1060 3GB (CNMeM is enabled with initial size: 10.0% of memory, cuDNN not available)
____________________________________________________________________________________________________
Layer (type)                     Output Shape          Param #     Connected to
====================================================================================================
flatten_1 (Flatten)              (None, 60)            0           flatten_input_1[0][0]
____________________________________________________________________________________________________
dense_1 (Dense)                  (None, 64)            3904        flatten_1[0][0]
____________________________________________________________________________________________________
elu_1 (ELU)                      (None, 64)            0           dense_1[0][0]
____________________________________________________________________________________________________
dense_2 (Dense)                  (None, 64)            4160        elu_1[0][0]
____________________________________________________________________________________________________
elu_2 (ELU)                      (None, 64)            0           dense_2[0][0]
____________________________________________________________________________________________________
dense_3 (Dense)                  (None, 64)            4160        elu_2[0][0]
____________________________________________________________________________________________________
elu_3 (ELU)                      (None, 64)            0           dense_3[0][0]
____________________________________________________________________________________________________
dense_4 (Dense)                  (None, 3)             195         elu_3[0][0]
____________________________________________________________________________________________________
activation_1 (Activation)        (None, 3)             0           dense_4[0][0]
====================================================================================================
Total params: 12419
____________________________________________________________________________________________________
None
Training for 27000000 steps ...
     2700/27000000: episode: 1, duration: 5.134s, episode steps: 2700, steps per second: 526, episode reward: -1282.000, mean reward: -0.475 [-100.000, 100.000], mean action: 1.164 [0.000, 2.000], mean observation: 72.092 [-304.000, 549.654], loss: --, mean_absolute_error: --, mean_q_new: --
Traceback (most recent call last):
  File "dqn.py", line 97, in <module>
    main(args)
  File "dqn.py", line 78, in main
    agent.fit(env, nb_steps=STEPS_PER_EPISODE*10000, visualize=True, verbose=2, action_repetition=args.frameskip, callbacks=[log])
  File "/home/ryan/workspace/keras-rl/rl/core.py", line 97, in fit
    metrics = self.backward(reward, terminal=done)
  File "/home/ryan/workspace/keras-rl/rl/agents/dqn.py", line 279, in backward
    metrics = self.trainable_model.train_on_batch([state0_batch, targets, masks], dummy_targets)
  File "/usr/local/lib/python2.7/dist-packages/Keras-1.1.0-py2.7.egg/keras/engine/training.py", line 1221, in train_on_batch
    outputs = self.train_function(ins)
  File "/usr/local/lib/python2.7/dist-packages/Keras-1.1.0-py2.7.egg/keras/backend/theano_backend.py", line 717, in __call__
    return self.function(*inputs)
  File "/home/ryan/.local/lib/python2.7/site-packages/theano/compile/function_module.py", line 871, in __call__
    storage_map=getattr(self.fn, 'storage_map', None))
  File "/home/ryan/.local/lib/python2.7/site-packages/theano/gof/link.py", line 314, in raise_with_op
    reraise(exc_type, exc_value, exc_trace)
  File "/home/ryan/.local/lib/python2.7/site-packages/theano/compile/function_module.py", line 859, in __call__
    outputs = self.fn()
ValueError: GpuElemwise. Input dimension mis-match. Input 1 (indices start at 0) has shape[1] == 1, but the output's size on that axis is 32.
Apply node that caused the error: GpuElemwise{Composite{Abs((i0 - i1))},no_inplace}(GpuDimShuffle{x,0}.0, GpuFromHost.0)
Toposort index: 92
Inputs types: [CudaNdarrayType(float32, row), CudaNdarrayType(float32, matrix)]
Inputs shapes: [(1, 32), (32, 1)]
Inputs strides: [(0, 1), (1, 0)]
Inputs values: ['not shown', 'not shown']
Outputs clients: [[GpuCAReduce{add}{1,1}(GpuElemwise{Composite{Abs((i0 - i1))},no_inplace}.0)]]

HINT: Re-running with most Theano optimization disabled could give you a back-trace of when this node was created. This can be done with by setting the Theano flag 'optimizer=fast_compile'. If that does not work, Theano optimizations can be disabled with 'optimizer=None'.
HINT: Use the Theano flag 'exception_verbosity=high' for a debugprint and storage map footprint of this apply node.

Yes, there's currently a bug if you use the Theano backend and with a metric. This will be fixed before this gets merged.

I am pretty sure I tracked it down to this commit -> 8231b72ead028e7ff1e1c7c51bfa8c7587151711

Yes this is because of the way we mask the targets in the Lamba layer. I have a plan how this could be addressed siding worry about it too much for now.

Ok I just figured I would help test what seems like the dev branch because when I start submitting PRs for new agents and feature requests I wanted to have a stable dev branch to work off of.


pids: remove hardcoded unmanaged provider name

refers to inveniosoftware/invenio-rdm-records#580
closes #285

@jrcastro2 can you please review this PR?

Overall LGTM.
There is just a small bug that needs to be fixed, when saving a wrong DOI and then clicking the "No" radio button for DOI I would expect to save it without a DOI, instead it still saves the previous DOI and the error message is still visible.
pid-bug.mp4

Thanks a lot for taking the time of checking it out and testing it. I have simply removed the error from the Managed part because it does not make sense in this case. We will see if we need errors there, but it should not be the case.


CI Fixes

Description
There have been a few PRs lately attempting to fix issues in the CI. Most solve the issues but then the CI breaks in some other obscure way. Here I have attempted to do a pass over the CI and make some quality of life improvements whilst I was at it.
Doxygen Check

Reduced to 10-20s from 1-5m
Uses ghcr.io/khronosgroupactions/doxygen:1.9.5

Copyright Check

Reduced to 10-20s from 1-5m
Uses ghcr.io/khronosgroupactions/copyright-check:latest

Snake Case Check

Reduced to 10-20s from 1-5m
Uses ghcr.io/khronosgroupactions/snake-case-check:latest

Clang Format and Clang Tidy

Updated to 15.0.6
Uses ghcr.io/khronosgroupactions/clang-tools:15.0.0
Reduced Image Size improves checkout speed
Use scripts supplied by clang-extra-tools as these are more up to date than those in this project

Android Build

Moved Android build to a different image
Uses ghcr.io/khronosgroupactions/android-ndk-build:r25b
Image now builds in 4 minutes instead of 30min-1hour
Image is now 4GB instead of 7GB+

Known Issues

git config --global --add safe.directory /__w/Vulkan-Samples/Vulkan-Samples is sometimes required

This should be handled by actions/checkout@v3 but seems to intermittently fail

@ggfan for some reason I can't add you to the reviewer list, please take a look at this when you get chance :)
This still requires your validation layer fix

Just seeing the one CI failure on Clang Tidy Check.  We discussed this on the Samples call and just need guidance from @tomadamatkinson if we're OK merging this with this failure (external warnings).  LMK as this one will fix several other CI failures.  Thx

LGTM, thank you so much!

OK, I'll go ahead and merge this.  LMK if it causes issues and I can revert.  Thanks!


Local env may require the same package versions as in .mlem file for build

error at deploying model:
ValueError: Conflicting versions for package numpy: 1.21.6 and 1.23.5 (model saved in one place, getting deployed from some mlem env with numpy 1.23.5)

I got this when I trained a model on one env, but then started deploying it in another env. I assumed we won't check this (and that we even don't need numpy installed in this case). So: we should work around this and don't require numpy installed as well.
How to repro (rough draft, ping me if this doesn't reproduce):
pip install numpy==1.21.6
import mlem

def f(x):
    import numpy
    return x

mlem.api.save(f, "model", sample_data=1)

now:
$ pip install numpy==1.23.5
$ mlem deploy run flyio ...  # but maybe `mlem build docker` will work

@aminalaee, while we're waiting on the answer on the other issue, could you please TAL at this one? I believe this is an important one. This could require some dive, but happy to help investigating. Happy to also clarify the issue if it's not explained clearly.

@aguschin  I couldn't reproduce this issue. Maybe some steps for reproducing are missing or the issue is resolved?

Hmm, interesting. I'll try to reproduce this again and post it. Ironically, I'm blocked by #661, but not sure if you can fix that one - reproducing it will require having at least two orgs on fly.
On the second thought - I can add you to my org, then you'll be in 2 orgs at the same time - and the issue will be reproduced. Please ping me if you would like to check that one out while I'm reproducing this one. To invite you, I'll need your email, so you can send it to me to<EMAIL_ADDRESS>or t.me/agusch1n


Ignoring .stylintrc

Atom Plugin:
In the root of my project I have .stylintrc with the following:
{   "insertColons": false,   "insertSemicolons": false,   "insertBraces": false }
However all of these are still applied.
.foo { 	display: none; }

Hi @aharris, thanks for your interest in Stylus Supremacy.
I believe you have misunderstood the formatting options (insertColons, insertBraces and so on) with Stylint rules (colons, braces and so on). Content in .stylintrc file should look like this:
{
  "colons": "never",
  "braces": "never"
}

Please refer to the formatting-option and Stylint-rule conversion table. Basically, you have to use the TEAL column in .stylintrc.
Let me know if I can improve our documentation web site so it prevents others from further confusion.
Cheers!

Thanks for that. I missed this mapping.


Add X509v3_addr_get_safi() function

Implementation of the X509v3_addr_get_safi() function, which is a getter for the SAFI.
This presents the same issue as X509v3_addr_get_afi() with returning 0 as an error instead of the reserved SAFI. Maybe we should have something like
int X509v3_addr_get_safi(unsigned *safi);
return 1 on success and modify the safi's value if we want to fully comply with the standard, or maybe that's too over-engineered.
Fixes #18529 in master branch.

Maybe we should have something like
int X509v3_addr_get_safi(unsigned *safi);
return 1 on success and modify the safi's value if we want to fully comply with the standard, or maybe that's too over-engineered.

Another possibility is to make this function return int instead of unsigned int, and return -1 on error. The asymmetry between how X509v3_addr_get_safi would work and how X509v3_addr_get_afi works would be annoying though, so I'm not sure.

This PR is waiting for the creator to make requested changes but it has not been updated for 30 days.  If you have made changes or commented to the reviewer please make sure you re-request a review (see icon in the 'reviewers' section).

This PR is waiting for the creator to make requested changes but it has not been updated for 61 days.  If you have made changes or commented to the reviewer please make sure you re-request a review (see icon in the 'reviewers' section).

This PR has been closed.  It was waiting for the creator to make requested changes but it has not been updated for 90 days.


Update strain names to gn

a new issue in on updating the strain names to the gn database

My new updates on the issues regarding uploading genetype names to the GN database