.Net Core 2.0 + 2.2 High Memory Usage On Startup

Describe the issue
When spinning up an .Net Core Application after an application pool recycle in IIS, the process seems to go on a few memory grabbing runs but doesn't appear to be using it - This doesn't look like a memory leak as the memory is always reclaimed.
For example, the memory usage will cycle upwards to around 3-4GB, back down to about 400mb, then up and down between these a handful more times before settling down to ~200mb. The application starts as normal through this process, and can be accessed and used as normal.
Normal usage averages at around 200mb.
We've managed to take a dump at it's peak;

To Reproduce

.Net Core 2.0
.Net Core Runtime 2.0.7
Windows Server 2016
Workstation GC Mode

We haven't been able to reproduce this outside of our production server environments.
Is there a way we can determine what is causing this high memory usage during startup?
Edit
I've also been able to reproduce this with;

.Net Core 2.2
.Net Core Runtime 2.2.4
Windows Server 2016
Workstation GC Mode

We've managed to take a dump at it's peak;

Would you be willing to share the dump itself privately with us so we can examine it further?

Yeah we can do that. How’s best to get that to you, and do you have a preferred zip format?

You can send it to me at [my github<EMAIL_ADDRESS>I should be fine with most formats, as long as 7-zip can open it ;).

Thanks, I've send a share link over on OneDrive :)

@anurse
We've been looking at this further and we currently believe this is related to #2737.
Using the recommended workaround on that issue, to switch the ServiceProviderMode to Runtime appears to have resolved this issue in our Staging environment.
As there is no documentation on ServiceProviderMode (that I can find ...) can you offer some guidance on the difference between the two modes?
I can see that the default Dynamic mode is compiled whereas runtime is resolved at request, am I correct in thinking we should only be expecting a small performance hit by switching mode until a fix is released?

You'd have to measure in your scenario to know for sure. Here's a brief summary of the modes:

Runtime means that every time a service needs to be created, we basically use Activator.CreateInstance to do so. For singletons, this has minimal impact since they are created once per app. For transient and scoped services, reflection is much slower than directly calling new, so it has a noticible impact
Dynamic means that the first time a service needs to be created, we compile IL into a new DynamicMethod that will actually use new (or rather than newobj IL opcode ;)) to create an instance of the type. We then store that delegate and use it to construct instances each time. It's still a little slower than directly calling new but it's much faster than constructing the instance using reflection each time.

So I would expect some performance impact from switching modes.

Hi Andrew,
I can confirm this was the route cause of the high CPU on startup & the OutOfMemoryException's we were seeing throughout the day when the application was starting up again.
We are now averaging a slightly higher CPU usage, but thankfully it's nothing Drastic.
We'll keep an eye on #2737 relating to this - Thanks.

Using the Dynamic mode was the root cause? Do you have a large number of DI services? I believe you can query the count of the IServiceCollection at the end of your ConfigureServices to check this.
The fact that switching dramatically reduces CPU/Memory load indicates a very large number of DI services, all of which are used at startup, meaning we have to compile a lot of these dynamic methods.

@anurse
I’m going to reply on behalf of @adamriddick.
No we do not have a large number of DI services. The application is quite small. It is a custom version of Identity Server 4.
The more we look at it it looks like a bug/issue with the dynamic mode.
Did you manage to look at the dump Adam sent? That shows the issue quite well.

Interesting. I did look at the dump and responded indicating that it seemed to be missing data. I can try to take another look.

Yeah, I double checked the dump and it looks like it may be after the memory spike is cleaned up. For example, I see the total GC heap size reported as only about ~130MB:
0:000> !EEHeap -gc
Number of GC Heaps: 1
generation 0 starts at 0x0000016886f93a60
generation 1 starts at 0x0000016886aeea88
generation 2 starts at 0x0000016880001000
ephemeral segment allocation context: none
         segment             begin         allocated              size
0000016880000000<PHONE_NUMBER>001000<PHONE_NUMBER>3e45d0  0x73e35d0(121517520)
Large object heap starts at 0x0000016890001000
         segment             begin         allocated              size
0000016890000000<PHONE_NUMBER>001000<PHONE_NUMBER>5498b0  0x15488b0(22317232)
Total Size:              Size: 0x892be80 (143834752) bytes.
------------------------------
GC Heap Size:            Size: 0x892be80 (143834752) bytes.

I do see 4GB of Private Bytes recorded in the dump (the MEM_PRIVATE category in your results above), but it appears the dump occurred after a garbage collection but before the memory was returned to the OS so I'm not really able to see what is allocating the large amounts of memory.
Did you find more relevant data in the dump? That might help point me in the right direction. Is it possible for you to share the application code at all?

@anurse
Ah ok - I'll go through the dump again and try and locate what I found interesting.  But we saw 15000 items in a collection that I believe was related to the container and dynamic mode.  I will dig it up.
If we connect privately, we can share application code.
I am in Sydney, Adam is in UK.  So hopefully there should be crossover where we can jump on a call or something.

@anurse
Here is what I found in the dump file.  To be honest, now that I look at it, I really don't know whether this is normal or not, but I thought I would enclose it anyway.

Feel free to send me an email at [my github<EMAIL_ADDRESS>I'm happy to do what I can to help. We've got quite a bit going on with the upcoming 3.0 release, but I'll take a look where I can. If you have a time-sensitive need for support, Microsoft Support might be more appropriate.
Feel free to share additional data with me there. Probably the code I'm most interested in is where you register your services.
The TrueReadOnlyCollection<Expression> does come from the System.Linq.Expressions code which is what we use to generate DI factory methods, so it does seem somewhat relevant. Thanks for pointing me in this direction.

@anurse
Email sent :-)

I just wanted to provide an update here.
@anurse has been very helpful over email, but I am still a little stuck without knowing the ins and outs of the framework.  So hopefully someone else can point me in the right direction.

When we use the default ServiceProviderMode (Dynamic), we see two memory spikes on the first load of the razor views.  NB. Not on application start
When we set the ServiceProviderMode to Runtime, we do not see the memory spikes.  Standard memory tools such as dotMemory and Ants Memory Profiler have not been helpful as the memory appears to be unmanaged so these tools do not really show it.

Here is a standard memory trace from both dotMemory and ANTS.

The two spikes are very visible.
The most success I have had is using DebugDiag which said the following:
\clrjit.dll is responsible for 797.62 MBytes worth of outstanding allocations. The following are the top 2 memory consuming functions:clrjit!ArenaAllocator::allocateNewPage+93:  797.62 MBytes worth of outstanding allocations.

The report is here:
dotnet.exe__PID__23136__Date__06_27_2019__Time_02_46_41PM__22__Leak Dump - Virtual Bytes_MultipleRules.mht.zip
Can anyone point me in the right direction to answer the following questions.  @anurse has said that this finding is in line with his thoughts that the dynamic code-gen process (Dynamic mode) is responsible for this.  He said that in the memory dump, there were 16000 instances of the linq query to generate one particular class (DBContext).
My questions are:

Why would there be 16000 instances of the linq to generate a single class?  I don't know where to start looking into this.
Why would clrjit.dll be responsible for so many allocations?

2. Why would clrjit.dll be responsible for so many allocations?

This would make sense if there are a lot of code-generation paths going on.

Why would there be 16000 instances of the linq to generate a single class?  I don't know where to start looking into this.

This is likely the root cause of 2. I'm not sure where to start yet either but it's something we can continue to dive in to (cc @davidfowl)

I wonder if something is wrong with the EF configuration causing it to do weirdness (like making a bunch of internal service providers).
cc @ajcvickers @roji

@davidfowl What does the code that registers EF in D.I. look like?

@AdamRiddick and @gregpakes shared some of the code with me offline, I can loop you both in on the thread.


Accelerated Networking Documentation

Thanks for this page! One update could be to list which ones support accelerated networking. I was under the assumption that only VMs that have the option of >= 30,000 Mbps can do accelerated networking. For example, DSV2 supports accelerated networking, but it isn't mentioned on this page.

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

ID: 9bb1d37c-6337-0906-42ff-b5dbc6afdc8b
Version Independent ID: 72a96e16-1d2d-4c01-45be-3caf82470592
Content: Azure Windows VM sizes - General purpose
Content Source: articles/virtual-machines/windows/sizes-general.md
Service: virtual-machines-windows
GitHub Login: @jonbeck7
Microsoft Alias: jonbeck

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

@MicahMcKittrick-MSFT please assign to @steveesp who owns Accelerated Networking.

Is there a status on this?  Trying to find out what machines support Accelerated Networking is difficult at this point.

#reassign @steveesp

@steveesp and  adding @laurenhughes in case we need to add something to the existing VM size docs

This is documented here: https://docs.microsoft.com/en-us/azure/virtual-network/create-vm-accelerated-networking-cli#limitations-and-constraints

Documented here:
https://docs.microsoft.com/en-us/azure/virtual-network/create-vm-accelerated-networking-cli#limitations-and-constraints
/Steve
From: Cynthia Nottingham<EMAIL_ADDRESS>Sent: Thursday, August 8, 2019 4:55 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Steve Espinosa<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Accelerated Networking Documentation (#13906)
@steveesphttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fsteveesp&data=02|01|steveesp%40microsoft.com|f08554b2f61b4358725008d71c5bcb52|72f988bf86f141af91ab2d7cd011db47|1|0|637009052888205385&sdata=Bqw5du9fVk7KGlZ3%2FOjt3lhEm35r5VBR0hpDPUOIdgo%3D&reserved=0 and adding @laurenhugheshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Flaurenhughes&data=02|01|steveesp%40microsoft.com|f08554b2f61b4358725008d71c5bcb52|72f988bf86f141af91ab2d7cd011db47|1|0|637009052888215392&sdata=uU5tMbb85O0u8YjZN7URT%2BX1wuuEfKkDJc3DK40C21Y%3D&reserved=0 in case we need to add something to the existing VM size docs
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F13906%3Femail_source%3Dnotifications%26email_token%3DAE7GG5H7RRLSTD4F6C55F43QDSW4NA5CNFSM4FRKAICKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD35GVOA%23issuecomment-519727800&data=02|01|steveesp%40microsoft.com|f08554b2f61b4358725008d71c5bcb52|72f988bf86f141af91ab2d7cd011db47|1|0|637009052888215392&sdata=kKX3gyRu5utctr1az%2BAslCgPMmOKCrWrChsy%2BPaUAT8%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAE7GG5B4ZQ5UXJRMTRXYI3DQDSW4NANCNFSM4FRKAICA&data=02|01|steveesp%40microsoft.com|f08554b2f61b4358725008d71c5bcb52|72f988bf86f141af91ab2d7cd011db47|1|0|637009052888225396&sdata=g5SM8SIj6v8AI3X%2BQoN9UTYNotkQSkIP5e1EbIaFENw%3D&reserved=0.

@BlakeYarbrough Thank you for your feedback. It looks like the docs have been updated in this space, so I am going to close the issue for now. Please feel free to reopen this issue or start a new one if you have more feedback. Thanks again!
#please-close


ts: Migrate list_widget to TypeScript.

Migrates the list_widget to TypeScript.

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

@zulipbot add "mentor review"

@zulipbot add "area: typescript migration"

Left some more comments. Thanks for putting this all together! @Lalit3716

@PIG208 Thanks for the review!
Updates:

Made predicate and filterer options mutually exclusive.
Refined some types.

This is ready for another look!

https://github.com/zulip/zulip/blob/5734b9abc4064b38073c301fdccd07c3c89b117a/web/src/list_widget.ts#L436
The first condition is redundant.

This seems like a bug unrelated to the PR:
https://github.com/zulip/zulip/blob/5734b9abc4064b38073c301fdccd07c3c89b117a/web/src/list_widget.ts#L293-L298
$.find always returns a truthy value, so the check that follows is redundant.

I went over the PR again and posted some comments.
The prep commit sequence (first 4 commits) should be ready for an integration review.
The rest of them have some minor issues to be addressed. I ran eslint with a stricter setting to double-check. Eventually, we should have more confident assurance of its correctness as we progress further into the migration.

Updates:

Addressed reviews left by @PIG208

Question:
Regarding this comment comment I think we want to check the length of the JQuery collection here instead of checking the truthiness of the JQuery collection itself ?

Question: Regarding this comment I think we want to check the length of the JQuery collection here instead of checking the truthiness of the JQuery collection itself ?

Yeah, that sounds correct to me.

@PIG208 Made a new commit for the suggested change, ready for another look :)

Looks good to me!

Posted a handful of comments, otherwise lgtm.

@timabbott Addressed your review and left some comments for you. Also I have extracted two more commits for some changes which you pointed out in your review :)

This looks great, squashed the commits marked for squashing and merged, thanks for all the hard work on this @Lalit3716!


Library won't work with Sure Electronics 2416 Red from 2010

Hi,
recently I unearthed two Sure Electronics 2416 Red from 2010 and tested them with Bill Westfield's code (circa 2008 - "Random Walk" and other examples) and they both worked without a problem.
Unfortunately, they won't work with neither your library, nor the Adafruit HT1632 library. None of the examples work, nor any of the Adafruit ones. Tried with Arduino UNO, and an R3 Elegoo to make sure.
Has something changed since the 2008 code was written that would make them incompatible with your library?
What other information could I provide to find what the problem is?
As far as I know, I did the proper selection in the HT1632.h file and commented out the other boards. I used this one:
// SureElectronics 24x16 Monochrome LED Dot Matrix Unit Board
#define TYPE_2416_MONO 1

Thank you in advance.
Athanasios

a bit more context in case it matters:
The Chip is the old HT1632 and not the "C" variant.
The board has a code of: DE-DP016 VER1.1
Cheers

Oh man I haven't worked on this in ages. Going by this comment, you could try changing the initialization code to send the RC_MASTER_MODE command.
I remember the original datasheet had the entire initialization procedure laid out. You could compare that with what the initialize function does.

Oh man I haven't worked on this in ages. Going by this comment, you could try changing the initialization code to send the RC_MASTER_MODE command.
I remember the original datasheet had the entire initialization procedure laid out. You could compare that with what the initialize function does.

Thank you very much for the reply. I'll try your suggestion and post any findings here for others to access.
Also, thank you for making this library in the first place!
Cheers

I replaced the init code, and now I get some indications of life, but the examples and a test Sketch I wrote gives me wonky results.
The Heart example, only blinks on the Top right tile, and skips lines.
The sketch I wrote loops x (0-23) and y (0-15) using the HT1632.setPixel(  x , y ) and only lights LEDs up on the top 3 tiles and in a rather odd fashion.
I know this is not enough to get a definitive answer, so all I can ask for is "what am I looking for in the code"? I'm not a developer, and my coding skills are "spaghetti" grade (I can read code and copy paste).
I could post a video of what I get, and my code, but I don't want to waste your time :-)
Thanks

Solved. For anyone interested, you need to edit the "HT1632.cpp" under the aforementioned comment, and add:
writeCommand(HT1632_CMD_MSTMD);
change the:
writeCommand(HT1632_CMD_COMS01) to writeCommand(HT1632_CMD_COMS11);
(01 to 11).
This solution is for the board I mentioned in the OT, and make sure you edit the "HT1632.h" file appropriately after reading the comments.
Thank you Gaurav very much for providing the lead.

Awesome! I'm glad it worked.


Constants not loaded correctly when NODE_ENV set to "development"

Hi,
there seems to be a bug with the loading of basic constants affecting dev environments. In my case the constants were mostly loaded as "undefined" which leads to an url like this 'undefined://undefinedundefined/v2' which is obviously not working.
The issue seems to be for me in line 10 with process.env[key] !== "undefined", I think this should be changed to process.env[key] !== undefined for a correct comparison.
https://github.com/mondaycom/monday-sdk-js/blob/c27a313b0aa4209f4c56028fcd4e7e88cc1a9314/src/constants.js#L9-L11
I have in my .env file NODE_ENV=development.
I included the following vars in my .env which fixed it as a workaround for now:
# Temporary fix for monday.com api sdk
MONDAY_COM_PROTOCOL=https
MONDAY_COM_DOMAIN=monday.com
MONDAY_SUBDOMAIN_API=api.
MONDAY_OAUTH_SUBDOMAIN=auth.

I'm happy to assist in all follow-up questions 😊
Edit: It seems like there is already an open pr for that: #118

Seems to be fixed, closing.


VIN 1FBNE3BE1CDB23596 doesn't return any results

VIN('1FBNE3BE1CDB23596') returns an empty list []
additionally if doing a pyvin.clean_vins('1FBNE3BE1CDB23596') it also returns an empty
My guess is the VIN validator is considering this an invalid VIN, which is not the case. it is valid:
https://vpic.nhtsa.dot.gov/api/vehicles/DecodeVinValues/1FBNE3BE1CDB23596
@arpuffer not sure if you're still working on this, but here it is. I am not a software engineer myself, but a Product Manager dangerous enough with code, I would love to be able to solve this myself but I couldn't figure that out. Thank you very much in advance!

Check ErrorText--this VIN is not valid and while VPIC will attempt a decode it also tells you that it's invalid.  On the web UI things look a little more helpful
Error Text: 1 - Check Digit (9th position) does not calculate properly; 3 - VIN corrected, error in one position (assuming Check Digit is correct); 14 - Unable to provide information for some of the characters in the VIN, based on the manufacturer submission.
I have not been active on this project in a very long time, but it may be a reasonable change to have pyvin skip VIN validation and send to the VPIC API anyway. For the moment this code is functioning as expected, if a VIN is invalid we will not ask VPIC to decode it.

@lucasmocellin closing this issue in favor of https://github.com/arpuffer/pyvin/issues/9
PyVIN is currently working as expected, but I think the next version should work with VINs like the one you tried.

that makes sense, haven't noticed that ErrorText field. Thanks for the quick response and the complete explanation @arpuffer .


WSS support is broken

SslSource is not actually working because we don't setup root certificates nor use ctx.set_verify_mode(ssl::verify_none).
Proposed solution is to only support Ssl on Linux and load certificates from the standard paths. If Mac is easy enough to support then we could do that too for completeness sake.

Was fixed in #1140


Update to Argotronic.ArgusMonitor <IP_ADDRESS>65

Resolves #103354

Microsoft Reviewers: Open in CodeFlow

I finally confirmed.

@wingetbot run

[Policy] reset feedback

I can see that file.

Not necessarily the number 3, but only one manifest folder. .yaml files can be more than 3 depending on if we have more locale files for the package


invalid signature: for uri  calculated digest is '' but the xml to validate supplies digest ''

Good Day,
I have seen this reported a few different time with no resolution. I am hoping I would get some assistance in resolving this.
I am new to this type of feature, so any guidance or assistance is more than welcome. I have battling this issue for a few days now and I have no idea what to do next.
I am using version "xml-crypto": "^4.0.1"
node version v16.14.0
Error displayed as
invalid signature: for uri  calculated digest is 2U1suBt1sOA2olbnbMK1gC/3FHk= but the xml to validate supplies digest OGXcEIgUP1W+Hv9ghexl8gdMtrI=
Generated XML
<samlp:Response xmlns:samlp="urn:oasis:names:tc:SAML:1.0:protocol">
    <saml:Assertion MajorVersion="1" MinorVersion="1"
        AssertionID="_1" Issuer="mydomain.com"
        IssueInstant="2023-08-13T03:01:27.265Z" xmlns:saml="urn:oasis:names:tc:SAML:1.0:assertion"
        Id="_0">
        <saml:Conditions NotBefore="2023-08-13T03:01:27.266Z"
            NotOnOrAfter="2023-08-14T03:01:27.263Z" />
        <saml:AuthenticationStatement AuthenticationMethod="urn:oasis:names:tc:SAML:1.0:am:password"
            AuthenticationInstant="2023-08-13T03:01:27.268Z">
            <saml:Subject>
                <saml:NameIdentifier Format="urn:oasis:names:tc:SAML:1.1:nameid-format:unspecified"
                    NameQualifier="urn:mydomain.com">123356</saml:NameIdentifier>
                <saml:SubjectConfirmation>
                    <saml:ConfirmationMethod>urn:oasis:names:tc:SAML:1.0:cm:bearer</saml:ConfirmationMethod>
                </saml:SubjectConfirmation>
            </saml:Subject>
        </saml:AuthenticationStatement>
        <saml:AttributeStatement>
            <saml:Subject>
                <saml:NameIdentifier Format="urn:oasis:names:tc:SAML:1.1:nameid-format:unspecified"
                    NameQualifier="urn:mydomain.com">123356</saml:NameIdentifier>
                <saml:SubjectConfirmation>
                    <saml:ConfirmationMethod>urn:oasis:names:tc:SAML:1.0:cm:bearer</saml:ConfirmationMethod>
                </saml:SubjectConfirmation>
            </saml:Subject>
            <saml:Attribute AttributeName="version" AttributeNamespace="mydomain.com">
                <saml:AttributeValue>1</saml:AttributeValue>
            </saml:Attribute>
        </saml:AttributeStatement>
    </saml:Assertion>
    <samlp:Status>
        <samlp:StatusCode Value="samlp:Success" />
    </samlp:Status>
    <Signature xmlns="http://www.w3.org/2000/09/xmldsig#">
        <SignedInfo>
            <CanonicalizationMethod Algorithm="http://www.w3.org/TR/2001/REC-xml-c14n-20010315" />
            <SignatureMethod Algorithm="http://www.w3.org/2000/09/xmldsig#rsa-sha1" />
            <Reference URI="#_0">
                <Transforms>
                    <Transform Algorithm="http://www.w3.org/2000/09/xmldsig#enveloped-signature" />
                </Transforms>
                <DigestMethod Algorithm="http://www.w3.org/2000/09/xmldsig#sha1" />
                <DigestValue>OGXcEIgUP1W+Hv9ghexl8gdMtrI=</DigestValue>
            </Reference>
        </SignedInfo>
        <SignatureValue>
            HKJ/iE4XpJlF219H0zP8yK6SZO/Haz1234E09GTwD...[redacted]...tsRlIJX9nMtdg==</SignatureValue>
    </Signature>
</samlp:Response>

I used the following function to strip the spaces from the XML
function removeXMLSpaces(string){
  let xmlString = string;
  xmlString = xmlString.replace(/>\s*/g, ">");
  xmlString = xmlString.replace(/\s*</g, "<");
  xmlString = xmlString.replace(/\r\n?/g, "\n");
  xmlString = xmlString.replace(/(\r\n\t|\n|\r\t)/gm, "");
  return xmlString;
}

Here is my SignXML function used
function signXml(xml, options) {
  console.log({ options });

  const sig = new SignedXml(options);

  // sig.keyInfoProvider = new KeyProvider();

  sig.signatureAlgorithm = "http://www.w3.org/2000/09/xmldsig#rsa-sha1";

  sig.canonicalizationAlgorithm = "http://www.w3.org/TR/2001/REC-xml-c14n-20010315";

  sig.addReference({
    xpath: "//*[local-name(.)='Assertion']",
    transforms: ["http://www.w3.org/2000/09/xmldsig#enveloped-signature"]
  });

  sig.computeSignature(xml);

  fs.writeFileSync(process.cwd() + "/xml/signed.xml", sig.getSignedXml());

  let currentSignedXML = sig.getSignedXml();
  currentSignedXML = currentSignedXML.replace("Id=\"_0\"","").replace("URI=\"#_0\"","URI=\"\"");

  return currentSignedXML;
}

Let me know if I need to provide more details

So...i found the issue. I am not sure if this requires an update to validation.
For it to work, I had to add the transform for C14n. On reading the documentation, I believe that this was a default value but on inspecting the code in the package, I saw the value was missing in the validationXML which caused the mismatch.
Also note that I changes the xpath to "Response" and took @srd90 suggestion for appending the signature.
function signXml(xml, options) {
  const sig = new SignedXml(options);

  sig.signatureAlgorithm = "http://www.w3.org/2000/09/xmldsig#rsa-sha1";

  sig.canonicalizationAlgorithm =
    "http://www.w3.org/TR/2001/REC-xml-c14n-20010315";

  sig.addReference({
    xpath: "//*[local-name(.)='Response']",
    transforms: ["http://www.w3.org/2000/09/xmldsig#enveloped-signature", "http://www.w3.org/TR/2001/REC-xml-c14n-20010315"],
  });

  sig.computeSignature(xml, {
    location: {
      reference: "//*[local-name(.)='Status']",
      action: "after",
    },
  });

  let currentSignedXML = sig.getSignedXml();

  return currentSignedXML;
}

Can you site where in the documentation it mentions that default value, or, even better, put up a PR with a test that makes sure the defaults correspond to the spec?

@LoneRifle , do you have any thoughts on this? In all the refactoring, I don't think we touched anything related to any of these defaults in the code, but I'd have to double-check. In any case, the README should at least be clear to a reader, which it appears it currently isn't.

@cjbarth @srd90

Adding default at the beginning gave me the impression that if I did not set the value, this would be automatically chosen. Also in the example provided, nothing suggests that adding a value for canonicalizationAlgorithm may need to be added to addReference. If it did, it wasn't obvious to me

@CarlaMck77 , we're more than happy to help anyone through the process of submitting PRs. Having said that, in harmony with the approach taken with node-saml and passport-saml, we are removing as many defaults as possible from this project to force the consumer to think about the actions they are taking. Hopefully this will address some of the concerns/issues you have (had).
In any case, more PRs, especially those that clarify the documentation or increase test coverage, are very welcome.

@CarlaMck77 Again, thanks for pointing this out and @srd90 , thank you for providing specific details. I believe with the latest PRs that have landed there are no longer any confusing defaults set. If so, I believe we are ready for a semver-major release (pending a round of dependency updates). Please let me know if I've missed anything.


Timestamps fail to decode due to microseconds.

Common pg built-ins like now() add microseconds.
This causes decode_timestamp https://github.com/lpil/pgo/blob/3c7926125ff3050c1ac378eede6513762e0c0571/src/gleam/pgo.gleam#L531 to error as it's expecting #(#(Int, Int, Int), #(Int, Int, Int)) and receives #(#(Int, Int, Int), #(Int, Int, Float)).
Is this intentional?
Feels like changing the last element to a Float would work in both cases.

Hello! We've been having some discussion to determine what to do about this, and with time types in general. Sorry about the problem

Yeah, plenty of ways around it. Made the magic of squirrel decidedly less magical is all!
Thanks for taking a look.

This should be fixed now that pot has its own time type with microseconds!

Neat!

Thanks folks!


Make XXXWriter Pool configurable

It would be nice to have more control of Writers Pool, ie GraphiteWriterFactory.
When moving from GraphiteWriter to GraphiteWriterFactory, I noticed 60 establissed connections instead of 5/6, and it may be too high.
Ad minima, we should be able to configure

Pool Size
Max Idle
Min Idle

Strange, strange, strange... The pool of TCP connections used by GraphiteWriterFactory has a size of 1 (it still make sense to make it configureable). The issue is probably that multiple instances of GraphiteWriter2 are created. ServerListBuilder should take care of reusing the same instance (as long as its parameters are the same).
It looks like there is a bug just as much as a new feature request.


Houses vs Maxhouses | removing houses, but not allowing build more

Since playing with the multis again, i find out that while building house and then repacking it, it will correctly remove house from .houses.
I have correctly houses 0 and maxhouses=5. But since i builded and repacked 5 houses already, while testing, now I cannot build anymore.
I assume somehow even I have 0 actual houses, i fall into FALSE in this code.
This state get fixed with restarting Sphere.
    uint8 iMaxHouses = pChar->m_pPlayer->_iMaxHouses;
    if (iMaxHouses == 0)
    {
        if (pChar->GetClientActive())
        {
            iMaxHouses = pChar->GetClientActive()->GetAccount()->_iMaxHouses;
        }
    }
    if (iMaxHouses == 0)
    {
        return true;
    }
    if (GetHouseCountTotal() + iHouseCount <= iMaxHouses)
    {
        return true;
    }
    pChar->SysMessageDefault(DEFMSG_HOUSE_ADD_LIMIT);
    return false;

I think there was a fix for this issue, or at least I tried xD


Link to Page displaying JSON (406 code)

Preflight checklist

[X] I could not find a solution in the existing issues, docs, nor discussions.
[X] I agree to follow this project's Code of Conduct.
[X] I have read and am following this repository's Contribution Guidelines.
[X] This issue affects my Ory Cloud project.
[X] I have joined the Ory Community Slack.
[X] I am signed up to the Ory Security Patch Newsletter.

Describe the bug
With the Ory cloud free plan, when clicking on 'Invoices & Active Subscriptions' the page opens a new tab displaying JSON due to a 406 code. With a paid plan, you are redirected to stripe (i assume this is the intended behaviour).
See screen capture on Imgur
Reproducing the bug
If you have a free Ory Cloud account, navigate to 'Usage & Billing' then click on 'Invoices & Active Subscriptions'. A new tab opens displaying an error 406 in JSON format.
Relevant log output
No response
Relevant configuration
No response
Version
Cloud Console
On which operating system are you observing this issue?
macOS
In which environment are you deploying?
Ory Cloud
Additional Context
No response

@MicLeey the invoice button should not be visible or disabled if the user does not have yet paid anything. If no checkout was initiated, the user will not have a stripe account - hence the error from the backoffice.
Please note that being on the free plan is not enough for validation, as a user might have purchased something a year ago, switched to free, then needed invoices retroactively. The only question that matters is: does the identity have a stripe customer id?


Context menu got ruined in the last update.

Up until know, after I click the IP icon, in the tab that emerges with images I could right click on the images and then have the option to View Image, Open Image in Tab, Search By Image, etc...
Now its all gone, Firefox doesn't recognize them as images, and I HAVE TO download them. This is completely unpractical and I do not understand why was it changed. It was useful especially when I navigate Instagram for example, when I just want to take the clean image and not download it immediately.
Can I somehow use the previous version?

Well, I didn't expect that someone uses this extension as a simple image viewer. Starting from 0.8, images are displayed in buttons which probably causes this issue.

right click on the images and then have the option to View Image, Open Image in Tab, Search By Image, etc...

This is definitely a regression. Users should be able to copy the image URL with the context menu.

Can I somehow use the previous version?

You can download old versions from here:
https://addons.mozilla.org/en-US/firefox/addon/image-picka/versions/


Add 'file' support

@guozhenduo Can you provide a link to the homepage for this language?

https://github.com/guozhenduo/file
在 2020-08-29 19:18:08，"Richie Bendall"<EMAIL_ADDRESS>写道：
@guozhenduo Can you provide a link to the homepage for this language?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.


newbee question: How to do an offline application in c#/.NET?

Hi there,
I hope I do not bother you too much with a 100% new bee question. I'd like to use "deep zoom" within my offline standalone Windows .NET application (not silverlight, but .NET forms only) programmed in C#. I already used the DeepZoomTools.dll to generate a subdirectory which contains all these nice litte images which may certainly could be used for deep zooming. So far my project looked promising. But now I do not know how to continue. What I dream about is to program an image viewer (not silverlight based and not dependend of an internet browser) within my own little application. What I really need is a library which offers some functions for example returning the bunch of images needed for display my "deep zoom image" at current zoom or drag position.
Do you know if such a library compatible with .NET / C# is available somewhere?
(If only commercial libraries are available, I'd appreciate a hint, also)
Kindest regards and happy programming

Can you use JavaScript in your application, or include a web component? If so, OpenSeadragon should work for you.
I'm not aware of C# native zooming libraries, though they may certainly exist. Perhaps other people have thoughts they can add.

Well, it's definitely possible. I don't know that much about the C# world, so it may already exist. If it doesn't, though, you should be able to write it yourself based on the logic from OpenSeadragon.
Do you already have an image tiling system in OpenGL, and you just need to load the deep zoom format? Or do you also need to build the tiling and zooming code?

Hi IanGilman,
thanks a lot for your encouraging words. What I'd really love to implement is to "feed" my OpenGL engine with the bunch of images being output from the OpenSeadragon library or other library which does that for a DeepZoomImageformat. I'd do the tiling by my own, afterwards.
Something like that:
Bitmaps [ ] BunchOfBitmaps= HiNiceDeepZoomImageLibraryGetMeTheBunchOfImagesYouThinkAreNeeded(IamHereAtX,IamHereAtY,ThatIsMyZoom,ThereIsWhereIwantToGo,WithThisZoom);
If something commercial is on market (library to solve this), any hint is appreciated.
Kind regards

http://deepzoom.codeplex.com/ ?

I wrote a similar library in Java https://github.com/usnistgov/pyramidio (see the Read a DZI pyramid section).
Considering how close Java and C# syntax are, maybe you can translate it in C# for your needs?

Hi AvandeCreme,
WOOOOOOOOOOOOOOW, thanks a lot for your valuable hint. I'll give that a try. It really sounds promising. Super :-)


Test Failure

Hey, first of all thanks for this great library! I use this in an app of mine- it seems to work very well.
However, I suddenly seem to get test failures (macOS 10.4.4, Xcode 10.2).
The falling test is testRandomDataBlob, specifically the .lzfse decompression test. Apples data compression lib seems to make it hard to find out what the actual error is though. Do you have any idea whats up with that?
Btw the way this test is set up makes it look like more tests are failing after a first test failure.  However really only lzfse decompression fails.

Thanks!
Hmnnn.... OK. That is not very nice. I admit that the tests are bad structured. Testcases separated for each compression method would be better to catch that. My idea was to just test my wrapper code extensively and just assume that Apples code works. Well we see how good that worked out...
Generating a big blob of random data is also not very reproducible. I also had other complains that Apple's lz4 implementation is not compatible to other implementations. And my test showed that 50% of the time it doesn't compress the data at all. But obviously I can't do anything about Apples code.
I will restructure the tests and will start to use random data with a static seed to be at least reproducible in the next release. But I'm afraid that is all I can do for you. My general advice is: Just stick with inflate/deflate (zlib). I has proven itself in the last decades and just works while offering a good ratio of time to compression rate.
(I will close this issue when I have restructured the unit tests)

My idea was to just test my wrapper code extensively and just assume that Apples code works.

Ah, so you think the problem really is with Apples library? That would be quite worrying. 😧

My general advice is: Just stick with inflate/deflate (zlib).

I’m currently using gzip, hoping that’s solid too.

And my test showed that 50% of the time it doesn't compress the data at all.

I’m not sure if you’re saying that the library failed to do anything at all, or just that the resulting data is not smaller than the input. If the latter: I’m not a specialist in this area, but I think it is a basic property of information theory that random data generally can not be compressed.
Thanks for your quick reply and the great work you did with his library!

Once again thank you very much for your help. I also added better test cases in the 3.5.0 release. Sticking with random data for now though.


Merkle Tree construction and membership proofs

This task consists of 2 gadgets:
Native Merkle Tree implementation

[x] Construction of the whole tree from array
[x] Construction of membership path at specific index

Gadgets Merkle tree implementation:

[x] Verification of membership given the path

Given an array of leaves -- the array length will be 2 ** H, where H is height of the tree. E.g: H = 9, array length = 2 ** 9 = 512

Construct the whole binary merkle tree from this array, by hashing the pairs of 2. -- (0, 1), (2, 3), ..., (510, 511)
Costruct the membership path - given the index or leaf value, find the pair, and re-construct the root hash, but along the way, save the path values into the path_arr.

Merkle Tree native and circuit implementation - #123


The test file for Hello World is overly confusing and can turn off newcomers

Compare Python's Hello World test file https://github.com/exercism/python/blob/master/exercises/hello-world/hello_world_test.py
To Perl 6's https://github.com/exercism/perl6/blob/master/exercises/hello-world/hello-world.t
Is there anyway to simplify Perl 6's testing files? Ideally, a test file that a student downloads would teach them how to set up a test. This file is sort of unreadable.
is hello, |.<expected description> for @($c-data<cases>);
In order to test in Perl 6, first you must learn the entire language? Kind of defeats the purpose of Perl allowing for "baby talk" as well as Shakespeare.

PR open @ https://github.com/exercism/perl6/pull/298

What if we shortened the entire thing to:
#!/usr/bin/env perl6
use v6;
use Test;
use HelloWorld;
plan 1; # This is how many tests we expect to run.

is hello, "Hello, World!";

The absence of use lib is not ideal as it would require the user to set PERL6LIB themselves.
There is an existing PR to remove versioning @ https://github.com/exercism/perl6/pull/228, the reason it was there originally is because if a test file were changed, and someone attempted to run it against an older solution, comments would be made regarding failing tests. When a test is run the version check alerts the user not to expect all tests to succeed due to potential changes. It likely isn't needed anymore but need to make sure that it isn't.


Support generating comment as an output instead of posting it directly

I'm also uploading the html coverage report, and would like to append the artifact URL to the comment this action leaves, before posting the comment myself.

Nice idea, and shouldn't be too difficult to implement given the comment is already written into a file.
Would you like to give it a try and submit a PR?


Fix "was called on null" error

Returns null when any min/max method is called on no elements, prevents "The getter 'getterName' was called on null" error and similar.

That's great! Could you maybe also add a test which confirms the behaviour?

That's great! Could you maybe also add a test which confirms the behaviour?

Sorry working hard on our app (cashcoach.io), so I would appreciate if you merge it as is. We use a fork with this very fix for now, so it would be great to switch back to the upstream.

Thanks for you contribution. I ran into this myself. This has been automatically fixed when we migrated to nullsafety.


How To Customize Show Page

Hi guys
when  I added an image and some Data from select2 the crud show them as a text
so How To Customize Show Page ??

Hi @ayubDev ,
Could you please rephrase the question? I don't understand.
PS. For development questions (non-bugs), it's recommended that you post on Stack Overflow (using the backpack-for-laravel tag) or Gitter, after you're sure you can't solve the problem yourself and there isn't an answer to that already. There are a lot more people on Gitter and StackOverflow. Plus:

on StackOverflow people are rewarded with points for helping you out;
on StackOverflow your conversation will be better indexed by search engines;
for super-urgent issues, nothing beats a chatroom, like Gitter; if anyone's online, of course;
we can keep the Github Issues only for bugs that affect everyone, so we can crush them faster;

If you want to help people out with development question, that would be a HUGE help for the project. You can answer people:

synchronously - by leaving your Gitter app open;
asynchronously - by adding a StackOverflow filter for the backpack-for-laravel tag to receive questions by email;

Answering questions improves communication skills, makes you better understand the inner workings and improves your self-esteem. Just sayin' :-)

thank you @tabacitu , and I'll to do what you told me ^^

Thanks, have a nice day!


Fix website repo link

The homepage is in a repo separate from the main CNPG repo.

Superseded by #232


Added user description

This Pull Request resolves Issue #1
Description:
In this Pull Request we have done the following additions/edits for the code:
*Added user description.

Thank you for contributing!


Type fix for 64-bit compilation

Without this fix, I get an error saying
main.c: In function ‘open_device_with_vid’:
main.c:52:4: warning: format ‘%d’ expects argument of type ‘int’, but argument 2 has type ‘size_t’ [-Wformat=]
    printf("Found device %d idVendor=0x%04x idProduct=0x%04x\n", i, desc.idVendor, desc.idProduct);
    ^

when compiling on a x64 Ubuntu 14.04 host.
Given that it's only iterating the results from libusb_get_device_list, I suspect that an int is still more than sufficient ;-)

I used an uint32_t when I fixed that. Don`t use signed ints for array offsets.
I also have a missing "/n" in the help text on line 28.

Thanks @ED6E0F17 , I force-updated the PR.

This code is being updated significantly in the next week so will have to
have a look at this afterwards...
Gordon
On Wed, Jan 18, 2017 at 6:28 PM Andrew Scheller<EMAIL_ADDRESS>wrote:

Thanks @ED6E0F17 https://github.com/ED6E0F17 , I force-updated the PR.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/raspberrypi/usbboot/pull/5#issuecomment-273559024,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB9CHB2Rc2swLgjgMuVPkoakabmFAmymks5rTlnegaJpZM4LnCEQ
.

--
Director of Software Engineering, Raspberry Pi (Trading) Limited
t:  +44 (0) 1223 322633
m: +44 (0) 7450 946289
e<EMAIL_ADDRESS>w: www.raspberrypi.org

OK, added this to my push...  Thanks


Add check command in _WD_ElementActionEx

Hi, I have added support for new check and uncheck commands in the _WD_ElementActionEx function. Please review the code and I'll amend it as appropriate for merge :)

This allows setting a checkbox to a known state, and also since this uses JS it doesn't have the limitations of _WD_Click.

What kind of limitations you mean ?

By limitations I don't mean the function's code, but the limitations of the WebDriver itself in how it executes this function. I am not sure about other browsers, but Chrome doesn't like clicking on elements if they are obscured or overlapped by another element or if they aren't rendered to be visible in the page.
Using JavaScript entirely avoids this class of errors as it doesn't care about that at all :grin:


User/wang edward/248/imu gps urdf

https://github.com/uwrobotics/uwrt_mars_rover/issues/248

This is tested and working? Maybe try to refer to meshva's PR here on how to parameterize the GPS and the IMU: https://github.com/uwrobotics/uwrt_mars_rover/pull/252. The most important part is that we have some config file that we can change the locations and speciifc parameters for those sensors.

added and slightly modified in my PR. Closing.


Remove unsupported Python 2.6, 3.2 and 3.3

Reasons for dropping old ones

https://en.wikipedia.org/wiki/CPython#Version_history

2.6

https://snarky.ca/stop-using-python-2-6/
http://www.curiousefficiency.org/posts/2015/04/stop-supporting-python26.html
Current pip 9 deprecates Python 2.6 support, pip 10 won't support it (https://github.com/pypa/pip/issues/3955)
Not much PyPI traffic (June 2016) https://github.com/pypa/pip/issues/3796

3.2

pip 8 no longer supports Python 3.2 (link)
Coverage 4 no longer supports Python 3.2 (link)
Requests no longer supports Python 3.2
Virtually no PyPI traffic (June 2016) https://github.com/pypa/pip/issues/3796

3.3

pip 10 will deprecate Python 3.3 support, pip 11 won't support it (https://github.com/pypa/pip/issues/3796)
Very little PyPI traffic (June 2016) https://github.com/pypa/pip/issues/3796

Here's the pip installs for pronouncing from PyPI for the last year, showing virtually no 2.6 or 3.3, and no 3.2 (I've no idea what that single 2.8 is!):
$ pypinfo -d 365 --percent --pip pronouncing pyversion
python_version percent download_count
-------------- ------- --------------
2.7              60.9%          1,005
3.5              18.8%            310
3.6              14.8%            244
3.4               5.3%             88
2.6               0.1%              2
3.3               0.1%              1
2.8               0.1%              1


Implements transition

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.18% :tada:

Comparison is base (94865c9) 29.54% compared to head (76a1768) 29.72%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #26      +/-   ##
==========================================
+ Coverage   29.54%   29.72%   +0.18%
==========================================
  Files          11       12       +1
  Lines         775      777       +2
==========================================
+ Hits          229      231       +2
  Misses        546      546

Files Changed
Coverage Δ

src/sampler.jl
94.57% <100.00%> (+0.04%)
:arrow_up:

... and 1 file with indirect coverage changes

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


Looks like minor version update beyond this breaks uglify

The symptom at build time is the following.
At runtime the site simply fails to load except for the header.
Running "uglify:generated" (uglify) task
WARN: Output exceeds 32000 characters
File dist/scripts/vendor.js created: 4.45 MB → 1.54 MB
File dist/scripts/scripts.js created: 546.78 kB → 315.5 kB
>> 2 files created.

Please make sure that you've checked the following before submitting your pull request. Thanks!

[x] Check that you pass the basic compilation and unit tests by running grunt

Looks okay, definitely does not show the output exceeds warning after changing uglify version.


implementing sub columns into table

I was looking through some of the documentation for the table and I had a question when I was reading through it. Is it possible to create sub-columns for one of the header column? Some of my data has sub-columns in it and I would like to display the sub columns along with their respective parent columns.

HI @VEDA95, we dont support sub column currently, however, I think you have some nest columns in your data, so you want to apply them to the table, right.  There's same issue that somebody open, you can check https://github.com/AllenFang/react-bootstrap-table/pull/508  and https://github.com/AllenFang/react-bootstrap-table/issues/50.
In my opinion, I may support the nest data in future, but nest column is not, so PR is welcome 👍
I'll close this issue, if you have any concern, feel free to discuss with me. Thanks :)


client: undo set read-timeout for failed GetNameVersion

Undo temporary read-timeout in GetNameVersion(). (Note that code in this function does not take into account the possibility for another read-timeout already set. This fix only makes GetNameVersion() behave as expected under the same assumptions already present.)
update: you could consider calling panic on the error of tk.conn.SetReadTimeout. The only error possible seems to be for invalid values, and those are handled already. The panic would never trigger, but it would alert you in case the serial-port API changes unexpectedly.

you can remove return-type for SetReadTimeout,
panic in case of error at tk.conn.SetReadTimeout,
in GetNameVersion, call defer SetReadTimeout(0) immediately after SetReadTimeout(2),
remove other individual calls SetReadTimeout(0) as defer already handles every exit,

Resetting the read timeout to 0 before returning error looks like a good idea in general (no matter what this patch might be otherwise needed for).
And the update regarding panic:ing on SetReadTimeout (and simplifying of code paths) also seems reasonable, if indeed the only case it can error is on being passed an illegal value.

update: you could consider calling panic on the error of tk.conn.SetReadTimeout. The only error possible seems to be for invalid values, and those are handled already. The panic would never trigger, but it would alert you in case the serial-port API changes unexpectedly.
you can remove return-type for SetReadTimeout,
panic in case of error at tk.conn.SetReadTimeout,
in GetNameVersion, call defer SetReadTimeout(0) immediately after SetReadTimeout(2),
remove other individual calls SetReadTimeout(0) as defer already handles every exit,

I like this approach, very clean. As you say, it seems to only have one possible error, which is already handled.
This is however a breaking change. Will discuss here to see how to go about it.

Sorry for the delay.
Tkeyclient is now on licensed under BSD-2-Clause.
So to be able to merge this PR we need two things:

rebase this branch on main
we require commits to be signed, so please sign your commit

If you accept the new license of BSD-2-Clause, either do the two points above, or simply write here that you accept the new terms, and I can fix the rebasing and signing (keeping you as the author of the commit of course).

Another PR https://github.com/tillitis/tkeyclient/pull/16 is now merged. Closing.


Injecting credentials broken

Issue description
Upgrading to the latest version of pipenv via HomeBrew breaks injecting credentials using environment variables.
Expected result
pipenv install installs dependencies from private repositories.
Actual result
[pipenv.exceptions.ResolutionFailure]: Warning: Your dependencies could not be resolved. You likely have a mismatch in your sub-dependencies.
  You can use $ pipenv run pip install <requirement_name> to bypass this mechanism, then run $ pipenv graph to inspect the versions actually installed in the virtualenv.
  Hint: try $ pipenv lock --pre if it is a pre-release dependency.

Steps to replicate
pipenv install

We experience a similar issue: after upgrading to 2023.8.26, pipenv lock fails with the error message above.
However, pipenv install with a previously created Pipfile.lock still works.
We are using 90 packages from PyPI and one package from a private Gitlab package index.
Unfortunately I cannot share the private registry (it's private :D) but a sample Pipfile which triggers the problem:
[[source]]
name = "pypi"
url = "https://pypi.org/simple"
verify_ssl = true

[[source]]
name = "gitlab"
url =<EMAIL_ADDRESS>verify_ssl = true

[packages]
some-private-package = {version = "==1.23.42", index = "gitlab"}

[requires]
python_version = "3.10"

@dhonorez Would be helpful to know which kind of dependencies are affected -- I am assuming vcs, but I think there are tests around  this, do you have an example Pipfile entry?

Also, do you know if this just started in 2023.8.26 or if it started in the august releases?

We just tested 2023.8.25 and it is working fine.

@matteius The entry that fails is the following:
woodwing-client = { version = "==12.2.0", index = "artifactory" }

Hmm, I just subbed out part of my pytorch index with an env var and it seemed to work fine:
$ cat Pipfile
[[source]]
url = "https://pypi.org/simple"
verify_ssl = true
name = "pypi"

[[source]]
url = "https://${PYTORCH}/whl/cu117"
verify_ssl = true
name = "downloadpytorch"

[packages]
torch = {version = "==2.0.1", index = "downloadpytorch"}

[dev-packages]

[requires]
python_version = "3.11"

I am not sure where its breaking in your case -- anything stand out in the diff? https://github.com/pypa/pipenv/compare/v2023.8.25...v2023.8.26
My hunch is maybe the cached_property is causing it, but I am honestly at a loss currently without being able to reproduce it.

@matteius is your private repo protected with authentication? Because I read pipenv does something to hash the credentials that are provided, I suspect maybe something goes wrong while hashing?

is your private repo protected with authentication?

No, this is just the public pytorch repo, I thought this has something to do with using an env var in the source URL.

I read pipenv does something to hash the credentials that are provided, I suspect maybe something goes wrong while hashing?

I am not sure -- do you have a link to this topic?
Just from a code perspective, it has to be something  that changed in that diff between .25 and .26 ... nothing stands out in that department to me yet.

is your private repo protected with authentication?

No, this is just the public pytorch repo, I thought this has something to do with using an env var in the source URL.

It's definitely an issue with credentials, not with injecting env vars.

I read pipenv does something to hash the credentials that are provided, I suspect maybe something goes wrong while hashing?

I am not sure -- do you have a link to this topic?

(https://pipenv.pypa.io/en/latest/credentials/#injecting-credentials-into-pipfile-via-environment-variables)

Just from a code perspective, it has to be something that changed in that diff between .25 and .26 ... nothing stands out in that department to me yet.

Are you able to check if this fixes it?  https://github.com/pypa/pipenv/pull/5882
I am not 100% sure it will, but of any change between 25 and 26 this is my hunch.

@matteius No it the other cached_property that's causing the bug.  https://github.com/pypa/pipenv/commit/87162712a94c8a168afbd9688e16106355453803#diff-62e85156fc3b274046cc03dabfe6e3ceead503f330b98fb3611645216b4b6e1fR301

Please try 2023.8.28

Resolved for me, thanks!


[WIP] Add channel awareness

Q
A

Bug fix?
no

New feature?
yes

BC breaks?
yes

Deprecations?
no

Related tickets
#136

License
MIT

Will there also be migrations for existing data?


New config options need to be merged into the global config

Right now the global config is merged with the default config so that any new settings are present in the config. This does not change the default config on disk however.
Should the merge be written back to the main config, so that new settings are written to disk?

This is being done but I am unsure if NOT writing the home config by default is probably the best option. So on a run Versup will fetch its default config, then the user's home directory one (if it exists) and merge it. Finally it will take the .versup.json in the project dir and merge that in.
This way we avoid a user having to deal with a config file they might not even need, and when attributes get removed or added to the config, they are added to the default one. If there's a home directory settings file, the user will have placed it there and know about it and edit it.
As a final option (not entirely sure about this) would be to add a command to dump the default config to the home dir.


add cloning \w method for cloning sheets

Issue: https://github.com/abap2xlsx/abap2xlsx/issues/1004
Added the zcl_excel_base prototype class, implemented deep cloning in Excel components, added the clone_worksheet method in the main Excel class.

note that abap2xlsx aims for 702 compatibility, so inline declarations are not possible

note that abap2xlsx aims for 702 compatibility, so inline declarations are not possible

Working on it.

Please review.

I created the below program to test your PR and it produced the error 15_08_RangeFromReader.xlsx Failed to match the regular expression. To reproduce, you must run ZDEMO_EXCEL before running the demo program. It happens in the method GET_SHEET_TITLE of ZCL_EXCEL_RANGE, which is called by SET_TITLE of ZCL_EXCEL_WORKSHEET.
I'm not sure why it's needed to change the logic, can't we keep the same as before?
NB: I also just added a commit to your branch to cancel very minor changes which are not related to cloning a sheet, it shouldn't produce any side-effect.
  TRY.
      DATA(file_names) = VALUE string_table(
      ( |15_31_AutosizeWithDifferentFontSizesFromReader.xlsx            | )
      ( |15_24_Sheets_with_different_default_date_formatsFromReader.xlsx| )
      ( |15_13_MergedCellsFromReader.xlsx                               | )
      ( |15_08_RangeFromReader.xlsx                                     | )
      ( |15_07_ConditionalAllFromReader.xlsx                            | )
      ( |15_05_ConditionalFromReader.xlsx                               | )
      ( |15_04_SheetsFromReader.xlsx                                    | )
      ( |15_03_iTabFromReader.xlsx                                      | )
      ( |15_02_StylesFromReader.xlsx                                    | )
      ( |15_01_HelloWorldFromReader.xlsx                                | )
      ( |Image_Header_Footer.xlsx                                       | )
      ( |Comments.xlsx                                                  | )
      ( |49_Bind_Table_Conversion_Exit.xlsx                             | )
      ( |44_iTabNotEmpty.csv                                            | )
      ( |44_iTabEmpty.csv                                               | )
      ( |ABAP2XLSX Inheritance.xlsx                                     | )
      ( |40_Printsettings.xlsx                                          | )
      ( |39_Charts.xlsx                                                 | )
      ( |38_SAP-Icons.xlsx                                              | )
      ( |37- Read template and output.xlsx                              | )
      ( |36_DefaultStyles.xlsx                                          | )
      ( |35_Static_Styles.xlsx                                          | )
      ( |34_Static Styles_Chess.xlsx                                    | )
      ( |33_autofilter.xlsx                                             | )
      ( |31_AutosizeWithDifferentFontSizes.xlsx                         | )
      ( |30_CellDataTypes.xlsx                                          | )
      ( |28_HelloWorld_Sheet1.csv                                       | )
      ( |28_HelloWorld_Sheet2.csv                                       | )
      ( |27_ConditionalFormatting.xlsx                                  | )
      ( |24_Sheets_with_different_default_date_formats.xlsx             | )
      ( |23_Sheets_with_and_without_grid_lines.xlsx                     | )
      ( |22_itab_fieldcatalog.xlsx                                      | )
      ( |21_BackgroundColorPicker.xlsx                                  | )
      ( |19_SetActiveSheet.xlsx                                         | )
      ( |18_BookProtection.xlsx                                         | )
      ( |17_SheetProtection.xlsx                                        | )
      ( |16_Drawings.xlsx                                               | )
      ( |14_Alignment.xlsx                                              | )
      ( |13_MergedCells.xlsx                                            | )
      ( |12_HideSizeOutlineRowsAndColumns.xlsx                          | )
      ( |10_iTabFieldCatalog.xlsx                                       | )
      ( |09_DataValidation.xlsx                                         | )
      ( |08_Range.xlsx                                                  | )
      ( |07_ConditionalAll.xlsx                                         | )
      ( |06_Formulas.xlsx                                               | )
      ( |05_Conditional.xlsx                                            | )
      ( |04_Sheets.xlsx                                                 | )
      ( |03_iTab.xlsx                                                   | )
      ( |02_Styles.xlsx                                                 | )
      ( |01_HelloWorld.xlsx                                             | ) ).
      DATA(lv_workdir) = ``.
      cl_gui_frontend_services=>get_sapgui_workdir( CHANGING sapworkdir = lv_workdir ).
      cl_gui_cfw=>flush( ).

      LOOP AT file_names INTO DATA(file_name).
        file_name = replace( val = file_name regex = ' +$' with = '' ).
        DATA(file_path) = lv_workdir && '\' && file_name.
        DATA(reader) = NEW zcl_excel_reader_2007( ).
        DATA(workbook) = reader->zif_excel_reader~load_file( file_path ).

        DATA(iterator) = workbook->get_worksheets_iterator( ).
        TYPES ty_sheets TYPE STANDARD TABLE OF REF TO zcl_excel_worksheet WITH EMPTY KEY.
        DATA(sheets) = VALUE ty_sheets( ).
        WHILE iterator->has_next( ).
          DATA(sheet) = CAST zcl_excel_worksheet( iterator->get_next( ) ).
          APPEND sheet TO sheets.
        ENDWHILE.

        LOOP AT sheets INTO sheet.
          DATA(new_sheet) = workbook->clone_worksheet( sheet ).
          new_sheet->set_title( sheet->get_title( ) && '2' ).
        ENDLOOP.

        DATA(writer) = NEW zcl_excel_writer_2007( ).
        DATA(xlsx_content) = writer->zif_excel_writer~write_file( workbook ).
        file_path = file_path && '2'.
        PERFORM write_bin_file USING file_path xlsx_content.
      ENDLOOP.
    CATCH zcx_excel INTO DATA(error).
      MESSAGE |{ file_name } { error->get_text( ) }| TYPE 'I' DISPLAY LIKE 'E'.
  ENDTRY.
FORM write_bin_file
      USING
        i_filename      TYPE csequence
        i_file_xstring  TYPE xstring.
  DATA lt_xstring TYPE TABLE OF x255.
  DATA l_length TYPE i.
  DATA l_filename TYPE string.
  l_filename = i_filename.
  CALL METHOD cl_swf_utl_convert_xstring=>xstring_to_table
    EXPORTING
      i_stream = i_file_xstring
    IMPORTING
      e_table  = lt_xstring.
  l_length = xstrlen( i_file_xstring ).
  CALL METHOD cl_gui_frontend_services=>gui_download
    EXPORTING
      bin_filesize = l_length
      filename     = l_filename
      filetype     = 'BIN'
    CHANGING
      data_tab     = lt_xstring.
ENDFORM.

I've reproduced the error you described and it turned out that my regex was not written correctly and that was a reason of an exception.
Additionally, I discovered another problem which has been fixed as well.
I was not able to test the following files and they were not available in the Web Repository. I commented them out, so you can test them youself to be sure.

44_iTabNotEmpty.csv
44_iTabEmpty.csv
ABAP2XLSX Inheritance.xlsx
40_Printsettings.xlsx
37- Read template and output.xlsx
28_HelloWorld_Sheet1.csv
28_HelloWorld_Sheet2.csv

I'm not sure why it's needed to change the logic, can't we keep the same as before?

As far as I remember, when you read ranges from an xlsx file, they are provided to in the form of strings.
When the worksheet is cloned, and its ranges at the same time, the worksheet title in all of those ranges has to be changed. Without the regex-logic I was not able to read only the worksheet title from them to replace it easily. I would have to replace the whole range value.
As I already created those methods for cloning, I replaced the logic in the set_title method as well. It is a bit shorter, but you can restore the old logic if you prefer it.

Without the regex-logic I was not able to read only the worksheet title from them to replace it easily.

As the clone method is public, ranges can be cloned separately from worksheets - in those cases the regex logic is necessary.
That's what I meant.

Anything in this topic?

That fixed the issue. I compared the sheets manually, and saw these differences:

1015_04_Sheets.xlsx: the worksheet cloned from the hidden Sheet3 is made visible
The Excel feature "Data Validation" has been broken, but it happens just by running the demo programs (just run ZDEMO_EXCEL to generate all the files), not even cloning. When opening any of the 2 files below, this pops up (it doesn't happen with the current main branch):

03_iTab.xlsx: in cell 'Internal table'!C4, the list of possible values doesn't show up
09_DataValidations.xlsx: in cell 'Table with Data Validation'!A12, the list of possible values doesn't show up
27_ConditionalFormatting.xlsx: in cell 'Conditional formatting'!C2, the list of possible values doesn't show up

1015_12_HideSizeOutlineRowsAndColumns.xlsx: the cloned worksheet makes all columns visible (zdemo_excel12: possibly lo_worksheet->zif_excel_sheet_properties~hide_columns_from = 'M'.?)
1015_22_itab_fieldcatalog.xlsx: the cloned worksheet loses the "white background" (zdemo_excel22: possibly lo_worksheet->zif_excel_sheet_properties~set_style( lv_style_guid ).?)
1015_33_autofilter.xlsx: the cloned worksheet loses the filters
1015_39_Charts.xlsx: there's an Excel error when opening the workbook
1015_40_PrintSettings.xlsx: the print settings "do not repeat" are not cloned

Fixing all these differences may take a while, maybe we can simply say that the worksheet cloning doesn't support those features.
Note that few differences are not due to cloning but to features working in the writer but not in the reader (it can be seen also by running zdemo_excel37), so cloning can't work on these features (as expected):

Tables (colors, column formulas)

1015_10_iTabFieldCatalog.xlsx
etc.

@AndreaBorgia-Abo If you have some time, could you give your opinion about the code concerning the creation of the super class zcl_excel_base, many classes will inherit from it. I think it's something smart for limiting the number of modifications but not so good from design perspective. I don't know if this can be problematic (whatever it's for existing installations, for maintenance...)

@sandraros maybe we could discuss it during the Community Call, otherwise I'm very short on time right now and, more to the point, I'm not sure I can provide reasonable feedback on OO design issues in a reasonable timeframe.


customTextToDisplayAtTime called way too frequently

customTextToDisplayAtTime is getting called every 10 milliseconds or so in my app. Seems way to frequent for a timer where the smallest time I'm showing is seconds. Is there any way to change this?

All you have to do, is to change the value of "kDefaultFireIntervalHighUse".

how?
we can't change it, it is private a method

@CavalcanteLeo Fork the repo.

i will send a pull request later to change it


ORC-1556: Add Rocky Linux 9 Docker Test

What changes were proposed in this pull request?
This PR aims to add Rocky Linux 9 Dockerfile.
Why are the changes needed?
To provide a test coverage for Oracle 9-compatible OS environment.

https://docs.rockylinux.org/release_notes/9_3/
https://access.redhat.com/documentation/en-us/red_hat_enterprise_linux/9/html/9.3_release_notes/index

How was this patch tested?
Pass the CIs and check the logs.

https://github.com/apache/orc/actions/runs/7314447285/job/19927098261?pr=1706

...
Status: Downloaded newer image for apache/orc-dev:rocky9
-- The C compiler identification is GNU 11.4.1
-- The CXX compiler identification is GNU 11.4.1
...
Java version: 17.0.9, vendor: Red Hat, Inc., runtime: /usr/lib/jvm/java-17-openjdk-<IP_ADDRESS>.9-2.el9.x86_64
Default locale: en_US, platform encoding: ANSI_X3.4-1968
OS name: "linux", version: "6.2.0-1018-azure", arch: "amd64", family: "unix"

Merged to branch-1.7.


feature: new deploy engine

Abstract
We experienced numerous problems with Helm deploys, much of them were very hard to solve. Also, the extensibility of the code that managed deploys in Helm was pretty limited, making implementation of new features too expensive.
We decided to develop a new alternative deploy engine which is 100% backwards compatible with Helm/werf, but with the problems of the previous implementation solved and with extensibility and maintainability improved. New deploy engine will become the default one when stabilized.
New deploy engine can be activated using WERF_EXPERIMENTAL_DEPLOY_ENGINE=1 since v1.2.234, no other changes required.
Major new features roadmap

[x] 3-way merge replaced with native Server-Side Apply;
[x] CRDs from /crds now deployed on install and upgrade, not just install;
[x] CRDs from /crds now can be created and updated, not only created;
[x] resource updates are skipped if corresponding cluster resources already as desired;
[x] if no changes in the whole release then the release is skipped;
[x] resources stored in previous release that can't be server-side validated (Kubernetes upgraded, CRD removed, ...) won't brick further upgrades;
[x] newly created resources won't be deleted on failed release;
[x] report of what was actually done in the end of deployment;
[ ] werf lint;
[ ] werf plan/diff;
[ ] easier management of immutable resources;
[ ] easier management of resources with "helm.sh/resource-policy": keep;

Released in experimental mode in alpha channel.

Here is a PR, where most of the changes are: https://github.com/werf/3p-helm/pull/254/files

The werf's new deployment engine is called Nelm now, and further related announcements can be found in this discussion: https://github.com/werf/werf/discussions/5657
There's also a separate repo for Nelm: https://github.com/werf/nelm

The new deployment engine is generally complete and stabilized, although we plan for a second iteration to improve on it. It can be enabled in werf v1.2 with export WERF_NELM=1 and it will be the only deployment engine in soon-to-be released v2.0.


Put coord type in advent.h

Put coord type in advent.h. This type is used in day 6 and day 8.

coord has been added to advent.h in branch "puzzles/2024/day10", todo add tests for coord and merge branch (in no particular order).


2024-02-01 and 2024-02-02 Meeting Agendas

:robot: JJ

Got better training environment; figured out how reset gazebo without having trouble
Demonstrate this
Still need to simulate laser sensors
Still need to get discretized navigation (I think this will come first, because it's much simpler and I already know how to do it)

:atom: Willey

Still need to understand rotational matrices and quaternions
Demonstrate enhanced training
Discuss new discretized method that will make drone rely on local IMU to navigate; this will get us farther along in the process

TODO

Get 'discretized' movement method, moving in increments of half meters or centimeters (not quite sure)
Simulate laser rangefinders

:atom: Meeting w Willey

Is it possible to shut off an accelerometer
We discussed discretized method


Use more informative exception

What does this PR do?
Also

Adds e2e support
Adds Python 3 support
Adds to Travis

Codecov Report

Merging #2414 into master will increase coverage by 14.03%.
The diff coverage is 88.88%.

@@            Coverage Diff             @@
##           master   #2414       +/-   ##
==========================================
+ Coverage   81.56%   95.6%   +14.03%
==========================================
  Files         258       5      -253
  Lines       16432      91    -16341
  Branches     1580       7     -1573
==========================================
- Hits        13403      87    -13316
+ Misses       2606       1     -2605
+ Partials      423       3      -420


UKCP Guide Typo

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


[BIOMAGE-1266] - Rotate X axis labels

Background
Link to issue
https://biomage.atlassian.net/browse/BIOMAGE-1266
Link to staging deployment URL
https://ui-agi-ui431.scp-staging.biomage.net/
Links to any Pull Requests related to this
N.A.
Anything else the reviewers should know about the changes here
Rotate X axis labels by default for :

Frequency plot in Data Integration
Frequency plot in Plots and Tables
Violin plot

The new settings (rotate X axis by deafult) will only be set for new experiments because old experiments will use saved plot settings (non-rotated X axis)
Changes
Code changes
Definition of DONE
Your changes will be ready for merging after each of the steps below have been completed:
Testing

[ ] Unit tests written
[X] Tested locally with Inframock (with latest production data downloaded with biomage experiment pull)
[X] Deployed to staging

To set up easy local testing with inframock, follow the instructions here: https://github.com/biomage-ltd/inframock
To deploy to the staging environment, follow the instructions here: https://github.com/biomage-ltd/biomage-utils
Documentation updates
Is all relevant documentation updated to reflect the proposed changes in this PR?

[ ] Relevant Github READMEs updated
[ ] Relevant wiki pages created/updated

Approvers

[ ] Approved by a member of the core engineering team
[ ] (UX changes) Approved by vickymorrison (this is her username, tag her if you need approval)

Just before merging:

[ ] After the PR is approved, the unstage script in here: https://github.com/biomage-ltd/biomage-utils is executed. This script cleans up your deployment to staging

Optional

[ ] Photo of a cute animal attached to this PR

Are the labels rotated in frequency in plots and tables? - i see this

Are the labels rotated in frequency in plots and tables? - i see this

The settings will only take place for new experiments. Old experiments use saved plot settings, which unfortunately is set to not rotated by default. To change the value for old experiments, we'd have to hard code it, or do a migration to change the value in the plot settings,


Feat: add BUILDRAINER for pipelinemode

Hi @monperrus :) , BUILDRAINER as a pipeline mode for RTScanner.

close for new changes.


Rule to forbid component dot notation?

Hi, with server components the component dot notation (Group.Foo) is not supported, it would be useful to have an eslint rule that forbids its usage to avoid issues like the one described here:
https://github.com/vercel/next.js/issues/51593#issuecomment-1600289701
The React error is:
Cannot access .Foo on the server. You cannot dot into a client module from a server component. You can only pass the imported name through.

From what I see it's coming from React itself. I'll dig further and get back to you

I searched with grep and found it in react-server-dom-webpack. I can't find the source location on the Git repository though.

To be fair I would be okay with a rule that forbids them all the time. I wouldn't want to mess with refactors in case a component has to be moved from server to client.


Extension issue

Issue Type: Bug
Extension Name: python
Extension Version: 2021.9.1191016588
OS Version: Windows_NT x64 10.0.19041
VS Code version: 1.60.0

@galmae8619 See if you find any logs from output > Python, or Developer tools window and share it with us.

Thanks for the bug report, but unfortunately we need some more information from you in order to triage this issue appropriately. Please create a new bug by providing a filled-in copy of our issue template.


same game again and again

Hi there,
Thanks for developing this wonderful tools. But I have a question: when I let Mugo match against itself, the game of each time is exactly same. How to let Mugos play differently?
Thanks again.

Try using 'randompolicy' as a option instead of 'policy'. Policy will take the most likely move, but randompolicy will select randomly, weighted by the NN's output probabilities.


What Spotify premium cracked version works?

Describe the issue
Hello I tried to install Spotilife C but I doesn’t work someone can tell what Spotify cracked version works?
Instructions to reproduce
Install the ipa of Spotlife C and it crash when I open it
What version of LiveContainer are you using?
Version 2.0-release (main/ebea623)
Other
No response

I tried to install the vanilla ipa of Spotilife C but he gives this error
Use of the class <INPreferences: 0x300a14960> from an app requires the entitlement com.apple.developer.siri. Did you enable the Siri capability in your Xcode project?
Call stack: (
0   CoreFoundation                      0x00000001a397eb34 3A5F992A-D1CD-312E-BD2E-F7C66343A417 + 969524
1   libobjc.A.dylib                     0x000000019b7faf78 objc_exception_throw + 60
2   Intents                             0x00000001ad54cbac B9269871-211A-3E53-A572-D69B95D0A0A2 + 1006508
3   libdispatch.dylib                   0x00000001ab78edd4 B7D9D0A6-2414-37F2-9D03-1A25A474E2CF + 15828
4   libdispatch.dylib                   0x00000001ab790654 B7D9D0A6-2414-37F2-9D03-1A25A474E2CF + 22100
5   Intents                             0x00000001ad54ce1c B9269871-211A-3E53-A572-D69B95D0A0A2 + 1007132
6   Intents                             0x00000001ad54ca40 B9269871-211A-3E53-A572-D69B95D0A0A2 + 1006144
7   Intents                             0x00000001ad54cfbc B9269871-211A-3E53-A572-D69B95D0A0A2 + 1007548
8   Spotify                             0x000000010b30f6d8 Spotify + 4306648
9   Spotify                             0x000000010b30e18c Spotify + 4301196
10  Spotify                             0x000000010b30e6c4 Spotify + 4302532
11  Spotify                             0x000000010b40be98 Spotify + 5340824
12  Spotify                             0x000000010b40bf70 Spotify + 5341040
13  Spotify                             0x000000010b40bf00 Spotify + 5340928
14  Spotify                             0x000000010b34e4b8 Spotify + 4564152
15  Spotify                             0x000000010b40bf3c Spotify + 5340988
16  Spotify                             0x000000010b40bb48 Spotify + 5339976
17  Spotify                             0x000000010b40b994 Spotify + 5339540
18  Spotify                             0x000000010c4887cc Spotify + 22628300
19  Spotify                             0x000000010c48abf4 Spotify + 22637556
20  Spotify                             0x000000010c488dc4 Spotify + 22629828
21  Spotify                             0x000000010b27c04c Spotify + 3702860
22  libdispatch.dylib                   0x00000001ab78d13c B7D9D0A6-2414-37F2-9D03-1A25A474E2CF + 8508
23  libdispatch.dylib                   0x00000001ab78edd4 B7D9D0A6-2414-37F2-9D03-1A25A474E2CF + 15828
24  libdispatch.dylib                   0x00000001ab796400 B7D9D0A6-2414-37F2-9D03-1A25A474E2CF + 46080
25  libdispatch.dylib                   0x00000001ab796f30 B7D9D0A6-2414-37F2-9D03-1A25A474E2CF + 48944
26  libdispatch.dylib                   0x00000001ab7a1cb4 B7D9D0A6-2414-37F2-9D03-1A25A474E2CF + 93364
27  libdispatch.dylib                   0x00000001ab7a1528 B7D9D0A6-2414-37F2-9D03-1A25A474E2CF + 91432
28  libsystem_pthread.dylib             0x00000001ff874f20 _pthread_wqthread + 288
29  libsystem_pthread.dylib             0x00000001ff874fc0 start_wqthread + 8
)

Apps with tweaks in them currently don't work

Try this https://github.com/whoeevee/EeveeSpotify this works perfectly.

Try this https://github.com/whoeevee/EeveeSpotify this works perfectly.

But it's not working for me. Premium patch isnt applying. I am using it jitless.

Try this https://github.com/whoeevee/EeveeSpotify this works perfectly.

I tried it just now it crashes then when i open it nothing happens

For me it also didn't work in the version 2 release.
With this new build it does though:
https://github.com/khanhduytran0/LiveContainer/actions/runs/10113010321/artifacts/1744235554
Does the error still occur for you with this?

It would help if you tried this. It works perfectly fine for me -
https://github.com/whoeevee/EeveeSpotify

It would help if you tried this. It works perfectly fine for me -
https://github.com/whoeevee/EeveeSpotify

Hi, it works but every time i run Spotify i have a popup error: Impossible to open the link, etc..
How to fix this error?


Update to lightning 2.0.2, remove bolts dependency and fixes

Can confirm that this fixes the issue for me, thanks!


Rework and simplify resource flow in drbg_add

Not a bug fix, but making things a lot easier to understand (I hope).

Thanks, I'll have a look. But let me tie up some other loose ends first. Currently my resources are limited due to family affairs...

Merged to master as f9e4392 and 1.1.1 as 6101850, thanks!


Fix build with GHC 8.10

This will fix #10.
I'm not sure what the problem is yet. But it turns out that I wasn't actually building anything in CI. I was used to Stack's CLI, which builds things when you run stack test even if there are no tests. Cabal's CLI is different because if you run cabal test without any tests, it doesn't build anything. So with any luck I should see this fail in CI shortly.

I think GHC 8.10 might be a bit of a red herring. Really this problem happens because hlint-3.3.x switched to ghc-lib-parser-9.0.x.
One way to fix this would be to support arbitrary versions of GHC and HLint at the same time, which is what Splint currently does. However HLint seems to have move to an approach where version X uses ghc-lib-parser version Y. So if I want to support, say, HLint 3.4 on GHC 8.10, I have to consume ghc-lib-parser-9.2.x's output and use it as input for ghc-8.10.x. Frankly I'm not interested in supporting that matrix.
A simpler solution is to pin a major version of HLint and then allow multiple versions of GHC. I suspect that's a much more common use case anyway.

What I want to be able to say is: "If hlint's version of ghc-lib-parser matches the version of ghc, then use a 'native' parser. Otherwise use a 'fallback' parser." Unfortunately Cabal's conditionals aren't expressive enough for that. I think I can do that with CPP (that's how I did things before #8), but I'm not sure if it's worth the trouble.


Internet Explorer (any version) getClientRects

In our app we're using getClientRects to get the bounding boxes of the current text selection. We're using this and telling the server where they are. The issue is that pdf.js positions elements absolutely and this seems to break internet explorers ability to have getClientRects function correctly. The solution that I was able to find was to have all elements float left with negative margins and then position them via margin-left/top. I created a stackoverflow question with an example as well as the solution I was able to find here: http://stackoverflow.com/questions/27132674/internet-explorer-getclientrects-absolute-positioned-elements

I can confirm your solution works! Thank you!

The issue is that pdf.js positions elements absolutely and this seems to break internet explorers ability to have getClientRects function correctly.

We won't be making changes to the PDF.js library to specifically cater to old browsers, such as e.g. Internet Explorer; hence closing the issue as out-of-scope.

@Snuffleupagus Just FYI, this is potentially also an issue in Edge and not just old IE browsers:
https://developer.mozilla.org/en-US/docs/Web/API/Element/getClientRects

IE and Edge return a (ClientRectList with) MSDN: ClientRect objects (which do not contain x and y properties) instead of DOMRect objects

@NogsMPLS  I figure this may need to be reopened. This does not work with Edge 44 anymore.

This is related with this Edge bug I think: https://developer.microsoft.com/en-us/microsoft-edge/platform/issues/15068329/

@MathewBerg  perhaps you can reopen this issue? maybe someone else may be able to provide a fix.

It seems I cannot, as this was specifically related to older versions of IE and not Edge I suggest you create a new issue and reference this one.

i saw an issue created regarding the new versions. ill reference this one then. thank you

#5513


run under convox/docker

convox start

 QAE Support: Server Maintenance

Can we change the docker_compose file to docker_compose.yml.local which you can specify on start.
This is because this is the file convox uses on deployment.

Do not merge please. I'm still failing with
web    | /usr/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/runtime.rb:80:in rescue in block (2 levels) in require': There was an error while trying to load the gem 'skylight'. (Bundler::GemRequireError)`

Remove skylight, it uses rust so could be missing a dependency.
sent from my phone.
On 10 Mar 2016 00:49, "Mario Andrés"<EMAIL_ADDRESS>wrote:

Do not merge please. I'm still failing with
web |
/usr/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/runtime.rb:80:inrescue
in block (2 levels) in require': There was an error while trying to load
the gem 'skylight'. (Bundler::GemRequireError)`
—
Reply to this email directly or view it on GitHub
https://github.com/bitzesty/qae/pull/788#issuecomment-194592001.

@matthewford I simplified circle file by adding sql structure dump


scipy.sparse.linalg.eigs segmentation fault on complex matrix input

When I run scipy.sparse.linalg.eigs on a matrix with all real entries it produces a result without error, but when I run it on a matrix with complex entries it produces a segmentation fault. I've tried running this code snippet with each of the seven sparse matrix classes available in scipy.sparse and I get the same result for all of them.
Reproducing code example:
import numpy as np
import scipy.sparse as sparse
from scipy.sparse import linalg

mat = np.array([[1,1j,1+1j],
                [-1j, 2, 0],
                [1-1j, 0, 3]])
print np.linalg.eig(mat)[0]
print linalg.eigs(sparse.dok_matrix(mat, dtype=complex), 1)```

### Error message:

[-0.11490754 +1.35212033e-17j  3.86080585 -2.05623954e-16j
2.25410169 -2.99418538e-17j]
Segmentation fault (core dumped)

### Scipy/Numpy/Python version information:

('0.19.1', '1.11.1', sys.version_info(major=2, minor=7, micro=11, releaselevel='final', serial=0))

Not reproducible. Reason most likely is that you scipy installation is
broken in some way. Platform / where did you install it from?

I'm running lubuntu on an Oracle VM Virtualbox and my python installation is 2.7 from anaconda.
pip show scipy gives:
Name: scipy Version: 0.19.1 Summary: SciPy: Scientific Library for Python Home-page: https://www.scipy.org Author: SciPy Developers Author-email<EMAIL_ADDRESS>License: BSD Location: /home/michael/anaconda2/lib/python2.7/site-packages Requires: numpy
How would you recommend installing python/scipy on linux? Thanks for your help!

Anaconda should be fine in general. Since the problem appears specific
to the binaries+platform configuration, you may want to take this up
with anaconda. The upstream scipy project cannot fix it.

Alright, that makes sense. Nonetheless, would you be willing to tell me the platform/install that you have on which this code does not produce a seg fault? I might be able to redo my installation to be more like yours.

@mgscheer Both works on
Windows 10, Python 3.6.2, NumPy 1.13.1, SciPy 0.19.1
Ubuntu 16.04 Python 3.6.1, NumPy 1.12 Scipy 0.19
On Windows wheel install, on Ubuntu built directly.

Thanks a lot to both of you!


AIY vision kit error can't run the demo

I flashed the latest image to the SD card. And I didn't sudo apt-get update or sudo apt-get upgrade, because some issues have shown that update or upgrade would cause any other problems.
But even I checked the connecters "thousands times", it always had failure to run the program correctly. So I made a video for this mistake about what it happened after I run the demo:
https://youtu.be/zpCFq9uftHY
The log is here:

-- Logs begin at Sun 2019-08-18 10:20:56 UTC. --
Aug 18 10:59:37 raspberrypi python3[13978]:     if cb(*exc_details):
Aug 18 10:59:37 raspberrypi python3[13978]:   File "/usr/lib/python3.5/contextlib.py", line 261, in _exit_wrapper
Aug 18 10:59:37 raspberrypi python3[13978]:     return cm_exit(cm, *exc_details)
Aug 18 10:59:37 raspberrypi python3[13978]:   File "/usr/lib/python3/dist-packages/picamera/camera.py", line 758, in exit
Aug 18 10:59:37 raspberrypi python3[13978]:     self.close()
Aug 18 10:59:37 raspberrypi python3[13978]:   File "/usr/lib/python3/dist-packages/picamera/camera.py", line 752, in close
Aug 18 10:59:37 raspberrypi python3[13978]:     raise exc
Aug 18 10:59:37 raspberrypi python3[13978]:   File "/usr/lib/python3/dist-packages/picamera/camera.py", line 1995, in _control_callback
Aug 18 10:59:37 raspberrypi python3[13978]:     "No data recevied from sensor. Check all connections, "
Aug 18 10:59:37 raspberrypi python3[13978]: picamera.exc.PiCameraRuntimeError: No data recevied from sensor. Check all connections, including the SUNNY chip on the camera board

I have no idea what  I can do for troubleshooting. Please help me. Thanks!

@DaXiongAhey Can you try to connect the camera directly to RPI and run below command :
raspistill -o file.jpg
Please share what happens.

Please try to use new SD card image aiyprojects-2019-11-13.img.xz at Releases section.


My Files page loops on the loading animation

When a user doesn't have any downloaded files, the loading animation loops for a long time, perhaps forever.

Looking at the network tab, its sending a request every second:
{"jsonrpc":"2.0","method":"get_lbry_files","params":[{}],"id":0}
And getting back the response:
{"id": null, "result": false, "error": null}


Started to support writing of binary VTK legacy files

NOT READY to MERGE.
I just needed faster writing of point clouds and this quick and dirty implementation (not polished for merging yet) is already about 20 times faster than the ASCII version. And e.g. paraview is reading it way faster.
@pomerlef , do you think this is a useful extension (even though it is legacy VTK files anyway and reading the same files would not be supported without further effort) or would you rather switch to libvtk to read and write vtk files. Was there a good reason not to do that in the first place?

@HannesSommer : the general strategy is to keep the dependency list as small as possible. We provide basic functionalities from inside the library and, for more advance features, we create a binder like:

pointmatcher-ros: pull out by Ralf
libpointmatcher_vtk: I've done that for Alstom, there is not much in it.

That's the general guideline but, as you can see, we didn't went too far in that direction and it's apparently not standardize for the naming. That being said, everything that increase the speed of the workflow is good. If you are in the order of 20x then we should take the time to polish that. This feature was requested a couple of time in the issue list too.
Is it a lot of work to support it both way (read/write)? With the trend of issue I saw, it will confuse a lot of users if not.

Ah, thanks for the quick reply. I totally get the aim of small dependency footprint. It is actually probably an 1h of work for me to polish it enough and add write support - at least for most of the data (see below). I can do that.
A few questions remain, though:

Colour:
How important is storing colour? (see https://github.com/ethz-asl/libpointmatcher/pull/132/files#diff-65f29f99818f583e6f26c71cfaade3cdR505). Would it be fine to start with binary only for point clouds without colour feature? That would mean falling back to ASCII if there is colour in there. Adding colour would be 30min extra I suppose and could in theory be done later.

new save binary flag:
What about this interface extension https://github.com/ethz-asl/libpointmatcher/pull/132/files#diff-5cc32d1d2c455d67872c78be20145a26R201. Is that fine, would you do it different? This gets passed down till here https://github.com/ethz-asl/libpointmatcher/pull/132/files#diff-03f31f751a0bac300a04bd031c66ec48R878. The fact that the others don't support binary I don't know how to treat the libpointmatcher way :). What would it be? Throw an exception (like binary not supported for the BLA format) or silently or with a warning switch to non binary if binary is not supported.

New writeBinary parameter:
Same question here:https://github.com/ethz-asl/libpointmatcher/pull/132/files#diff-d702c135dcab9ea187a2353e809ffd82R168

ASCII data type (float or double):
In this line https://github.com/ethz-asl/libpointmatcher/pull/132/files#diff-65f29f99818f583e6f26c71cfaade3cdR210 there is currently a change of the existing semantic for the ASCII output: in case of T equals double it would write double instead of float. Nothing else would change for ASCII output. However, if such a file gets read e.g. with paraview it would probably read a double point cloud instead of a float pointcloud afterwards. Why did you hard code float there? (I think currently libpointmatcher will just use T when reading ASCII data - no matter whether there is float or double written in the vtk file)

ok
throw an exception with the description of the error and what to do.
see 2
just like that, I don't recall why float is hard coded. It seems better to use T so try it like that. Test if there is a problem in Paraview for double and float formats, just to make sure.

Thanks for looking into that. The IO.cpp is halfway through a refactoring process (since 1-2 years), so it's a bit of a mess. At some point, I will be bored enough to finish that...

It's the second time I'm seeing a unit test failure on DataFilterTest.ElipsoidsDataPointsFilter. It works on my MacOS machine so I didn't catch it before. It seems to fail on my Ubuntu machine.
@HannesSommer do you observe the same thing?
@rdube was that test working when you added it? Is it a recent thing?

I think the result is random. Try
./utest/utest --gtest_filter=DataFilterTest.ElipsoidsDataPointsFilter --gtest_repeat=100 --path <POINTMATCHER_SRC>/example/data/
and it will fail sometimes.

Probably some thresholds need adapted or the random corner cases need to get filtered out.

ouuu lucky me. Indeed, it fails in two ways:

C++ exception with description "no outlier to filter" thrown in the test body.2.The difference between validTrans and testTrans is 0.059576287865638733, which exceeds 0.05, where
validTrans evaluates to 0.22068966925144196,
testTrans evaluates to 0.28026595711708069, and
0.05 evaluates to 0.050000000000000003.```

Finally I've found some time to but some work in this.
I've tried to implement all your suggestions.
I've reached the feature set of the VTK ASCII part (so reading writing including colour).
I've amended the unit test with a VTKBinary test.
The correctness of the written files I've tried to validate a bit with paraview. But this wasn't near to heavy testing! In my normal work flow I don't use colours and I don't read VTK with pointmatcher at all (only paraview). So the question is how do we test this? Can we mark this as a experimental feature? Do you, @pomerlef , have normal work that involves colour and reading VTK?

Me personally, no. It's mainly for people using the Kinect. I'll send you a file by mail in ASCII with a color field to test. The upload attachment size is limited to 10MB here.

If it's only few corner cases with the colors that you're not sure about, I wouldn't mind to add it in the main branch and deal with deal with bugs that might raise from it later. If somebody finds a bug, that means that he was in a better situation than us to test it.