Selenium: Restore getDockerImageForCapabilities method to public scope

Per https://github.com/testcontainers/testcontainers-java/commit/1e597ccd4fa1a7113d3077599e0b13237c1fef51#r43961779
Restoring this method to public scope.
I have marked with @Deprecated. I think this method will end up going away in the future, but we need to spend a bit of time thinking about it. BrowserWebDriverContainer is a slightly quirky class because of the automatic image inference is does, and we need to think about how things fit together.

Yes indeed, I would make getImage() return the image used for all containers. When BrowserWebDriverContainer is used with a a set image it would return it and when it's computed it would return the computed image. TBH I was surprised when I saw that getImage() was not returning what I was expecting.
Thanks @rnorth !


Does edge weight data type need to be double?

Hi!
Is there a reason for the data type of edge weight being a double? I found multiple libraries doing that, and I am wondering if for whatever reason that is a must, or a 64bit integers (or even unlimited integers with some bignum arithmetic library) would also do the job.
Great work, by the way!

Hi.
The reason is that double is more general than integer. (I'm not sure I
understand your question correctly)
If you input integers to the algorithm, it's gonna work just fine.
Maybe your question is why doesn't the algorithm use just integer numbers
all around. The reason is that even if the inputs were integers, by the
nature of the primal-dual algorithm, real valued quantities would arise.
You could of course replace the doubles with large enough integers, but
that is not ideal, unless you really have to for some reason.
Regards.
Dilson
On Sun, Sep 22, 2024 at 7:31 AM Karel Jílek @.***>
wrote:

Hi!
Is there a reason for the data type of edge weight being a double? I found
multiple libraries doing that, and I am wondering if for whatever reason
that is a must, or a 64bit integers (or even unlimited integers with some
bignum arithmetic library) would also do the job.
Great work, by the way!
—
Reply to this email directly, view it on GitHub
https://github.com/dilsonpereira/Minimum-Cost-Perfect-Matching/issues/6,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACDRUD77M4XA7WZDTENQU2DZX2MAJAVCNFSM6AAAAABOUN42LGVHI2DSMVQWIX3LMV43ASLTON2WKOZSGU2DAOJZGYZDMMA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hi Dilson, thank you for quick answer! I would like to use the algorithm you wrote, but my issue is that my edge weights are too large for doubles, therefore I wonder if there would be any way to have the weight data type as a template parameter for example. I did not study the details of the algorithm so I am not really following the issue with the primal-dual algorithm you mention, so I will just trust you there if you say that it has to be doubles no matter what :).
Best,
Karel

Hi Dilson,
by trial and error, I used arbitraty precision floats and the algorithm seems to work fine. If it is easy enough for you to see, could you confirm that it actually does not break the soundness? And would the same be true for unlimited integers?
Best,
Karel

If the floats you're using offer a reasonable amount of decimal places,
then you're likely fine.
As for unlimited integers, if you multiply your input by some large power
of 10 and then truncate them, then it should work (even if the input is
already composed of large numbers, you still have to multiply them by a
large power of 10).
I cannot offer you a guarantee, though.
On Tue, Oct 1, 2024 at 1:21 PM Karel Jílek @.***> wrote:

Hi Dilson,
by trial and error, I used arbitraty precision floats and the algorithm
seems to work fine. If it is easy enough for you to see, could you confirm
that it actually does not break the soundness? And would the same be true
for unlimited integers?
Best,
Karel
—
Reply to this email directly, view it on GitHub
https://github.com/dilsonpereira/Minimum-Cost-Perfect-Matching/issues/6#issuecomment-2386449984,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACDRUD4LW72PQEHNF27FYULZZLDZHAVCNFSM6AAAAABOUN42LGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBWGQ2DSOJYGQ
.
You are receiving this because you commented.Message ID:
@.***
com>

What should I see behind "reasonable" and "large power of 10"? Is it more that double of the bits would be enough, or rather even more? If, say, my highest edge cost would be 290, would it be enough to multiply the inputs by another 290 (roughly 10**30), or rather even more? I understand you cannot offer me a guarantee, but your qualified guess is still way better than my qualified guess.

7 or 8 decimal places should be enough. So if you multiply your input by
10**8 you're likely fine (if you can correctly handle these large numbers)
On Wed, Oct 2, 2024 at 11:57 AM Karel Jílek @.***>
wrote:

What should I see behind "reasonable" and "large power of 10"? Is it more
that double of the bits would be enough, or rather even more? If, say, my
highest edge cost would be 290, would it be enough to multiply the
inputs by another 290 (roughly 10**30), or rather even more? I
understand you cannot offer me a guarantee, but your qualified guess is
still way better than my qualified guess.
—
Reply to this email directly, view it on GitHub
https://github.com/dilsonpereira/Minimum-Cost-Perfect-Matching/issues/6#issuecomment-2388879861,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACDRUD5NYHYPDZT65VZMCNLZZQCXLAVCNFSM6AAAAABOUN42LGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBYHA3TSOBWGE
.
You are receiving this because you commented.Message ID:
@.***
com>

I will try with that, thank you!

Thank you for the support, it actually seems to work stellar! Just if you care what people use your thingy for :) https://github.com/karlosss/swisspair

That's pretty cool. Good job.
On Thu, Oct 3, 2024 at 1:17 PM Karel Jílek @.***> wrote:

Thank you for the support, it actually seems to work stellar! Just if you
care what people use your thingy for :)
https://github.com/karlosss/swisspair
—
Reply to this email directly, view it on GitHub
https://github.com/dilsonpereira/Minimum-Cost-Perfect-Matching/issues/6#issuecomment-2391818872,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACDRUDY4HZF4YVTX3LPJQM3ZZVUZPAVCNFSM6AAAAABOUN42LGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOJRHAYTQOBXGI
.
You are receiving this because you commented.Message ID:
@.***
com>


Converts my https urls to http before submitting to the Prerender Service

Hi,
I am using this for one of my projects. Below is my java configuration:
@Configuration
public class SEOFilterConfig {

  @Value("${prerender.service.url}")
  private String prerenderServiceUrl;

  @Bean
  public FilterRegistrationBean someFilterRegistration() {
    FilterRegistrationBean registration = new FilterRegistrationBean();
    registration.setFilter(seoFilter());
    registration.setName("prerender");
    registration.addUrlPatterns("/*");
    registration.addInitParameter("prerenderServiceUrl", prerenderServiceUrl);
    registration.addInitParameter("prerenderToken", "mytoken");
    return registration;
  }

  @Bean(name = "seoFilter")
  public Filter seoFilter() {
    return new PreRenderSEOFilter();
  }
}

My pom is as follows:
<dependency>
	<groupId>com.github.greengerong</groupId>
	<artifactId>prerender-java</artifactId>
	<version>1.6.4</version>
</dependency>

Everything is working but there is only one problem, all my urls are getting converted to non-https before being submitted to prerender service as I can see in https://prerender.io/crawl-stats here.
Please suggest a way out of this. Thanks.

You might want to use the forwardedURLHeader option and use the header that you pick to set the original https URL to be used when sent to Prerender:
https://github.com/greengerong/prerender-java#forwardedurlheader
Otherwise, you could add an option to the java middleware to force the protocol to https if you wanted to open a pull request! :)

Sure! In the getRequestUrl function:
https://github.com/greengerong/prerender-java/blob/master/src/main/java/com/github/greengerong/PrerenderSeoService.java#L181
You would want it to look something like:
    private String getRequestURL(HttpServletRequest request) {
        if (prerenderConfig.getForwardedURLHeader() != null) {
            String url = request.getHeader(prerenderConfig.getForwardedURLHeader());
            if (url != null) {
                return url;
            }
        }

        if (prerenderConfig.getProtocol() != null) {
            String url = request.getRequestURL().toString();
            return url.replace(request.getScheme(), prerenderConfig.getProtocol());
        }

        return request.getRequestURL().toString();
    }

You'd need to add the config methods in the PrerenderConfig.java file. I didn't test the above code so you'll want to test and modify that if needed. Also, if you could add some documentation on the readme.md, that would be awesome too! Let me know if you have any questions.
Thanks a ton!

Thanks for the pointer, I have created a PR. Let me know in case something is missing. Thanks!

Thanks so much!

merged


Add contributing page

We should also list easy things that others can do, for example:

Better documentation,
Adding tests
Adding client libraries for languages
Random other easy tasks

For example, switching parent panics over to the Result<_, GraphError::MissingParent(node)>

We should also have a list of contributors.


Add support for plain/text response type

Fixes #376

I need to take a closer look later but I believe this is supposed to handle the case of non-application/json.

The problem is when there is no schema in the response type (like the example below) than the responseSchema is undefined on this line and than we get the error from issue #376.
paths:
  "/test":
    get:
      summary: Get string value
      responses:
        "200":
          description: Returns value as a string
          content:
            text/plain:
              example: example

I see! In that case, could we just reorganize the code so that we still use this but it checks if a responseSchema exists first?

Yes, it should fix the issue.

@JKostov I've addressed this with #391. Thank you for all your help!


feat(app): show session resource usage

closes #727
NOTE to reviewers: 90% of the changes here are the result of moving files around - i.e. instead of having one large file that contains all the schemas I broke all of these up. I will annotate the PR with comments to indicate what is exactly new code. Anything that is not annotated as new code in the diff is actually old code that just got moved.
~TODO: Fix integration tests when there is agreement on the units and final format of responses with the UI.~

@lorenzo-cavazzi this is ready now but needs a change on the ui. Because the resources response from the api doesn't just send the requests but also requests and usage.
As we discussed the units are the same as before.

The changes in the response are as follows:
OLD:
    annotations = fields.Nested(UserPodAnnotations())
    name = fields.Str()
    state = fields.Dict()
    started = fields.DateTime(format="iso", allow_none=True)
    status = fields.Nested(ServerStatus())
    url = fields.Str()
    resources = {"cpu":"1", "memory":"1G", }  <<<< Changes are only here see below
    image = fields.Str()

NEW:
    annotations = fields.Nested(UserPodAnnotations())
    name = fields.Str()
    state = fields.Dict()
    started = fields.DateTime(format="iso", allow_none=True)
    status = fields.Nested(ServerStatus())
    url = fields.Str()
    resources = { requests: {cpu:"1", memory:"1G"}, usage: {cpu:"0.25", memory:"0.50G"} }  <<<
    image = fields.Str()

@rokroskar yes we can. I have to resolve conflicts and bring it up to date.

TODO: Ensure the code in the schemas folder is up to date with the current state of the schemas.


IE bug

when we use this library in IE, it asks for user permission I know we can disable it, but by-default its enabled
if the user deny access to clipboard, the API show trigger error, instead it trigger success even nothing is copied in clipboard...

Thanks @aliarshad9691, this problem is already described in the Known Issues wiki


Update to latest syntax

Thank you for the pull request. I've just pushed https://github.com/diesel-rs/diesel/commit/4267ca42437b4ef76e4ace49a9a1ecd080243bee which was already done when you opened this (but was blocked waiting on a release of dotenv).


Support Q CLI and Management Console prompts

Is your feature request related to a problem? Please describe.
The q translate command lets you write a natural language instruction such as “copy all files in my current directory to Amazon S3”. Amazon Q will then translate it to an instantly executable shell code snippet. The q translate command is useful in those common situations where the correct bash syntax is easy to forget. Examples include reversing a git commit, finding strings inside files with grep, or compressing files with tar.
Describe the solution you'd like
We need to add a third dimension (in addition to SDLC stage and category) that captures the surface/channel/client. Some  Prompts in the management console cannot work in the IDE (e.g. the operational prompt above).
Prompt categories are bound to the channel. Let's break this down:

IDE related categories: DEV Agent, Chat, Inline Chat
CLI related categories: Translate, chat
Management console related categories: Chat

This is how it looks like on my DEV environment.


Insert NULL into always encrypted column is not working

Environment

Python: 3.9.16
pyodbc: 4.0.39
OS: Win 11
DB: SQL Server 16,  Always Encrypted
driver: ODBC Driver 18 for SQL Server

Issue
I am using pyodbc to write into an always encrypted database.
If I want to write a NULL value into an encrypted column it is not working:
conn = pyodbc.connect("DSN=SQLXXXX;"
                      "database=DBNAME;"
                      "Column Encryption Setting=Enabled;"
                      )
cursor = conn.cursor()

query = """INSERT INTO test_table (designation, label) VALUES (?, ?)"""

cursor.execute(query, ('Test', None))

Both columns are encrypted.
The error message I get for the column I am trying to insert NULL/None is:

pyodbc.DataError: ('22018', "[22018] [Microsoft][ODBC Driver 18 for SQL Server][SQL Server]Operand type clash: varchar(1) encrypted with (encryption_type = 'DETERMINISTIC', encryption_algorithm_name = 'AEAD_AES_256_CBC_HMAC_SHA_256', column_encryption_key_name = 'XX', column_encryption_key_database_name = 'DBNAME') collation_name = 'Latin1_General_BIN2' is incompatible with nvarchar(50) encrypted with (encryption_type = 'DETERMINISTIC', encryption_algorithm_name = 'AEAD_AES_256_CBC_HMAC_SHA_256', column_encryption_key_name = 'XX', column_encryption_key_database_name = 'DBNAME') (206) (SQLExecDirectW); [22018] [Microsoft][ODBC Driver 18 for SQL Server][SQL Server]Statement(s) could not be prepared. (8180)")

It is working fine if I execute a similar parameterized statement in the SQL Management Studio and insert a NULL.
Is this a bug or am I doing something wrong? It seems like it is encrypting the None to a varchar(1) string. How can i prevent this?

Try using
cursor.setinputsizes([None, (pyodbc.SQL_WVARCHAR, 50, 0)])

before you call .execute()

Thank you very much. This solved my problem.
However, I observed another odd behaviour. This works for the given example query. But if my query looks like this, I get the same error message again:
query = """IF NOT EXISTS
(SELECT designation FROM test_table WHERE designation = ?)
BEGIN
INSERT INTO test_table (designation, label) VALUES (?, ?)
END"""

Do you have any idea why that might be?

You need to set the type for the appropriate parameter(s). See the documentation for setinputsizes.

In that case you would need to use an extra None to modify the third parameter instead of the second
cursor.setinputsizes([None, None, (pyodbc.SQL_WVARCHAR, 50, 0)])


Provide a UnderlineVisible and/or UnderlineActiveColor bindable property

Is your feature request related to a problem? Please describe.
As of now, one can control the color of the MaterialTextField's underline using the TintColorProperty. Any change made to this prop also affects the floating placeholder. But, sometimes, you may wish to not have a underline at all, eg. when you place a MaterialTextField inside a TextCell. I attached an example at the Additional Context. It's from an iOS app, but the Android one is equal. You can see there's a really opaque line under the textfield, but it belongs to the cell itself.
Describe the solution you'd like
Having an option to control the inactive/active color would be nice, as one could simply set it as transparent.
Describe alternatives you've considered
An alternative could be an IsUnderlineVisible property, which would control whether the line is rendered or not.
Additional context
<img src="https://is5-ssl.mzstatic.com/image/thumb/Purple128/v4/31/80/a9/3180a963-a930-3b84-57ec-49b58185c91c/mzl.bafeurhs.png/230x0w.jpg" />

PR for disable underline: https://github.com/contrix09/XF-Material-Library/pull/93

You can disable the underline from showing/animating using the new property ShouldAnimateUnderline.


[REGRESSION]: VSTest: Change from TestCaseFilter:FullyQualifiedName to TestCaseFilter:ClassName&Name breaks Rerun Failed Tests option

New issue checklist

[x] I searched for existing GitHub issues
[x] I read pipeline troubleshooting guide
[x] I checked how to collect logs

Task name
VSTest
Breaking task version
3.246.1
Last working task version
3.231.0
Regression Description
Getting

System.Exception: No tests were discovered while attempting to rerun failed tests. This is likely to happen if these tests are xunit or nunit datadriven/parameterized tests or tests with custom display names with spaces or special characters which are not supported with rerun.

on a pipeline where test rerun was previously working.
Tests are MSTest, not datadriven or parameterized, no custom display names.
Classic Pipeline, Windows 7 agent.
Seems to be a change in the TestCaseFilter arugment:
/TestCaseFilter:"FullyQualifiedName=TestCxEpjSerializer.CxEpjSerializerTests.GetEvents_MeetsPerformanceExpectations" - works
/TestCaseFilter:"(ClassName=TestSECSConnect.CxSMSTests&Name=SECSI_NormalMessaging_Works)" - does not work
Environment type (Please select at least one enviroment where you face this issue)

[x] Self-Hosted
[ ] Microsoft Hosted
[ ] VMSS Pool
[ ] Container

Azure DevOps Server type
dev.azure.com (formerly visualstudio.com)
Azure DevOps Server Version (if applicable)
No response
Operation system
Windows 7
Relevant log output
2024-10-11T20:32:08.3832790Z **************** Rerunning failed tests for Test run 1095563 *********************
2024-10-11T20:32:08.3832790Z Max attempts: 3; Current attempt: 1; Failed test cases threshold: 30; Failed test cases max limit: 0
2024-10-11T20:32:08.3832790Z [command]C:\Program Files (x86)\Microsoft Visual Studio\2019\Enterprise\Common7\IDE\CommonExtensions\Microsoft\TestWindow\vstest.console.exe "@C:\Temp\buildagent_work_temp\1k2kqbofe5a.tmp"
2024-10-11T20:32:08.4929018Z Microsoft (R) Test Execution Command Line Tool Version 16.11.0
2024-10-11T20:32:08.4929018Z Copyright (c) Microsoft Corporation.  All rights reserved.
2024-10-11T20:32:08.4929018Z vstest.console.exe /TestCaseFilter:"(ClassName=TestSECSConnect.CxSMSTests&Name=SECSI_NormalMessaging_Works)"
2024-10-11T20:32:08.4929018Z "C:\Temp\buildagent_work\4\s\Source\Shared\TestSECSConnect.dll"
2024-10-11T20:32:08.4929018Z /Settings:"C:\Temp\buildagent_work\4\s\Source\dcbmssrluio.tmp.runsettings"
2024-10-11T20:32:08.4929018Z /Logger:"trx"
2024-10-11T20:32:08.4929018Z /TestAdapterPath:"C:\Temp\buildagent_work\4\s"
2024-10-11T20:32:08.4929018Z /Diag:"C:\Temp\buildagent_work_temp\Attempt-3_drsndv.diag"
2024-10-11T20:32:08.4929018Z /logger:trx /platform:x64
2024-10-11T20:32:08.8687514Z Starting test execution, please wait...
2024-10-11T20:32:08.8687514Z Logging Vstest Diagnostics in file: C:\Temp\buildagent_work_temp\Attempt-3_drsndv.diag
2024-10-11T20:32:09.0096950Z A total of 1 test files matched the specified pattern.
2024-10-11T20:32:09.9806398Z The test execution was delegated to legacy TestPlatform runner. You are seeing this message most likely because you are running MSTest v1 tests with a testsettings file, Generic tests, Ordered tests or CodedUI tests. All of those test types, and the legacy TestPlatform runner are legacy. It is recommended that you migrate to MSTest v2.
2024-10-11T20:32:10.2312062Z TestSettings are legacy. It is recommended that you use RunSettings to configure test runs. To learn more, see http://aka.ms/runsettings
2024-10-11T20:32:10.5600746Z No test matches the given testcase filter (ClassName=TestSECSConnect.CxSMSTests&Name=SECSI_NormalMessaging_Works) in C:\Temp\buildagent_work\4\s\Source\Shared\TestSECSConnect.dll
2024-10-11T20:32:10.6853578Z Results File: C:\Temp\buildagent_work\4\s\TestResults\QEBuildService_QEBACOMMON_2024-10-11_14_32_10.trx
2024-10-11T20:32:12.9874366Z ##[debug]Exited vstest.console.exe with code 0.
2024-10-11T20:32:12.9874366Z ##[debug]PERF: ExecuteVsTestPhase.InvokeVSTest: took 4577.1487 ms
2024-10-11T20:32:12.9874366Z Vstest.console.exe exited with code 0.
2024-10-11T20:32:12.9874366Z **************** Completed test execution *********************
2024-10-11T20:32:12.9874366Z ##[debug]Current Phase: MS.VS.TestService.VstestConsoleAdapter.ExecuteVsTestPhase Phase Result: MS.VS.TestService.VstestConsoleAdapter.ExecutionVsTestPhaseResults
2024-10-11T20:32:12.9874366Z ##[debug]PERF WARNING: Running the phase MS.VS.TestService.VstestConsoleAdapter.ExecuteVsTestPhase: took 4577.2488 ms
2024-10-11T20:32:12.9874366Z ##[debug]Current phase: MS.VS.TestService.VstestConsoleAdapter.PublishTestResultPhase
2024-10-11T20:32:12.9874366Z ##[debug]Starting Publish Test Results: TIA: False
2024-10-11T20:32:12.9874366Z ##[debug]Starting PublishTestResult from Result Files
2024-10-11T20:32:12.9874366Z Test results files: C:\Temp\buildagent_work\4\s\TestResults\QEBuildService_QEBACOMMON_2024-10-11_14_32_10.trx
2024-10-11T20:32:12.9874366Z ##[debug]Updating Test results: 1095563
2024-10-11T20:32:12.9874366Z ##[debug]Entering ParseTestResultFiles
2024-10-11T20:32:12.9874366Z ##[debug]runContext.ReleaseURI is null
2024-10-11T20:32:12.9874366Z ##[debug]runContext.ReleaseEnvironmentUri is null
2024-10-11T20:32:12.9874366Z ##[debug]Reading test results from file 'C:\Temp\buildagent_work\4\s\TestResults\QEBuildService_QEBACOMMON_2024-10-11_14_32_10.trx'.
2024-10-11T20:32:12.9874366Z ##[debug]Setting run start and finish times.
2024-10-11T20:32:12.9874366Z ##[debug]Attachment location: C:\Temp\buildagent_work\4\s\TestResults\QEBuildService_QEBACOMMON_2024-10-11_14_32_10\In.
2024-10-11T20:32:12.9874366Z No Result Found to Publish 'C:\Temp\buildagent_work\4\s\TestResults\QEBuildService_QEBACOMMON_2024-10-11_14_32_10.trx'.
2024-10-11T20:32:12.9874366Z ##[debug]Total test results: 0.
2024-10-11T20:32:12.9874366Z ##[debug]Leaving ParseTestResultFiles
2024-10-11T20:32:12.9874366Z Updating test results: 0
2024-10-11T20:32:12.9874366Z ##[debug]Entering UpdateTestResultsAsync
2024-10-11T20:32:13.0500782Z Publishing test results to test run '1095563'.
2024-10-11T20:32:13.6139822Z ##[debug]PERF: GetTestCaseResults: took 544.585 ms
2024-10-11T20:32:13.6139822Z ##[debug]PERF WARNING: GetTestCaseResults: took 544.585 ms
2024-10-11T20:32:13.6139822Z TestResults To Publish 0, Test run id:1095563
2024-10-11T20:32:13.6139822Z ##[debug]Leaving UpdateTestResultsAsync
2024-10-11T20:32:13.6139822Z Updated test results: 0
2024-10-11T20:32:13.6139822Z Publishing Attachments: 1
2024-10-11T20:32:13.6139822Z ##[debug]Entering CreateTestRunAttachmentsAsync
2024-10-11T20:32:13.6766430Z ##[debug]Leaving CreateTestRunAttachmentsAsync
2024-10-11T20:32:13.6766430Z ##[debug]Uploading test run attachements individually
2024-10-11T20:32:13.6766430Z ##[debug]Entering UploadTestRunLogAsync
2024-10-11T20:32:13.6766430Z ##[debug]Blob name: GeneralAttachment/QEBuildService_QEBACOMMON_2024-10-11_14_32_10.trx .
2024-10-11T20:32:13.8646254Z ##[debug]File uploaded successsfully on LogStore QEBuildService_QEBACOMMON_2024-10-11_14_32_10.trx
2024-10-11T20:32:13.8646254Z ##[debug]Completed PublishTestResult
2024-10-11T20:32:13.8646254Z ##[debug]PERF: PublishTestResultPhase.Publish: took 890.0333 ms
2024-10-11T20:32:13.8646254Z ##[debug]Completed Publish Test Results.
2024-10-11T20:32:13.8646254Z ##[debug]Current Phase: MS.VS.TestService.VstestConsoleAdapter.PublishTestResultPhase Phase Result: MS.VS.TestService.VstestConsoleAdapter.PublishTestResultPhaseResults
2024-10-11T20:32:13.8646254Z ##[error]Error occurred while publishing test results : System.Exception: No tests were discovered while attempting to rerun failed tests. This is likely to happen if these tests are xunit or nunit datadriven/parameterized tests or tests with custom display names with spaces or special characters which are not supported with rerun. Please reach out to us at https://github.com/microsoft/azure-pipelines-tasks/issues if you are hitting this for tests other than the ones mentioned before.
2024-10-11T20:32:13.8646254Z ##[debug]Processed: ##vso[task.logissue type=error;]Error occurred while publishing test results : System.Exception: No tests were discovered while attempting to rerun failed tests. This is likely to happen if these tests are xunit or nunit datadriven/parameterized tests or tests with custom display names with spaces or special characters which are not supported with rerun. Please reach out to us at https://github.com/microsoft/azure-pipelines-tasks/issues if you are hitting this for tests other than the ones mentioned before.
Full task logs with system.debug enabled
unsuccess_log.txt
success_log.txt
Repro steps
No response

A few things we've tried that have not resolved the issue:

Removing Legacy settings/.testrunconfig so that MSTest v2 can be used
Using VS2022 Test Platform

I have been able to work around the issue by changing the "Batch tests" option under "Advanced execution options" to "Based on test assemblies" - in this case, it is able to rerun failed tests.


When does it start over?

Hi there! thanks for this. I have a question. Using a seed, they will eventually repeat. Is there any way to know when this will happen ?

Phonetic's seeding system isn't based on an actual language-provided RNG, but rather MD5. It was done in this fashion to ensure that the same algorithm could be re-implemented reliably in other languages, as well as produce the same results no matter what JS engine and version is being run.
The seed is used to generate the first MD5 binary blob, which is digested into an unsigned 32-bit int, and this is used to calculate the first syllable. This process is repeated every time a new syllable is added. If a seed isn't provided, the output of Math.random() is used for the seed -- this is safe because implementations on other JS engines or other languages don't need to choose a random seed in the same way, as long as it's a sufficiently random number.
The MD5 is re-generated every time Phonetic is called, so the number of syllables required in a single name to produce any discernible pattern given any random seed would approach infinity. If your concern is the randomness of the generated seed in the event that a static seed isn't provided, that relies entirely on the Math.random() implementation in V8. Its randomness is not considered cryptographically secure, however the distribution and range of its generated values has proven more than sufficient for most non-crypto applications.
Sorry for the long-winded answer -- let me know if I misunderstood your concern!


Circular references with EmbeddedDocumentField

Hi,
I had a search but couldn't find this particular issue. I have a schema which contains an embedded document which has further embedded documents in it, which reference the same embedded document type. This causes an infinite recursion in _unique_with_indexes on TopLevelDocumentMetaClass.
Here's a simple version of the schema which causes the issue:
class RecursiveObject(mongoengine.EmbeddedDocument):
    obj = mongoengine.EmbeddedDocumentField('RecursiveObject')

class RecursiveDocument(mongoengine.Document):
    recursiveobj = mongoengine.EmbeddedDocumentField(RecursiveObject)

I did a quick fix for this which doesn't recurse into _unique_with_indexes if the field is an EmbeddedDocumentField with the same document type as the current field, but not sure if that's ideal for general use:
(at the end of _unique_with_indexes):
if field.__class__.__name__ == "EmbeddedDocumentField" and field.document_type != new_class:

Cheers, Jon

Jon - many thanks!  This has been fixed in dev


fit to page not working for me

wetty is always an 80x24 terminal in the upper left corner of the browser window in both google-chrome and firefox. the fit to page box is checked in settings. where have i gone wrong?

I am also facing the same issue. Developers please help on the same.

same problem for me

After downgrade to version 2.5. it works. the terminal fit to page on chrome

tried wetty-2.5 and wetty-2.6 and nodejs 18 and 20, but it doesn't work on ubuntu 23.10

Yep, same here.
NodeJS 18 and latest version of wetty. Chek "fit to page" bix or changing number of columns and row change nothing.

Works with wetty 2.5.0 and node-js 18.17.0
Otherwise I had same problem.
OS: Arch linux arm 64 bit

Same here :o

Still valid issue.


Toggle button (bars)  not shown if sidebar is empty

The Page component is slightly modified in a such way that if one decides to make a page without a sidebar (aka Horizontal navigation) then the bars button will not be displayed.

Thanks for the PR!

Released as 0.6.1


Make Svcat print help text when a command is entered incorrectly

Currently it just tells me the one specific thing I did wrong:
$ svcat provision
Error: an instance name is required

This leads to getting it wrong several times in a row to figure out what the actual syntax is:
> svcat provision foobar
Error: provision request failed (ServiceInstance.servicecatalog.k8s.io "foobar" is invalid: [spec.clusterServiceClassExternalName: Required value: exactly one of clusterServiceClassExternalName, clusterServiceClassExternalID, or clusterServiceClassName required, spec.clusterServiceClassExternalID: Required value: exactly one of clusterServiceClassExternalName, clusterServiceClassExternalID, or clusterServiceClassName required, spec.clusterServiceClassName: Required value: exactly one of clusterServiceClassExternalName, clusterServiceClassExternalID, or clusterServiceClassName required])

It would be helpful if it spit out the help text with examples so I could know how to enter it correctly, as currently I pretty much always follow up an incorrectly entered command with -h.
Desired UX:
> svcat provision
Error: an instance name is required

Usage:
  svcat provision NAME --plan PLAN --class CLASS [flags]

Examples:

  svcat provision wordpress-mysql-instance --class mysqldb --plan free -p location=eastus -p sslEnforcement=disabled
  svcat provision wordpress-mysql-instance --class mysqldb --plan free -s mysecret[dbparams]
  svcat provision secure-instance --class mysqldb --plan secureDB --params-json '{
    "encrypt" : true,
    "firewallRules" : [
        {
            "name": "AllowSome",
            "startIPAddress": "<IP_ADDRESS>",
            "endIPAddress" : "<IP_ADDRESS>"
        },
        {
            "name": "AllowMore",
            "startIPAddress": "<IP_ADDRESS>",
            "endIPAddress" : "<IP_ADDRESS>"
        }
    ]
}
'

Flags:
      --class string         The class name (Required)
  -n, --namespace string     The namespace in which to create the instance (default "default")
  -p, --param stringSlice    Additional parameter to use when provisioning the service, format: NAME=VALUE. Cannot be combined with --params-json
      --params-json string   Additional parameters to use when provisioning the service, provided as a JSON object. Cannot be combined with --param
      --plan string          The plan name (Required)
  -s, --secret stringSlice   Additional parameter, whose value is stored in a secret, to use when provisioning the service, format: SECRET[KEY]

Global Flags:
      --kube-context string   name of the kube context to use
      --kubeconfig string     path to kubeconfig file. Overrides $KUBECONFIG

This could be probably be fixed by changing the stuff we pour into cobra in cmd/svcat somewhere.

made a minor change, and UT now succeed , hopefully better now ...
ok      github.com/kubernetes-incubator/service-catalog/cmd/svcat       7.238s


Don't hardcode Cosign attest type

Fixes #1532.
Since this seems to be intentional, I also raised the issue to describe the issue and maybe keep the discussion there?
Depending on how the discussion goes, maybe merge or close this PR. I don't know - the fix is trivial though :)

@spiffcs probably has the most context here, but I believe the issue is we can no longer verify attestations if they are not using the custom type

Yes, see the issue :)
From what I gathered is that the issue here is
a) a missing type in the verification command
and/or
b) Cosign itself - not Syft.
Cosign default to custom if no type is specified, so I don't know why this has been hard coded?

👋 Hi @Nirusu see this issue we filed with cosign upstream:
https://github.com/sigstore/cosign/issues/2494
As of this feature the only way I could get verify-attestation to behave correctly was by using a custom type - this is why it's been hardcoded this way.

I read the issue you linked - I do agree that we probably should just expose the type at this point and let the downstream commands work as expected giving the user the most freedom

I'll add that change - thanks for the report!

I saw your issue (see my chain of thoughts in #1532), but that's what I don't get.
You ran:
$COSIGN_EXPERIMENTAL=1 cosign attest --predicate test_spdx_scratch.json --type spdxjson caphill4/scratch:latest
COSIGN_EXPERIMENTAL=1 cosign verify-attestation caphill4/scratch:latest

The attest is specified with a predicate, the verify-attestation without.
That doesn't make sense to me.
If you specify the predicate, why add not add it to the verify-attestation? It does not auto-detect the predicate, that's actually broken.
If you want custom as a type anyway, why specify it? Cosign defaults to custom if nothing is specified (hence this PR).
So I don't really see what you were going here for - not specifying it should already give you the same custom behavior.
Specifying it as a predicate in both commands - attest and verify-attestation - should also work (not 100% sure now, though).
But only in one command? Yes, auto-detection is broken, but I would still pass that as user error if you define another type but ultimately don't care about the type.


Set the correct length for stax address display

Make it consistent across all devices

Codecov Report
All modified lines are covered by tests :white_check_mark:

Comparison is base (5547e69) 91.96% compared to head (452a2cd) 91.96%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #51   +/-   ##
=======================================
  Coverage   91.96%   91.96%
=======================================
  Files           8        8
  Lines         336      336
=======================================
  Hits          309      309
  Misses         27       27

Flag
Coverage Δ

unittests
91.96% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Build successful, but traceback says //mediapipe_api\mediapipe_proto_srcs.zip is not a zip file

Plugin Version or Commit ID
v0.10.0
Unity Version
2021.3.3f1
Your Host OS
Windows 11
Target Platform
UnityEditor
[Windows Only] Visual Studio C++ and Windows SDK Version
Windows SDK: 10.0.19041.0
[Linux Only] GCC/G++ and GLIBC Version
No response
[Android Only] Android Build Tools and NDK Version
No response
[iOS Only] XCode Version
No response
Command Sequences
ran build command (path to python.exe build.py build --desktop cpu --opencv=cmake -vv
Log
C:\Users\kmark\UnityProjects\MediaPipeUnityPlugin-master>C:\Users\kmark\AppData\Local\Programs\Python\Python310\python.exe build.py build --desktop cpu --opencv=cmake -vv
INFO (build.py): Building protobuf sources...
DEBUG (build.py): Running bazel --output_user_root C:/_bzl build -c opt --action_env PYTHON_BIN_PATH="C://Users//kmark//AppData//Local//Programs//Python//Python310//python.exe" --action_env ProgramData --action_env PROCESSOR_ARCHITECTURE --action_env PROCESSOR_IDENTIFIER --action_env PROCESSOR_LEVEL --action_env PROCESSOR_REVISION --verbose_failures //mediapipe_api:mediapipe_proto_srcs
DEBUG: C:/_bzl/k57vclnn/external/org_tensorflow/third_party/repo.bzl:124:14:
Warning: skipping import of repository 'com_google_absl' because it already exists.
DEBUG: C:/_bzl/k57vclnn/external/org_tensorflow/third_party/repo.bzl:124:14:
Warning: skipping import of repository 'com_google_benchmark' because it already exists.
DEBUG: C:/_bzl/k57vclnn/external/org_tensorflow/third_party/repo.bzl:124:14:
Warning: skipping import of repository 'pybind11_bazel' because it already exists.
DEBUG: C:/_bzl/k57vclnn/external/org_tensorflow/third_party/repo.bzl:124:14:
Warning: skipping import of repository 'com_google_protobuf' because it already exists.
DEBUG: C:/_bzl/k57vclnn/external/org_tensorflow/third_party/repo.bzl:124:14:
Warning: skipping import of repository 'com_google_googletest' because it already exists.
DEBUG: C:/_bzl/k57vclnn/external/org_tensorflow/third_party/repo.bzl:124:14:
Warning: skipping import of repository 'com_github_gflags_gflags' because it already exists.
DEBUG: C:/_bzl/k57vclnn/external/org_tensorflow/third_party/repo.bzl:124:14:
Warning: skipping import of repository 'rules_python' because it already exists.
DEBUG: C:/_bzl/k57vclnn/external/org_tensorflow/third_party/repo.bzl:124:14:
Warning: skipping import of repository 'build_bazel_rules_apple' because it already exists.
DEBUG: C:/_bzl/k57vclnn/external/org_tensorflow/third_party/repo.bzl:124:14:
Warning: skipping import of repository 'build_bazel_rules_swift' because it already exists.
DEBUG: C:/_bzl/k57vclnn/external/org_tensorflow/third_party/repo.bzl:124:14:
Warning: skipping import of repository 'build_bazel_apple_support' because it already exists.
DEBUG: C:/_bzl/k57vclnn/external/org_tensorflow/third_party/repo.bzl:124:14:
Warning: skipping import of repository 'xctestrunner' because it already exists.
DEBUG: C:/_bzl/k57vclnn/external/org_tensorflow/third_party/repo.bzl:124:14:
Warning: skipping import of repository 'pybind11' because it already exists.
INFO: Analyzed target //mediapipe_api:mediapipe_proto_srcs (0 packages loaded, 0 targets configured).
INFO: Found 1 target...
Target //mediapipe_api:mediapipe_proto_srcs up-to-date:
bazel-bin/mediapipe_api/mediapipe_proto_srcs.zip
INFO: Elapsed time: 3.705s, Critical Path: 0.02s
INFO: 1 process: 1 internal.
INFO: Build completed successfully, 1 total action
//mediapipe_api\mediapipe_proto_srcs.zip
DEBUG (build.py): Unarchiving '//mediapipe_api\mediapipe_proto_srcs.zip' to 'build\Scripts\Protobuf'...
Traceback (most recent call last):
File "C:\Users\kmark\UnityProjects\MediaPipeUnityPlugin-master\build.py", line 460, in
Argument().command().run()
File "C:\Users\kmark\UnityProjects\MediaPipeUnityPlugin-master\build.py", line 111, in run
self._unzip(
File "C:\Users\kmark\UnityProjects\MediaPipeUnityPlugin-master\build.py", line 96, in _unzip
shutil.unpack_archive(source, dest)
File "C:\Users\kmark\AppData\Local\Programs\Python\Python310\lib\shutil.py", line 1273, in unpack_archive
func(filename, extract_dir, **kwargs)
File "C:\Users\kmark\AppData\Local\Programs\Python\Python310\lib\shutil.py", line 1172, in _unpack_zipfile
raise ReadError("%s is not a zip file" % filename)
shutil.ReadError: //mediapipe_api\mediapipe_proto_srcs.zip is not a zip file
Additional Context
No response

ran build command (path to python.exe build.py build --desktop cpu --opencv=cmake -vv

Is that all? Didn't you set an empty string to _BAZEL_BIN_PATH in the build.py?

ran build command (path to python.exe build.py build --desktop cpu --opencv=cmake -vv

Is that all? Didn't you set an empty string to _BAZEL_BIN_PATH in the build.py?

I had not. but just tried that and am still getting the same error.

just tried that

No, no, you MUST NOT do that. I thought that was the cause.
https://github.com/homuler/MediaPipeUnityPlugin/blob/4302f4a174404ce5fff65fa2ce219ed71f8b9094/build.py#L111
Your log says that //mediapipe_api\mediapipe_proto_srcs.zip ==  os.path.join(_BAZEL_BIN_PATH, 'mediapipe_api', 'mediapipe_proto_srcs.zip') but I think it can't be since _BAZEL_BIN_PATH is not empty (bazel-bin).
Do you know why it is in your environment?

So I realised I needed it to equal bazel-bin already, and I also figured out what the issue had been before and I'm embarrassed to admit it but it was totally my fault. I was having issues where it turned out cmd.exe was using python.exe from msys2 and it was therefore automatically changing // to / at one point and in my debugging efforts I added '//' before mediapipe_api in a few places and forgot to remove them again when it didn't work. Removing them fixed the error. So sorry to waste your time!
I got the build to complete successfully, however, in Unity I am getting several errors about different types not having implicit reference conversions, ie. in BoxTrackingGraph.cs, _trackedDetectionsStream(Mediapipe.DetectionVectorPacket) cannot be implicitly converted to Mediapipe.packet<System.Collections.Generic.List<Mediapipe.Detection>>

They all seem to have to do with various classes not being able to be used as type parameter TPacket because of no reference conversion.

hmm. I extracted your repo to a new spot and created a new Unity project and redid the build command from that location, and when I opened the Unity project, I don't see any errors. I think I might have had Unity open when I built the libraries the first time, and Unity was in safe mode due to the compiler errors. So, again, embarrassing on my part, but I'm hoping it's maybe useful for you to know?

I think running python build.py uninstall then python build.py build ... will fix the errors.


Android login implementation

Android Facebook Implementation
1. Setup Android's Facebook SDK in config files
Added Facebook SDK as a dependency into Android client via Maven repository
Affected files (Some changes are generated)

client/android/WorldScope/.idea/vcs.xml
client/android/WorldScope/app/app.iml
client/android/WorldScope/build.gradle
client/android/WorldScope/app/build.gradle
client/android/WorldScope/app/src/main/AndroidManifest.xml

2. Removed LoginActivity and its XML layout
These 2 activities were placeholders and are now removed
Affected files

client/android/WorldScope/app/src/main/java/com/litmus/worldscope/LoginActivity.java
client/android/WorldScope/app/src/main/res/layout/activity_login.xml

3. Implemented Facebook Login and imported icon resources
Implemented a new FacebookLoginActivity and FacebookLoginFragment with its corresponding layouts
Affected files

client/android/WorldScope/app/src/main/java/com/litmus/worldscope/FacebookLoginActivity.java
client/android/WorldScope/app/src/main/java/layout/FacebookLoginFragment.java
client/android/WorldScope/app/src/main/res/layout/activity_facebook_login.xml
client/android/WorldScope/app/src/main/res/layout/fragment_facebook_login.xml

Updated strings resources to remove previous LoginActivity strings and added new strings for Facebook
Affected files

client/android/WorldScope/app/src/main/res/values/strings.xml

Other resources such as icons are also added

Good to be merged.


feat: expose decorator options

What kind of change does this PR introduce?
Feature
Description
List of arguments to be created due to the introduction of this PR

--decorator flag in subcommand start specifies the type of decorator to use on the main worker and event worker. If not specified, the decorator feature is disabled. (possible inputs: tc39, typescript, typescript_with_metadata)
--decorator flag in subcommand bundle specifies the type of decorator to use when bundling. If not specified, the decorator feature is disabled. (possible inputs are same above)

Apart from the CLI arguments above, when you create a User Worker via the API in the main service, you can specify the type of decorator to use on the User Worker.
#[derive(Debug, Clone, Copy, Serialize, Deserialize)]
#[serde(rename_all = "snake_case")]
pub enum DecoratorType {
    /// Use TC39 Decorators Proposal - https://github.com/tc39/proposal-decorators
    Tc39,
    /// Use TypeScript experimental decorators.
    Typescript,
    /// Use TypeScript experimental decorators. It also emits metadata.
    TypescriptWithMetadata,
}

...

// type DecoratorType = "tc39" | "typescript" | "typescript_with_metadata"
const decoratorType = "typescript_with_metadata";

return await EdgeRuntime.userWorkers.create({
	...
	decoratorType
});

type
experimentalDecorators
emitDecoratorMetadata

tc39
-
-

typescript
✅
❌

typescript_with_metadata
✅
✅

There are differences between Typescript's implementation of the decorator specification and the decorator specification specified by Ecma TC39, see the link below for a description of the differences:
https://github.com/tc39/proposal-decorators/issues/329
Note that the typescript_with_metadata option allows concrete types in design time to be output with metadata during the transpiling process. This option is equivalent to emitDecoratorMetadata of tsconfig.json.
For more information on this topic, see the link below:
https://www.typescriptlang.org/tsconfig#emitDecoratorMetadata
Resolves #296

We would need to also update Supabase CLI for this to work on deployed edge-functions.


Release rep

Prepares release for Zenodo (see #6).
Mostly small things and refactorings. However, please have a look at the changes to forms.csv. It seems that there were line breaks in the quoted Arabic script before the CLDF generation. Can you confirm that the current state of forms.csv is as intended? If yes, I could prepare a release of this after the PR has been merged.

@chrzyki I think I fixed the arabic issue and launched cldfbench again, should be fine now.


Improve nested parsing of needs content, with correct source mapping

This is a WIP based on @PhilipPartsch original PR (#1150), where the main goal is to have sphinx warnings point to a sensible location in the source documentation (file-path and line-number)
The main difference being that here, where possible we simply pass through the StringList object (a "source-mapped" representation of the content) from the actual directive and pass that.
This works best for needs directives that are specified directly on the page, e.g.
.. req: title
   :id: my_id

   This is the content, which can have **nested syntax** that we want to parse

If the content is not directly on the page, then this is where I have found things becoming tricky.
This could either be because, the need contains jinja content, e.g.
.. req: title
   :id: my_id
   :jinja_content: true

   {% for i in range(11) %}
   This is the content.
   After it is "pre-rendered" by jinja, then the content might not align with the original source.
   {% endfor %}

or because it is an external need, loaded from a JSON file, e.g.:
needs_external_needs = [
      {
        'base_url': 'http://mydocs/my_project',
        'json_url':  'http://mydocs/my_project/needs.json',
        'version': '1.0',
        'id_prefix': 'ext_',
        'css_class': 'external_link',
      },
 ]

For both of these, ideally we want to construct our own source-mapping object to pass to docutils/sphinx: https://github.com/live-clones/docutils/blob/582f8c946f776da7db652c06e0b49521f9dc3fdf/docutils/docutils/statemachine.py#L1323
For example, we might want to simply point every line to the start of the directive:
sourcemap = StringList(content.splitlines(), items=[(source, lineno) for _ in content.splitlines()])
state.nested_parse(sourcemap, ...)

The appears to me, to be a minor technical annoyance here, in that docutils ignores this source-mapping, when it comes to issue warnings, for example here: https://github.com/live-clones/docutils/blob/582f8c946f776da7db652c06e0b49521f9dc3fdf/docutils/docutils/parsers/rst/states.py#L429
instead of using StateMachine.get_source_and_line(lineno), which returns what we set in sourcemap, to get the correct place to report about, it actually uses abs_line_number, which just report the line offset in.
This is probably a bit technical for people reading this, but I write it here asI think I will eventually upstream the question as a bug to docutils.

Cool.
Hint: needs_external_needs are not an issue to been parsed, as external needs are been skipped before parsing. See current api/need.py.
I believe we have to have a look about needimport, especially as they are from a external source but not marked as is_external.


Abstracts database storage & Status page UI

Added an IStore interface and an implementation of the current json storage.
This will help adding support for other databases like sqlite
Added also a Status page which displays all the peers
sample of how it looks like here:
https://prnt.sc/1p5oagi

Thanks @gosom for the PR. I've left a few comments there :-)


Broken URL at waseem0605.netlify.app

Reference link to CV/resume pdf download broken at portfolio site, Tested on two different browsers and internet connections.
Tested on these browsers:

Firefox 117.0.1 x64
Edge Version 117.0.2045.36 (Official build) (64-bit)

Fixed. Can check now...!


Update Microsoft.OAuthInput.schema

Remove the reference to InputDialog base.
#2673

@luhan2017 - we need to re-target this to master branch.

@luhan2017 - we need to re-target this to master branch.

OK, all the input dialogs didn't have a required section, we may need to add them all.

OK, all the input dialogs didn't have a required section, we may need to add them all.

Yes. Let's do a required properties for all inputs for 4.7.

Pull Request Test Coverage Report for Build 82545

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.5%) to 77.587%

Totals

Change from base Build 82538:
0.5%

Covered Lines:
6404

Relevant Lines:
8254

💛  - Coveralls

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.Luis.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.QnA.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.ApplicationInsights.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Dialogs.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Configuration.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Connector.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Schema.dll compared against version 4.5.3

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.Luis.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.QnA.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.ApplicationInsights.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Dialogs.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Configuration.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Connector.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Schema.dll compared against version 4.5.3


update to cpp seller guide

adding managed applications and saas; updating reports and best practices

#sign-off

@divacc : Thanks for your contribution to the Azure documentation! The author, @rupeshazure, has been notified to review your proposed change.

@: Only the author, @rupeshazure, can sign off on this pull request.

@: Only the author, @rupeshazure, can sign off on this pull request.

@: Only the author, @rupeshazure, can sign off on this pull request.

@: Only the author, @rupeshazure, can sign off on this pull request.

@: Only the author, @rupeshazure, can sign off on this pull request.

@: Only the author, @rupeshazure, can sign off on this pull request.

@: Only the author, @rupeshazure, can sign off on this pull request.

@: Only the author, @rupeshazure, can sign off on this pull request.

@: Only the author, @rupeshazure, can sign off on this pull request.

@: Only the author, @rupeshazure, can sign off on this pull request.

@: Only the author, @rupeshazure, can sign off on this pull request.

@: Only the author, @rupeshazure, can sign off on this pull request.

@: Only the author, @rupeshazure, can sign off on this pull request.

@: Only the author, @rupeshazure, can sign off on this pull request.

@: Only the author, @rupeshazure, can sign off on this pull request.

@: Only the author, @rupeshazure, can sign off on this pull request.

@: Only the author, @rupeshazure, can sign off on this pull request.

@: Only the author, @rupeshazure, can sign off on this pull request.

@: Only the author, @rupeshazure, can sign off on this pull request.

@: Only the author, @rupeshazure, can sign off on this pull request.

@: Only the author, @rupeshazure, can sign off on this pull request.

@: Only the author, @rupeshazure, can sign off on this pull request.

@: Only the author, @rupeshazure, can sign off on this pull request.

@: Only the author, @rupeshazure, can sign off on this pull request.

@: Only the author, @rupeshazure, can sign off on this pull request.

@: Only the author, @rupeshazure, can sign off on this pull request.

@: Only the author, @rupeshazure, can sign off on this pull request.

This article has been deleted; the URL was redirected to https://docs.microsoft.com/en-us/azure/marketplace/marketplace-publishers-guide.


New UI

I'm considering do a new UI for app, but need to know some thins.
What community need in a new interface.

I would even be willing to do some of this work, but I haven't heard a clear feature request yet.  Yes, I am not a graphic designer, but I would rather have specific issues for feature requests.

I'm UX designer, I could create a new interface but first need to understand what do u think for this project
srry for bad english

Oh nice! Contributions or mocks welcome! :+1:
I think the feature set is pretty well locked in (and you can toy around with KeePassX on the desktop to see the full feature set this is based on), but I'm sure there can be improvements to visual presentation (laying out the page differently, or possibly having the flow of interactions be different).
My only request is that I don't particularly want to add in a JS framework into this app.  There's already enough security concerns that I don't want make the system more complex by adding client-side state.


style: previous PR broke style guidelines

Corrected quoting in error message.

Hey @gabe-camp thanks for the PR.
There's a problem with your previous commits. The solution is a bit complicated so the simplest I can recommend is to introduce your changes in a new PR branched from master.
git fetch --all
git checkout -b style-message origin/master

Apply your new changes and submit another PR.

Apply your new changes and submit another PR.

ok


Throws an error if nonce is sent as a string

It would be more robust if this microservice handle receiving the nonce as either a string or a number.

This is taken care of in: https://github.com/oceanprotocol/dbs_arweave/commit/7f72e18036fe47a33a2fdd72bf8024e0afc533c8


Update repositories.md

Hi to whoever is reading this! 🤗
What's in this PR?

Fix path to Pull requests advanced usage as was throwing 404
Capitalize Dataset as done with Models and `Spaces

Thanks!

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Open buffer failing to update when its file is changed on disk

Check for existing issues

[X] Completed

Describe the bug / provide steps to reproduce it
If a file is changed on disk while open on a Zed tab, the buffer is not updated to reflect those changes. Instead, a blue button is shown in the tab to mark the file as modified as if the modification had come from editing it in Zed.
Environment
Zed: v0.98.2 (stable)
OS: macOS 13.4.1
Memory: 32 GiB
Architecture: aarch64
If applicable, add mockups / screenshots to help explain present your vision of the feature
No response
If applicable, attach your ~/Library/Logs/Zed/Zed.log file to this issue.
If you only need the most recent lines, you can run the zed: open log command palette action to see the last 1000.
No response

I can't reproduce this. The current behavior is that when a file changes on disk, if the buffer is currently unmodified, it will be reloaded. If the buffer has unsaved changes, then it will be put into a conflicting state (indicated by a yellow dot). Can you share some detailed reproduction steps?

This looks like a duplicate of #1971.


Update port for azure-template-cpp

Update port for azure-template-cpp

Is this a draft or is this intended to add a new port?

Is this a draft or is this intended to add a new port?

@ras0219 New port, the PR title opened by the robot is fixed.

@ras0219 New port, the PR title opened by the robot is fixed.

/azp run

/azp run

We can close these stale PRs.
cc @antkmsft

Closing this PR since duplicated to #16778.


Update google-chat to 18.4.101

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} reports no offenses.
[x] The commit message includes the cask’s name and version.

Thanks for the PR!


Add OpenAI API Key-based version of Jsonformer

Currently, the Jsonformer class is using a local transformer model and tokenizer to generate data in JSON schema format. However, it would be useful to have a version of the class that uses OpenAI's Language Models to generate data. Therefore, I would like to request the implementation of a new version of the class that takes OpenAI API keys and a model name as parameters, in addition to the existing parameters of the Jsonformer class.
The OpenAI API keys should be used to authenticate the requests made to OpenAI's API. The model name should be used to specify which LLM to use for data generation. The new version of the class should work similar to the current implementation but use OpenAI's API to generate data
This feature would allow for the generation of data using state-of-the-art LLMs from OpenAI.
Thank you for considering this feature request.

This is kind of obsolete now with function calls feature from OpenAI

This is kind of obsolete now with function calls feature from OpenAI

I'm going to provide some test data shortly. But I still think this will be much cheaper at scale

This is kind of obsolete now with function calls feature from OpenAI

it's not obsolete. Function calling is but one application of many of jsonformer.

Some related work that considers GPT-4: https://github.com/newhouseb/clownfish


ReactNativeRenderer.mountSafeCallback throws isMounted() deprecation errors

Hi there,
I'm getting a handful of isMounted deprecation errors since upgrading to 0.55.3.
The stack trace looks like this:

  | console.warn | @ | YellowBox.js:80
  | printWarning | @ | react.development.js:88
  | lowPriorityWarning | @ | react.development.js:107
  | get | @ | react.development.js:272
  | (anonymous) | @ | ReactNativeRenderer-dev.js:3518
  | __invokeCallback | @ | MessageQueue.js:398
  | (anonymous) | @ | MessageQueue.js:137
  | __guardSafe | @ | MessageQueue.js:314
  | invokeCallbackAndReturnFlushedQueue | @ | MessageQueue.js:136
  | (anonymous) | @ | debuggerWorker.js:70

The function in question looks like this right now:
/**
 * In the future, we should cleanup callbacks by cancelling them instead of
 * using this.
 */
function mountSafeCallback(context, callback) {
  return function() {
    if (!callback) {
      return undefined;
    }
    if (typeof context.__isMounted === "boolean") {
      // TODO(gaearon): this is gross and should be removed.
      // It is currently necessary because View uses createClass,
      // and so any measure() calls on View (which are done by React
      // DevTools) trigger the isMounted() deprecation warning.
      if (!context.__isMounted) {
        return undefined;
      }
      // The else branch is important so that we don't
      // trigger the deprecation warning by calling isMounted.
    } else if (typeof context.isMounted === "function") {
      if (!context.isMounted()) {
        return undefined;
      }
    }
    return callback.apply(context, arguments);
  };
}

so it seems there was some effort made to suppress this warning from being thrown, but because context.isMounted has a getter with side effects ಠ_ಠ, this gets thrown anyway:
var deprecatedAPIs = {
    isMounted: ['isMounted', 'Instead, make sure to clean up subscriptions and pending requests in ' + 'componentWillUnmount to prevent memory leaks.'],
    replaceState: ['replaceState', 'Refactor your code to use setState instead (see ' + 'https://github.com/facebook/react/issues/3236).']
  };
  var defineDeprecationWarning = function (methodName, info) {
    Object.defineProperty(Component.prototype, methodName, {
      get: function () {
        lowPriorityWarning$1(false, '%s(...) is deprecated in plain JavaScript React classes. %s', info[0], info[1]);
        return undefined;
      }
    });
  };

I've suppressed the warnings myself for now, but since it seems that someone took some effort to make sure these warnings don't happen, I thought I'd go ahead and open the ticket.

I've got that too, but it's expected. See release notes ...
https://github.com/react-native-community/react-native-releases/blob/master/CHANGELOG.md#054

Duplicate of #18868


[team] New PMC jinyong and new committer

please make sure you information correct @JinyLeeChina @WangJPLeo

I think it is ok, maybe we can merge it , PTAL


liferay-theme-tasks 10.0.0-alpha.1+gulp 4.0.0 - UnhandledPromiseRejectionWarning when running 'gulp watch'

Steps to reproduce:

Generate a theme with JS Theme Toolkit v9.5.0.
Edit the theme's package.json and point liferay-theme-tasks to "10.0.0-alpha.1" and gulp to "^4.0.0".
rm -rf node_modules (just in case).
rm package-lock.json (just in case).
Run npm install again.
Navigate to theme's root folder.
Make an edit to to file _custom.scss.

html body {
    background-color:green;
}

Run 'gulp deploy'.
Start liferay.
Apply theme to a site.
Navigate to theme’s root folder.
Run 'gulp watch'.

Expected Result:
No warning message in console.
Actual Result:
UnhandledPromiseRejectionWarning is thrown, but this warning doesn't affect the function.

I cannot reproduce it and I cannot see any information in the stack trace of what could be happening.
Can you provide more info @zhangyan0701?
If not, maybe you can test it in a different machine to see what happens. Or we can close it, given that it doesn't alter the functionality...

Hi @izaera,
Thanks for working on this! I changed to test on CentOS 7 VM instead of a docker image, and didn't see this warning message. The gulp watch worked as expected. I will close this issue.
Regards!


"Please add a @NgModule annotation." error with AOT, full path vs short package.json main

Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request

Versions.
Windows 7
@angular/cli: 1.0.2
node: 6.9.0
os: win32 x64
@angular/common: 4.1.1
@angular/compiler: 4.1.1
@angular/core: 4.1.1
@angular/forms: 4.1.1
@angular/http: 4.1.1
@angular/platform-browser: 4.1.1
@angular/platform-browser-dynamic: 4.1.1
@angular/router: 4.1.1
@angular/cli: 1.0.2
@angular/compiler-cli: 4.1.1
Repro steps.
ng serve works ok, but aot compilation fails in case:
import module like import { Ng2FlatpickrModule } from 'ng2-flatpickr/lib/ng2-flatpickr.module.js';
-> No errors
import module like import { Ng2FlatpickrModule } from 'ng2-flatpickr;
-> "Please add a @NgModule annotation." error
The log given by the failure.

ERROR in Unexpected value 'Ng2FlatpickrModule in .../node_modules/ng2-flatpickr/lib/ng2-flatpickr.module.js' imported by the module 'AppModule in .../src/app/app.module.t
s'. Please add a @NgModule annotation.
ERROR in ./src/main.ts
Module not found: Error: Can't resolve './$$_gendir/app/app.module.ngfactory' in 'D:\Projects\work\unleesh-frontend\src'
@ ./src/main.ts 6:0-74
@ multi ./src/main.ts

Desired functionality.
There should be no errors
Mention any other details that might be useful.
Here is my fork https://github.com/FriOne/ng2-flatpickr

This really sounds like a problem with how that library is published. We don't yet have a full guide for how to do it, but there's WIP version that you can direct the author to in https://github.com/angular/angular/pull/16486

@filipesilva, did you worked with webpack to create the lib. I see that you're using rollup to bundle your code in your lib quickstart.
I'm facing missing NgModule annotation issue in my custom library and I'm trying to debug what is the root case. As far as I understood the issue is that awsome-typescript-loader/ts-loader for webpack does not parse the code compiler-friendly -  the module is exported with all the required properties, but it's not exported as a NgModule object. Thus the compiler fails to load the exported module as it does the typecheking expecting instance of NgModule.
Currently I want to check @ngTools/webpack or ngc (either vanilla or throught ngc-webpack). You are using ngc, what would confirm my theory.

Facing same issue,  ERROR in Error: Unexpected value 'JSONEditorModule in /node_modules/ng2-jsoneditor/dist/ng2-jsoneditor.d.ts' imported by the module 'AppModule in /src/app/app.module.ts'. Please add a @NgModule annotation.
This is a third-party library and there is no annotation for module or component. How would I update that library? or is there any other way to add the annotation for that module?


default values for environment variables

Hi,
Is there a way to default a variable in binding.gyp yet allow a different user externally set a different value.
https://github.com/HDF-NI/hdf5.node/blob/master/binding.gyp#L3
Some people have a hard coded path to native dependencies while others like a more app packaged deployment and/or the ability to have more than one version installed
"hdf5_home_linux": "/usr/local"

How to keep a default yet make it adaptable for other choices?

In theory, you can pass custom defines to node-gyp like this:
$ node-gyp -Dhdf5_home_linux=/foo/bar rebuild

You need to declare the variable as configurable in your binding.gyp, like this:
'variables': { 'hdf5_home_linux%': '/usr/local' }

Having said that, I had a look at the code that implements argument parsing and it looks broken to me.  I'll see if I can come up with a PR.
Also, how to get npm to pass the -D option to node-gyp is something I'll have to leave as an exercise to the reader because I don't know. :-)

Thank you for looking into this.  I'll test further

I found that this does work to make npm (node-v5.0.0) pick up the variable as a switch.
npm install  --hdf5_home_linux=<pathtoyourhdf5install>

for example where this was a dependent project that used my module

For anyone coming here through a search engine, you can alternatively pass options through the GYP_DEFINES environment variable:
$ env GYP_DEFINES="-Dname=value" npm install

That should work with any node-gyp version.

For anyone coming here through a search engine

The GYP_DEFINES is enabled by command line option "--ignore-environment", which by default is true.

$ env GYP_DEFINES="name=value name2=value" npm install

Anybody know where's the documentation for this? 🙄


Updating some libs

Bug Report or Feature Request (mark with an x)
- [ ] bug report -> please search issues before submitting
- [X] feature request

Desired functionality.
Update some libs:

autoprefixer 6.5.3 -> 6.7.7
debug 2.1.3 -> 2.6.3
diff 3.1.0 -> 3.2.0
html-webpack-plugin 2.19.0 -> 2.28.0
rsvp 3.0.17 -> 3.5.0
rxjs 5.0.1 -> 5.2.0
webpack 2.2.0 -> 2.3.2
webpack-dev-server 2.3.0 -> 2.4.2
webpack-merge 2.4.0 -> 4.1.0
ts-node 2.0.0 -> 3.0.2
tslint 4.0.2 -> 5.0.0

I believe that with the update of these libraries, the Angular CLI, could make use of the bugfix that these latest versions corrected and the new functionalities implemented, making the applications more stable.

Would you be available to make a Pr that adds these updates? These files would need the update:

https://github.com/angular/angular-cli/blob/master/package.json
<EMAIL_ADDRESS>

FYI, tslint will need some additional configuration updates, as well. I mention this, as I tried to upgrade another project to 5.x (from 4.x), and 💥


Refactor

What does this PR do?
Delete unnecessary files and add first iteration for the search
Any background context you want to provide?
Please review deleted files, maybe I deleted some files wrong

I think that initialState is good to have a store tree at a glance. helps to understand site structure. For example
{
  searchPage: {
    searchText: '',
    isSearching: false
  },
  resultsPage: {
    resultList: []
  }
}
''''

Don't you think?

Hard to markup on mobile :p

Yep, we can do it in the future.

I need to merge this PR, so...


uploading sanitized image file from angular2 to firebase storage

I'm using angular2 DOM sanitizer to read an image file. I want to upload this file to the firebase storage. However, I am unable to find a way to convert the sanitized image to a blob or something that can be uploaded.
I'm using the function to read the image file.
readImage2(inputValue: any): Observable<any> {
    var file:File = inputValue.files[0];
    return this.ng2ImgMaxService.compress([file], this.maxImageSize).mergeMap( (result) => {
      const compressedImage = window.URL.createObjectURL(result);
      const compressedImageTrusted=this.sanitizer.bypassSecurityTrustUrl(compressedImage);
      return Observable.create((observer) => {
          observer.next({imageUrl: compressedImageTrusted});
      })
  })
}

In this code, I want to upload the compressedImageTrusted file to firebase storage. This file has a type SafeUrlImplwhich I'm not sure how to convert to something that I can upload to a firebase storage. Any help would be appreciated.

I should have raised the issue in angularfire2 repo..


feature add scanRows scanBytes in auditlog

Proposed changes
info from profile
sql
select count(*) from web_sales;
+----------+
| count(*) |
+----------+
|   719384 |
+----------+

profile
-  ScanByteRead:  6.56  MB
-  ScanRowsRead:  719.384K  (719384)

auditlog
State=EOF|ErrorCode=0|ErrorMessage=|Time(ms)=49|ScanBytes=6881280|ScanRows=719384|ReturnRows=1|StmtId=16|QueryId=d93ad95d62484a3f-a81acb47b7835a9c|IsQuery=true|isNereids=true|feIp=<IP_ADDRESS>|Stmt=select count(*) from web_sales|CpuTimeMS=49

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall

run p0

#25435


[Bug]: Modern.js Builder 使用 vue3 rspack 热更新报错

Version
Mac os
Chrome 116

Details
随意修改模版里面的值保存后重新编译，热更新出现下面的错误
10% building ./HelloWorld.vue.css?vue&type=style&index=0&id=469af010&scoped=true&lang=css!=!-!../.thread 'tokio-runtime-worker' panicked at 'Module(missing|/Users/allen/Documents/tele-zone/builder-vue3-rspack/./HelloWorld.vue?vue&type=style&index=0&id=469af010&scoped=true&lang=css) should be added before using', crates/rspack_core/src/chunk_graph/chunk_graph_module.rs:68:26
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
Reproduce link
https://github.com/web-infra-dev/modern-js-examples/tree/main/examples/builder-vue3-rspack
Reproduce Steps
修改 HelloWorld 组件

@94ashui You can lock the version of modern.js packages to use Rspack 0.2.12:

After my debugging, I found this one is caused by PR: https://github.com/web-infra-dev/rspack/pull/4026. Would you please to take a look? @underfin

@chenjiahan Just for heads-up: v0.3.0 is working as well.


Tips for avoiding exploding gradients in the generator?

When training, I'm getting NaN errors on some layers. The actual layer depends on my seed but, for example, I just got the error :
Nan in summary histogram for: generator/decoder_7/batchnorm/offset/values

Sometimes I'll also get them on the encoder gradients. At least in those cases, inspecting the corresponding gradients via tensorboard/histograms shows huge values.
Did you come across this at all and any tips for how to adjust for these?
I'm training for colorization on my own dataset. The error tend to happen within the first 2000 images. Viewing the resulting generated imaged in tensorboard up to that point show promising results. I've played around with varying the 0.2 parameter getting used in the leaky relu layers but am not seeing any obvious improvements.

I haven't seen this problem myself, but you can try clipping the gradients with something like this: grads = [tf.clip_by_value(g, -5, 5) for g in grads]

Hi @usersub0 , You can check wgan_gp, which provide better way than weight clips referenced by @christopherhesse

Have you solved it? Im running into the same issue with colorization ...


Adding the ability to have volume either temporary of with file system binding

Adding the ability to set a temporary volume and show an example for https://github.com/testcontainers/testcontainers-java/issues/7420

#7420


debugger.html - window.require("BatchedBridge")["processBatch"] is undefined

Hi RN team,
I'm experiencing this issue after upgrading to 0.4.0. I confirmed from one other person in IRC that this is happening to them as well.
The packager will load the app fine, but when the app opens in simulator or on a device, there is only a white screen. When opening chrome dev tools, it will show that there is an error in debugger-ui.html at line 54: Uncaught TypeError: Cannot read property 'apply' of undefined
'executeJSCall:method:arguments:callback:': function(message, sendReply) {
  var returnValue = [[], [], [], [], []];
  try {
    if (window && window.require) {
      // it breaks here
      returnValue = window.require(message.moduleName)[message.moduleMethod].apply(null, message.arguments);
    }
  } finally {
    sendReply(JSON.stringify(returnValue));
  }
}

putting a catch there reveals a little more information:
TypeError: Cannot read property 'apply' of undefined
    at messageHandlers.executeJSCall:method:arguments:callback: (http://localhost:8081/debugger-ui:54:79)
    at WebSocket.ws.onmessage (http://localhost:8081/debugger-ui:82:5)

This is what executeJSCall:method:arguments:callback: is called with before it breaks:

I hope this helps. In the meantime I will downgrade to 0.3.11. Let me know if you need anymore info.

When I run this same code in 0.3.11 it looks like it is calling with moduleMethod: "callFunctionReturnFlushedQueue" instead of "processBatch"

Thanks for reporting, I'll take a closer look soon

Same here.
Steps to reproduce:

Install<EMAIL_ADDRESS>and react-native init test-app && cd test-app.
Install react-native-icons@latest https://github.com/corymsmith/react-native-icons/
Run the app, get white screen + error Uncaught TypeError: Cannot read property 'apply' of undefined  in console.

Ok, I have an update. @unknownexception your comment made me suspect it was versions of RN that packages are using.
I have three packages in my app that are dependent on RN:
"react-native-navbar"
"react-native-side-menu"
"react-native-video"

After installing them locally and pointing them to 0.4.0, my app is working with 0.4.0! :tada: :

@mjw56, thanks, now it works. I hope npm@3 will help to deduplicate dependencies (it seems peerDependency has failed and nobody uses it anymore).

@unknownexception ok great, I'm wondering if there is anything we could do in the meantime to prevent something like this from happening again in the future? @frantic

Something like peer dependencies (or whatever succeeds it) makes sense. I'll go out on a limb and say all of the React Native modules so far shouldn't specify react-native as a regular dependency. Even if React's native side correctly handles multiple copies of the JS side, it's important to keep the JS bundle small for boot times especially on an iPod touch or 4s.
One way React could guard against this would be to set a global flag when first loaded, and warn/throw if the flag is already set.

@ide so is that the way it is happening now is that multiple copies of RN are being loaded on the device for every module which has RN as a dependency?

I believe so unless you've run npm dedupe. I haven't looked at an actual JS bundle to verify that though.

So this issue was resolved with either a module declaring a react-native peerDependency or upgrading their react-native version to 0.4.0 for an app using 0.4.0. I am going to close this for now. I'm not sure if something should be added to the docs about this, I'd be happy to send a PR.


Outlook mail - LLM response does not let the user know that there are more  emails returned  when email queries return more than 10 entries.

Steps to reproduce the problem:

Create an agent with outlook tool bundle.
Ask to return email list that will result in more than 10 emails being returned like list emails from last 7 days

LLM returns first 10 emails from the list.
There is no indication to the user that there are more emails though if I ask to return more emails like list 5 more emails from the list , it gets the next set of emails as expected.
Expected Behavior:
When LLM shows 10 emails when the returned email list is longer , it should also inform the user that there are more emails that the query has actually returned.
Debug calls -
calls - 2024-11-12T222820.981.json
calls - 2024-11-12T222815.134.json

Are you saying that you are now seeing an issue with the outlook tool where it's repeating data over and over again in its response?

@drpebcak No.I dont see it repeat the same data again and again.
LLM is not able to tell the users that there are more than the 10 emails that get listed when there are lot of emails to be returned. In this same scenario , LLM has been able to inform users about there being more emails.
I was pointing out that i was seeing better handling of data in case of google sheets tool than xlsheet tool.


Bug/feature in find_following_working_day

find_following_working_day returns the next non-weekend day, but not necessarily the next working day.
E.g. if a Sunday and a Monday are public holidays (assuming SAT/SUN weekend), and one wants to roll the Sunday to the next working day then it should return Tuesday.  Instead it returns Monday.
Since this function is often used for determining which days are holidays, it is non-trivial to just incorporate the holiday calendar as you get a chicken and egg situation (you can't check the holiday calendar when trying to calculate the holiday calendar).

Mmmmm I see... you're right, this is a chicken & egg situation, here.
AFAICT, we're adding fixed holidays first, and then variable days. The "find_following_working_day" function should not be used for calculating holidays, but should rather only be used when the holiday calendar for this year is already computed.
I need to dig into it and try to find a case where this causes a bug.

The first two days of Lunar New Year 2013 were Sunday/Monday, so in (at least) Singapore and Hong Kong the holiday was Monday/Tuesday.
I have written a Singapore calendar with this case as a failing test.  I can submit it as a PR if you like.

I have written a Singapore calendar with this case as a failing test. I can submit it as a PR if you like.

Oh, that'd be awesome, please do!

Done.  Note that I know there is some stuff to cleanup - e.g. the super() calls double-populate some holidays - but the test case works as required.

Any thoughts on this Sir?  Many thanks.

Hello,
Sorry about the lack of feedback, I've been a bit busy. I think I may start out of your PR/branch and rework it a bit before integrate it into the workalendar lib.
Thanks a lot for your time.

@nedlowe quick and important question ; it looks like the Chinese New Year day in 2013 was on February the 10th. But on your branch, it's set on the 11th, and the shift day on the 12th:
self.assertIn(date(2013, 2, 11), holidays)  # CNY

That's weird, because the 11th February 2013 falls on a Monday, so I think it's an error on your side, but just to make sure... Which one should I trust?

I'm tempted to implement the following (pseudo-code):
days = super(Singapore, self).get_variable_days(year)
pre_shifted_days = [
    (LunarCalendar.lunar(year, 1, 1), "Chinese New Year"),
    (LunarCalendar.lunar(year, 1, 2), "Chinese New Year"),
    (LunarCalendar.lunar(year, 4, 15), "Vesak Day")
    (date(year, 1, 1), "New Year"),
    (date(year, 5, 1), "Labour Day"),
    (date(year, 8, 9), "National Day")
]
# Singapore rolls all Sunday holidays.
# I feel this should be baked deeper in the class hierarchy
for holiday, label in pre_shifted_days:
    # Whether they're on Sunday or not, they can be added to the calendar
    days.append((holiday, label))
    if holiday.weekday() == SUN:
        days.append((holiday + timedelta(days=1), label + ' shift'))

this way, you'll have both "official" holidays, and shifted days in the variable days.

Hey no worries, was just checking in :-)
Here is the official Ministry of Manpower calendar for 2013: http://www.mom.gov.sg/newsroom/press-releases/2012/singapore-public-holidays-2013
So CNY was 10/11th.  The 10th was a Sunday so rolled to the next available date (12th).
The possible implementation doesn't solve this issue (unless I'm missing something) since the holiday on the 10th (Sunday) rolls to the 11th.

So CNY was 10/11th. The 10th was a Sunday so rolled to the next available date (12th).

oh! now I see. I guess that the right way to tackle this problem is to make a dedicated method in Singapore calendar. Will work on it as soon as I can

I'm happy to do it for Singapore, but I guess my point was this is probably
not a Singapore specific thing.  Same for Sunday rolls in general I guess...
On Wed, 24 May 2017 at 18:20, Bruno Bord<EMAIL_ADDRESS>wrote:

So CNY was 10/11th. The 10th was a Sunday so rolled to the next available
date (12th).
oh! now I see. I guess that the right way to tackle this problem is to
make a dedicated method in Singapore calendar. Will work on it as soon as I
can
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/novafloss/workalendar/issues/194#issuecomment-303681951,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATTDfF83Ddu5ZrPDF9XtLLHki3YAyKgBks5r9ARogaJpZM4NY3JI
.

closed by #198

Singapore calendar available in the v1.3.0 release


Autonomus driving button

User story:
As a user, i want to be able to make the car move autonomously  through the app.
Acceptance Criteria:
A button on the app, that is when pressed, the car starts the autonomous driving.

Working on that in #46


[ci] release

This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.
Releases
<EMAIL_ADDRESS>Patch Changes

#2633 5adb720 Thanks @HiDeoo! - Fixes a VoiceOver issue with Safari where the content of a <script> element could be read before the sidebar content.

size-limit report 📦

Path
Size

/index.html
6.15 KB (0%)

/_astro/*.js
22.41 KB (0%)

/_astro/*.css
13.73 KB (0%)


Allow better support for test frameworks other than JUnit

See #86 for background

@rnorth I can take TestNG support since one of our projects is using it :) See #132

Cool, thanks! In terms of structuring this, I think there's a reasonably chunky refactoring coming up in the future so it might be best to consider now. I'm thinking that:

all docker support should be moved into a new maven module (named, e.g. containercore) that is completely test framework agnostic. Part of the rationale also includes potentially using the library in non-test contexts, BTW, which is admittedly a bit of a revision on my previous ROADMAP thinking ;)
we'd have a separate maven module for JUnit 4 support. Note that JUnit 5 support would eventually have to be another module as @ Rule support is going away :(
there would be another maven module for TestNG support. I don't have strong feelings about how the internals of this should be structured, but basically it looks like providing suitable base class(es) should be fairly idiomatic.
The degree of change probably warrants this being a 'v2.0.0' release

This diagram roughly illustrates my thinking (please excuse the UML ;)):

What do you think?

@rnorth looks good! I'm thinking about some refactorings before we start (especially in case of v2.0.0), will write them down to discuss a little bit later

I'd be very interested in a version of which does not include any JUnit dependencies. I'm currently developing a Docker extension for Spock (https://github.com/kiview/spock-docker-extension) and I'm using the Groovy Docker Client (https://github.com/gesellix/docker-client).
Lot's of the feature I still need add to my extension are already available in testcontainers in a quite accessible and convenient manner. So I wonder how I could best integrate testconainters into my project, or help in modularising the current version of the library. As for the time being I'm still hesistant in integrating this project as long as this will include JUnit dependencies.

@kiview thanks for the comment - sorry for the delay in responding!
Yes, breaking the JUnit dependency is absolutely something I'd like to do, as you can see from this ticket.
It would probably be reasonable to do #240 at the same time; either as a separate JAR for DockerClientFactory, or make it a consciously public (exposed) member of the containercore lib. Personally I feel the latter would be reasonable. What do you think @bsideup?

@rnorth sounds good for me. I have some free time this week, so I can take this task if you don't mind :) I see two tasks here - one for "JUnit-free core" and another one "separate DockerClientFactory", so we can split the work. WDYT?

@bsideup sure, that'd be great! Does the diagram above make sense, or do you want to discuss any refinements?
DockerClientFactory would probably have to move into the new containercore JAR anyway, but yeah I'd imagine the tidy-up work to make DCF be a real reusable class can follow.

@rnorth it looks good, but I can't say exactly until I start :)
Also, looks like there is only one open external PR, so I think it's a good time to start 2.x

I'll try to explain my use cases when using this library in conjunction with the spock extension:

Convenient interface for creating and configuring containers
Ability to pause and resume containers during tests
Easy inspection of containers during tests (like getting IP)
Docker-Compose support

Bonus points for Windows support with named pipes ;)
Another thing which would be awesome would be the possibility to run my tests inside a container, so I can use features like docker networking to access my testcontainers. But this seems like another project ;) (Gradle plugin maybe?)

I'll be a bit easier to satisfy :p
I started working on JUnit5 extensions because it allows the user to choose in which order the executions will be executed (the order of the declaration).
My goal is very simple : have a convenient way to manage containers as an extension. A use case example is to start a database container before starting an ORM framework like hibernate or Spring data.
To do so, my current code just needs to be able to create a adapter with this simple interface : https://github.com/FaustXVI/junit5-docker/blob/master/src/main/java/com/github/junit5docker/DockerClientAdapter.java
Maybe this interface could be a starting point ? @kiview would it fits your needs (for now) ?

My currently used interface hast indeed quite similar methods:
https://github.com/kiview/spock-docker-extension/blob/master/src/main/groovy/com/groovycoder/spockdockerextension/DockerClientFacade.groovy
I even think about keeping this facade class and simply substitute the used docker library with testcontainers, since testcontainers already gives mehr a high level of abstraction. This would mean,  I would use testcontainers as a high level Docker client.

@rnorth @FaustXVI is pure JUnit5 support (without vintage engine) support on the roadmap? @Rule and @ClassRule are going away... is this likely to be a difficult problem to solve. Thanks!

Today I pushed an experiment with JUnit 5 to the branch:
https://github.com/testcontainers/testcontainers-java/tree/junit5/modules/junit5
Feedback is welcome :)

@bsideup when do you think TC2.0 will be releases? Has it already started?

P.S. personally, I use TC a lot, but I also start them with static blocks and not using the rules:
https://github.com/bsideup/testcontainers-microservices-example/blob/5d28ca32d29d9fd7f3101044a02ec2660b0678fd/src/test/java/org/testcontainers/example/demo/support/AbstractIntegrationTest.java#L38

@bsideup Nice solution, but what about stopping. Why you are not stopping the containers?

Testcontainers will stop it for you automatically after the tests :)
On Thu, 17 May 2018 at 13:30, Aleksandr Podkutin<EMAIL_ADDRESS>wrote:

P.S. personally, I use TC a lot, but I also start them with static blocks
and not using the rules:
https://github.com/bsideup/testcontainers-microservices-example/blob/5d28ca32d29d9fd7f3101044a02ec2660b0678fd/src/test/java/org/testcontainers/example/demo/support/AbstractIntegrationTest.java#L38
@bsideup https://github.com/bsideup Nice solution, but what about
stopping. Why you are not stopping the containers?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/testcontainers/testcontainers-java/issues/87#issuecomment-389835091,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABAIitGhLGvBtOhG78cyZUSKwSQnNZe7ks5tzV9bgaJpZM4HwB4A
.

It was my first thought, just wanted to confirm 😄
Thank you.

@colinmorelli I did something along the lines of below pattern, which seems to work reasonably well without reliance on ClassRule mechanism (pattern is likely relevant for use in non-junit test frameworks):
public class PostgresContainer {

  private static final PostgreSQLContainer container;

  static {
    container = new PostgreSQLContainer();
    if (!container.isRunning()) {
      container.start();
    }

    // explicit container stop on VM shutdown
    Runtime.getRuntime().addShutdownHook(new Thread(container::stop));
  }
  public static PostgreSQLContainer getContainer() {
    return container;
  }
}

Then you reference the static getContainer() method across your test suite whenever you need it, and it is an effective singleton for the test suite JVM in the usual case.

@tom-haines actually, !container.isRunning() will never give false, and the shutdown hook is not needed, Testcontainers will take care of it :)
So it is as simple as:
public class PostgresContainer {
  private static final PostgreSQLContainer container = new PostgreSQLContainer();

  static {
      container.start();
  }

  public static PostgreSQLContainer getContainer() {
    return container;
  }
}

@rnorth Was redirected here by #1525 - although this ticket focusses on better support for other test frameworks, I would love to see for it to be completely test framework agnostic.
I was thinking of the use case where you run testcontainers as the dependency within your spring-boot app in development. I just do not want to bother with (re)starting containers when starting my app.
The idea is is that starting mvn spring-boot:run -Pdev will start the containers where the dev profile contains the the testcontainers dependency.
<profiles>
	<profile>
		<id>dev</id>
		<dependencies>
			<dependency>
				<groupId>org.testcontainers</groupId>
				<artifactId>testcontainers</artifactId>
			</dependency>
		</dependencies>
	</profile>
</profiles>

This actually already works - but having all junit Test classes inside the application in the IDE just makes things confusing.
The maven profile is there to avoid having this dependency included in the final artifact (don't want this in production, right).
Adding an exclusion block for junit also is not a solution - it fails with a "weird" compile time error
[ERROR] /Users/slyoldfox/Code/file-manager-module-test/src/main/java/com/samples/filemanager/FileManagerTestApplication.java:[78,61] cannot access org.junit.rules.TestRule
[ERROR]   class file for org.junit.rules.TestRule not found
This is of course due to:
public class FailureDetectingExternalResource implements TestRule.
Ideally I see a wonderful use case for testcontainers (your should call it bootcontainers ;-)) - where it can be used outside test contexts to quickly boot containers from within the app.
Think spring-boot - for testcontainers ...

@slyoldfox see https://bsideup.github.io/posts/local_development_with_testcontainers/

@bsideup thanks for the link and interesting read - but how does work around the junit dependency being pulled in? I assume you pulled in testcontainers as a compile time dependency to make this work?
Your final artifact would then have testcontainers (and all their transitive dependencies like junit) in it?
I am trying to avoid having junit (and testcontainers) on the classpath in anything else than the "dev" profile.

@slyoldfox as described in the article, the test application is in testCompile scope (where Testcontainers) is available.

@bsideup thanks for the clarification! :-) While I understand the setup now - and for the sake of discussion, your setup might work but poses some possible issues if your TestApplication includes test dependencies that interfere in development. Since you are executing this in testCompile scope, all the test dependencies will be included in your classpath - which is exactly what I want to try and avoid (I don't want assertj and mockito doing all their magic for example - even though they would probably do nothing harmful).
For me, your Initializer class should be in the compile scope instead of the testCompile scope. And the test should re-use the Initializer class from the compile scope (so the way around). This allows or re-use of this class in third party modules - if you intend to reuse this class in other third party projects.
Further more this Initializer class could then be configured by Spring Boot using @ConditionalOnClass annotations and setup by an *AutoConfiger class (specified in spring.factories). Which would allow early initialization of the containers and there would be no need to modify the SpringApplication.run(DemoApplication.class, args); contract.
Anyway all this won't work until the JUnit classes are decoupled from testcontainers - or until you include junit in compile scope.
It would make it a powerful tool though - outside of testing.

I don't want assertj and mockito doing all their magic for example - even though they would probably do nothing harmful

when you run psvm-like process from the test scope, there won't be any automagically applied Mockito or AssertJ code

Further more this Initializer class could then be configured by Spring Boot using @ConditionalOnClass annotations and setup by an *AutoConfiger class (specified in spring.factories)

That's just another option how to do that. You're free to use any mechanism available in Spring Boot.

For me, your Initializer class should be in the compile scope instead of the testCompile scope

You're free to put it wherever you like. Just keep in mind that Testcontainers is a library for... well... testing, so there is no guarantee that it won't have test-focused dependencies.
I'd personally advice against mixing production and development code.