Add support for react-native-web

Is your feature request related to a problem? Please describe.
The react-native-web build does not work
Describe the solution you'd like
Support react-native-web
Describe alternatives you've considered
Currently I'm mocking DdSdk and other objects in the web build with a webpack alias. I could try writing code to make requests either with the browser SDK or an http client. But it think it would make more sense if it lived in here, even if it were basically a proxy to the browser SDK.

Hi @onlyanegg, thanks for reaching out on this topic!
It is in our backlog, but it isn't places on the roadmap yet.
In the meantime I would suggest that you try writing a proxy to the browser SDK for the web part.

Ok, thank you :)

+1 on this
I've been using a custom cross-platform abstraction -- and I'm very glad to have the functionality I am able (thanks!) But I will say that it's been quite difficult at times, to keep things in aligned -- after a couple bugs slipped in, I've started to ensure that any update includes an exhaustive accounting of configuration changes / types def updates to ensure consistency between the platforms.
Even if a fully fledged solution doesn't make it onto the roadmap this time around, it would be tremendously useful to start aligning the APIs.

API Mismatch Summary
Native- vs web-specific config

only relevant to particular platform - legit as it's not easily confused with

eg nativeCrashReportEnabled

seemingly missing feature

eg version (recently added!)

Overlapping config

same key / same value <-- this is the dream :)
different key / same value <-- requires some aliasing, but not too bad

eg service vs serviceName

same or slightly different key / slightly different value 🤕

eg firstPartyHosts vs allowedTracingOrigins or siteParameter vs site

different key / same value, BUT unclear if features actually overlap 😫

eg trackErrors vs forwardErrorsToLogs

Misc Other Issues

Method mismatch - names but also sync vs async
Companion libraries eg @datadog/browser-logs @datadog/mobile-react-navigation
CI / SourceMap orchestration

Hey, @leggomuhgreggo! Would you be open to sharing your abstraction? That could be super useful for me (and maybe others?) at least as a reference.

@onlyanegg the abstraction is a library in a private monorepo, but I threw the raw code in a repo here.

Thanks, @leggomuhgreggo!

Updated the example x-platform RUM client repo

Hey, @louiszawadzki! I just wanted to bump this. I'm having to do a bit of a refactor to our implementation because of and Expo update. Has supporting web in this lib gained any traction?


BUILD-159: initial CLI only implementation of this operator

See README for how to run this operator manually

@adambkaplan @coreydaley do either of you have the green button for this initial merging?
I need this basic seeding in order to have enough there for the subsequent openshift/releaes PR that will enable prow, etc.
Of course, review and provide feedback on my copy and hack of https://github.com/openshift/gcp-pd-csi-driver-operator as you all deem fit
Some quick notes:

that repo had no unit tests;
that repo's e2e was based on the k8s csi e2e's baked into openshift/origin's openshift-tests binary .... as https://github.com/openshift/csi-driver-projected-resource by design is not a fully functional CSI driver, we'll have to see how that shakes out when the next Jira to fully integrate with the CSO happens ... bottom line, I did not copy the e2e test ... fwiw, the openshift/release definition at https://github.com/openshift/release/blob/master/ci-operator/config/openshift/gcp-pd-csi-driver-operator/openshift-gcp-pd-csi-driver-operator-master.yaml to not enable make test-e2e
as we have not cloud dependency, I just have the node level stuff from gcp-pd ... our csi driver is missing parts, parts it does not need, to satisfy what the gcp-pd controller pod was doing

But as I noted in the Jira, running the command line as documented in the README provisions our driver such that I could provision a pod with a Share

/assign @adambkaplan
/assign @coreydaley

@gabemontero I do not have green button access
/lgtm

ok thanks @coreydaley
don't think this will work like lgtm, but just in case
/approve

It may not since there is no existing OWNERS file

repo needs prow setup in general it appears.

repo needs prow setup in general it appears.

thanks @bparees
yep working on that now ... with the repo initially seeded, the rehearsal jobs will have something tangible to play with

repo needs prow setup in general it appears.

thanks @bparees
yep working on that now ... with the repo initially seeded, the rehearsal jobs will have something tangible to play with

fyi https://github.com/openshift/release/pull/19713 is up
I'll assign folks once I get all the CI jobs passing


DNS Swagger description updates

Update to DNS Swagger descriptions.
Also removes 'If-None-Match' from Zones_Delete and RecordSets_Delete, since If-None-Match is not applicable to delete operations.

This change is

Hi @jtuliani, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (jonatul). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

@tbombach OK, dnsZones capitalization should be fixed now...pls can you merge?

Merged

 https://github.com/Azure/azure-sdk-for-python/pull/853

 https://github.com/Azure/azure-sdk-for-node/pull/1960

 https://github.com/Azure/azure-sdk-for-ruby/pull/541


Refactor default backend handling and add better events

We were setting up and tearing down the default http backend used in url maps that don't have a user specified default backend in weird places that didn't make sense. This pr also sends better events that include health check information, and sets better timeouts on readiness-probes converted to health checks.

This change is

LGTM

I do not have permission to merge or apply label.

Oh ,thanks


Update publish.sh

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


Using the getConfigInMultipleLangs method instead of getInt (Deprecated)

Questions
Answers

Description?
Using the getConfigInMultipleLangs method instead of getInt (Deprecated) for fixing the CI. See https://github.com/PrestaShop/ps_banner/pull/54#issuecomment-1572071201

Type?
bug fix

BC breaks?
no

Deprecations?
no

Fixed ticket?
-

How to test?
CI is green

dev validation ok

thanks @M0rgan01


seperates task and container override

Sepererates the way we override task level cpu and memory from the prefect container. If no container cpu and/or memory is passed, it uses task level.
Closes  #264
Example

Checklist

[ x ] References any related issue by including "Closes #" or "Closes ".

If no issue exists and your change is not a small fix, please create an issue first.

[ x ] Includes tests or only affects documentation.
[x ] Passes pre-commit checks.

Run pre-commit install && pre-commit run --all locally for formatting and linting.

[ ] Includes screenshots of documentation updates.

Run mkdocs serve view documentation locally.

I think it'll be important for us to keep in mind that this'll make it easier for users to accidentally supply values to a single container that are larger than the values supplied to the task, especially when they leave the task-level resource settings empty and then request resources for their container that are larger than the default.
I'm ok with letting AWS raise the error for us in cases where users specify many of their own containers and get the math wrong, but the case above seems like an easy pitfall for those not familiar with ECS.
@jeanluciano Do you think we should check if the Prefect container's resources are larger than the task's resources and reject the run with a nice error? Or maybe add a note on the new settings that they can't exceed the task level resources.


chore(deps): update react monorepo to v16.14.0

This PR contains the following updates:

Package
Type
Update
Change

react (source)
peerDependencies
minor
16.13.1 -> 16.14.0

react (source)
devDependencies
minor
16.13.1 -> 16.14.0

react (source)
dependencies
minor
16.13.1 -> 16.14.0

react-dom (source)
dependencies
minor
16.13.1 -> 16.14.0

react-test-renderer (source)
devDependencies
minor
16.13.1 -> 16.14.0

Release Notes

facebook/react
v16.14.0
Compare Source
React

Add support for the new JSX transform. (@lunaruan in #18299)

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Disabled by config. Please merge this manually once you are satisfied.
:recycle: Rebasing: Whenever PR becomes conflicted, or if you modify the PR title to begin with "rebase!".
:no_bell: Ignore: Close this PR and you won't be reminded about these updates again.

[ ] If you want to rebase/retry this PR, check this box

Available now for Enterprise: Renovate Pro with real-time webhook handling and priority job queue.

:tada: This PR is included in version 6.1.10 :tada:
The release is available on:

v6.1.10
GitHub release

Your semantic-release bot :package::rocket:


Summary row fixes

What kind of change does this PR introduce? (check one with "x")

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

What is the current behavior? (You can also link to an open issue here)

[x] https://github.com/swimlane/ngx-datatable/issues/1412
[x] https://github.com/swimlane/ngx-datatable/issues/1411
[x] https://github.com/swimlane/ngx-datatable/issues/1417

What is the new behavior?
Does this PR introduce a breaking change? (check one with "x")

[x] Yes
[ ] No

Other information:
Breaking change notice: the default summary function will no longer work with columns with a type different than number - instead of string concatenation, there will be an empty cell.

Hello again, In this PR I want to fix all currently opened issues connected with a summary row. I've created checkboxes in current behaviour section to keep track of progress.

@SirWojtek thank you! Please ping me when you think this is ready for review.

@marjan-georgiev I think the PR is ready to be reviewed. Please take a look at it when you'll have some time.

Looks good, also tested using it with scrollbarV.
Since there is a breaking change, the next release will be 13.0.0.
@SirWojtek thank you for all the effort.


Edit / add synonyms to model

Any possibility of adding support to edit "Synonyms" for PBI models? I have tried adding synonyms and see no change in the Tabular Editor. Would be fantastic to do this from the editor as well.

I’d like to support this, but last time I checked, Power BI was using a weird XML-based format to store the linguistic metadata embedded within the Model metadata. The AMO libs used by Tabular Editor to manipulate the TOM does not support this format. It does expose a set of properties and classes related to linguistic metadata, but it does not seem to work with Power BI files. Also, adding linguistic metadata to SSAS Tabular models is not supported by Power BI (as Q&A is not supported with Live connections), so it’s quite a mess at the moment, and I’m not really sure what to make of it.

I'd suggest to wait - long term, they should implement it in JSON or at least make it supported. Doing something now means trying to reverse engineer their format / unless it is documented somewhere...

I'm not sure if it is out of preview yet but Q&A is supported for SSAS live connection (https://docs.microsoft.com/en-us/power-bi/service-q-and-a-direct-query)

Thanks - I was not aware that Q&A is now supported for live connections. I will investigate whether the LinguisticMetadata property of the TOM actually works with Power BI Q&A, because in that case, it most definitely makes sense to let you edit these in Tabular Editor.

I'd also be interested in this feature. We develop tabular models deployed on Azure AS. Q&A with Live connection does work pretty well in PBI, we are only missing the possibility of defining synonyms :) Even if not intensively used in production, Q&A is quite good for a nice "wow effect" with some of our management, and it does look promising.
Available for testing if needed.
Many thanks !

Hi,
We are also interested in this feature.
Is there any change in this?
Kind regards,
Benito2313

Unfortunately, I haven't been able to find any documentation on how to use the LinguisticMetadata property. I could certainly expose the property within Tabular Editor, but you'd still need to figure out how the JSON or XML content should be changed in order to add synonyms, etc. Today, you can access the LinguisticMetadata property through scripting only:
// WARNING: When we're accessing the underlying TOM object using .GetMetadataObject(), we're circumventing
// Tabular Editor's undo/redo framework. Any changes made to the tomCulture object in this script, can not
// be undone!
var tomCulture = Model.Cultures["da-DK"].GetMetadataObject() as TOM.Culture;
tomCulture.LinguisticMetadata.Content.Output();

Would it suffice if Tabular Editor exposed the LinguisticMetadata properties for a specific culture in the Property Grid? This would allow you to edit the XML or JSON content as a string. I'd love to provide a better experience, but this is not possible until there's full documentation (or even better, an API!) for the XML/JSON formats!

Tabular Editor 2.13.2 now exposes the linguistic metadata content in the property grid. You can modify the value in the grid, or you can use scripting to set the Content property to any XML or Json value:

Hi,
Thnx for adding this feature.
Is there anything that can be done by adding a field called synonyms ? just like with Annotations, that we can add multiple synonyms.

And also could u add it to the translations tab:

Because you would also like to translate this.
There is some information about it, but i dont know if its enough
https://docs.microsoft.com/en-us/openspecs/sql_server_protocols/ms-ssas-t/dd77faf6-2bef-457e-b7a0-3add19aaab9e
https://docs.microsoft.com/en-us/openspecs/sql_server_protocols/ms-ssas-t/f8924a45-70da-496a-947a-84b8d5beaae6
I also have an output from a pbix file with the formating:

I know this is attached to an translation, but you would like to store the content in a main place just like you do with the description and name and so on.
And when i use metadata translator i would like to also retrieve those synonyms to also translate them.
Kind regards,
Ben

Version 2.15.2 now attempts to parse the culture Content property, in order to show a comma-separated list of synonyms defined for each table, column, measure, hierarchy and level.
This works as of the March 2021 release of Desktop, but there are no guarantees that this will continue working. Specifically, since the Linguistic Schema Definition Language (which is what Microsoft call the JSON data in the Content property) is not really documented, Microsoft could basically change the structure in a later version of Desktop, potentially breaking Tabular Editor's support for adding/editing synonyms. You have been warned! :-)

Hi,
Thnx for the quick update :)
But where can i add a synonym in the model in tabular editor? I dont want to do this in powerbi desktop, this because we work with AS.
Kind regards,
Ben

@benito2313 I'm not sure if synonyms defined in an SSAS/Azure AS model are supported by Power BI at all. Do you have any links that indicates that this should be the case?
Tabular Editor will let you add/edit synonyms on objects once a JSON-based LSDL schema has been specified on the Content property on your culture. For a Power BI Desktop model, this schema would be specified by Power BI, with Tabular Editor only adding/removing synonym terms (but there's a lot of other stuff going on in the LSDL, which Tabular Editor doesn't touch).
So in your case, you can try to add the following empty LSDL schema as the Content of your culture (I guess Q&A only works with en-US at the moment):
{"Version":"1.2.0","Language":"en-US","DynamicImprovement":"None","Entities":{}}

Once you add this string, Tabular Editor will display a "Synonyms" property on all objects that can have synonyms assigned:

Modifying the "Synonyms" value will basically just update the LSDL schema to include the synonyms:

But again - there are no guarantees that this actually works, since I couldn't find anywhere that says synonyms are supported for live connections against SSAS/Azure AS. But feel free to try it and kindly let us know the result. Thanks!

This look great i'm going to check it out if it works in pbi.
In the mean while i have this link https://powerbi.microsoft.com/en-us/blog/editing-q-a-linguistic-schemas/
Also at the end of the blog there is a download link to the schema.

Please let me know if it was helpful, or that i need to search further for you.
Kind regards,
Ben

Hi Daniel,
I was wondering if the schema is updated for synonyms in tabular editor?
As @Aaron-Meyers suggested in https://github.com/microsoft/Analysis-Services/issues/100 :
The TOM LinguisticMetadata schema can be found at aka.ms/lsdlschema or json.schemastore.org/lsdlschema
I saw there is a new version (3.3.) and I was wondering if it could be updated in TE?
https://json.schemastore.org/lsdlschema-3.3.json
I have also found more documentation from Microsoft that was added/modified on 30-07-2021.
https://docs.microsoft.com/en-us/power-bi/natural-language/q-and-a-tooling-advanced#prerequisites
It’s on line “Download sample .yaml and .pbix files” , at this part of the post you can download a zip with three files

Linguistic Schema Spec.docx

in this file the schema is fully explained!

Sample PBIX
Sample yaml
The direct link to the zip is https://go.microsoft.com/fwlink/?linkid=871858 .

So to summarize we have the schemas and the explanation of the schema.
Is this something that you can use to add more functionality to TE?
Kind regards,
Ben

Hi @benito2313
There will not be better linguistic schema support in Tabular Editor, as long as Microsoft does not provide an API for programatically accessing the schema. Implementing my own API based on the linguistic schema specs would be a tremendous effort, and it would have to me maintained every time there is an update of the schema (which happens often!).
This is in contrast to the TOM itself, where Microsoft provided an excellent API, and Tabular Editor simply builds on top of this. In other words, whenever Microsoft makes any updates to the TOM, I only need to download the latest version of the API and recompile Tabular Editor. In other words, I do not have to manually manipulate the model metadata JSON - the API takes care of that.
Until something similar is available for the LSDL (ideally, some .NET library that can load the LSDL schema from JSON/XML/YAML into an object model and vice versa), please do not expect any improvements in this area.

Hi @benito2313
There will not be better linguistic schema support in Tabular Editor, as long as Microsoft does not provide an API for programatically accessing the schema. Implementing my own API based on the linguistic schema specs would be a tremendous effort, and it would have to me maintained every time there is an update of the schema (which happens often!).
This is in contrast to the TOM itself, where Microsoft provided an excellent API, and Tabular Editor simply builds on top of this. In other words, whenever Microsoft makes any updates to the TOM, I only need to download the latest version of the API and recompile Tabular Editor. In other words, I do not have to manually manipulate the model metadata JSON - the API takes care of that.
Until something similar is available for the LSDL (ideally, some .NET library that can load the LSDL schema from JSON/XML/YAML into an object model and vice versa), please do not expect any improvements in this area.

This makes sense and I'll see what I can do about this. Of course, we have an API internally and besides just providing a nicer interface for working with the JSON it also implements version upgrade which I think would be helpful for Tabular Editor as well. Realistically I have no idea when we can get to exposing this but it's good to know there is interest :)

@aaron-meyers it’s been a while. Any updates on your last comment? :-)


Add subtitle to search query

search indexes are already extended by subtitle
  if (doc.subtitle) {
    index("subtitle", doc.subtitle);
  }

@fuenkchen while you are at it, could you add _id as well? i think its already in the index as well.


Mat1 and Mat2 shapes cannot be multiplied.

Hi, Very Impressive work you have done, i wanted to mention that i am using pre computed resnet features and now trying to run the anomaly detection model training code but i am getting this error:
RuntimeError: mat1 and mat2 shapes cannot be multiplied (1920x2048 and 4096x512)
Can you please help me why am i getting this error?
Regards,

Hi, can you give me the command you are trying to run?

Hey, Resolved The Error, Thanks..


fixes #11

Hi,
this PR fixes issue #11, WRT dependency management.
Changelog

Added a requirements.txt file
Added a .gitignore file
Modified README.md to add installation instructions

!bounty 10


Improve nested html! expansion by omitting VNodes

This doesn't directly resolves the feature request #819 but addresses one underlying issue that provoked #819.

@jstarry I am getting close to wrapping this PR and making it completely work. Once the PR is done, the snippet below will work in the following way:
Example:
let mut test2: VNode = html! {
    <Body>
        {
            let blue = "hello-world";
            return html! {
                <Row>
                    <Cell>{ 1 }</Cell>
                </Row>
            };
        }
        {(0..2).map(|_| {
            return html! {
                <Row>
                    <Cell>{ 1 }</Cell>
                </Row>
            }
        }).collect::<Vec<VChild<Row>>>()}
    </Body>
};

The code that ultimately gets generated will act at though someone just wrote the <Row>...</Row> components straight inside of the body of the <Body></Body> like this:
let mut test2: VNode = html! {
    <Body>
        <Row>
            <Cell>{ 1 }</Cell>
        </Row>
        <Row>
            <Cell>{ 1 }</Cell>
        </Row>
        <Row>
            <Cell>{ 1 }</Cell>
        </Row>
    </Body>
};

This means that effectively HtmlIterable will absolutely no longer be needed and it will be possible to merge HtmlNode and BlockContent into one. The question is whether or not you would be okay if I completely deprecate those inside of this PR?

Also, not sure why the CI is failing but running the test suite locally, all of the existing tests pass with the PR already as is.

Also, not sure why the CI is failing but running the test suite locally, all of the existing tests pass with the PR already as is.

Yeah, don't worry about CI right now. rust 1.40.0 broke building with emscripten

This means that effectively HtmlIterable will absolutely no longer be needed and it will be possible to merge HtmlNode and BlockContent into one. The question is whether or not you would be okay if I completely deprecate those inside of this PR?

Sounds great to me, feel free

@jstarry Awesome. I am committing and pushing last update right now to fix a bug that wasn't allowing for different types of nested children and this is ready. I will deprecate those in a separate PR. To keep things cleaner.

@trivigy I fixed some rebase conflicts, can you check them out? Let's get this in tonight :)

@trivigy I fixed some rebase conflicts, can you check them out? Let's get this in tonight :)

already on it. :nerd_face:

Thanks!!


ESLint files Pattern

When i run npm run lint, i get the following message
> office-addin-lint check

Oops! Something went wrong! :(

ESLint: 6.8.0.

No files matching the pattern "src/**/*.js" were found.
Please check for typing mistakes in the pattern.

I'm writing a taskpane for Excel with Typescript and React.
I haven't changed anything on the eslint configuration.
I'm have this on my package.json
 "office-addin-lint": "^1.0.21",

Thanks.

Do you have your sources files in an src folder? Do they have a .js extension?

Yeah, i used the generator, so the structure is the default one for the React Typescript project.
Also, the docs mention that the default pattern is src/**/*.{ts,tsx,js,jsx}
Here is an example i created that also fails with the same error on the linter.
https://github.com/abimaelmartell/excel-stock-price-pane
As you can see i have tsx files on the src.

Usually setting "no-error-on-unmatched-pattern": 0 in .eslintrc.json fixes this, but oddly this issue is persisting for me even after adding it to my project root's config file and the config file found in office-addin-lint/config. (I'm doing the same thing as @abimaelmartell, trying to write a taskpane for Excel with Typescript+React.)

@abimaelmartell Thank you again for reporting this issue. We are not able to reproduce the issue in house. Are you able to create new add-ins using the template, and they all exhibit the same issue?

No additional information provided.

FFR: The true answer is here
office-addin-lint prettier --files="src/**/*.{tsx,ts}"
hth

Is the issue here that TypeScript is used and there are no *.js files? @millerds

@akrantz the issue is that the documentation is unclear how to specify which files to scan for the linter option.

@akrantz, the lint command works just fine when there aren't any .js files present (like in the react template repo) so I'm don't believe it's a case of which files are present
@rahulahoop the lint package did not include a way to customize the linting that is done as it used its own built-in configuration (the local .eslintrc.json files was for the live linting in VS Code or other editors).  However, I just finished publishing an update to the lint package that now looks for the local .eslintrc.json file and uses that over the built in one.
I would also note that there have been a number of updates to the linting tools since this issue was originally opened 1.5 years ago.  I would like to see the current versions of the packages used to see if the issue continues (without any local .eslintrc.json overrides).

@abimaelmartell @Keating950 @rahulahoop . . . if you are still seeing issues with this using current versions of the templates and script packages please reopen.


(maint) fix install repos on to use PC1 urls

this change helps the install utilities find repo_configs from our new
PC1 URLS on build.puppetlabs.lan.  this WILL break again (PC2?) and we need to
refactor and repackage all this code elsewhere, sometime, RSN.

This change was made in stable, but not master (because we're not merging stable to master any time soon). Does this really need to go master?
Also the version in stable checks PC1 -> products -> devel, but this one does products -> PC1 -> devel.

this or something similar should go in when master pipelines are setup again
i have no clue of what order of precedence the dirs should be in.


chore: upgrade to prettier 3

Codecov Report

Merging #352 (31e4983) into main (f7a9660) will increase coverage by 0.00%.
The diff coverage is 83.92%.

@@           Coverage Diff           @@
##             main     #352   +/-   ##
=======================================
  Coverage   88.30%   88.31%
=======================================
  Files          72       72
  Lines        4566     4569    +3
  Branches      576      576
=======================================
+ Hits         4032     4035    +3
  Misses        533      533
  Partials        1        1

Impacted Files
Coverage Δ

...ackages/swagger-ts/src/generators/TypeGenerator.ts
66.76% <37.50%> (ø)

packages/core/src/build.ts
78.86% <50.00%> (ø)

...kages/swagger/src/generators/OperationGenerator.ts
89.32% <90.00%> (+0.12%)
:arrow_up:

packages/ts-codegen/src/codegen.ts
93.11% <94.73%> (ø)

packages/core/src/config.ts
100.00% <100.00%> (ø)

...kages/core/src/managers/fileManager/FileManager.ts
80.15% <100.00%> (ø)

packages/core/src/managers/fileManager/types.ts
100.00% <100.00%> (ø)

...s/core/src/managers/pluginManager/PluginManager.ts
93.05% <100.00%> (ø)

packages/core/src/utils/read.ts
92.06% <100.00%> (ø)

packages/core/src/utils/write.ts
82.22% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


New Issue and PR templates plus contribution docs

Please check if this PR fulfills these

[x] The commit message is descriptive of changes
[x] You have tried at least one local test (this includes viewing the changes on your local fork)
[x] Docs have been added / updated as needed (for bug fixes / features)

What kind of change does this PR introduce?

[ ] Typo / Doc Update
[ ] Bug Fix
[x] Feature / New Item

What is the new behavior (if this is a feature change)?

[ ] Added Interview Technical Question
[ ] Added Interview Whiteboard Question
[ ] Corrected existing items
[x] Other

Other information:

Adding Issue Template
Adding Pull Request Template
Adding Contributing File
Issue #27

@izulien I made a few suggestions in my review. This is an awesome addition to the project. Thank you so much for all of your work!

Items corrected, though there was a fun merge conflict that got a bit wonky.  Guess that's what I get for not pulling before making changes ;)

Fantastic! Thank you so much for all of the effort you put into this @izulien!


Fix typos in docs/reference/builder.md.

Fix typos in docs/reference/builder.md.
Signed-off-by: Ding Fei<EMAIL_ADDRESS>

@thaJeztah Thanks for the review. Requested changes are squashed into the first commit, please take a look again.


Small text update - missing "> Setup"

The text is:  "2. Go to System administration > Operational Insights."
The text should be : "2. Go to System administration > Setup >Operational Insights."

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

ID: 1ea563b0-eb9c-bc38-f17a-f24913eb426f
Version Independent ID: 1f9c1430-a53c-9a63-77fe-dc4b44b7c3a7
Content: Commerce component events for diagnostics and troubleshooting - Commerce | Dynamics 365
Content Source: articles/commerce/dev-itpro/retail-component-events-diagnostics-troubleshooting.md
Service: dynamics-365-commerce
Product: ****
Technology: ****
GitHub Login: @ShalabhjainMSFT
Microsoft Alias: shajain

Bug 754626 has been logged to address this issue.

This should likely be assigned to Jarad Shannon.

Bug 757246 has been logged to address this issue.

This update will be published today. Thank you.


Lowercase Timestamps variants

Description
I changed timestamp variants form UPPERCASE to TitleCase. Fixes #39
Checklist

[ ] Formatted code using cargo fmt --all
[ ] Linted code using clippy cargo clippy --all-targets --all-features -- -D warnings
[ ] Updated README.md using cargo readme > README.md
[ ] Reviewed the diff. Did you leave any print statements or unnecessary comments?
[ ] Any unfinished work that warrants a separate issue captured in an issue with a TODO code comment

Thanks for this PR! 🎉
The CI is failing due to code style. Please run cargo fmt --all and commit, so we can merge this!


[SPARK-45224][PYTHON] Add examples w/ map and array as parameters of sql()

What changes were proposed in this pull request?
In the PR, I propose to add a few more examples for the sql() method in PySpark API with array and map parameters.
Why are the changes needed?
To inform users about recent changes introduced by #42752 and #42470, and check the changes work actually.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
By running new examples:
$ python/run-tests --parallelism=1 --testnames 'pyspark.sql.session SparkSession.sql'

Was this patch authored or co-authored using generative AI tooling?
No.

Merging to master. Thank you, @HyukjinKwon and @cloud-fan for review.


What are units of accelerometer and gyroscope?

Hello,
in documentation is written about accelerometer and gyroscope, but nothing about their units.
Is gyroscope in deg/s or rad/s?
Is accelerometer in m/s2?
Thanks a lot.
Have a nice day.

Gyroscopes always report rad / s.
Accelerometers report length-units / s^2. The actual length units are generally undefined, though they can often be inferred from the value of gravity under the assumption that you are on Earth. For the default (0, 0, -9.81), if you are on the surface of Earth, the unit of length can be inferred to be Metres. However if you are in free space with no gravity, the length unit is completely open to your own interpretation, as are units of force and mass etc, as long as they are all consistent. To phrase this with an example: the dynamics of a 1 metre spaceship that weighs 1 Kg and has a 1 Newton thruster are exactly equivalent to those of a 1 cm spaceship that weighs 1 gram and has a 1 Dyn thruster.

That said, since this is a common question, we will add a section in the documentation to clarify.
Thanks for bringing this to our attention.

Following up, see here

@yuvaltassa
Thanks, it's very clear... 🙂
Thanks.


.Net GoogleAI/VertexAI public api for text and chat

Description

State of connector progress https://github.com/microsoft/semantic-kernel/issues/4680
Spit from PR https://github.com/microsoft/semantic-kernel/pull/4407
Contribution Checklist

[x] The code builds clean without any errors or warnings
[x] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[x] All unit tests pass, and I have added new tests where possible
[x] I didn't break anyone :smile:

@RogerBarreto


ImagesTable: Ensure created_at date for a new image

This adds a temporary value of current date to created_at for newly created images.
This is not the most elegant solution and will be reworked after the ImagesTable refactor.

/retest


Added Classic Computer Science Problems in Python

Hello,
Would you please consider adding this book to your other Manning publications.
Thank you for your time

Thank you! :cake: :snake:


Use polynomial approximation for tanh, sin, cos, log and exp

Before this patch, math.tanh is expanded into math.exp, math.sub, math.add, and math.div by calling populateExpandTanhPattern, which is not efficient.
This patch calls populateMathPolynomialApproximationPatterns that does polynomial approximation, which leads to better inference performance.
Another difference is that populateMathPolynomialApproximationPatterns works with vector type while populateExpandTanhPattern does not.
LSTM and GRU use math.tanh as an activation, so they would become a little faster.
populateMathPolynomialApproximationPatterns also supports sin, cos, log, and exp.
Signed-off-by: Tung D. Le<EMAIL_ADDRESS>

@tungld Please don't forget that we are now merging against main, not master anymore.

Jenkins Linux ppc64le Build #1943 [push] Use polynomial approxima... started at 01:38

Jenkins Linux s390x Build #2834 [push] Use polynomial approxima... started at 01:35

Jenkins Linux amd64 Build #2836 [push] Use polynomial approxima... started at 00:35

Jenkins Linux s390x Build #2834 [push] Use polynomial approxima... passed after 44 min

Jenkins Linux ppc64le Build #1943 [push] Use polynomial approxima... passed after 56 min

Jenkins Linux amd64 Build #2836 [push] Use polynomial approxima... passed after 57 min


Does this plugin provide 3D view mode?

I'm submitting a ...  (check one with "x")

[x] question
[ ] any problem or bug report

OS:  (check one with "x")

[ ] Android
[x] iOS
[ ] Browser

Question:
I want to use 3D view functionality like in google maps.
I have not seen anything mentioned in this plugin documentation?
Does this plugin provide that feature or will it be provided in the future?

A reply would be highly appreciated
Thank you for your cooperation.

map.moveCamera({
  zoom: 19,
  tilt: 30
});


Slightly ambiguous let() wiki documentation.

In the Wiki, let is described as:

let(subject, aBlock) declares a local helper variable that is re-initialized before every example.

I have a few questions based on this description:

Is "example" synonymous with it block?
Is the execution of a let limited to the context it is in, as well as all sub-contexts (just like beforeEach)?

My understanding of how let works is that this test code...
context(@"given a context", ^{
    let(someMock, ^id{
        return [some nullMock];
    });
});

...is equivalent to this test code:
context(@"given a context", ^{
    __block id someMock;

    beforeEach(someMock, ^id{
        someMock =  [SomeClass nullMock];
    });
});

Is this accurate?

Yep, that's the intended use case. Just like a beforeEach, let variables are reinitialised before each example (i.e., it block) runs.
Closing this for now. If there's anything further you'd like to clarify, please ask!

Thanks!


Find cycle in a Linked List

Submission Title
Find cycle in a Linked List
Submission Type
Data Structure
Submission Description
Given a linked list, check if the linked list has loop or not
Submission Directory
Main_Folder => Data_Structures => Linked_List => Singly_Linked_List => DetectCycle.cpp
Contributors

[x]  C++ -
[x]  C -
[x]  Python -
[x] Java -
[ ] JavaScript -
[ ] TypeScript -
[ ] Golang -
[ ] Ruby -
[ ] php -
[ ] Swift -
[ ] C# -

Can I work on this issue, please review.

Can I code this in cpp

Hello, can I work on this issue?

Can i implement this using C++?

Can I contribute in Java?

Can I implement this in Python?

Hi @garimamishraa
Please opt a different language/issue :v:

Can I write in C#? Please assign it to me.

Hi, it seems that the C++ code has already been submitted and merged, can I submit the solution in Golang?

Hey, I would like to submit it with TypeScript

hey can i do this in javascript??

I would like to work on this in C++. Can you please assign it to me?

I'd like to do it with Ruby 🙌

I would like to do it with python with detailed explanation using tortoise method:-)

I would like to do it in swift using tortoise method:-)

can you please assign it to me?

Hey @Shreyash0713 ., i would like to work on this issue in Golang. Can you assign this to me..

thanks everyone :v:


Use codecov instead of coveralls

Close #17.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@3f41844). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #18   +/-   ##
=========================================
  Coverage          ?   94.79%
=========================================
  Files             ?        5
  Lines             ?       96
  Branches          ?       16
=========================================
  Hits              ?       91
  Misses            ?        4
  Partials          ?        1

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 3f41844...3e9bfed. Read the comment docs.


add tolerations

Signed-off-by: Benjamin<EMAIL_ADDRESS>

This is a great example! It does seem pretty specific to your setup. I'd prefer if this was just documentation maybe, what do you think?

I've made some modifications to make it more generic and to suit more toleration cases.
https://github.com/coreos/kube-prometheus/pull/102/commits/6f48bdb5a638cd8c4e122d68ae7e22427b4b476a
Let me know which doc you prefer to add it to , thanks. :)
Ben

@brancz I've moved the toleration to a new file named tolerations.libsonnet in examples folder.
Is it ok for you ?
Thanks
Ben

yep that looks good! thanks!


Cracking Hash-Code #11

Description
Cracks the password for hash-code
Fixes #(11)
Replace issue_no with the issue number which is fixed in this PR
Have you read the Contributing Guidelines on Pull Requests?
Yes

@avinashkranjan Sir please merge it now

@avinashkranjan Sir please merge it now

@avinashkranjan sir I was the first one to send the PR. Then u should u reviewed it first.

@avinashkranjan sir I was the first one to send the PR. Then u should u reviewed it first.

We received a better script for the same issue so, closing this now...

We received a better script for the same issue so, closing this now...

U could have said it 5 days back when I first sent it

U could have said it 5 days back when I first sent it

@avinashkranjan sir I was the first one to send the PR. Then u should u reviewed it first.

It's not about first and second it about which is more better... And Since It was a level-5 issue and you just used md-5 when I mentioned minimum two..

@avinashkranjan sir I was the first one to send the PR. Then u should u reviewed it first.

It's not about first and second it about which is more better... And Since It was a level-5 issue and you just used md-5 when I mentioned minimum two..

U could have said it 5 days back when I first sent it

You could have worked on other issues till then..I'll review PRs when I get time..

U could have said it 5 days back when I first sent it

You could have worked on other issues till then..I'll review PRs when I get time..


fix bug pull request files will be broken if head repo was transfered to another user or orgnization

Fix #8563, but this PR will not fix the broken pull requests because the bug because we cannot add a migration on this release branch. There are two workarounds to fix that.

Transfer the head repository to someplace and transfer it back.
Update the database to change the head_user_name to the correct one on table pull_request.

This PR is a manually back port from #8564 .

Would it not be possible to replace every call to headusername with the equivalent looking up from the headrepo id?

@zeripath Did you mean we will not drop that column, but we could do not use that column?

@lunny that could probably be better

we can leave that column for 1.9 and drop only in 1.10 but fix not to use it already in 1.9

@lafriks drop column cannot be in v1.10 I think, only in v1.11 since release v1.10 branched.

It can, as long as no PR is merged that has migration that won't be backported and migrations does not go out of sync for 1.10 and master branches

OK. So please review #8564 at first.


Implement Math Algorithms

Math algorithms to be implemented are:

[x] GCD
[x] LCM
[x] Multiplication Expo Squaring
[ ] Find Divisors
[ ] Fibonacci
[ ] Fisher Yates
[ ] Check if number is a prime
[ ] Calculate square root using Newton's method
[x] Extended Euclid Algorithm
[ ] Mod Inverse
[ ] Mod Inverse using Fermet
[ ] Euler's Totient Function
[ ] Euler's Inverse method
[ ] Inverse of first 'n' numbers
[ ] Combination Modular

Working on this one

is this issue still available? can i work on this?

@manrajgrover  I would like to do Euler's Totient Function

/request I would like to work on the Fibonacci algorithm!

I will do Find Divisors

I would like to work on check if the number is a prime. Thank you!

I will do Fisher Yates!

I will do  Inverse of first 'n' numbers

I will do Mod Inverse, Mod Inverse using Fermet, and Fisher Yates

I will implement Leonardo Numbers Sequence

I will work on the inverse of first n numbers and check if a number is prime.

I am taking up Mod Inverse and Mod Inverse using Fermat.

I will do the Combination Modular.

I would like to do Fibonacci and check if prime

is this issue closed? or which functions are missing?

i can do find divisors and square root by newton's method

Is this still open. I would like to contribute if anything not done yet.

I want to contribute.

I also want to contribute.

Is there any issue which requires contribution?? I want to contribute.

I want to contribute. Is there anything I can work on ?

If any of the above tasks are left to solve , kindle let me know!__

Hello, is any of tasks available?

Hello ,
Is there any task available now??

Implemented Fisher Yates: https://github.com/manrajgrover/algorithms-js/pull/154

Implemented Fibonacci: #155

Implemented Check if Number is a Prime: 155


Upgrade YAML requirements

I can't install PyYAML so I need this

I also wonder if It can be more lenient with the versions

this shouldn't happen, can you explain what exactly you do and post which errors you receive?
normally you would make a virtual environment and install all the packages you need into it, including osu-wiki-tools

I get this:
Terminal output
~ $ pip install pyyaml==6.0                             Collecting pyyaml==6.0
  Using cached PyYAML-6.0.tar.gz (124 kB)                 Installing build dependencies ... done
  Getting requirements to build wheel ... error
  error: subprocess-exited-with-error                                                                             × Getting requirements to build wheel did not run successfully.
  │ exit code: 1
  ╰─> [54 lines of output]
      running egg_info
      writing lib/PyYAML.egg-info/PKG-INFO                    writing dependency_links to lib/PyYAML.egg-info/dependency_links.txt
      writing top-level names to lib/PyYAML.egg-info/top_level.txt                                                    Traceback (most recent call last):
        File "/data/data/com.termux/files/usr/lib/python3.11/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 353, in <module>
          main()                                                File "/data/data/com.termux/files/usr/lib/python3.11/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 335, in main
          json_out['return_val'] = hook(**hook_input['kwargs'])
                                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/data/data/com.termux/files/usr/lib/python3.11/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 118, in get_requires_for_build_wheel
          return hook(config_settings)
                 ^^^^^^^^^^^^^^^^^^^^^
        File "/data/data/com.termux/files/usr/tmp/pip-build-env-9bdkpc5l/overlay/lib/python3.11/site-packages/setuptools/build_meta.py", line 325, in get_requires_for_build_wheel
          return self._get_build_requires(config_settings, requirements=['wheel'])
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/data/data/com.termux/files/usr/tmp/pip-build-env-9bdkpc5l/overlay/lib/python3.11/site-packages/setuptools/build_meta.py", line 295, in _get_build_requires
          self.run_setup()
        File "/data/data/com.termux/files/usr/tmp/pip-build-env-9bdkpc5l/overlay/lib/python3.11/site-packages/setuptools/build_meta.py", line 311, in run_setup
          exec(code, locals())
        File "<string>", line 288, in <module>
        File "/data/data/com.termux/files/usr/tmp/pip-build-env-9bdkpc5l/overlay/lib/python3.11/site-packages/setuptools/__init__.py", line 103, in setup
          return distutils.core.setup(**attrs)
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/data/data/com.termux/files/usr/tmp/pip-build-env-9bdkpc5l/overlay/lib/python3.11/site-packages/setuptools/_distutils/core.py", line 185, in setup
          return run_commands(dist)
                 ^^^^^^^^^^^^^^^^^^
        File "/data/data/com.termux/files/usr/tmp/pip-build-env-9bdkpc5l/overlay/lib/python3.11/site-packages/setuptools/_distutils/core.py", line 201, in run_commands
          dist.run_commands()
        File "/data/data/com.termux/files/usr/tmp/pip-build-env-9bdkpc5l/overlay/lib/python3.11/site-packages/setuptools/_distutils/dist.py", line 969, in run_commands
          self.run_command(cmd)
        File "/data/data/com.termux/files/usr/tmp/pip-build-env-9bdkpc5l/overlay/lib/python3.11/site-packages/setuptools/dist.py", line 963, in run_command
          super().run_command(command)
        File "/data/data/com.termux/files/usr/tmp/pip-build-env-9bdkpc5l/overlay/lib/python3.11/site-packages/setuptools/_distutils/dist.py", line 988, in run_command
          cmd_obj.run()
        File "/data/data/com.termux/files/usr/tmp/pip-build-env-9bdkpc5l/overlay/lib/python3.11/site-packages/setuptools/command/egg_info.py", line 321, in run
          self.find_sources()
        File "/data/data/com.termux/files/usr/tmp/pip-build-env-9bdkpc5l/overlay/lib/python3.11/site-packages/setuptools/command/egg_info.py", line 329, in find_sources
          mm.run()
        File "/data/data/com.termux/files/usr/tmp/pip-build-env-9bdkpc5l/overlay/lib/python3.11/site-packages/setuptools/command/egg_info.py", line 551, in run
          self.add_defaults()
        File "/data/data/com.termux/files/usr/tmp/pip-build-env-9bdkpc5l/overlay/lib/python3.11/site-packages/setuptools/command/egg_info.py", line 589, in add_defaults
          sdist.add_defaults(self)
        File "/data/data/com.termux/files/usr/tmp/pip-build-env-9bdkpc5l/overlay/lib/python3.11/site-packages/setuptools/command/sdist.py", line 112, in add_defaults
          super().add_defaults()
        File "/data/data/com.termux/files/usr/tmp/pip-build-env-9bdkpc5l/overlay/lib/python3.11/site-packages/setuptools/_distutils/command/sdist.py", line 251, in add_defaults
          self._add_defaults_ext()
        File "/data/data/com.termux/files/usr/tmp/pip-build-env-9bdkpc5l/overlay/lib/python3.11/site-packages/setuptools/_distutils/command/sdist.py", line 336, in _add_defaults_ext
          self.filelist.extend(build_ext.get_source_files())
                               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "<string>", line 204, in get_source_files
        File "/data/data/com.termux/files/usr/tmp/pip-build-env-9bdkpc5l/overlay/lib/python3.11/site-packages/setuptools/_distutils/cmd.py", line 107, in __getattr__
          raise AttributeError(attr)
      AttributeError: cython_sources
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error

× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> See above for output.

note: This error originates from a subprocess, and is likely not a problem with pip.

not sure what your setup is (OS, and Python distribution -- which pip; pip --version), but this seems to be an issue for a specific combination of PyYAML + OS + interpreter, where pip cannot compile an installation package from ground up. I'd usually recommend installing python3-dev and retrying, but there seems to be more than that:

https://github.com/yaml/pyyaml/issues/601
https://github.com/pytorch/pytorch/pull/115956

I also wonder if It can be more lenient with the versions

technically yes, since >= exists, but I'd argue that dependency pinning ensures stability

this'll be in the next release whenever that happens, unless this is somehow required atm. use your local build for now


HTML and CSS must follow semantic rules

WHEN I view the source code
THEN I find semantic HTML and CSS elements

All code has been updated to follow semantic elements


Update heroku docs

Codecov Report

Merging #1252 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1252      +/-   ##
==========================================
- Coverage   70.24%   70.23%   -0.02%
==========================================
  Files         118      118
  Lines        6356     6356
  Branches      812      812
==========================================
- Hits         4465     4464       -1
  Misses       1730     1730
- Partials      161      162       +1

Impacted Files
Coverage Δ

saleor/core/utils/random_data.py
81.57% <0%> (-0.38%)
:arrow_down:

Continue to review full report at Codecov.

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


Added db_get_param for rendering the .databrickscfg file

renders the .databrickscfg file for parameters used in brickster

Modified db_wsid(), db_token() and db_host() to use db_get_param()

I think the flow should work slightly differently to what is currently proposed.
Want to support the reading of both a .databrickscfg file and also .Renviron which is the default and surface the switching of those ideally through setting of options.
My current thinking is similar to this:
db_host <- function(id = NULL, prefix = NULL, env = getOption("db_env")) {

  # if option `use_databrickscfg` is `TRUE` then fetch the associated env.
  # env is specified via `db_env` option, if missing use default.
  # this behaviour can only be changed via setting of config
  if (getOption("use_databrickscfg", FALSE)) {

    # TODO:
    # depending on value of `env` get appropriate data from `.databrickscfg`

  }

  if (is.null(id) && is.null(prefix)) {

    if (is.null(env)) {
      host <- Sys.getenv("DATABRICKS_HOST")
    } else {
      host <- Sys.getenv(paste("DATABRICKS_HOST", upper(env), sep = "_"))
    }

    if (host == "") {
      stop(format_error(c(
        "`DATABRICKS_HOST` not found in `.Renviron`:",
        "x" = "Need to specify `DATABRICKS_HOST` within `.Renviron` file."
      )))
    }
  } else {
    host <- paste0("https://", prefix, id, ".cloud.databricks.com")
  }

  host
}

In this case db_host() would be expected to behave is does today without change unless specifying options.
Setting option(db_env = "prod") would then search for DATABRICKS_HOST_PROD in .Renviron.
Setting both option(db_env = "prod") and option(db_env = "use_databrickscfg") would search .databrickscfg for host under the "prod" profile.
Setting only option(db_env = "use_databrickscfg") would use the default profile's host in the .databrickscfg file.

OK, let me summarise what i understand from your suggestions:

Both .Renviron and .databrickscfg should be supported
Switching between different configs is controlled through two options:db_env and use_databrickscfg.
If option use_dbconfig is not set (it's NULL), the we use .Renviron
The option db_env is used to either switch between profiles in .databrickscfg or to select variable from .Renviron defined for that specific environment (by adding a suffix _<db_env> i.e. DATABRICKS_HOST _PROD

The side effect of this approach is to have exactly the same profile names and environment suffixes.

Thanks @skodman.
I've been mulling over the changes and I've decided on a direction, I'm going to merge these changes into the multi-env-support branch and then tweak things before re-raising a PR into main.
Once I've made my changes will still need help to add unit tests and check behaviour on windows before this reaches main branch.

Refer to https://github.com/zacdav-db/brickster/pull/22 for new pull request.


tf.io.gfile.GFile does not raise an error when given a directory

Python's open does not accept directories. The following results in an IsADirectoryError:
    with open('.', 'r') as f:
        print(f.readlines())

However, if you use tf.io.gfile.GFile instead, you will get an empty list and no error:
    with tf.io.gfile.GFile('.', 'r') as f:
        print(f.readlines())

Not sure if this is a feature or a bug, but it is not the behavior I would expect, given that the documentation says that tf.io.gfile is meant to provide an API close to Python's file IO objects.

@MiladShahidi
Thanks for filing the issue.
Please, fill issue template.
Please, share colab link or simple stndalone code with supporting files to reproduce the issue in our environment. It helps us in localizing the issue faster.
Are you interested in submitting PR for this?
Thanks!

@MiladShahidi
Thanks for filing the issue.
Please, fill issue template.
Please, share colab link or simple stndalone code with supporting files to reproduce the issue in our environment. It helps us in localizing the issue faster.
Are you interested in submitting PR for this?
Thanks!

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): MacOS Mojave 10.14.6
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 2.4.0
Python version:  3.7.5
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:

Describe the current behavior
The tf.io.gfile.GFile function does not raise an error when given a directory, whereas Python's open does. Given that the documentation says the role of the gfile module is to provide an API that's close to Python's file IO API, it would be reasonable to expect it to raise an error when the provided address points to a directory.
Describe the expected behavior
the GFile method should raise an error when given a directory.
Standalone code to reproduce the issue
https://colab.research.google.com/drive/1qR5b7M5SbpepOX9b9Jlru3fPOfCWqVxa?usp=sharing

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): MacOS Mojave 10.14.6
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 2.4.0
Python version:  3.7.5
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:

Describe the current behavior
The tf.io.gfile.GFile function does not raise an error when given a directory, whereas Python's open does. Given that the documentation says the role of the gfile module is to provide an API that's close to Python's file IO API, it would be reasonable to expect it to raise an error when the provided address points to a directory.
Describe the expected behavior
the GFile method should raise an error when given a directory.
Standalone code to reproduce the issue
https://colab.research.google.com/drive/1qR5b7M5SbpepOX9b9Jlru3fPOfCWqVxa?usp=sharing

Thank you for your response @ravikyram. I'd be more than happy to submit a PR for this.

Thank you for your response @ravikyram. I'd be more than happy to submit a PR for this.

@MiladShahidi
Thanks for your interest in submitting PR.
I have tried in colab with TF version 2.3, 2.4 and nightly version(2.5.0-dev20210110) and was able to reproduce the issue.Please, find the gist here.
Please, submit a PR .Thanks!

@MiladShahidi
Thanks for your interest in submitting PR.
I have tried in colab with TF version 2.3, 2.4 and nightly version(2.5.0-dev20210110) and was able to reproduce the issue.Please, find the gist here.
Please, submit a PR .Thanks!

Known issue, should be fixed in 2.5 with the switch to modular filesystems.

Known issue, should be fixed in 2.5 with the switch to modular filesystems.

Thanks @mihaimaruseac. So, should I still submit a PR? Or is it already taken care of?

Thanks @mihaimaruseac. So, should I still submit a PR? Or is it already taken care of?

I think this is handled already, thank you for offering to send a PR.
https://cs.opensource.google/tensorflow/tensorflow/+/master:tensorflow/c/experimental/filesystem/plugins/posix/posix_filesystem.cc;l=208-212;drc=f64e839687b6129dfe2a9e21ef965593c7f3a3c2

I think this is handled already, thank you for offering to send a PR.
https://cs.opensource.google/tensorflow/tensorflow/+/master:tensorflow/c/experimental/filesystem/plugins/posix/posix_filesystem.cc;l=208-212;drc=f64e839687b6129dfe2a9e21ef965593c7f3a3c2

I executed the code in Nightly 2.6 and  not getting any error message when passing directory. Please find the gist here.Thanks!


emotify-aryanbaburajan

Hi, I've been programming for over 3 years now, and am familiar with plenty of frameworks and languages. I've developed a few video games and web applications.
This is a random emoji generator which generates a random emoji based off a number of combinations. (A total of 18 combinations are possible!)

Thanks for the submission! There have been a few emoji generators already but I appreciate how this one uses shapes with area and not just open curves to represent features.
 You can order it here.

@leomcelroy Thank you!
I have an issue with the form:

Upload a photo of your student ID or class schedule (with date) so we know you're a student!

My school provides neither student ID cards or any form of class schedule. I just finished my 11th exams, so would uploading a scan of my 11th final exam's hall ticket be enough? It also contains my name and photo.


The layout of 'Upload Blob' dialog is odd with 'Use AzCopy for Improved Upload and Download' checked

Storage Explorer Version: 1.5.0
Platform/OS Version: Windows 10/ Linux Ubuntu 16.04/ MacOS High Sierra
Architecture: ia32/x64
Build Number: 20180926.5
Commit: d6717c99
Regression From: Not a regression
Steps to Reproduce:

Navigate to 'Experimental' menu -> Check 'Use AzCopy for Improved Upload and Download'.
Open one blob container then open Upload dialog.

Expected Experience:
The layout of Upload dialog shows well.
Actual Experience:
The layout of Upload dialog is odd.

This was a build issue.


Update backtick references in pdoc

Codecov Report

Merging #109 (4aef2e6) into main (336d5f3) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 4aef2e6 differs from pull request most recent head 4ff3c89. Consider uploading reports for the commit 4ff3c89 to get more accurate results

@@           Coverage Diff           @@
##             main     #109   +/-   ##
=======================================
  Coverage   97.22%   97.22%
=======================================
  Files          41       41
  Lines        2124     2124
=======================================
  Hits         2065     2065
  Misses         59       59

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Use com.typesafe.sbt as the artifact group id

Minor annoyance: I always forget how eed3si9n is spelt :P
Moreover, most plugins in the sbt github organization already fall under com.typesafe.sbt

Moreover, most plugins in the sbt github organization already fall under com.typesafe.sbt

Given that most of the plugins are not actively maintained by Lightbend (e.g. sbt-native-packager), I would say we should go the opposite direction and give them some other group id to distinguish them from the artifacts that Lightbend maintains.
http://www.scala-sbt.org/0.13/docs/Community-Plugins.html:

The sbt organization is available for use by any sbt plugin. Developers who contribute their plugins into the community organization will still retain control over their repository and its access. The goal of the sbt organization is to organize sbt software into one central location.

As noted above, sbt GitHub organization is open to hosting plugins created by community members, and I made sbt-assembly by forking Coda Hale's assembly-sbt in 2011 long before joining Lightbend.

Si @evacchi lo stato è pessimo https://github.com/sbt/sbt-release/issues/196


Fix signature for Array#at

Motivation

If you want a similar method that will raise instead of returning nil
sometimes, you can use Array#fetch instead.
Test plan

See included automated tests.

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build results here:
→ https://go/builds/bui_LEi2wldfWMLchh
→ https://go/builds/bui_LEi2EexjjP2o7T


mag eHp and general mitigation bug

Hi
If you try to make a build with say hp * spr > 25200000 (aigion trial stuff lol) and you have general mitigation set to 50%..
The builds you get will have a mag ehp  of double that amount. (as if there was no mitigation at all)
Below is a build I'm doing with 50% global mitigation. (i'm calculating for 12600000 as that's how I get the correct mag eHP of 25200000)

That's not wrong. I mean, the displayed in the builder m eHP is not just HP * SPR, but also takes mitigation into account.

yes but if you see that I'm aiming for (in that image) 12600000 eHP buit the build tries to hit 25200000 eHP

The MeHP value displayed isn't actually HPSPR, it is HPSPRMitigtion; the build displayed shows 17717 HP and 743 SPR, which is 13,163,731 (slightly over the 12.6m value you are aiming for). So the builder is building for the goal you asked it for, but you asked it for HPSPR specifically, not HPSPRMitigation. Since you can't currently enter mitigation as a goal like that, you just have to manually lower your goal to account for the mitigation you have.

I think @Darwe-Canine said it all. Do you have something to add, @cloudix1 ? Else I'll close the issue.


Support GroupsAccumulator accumulator for User Defined Functions

Is your feature request related to a problem or challenge?
Many of DataFusion's built in accumlators use the GroupsAccumulator interface for a significant performance improvement
See the blog post for more details
However, it is not currently possible to use [GroupsAccumulator] with user defined Aggregate functions , which is less than ideal for 2 reasons:

User's can't take advantage of the faster interface
We can't port built in Aggregate functions to use the same API https://github.com/apache/arrow-datafusion/issues/8708

Describe the solution you'd like
I would like to extend the AggregateUDFImpl trait added by @guojidan  in https://github.com/apache/arrow-datafusion/pull/8733 with a new method that returns a GroupsAccumulator

Add new methods to AggregateUDFImpl
Wire in the the call into the impl for AggregateExpr https://github.com/apache/arrow-datafusion/blob/dd58c5a7b0069e3bade997abad9ec6bed8c8a1c3/datafusion/physical-plan/src/udaf.rs#L75
Update advanced_udaf.rs example to create a GroupsAccumulator

Adding new methods to AggregateUDFImpl:
Following the model of PartitionEvaluator and groups_accumulator_supported
pub trait AggregateUDFImpl {
...
    /// If the aggregate expression has a specialized
    /// [`GroupsAccumulator`] implementation. If this returns true,
    /// `[Self::create_groups_accumulator`] will be called.
    fn groups_accumulator_supported(&self) -> bool {
        false
    }

   /// Return a specialized [`GroupsAccumulator`] that manages state
    /// for all groups.
    ///
    /// For maximum performance, a [`GroupsAccumulator`] should be
    /// implemented in addition to [`Accumulator`].
    fn create_groups_accumulator(&self) -> Result<Box<dyn GroupsAccumulator>> {
        not_impl_err!("GroupsAccumulator hasn't been implemented for {self:?} yet")
    }

}

Describe alternatives you've considered
No response
Additional context
No response

I will deal this issue after #8733 finished and merged

GroupsAccumulator is in datafusion-physical-expr package, datafusion-physical-expr package depend datafusion-expr package, so if we use GroupsAccumulator in datafusion-expr package will cause reference loops 😢

GroupsAccumulator is in datafusion-physical-expr package, datafusion-physical-expr package depend datafusion-expr package, so if we use GroupsAccumulator in datafusion-expr package will cause reference loops 😢

Is it possible to move GroupsAccumulator into datafusion_expr (in the same place as Accumulator)?


0.1.4

This pull request contains features/fixes for the following issues:

Issue 62
Issue 63
Issue 64

Codecov Report

Merging #68 into master will decrease coverage by 1.22%.
The diff coverage is 57.89%.

@@            Coverage Diff            @@
##           master     #68      +/-   ##
=========================================
- Coverage   83.72%   82.5%   -1.23%
=========================================
  Files           5       5
  Lines         424     440      +16
=========================================
+ Hits          355     363       +8
- Misses         69      77       +8

Impacted Files
Coverage Δ

pyscc/controller.py
86.17% <100%> (ø)
:arrow_up:

pyscc/element.py
80.41% <50%> (-1.81%)
:arrow_down:

Continue to review full report at Codecov.

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


How do I go about using this with the SLAB service?

You should be able to follow the steps here for setting up the out of process service.  I don't have an example configuration file handy for that, but you might want to have a look at the configuration handler.
Have a look at this.
https://github.com/fullscale180/slab-sinks/blob/master/src/SemanticLogging.Elasticsearch/Configuration/ElasticsearchSinkElement.cs
We have not really been using SLAB anymore because we have not seen any movement from MS on the project in a long time.

What are you guys using as alternative? any suggestions


Manage slug releases

Releases must be managed by the command line

[ ] View build logs from a release
koli release my-deploy v1 --build-logs
koli release my-deploy/v1 --build-logs

First lookup for pods with the given release selector sys.io/release=1
If the pod doesn't exists, dowload the log files from S3

Related https://github.com/kolibox/controller/issues/9

The koli command line it's not on the road map yet, closing for now.


refreshVersions task incorrectly marks plugin.android as unused in versions.properties

[x] I have read the guidelines for submitting issues

🐛 Describe the bug
Android Gradle Plugin version gets marked as unused in versions.properties
⚠ Current behavior
plugin.android gets marked as ## unused:
## unused
plugin.android=7.3.0

✅ Expected behavior
plugin.android correctly gets updates shown for it and does not have an ## unused comment
💣 Steps to reproduce

Using the New Project wizard in Android Studio Dolphin, create a new Basic Activity project.
Setup the refreshVersions plugin in the project
Run refreshVersionsMigrate --mode=VersionsPropertiesOnly
Run refreshVersions

📱 Tech info

Build Scan URL: https://scans.gradle.com/s/2btew7iahbgqg

Havin the same issue..

I've also seen the same happen on another project (not on my machine). If anyone has a clue as to why it's happening, it'd be helpful to fix it.

This happens to me when I have a project with no buildscript block bringing in the android dependency.

I finally got a reproducer thanks to @attila-fazekas https://github.com/Splitties/refreshVersions/issues/700#issuecomment-1694591607
I'm looking into it.

I’m experiencing the same issue. My Android Plugin is marked as unused, and while all my dependencies are receiving updates, the Gradle plugin itself isn't. I was on version 8.5.0, but it's not being updated even though 8.6.1 is available as a stable release.


Provide cjs, esm and umd bundles instead of transpiled files

added rollup to bundle cjs, esm and umd (with all deps expect react)
moved everything in single dist folder
added size snapshot which provides info about actual, minified and gzipped size
enable treeshake option for es which shows as the size of import {/*nothing*/} from 'bundle'

every new commit will show the difference

Thank you SO much! This looks great! Rollup is so nice 🎉


request() helper not working properly when Laravel is optimized

Laravel Version
11.7.0
PHP Version
8.2.12
Database Driver & Version
No response
Description
Laravel request() helper won't work when app is optimized.
public function resolveAuthMiddleware()     {         return strtolower(request()->header('Request-From')) == "portal"             ? "auth:sanctum" : "api.auth"; }
I have a logic to check which middleware to use depending on a header Request-From value. This works perfectly when the app is not optimized. But when i run php artisan optimize, the ternary operator returns false giving me api:auth. Again when i run php artisan optimize:clear i have auth:sanctum.
PS: The request header was available when checked with telescope
Steps To Reproduce

Generate a method to conclude which middleware to use
public function resolveAuthMiddleware()  {      return strtolower(request()->header('Request-From')) == "portal"          ? "auth:sanctum" : "api.auth";  }

Make an HTTP request with header Request-From: Portal

Compare the value when optimized and when not optimized

Hey there,
Can you first please try one of the support channels below? If you can actually identify this as a bug, feel free to open up a new issue with a link to the original one and we'll gladly help you out.

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

Thanks!


Some errors thrown by runtime layer don't log the "runtimeVersion" property

There a few places in the runtime layer where the runtimeVersion property is not added to the error that is logged in telemetry:

When the runtime calls container.close. For example - https://github.com/microsoft/FluidFramework/blob/608f93485d2954121ff4fb64d1deafceb2f3b6c2/packages/runtime/container-runtime/src/containerRuntime.ts#L794.
When the runtime throws an error, the error results in closing the container and the container logs an error. For example - https://github.com/microsoft/FluidFramework/blob/608f93485d2954121ff4fb64d1deafceb2f3b6c2/packages/runtime/container-runtime/src/containerRuntime.ts#L542

@markfields FYI, I assigned this to you.

Tracked by AB#441


Review old warning messages for precision mismatch in schema validation

It seems that this warning below is longer needed, we need to review it and remove or update it.
data-validation validate schema -sc oracle21 -tc bq -tbls pso_data_validator.dvt_core_types --allow-list-file=professional-services-data-validator/samples/allow_list/oracle_to_bigquery.yaml
08/21/2023 01:43:24 PM-WARNING: Source and target data type has precision mismatch: decimal(20,0) - decimal(38, 9)
08/21/2023 01:43:24 PM-WARNING: Source and target data type has precision mismatch: decimal(38,0) - decimal(76, 38)
08/21/2023 01:43:24 PM-WARNING: Source and target data type has precision mismatch: decimal(10,2) - decimal(38, 9)

Implemented on: https://github.com/GoogleCloudPlatform/professional-services-data-validator/blob/develop/data_validation/schema_validation.py#L189

We log this warning when the schema of the same data type is more relaxed and the data still fits. We consider it a 'success' in schema validation, but log a warning since it's not an exact match.
See this comment for more details.

I agree with the linked comment, but I don't think the code reflects the comment. If I read the code correctly we do this:

If source = target exactly then success
If source/target match an allow list pair then
2.1. If target precision lower then fail
2.2 If target precision higher then success with warning
2.3 If precision equal then success
If not an exact match and not an allow list pair then fail

I think the original comment was asking for:

If source = target exactly then success
If source/target match an allow list pair then success
If not an exact match and not an allow list pair then
3.1. If base type is same and target precision lower then fail
3.2 If base type is same and target precision higher then success with warning
3.3 Else fail

I summary it looks to me that the precision checking is in the wrong place. Hopefully I misread it.

@nj1973 Ah you're right - we are checking in the wrong place. Thanks for the explanation!


Upscaling a video on linux generates errors and produces blank frames

Description
When trying to upscale a video with the following command, there is an error each time it attempts to process a frame:
./start.cmd         --model upscayl-ultrasharp         --scale 4         --format png         --video         --input ../input.mkv         --output my-video-upscaled.mp4
Error:

Stream mapping:
  Stream #0:0 -> #0:0 (hevc (native) -> png (native))
Press [q] to stop, [?] for help
[0 NVIDIA GeForce RTX 3090]  queueC=2[8]  queueG=0[16]  queueT=1[2]
[0 NVIDIA GeForce RTX 3090]  bugsbn1=0  bugbilz=0  bugcopc=0  bugihfa=0
[0 NVIDIA GeForce RTX 3090]  fp16-p/s/a=1/1/1  int8-p/s/a=1/1/1
[0 NVIDIA GeForce RTX 3090]  subgroup=32  basic=1  vote=1  ballot=1  shuffle=1
[1 llvmpipe (LLVM 15.0.7, 256 bits)]  queueC=0[1]  queueG=0[1]  queueT=0[1]
[1 llvmpipe (LLVM 15.0.7, 256 bits)]  bugsbn1=0  bugbilz=0  bugcopc=0  bugihfa=0
[1 llvmpipe (LLVM 15.0.7, 256 bits)]  fp16-p/s/a=1/1/1  int8-p/s/a=1/1/1
[1 llvmpipe (LLVM 15.0.7, 256 bits)]  subgroup=8  basic=1  vote=1  ballot=1  shuffle=1
Output #0, image2, to '../input-upscaled_workspace/sample.png':
  Metadata:
    encoder         : Lavf61.7.100
  Stream #0:0: Video: png, rgb24(pc, gbr/bt470bg/bt709, progressive), 1280x720 [SAR 1:1 DAR 16:9], q=2-31, 200 kb/s, 25 fps, 25 tbn (default)
      Metadata:
        DURATION        : 00:48:24.301000000
        encoder         : Lavc61.19.100 png
[image2 @ 0x55a6cbfa4d00] The specified filename '..input-upscaled_workspace/sample.png' does not contain an image sequence pattern or a pattern is invalid.
[image2 @ 0x55a6cbfa4d00] Use a pattern such as %03d for an image sequence or use the -update option (with -frames:v 1 if needed) to write a single image.
[out#0/image2 @ 0x55a6cbfd5480] video:3KiB audio:0KiB subtitle:0KiB other streams:0KiB global headers:0KiB muxing overhead: unknown
frame=    1 fps=0.0 q=-0.0 Lsize=N/A time=00:00:00.04 bitrate=N/A dup=13 drop=0 speed=0.736x
0

I don't understand what this error means or how to resolve it. I'm guessing frames are processed sequentially and for some reason, it can't get the frame from the input video. Are mkv files supported?
Expected Behavior
The video will be upscaled without error and the progress indicator will indicate actual progress.
Current Behavior
The command loops with the above error. The work folder is created but fills up with blank / black frames.
Steps to Reproduce [COMPULSORY]
Run the command in the description on Ubuntu 22.04 with an RTX 3090. I've already tried to ask about which hardware is supported but received no reply.
Associated Data Files
No response

OS?


vuestic-alert onClose

Added an action that can be executed on closing an alert.
Usage

Description

Markup:

// Your code

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[* ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Improvement/refactoring (non-breaking change that doesn't add any feature but make things better)

Addressing this in the scope of https://github.com/epicmaxco/vuestic-admin/issues/406
Thanks!


Ej/fix dependency

I think this keeps the functionality that craig added and fixes the dependencies.
This worked for me but tests that it works for you before merging it. Delete node_modules and .expo folder and then run npm install
@hreddy9 @craignyazema3 test this please

basically I was able to solve the problem  by running expo install expo-status-bar, expo install expo-permissions, expo install expo-media-library, expo install expo-image-picker, expo install expo-camera, expo install expo-barcode-scanner


Changing background color to light teal

Experimenting with a new background color... what do you think?

Can we make this bigger?
Browser metadata
Path:      /static
Browser:   Safari 15.6.1 on Mac OS 10.15.7
Viewport:  1280 x 695 @2x
Language:  en-US
Cookies:   Disabled

Open in BrowserStack

Open Deploy Preview · Mark as Resolved


dependency updates

#153  👈
main

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

Join @Enalmada and the rest of your teammates on  Graphite


[e2e]: generic schedule main multi-uses slsa3

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/3247671219
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.schedule.main.multi-uses.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.schedule.main.multi-uses.slsa3.yml
Trigger: schedule
Branch: main
Date: Fri Oct 14 05:35:54 UTC 2022

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/3247671219
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.schedule.main.multi-uses.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.schedule.main.multi-uses.slsa3.yml
Trigger: schedule
Branch: main
Date: Fri Oct 14 05:36:08 UTC 2022

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/3252007585
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.schedule.main.multi-uses.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.schedule.main.multi-uses.slsa3.yml
Trigger: workflow_dispatch
Branch: main
Date: Fri Oct 14 18:24:42 UTC 2022
Tests are passing now. Closing this issue.


Is there anyone convert it into py3.x  version?

it is grateful if anyone rewrite it by py3.x .

Hi all, I'm working on a migration to Python 3 and Tensorflow 1.x. Seems to be working fine under Python 3.5.4 and tensorflow-gpu-1.15.2. Check my fork.
Anyway I'll send a pull request to the authors so they can test it and confirm it is fine (there were several changes to the RNN API since this original work has been done, and still some others to do if looking forward to a migration to tf 2.x).

Hi all, I'm working on a migration to Python 3 and Tensorflow 1.x. Seems to be working fine under Python 3.5.4 and tensorflow-gpu-1.15.2. Check my fork.
Anyway I'll send a pull request to the authors so they can test it and confirm it is fine (there were several changes to the RNN API since this original work has been done, and still some others to do if looking forward to a migration to tf 2.x).

Hi, @genaris!
I am trying to reproduce this project, and get in trouble with the bugs.
When executing your work, I still can't succeed.
I failed in prepare_physionet.py:
_, _, ann = list(zip(*reader_ann.records()))
Returns
TypeError: catching classes that do not inherit from BaseException is not allowed
I reinstalled the dependency mne version to 0.15.2, but it does not work.
And my Python is 3.6
Could you please help me out? Thank you!

Hi @wangzhang021 , you are right. prepare_physionet.py was not updated (I've downloaded data with original py2.7 setup), so the repo was not in a consistent state.
I've just commited some changes: it should work now on python3 with mne 0.2.0 (latest version). However, it is still work in progress, as prepare_physionet.py code is using a deprecated parameter (scaling_time) that will not be available on mne 0.2.1. Hopefully I'll update it in the following days.

Hi all, I'm working on a migration to Python 3 and Tensorflow 1.x. Seems to be working fine under Python 3.5.4 and tensorflow-gpu-1.15.2. Check my fork.
Anyway I'll send a pull request to the authors so they can test it and confirm it is fine (there were several changes to the RNN API since this original work has been done, and still some others to do if looking forward to a migration to tf 2.x).

Hi, @genaris!
I can't wrap my head around this problem I am getting here. I am running on Tensorflow 2.x (cause i see 'tf.compat.v1.x') and I am really not seeing why this error appears. Is there something I am missing?
I failed in train.py:
This is the relevant part of the code (in trainer.py) where the error appears:
def train(self, n_epochs, resume):
    with tf.Graph().as_default(), tf.compat.v1.Session() as sess:
        # Build training and validation networks
        train_net = DeepFeatureNet(
            batch_size=self.batch_size,
            input_dims=self.input_dims,
            n_classes=self.n_classes,
            is_train=True,
            reuse_params=False,
            use_dropout=True
        )
        valid_net = DeepFeatureNet(
            batch_size=self.batch_size,
            input_dims=self.input_dims,
            n_classes=self.n_classes,
            is_train=False,
            reuse_params=True,
            use_dropout=True
        )

        # Initialize parameters, I think this step hadnt 'input_var'
        train_net.init_ops()
        valid_net.init_ops()

So, I find 'input_var' in model.py:
def _build_placeholder(self):
    # Input
    name = "x_train" if self.is_train else "x_valid"
    self.input_var = tf.compat.v1.placeholder(
        tf.float32,
        shape=[self.batch_size, self.input_dims, 1, 1],
        name=name + "_inputs"
    )
    # Target
    self.target_var = tf.compat.v1.placeholder(
        tf.int32,
        shape=[self.batch_size, ],
        name=name + "_targets"
    )

This is the traceback I am getting:
File ":\deepsleepnet\deepsleep\trainer.py", line 237, in train
train_net.init_ops()
File ":\deepsleepnet\deepsleep\model.py", line 199, in init_ops
network = self.build_model(input_var=self.input_var)
File ":\deepsleepnet\deepsleep\model.py", line 90, in build_model
network = self._conv1d_layer(input_var=input_var, filter_size=50, n_filters=64, stride=6, wd=1e-3)
File ":\deepsleepnet\deepsleep\model.py", line 61, in _conv1d_layer
n_batches = input_shape[0].value
AttributeError: 'int' object has no attribute 'value'
And my Python is 3.6, tensorflow.version=2.1.0
Could you please help me out? Thank you!


Running out of RAM on Google Colab Comparison Notebook

I'm running on TPU in High Memory mode and still crashing.
https://i.imgur.com/tqcqSin.png
I've commented out the Google and AWS classifiers. Just trying to run keras_ocr.
https://colab.research.google.com/drive/1FMS3aUZnBU4Tc6bosBPnrjdMoSrjZXRp#scrollTo=2jXR7Bv7aW0D&line=7&uniqifier=1
Logs from Colab

Apr 25, 2020, 1:41:14 PM
WARNING
warn("IPython.utils.traitlets has moved to a top-level traitlets package.")

Apr 25, 2020, 1:41:14 PM
WARNING
/usr/local/lib/python3.6/dist-packages/IPython/utils/traitlets.py:5: UserWarning: IPython.utils.traitlets has moved to a top-level traitlets package.

Can you help me diagnose and resolve the issue? Is anyone able to run the comparison on the Notebook? Original notebook here: https://drive.google.com/file/d/1FMS3aUZnBU4Tc6bosBPnrjdMoSrjZXRp/view?usp=sharing

This sounds like it may be related to https://github.com/faustomorales/keras-ocr/issues/65, which showed that there was a memory leak in recent versions of TensorFlow. If you only need inference, one workaround might be to use TensorFlow 1.15.2 until the memory leak is resolved.

Thank you for your response. I actually need to retrain the model to accept some punctuation and spacing. Am I correct that I need to use tf 2+ to retrain?

I think that you may be able to get away with TF 1.15.2 for training but it seems like the memory leak is only a problem for inference and not for training. So I would suggest (1) training with 2.x, (2) save weights, (3) load weights using TF 1.15.2, (4) carry out inference. Does that make sense?
I realize all this is far from ideal and really am sorry for the inconvenience.

Please, no apologies necessary. Thank you for the advice. I’ll try
everything out and report back in a few days.
On Wed, Apr 29, 2020 at 12:19 AM Fausto Morales<EMAIL_ADDRESS>wrote:

I think that you may be able to get away with TF 1.15.2 for training but
it seems like the memory leak is only a problem for inference and not for
training. So I would suggest (1) training with 2.x, (2) save weights, (3)
load weights using TF 1.15.2, (4) carry out inference. Does that make sense?
I realize all this is far from ideal and really am sorry for the
inconvenience.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/faustomorales/keras-ocr/issues/69#issuecomment-620980560,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABU2FGFXIWAUOWXZ2B2KDM3RO6TFZANCNFSM4MQ22RZQ
.

Closing due to inactivity. Please reply back if this issue has not been resolved with newer versions of TensorFlow.


Fallback Image Capture With Larger Window

if changing the width to 640 and height to 480 and trying to take a picture with the flash fallback the image is not drawn to the canvas.
i am not sure if this is because of the size or the resolution of my webcam not matching.

I tried with a better camera and still the same issue with the fallback

I found this from the jquery webcam plugin docs. Looks like you need to recompile the swf.
http://www.xarg.org/project/jquery-webcam-plugin/
width
The width of the flash movie.
height
The height of the flash movie. Both parameters have to be changed in the Flash file as well. Follow the instructions below to recompile the swf after the size change.


Check All Headers In Documentation

Check headers are correct

TLDR; Headers should use the #, ## etc to format different levels of headers, and not be underlines using ===== or ``-----`, or be psuedo header using bold

Check all headers on the documentation - headers should use the hash style of declaration rather then be underlined with equals or dashes. The more hashes, the more of a subheading. Eg:

# is equal to

## is equal to

### is equal to

#### is equal to

##### is equal to

Headings should be appropriate for their level in the documentation.
Psuedo headers using bold tags ** should be replaced with appropriate level of heading tag.

Fixed with 179afa3bd16bb2ab58c98a280e5226e5369ed635


Investigate and propose method for handling of discrete action spaces by OpenVLA

OpenVLA is designed to work with continuous action spaces. Come up with a method to adapt this to discrete action space control tasks.

Same as #238


How do I get FFT size of more than 32768?

By default it allows up to 16384 samples. If I edit audioSpectrumAnalyzer/src/main/res/values/arrays.xml, I can also get 32768 samples.
But if I try further values (65536, 131072), it stops working (shows empty 1D or 2D plot) if I select those values.
What else do I need to change to make it work for large windows?

There are few possible reasons, I didn't test them yet.

The drawing function drawBitmap in Canvas class has a hardware limit (down to the OpenGL/GPU limit), probably around 32768. So if you want to draw a super wide picture, you might try split it or down-sampling it (the log plot already implemented a down-sampling algorithm).
May be there is a limit in FFT routine???
A bug in my code.
memory allocation failure???


restore doesn't restore properties properly

object[property] =  occurs in numerous places in the code base to restore properties, but this does not restore properties properly in modern javascript.
Object.getOwnPropertyDescriptor & Object.defineProperty should be used instead.

Can you provide an example where the current implementation fails?

var obj = {};
Object.defineProperty(obj, 'nonWritable', {value: function(){
  return 1;
}});
var stub = sinon.stub(obj, 'nonWritable').returns(2);
console.log(obj.nonWritable());
stub.restore();

1 is printed instead of 2.

Sure, in the following code:
var obj = {};
Object.defineProperty(obj, 'nonWritable', {value: function(){
  return 1;
}});
var stub = sinon.stub(obj, 'nonWritable', function(){return 2;});
console.log(obj.nonWritable());
stub.restore();

1 is printed instead of 2.

This is an example of sinon.stub not working with Object.defineProperty, but I see your point ;-)


New package: EmpiricalMeasures v0.1.0

Registering package: EmpiricalMeasures
Repository: https://github.com/davibarreira/EmpiricalMeasures.jl
Created by: @davibarreira
Version: v0.1.0
Commit: 5e5eebca72adc6b7147cffa5dcc01e909c0f2048
Reviewed by: @davibarreira
Reference: https://github.com/davibarreira/EmpiricalMeasures.jl/issues/2

Please, remove this PR. This package might end up absorbed in Distributions.jl.


Use virtual table for subtitles

I tried this using react-virtualized and the UX actually seems to degrade (e.g. scroll bar seems to shift up or down and change size while scrolling), but maybe I was implementing the table incorrectly.

Less of a problem now since the subtitle list rarely re-renders.


Re-enable multithreading in D version, use default configuration of vibe.d

On my linux box, go version uses 14 threads (7 for 'go' process, and 7 for 'main' process). So, removing of multithreading from D test app looks strange.
P.S: please use 'dub upgrade --force-remove' after changing of dub.json to let dub choose new version if you already used dub with older version specified in dub.json

Multithreaded version fails on macOS, it just crashes with the "Program exited with code -11". I couldn't reproduce it without "--release" option, looks like some compiler optimizations have side effects. However, the brewed version of ldc2 is 1.3.0, I guess I have to try to use the latest version.
Anyway, I'll merge for now, but I think some changes will be needed anyway.


infrastructure: migrate everything to a self-hosted stack

self reliance
better visibility into systems at a cheaper cost

super happy with the fact that this is done! - https://github.com/viveknathani/nattukaka


Update akka-persistence-cassandra to 1.0.6

Updates com.typesafe.akka:akka-persistence-cassandra from 1.0.5 to 1.0.6.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.typesafe.akka", artifactId = "akka-persistence-cassandra" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "com.typesafe.akka", artifactId = "akka-persistence-cassandra" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 79.51%. Comparing base (c04d33c) to head (cc98cad).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1325      +/-   ##
==========================================
+ Coverage   79.35%   79.51%   +0.15%
==========================================
  Files         144      144
  Lines        5008     5008
  Branches      670      670
==========================================
+ Hits         3974     3982       +8
+ Misses       1034     1026       -8

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Attachments/blocks not working ("no_text")

Your checklist for this issue

[x] Jenkins version: 2.251

[x] Plugin version: 2.40

[x] OS: Amazon Linux 1

Description
Having followed the setup instructions and generating a custom bot token, it appears as though neither attachments nor blocks are working, returning {"ok":false,"error":"no_text"} from Slack.
I have tested the token using the Python Slack SDK and have confirmed no issues with permissions.
Plugin is configured with workspace, credential, default channel, and custom app bot user all set, and works for simple messages, however the example snippets with blocks/attachments in the README fail.
Sample pipeline attached
// send a message to a channel (this works)
slackSend(channel: '#test', message: "This is a test message")

// send a message with attachments to a channel (this doesn't work)
def attach = [
    [
        text: "This is some text in the attachment",
        fallback: "This is fallback text",
        color: "#2eb886",
        title: "this is the attachment title",
        fields: [
            [title: "field1", value: "field1 val", short: false]
        ]
    ]
]
slackSend(channel: '#test', attachments: attach)

// send a message with blocks to a channel (this doesn't work)
blocks = [
	[
		"type": "section",
		"text": [
			"type": "mrkdwn",
			"text": "Hello, Assistant to the Regional Manager Dwight! *Michael Scott* wants to know where you'd like to take the Paper Company investors to dinner tonight.\n\n *Please select a restaurant:*"
		]
	],
    [
		"type": "divider"
	],
	[
		"type": "section",
		"text": [
			"type": "mrkdwn",
			"text": "*Farmhouse Thai Cuisine*\n:star::star::star::star: 1528 reviews\n They do have some vegan options, like the roti and curry, plus they have a ton of salad stuff and noodles can be ordered without meat!! They have something for everyone here"
		],
		"accessory": [
			"type": "image",
			"image_url": "https://s3-media3.fl.yelpcdn.com/bphoto/c7ed05m9lC2EmA3Aruue7A/o.jpg",
			"alt_text": "alt text for image"
		]
	]
]
slackSend(channel: "#test", blocks: blocks)

Im also experiencing this issue.
Even when using the examples in the documentation, I get the no_text error.

Unfortunately not :( Sorry

I came across this old issue and it is probably no longer relevant,
To make sure, I verified the attachment example above and it works as expected
def attach = [
    [
        text: "This is some text in the attachment",
        fallback: "This is fallback text",
        color: "#2eb886",
        title: "this is the attachment title",
        fields: [
            [title: "field1", value: "field1 val", short: false]
        ]
    ]
]
slackSend(channel: "@<my-slack-user-id>", attachments: attach, tokenCredentialId: '<my-slack-cred>')

Jenkins 2.429
Slack Notification Plugin 684.v833089650554

Thanks