Add GDPR Confirmation Modal

Partial PR for making the platform GDPR compliance. Takes User's confirmation after he sign in/sign up (the platform cannot be used without agreeing to the Privacy Policy)

Coverage decreased (-21.3%) to 66.078% when pulling 100cf065ee15c6ebebad22b54f19b045d14caeef on aayush-05:addGDPRConfirmation into b981bb0db5d0952018229cad5e29ed24db8a26d2 on CircuitVerse:master.

The design part looks fine to me


Jinvoo valve flash with tua convert [ jinvoo_SM-AW713 ]

Do have a
https://templates.blakadder.com/jinvoo_SM-AW713.html
Should be possible to flash..
Only how do i ghet this thing in he learning mode?

Having a template does not mean it is possible to flash. Hardware and software on these devices are constantly changing, so there is no guarantee it is possible to flash even if it was possible before.
Please refer to your vendor documentation for how to set your device into pairing mode.


[BEAM-10556] Add flag to ignore rawtype errors and add to modules

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[x] Update CHANGES.md with noteworthy changes.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

What's the reason to turn off the rawtype error checks?

All the modules where it is disabled have rawtype usage, so it must first be fixed before the error can be turned on.

I should have clarified: after this PR, the default is that rawtypes will break the build. So new modules will not have them. Existing modules have them.

 See a timeout (flake?) in Euphoria tests: https://scans.gradle.com/s/gx3exaswuez5w/tests/:sdks:java:extensions:euphoria:test/org.apache.beam.sdk.extensions.euphoria.core.translate.TimestampExtractTransformTest/testTransform#1

Run Java PreCommit

Run Java PreCommit

Run Java PreCommit

Run Portable_Python PreCommit

Run Python2_PVR_Flink PreCommit

Ah, finally green, except :sdks:java:io:azure was added and it has rawtype errors and needs disabling.

Run Java PreCommit

Noting that this is green and the conflicts in sdks/java/extensions/sketching/build.gradle are just two adjacent edits to the parameters to applyJavaNature. Since this PR is essentially trivial and has been held up by flakes and conflicts, I intend to merge it after resolving the conflict and confirming that compileJava compileTestJava succeeds.


Enhance Integrated Terminal with Split View and Independent Profiles

Introduce the ability to split the integrated terminal horizontally and vertically, allowing users to run multiple terminal sessions side by side. Each split should support independent profiles, themes, and environment variables, enabling developers to work with different configurations simultaneously.
This feature would improve productivity by reducing the need to switch between terminal tabs and external terminal applications.
Key Features:
Split View: Allow users to split the terminal horizontally and vertically.
Independent Profiles: Enable each split to run with different terminal profiles.
Customization: Allow users to customize themes and environment variables per split.
Seamless Navigation: Add keyboard shortcuts for easy navigation between splits.
Persistence: Save and restore terminal splits and configurations across VS Code sessions.
This enhancement would make the integrated terminal more powerful and versatile for developers working on multi-environment projects.

You can split in 1 dimension in the terminal panel, if you need more than this you can use terminal editors https://code.visualstudio.com/docs/terminal/basics#_terminals-in-editor-area


How to use custom layout instead of default crop layout ?

You can either leverage the usual Android resource override mechanism by creating a layout resource in your project called cropimage.xml or fork the library to customize it.


xq vs jaq

Hi, I am curious why xq is being developed when https://github.com/01mf02/jaq existed and was quite functional before xq was started.  Was it not known? Now it is known, what is the goal difference?
I have a few simple jq queries which work with both xq and jaq, and I am wondering which I should put into my CI scripts.  xq is smaller, yet sufficient, but the README tells me not to use xq. ;-(

I'm not the author of xq and not sure about the original motivation, but let me comment here as an author of gojq. The design of xq and jaq are quire different. While jaq redesign the execution model and fixing various behaviors for correctness and performance goal, xq is just a Rust implementation of jq and its execution model is very similar to the original (gojq is similar too). Based on the goal about correctness, jaq's behavior is quite different from the original even in simple query like jaq -n '.foo'. And even a basic and important filter like path/1 is missing in jaq. The assignment and update-assignment operators of jq are implemented based on this path/1 filter, but jaq updates the value without construction paths for higher performance. This difference sometimes causes jaq to panic; jaq -n '[[]] | .. = ..' overflows its stack. On the other hand, xq implements very similar execution model than jq, and xq passes more compatibility tests than jaq. The xq command implements path functions (path/1, getpath/1, setpath/2), so it is capable to include the pick/1 filter easily (introduced in 1.7); xq -n 'def pick(f): . as $v | reduce path(f) as $p (null; setpath($p; $v | getpath($p))); {x:1,y:2,z:3} | pick(.y)' works as expected (Needs much more Rust coding to implement this in jaq). However, I'm not blaming, this is just design difference. I'd like to express my respect to the jaq author, the paper Denotational Semantics and a Fast Interpreter for jq describes very well about the semantics about jq, and we can learn a lot from it.

Was it not known? Now it is known, what is the goal difference?

I was not aware of jaq until I found it later, but that wouldn't have affected the decision of making xq, because the motivation was to learn/understand jq as itchyny mentioned.

I have a few simple jq queries which work with both xq and jaq, and I am wondering which I should put into my CI scripts. xq is smaller, yet sufficient, but the README tells me not to use xq. ;-(

I'd personally recommend jq or gojq over xq as they're well-used and maintained better. If Rust tools is really preferred, I'd personally use xq because I know xq, and it can be replaced to jq/gojq later easily if something went wrong. Though I'm very biased. jaq is used/maintained more than xq, and it's perfectly reasonable to choose jaq.


Fix slow VoucherCode-Query in Frontend

Hello, i found an issue while searching for vouchers.
When you got a large number of vouchers (e.g. s_emarketing_voucher_codes  > 1GB) it takes a long time so serach for them (~2-5sec). Its much faster when you split up the join in two separated Sql-requests.

oh, someone else fixed this. ^^
https://github.com/shopware/shopware/pull/400/files
i just lost a holiday because of that query. again.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/#/issues/SW-15611.
Please use this issue to track the state of your pull request.

@wesolowski @jhofm merged with commit https://github.com/shopware/shopware/commit/03a06be77eecfb26eb9563310e5291b17f1a7a24.
Thanks for your contribution :+1:


UI: Support update prometheus config in enable monitoring page

UI for https://github.com/rancher/rancher/issues/17340

@loganhz could we change the Edit button to a Save button? When I click on the Edit button in my mind I am expecting a new page or modal to pop-up asking for what I want to change and then save. If we just make this a save button I think it will make more sense that we are already on the page for editing.

It will be addressed in https://github.com/rancher/ui/pull/2673

Version: Master 2.2 (2/19/20)
Confirmed issue. Refer to #17734 where this was tested.
The Edit button is now a normal Save button.


resume loading screen

I'm using your module and it is working well. Thanks...
When I make a request for some data and it returns a 401 I hide my loading spinner so I can show my login modal.
Once I've logged in I want to show the loading spinner again. Is there a broadcast or something that I can leverage off of that tells me it is now re running requests?
similar to authService.loginConfirmed?
any thoughts?
Thanks.

ok, I can get it working but thought I would see if you can recommend a better option.
I've added an additional callback function in your factory and included $rootScope
  retryAll: function(updater) {

    for (var i = 0; i < buffer.length; ++i) {
      retryHttpRequest(updater(buffer[i].config), buffer[i].deferred);
	  $rootScope.$broadcast('event:auth-retryAll');
    }
    buffer = [];
  }

		$scope.$on('event:auth-retryAll', function() {
			$ionicLoading.show({template: '<ion-spinner icon="ripple"/>'});
		});

When you use this module and you request for some data and it returns a 401, then that response in intercepted by the module and you won't catch the 401 response, so your spinner should keep spinning and stop only when you have authorized and the retry operation succeeded.
Bottom line is: do not make your app interrupting anything because of responses captured by this module. Just hide everything behind login panel and make this login panel disappears when it's done.

I thought about that but I'm using ionic and the spinner ionicloading I'm using is made to go on top and and clicks are disabled etc.
The spinner itself does keep working I'm canceling it manually to show the login modal.
Might re visit it and see what I can do.


Fix for "Chefspec template rendering fails when cookbook_name != directory name"

closes #5089
closes #5096

@chef/client-core ready for review


JS: Context sensitive exploratory flow

This changes exploratory flow to respect call/return matching (in practice), but still disregard store/load and flow labels. The call/return matching is slightly more overapproximate in the handling of captured variables than the full flow version, but for performance purposes that's fine.
The forward phase constantly keeps track of a "context", which is either

The most recently used parameter, meaning we may not use return steps, but may be used to construct summaries;
A function f, meaning we may use return steps out of any function nested within f (used to model captured variables);
A pseudo-node, meaning no call has been entered

We can freely step out of a return if the function we're returning from is more deeply nested than the context. For example:
outer(taint())
function outer(x) { // 'x' has depth 1
  function inner() {
    return x;      // returning from depth 2
  }
  return inner();  // reachable with 'x' as the most recently used parameter
}

When we step into a captured variable, the context becomes the enclosing function, allowing flow like:
function outer() {
  var shared;
  function a() {
    shared = taint();
  }
  function b() {
    return shared + 'blah';
  }
  function c() {
    sink(b()); // free to return out of b() because context is 'outer'
  }
}

The evaluation (sorted by tuples here) is a bit hard to read. I'm a little uncertain about whether it's worth landing this.

The evaluation (sorted by tuples here) is a bit hard to read. I'm a little uncertain about whether it's worth landing this.

I think it looks worth landing.
It definitely deserves another evaluation after https://github.com/Semmle/ql/pull/3095 has landed.
And it looks like bwip-js was an outlier in your evaluation, as it has been in my evaluations.
bwip-js  1128  1177    0.977828  2.41258  1.04344.
It is getting hard to distinguish between the exploratory and non-exploratory data-flow in Configuration.qll (e.g. capturedVariableScope, which is only used for exploratory flow, but that is not very clear).
I'm in favor of some sort of refactorization (into separate modules? or files?) such that there is a more clear separation between exploratory and non-exploratory data-flow.
(And I would suggest not doing that until after https://github.com/Semmle/ql/pull/3095 has landed).

I'm in favor of some sort of refactorization (into separate modules? or files?) such that there is a more clear separation between exploratory and non-exploratory data-flow.

That sounds like a good idea, there's a lot of stuff in Configuration.qll, but let's not do it in this PR.

Right, the combined evaluation finished a few days ago.

Right, the combined evaluation finished a few days ago.

Its like wallclock, dpm and tuples tell 3 different stories.
But the on the wallclock the new version spent about 4 minutes longer than the old version.
I think we could use an evaluation with more benchmarks.

Updated evaluation looks about the same

Updated evaluation looks about the same

The wallclock and tuple count are slightly lower on average in that one, only dpm is showing a slowdown. So I think it looks slightly better.
I think we can land this if you do a big-apps evaluation that doesn't show a regression.
But I'm not sure enough about it to press the button myself until someone else looks at this.

I agree about the need for more evaluations; I wasn't feeling overly confident about this either. I'll run more evaluations and investigate the benchmarks that came back slower.

Big-apps evaluation doesn't look all that convincing. (And TypeScript scores a whopping 5.0 DPM 🤣)

Closing the PR for now as the performance isn't quite what I was hoping for. I might revisit it after some other performance improvements have landed.


Add a "Save Link" button that will provide a resusable link to the current example

Should be doable with slightly more robust URL encoding options (#80) ...

closed in #86


dokku config:set but without restart (in a pre-release hook)

Hi,
I need to be able to call dokku config:set but without restarting the application, because I'm already in the process of restarting the application. I'm in a pre-release hook, see:
https://github.com/vincentfretin/dokku-elasticsearch-plugin/blob/master/pre-release#L37
How can we modify the config:set code to handle this case?
For now the application restarts twice if the elasticsearch container is not started.

@josegonzalez thoughts on this?

Yeah, that seems like a reasonable request. Not sure on the best implementation, as it would need to work over ssh as well. Also, afaik no normal heroku plugin has the same requirements - when linked, a plugin sets configuration, it doesn't mount a container.

Should we just decouple the restart from config:set? That's probably reasonable since we now have a simple restart command.

Yes please. I think the current behavior of restarting is not desirable. Especially since the "auto-restart" only does a redeploy and not a rebuild. It'd be nice to have a rebuild command.

Well config:set follows what heroku does - redeploy after configuration change. I'm still not sure as to why  the pre-release hook for that mongo plugin is setting configuration - shouldn't that go in a link step or something?

I suppose you could just interface with $DOKKU_ROOT/$APP/ENV directly but that kind of breaks the api boundary...
From https://devcenter.heroku.com/articles/config-vars: "Whenever you set or remove a config var, your app will be restarted."
Given that we're complying with Heroku's API, I don't think we should change config:set. Perhaps we can provide a "protected" function that does config setting/unsetting without restart that is only exposed via pluginhook?

Why is that mongo plugin running config:set during a deploy?

You mean elasticsearch, right? I don't see mention of mongo here.

Sorry yeah elasticsearch

That's a good point though. This could be done with a container link instead and bypass the entire config:set api. The deployed app would just use the standard docker env vars for linked containers.

This seems to be the agreed upon plugin to do this sort of thing. https://github.com/rlaneve/dokku-link

Well we do have the plugin-spec issue, as well as a PR for plugins as addons...

Closing as there seems nothing to be done for this particular issue at this time. Feel free to reopen if there's anything else to discuss here.

I am requesting that this issue be re-opened. At least a flag or option on config:set that does not redeploy is needed. I am already using Dokku Link, however config:set is needed to provide applications with a quick method of connecting to the plugin (URL format instead of host and port).

@sekjun9878 would you mind submitting a PR for this?

I have something in a dev branch. Let me clean it up and submit the PR for review.

@vincentfretin this has been fixed and documented in the plugin development section of the docs. Let us know if it works for you!

:D thanks!

@josegonzalez Thanks, I'll try it.


systemtests - olm csv links

Type of change

Systemtest

Description

Checklist

[ ] Update/write design documentation in ./documentation/design
[x] Write tests and make sure they pass
[ ] Update documentation
[ ] Check RBAC rights for Kubernetes / OpenShift roles
[x] Try your changes from Pod inside your Kubernetes and OpenShift cluster, not just locally
[ ] Reference relevant issue(s) and close them after merging
[ ] Update CHANGELOG.md

@enmasse-ci run tests profile=olm testcase=OLMLinksTest olm

:heavy_check_mark: Test Summary :heavy_check_mark:
TEST_PROFILE: olm
TEST_CASE: OLMLinksTest
TOTAL: 8
PASS: 8
FAIL: 0
SKIP: 0
BUILD_NUMBER: 300


I get blocked from Worx

Hello!
I wonder if there is a way to slow down the worx adapter? Currently it is updating every 3 second and Worx has blocked me several times due to this. They say I spam them with requests.

Here is part of my log:
worx.0 2019-08-23 22:28:44.099 warn Connection blocked from Worx, please try again in 24h
worx.0 2019-08-23 22:28:41.861 info starting. Version 0.4.0 in C:/WINDOWS/system32/node_modules/iobroker.worx, node: v10.16.0
host.DESKTOP-BF47MV5 2019-08-23 22:28:40.184 info instance system.adapter.worx.0 started with pid 49688
host.DESKTOP-BF47MV5 2019-08-23 22:28:10.177 info Restart adapter system.adapter.worx.0 because enabled
host.DESKTOP-BF47MV5 2019-08-23 22:28:10.177 error instance system.adapter.worx.0 terminated with code 0 (OK)
Caught 2019-08-23 22:28:10.177 error by controller[0]: at process._tickCallback (internal/process/next_tick.js:63:19)
Caught 2019-08-23 22:28:10.177 error by controller[0]: at emitErrorAndCloseNT (internal/streams/destroy.js:59:3)
Caught 2019-08-23 22:28:10.177 error by controller[0]: at emitErrorNT (internal/streams/destroy.js:91:8)
Caught 2019-08-23 22:28:10.177 error by controller[0]: at TLSSocket.emit (events.js:203:15)
Caught 2019-08-23 22:28:10.177 error by controller[0]: at TLSSocket.handleTLSerrors (C:\WINDOWS\system32\node_modules\iobroker.worx\node_modules\mqtt\lib\connect\tls.js:26:18)
Caught 2019-08-23 22:28:10.177 error by controller[0]: at MqttClient.emit (events.js:198:13)
Caught 2019-08-23 22:28:10.177 error by controller[0]: at MqttClient. (C:\WINDOWS\system32\node_modules\iobroker.worx\lib\api.js:259:13)
Caught 2019-08-23 22:28:10.176 error by controller[0]: ReferenceError: reject is not defined
worx.0 2019-08-23 22:28:10.153 info cleaned everything up...
worx.0 2019-08-23 22:28:10.153 error at process._tickCallback (internal/process/next_tick.js:63:19)
worx.0 2019-08-23 22:28:10.153 error at emitErrorAndCloseNT (internal/streams/destroy.js:59:3)
worx.0 2019-08-23 22:28:10.153 error at emitErrorNT (internal/streams/destroy.js:91:8)
worx.0 2019-08-23 22:28:10.153 error at TLSSocket.emit (events.js:203:15)
worx.0 2019-08-23 22:28:10.153 error at TLSSocket.handleTLSerrors (C:\WINDOWS\system32\node_modules\iobroker.worx\node_modules\mqtt\lib\connect\tls.js:26:18)
worx.0 2019-08-23 22:28:10.153 error at MqttClient.emit (events.js:198:13)
worx.0 2019-08-23 22:28:10.153 error at MqttClient. (C:\WINDOWS\system32\node_modules\iobroker.worx\lib\api.js:259:13)
worx.0 2019-08-23 22:28:10.153 error ReferenceError: reject is not defined
worx.0 2019-08-23 22:28:10.152 error uncaught exception: reject is not defined

please stop adapter for 24 hours. install the version from git, activate debug log and post the log again

Thanks!
Is the 0.4.0 the last version of the adapter?
Regards Roger
Fra: MeisterTR<EMAIL_ADDRESS>Sendt: lørdag 24. august 2019 20.03
Til: MeisterTR/ioBroker.worx<EMAIL_ADDRESS>Kopi: rogerbl<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Emne: Re: [MeisterTR/ioBroker.worx] I get blocked from Worx (#21)
please stop adapter for 24 hours. install the version from git, activate debug log and post the log again
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/MeisterTR/ioBroker.worx/issues/21?email_source=notifications&email_token=ACTFYABCBYRLOZAL7RDEAC3QGFZWFA5CNFSM4IPGLW4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5CE2XI#issuecomment-524569949, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ACTFYABZS55FN7YJWUQX4PTQGFZWFANCNFSM4IPGLW4A.

Yes but you have to install the version from git because I have changed logging and fixed an error

Ok, thanks!
Regards Roger
Fra: MeisterTR<EMAIL_ADDRESS>Sendt: lørdag 24. august 2019 20.54
Til: MeisterTR/ioBroker.worx<EMAIL_ADDRESS>Kopi: rogerbl<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Emne: Re: [MeisterTR/ioBroker.worx] I get blocked from Worx (#21)
Yes but you have to install the version from git because I have changed logging and fixed an error
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/MeisterTR/ioBroker.worx/issues/21?email_source=notifications&email_token=ACTFYABJHW2BC626JLHUQ23QGF7VRA5CNFSM4IPGLW4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5CFTEA#issuecomment-524573072, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ACTFYAFKEMFNEMFTLA53NSDQGF7VRANCNFSM4IPGLW4A.

Perfect!
Now it works fine! What a good job you have done!
I have been connected since I updated the adapter last night!
Thanks a lot! 😊
Regards Roger
Fra: MeisterTR<EMAIL_ADDRESS>Sendt: lørdag 24. august 2019 20.54
Til: MeisterTR/ioBroker.worx<EMAIL_ADDRESS>Kopi: rogerbl<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Emne: Re: [MeisterTR/ioBroker.worx] I get blocked from Worx (#21)
Yes but you have to install the version from git because I have changed logging and fixed an error
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/MeisterTR/ioBroker.worx/issues/21?email_source=notifications&email_token=ACTFYABJHW2BC626JLHUQ23QGF7VRA5CNFSM4IPGLW4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5CFTEA#issuecomment-524573072, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ACTFYAFKEMFNEMFTLA53NSDQGF7VRANCNFSM4IPGLW4A.

Hi again!
I was again blocked …
Regards Roger
Fra: MeisterTR<EMAIL_ADDRESS>Sendt: lørdag 24. august 2019 20.03
Til: MeisterTR/ioBroker.worx<EMAIL_ADDRESS>Kopi: rogerbl<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Emne: Re: [MeisterTR/ioBroker.worx] I get blocked from Worx (#21)
please stop adapter for 24 hours. install the version from git, activate debug log and post the log again
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/MeisterTR/ioBroker.worx/issues/21?email_source=notifications&email_token=ACTFYABCBYRLOZAL7RDEAC3QGFZWFA5CNFSM4IPGLW4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5CE2XI#issuecomment-524569949, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ACTFYABZS55FN7YJWUQX4PTQGFZWFANCNFSM4IPGLW4A.

Hi again!
I don’t see any error messages connected to your adapter, but worx blocked me anyway.
Will try to disable other adapters to see if they are influencing the worx adapter.
Specially the mqtt client adapter which I use.
I’ll let you know what I find out.
Regards Roger
Fra: MeisterTR<EMAIL_ADDRESS>Sendt: lørdag 24. august 2019 20.54
Til: MeisterTR/ioBroker.worx<EMAIL_ADDRESS>Kopi: rogerbl<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Emne: Re: [MeisterTR/ioBroker.worx] I get blocked from Worx (#21)
Yes but you have to install the version from git because I have changed logging and fixed an error
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/MeisterTR/ioBroker.worx/issues/21?email_source=notifications&email_token=ACTFYABJHW2BC626JLHUQ23QGF7VRA5CNFSM4IPGLW4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5CFTEA#issuecomment-524573072, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ACTFYAFKEMFNEMFTLA53NSDQGF7VRANCNFSM4IPGLW4A.

Hi!
After I moved the iobroker installation to a raspberry pi, worx don’t block
me anymore.
The other iobroker was on a win 10 pc.
Thanks for your good work! 😊
Regards Roger
Fra: MeisterTR<EMAIL_ADDRESS>Sendt: lørdag 24. august 2019 20.54
Til: MeisterTR/ioBroker.worx<EMAIL_ADDRESS>Kopi: rogerbl<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Emne: Re: [MeisterTR/ioBroker.worx] I get blocked from Worx (#21)
Yes but you have to install the version from git because I have changed logging and fixed an error
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/MeisterTR/ioBroker.worx/issues/21?email_source=notifications&email_token=ACTFYABJHW2BC626JLHUQ23QGF7VRA5CNFSM4IPGLW4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5CFTEA#issuecomment-524573072, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ACTFYAFKEMFNEMFTLA53NSDQGF7VRANCNFSM4IPGLW4A.

Hi!
when the edgecut command is issued on a mower without edgecut option, the adapter spam worx server and it blocks the connection for 24h.
My mower is a S300 without edgecut option.
I tested it on both my iobroker installations.
So win10 was not the problem. 😊 Good to know!
Regards Roger
Fra: MeisterTR<EMAIL_ADDRESS>Sendt: lørdag 24. august 2019 20.03
Til: MeisterTR/ioBroker.worx<EMAIL_ADDRESS>Kopi: rogerbl<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Emne: Re: [MeisterTR/ioBroker.worx] I get blocked from Worx (#21)
please stop adapter for 24 hours. install the version from git, activate debug log and post the log again
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/MeisterTR/ioBroker.worx/issues/21?email_source=notifications&email_token=ACTFYABCBYRLOZAL7RDEAC3QGFZWFA5CNFSM4IPGLW4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5CE2XI#issuecomment-524569949, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ACTFYABZS55FN7YJWUQX4PTQGFZWFANCNFSM4IPGLW4A.

interesting is a debug log. normaly you shouln,t blocked from worx.  So i thing something is wrong

Yes, I understand , but there is not much to see in the degbug log.
Sendt fra E-posthttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10

Fra: MeisterTR<EMAIL_ADDRESS>Sendt: Thursday, August 29, 2019 7:42:33 PM
Til: MeisterTR/ioBroker.worx<EMAIL_ADDRESS>Kopi: rogerbl<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Emne: Re: [MeisterTR/ioBroker.worx] I get blocked from Worx (#21)
interesting is a debug log. normaly you shouln,t blocked from worx. So i thing something is wrong
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/MeisterTR/ioBroker.worx/issues/21?email_source=notifications&email_token=ACTFYAAVBJBR5BMZ3VNR53LQHADATA5CNFSM4IPGLW4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5PITNA#issuecomment-526289332, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ACTFYACOLNNETDUWI7P4NHLQHADATANCNFSM4IPGLW4A.


Enforce proper dependencies in sbeppc_make_schema_target

It's worth mentioning that the file listed in OUTPUT of a custom command should be included by every other generated file. Otherwise, the situation is possible that consumer target does not include it directly or indirectly and recompilation won't be triggered when schema changes. When SCHEMA_NAME is known, such file is OUTPUT_DIR/SCHEMA_NAME/schema/schema.hpp because it's included by all other generated headers. Otherwise, a fake anchor file is created and injected into schema.hpp using new --inject-include CLI option.


[BBC] Error: JSON object must be str, bytes or bytearray, not dict

Checklist

[X] I'm reporting a broken site
[X] I've verified that I'm running yt-dlp version 2022.06.29 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
anywhere
Provide a description that is worded well enough to be understood
ERROR: the JSON object must be str, bytes or bytearray, not dict
Guess the structure of the response changed or something.
Video is located at this link:
https://vod-dash-ww-live.akamaized.net/usp/auth/vod/piff_abr_full_sd/0b8189-p065hw19/vf_p065hw19_48490131-03e6-4f47-8a06-6e90f10bfe2c.ism/pc_hd_abr_v2_dash_master.mpd?__gda__=1657378563_1d46032da1fe3a0a011f341958d191b5
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
./yt-dlp -vU -F "https://www.bbc.co.uk/teach/school-radio/assemblies-the-good-samaritan-modern-setting-ks2/zjsx2v4"
[debug] Command-line config: ['-vU', '-F', 'https://www.bbc.co.uk/teach/school-radio/assemblies-the-good-samaritan-modern-setting-ks2/zjsx2v4']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2022.06.29 [9d339c41e] (zip)
[debug] Plugins: ['SamplePluginIE', 'SamplePluginPP']
[debug] Python 3.9.13 (CPython 64bit) - macOS-12.4-arm64-arm-64bit
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: avconv -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] Checking exe version: avprobe -bsfs
[debug] exe versions: none
[debug] Optional libraries: sqlite3-2.6.0
[debug] Proxy map: {}
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version: 2022.06.29, Current version: 2022.06.29
yt-dlp is up to date (2022.06.29)
[debug] [bbc] Extracting URL: https://www.bbc.co.uk/teach/school-radio/assemblies-the-good-samaritan-modern-setting-ks2/zjsx2v4
[bbc] zjsx2v4: Downloading webpage
ERROR: the JSON object must be str, bytes or bytearray, not dict
Traceback (most recent call last):
  File "./yt-dlp/yt_dlp/YoutubeDL.py", line 1425, in wrapper
    return func(self, *args, **kwargs)
  File "./yt-dlp/yt_dlp/YoutubeDL.py", line 1495, in __extract_info
    ie_result = ie.extract(url)
  File "./yt-dlp/yt_dlp/extractor/common.py", line 640, in extract
    ie_result = self._real_extract(url)
  File "./yt-dlp/yt_dlp/extractor/bbc.py", line 1189, in _real_extract
    initial_data = self._parse_json(initial_data, playlist_id, fatal=False)
  File "./yt-dlp/yt_dlp/extractor/common.py", line 942, in _parse_json
    return json.loads(
  File<EMAIL_ADDRESS>line 339, in loads
    raise TypeError(f'the JSON object must be str, bytes or bytearray, '
TypeError: the JSON object must be str, bytes or bytearray, not dict

The extractor fails when looking for an assignment to __INITIAL_DATA__ but these pages have the Morph structure, where the needed hydration data is the second parameter of one of several calls to Morph.setPayload(). In this case, the first parameter is /data/bbc-morph-blocks-data/preview/false/projectName/blocks-school-radio/timestamp/0/version/2.95.0/zid/zm3ptrd where zm3ptrd is the id of the <div> that contains the video. The second is this (prettified):
{
  'meta': {
    'responseCode': 200,
    'errorMessage': null,
    'headers': {
      'content-type': 'application/json'
    }
  },
  'body': {
    'type': 'video-block',
    'id': 'zm3ptrd',
    'title': '',
    'caption': '',
    'pid': 'p065hw17',
    'transcript': '',
    'video': {
      'title': 'The Good Samaritan',
      'holdingImage': 'https://ichef.bbci.co.uk/images/ic/$recipe/p065hxv2.jpg',
      'duration': 'PT5M22S',
      'vpid': 'p065hw19',
      'guidance': null,
      'mediaType': 'video',
      'aspectRatio': '16:9'
    }
  }
});
}

And then the PID will lead you to the desired content from  https://www.bbc.co.uk/programmes/p065hw17.

In https://github.com/ytdl-org/youtube-dl/pull/28577, the extractor looked for just asset-data, page-component-data, asset-with-media or guide-data in the first parameter. Probably we should parse all the payloads and extract media from any that have known structures. In this case we have payload['body']['type'] == 'video-block which may be a hint, but unfortunately also produces other matches (p08r2q32, p07q3hln, p09cydh5). OTOH the page does appear to have 4 video blocks.
One issue that stymied the PR was the deficient semantics of --no/yes-playlist, whereby it was impossible to make the extractor target the main video by default, with --yes-playlist targeting all the page videos as a playlist, because ydl.params['no_playlist'] is truthy/falsy and the extractor can't tell whether the user said --yes-playlist or just didn't say --no-playlist. It would be easy to  the semantics of --no/yes-playlist like this, with a three valued logic for ydl.params['no_playlist']:

no --x-playlist, or --yes-playlist ... --no-playlist: ydl.params['no_playlist'] is None
--yes-playlist: ydl.params['no_playlist'] is False
--no-playlist otherwise: ydl.params['no_playlist'] is True

There was a concern that people may be testing if ydl.params['no_playlist'] is not None: instead of just if ydl.params['no_playlist']: and would get the wrong value in case 3. I would say they might deserve to.

Here's another test link with the same error message:
https://www.bbc.com/reel/embed/p08bl6tl
Log:
./yt-dlp -vU -F "https://www.bbc.com/reel/embed/p08bl6tl"
[debug] Command-line config: ['-vU', '-F', 'https://www.bbc.com/reel/embed/p08bl6tl']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2022.08.14 [55937202b] (zip)
[debug] Plugins: ['SamplePluginIE', 'SamplePluginPP']
[debug] Python 3.9.13 (CPython 64bit) - macOS-12.5-arm64-arm-64bit
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] exe versions: none
[debug] Optional libraries: sqlite3-2.6.0
[debug] Proxy map: {}
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version: 2022.08.14, Current version: 2022.08.14
yt-dlp is up to date (2022.08.14)
[debug] [bbc] Extracting URL: https://www.bbc.com/reel/embed/p08bl6tl
[bbc] p08bl6tl: Downloading webpage
ERROR: the JSON object must be str, bytes or bytearray, not dict
Traceback (most recent call last):
File "./yt-dlp/yt_dlp/YoutubeDL.py", line 1441, in wrapper
return func(self, *args, **kwargs)
File "./yt-dlp/yt_dlp/YoutubeDL.py", line 1517, in __extract_info
ie_result = ie.extract(url)
File "./yt-dlp/yt_dlp/extractor/common.py", line 666, in extract
ie_result = self._real_extract(url)
File "./yt-dlp/yt_dlp/extractor/bbc.py", line 1190, in _real_extract
initial_data = self._parse_json(initial_data, playlist_id, fatal=False)
File "./yt-dlp/yt_dlp/extractor/common.py", line 970, in _parse_json
return json.loads(
File<EMAIL_ADDRESS>line 339, in loads
raise TypeError(f'the JSON object must be str, bytes or bytearray, '
TypeError: the JSON object must be str, bytes or bytearray, not dict

With this fix https://github.com/ajj8/yt-dlp/commit/04b3418f2633ef76352a621765b743fff6d1b243 the error is: ERROR: [bbc] p08bl6tl: Unable to extract playlist data
Log below:
./yt-dlp -vU -F "https://www.bbc.com/reel/embed/p08bl6tl"
[debug] Command-line config: ['-vU', '-F', 'https://www.bbc.com/reel/embed/p08bl6tl']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2022.08.14 [55937202b] (zip)
[debug] Plugins: ['SamplePluginIE', 'SamplePluginPP']
[debug] Python 3.9.13 (CPython 64bit) - macOS-12.5-arm64-arm-64bit
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] exe versions: none
[debug] Optional libraries: sqlite3-2.6.0
[debug] Proxy map: {}
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version: 2022.08.14, Current version: 2022.08.14
yt-dlp is up to date (2022.08.14)
[debug] [bbc] Extracting URL: https://www.bbc.com/reel/embed/p08bl6tl
[bbc] p08bl6tl: Downloading webpage
ERROR: [bbc] p08bl6tl: Unable to extract playlist data; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
File "./yt-dlp/yt_dlp/extractor/common.py", line 666, in extract
ie_result = self._real_extract(url)
File "./yt-dlp/yt_dlp/extractor/bbc.py", line 1289, in _real_extract
self._search_regex(
File "./yt-dlp/yt_dlp/extractor/common.py", line 1218, in _search_regex
raise RegexNotFoundError('Unable to extract %s' % _name)

Hi, rather than opening a new ticket - I'm going to post my error into this topic as it's almost identical to the previous posts, just a different part of the BBC website. I'm running the most recent version of yt-dlp through yt-dlg.
url = "https://www.bbc.co.uk/newsround/62587298"
LOG:
2022-08-19 08:22:40,568-DEBUG-Worker_1-[debug] Command-line config: ['--newline', '-i', '-o', '#####\%(title)s.%(ext)s', '-f', 'mp4', '-v', '--ignore-config', '--hls-prefer-native', 'https://www.bbc.co.uk/newsround/62587298']
2022-08-19 08:22:40,568-DEBUG-Worker_1-[debug] Encodings: locale cp1252, fs utf-8, pref cp1252, out cp1252 (No ANSI), error cp1252 (No ANSI), screen cp1252 (No ANSI)
2022-08-19 08:22:40,568-DEBUG-Worker_1-[debug] yt-dlp version 2022.08.19 [48c88e0] (win32_exe)
2022-08-19 08:22:40,568-DEBUG-Worker_1-[debug] Python 3.8.10 (CPython 64bit) - Windows-10-10.0.22000-SP0
2022-08-19 08:22:40,568-DEBUG-Worker_1-[debug] Checking exe version: ffmpeg -bsfs
2022-08-19 08:22:40,568-DEBUG-Worker_1-[debug] Checking exe version: ffprobe -bsfs
2022-08-19 08:22:40,568-DEBUG-Worker_1-[debug] exe versions: ffmpeg 2022-08-03-git-d3f48e68b3-essentials_build-www.gyan.dev (setts), ffprobe 2022-08-03-git-d3f48e68b3-essentials_build-www.gyan.dev
2022-08-19 08:22:40,568-DEBUG-Worker_1-[debug] Optional libraries: Cryptodome-3.15.0, brotli-1.0.9, certifi-2022.06.15, mutagen-1.45.1, sqlite3-2.6.0, websockets-10.3
2022-08-19 08:22:40,568-DEBUG-Worker_1-[debug] Proxy map: {}
2022-08-19 08:22:40,568-DEBUG-Worker_1-[debug] [bbc] Extracting URL: https://www.bbc.co.uk/newsround/62587298
2022-08-19 08:22:40,568-DEBUG-Worker_1-ERROR: the JSON object must be str, bytes or bytearray, not dict
2022-08-19 08:22:40,568-DEBUG-Worker_1-Traceback (most recent call last):
2022-08-19 08:22:40,568-DEBUG-Worker_1-  File "yt_dlp\YoutubeDL.py", line 1442, in wrapper
2022-08-19 08:22:40,568-DEBUG-Worker_1-  File "yt_dlp\YoutubeDL.py", line 1518, in __extract_info
2022-08-19 08:22:40,568-DEBUG-Worker_1-  File "yt_dlp\extractor\common.py", line 666, in extract
2022-08-19 08:22:40,568-DEBUG-Worker_1-  File "yt_dlp\extractor\bbc.py", line 1190, in _real_extract
2022-08-19 08:22:40,568-DEBUG-Worker_1-  File "yt_dlp\extractor\common.py", line 970, in parse_json
2022-08-19 08:22:40,568-DEBUG-Worker_1-  File "json_init.py", line 341, in loads
2022-08-19 08:22:40,568-DEBUG-Worker_1-TypeError: the JSON object must be str, bytes or bytearray, not dict
2022-08-19 08:22:40,568-DEBUG-Worker_1-Child process exited with non-zero code: 1

Is @ajj8 fix working or still have that issue? How can I test his code?

https://www.bbc.com/reel/embed/p08bl6tl: as with previous Reel pages, this has hydration data in the data-json attribute of the script element with id="initial-data", but the video data is in the root of the .initData member instead of in a list .initData.items.
https://www.bbc.co.uk/newsround/62587298: this is a Morph page but, instead of a vpid in the .body.video member of one of the payloads, it has a pid in the .body.media member of one of the payloads, but no vpid, so it has to be passed through url_result() as http://www.bbc.co.uk/programmes/{pid}.
I've added the fixes for these and the corresponding tests to my refactored BBC extractor that will be coming to a yt-dl PR soon (including code taken from ajj8's PR here, as well as outstanding PRs from yt-dl). It should be a drop-in to yt-dlp apart from time machine cosmetics.
My solution to the Morph playlist vs video problem is this: when the page has a playlist (not like https://www.bbc.co.uk/newsround/62587298),

if a video is found for which /{pid}/ is in the URL, return that video (eg, https://www.bbc.com/reel/video/p07c6sb6/how-positive-thinking-is-harming-your-happiness);
otherwise, return the playlist (eg https://www.bbc.com/reel/playlist/rethink).

One more repro for the heap:
$ yt-dlp -vv 'https://www.bbc.com/news/world-us-canada-49667778'
[debug] Command-line config: ['-vv', 'https://www.bbc.com/news/world-us-canada-49667778']
[debug] User config "/home/ryan/.config/yt-dlp.conf": ['--rate-limit', '2000K', '--output', '%(title).200B%(title.201B&…|)s.%(ext)s']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [24f827875]
[debug] Lazy loading extractors is disabled
[debug] Python 3.11.6 (CPython x86_64 64bit) - Linux-6.6.7-zen1-1-zen-x86_64-with-glibc2.38 (OpenSSL 3.2.0 23 Nov 2023, glibc 2.38)
[debug] exe versions: ffmpeg 6.1 (setts), ffprobe 6.1, rtmpdump 2.4
[debug] Optional libraries: Cryptodome-3.19.0, brotli-1.1.0, certifi-2023.11.17, mutagen-1.47.0, requests-2.31.0, sqlite3-3.44.2, urllib3-2.1.0, websockets-12.0
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests, websockets
[debug] Loaded 1797 extractors
[bbc] Extracting URL: https://www.bbc.com/news/world-us-canada-49667778
[bbc] world-us-canada-49667778: Downloading webpage
ERROR: [bbc] world-us-canada-49667778: Unable to extract playlist data; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
  File "/home/ryan/src/other/yt-dlp/.venv/lib/python3.11/site-packages/yt_dlp/extractor/common.py", line 718, in extract
    ie_result = self._real_extract(url)
                ^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/ryan/src/other/yt-dlp/.venv/lib/python3.11/site-packages/yt_dlp/extractor/bbc.py", line 1291, in _real_extract
    self._search_regex(
  File "/home/ryan/src/other/yt-dlp/.venv/lib/python3.11/site-packages/yt_dlp/extractor/common.py", line 1266, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
$

OP's issue was partially addressed by 19741ab8a401ec64d5e84fdbfcfb141d105e7bc8

@rpdelaney posted:

[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [24f827875]
[debug] Lazy loading extractors is disabled
[debug] Python 3.11.6 (CPython x86_64 64bit) - Linux-6.6.7-zen1-1-zen-x86_64-with-glibc2.38 (OpenSSL 3.2.0 23 >Nov 2023, glibc 2.38)

... My OS (Vista SP2 x86) is no longer officially supported by yt-dlp, but using the 32-bit binary (py3.7-based) of the 2023.11.16 stable release (like you, basically :wink: ), I get different, successful, results with the very same BBC URL:

yt-dlp_x86 -vF "https://www.bbc.com/news/world-us-canada-49667778"
[debug] Command-line config: ['-vF', 'https://www.bbc.com/news/world-us-canada-49667778']
Deprecated Feature: Support for Windows Vista/Server 2008 has been deprecated. See  https://github.com/yt-dlp/yt-dlp/issues/7803  for details.
                    You may stop receiving updates on this version at any time! Please update to Python 3.8 or above
[debug] Encodings: locale cp1253, fs utf-8, pref cp1253, out utf-8 (No VT), error utf-8 (No VT), screen utf-8 (No VT)
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [24f827875] (win_x86_exe)
[debug] Python 3.7.9 (CPython x86 32bit) - Windows-Vista-6.0.6003-SP2 (OpenSSL 1.1.1g  21 Apr 2020)
[debug] exe versions: none
[debug] Optional libraries: Cryptodome-3.19.0, brotli-1.1.0, certifi-2023.07.22, mutagen-1.47.0, requests-2.31.0, sqlite3-3.31.1, urllib3-2.0.7, websockets-11.0.3
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests
[debug] Loaded 1901 extractors
[bbc] Extracting URL: https://www.bbc.com/news/world-us-canada-49667778
[bbc] world-us-canada-49667778: Downloading webpage
[bbc] p07mf88w: Downloading media selection JSON
[bbc] p07mf88w: Downloading m3u8 information
[bbc] p07mf88w: Downloading m3u8 information
[bbc] p07mf88w: Downloading m3u8 information
[bbc] p07mf88w: Downloading m3u8 information
[bbc] p07mf88w: Downloading MPD manifest
[bbc] p07mf88w: Downloading MPD manifest
[bbc] p07mf88w: Downloading MPD manifest
[bbc] p07mf88w: Downloading MPD manifest
[download] Downloading playlist: Aaron Persky: Brock Turner judge fired from coaching girls
[bbc] Playlist Aaron Persky: Brock Turner judge fired from coaching girls: Downloading 1 items of 1
[download] Downloading item 1 of 1
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), channels, acodec, size, br, asr, proto, vext, aext, hasaud, source, id
[info] Available formats for p07mf88w:
ID                              EXT RESOLUTION FPS |   TBR PROTO | VCODEC VBR ACODEC     ABR ASR MORE INFO
---------------------------------------------------------------------------------------------------------------------------------
mf_akamai-audio_eng=96000-0     m4a audio only     |   96k dash  | audio only     mp4a.40.5  96k 48k [en] DASH audio, m4a_dash
mf_akamai-audio_eng=96000-1     m4a audio only     |   96k dash  | audio only     mp4a.40.5  96k 48k [en] DASH audio, m4a_dash
mf_cloudfront-audio_eng=96000-0 m4a audio only     |   96k dash  | audio only     mp4a.40.5  96k 48k [en] DASH audio, m4a_dash
mf_cloudfront-audio_eng=96000-1 m4a audio only     |   96k dash  | audio only     mp4a.40.5  96k 48k [en] DASH audio, m4a_dash
mf_akamai-video=281000-0        mp4 384x216     25 |  281k dash  | avc3.42C015  281k video only         DASH video, mp4_dash
mf_akamai-video=281000-1        mp4 384x216     25 |  281k dash  | avc3.42C015  281k video only         DASH video, mp4_dash
mf_cloudfront-video=281000-0    mp4 384x216     25 |  281k dash  | avc3.42C015  281k video only         DASH video, mp4_dash
mf_cloudfront-video=281000-1    mp4 384x216     25 |  281k dash  | avc3.42C015  281k video only         DASH video, mp4_dash
mf_akamai-400-0                 mp4 384x216     25 |  400k m3u8  | avc1.42C015  281k mp4a.40.5  96k
mf_akamai-400-1                 mp4 384x216     25 |  400k m3u8  | avc1.42C015  281k mp4a.40.5  96k
mf_cloudfront-400-0             mp4 384x216     25 |  400k m3u8  | avc1.42C015  281k mp4a.40.5  96k
mf_cloudfront-400-1             mp4 384x216     25 |  400k m3u8  | avc1.42C015  281k mp4a.40.5  96k
mf_akamai-video=437000-0        mp4 512x288     25 |  437k dash  | avc3.4D401E  437k video only         DASH video, mp4_dash
mf_akamai-video=437000-1        mp4 512x288     25 |  437k dash  | avc3.4D401E  437k video only         DASH video, mp4_dash
mf_cloudfront-video=437000-0    mp4 512x288     25 |  437k dash  | avc3.4D401E  437k video only         DASH video, mp4_dash
mf_cloudfront-video=437000-1    mp4 512x288     25 |  437k dash  | avc3.4D401E  437k video only         DASH video, mp4_dash
mf_akamai-565-0                 mp4 512x288     25 |  565k m3u8  | avc1.4D401E  437k mp4a.40.5  96k
mf_akamai-565-1                 mp4 512x288     25 |  565k m3u8  | avc1.4D401E  437k mp4a.40.5  96k
mf_cloudfront-565-0             mp4 512x288     25 |  565k m3u8  | avc1.4D401E  437k mp4a.40.5  96k
mf_cloudfront-565-1             mp4 512x288     25 |  565k m3u8  | avc1.4D401E  437k mp4a.40.5  96k
mf_akamai-video=827000-0        mp4 704x396     25 |  827k dash  | avc3.4D401E  827k video only         DASH video, mp4_dash
mf_akamai-video=827000-1        mp4 704x396     25 |  827k dash  | avc3.4D401E  827k video only         DASH video, mp4_dash
mf_cloudfront-video=827000-0    mp4 704x396     25 |  827k dash  | avc3.4D401E  827k video only         DASH video, mp4_dash
mf_cloudfront-video=827000-1    mp4 704x396     25 |  827k dash  | avc3.4D401E  827k video only         DASH video, mp4_dash
mf_akamai-979-0                 mp4 704x396     25 |  979k m3u8  | avc1.4D401E  827k mp4a.40.5  96k
mf_akamai-979-1                 mp4 704x396     25 |  979k m3u8  | avc1.4D401E  827k mp4a.40.5  96k
mf_cloudfront-979-0             mp4 704x396     25 |  979k m3u8  | avc1.4D401E  827k mp4a.40.5  96k
mf_cloudfront-979-1             mp4 704x396     25 |  979k m3u8  | avc1.4D401E  827k mp4a.40.5  96k
mf_akamai-video=1604000-0       mp4 960x540     25 | 1604k dash  | avc3.4D401F 1604k video only         DASH video, mp4_dash
mf_akamai-video=1604000-1       mp4 960x540     25 | 1604k dash  | avc3.4D401F 1604k video only         DASH video, mp4_dash
mf_cloudfront-video=1604000-0   mp4 960x540     25 | 1604k dash  | avc3.4D401F 1604k video only         DASH video, mp4_dash
mf_cloudfront-video=1604000-1   mp4 960x540     25 | 1604k dash  | avc3.4D401F 1604k video only         DASH video, mp4_dash
mf_akamai-1802-0                mp4 960x540     25 | 1802k m3u8  | avc1.4D401F 1604k mp4a.40.5  96k
mf_akamai-1802-1                mp4 960x540     25 | 1802k m3u8  | avc1.4D401F 1604k mp4a.40.5  96k
mf_cloudfront-1802-0            mp4 960x540     25 | 1802k m3u8  | avc1.4D401F 1604k mp4a.40.5  96k
mf_cloudfront-1802-1            mp4 960x540     25 | 1802k m3u8  | avc1.4D401F 1604k mp4a.40.5  96k
[download] Finished downloading playlist: Aaron Persky: Brock Turner judge fired from coaching girls

So, what's going on differently in your case? ...

So, what's going on differently in your case? ...

I get the same error as rpdelaney due to geo-restriction

I get the same error as rpdelaney due to geo-restriction

My physical location is Southern Europe :wink: (not the UK); I was under the impression www.bbc.com offerings (and, certainly, news items/clips) were globally available (from overseas), as opposed to many www.bbc.co.uk offerings reserved for just within the UK; in any case, if it's geo-restriction the reason rpdelaney fails to access that videoclip, is this still relevant to this issue?
From experience, when "I" try to access geo-fenced BBC content (e.g on iPlayer), I usually end-up with a 403 forbidden and/or geolocation error, which is indicative enough; wouldn't it be better if yt-dlp raised a similar error/notification in the case of
https://www.bbc.com/news/world-us-canada-49667778
too? (Unable to extract playlist data sounds pretty vague/generic to me...)

ah my bad, it's not geo-restriction. but the html that is served to my region seems completely different than anything the extractor is expecting. all the useful info is in a nextjs __NEXT_DATA__ block

it's not geo-restriction. but the html that is served to my region seems completely different than anything the extractor is expecting

Spot on! :+1: Europe and The Americas are being served different pages/embedded players; only the "European" version is compatible with the extractor:
European IP:

US IP:

In both cases, the content does play back inside the browser...


Updates to flowlist causing null flow exchanges

Due to some changes in FEDEFL, some mappings used in eLCI are now outdated, namely radioisotopes from upstream inventories (not StEWI based) including:
Radon-222
Thorium-230
Lead-210
Polonium-210
Radium-226
Cesium-137
Tritium
Radium-228
Radium-224
Iodine-131
Thorium-232
Strontium-90
Iodine-129
This resulted in exchanges with NULL values and display issues in openLCA. eLCI was generated specifying v.1.0.1 of FEDEFL.
Updating the mapping file should fix this issue, but raises the question of whether mapping files need version control, if users are able to adjust the version of FEDEFL used.

I believe this issue is now resolved based on #84 correct? @jump2conclusionsmatt

Agreed.


output null when tests don't have name

I just noticed this by accident when testing mem-map and couldn't find a similar issue.

AVA version: v0.11.0
The reason seems to be that I don't have a name for my test.

Should be [anonymous]

Actually - if there's just one test - it should be nothing.

This is the offending line. Should it just return an empty string?

Yep. Empty string


Define indent/outdent behavior for single-line selected text

Coming from atom/snippets#239, I have now realized that there isn't a properly defined behavior for selecting text on single line and indenting/outdenting it:

Here is why I don't think this has been defined yet:

There is no spec for single-line selected text being indented via indent(): https://github.com/atom/atom/blob/a7736b81e3f6628f2a9e685f28f4c1df9a5e4615/spec/text-editor-spec.coffee#L3673-L3789

indentSelectedRows (which is used in indent()) is poorly documented:
https://github.com/atom/atom/blob/a7736b81e3f6628f2a9e685f28f4c1df9a5e4615/src/selection.coffee#L671-L679

If the selection spans multiple rows, indent all of them.

It's possible to interpret this in a way that if the selection does not span across multiple rows, then nothing gets indented. Insert your favorite programmer-goes-to-grocery-store-joke here.

The current behavior is not intuitive and unexpected for anyone coming from a different editor. With the exception of Brackets, other text editors tend to agree that selecting text and pressing tab within a single line should not indent the whole line, but insert/overwrite with a tab.

Outdent is done through outdentSelectedRows, which has a spec for single line selected text. However it seems to be slightly mislabeled, there is actually less than one line selected. That is not a big deal and can be easily fixed, but it's possible that something is missing from here too.
At minimum I am looking to define behavior for following cases:

Selecting whole line and issuing indent().
Selecting substring within line with length greater than 0 and issuing indent().
Selecting whole line and issuing outdentSelectedRows().
Selecting substring within line with length greater than 0 and issuing outdentSelectedRows().

There might be additional cases that might need be considered, but I don't want this to be come overly ambitious, so starting with a more managable goal might be better.
The motivation for all of this comes from behavior related to text selections that come from traversing tab stops within snippets, where indenting/outdenting may lead to unexpected results:

Personally I think that outdent behavior can be kept as-is and it is sufficient to simply improve the spec and cover cases 3 and 4. Outdenting does not break out of snippet's tab stops whereas indenting does, so indicating change with a slightly more destructive (but predictable) operation should be obvious enough that it's impossible to return to snippet's tab stops.
Visual Studio Code's behavior is a suitable candidate, which indents if the whole line is selected, but inserts a tab when only part of line is selected. Outdent is handled in the same way as Atom does it now. If there aren't any objections or better ideas, I might just implement that and propose this as the final solution.
Code at the time of writing in pull request is just to get the ball rolling and does not represent the final proposed solution in any way. But it does replace everything with a tab if at most only one line has been selected.

In the meanwhile I implemented vscode-like behavior, so it can be tested, turned out to be easier than expected. I'll clean up commit list after feedback.

Thanks @Ingramz for the work you've put into this.
We agree that the behavior could definitely be better defined. We would love to hear your thoughts on some of the edge-cases and potential drawbacks of whatever approach you are ultimately considering here. It looks like there are a lot of interactions between selections, indenting/outdenting and snippets that could be tricky. As you get closer, please update the main body of the PR with the full details requested in the PR template and ping us again :+1:

@Ingramz Hello, In fact, I also found out that in SublimeText and Notepad++, only when we selected one or more rows, pressing the tab key will indent, otherwise it is just a normal tab operation. To solve this problem, I already spend four hours, but there is no any result. I am very happy to find your Issue, now two years have passed. I found this problem in Atom still unresolved. Is there any way to solve it ?


Incorrect bean property

Hello,
I'm trying to read a CSV file with the header but seems that name of the set method has an character in lowercase:
org.supercsv.exception.SuperCsvReflectionException: unable to find method setidOrder(java.lang.Integer)
The name of the bean property is idOrder and the Header is idOrder so the set name of the method should be setIdOrder.
Any suggestion?
Thank you for your support,
Raffaele Latela
201805081341477_ORDERS.csv.zip

This problem is caused by reading the UTF-8 bom file, and the file header has BOM information。

Hi, @rlatela
This problem is caused by reading the UTF-8 bom file, and the file header has BOM information. For UTF-8, the BOM is EF BB BF. For UTF-16, the BOM is FE FF. So when you read the csv file, It is a problem with the header caused by the BOM informatio.
You can solve this problem by converting the file to UTF-8 without BOM format.
I will continue to focus on this issue, Thank you!

#164 is the PR fix this problem


Move to testing source

Fixes #18

Codecov Report

Merging #19 into master will increase coverage by 3.82%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #19      +/-   ##
==========================================
+ Coverage    90.9%   94.73%   +3.82%
==========================================
  Files           2        3       +1
  Lines          11       19       +8
==========================================
+ Hits           10       18       +8
  Misses          1        1

Impacted Files
Coverage Δ

src/Page.js
100% <0%> (ø)
:arrow_up:

src/index.js
100% <0%> (ø)

src/fetchPage.js
90% <0%> (+6.66%)
:arrow_up:

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 fa8e4e9...b53bf4e. Read the comment docs.


Traffic Flow from Spok1 to SPoke2

In the diagram below spoke-connectivity, you have NVA.  With each Spoke1 & Spok2 each has a UDR to send traffic to the other VNet.
If VNet Peering doesn't allow transitive traffic.  How does this traffic flow from Spok1 to Spoke 2.  Is it through the internet?  That is not communicated in this document.

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

ID: 104eba32-c77e-b013-e161-ff616d16a0f7
Version Independent ID: 3bc27d0a-ef32-f5f6-4c85-fe8e2e45155e
Content: Implement a hub-spoke network topology in Azure - Azure Reference Architectures
Content Source: docs/reference-architectures/hybrid-networking/hub-spoke.md
Service: architecture-center
Sub-service: reference-architecture
GitHub Login: @telmosampaio
Microsoft Alias: pnp

Hi @DMoonesinghe Thank you for your feedback! We will review and update as appropriate.

@DMoonesinghe You are correct, and this is discussed in the peering section of the doc.
Getting the 2 spokes to communicate via a NVA in the HUB VNET is also an option, and it is discussed here.

@DMoonesinghe We will now proceed to close this thread for now. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Where to doc methods

Discussion
In #73, @AFg6K7h4fhy2 proposes documenting methods as part of the class docstring and the associated method (so double docs). I'm not sure this is the right approach. I like having more info, but it could introduce an extra burden for the dev. So I open this for discussion.
Context
[Short paragraph describing how the issue arose and constraints imposed by the existing code architecture]
Required features
None yet, but it may lead to some refactoring.

cc @dylanhmorris @natemcintosh this is in your Py dev alley.

We should not reference the methods in the class docstring. Rather, we should give each class method its own docstring and configure sphinx.ext.autodoc etc appropriately to make sure the methods are enumerated in the class's rendered documentation. See e.g. the documentation for the DataFrame class in polars and the corresponding source code

Acknowledging that I have seen this. DHM's suggestion makes sense, somewhat in terms of adhering to don't repeat yourself.
Further supporting this is Numpydocs statement (extending to the photo):

In general, it is not necessary to list class methods. Those that are not part of the public API have names that start with an underscore. In some cases, however, a class may have a great many methods, of which only a few are relevant (e.g., subclasses of ndarray). Then, it becomes useful to have an additional Methods section:

I see. I'm still aligned with @dylanhmorris about where to put the docstring: So, for the moment, let's avoid referencing the method in the class docstring. I'm closing and adding a new issue removing methods documentations for the next sprint.


Make Enemies / Death Pits

Add enemies to the game

Find sprites for the enemies

Death Pits: Add death pits that restart the player (COMPLETED)

Death Pits were added

This was done by have a function with everything in it so when you reach a certain y value you rerun the function to reset the level and the player.


[$1000] System throws an error when user request an IOU to the secondary login account

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Sign in with account A
Add account B as a secondary login in the contact methods section
Open new tab (window) & sign in with account C
Click on 'FAB' icon
Click on 'Split bill' link
Enter amount, and click on 'Next' button
Enter secondary login account B in the search field, and select the account
Click on 'Next' button
Click on 'Split' button

Expected Result:
Systems should send a successful IOU request
Actual Result:
System throws an error message
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: 1.3.13
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/218d4c6b-15d2-40cf-8a48-211fa3cbfb31
https://github.com/Expensify/App/assets/43996225/10a38a39-31ef-437b-a292-77f85c1ff7c6
Expensify/Expensify Issue URL:
Issue reported by: @natnael-guchima
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1683793394254349
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01de99bdd46652a0b4
Upwork Job ID:<PHONE_NUMBER>396896768
Last Price Increase: 2023-05-15

Thanks for this!

Proposal
Please re-state the problem that we are trying to solve in this issue.
System throws an error when user request an IOU to the secondary login account
What is the root cause of that problem?
We are blocking user from creating IOU to their login list on the server, but doesn't validate it from client. So when an user tried to create IOU to the secondary account, it will show the generic error.
What changes do you think we should make in order to solve the problem?
https://github.com/Expensify/App/blob/293f23b57be2bca3f3eb0af903d5e0c2894cb36a/src/components/MoneyRequestConfirmationList.js#L263
Right now, in MoneyRequestConfirmationList we only disable the bottom button if the selected participants length equals to 0. We can improve that by checking if inside  selected participants have user from loginList then disable the button, we can also show a message to let user know they shouldn't create IOU request to themself.
So the shouldDisableButton condition will look like this:
 const isIncludeCurrentUSer = _.some(selectedParticipants, OptionsListUtils.isCurrentUser);
 const shouldDisableButton = selectedParticipants.length === 0 || isIncludeCurrentUSer;

We can also add a error message at the Footer of the component to let user know they shouldn't request money to themself.
                footerContent={
                    <>
                        {shouldShowSettlementButton ? (
                            <SettlementButton
                                isDisabled={shouldDisableButton}
                                onPress={this.confirm}
                                shouldShowPaypal={Boolean(recipient.payPalMeAddress)}
                                enablePaymentsRoute={ROUTES.IOU_SEND_ENABLE_PAYMENTS}
                                addBankAccountRoute={ROUTES.IOU_SEND_ADD_BANK_ACCOUNT}
                                addDebitCardRoute={ROUTES.IOU_SEND_ADD_DEBIT_CARD}
                                currency={this.props.iou.selectedCurrencyCode}
                            />
                        ) : (
                            <ButtonWithMenu
                                isDisabled={shouldDisableButton}
                                onPress={(_event, value) => this.confirm(value)}
                                options={this.getSplitOrRequestOptions()}
                            />
                        )}
                        {isIncludeCurrentUSer && <FormHelpeMessage message={"You can't send IOU request to yourself"} />}
                    </>
                }

What alternative solutions did you explore? (Optional)
Do nothing
Result

Not overdue, waiting on the proposals and for @deetergp view on which is the best :)

I'm removing both the External and Help Wanted labels, as these errors are coming from the deepest depths our back end. Additionally, I think there needs to be a deep discussion about how we want this to work before we start planning how to fix it.

This is being discussed here.

There has been more discussion, but no real decision on how we're going to approach it. I'm going to downgrade this to a weekly but it will likely get closed.

Cool, thanks @deetergp . I am going OOO so if it doesn't get closed and you need BZ management, please reapply the label!

For more context, after this PR https://github.com/Expensify/App/pull/19431#issuecomment-1563907063 is merge. We will block user from messaging their other contact method email and they can't send IOU request to these account any more. I think we can close this one.


CreateLoadBalancer is failing on the free subscription.

Operation Name: CreateLoadBalancer
OPC request ID: 3ee932500afb2d41186ac6ac1889afc9/5DCE56D4ACD6AD37E76D4536CF567C84/F40476563924E91390C32A7BEB770421
on lb.tf line 9, in resource "oci_load_balancer" "lb01"
9: resource "oci_load_balancer" "lb01" {
Error: 400-LimitExceeded, NAT gateway limit per VCN reached
Suggestion: Request a service limit increase for this resource Core Nat Gateway
Documentation: https://registry.terraform.io/providers/oracle/oci/latest/docs/resources/core_nat_gateway
Request Target: POST https://iaas.us-ashburn-1.oraclecloud.com/20160918/natGateways
Provider version: 4.72.0, released on 2022-04-21.
Service: Core Nat Gateway
Operation Name: CreateNatGateway
OPC request ID: d24d38102605af7c867ef1d9cad0ebcb/B2D6F003113B31DB52977E3D20D8761F/DED4EBFAA800B4D5475F539682511CE2
on network.tf line 22, in resource "oci_core_nat_gateway" "vcn01_nat_gateway"
22: resource "oci_core_nat_gateway" "vcn01_nat_gateway" {

@mahendrabaghel Could you please retry the latest release: https://github.com/oracle-devrel/terraform-oci-arch-tomcat-autonomous/releases/tag/v1.5

Hi,
It worked but with Free Tier VCN is allowed one NAT Gateway. The issue seems to be with Loadbalancer max size greater than 10MB. Free tier only allows 10MB. Can we try with private subnets for Tomcats?
Thanks,
Mahendra

@mahendrabaghel I am afraid you are not right. FreeTier doesn't cover NAT Gateway. That is why in FreeTier scenario, Tomcats are not in private subnets. In the case of LoadBalancer, please check this [page](Flexible Load Balancer: 1 instance, 10 Mbps). It is clear FreeTier covers Flexible LB 10Gb/s.


SPIKE incomplete section pattern

To make it easier to deal with incomplete sections on the form we show only completed fields and wrap incomplete sections in a blue border.
We add the domain objects SectionGroup which contains many Section which contains many Forms to represent the form structure.
A Form is the atomic unit of completion and forms are complete if they are valid? so it is relatively easy to see which is the next question. All the state of the form completion is already stored in the referral object, so there's no need to add additional state.
For consistency and for the system to work correctly Form objects need to be initialized with a referral object rather than specific parameters.
Form consistency is implemented with the ReferralFormSection module (needs a rename), and we've also added Section and SectionGroup base classes which provide default paths and translations (some of these are overridden, but we could go back and homogenise these at a later date)
The ViewComponents for rendering summaries of the sections now require a section object so they know completed they are and whether to render the blue borders around them.
https://trello.com/c/jhcJstSI/1315-spike-incomplete-section-pattern

I've simplified the PR so it only covers a single section and polished off the rough edges a bit


[2.1.3] Never throw ODE from MemoryPoolBlock.Dispose()

aspnet/KestrelHttpServer#2638

@pranavkm Given this is referenced by source in the Kestrel repo, do we have to do anything special to make sure this change is picked up in our 2.1.3 builds?

@natemcmaster \ @ryanbrandenburg would be the folks to ask 👍


Docs - Examples integrations Javascript's Library

Hi!
@etimberg @simonbrunel @zachpanz88
what do you think of the suggestion?
Tks!

I don't really see the point of adding another section to the documentation. The docs serve well as a reference for getting charts up and running and there are already multiple plugins/bindings that make it easy to use with other libraries.


Can not find the texlab after using setup{}

Description
I add
require('lspconfig').texlab.setup {
  settings = {
    latexindent = {
      modifyLineBreaks = true,
    },
  },
}

to my init.lua. Then open a tex file. An error accur Spawning language server with cmd: texlab failed. The language server is either not installed, missing from PATH.  However, when the setup {} function is removed from the init.lua, the texlab initializes correctly.
Neovim version
NVIM v0.6.1
Build type: Release
LuaJIT 2.1.0-beta3
Nvim-lspconfig version
ad9903c66bac88f344890acb6532f63f1cd4dac3
Operating system and version
WSL Ubuntu 20.04.4
Affected language servers
texlab
Steps to reproduce

install the texlab by nvim-lsp-installer, which is installed in ~/.local/share/nvim/lsp_servers/latex by default and not in the $PATH.
Then texlab can be correctly initialized.
add

require('lspconfig').texlab.setup {
  settings = {
    latexindent = {
      modifyLineBreaks = true,
    },
  },
}

to the init.lua. Then texlab failed to be initialized when open a tex file.
4. add ~/.local/share/nvim/lsp_servers/latex to the $PATH. Then the texlab is initialized correctly. However, modifyLineBreaks = true, does not work.
Actual behavior
When open a tex file an error accur Spawning language server with cmd: texlab failed. The language server is either not installed, missing from PATH.
Expected behavior
Correctly initialize the texlab when open a tex file and modifyLineBreaks = true works.
Minimal config
lunarvim is used.

LSP log
[START][2022-04-27 19:15:54] LSP logging initiated [INFO][2022-04-27 19:15:54] .../vim/lsp/rpc.lua:261	"Starting RPC client"	{  args = {},  cmd = "texlab",  extra = {    cwd = "/home/LunarVim/temp_try"  }} [INFO][2022-04-27 19:15:54] .../vim/lsp/rpc.lua:261	"Starting RPC client"	{  args = {},  cmd = "texlab",  extra = {    cwd = "/home/LunarVim/temp_try"  }} [START][2022-04-27 19:16:41] LSP logging initiated

install the texlab by nvim-lsp-installer, which is installed in ~/.local/share/nvim/lsp_servers/latex by default and not in the $PATH.

Issues that require a plugin to reproduce are not issues that should be filed here.


image added

Docs Build status updates of commit 7df3a7e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-community-training/media/ContentLocalization1.png
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

PRMerger Results

Issue
Description

Added or Deleted File(s)
This PR contains added or deleted file(s).

Image File
This PR contains an image file(s).


assets/manifests/coredns\cluster-role-binding.yaml fails to generate on Windows

Bug
UPDATE: It seems that the problem only occurs on windows.
assets/manifests/coredns\cluster-role-binding.yaml is missing.
Only last separator is backslash, so It looks like a bug of the template provider.
Environment

Platform: aws
OS: container-linux
Release: Typhoon version v1.15.3, v1.16.1
Terraform: v0.12.9
Plugins:

provider.aws v2.31.0
provider.ct v0.4.0
provider.local v1.4.0
provider.null v2.1.2
provider.template v2.1.2
provider.tls v2.1.1

Terraform is running on windows machine.
Problem
Error: open assets/manifests/coredns\cluster-role-binding.yaml: The system cannot find the path specified.

  on .terraform\modules\tempest.bootkube\assets.tf line 17, in resource "template_dir" "manifests":
  17: resource "template_dir" "manifests" {

manifest directory is created, but its contents is empty.
$ find ./assets/
./assets/
./assets/auth
./assets/auth/kubeconfig
./assets/auth/kubeconfig-kubelet
./assets/auth/tempest-config
./assets/bootstrap-manifests
./assets/bootstrap-manifests/bootstrap-apiserver.yaml
./assets/bootstrap-manifests/bootstrap-controller-manager.yaml
./assets/bootstrap-manifests/bootstrap-scheduler.yaml
./assets/manifests
./assets/manifests-networking
./assets/manifests-networking/bgpconfigurations-crd.yaml
./assets/manifests-networking/bgppeers-crd.yaml
./assets/manifests-networking/blockaffinities-crd.yaml
./assets/manifests-networking/cluster-role-binding.yaml
./assets/manifests-networking/cluster-role.yaml
./assets/manifests-networking/clusterinformations-crd.yaml
./assets/manifests-networking/config.yaml
./assets/manifests-networking/daemonset.yaml
./assets/manifests-networking/default-ipv4-ippool.yaml
./assets/manifests-networking/felixconfigurations-crd.yaml
./assets/manifests-networking/globalnetworkpolicies-crd.yaml
./assets/manifests-networking/globalnetworksets-crd.yaml
./assets/manifests-networking/hostendpoints-crd.yaml
./assets/manifests-networking/ipamblocks.crd.yaml
./assets/manifests-networking/ipamconfigs-crd.yaml
./assets/manifests-networking/ipamhandles-crd.yaml
./assets/manifests-networking/ippools-crd.yaml
./assets/manifests-networking/networkpolicies-crd.yaml
./assets/manifests-networking/networksets-crd.yaml
./assets/manifests-networking/service-account.yaml
./assets/tls
./assets/tls/admin.crt
./assets/tls/admin.key
./assets/tls/apiserver.crt
./assets/tls/apiserver.key
./assets/tls/ca.crt
./assets/tls/ca.key
./assets/tls/etcd
./assets/tls/etcd/peer-ca.crt
./assets/tls/etcd/peer.crt
./assets/tls/etcd/peer.key
./assets/tls/etcd/server-ca.crt
./assets/tls/etcd/server.crt
./assets/tls/etcd/server.key
./assets/tls/etcd-ca.crt
./assets/tls/etcd-ca.key
./assets/tls/etcd-client-ca.crt
./assets/tls/etcd-client.crt
./assets/tls/etcd-client.key
./assets/tls/kubelet.crt
./assets/tls/kubelet.key
./assets/tls/service-account.key
./assets/tls/service-account.pub

Desired Behavior
Cluster starts successfully
Steps to Reproduce
module "tempest" {
  source = "git::https://github.com/poseidon/typhoon//aws/container-linux/kubernetes?ref=v1.15.3"

  # AWS
  cluster_name = "tempest"
  dns_zone     = "stating.example.com"
  dns_zone_id  = "<my_zone_id>"

  # configuration
  ssh_authorized_key = "ssh-rsa ..."
  asset_dir          = "${path.module}/assets/"

  # optional
  worker_count = 1
  worker_type  = "t2.micro"

  controller_type = "t3.small"
}

looks like a bug of the template provider. It seems that the problem only occurs on windows.

Looks like you've already identified the suspect template template_dir. Try to isolate a standalone template_dir example with a nested directory to report the path separator issue to upstream terraform-provider-template.

The problem also occurs when I run terraform-render-bootstrap directly, so I attached it as an example.

Great, thanks for reporting. I'd say track that issue to await a fix in the template provider for Windows.

The problem also occurs when I run terraform-render-bootstrap

Yes, as that's where template_dir is used. You can likely produce a much smaller reproducible example:
resource "template_dir" "manifests" {
  source_dir      = "input"
  destination_dir = "output"
  vars = {}
}

$ tree input
input/manifests/
├── manifests
│   ├── cluster-role-binding.yaml
├── somefile.yaml

I don't make use of Windows systems, so perhaps you can check if that demonstrates the problem.

I've updated example with minimal one. Unfortunately the issue is not getting attentions by maintainers.
At the same time, I've found a workaround for the issue while I was trying to customize terraform-render-bootstrap for putting generated files on Amazon S3.
You can create set of files from templates on S3 using for_each.
resource "aws_s3_bucket_object" "manifests" {
  for_each        = fileset("${path.module}/resources/manifests", "**/*.yaml")

  bucket = aws_s3_bucket.assets.id
  key    = "/manifests/${each.value}"
  content = templatefile("${path.module}/resources/manifests/${each.value}", {
    hyperkube_image        = var.container_images["hyperkube"]
    coredns_image          = var.container_images["coredns"]
    control_plane_replicas = max(2, length(var.etcd_servers))
    pod_cidr               = var.pod_cidr
    cluster_domain_suffix  = var.cluster_domain_suffix
    cluster_dns_service_ip = cidrhost(var.service_cidr, 10)
    trusted_certs_dir      = var.trusted_certs_dir
    server                 = format("https://%s:%s", var.api_servers[0], var.external_apiserver_port)
  })
}

I've tried again with local_file resource and it could handle subdirectories without problems.
resource "local_file" "manifests" {
  for_each = fileset("${path.module}/resources/manifests", "**/*.yaml")

  filename = "${var.asset_dir}/manifests/${each.value}"
  content  = templatefile("${path.module}/resources/manifests/${each.value}", {
    hyperkube_image        = var.container_images["hyperkube"]
    coredns_image          = var.container_images["coredns"]
    control_plane_replicas = max(2, length(var.etcd_servers))
    pod_cidr               = var.pod_cidr
    cluster_domain_suffix  = var.cluster_domain_suffix
    cluster_dns_service_ip = cidrhost(var.service_cidr, 10)
    trusted_certs_dir      = var.trusted_certs_dir
    server                 = format("https://%s:%s", var.api_servers[0], var.external_apiserver_port)
  })
}

But if we switch to this method, what value output and content hash should be? Should we concat generated content on memory and make sha1 hash?
I tried putting generated yaml files on S3, because current approach that uses local_file and template_dir has some issues.

When working with multiple developers or on CI environment (using remote state store like S3), files get generated on each machine, making a lot of noise on plan result. Especially on CI environment, this will occur every time when plan command is invoked.
I just don't want to put sensitive information like certifiactes on my local machine. They should be downloadable to only developers/machines which has right IAM permissions. I'd rather let master nodes download certificates when they launch.

@dghubble
I created a PR for generating bootstrap files without using template_dir:
https://github.com/poseidon/terraform-render-bootstrap/pull/157
This solved the problem of templating on windows. I've faced another problem though: I couldn't connect to AWS EC2 with pageant. Terraform won't use ssh keys stored in pageant.
I modified typhoon, and provided the content of ssh private key to make provisioner work without agent.

For now, the recommendation is still to await a fix from the upstream template provider or use a non-Windows system. Moving to templatefile may be on the horizon, but for other reasons, not for whether its better/worse on Windows. It affects plugin dependencies in downstream modules, so if I go that route it would be a part of broader changes/testing and likely not in a point release.
As for the other questions, please keep focused on the issue. I'll briefly comment on two items to close them:

Rendering to aws_s3_bucket_objects in terraform-render-bootstrap is not a goal (and tangential to your rendering issue). The bootstrap module is pulled in as a dependency for multiple cluster modules (GCP, Azure, DO, bare-metal) and its not appropriate to take on a cloud provider dependency. AWS occupies no special prominence. Please drop references to this being a goal/direction*.
Terraform SSH agent support is used intentionally, it prevents your private key being present in state files (indeed, in security conscious setups with Yubikeys its not even possible to provide a private key as it doesn't reside on-disk). Probably revisit any security shortcuts you've taken, privately.

I created another issue to talk about how we should generate manifests/certs: #584 584
Regrading SSH issue, I understand that agent is better for security,  but I just couldn't make it work on windows. Only environment that I could make it work is WSL ubuntu, which has ssh-agent. To make ssh-agent available for terraform running on Docker for windows, I guess some special configuration is required.

Closed by #587


Executing the same promise chain multiple times

PromiseKit version: 4.3.1
Xcode version: latest non-beta version

I'm using Firebase to retrieve items from a list in the database, so the same async completion handler will be called multiple times. How do I wrap such a function? The obvious way seems to be to just put the promise chain inside the completion handler, but that's not clean.
func foo() {
    ref.observe(.childAdded) { snapshot in
        // this is called many times, and I need it to execute the same promise chain each time
    }
}

func bar() {
  // I want this
    ref.observe(.childAdded).then {
        // do something for each call of observe()'s callback
    }.then {
        // do something else for each call
    }
}

The only easy way to do this is to add the chain to the completion handler.
The harder way would be to write your own .then for the firebase object returned from observe which returns your own “promise-like” objects that do the same as above but you'll get your cleaner code.
Promises only complete once, which is their feature, and in places like this, their curse.

Thanks. I'll just put them in the completion handlers, then.


Merge main

Does anyone have the access to Create a Merge Commit instead of Squash and merge? The PR merges the codes of main to feature/v4. So we don't want to squash it. thanks


fix the browser compatibility; this closes #40

Fixing the compatibility issue by adding semantic UI markup and CSS

looks fine on firefox, I recommend changing the border to 3px. push your new commit with the change.
the width doesn't look good we need to make it smaller.
please open a new issue for the width. we will deal with it later.


MNT: Re-rendered with conda-build 3.18.8, conda-smithy 3.4.1, and con…

…da-forge-pinning 2019.07.22

Checklist

[ ] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

There are too few lines.  There should be one empty line at the end of the file.


Setting the target SOC seems not work

Setting the target SOC seems not work with my eUP.
up/vehicles/XXX/domains/charging/chargingSettings/targetSOC_pct_writetopic
has no impact.
There is also no readable topic for:
up/vehicles/XXX/domains/charging/chargingSettings/targetSOC_pct
Can anybody confirm that it is not working?

what type of car is it?

It's an eUPBRAm 13.09.2024 07:56 schrieb Till Steinbach @.***>:
what type of car is it?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

The e-up is a bit special. Can you control the target SoC through the Volkswagen App?

I just checked and it's also not working.Then I guess it's not possible.Am 13.09.2024 15:11 schrieb Till Steinbach @.***>:
The e-up is a bit special. Can you control the target SoC through the Volkswagen App?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

Maybe it is added later. When you can control it some time in the future through the app let me know again.


Make checkPropertyNotUsedBeforeDeclaration ignore properties of properties

Use isAccessExpression to cover both PropertyAccess and
ElementAccess.  Also use it in a few other places that used both
explicitly.  (And also fix a random weird formatting.)
Fixes #32721.

@RyanCavanaugh, I'm assuming that it's needed at both levels, right?  I did that with a utility that I found that checks for both, and updated some other places that checked for both of these explicitly.

Looks like there are some lint errors


Header comment still saying "BSD-style license" in xcodeproj

To reproduce, use react-native init. Even for the latest v0.54 version, the header comments in AppDelegate.h, AppDelegate.m, main.m, etc, are still saying:

/**

Copyright (c) 2015-present, Facebook, Inc.
All rights reserved.

This source code is licensed under the BSD-style license found in the
LICENSE file in the root directory of this source tree. An additional grant
of patent rights can be found in the PATENTS file in the same directory.
*/

This also shows up for Android builds.

The license change landed in master after the 0.54 RC had already been cut. If you build from source (i.e. master), you should not see the previous BSD header. If all goes well, the change should make it to a stable release in v0.55.
Note that we have checks in place that should catch any regressions here, and they are green at this moment.


Radix Sort in C++

Description 📜

In this, we will get a brief idea about the The Radix Sort Algorithm

Domain of Contribution 📊

[x] C/CPP
[x] Competitive Programming
[x] DSA

/assign

/assign

Hello
My name is Rohit Goyal
I would like to work on the documentation part
Pls assign me this issue
i am from DSA batch-2


Update dependencies on foxy branch

As we've released ur_msgs and ur_client_library to foxy, we should remove those from the not-released repo files.

merged, closing


Fix AddContextListener to be async

I cannot think of any other looking at the API surface; but any other API that needs to be async?

I cannot think of any other looking at the API surface; but any other API that needs to be async?

They theoretically should have been done in the FDC3 v2 changes but I missed this one.  It was discovered while doing the example app.


update(README): fix typo

Signed-off-by: Michele Zuccala<EMAIL_ADDRESS>What type of PR is this?
/kind documentation
Any specific area of the project related to this PR?
What this PR does / why we need it:
Which issue(s) this PR fixes:
Small typo.
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: zuc
Once this PR has been reviewed and has the lgtm label, please assign maxgio92 for approval by writing /assign @maxgio92 in a comment. For more information see:The Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/assign @leogr

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr, zuc
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Concurrent & Prefixed Logging

Given that we have background queues that will want to log, we should be able to use ASL to log with separate clients on these queues. Also added some prefixing and additional video logging.

@facebook-github-bot import

Reviewed IRL by @mmmulani


fix credit card bug which accepted string alphabets #416

 Bugfix: master branch  #416

Q
A

Documentation
no

Bugfix
yes

BC Break
no

New Feature
no

RFC
no

QA
no

Description
now the Credit card validation rejects non-numeric characters and just accept spaces & dashes which are placed individually
Example:
these following credit cards returned false:

"26f71e56d2e54fabae2ca87e5c524fef"
"26715--62542--875524"
"26715   625428755  24"
"26715 -62542-875524"

Hello guys!
I forked this package cause owner disappeared. Hope, he will be back, but it would be easier to merge these changes back if he is back
Link to my repo: create issue there and we'll discuss it.


Consider default cert domain in certificate store

What does this PR do?
This allows for the default certificate to be considered when getting all domains from a TLS certificate store.
Motivation
When getting all domains for a store, the default cert domain should be included in the domains list. This is applicable when generating ACME domains, it should not generate a certificate for a domain that falls under the default domain.
More

[x] Added/updated tests
[ ] Added/updated documentation

Additional Notes
Co-authored-by: Nicolas Mengin<EMAIL_ADDRESS>

Fixed


Config References don't get updated for inputActions until they get updated the next time

[Issue created by DocMoebiuz: 2021-04-18]

UPDATE:
I think we can test this with you file from the "Tutorial Video for the XPDR"
Baldy you never show the situation in the Vid while your in the INPUT Tab,
If possible we can simply confirm this via Shared Screen


Add support for specifying swift_versions in cocoapods 1.7

Looks like travis-ci is still using a prior version of CocoaPods when running the build (and not the latest stable).
We could either update the .travis.yml to install the latest CocoaPods with a before step: gem install cocoapods, or add a Gemfile to the project that can lock the CocoaPods version with Bundler?
Unsure of the backwards compatibility of this change, for users that remain on CocoaPods 1.6.

Have updated the .travis.yml to update CocoaPods to the latest version before running the pod spec lint, pod lib lint commands.
Have also double-checked the backwards compatibility of this change.
For users that remain on version 1.6 (or prior) version of CocoaPods, this is considered a breaking change as it will fail to install the pod.
Tried using variants of swift_version, and swift_versions with single (and array) version specifiers but couldn't find anything that was backwards compatible.
However, moving across to using swift_versions with the array based version specifiers will allow end-users of the Pod to be able to compile it with Swift 5.0 instead of Swift 4.0 by default.

Looks like travis-ci is failing the build due to a warning in one of the swift files (from the 1.5.1 tag), when compiled with Swift 5.
If we were to update the Podfile to point at the new 1.6.0 release tag, then this should pass on travis-ci. But would depend on if how you want to manage that with the #213 PR as well.

I've run a build for this PR again on travis ci.

I'm planning to release this on v1.7.0. Thanks, @joshuafinch 👍

I'm sorry for late. I merged this PR to master for v1.7.0! Thank you.


added terms for #22374 for #22373

@balhoff We can merge the PR with these errors?

Tricky error to find in the log:
ERROR: CHEBI IDs should be mapped to pH 7.3:
168396

CHEBI:168396 is not in the pH 7.3 set.

Thanks @balhoff
A side question: CHEBI: 168396 is not in the imports, ie it's somehow imported automatically. Could we make it that only allowed CHEBI chemicals would be imported? or available to GO editors?
Thanks, Pascale


feat(fabric) Update manage-user-certificate.yaml playbook

Description
As a developer I want to update the manage-user-certificate.yaml playbook, since it uses roles that have been decoupled from ansible and are no longer available
Acceptance Criteria
Deploy the playbook with the corresponding charts that replaced roles.

I'll work on this


sorbet: Update RBI files.

Autogenerated by the sorbet workflow.

Review period will end on 2020-12-02 at 01:14:32 UTC.

Review period ended.

Review period ended.


Does the framework support ChatGLM now?

想试试chatglm在经过RLHF后的效果。

https://github.com/yangzhipeng1108/DeepSpeed-Chat-ChatGLM

@yangzhipeng1108 你的这个项目这个好像还没跑通啊

Due to the countless number of existing models, we won't be able to test our framework on all of them. It'd be helpful if you could just simply try yourself first, and submit an issue (and how to reproduce it) whenever you face one.


Merge release/0.8.1 into main

Description
Merge release/0.8.1 into main
Status
Ready/Work in progress/Hold
Types of changes

[x] Non-breaking change (fix or new feature that would not break existing functionality).
[ ] Breaking change (fix or new feature that would cause existing functionality to change).
[ ] New tests added to cover the changes.
[ ] All tests passed locally by running ./src/run-tests-in-docker.sh.
[ ] Documentation comments included/updated.
[ ] User guide updated.
[ ] NGC publishing metadata updated.
[ ] I have updated the changelog

Codecov Report

Merging #84 (95b3522) into main (28e9c99) will increase coverage by 0.03042%.
The diff coverage is 73.91304%.

@@                 Coverage Diff                 @@
##                main         #84         +/-   ##
===================================================
+ Coverage   85.76867%   85.79909%   +0.03042%
===================================================
  Files            152         152
  Lines          13133       13140          +7
  Branches         729         728          -1
===================================================
+ Hits           11264       11274         +10
+ Misses          1601        1597          -4
- Partials         268         269          +1

Impacted Files
Coverage Δ

src/Server/Program.cs
0.00000% <0.00000%> (ø)

src/Server/Repositories/ResultsApi.cs
93.02326% <76.92308%> (+6.56492%)
:arrow_up:

.../Server/Repositories/InferenceRequestRepository.cs
93.19728% <100.00000%> (+0.04660%)
:arrow_up:

src/Server/Services/Jobs/DataRetrievalService.cs
81.59722% <100.00000%> (-0.57062%)
:arrow_down:

src/Server/Test/Integration/DicomAdapterFixture.cs
95.89744% <100.00000%> (ø)

...er/Test/Unit/Processors/AeTitleJobProcessorTest.cs
66.94561% <100.00000%> (ø)

...rc/Server/Test/Unit/Repositories/ResultsApiTest.cs
100.00000% <100.00000%> (ø)

src/Server/Services/Jobs/JobSubmissionService.cs
94.56522% <0.00000%> (+1.08695%)
:arrow_up:

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 28e9c99...95b3522. Read the comment docs.


Remove uniqueness for email

Please let me know how to remove duplicate email validation from this gem
Thanks.

@chaitanya-nyros hello.
Do you want sign up user with the same email more at once?

Yes I want signup user with same email more at once
But the provider is same.
I want to work this gem as bellow. Could you please guide me on this
Same email and provider bind to scope like organisation_id

@chaitanya-nyros
I am not sure, but you can't sing up with the same email more than once. I think it is impossible.
You can try to set max_number_of_devices option and use the same email to sign in in different devices.
https://github.com/lynndylanhurley/devise_token_auth/blob/fcb1e6d22666175aa31945f7b4f0593dc111ff1c/lib/generators/devise_token_auth/templates/devise_token_auth.rb#L14-L16

Okay then is it possible to create user with same email and different provider?

@chaitanya-nyros

Okay then is it possible to create user with same email and different provider?

Try. I have just assumed this.
You should also consider OmniAuth.
Actually I don't understand what you want from simple authentication by unique email.
You should also ask someone else.

Yes I tried it is working thanks :)

You can have the same email with different providers, if you want inside the same provider, you can override email_provider? in user and return false, but that won't work because we always return the first user matched with that email.

@MaicolBen
I customized the gem to work with same emails and different providers.
Thanks.