.setOnItemClickListener((dialog1, item, view, position)  returning wrong index

I am using an custom adaptor and returning an arraylist of a custom class, when I am using
adaptor.getItem(position); the position returned is 1 above the item selected. It looks like it is not taking into account that Arraylist is 0 indexed?

Thanks for the feedback. Let me check this, I think someone else had the same issue before and was going to contribute, seems it's up to me now.

Thanks for the reply, I will create a fork and take a look as well, hopefully get my head round it!

That would be great!

Actually I faced the same problem, but I solved it by using adapter extends from BasicAdapter, rather than ArrayAdapter .
Enjoy!

ListView as default returns 1 if the list has headerview. If that was the case, I changed this behaviour with 1.1 version. GridHolder and ListHolder will return 0 regardless of headerview.

Im sorry to say that GridHolder still returns the wrong index, for example, I got 6 items to display, and I create a GridHolder using new GridHolder(3). But when I click the second or the third item, I both got a position  = 0;
thanks for your work, but this bug is very upsetting

Yup. Same issue here. no matter what I click on the gridHolder, the item position is always 0. Also, the ripple effect appears on the first item no matter what I click.

Update: As mentioned in issue #72 , this occurs when setExpanded() is set to true. making it false will fix the issue for now.


Error Pages

Labs fertig, Hugo so halb und Pubby funktioniert nicht weil die Seitenabfrage irgendwie zu kompliziert ist.

@sarafischer Can you check and see what error pages are missing, not working and where (labs, hugo, pubby)?

hugo: no error pages
pubby: no error pages
labs: 404 & 500 templates present
@bsesic which error pages do we need?

Which branch (labs) do I have to check?/Are there any changes in beta concerning this matter that are missing in deployment?

deployment for now. We cherry pick into beta later.

hugo: no error pages
pubby: no error pages
labs: 404 & 500 templates present
@bsesic which error pages do we need?

We need:
404
403
500
400
See the documentation for django (here)[https://docs.djangoproject.com/en/4.0/ref/views/].

Information for Hugo.

@sarafischer  fixed in pubby.

ErrorDocument 404 /404.html added to sites-available in Apache config.


Fix the boostnote recipe, wasn't able to fetch the URL of the .deb file

The script could't find the .deb url, corrected the bug

Thanks, @flavienbonvin.


Add remez to cupyx.scipy.signal

See #7403
As per an internal discussion in Gitter, the implementation of remez will defer its call to scipy.signal, since there wouldn't be much gains from trying to compute the exchange algorithm (which is mostly serial) in GPU. Also, the usual filter sizes are small in length, thus also reducing the potential gains from a GPU implementation

@takagi, I've just updated the PR to call remez from SciPy, as we discussed with @ev-br on Gitter

/test mini

I appreciate your effort that you implemented GPU kernels and considered their effectiveness.

What is still needed here?

Seems that we have a few CI failures. Would you take a look at them, @andfoy?
FAILED cupyx_tests/scipy_tests/signal_tests/test_fir_filter_design.py::TestRemez::test_fs_validation[mod0]
FAILED cupyx_tests/scipy_tests/signal_tests/test_fir_filter_design.py::TestRemez::test_fs_validation[mod1]

TypeError: object of type 'int' has no len()

@takagi I cannot reproduce any of these errors locally

/test mini

/test mini

/test mini


[Event Request] Codeunit::"Whse.-Post Shipment" - Function HandlePurchaseLine - OnBeforeFindWhseShptLineForPurchLine

We like to request the following Event:
Object: Codeunit::"Whse.-Post Shipment"
Function: HandlePurchaseLine
Publisher Event: OnBeforeFindWhseShptLineForPurchLine
Reason: To implement custom business logic
Where is the event placed:
local procedure HandlePurchaseLine(var WhseShptLine: Record "Warehouse Shipment Line")
var
PurchLine: Record "Purchase Line";
ModifyLine: Boolean;
IsHandled: Boolean;
begin
with WhseShptLine do begin
PurchLine.SetRange("Document Type", "Source Subtype");
PurchLine.SetRange("Document No.", "Source No.");
if PurchLine.Find('-') then
repeat
SetRange("Source Line No.", PurchLine."Line No.");
IsHandled := false;
OnBeforeFindWhseShptLineForPurchLine(WhseShptLine, PurchLine, WhseShptHeader, ModifyLine, IsHandled);
if not IsHandled then begin
if Find('-') then begin
Event Code:
[IntegrationEvent(false,   false)]         local procedure   OnBeforeFindWhseShptLineForPurchLine(var WarehouseShipmentLine: Record   "Warehouse Shipment Line"; var PurchLine: Record "Purchase   Line"; WhseShptHeader: Record "Warehouse Shipment Header"; var   ModifyLine: Boolean; var IsHandled: Boolean)         begin         end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Precision issues near poles for large values of nside

At the moment, our longitude/latitude to pixel conversion has a lower precision than that of the official HEALPix library close to the poles. This is visible when making a map of pixel values for 2**28 for the following range of coordinates:
lat_min = 89.999996
lat_max = 89.999997
lon_min = -5
lon_max = +5

Original HEALPix library (via healpy)

This package

I think I've tracked down the issue to the xyztohp function, and I think the issue could be with these lines:
        // solve eqn 20: k = Ns - xx (in the northern hemi)
		root = (1.0 - vz*zfactor) * 3.0 * mysquare(Nside * (2.0 * phi_t - pi) / pi);
		kx = (root <= 0.0) ? 0.0 : sqrt(root);

        // solve eqn 19 for k = Ns - yy
		root = (1.0 - vz*zfactor) * 3.0 * mysquare(Nside * 2.0 * phi_t / pi);
		ky = (root <= 0.0) ? 0.0 : sqrt(root);

		if (north) {
			xx = Nside - kx;
			yy = Nside - ky;
		} else {
			xx = ky;
			yy = kx;
		}

The issue is that doubles actually have less precision information than int64 (only 52 bits are allocated to significant precision), so by going through double and doing operations such as sqrt etc which further degrade precision, I think we are losing precision there. So this may not be a 32-bit vs 64-bit int issue but rather an issue with floating point precision. We should investigate whether we can improve the precision here, for example by using long double or finding other ways to avoid precision loss. For now, I will update the documentation to mention the resolution to which the results can be trusted.
@dstndstn - if you have any ideas on improving precision here, please feel free to comment!

Hi,
If this is at Dec = 89.99999, then I'm guessing you're seeing cancellation
error in (1. - vz), where (I'm guessing) vz is the unit-sphere z vector --
which will be approaching 1 as Dec approaches 90 degrees.  Maybe you could
use a different trig identity here to avoid the cancellation error --
sin(Dec) = cos(pi/2 - Dec) -- for vz > 0.9 or 0.5 or whatever.
cheers,
--dustin
On Mon, Sep 25, 2017 at 9:30 AM, Thomas Robitaille<EMAIL_ADDRESS>
wrote:

At the moment, our longitude/latitude to pixel conversion has a lower
precision than that of the official HEALPix library. This is visible when
making a map of pixel values for 2**28:
Original HEALPix library (via healpy)
[image: healpy_true]
https://user-images.githubusercontent.com/314716/30810655-aeabe51e-a1fd-11e7-82d0-e956b59b1029.png
This package
[image: healpy_false]
https://user-images.githubusercontent.com/314716/30810664-b4bbafac-a1fd-11e7-8cd9-bad400e11f53.png
I think I've tracked down the issue to the xyztohp function, and
specifically these lines:
    // solve eqn 20: k = Ns - xx (in the northern hemi)
  root = (1.0 - vz*zfactor) * 3.0 * mysquare(Nside * (2.0 * phi_t - pi) / pi);
  kx = (root <= 0.0) ? 0.0 : sqrt(root);

    // solve eqn 19 for k = Ns - yy
  root = (1.0 - vz*zfactor) * 3.0 * mysquare(Nside * 2.0 * phi_t / pi);
  ky = (root <= 0.0) ? 0.0 : sqrt(root);

  if (north) {
  	xx = Nside - kx;
  	yy = Nside - ky;
  } else {
  	xx = ky;
  	yy = kx;
  }

and in particular the issue is with the x value (y seems fine I think).
The issue is that doubles actually have less precision information than
int64 (only 52 bits are allocated to significant precision), so by going
through double and doing operations such as sqrt etc which further
degrade precision, I think we are losing precision there. So this may
not be a 32-bit vs 64-bit int issue but rather an issue with floating point
precision. We should investigate whether we can improve the precision here,
for example by using long double or finding other ways to avoid precision
loss. For now, I will update the documentation to mention the resolution to
which the results can be trusted.
@dstndstn https://github.com/dstndstn - if you have any ideas on
improving precision here, please feel free to comment!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cdeil/healpix/issues/34, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABBD_Z5p3TkcPkeib_Fg2WSXTGlLaSPJks5sl6sDgaJpZM4PivH7
.

I had a further look at this, and indeed the precision loss is in converting to unit-sphere coordinates.  In order to fix this, I think one would have to pass Dec, in addition to z, into xyztohp -- or maybe pass in (1-z) and z -- or event sqrt(1-z) and z if you want to get really fancy -- and then use trig identities.

In e1df57b I used "long double" trig functions and the result seems to fix this resolution problem.

I only did that for the one function used by the library, but presumably we could use this approach more widely.  I assume there is some computational cost for using the long double trig functions, so I don't know whether to use this everywhere or not.

@dstndstn - Do you understand this? Is "double" 64 bit and "long double" more? Or are both 64 bit and just the C functions called are higher precision somehow?
Maybe open a PR with the commit you have, and wait for @astrofrog to comment whether to merge as-is or apply more widely?

"long double" is longer than 64 bits.  How long depends on implementation;
https://en.wikipedia.org/wiki/Long_double

PR #93

PR #94 instead?

I don't know if it helps but in the case of the CDS Java HEALPix lib I developed, I replaced \sqrt(3(1-\sin(\delta))) by the equivalent \sqrt(6)\cos(\delta/2 + \pi/4) for the Collignon projection.

(In your lib, see more efficient algo to compute the Morton code here: https://graphics.stanford.edu/~seander/bithacks.html#InterleaveTableLookup)


Combining multiple providers from seperate libraries

The approach shown in this documentation doesn't show you how you might combine multiple providers, say from seperate nuget packages, into one singleton composite provider.
For anyone looking to do that, I created: https://github.com/dazinator/Dazinator.AspNetCore.Authorization - and have opened this github issue as I think that would be a good thing to have in the framework by default: https://github.com/aspnet/AspNetCore/issues/17493

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

ID: 150023ea-e35a-4a06-6f71-7ab63994b07f
Version Independent ID: 7b5c3832-a5a3-a27a-eed3-3ecfc3e8dbe2
Content: Custom Authorization Policy Providers in ASP.NET Core
Content Source: aspnetcore/security/authorization/iauthorizationpolicyprovider.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @mjrousos
Microsoft Alias: riande

I think a better example of how to write a custom policy provider is the one I have documented here: https://github.com/dazinator/Dazinator.AspNetCore.Authorization/blob/develop/README.md#note-to-provider-authors
It would be interesting to know if the asp.net core team agree with me and if you'd therefore update this documentation with this in mind?

@blowdart  let me know if you're interested in documenting this. I think this is out of the scope of the docs so closing for now.


Version of @angular/compiler-cli needs to be 2.3.1 or greater. Current version is "8.2.14".

Hi,
I am using angular 8.2.14 with node 8.11.
I am able to run the project.
I am using aot webpack 3.11 to build.
So while building I m facing error.
Version of @angular/compiler-cli needs to be greater than 2.3.1 current version is 8.2.14.
So below things I tried
Deleting node modules and pkg. Json and npm install no solution

Duplicate of #5558

I did not get any solution out of that, hence created this

https://github.com/angular/angular-cli/issues/5558#issuecomment-288270864 should solve your issue.
if the issue persists, please file another issue and follow the issue template.


HA player provieder - connection failed

What version of Music Assistant has the issue?
2.1.0b1
What version of the Home Assistant Integration have you got installed?
2024.5.1
Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?

[X] Yes

The problem
When i setting up the HA plugin and die HA player provider i get the following log entry:

I tried several player (which worked before on betas before the official release)
How to reproduce
Setup the HA player provider
Music Providers

Player Providers
Home Assistant
Full log output
please see above
Additional information
No response
What version of Home Assistant Core are your running
2024.5.5
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Linux

Please review the troubleshooting page in the docs again and supply the requested information

Please review the troubleshooting page in the docs again and supply the requested information

I updated my post. Please let me know when you need further information.

I have added more information. I hope you can analyze the problem.

Please test with version 2.0.7 (or beta 2.1.0b6) that has just been released

@collateral87 please advise if this is still a problem in2.0.7

Hi i can confirm, that v2.0.7 fixed the problem.


stop using fat arrow function so it can be used with webpack

See here for why https://github.com/facebookincubator/create-react-app/issues/3529

I've commented on the create-react-app issue, because I think this is a failing of their bundled tooling. This certainly isn't the only package out there to use ES2015+ features in the package(s) contained within the module path. If you are using tooling that only supports ES5 and still insists on importing the module path, then you can manually import the ES5 code from this package with import relativeDate from 'tiny-relative-date/lib/index' instead.

(P.S. Sorry for the horrendously late reply! I didn't have notifications turned on for this repo)


the parent object could not be found

this is on the source side, when running with all the parameters.
Getting this error.
VERBOSE : Creating Organization Relationship: Domain_Sync_9193 in domain.onmicrosoft.com
The parent object for Domain_Sync_9193 could not be found. Please check that domain.onmicrosoft.com \federation exists.

Kumar,
What did you specify in the parameters?
Thanks,
Roman
From: kumar-debug @.>
Sent: Friday, July 2, 2021 2:53 PM
To: microsoft/cross-tenant @.>
Cc: Subscribed @.***>
Subject: [microsoft/cross-tenant] the parent object could not be found (#68)
this is on the source side, when running with all the parameters.
Getting this error.
VERBOSE : Creating Organization Relationship: Domain_Sync_9193 in domain.onmicrosoft.com
The parent object for Domain_Sync_9193 could not be found. Please check that domain.onmicrosoft.com \federation exists.

You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fcross-tenant%2Fissues%2F68&data=04|01|Roman.Powell%40microsoft.com|a1edefa8ca134537acbb08d93da3c901|72f988bf86f141af91ab2d7cd011db47|0|0|637608595961927515|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=RvLcauoUukKOTA3Genv8EPfmlDHC6zB1APGtMcULVTk%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAPUHY6GUBSIMUUBZGMSVY7DTVYYMNANCNFSM47XNR2NQ&data=04|01|Roman.Powell%40microsoft.com|a1edefa8ca134537acbb08d93da3c901|72f988bf86f141af91ab2d7cd011db47|0|0|637608595961937471|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=bXmzskLyg3nrSkJ1WKsOrMY2oC3jpQGg0dfU2H6DGzs%3D&reserved=0.

The parameters are
-ResourceTenantDomain  -targetTenantDomain -TargetTenantID -sourceMailboxmovepublishedScopes -applicationID

Please include the values you used. If you do not want them on GitHub please send a separate email to<EMAIL_ADDRESS>Thanks,
Roman

it was a tenant issue, its resolved now.


Supabase Client does nothing

Hi is this a work in progress? From looking at the code it seems it only reads sites from lib/data. Any entries in Supabase aren't even looked at.

Hi jontstaz, the supabase is use for save the submit list. And we will release the new version to get the submit list and generate the web detail for the url.


Adding some coding conformity suggestions

Code is sometimes inconsistent in how it presents things.  Modifying the delegate to add some suggestions.

r=me


LTS: Disable ODSP E2E and Service Load Tests

the lts version, 1.0.0 is no longer supported for all features on odsp, specifically "Received summaryNack: Upgrade to a newer version of the Fluid client packages to summarize."

Are the submodule updates necessary?

i think we need odsp to actually fix this. we might need to make a similar change later, but i'm going to close for now


Provide option for only showing historical changes

This PR changes the behavior of the historical view so that the page only displays changes to the history (rather than the whole history). In order to maintain previous functionality, I've added a button which allows viewing the full history across all rosters.
I also changed the file types of some jinja templates we had to be consistent across the board (and also let my IDE recognize them as HTML).
https://user-images.githubusercontent.com/10214785/147322646-29b22fc1-1fde-449a-a5ba-06cad969d474.mp4

Oh dang, that seems like it'd be a way better approach 😂🤦🏼♀ chalk that up to watching bachelorette while I worked on this haha. I'll try to see if I can move that logic out of the template.

@sarayourfriend does that seem better? I feel like it's more complicated this way but I'm not an objective third party :thinking:

I think it's simpler out of the template (for one it should be easier to find where this is happening in the future I think?). I suppose having to check historical makes it slightly more complicated but I'm wishing we'd separated that route out a little more but 😬 I don't think it's a the point that it would justify refactoring all of that just to accomodate this change more cleanly.


Option to allow Microsoft 365 Apps Updates does not mention channel switch in blurb.

The Autopatch option to "Allow" "Microsoft 365 Apps Updates" does not mention in the blurb in Intune that it switches the channel to Monthly Enterprise Channel. This is a big change that customers need to know before they flip that switch.

Can a short blurb mentioning that change be added to the Intune description? i.e.:
When set to Allow, Windows Autopatch will move all Autopatch devices to the Monthly Enterprise Channel and begin managing updates automatically. To manage them yourself, select the switch to turn it off. You must be an Intune Administrator to make changes to the setting. Learn more about Microsoft 365 apps updates.
Thanks!
John Rea

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

ID: ec8a5572-1135-3c0b-06d1-b8e1f258c17a
Version Independent ID: ec8a5572-1135-3c0b-06d1-b8e1f258c17a
Content: Microsoft 365 Apps for enterprise - Windows Deployment
Content Source: windows/deployment/windows-autopatch/operate/windows-autopatch-microsoft-365-apps-enterprise.md
Product: windows-client
Technology: itpro-updates
GitHub Login: @tiaraquan
Microsoft Alias: tiaraquan

Thank you for your feedback. We've updated the documentation.


async shouldReadFromCache and shouldWriteToCache test

This test looks good to me, so if @centogram merges it I'll be able to merge upstream!


ARC-370 Fixed JWT Authentication for iframe endpoints

I've added validation to JWT tokens for iframe and calls we are making from iFrame
Also changed iframe script to generate context tokens for the calls (instead of reusing the iframe token itself)

Isn't there some kind of library we can use so we don't have to implement our own JWT verification?  Seems like something that would be standardized and used a lot.

Closing this PR hence its changes were incorporated in this one https://github.com/atlassian/github-for-jira/pull/599


changefeed lag always keep at 6~7s after inject one of az failure for 10mins

What did you do?
1、create changefeed
2、run sysbench
sysbench --db-driver=mysql --time=0 --mysql-db=sysbench_64_7000w --mysql-host=tc-tidb.endless-ha-test-ticdc-tps-2220745-1-732 --mysql-port=4000 --threads=64 --tables=64 --table-size=70000000 --mysql-ignore-errors="2013,1213,1105,1205,8022,8028,9004,9007,1062" --mysql-user=root --report-interval=10 oltp_read_write run
3、inject one of az failure for 10mins
What did you expect to see?
changefeed lag can recover to less than 3s after fault recover
What did you see instead?
changefeed lag always keep at 6~7s after inject one of az failure for 10mins

Versions of the cluster
git hash：c32bf531576cb83ab2722deb5848fe09a79befb9
current status of DM cluster (execute query-status <task-name> in dmctl)
No response

/remove-area dm
/area ticdc

/severity major

/assign hicqu


test: try Jest's --onlyChanged for functional tests

Only for GitHub Actions
This checks if all the files modified (or added) by a PR are located inside packages/client/tests/functional
If true we only run these tests, thanks to Jest's --onlyChanged
else we run like before (all client tests)

Merging now and trying it with https://github.com/prisma/prisma/pull/15563

Only worked locally but not in GitHub Actions 😢


Target lower FSharp.Core version

Can this project target a lower FSharp.Core version to prevent package downgrade warnings in downstream libraries?

Closing and adding a new issue - We should update the whole repo to use .NET SDK and a reasonable FSHarp.Core.


articles/terraform/create-vm-scaleset-network-disks-hcl.md needs to specify standard SKU for PIP

The main.tf script in the article in articles/terraform/create-vm-scaleset-network-disks-hcl.md (seen here https://learn.microsoft.com/en-us/azure/developer/terraform/create-vm-scaleset-network-disks-hcl) need to set the SKU for the PIP to Standard or it will fail.
From this:
resource "azurerm_public_ip" "vmss" {
 name                         = "vmss-public-ip"
 location                     = var.location
 resource_group_name          = azurerm_resource_group.vmss.name
 allocation_method            = "Static"
 domain_name_label            = random_string.fqdn.result
 tags                         = var.tags
}

To this:
resource "azurerm_public_ip" "vmss" {
 name                         = "vmss-public-ip"
 location                     = var.location
 resource_group_name          = azurerm_resource_group.vmss.name
 allocation_method            = "Static"
 sku                          = "Standard"
 domain_name_label            = random_string.fqdn.result
 tags                         = var.tags
}

This is the error if you don't set it:

Error: creating/updating Load Balancer: (Name "CacheLoadBalancer" / Resource Group "poc"): network.LoadBalancersClient#CreateOrUpdate: Failure sending request: StatusCode=400 -- Original Error: Code="PublicIPAndLBSkuDoNotMatch" Message="Standard sku load balancer /subscriptions/id/resourceGroups/poc/providers/Microsoft.Network/loadBalancers/CacheLoadBalancer cannot reference Basic sku publicIP /subscriptions/id/resourceGroups/poc/providers/Microsoft.Network/publicIPAddresses/CachePublicIP." Details=[]

Unfortunately, we have not been able to review this issue in a timely manner. We sincerely apologize for the delayed response. We are closing this issue. If you feel that the problem persists, please respond to this issue with any additional information.
Please continue to provide feedback about the documentation. We appreciate your contributions to our community.
#please-close


add translation to latvian

Added translation to latvian

@xanthopsia also add this language  at LanguageSelector component

Please add to LanguageSelector.tsx, and then I'll approve.

@xanthopsia Please remove the swap file created by the VIM kofta/src/app/components/.LanguageSelector.tsx.swp

@xanthopsia Please remove the swap file created by the VIM kofta/src/app/components/.LanguageSelector.tsx.swp

I don't see the .swp file?

@xanthopsia Please remove the swap file created by the VIM kofta/src/app/components/.LanguageSelector.tsx.swp

I don't see the .swp file?

https://stackoverflow.com/a/13361874/3838600


fix(ui): left-align column text

Addresses the weird gap between words in column issue from #636. See the last image and paragraph here: https://github.com/LN-Zap/zap-desktop/issues/636#issuecomment-450604350

Coverage remained the same at 19.049% when pulling cf95c8f2e1029db0ab9cd1419ba208fc6d815b27 on bolatovumar:fix/636 into ebce0a767ce89b0355e9e91400dd4196f2f8839a on LN-Zap:master.

Coverage remained the same at 19.049% when pulling cf95c8f2e1029db0ab9cd1419ba208fc6d815b27 on bolatovumar:fix/636 into ebce0a767ce89b0355e9e91400dd4196f2f8839a on LN-Zap:master.

Coverage remained the same at 19.049% when pulling cf95c8f2e1029db0ab9cd1419ba208fc6d815b27 on bolatovumar:fix/636 into ebce0a767ce89b0355e9e91400dd4196f2f8839a on LN-Zap:master.

Coverage remained the same at 19.049% when pulling cf95c8f2e1029db0ab9cd1419ba208fc6d815b27 on bolatovumar:fix/636 into ebce0a767ce89b0355e9e91400dd4196f2f8839a on LN-Zap:master.

I tried out using 16px font size (currently set at 10px) on my MacBook Air (non-retina) and it looks much more readable. My only concern for increasing the font size here would be that it might show up too big on retina screens. I doubt it would be an issue but would like someone to confirm.

And as far as legibility for non-retina screens goes I found that using font-weight: 400; instead of the currently used font-weight: 300; works wonderfully. Not sure whether retina screens would be negatively impact by this change though.

@bolatovumar regarding the font-sizes - I think the best way to do this is to take a more holistic approach and make our font-sizes (and other page elements) responsive via breakpoints.
I don't think we really want to adjust the default font-sizes or weights, but we can tailor for smaller/larger screen sizes.
Rebass (our theme system, https://github.com/rebassjs/rebass) already supports responsive theming and I actually already have a PR in my local branch that makes a start at making our font-sizes responsive using this approach. If you are interested on taking this on let me know and I can share what I have with you.

@mrfelton do you mean you would prefer to adjust font sizes/weights for NON-RETINA displays through media queries that target these displays rather than enable the change for all displays?
The issue with font sizes and weights here is not due to the viewport or resolution being too small but due to the font-sizes themselves being too small and therefore rendering less than ideally.
For example, I'm having trouble reading some of the text without glasses (especially within the menu where the fiat/theme/language are set). It's a lot better with the glasses but still quite small. I generally find that using <12px font sizes in UIs is a no-go from the accessibility perspective especially if light font weights are used.
The type scale for the Material Design system from Google shows how they use Roboto at various sizes (https://material.io/design/typography/the-type-system.html#). Notice that the 10px size is used in all-caps only and in the Regular font-weight.

@bolatovumar Yes, I was suggesting that we adjust font sizes for specific breakpoints but leave the defaults / base sizes as is. Reason being that we've been through a design process and these were the sizes and weights that were picked by the designers, so changing this would have an impact on the designs and overall look and feel. Not saying that the current font sizes are right or perfect, just that our current look and feel has been designed with these sizes.
The reason that I was looking at adjusting the font sizes via breakpoints is that we know that when the app is displayed in full screen mode on a large hi-res retina display, the fonts look very small and there is a lot of space on the screen - so making fonts larger when there is is more screen space would be nice.  But, making the font sizes larger on smaller screens / resolutions could cause some issues in some parts of the app, with things not fitting onto the page fully without requiring the user to scroll - which is something that we want to avoid if possible.
I generally work on a mac with a retina display, at a res of 1440 x x900 and there the font size is good (for me). But I think your point about accessibility is a valid one and we certainly don't want users having to squint at the screen to read things.
@bolatovumar could you open a separate issue to carry on this discussion about font sizes please, so that it doesn't get lost in this closed PR? Thanks.


Dates are off

As Greenchoice publishes the readings one day behind
When new data is available from the API, the datetime from Greenchoice should be used and stored, not now().

This seems to actually be somewhat complicated..
https://community.home-assistant.io/t/energy-counter-delayed-data/332108/14
https://community.home-assistant.io/t/batch-receive-time-series-data/171915

@dedmen thnx for diving into this and sharing the information.
It doesn't look promising (even looks like a serious limitation in the HA architecture).
Allthough I'm still a newbie with HA, it's architecture and with Python (yet), I am an IT Professional so let's see if I can help.

I actually got it implemented in C# today.
Sending manual state_update events with last_updated property does work and correctly lists states in history.
Problem is the last state value isn't being set by it.
So next day, it's hard to retrieve the last meter state of the previous day.
And the hourly API that I use doesn't have full meter readings, but restarts from zero every day.
And total_increasing state class doesn't work when sending manual state_changed events.
So to keep proper state, I need to add them to the previous days maximum.
And reading the full meter readings isn't helping because they are rounded to integer, but my daily usage is sometimes less than 1.
Could always store the last maximum in a separate file. Will probably try that tomorrow.
When I got it working sufficiently I'll publish the C# code for it.

Works great

Its a bit of a mess, but good enough for my needs https://github.com/dedmen/HomeAssistantTools/blob/main/apps/HassModel/HelloWorld/GreenchoiceMeterReader.cs
I could even use it to fill in the last 30 days of historic data

Okey a couple days later.
Data collection works mostly (I have some trouble with my scheduling).
The History view works fine as shown above.
But the "Energy" system from home assistant doesn't work at all.
I suspect that system needs the SetState API, and doesn't work with state_update events.
SetState API doesn't support timestamps in the past. So we're back on missing functionality in HA core. And seeing the above linked community forum threads, while the interest for it exists, we'll probably not see a implementation for it.
https://community.home-assistant.io/t/batch-receive-time-series-data/171915/12

Hi Dedmen,
I didn't even get to setup a dev environment, swampt with other tasks and work :(.
Thanx for the updates though.
I suspect that the energy dashboards use statistical data rather than sensor data.
I read something about that in the links you shared.
That information also implied that you would have to rebuild these statistics and this is were I suspect the rabbithole starts because as I've read, building the statistics depends on the type of data store used to store the sensor data (and statistics). I.e. this would require different scripts for different databases (SQL for MariaDB, InfluxQL/Flux for InfluxDB).
Allthough it can be done and might not even be that hard at first, I would not recommend going down this path because the solution could easily break upon update or people expecting your integration to work for their particular setup which might be very different from the mainstream HA setups.
Many thnx though.

Hi @dedmen, I'd like to ask whether your implementation is working stable? Would you upstream it? I'd LOVE to have hourly data on my dashboard.
Thanks for all your hard work!

It is not perfect.
To have the accumulating work correctly, I need to remember the numbers of the previous day. My solution for that is to just store the numbers in a text file, and its somewhat broken, on some days it just resets. I need to redo that.

I have the daily meter readings, and would be able to catch up on a missing day, but I didn't implement that (Daily readings are rounded and less precise, than the hourly ones. So at end of day, they will not match up exactly)
But just checking, I didn't get any data for 3 weeks.. Greenchoice changed their API, but that's a quick fix.
My API implementation works correcly, but handling the received data is still buggy.
Also my implementation, is completely NetDaemon based C#, I don't know enough python to port it into a python addon. So I will not be porting it into this.
But anyone knowing python and C#, could port my code over.
https://github.com/dedmen/HomeAssistantTools/blob/main/apps/HassModel/HelloWorld/GreenchoiceAPI.cs

Hey all, thanks for sharing your experiences. I just started using this integration but nothing shows up yet in the Energy tab. I guess I just have to wait a bit for enough data to be collected, and in the meantime I was looking into whether it's possible to easily import historic data into Home Assistant, but it still looks like an unresolved problem. It seems that some people in the community have resorted to accessing the Home Assistant database directly and backfilling data there: https://community.home-assistant.io/t/how-to-import-historical-energy-data/556356. Hope this helps someone! End please keep updating the threads with updates and more tips. Thanks 🙏


Build fails on arch linux , gcc 5.1.0 with biicode

This is clean vanilla project settings using biicode and everthing else works till I build. Then there is this error.
Also when I manually tried to build with clang , all of the errors disappeared and another wierd one appeared stating something
Scanning dependencies of target florianwolters_catch
[ 20%] Building CXX object florianwolters_catch/CMakeFiles/florianwolters_catch.dir/projects/SelfTest/TestMain.cpp.o
In file included from /home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/projects/SelfTest/TestMain.cpp:10:
In file included from /home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/catch.hpp:47:
In file included from /home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/internal/catch_impl.hpp:19:
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/catch_runner.hpp:63:1: error: version control conflict marker in file
<<<<<<< HEAD
^
1 error generated.
florianwolters_catch/CMakeFiles/florianwolters_catch.dir/build.make:54: recipe for target 'florianwolters_catch/CMakeFiles/florianwolters_catch.dir/projects/SelfTest/TestMain.cpp.o' failed
make[2]: *** [florianwolters_catch/CMakeFiles/florianwolters_catch.dir/projects/SelfTest/TestMain.cpp.o] Error 1
CMakeFiles/Makefile2:110: recipe for target 'florianwolters_catch/CMakeFiles/florianwolters_catch.dir/all' failed
make[1]: *** [florianwolters_catch/CMakeFiles/florianwolters_catch.dir/all] Error 2
Makefile:85: recipe for target 'all' failed
make: *** [all] Error 2

Here is default gcc build error :
INFO: There is a new version of biicode. Download it at https://www.biicode.com/downloads
INFO: Processing changes...
INFO: Saving files from: florianwolters/catch
Running: "cmake"  -G "Unix Makefiles" -Wno-dev  ../cmake
-- The C compiler identification is GNU 5.1.0
-- The CXX compiler identification is GNU 5.1.0
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done

		BLOCK: florianwolters/catch
-----------------------------------------------------------
+ LIB: florianwolters_catch
-- Writing default cmake_dummy.cpp for building library
+ EXE: florianwolters_catch_projects_SelfTest_TestMain
+ EXE: florianwolters_catch_projects_SelfTest_ToStringWhich

		BLOCK: agauniyal/cbs-algorithm-sort
-----------------------------------------------------------
+ LIB: agauniyal_cbs-algorithm-sort
+ EXE: agauniyal_cbs-algorithm-sort_unit_test
-- Initializing variables to create tests with CTest
-- Added custom target for all the tests:
-- Following targets are defined like tests (excluded from build)
+ TEST: agauniyal_cbs-algorithm-sort_unit_test
-- Configuring done
-- Generating done
-- Build files have been written to: /home/agauniyal/projects/cbs-algorithm-sort/bii/build
Building: "cmake" --build .
Scanning dependencies of target florianwolters_catch
[ 20%] Building CXX object florianwolters_catch/CMakeFiles/florianwolters_catch.dir/projects/SelfTest/TestMain.cpp.o
In file included from /home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/internal/catch_impl.hpp:19:0,
                 from /home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/catch.hpp:47,
                 from /home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/projects/SelfTest/TestMain.cpp:10:
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/catch_runner.hpp:73:9: error: invalid suffix "f5615272b0fb51bcd7a669e68f3b4c7a87d93bb" on integer constant
 >>>>>>> 7f5615272b0fb51bcd7a669e68f3b4c7a87d93bb
         ^
In file included from /home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/internal/catch_expression_lhs.hpp:12:0,
                 from /home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/internal/catch_result_builder.h:91,
                 from /home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/internal/catch_capture.hpp:11,
                 from /home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/catch.hpp:28,
                 from /home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/projects/SelfTest/TestMain.cpp:10:
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/internal/catch_evaluate.hpp:43:17: error: ‘nullptr_t’ in namespace ‘std’ does not name a type
     inline std::nullptr_t opCast(std::nullptr_t) { return nullptr; }
                 ^
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/internal/catch_evaluate.hpp:165:58: error: ‘template<Catch::Internal::Operator Op, class T> bool Catch::Internal::compare’ conflicts with a previous declaration
     template<Operator Op, typename T> bool compare( std::nullptr_t, T* rhs ) {
                                                          ^
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/internal/catch_evaluate.hpp:159:44: note: previous declaration ‘namespace Catch::Internal { }::compare’
     template<Operator Op, typename T> bool compare( T* lhs, int rhs ) {
                                            ^
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/internal/catch_evaluate.hpp:165:53: error: ‘nullptr_t’ is not a member of ‘std’
     template<Operator Op, typename T> bool compare( std::nullptr_t, T* rhs ) {
                                                     ^
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/internal/catch_evaluate.hpp:165:70: error: expected primary-expression before ‘*’ token
     template<Operator Op, typename T> bool compare( std::nullptr_t, T* rhs ) {
                                                                      ^
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/internal/catch_evaluate.hpp:165:72: error: ‘rhs’ was not declared in this scope
     template<Operator Op, typename T> bool compare( std::nullptr_t, T* rhs ) {
                                                                        ^
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/internal/catch_evaluate.hpp:165:76: error: expression list treated as compound expression in initializer [-fpermissive]
     template<Operator Op, typename T> bool compare( std::nullptr_t, T* rhs ) {
                                                                            ^
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/internal/catch_evaluate.hpp:165:44: warning: variable templates only available with -std=c++14 or -std=gnu++14
     template<Operator Op, typename T> bool compare( std::nullptr_t, T* rhs ) {
                                            ^
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/internal/catch_evaluate.hpp:165:78: error: expected ‘;’ before ‘{’ token
     template<Operator Op, typename T> bool compare( std::nullptr_t, T* rhs ) {
                                                                              ^
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/internal/catch_evaluate.hpp:168:66: error: ‘std::nullptr_t’ has not been declared
     template<Operator Op, typename T> bool compare( T* lhs, std::nullptr_t ) {
                                                                  ^
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/internal/catch_evaluate.hpp:168:44: error: redefinition of ‘template<Catch::Internal::Operator Op, class T> bool Catch::Internal::compare(T*, int)’
     template<Operator Op, typename T> bool compare( T* lhs, std::nullptr_t ) {
                                            ^
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/internal/catch_evaluate.hpp:159:44: note: ‘template<Catch::Internal::Operator Op, class T> bool Catch::Internal::compare(T*, int)’ previously declared here
     template<Operator Op, typename T> bool compare( T* lhs, int rhs ) {
                                            ^
In file included from /home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/internal/catch_expression_lhs.hpp:13:0,
                 from /home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/internal/catch_result_builder.h:91,
                 from /home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/internal/catch_capture.hpp:11,
                 from /home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/catch.hpp:28,
                 from /home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/projects/SelfTest/TestMain.cpp:10:
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/internal/catch_tostring.h:54:38: error: ‘std::string Catch::toString’ redeclared as different kind of symbol
 std::string toString( std::nullptr_t );
                                      ^
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/internal/catch_tostring.h:51:13: note: previous declaration ‘std::string Catch::toString(unsigned char)’
 std::string toString( unsigned char value );
             ^
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/internal/catch_tostring.h:54:23: error: ‘nullptr_t’ is not a member of ‘std’
 std::string toString( std::nullptr_t );
                       ^
In file included from /home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/internal/catch_impl.hpp:19:0,
                 from /home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/catch.hpp:47,
                 from /home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/projects/SelfTest/TestMain.cpp:10:
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/catch_runner.hpp: In member function ‘Catch::Totals Catch::Runner::runTests()’:
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/catch_runner.hpp:63:1: error: expected primary-expression before ‘<<’ token
 <<<<<<< HEAD
 ^
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/catch_runner.hpp:63:3: error: expected primary-expression before ‘<<’ token
 <<<<<<< HEAD
   ^
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/catch_runner.hpp:63:5: error: expected primary-expression before ‘<<’ token
 <<<<<<< HEAD
     ^
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/catch_runner.hpp:63:7: error: expected primary-expression before ‘<’ token
 <<<<<<< HEAD
       ^
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/catch_runner.hpp:63:9: error: ‘HEAD’ was not declared in this scope
 <<<<<<< HEAD
         ^
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/catch_runner.hpp:64:3: error: expected primary-expression before ‘==’ token
 =======
   ^
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/catch_runner.hpp:64:5: error: expected primary-expression before ‘==’ token
 =======
     ^
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/catch_runner.hpp:64:7: error: expected primary-expression before ‘=’ token
 =======
       ^
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/catch_runner.hpp:65:35: error: expected primary-expression before ‘skippedTestCases’
             std::vector<TestCase> skippedTestCases;
                                   ^
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/catch_runner.hpp:66:91: error: ‘skippedTestCases’ was not declared in this scope
             getRegistryHub().getTestCaseRegistry().getFilteredTests( testSpec, *m_config, skippedTestCases, true );
                                                                                ^
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/catch_runner.hpp:69:27: error: name lookup of ‘itEnd’ changed for ISO ‘for’ scoping [-fpermissive]
                     it != itEnd;
                           ^
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/catch_runner.hpp:69:27: note: (if you use ‘-fpermissive’ G++ will accept your code)
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/catch_runner.hpp:73:1: error: expected primary-expression before ‘>>’ token
 >>>>>>> 7f5615272b0fb51bcd7a669e68f3b4c7a87d93bb
 ^
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/catch_runner.hpp:73:3: error: expected primary-expression before ‘>>’ token
 >>>>>>> 7f5615272b0fb51bcd7a669e68f3b4c7a87d93bb
   ^
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/catch_runner.hpp:73:5: error: expected primary-expression before ‘>>’ token
 >>>>>>> 7f5615272b0fb51bcd7a669e68f3b4c7a87d93bb
     ^
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/catch_runner.hpp:73:7: error: expected primary-expression before ‘>’ token
 >>>>>>> 7f5615272b0fb51bcd7a669e68f3b4c7a87d93bb
       ^
In file included from /home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/internal/catch_impl.hpp:35:0,
                 from /home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/catch.hpp:47,
                 from /home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/projects/SelfTest/TestMain.cpp:10:
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/internal/catch_tostring.hpp: At global scope:
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/internal/catch_tostring.hpp:163:38: error: ‘std::string Catch::toString’ redeclared as different kind of symbol
 std::string toString( std::nullptr_t ) {
                                      ^
In file included from /home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/internal/catch_expression_lhs.hpp:13:0,
                 from /home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/internal/catch_result_builder.h:91,
                 from /home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/internal/catch_capture.hpp:11,
                 from /home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/catch.hpp:28,
                 from /home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/projects/SelfTest/TestMain.cpp:10:
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/internal/catch_tostring.h:199:13: note: previous declaration ‘template<class T, class Allocator> std::string Catch::toString(const std::vector<_Tp, _Alloc>&)’
 std::string toString( std::vector<T,Allocator> const& v ) {
             ^
In file included from /home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/internal/catch_impl.hpp:35:0,
                 from /home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/catch.hpp:47,
                 from /home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/projects/SelfTest/TestMain.cpp:10:
/home/agauniyal/projects/cbs-algorithm-sort/bii/deps/florianwolters/catch/include/internal/catch_tostring.hpp:163:23: error: ‘nullptr_t’ is not a member of ‘std’
 std::string toString( std::nullptr_t ) {
                       ^
florianwolters_catch/CMakeFiles/florianwolters_catch.dir/build.make:54: recipe for target 'florianwolters_catch/CMakeFiles/florianwolters_catch.dir/projects/SelfTest/TestMain.cpp.o' failed
make[2]: *** [florianwolters_catch/CMakeFiles/florianwolters_catch.dir/projects/SelfTest/TestMain.cpp.o] Error 1
CMakeFiles/Makefile2:110: recipe for target 'florianwolters_catch/CMakeFiles/florianwolters_catch.dir/all' failed
make[1]: *** [florianwolters_catch/CMakeFiles/florianwolters_catch.dir/all] Error 2
Makefile:85: recipe for target 'all' failed
make: *** [all] Error 2
ERROR: Build failed

stackoverflow.com/questions/30730517/g-5-1-0-not-building-project-clang-shows-unknown-error

For now , g++ build process completely fails while clang++ happens to produce the require build only if >>>>>HEAD and ====== part is removed from line 63 in that file. I would request you to update bicode version too since some users might use your file from their interface.

Note: Biicode version of Catch is created by Biicode user Florian Wolters (GitHub user @florianwolters).

@martinmoene I've commented on their forums , I hope it would get updated soon. Can you look into the gcc errors as well? Even after removing those git merge affected lines , it fails to build on gcc with given errors.

do you enable c++11?

yes ( via compiler flags , right? )

This looks like it's the same problem as #424 which was caused by a mismatch between compiler & standard library versions

I am also facing the problem that catch based projects no longer build under arch linux, which currently uses gcc 5.2.0.
Sorry for the german error messages.
/tmp/libjson-rpc-cpp/build/catch/src/catch/include/internal/catch_evaluate.hpp:43:17: Fehler: »nullptr_t« in Namensbereich »std« bezeichnet keinen Typ
     inline std::nullptr_t opCast(std::nullptr_t) { return nullptr; }
                 ^
/tmp/libjson-rpc-cpp/build/catch/src/catch/include/internal/catch_evaluate.hpp:165:58: Fehler: »template<Catch::Internal::Operator Op, class T> bool Catch::Internal::compare« steht mit vorheriger Deklaration in Konflikt
     template<Operator Op, typename T> bool compare( std::nullptr_t, T* rhs ) {
                                                          ^
/tmp/libjson-rpc-cpp/build/catch/src/catch/include/internal/catch_evaluate.hpp:159:44: Anmerkung: vorherige Deklaration »namespace Catch::Internal { }::compare«
     template<Operator Op, typename T> bool compare( T* lhs, int rhs ) {
                                            ^
/tmp/libjson-rpc-cpp/build/catch/src/catch/include/internal/catch_evaluate.hpp:165:53: Fehler: »nullptr_t« ist kein Element von »std«
     template<Operator Op, typename T> bool compare( std::nullptr_t, T* rhs ) {
                                                     ^
/tmp/libjson-rpc-cpp/build/catch/src/catch/include/internal/catch_evaluate.hpp:165:70: Fehler: expected primary-expression before »*« token
     template<Operator Op, typename T> bool compare( std::nullptr_t, T* rhs ) {
                                                                      ^
/tmp/libjson-rpc-cpp/build/catch/src/catch/include/internal/catch_evaluate.hpp:165:72: Fehler: »rhs« wurde in diesem Gültigkeitsbereich nicht definiert
     template<Operator Op, typename T> bool compare( std::nullptr_t, T* rhs ) {
                                                                        ^
/tmp/libjson-rpc-cpp/build/catch/src/catch/include/internal/catch_evaluate.hpp:165:76: Fehler: Ausdrucksliste als zusammengesetzten Ausdruck in Initialisierung behandelt [-fpermissive]
     template<Operator Op, typename T> bool compare( std::nullptr_t, T* rhs ) {
                                                                            ^
/tmp/libjson-rpc-cpp/build/catch/src/catch/include/internal/catch_evaluate.hpp:165:78: Fehler: expected »;« before »{« token
     template<Operator Op, typename T> bool compare( std::nullptr_t, T* rhs ) {
                                                                              ^
/tmp/libjson-rpc-cpp/build/catch/src/catch/include/internal/catch_evaluate.hpp:168:66: Fehler: »std::nullptr_t« wurde nicht deklariert
     template<Operator Op, typename T> bool compare( T* lhs, std::nullptr_t ) {
                                                                  ^
/tmp/libjson-rpc-cpp/build/catch/src/catch/include/internal/catch_evaluate.hpp:168:44: Fehler: Redefinition von »template<Catch::Internal::Operator Op, class T> bool Catch::Internal::compare(T*, int)«
     template<Operator Op, typename T> bool compare( T* lhs, std::nullptr_t ) {
                                            ^
/tmp/libjson-rpc-cpp/build/catch/src/catch/include/internal/catch_evaluate.hpp:159:44: Anmerkung: »template<Catch::Internal::Operator Op, class T> bool Catch::Internal::compare(T*, int)« wurde bereits hier deklariert
     template<Operator Op, typename T> bool compare( T* lhs, int rhs ) {
                                            ^
In file included from /tmp/libjson-rpc-cpp/build/catch/src/catch/include/internal/catch_expression_lhs.hpp:13:0,
                 from /tmp/libjson-rpc-cpp/build/catch/src/catch/include/internal/catch_result_builder.h:91,
                 from /tmp/libjson-rpc-cpp/build/catch/src/catch/include/internal/catch_capture.hpp:11,
                 from /tmp/libjson-rpc-cpp/build/catch/src/catch/include/catch.hpp:34,
                 from /tmp/libjson-rpc-cpp/src/test/test_stubgenerator.cpp:13:
/tmp/libjson-rpc-cpp/build/catch/src/catch/include/internal/catch_tostring.h:56:38: Fehler: »std::string Catch::toString« als andere Symbolart redeklariert
 std::string toString( std::nullptr_t );
                                      ^
/tmp/libjson-rpc-cpp/build/catch/src/catch/include/internal/catch_tostring.h:53:13: Anmerkung: previous declaration »std::string Catch::toString(unsigned char)«
 std::string toString( unsigned char value );
             ^
/tmp/libjson-rpc-cpp/build/catch/src/catch/include/internal/catch_tostring.h:56:23: Fehler: »nullptr_t« ist kein Element von »std«
 std::string toString( std::nullptr_t );

@cinemast: which build is that from?
I did make some changes to the detection of nullptr support in GCC just recently. It's possible those changes either caused your issue or may, indeed, fix them. Or it could be something else (but either way you can try with a global define of CATCH_CONFIG_NO_NULLPTR to see if that fixes it for you).

Of course I hadn't actually pushed those changes!
I've done that now: v1.2.1-develop.11

My German's not good enough to translate the error messages, but this looks exactly the same as #424. It's a problem with the compiler/stdlib setup, not with Catch.
@cinetmast Please try compiling this with g++ -std=c++11:
#include <cstddef>
const std::nullptr_t p;

If that doesn't compile, then the problem is with your compiler setup.

Ran a few errors through Google Translate. Exact same messages as @PureAbstracr referred to.

s/@PureAbstracr/@PureAbstract/ (I'm going to kill this Android phone keyboard../)

Thanks for the hints. I will try it and get back here.

Hi!
I finally got to test the suggestions. For me, it was the missing g++ -std=c++11. I don't know why it worked with gcc version < 5. I did not add the compile flag there. Anyway, for me it works now.


Stop SHA errors related to: Token Length Exception

Some websites and providers generate a secret key that technically is not the proper length to use in generating a OTP. If you pad from the left when such a secret key is detected you can fix the key length without altering the output. This is needed for example with websites like Amazon that currently return a 52 character secret key.
NOTE: This changes was only done in the jsOTP-es5.js and jsOTP.js files. These files still need to be minified and then minified gzipped.

Hi,
Your fix does not seem to work.
My suggestion is this:
var checklength = bits.length - bits.length % 8;
i = 0;
while (i + 4 <= checklength) {
chunk = bits.substr(i, 4);
hex = hex + parseInt(chunk, 2).toString(16);
i += 4;
}
Based on that in a working totp generator the last standalone HEX digit is ignored.


partial clone

Label -->Feature request/Enhancement.
Data repositories can be large it would be useful if OSFs python API allowed you to download just single files that you could use in a python script.
I can perform this functionality using Rs OSF API, but not with pythons API.
https://github.com/CenterForOpenScience/osfr/issues/91

(not a developer) but what you can do is use osfclients fetch to get single files

Oh cool. Thanks @behinger


~hentai

~hentai isn't a valid command anymore? Couldn't find any change regarding that on the Releases.

Are you hosting your own bot?

No, I am using the hosted one.

Can you try ~gelbooru and ~danbooru, and tell me if they work

~gelbooru is not working as well. ~danbooru is working

NadekoBot v0.971.zip
Can you tell me if this one fixes your issue? Probably just replacing the NadekoBot.exe will be enough.
Weird thing is that this is affecting NadekoBot on the server but not on my local PC. Maybe gelbooru got blocked in U.S? Just guessing.

The client-side one is working as intended. The server-side commands still aren't working. The U.S. block might be a possibility, as I don't live there.

What do you mean by client side and server side?

Ohh you said you don't host your own bot. Sorry. Yeah ok then ill try restarting nadeko with the new patch

Ok its fixed.


vpk file names do not support non-ascii characters

Describe the Bug
Open a vpk file with a filename containing non-ascii characters.
To Reproduce

Rename vpk filename as: "测试_test.vpk" or "テスト_test.vpk"
Open file via menu.
The error message pops up:
Unable to load this file. Please ensure that a game or another application is not using the file.

Operating System
Windows10

While I still want to fix this bug, please keep in mind the Source engine will not enjoy loading non-ascii filenames in a lot of cases

Having looked into this, this is almost impossible to fix with the way the library is currently written. Text encoding is a nightmare!!!! I will probably try to fix this at some point, but I can't make any guarantees. Closing as wontfix for now


Substitution of exe commands in tmux session not working as expected

Windows build number:  Microsoft Windows [Version 10.0.17763.437]
I have two distributions:
>wslconfig /l
Windows Subsystem for Linux Distributions:
Legacy (Default)
Ubuntu

Using Legacy distribution (which I installed about year and half ago, DISTRIB_DESCRIPTION="Ubuntu 16.04.6 LTS") I get:
$ tmux
$ echo $(cmd.exe /c echo Hello)
Hello
$

Using Ubuntu distribution installed from Microsoft Store (DISTRIB_DESCRIPTION="Ubuntu 18.04.2 LTS") I get:
$ tmux
$ echo $(cmd.exe /c echo Hello)

$

Not sure if a problem is in WSL, because without starting tmux session everything works fine.

There are sporadic issues between using Legacy edition and Store version. Legacy is not intended to be used. Uninstall it, reboot, and try again. If there's still an issue, provide an update though this may end up going to the Console issue page.

Looks like it was some kind of problem with Ubuntu 18.04 installation. Reinstalled it and now substitution works fine. Thanks!


UnLazy not working on imdb

Please add support for that website, I noticed that it does not work on any movie or list page, and does not detect anything (no numbers on icon in firefox). Your extension works perfect with wikihow tough and it really speeds up loading too, not just sparing my hands from scrolling!

oh thanks for notifying me,
I'll check right away.

nice discovery!
imdb puts the real SRC inside an attribute named loadlate.
also, if the page disables JavaScript, it tries to use the built-in lazyloading of the browser (new browsers have it) by placing loading="lazy" attribute on-to the IMG-element.

(really just note to myself..)

Right, I've updated the Chrome web-extension,
which you can find in https://github.com/eladkarako/chrome_extensions/releases/tag/LatestChrome
sadly Mozilla takes forever to sign web-extensions, so I'll just update it later (might took up to several days! I know right!?!?).

sadly Mozilla takes forever to sign web-extensions, so I'll just update it later (might took up to several days! I know right!?!?).
Hmmm... nevermind, it got approved right away for some reason.
get the latest XPI from here: https://github.com/eladkarako/chrome_extensions/releases/tag/LatestFirefox


Edit Attack roll formula

Is there a way to edit the formula for an attack roll when rolling an item?

Please head over to the #dnd5e channel on the Foundry discord for all general usage questions.


is there a way to get commissions?

The IB API includes in reqExecutions a way to get the commissions (fees) for each trade, but I don't see it being returned.

Looking at the source, there is support for CommissionReport, but unlike reqExecutions I don't see a way to request commissions. My goal is to get fees paid for executed trades.

From the docs(I suggest these over the ibpy source as ibpy is generated
from the java api), you should be able to register commissionReport and
then it will be called either automatically after a trade execution or when
reqExecutions is used.
Source:
https://www.interactivebrokers.com/en/software/api/apiguide/java/reqexecutions.htm
https://www.interactivebrokers.com/en/software/api/apiguide/java/commissionreport_event.htm
On Thu, Nov 24, 2016 at 11:07 AM, expectedvalue073<EMAIL_ADDRESS>
wrote:

Looking at the source, there is support for CommissionReport, but unlike
reqExecutions I don't see a way to request commissions. My goal is to get
fees paid for executed trades.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/blampe/IbPy/issues/35#issuecomment-262808498, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AIHXdqupwlwmxsVAqMFbbAw9wzuRN8Knks5rBbYwgaJpZM4K7yu-
.

I have reqExecutions and I added message.commissionReport as a callback, but I simply do not get calls on the callback intended for commissions.

To add, I have a callback registered for "ExecDetails" and that one works fine. I can get a list of all executed trades. But the commission callback does not get called.

I wrote a print statement on line 35 of opt/dispatcher.py and here is what I got:

Server Version: 76
TWS Time at connection:20161124 14:08:10 EST
managedAccounts {'accountsList': 'xxxxxxxxxxxx'}
nextValidId {'orderId': 1}
error {'errorCode': 2104, 'errorMsg': 'Market data farm connection is OK:usfarm.us', 'id': -1}
error {'errorCode': 2104, 'errorMsg': 'Market data farm connection is OK:usfarm', 'id': -1}
error {'errorCode': 2106, 'errorMsg': 'HMDS data farm connection is OK:ushmds', 'id': -1}
execDetails {'reqId': 1, 'execution': <ib.ext.Execution.Execution object at 0x000000000346D5C0>, 'contract': <ib.ext.Contract.Contract object at 0x000000000347EB00>}
execDetails {'reqId': 1, 'execution': <ib.ext.Execution.Execution object at 0x000000000347EAC8>, 'contract': <ib.ext.Contract.Contract object at 0x000000000347EB00>}
execDetails {'reqId': 1, 'execution': <ib.ext.Execution.Execution object at 0x000000000347EB00>, 'contract': <ib.ext.Contract.Contract object at 0x000000000347EAC8>}
...
execDetails {'reqId': 1, 'execution': <ib.ext.Execution.Execution object at 0x000000000347EB00>, 'contract': <ib.ext.Contract.Contract object at 0x000000000347EAC8>}
execDetailsEnd {'reqId': 1}

I don't see anything regarding commissions, and I have commissions registered.

I added another print in line 122 of ext/EReader.py
print msgId

and I do not see a 59 (COMMISSION_REPORT), but a lot of 11's (EXECUTION_DATA).

Could this be related to your issue with phone app executions RE orderIDs? If so, you could try changing your ibpy script to use the master client id when connecting. This is used to monitor executions across a wider scope but  for other api clients on the network and for TWS itself I believe.
Source (the section re Master API client ID):
https://www.interactivebrokers.com/en/software/tws/usersguidebook/configuretws/apisettings.htm

You are right. The Master API client ID solved it.


Question About Two-Stage Training Setup

Hello,
Thank you for sharing your excellent work on Transfuser++. I really appreciate the clarity and depth of your paper—it’s been very inspiring and helpful for my research.
I have a question regarding the two-stage training mentioned in your paper. For the second stage, it’s stated that the learning rate is set to 0.0003 and reduced to 0.00003 at the 30th epoch, with a total of 31 epochs. I’d like to confirm:
1.Are the learning rate and epoch settings the same for the first stage?
2.For the first stage, is the only required parameter change to set use_controller_input_prediction to 0?
Thank you in advance for your time and assistance!
Best regards,
Xinxing

yes
yes, --use_controller_input_prediction 0 --use_wp_gru 0 both need to be 0 for the first stage. In the second stage you set the load_file argument to the last model of the first stage and --continue_epoch 0.

yes
yes, --use_controller_input_prediction 0 --use_wp_gru 0 both need to be 0 for the first stage. In the second stage you set the load_file argument to the last model of the first stage and --continue_epoch 0.

Thank you very much for your prompt reply!! Will try it!


Add ability to call sqlite3_threadsafe to SQLitePCL.raw

Doesn't seem to be there. We're currently P/Invoking this for logging purposes, but can't anymore with Android N. It's a nice-to-have.

Planned for 0.9.4

Fixed in the 1.0 release


Healthcheck fails on OpenResty

I've been actively using version 3.5.5 for CrowdSec bouncers (firewall bouncer and django bouncer for django apps). However I wasn't able to implement nginx bouncer due to lack of lua package in version 3.5.5. So, I switched to 4.0.0-openresty-alpine-fat this weekend, however I couldn't get it running in a healthy state.
Healthcheck fails and the container stays in unhealthy condition. Pls find below the Health log from inspect:
----------------------------------------
"Health": {
                "Status": "unhealthy",
                "FailingStreak": 4,
                "Log": [
                    {
                        "Start": "2024-03-17T21:37:51.899899623+03:00",
                        "End": "2024-03-17T21:37:51.962438883+03:00",
                        "ExitCode": 7,
                        "Output": ""
                    },
                    {
                        "Start": "2024-03-17T21:38:21.967048446+03:00",
                        "End": "2024-03-17T21:38:22.016149571+03:00",
                        "ExitCode": 7,
                        "Output": ""
                    },
                    {
                        "Start": "2024-03-17T21:38:52.019909614+03:00",
                        "End": "2024-03-17T21:38:52.076661981+03:00",
                        "ExitCode": 7,
                        "Output": ""
                    },
                    {
                        "Start": "2024-03-17T21:39:22.080164811+03:00",
                        "End": "2024-03-17T21:39:22.130057679+03:00",
                        "ExitCode": 7,
                        "Output": ""
                    }
                ]
            }

------------------------------------

I have no clue what I'm doing wrong?

Thanks for reporting. We'll look into, but it may take us a couple of days.
@TafkaMax, could you take a look?

Hi,
The log is as follows:
-------------------
ted@ash1:~$ docker logs nginx
/docker-entrypoint.sh: /docker-entrypoint.d/ is not empty, will attempt to perform configuration
/docker-entrypoint.sh: Looking for shell scripts in /docker-entrypoint.d/
/docker-entrypoint.sh: Launching /docker-entrypoint.d/20-envsubst-on-templates.sh
20-envsubst-on-templates.sh: Running envsubst on /usr/local/openresty/nginx/templates/conf.d/default.conf.template to /usr/local/openresty/nginx/conf/conf.d/default.conf
20-envsubst-on-templates.sh: Running envsubst on /usr/local/openresty/nginx/templates/conf.d/logging.conf.template to /usr/local/openresty/nginx/conf/conf.d/logging.conf
20-envsubst-on-templates.sh: Running envsubst on /usr/local/openresty/nginx/templates/conf.d/modsecurity.conf.template to /usr/local/openresty/nginx/conf/conf.d/modsecurity.conf
20-envsubst-on-templates.sh: Running envsubst on /usr/local/openresty/nginx/templates/nginx.conf.template to /usr/local/openresty/nginx/conf/nginx.conf
20-envsubst-on-templates.sh: Running envsubst on /usr/local/openresty/nginx/templates/modsecurity.d/modsecurity-override.conf.template to /usr/local/openresty/nginx/conf/modsecurity.d/modsecurity-override.conf
20-envsubst-on-templates.sh: Running envsubst on /usr/local/openresty/nginx/templates/modsecurity.d/setup.conf.template to /usr/local/openresty/nginx/conf/modsecurity.d/setup.conf
20-envsubst-on-templates.sh: Running envsubst on /usr/local/openresty/nginx/templates/includes/proxy_backend_ssl.conf.template to /usr/local/openresty/nginx/conf/includes/proxy_backend_ssl.conf
20-envsubst-on-templates.sh: Running envsubst on /usr/local/openresty/nginx/templates/includes/proxy_backend.conf.template to /usr/local/openresty/nginx/conf/includes/proxy_backend.conf
20-envsubst-on-templates.sh: Running envsubst on /usr/local/openresty/nginx/templates/includes/location_common.conf.template to /usr/local/openresty/nginx/conf/includes/location_common.conf
/docker-entrypoint.sh: Launching /docker-entrypoint.d/25-listen-on-ipv6-by-default.sh
25-listen-on-ipv6-by-default.sh: info: Getting the checksum of /usr/local/openresty/nginx/conf/conf.d/default.conf
25-listen-on-ipv6-by-default.sh: info: /usr/local/openresty/nginx/conf/conf.d/default.conf differs from the packaged version
/docker-entrypoint.sh: Launching /docker-entrypoint.d/30-tune-worker-processes.sh
/docker-entrypoint.sh: Launching /docker-entrypoint.d/90-copy-modsecurity-config.sh
/docker-entrypoint.sh: Launching /docker-entrypoint.d/91-update-resolver.sh
/docker-entrypoint.sh: Launching /docker-entrypoint.d/92-update-real_ip.sh
/docker-entrypoint.sh: Launching /docker-entrypoint.d/94-activate-plugins.sh
/docker-entrypoint.sh: Launching /docker-entrypoint.d/95-activate-rules.sh
/docker-entrypoint.sh: Configuration complete; ready for start up
2024/03/18 18:32:48 [notice] 1#1: ModSecurity-nginx v1.0.3 (rules loaded inline/local/remote: 0/797/0)
ted@ash1:~$

--------------------

This may not be so meaningful so I'm also posting the terminal messages:
--------------------
nginx        | /docker-entrypoint.sh: Launching /docker-entrypoint.d/30-tune-worker-processes.sh
nginx        | /docker-entrypoint.sh: Launching /docker-entrypoint.d/90-copy-modsecurity-config.sh
nginx        | /docker-entrypoint.sh: Launching /docker-entrypoint.d/91-update-resolver.sh
nginx        | /docker-entrypoint.sh: Launching /docker-entrypoint.d/92-update-real_ip.sh
nginx        | /docker-entrypoint.sh: Launching /docker-entrypoint.d/94-activate-plugins.sh
nginx        | /docker-entrypoint.sh: Launching /docker-entrypoint.d/95-activate-rules.sh
nginx        | /docker-entrypoint.sh: Configuration complete; ready for start up
nginx        | 2024/03/18 18:32:48 [notice] 1#1: ModSecurity-nginx v1.0.3 (rules loaded inline/local/remote: 0/797/0)
Gracefully stopping... (press Ctrl+C again to force)
dependency failed to start: container nginx is unhealthy
ted@ash1:~/docker/proxy$
--------------------

I made a brief comparison between 3.5.5 and 4.0.0 and figured that generate-certificate script is not listed in "/usr/local/bin/" folder for 4.0.0 (healthcheck script is there). Could this be the reason?

Let me also add this:
ted@ash1:~/docker/proxy$ docker exec -it nginx curl http://localhost:80/healthz

<title>503 Service Temporarily Unavailable</title>

503 Service Temporarily Unavailable

openresty

ted@ash1:~/docker/proxy$

Thanks @erseldev. The missing certificates aren't good but shouldn't be the reason for the issue. However, it looks like there's no endpoint for the health check.


translation

some variable / function / etc names are in hungarian, should be translated to english

Solved in:
https://github.com/ABC-iRobotics/iRob-FaaR/commit/ab543816892655a66f59af6c0a500b6a0c6b141e


Time delay of moving CursorColumn when mode switching

When press 'esc' with highlighted CursorColumn, the cursor would move left by 1 character immediately, while the highlighted column moves with delay.
To reproduce this, just simply highlight CursorColumn in ~/.vimrc, create a line with more than 1 character, move cursor right, and press 'esc' key.

Albert wrote:

When press 'esc' with highlighted CursorColumn, the cursor would move
left by 1 character immediately, while the highlighted column moves
with delay.
To reproduce this, just simply highlight CursorColumn in ~/.vimrc,
create a line with more than 1 character, move cursor right, and press
'esc' key.

The cursor moving left is just an indication that Vim found the Esc
character, but it's actually still waiting on what follows.  It could be
the start of an escape sequence.
Not sure if it's worth trying to fix this.  It gets tricky when the Esc
indeed turns out to be the start of an escape sequence.  Moving the
cursor is simple, redrawing the CursorColumn is not that simple.
--
TIM: But follow only if you are men of valour.  For the entrance to this cave
is guarded by a monster, a creature so foul and cruel that no man yet has
fought with it and lived.  Bones of full fifty men lie strewn about its
lair ...
"Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

Mintty and some other terminal emulators support application escape key mode.
https://github.com/mintty/mintty/wiki/Tips#avoiding-escape-timeout-issues-in-vim
https://github.com/mintty/mintty/wiki/CtrlSeqs#escape-keycode
This can distinguish the Esc character and the start of an escape sequence.
It would be nice if Vim supports the application escape key mode.
(I'm not sure this can solve the reported issue, though.)

There is a wide spread solution: use 8-bit mode.  Then CSI is used to start an escape sequence.
Problem is not that the terminal can't do that (most can), but that most application can't interpret these CSI codes.


Add k8ssandra demo support to CLI

Summary: We want to add a K8ssandra demo to our list of demos. A few changes needed to be made to ensure the demo can be deployed properly:

We should check for cert-manager as a required dep. We decided not to deploy this with the demo, since deploying 2 cert-managers can break things. If we deploy cert-manager for the user, there's a bit of bookkeeping we'd need to do if the user runs px demo delete to decide whether or not we should also clean up cert-manager. Instead, we'll leave this up to the user for the time-being.
The demo deploys some CRDs. This can lead to race conditions if deploying everything in the manifest at once. To handle this, added a retry in the deploy block.

Relevant Issues: N/A
Type of change: /kind cleanup
Test Plan: Push to dev demo manifest, px demo deploy with and without cert-manager installed.

Can we add https://github.com/pixie-io/pixie/issues/680 to the relevant issue section in the description?


feat: new alias command

[x] Mathlib patch: mathlib4#6172

Implementation of the revised alias command discussed on Zulip.
Supersedes #184

I think you should rebase this on top of #184's commits, so it is easier to review what changed and ensure that modifications from that PR are reflected here (which does seem to be the case)

The original Mathlib alias had a comment to add an alias attribute. I'm not sure what that attribute was intended to do other than "I'm an alias of". I think even that basic functionality makes some sense but is it needed?

The attribute was intended to be usable so that third party code could programmatically determine that a declaration is an alias of another.

Should that be implemented as an attribute or as an environment extension? I'm thinking the latter since I don't see much need for @[alias ...].

Yes, it supports all modifiers and attributes the same way as def and theorem do!

Should that be implemented as an attribute or as an environment extension? I'm thinking the latter since I don't see much need for @[alias ...].

At some point I wrote a linter for duplicate lemmas and the attribute was useful as a way of skipping intentional duplicates. The interaction with to_additive  was a little subtle here and it was important for that application that to_additive could copy the alias attribute to an additive version of an existing alias.


Is detection of bash version really necessary when installing shell completions

Thank you for the great tool!
I'm wondering if it is necessary to detect the bash version when trying to generate the bash shell completions.
Generally speaking, I may be trying to maximize compatibility with an older system, where bash may be older than 4.4
I understand that it will only "work" with newer bash. but does the "generation" of the shell completions need newer bash?
Maybe a warning is more appropriate?
https://github.com/pallets/click/blob/ef11be6e49e19a055fe7e5a89f0f1f4062c68dba/src/click/shell_completion.py#L302

As a comparison, fish and zsh do not check for version (or as it seems, existence)

We previously had issues with different versions, and requests for features that did not work on older versions. Given that we are not Shell Completion programmers, we opted to limit the maintenance surface we were taking on. You're welcome to add a custom completion provider with a script that works for other shell versions.

thank you for your response.


Changes the settings form colors to match the current design.

@morebrownies I updated the settings form to match the colors in Zeplin, take a look and make sure this is right.
@rmisio if these are correct, can you make the same changes in the address PR?

Was this one merged with the other settings merges today and we can close this?
If not, then there are some conflicts to fix.

All of these changes have already been merged in.


Do not rise error in Interactions.addPluggable

Issue #1075
Description of changes:
Do not raise an error in Interactions.addPluggable if type pluggable.getProviderName() does not match, just silently ignore it.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

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

@@            Coverage Diff            @@
##           master   #1076      +/-   ##
=========================================
+ Coverage   87.18%   87.2%   +0.02%
=========================================
  Files          85      85
  Lines        4096    4095       -1
  Branches      783     783
=========================================
  Hits         3571    3571
+ Misses        502     501       -1
  Partials       23      23

Impacted Files
Coverage Δ

...kages/aws-amplify/src/Interactions/Interactions.ts
93.47% <ø> (+1.98%)
:arrow_up:

Continue to review full report at Codecov.

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


Binary release files shouldn't be part of the repo

The issue
Hi, I love this app, use it all the time, and I'm thankful for all the effort the creator and the community has put into it.
I would love to contribute to the project but there's one thing giving me some pause. If you clone the main branch, the repo will weight more than 7GB (almost 8 at the moment of writing) (and rapidly growing).
Caused by including the compiled binary with the code, it's growing about 40+MB each release.
I don't know the rationale behind including the binary this way, but it's not common for a project to do that.
Potential fix | use the release attached files
If this is being done to facilitate the auto-update feature of the app, I don't think there's an actual need for including the binary like that. You could just access the latest release using the latest tag. For example:
https://github.com/ShootMe/FallGuysStats/releases/latest/download/FallGuysStats.zip
You can even have other tags, for beta releases, etc. Some projects have a text file with links to different releases and validate those client side.
Benefits of committing binary builds to the repo?
I don't see any benefit to having it as a version-controlled file inside the actual code branch. But I'm open to hearing them.
If for some reason you need to have them in the repo, you could create a branch with the sole purpose of having this binary files. And never merge it back to main/master. Then developers could clone the main/master branch without downloading GBs of previous binary release files.
PD: I see there are 4 .zip files (2 different binaries but 4 .zip files). I don't know why, but I'm sure there is a solution without having to constantly commit those to master.

Delete the existing repository and try cloning again.


add servicemonitor

Towards: https://github.com/giantswarm/roadmap/issues/382
Add ServiceMonitor for the analytics-server so its scrapped by Prometheus.
Checklist

[ ] Update changelog in CHANGELOG.md.

Tested and got metrics in prometheus

It seems fine to me but can you also remove the monitoring annotations to the service ?

What do you want to remove namespace=monitoring annotation ?

Sorry for my lack of context. I already added the giantswarm.io/monitoring annotation on the service manifest: https://github.com/giantswarm/macropower-analytics-panel-server-app/blob/5e9e6d67f7dfa4f63ae9b8b310e262c4dcae62d3/helm/macropower-analytics-panel-server/templates/service.yaml#L8 so it needs to be removed to ensure the service monitor wors :)

Perfects not more duplicates :ok_hand:


Authenticate using User JWT-Token

Closes #238
Introduces a user_token parameter to the arango.client.ArangoClient.db method to build a JwtConnection object without a username/password
also introduces arango.connection.JwtConnection.set_token
import os
from arango import ArangoClient

token = os.environ["ARANGODB_USER_JWT"]
db = ArangoClient().db('test', user_token=token)

db.conn.refresh_token() # Raises `arango.exceptions.JWTRefreshError`, since `username/password` is not set
db.conn.set_token(token=...) # Valid

# TODO: Should we support this? (see PR comment below)
db = ArangoClient().db('test', username='root', password='test', user_token=token)

Codecov Report

Merging #293 (34aaa13) into main (744a6e9) will decrease coverage by 0.57%.
Report is 22 commits behind head on main.
The diff coverage is 92.05%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff             @@
##             main     #293      +/-   ##
==========================================
- Coverage   99.01%   98.44%   -0.57%
==========================================
  Files          26       26
  Lines        3958     4062     +104
==========================================
+ Hits         3919     3999      +80
- Misses         39       63      +24

Files
Coverage Δ

arango/aql.py
95.04% <100.00%> (ø)

arango/client.py
98.46% <100.00%> (+0.04%)
:arrow_up:

arango/cluster.py
100.00% <100.00%> (ø)

arango/exceptions.py
100.00% <100.00%> (ø)

arango/executor.py
99.35% <100.00%> (-0.13%)
:arrow_down:

arango/http.py
100.00% <ø> (ø)

arango/job.py
100.00% <100.00%> (ø)

arango/request.py
94.44% <100.00%> (ø)

arango/typings.py
100.00% <100.00%> (ø)

arango/utils.py
87.50% <100.00%> (-12.50%)
:arrow_down:

... and 3 more


Login problem  403 font

I use the lastest version
I  can't login, how i can fix it plz ? Thx

Hey @nisked, thank for raising this with me, I'm not sure why your getting that error.
I will ha e to look into it and correct the issue in an update as it seems to be contacting an outside server to generate the styling in screen.
I the mean time, is there anything that's blocking your browser from Web access just to rule that out? Like a firewall or ad blocker?

No i try with 3 browser and 2 internet connection :/

That's ok, thank you, I'll have a look and get back to you when I have something.
How are you setup? Using docker, windows, Linux?

Hey @nisked,
I have had a look over my version and I am not seeing this issue, I have double checked and cannot find this in the latest build.
This was however, an old issue from #11, I removed the link to the source you are erroring on.
Can you please double check your version of Requestrr? The version will show in the logs when starting up.

Hey @nisked,
Just following this up, are you still having this issue?

Hey @nisked,
I haven't heard form you regarding this issue, please reach out if you're still having issues.
This issue was corrected in V2.1.5 and I can't replicated it with the latest build.
I will mark this as closed if I receive no response.


feat(utils): add text email generation utility methods

Requirements:

generateTextEmailFromHTML function which will take the HTML string and convert it to plaintext version
generateTextEmail function which will accept the JSX Layout and convert that into plaintext format using generateEmail and generateTextEmailFromHTML functions

The Pull Request covers:

generateTextEmailFromHTML utility function
generateTextEmail function
Tests for generateTextEmailFromHTML and generateTextEmail methods

Resolves #60
Signed-off-by: Niloy Sikdar<EMAIL_ADDRESS>

I think you need to have tests for all types of components we have to make sure all the components turn to the correct text version of them.

I think you need to have tests for all types of components we have to make sure all the components turn to the correct text version of them.

Yeah, make sense, doing the changes.


xlink is errant calling exit() which forcibly destroys the main app itself -- should never force exit apps

After much debug and fixing thread/ownership issues in depthai startup/shutdown, I isolated that XLink is calling exit() which force terminates the main app itself. Naturally this is errant. XLink library should never exit() or abort() or quick_exit(). Instead it should use tools like assert(), returning error values from the function, etc.
Setup

Microsoft Windows [Version 10.0.19043.1348]
Xlink 4c14908
MSVS 2019 Community v16.11.6

Repro
Search codebase for exit( and all 3 are errant...
or...

Fix the threading/ownership issues in depthai startup/shutdown https://github.com/luxonis/depthai-core/issues/257
config, build=debug for depthai (which will pull in xlink)
Run repeatedly the example bootloader_version.exe in msvc debugger

Result
Crashes and/or app suddendly terminates with exit code=1
Expected
No crash, and clean exit with exit code = 0
Logs
After the hacks for threading/ownership above, enabling logging in XLink, adding a few mvLog() calls in XLink itself, and compiling XLink locally using DEPTHAI_XLINK_LOCAL I isolated the issue.
Found device with name: 14442C10C1A1C6D200-ma2480
[2021-11-15 16:16:36.412] [debug] connlinkid= 0    streamId= 0
creating... init::watchdogThread
Version: 0.0.15
DeviceBootloader about to be closed...
::close before join()
[2021-11-15 16:16:36.414] [debug] connlinkid= 0    streamId= 1
init::watchdogThread 2 before loop
init::watchdogThread 2 after loop 1
E: [global] [         0] [] addEvent:321	Condition failed: event->header.flags.bitField.ack != 1
E: [global] [         0] [] addEventWithPerf:333	 addEvent(event) method call failed with an error: 3
F: [global] [         0] [EventRead00Thr] dispatcherEventReceive:91	Duplicate id detected.
E: [global] [         0] [] XLinkReadData:183	Condition failed: (addEventWithPerf(&event, &opTime))
E: [global] [         0] [EventRead00Thr] handleIncomingEvent:568	Assertion Failed and exit(EXIT_FAILURE): stream
F: [global] [         0] [Scheduler00Thr] dispatcherResponseServe:776	no request for this response: XLINK_WRITE_RESP 1
init::watchdogThread 2 after readRaw
init::watchdogThread 2 before sleep in try
init::watchdogThread 2 after sleep in try
init::watchdogThread 2 exiting thread lambda...
The program '[11784] bootloader_version.exe' has exited with code 1 (0x1).

Look above at handleIncomingEvent:568	Assertion Failed and exit(EXIT_FAILURE): stream
This is ASSERT_XLINK(stream);
which is a macro that expands to (I added the extra "exit()" in the log so I can see behavior)
#define ASSERT_XLINK(condition) do { \
            if(!(condition)) { \
                mvLog(MVLOG_ERROR, "Assertion Failed and exit(EXIT_FAILURE): %s \n", #condition); \
                exit(EXIT_FAILURE); \
            } \
        } while(0)
    #endif // ASSERT_XLINK

I can see that original coder has a release/debug for these exit(). Since depthai and hunter compile the build_type as specified, and therefore apps link in that specified build type, it is errant to ever compile an exit() and provide that in any easy to use system via cmake, hunter, etc.
I see 3 places where exit() is called. I believe all of them should be altered to never exit().

remove the debug versions of those two macros
the 3rd function which hardcodes exit(1) can be replaced with something similar to..mvLog(MVLOG_ERROR, "connect(usbFdWrite,...) returned < 0\n");
close(usbFdRead);
close(usbFdWrite);
usbFdRead = -1;
usbFdWrite = -1;
return X_LINK_PLATFORM_ERROR;

After these XLink fixes, the example bootloader_version.exe is so far successful 100% of the time.
Further...other tests which would sometimes fail previously like rgb_preview.exe are also successful.
I do see other failures on other tests, so I'll keep searching for the source of those fails as a separate issue.

the specific issue of xlink calling exit() was resolved with https://github.com/luxonis/XLink/pull/20


Replace<Type, KeyOld, <KeyNew,TypeNew>> Utility Type

🔍 Search Terms
Replace<Type
Reaplce
replace type util
replace type utils
Replace<Type, KeyOld, <KeyNew,TypeNew>>
replace<type,
✅ Viability Checklist

[X] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[X] This wouldn't change the runtime behavior of existing JavaScript code
[X] This could be implemented without emitting different JS based on the types of the expressions
[X] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
[X] This feature would agree with the rest of our Design Goals: https://github.com/Microsoft/TypeScript/wiki/TypeScript-Design-Goals

⭐ Suggestion
Types utilities are marvellous, yet they don't have a simple Replace! It's not only nice to have, but would make it easier to replace types' key with new one with its associated type as below:
Replace<Type, KeyOld, <KeyNew,TypeNew>>

📃 Motivating Example
The type User has its updatedAt key replaced with updatedBy  along with the new type in NewUser type
type User = {
	id: string,
	firstName: string,
	lastName: string,
	createdAt: string,
	updatedAt: string,
}

Assuming we have type utility Replace<Type, KeyOld, <KeyNew,TypeNew>>
type NewUser = Replace<User, updatedAt, <updatedBy,{
	userId: string,
	updatedAt: string,
}>>

The NewUser type now looks like
type NewUser = {
	id: string,
	firstName: string,
	lastName: string,
	createdAt: string,
	updatedBy: {
		userId: string,
		updatedAt: string,
	},
}

💻 Use Cases

What do you want to use this for?
replacing types' key on the and making new ones
What shortcomings exist with current approaches?
using Omit or Exclude to make new type with the replaced key. The Replace<Type, KeyOld, <KeyNew,TypeNew>> is one easier utility.
What workarounds are you using in the meantime?
a combination of Omit and & or  Exclude and & to replace the current key

See https://github.com/microsoft/TypeScript/issues/55301


Add prettier

e90654f
hey  i didn't find any contributing guidelines so i just made a pull request can you check if that's all i have to do..? for adding prettier..


Where are "core.h" y "imcore.h" files?

This project uses imlab library. You can find the precompiled library and header files under library folder . If the precompiled library doesn't work for you, you should rebuild the library first. After creating the library files, you should set the IMLAB_LIBRARY_DIRECTORY variable in CMakeList.txt to the newly created library directory.

Appreciate it!


[SPARK-48332][BUILD][TESTS] Upgrade jdbc related test dependencies

What changes were proposed in this pull request?
The pr aims to upgrade jdbc related test dependencies, include:

com.mysql:mysql-connector-j from 8.3.0 to 8.4.0
com.oracle.database.jdbc:ojdbc11 from <IP_ADDRESS>.09 to <IP_ADDRESS>.05

Why are the changes needed?

com.mysql:mysql-connector-j from, release notes:
https://dev.mysql.com/doc/relnotes/connector-j/en/news-8-4-0.html

com.oracle.database.jdbc:ojdbc11, release notes:
https://download.oracle.com/otn-pub/otn_software/jdbc/23c/JDBC-UCP-ReleaseNotes-23ai.txt?AuthParam=1716161887_dbf7a096828486d544bee00a2383f42a
=======================================================================================
Known Problems Fixed in the Patch Release <IP_ADDRESS>.05
=======================================================================================
Bug 36279736 - CONNECTION TO A PROXY USER FAILS WITH INVALID USERNAME/PASSWORD WHEN WALLET IS PROVIDED
Bug 36187019 - LOB PROCESSING FAIL WHEN DB SET WITH EL8ISO8859P7 CHARACTER SET
Bug 36152805 - GET CONNECTION AGAINST NORMAL PDB WITH TFO=ON SHOULD FAIL WITH ORA-18739

Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass GA.
Was this patch authored or co-authored using generative AI tooling?
No.

Merged to master. Thank you all


⚠ T:Riza Api has degraded performance

In 433b245, T:Riza Api (https://infra.triza.dev/api) experienced degraded performance:

HTTP code: 200
Response time: 1527 ms

Resolved: T:Riza Api performance has improved in d427d71.


feat: allow to use port placeholder in client.port

Summary
Allow to use '<port>' placeholder in client.port.
Checklist

[x] Tests updated (or not required).
[x] Documentation updated (or not required).

Can we useport as the default value of client.port?

No, see https://github.com/web-infra-dev/rsbuild/pull/2639 for some backgrounds


Minor Mistake in Example Code

In the example you offer on the front page you define a "section0":
<!--sec data-title="Introduction" data-id="section0" data-show=true ces-->

but then the example button to go with it targets "section1":
<button class="section" target="section1" show="Show next section" hide="Hide next section"></button>

Obviously this is a minor issue, but it would be nice if it could be fixed so that new users could use the example code "out of the box."
P.S. Even when I fix section1=>section0 the example code still doesn't work for me.  For some reason the "hidden" section starts out visible (even if I change data-show=false or add a data-collapse=true), and clicking the button doesn't hide it.  Just thought you might want to know.

Thanks for the report. I will look into it and update the documentation accordingly.


 causes the page to crash on PageSpeed insights

Describe the bug
When using the new <SanityLive /> component from defineLive(), the website crashes when I run it through PageSpeed Insights. According to sentry.io the crash is inside the <SanityLive />-component when it tries to verify connection. See screenshots:

To Reproduce
Steps to reproduce the behavior:
Deploy a website that renders <SanityLive />. Run it through pagespeed insights.
Expected behavior
No crash
Which versions of Sanity are you using?
@sanity/icons                3.4.0 (up to date)
@sanity/image-url            1.0.2 (up to date)
@sanity/preview-url-secret   2.0.0 (up to date)
@sanity/ui                  2.8.13 (up to date)
@sanity/uuid                 3.0.2 (up to date)
@sanity/vision              3.62.3 (up to date)
sanity                      3.62.3 (up to date)

@dagstuan can you try https://github.com/sanity-io/next-sanity/releases/tag/next-sanity-v9.8.10 and tell me if it works out for you?

@stipsan Seems to work fine now! Thanks for the fix 🥳