Tried to render a non renderable component

Thanks for making this project. This will simply a lot of my tests and test configs.
I am starting to replace enzyme with it wherever possible

react-testing-library version: 1.6.0
node version: 9.4
npm (or yarn) version: 1.5.1
jest version: 22.4.2

Relevant code or config
import { render } from 'react-testing-library'

global.console = {
  error: jest.fn(),
}

...

test('should warn incase child is not provided', () => {
  render(<TestComponent />)
  expect(global.console.error).toHaveBeenCalledWith(`You must provide children`)
})

What you did:
Tried to render a non renderable component , to test if my custom console errors are thrown
What happened:
render throws error on non-renderable component rather than silently failing
Nothing was returned from render. This usually means a return statement is missing. Or, to render nothing, return null.

Reproduction repository:
currently i am playing with react-testing-library and trying to replace enzyme on a small react util i wrote.
This is a sample test currently written in enzyme, but on my local branch i am replacing it react-testing-library.
https://github.com/shrynx/react-render-function/blob/master/tests/index.test.js#L77
Problem description:
TestComponent shouldn't be able to render without Children, and for missing Children i am throwing errors from inside TestComponent which i want to test for using jest (i.e, test console.error message).
But the test fails at render itself throwing, rather than throwing my error.
Earlier i was using enzyne and used shallow in the same way i used render above and this used to work.
Suggested solution:
The render should fail silently or better if could be configured to not throw error, maybe like
const {container, unmount} = render(<Component />, {renderError: false})

I can try making a PR for it. should be easy to put try catch on ReactDOM.render and not throwing errors only  if user has configured not to.

@kentcdodds i added a PR implementing the mentioned fix
https://github.com/kentcdodds/react-testing-library/pull/19

Hi @shrynx! The reason is that it overcomplicates the library (both in implementation and use) and goes against the guiding principles. I would therefore recommend adding a try/catch in your test code around the render call as this will make things more explicit.
Based on your project I think you came up with an alternative. I'm glad. Good luck! And thanks for understanding :)


 Do you get it when calling pip-sync inside of a virtualenv?

Can you specify the contents of your *.in file here?

I don't know what method pip-tools uses to find out what's installed but in case it uses pip you may be interested to know that argparse (and a few others libs) are special cased in pip. See https://github.com/pypa/pip/issues/1570 and https://github.com/pypa/pip/issues/3132

I think this might be related to #328.

Is this still happening with newer pip-tools?

Closing this, a lot has changed. Comment if this is still an issue.


Release Google.Cloud.Metastore.V1 version 2.7.0

Changes in this release:
New features

Change netstandard2.1 target to netstandard2.0 (commit 82bea85)

Created release for Google.Cloud.Metastore.V1-2.7.0


Unable to set the name of a Doc Type added and IPublishedContent.Id is always 0

Great package - thanks.
I had hoped to reuse my document types and their current views unchanged in the grid layout via the Doc Type Editor.  I ran into 2 problems - my view was rendering the doctype.name property but this is always set to "doc type" now ..so i have to create a new doc type property to work around this.
The other problem was I was using the doctype "Id" property for a unique id on rendered html - but this is now always 0 for all doc types rendered. I can work around this with a unique id generator. I guess the doc types are not actually being stored as umbraco nodes so these issues may be "features".

@John-Blair Thanks for getting in touch!

The Name value its take from the Grid Editor itself, which is whatever is set for the entry in the "config/grid.editors.config.js" file, which out the box (for DTGE) is named "Doc Type". You can change that, or add other entries to the "grid.editors.config.js" file, (note, if the allowedDocTypes option matches a single DocType, then that's what will be used (automatically).

Failing that, you could look at the nameTemplate option, it's referenced in the docs here - but we are missing documentation on how it works explicitly.  From what I recall, it uses AngularJs expressions, e.g. "{{propertyAliasFromYourDocType}}".

The Id property, this is intentionally set to 0, as like you say these aren't "real" nodes, DTGE fakes them.  DTGE does set the Key property with a GUID, that could be used instead? (Although keep in mind that IPublishedContent doesn't have a Key property, (because that interface pre-dates its existence), so you'll need to call .GetKey() extension method to get the GUID value.

I hope all this helps.

Thanks for the very quick reply - appreciate the suggestions.

I have multiple doc types in the grid layout so found it more convenient to add a Title property as I want the user to be able to set it.
The Id property: You might be surprised to hear but this broke all my custom controls (tabs, accordions, carousels etc) as these are all doc types and I was using the node id for id and href attribute use. Thanks for your suggestion. In the end I decided to go with creating a UniqueId extension method for general use in my controls. FYI I made the UniqueId a long (instead of an int) and starting above max int so as not to  conflict with any possible umbraco node.  Code below in case anyone else hits this problem and wants a solution.

With the above changes I can now use the doc types and views either inside the grid layout or standalone with the same results.
namespace Ec.UmbracoExtensions
{
public static class ValueTypeExtensions
{
    // Some packages store document types without a node id - return 0.
    // Generate a "unique id" for use in html id attributes.
    // Using above max int as the starting point as this will not clash with any existing umbraco node id.
    private const long _longSeed = ((long) int.MaxValue) + 1;
    private static long _nextUniqueNodeId = _longSeed;

    // Make access thread safe.
    internal static readonly object idlock = new object();

    internal static long NextUniqueNodeId
    {
        get
        {
            lock (idlock)
            {
                if (_nextUniqueNodeId == long.MaxValue)
                {
                    // Reset seed.
                    _nextUniqueNodeId = _longSeed;
                }

                return _nextUniqueNodeId++;
            }

        }
    }

    public static long UniqueId<T> (this T id) where T : struct  {

        return NextUniqueNodeId;

    }

    // Get a unique block of ids for use in HTML controls for use in id and href synchronisation.
    // Just return the first of those ids - knowing the following sequential ids are reserved.
    public static long UniqueIdRange<T>(this T id, int rangeSize = 1) where T : struct
    {
        // Reserve a range of node ids in thread safe manner.
        // Note: Keep ids sequential even if mid range hits the max long id by resetting the starting id to the seed.
        lock (idlock)
        {
            // Handle wrap of ids early to keep ids sequential.
            if ( long.MaxValue - rangeSize < _nextUniqueNodeId) _nextUniqueNodeId = _longSeed;

            long result = _nextUniqueNodeId;
            _nextUniqueNodeId += rangeSize;

            return result;
        }
    }

}

}
Thanks
John Blair

From: Lee Kelleher<EMAIL_ADDRESS>Sent: 17 March 2019 12:47
To: umco/umbraco-doc-type-grid-editor
Cc: John Blair; Mention
Subject: Re: [umco/umbraco-doc-type-grid-editor] Unable to set the name of a Doc Type added and IPublishedContent.Id is always 0 (#139)
@John-Blairhttps://github.com/John-Blair Thanks for getting in touch!

The Name value its take from the Grid Editor itself, which is whatever is set for the entry in the "config/grid.editors.config.js" file, which out the box (for DTGE) is named "Doc Type". You can change that, or add other entries to the "grid.editors.config.js" file, (note, if the allowedDocTypes option matches a single DocType, then that's what will be used (automatically).

Failing that, you could look at the nameTemplate option, it's referenced in the docs herehttps://github.com/umco/umbraco-doc-type-grid-editor/blob/master/docs/developers-guide.md#example - but we are missing documentation on how it works explicitly. From what I recall, it uses AngularJs expressions, e.g. "{{propertyAliasFromYourDocType}}".

The Id property, this is intentionally set to 0, as like you say these aren't "real" nodes, DTGE fakes them. DTGE does set the Key property with a GUID, that could be used instead? (Although keep in mind that IPublishedContent doesn't have a Key property, (because that interface pre-dates its existence), so you'll need to call .GetKey() extension method to get the GUID value.

I hope all this helps.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/umco/umbraco-doc-type-grid-editor/issues/139#issuecomment-473662317, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AbeY8hU5U87zgvG8hmWGIMO8UK_MsAjvks5vXjlmgaJpZM4b4CBT.

Hi @John-Blair, I'm curious. When you say "custom controls", are you meaning that these were pre-built before using DTGE? I'm not sure why I have this impression that they are .NET Server Controls? But (hopefully) they are Razor partial-views? Like I say, I'm purely curious. 🙂
Maybe .GetHashCode() on the object might be unique enough? Save you the extra code. Anyway, you've written your code now and if it's working, all good. Cheers!

Hi Lee,
Yes, they are razor partial views (old naming habit from my earlier days of developing .net server controls).
I had developed these partial views before using the DTGE - so i just configured DTGE to point to the partial view directory and none of them worked initially due to the "id" = 0 issue.
I had considered .GetHashCode - but as this is only an "int" there is a chance this could return an existing umbraco node id - and i didn't want the possibility of clashing with any other 3rd party products where the node id was being used by their "controls" in the "id" attribute of the HTML - hence my use of a "long" starting at max int +1 and no possibility of collision (unless another developer has the same idea 🙂)
FYI The DTGE is a fantastic product and I was delighted at how little I had to change to now have all my "controls" - including multiple instances -  working in any grid layout!
Suggestion: W.r.t. DTGE, I would consider storing the instances of the doc types as actual umbraco nodes so they behave exactly like content nodes created from doc types, where Id and name properties could be set and available per usual use.
If that was not feasibly, another thing i'd consider is to add a "long UniqueId or GUID" to the Properties collection of the IPublishedContent object returned from DTGE - again as they are "fake" nodes I'd mandate the type as long or GUID instead of int. This would be useful for anyone else who experienced my issue. I'm curious if i am the only one experiencing an issue with id - maybe other partial view developers have a better way of generating unique "id" values in the html!
Thanks
John Blair

From: Lee Kelleher<EMAIL_ADDRESS>Sent: 18 March 2019 10:39
To: umco/umbraco-doc-type-grid-editor
Cc: John Blair; Mention
Subject: Re: [umco/umbraco-doc-type-grid-editor] Unable to set the name of a Doc Type added and IPublishedContent.Id is always 0 (#139)
Hi @John-Blairhttps://github.com/John-Blair, I'm curious. When you say "custom controls", are you meaning that these were pre-built before using DTGE? I'm not sure why I have this impression that they are .NET Server Controls? But (hopefully) they are Razor partial-views? Like I say, I'm purely curious. 🙂
Maybe .GetHashCode() on the object might be unique enough? Save you the extra code. Anyway, you've written your code now and if it's working, all good. Cheers!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/umco/umbraco-doc-type-grid-editor/issues/139#issuecomment-473857635, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AbeY8rLAg4KoUiv_7r7b7pPMgRU-oRV-ks5vX2zhgaJpZM4b4CBT.

Hi @John-Blair, I mentioned in my initial reply that each item has a GUID set, you can get the value using .GetKey().
re: using "real nodes", yup we'd love that too! It's something we considered when we started development on DTGE. It introduced many complications, in terms of performance and structure, e.g. where would these nodes live in the tree? We had many discussions with Umbraco core team about it.
Ultimately, DTGE is a hack (in a non-offensive way), in order to re-use Doc Types for smaller fragmented content items.
Hopefully one day, this approach may influence Umbraco core to adopt a similar concept. 😃

Hi Lee, Yes I agree - one for the umbraco team to adopt.
FYI I'd put the nodes in the Settings/Content Templates area of the tree.
[cid:7f813cd3-503b-4efc-8f29-975cd9eb9775]
Thanks
John Blair

From: Lee Kelleher<EMAIL_ADDRESS>Sent: 18 March 2019 11:40
To: umco/umbraco-doc-type-grid-editor
Cc: John Blair; Mention
Subject: Re: [umco/umbraco-doc-type-grid-editor] Unable to set the name of a Doc Type added and IPublishedContent.Id is always 0 (#139)
Hi @John-Blairhttps://github.com/John-Blair, I mentioned in my initial reply that each item has a GUID set, you can get the value using .GetKey().
re: using "real nodes", yup we'd love that too! It's something we considered when we started development on DTGE. It introduced many complications, in terms of performance and structure, e.g. where would these nodes live in the tree? We had many discussions with Umbraco core team about it.
Ultimately, DTGE is a hack (in a non-offensive way), in order to re-use Doc Types for smaller fragmented content items.
Hopefully one day, this approach may influence Umbraco core to adopt a similar concept. 😃
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/umco/umbraco-doc-type-grid-editor/issues/139#issuecomment-473876076, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AbeY8kSZq1ZI0ueCREgl16ASB3RJ9P1Yks5vX3s2gaJpZM4b4CBT.


added performance testing

Changes:

added performance.test.ts using jest
Mainly testing on Zerox latency on PDFs with different number of pages and different concurrency params
added mock on openai call, and set timeout to 5s

Current test results:

To run the performance test (make sure to npm install first to install jest):
npm run test:performance

LGTM 🚀


[Feat] OnBoarding Frontend Navigation Bar

required!

Done here
https://github.com/peppelinux/spid-cie-oidc-django/pull/24


Update plugin for Pipeline support

Ported to newer analysis-core and adapted to be pipeline-friendly

Have not yet verified this won't blow away existing results. Working up a 'live' environment to ensure this isn't the case at some point today or tomorrow.

It is great that you're doing this.  Did you also add a definition for a step for this publisher like it does
https://github.com/jenkinsci/workflow-basic-steps-plugin/blob/300fe6c02b41f072e50a501cfec3e2f425048446/src/main/java/org/jenkinsci/plugins/workflow/steps/ArtifactArchiverStep.java#L37-L53?

@bitwiseman Was planning on it. Haven't noticed a particular naming/param convention for other reporting plugins (particularly HealthAwarePublisher derived ones), planned to do a bit more research on that front to be sure I'm not violating any expectations.

@ConstantineXVI There doesn't appear to be one at this time, so naming is a bit all over the place.  I could make some suggestions but they'd be only my personal tastes. 😄

Hm, I'm not sure how to add in a pipeline step that reuses the "classic"[?] publisher class instead of reimplementing it. Entirely possible I'm just not finding the right docs on this.

Would this work?
https://github.com/jenkinsci/htmlpublisher-plugin/blob/73fdcc6e0a668d27bc390567f95cc8c9ec445cc1/src/main/java/htmlpublisher/workflow/PublishHTMLStep.java

Seems it would, yes

Very cool.

Confirmed old results and builds still work. I'm willing to call this sane.

:heart_eyes:

Does anything else need to be done to get this merged?

Probably not, I just need to test it out myself.

*ping

Sorry for the delay, I was waiting for maven to finish :racehorse:
I noticed there's no Brakeman warning trend graph for the pipeline project...is this expected?
Otherwise, looks fine to me.

I'll give that a peek. Suspect I could be setting better defaults for the DSL step.

OK, from my testing this looks like it might be an upstream issue/unimplemented feature in analysis-core with pipelines. May fix it anyway, but don't think I can do it here.

Thank you so much for these updates!
Are you interested in being a maintainer for this plugin?

Sure.

Okay, I'll just do this next release and then get you set up.

https://issues.jenkins-ci.org/browse/JENKINS-31202 (analysis-core pipeline trend graphs) already in progress by Ulli, just to close that loop


Information missing to Inserting and Querying partition tables

I think there should also be information around how to query partitioned tables as well as on How to insert record into proper partition.
Like when querying record, do we need to be aware of the partition?
Do we need to supply partition information on the Query?
What is i am not sure which partition the data is on, can it scan all partitions?
similarly, when inserting do i need to provide hint on which partition to store data or is this automatic based on the value on in the partitioned column?

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

ID: 4466c524-282b-adc0-d539-3eb85d956597
Version Independent ID: 08052bb4-d7bb-02ab-6d37-e3fcce20846c
Content: Create partitioned tables and indexes - SQL Server, Azure SQL Database, Azure SQL Managed Instance
Content Source: docs/relational-databases/partitions/create-partitioned-tables-and-indexes.md
Product: sql
GitHub Login: @VanMSFT
Microsoft Alias: vanto

@nssidhu -- thank you for your feedback and questions.  Please consider these resources:

SQL Server on Microsoft Q&A
DBA Stack Exchange
Stack Overflow

@VanMSFT -- Van, please look into this issue.

Hi @nssidhu, your question would be better suited to a support forum, but hopefully I can answer your questions here.

Like when querying record, do we need to be aware of the partition?

No.

Do we need to supply partition information on the Query?

No.

What is i am not sure which partition the data is on, can it scan all partitions?

It depends on the query plan. If your WHERE clause contains the partitioning key, or an index helps the engine figure out where the data is, the query optimizer can perform partition elimination and skip scanning all of the partitions if possible.

similarly, when inserting do i need to provide hint on which partition to store data

No.

[...] or is this automatic based on the value on in the partitioned column?

Correct.


Upload pyATS results to TRADe from docker container

@simingy
Hello, Siming Yuan!
Please advise me how can I upload pyATS results to TRADe from docker container?

Hi,
Sorry for the long delay.
The pyATS Docker image has the external version of pyATS, which does not support uploading to TRADe. What you are looking for is a Docker image with the internal version of pyATS. This is easy to build using the pyATS Image Builder.
https://github.com/CiscoTestAutomation/pyats-image-builder
You can make a YAML file something like this:
tag: myimage:latest
packages:
  - ats[full]
pip-config: |
  [global]
  format = columns
  no-cache-dir = false
  trusted-host =
      pypi.python.org
      pyats-pypi.cisco.com
  index-url = http://pyats-pypi.cisco.com/simple
  disable-pip-version-check = 1

This will build a new Docker image with the internal version of pyATS installed, which supports uploading to TRADe via REST api calls.

@bastell
Uploading to TRADe via REST api calls is exactly what I am looking for.
You mean use Log Uploader for it or some other API?
If other, please give me a link to the documentation of this API for uploading to TRADe via REST! =)

Hi,
Uploading to TRADe via REST happens automatically when using the internal version of pyATS, and is unsupported when using the external version of pyATS. The important difference is to install ats from the internal PYPI server, instead of pyats from the public PYPI server. The image ciscotestautomation/pyats has external pyats installed, so you will have to make your own image with ats installed. This is very straightforward using the pyATS Image Builder.


Sharepoint Client authority Exception

We're trying out the sharepoint client from SystemApp, but we're getting an exception, here an excerpt from the eventlog:
Message System.ArgumentException in AcquireTokenByAuthorizationCodeAsync (action id: 099c3d7f-584c-457a-9e57-f0550c68473a): 'authority' should be in URI format.  (Parameter 'authority')
The webclient opens the OAuth page and closes it, but seems to crash in the GetLists()-function
procedure TestSharepointClient()
    var
        SharePointList: Record "SharePoint List";
        SharePointClient: Codeunit "SharePoint Client";
        SharePointAuth: Codeunit "SharePoint Auth.";
        SharePointAuthorization: Interface "SharePoint Authorization";
    begin
        SharePointAuthorization := SharePointAuth.CreateAuthorizationCode('<Azure Tenant ID>', '<Client ID>', '<Client Secret>', '00000003-0000-0ff1-ce00-000000000000/.default');
        SharePointClient.Initialize('<Sharepoint Url>', SharePointAuthorization);
        SharePointClient.GetLists(SharePointList);
    end;

I've been trying in the BC22 public preview
Version: DE Business Central 22.0 (Platform 22.0.54512.0 + Application 22.0.54157.54529)

@jwikman was involved in some change for the sharepoint authorization. https://github.com/microsoft/ALAppExtensions/pull/21591
Maybe he can help in this specific topic.

@jwikman
thanks for you  reply. I've tried using a guid with  and without the curly brackets. what would the correct usage be?

now I'm getting a HttpRequestException  "Exceeded maximum number of requests (2)"

also the login page is opening every time. will this be a problem when using the sharepoint client in a task scheduler context?

Are you running OnPrem/Container or SaaS?

Ok. BC22 Public Preview should have the latest fixes I added, so that should be good.
I've never seen the Exceeded maximum number of requests (2) error. I have no clue on that, sorry...
Reg. the "login page opening every time": After my addition earlier this year, the authentication logic tries to fetch an access token from the token cache inside BC (using stored Refresh Tokens). If that fails, the authentication window is opened. Often the browser has cached tokens as well, so that window might just flash open and close. But if nothing cached in client either, the user must logon again.
The refresh token in BC should be usable for up to 90 days, but shorter times could be configured in AAD.
I'm using the same approach in one of our AppSource apps. And if the user authenticates once, it can be used in Job Queue for several months. I'm catching the authentication error and uses notifications for the user to re-authenticate when logged on the next time.
The best thing to have if you want to run in a Job Queue, obviously, is Service to Service authentication with the Client Credentials Flow. But that requires certificates, and System App does currently only support Client Credentials Flow with certificates that are password protected. And I think that implementing this without password do not comply with security best practices, so I'll wait until that is supported before implementing this in the SharePoint Authorization module. This is requested in https://github.com/microsoft/ALAppExtensions/issues/22261.

BC22 Public Preview sandbox container and the sharepoint is a CDX, so the webclient is in private mode

Try a SaaS sandbox instead, and see if that helps.
And make sure to add https://businesscentral.dynamics.com/OAuthLanding.htm as Redirect URI on the App Registration.

okay, if the AcquireTokenWithClientCredentials is not yet possible I can stop my testing for now.
Don't know what you mean by "requires certificates"? certificates where? We had implemented an OneDrive connection via GraphAPI with Oauth2 AcquireTokenWithClientCredentials and this works on containers without SSL

also, when I got your attention:
did you not have an issue with uploading files that are greater than 4MB? but I'm guessing your using the sharepoint API and not GraphAPI, right?

It is a requirement from SharePoint, we discovered while trying to implementing this. Using an access token that is created from ClientId + ClientSecret only (with the Client Credentials Flow) is not accepted by SharePoint.
You can read more at https://learn.microsoft.com/en-us/sharepoint/dev/solution-guidance/security-apponly-azuread#setting-up-an-azure-ad-app-for-app-only-access
I've created a draft PR (https://github.com/microsoft/ALAppExtensions/pull/22404/files) for Client Credentials with a certificate that is not password protected, but I don't want to publish that until we get proper support for password protected certificates. But you can copy that code and implement your own authentication to be used with the SharePoint Module if you are OK with certificates without password - we've tested and got it to work.

also, when I got your attention: did you not have an issue with uploading files that are greater than 4MB? but I'm guessing your using the sharepoint API and not GraphAPI, right?

The SharePoint module is using the SharePoint API, from what I understand.
A colleague of mine tested with a 19.5Mb file earlier, and that worked. He will test again with even larger files to see if we can hit any limits.

okay, thanks for all the info! :-)
We'll take another look when the client credential is officially implemented

Just tested upload/download with a 120Mb file - all fine. 👍


Keep newlines at end of file

When formatting files, git diff says "\ No newline at end of file". I prefer to always have a newline at the file end. At least that is what all projects I am working on use.

Does it mean the last char of the buffer is '\n'? Then I have tried to fix it.. in the newest commit

https://github.com/Decodetalkers/neocmakelsp/commit/dd06d5365970c731e27404b3250d536869704c3c can you take a try?

Ok, I will make a new tag, I think I have already handle it

/home/user/projects/neocmakelsp/target/debug/neocmakelsp -F -o CMakeLists.txt
That does not seem to work with the CMakeLists.txt of lets say https://invent.kde.org/frameworks/kcoreaddons/-/blob/master/CMakeLists.txt?ref_type=heads
-kde_configure_git_pre_commit_hook(CHECKS CLANG_FORMAT)
+kde_configure_git_pre_commit_hook(CHECKS CLANG_FORMAT)
\ No newline at end of file

Fixed, and I add unit test

Can confirm, thanks!


Add support for aws_apigatewayv2_route

Q
A

🐛 Bug fix?
no

🚀 New feature?
yes

⚠ Deprecations?
no

❌ BC Break
no

🔗 Related issues
https://github.com/cloudskiff/driftctl/issues/1249

❓ Documentation
no

Description
This is the first resource that I'm trying to add to driftctl, so I may have got various things wrong.
I've ticked "no" on the documentation question above, I'm not sure if that's right? Is there a list of resources we support that I should update too?
I haven't yet written middleware to avoid spurious drift in the case in which the terraform user defines an aws_apigatewayv2_api with an inline openAPI declaration, that will cause AWS to create routes. There is already middleware for this exact case in the v1 API gateway, so I can probably extend that to support v2. Is there value in merging this first, or will the possibility of spurious drift confuse users, in which case it might be best to add middleware here before merge? I'm happy either way, but figured I'd ask for feedback early.
Closes https://github.com/cloudskiff/driftctl/issues/1249

I've ticked "no" on the documentation question above, I'm not sure if that's right? Is there a list of resources we support that I should update too?

For each provider, we maintain a list of supported resources and whether deep mode is available or not, in the documentation. Please make a PR on the relevant repository for that resource 🙏🏻  (example).

Is there value in merging this first, or will the possibility of spurious drift confuse users, in which case it might be best to add middleware here before merge?

I think it's useless to add this resource without the relevant middleware because we don't want the user to have false positives anyway. It's better to spend more time on it rather than releasing an incomplete support of the resource IMO.

Hey @sundowndev, thanks for the docs! I'll be sure to file an update after I add the middleware to this.

I think it's useless to add this resource without the relevant middleware

Minor semantic point, but I disagree that it's useless: it's useful to users who don't use openAPI declarations, but yes now that I think again it arguably offers negative, not neutral experience to users who do. False positives are perhaps worse than no support at all!
This doesn't really matter because the result is the same: I'll take this back to draft and add middleware 🙂

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

Codecov Report

Merging #1252 (971453f) into main (2b7dae2) will increase coverage by 0.02%.
The diff coverage is 90.69%.

@@            Coverage Diff             @@
##             main    #1252      +/-   ##
==========================================
+ Coverage   81.68%   81.71%   +0.02%
==========================================
  Files         392      393       +1
  Lines       14162    14205      +43
==========================================
+ Hits        11568    11607      +39
- Misses       2301     2304       +3
- Partials      293      294       +1

Impacted Files
Coverage Δ

pkg/remote/aws/init.go
0.00% <0.00%> (ø)

pkg/resource/resource_types.go
100.00% <ø> (ø)

pkg/remote/aws/apigatewayv2_route_enumerator.go
89.65% <89.65%> (ø)

...g/remote/aws/repository/apigatewayv2_repository.go
88.37% <100.00%> (+5.03%)
:arrow_up:

@wbeuil I think this is ready for review again. Although I'm not sure exactly what you mean by "It misses the test inside the pkg/resource/aws/metadata_test.go file for non deep-mode flag", so if it means something other than what was in the line notes in that same review, I haven't done that yet.
I've implemented the middleware, let me know what you think.

Thanks @wbeuil, I've made those changes, this should be ready for another review


Chore/type fixing

Fixed a bunch of typing issues (most of which were identified with Mypy).
Didn't fix everything according to Mypy, but it does cover most, and practically every error in Pycharm.
Also, Pylance should now do type hints correctly wrt StreamingSeries and StreamingDataFrames

Done, ready to merge, just need approval


Lens background-image distorted

The image of the lens setted as the lens background-image seems to distort due to css classes at image to be sized.

Image seems to work properly if thus classes is removed (as well as the css properties of the class) as the image uses the real width x height of the file (overflows from red border-box line)

Ok, so, after checking issue #3 i've set the background size using the thumb.width and thumb.height
lets see how it works.


termuiimg update

Hey I came across your package for the termui widget.
How do you update the image in place?
I am generating new QR codes of the same size (width/height), and I wish to update the image in the console.
Thanks in advance.

Thanks for testing this library!
I added a SetImage method and it now uses (and exposes) *term.Canvas as the widget core. There are still issues with this package and the module in whole. If you use termimg together with a TUI library, both will read from the same input, I am trying to fix that. For now you have to close the termimg instances before the TUI can parse input.


WIP Introduce patched state update

Closes #352
Partially covers #268 and #295

Roadmap

[x] Introduce synchronous way of applying state patches
[x] Introduce asynchronous way of applying state patches
[x] Deprecate Form.updateFieldsWith
[x] Deprecate Form.fieldsDidUpdate
[x] All tests are passing

[x] Field unmounting
[x] Touched: A field becomes touched after focus and blur
[x] Form-wide errors: All test suits
[x] Conditional schema (fixed by 93a9f377454fc823085870d9bb6c6746f0f5c995)
[x] Reactive props

[x] Obsolete value of value during field change (fixed by 223ed93)
[ ] Solve intermediate state composition issue (and recordUtils call signatures)

Although patched state updates eradicate field value concurrency issue that required to have a delay set while typing into inputs during tests, it still doesn't help with random integration tests failing during their CLI execution. That matter may require more investigation, and I don't plan to do that as a part of this pull request.

I've tried to simplify record utils as a part of function call signature adjustment, and I can conclude that indeed a lot of payload coming into utility functions can be omitted, or replaced with the existing payload. I can't say it's perfect now, but some complicated utils (updateValidityState) where drastically simplified, removing the need for intermediate states.


Added optional type definitions for Change action type

Fixes #3620

Codecov Report

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

@@          Coverage Diff           @@
##           master   #3624   +/-   ##
======================================
  Coverage     100%    100%
======================================
  Files          70      70
  Lines        1571    1577    +6
======================================
+ Hits         1571    1577    +6

Impacted Files
Coverage Δ

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

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

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

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

Continue to review full report at Codecov.

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

Ready to merge


make simple indicator not light dependent

Make LightDependentProtein a subclass of Protein. Need function for synapse source: ng in Protein and light agg ng in LightDependentProtein
make constructor functions include option for simple or light_dependent, e.g.,
def jgcamp7f(light_dependent=False):
    if light_dependent:
        return LDGECI(**params)
    else:
        return SimpleGECI(**params)

gcamp = jgcamp7f(light_dependent=[True|False])

#39


Maven central version

Shows 4.0.1 as  the latest, but main branch is more recent, eg. getRestricted() doesn't exist in 4.0.1
Will there be a new version made available?
btw https://search.maven.org/search?q=g:"com.dustinredmond.fxtrayicon" AND a:"FXTrayIcon" doesn't bring anything useful back.
thanks

I could have sworn I pushed a new release. Let me do that now. Thanks for opening this issue!

Yep i usually check on here: https://mvnrepository.com/artifact/com.dustinredmond.fxtrayicon/FXTrayIcon

4.2.0 is available - works! Many thanks!


proper error message should be thrown when reading non-rdf triples

Steps to reproduce::
writerClient = DatabaseClientFactory.newClient("localhost", restPort,
dbName, "rest-writer", "x", Authentication.DIGEST);
gmWriter = writerClient.newGraphManager();
File file = new File("test-complete/src/test/java/com/marklogic/client/functionaltest/data/semantics/non-iri.xml");
FileHandle filehandle = new FileHandle();
filehandle.set(file);		 gmWriter.write("htp://test.sem.graph/tripxml1",filehandle.withMimetype(RDFMimeTypes.TRIPLEXML));
StringHandle handle = gmWriter.read("htp://test.sem.graph/tripxml1", new StringHandle());
Current Exception::
com.marklogic.client.FailedRequestException: Local message: failed to read resource at graphs: Internal Server Error. Server Message: Request failed. Error body not received from server
Expected :: More meaningful message with SEM-NOTRDF, from the server side.
SEM-NOTRDF: "5.11"^^http://www.w3.org/2001/XMLSchema#double "5.11"^^http://www.w3.org/2001/XMLSchema#double "3.11"^^http://www.w3.org/2001/XMLSchema#double . (err:FOER0000):

Issue fixed by #323


HM-4508: Date on metrics cards

This pull request adds the text As at {month} {year} to the metrics cards for percentage awarded to SMEs and total contract value on the landing page. The month and year are retrieved from the database via the API; if no value is received, the text is not displayed.

Empty paragraph has been removed and whitespace has been instead added using CSS margin. Some length units had to be changed from em to rem to allow the margin to match the height of the paragraph element containing the "As at  " text in the other cards.


ci: added model evaluation workflow for github action

The model-evaluation workflow only accepts manual trigger, it will run a complete cycle of deploy, run evaluation and undeploy.

This CI workflow relies on the scripts on the evaluation branch, this workflow will not work properly without the scripts on the evaluation branch merged

Purpose

Does this introduce a breaking change?
When developers merge from main and run the server, azd up, or azd deploy, will this produce an error?
If you're not sure, try it out on an old environment.
[ ] Yes
[x] No

Does this require changes to learn.microsoft.com docs?
This repository is referenced by this tutorial
which includes deployment, settings and usage instructions. If text or screenshot need to change in the tutorial,
check the box below and notify the tutorial author. A Microsoft employee can do this for you if you're an external contributor.
[ ] Yes
[x] No

Type of change
[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[x] Other... Please describe: CI configuration

Code quality checklist
See CONTRIBUTING.md for more details.

[x] The current tests all pass (python -m pytest).
[ ] I added tests that prove my fix is effective or that my feature works
[ ] I ran python -m pytest --cov to verify 100% coverage of added lines
[ ] I ran python -m mypy to check for type errors
[ ] I either used the pre-commit hooks or ran ruff and black manually on my code.

@GrayNekoBean, rebased on top of main (it was pointing to a random branch, hf-client-integration).

I'll be here explain one by one.

To begin with, we need instructions for testing this. The workflow relies on an evaluation script that's not merged yet. Where can one find this script?

The scripts are on the evaluation branch, it will need the whole directory of ./evaluation/. But even we have those scripts, it's hard to test them locally, even with this tool, you will need to download a 75G large image to test this workflow locally. That's also the reason I cannot test it locally after I introduced a certain step.

What do they need to do to use it? How can they trigger the workflow?

Here's a command to manually trigger this CI workflow but using the original ci branch (safely):
gh workflow run full-evaluation.yml --repo ucl-contoso-chat/ucl-openai-search --ref ci -f provision=false -f deploy=false -f purge=false

I'll add this to my pr message if you think this is okay.

What sort of information do they absolutely need to provide? I see that the action reads loads of things from the GH environment, are these already set?

There're some arguments to set whether to deploy or undeploy the online services. They all have a default value and I could add more instructions to them in the PR message. They can be specified using the command line arguments. For the GH environments, yes they are already properly set in this repository and works globally regardless of the branch. No more actions required to set environment variables.

Apart from that, I see that the workflow duplicates all code found in azure-dev.yaml. Is there any reason why we can't reuse that?

Yes, in fact many fractions of code are from the azure-dev.yaml, earlier I've also thought about invoking that workflow file via call_dispatch but as the arguments were introduced and they separately control two steps for "provision" and "deploy", this way would not work properly anymore.

I don't get why we allow controlling the deploy and provision steps individually. Provisioning without deploying makes no sense. Deploying without provisioning is a valid scenario, when we wanna use existing infrastructure but update the application code for some reason. But if the infrastructure exists, the provisioning step will be a noop anyway. Even if these flags made sense, you would add them to azure-dev and have one workflow trigger the other instead of copy-pasting 100 lines of code.

you will need to download a 75G large image to test this workflow locally

This sounds terribly alarming. I'm guessing it refers to the evaluation side so we'll discuss it there when the time comes.

you will need to download a 75G large image to test this workflow locally

This sounds terribly alarming. I'm guessing it refers to the evaluation side so we'll discuss it there when the time comes.

Sorry if this sentence sounds misguidance. That's the tool's problem, nothing related to our codebase. It's only because that tool needs to download a larger docker image to run the github action workflow locally, before it only needs a smaller one, but in this workflow it runs an "az login" step which requires its largest image.

I don't get why we allow controlling the deploy and provision steps individually. Provisioning without deploying makes no sense. Deploying without provisioning is a valid scenario, when we wanna use existing infrastructure but update the application code for some reason. But if the infrastructure exists, the provisioning step will be a noop anyway. Even if these flags made sense, you would add them to azure-dev and have one workflow trigger the other instead of copy-pasting 100 lines of code.

okay I'll consider changing the mechanism of these two arguments and using workflow trigger

There will be more changes to our codebase in the foreseeable future, so constantly changing the CI workflow to keep it in sync with other branches is not an efficient exercise at this time. Additionally, I've noticed that frequent redeployments by CI could be interruptive to other team members' work. Therefore, work around the CI branch will be halt until the other branches are confirmed as complete.

I've followed the instructions found here to create a Microsoft Entra application for our CI deployments and generate federated credentials for it so that we don't use our own client ID. For now, I'm setting the branch to ci-config in the subject identifier, but once we merge this will only be main.

@GrayNekoBean, PTAL at my changes. The commits won't help you much, cause I was testing the workflow and making changes on the fly, but it should be easy enough to tell what's altered by looking at the final diff.
The main change introduced was making the workflow self-contained, i.e. handling the deployment and purging of the resources apart from the evaluation. Previously, I argued that we should be calling azure-dev to handle provisioning/deploying, but testing showed that's not feasible in our case! Running provision and deploy separately skips crucial steps of the pipeline defined in azure.yaml, most importantly postprovisioning, which entails running prepdocs to populate AI Search. This can be seen in this failed pipeline which reports that the index was not found. The reason why that had never shown before was that the original version of the code relied on the synthetic data being committed to the repo (therefore skipping the generation step that actually makes use of AI Search). Of course, that approach was misguided; we are now running the generation script before proceeding with the evaluation steps.
The other major change involved cleaning up most of the environment/repo variables, since they only created a mess, making debugging impossible (this makes the workflow less configurable of course). Using the client ID retrieved from your local environment meant that I couldn't run the deployment for different environments. I believe I fixed that with the approach described here. Was this PR ever actually tested e2e? Did it ever go through a fresh deployment? Did azd ever work (I had to add the separate login logic to allow using azd, e.g. when purging)? Note also how I'm using a script to generate a .env file based on the actual values generated by the deployment, thus not having to rely on values hardcoded in the repo environment.
Apart from that, I reinstated a flag like the one you had before to skip deployments (arguably, it's much more convenient for testing, still doubt its practicality in any real use case) and I made the azd environment and number of questions for evaluation configurable. Finally, I made it so that purging can be triggered (when specified by the user) even in cases where other steps fail (crucial in order to avoid leaving stale resources behind when a deployment fails).


Unable to merge microsoft.bcl assemblies

Microsoft has made three assemblies available to support async await and HttpClient on v4. These are:
microsoft.bcl
microsoft.bcl.async
microsoft.net.http
I'm unable to successfully merge a project when these assemblies are included.
The exception I get is:
1>  Mono.Cecil.AssemblyResolutionException: Failed to resolve assembly: 'System.Core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'
1>     at Mono.Cecil.BaseAssemblyResolver.Resolve(AssemblyNameReference name, ReaderParameters parameters)
1>     at Mono.Cecil.BaseAssemblyResolver.Resolve(AssemblyNameReference name)
1>     at Mono.Cecil.DefaultAssemblyResolver.Resolve(AssemblyNameReference name)
1>     at Mono.Cecil.MetadataResolver.Resolve(TypeReference type)
1>     at Mono.Cecil.ModuleDefinition.Resolve(TypeReference type)
1>     at Mono.Cecil.TypeReference.Resolve()
1>     at ILRepacking.ReferenceFixator.FixReferences(Collection`1 attributes)
1>     at ILRepacking.ReferenceFixator.FixReferences(MethodDefinition meth)
1>     at ILRepacking.ReferenceFixator.FixReferences(TypeDefinition type)
1>     at ILRepacking.ILRepack.Repack()
1>     at ILRepacking.Application.Main(String[] args)
I made a test project in VS to demonstrate the issue which is available here:
https://backup.filesanywhere.com/fs/v.aspx?v=8d6a648d5a607077af6e
Any help would be greatly appreciated!
Steve

Duplicate #58
Will be fixed in 2.0, though I think you can work around it if you somehow force ilrepack to execute with v4 clr.


Is there a way to get intermediate results from the MPSolver?

Hello,
I've been using the MPSolver (CBC) to solve a MIP and I've been wondering if there is any way to get intermediary results while it's running? Is there a way to set a callback or another way to get information about the current best found solution?
I've been searching trough the documentation and API but wasn't able to find anything.
Thanks!

Which language do you use ? (c++, python, C#, java )

AFAIK the MPSolver is only a wrapping the CBC solver
see: https://github.com/google/or-tools/blob/stable/ortools/linear_solver/cbc_interface.cc so unfortunately there is no callback.
If you can use the CP-SAT solver instead then you'll have solution callback support...

Hey @JeroenGar ,
This solution is a bit hacky, but it works:

Set a time limit on the solver via set_time_limit that is less than the known solve time, or simply very small.
Write a solve loop that queries if the solution is optimal, and if not, adds an additional time increment onto the time limit.

A lot of code left out, but you can see the strategy below. For full code, go to https://github.com/rychien-official/payoff/blob/master/source.py.
    # Solve ----------------------
    if verbose[0]==True:
        status = solver.NOT_SOLVED   # set status flag to not solved
        i = 1   # set index flag
        while status != solver.OPTIMAL:
            status = solver.Solve()   # query solver status
            wall_time = solver.wall_time()  # query current solver time
            if status in (solver.FEASIBLE, solver.OPTIMAL):
                interest_cost = \
                    solver.Objective().Value() - sum(principal_initial) - sum(
                        [remnant[i,t].solution_value()
                            for i in range(0, num_loans)
                            for t in range(0, term_months[i])])    # interest cost w/ optimal schedule
            else:
                interest_cost = "__not_solved__"
            print("Optimal solution is: $" + str(interest_cost)
                + " after " + str(wall_time/1000) + " milliseconds.")
            i = i+1   # update flag
            solver.set_time_limit(time_limit_milliseconds * i)   # update time limit
    else:
        status = solver.Solve()

The full code results in:
>>> opt_pay_schedule()
Optimal solution is: $__not_solved__ after 0.125 milliseconds.
Optimal solution is: $__not_solved__ after 0.192 milliseconds.
Optimal solution is: $6059.492170638128 after 0.326 milliseconds.
Optimal solution is: $6055.741729656031 after 0.51 milliseconds.
Optimal solution is: $6055.741729656031 after 0.736 milliseconds.
Optimal solution is: $6055.741729656031 after 1.017 milliseconds.
Optimal solution is: $6055.741729656031 after 1.346 milliseconds.
Optimal solution is: $6055.741729656031 after 1.743 milliseconds.
Optimal solution is: $6055.741729656031 after 2.193 milliseconds.
Optimal solution is: $6055.741729656031 after 2.638 milliseconds.

Python 3.7.3 and Coin OR CBC.
BR,
Ryan

Thanks for the responses! I'm using Java.
@Mizux I'll check if I can use the CP-SAT solver.
@rychien-official This could also be a good solution. Would there be a significant performance penalty if the search is interrupted every few seconds?

@JeroenGar there definitely is a performance penalty:
# Import
import sys
sys.path.append("C:/gitrepo/payoff/")
from source import *
import numpy as numpy
import timeit

def verbose_opt():
    opt_pay_schedule(verbose=[True, 50])

def silent_opt():
    opt_pay_schedule(verbose=[False, 50])

verbose_opt_time = timeit.timeit(verbose_opt, number=10)
print(verbose_opt_time)

silent_opt_time = timeit.timeit(silent_opt, number=10)
print(silent_opt_time)

We get these timings:
>>> verbose_opt_time
26.80526470000001
>>> silent_opt_time
5.457788499999992

So the verbose option with interrupts to get intermediate results is 5x slower, given interrupts every 50 milliseconds.
Perhaps the smarter strategy is to run a solve without interrupts first, query the total runtime, then set the interrupts based on that, to limit performance degredation. Maybe using a weighted time formula, such that we let the solver get very close to optimal before starting to interrupt.
Run on a cheap laptop, Intel 7200u processor and 8gb ram.

Based on the following math, I interpret that the solver starts from scratch each time.
Given interrupts of 50 ms and 500ms for an optimal solve:
2750ms = 50+100+150+200+250+300+350+400+450+500

The above is approximately equal to 2.6 seconds, which over a sample of 10 runs would sum to approximately 26 seconds.

@rychien-official
Thanks, but I think this approach is ideal for my particular use-case since I'm dealing with models that often require a bit of time to solve.
@Mizux
I've modified the problem to use the CP-SAT solver and the callback functionality works flawless!


Add polling to ext service test

Fixes #146
Signed-off-by: Andreas Neumann<EMAIL_ADDRESS>

My only concern is that "false is not true" is not a great message when a test fails.
The Infofs do help, but they have the downside of spamming the output.
I wonder whether we could come up with something even better? Perhaps return a structured object from the func and do a more complex assertion (which hopefully will have a more meaningful error message)?

I mostly agree. I tried to figure something out, but it's all pretty annoying. The structured object return might be an option, but it adds a lot of complexity.
I don't think the log spam is too much, i've added a bit more detailed logs and removed the spammy output that was there before.


[Bug]: The workspace settings/options pop up does not close when we scroll through the page.

Is there an existing issue for this?

[X] I have searched the existing issues

Description
If we open the workspace settings/options pop up by clicking the 3 dots, it does not close when we scroll through the page.
https://www.loom.com/share/ea29dabf3d03458ab871669be8dd7fa3
Steps To Reproduce

Go to the homepage.
For any workspace click on the 3 dots next to the + New button.
Now scroll through the page.

Expected: The settings pop up should automatically close when we scroll through the homepage.
Public Sample App
No response
Version
Cloud

With the recent revamp of the homepage, the issue is no longer relevant as multiple workspaces are no longer displayed one below the other. Therefore, closing the ticket.


Fail CI on failed fmt or other linting file changes

Summary of changes
Changes introduced in this pull request:

Changes the CI lint step to check for changes resulted from e.g. cargo fmt. Before they were silently discarded.

Just change the lint target to run cargo fmt --all -- --check. It should not actually do any formatting.

Just change the lint target to run cargo fmt --all -- --check. It should not actually do any formatting.

Yes, but then anyone wanting to run the lint will have to cargo fmt --all himself - trying not to introduce breaking changes to existing contributors :smile: . What do you think?

Just change the lint target to run cargo fmt --all -- --check. It should not actually do any formatting.

Yes, but then anyone wanting to run the lint will have to cargo fmt --all himself - trying not to introduce breaking changes to existing contributors smile . What do you think?

I think it is a mistake that lint modified the code in the first place. In my mind, linting should check for problems, not automatically fix them.


Trouble loading the streamlit_option_menu component

I am running an app on Steamlit cloud. Sometimes I receive the following error:
Your app is having trouble loading the streamlit_option_menu.option_menu component.
(The app is attempting to load the component from ****, and hasn't received its "streamlit:componentReady" message.)
If this is a development build, have you started the dev server?
If this is a release build, have you compiled the frontend?

Same issue here, never happens on a development server, only on production deploys, and right after the deploy. It seems that some time later it starts working normally.
Versions on my setup:
$ pip list | grep streamlit
streamlit             1.12.2
streamlit-option-menu 0.3.2

I have found a solution for my case -- described here HTH.

I have found a solution for my case -- described here. HTH.

How this work on streamlit cloud, can you give an example?

@wraza2k9: Not using Streamlit Cloud, so take my thoughts with a grain of salt. My guess is it won't be possible with a default deploy as it probably expects you to provide path to the application main page and then launches that. Deploying via a docker container on Kubenetes is what I do, having this clause at the end of the Dockerfile, to get the non-standard launcher executed:
CMD ["python3", "bin/run"]

You probably know this page -- if not, take a look, maybe you can find some alternative there that's going to work for your case.
HTH 🤞

In nginx add mapping for component if it is missing.
eg.
location ^~ /static {
proxy_pass http://<IP_ADDRESS>:8501/static/;
}
location ^~ /st-allowed-message-origins {
proxy_pass http://<IP_ADDRESS>:8501/st-allowed-message-origins;
}
location ^~ /component {
proxy_pass http://<IP_ADDRESS>:8501/component;
}
location ^~ /healthz {
proxy_pass http://<IP_ADDRESS>:8501/healthz;
}
location ^~ /vendor {
proxy_pass http://<IP_ADDRESS>:8501/vendor;
}

I have found a solution for my case -- described here. HTH.

I gave that a try and it seems to have fixed my issue as well.
I was deploying a Streamlit app with the streamlit_option_menu to the Digital Ocean App Platform with a custom Dockerfile build.
Implementing your hack seems to have done the trick of getting rid of that warning message on the initial startup.
Thanks again.

Found this somewhere. And it does the magic.
import mimetypes
mimetypes.add_type('application/javascript', '.js')
mimetypes.add_type('text/css', '.css')

Found this somewhere. And it does the magic.
Add this in your code.
import mimetypes
mimetypes.add_type('application/javascript', '.js')
mimetypes.add_type('text/css', '.css')

Excuse me, could you be a little more specific?

Add the code snippet to the beginning of the body of your code to resolve the issue. I can't remember the source. But it solved my problem when I had similar issue.

import mimetypes
mimetypes.add_type('application/javascript', '.js')
mimetypes.add_type('text/css', '.css')

It works. Dm it! Who knows why?


Start-up animation delay

What config to change
According to the 5.6.0 (latest update for the modpack), FO now has a previously-non-existent 0.5 seconds delay for the splash screen.

The startup animation splash is now set to 0.5 seconds (compared to vanilla 2 seconds, previous FO version 0 seconds)

The value can be adjusted in Remove Reloading Screen settings

It should be changed back to zero seconds.
Why should it be changed
I see no reason to keep an artificial delay of 0.5 seconds for the splash screen.
It doesn't matter the context: the less delay it takes to load, the better.
Additional details
No response

The change was done to find a balanced solution that would satisfy the 59% who voted for Fadeless keeping and 41% for removal at #737. So instead of removing the animation entirely, I made it 4x faster than vanilla and configurable.
I can still consider tweaking the value even faster or adjusting other settings of RRLS, but at this point I don't think setting it to 0 seconds is a good solution.

I encourage you to join our Discord and read the full conversation of the "Remove Fadeless" thread to get the full context on why this change was made and what other alternatives were proposed.

Decided that the current value of 0.5s is still a good balance of "improved speed" vs "being jarring". This matches the previously ran public survey of FO users' benefits.
Luckily, the value is also easy to change for those who prefer 0s, 2s or something else - Mods, search "Remove Reloading Screen", 🎛, adjust the slider, save.


Fix name to either visualizer or oscilloscope

Description
Fix name to either visualizer or oscilloscope. Right now, the doc references oscilloscope as visualizer or oscilloscope
Type of correction

[ ] Documentation structure change
[ ] Documentation update
[ ] Permission Issue(Any page you do not have access to)
[ ] Inaccurate content
[ ] Inappropriate content
[ ] Outdated/Deprecated content

Link

Add Link to the page where the problem exits here

Screenshots

for now, get the visualizer out of the docs.

Done.


[WIP] Support SegmentLevel MinMax for better Pruning and less driver memory usage for cache

Why is this PR needed?
What changes were proposed in this PR?
Does this PR introduce any user interface change?

No
Yes. (please explain the change and update document)

Is any new testcase added?

No
Yes

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/6/

Build Failed  with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/288/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/1992/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/1990/

Build Failed  with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/292/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/1996/

Build Success with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/295/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/1999/

Build Success with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/297/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2001/

Build Failed  with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/307/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2010/

Build Success with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/308/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2011/

Build Success with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/319/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2021/

Build Success with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/328/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2030/

Build Success with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/337/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2039/

Build Success with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/345/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2047/

Build Success with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/353/

Build Success with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/356/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2057/

retest this please

Build Success with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/359/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2061/

Build Success with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/365/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2067/

Build Success with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/374/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2076/

Build Success with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/384/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2085/

Build Success with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/388/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2089/

Build Success with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/389/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2090/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2095/

Build Success with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/394/

Build Success with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/395/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2096/

Build Success with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/396/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2097/

retest this please

Build Success with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/421/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2121/

Build Success with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/424/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2124/

Build Success with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/426/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2126/

retest this please

Build Success with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/431/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2131/

Build Success with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/436/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2136/

Build Success with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/444/

retest this please

Build Success with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/445/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2145/

Build Failed  with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/472/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2171/

Build Failed  with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/478/

Build Failed  with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/480/

retest this please

Build Success with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/482/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2182/

retest this please

Build Success with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/495/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2195/

retest this please

Build Success with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/552/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2253/

@akashrn5 please review

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2332/

Build Failed  with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/626/

Build Failed  with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/628/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2334/

Build Failed  with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/632/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2339/

Build Failed  with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/684/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2390/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2391/

Build Failed  with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/685/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2396/

Build Failed  with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/690/

Build Failed  with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/695/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2401/

Build Failed  with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/699/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2405/

@Indhumathi27 : make that carbon property deafult value to false
and In query flow, If the table is transactional table and segment minmax is not set. Throw runtime exception. So, that CI can catch all the missed scenarios.

Build Failed  with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/706/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2413/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2414/

Build Failed  with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/707/

Build Failed  with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/710/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2417/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2423/

Build Failed  with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/716/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2428/

Build Failed  with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/721/

retest this please

Build Failed  with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/743/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2451/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2453/

Build Failed  with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/745/

Build Failed  with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/747/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2455/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2462/

Build Failed  with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/754/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2464/

Build Failed  with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/756/

Build Success with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/759/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2467/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2479/

Build Success with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/771/

Build Success with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/804/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2509/

LGTM

@Indhumathi27 : make that carbon property deafult value to false
and In query flow, If the table is transactional table and segment minmax is not set. Throw runtime exception. So, that CI can catch all the missed scenarios.

is it verified ?

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2536/

Build Failed  with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/829/

retest this please

Build Success with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/830/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2537/

LGTM


fix(project): update secrets baseline

Signed-off-by: James Hart<EMAIL_ADDRESS>PR summary
To exclude package-lock.json
Fixes: https://github.com/IBM/container-registry-java-sdk/issues/3
PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the Angular Commit Message Guidelines.
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[ ] Build/CI related changes
[ ] Documentation content changes
[ ] Other (please describe)

What is the current behavior?

What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

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


initial travis support

For #5

@jrperritt Ping for review.
Here's an example of a failed run: https://travis-ci.org/gophercloud/utils/builds/302817871


[BUG]: Test Progression Bar Overflow

Description
When typing out longer text on a 4k screen the test progression bar goes off the screen

(optional) What browsers are you seeing the problem on?
Firefox
Code of Conduct

[X] I agree to follow this project's Code of Conduct

i will give this a shot

I want to propose a solution.
Beside progress bar, add a % base number to indicate to finished rate.
When screen size is smaller than maybe md size screen, than only show % base indicator.
% number will provide better feedback on smaller screen and maybe on mobile.


Embeddings not working that well

Similarity: 0.9279538989067078, Expected output: The user wants to code a Scrabble game using Go., Actual output: Updated Summary: The user wants to code a Scrabble application using Go.
Similarity: 0.9863240122795105, Expected output: The user wants to code a Scrabble game using Go., Actual output: The user wants a todo list application written in python.
Similarity: 0.9259848594665527, Expected output: The user wants to code a Scrabble game using Go., Actual output: Updated Summary: The user wants to code a Scrabble game using Go.
Similarity: 0.9259848594665527, Expected output: The user wants to code a Scrabble game using Go., Actual output: Updated Summary: The user wants to code a Scrabble game using Go.

Weird that the incorrect output has a higher cosine similarity score. Probably should try a different embedding model and/or a different way to estimate the distance between the two sentences.

mxbai-embed-large works a lot better.
The threshold seems to be around 0.9
Similarity: 0.974323551168669, Expected output: The user wants to code a Scrabble game using Go., Actual output: Updated Summary: The user wants to code a Scrabble application in Go.
1.0
Similarity: 0.9557684741757109, Expected output: The user wants to code a Scrabble game using Go., Actual output: Updated Summary: The user wants a Scrabble game coded in Go.
1.0
Similarity: 0.8998050841715695, Expected output: The user wants to code a Scrabble game using Go., Actual output: Updated Summary: The user prefers to code a Scrabble game using Go instead of a todo list application in Python.
1.0
Similarity: 0.8740119465489894, Expected output: The user wants to code a Scrabble game using Go., Actual output: Updated Summary: The user prefers to code a Scrabble application using Go instead of a todo list application in Python.


metadata editor flickers after activating the toolbar button 'Count the number of lines and other expensive functions'

System information:

Win10, DBeaver 21.2.2

Describe the problem you're observing:
Some widgets in metadata editor are flicker after clicking the toolbar button 'Count the number of lines and other expensive functions'
https://user-images.githubusercontent.com/43252954/136336151-75478b6e-010d-42a9-a656-7019e634ab6b.mp4

Fixed

Verified


Plugin: Make a terminal plugin for use on Linux with Terminado

This is a request I received at SciPy Latin America.
Having a terminal as part of an IDE is quite handy, for example to do git commits or start/stop OS services. Projects like Geany have one, and it was mentioned to me that that's one of the main reasons people prefer it over Spyder :-)
We should try to use an external project (e.g. qbash) to do this, probably as an external plugin.
Note: Issue #306 has some discussion about this too.

@thewhitetulip this is something you are still willing to go for?

Yes, I am, but I am not getting time out of work. I can squeeze some time in later this month though.

It would fit nicely as a plugin, as it's just a widget with a few connections to spyder internals.

Yes, @Nodd also another thing that makes it more suitable as a plugin is that the project this is going to be based on is a GPL2 licensed, so we can't really use it in spyder directly. I will ask for assistance when I start working on it, probably this month end.

This would be very easy to do now with the plugins interface.
I created a repo

@goanpeca, I renamed it to spyder.terminado because the idea is to embed Terminado (not QBash), as we've talked about :-)

Ahh ok :-)

It's easier for us because Terminado provides terminals for the Jupyter notebook.
So it's a project that's going to be well maintained in the future :-)

Agree :+1:

This is going to be developed by @andfoy in an external plugin.


Use Proxy

What does this PR do?
Specifies a proxy from which the sources should be loaded/proxied through.
Since many Geonodes do not have CORS support we need a proxy do display them all.
It now allows us to use almost any geonode config and test it out.
Screenshot

Related Issue
closes #11

not possible to use the github version for now. the i18n part needs the build


Update tomcat dependency to 9.0.29

Fixes JIRA #TOMEE-2745.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?


Train loss increasing during training

Wandb log:

Model from commit 37129af4df4360d64b8518d4da9fe45da8c71df9 with pmap script
Started increasing at around 19B tokens mark

Will run with smaller model to see if same behavior is occurring, possible sources of error:

Incorrect dropout RNGs being applied (doesn't seem likely, I checked this)
Webdataset + multiepoch resume could be problematic

Training 50M param model for 4 epochs on V2-8, should be done in 2 hrs

50M run for 4 epochs on pod completed, this training curve looks as expected. Need to investigate code for resuming runs next.

Could also be that this is the result of some issues with the data/model scale? Following OPT, will try and log l2 norm of final layer activations, they found this to be good indicator for loss divergences. I'm a bit skeptical this is what it would be, but working on this is still good practice

Made a small change to how data resuming is handled:

Splitting out rng keys while resuming in dataset. Before, rng keys weren't split out when resuming from a previous ckpt, meaning rng state was reused.

Confirmed that this works properly by training a 125M model and forcing a resume about 50% through training. Model trained properly from this ckpt and loss didnt diverge. Considering this issue closed for now

Turns out there was an odd duplication bug with the webtext data. Remade dataset from scratch, will see how new models perform


adding connectedmachine to index

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

If this PR is for a new extension or change to an existing extension, use the following to try out the changes in this PR:
docker run -it microsoft/azure-cli:latest
export EXT=<NAME>
pip install --upgrade --target ~/.azure/cliextensions/$EXT "git+https://github.com/Azure/azure-cli-extensions.git@connectedmachine-to-index#subdirectory=src/$EXT&egg=$EXT"


dynamic set rebalance speed with API

Assume our Database contain many GB or TB datas, when we start our tikv nodes one by one, the earlier started nodes will campaign election earlier than others and win the election then become region leader, result in most regions' leader located at few nodes. What we want is the leader transfer from these few nodes to others as soon as possible, so we need high rebalance speed. But when the DB is providing OLTP services, we should limit the rebalance speed.

See #188
I think in our algorithm, we can try to do balance first, but if the balance condition is not matched, we can try another way that is only leader transfer.
In your scenario,  this strategy can work well.
So how do you think of this? Any other suggestions?
@zhangjinpeng1987  @shenli  @siddontang

I guess @zhangjinpeng1987 wants to adjust the speed dynamically, e.g, now we do auto-balance every 10s, but we can reduce or increase this, like every 1s or every 30s dynamically.


Callbacks bound to lifecycle events don't fire when expected.

I'm submitting a ...
[x ] bug report
[ ] feature request
[ ] question about the decisions made in the repository

Do you want to request a feature or report a bug?
Bug

What is the current behavior?
Callbacks bound to lifecycle hooks don't fire when expected.

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem via https://plnkr.co or similar.
Drag an item from the 'Available List' to the 'Second Team'.  Move the cursor through the 'Second Team' dropzone, and release the cursor outside of the 'Second Team' dropzone.  The item should be added to the 'Second Team' list, however neither the onDragSuccess event nor the onDragEnd event is fired.  onDragSuccess only fires if you release the cursor while it is positioned inside of a dropzone, on top of the item you were dragging.  onDragEnd only fires if you drag and release an item still in it's current dropzone.

https://plnkr.co/edit/0PVipgYqx4cNOXJBienj?p=preview

What is the expected behavior?
The onDropSuccess event is fired whenever an item is successfully added to a list.  The onDragEnd event fires whenever an item stops being dragged anywhere.

What is the motivation / use case for changing the behavior?
I want to update an object whenever a change is made to the list.  Theoretically I should be able to use the onDropSuccess event, however I have found you can add an item to a list without triggering the onDropSuccess event.  While trying to use the onDragEnd event instead, I discovered that onDragEnd only fires if you drag an item but don't leave it's current list.

Please tell us about your environment:

Angular: 2.4.7
ng2-dnd: 2.2.2
Browser: Chrome Version 56.0.2924.76 (64-bit)

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, gitter, etc)
I have included a .gif demonstrating the issue, but man, what a terrible looking .gif.  If my explanation is unclear I will try recording something of better quality, hopefully that will help...

I'm experiencing the exact same issue.

I'm also having this issue.


The test case L2_EnableAndGetDynamicContrast is missing a call to the TvTerm function, which is required to properly terminate the session.

Problem/Opportunity
The TvTerm function is essential for cleaning up and terminating the TV session. Its absence in the test case can lead to resource leaks or inconsistent system states, especially in subsequent tests.
Steps to reproduce
No response
Expected Behavior
The test should invoke the TvTerm function at the end to ensure proper cleanup of the session.
Actual Behavior
The TvTerm function is not called, leaving the session improperly terminated, potentially causing resource leaks or test environment issues.
Notes (Optional)
No response

available in tag 2.1.3


Provide API for extension developers to hide and replace user typed text

If the user types "c" but the Intellicode completion is "Console", we'd like to be able to hide the user typed "c" and replace with "Console". We want a bit more flexibility (e.g. user types "console.wl" and we want to display "WriteLine" and then have the ability for the user to backspace and have that only delete "l" from the buffer).

Unfortunately, this is quite challenging.
Also, it could be very confusing. We already experimented with allowing ghost text to be inserted before the cursor (e.g. when the user types CW and the suggestion is Console.WriteLine, then onsole. and riteLine would be shown as ghost text).
However, because it caused a lot of jumping, we decided to only show ghost text after the cursor.
Can you provide screenshots/mocks how you would solve this UX problem of jumping text?


Hamburger Buttons should fire commands too in addition to page navigation

It would be great if we could fire commands to the current page in the shell from the hamburger buttons!

What kind of commands? And, more importantly, why can't you?

In a lot of our apps, we leverage a MainPageViewModel that uses delegate commands & parameters to filter groups in a hub.
I dug deeper into what we now get in T10 out of the box and saw this new fangled Event Binding with x:Bind. Having read up a little on the subject, I think it will really meet a lot of our needs. What I wasn't clear on was how to manage parameters?

There is already a ButtonType.Command that can be used to raise a Tapped event. Can't you x:bind that to a view-model method?

Great - thanks!


bootpy: Make it possible to add more than one network or block device

Currently not possible, afaik

It's possible, but it's not immediately apparent how to do this... some documentation or example (update dualnic example!) to do this is necessary.


Add some requested CLI cookbook entries

Addresses https://github.com/stellar/stellar-cli/issues/1438
Adds cookbook entries for:

A lifecycle of a contract (requested by the community)
Payment for XLM, classic asset, soroban assets

We cannot yet do the following because the commands are not available on the CLI:

Issuance for Stellar and Soroban assets  (specifically we need trustline functionality)
Transaction (should mostly be available once this PR is merged https://github.com/stellar/stellar-cli/pull/1406

Preview is available here:http://developers-pr840.previews.kube001.services.stellar-ops.com

Preview is available here:http://developers-pr840.previews.kube001.services.stellar-ops.com

LGTM, just added a couple of small comments

@BlaineHeffron Ideally this could be in a single markdown file called cookbooks.mdx, with different paths for each recipe.

@BlaineHeffron I was thinking that this Cookbook.mdx could be added to the CLI repo next to the  FULL_HELP_DOCS.md and we could copy it over the docs, similar to the CLI Manual is copied over: https://github.com/stellar/stellar-docs/pull/755

@janewang Makes sense, I'll work on getting that set up.

Preview is available here:http://developers-pr840.previews.kube001.services.stellar-ops.com

Preview is available here:http://developers-pr840.previews.kube001.services.stellar-ops.com

Preview is available here:http://developers-pr840.previews.kube001.services.stellar-ops.com

Something went wrong with PR preview build please check

@janewang I opened a draft PR for a Cookbook.md file. https://github.com/stellar/stellar-cli/pull/1541  I also added to the script an entry to copy over this Cookbook.md file to stellar-cli-cookbook.mdx alongside the stellar-cli.mdx file. I also added a link to it in the docs. I am wondering if you would like to copy all the existing stellar-cli guides that are currently here? https://developers.stellar.org/docs/build/guides/cli

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Hey @BlaineHeffron The other file that's being copied over is this file: https://github.com/stellar/stellar-cli/blob/main/FULL_HELP_DOCS.md. I think we only need these two files to be copied.

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

This PR can only be merged after https://github.com/stellar/stellar-cli/pull/1541 is merged. In that PR I moved the guide mdx files to the CLI repo in a new  /docs directory. In this PR I altered the script to copy over the mdx files from the /docs directory.

Something went wrong with PR preview build please check

Preview is available here:http://developers-pr840.previews.kube001.services.stellar-ops.com

Preview is available here:http://developers-pr840.previews.kube001.services.stellar-ops.com

Preview is available here:http://developers-pr840.previews.kube001.services.stellar-ops.com

You can run yarn check:mdx to fix the markdown linting error

The problem is that I needed to add the yarn stellar-cli:build command to the git actions so that markdown files that link to the CLI cookbook entries wouldn't error during build. This means that the CLI docs markdown file gets linted and apparently it is producing warnings. This is the auto generated docs file from the CLI. Either we would have to fix the markdown formatting errors from the generated document or we just add stub files for all of the CLI cookbook entries and don't actually run yarn stellar-cli:build in the git actions.

Something went wrong with PR preview build please check

Preview is available here:http://developers-pr840.previews.kube001.services.stellar-ops.com

@BlaineHeffron @janewang
Hi, friends!!
Sorry, i'm late to the party here, had a couple hectic weeks. Can I get a little more context about what brought this change about? I think I've picked up on reasonably on what is taking place in this change, but I'm a little unclear about the why. Is there a particular reason having these cookbook guides live alongside the CLI codebase is better off than having them live in the docs?
One of the reasons I'm asking is: now we're seeing the same broken link error Blaine mentioned in this comment if we're just using yarn start to preview locally while editing.
Thanks!

What @ElliotFriend said! I'm getting broken links when linking to the CLI guides and I'm not sure how to rectify the link formatting. We can't merge any other docs PRs until we fix these links.

There is a test that runs through the command to make sure that they are valid on the CLI side. That test could only reasonably live in the CLI repo. That's why Blaine created the file on the CLI side and copied over to the docs.
Can someone point me to the broken links on the docs side? I heard it mentioned but didn't see the links.

@janewang you can see some of the fixes Bri was needing in some of the changed files in pr #982

Thank you @briwylde08!


Find a good name for the internal builtin test repository

The name internal seems to be misleading. Find a good name for the builtin test repo that contains an artificial hello_world.

Closing this because there's no builtin test repository.


What happens to an existing session when I create a new session?

It's not clear to me what happens to my working session if I chose to create a new session from the File menu.

We should warn users that the old session is going to be gone.

@StasJ I'm working on this issue to add a pop up box to warn users.  How to use that?

The warning popup appears when we load a new session from the command line (ie >vaporgui /path/to/mySession.vs3 results in a warning that we're about to lose a session).
I'd like to suggest the following:

Avoid the warning in this situation since the previous session is in a blank state and there's nothing to lose.
Allow the user to perform a "Save session as" operation from the warning menu, so that they could either proceed, cancel, or proceed+save.
Change the word "lose" to "lost" in the popup.

hmm, the current warning message isn't intelligent enough to distinguish this case, so it pops up more often than necessary.
To eliminate these unnecessary pop-ups, we'll need to plant a flag in the ParamsMgr class to keep track of any changes, and it's a fair amount of work.
Let's discuss if we want to pursue that direction.

It seems VAPOR is intelligent enough to pop up warning messages only when settings are changed. I'm going to close it unless new misbehaviors are found.


fix: use proper EventFormat label in plot_who_can_still_win_wdc.py

The exemples gallery currently includes an exemple to determine who can still win the wdc.
The EventFormat value that is used is sprint.
However, sprint_shoutout should be used, as sprint will resolve 0 events.

Thanks, @papshed for opening this PR and @borolepratik for reviewing. There actually already was #414 and I just didn't get around to merging it, because I wanted to check some related stuff. I finally merged it right now, so this is fixed.
Sorry for the duplicate work on your side. But you are welcome to contribute to FastF1 at any time.

No worries at all, thanks for merging!
Cheers!


Add auto_align feature for automatic column alignments by dtype

Automatic alignment of columns should occur during data ingest. This is made optional by the auto_align argument in the GT class (set to True by default).

This is now working fairly reasonably.


Request - It will be great to add configuration for file.svg custom color

I'm a big fan of the theme. Color customization of the plain folders was a selling point for me.
I think the theme will be even better if there is a configuration like "icons.file.color" related to the plain file's icon file.svg. So the user will have full control over the plain folder and file colors and will be able to match them.
Thank you very much for the work you put into the theme.

The newest version (1.12.0) now has the same setting as the standard folder for the standard file icon!


TRTorch on Jetson Nano

Is there a build process outlined for getting this library built on a Jetson Nano or another ARM64 platform (TX2, Xavier, etc.)? The Bazel WORKSPACE seems rather dependent on X86 binaries and since I am new to Bazel I am having a hard time figuring out how to get it to build on Jetson Nano that already has TensorRT, CUDA, cuDNN, and libtorch/pytorch installed. Thanks

Try using the local dependencies path for bringing sources for cuDNN and TensorRT. https://nvidia.github.io/TRTorch/tutorials/installation.html#building-using-locally-installed-cudnn-tensorrt.
If you have a libtorch build for Jetson, you can replace the http_archive with a similar local_repository target. If you are building for Python or are using the PyTorch distribution of libtorch (the one that comes with the wheel file) replace the pre_cxx11_abi libtorch target with something like this.
new_local_repository(
    name = "libtorch_pre_cxx11_abi",
    path = ".../site-packages/torch",
    build_file = "@//third_party/libtorch:BUILD",
)

otherwise replace the CXX11 ABI target with the path to your library like this:
new_local_repository(
    name = "libtorch",
    path = "<PATH TO LIBTORCH ROOT>",
    build_file = "@//third_party/libtorch:BUILD",
)

We are working on a better workflow, including hopefully cross compilation support but it is lower priority than other tasks right now.

Ah, finally got it working. I made those changes and then had to change some of the BUILD files for the third party libraries so it would find them correctly. I have all those edits made on a branch in my fork just in case anyone else would like to use them until cross compilation is supported in a cleaner way. https://github.com/BytesRobotics/TRTorch/tree/jetson-nano

Ah, finally got it working. I made those changes and then had to change some of the BUILD files for the third party libraries so it would find them correctly. I have all those edits made on a branch in my fork just in case anyone else would like to use them until cross compilation is supported in a cleaner way. https://github.com/BytesRobotics/TRTorch/tree/jetson-nano

Hi, Did you had to switch to tensorrt 8.0 on jetson in order to install TRTorch ?

Your TensorRT version should be determined by your Jetpack version. By adding the jetpack version as a flag you can use the previous version (4.5) if you are still using it. https://nvidia.github.io/TRTorch/tutorials/installation.html#building-natively-on-aarch64-jetson


Enable running Features/Scenarios from Gherkin file

This run button should work

Example impl
https://github.com/JetBrains/intellij-plugins/blob/master/cucumber-java/src/org/jetbrains/plugins/cucumber/java/run/CucumberJavaRunConfiguration.java

Hi. I was trying to configure this in my IDE, but still no option.
I don't know how to configure godog for .feature files.
I see that there is new template for Go test but that's all.

There is demo repo, where you can find example of feature file and step definitions for it.
https://github.com/nikolaymatrosov/bdd-godog

Here is the screenshot showing how to create test run configuration directly from .feature file.


Add support for wowhead "transmog sets"

Well done on this.
Currently links like http://www.wowhead.com/transmog-set=450 don't work. Would be could if they did. :D
Example #2: http://www.wowhead.com/transmog-set=987

Added. Download the binary here: https://github.com/kdar/morphgen/releases

Thanks for this, worked perfectly


Invalid byte sequence

When I try to download a file that contains an "ñ" character, I get an invalid byte sequence error:
view $ inproc "curl" ["https://www2.census.gov/geo/relfiles/cdsld13/35/co_cd_35.txt"] empty
Is there a way to encourage this to somehow start using unicode?

@skatenerd: Yes.  You can use the following command to specify that your program should assume a UTF-8 encoding globally:
GHC.IO.Encoding.setLocaleEncoding System.IO.utf8

@skatenerd: You're welcome! :slightly_smiling_face: