WebSocket registerEvents is undefined when running master

Environment
 React Native Environment Info:
    System:
      OS: macOS High Sierra 10.13.6
      CPU: x64 Intel(R) Core(TM) i5-6267U CPU @ 2.90GHz
      Memory: 2.07 GB / 16.00 GB
      Shell: 5.3 - /bin/zsh
    Binaries:
      Node: 8.7.0 - /usr/local/bin/node
      Yarn: 1.3.2 - /usr/local/bin/yarn
      npm: 5.4.2 - /usr/local/bin/npm
      Watchman: 4.7.0 - /usr/local/bin/watchman
    SDKs:
      iOS SDK:
        Platforms: iOS 11.4, macOS 10.13, tvOS 11.4, watchOS 4.3
    IDEs:
      Xcode: 9.4.1/9F2000 - /usr/bin/xcodebuild
    npmPackages:
      react: 16.4.1 => 16.4.1
      react-native: /Users/grabbou/Repositories/react-native/react-native-1000.0.0.tgz => 1000.0.0
    npmGlobalPackages:
      create-react-native-app: 1.0.0
      react-native-cli: 2.0.1
      react-native-create-library: 3.1.2
      react-native-fbads: 4.2.0
      react-native-git-upgrade: 0.2.7

Description
The following error is produced as soon as you run the app:

The following line https://github.com/facebook/react-native/blob/master/Libraries/WebSocket/WebSocket.js#L147 produces an error - for some reason _registerEvents() is not available on this, even though you can see it defined as a class method. Is this something related to Babel transpilation?

Reproducible Demo
Running freshly created app with 0.57-stable (react-native init). Interestingly, RNTester (that runs on the same React Native version) works just fine. I'll keep debugging babel differences in setups to see why it could be occuring. Help appreciated.

I've seen this as well when experimenting with different versions of Babel but still using version 0.56 of React (by overriding the dependency resolution).
Could be that a simple bump of Babel could fix this, could you try adding the following to your package.json to force a newer babel?
  "resolutions": {
    "**/@babel/code-frame": "7.0.0-beta.56",
    "**/@babel/core": "7.0.0-beta.56",
    "**/@babel/generator": "7.0.0-beta.56",
    "**/@babel/helper-annotate-as-pure": "7.0.0-beta.56",
    "**/@babel/helper-builder-binary-assignment-operator-visitor": "7.0.0-beta.56",
    "**/@babel/helper-builder-react-jsx": "7.0.0-beta.56",
    "**/@babel/helper-call-delegate": "7.0.0-beta.56",
    "**/@babel/helper-define-map": "7.0.0-beta.56",
    "**/@babel/helper-explode-assignable-expression": "7.0.0-beta.56",
    "**/@babel/helper-function-name": "7.0.0-beta.56",
    "**/@babel/helper-get-function-arity": "7.0.0-beta.56",
    "**/@babel/helper-hoist-variables": "7.0.0-beta.56",
    "**/@babel/helper-member-expression-to-functions": "7.0.0-beta.56",
    "**/@babel/helper-module-imports": "7.0.0-beta.56",
    "**/@babel/helper-module-transforms": "7.0.0-beta.56",
    "**/@babel/helper-optimise-call-expression": "7.0.0-beta.56",
    "**/@babel/helper-plugin-utils": "7.0.0-beta.56",
    "**/@babel/helper-regex": "7.0.0-beta.56",
    "**/@babel/helper-remap-async-to-generator": "7.0.0-beta.56",
    "**/@babel/helper-replace-supers": "7.0.0-beta.56",
    "**/@babel/helper-simple-access": "7.0.0-beta.56",
    "**/@babel/helper-split-export-declaration": "7.0.0-beta.56",
    "**/@babel/helper-wrap-function": "7.0.0-beta.56",
    "**/@babel/helpers": "7.0.0-beta.56",
    "**/@babel/highlight": "7.0.0-beta.56",
    "**/@babel/plugin-external-helpers": "7.0.0-beta.56",
    "**/@babel/plugin-proposal-class-properties": "7.0.0-beta.56",
    "**/@babel/plugin-proposal-object-rest-spread": "7.0.0-beta.56",
    "**/@babel/plugin-proposal-optional-chaining": "7.0.0-beta.56",
    "**/@babel/plugin-syntax-class-properties": "7.0.0-beta.56",
    "**/@babel/plugin-syntax-dynamic-import": "7.0.0-beta.56",
    "**/@babel/plugin-syntax-flow": "7.0.0-beta.56",
    "**/@babel/plugin-syntax-jsx": "7.0.0-beta.56",
    "**/@babel/plugin-syntax-nullish-coalescing-operator": "7.0.0-beta.56",
    "**/@babel/plugin-syntax-object-rest-spread": "7.0.0-beta.56",
    "**/@babel/plugin-syntax-optional-chaining": "7.0.0-beta.56",
    "**/@babel/plugin-transform-arrow-functions": "7.0.0-beta.56",
    "**/@babel/plugin-transform-async-to-generator": "7.0.0-beta.56",
    "**/@babel/plugin-transform-block-scoping": "7.0.0-beta.56",
    "**/@babel/plugin-transform-classes": "7.0.0-beta.56",
    "**/@babel/plugin-transform-computed-properties": "7.0.0-beta.56",
    "**/@babel/plugin-transform-destructuring": "7.0.0-beta.56",
    "**/@babel/plugin-transform-exponentiation-operator": "7.0.0-beta.56",
    "**/@babel/plugin-transform-flow-strip-types": "7.0.0-beta.56",
    "**/@babel/plugin-transform-for-of": "7.0.0-beta.56",
    "**/@babel/plugin-transform-function-name": "7.0.0-beta.56",
    "**/@babel/plugin-transform-literals": "7.0.0-beta.56",
    "**/@babel/plugin-transform-modules-commonjs": "7.0.0-beta.56",
    "**/@babel/plugin-transform-object-assign": "7.0.0-beta.56",
    "**/@babel/plugin-transform-parameters": "7.0.0-beta.56",
    "**/@babel/plugin-transform-react-display-name": "7.0.0-beta.56",
    "**/@babel/plugin-transform-react-jsx": "7.0.0-beta.56",
    "**/@babel/plugin-transform-react-jsx-source": "7.0.0-beta.56",
    "**/@babel/plugin-transform-regenerator": "7.0.0-beta.56",
    "**/@babel/plugin-transform-shorthand-properties": "7.0.0-beta.56",
    "**/@babel/plugin-transform-spread": "7.0.0-beta.56",
    "**/@babel/plugin-transform-sticky-regex": "7.0.0-beta.56",
    "**/@babel/plugin-transform-template-literals": "7.0.0-beta.56",
    "**/@babel/plugin-transform-unicode-regex": "7.0.0-beta.56",
    "**/@babel/register": "7.0.0-beta.56",
    "**/@babel/template": "7.0.0-beta.56",
    "**/@babel/traverse": "7.0.0-beta.56",
    "**/@babel/types": "7.0.0-beta.56"
  },

If this solves it, we could just bump Babel in Metro, bump Metro, and then bump Metro here in React Native 😄

Here is a pull request to Babel, that was also needed for an unrelated bug with exporting interfaces from a TypeScript file: https://github.com/facebook/metro/pull/216

I'm concerned this is potentially related to the react-native init command not creating a "properly" setup project, like maybe the babel version is wrong or is not using the right babel-preset... something like that.
@grabbou can you upload the RN init'd project that the command generated?
BTW the correct version of babel to set the package.json is beta.54 (https://github.com/facebook/metro/commit/1ef4e4e92b77ee6718f09cfa718afef8711fad59)
cc @rafeca @CompuIves

This can be related to react-native init adding babel-preset-react-native as a dependency.
Instead of that, it should add the metro-react-native-babel-preset package (and it should be locked to the same version as the Metro version used by React Native).
Another thing that I've just realized is that the .babelrc template file should be also updated to point to the metro-react-native-babel-preset package.

Unfortunately, the error still happens after performing the suggested changes. I am going to put them all into a PR anyway so that master can be up to date.

CC: @hramos and @axemclion - you might be able to make this issue more visible - it's currently blocking the release. I don't know the React Native RNTester works without issues while react-native init app breaks, but there has to be something different.

Related: https://github.com/facebook/react-native/commit/7e61f433ef3919ac828aff3c22a7bf82acf80a08
which should then cherrypick back to master.

Just tried again with using latest master, and this is what happens after I git clone and run ./scripts/test-manual-e2e.sh locally (with simulators):
:white_check_mark: RNTester runs on Android
:white_check_mark: RNTester runs on iOS
:question: RNTestProject builds correctly on Android (but goes to white screen - I guess it’s the same issue faced by Mike a while back, so can't really understand what's going on)
:exclamation: RNTestProject builds but then I hit this issue
It's surely related to babel, and the config.

I've tried changing the babel-preset dept from "babel-preset-react-native": "^5", to "metro-react-native-babel-preset": "0.43.5",, then yarn and npm start --reset-cache but it's still failing with the same error for this .babelrc configuration:
{
  "presets": ["module:metro-react-native-babel-preset"]
}

that was suggested here.
Even if I completely remove the .babelrc file and entry in the package.json, which I seem to recall @rafeca saying that is a viable solution that will "force" metro to use its internal dep, I still get the issue.
Just to make sure that it is in fact a babel related issue I double checked where the error was coming from (here) and if I "bruteforcefully" comment out that section, the error actually changes to  TypeError: instance.render is not a function.
I have spent a couple hrs now on it and I'm quite confident that it's either:

babel being missconfigured, and the metro-react-native-babel-preset not getting picked up during bundling
metro-react-native-babel-preset being missconfigured, maybe missing some transformer or something.

Either case, I think @hramos that the metro team should take a closer look at it, it is also probably related to the many breaking changes Babel7 has undergone from beta.47 to beta.56 (which is the one RN is currently using).
Or maybe I'm missing something simpler and @hzoo has a 1-line solution to all our issues 😇

Thanks for the detailed investigation @kelset!
Thanks to this I've been able to debug the issue and found that the root cause is the Babel preset that's on the template's .babelrc file.
I think that @kelset was not able to find it because npm run start --reset-cache does not seem to work properly and still uses the cached transformed files (that's a separate issue).
So there are two potential solutions here:

Remove completely the .babelrc file from the template (so Metro will pick its own one).
Modify the .babelrc file from the template to have the module:metro-react-native-babel-preset preset.

Here is a PR for switching the preset 👉 #20653

I've updated the 0.57 branch to be in sync with master and tested the changes, and I don't know why but I still, following the list of steps I listed above, get the RNTestProject initialized with the old babel-preset. I feel it's a local caching issue, so it would be great if someone could test if now master or the branch creates the new project correctly.
I'll keep this open until we are sure it's fixed.

Hmm, running npm pack locally, and then generating a new project locally using the following commands produces the correct .babelrc for me.
react-native init Foobar --version /Users/linus/coding/react-native/react-native-1000.0.0.tgz

This should be how the RNTestProject is generated 🤔

Yes I can confirm now that it was a caching issue on the machine, I've just tested with another laptop and the project gets generated correctly 🎉
I'll close this and try to have the first RC0 release for 0.57 out asap.

I am also getting this error. I upgraded RN from 0.54 to 0.56 after fixing alot of babel issues now getting this one.
please let me know what is proper solution

@kelset how do you resolve caching issue? Please, tell

@NelGarbuzova I just restarted the machine, It looks like the error is different now I get an error on StackNavigator (undefined is not a function) with which I assume it has reseted the cache.

@NelGarbuzova I just restarted the machine, It looks like the error is different now I get an error on StackNavigator (undefined is not a function) with which I assume it has reseted the cache.

Restart the machine
It works for me

@NelGarbuzova I just restarted the machine, It looks like the error is different now I get an error on StackNavigator (undefined is not a function) with which I assume it has reseted the cache.
It works for me...
Restart the machine ! ... (why ?! :|)

@NelGarbuzova I just restarted the machine, It looks like the error is different now I get an error on StackNavigator (undefined is not a function) with which I assume it has reseted the cache.

It works for me...
Restart the machine ! ... (why ?! :|) any way thank you ...


create an app gateway for the atlantis container instance

https://learn.microsoft.com/en-us/azure/web-application-firewall/ag/application-gateway-web-application-firewall-portal

Transitioned to JIRA


Relative wind

Clearer pull request for the relative wind changes in the code and the namelist file hycom_opt.

This has been included in the upstream version of HYCOM as of august this year. I suggest that I bring that into version 2.3. And leave out this version. It is implemented the same way.
@AntoineBernigaud @Jipingnersc
Any thoughts on this?
We can still implement this in HYCOM 2.2

Hi Till,
It would be a nice contribution to V2.3, and welcome from my point.
Is there any difference between the upstream update and our tested implementation?
Cheers,
Jiping

On 18 Sep 2024, at 12:12, TRasmussen @.***> wrote:
This has been included in the upstream version of HYCOM as of august this year. I suggest that I bring that into version 2.3. And leave out this version. It is implemented the same way.
@AntoineBernigaud https://github.com/AntoineBernigaud @Jipingnersc https://github.com/Jipingnersc
Any thoughts on this?
We can still implement this in HYCOM 2.2
—
Reply to this email directly, view it on GitHub https://github.com/nansencenter/NERSC-HYCOM-CICE/pull/213#issuecomment-2358064484, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGDZNKIXUZS2YWEX7VLUSUTZXFG2NAVCNFSM6AAAAABONKUZNSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJYGA3DINBYGQ.
You are receiving this because you were mentioned.

Hello,
I do not have much thoughts about it, it might be easier indeed to just take the upstream version of HYCOM into version 2.3 :)
Cheers,
Antoine

The only difference (as I see it) is that it is integrated into blkdat.input.


Terminal Activation tests are flaky

This test fails often and requires re-run.
1) Activation of Environments in Terminal
       Should activate with virtualenv:

      Environment not activated
      + expected - actual

      -false
      +true

      at testActivation (out/test/common/terminals/environmentActivationProviders/terminalActivation.testvirtualenvs.js:75:74)
      at Context.<anonymous> (out/test/common/terminals/environmentActivationProviders/terminalActivation.testvirtualenvs.js:95:9)

Disabled these tests for now.

@anthonykim1 @karthiknadig - is this still the case?

We can close this, this will be handled in the new extension


Vercel deployment broken

Recently I changed this package to use the "standard" environment variables for AWS access.  However, in https://github.com/fox1t/turborepo-remote-cache/pull/30#issuecomment-1164478682 it was revealed that for Vercel deployment for whatever reason they do not allow those environment variables to be used.
Some fallback environment variables will likely be needed to support Vercel deployment.
Docs:

https://vercel.com/docs/concepts/projects/environment-variables#reserved-environment-variables
https://vercel.com/support/articles/how-can-i-use-aws-sdk-environment-variables-on-vercel

Vercel deployments will not work until that is resolved.
This may be as simple as changing vercel.ts like this:
import type { VercelRequest, VercelResponse } from '@vercel/node'
import { createApp } from './app'

// Propage any environment vars APP_AWS_* to AWS_* to workaround
// vercel restriction on environment variables
Object.keys(process.env).filter(k => k.startsWith('APP_AWS_')).forEach(k => {
  process.env[k.substring(4)] = process.env[k];
})

const app = createApp()

export default async function (req: VercelRequest, res: VercelResponse) {
  await app.ready()
  app.server.emit('request', req, res)
}

I am not sure that this is the correct snippet. We should remap S3_ to AWS_.


Updated troubleshooting.md with recommendation of using args for heroku

I decided to bring @ebidel advice in #758 issue to the troubleshooting.md file, since it isn't specified explicitly, only in mentioned issue and that "simple guide" below.
This argument stuff really helped me with getting puppeteer up and running on Heroku and I was frustrated when I realized that this solution is mentioned everywhere but in troubleshooting.md
P.S.
Really thankful for such an awesome tool. I ❤it!

@googlebot I signed it!


Create a Getting Started guide

Create some wiki pages for:

Getting started / installation
Creating a navigation UI
Creating a tab UI (with cross-platform UI)
Modals
Autoform
Accounts templates

+1
Way to go with this project by the way, this is amazing! I'd love to help, but I'm kind of stuck right now because I'm not sure how you get all that animation!

@ccorcos do the example apps help in the meantime? If not, I'll try to write something up tonight or tomorrow:
https://github.com/meteoric/contacts
https://github.com/meteoric/meteorhunt

They do a little bit, but it would be super helpful just to have some kind of guide that helps me understand the design of this project. I don't know much about block helpers {{#iconic-something}} and it seems those are used everywhere.

To answer your question, a 1-2 page guide would be enormously helpful ;)

+1 to guide

Ok guys, here's something quick I whipped up: https://github.com/meteoric/meteor-ionic/blob/master/GUIDE.md
Let me know if it's helpful at all, and what else you'd like to see.

This is pretty awesome. At first I was thinking that surely the styles would be enough, but you quickly realise that there's a lot more to a native feeling app than just the styles. Your demos are very responsive which is great. Eagerly awaiting more directive conversions & docs.

@nickw great start on the guide. I like how you explained Blaze blocks really well by contracting them to inclusions.
I just built a similar package to meteor-ionic but for Material Design (not a public project yet, it was for client work). Looking forward to contributing. Will be building a meteoric app this weekend and will record my development then circle back and make a step-by-step how-to. Bug me next Tuesday if you do not see anything, haha

One thing I just noticed is that the backUrl in a nav view is set only when you specify some kind of data in the "data" parameter in your router setting. If you don't set data, it won't work.

@rjakobsson can you add a new issue for this? I'm not sure how it's happening since we're grabbing the backUrl directly from the template options, and falling back to history.back(). If you can post some code in that issue that would be awesome.

@nickw Yes I'll do that.

@benstr Bugging you as requested. I need all the step-by-steps I can get my hands on ;)

@hettema my optimism always gets the best of me. LOL
I got the bones started, check it out, meteoric-projects.

@benstr wow looks great

Thanks @nickw. PRs totally welcome by anyone. I just closed on two large projects. With my new schedule I will probably release one chapter a week.

@benstr dude, that looks awesome!

@benstr I am interested in your package based on material design. can you please make it available or provide pointers as to how you had it done


Bump tensorflow-io-gcs-filesystem to 0.21.0

This PR bumps tensorflow-io-gcs-filesystem to 0.21.0 so that TF 2.7
branch cut can carry the latest change which includes the fix that removes the extra
build directory in the python pip package (https://github.com/tensorflow/io/pull/1497)
Signed-off-by: Yong Tang<EMAIL_ADDRESS>

@mihaimaruseac Wondering if you can take a look at this PR. It will be great if this PR can be part of the 2.7 branch as it does fixes an issue in pip packages (tensorflow/io#1497)

Sorry, I was OOO.
Can you make a cherry-pick once this lands, please?

Please add me to the cherrypick to review and get approved before a release on 2.7 branch


Update Appstore version 2.4

Describe the bug
问题描述
Old version 2.3.6 in the App Store is broken. Could you kindly update to Appstore's most recent version 2.4?
To Reproduce
如何复现
Steps to reproduce the behavior:

Go to 'Apptore'
Click on Fission_Chatbox
Go to Sari
Extension does not work anymore

Expected behavior
期望行为
Loading nothing when using 2.3.5 version. Tried .dmg file on Github, it not install Safari extension.
Please complete the following information):
请补全以下内容

OS: Sonoma MacOS
Browser: Safari
Extension Version: 2.3.6

I have encountered an issue where the Appstore version of the application is working fine. However, the version downloaded from the website (2.4.2) doesn't display any error messages but gets stuck with the message "Waiting for Response" for a long time. On the other hand, the Appstore version prompts me to log in to OpenAI, and once I do that, it starts working without any issues.

https://github.com/josStorer/chatGPTBox/wiki/Install#app-store
Since v2.3.6, Apple App Store has consistently rejected new versions of this extension for the reason "4.3.0 Design: Spam" and has not provided any substantial explanation for the problem. Despite I've do some efforts, I still failed to overcome this issue. I no longer plan to continue updating this extension for Safari. Please use some other browsers or manually install the extension for Safari.

@josStorer This is how I installed the latest version. However, I am not receiving any error messages that would require me to log in to get it up and running. Still only the AppStore version is working with some bugs and crashing.

I will check later on the problem you encountered on the latest version for Safari. The previous reply is regarding this issue.

@Erik262 Fixed, the previous issue was due to Safari's inconsistent support for certain APIs compared to Firefox and Chromium, which caused background.js to not initialize correctly.
However, please note that due to the recent addition of the "arkose_token" requirement in chatgpt web mode, Safari is currently unable to obtain the arkose token due to permission problems. Therefore, in Safari, you can only use other modes, such as API mode.

@josStorer I tried the latest version, but it doesn't work. I uninstalled and reinstalled, including restarting my Mac and Safari. However, I'm still encountering issues:

try 2.4.4 https://github.com/josStorer/chatGPTBox/releases/download/v2.4.4/safari.dmg

@josStorer It works! As you said, "due to Safari's inconsistent support for certain APIs, it may not work". This is the case, but why does the App Store version work? 🧐

@Erik262 because that inconsistent API was added in the last few months for Bing support, the version in app store has not yet included that API


Document for FindAndCount()

Ref in #1362

Codecov Report

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

@@            Coverage Diff             @@
##           master    #1365      +/-   ##
==========================================
- Coverage   57.54%   57.52%   -0.02%
==========================================
  Files          44       44
  Lines        7838     7838
==========================================
- Hits         4510     4509       -1
  Misses       2776     2776
- Partials      552      553       +1

Impacted Files
Coverage Δ

xorm.go
68.18% <0%> (-1.52%)
: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 18b3248...4bad228. Read the comment docs.


MAISTRA-1160 Add documentation on how to use an existing Jaeger instance

@neal-timpe maybe you can take a look; we should also have a similar section in the product docs

@dgn Would it be possible to add a description of how to get the htpasswd secret (a secret mentioned in docs: Maistra uses a secret called htpasswd)? This secret should be added to jaeger instance but the doc does not say how to get it from maistra.

@dgn Would it be possible to add a description of how to get the htpasswd secret (a secret mentioned in docs: Maistra uses a secret called htpasswd)? This secret should be added to jaeger instance but the doc does not say how to get it from maistra.

Maybe 'uses' is the wrong term. The operator will create that secret in the control plane namespace.

@dgn I got it now, thank you. So since the jaeger is running in the same namespace as the Maistra control plane, it will find the secret called htpasswd created by maistra operator. No need for manual creation.

I added instructions on how to retrieve and set the Jaeger dashboard URL

@JStickler said she is looking into it.

General comment here. I will highlight what stands out to me, but this should be following our documentation standard: https://github.com/openshift/openshift-docs/blob/master/contributing_to_docs/doc_guidelines.adoc#writing-concepts


Fix command to build & serve site locally using docker (bad workdir)

Workdir option in docker run ... jekyll is wrong. Workdir should be /srv/jekyll, not $(pwd)

Hi @zbbfufu. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

I signed it!
/assign @geeknoid


Improve mixed styles values (eg: margin auto, width 100%)

margin and width (as example) support float and some strings or other values (eg: percentage are supported by yoga engine for width/height).
I am still new to reason but would like to improve the situation. Any pointers on where to start?

it's handled in the styles PR.

Awesome, I missed that. Thanks!


performance: improve MySQL's global lock batch insert

Ⅰ. Describe what this PR did
update 23000 rows
original:

now:

Ⅱ. Does this pull request fix one issue?

fixes #2636
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #2648 into develop will decrease coverage by 0.20%.
The diff coverage is 40.62%.

@@              Coverage Diff              @@
##             develop    #2648      +/-   ##
=============================================
- Coverage      50.91%   50.70%   -0.21%
+ Complexity      2815     2814       -1
=============================================
  Files            558      558
  Lines          17941    17938       -3
  Branches        2129     2094      -35
=============================================
- Hits            9134     9096      -38
- Misses          7938     7949      +11
- Partials         869      893      +24

Impacted Files
Coverage Δ
Complexity Δ

...a/core/store/db/sql/lock/AbstractLockStoreSql.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

...eata/core/store/db/sql/lock/MysqlLockStoreSql.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

...a/server/storage/db/lock/LockStoreDataBaseDAO.java
53.43% <50.00%> (-3.57%)
26.00 <2.00> (ø)

...obuf/convertor/BranchRegisterRequestConvertor.java
90.47% <0.00%> (-9.53%)
3.00% <0.00%> (ø%)

...otobuf/convertor/BranchReportRequestConvertor.java
90.90% <0.00%> (-9.10%)
3.00% <0.00%> (ø%)

...obuf/convertor/BranchRollbackRequestConvertor.java
92.00% <0.00%> (-8.00%)
3.00% <0.00%> (ø%)

...otobuf/convertor/GlobalBeginResponseConvertor.java
92.59% <0.00%> (-7.41%)
3.00% <0.00%> (ø%)

...otobuf/convertor/GlobalCommitRequestConvertor.java
93.75% <0.00%> (-6.25%)
3.00% <0.00%> (ø%)

...otobuf/convertor/GlobalStatusRequestConvertor.java
93.75% <0.00%> (-6.25%)
3.00% <0.00%> (ø%)

...obuf/convertor/GlobalRollbackRequestConvertor.java
93.75% <0.00%> (-6.25%)
3.00% <0.00%> (ø%)

... and 23 more

Codecov Report

Merging #2648 into develop will decrease coverage by 0.07%.
The diff coverage is 39.39%.

@@              Coverage Diff              @@
##             develop    #2648      +/-   ##
=============================================
- Coverage      50.92%   50.85%   -0.08%
+ Complexity      2818     2814       -4
=============================================
  Files            558      558
  Lines          17941    17956      +15
  Branches        2129     2134       +5
=============================================
- Hits            9136     9131       -5
- Misses          7938     7954      +16
- Partials         867      871       +4

Impacted Files
Coverage Δ
Complexity Δ

...a/core/store/db/sql/lock/AbstractLockStoreSql.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

...eata/core/store/db/sql/lock/MysqlLockStoreSql.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

...a/server/storage/db/lock/LockStoreDataBaseDAO.java
53.69% <48.14%> (-3.31%)
26.00 <2.00> (ø)

...torage/file/store/FileTransactionStoreManager.java
56.77% <0.00%> (-0.65%)
28.00% <0.00%> (-1.00%)

...o/seata/server/coordinator/DefaultCoordinator.java
54.63% <0.00%> (-0.52%)
28.00% <0.00%> (-2.00%)

...in/java/io/seata/server/session/GlobalSession.java
83.71% <0.00%> (-0.46%)
71.00% <0.00%> (-1.00%)

mysql: packet for query is 4M. update mysql conf max_allowed_packet property.
sqlserver: values max quantity limit is 2100.
postgresql: sql max packet limit is 512M.

close


Code review for features branch

In this project we have built a website where you can signup, login and add posts:

We have used devise gem for authenticate users and create/destroy sessions.
We have created a posts model with a foreign key in the database to 'users' table
We have created a post controller with index, new and create actions
We have created the associations between models
We have created the views for the posts controller actions, and a partial form
We have added logic to our posts views so only a signed user can see the name of the user who created the post
Only logged users can create posts

The project has passed rubocop test

We have fixed all the rubocop errors, see last commit, and at our end we see 'no offenses' after testing the code with rubocop. The errors remain here, in the pull request, but we cannot fix them because in our end everything looks good.

Thanks, @shshamim63 for an awesome and descriptive review!
Changes are done after review:

Fixed carriage return rubocop errors.
Added custom helper method (enforce_log_in) in app/controllers/application_controller.rb to enforce login before creating new posts. Previously devise helper method authenticate_user! was used.


feat: introduces sync gRPC server

udf multi-threaded sync server
udsink multi-threaded sync server
udf and udsink sync servers with migration threadpool

Note: Since most of the changes are in the server function, the codecov is failing.
Closes #25

I wonder whether we should do an integer multiple of CPUs available as the default value? The number of CPUs as thread pool size is too low.


Ephemeral storage and Dasv5 and Dadsv5-series VM's

We are looking to move away from using Standard_D4as_v4 in our AKS clusters but the usage of Ephemeral storage is mandatory for us.
The documentation says we need to pay attention to
The sizes for VM cache are available in the Azure documentation in parentheses next to IO throughput ("cache size in GiB").

But that doesn't exist for any variant of the v5 version. I find it hard to believe that a newer generation process/machine would lack features of the old model.
Is this something that is missing in the documentation? If so what are the disk sizes of the VM's?
Thanks in Advance

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

ID: d5211289-a99f-3f6f-83c6-bd6d89844ef3
Version Independent ID: ef06c30a-be36-8b6a-d870-8f8199c6fc99
Content: Dasv5 and Dadsv5-series - Azure Virtual Machines
Content Source: articles/virtual-machines/dasv5-dadsv5-series.md
Service: virtual-machines
Sub-service: sizes
GitHub Login: @mamccrea
Microsoft Alias: mamccrea

@urbantz-technical-support
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

@mamccrea
Please review it .

@Aarthi-Vijayaraghavan, can you please advise?

Hi @urbantz-technical-support, I understand your concern. The Dasv5 SKU do not support ephemeral as the SKUs do not come with cache or temp space to place the Ephemeral OS disk. If you want to use Ephemeral OS disk on a v5 SKU the next option would be to use Dadsv5 SKUs as they have temp space to place the Ephemeral OS disk.

Hi @Aarthi-Vijayaraghavan thanks for the response.
Unfortunately this doesn't really satisfy us. Am I to understand the the the field Temp storage (SSD) GiB is the size of the cache?
Why on other platforms is it described in parenthesis next to the maximum iops, like the AKS documentation mentions, but here it is a different column?

@urbantz-technical-support - Ephemeral OS disks can be placed on the cache or temp storage of the respective VM sizes.
The sizes described in parenthesis is for cache sizes (highlighted in red). Temp storage (highlighted in green) is mentioned as a separate field/column.

If the OS image size is less than or equal to the temp storage size, then you can place the ephemeral OS disk via temp disk placement option available.

Hey @Aarthi-Vijayaraghavan
Thanks for the response. We understood that and in fact a very similar line appears in the aks documentation regarding ephemeral storage.
My problem is you claim the Dadsv5 series of VM's support this yet the table that the docs show is this one.

It looks to us like there is not cache here. Does it mean we should instead treat the Temp storage as the cache here? Why is this generation the only one that doesn't have cache size in parenthesis.
I also have a follow up. You claim that

Ephemeral OS disks can be placed on the cache or temp storage of the respective VM sizes.

Which is? Those 2 disks appear to have very different sizes in the paragraph you paste. As an example Standars_D2s_v3 has 16GiB of Temp storage but 50GiB of cache size. Should my os image be smaller than 16GiB or 50GiB.
I hope I have transmitted our confusion as understandable as possible. :)

Any updates?

You're finding about Dadsv5 not having cache is true. We will not be supporting cache for Dadsv5 series. If you plan to use Dadsv5 series the only option is to use temp storage for hosting Ephemeral OS disks. I am not sure what you mean by treating temp storage as cache given that both are different.
For the other question, Temp and cache size do differ based on the VM size. Where to place is up to the customer's choice based on their needs and the OS image size the customer is using.
I hope these answers all your questions.

Hi  @Aarthi-Vijayaraghavan
the docs https://learn.microsoft.com/en-us/azure/virtual-machines/dasv5-dadsv5-series say “Premium Storage caching: Supported”, but I think you’re saying that caching is not supported. Perhaps this is a mistake in the documentation? Clarification would be really great, thanks!

#reassign:@mattmcinnes

@mattmcinnes FYI

Thanks for your contribution to our documentation
We sincerely apologize for the delayed response. Unfortunately, we have been unable to review this issue in a timely manner. However, we are making overall enhancements to our content. We are closing this issue for now as there has been no activity for a while. If you feel that the problem persists, please respond to this issue with additional information.
?
Please continue to provide feedback about the documentation. We appreciate your contributions to our community.
#please-close


fix(jest-config): accept snapshotFormat.compareKeys option

The configuration docs states that:

snapshotFormat [object]
[...]
Allows overriding specific snapshot formatting options documented in the pretty-format readme. For example, this config would have the snapshot formatter not print a prefix for "Object" and "Array":

However, I've found out that it's not possible to configure snapshotFormat.compareKeys from within jest.config.js file.
The reason behind this is that you are outsourcing correct types for options to the DEFAULT_OPTIONS from the pretty-format that are declared here:
https://github.com/facebook/jest/blob/c8a84566be774a35a56c26a4595c553df5fc6fc1/packages/pretty-format/src/index.ts#L401
and this is used during validation. But the correct type for snapshotFormat.compareKeys doesn't match the typeof DEFAULT_OPTIONS.compareKeys.

This won't actually works as we serialize the options so we can pass them to workers/child processes. So functions won't survive.
If we are to support compareKeys (and plugins) it need to point to a file that exports a function

Pushed 6baf127e5eef729b39baeb11471a49862e0140de correcting the docs.

Would you perhaps be open to introducing a new option to the pretty-format that would just disable sorting?

Sure. I think I'd prefer overloading the existent function parameter, tho. E.g. supporting some magic string (alphabetical, none) from config. Then pretty-format itself can in addition support a function.

Codecov Report

Merging #12387 (6baf127) into main (c8a8456) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 6baf127 differs from pull request most recent head 47df142. Consider uploading reports for the commit 47df142 to get more accurate results

@@           Coverage Diff           @@
##             main   #12387   +/-   ##
=======================================
  Coverage   68.48%   68.48%
=======================================
  Files         325      325
  Lines       16971    16971
  Branches     5061     5061
=======================================
  Hits        11622    11622
  Misses       5317     5317
  Partials       32       32

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 c8a8456...47df142. Read the comment docs.


python3Packages.aioshelly: 0.6.1 -> 0.6.2

Motivation for this change
Update to latest upstream release 0.6.2
Change log: https://github.com/home-assistant-libs/aioshelly/releases/tag/0.6.2
Enable Home Assistant's test.
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

FAILED tests/components/shelly/test_config_flow.py::test_form[pyloop] - Asser...
FAILED tests/components/shelly/test_config_flow.py::test_title_without_name[pyloop]
FAILED tests/components/shelly/test_config_flow.py::test_form_auth[pyloop] - ...
FAILED tests/components/shelly/test_config_flow.py::test_form_errors_test_connection[pyloop-error0]
FAILED tests/components/shelly/test_config_flow.py::test_user_setup_ignored_device[pyloop]
FAILED tests/components/shelly/test_config_flow.py::test_form_auth_errors_test_connection[pyloop-error0]
FAILED tests/components/shelly/test_config_flow.py::test_form_auth_errors_test_connection[pyloop-error1]
FAILED tests/components/shelly/test_config_flow.py::test_form_auth_errors_test_connection[pyloop-error2]
FAILED tests/components/shelly/test_config_flow.py::test_zeroconf[pyloop] - A...
FAILED tests/components/shelly/test_config_flow.py::test_zeroconf_sleeping_device[pyloop]
FAILED tests/components/shelly/test_config_flow.py::test_zeroconf_sleeping_device_error[pyloop-error0]
FAILED tests/components/shelly/test_config_flow.py::test_zeroconf_sleeping_device_error[pyloop-error1]
FAILED tests/components/shelly/test_config_flow.py::test_zeroconf_confirm_error[pyloop-error0]
FAILED tests/components/shelly/test_config_flow.py::test_zeroconf_confirm_auth_error[pyloop]
FAILED tests/components/shelly/test_config_flow.py::test_zeroconf_require_auth[pyloop]

https://gist.github.com/mweinelt/340848986c8c9dac526c0f3b24c77aef

Result of nixpkgs-review pr 118401 run on x86_64-linux 1

1 package failed to build:

home-assistant

2 packages built:

python38Packages.aioshelly
python39Packages.aioshelly

Tries to join a multicast group and fails with OSError. Likely not possible in the sandbox.

Yes, they do not pass.

@fabaff pushed 0 commits.

What are you doing?

@fabaff pushed 0 commits.

What are you doing?

I think that happens when you just remove a commit.

Result of nixpkgs-review pr 118401 at 1a037df4 run on aarch64-linux 1

2 packages built successfully:

python38Packages.aioshelly
python39Packages.aioshelly

Merging without the tests for now.


Master branch keyboard doesn't display symbols on iOS

Steps to Reproduce
Switch to master, run flutter upgrade. Add a TextFormField and launch the app in release on iOS.
Expected results: A full set of symbols is available when tapping on the 123/symbols button
Actual results: A numpad is displayed mimicking the behaviour of keyboardType: TextInputType.numberWithOptions() (where before it would display the full set of symbols)
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel master, 1.21.0-6.0.pre.141, on macOS 11.0 20A5323l, locale en-GB)

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 11.6)
[✓] Chrome - develop for the web
[✓] Android Studio (version 4.0)
[✓] VS Code (version 1.47.3)

[✓] Connected device (4 available)

• No issues found!

The widget:
  Padding(
                  padding: EdgeInsets.all(10),
                  child: TextFormField(
                    controller: _emailController,
                    decoration: InputDecoration(
                      labelText: "Email Address",
                      border: OutlineInputBorder(),
                    ),
                  ),
                ),

this is still a very large issue and one causing a lot of issues from clients. is there a fix for this? Have tried many things, but still ios on iphone does not show punctuation, but it does on ipad and android.

this came after updating flutter to master branch, same logs as you, same exact issue as yours.

Hi @Pebsie,
There's similar ticket describing your case, 62659 and the fix has been merged. See if this resolves your issue.


Native object lift

Fixes #209
Package for external testing: react-native-powerauth-mobile-sdk-2.5.5.tgz

Tested by our client - works OK for them


Add protobuf_javascript_esm module.

Checked with python3 tools/bcr_validation.py --check=protobuf_javascript_esm.
The module in question is my fork of the protobuf-javascript repo. The changes in the fork are unlikely to be accepted upstream (see https://github.com/protocolbuffers/protobuf-javascript/pull/156).

Hello @bazelbuild/bcr-maintainers, modules without existing maintainers (protobuf_javascript_esm) have been updated in this PR. Please review the changes.

Maybe protobuf_javascript_gonzojive is a better name to indicate this is a fork?

Following other examples in the registry, <org>_protobuf_javascript would fit well.

Hello @bazelbuild/bcr-maintainers, modules without existing maintainers (gonzojive_protobuf_javascript) have been updated in this PR. Please review the changes.

Done.

please fix check failures.

I suspect it will work now. I removed debian10 due to a C++ error in protobuf that I'm not sure how to resolve.


NoSuperfluousPhpdocTagsFixer - Attribute handling

fixes https://github.com/FriendsOfPHP/PHP-CS-Fixer/issues/6349

Coverage increased (+0.002%) to 93.123% when pulling 3bb2700edfd7d45c16bae0231e0c5848e32fd77d on SpacePossum:master_6349 into aa805a8a4d8384c71a1367fceb57a081a4d969e4 on FriendsOfPHP:master.


STORM-627 - Storm-hbase configuration error

correct problem reading configuration data as reported in STORM-627 - Storm-hbase configuration error

+1.

+1

@msdotnetclr  Thanks for the patch.


[dv] Multi-reg handling: update to #7185

See notes in #7185 for what's going on.

This looks great. I'm taking next 6 days off too. Will try this out when I'm back. Thank you very much.


ARROW-15415: [C++] Fixes for Windows Debug build

This fixes various issues I encountered while trying to create a debug build on Windows using MSVC and vcpkg:

Python debug and release libraries are passed, but our Cmake scripts only expect one or the other. This is the same issue reported in ARROW-13470
Since vcpkg upgraded gtest to 1.11.0, there is again a mismatch between the bundled gtest and the vcpkg versions. So we get the same error as was found in ARROW-14393. The only reason it was using bundled gtest was a carveout for an issue with Conda (does it even apply anymore?), so I made that if-statement more specific so it didn't interfere with vcpkg dependencies.
Thrift could not find debug static libraries, because it was missing the "d" suffix. It should be libthriftmdd.lib, but was finding libthriftmd.lib.
Additionally, the recommended clcache program from our Windows developer docs is no longer maintained. I found its dependency pyuv doesn't install on Windows anymore, and is also no longer maintained. I've dropped mention of it from the docs.

There is a fork of clcache at https://github.com/Nuitka/clcache/ . Note we use clcache on AppVeyor, so it still works, presumably. See e.g. https://ci.appveyor.com/project/ApacheSoftwareFoundation/arrow/builds/42300017/job/imhh1jrmclhooba9#L2859

cc @kou

Closing in favor of #13108


Spike: Using OpenAPI specs for Admin API

Scope:

Identify the way we could make use of OpenAPI specs of Admin API (generated clients, validations, etc.)
Draft a PoC integration with an Admin API exposing OpenAPI specs (these can be faked for this PoC)
Propose to Kong Gateway maintainers introduction of OpenAPI specs that got battle-tested in the PoC above

Closing as stale: we previously decided that we wouldn't prioritize this as the cost/value was not expected to be very high in comparison to other efforts we were trying to prioritize at the time. If we find that this has changed or there's newfound requirements where this work becomes an implication, let's re-open.


Reset Discord status after a while of being in menu

Closes #667

Thank you for the remarks. I think all the issues should be cleared up

This should probably just not display time anymore instead of removing entire rich presence

@deathbeam I Removed the timer

Code LGTM, but the test could use improvement imo

I implemented the changes you requested. Thanks for your input


[linux] pin Hashicorp tools due to license change

Description
pin Hashicorp utilities due to license change.
Related issue:
https://github.com/actions/runner-images-internal/issues/5325
Check list

[x] Related issue / work item is attached
[ ] Tests are written (if applicable)
[ ] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

/azp run ubuntu2004,ubuntu2204


Youtube Videos Comments Dataset Added

Youtube Videos Comments Dataset contains contains comments from youtube videos, I think this dataset will help people understand which content to upload and how to reach the audience. Youtube is very very popular media platform everyone knows.

Please Accept this under hacktoberfest I need one PR to complete Hacktoberfest

please consider this under Hacktoberfest I need only one PR to complete my hacktoberfest

@Harshil0512 |  Hacktoberfest is over, this PR will anyhow not gets count.

it's showing in my Hacktober fest page you just need to add hacktoberfest accepted tag

This is my last hope please help me

@Harshil0512  | We still have the hacktoberfest label in our repository, if they are still accepting PR. This PR will automatically get included.

It's showing in Not Accepted State as it's not accepted under Hacktoberfest but yesterday my one PR was accepted and gone into waiting state as it was accepted under Hacktoberfest


Is this project dead?

There are a lot of open PRs that fix real bugs but haven't been merged in a while. Is there anyway to merge them perhaps in a fork and re-upload to npm?

project is dead, look at the owner's profile https://github.com/diegonetto

Do you think someone could take this over? I want to fork this and have my own version that fixes many of the bugs that were fixed from the pending pull requests, but honestly, I think it'd be best if we had someone take this over. Perhaps if we ask nicely enough, maybe we can request yeoman make an official one, based from this one,?

I'd be willing to put the time in to take this over, but I don't know if @diegonetto is going to be around to look at this.

:+1:

Hey everybody. We've been developing a similar tool. Regularly maintained. If you're interested check it ou: https://github.com/mwaylabs/generator-m-ionic

I have been using generator-m-ionic linked by @gruppjo above since my original query. Can vouch for its awesomeness.


Ensure thread safety

This PR ensures all classes and functions are thread-safe by adopting the @MainActor attribute.
With the adoption of async/await we've introduces potential race conditions and made some of the classes not thread safe.
A MainActor is a globally unique actor who performs his tasks on the main thread.
I think adopting MainActor makes sense because:

it ensures thread safety throughout the library.
BridgeDelegate is usually instantiated in a UIViewController, which conforms to MainActor itself, thus no additional code changes are needed.
BridgeComponents are usually used to interact with UI, which must run on the main thread.

This PR also fixes an error when using the async version of evaluateJavaScript. More info here.

I've tested these changes in the Calendar app, and so far haven't seen any regressions or issues.

@joemasilotti Unfortunately I'm running into some issues. Will continue tomorrow.

@joemasilotti Could you please give this branch a try in one of your projects? I'm running into an issue and can't seem to get it to work anymore. The components are successfully registered when the bridge sends 'ready', but after that, no more messages are received from the bridge. I've double-checked the code, but I'm unable to identify what might be causing this. I'm running out of ideas.
@zoejessica Could you please double check the code?

I'm not seeing any components but I do see bridgeNotInitializedForWebView in the console.
I think making Bridge.initialize() async is breaking it. I need to return the web view right away to work with Turbo Navigator but it isn't initialized until the completion block calls.
Turbo.config.makeCustomWebView = { configuration in
    configuration.defaultWebpagePreferences?.preferredContentMode = .mobile

    let webView = WKWebView(frame: .zero, configuration: configuration)
    webView.makeInspectableInDebugBuilds()
    Bridge.initialize(webView) { }
    return webView
}

If this change is required we can refactor the makeCustomWebView block to be async as well.

@joemasilotti I've seen bridgeNotInitializedForWebView when the web view is required right away. Subsequent calls do work though, however the components are still broken.
Man, it should be trivial to start a Task on the @MainActor in a synchronous way, when the task is started from the main thread. But nope. I'm still looking for alternatives.

Man, it should be trivial to start a Task on the main actor in a synchronous way, when the task is started from the main thread.

Agreed, I just want a blocking call!
Does initializing the Bridge have to be asynchronous? What if there was a non-async option, too?

Does initializing the Bridge have to be asynchronous? What if there was a non-async option, too?

Having @MainActor attribute enforces it. However, I can remove it from the class level, and only mark the async function with it.

Excellent, everything is working for me, too! Will merged when the tests pass.

I had to manually merge this on the command line for some reason. Thanks @svara!


Simplify boolean return pattern?

Code like:
        if (globalDelegationHashes[hash].length >0) {
            return true;
        } else {
            return false;
        }

could be simplified to:
        return globalDelegationHashes[hash].length > 0

this pattern appears a few times in the code, so this would reduce the LOC meaningfully.

It will also reduce cost to deploy  the initial contract and overall simplify the contract

merged with prxt PR.


Running tests skips build artifact upload process

Before opening, please confirm:

[X] I have checked to see if my question is addressed in the FAQ.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

App Id
dm6pmstqyhyh5
Region
eu-west-2
Amplify Hosting feature
Build settings, E2E tests
Describe the bug
Running tests during the pipeline with Playwright causes a skip uploading the build artifacts resulting in a deployment failure due to no files and ends the frontend build at 2022-06-15T11:41:56.935Z [INFO]: # Caching completed instead of uploading them.
The provision, build, and test stages all pass, then deployment fails due to no build files.
Expected behavior
After 2022-06-15T11:41:56.935Z [INFO]: # Caching completed, the following appears in the frontend build on successful deployments.
I'm uncertain about the issue with the customHeaders. The file is empty on Amplify. This only appears when the artifact upload does.
2022-06-15T11:41:57.017Z [INFO]: !!Invalid custom header config file. The `customHeaders` key should be an array
2022-06-15T11:41:57.048Z [INFO]: # Starting build artifact upload process...
2022-06-15T11:41:57.523Z [INFO]: # Build artifact is: 5MB
2022-06-15T11:41:57.523Z [INFO]: # Uploading build artifact '__artifactsHash.zip'...
2022-06-15T11:41:57.524Z [INFO]: # Build artifact is: 5MB
2022-06-15T11:41:57.525Z [INFO]: # Uploading build artifact '__artifacts.zip'...
2022-06-15T11:41:57.690Z [INFO]: # Build artifact upload completed
2022-06-15T11:41:57.691Z [INFO]: # Starting environment caching...
2022-06-15T11:41:57.692Z [INFO]: # Uploading environment cache artifact...
2022-06-15T11:41:57.785Z [INFO]: # Environment caching completed

Reproduction steps
Reproducible by using the following build settings and running a build deployment.
Build Settings
version: 1
backend:
  phases:
    build:
      commands:
        - '# Execute Amplify CLI with the helper script'
        - envCache --set stackInfo ""
        - amplifyPush --simple
frontend:
  phases:
    preBuild:
      commands:
        - npm ci
    build:
      commands:
        - npm run build
  artifacts:
    baseDirectory: /dist
    files:
      - '**/*'
  cache:
    paths:
      - node_modules/**/*
test:
  phases:
    preTest:
      commands:
        - echo fs.inotify.max_user_watches=65536 | tee -a /etc/sysctl.conf && sysctl -p
        - npm ci
        - npm install pm2
        - npm i -D @playwright/test
        - npx playwright install
        - npx pm2 start npm -- start
    test:
      commands:
        - npx playwright test
    postTest:
      commands:
        - npx pm2 kill
  artifacts:
    baseDirectory: /test-results
    files:
      - '**/*'

Additional information
This is a similar issue to: https://github.com/aws-amplify/amplify-hosting/issues/41 https://github.com/aws-amplify/amplify-hosting/issues/963
I've tried a number of different build settings, but nothing changes unless the tests aren't run.

Hi @itswyll 👋🏽 thanks for providing reproduction steps. We are investigating this.

+1

The team has investigated this further and unfortunately we only support Cypress E2E testing at this time. We are working on updating our documentation to reflect this. We are also adding a feature request to extend support for Playwright tests. We will update this issue: #2857 once we provide support.


can't build alloy smelter in the craftingtable

I use the the world of mianite modpack with the techniclauncher.
When i put everything into the craftingtable (i tried with every copper ingot ingame) there is nothing --> no alloy smelter (even in creative mod i can't make them in the craftingtable)
On a sidenote when i try to open the recipes it crashes (but i think thats a problem of notenoughitems)

what version are you using ? this seems like a very old one

If we are talking about enderio i was using enderio-1.7.10-<IP_ADDRESS>7
I downloaded the newest EnderIO-1.7.10-<IP_ADDRESS>9.jar but still i cant get it to work.
I use 1.7.10 (if you want to know my minecraft version)
(Tbh i thought TechnicLauncher would auto update my mod packs :-/ )

there is a option in the config to disable recipe from Mod ingots and only use ingots from Vanilla & EIO
the current recipe in version 2.2.8 349 for Alloy Smelter is this : https://www.dropbox.com/s/chelzsvr20tzwv7/123.jpg?dl=0
if you didn't change the config file instead of iron ingot in the first recipe use copper ingot

Thank you very much. I got the wrong recipe (o.0)
Finally i got my smelti (I even got a nick name for him)


Add styling options to table header guidance

This is an update to the guidance to include the additional table styles created in pull request #2048

:heavy_check_mark: You can preview this change here:
:hammer: Explore the source changes: 55329b2e1d4c93a3c978c1e6eba580bd968addaa
:mag: Inspect the deploy logs: https://app.netlify.com/sites/govuk-design-system-preview/deploys/5ffdae3f6fb4a70007bcf71c
:sunglasses: Browse the preview: https://deploy-preview-1448--govuk-design-system-preview.netlify.app

:heavy_check_mark: You can preview this change here:
:hammer: Explore the source changes: 55329b2e1d4c93a3c978c1e6eba580bd968addaa
:mag: Inspect the deploy logs: https://app.netlify.com/sites/govuk-design-system-preview/deploys/5ffdae3f6fb4a70007bcf71c
:sunglasses: Browse the preview: https://deploy-preview-1448--govuk-design-system-preview.netlify.app

Closing this pull request as it is now being covered in this pull request

Closing this pull request as it is now being covered in this pull request


Scan Local Network for Active Servers

I could really use an option to scan the local network for servers listening on the selected port, or at least an option to ping servers by IP and port, without the long timeout that comes with trying to connect to a server not knowing if it's even online...
Not sure if I'm clear enough but I'm trying to make a game with LAN/WLAN multiplayer that has autodiscovery of available hosts.
If it's possible to do that as is I'd be extremely grateful for any help.

Sounds like you're looking for the LanDiscovery class 😉
For a short explanation on setting it up, take a look at this.


Fix env parsing

Fixes #253
Alternative to #257
Turns YES and NO that get parsed to bools in yaml, back into YES and NO strings.
Also turns any other type into a string (eg Int)

@turekj


Create new ARC20 spec

This ARC introduces a design for minimal Fungible Tokens just like ERC20. It allows for transferring tokens and approving programs to transfer tokens on your behalf.
A big thank you to Valentin Seehausen, Evan Marshall, FullTimeMike and authors of the previous two ARC20 specs Ghostant-1017 and EdVeralli.
Discussion link: https://github.com/AleoHQ/ARCs/discussions/42

Let's not use the outdated approach of approve() and transfer_from(). The probem with this is that this is a two step procedure, while it is possible to do it in one step like ERC-2612 using permit().

Let's not use the outdated approach of approve() and transfer_from(). The probem with this is that this is a two step procedure, while it is possible to do it in one step like ERC-2612 using permit().

Agree with r001. Would also second the importance of being able to transfer tokens into smart contracts, which I believe is part of r001's wip proposal.

Hi @r001 & @asharma13524, thank you for the thoughts! I had some discussions about this with the people mentioned at the top of this PR and almost ended up in a flamewar. I decided to pursue the ERC20 route for now because it is well known, and it makes sense to call it ARC20 if it looks like ERC20. Also note that due to the nature of the batch zero knowledge proofs on Aleo, it should be possible in the near-term to execute both approve and transfer_from for effectively the price of a single execution.
More proposals like ERC-2612 are very much welcome, maybe it will have merit to name it ARC-2612 as well!

Also note that due to the nature of the batch zero knowledge proofs on Aleo, it should be possible in the near-term to execute both approve and transfer_from for effectively the price of a single execution.
I don't want to push things hard, but please also consider that:

Takes additional system resources to call approve() which in turn updates approvals mapping. While all this is not necessary by  applying signatures.
Makes the codebase unnecessarily more difficult by having approve() transition and allowance mapping.

@vicsn @evanmarshall  Do you think join and split functions should be included in the ARC20 standard ? As I can see they are currently not.

You might want to consider an update to the unapprove logic. As it stands, if someone wants to unapprove the entire balance of another user, they have to specify the exact amount the user is currently approved for. This could cause some some annoying usability issues as it creates a race condition on the current approval. Instead, if an amount greater than the current approval is specified, the approval should just be reduced to zero (rather than reverting).


SemanticsTester and its friends are not public

Use case
Unit testing semantic trees is difficult without using SemanticsTester, TestSemantics and hasSemantics() (all found in semantics_tester.dart). This is what flutter seems to do for testing semantics, but these classes are not public so our team is forced to copy this file into our own code base.

Proposal
Please make semantics_tester.dart public so that we can use SemanticsTester, TestSemantics and hasSemantics(), etc for testing semantics. Thanks!

Any reason why this is not public? It's really useful.

Found the right way to do this. For anyone else looking for the same use matcherSemantics.

Found the right way to do this. For anyone else looking for the same use matcherSemantics.
I think this issue can be closed. @darrenaustin @iapicca

Maybe we should also hear what the report has to say @rzagelow

This looks like it will work.. I'll report back in a couple days to confirm.. I do see what I think is a mistake in the documentation example where "matchesSemanticsNode" is referenced.. I think it should be "matchesSemantics".. here is the sample code I'm talking about:
final SemanticsHandle handle = tester.ensureSemantics();
expect(tester.getSemantics(find.text('hello')), matchesSemanticsNode(label: 'hello'));
handle.dispose();

Sorry for the delayed response..

This worked perfect for me @abhaysood!.. thanks!.. we can safely close @iapicca!.. Thanks both of you!!

Closing as matchesSemanticsNode works for this case.


Missing Denom Assets crash the API server

We parse Denoms into their human-friendly amount by:

Pulling in the Denoms' DenomAssets'
Finding the highest exponent
Dividing the Amount by 10^(highest denom exponent)

This allows us to convert e.g. 6ujuno -> .000006 JUNO in a generalized way.
However, when a Denom is not found in our asset list during Transaction indexing, we add an empty Denom with no DenomAssets and a symbol of UNKNOWN.
Then, during the run of the API CSV generator, we throw an error when there are no DenomAssets for an UNKNOWN Denom.
We either need to:

Provide coverage for all UNKNOWN Denoms and their DenomAssets
Provide a fallback mechanism so the application can continue on

In #411, I have removed the fatal errors that were crashing the API server.
In #410, I have added a validation method to update-denoms that ensures all denoms have at least 1 corresponding denom_unit with exponent = 0

This should prevent the API server from crashing due to denom and denom_unit errors, and will at least return a 500 response to the client.
It will also provide CSVs even if a symbol is in an UNKNOWN state during generation, they will just not get converted to the 6th exponent and will not have a symbol applied.
I am still exploring what our fallback mechanism should be.


Scaling or moving windows too quickly can cause them to "get stuck" sometimes

First of all, this is an awesome library and I am not downplaying that.
But if the mouse is moved out of the draggable area on one "frame" the window ignores any input until the user brings the mouse back, upon which it'll snap which is highly annoying.

Thanks for the nice words, I'am appreciate it. This issue only occurs when using iframes as content body. I made some event capturing tests and find out that the mouse event will be "consumed" by the iframe when fast moving and release the mouse in a moment where the cursor is hovering the iframe (same window as well as other windows with an iframe), and does not bubble outside the iframe document, so the "mouseup" event is getting lost. I could fix that by adding additional "mouseleave" or "mousemove" event, but in then fast moving gets always interrupted immediately (but the issue has gone). That fix feels more like a downgrade, especially when not using iframe contents. I need to find another way to fix that.

I'm not entirely sure how any of this works but if this is possible try it:
Add div with a zindex lower than iframe
when we capture mousedown, we raise this transparent div to above the iframe

F

F

G

@photonstorm Thanks a lot for this hint. It is now fixed also.


remove transitive config code that resides in spec

not used anymore, we don't need direct dependencies since all dependencies are captured indirectly via dependend building blocks
...should be thrown out

Already done...


Refactor TestRequest_Usage to be table-driven test

I've been giving this one a go - feel free to assign to me if that works for you.


Explicitly track metadata spreadsheet downloads in HCA DCP analytics report

Need
The HCA DCP has added a new analytics report to the data browser, holding a spreadsheet with full metadata. We would like to track downloads of this spreadsheet to help understand its utility.
Approach
Add a specific analytics event for this download and capture if it is from:

The project entity list
The project detail page

TODO: Determine if we can track this on the files direct download page.

complete


Add util function to clean up created entities from DB during testing

DeleteCreatedEntities records all created entities on the gorm.DB connection
and returns a function which can be called on defer to delete created entities
in reverse order on function exit.
Usage:
func TestDatabaseActions(t *testing.T) {

  // setup database connection
  db := ....
  // setup auto clean up of created entities
  defer DeleteCreatedEntities(db)()

  repo := NewRepo(db)
  repo.Create(X)
  repo.Create(X)
  repo.Create(X)
}

Output:
Deleting from x 6d143405-1232-40de-bc73-835b543cd972
Deleting from x 0685068d-4934-4d9a-bac2-91eebbca9575
Deleting from x 2d20944e-7952-40c1-bd15-f3fa1a70026d

@tsmaeder Similar in function to your undo scripts, but maybe a bit easier to use. Not studied the usage of undo scripts in detail, but at least from a 'clean up created' perspective, this function seems to do the job.

Current coverage is 66.45% (diff: 0.00%)

Merging #535 into master will decrease coverage by 0.28%

@@             master       #535   diff @@
==========================================
  Files            62         63     +1
  Lines          3286       3300    +14
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           2193       2193
- Misses          857        871    +14
  Partials        236        236

Powered by Codecov. Last update 9bd5c82...a6dcec9


Undrainable Tanks

Forge: 2811
Mekanism(main/generators/tools): <IP_ADDRESS>4
Fluids cannot be pulled out of a dynamic tank either via mekanism's own piping or via that from other mods (I tested with a barebones profile prior to this report, it's a mekanism issue). May be related to the prior unfillable tank issue.

Please test with latest version.

I can't type, that was supposed to be an 8, not a 4 #longday


optimization for loop

由于改动较大文件不存在了，我手动修改了，感谢一丝不苟的少年哈。

OK
On Sat, Dec 22, 2018 at 11:04 PM 布兰柯基<EMAIL_ADDRESS>wrote:

Closed #729 https://github.com/Blankj/AndroidUtilCode/pull/729.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Blankj/AndroidUtilCode/pull/729#event-2042135282, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AfEC10e0D-9TBqziJhgusCRE4RtykTqTks5u7kn3gaJpZM4Y27Bd
.


SMT CarbonFootptint (IDTA 2023-0-9) could not be loaded into AASPE

The AASX-File of the Template has been updated April 30, 2024. Since then it can not be loaded into the Package Explorer (Version v2024-06-10)
Error:  375 gave: While opening the package to read AASX C:\Users\RIS\AppData\Local\Temp\tmp5AA9.aasx at \AdminShellPackageEnv.cs:line 366 gave: End of Central Directory record could not be found.)

@foprs Please use the latest version of the AASPE tested both the AASX and JSON files they are working as intended.


Number Sequence is very slow on Cloud environments

Please include the following with each issue:
1. Describe the bug
NumberSequence is slowest way to get next number in Cloud Business Central.
Each NumberSequence.Next take 3ms

In my analysis, I came to the conclusion that the issue is a slow hard disk for Cloud Business Central.
One of the possible ways to improve performance it's to add CACHE parameter upon NumberSequence.Insert
In this case we can reduce number of IOs for SQL Server NumberSequence object
More information in article https://vld-nav.com/number-generation-performance-cloud
I don't have partner account, so if you think this isn't the right place, I won't be able to create a ticket.
2. To Reproduce
Steps to reproduce the behavior:
Create number sequence and call it in loop in Cloud Business Central.
NumberSequence.Insert('TestPerformance');
for i := 1 to 100000 do
    NumberSequence.Next('TestPerformance');

3. Expected behavior
It should work at least faster than Rec.Modify()/UPDATE statement in AL/SQL.
4. Actual behavior
It's super slow.
5. Versions:
Latest available Cloud Business Central

We cannot help you if you do not have a partner account.
Try with the repos for AL developers (e.g the repo for the system application) . Maybe they have better ideas and experience.
Nevertheless it is not for this repo.


Support OpenAI models via Azure deployments

🚀 The feature
Support OpenAI models via Azure deployments
Motivation, pitch
We don't have key of directly accessing the openai models.

Hey @wxp16, this issue is already resolved. We have also added a jupyter notebook for you to test the Azure OpenAI integration with Embedchain: https://github.com/embedchain/embedchain/blob/main/notebooks/azure-openai.ipynb
You can find the docs for OpenAI Azure integration here: https://docs.embedchain.ai/components/llms#azure-openai


Move artifacts gathering around

moves logging of node and pod info into artifact gathering in teardown
removes running diagnostics at least for now
avoid version check panic

Depends on https://github.com/Azure/acs-engine/pull/3152

/assign jim-minter

/unassign

Rebased

/label orchestrator/openshift

Flaked on https://github.com/Azure/acs-engine/issues/2952

/lgtm
/unassign kargakis

/unassign jim-minter

@jackfrancis @CecileRobertMichon please merge

/approve


Fix restart handling testing

Closing as this needs more work done on it that I don't currently have time for


WebView2 error when loading local app package file

Describe the bug

I cannot seem to get WebView2 to load a file from the app package. Setting the source to a file in app package results in error in console.
Steps to reproduce the bug
Start a blank WinUI project.
Install necessary packages.
Add index.html to app package, containing random html code.
Add WebView2 in XAML
WebView2 x:Name="MyWebView_3" Source="ms-appx-web:///help/index.html"/
Run app and open debug console
Expected behavior

Expect index.html to be located and loaded.
Screenshots

Version Info

NuGet package version: 5.9.0

Microsoft.WinUI 3.0.0.-preview3.201113.0

Windows app type:

UWP
Win32

Yes

Windows 10 version
Saw the problem?

Insider Build (xxxxx)

May 2020 Update (19041)
Yes

November 2019 Update (18363)

May 2019 Update (18362)

October 2018 Update (17763)

April 2018 Update (17134)

Fall Creators Update (16299)

Creators Update (15063)

Device form factor
Saw the problem?

Desktop
Yes

Xbox

Surface Hub

IoT

@adambarlow Fyi

Looks like a duplicate of https://github.com/microsoft/microsoft-ui-xaml/issues/1967 (this issue has an additional error output description).

Closing as a dup, as noted in the previous comment.


Don't log full exception traceback in direct runner.

It will be logged later if not caught.
This reduces duplication for failing tests, and noise for tests
expecting failure.
Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify.
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

R: @aaltay

LGTM

Irrelevant Spark failure.

Coverage decreased (-0.06%) to 70.531% when pulling 07faa819228586cd4e4427106404a993eb8045ee on robertwb:py-exceptions into 152c5bcc7f6fb53565ed132653384f4244542b59 on apache:master.


cannot install in python3.6

Dear Catalyst Maintainers,
I have tried to install catalyst several times, it always fail.  Really need help. Python 3.6 version.  please
Before I tell you about my issue, let me describe my environment:
Environment

Operating System: (Windows Version or $ uname --all)
Python Version: $ 3.6 python --version
Python Bitness: $ python -c 'import math, sys;print(int(math.log(sys.maxsize + 1, 2) + 1))'
How did you install Catalyst: (pip, conda, or other (please explain))
Python packages: $ pip freeze or $ conda list

Now that you know a little about me, let me tell you about the issue I am
having:
Description of Issue

What did you expect to happen?
install sucessfully
What happened instead?

Here is how you can reproduce this issue on your machine:
Reproduction Steps

conda create python36
pip install enigma-catalyst matplotlib

...
What steps have you taken to resolve this already?
...
Anything else?
...
Sincerely,
$ whoami

I have installed the catalyst.  While, when i import it.   it shows runfile('C:/Anaconda3-4.2/lib/site-packages/catalyst/utils/run_algo.py', wdir='C:/Anaconda3-4.2/lib/site-packages/catalyst/utils')
Traceback (most recent call last):
File "", line 1, in
runfile('C:/Anaconda3-4.2/lib/site-packages/catalyst/utils/run_algo.py', wdir='C:/Anaconda3-4.2/lib/site-packages/catalyst/utils')
File "C:\Anaconda3-4.2\lib\site-packages\spyder\utils\site\sitecustomize.py", line 866, in runfile
execfile(filename, namespace)
File "C:\Anaconda3-4.2\lib\site-packages\spyder\utils\site\sitecustomize.py", line 102, in execfile
exec(compile(f.read(), filename, 'exec'), namespace)
File "C:/Anaconda3-4.2/lib/site-packages/catalyst/utils/run_algo.py", line 13, in
import catalyst
File "C:\Anaconda3-4.2\lib\site-packages\catalyst_init_.py", line 25, in
from .utils.run_algo import run_algorithm
File "C:\Anaconda3-4.2\lib\site-packages\catalyst\utils\run_algo.py", line 16, in
from catalyst.exchange.exchange_pricing_loader import ExchangePricingLoader, \
File "C:\Anaconda3-4.2\lib\site-packages\catalyst\exchange\exchange_pricing_loader.py", line 17, in
from catalyst.exchange.utils.factory import get_exchange
File "C:\Anaconda3-4.2\lib\site-packages\catalyst\exchange\utils\factory.py", line 4, in
from catalyst.exchange.ccxt.ccxt_exchange import CCXT
File "C:\Anaconda3-4.2\lib\site-packages\catalyst\exchange\ccxt\ccxt_exchange.py", line 7, in
import ccxt
File "C:\Anaconda3-4.2\lib\site-packages\ccxt_init_.py", line 29, in
from ccxt.base.exchange import Exchange                     # noqa: F401
File "C:\Anaconda3-4.2\lib\site-packages\ccxt\base_init_.py", line 24, in
from ccxt.base import exchange
File "C:\Anaconda3-4.2\lib\site-packages\ccxt\base\exchange.py", line 78, in
from web3 import Web3, HTTPProvider
File "C:\Anaconda3-4.2\lib\site-packages\web3_init_.py", line 7, in
from eth_account import Account  # noqa: E402
File "C:\Anaconda3-4.2\lib\site-packages\eth_account_init_.py", line 1, in
from eth_account.account import Account  # noqa: F401
File "C:\Anaconda3-4.2\lib\site-packages\eth_account\account.py", line 10, in
from eth_keyfile import (
File "C:\Anaconda3-4.2\lib\site-packages\eth_keyfile_init_.py", line 7, in
from eth_keyfile.keyfile import (  # noqa: F401
File "C:\Anaconda3-4.2\lib\site-packages\eth_keyfile\keyfile.py", line 10, in
from eth_keys import keys
File "C:\Anaconda3-4.2\lib\site-packages\eth_keys_init_.py", line 15, in
from .main import (  # noqa: F401
File "C:\Anaconda3-4.2\lib\site-packages\eth_keys\main.py", line 3, in
from eth_keys.datatypes import (
File "C:\Anaconda3-4.2\lib\site-packages\eth_keys\datatypes.py", line 8, in
from eth_utils import (
File "C:\Anaconda3-4.2\lib\site-packages\eth_utils_init_.py", line 24, in
from .applicators import (  # noqa: F401
File "C:\Anaconda3-4.2\lib\site-packages\eth_utils\applicators.py", line 29, in
def combine_argument_formatters(*formatters: List[Callable[..., Any]]) -> Formatters:
File "C:\Anaconda3-4.2\lib\typing.py", line 1025, in getitem
tvars = _type_vars(params)
File "C:\Anaconda3-4.2\lib\typing.py", line 284, in _type_vars
_get_type_vars(types, tvars)
File "C:\Anaconda3-4.2\lib\typing.py", line 279, in _get_type_vars
t._get_type_vars(tvars)
File "C:\Anaconda3-4.2\lib\typing.py", line 786, in _get_type_vars
_get_type_vars(self.args, tvars)
File "C:\Anaconda3-4.2\lib\typing.py", line 277, in _get_type_vars
for t in types:
TypeError: 'ellipsis' object is not iterable

Hi @studyquant ,
The error you received seems to be the same discussed at #461, which resulted from a problematic Python version.
If after upgrading to Python 3.5.3+ or using 3.5.1 you still experience this issue, please share the following information:

Your operating system
Python version
Catalyst version

Lena

Thanks, I have sucessfully instelled the catalyst


Using SupContrast loss for non-image application

Hey
I am about to use the SupConLoss for my specific application:
As I am embedding some graphs and would like to make the embeddings of similar graphs to get more similar, taking advantage of this loss might be helpful. but I have faced some problems:

the input of forward function should have a shape like: bsz, n_views, n_features however, I don't have copies for each embedding (equivalent to "images' crops"). So: my model's output has a shape like: (bsz, d_embedding) and I assumed I can match it to you loss function by unsqueeze(1) which makes it shaped like: (bsz, 1, d_embedding)
The above mentioned input, results in nan
As I have tracked the variables in your loss function:
Also as I use mask, it has one and zero values. mask(i,j)=1 which means to me that the i'th and j'th embedding in my batch should get close.

In case of using 'label' argument, I am about to make those embeddings with same label to get close to each other. (mask is  then automatically created)

Are these assumptions right?
appreciate

hello, have you solved this problem?


How to get video duration on Android?

Hi!
I need to get the total duration of a video and I'm not clear how to do it, without using Youtube Data API.
I was researching a bit and in the documentation of Youtube API Player is the method getDurationMillis but my implementation was not successful.
Is there any way to implement this correctly? It is only implemented on iOS and is required on android too.
My implementation was the next.. based on https://github.com/inProgress-team/react-native-youtube/pull/146
YouTubeManager.java
....
public int getDuration(YouTubeView view) {
    return view.getDuration();
}
....

YouTubeView.java
....
public int getDuration() {
    return mYoutubeController.getDuration();
}
....

YouTubePlayerController.java
....
public int getDuration() {
    return mYouTubePlayer.getDurationMillis() / 1000;
}
....

YouTubeModule.java
....
@ReactMethod
public void duration(final int reactTag, final Promise promise) {
    try {
        UIManagerModule uiManager = mReactContext.getNativeModule(UIManagerModule.class);
        uiManager.addUIBlock(new UIBlock() {
            public void execute (NativeViewHierarchyManager nvhm) {
                YouTubeView youTubeView = (YouTubeView) nvhm.resolveView(reactTag);
                YouTubeManager youTubeManager = (YouTubeManager) nvhm.resolveViewManager(reactTag);
                 int duration = youTubeManager.getDuration(youTubeView);
                 promise.resolve(duration);
            }
        });
    } catch (IllegalViewOperationException e) {
            promise.reject(E_MODULE_ERROR, e);
    }
}
....

YouTube.android.js
....
duration() {
  return new Promise((resolve, reject) =>
    NativeModules.YouTubeModule
      .duration(ReactNative.findNodeHandle(this._nativeComponentRef))
      .then(duration => resolve(duration))
      .catch(errorMessage => reject(errorMessage)),
  );
}
....

I'll handle it soon when I find some time

@davidohayon669 any news about this? I think it is necessary also on Android.

@hasaezs 8d579baf1f1b55293c38cc937a39b173d221352a next version

Is it possible to get video duration in youtube standalone player on Android?


json error

i get this error every time in every course

can u pls provide the link

pls provide th

all the courses the bot isnt working anymore
i restart the program but i still have the same problem in all coupon websites

can u pls provide the course link which is causing trouble

every course all the courses

Can i get ur system info

System Info:
OS: Windows 10 pro 64 bits version 19041
python version 3.7.0
d [e.g. chrome, safari] : Version 87.0.4280.88 (Build officiel) (64 bits)

Will take some time any way will releases a better updated version.

i get this eroor with the new version . i don t know why even if i installed all the requirement libraries

pip install html5lib

sorry for it

I guess it fixed


Enhancement idea: Support multiple languages.

Dwarf Fortress can't be translated. But if you can control it from Armok Vision then translating Armok Vision into multiple languages can be good idea. @JapaMala What do you think of it? PS. Sorry about my bad English.

I barely support even a single language as-is, but it's something to consider down the road.

@JapaMala I wasn't thinking about you supporting many languages. I was thinking about building this application so it an support multiple languages. (Doesn't have hardcoded text).


Sales analytics .pbix file isnt opening. Error message "Required property 'connections' not found in JSON. Path"",line 1, position 136

[Enter feedback here]

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

ID: d92bb764-442b-4116-0c78-b64ae77a00e3
Version Independent ID: 118c80e2-c1a1-8d40-3f6d-b488c080fa9f
Content: Customize Power BI template apps to work with Dynamics 365 Sales
Content Source: ce/sales-enterprise/customize-template-apps.md
Service: dynamics-365-sales
GitHub Login: @sbmjais
Microsoft Alias: shjais

Thanks for your feedback. We are closing this issue because we are either unable to find relevant information or reproduce this. If you're still experiencing the issue described here, please reopen or create a new issue.


Typos

Description:
Fixed errors and inaccuracies in the project's textual materials.
Type of Change:
Fixing errors (typos, grammatical mistakes, etc.)
Testing:
Changes have been reviewed to ensure no new errors were introduced.
Impact:
Improved readability of texts and documentation.
Additional Notes:
Regular text audits are recommended to maintain a high standard of quality.

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


Fix list_profiles command in docs

➜  ~ bay list-profiles
Usage: bay [OPTIONS] COMMAND [ARGS]...

Error: No such command "list-profiles", are you trying to run: list_profiles?

Looks like build has been failing since December, but not specs ran.
----------------------------------------------------------------------
Ran 0 tests in 0.000s

OK


Add the ability to extend Wasm for Envoy embedders.

Signed-off-by: John Plevyak<EMAIL_ADDRESS>

This is the companion to https://github.com/istio/proxy/pull/2916


Improve certificates

This PR consists of two commits.

The first commit updates the certificates.md file by replacing a reference to source code with a reference to the k8s API type.
The second commit normalize the markdown for the certificates.md page (long line wrapping + table beautify), no content is changed in this commit.

Thanks
This is almost a pure cleanup PR, so:
/lgtm
/approve


Change Authors / Contributors to Team.

How about we change that to just "Contributors"? I would reserve 'team' for the core contributors and people that have an ongoing involvement in the project.

Ok, "Contributors" is fine, too.


Sampling & switching scripts (for charmm systems) + zinc_id modification

Description

ZINC IDs were modified
Sampling and switching scripts for charmm systems (+test functions) added
add sampling script for charmmff systems

Todos

[x] modify zinc_systems list in constant.py (added zeros -> compatible with directory names)
[x] add sampling_charmff.py script
[x] add switching_charmff.py script
[x] add test function in test_neq.py for switching: test_switching_charmmff() and helper function load_system_and_samples_charmmff()
[x] add submit_sampling_charmmff.sh

Status

[ ] Ready to go

Codecov Report

Merging #8 (700b246) into main (48bda60) will not change coverage.
The diff coverage is n/a.


encfs: 1.9.2 -> 1.9.4

Semi-automatic update. These checks were performed:

built on NixOS
ran /nix/store/cfhzvmll3rjhgrhl4ypv40ap726dlgh2-encfs-1.9.4/bin/encfs --version and found version 1.9.4
ran /nix/store/cfhzvmll3rjhgrhl4ypv40ap726dlgh2-encfs-1.9.4/bin/encfsctl --version and found version 1.9.4
found 1.9.4 with grep in /nix/store/cfhzvmll3rjhgrhl4ypv40ap726dlgh2-encfs-1.9.4
found 1.9.4 in filename of file in /nix/store/cfhzvmll3rjhgrhl4ypv40ap726dlgh2-encfs-1.9.4

@GrahamcOfBorg build encfs

Failure on x86_64-darwin (full log)
Partial log (click to expand)
Package ‘encfs-1.9.4’ in /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/tools/filesystems/encfs/default.nix:29 is not supported on ‘x86_64-darwin’, refusing to evaluate.

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowBroken = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowBroken = true; }
to ~/.config/nixpkgs/config.nix.

Success on aarch64-linux (full log)
Partial log (click to expand)
shrinking /nix/store/3r1njg1mqsi50vdq5204prlqrj6220d6-encfs-1.9.4/bin/encfsctl
shrinking /nix/store/3r1njg1mqsi50vdq5204prlqrj6220d6-encfs-1.9.4/bin/encfs
shrinking /nix/store/3r1njg1mqsi50vdq5204prlqrj6220d6-encfs-1.9.4/lib/libencfs.so.1.9.4
gzipping man pages under /nix/store/3r1njg1mqsi50vdq5204prlqrj6220d6-encfs-1.9.4/share/man/
strip is /nix/store/lvx1acn1ig1j2km8jds5x3ggh3f2wa8v-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/3r1njg1mqsi50vdq5204prlqrj6220d6-encfs-1.9.4/lib  /nix/store/3r1njg1mqsi50vdq5204prlqrj6220d6-encfs-1.9.4/bin
patching script interpreter paths in /nix/store/3r1njg1mqsi50vdq5204prlqrj6220d6-encfs-1.9.4
/nix/store/3r1njg1mqsi50vdq5204prlqrj6220d6-encfs-1.9.4/bin/encfssh: interpreter directive changed from "/bin/sh" to "/nix/store/lnws52qanbi5vz8qg24v4ym57nznfndg-bash-4.4-p12/bin/sh"
checking for references to /build in /nix/store/3r1njg1mqsi50vdq5204prlqrj6220d6-encfs-1.9.4...
/nix/store/3r1njg1mqsi50vdq5204prlqrj6220d6-encfs-1.9.4

Success on x86_64-linux (full log)
Partial log (click to expand)
shrinking /nix/store/cfhzvmll3rjhgrhl4ypv40ap726dlgh2-encfs-1.9.4/bin/encfs
shrinking /nix/store/cfhzvmll3rjhgrhl4ypv40ap726dlgh2-encfs-1.9.4/bin/encfsctl
shrinking /nix/store/cfhzvmll3rjhgrhl4ypv40ap726dlgh2-encfs-1.9.4/lib/libencfs.so.1.9.4
gzipping man pages under /nix/store/cfhzvmll3rjhgrhl4ypv40ap726dlgh2-encfs-1.9.4/share/man/
strip is /nix/store/b0zlxla7dmy1iwc3g459rjznx59797xy-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/cfhzvmll3rjhgrhl4ypv40ap726dlgh2-encfs-1.9.4/lib  /nix/store/cfhzvmll3rjhgrhl4ypv40ap726dlgh2-encfs-1.9.4/bin
patching script interpreter paths in /nix/store/cfhzvmll3rjhgrhl4ypv40ap726dlgh2-encfs-1.9.4
/nix/store/cfhzvmll3rjhgrhl4ypv40ap726dlgh2-encfs-1.9.4/bin/encfssh: interpreter directive changed from "/bin/sh" to "/nix/store/zqh3l3lyw32q1ayb15bnvg9f24j5v2p0-bash-4.4-p12/bin/sh"
checking for references to /tmp/nix-build-encfs-1.9.4.drv-0 in /nix/store/cfhzvmll3rjhgrhl4ypv40ap726dlgh2-encfs-1.9.4...
/nix/store/cfhzvmll3rjhgrhl4ypv40ap726dlgh2-encfs-1.9.4


Working a Getting Started guide with IntelliJ IDEA (initial draft)

Resolved via https://github.com/spring-guides/gs-intellij-idea/commit/f56a02ea1e084a9e8d969a36d76d836451225692. Checkout https://spring.io/guides/gs/intellij-idea/. Thanks!


Change details about a subject

I miss a "Rediger emne detaljer" where the text that matches a certain subject can be changed.

Implemented in 3e5ea4ff8f684a46c422ca7bc37f0fb295268cc3


Support for AWS-delegated credentials for Elasticsearch

Is your feature request related to a problem? Please describe.
Currently, Elasticsearch configuration has an optional field to input username and password credentials for the Elasticsearch instance. For AWS-managed instances, not all customers create a master user and password.
This Feature Request is to delegate the credentials handoff using boto3 for python clients. There could be a flag when setting up the config useAWSCredentials: bool which would use credentials from the container (code sample below) and AuthN to the Elasticsearch instance.
Describe the solution you'd like
An alternate methodology to authN to the ES Instance from a service would be using the boto3 Session credentials stored by default in every container. The role delegated to the container determines if the credentials being used can access the service instead of using an explicit username and password.
Using this solution AWS4Auth library would enable customers to do hands-off delegation of credentials.
Describe alternatives you've considered
NA
Additional context
Relevant code context:
        from elasticsearch import Elasticsearch, RequestsHttpConnection

        credentials = boto3.Session().get_credentials()

        http_auth = AWS4Auth(region='us-east-1', service='es', refreshable_credentials=credentials)

        self.elasticsearch_client = Elasticsearch(
            [
                {"host": self.config.es_host, "port": self.config.es_port},
            ],
            http_auth=http_auth,
            scheme=self.config.scheme,
            use_ssl=self.config.use_ssl,
            verify_certs=self.config.verify_certs,
            ssl_context=ssl_context,
            ca_certs=self.config.ca_certs,
            connection_class=RequestsHttpConnection //Required for compatibility with AWS4Auth delegated creds
        )

Thanks for opening this, we'll be bringing that on 0.12. Thanks


undefined constant SQLite3::Database

I'm very new to Crystal, so apologies if this is silly question.
I'm getting the below error
Error in ./libs/kemalyst-model/adapter/sqlite.cr:6: undefined constant SQLite3::Database

  @pool : ConnectionPool(SQLite3::Database)
                         ^~~~~~~~~~~~~~~~~

shards.yml
dependencies:
  kemal:
    github: sdogruyol/kemal
    branch: master
  slang:
    github: jeromegn/slang
  kemalyst-model:
    github: drujensen/kemalyst-model
    branch: master
  sqlite3:
    github: crystal-lang/crystal-sqlite3
    branch: master

database.yml
sqlite:
  database: config/test.db

user.cr
require "kemalyst-model/adapter/sqlite"

class User < Kemalyst::Model
  adapter sqlite

  # table name is set to post_comments and timestamps are disabled.
  sql_mapping({
    first_name: ["VARCHAR(255)", String],
    surname: ["VARCHAR(255)", String],
    address: ["TEXT", String],
    postcode: ["VARCHAR(255)", String]
  })

end

Many thanks.

@crisward It looks like  manastech/crystal-sqlite3 has moved to crystal-lang/crystal-sqlite3 and has changed.  The good news is that they have created a common base class to build new drivers on so my adapters may not be necessary anymore.  When I get some free time, I will look into it.

Cool, Thanks...

Release 0.2.0 with new Crystal DB Drivers.


Tagging issues with files less than 200 pixel height. While tagging such images, the moment you release the mouse after drawing the rectangle and clicking the tag value the drawn rectangle suddenly becomes small.

Steps to Reproduce

Pick a less than 200 pixel height image
Draw the rectangle, release the mouse
Click on the tag to select
Suddenly the rectangle will shift upwards and become very small in height

Workaround

Pick a less than 200 pixel height image
Draw the rectangle, release the mouse
Resize the rectangle, any resizing motion will do, even if it comes back to the same size.
Click on the tag to select, Now the rectangle won't resize

Thanks for bringing this to our attention I will work to reproduce and let you know what I find.


Is it possible to edit the LyCORIS block weights and save?

With webui extension "SuperMerger" we can edit the block weights for a lora and then save as a new one. But the extension dose not support LyCORIS. Is there any way we can do that like lora?

You should submit this issue to SuperMerger but not here XD


Example binary runner

Suggestion of an example of consumer using the binary runner. Trying to reuse existing pre/post/examples.
The build fails because of this still forcing the job name, maybe you'll have a better idea than what I tried/implemented.

I was thinking of trying to find a way to capture automagically the job name from the input schema, and use that rather than repeating ourselves in the POST description.

I was thinking of trying to find a way to capture automagically the job name from the input schema, and use that rather than repeating ourselves in the POST description.

Sure why not. I can give you a hand if needed.


update all dependencies to @latest, drop node@8 CI testing, add node@16

Got paranoid after seeing this news  and decided to update all our dependencies to the latest version of things.
npm run build and npm test appear to work.
Edit: npm run lint  broke.  Fixing this involved switching to @eslint/babel-parser, and then eslint --fix and prettier -w to patch formatting differences.
... and apparently jest@latest doesn't work with node@8.    So I've updated our CI platforms to drop node 8 and add node 16.
Note: I don't know why Github is saying it needs the "ci (8.x) Expected" check here, since I removed node 8 from the node-versions in the ci.yml workflow. 😕

Okay, removed node 8 from checks.  Also renamed master to main.

@ctavan ping


Simplify the verification

We have now mostly migrated to pydantic. Time to use this and simplify the verification procedure that also contains the check_json or verify_experiment. This should make the code quite a bit more transparent.

We are currently doing:

check_json_dict
check_dimension
get_exp_input_dict

It gets especially complicated within check_json_dict

First we check that all the keys are fine.
then we raise check_experiment
then we raise check_instructions