Feature uri scheme fixes

This pull request implements the following fixes to the WebView uri scheme handling:
Ensure schemes read from the uno project are suffixed with :// before being taken into account by webviews. This might be seen as redundant so is in own patch.
Ensure schemes are only matched against the URL prefix: In the past the scheme "foo" would match on foo anywhere in the url.
Ensure no attempts to match schemes takes place if there is no explicitly set URI scheme handler for the webview.
This fixes #3859
This PR contains:

[x] Changelog

@Sunjammer bump

Oh ffs I was convinced this thing got merged. I'll get on it.

@Sunjammer: The diff now is pretty much bonkers. Could you rebase this on master rather than doing these sketchy merges?

Basically, it's not this branch that is supposed to merge release-1.2 into master.

For sure.  I'll clean her up.

Re the last merge commit, should I not be using GitHub's conflict resolver to fix changelog conflicts?

@Sunjammer: I don't see a problem with doing that.

@mortoray Look better now?

All I really did (I thought) was remove the merge markers so GitHub would let me mark the conflict as resolved. I absolutely effing loathe changelog merges, can't seem to do them right ever.

@Sunjammer: Seems you did more; you left at least one marker, and somehow deleted a line. Merging changelogs isn't my favourite thing either, but we still need to get it right.

Once more unto the breach


Binance Smart Chain (BSC) usage

So BSC has mempool too, the issue is with Theo on BSC. Has anyone got some access to direct BSC node that processes transactions (block producer)? How otherwise to bypass BSC mempool and send transaction directly to block producers, are there any paid services to get direct producer's node endpoint?
https://www.reddit.com/r/ethdev/comments/n11jm1/ethereum_can_be_front_run_but_can_someone_front/

No private trxs, at least public ones. Just use higher gas price


Add documentation on Sortable adapter

Adapter PR: https://github.com/prebid/Prebid.js/pull/2824

Hi @bretg , I'll be helping out while Shannon is on vacation for the next few days.
Good idea, I've made the requested changes, let me know what you think.

released in 1.17


Get md from HMAC_CTX

Would you mind a new opaque api const EVP_MD *HMAC_CTX_md(const HMAC_CTX *ctx) for openssl-1.1?
BTW, the source ctx of HMAC_CTX_copy can be const.

Possibly. Why do you need this?

Yes, I want to copy a given HMAC_CTX, just the algorithm not the states.

Added in commit a621181. Closing.


mempool/lanes: Write lane-specific unit tests

Depends on #3978 and #3980.

Allow kvstore to initialise with lanes (lanes = nil)
Unit test scenarios with nil as lanes (in clist_mempool_test.go)

Closed with #3484 and #4008


Max Header Size

Is their a maximum header size for requests/responses in Fresh?

Fresh doesn't impose any limits anywhere. This is more of a question about deno and I see that you've already asked the question over there https://github.com/denoland/deno/issues/21047


[BUG] Add-PnPContentTypesFromContentTypeHub - Not syncing to the current site

Add-PnPContentTypesFromContentTypeHub - Not syncing to current site
Reporting an Issue or Missing Feature
Issue
Expected behavior
To sync the published content types in to the current site
Actual behavior
Command runs without any error, but no Content Types are synced in to the current site
Steps to reproduce behavior
  Connect-PnPOnline -Url $siteUrl -UseWebLogin
 $ctHubUrl = Get-PnPContentTypePublishingHubUrl
Connect-PnPOnline -Url $ctHubUrl -UseWebLogin
$CTIds = Get-PnPContentType
$list = New-Object Collections.Generic.List[string]
foreach($id in $CTIds){
    $list.Add($id.Id.StringValue)
}
Connect-PnPOnline -Url $siteUrl -UseWebLogin
Add-PnPContentTypesFromContentTypeHub -ContentTypes $list
What is the version of the Cmdlet module you are running?
1.7.0
Which operating system/environment are you running PnP PowerShell on?

[x] Windows

@kilasuit - We are running into the same issues reported here, and also on this ticket.  I have some questions about your "do until" example.

Have you gotten past the position 248 error? If so, what was the solution?
What are you targeting with $ContentTypesMatchString?  Are you just check for specific named content types from the list of IDs?

@robertcaretta

Have you gotten past the position 248 error? If so, what was the solution?

The above code sample doesn't care if it errors or not with -ErrorAction SilentlyContinue and as I am looping and retrying until I know it's worked, I chose not to care about it erroring

What are you targeting with $ContentTypesMatchString? Are you just checking for specific named content types from the list of IDs?

I have a prebuilt string that contains the names of all of the content types I want pulled into the site like so "DocOwner|DocAuthoringTeam"

@kilasuit - Thank you for the quick response! We updated the SPO and PnP modules to the latest versions.  We tried the "do until" approach and unfortunately it just ran without success. So we terminated the command.
Here is the change Microsoft made recently regarding the content type publishing.
https://www.microsoft.com/en-us/microsoft-365/roadmap?filters=&searchterms=70795
It appears we may be in a catch-22 scenario where we can't add the content type, and until we add it the new sync process will not start.

Hi All,
I have been struggling with this issue for long time. Considering that the content type auto sync mechanism will be end by mid of feb I need to find a solution by that time. Not getting enough support from Microsoft regarding this issue. My code is as below
$ctHubUrl = Get-PnPContentTypePublishingHubUrl
Connect-PnPOnline -Url $ctHubUrl
$CTIds = Get-PnPContentType
Write-Output 'Count:' $CTIds.Count
$list = New-Object Collections.Generic.List[string]
foreach($id in $CTIds)
{
$list.Add($id.Id.StringValue)
}
Add-PnPContentTypesFromContentTypeHub -ContentTypes $list -Connection $site
Start-Sleep -s 60
$SiteContnetTypes= Get-PnPContentType -InSiteHierarchy -Connection $site
Write-Output 'Count:' $CTIds.Count
After running this code I got the error  'The type of data at position 280 is different than the one expected.' But I confirmed that the count of content types in the hub and new site are the same using above code.
To verify the same using the UI I checked the Content type gallery (/_layouts/15/SiteAdmin.aspx#/contentTypes) of the new site, I can see that all content types are added but if check the classic UI (/_layouts/15/mngctype.aspx) custom content types are not appearing there.
Is this an expected behavior?
What I want to do is, I want to add a content type to the list using site design script. Previously we were relaying on 1 hour delay to sync the content types to the new site. Now we removed it and tried the above code, but the problem is site design is not adding the content type to the list. I also tried the command Add-PnPContentTypeToList, which is also not working.
This will work if we wait for 1 hour and  classic UI is populated with all the content types. Any idea on this issue?

I also noticed that even if we don't run the above code, immediately after creating the site  Content type gallery (/_layouts/15/SiteAdmin.aspx#/contentTypes) will have all the custom content types populated by default. Not sure if this be the case once MS turn off content type sync in feb.

Side note, has anyone looked into these SPO content type replication cmdlets? There is a Get and a Set.
https://docs.microsoft.com/en-us/powershell/module/sharepoint-online/?view=sharepoint-ps
When I run the get cmdlet "Get-SPOTenantContentTypeReplicationParameters" I receive an error that "specified method is not supported".   I am using the latest version of the module too.

@robertcaretta / @dreamweaver9962 - The reason why I chose a do until loopin my sample above was that I had varience in time for it to be successful and this was the simplest way to code defensively around waiting for it with a guessed delay for it to be as I expected it to be.
@dreamweaver9962 - I'd be tempted to suggest doing this for only a few content types at a time instead of all of them as there are many that you'll have in the Content Type hub that are default anyway and do not need to be force replicated to new site collections

@kilasuit I tried with three content types, still no luck. Could you please check /_layouts/15/mngctype.aspx this link after running your code, just to confirm that the newly added content types are appearing here as well.

@dreamweaver9962 - this is running on a client's tenant where I tested this extensively before they signed off that it does work as intended and were further added to any libraries that used them.
As the older content type page /_layouts/15/mngctype.aspx are being replaced for the newer one, I can't say I ever checked that as I was more caring about my site creation code doing what I wanted it to do for this particular client.

@kilasuit As I mentioned earlier the new Content-type gallery will have all the content types even without running the code,it might be because auto-sync is still working. After adding the content types to the site are you adding the content type to the list? I'm facing the issue in that step I tried using PNP (Add-PnPContentTypeToList), Flow, and site script. All were failing because the content types are not available on the site. I think it's basically checking this path /_layouts/15/mngctype.aspx rather than the modern content type hub.

@dreamweaver9962 - like I mentioned earlier on I am doing this in a do until loop and only  once that until statement is true will my script then move on and add the ContentTypes to the Libraries as needed, all in a PowerShell script using 1.8.0 of the PnP.PowerShell module.
The process I have is clear in code but to break it down

I attempt to add the content types to the site using Add-PnPContentTypesFromContentTypeHub choosing to sliently continue on any warnings or errors

I sleep for 30 seconds
I check if the content types are available using Get-PnPContentType as the above task withing Add-PnPContentTypesFromContentTypeHub isn't synchronous but actually asynchronous which really goes against typical PowerShell scripting expectations and can take any length of minutes to complete (hence a do until loop and not a static single sleep length)

If not I loop back through the above and continually retry to add them until my check confirms that they are available due to the asynchronous nature of Add-PnPContentTypesFromContentTypeHub
-Once they are I then create the required lists & libraries

I then add the contentTypes to the correct libraries using Add-PnPContentTypeToList cmdlet

I think it's basically checking this path /_layouts/15/mngctype.aspx rather than the modern content type hub.

Sorry that's not what I have seen to be the case as this has been working on a clients prod tenant, thier test tenant and multiple other test tenants (which I no longer have access to) using code to check as detailed above and then add to lists/libraries when available.

@chaitanya-SharePoint / @robertcaretta /  @kilasuit  / @dreamweaver9962  - is this still happening in your tenants ? Maybe you can try with the latest 1.9.0 or nightly build and let us know ? Microsoft did add some methods to publish/unpublish as well as get the status of the content type. Maybe that can help you out ?
https://pnp.github.io/powershell/cmdlets/Get-PnPContentTypePublishingStatus.html
https://pnp.github.io/powershell/cmdlets/Unpublish-PnPContentType.html
https://pnp.github.io/powershell/cmdlets/Publish-PnPContentType.html

Closing as no response.


Super cool, plurals?

Very cool lib. Does it have plans for support for plurals?

Plurals feature would be great, but since iOS doesn't have native support I'm still trying to figure out a good way implement it.
I was thinks to follow Android way but using a prefix in the string:
https://developer.android.com/guide/topics/resources/string-resource.html?hl=pt-br#Plurals
"PLURALS_ZERO_MY_LABEL_NAME" = "No song found";
"PLURALS_ONE_MY_LABEL_NAME" = "%d song found"";
"PLURALS_OTHER_MY_LABEL_NAME" = "%d songs found"";

Localizable.MY_LABEL_NAME(0) // No sound found
Localizable.MY_LABEL_NAME(1) // 1 song found
Localizable.MY_LABEL_NAME(2) // 2 songs found

I'm open for any suggestion :)

Thanks for your optimism and giving hope here! I dont have experience with localizing in native, but I agree with you it totally makes sense to follow the Android way if iOS doesn't.


bantosang

bantosang created by<EMAIL_ADDRESS>on 2018-01-31 13:43:12

<EMAIL_ADDRESS>replied, what is the best english word for BANTOSANG?

<EMAIL_ADDRESS>replied, Alii Aleksandr,
bantosang is a Japanese word for cashier. store cashier.
Masaharu

<EMAIL_ADDRESS>replied, ke kmal mesaul, mr. Masaharu!another question for everyone: do young Palauans use this word nowadays, or not?

<EMAIL_ADDRESS>replied, Alii Aleksandr,
Yes but not very much. The teenagers now do not use it.
they use cashier.
Sulang,
Masaharu

<EMAIL_ADDRESS>replied, Bantosang-cashier
On Thursday, February 1, 2018 3:00 PM, Aleksandr Kuznetsov (Debugle<EMAIL_ADDRESS>wrote:

Emails/html
|
|  --- Write ABOVE THIS LINE to reply ---
Aleksandr Kuznetsov posted a new comment on bantosang :
ke kmal mesaul, mr. Masaharu!another question for everyone: do young Palauans use this word nowadays, or not? View this on DebugleStop receiving email notifications about this item.  |
|


How do I automatically reload Apache after renewal

I saw the a new certificate got issued on renewal but was not showing up online, I reloaded Apache and there it was. Is it possible to reload apache after successful cert renewal?

Sure! There's a few options to do this:

You can use Certbot's Apache plugin to install the certificate, which makes Certbot to automatically restart Apache when certificate is renewed.
You can create a custom script that restarts Apache (or other services) when the certificate is renewed. Make the script executable, and place it to /etc/letsencrypt/renewal-hooks/deploy and it gets run when a certificate on your system gets renewed.

certbot renew --deploy-hook "systemctl reload apache2" (you may need to change the command in case you are not using Debian/Ubuntu)

what if I'm using docker)))
where everything is in separate container)))


Retina sprite positions in SCSS are incorrect

In the latest version (0.9.1), I get the coordinates in my SCSS file based on the retina size, instead of the regular size of the images, with the following gulp config:
return gulp.src('app/images/sprite/*.png')
		.pipe(sprite({
			base64: true,
			style: '_base64.scss',
			processor: 'scss',
			format: 'png',
			retina: true,
			prefix: 'sprite',
			orientation: 'vertical'
		}))
		.pipe(
			gulp.dest('app/styles'));

Instead of the correct coordinates:
$logo-footer: -2px -2px 67px 29px;

I get
$logo-footer: -4px -4px 134px 58px;

Am I doing something wrong, or should all the coordinates be divided by 2 when the retain option is on?

I see this exact behavior. I downgraded to 0.9.0 and it worked. Curious to know the source of the problem and a better solution.

In my case I'm using a custom template with minor modifications to the CSS processor, FWIW.

Mh strange. The test seem to be fine and actually the code for calculating the positions didn't change. I'll look into it.

Yeah I'm not sure... I'm not doing anything too wacky but I'm happy to provide my gulp task config and previous/post output if it's helpful.

This was a nice one. Due to the update of lodash to the new 3 version something like lodash(somearray).each() doesn't work anymore. Unfortunately that failed without any error :(

Fixed in 0.9.2

Does npm need time to update? npm ERR! version not found<EMAIL_ADDRESS>

I am not sure what's going on. npm info tells me:
versions:
[ '0.4.0',
'0.5.0',
...
'0.9.0-beta2',
'0.9.0',
'0.9.1' ],
time:
{ modified: '2015-02-05T17:58:49.101Z',
created: '2014-02-09T01:05:34.326Z',
'0.4.0': '2014-02-09T01:05:34.326Z',
....
'0.9.0': '2014-11-16T19:46:44.028Z',
'0.9.1': '2015-02-05T09:20:29.862Z',
'0.9.2': '2015-02-05T17:48:48.849Z' },

Looks unpublished to me, see output from http://registry.npmjs.org/css-sprite

Mh so I guess I have to publish 0.9.3. Because I can't do it with 0.9.2 anymore.
❯ npm publish
npm ERR! publish Failed PUT 403
npm ERR! Error: forbidden Attempting to modify version 0.9.2,
npm ERR! which was previously published on 2015-02-05T17:48:48.849Z.

The package is online http://registry.npmjs.org/css-sprite/-/css-sprite-0.9.2.tgz, so I guess it's just a caching issue. I will wait till the morning and if nothing happens I republish with a new version number.

Yeah, I've seen that before with npm. You can open an issue with them,
they're usually pretty responsive.
On Thu, Feb 5, 2015 at 3:18 PM, Alexander Slansky<EMAIL_ADDRESS>wrote:

The package is online
http://registry.npmjs.org/css-sprite/-/css-sprite-0.9.2.tgz, so I guess
it's just a caching issue. I will wait till the morning and if nothing
happens I republish with a new version number.
—
Reply to this email directly or view it on GitHub
https://github.com/aslansky/css-sprite/issues/47#issuecomment-73130750.

Yeah, I opened an issue with them, because the module is also gone from my profile page on npmjs.com. Let's hope the problems gets resolved quickly.

Any news?

I published 0.9.3 a few minutes ago and it's available. So your good to go.


adds ability to translate formats that contain escape characters

https://github.com/MadMG/moment-jdateformatparser/issues/7#issuecomment-252155592

Doesn't work for escaping single quotes themselves.
E.g. "hh 'o''clock' a, zzzz" is supposed to display as "12 o'clock PM, Pacific Daylight Time"
but escaping the single quote in o'clock fails.
However, this fix will work on escaping other characters e.g. "ddHHmm'Z'MMMyyyy" will display as "062000ZMay2036"
Tests should also be added.

Thanks a lot for your contribution @arikimolloy but could you please revert all other changes which are not related to the added functionality itself? This type of PR makes it really cumbersome and in later stages nearly impossible to tell what was really added, as you basically re-arranged the whole file. 😄

My bad I think it was my formatter. Now it should only show the actually changed code.

Awesome - thanks a ton @arikimolloy 👍
LGTM - what about you @MadMG, are you ok with that?

@herom  sorry for the late repsonse.... 👍


[JaVers] wrong audited entities types returned by REST endpoint

I've enabled entity aduting on my existing JHipster project.
My entities names all ends with the Entity suffix (e.g. TimelogEntity), but the JaversEntityAuditResource seems to not take care of it.
Generated code:
    /**
     * fetches all the audited entity types
     *
     * @return
     */
    @RequestMapping(value = "/audits/entity/all",
        method = RequestMethod.GET,
        produces = MediaType.APPLICATION_JSON_VALUE)
    @PreAuthorize("hasRole(\"" + AuthoritiesConstants.ADMIN + "\")")
    public List<String> getAuditedEntities() {

      return Arrays.asList("Timelog");
    }

This results in the following exception being thrown when trying to load the change list of an entity from the UI:
java.lang.ClassNotFoundException: it.blutec.bludesk.domain.Timelog
Suggested fix: consider the presence of the suffix in the code generation process.
    /**
     * fetches all the audited entity types
     *
     * @return
     */
    @RequestMapping(value = "/audits/entity/all",
        method = RequestMethod.GET,
        produces = MediaType.APPLICATION_JSON_VALUE)
    @PreAuthorize("hasRole(\"" + AuthoritiesConstants.ADMIN + "\")")
    public List<String> getAuditedEntities() {

      return Arrays.asList("TimelogEntity");
    }

Closing, since we migrate to a new blueprint version.
If you found any issue with latest version, please open a new issue.


Make sure to add assemble task dependencies after base plugin evaluation.

#821 added support running the assemble task of project dependencies because otherwise, the project dependency JARs aren't available for jib to run. It eagerly looks for the assemble task, however, which does not work if the jib project happens to be evaluated before the dependency project. In gradle, projects are evaluated in alphabetical order, so the naming of the projects can easily cause the projects to not be evaluated in such an order.
This change ensures the task dependency is only made after BasePlugin is applied, which is when it's guaranteed for the assemble task to exist.
From what I could tell, there is no way to reproduce this evaluation order problem in a unit test since everything is evaluated eagerly. So I added an integration test that checks for regressions instead.
Fixes #815

@coollog I'm going to submit this if the style fine.

It looks like the cla check hung. I think we just need to @googlebot to rerun it?

LGTM


Can an Ariston Clas One Wifi ....

Can an Ariston Clas One Wifi be used without a thermostat and trigger it with the temperature sensors linked in HA?

Can an Ariston Clas One Wifi be used without a thermostat and trigger it with the temperature sensors linked in HA?

It is good question! Are you able using a Clas One Wifi to read the gas consumption (smq)  using this integration ?
Thanks
Marco.


Test Kubernetes auth executable is in the path

Is your enhancement related to a problem? Please describe
Kubernetes clusters from Cloud providers generally need an executable installed on the user system, to negotiate auth tokens.
It can happen that users add the path of these executables to the PATH for shell sessions, but this change is not applicable to Podman desktop.
Because of this, Podman desktop is not able to get new tokens and clusters are not accessible.
Describe the solution you'd like
It may be interesting that Podman Desktop check if the command defined for authentication is accessible from the Podman context, and if not, display a warning in the Settings > Kubernetes page
Describe alternatives you've considered
No response
Additional context
See issue https://github.com/podman-desktop/podman-desktop/issues/6517

A perfect example of this is needing the aws command to get a user token.
On MacOS users will likely install via brew: brew install awscli
This results in aws being installed at /opt/homebrew/bin/aws which is usually in the users PATH. However, it's not for Podman Desktop.
A short term fix is to give the full command location in your ~/kube/config after generating it with aws eks update-config.


Error while reading line from the server

CommunicationExceptions are thrown with the message 'Error while reading line from the server' from time to time. The Predis part of an example stack trace:
Message: Error while reading line from the server
Thrown at: /var/www/lib/redis/Predis.php:1568
Trace:
#0 /var/www/lib/redis/Predis.php(1636): Predis\Connection->onCommunicationException('Error while rea...')
#1 /var/www/lib/redis/Predis.php(724): Predis\Connection->readLine()
#2 /var/www/lib/redis/Predis.php(1577): Predis\FastResponseReader->read(Object(Predis\Connection))
#3 /var/www/lib/redis/Predis.php(1586): Predis\Connection->readResponse(Object(Predis\Commands\Auth))
#4 /var/www/lib/redis/Predis.php(225): Predis\Connection->executeCommand(Object(Predis\Commands\Auth))
#5 /var/www/lib/redis/RedisProxy.php(21): Predis\Client->__call('auth', Array)
#6 /var/www/lib/redis/RedisProxy.php(21): Predis\Client->auth('password')

We couldn't find anything in common in the affected scripts. Some errors are from a local Redis server, some from a remote one. An example that sometimes generates this error:
The script is run by Apache, browsers call it directly by URL, so there can be a lot of instances running at once. The following Redis commands are run via Predis (in this order; a, b , c and d are different tables):

AUTH
SELECT a
EXISTS
(if a condition is met) SETEX
SELECT b
RPUSH
if a condition is met

SELECT b
INCR
SELECT c
GET
SELECT d
RPUSH

The bold commands are those that have ever generated the error. The majority (99%) is on AUTH.
Environment:
$ redis-server -v
Redis server version 2.2.4 (00000000:0)

Predis version 0.6.6

$ php -v
PHP 5.3.3-0.dotdeb.1 with Suhosin-Patch (cli) (built: Oct  1 2010 08:49:29)
Copyright (c) 1997-2009 The PHP Group
Zend Engine v2.3.0, Copyright (c) 1998-2010 Zend Technologies
    with Xdebug v2.1.0, Copyright (c) 2002-2010, by Derick Rethans

$ /usr/sbin/apache2ctl -v
Server version: Apache/2.2.9 (Debian)
Server built:   Apr 20 2010 15:40:17

Abesto, sorry to bother - but we are experiencing the same issue i was wondering how did u go about find out that "The cause of the issue was a faulty Realtek driver" ??
Thanks!

@rookie7799 No problem at all. /var/log/kernel.log contained "interface up" messages at the same times that the errors occured; the link was down for some time before these messages.
Mention of the problem and a detailed solution at Hetzner: http://wiki.hetzner.de/index.php/Installation_des_r8168-Treibers/en
Sorry for taking so long to answer.

Sorry for bringing this up again after so much time, but I think I have a different reason why I get this error message. I tried debugging the stream as mentioned earlier and got this:
array(7) {
  ["stream_type"]=>
  string(14) "tcp_socket/ssl"
  ["mode"]=>
  string(2) "r+"
  ["unread_bytes"]=>
  int(0)
  ["seekable"]=>
  bool(false)
  ["timed_out"]=>
  bool(false)
  ["blocked"]=>
  bool(true)
  ["eof"]=>
  bool(false)
}

read_write_timeout is set to -1, and we use one master and one slave with replication => true. Judging by the IP address, the error is triggered on the master.
Any idea about blocking mode connection? I saw other PHP Redis library has this configurable.
We just switched from phpredis and this is a cron job, the timeout happens exactly after 60 seconds.
Thank you!

This is how I managed to capture the debug info:
          catch (Predis\CommunicationException $exception) {
                $stream = $exception->getConnection()->getResource();
                $errorMessage = var_export(stream_get_meta_data($stream), true);
                //Log::info($errorMessage);
                continue;
            }

I get the following:
array (
  'timed_out' => false,
  'blocked' => true,
  'eof' => false,
  'stream_type' => 'tcp_socket/ssl',
  'mode' => 'r+',
  'unread_bytes' => 0,
  'seekable' => false,
)

I have set read_write_timeout to -1, timeout in redis.conf to 0, second param to blpop as 0. Has anyone managed to find out more about this?

Hi @pravindahal
Thanks for posting the debug capturing code. Did you get anywhere towards a resolution?
I have a similar issue:-
array (
  'stream_type' => 'tcp_socket/ssl',
  'mode' => 'r+',
  'unread_bytes' => 0,
  'seekable' => false,
  'timed_out' => false,
  'blocked' => true,
  'eof' => false,
)

No Ben, I haven't been able to resolve this.
Ben Cromwell wrote:

Hi @pravindahal https://github.com/pravindahal
Thanks for posting the debug capturing code. Did you get anywhere
towards a resolution?
I have a similar issue:-
|array (
'stream_type' => 'tcp_socket/ssl',
'mode' => 'r+',
'unread_bytes' => 0,
'seekable' => false,
'timed_out' => false,
'blocked' => true,
'eof' => false,
)
|
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nrk/predis/issues/32#issuecomment-263821705, or
mute the thread
https://github.com/notifications/unsubscribe-auth/AA2nC7YRPNChZdEOx9c1N3pBJNbr0KW3ks5rDT5XgaJpZM4HN5ze.

I find myself back here almost a year later and feel compelled to comment.
Composer doesn't work (install/update packages) with that socket setting changed.
No idea why the package doesn't seem to respect the configuration settings and is unable to override PHP's default. Leaving this comment here in case anyone is reading this and knows the solution!

default_socket_timeout

In my case configuring Redis was not enough setting default_socket_timeout=-1 in php.ini makes it to work.
thanks @bencromwell.


Preserve % in stripped comments

LaTeX's % also comments out the newline, which has important properties in many contents (e.g. omitting whitespace between text on multiple lines). So it's much safer to keep the % and only strip the contents of the comment.

fixed


Bar-chart: show chart value on the graph

Hello, would it be possible to show the value right on the graph when displaying a bar chart?

In that case there would be no need for the y-axis and it would be great to have an option to hide it.

Sadly the history explorer card has been declared end of life and won't be developed anymore. So I will have to close this issue / FR without resolution. This repository will be archived and set to read-only shortly.
See this post on the HA forum for more information:
https://community.home-assistant.io/t/new-interactive-history-explorer-custom-card/369450/978


Configure CORS for the batching server

Context
The batching server currently does not configure CORS and by default, all requests from a browser are disabled. We want to enable CORS in order to be able to query prices from the portal frontend.
This is needed for https://github.com/pendulum-chain/portal/issues/439.
TODO

Add actix_cors middleware to the batching server.
Configure it so that it can be used to query data from the  /currencies POST endpoint.
It should at least allow the origin portal.pendulumchain.org. We can also think about enabling netlify deployments in the CORS settings.

@pendulum-chain/product this is blocking https://github.com/pendulum-chain/portal/issues/439 and in order to unblock it we should work on this very soon.

Hey team! Please add your planning poker estimate with Zenhub @b-yap @bogdanS98 @ebma @gianfra-t @TorstenStueber


double overlapping context menus (on Mac)

hey @NomarCub thank you for making this !! I have wanted this for a while.  Hope they make it a native function one day.
One thing though, not sure it's fixable... on macOS, I now get 2 menus when right clicking a link:  the system-level one and the Electron one beneath it.
see video:
https://user-images.githubusercontent.com/1992842/125866102-fa4bb5ca-b346-4c84-8f7a-6415cf143c95.mp4

I made a new release with a possible fix.

It works! 🚀


Investigate why GPT2-Large model isn't working on ORT benchmark script

Related to #2174
When running the GPT2 large model in MIGraphX everything is fine. Need to determine why this fails on the onnxrt side.
Either with MIGraphX EP, Benchmark script or some other 3d party issue.

Need to still investigate


Bitfinex ProcessTradeUpdate out of range exception

Expected Behavior

No out of range exception

Actual Behavior

Brokerage.OnMessage(): Error - Code: -1 - Parsing wss message failed. Data: [23568,[2.6316,0,1]] Exception: System.IndexOutOfRangeException: Index was outside the bounds of the array.
  at QuantConnect.Brokerages.Bitfinex.BitfinexSubscriptionManager.ProcessTradeUpdate (QuantConnect.Data.Channel channel, System.String[] entries) [0x00058] in <f9964629229b45f38eadcfc9ba8c2068>:0
  at QuantConnect.Brokerages.Bitfinex.BitfinexSubscriptionManager.OnUpdate (System.Int32 channelId, System.String[] entries) [0x0009b] in <f9964629229b45f38eadcfc9ba8c2068>:0
  at QuantConnect.Brokerages.Bitfinex.BitfinexSubscriptionManager.OnMessage (System.Object sender, QuantConnect.Brokerages.WebSocketMessage e) [0x00139] in <f9964629229b45f38eadcfc9ba8c2068>:0

Potential Solution

Review implementation avoid out of range exception

Reproducing the Problem

Believe the exception has the data in the message that produced the issue Data: [23568,[2.6316,0,1]]

System Information

N/A
Checklist

[x] I have completely filled out this template
[x] I have confirmed that this issue exists on the current master branch
[x] I have confirmed that this is not a duplicate issue by searching issues

[x] I have provided detailed steps to reproduce the issue

Very strange...
The message format is correct: [23568,[2.6316,0,1]] is an order book update but was processed as a trade 😕
Trades should only be processed here:
https://github.com/QuantConnect/Lean/blob/master/Brokerages/Bitfinex/BitfinexSubscriptionManager.cs#L383
Order book updates should only be processed here:
https://github.com/QuantConnect/Lean/blob/master/Brokerages/Bitfinex/BitfinexSubscriptionManager.cs#L410
then here:
https://github.com/QuantConnect/Lean/blob/master/Brokerages/Bitfinex/BitfinexSubscriptionManager.cs#L613
For some reason, an order book update came through the "trades" channel and was passed in here:
https://github.com/QuantConnect/Lean/blob/master/Brokerages/Bitfinex/BitfinexSubscriptionManager.cs#L617
Seems the channel id mapping was incorrect, maybe it happened after a disconnect/reconnect?
The only potential missing lock I'm seeing in the code is here:
https://github.com/QuantConnect/Lean/blob/master/Brokerages/Bitfinex/BitfinexSubscriptionManager.cs#L148


balance is zero when disconnected

Reproduce:

Disconnect the Internet connection of the phone. Run AlphaWallet.
Observe a non-zero Ether balance
Go to Settings -> My wallets
observe that the balance displayed there is 0.00 Eth.

Can confirm this is no longer an issue


Patch 1

Next time try to create an issue for a problem and let me assign you that first. You should not spam pull requests for unnecessary needs.


+++ Future goals +++

Future goals
Features:

[x] My Account page
[x] App Settings page
[ ] Events page
[x] Topics page
[x] Groups page
[x] File/Image Upload in posts
[ ] Top-Comment function
[x] Notifications
[ ] Block users
[x] Report posts
[ ] Report users

[ ] Poll function

[ ] More authentication methods (Active Directory, Auth0, ...)

[ ] Single Post Page

Please add a directory (and Sync with ActiveDirectory).  That would be so cool. This is a real niche.


Integration with Gitea

Thanks for your code. It's very good.
I was wondering how can I integrate it with Gitea's webhook ?
(Said differently, how can I trigger a pipeline from outside the admin interface via a HTTP call ?)

You can do that with the pipeline remote trigger action.
If you access your pipeline, you'll see a trigger token like this:

Use that to trigger a pipeline by calling the rest endpoint without the need for an authenticated client like this:
curl -X POST http://localhost:8080/api/v1/pipeline/2/6ffb6acf-8438-5cd7-b819-0d18e9c47b68/trigger

Uh, that was actually not working correctly. :/ Bummer.
Here is the fix: https://github.com/gaia-pipeline/gaia/pull/251
But I'm afraid that it will not be released any time soon.... so you'll have to build your own gaia to make it work....

This has been fixed and released.

Indeed, it's working with the trigger URL.

Yay!


Add rust friendly assign parameters methods

Summary
Follow up on #12794.
These commits expose some rust-friendly assign_parameters methods in CircuitData allowing us to perform parameter assignments without having call to python directly.
Details and comments
The following commits add the methods assign_paramters_from_slice and assign_parameters_from_mapping which enable the assignment of parameters in rust in a friendly way instead of using hard-Python structures (Py<_> or Bound<'_, _>. Here are their use cases:

assign_paramters_from_slice: Use whenever a collection of Param needs to be assigned. The amount of parameters sent needs to be of the same size as the the number of parameters in the CircuitData instance.
assign_parameters_from_mapping: Use whenever of mapping between Param instances from the CircuitData and Param instances coming from Gates or Operations. This can be used for partial assignment, and the number of parameters can be smaller or equal to the amount of parameters in the CircuitData instance.

Known issues

The mapping of Params : Params may not be optimal as we wouldn't know if the types of parameters we're binding are valid, Param::Float does not posses a uuid and therefore is not a valid key for the mapping.
The mapping of assign_parameters_from_mapping does not accept instances of Map<Param, Param> due to param not implementing the hash property in Rust.

Other comments

Feel free to point out any issues in the comments and I will quickly address.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>0
Details

14 of 29 (48.28%)  changed or added relevant lines in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.004%) to 89.755%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

crates/circuit/src/circuit_data.rs
14
29
48.28%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/lex.rs
2
92.73%

Totals

Change from base Build<PHONE_NUMBER>0:
0.004%

Covered Lines:
67361

Relevant Lines:
75050

💛  - Coveralls


Bootstrap 4 alpha support

Hello,
This little MR adds support for Bootstrap 4 and destructively ends support for Bootstrap 3. I've removed any less conversion and now pull the scss files directly from the 4-dev branch.
Feel free to take a look and work with the latest bootstrap 4 alpha code in your Rails application. I suggest this gets released as version 4 and then follow bootstrap versioning (since this gem only pulls data and package it and does not do any fancy stuff anymore with converting less to scss)
Still to do

[x] Removed unused test
[ ] Updated Readme to reflect the new bootstrap 4 changes
[x] Wait for upstream fix of bootstrap flex compilation (https://github.com/twbs/bootstrap/issues/17046)

Thanks for this! We haven't yet had much of an internal discussion about what the plans for this project are when v4 goes live.

Maintaining it will be a piece of cake since it only repacks the scss and js files so not many issues will pop up.
Only concern is the ever changing pace of Rails/Sprockets ofcourse :)
The real beauty is that this gem ONLY includes Bootstrap and does not import any "helpers".
Hope we keep the official gem alive!
Regards

Hi @JanStevens, thanks a lot for this work! I have started a different repo for the v4 rubygem from this branch as per the discussion in https://github.com/twbs/bootstrap/issues/17291. The new repo will be published soon. Cheers, Gleb

@glebm Can I keep pushing to this branch or wait for the new repo? Was planning on adding generators for rails so people can easily install bootstrap (or one of its variants)
Cheers!

@JanStevens Please wait for the new repo as I've made some commits as well, shouldn't be long!

Published in https://github.com/twbs/bootstrap-rubygem

Since the scope of the project has been reduced to just the rubygem, the docs will need rewriting. For now I've put a stub README.md to indicate that the repo is a work in progress.


Re-export fuels-rs as single crate

Important: This is a very critical breaking change, review this carefully.
Closes https://github.com/FuelLabs/fuels-rs/issues/164.
This PR makes necessary changes to turn fuels-rs into a single crate, instead of having to import fuels-contract, fuels-signers, fuels-core, etc.
The only exception is fuels-abigen-macro, since it's a proc_macro package, it has be separated in its own package, for a lot of very tricky reasons. Still, this feels much more ergonomic, turning:
use fuels_abigen_macro::abigen;
use fuels_contract::contract::Contract;
use fuels_contract::errors::Error;
use fuels_contract::parameters::{CallParameters, TxParameters};
use fuels_core::constants::NATIVE_ASSET_ID;
use fuels_core::Token;
use fuels_signers::util::test_helpers::{
    setup_address_and_coins, setup_test_provider, setup_test_provider_and_wallet,
};
use fuels_signers::{provider::Provider, LocalWallet};

Into:
use fuels_abigen_macro::abigen;
use fuels_rs::prelude::*;

Where the prelude exports all necessary types/functions/names to bootstrap a fuels-rs project. Users can also import things separately through fuels_rs::{...}. The point is that now everything is under fuels_rs:: and that will be the crate that will be published on crates.io. And fuels-abigen-macro for reasons mentioned above.

Reasoning for proc macro needing to be its own crate: https://stackoverflow.com/questions/56713877/why-do-proc-macros-have-to-be-defined-in-proc-macro-crate/56714256#56714256

Converting to draft so it doesn't get merged by accident.

32342e6 resolves conflicts with master. Additional changes to make it build:

update fuels-rs version to 0.8.1 to match the dependencies version on master
add core::Tokenizable and signers::Signer to re-exports

cargo t --workspace --all-features passes locally.

Shouldn't fuels-rs just be a workspace member?

Is there a circular dependency, since fuels-rs depends on fuels-abigen-macro, and fuels-abigen-macro depends on fuels-rs?

Hmm, not sure. Two reasons: I'm using ethers-rs organization as an inspiration and the way it is right now worked w.r.t shrinking the number and complexity of imports (main goal);

Yeah, that's a valid concern, but again... everything seems to be working fine? It might be because fuels-abigen-macro only depends on fuels-rs for testing (dev-dependencies instead of dependencies)?

Is it okay for something to be both a package and a workspace?

Is it okay for something to be both a package and a workspace?

I'm not sure about this. Summoning @Voxelot. Thoughts on it?

Ethers-rs shouldn't necessarily be copied blindly.

Very true. However, I don't have tons of experience with package management in Rust, especially because every couple of years it changes; I was used to the pre-2018 way of structuring/packaging, and now different projects seem to use different approaches.
Findings working references is the way I'm approaching this, I'm also completely open to suggestions. I followed (not blindly copied 😉) ethers-rs because it's both relatively easy to understand and... it works!

Alright, made another big change here https://github.com/FuelLabs/fuels-rs/pull/168/commits/be2a845474a398ad34ea7a41c37a1786397f7381. This time inspired by another great project's structure: Yew. It turns fuels-rs into a workspace-only.
Let me know whether that's better or worse (it's the same from a DevEx perspective), we can revert to the previous commit if this isn't preferable.
Now it looks like this:
├── Cargo.lock
├── Cargo.toml
├── LICENSE
├── README.md
├── packages
│   ├── fuels-abigen-macro
│   │   ├── Cargo.toml
│   │   ├── src
│   │   └── tests
│   ├── fuels-contract
│   │   ├── Cargo.toml
│   │   └── src
│   ├── fuels-core
│   │   ├── Cargo.toml
│   │   └── src
│   ├── fuels-rs
│   │   ├── Cargo.toml
│   │   └── src
│   └── fuels-signers
│       ├── Cargo.toml
│       └── src
└── tools
    └── fuels-abi-cli
        ├── Cargo.toml
        ├── examples
        └── src

I think that with the latter system, we will still need to use /packages/fuels-rs/examples/foo instead of /examples/foo in #101. We're back to using a virtual manifest, which does not support top-level examples natively :(

I think that with the latter system, we will still need to use /packages/fuels-rs/examples/foo instead of /examples/foo in #101. We're back to using a virtual manifest, which does not support top-level examples natively :(

I think it does support top-level examples? Look here https://github.com/yewstack/yew/blob/master/Cargo.toml#L10.
We would have a top-level (sibling to packages/ and tools/) called examples/ and the examples would go in there. Seems to be a working strategy for Yew.

Yes, you're right—though that would require a separate crate for each example. I was leaning towards example targets (cargo run --example foo), but workspace members are fine too, makes sense :)

I agree that example targets are preferable where that makes sense, but may not be a great fit for every usecase.
Overall I think this new packaging structure looks fine, although I did notice there's a lot of unused harness.rs files under fuels-abigen-macro/tests/test_projects/*. At first I was wondering if those should be included in the workspace testing, but it appears they are just copied and pasted artifacts from the std lib?

Should they just be .gitignored then?

Maybe not ignored, but could be manually removed if they're not relevant to the particular tests?

If they're just autogenerated files that never get run, then they should be cleaned up. Otherwise we should figure out how to include them in the workspace restructure if the plan is to include them.

Merging this just to avoid further conflicts


Update Dockerfile

The app doesn't run in Docker on base image microsoft/dotnet:latest because Microsoft.NETCore.App 1.0.1 isn't installed. Only 1.0.0 is installed. The app runs fine on base image microsoft/dotnet:1.0.0-preview2-onbuild.

Hi @korneel, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, DNFBOT;

Thanks for this!


Associate entry files to version number

When rolling new JS/CSS versions, we sometimes have to force-refresh to get the full update. Sometimes the CSS takes effect while some of the JS doesn't or vice-versa.
To prevent this we may want to version the entry files and all references to them to force a fresh load when rolling out new entry points.
@ascott  @williaster, any simpler way to achieve this?

we should be able to handle this in the webpack build process afaik.

https://github.com/airbnb/caravel/pull/1060

nice work @vera-liu !


Improve examples

Codecov Report
Base: 72.60% // Head: 72.60% // No change to project coverage :thumbsup:

Coverage data is based on head (2226c82) compared to base (ece5804).
 Consider uploading reports for the commit e61d72d to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #204   +/-   ##
=======================================
  Coverage   72.60%   72.60%
=======================================
  Files          33       33
  Lines        2241     2241
=======================================
  Hits         1627     1627
  Misses        532      532
  Partials       82       82

Impacted Files
Coverage Δ

middleware/client_ip.go
100.00% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Just a question about h265

I am currently debating converting my files over to h265 to save some disk space.  I have been experimenting doing some conversions just to see what the space saving would be.
I have done a test with a few movies and have noticed that there is no file size difference between h264 and h265 (using the manual command line for testing purposes).  But if I use this command:
ffmpeg -i input.avi -c:v libx265 -preset medium -crf 23 -c:a aac -b:a 256k -strict -2 output.mp4
I get about a 50% savings in file size.  I have tried to match the settings in the ini file, but I am wondering if there is any place else I would need to check/change.
Love the program and cheers!

i would be interested in this also.

I think this feature should be added. H265 is becoming more wide spread.

This can be achieved with the preopts and postopts settings and the h265 video-codec settings now


Fix the requirements that broke Read-The-Docs

In a recent update, the requirements_dev.txt file was moved to the more standard across my projects requirements/development.txt. However, at the time, it was not caught that this is the venv used to build the docs:
 /home/docs/checkouts/readthedocs.org/user_builds/github-commit-status/envs/latest/bin/python -m pip install --exists-action=w --no-cache-dir -r requirements_dev.txt
ERROR: Could not open requirements file: [Errno 2] No such file or directory: 'requirements_dev.txt'

Fixed with PR https://github.com/glenjarvis/github-commit-status/pull/810.


Previous Quantity Value in Aggregated Doc

Addresses Issue https://github.com/cloudfoundry-incubator/cf-abacus/issues/385

Coverage decreased (-3.4%) to 87.861% when pulling 4b13743456300ef378e35c31647f88223378fda0 on rajkiranrbala:PreviousValueAggr into 0e9a649ed27c6418af470c9145182773989fdc79 on cloudfoundry-incubator:master.


Add viewer and editor roles for TFJob to rbac

What type of PR is this?
/kind bug
What this PR does / why we need it:
I forgot to add those roles to rbac in https://github.com/kubernetes-sigs/kueue/pull/1068.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

Add viewer and editor roles for TFJob to rbac

/release-note-none
Not user facing changes.

/release-note-none
Not user facing changes.

Updated release note.


[security] prevent XSS on FAB list views

@jefffeng @ascott @bkyryliuk

Looks good, just a minor comment, looks like return self.druid_datasource.link should stay.

LGTM


Changelog for 9.3.0 release

We need to make a 9.3.0 release soon to get it into Ubuntu 20.10 (#367). Here's what's changed since 9.2.0:

https://github.com/osrf/sdformat/compare/sdformat9_9.2.0...sdf9?expand=1

Codecov Report

Merging #368 into sdf9 will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             sdf9     #368   +/-   ##
=======================================
  Coverage   86.27%   86.27%
=======================================
  Files          59       59
  Lines        9177     9177
=======================================
  Hits         7917     7917
  Misses       1260     1260

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 94055d1...1031831. Read the comment docs.


eth_getBlockByNumber issue causing errors and multiple retries

It seems that a request come for eth_getBlockByNumber with a param of latest, we can see that the block number is 8, then the next request for eth_getBlockByNumber with a param of 9! it errors out because this block does not yet exist.  Is there an increment error here or another yet to be parsed block?

Truffle comments on running tests:

2) Contract: Test
      "after each" hook: after test for "Invalid token amounts":
    blocktag should be number | 'latest' | 'earliest' (argument="blockTag", value="0x6d", code=INVALID_ARGUMENT, version=bodhi.js/providers/2.1.4)

A count of this error found in running a test suite occurs more than 2700 times.

Under WH project, I'm still getting this error 30970 times.
[1638318896612] DEBUG (1780071 on XPS-15-7590): incoming request
    id: 36100
    jsonrpc: "2.0"
    params: [
      "0x22a",
      false
    ]
    method: "eth_getBlockByNumber"
[1638318896613] ERROR (1780071 on XPS-15-7590): request error
    methodName: "eth_getBlockByNumber"
    params: [
      "0x22a",
      false
    ]
    err: {
      "type": "Error",
      "message": "Unable to retrieve header and parent from supplied hash",
      "stack":
          Error: Unable to retrieve header and parent from supplied hash
              at assert (/home/bodhi.js/common/temp/node_modules/.pnpm/@polkadot+util@7.9.2/node_modules/@polkadot/util/assert.cjs:36:11)
              at ApiPromise._getBlockRegistry (/home/bodhi.js/common/temp/node_modules/.pnpm/@polkadot+api@6.9.2/node_modules/@polkadot/api/base/Init.cjs:227:22)
              at processTicksAndRejections (internal/process/task_queues.js:95:5)
    }
[1638318896613] DEBUG (1780071 on XPS-15-7590): request completed
    id: 36100
    jsonrpc: "2.0"
    error: {
      "code": 6969,
      "message": "Error: Unable to retrieve header and parent from supplied hash"
    }

Under WH project, I'm still getting this error 30970 times.
[1638318896612] DEBUG (1780071 on XPS-15-7590): incoming request
    id: 36100
    jsonrpc: "2.0"
    params: [
      "0x22a",
      false
    ]
    method: "eth_getBlockByNumber"
[1638318896613] ERROR (1780071 on XPS-15-7590): request error
    methodName: "eth_getBlockByNumber"
    params: [
      "0x22a",
      false
    ]
    err: {
      "type": "Error",
      "message": "Unable to retrieve header and parent from supplied hash",
      "stack":
          Error: Unable to retrieve header and parent from supplied hash
              at assert (/home/bodhi.js/common/temp/node_modules/.pnpm/@polkadot+util@7.9.2/node_modules/@polkadot/util/assert.cjs:36:11)
              at ApiPromise._getBlockRegistry (/home/bodhi.js/common/temp/node_modules/.pnpm/@polkadot+api@6.9.2/node_modules/@polkadot/api/base/Init.cjs:227:22)
              at processTicksAndRejections (internal/process/task_queues.js:95:5)
    }
[1638318896613] DEBUG (1780071 on XPS-15-7590): request completed
    id: 36100
    jsonrpc: "2.0"
    error: {
      "code": 6969,
      "message": "Error: Unable to retrieve header and parent from supplied hash"
    }

How to reproduce？

WH project is this one https://github.com/certusone/wormhole/tree/dev.v2/ethereum

after yarn we need to add mandala network settings like here https://github.com/AcalaNetwork/bodhi.js/blob/master/examples/truffle/truffle-config.js
then
yarn migrate --network mandala

Also for wormhole, you'll need to run this at least once after running truffle compile
mkdir -p build/contracts && cp node_modules/@openzeppelin/contracts/build/contracts/* build/contracts/

The test cases for the wormhole contract all pass with the latest code (set to 1 second per block).


Improve Error Response

Improve Error Response
Expected behavior
Error thrown with JSON error response and headers
Actual behavior
At the moment an error is thrown with just this info response.status, response.statusText like in #15
Steps to reproduce the behavior
Receive an status not in the range 200-299

What


Decouple the code of next in azure-cli-core

In order to decouple the code of az next in azure-cli-core and reduce the impact on the performance and stability of CLI，I set up this issue to track some items that need to be improved:

Register those methods (log_cmd_history,log_latest_error_info,clean_exception_history) in utils as events to move them into next extension
Use environment variables AZURE_CORE_USE_COMMAND_INDEX and reload command table to skip command index.
Call log_latest_error_info in a more appropriate way

Related PR: #15964

Checkbox syntax can be a very good helper 😉:
- [ ] Register those methods (`log_cmd_history`,`log_latest_error_info`,`clean_exception_history`) in utils as events to move them into `next` extension

Core


v0.5

v0.5 branch
This branch proposes a new functionality, subset.
The subset argument allows a user to only run the root-finding procedure on a subset of the provided parameters. This feature is intended for the use of layered estimating equations, where pieces can be optimized outside (or pre-washed) of the stacked estimating equations. An example would be the propensity score for the IPW mean.

Items to fix in RTD

[ ] Estimating equations in functions for log-logistic models
[ ] Bridge penalty should be sgn and is missing lambda

After a bit more exploring, I figured out one trick to reduce the runtime (convert to np.array instead of a list). However, a benefit doesn't become apparent until there are lots of additional parameters. Before that point, subset is slower (which I think is fine for the actual use-cases of subset)


Need to add dependcy analysis

Need to add dependency analysis plugin like was done in apache/fluo#435

This will help avoid issues like #91


GDScript 2.0: Error when a node name is an integer and you try to fetch it via a script

Godot version
4.0.dev 15062513c01a835f59d9c905e5e65a7002d6c370
System information
Linux binogure 5.14.0-2-amd64 #1 SMP Debian 5.14.9-2 (2021-10-03) x86_64 GNU/Linux
Issue description
GDScript raises an error when a node name is an integer and you try to fetch it via a script

Steps to reproduce
Create a scene, and add any type of node as a child, then name that node 1, attach a script to the main node, and try the following code:
func _ready:
  $1.get_name()

Minimal reproduction project
No response

This seems intentional to me. There's a specific case in the parser to exclude literals (which I think just means 'identifiers' interpreted as ints?), and literals aren't in the list of types that work in a NodePath. Should this just be adapted to not exclude literals (and add them to the accepted list), or is there an important reason for this behaviour that means a special case should be added for some types of literals?

It works if you use a string or get_node instead:
$"1".get_name()           # OK
get_node("1").get_name()  # OK
$1.get_name()             # Error

Since $1 is supposed to be a shorthand for get_node, I'd say this is a bug.

Good point, I agree there isn't an obvious reason this shouldn't be allowed. There's no danger of the user's intention being something else or the literal getting interpreted wrongly, since the number would always be placed as part of a / separated node path after $. I'll make a pull request on changing this, and see if it works out or there are any objections.

Turns out it's (at least partly) because a . placed after an integer is interpreted as a decimal point, which obviously causes problems in parsing the code. Since this is a more fundamental limitation than I realised, I will have a look at it again later since there's quite a lot of other work to get done right now (;

This isn't really a bug IMO, and indeed it works with $"1".name so it's not a big deal.
But maybe support could be added for it if anyone wants to have a look.

I aknowledge this might be a simple corner case (there are more trickier corner cases for sure, #42410 #72973 #73273 ). With more GDScript user you will encounter more corner cases such this one.
Fixing all the corner cases will take a lot of time, but at the end it will make GDScript rock solid.


"mixed|null" throw error

PHPCS expects a type hint if we use "mixed|null" as annotation:
/**
 * Gets argument
 *
 * @param string $key
 *
 * @return mixed|null
 */
public function getArgument(string $key);

Message:

25 | ERROR | Function/Method getArgument does not have a return type.

I think with PHP8 it isn't a problem anymore. Now we can set "mixed" as type hint.

I think with PHP8 it isn't a problem anymore. Now we can set "mixed" as type hint.


Randomize order of members shown on page

Is your feature request related to a problem? Please describe.
Display the members in a random order when a user goes to the members page.
Describe the solution you'd like
Currently users are being displayed in alphabetical order.
Goal: So all members get a chance to be showcased on the front page.
Describe alternatives you've considered
Additional context
This feature will be supplemented when the search bar is implemented.

@latinadeveloper We already have randomized member ordering on the homepage (/) so members do get a chance to be showcased on the primary page.
On the members page (/members), we do sort by first name because there's a lot of people being displayed. Once we do have search and filter there, it will be a lot nicer though.


Move Panel to Bottom only works if the panel doesn't have the focus

Type: Bug

View: Toggle terminal (to open the terminal panel)
View: Toggle maximized panel
View: Move panel to bottom

You should see that nothing happens because you had the focus inside of the terminal.
However, if you try to execute View: Toggle maximised panel you will see that it works even when the focus is in the terminal.
Workaround:

View: Toggle terminal (to open the terminal panel)
View: Toggle maximized panel
Click on the top bar wehre OUTPUT | DEBUG CONSOLE | PORTS | TERMINAL is displayed to force lose the focus of the terminal
View: Move panel to bottom now the panel return at its original place at the bottom

https://github.com/microsoft/vscode/assets/4760796/48c029b8-c564-43d2-9893-34c276266286
I would expect View: Move panel to bottom to works independently if the panel as the focus or not like View: Toggle maximised panel works.
I was able to reproduce it using VSCode insiders without any extensions installed.
 Use the View: Toggle maximized panel to unmaximize the panel.
There seems to be a focus bug which I have fixed


[AIClock] Added clkinfos

This PR adds clkinfo to the AIClock. As I prefer touch events rather then gestures (which I use for pattern launch) I implemented it classical and not via addInteractive.
I also updated some "base" icons which are currently used by BWClock and AIClock as the coloring was for some reasons not correct -- now it seems to work fine for both clocks.

the coloring was for some reasons not correct -- now it seems to work fine for both clocks.

I suspect it had to do with this (comment)?

Yea true I simply copied things from BW -- now it should be fine :)

Looks great, thanks for tweaking those icons too!
If you're finding that you're duplicating that clkinfo navigation code for your clocks then we could always add that to the library as well? I guess at some point we could look at making the interaction mode configurable


[QUESTION] How do you instance an entire collection?

State your question below
Is it possible to instance an entire collection of Atoms variables (including all the Atoms variables in the dictionary)? It doesn't seem to be the case and I'm struggling to find a way to send over multiple Atoms variables (in my cabled data transfer sim).
Add any additional information / background
I'm trying to create a simulation where objects that are physically connected can exchange data via a transmitter and receiver. I use Atoms collections to specify a data template, but the problem is, if I want to reuse the same collection (i.e. two similar devices sending data in the same format) I need to be able to make an instance of the entire collection.

First you want to instance the Atom Variables and Collections in runtime or in editor time?
For editor time (like, just to automate the process), you can use unity methods to create the assets for you. (Its possible to do it in runtime, but its trickier.
You can load your collection using: AssetDatabase.LoadAssetAtPath and then add elements to it. Be aware of using AtomsVariables with collections, as internaly, the key is the value (string) of the StringVariable at the time it adds on the collection, not the atomvariable itself.  There is also a method to create objs with AssetDatabase.CreateAsset
I'm not a maintainer of the unity-atoms, but I suggest you reach the discord channel to discuss.

I think @Kyrw is quite a complete answer to the question above. @quezdk was your question answered in the discord as well or solved already?

closed because of inactivity


chore: update repo data

Updated the repo name as defined in the usage for installing the module.
Also added a badge for made by ChainSafe.
It would be great that you release an initial version of this module in npm after this commit.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #24   +/-   ##
=======================================
  Coverage   83.22%   83.22%
=======================================
  Files           8        8
  Lines         453      453
=======================================
  Hits          377      377
  Misses         76       76

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 b7094c7...248da43. Read the comment docs.


Replace node-html-parser with Cheerio

The node-html-parser library seems to have trouble parsing some of our docs, so this PR switches to Cheerio so we can evaluate that instead.

nice, i see the missing page in the content.json is in the preview now.


TypeError: Cannot read property 'pit' of undefined

/Users/username/Documents/SourceTree/MorScout/server/server.js:25
 = readJSON("dataPoints.json")[(entry.meta.match == "pit") ? "pit" : "match"];
                                                                     ^
TypeError: Cannot read property 'pit' of undefined
    at validEntry (/Users/username/Documents/SourceTree/MorScout/server/server.js:25:79)
    at IncomingMessage.<anonymous> (/Users/username/Documents/SourceTree/MorScout/server/server.js:172:11)
    at IncomingMessage.emit (events.js:95:17)
    at IncomingMessage.<anonymous> (_stream_readable.js:764:14)
    at IncomingMessage.emit (events.js:92:17)
    at emitReadable_ (_stream_readable.js:426:10)
    at emitReadable (_stream_readable.js:422:5)
    at readableAddChunk (_stream_readable.js:165:9)
    at IncomingMessage.Readable.push (_stream_readable.js:127:10)
    at HTTPParser.parserOnBody [as onBody] (http.js:142:22)

I think I found the problem.  "server/dataPoints.json" needs a comma at the end of line 9.

Thank you!


Optimize experimental Kafka scaler and fix consumer group logic

TL;DR:

Only request metadata on the topics needed
Fix the logic to fetch topics from the consumer groups

I know we decided to implement the IAM auth in the sarama based scaler (the original) but there is still an optimization that would serve current users of the experimental scaler and potentially lower their Keda load and their broker.
Firstly, the Metadata request is not scoped for the requested topics (which can be empty):
https://github.com/kedacore/keda/blob/bcaf5c07e785e3e58e3be4e3707b518fdef6acde/pkg/scalers/apache_kafka_scaler.go#L425-L427
If the list of topics is empty, we enter the branch to detect the topics & permissions based on the consumer group activity here:
https://github.com/kedacore/keda/blob/bcaf5c07e785e3e58e3be4e3707b518fdef6acde/pkg/scalers/apache_kafka_scaler.go#L441C3-L441C14
On my AWS MSK cluster, the version for the response is not supported by the segment-io library which causes the following to error out, as seen in https://github.com/segmentio/kafka-go/issues/1212 (probably):
Got non-zero number of bytes remaining: 10

Alright, for the case of debugging, let's ignore the error. What happens next? From what I can see, this variable describeGrp is unused, and because the MetadataRequest is empty, the whole list of topics is processed and returned with their partitions. All the time. Which then cause Keda to request all consumer & producer offsets again and again, etc...
So to me, currently the behavior is buggy. The describeGrp should be processed to extract the list of topics and partitions for the consumer group which I also correct in this PR.
Checklist

[ ] When introducing a new scaler, I agree with the scaling governance policy
[ ] I have verified that my change is according to the deprecations & breaking changes policy
[ ] Tests have been added
[ ] Changelog has been updated and is aligned with our changelog requirements
[ ] A PR is opened to update our Helm chart (repo) (if applicable, ie. when deployment manifests are modified)
[ ] A PR is opened to update the documentation on (repo) (if applicable)
[ ] Commits are signed with Developer Certificate of Origin (DCO - learn more)

Fixes #

Relates to #5531

@dttung2905 thanks you for the review! I will look into adding unit tests, the issue being that this lib does not have an interface to create mock on. I will investigate and see what's possible to do.

any update on this, please?


[KeyVault] update autorest generated code from 7.0-preview spec

Codecov Report

Merging #1882 into keyvault_1.0_preview will increase coverage by 0.01%.
The diff coverage is 83.11%.

@@                   Coverage Diff                    @@
##           keyvault_1.0_preview    #1882      +/-   ##
========================================================
+ Coverage                 52.71%   52.73%   +0.01%
========================================================
  Files                      4645     4645
  Lines                    114709   114784      +75
========================================================
+ Hits                      60471    60533      +62
- Misses                    54238    54251      +13

Impacted Files
Coverage Δ

azure-keyvault/azure/keyvault/key_vault_client.py
77.61% <ø> (ø)
:arrow_up:

...keyvault/azure/keyvault/models/deleted_key_item.py
100% <ø> (ø)
:arrow_up:

...lt/azure/keyvault/models/certificate_attributes.py
100% <ø> (ø)
:arrow_up:

...eyvault/azure/keyvault/models/secret_item_paged.py
100% <ø> (ø)
:arrow_up:

...lt/azure/keyvault/models/deleted_key_item_paged.py
100% <ø> (ø)
:arrow_up:

...eyvault/azure/keyvault/models/secret_attributes.py
100% <ø> (ø)
:arrow_up:

...e-keyvault/azure/keyvault/models/key_attributes.py
100% <ø> (ø)
:arrow_up:

...zure/keyvault/models/storage_account_item_paged.py
80% <ø> (ø)
:arrow_up:

.../azure/keyvault/models/deleted_certificate_item.py
100% <ø> (ø)
:arrow_up:

...e-keyvault/azure/keyvault/models/key_item_paged.py
100% <ø> (ø)
:arrow_up:

... and 73 more

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 e5ddcee...62ce518. Read the comment docs.

This was generated from the 2016-10-01 spec, not the 7.0-preview.  Closing


Files with the .vla extension trigger the program to open

This feature requires administrative rights, perhaps this should be moved to the installer, rather than the runtime.

Clearing from milestone for now.


Adjust Constraint

Maybe this is a simpler way to fix #16.

It's cool! Thanks~ 😃

Great! Thanks again! :+1:


tsify compilation errors prevent bundle from being built

I am using karma-browserify and the tsify plugin.
I have noticed that when I run the tests and there is a typescript compilation error, the bundle is never created and the tests won't be run. When there are no compilation errors the bundle is created and the tests run.
Whilst it should be encouraged to not have compilation errors, I've run into cases where I would not like my tests to not be executed just because there is a compilation error. Therefore, I would like to have an option in the settings, which allow me to bundle my typescript despite there being compilation errors.
I have quickly looked into the code I could discover the following:

tsify fires error event, which is then
caught by bro.js.

From there onwards, I have hard time following the events and different internal states. If you could point me to the right direction, I can try to fix it myself. This is, assuming it is actually in the responsibility of your plugin.

Could you try to build an integration test for the problem you describe and provide that as a PR? Doing that is pretty straight forward.
With that in place we can give you further assistance on what may go awry.

of course, sounds like a plan. Let me come back to you.

Thanks for the PR.
I do not quite understand how there could be valid JavaScript code compiled as the result of a compilation error.
How should that work?

Not including typing files for JavaScript dependencies which live on the global scope will mean that TypeScript at compile time fails as it does not know about the existence of those libraries. But when things are bundled the actual source code ends up in the bundle, making the code sound at run time. TypeScript declaration are merely a tool to provide developers with static analysis at build time. They do not and cannot infer about the soundness at runtime.
We have the case that we are monkey patching some "incompletely typed" JavaScript of angular2. Doing so we write valid JavaScript that works, but because of the lack of typing does not "compile" without warnings.
Am I making sense?

The problem here is the distinction between:

tsify fires error event

and

the lack of typing does not "compile" without warnings

An error is an error. What you really need here is a way to tell TypeScript that you know about those errors and they shouldn't be raised. There's nothing preventing you from easily implementing a wrapper to do this.
I don't really know much about the TS compiler but given the info in this thread I'm 👎  on karma-browserify changing to accommodate this scenario.

What you really need here is a way to tell TypeScript that you know about those errors and they shouldn't be raised. There's nothing preventing you from easily implementing a wrapper to do this.

The TypeScript compile does not provide a way to ignore errors unfortunately. But could you elaborate on how to implement a wrapper?
I can see where you are coming from - I still think it is suboptimal behaviour to prevent the tests from running due to failures in the static analysis of TS code. I do not think that much would be lost by simply running them despite the compile errors; perhaps when a flag is activated.

The TypeScript compile does not provide a way to ignore errors unfortunately. But could you elaborate on how to implement a wrapper?

It'd be kind of a pain. You'd have to get really hacky and patch an .emit function where the error is being emitted. I don't really know enough about TypeScript to tell you.
Based on the discussion here I'm going to close this out. I know it sucks to be stuck right now but I don't see a way to generalize here.

Thanks guys for sorting that out.
@bengro Based on the discussion I feel time is better invested to request that feature in tsify or the typescript compiler.


Error in changelog for PR #42

The following message was added to "0.2.1" section instead of "unrealeased" section: "Versioned datasets artifacts logging are handled correctly (#41)". Can you add a patch @akruszewski please?

@Galileo-Galilei Sure, will do.


Updates to switch to the new AWS account

The S3 bucket becomes a configuration parameter, and the docker images are in a different place.
I'll update the config file in the wiki.

Thanks, I updated the config files in the wiki. You'll need to update ~/.kube/config, the ~/.aws/credentials file, and the config file here.
@s-jse, @Mehrad0711, are we good here? Did you take note?


M3 976 Imagize Disk

Add option to Imagize Disk from Linode > Settings > Disks
Re-uses the ImagesDrawer with a new mode "imagize"

Coverage decreased (-0.06%) to 27.416% when pulling d731e4243beeaa3b6f9e69d6a01ad681efa31ed9 on asauber:M3-976-imagize-disk into a0fb33371490d5a29f2a1d796d9c6dd73d2cb93d on linode:develop.


Enable precomputed similarity matrices for clustering

Hi, I just opened PR #87. Context is that I had a precomputed distance matrix that I wanted to cluster with. I used T-SNE to scale it down to two dimensions for the filter function, but I didn't see a way to use metric='precomputed' with DBSCAN for the clustering with inverse_X because of the hypercube slicing making things un-square. So this PR is what I did to tell mapper to give a square matrix to the clusterer.

Thanks for working on this!
The distance matrix stuff hadn't been working for a while.  Once the unit tests are passing and there are some new ones in place, I'll merge this is in.

#87 has been merged. Thank you!


Example of label transfer with saturn?

Hi @Yanay1 ,
Is there an example in one of the vignettes for how to do label transfer using SATURN?
Thanks!
Dario

To do label transfer there are a variety of off the shelf methods you can use. One that we use for scoring is to use sklearn logistic regression classifier to transfer labels from one specie to the other.
In that case, the features (X) would be the SATURN embeddings and the labels (Y) would be the cell types. The training data would be the embeddings from one species, and you would apply it to the embeddings from the other.
Other approaches like using clustering or nearest centroid could also work.


fixes #2831, circular dependency issue

My approach was to track what schemas, based off of location, has already been compared. This  makes it where we can complete the DIFF on a specs that have circular references as expected.

Gahh, my bad, removed!


How to get this to work in lightline

Was following some of the suggestions here https://github.com/w0rp/ale/issues/279
In .vimrc, I have
let g:ale_statusline_format = ['⨉ %d', '⚠ %d', '⬥ ok']

let g:lightline = {
\    'colorscheme': 'powerline',
\    'active': {
\      'left': [ [ 'mode',  'ale' ], [ 'filename' ], ['modified'] ],
\      'right': []
\    },
\    'inactive': {
\      'left': [ [ 'filename' ], ['modified'] ],
\      'right': []
\    },
\    'mode_map': {
\      'n' : '[n]',
\      'i' : '{i}',
\      'R' : '/r/',
\      'v' : '(v)',
\      'V' : '(v)',
\      "\<C-v>": '(v)',
\      's' : '|s|',
\      'S' : '|s|',
\      "\<C-s>": '|s|'
\    },
\    'component_expand': {
\      'ale': 'LightLineAle'
\    },
\    'component_function': {
\      'filename': 'RelativePathToFilename'
\    }
\  }

function! LightLineAle()
  return winwidth(0) > 90 && &ft !~? s:except_ft && exists("*ALEGetStatusLine") ?
    \ ALEGetStatusLine() : ''
  endif
endfunction

augroup UpdateAleLightLine
  autocmd!
  autocmd User ALELint call lightline#update()
augroup END

I cant seem to see the ale status in lightline. Any feedback would be appreciated

I don't know much about lightline configuration myself, but I'm sure there will be another user who might know more. I'll have a look into this eventually if no one else is sure either.

I'll recommend using the other function for getting the status that ALEGetStatusLine uses. I'd like to remove that function eventually.

Which function would that be?

Btw, seems to work without the if condition. i.e. just having return ALEGetStatusLine()

FYI, here is my setting for Lightline + ALE. https://github.com/delphinus/lightline-delphinus
I want to show status in different color in OK or Warning or Error, and so I should have written a bit complex functions to achieve that.

@krzkrzkrz This is how that function is defined:
function! ALEGetStatusLine() abort
    return ale#statusline#Status()
endfunction

I recommend calling ale#statusline#Status() instead. I probably won't remove the former function for a long time, but I will remove it eventually. It's just there for supporting old vimrc configurations from people.

I just got this to work with help from the author of lightline: https://github.com/itchyny/lightline.vim/issues/236

Cool. :+1:


Add set of tests to unjsonify all "extra" values from CKAN package

[tests,package,#19][s]: Add tests to unjsonify extras

test_unjsonify_all_extra_values_in_nested_dicts
test_unjsonify_all_extra_values_in_nested_lists
test_unjsonify_all_extra_values_in_nested_mixed_types

This makes sure that everything gets unjsonified at every nested level with different types (dict, list and a mix of both).

I went ahead and made some refactoring changes. While I was there, I fixed those pesky warnings from Pylama to make the build pass again: https://github.com/frictionlessdata/frictionless-ckan-mapper/pull/28


Enable specific history update announcements (WS)

Right now there's a generic history-update announcement and a live-update announcement.
We need a specific announcement for every historyId. Clients should listen no more for live-updates but for the historyId they're currently set to receive.
Phase out the live-update, as it would be gobbled up by the above.

Closing as no longer relevant


Error 0x80072746

Using: C:\Program Files\WindowsApps\yuk7.archwsl_<IP_ADDRESS>_x64__35zwpb4sx6e50\rootfs.tar.gz
Installing...
ERR: Uma ligação existente foi forçada a fechar pelo anfitrião remoto.
HRESULT: 0x80072746
Press enter to exit...

How to fix it?

This is most likely because your install is placed under C:\Program Files
The ArchWSL documentation says "For example, C:\Program Files cannot be used since the rootfs cannot be modified there."
More on that here

I'd recommend placing your install at the root of your C drive or somewhere else. I just made a folder at C:\Arch\ and put everything in there

Oh, thank u


No subtitles found

I cannot download any subtitles ;/ Everytime is no English subtitles found,
my config file: https://pastebin.com/8xcC4ngp
Debug: https://pastebin.com/W7bdRyeh

Here's a few quick ideas:

Fix your mpv.conf: Remove the unparseable extra characters: : jpn, jp, en.
Can't load unknown script: /home/shirobachi/.config/mpv/scripts/.autosub.lua.swp: Rename the script to the more conventional autosub.lua.
Comment out the provider line again, to check if it's related to that.

What I changed?

empty mpv.conf (i was coppied from some tutorial Ill have fun w/ conf another time)
file .autosub.lua.swp was probably made by vim (terminal editor) I just deleted this file not in script dir is ONLY file with script what is that repo about
commented credentials to providers
Now I do not see the error, just after some time like 15 second test searching subs dissapear (subs not dl)

My conf: https://pastebin.com/pCAgdYER
Debug output: https://pastebin.com/c9XCXtm8
Debug after uncomment provider: https://pastebin.com/R8vQSZKM
Thanks for help! <3

Remove the log(a) you added on line 102.
a is a table data structure, and cannot be passed to a function that requires a string.

Oh right, I forgot ;/
Debug: https://pastebin.com/RNNSuTx3
conf: https://pastebin.com/U0f0w88K

The script is working as it should now. As you can see in the debug output, the problem lies with subliminal.
Like I suggested before, try different setups, e.g. without the provider line, with other video files, etc. And stop vim from creating swap files where mpv will try to pick them up.
I recommend fixing subliminal first by running without the autosub script, and reaching out to the subliminal folks if you can't get it to work.

Okay, thank you so much <3

Just wanted to inform that I resolved problem, it was my password with some illegal characters. Now I using only chars a-Z0-9 and it work perfectlly :)

Glad you got it working. This is exactly why I suggested running without the provider line; a less frequently used feature like this is the more obvious place where things could go wrong.
Out of curiosity, did this password work on the Opensubtitles website itself? Do they allow some rather uncommon Unicode characters in their password field?

Yes I logged in to Opensubtitles by browser.
At first I tried with no providers but it didn't work probably by my fault :D
Anyway thanks for your work <3

Happy to help.
Anyways, if you can confirm there are specific passwords that do not work with this script, but do work with the subliminal command line tool (without script), I can consider passing the password as unicode to subliminal. Since this is very much an edge case, I won't bother doing this unless someone specifically requests it.

TBH I cannot confirm for sure, but while I was struggling with the script you suggested to try use it with no script first because debug was saying it's problem on subliminal site. That time I had problem until I changed password that's why I think that problem with reading password is on subminal site or even provider.
But you know I can be wrong

This worked for me https://github.com/Diaoul/subliminal/issues/1044#issuecomment-1015226261


Commit da03e4e breaks tesserocr compilation with header files

Related issue on Tesserocr: here
Environment

Tesseract Version: 4.00alpha
Commit Number: latest as of 2017-07-19
Platform: Ubuntu 16.04

Current Behavior:
Tesseract header files since commit da03e4e added the use of string in the unichar.h header file that is used during compilation by tesserocr. This causes the compilation of tesserocr to throw an error because the std library is not imported.
In file included from tesserocr.cpp:484:0:
/usr/local/include/tesseract/unichar.h:164:10: error: ‘string’ does not name a type
static string UTF32ToUTF8(const std::vector<char32>& str32);
          ^

Changing static string UTF32ToUTF8(const std::vector<char32>& str32); to static std::string UTF32ToUTF8(const std::vector<char32>& str32); in /usr/local/include/tesseract/unichar.h:164 fixes the problem for me.
I am aware that you do not provide support for product that aren't the command-line Tesseract but I thought you might want to know since it worked before the commit.
Expected Behavior:
Hopefully make it work again, if that's not an option, confirm that this is expected behavior.
Reproducing the error

Compile Leptonica from source
Compile Tesseract from source (I used --disable-openmp if that matters)
Install tesserocr with pip3 install tesserocr (obviously you need pip3 installed)

Try this code before including Tesseract header files:
#include <string>
using std::string;

According to @theraysmith, the Google coding standards require string instead of std::string.

Cant we do something in our side which Ray will be okay with?

That's my question too, while I do understand the desire to respect Google's coding guidelines, this probably won't bite only Tesserocr users...

The problem is that -DUSING_STD_NAMESPACE is now needed for all of the tesseract library, not just for training. I fixed it in the Makefile.am files, but corresponding fixes are needed for windows and other non-autotools platforms.

@theraysmith Is it? Because I had that problem about 3 hours ago on Ubuntu. Tesseract does build, it's the modules built with its header files that doesn't.
I'll try adding -DUSING_STD_NAMESPACE and report back.

The same define will be required for anything that uses the library too, as
the define causes a using std::string to be declared in platform.h, which
is exactly what you are missing.
On Wed, Jul 19, 2017 at 3:14 PM, Edouard Belval<EMAIL_ADDRESS>wrote:

@theraysmith https://github.com/theraysmith Is it? Because I had that
problem about 3 hours ago on Ubuntu. Tesseract does build, it's the modules
built with its header files that doesn't.
I'll try adding -DUSING_STD_NAMESPACE and report back.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tesseract-ocr/tesseract/issues/1045#issuecomment-316534228,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AL056Wm5F3tHXCEBRfmSncrNj3xLaO1Jks5sPn-tgaJpZM4OdRN0
.

--
Ray.

The same define will be required for anything that uses the library too,

We need to document that requirement somewhere.

It looks like USING_STD_NAMESPACE must always be defined. I wonder why the Tesseract code does not add using std::string unconditionally (making USING_STD_NAMESPACE unnecessary).

Ray already fixed the issue with USING_STD_NAMESPACE. @Belval, please try the latest version from git.

Indeed the issue is fixed (at least for tesserocr). Thank you!

Problem is back, clean pull and build from master causes the same error described in the issue.

There is probably an include of platform.h needed somewhere.
Can you give us the compiler error?

@Belval, are you using cmake and do you see the problem since commit c67c2e9f416e0b41b5e055e67ba5281e04dd5e6c (but not before that commit)?

@Belval? Please answer to the 2 questions.

@amitdo Sorry, issue fell off my radar because of another problem.
@theraysmith It's the exact same compiler error that was in the original issue.
@stweil  According to this issue the last "good" version was this commit. I did not try anything else than the latest version. Will report back with the results.

Quoting my comment from here:

I installed the latest pull from the master branch. These changes were added in unichar.h on 2017-07-14 (tesseract-ocr/tesseract@da03e4e9105b6262706d40ef2b4436eae4ebe19f) and unicharset.h on 2017-07-24 (tesseract-ocr/tesseract@b0ead95d64a3667339775b2f99ac37e97e90c2a0).

Setting USING_STD_NAMESPACE didn't make a difference (same error message).
I had to manually replace string with std::string in both unichar.h and unicharset.h to get it to compile properly.

The correct macro which must be defined is USE_STD_NAMESPACE.

@stweil I'll try it. If it works you might want to edit your comment above to make sure that no one else will make that mistake.

-DUSE_STD_NAMESPACE works 👍

@sirfz Can confirm, I built it successfully too.

I am finding same issue in tesseract fb359fc

#include <string>
using std::string;

before tesseract's includes works again for current master ebbfc3ae8df8 (thanks to @stweil)

I had to patch string -> std::string in one of the headers for our Debian package.

@jbreiden, It better to solve issues in upstream (this reoo).

https://github.com/tesseract-ocr/tesseract/commit/aee910a7bfcbe0366806b68dc7f20535764c06d4#r23118118
https://github.com/tesseract-ocr/tesseract/blob/aee910a7bfcbe0366806b68dc7f20535764c06d4/ccutil/platform.h#L57
// Fix to map between google use of string without std and everywhere else.
#ifdef USE_STD_NAMESPACE
#include <string>
using std::string;
#endif

@jbreiden,
What about changing this to:
// Fix to map between google use of string without std and everywhere else.
#ifndef GOOGLE
#include <string>
using std::string;
#endif

I'd prefer #ifndef DO_NOT_USE_STD_NAMESPACE because it is more specific, but yes, that modification would solve the problem for most people. I have no idea whether it would fit into the Google environment.

I have no idea whether it would fit into the Google environment.

but @jbreiden should know...

I am not a language lawyer, and don't know off the top of my head and find out. But it should be very easy for me to test and find out. Unfortunately due to holidays I won’t be able to do that until January 7. If super urgent Dan might be able to help but this type of thing is not his strong suit either. One thing is clear, we need to do something. Because of the OpenCV dependency, Tesseract is now critical infrastructure in Linux distributions.

It's not urgent. Enjoy the holidays!

I spoke with Ray about this. He is happy for Tesseract to use std::string, as long as it compiles fine and talks with Google internal code that uses a different type of string implementation. Apparently there was trouble with this some time ago, but I I tested it recently and it appears to work fine.  @theraysmith  and I I are now in agreement; go ahead and use std::string in unichar.cpp and unichar.h. Thank you.
P.S. This change has already in patched into the Debian & Ubuntu packages.

@jbreiden, Thanks.

Thanks also to @theraysmith, of course!


⚠ Bofrost.de has degraded performance

In 47edc6b, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1039 ms

Resolved: Bofrost.de performance has improved in e96cb53 after 6 minutes.


badrobot: init at 0.1.2

Description of changes

Package badrobot for nixpkgs - Kubernetes Operator Security Audit Tool
https://github.com/controlplaneio/badrobot
Tweezering Kubernetes Resources: Operating on Operators - Kevin Ward, ControlPlane
15:25 - 16:00 CEST - KubeCon Valencia

Spoiler warning
slides

Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[X] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[X] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[X] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[X] Fits CONTRIBUTING.md.

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

1 package built:

badrobot


First version of the probe

Summary

Related issue :

moving to Draft as there are lots of linter errors to fix


Default disable raft engine

What is changed and how it works?
Issue Number: Ref #12255
What's Changed:

Related changes
Check List
Tests

Unit test
Integration test

Release note
None

/merge


mixed messages: currently getting access or am I disconnected?

Version 0.6.4
To reproduce:

start getting access
put computer to sleep
wake computer up a bit later (long enough to loose tcp/udp connections)

Observe:

the uproxy window says I'm getting access
the tab says I'm disconnected.
the reality is that I'm disconnected. :)

Here's a screenshot:

when disconnected, the UI should now be reset with the error in the popup
https://github.com/uProxy/uproxy/pull/1467


Auto-deploy query server on merged pull request on Heroku

Please set up the query server on Heroku from the master branch.

@mariobehling we already have that -> https://query-server.herokuapp.com/

Oh, missed it. It should be added to the Readme then, please.
Thanks.

@mariobehling  It is there is the repo description. Should I add to the Readme file also?


Pause method

It is necessary to have an ability to stop broadcast and then start it from the place, where we stoped it.

i like this idea!

PLEASE ADD THIS. I'm working on an online radio service for the game Rust for server owners to add custom radio stations and I want a way to pause stations.


Incorrect query generation for cast integer to string for DB2 row validation.

There is incorrect query generation while doing row validation for DB. When casting to string we are casting to CLOB and query is:
SELECT t6.guestname AS guestname, t6.content AS content, t6.entryid AS entryid, t6.guestname AS cast__guestname, t6.content AS cast__content, CAST(t6.entryid AS CLOB) AS cast__entryid
FROM db2inst1.entries AS t6

But we are facing an error that:
A value with data type "SYSIBM.INTEGER" cannot be CAST to type "SYSIBM.CLOB".
I think instead of casting to CLOB casting to VARCHAR(n) will be a solution.
Some samples from DB2 terminal:

What's the exact DVT command/config you're using when getting this error?


Update sharepoint-powershell.md

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.


Hotfix #18 Create Business Tests Fix

I don't know in which order English letters go in the alphabet, sorry.
I think it is useless to order imports.
I am trying as hard as possible to stick to the flake8 code style, and I do this. every line has a required length

Done ordering and fixed dicts in tests


Reduce texture aggregation methods to 2D and 3D

We could reduce the texture aggregation choices to 2D and 3D.
Selecting 2D would extract:

2D averaged
2D slice-merged

Selecting 3D would extract:

2.5D direction-merged
2.5D merged
3D averaged
3D merged

Let's follow the naming convention from IBSI: https://github.com/theibsi/ibsi_1_data_analysis/blob/4feb75e02c154b295fcee7def0dc70e47b176e6e/naming_tables/20190830.xlsx


Add support for Ledger in the [Connect] menu

Metamask is having issues connecting hardware wallets.
It'd be great to have direct Ledger support on Juicebox.
This is ours:
https://discord.com/channels/@me/873913220862181446/877915927121920030
Something like Yearns would be nice:

FYI, most dapps are using Blocknative (so is Yearn above) - we should probably look into it.


Not listening on port 80

Hi
I use traefik on multiple docker envs without problems, but in one it didn't work
I simplify to your docker-compose example, and this is what happen :
cat docker-compose.yml
version: "2"
services:
traefik:
image: traefik
command: --web --docker --docker.domain=docker.localhost --logLevel=DEBUG
ports:
- "80:80"
- "8080:8080"
volumes:
- /var/run/docker.sock:/var/run/docker.sock
- /dev/null:/traefik.toml
whoami1:
image: emilevauge/whoami
labels:
- "traefik.backend=whoami"
- "traefik.frontend.rule=Host:whoami.docker.localhost"
whoami2:
image: emilevauge/whoami
labels:
- "traefik.backend=whoami"
- "traefik.frontend.rule=Host:whoami.docker.localhost"
docker-compose up -d
Creating network "traefikdemo_default" with the default driver
Creating traefikdemo_traefik_1
Creating traefikdemo_whoami2_1
Creating traefikdemo_whoami1_1
docker port traefikdemo_traefik_1
80/tcp -> <IP_ADDRESS>:80
8080/tcp -> <IP_ADDRESS>:8080
curl -H Host:whoami.docker.localhost http://<IP_ADDRESS>
curl: (56) Recv failure: Connection reset by peer
Service seems to not listen on port 80
traefik listen well on port 8080 :
curl http://<IP_ADDRESS>:8080/api
{"docker":{"backends":{"backend-whoami":{"servers":{"server-traefikdemo_whoami1_1":{"url":"http://<IP_ADDRESS>:80","weight":1},"server-traefikdemo_whoami2_1":{"url":"http://<IP_ADDRESS>:80","weight":1}}}},"frontends":{"frontend-Host-whoami-docker-localhost":{"backend":"backend-whoami","routes":{"route-frontend-Host-whoami-docker-localhost":{"rule":"Host:whoami.docker.localhost"}},"passHostHeader":true}}}}
I test directly on container IP and it seems the conatiner itself is listening only on port 8080, not 80 :
exec 3<>/dev/tcp/<IP_ADDRESS>/80; echo $?
bash: connect: Connection refused
bash: /dev/tcp/<IP_ADDRESS>/80: Connection refused
1
exec 3<>/dev/tcp/<IP_ADDRESS>/8080; echo $?
0
How can I debug why traefik is not listneing on port 80 ?
Env information :
Docker : Server Version: 1.11.1
docker-compose version 1.7.1, build 0a9ab35
traefik        latest       bf7f4f01be40    2 days ago    19.91 MB
Thanx

Hi @WTFKr0, this has been fixed in https://github.com/containous/traefik/pull/428. I will publish the rc2 soon. Sorry for that regression :/

OK good to hear that !
Thanx

@emilevauge Just ran into this issue myself. Looks like the official traefik in dockerhub is still busted. Your emilevauge/traefik is working great. Documentation however all uses the official Traefik and causes massive troubleshooting headaches

@mjgorman the dockerhub version will be updated soon ;) https://github.com/docker-library/official-images/pull/1822

I have the same issue on Mac, using exactly the same docker-compose as above. Any help?
MacBookPro:docker $ docker-compose down && docker-compose up -d
Stopping docker_traefik_1 ... done
Stopping docker_whoami1_1 ... done
Removing docker_traefik_1 ... done
Removing docker_whoami1_1 ... done
Removing network docker_default
Creating network "docker_default" with the default driver
Creating docker_whoami1_1 ...
Creating docker_traefik_1 ...
Creating docker_whoami1_1
Creating docker_traefik_1 ... done

Port info:
80/tcp -> <IP_ADDRESS>:80
8080/tcp -> <IP_ADDRESS>:8080

@dakkusingh chances are you are affected by a different problem. Join us on Slack for real time help or create a question on Stackoverflow
If you are really certain that this is a bug (either a new one or a regression), please open a new issue.

Hi Timo,
Thank you, you are correct, my issue didnt seem to be related to this. Although the symptoms were very similar. This is resolved now.


Create Post “pre-construction-termite-protection”

Automatically generated by Netlify CMS

👷 Deploy Preview for chic-shortbread-66a66b processing.

Name
Link

🔨 Latest commit
098ba50a26761bc876e480109446228c683fd489

🔍 Latest deploy log
https://app.netlify.com/sites/chic-shortbread-66a66b/deploys/653b07dc7032aa0008635478