How to enable generateTestResultAttributes programatically using Java ?

TestNG Version
7.9.0
Expected behavior
Custom attributes should be present in the XML report.
Actual behavior
Custom attributes are not present in the XML report.
Is the issue reproducible on runner?

[x] Shell
[ ] Maven
[ ] Gradle
[ ] Ant
[ ] Eclipse
[x] IntelliJ
[ ] NetBeans

Test case sample
I have a setup that i will build my project as a Jar with dependencies and run the automated test cases with java commands like below. I used OpenJDK 21.
java \
-Diterations=$ITERATIONS \
-DconfigFilePath=$env/mac/mac_install.yml \
-DxmlSuiteFile=testng_tiny_install_app.xml \
-Ddataset=$env/mac/web_chrome_$env.properties \
-jar $WORKSPACE/target/kepler-client-test-automation-jar-with-dependencies.jar

I saw a comment there is a way using maven-sure-fire plugin to enable generateTestResultAttributes. But is there a way to enable the same programmatically without using maven-sure-fire plugin ?
After executions are completed the testng-results.xml file has no attributes.
Runner
private static boolean runSuites(
      List<XmlSuite> xmlSuites, Integer iterations, Integer intervalBetweenIterations) {
    log.info("Running Suites has started");
    // Main Runner
    TestNG testRunner;
    boolean hasFailure = false;

    // Start the runner
    testRunner = new TestNG();
    if (SystemUtil.getIsReportPortalEnabled()) {
      testRunner.addListener(new ReportPortalTestNGListener());
    }
    log.debug("Initiating a TestNG Runner");
    updateXmlSuiteIndex(xmlSuites);
    testRunner.setXmlSuites(xmlSuites);
    log.debug("TestNG Runner has started");
    testRunner.run();
    hasFailure |= testRunner.hasFailure();
    log.debug("TestNG Runner has ended");
    return hasFailure;
  }

  private static void updateXmlSuiteIndex(List<XmlSuite> xmlSuites) {
    xmlSuites.forEach(
        xmlSuite -> {
          if (xmlSuite.getChildSuites().isEmpty()) {
            xmlSuite.getParameters().put("index", IndexCounter.increment() + "");
            xmlSuite.setGuiceStage("DEVELOPMENT");
          } else {
            xmlSuite.getChildSuites().forEach(AutomatedTestingRunner::setIndex);
          }
        });
  }

Test Method
@Test
public void welcomeTesting() {
    if (OsHelper.IOS_PLATFORM
        && WaitHelper.isElementAppearedWithinTimeout(uiController, ftuePage.getAllowButton(), 5L)) {
      waitAssertTap(uiController, ftuePage.getAllowButton());
    }
    assertElementIsVisible(uiController, ftuePage.getWelcomeFtueDescription());
    ITestResult result = Reporter.getCurrentTestResult();
    result.setAttribute("test", "XRY-6");
  }

Please help me where exactly i have to set the generateTestResultAttributes flag to true, to make the attributes available in testng-results.xml file.
Thanks!
Contribution guidelines
Incase you plan to raise a pull request to fix this issue, please make sure you refer our Contributing section for detailed set of steps.

Assuming that we have created a jar and test-jar based on the instructions as explained in the official documentation you would basically get this done by using the below command
java -jar target/uber-testjar-demo-1.0-SNAPSHOT-test-jar-with-dependencies.jar \
-testjar target/uber-testjar-demo-1.0-SNAPSHOT-tests.jar \
-reporter org.testng.reporters.XMLReporter:generateTestResultAttributes=true


What is the name for this plugin at the NPM website ?

Or is it deliberate that this plugin can not be found by the search function at the NPM website ? :
https://www.npmjs.com/search?q=rollup-plugin-browsersync

Hi!
I think NPM search has been malfunctioning lately. That same URL you pasted works for me today. That is the right name.
Closing for now.

Hi,
Must have been. I'll check it out later today. :)


Submit to awesome upstream

I meant to send them a pull request a while ago, but I had to wait until this project was old enough.
Anyway, I would do this myself, but I have a quick question. Should we categorize this under Entertainment or Miscellaneous?

I've already submitted a pull request and filed it under Miscellaneous.
You can see it here. But no word from @sindresorhus regarding the pull request 😥

💥  💥  💥 We've been merged! Hip hip hooray! 💥  💥  💥


BUG: No affect both index and column(GH 13522)

[x] closes #13522
[x] tests added / passed
[x] passes git diff upstream/master | flake8 --diff
[x] whatsnew entry

Current coverage is 85.27% (diff: 100%)

Merging #14005 into master will increase coverage by <.01%

@@             master     #14005   diff @@
==========================================
  Files           139        139
  Lines         50211      50213     +2
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits          42819      42821     +2
  Misses         7392       7392
  Partials          0          0

Powered by Codecov. Last update 5f49638...285bb84

can you run an asv and see if anything changed signficantly. lgtm otherwise.

@jreback all benchmarks not significantly changed beside groupby.groupby_frame_singlekey_integer.time_sum. It ranges from 1 to 1.2

thanks!


[CLI] extension update command

This PR adds a cli command to update all extensions code --list-extensions , which works without opening VSCode as GUI.
This fixes the issue https://github.com/microsoft/vscode/issues/56578 .
For reference, this is the following of the ticket re-opening after the discussion in this issue https://github.com/microsoft/vscode/issues/196155#issuecomment-1825617895 .
This have been linted using yarn and tested with the provided test.sh script.
I also tested that with classic extensions, but due to difficulties to test, I have not tested any prerelease upgrades, and not tested with vscode server. For prerelease, as soon as I will have some outdated one in my test setup I will be able to update this comment accordingly.
I am also not a TypeScript developer so if I wrote someting not optimal in the language, don't hesitate to state it below.

Rust changes look ok 👍

I choosed the simplest solution and renamed the one in extensionManagementCLI.ts to CLIInstallExtensionInfo , but this could be the topic of a refactoring.

This is ok for me. Let me know once you have incorporated all changes.

I already incorporated all changes you requested, so if renaming as CLIInstallExtensionInfo works for you, from what I see it should be ready to merge

LGTM, can you please resolve the merge conflicts?

but prints a DepreciationWarning (probably inside the extensionManagementService.installGalleryExtensions function):

This is being tracked separately. Tagging this as verified based on above comment

Does this update all extensions? or only all the extensions for the specified profile (--profile) / the default if no profile is specified?

Profile should be passed to update for a specific profile, otherwise extensions from default profile are updated.


multiple admin-UIs behind port-forwarding / each logging out other sessions

I'm using multiple OPNsense-appliances (OS) which I manage in-band through a vpn-tunnel. To access them I connect through a port-forwarding on the vpn-endpoint on different ports.
                            ___________
to OS 1> ---[WAN-IP:1337]---| OS VPN  |===<to VPN-IP:443>===[OS 1]
to OS 2> ---[WAN-IP:1338]---| NATTING |===<to VPN-IP:443>===[OS 2]
to OS 3> ---[WAN-IP:1339]---|_________|===<to VPN-IP:443>===[OS 3]

I can access every appliance from the outside separately, but when I sign into the web-ui of appliance 2, i get signed out of appliance 1 and so on. I think this has to do with the session-cookies and the cookie-domain (WAN-IP of OS VPN), which is the same for all appliances.
Is there a way to change this behavior? Maybe use the appliance-hostname in the cookie-name to differentiate them?

You could work around the issue by setting up three hostnames for that IP and using a different one for each opnsense. Cookies are shared on the same domain regardless of port, but they aren't shared between different domains which point to the same IP.

@fichtner what about a appliance-specific cookie-name to distinguish between those appliance without the mentioned workaround using different hostnames?

I'm not sure and I can't take a look in the next couple of weeks. CC @AdSchellevis

It's not that important, more a nice-to-have feature ;-)

Maybe Nginx or HAProxy is worth a try?

Timeout.


Updates for proper handling of LatLong columns

In the previous Woodwork implementation using DataTables and DataColumns, any columns that were designates as LatLong columns would be reformatted and properly stored in a standard format. We need to implement this functionality in the new accessor approach as well.
Note that reformatting of a standalone series cannot be done in place, and may require the use of the helper function created in #554.

@thehomebrewnerd , putting this to top of dev backlog. Let me know if it should go somewhere else.

@tyler3991 This is something we will need for feature parity with Woodwork prior to the accessor implementation, but I feel like this is lower priority that many of the other issues, since it only impacts one very specific data type, which isn't as common as most others.

Ok, got it. Thanks @thehomebrewnerd.

This code can be adapted for this conversion:
    if _get_ltype_class(self.logical_type) == LatLong:
        # Reformat LatLong columns to be a length two tuple (or list for Koalas) of floats
        if dd and isinstance(self._series, dd.Series):
            name = self._series.name
            meta = (self._series, tuple([float, float]))
            self._series = self._series.apply(_reformat_to_latlong, meta=meta)
            self._series.name = name
        elif ks and isinstance(self._series, ks.Series):
            formatted_series = self._series.to_pandas().apply(_reformat_to_latlong, use_list=True)
            self._series = ks.from_pandas(formatted_series)
        else:
            self._series = self._series.apply(_reformat_to_latlong)

Closed by #598


Add support for specifying the runner using env-var.

Description
We want to support multiple runners in the future (like Jest, Cypress, etc), but to do this we have to be able to tell the test splitter which runner we want to use. We could be smart and infer this from the TEST_CMD, but for now let's just KISS and make it configurable via an env var called BUILDKITE_SPLITTER_TEST_RUNNER. If the runner is set to anything but rspec, the splitter will exit with 16 and appropriate message.
Changes

Add BUILDKITE_SPLITTER_TEST_RUNNER env var, default rspec.

Testing
Test splitter will still work

There's probably a bunch more tidy-up I can do WRT the shared interface or the runners, but this at least gets us a small step towards a jest-runner world.

We've got some stuff server side that hardcodes the runner that could be cleaned up after this is released and used by bk/bk/bk https://github.com/buildkite/buildkite/blob/main/app/models/test_splitting/test_plan/params_transformer.rb#L16

Oh nice, I might also update the test plan payload being sent to include the runner so we can remove that.

In fact I've just realised that splitter versions prior to the one that sends the runner in the payload will still need a fallback case, so that code would have to stay in bk/bk, but read from the params if present.

@DrJosh9000 @moskyb any chance one of you could give this a quick look to make sure I'm not doing anything I shouldn't be doing?

LGTM, my only suggestion would be to add a go generate check to the pipeline similar to https://github.com/buildkite/agent/blob/c714bbfc6a2f58d6bbb5f446fbf8bdf4933254c9/.buildkite/steps/check-code-committed.sh#L25-L35

Thanks @DrJosh9000, I'll do that now!


Needed Contribution, Issue and Pull Request Guidelines and Templates

Issue Description
Problem
The current issue templates in the repository are outdated and insufficient, causing confusion among contributors and hindering the smooth process of opening and resolving issues. The lack of detailed instructions and examples is impeding the quality of issue reports, making it difficult for contributors to provide necessary information.
Proposed Solution
We need to update the contribution guidelines and issue templates to establish a standardized approach for contributors. The guidelines should cover essential aspects such as PR format, code quality standards, commit message conventions, and general issue reporting guidelines.
Changes to be Documented

Contribution Guidelines:

Clarify the process for submitting issues and pull requests.
Specify the expected format for pull requests.
Define code formatting and quality standards.
Outline commit message format and conventions.

Issue Templates:

Provide detailed instructions for opening bug fixes, feature requests, and other types of issues.
Include examples of well-structured issue reports.
Address common questions and concerns in the templates.

Expected Benefits

Standardization:

Ensure that all contributors follow the same set of guidelines.
Facilitate a uniform and streamlined contribution process.

Clarity:

Reduce confusion by providing clear and comprehensive instructions.
Enhance the quality of issue reports and pull requests.

Efficiency:

Expedite issue resolution by improving the information provided in issue reports.
Streamline the review and merging process for pull requests.

Action Items

Collaborate on defining and documenting contribution guidelines.
Revise and enhance the existing issue templates.
Include examples and illustrations to aid contributors in understanding the guidelines.
Review and approve the updated guidelines and templates.

Yes, this can be resolved with three separate templates for pull request, feature_request, bug_fixes and along with a detailed contribution document explaining on how to make a pull request using the templates along with the guidelines to be followed to contribute to this open source project.

Thanks @anshiiiii  just what we were expecting.

Congratulations on Contribution!


Add a table of contents (TOC) to the changelog template

What does this PR change? What problem does it solve?
This PR adds a Table of Contents (TOC) to the changelog template for improved readability of the resulting CHANGELOG.md file. It does not do any change to the code of restic. Note that this change not only helps this repo, but also the calens repo as this changelog.tmpl file is referenced. The changes shown are already successfully used in production. Note that the change originates to the work of @micbar who recently updated calens to allow prereleases.
Note that the size of this change is really small but due to also committing the final CHANGELOG.md for immediate view when merged, the number of changes looks big.
For the TOC, following changes have been applied:

Add template code to automatically generate the TOC
Change the markdown headlines from === or --- (not linkable) to correct sections identifyers # (linkable).

Was the change previously discussed in an issue or on the forum?
Checklist

[x] I have read the contribution guidelines.
[x] I have enabled maintainer edits.
[ ] I have added tests for all code changes.
[ ] I have added documentation for relevant changes (in the manual).
[ ] There's a new file in changelog/unreleased/ that describes the changes for our users (see template).
[ ] I have run gofmt on the code in all commits.
[ ] All commit messages are formatted in the same style as the other commits in the repo.
[x] I'm done! This pull request is ready for review.

@MichaelEischer

Sorry for the delay and thanks for the TOC! I had to replace the PR with https://github.com/restic/restic/pull/4598 as some branch protection rule on Github prevented me from tweaking the PR.


[Bug]: DatePicker modal is not closed once the date is chosen

Tremor Version
3.4.0
Link to minimal reproduction
https://www.tremor.so/docs/components/daterangepicker
Steps to reproduce

Click on the single DatePicker,
Choose a new Date,

What is expected?
Once the date is chosen the modal with calendar should be closed.
What is actually happening?
The modal with calendar is still visible - it closes once the user clicks outside of it.
What browsers are you seeing the problem on?
No response
Any additional comments?
I was playing around with the new DatePicker right after Tremor update to v3.4.0

Thanks Bart, working on it!


Missing development files

Hello! There's not link nor is the lzss library used for compression included in the repo. It's impossible to contribute properly without it, and I am curious as to how this works in first place.

https://github.com/BrightYC/Cyan/commit/54b2155fe0c84e8cda5482228337b116b549c103


Adding the Hessian and Laplacian to the NURBS shapefunctions [laplacian_dev]

The PR adds seconds derivatives  to the NURBS basis.
This  includes non-uniform weights  and general non-affine mappings.

PR
Author
Editor
Reviewers
Assignment
Approval
Merge

https://github.com/mfem/mfem/pull/586
@IdoAkkerman
@tzanio
@kmittal2 + @psocratis
01/19/20
⌛due 02/09/20
⌛due 02/16/20

Lets wait on #513 and #476 before assigning this.

Something must have changed in the build system. Changing changelog broke it...

One travis job was hanging... I restarted it

@kmittal2 and @psocratis, reminder that the reviews for this PR are due in less than a week

Hi @IdoAkkerman , can we also test this new functionality using a 3D example?

That should work as well. I will add a 3D example.

It did not work!! But i was able to fix it.
Thank you for the suggestion.

Also I noticed that only the serial example has been updated. Can we use this new functionality in parallel? If yes, can you update ex1p? If not, can you please indicate in the changelog that the new functionality has been implemented for the serial version only.

ex1p and ex11p are 2 parallel examples. I believe they work.

There is still active discussion so I extended the review deadline by a week.

I fixed the memory leak. Which was a small thing in the NURBS class. Not super related in this PR.
But quite non-intrusive.

@IdoAkkerman the changes are looking good so far. Although I noticed that you changed the names of the examples from nurbs_ex1 to ex1 in https://github.com/mfem/mfem/pull/586/commits/c598010352910ec1146f5c1f016b3847ac757a6b and then back to nurbs_ex1 in https://github.com/mfem/mfem/pull/586/commits/f9235d610a58382d27b5943baf321e120461b1bf.

Yes, with ex1 cmake gave an error, ex it clashed with the ex1 in the examples directory.
That is why I changed it back.

@tzanio Comments given on or before Feb 13 I can reply to on the 14th. After that I will be on holiday for a week.

Yes, with ex1 cmake gave an error, ex it clashed with the ex1 in the examples directory.
That is why I changed it back.

Yes, but why do we need to do this in ex1.cpp, ex1p.cpp, and ex11p.cpp? e.g. https://github.com/mfem/mfem/pull/586/files?file-filters[]=.cpp#diff-d78ef2fc830434c027e1185d92b454bfR3

Yes, but why do we need to do this in ex1.cpp, ex1p.cpp, and ex11p.cpp? e.g. https://github.com/mfem/mfem/pull/586/files?file-filters[]=.cpp#diff-d78ef2fc830434c027e1185d92b454bfR3<

ex1,ex1p an ex11p are executable names that are all ready used in the examples directory.
CMAKE apperently does not allow an executable name to be used more then once, zo I had to use something else then  ex1,ex1p an ex11p. That is why I opted for the prefix nurbs_.

In my case the current CMakeLists.txt will provide executable with the name nurbs_ex1, nurbs_ex1p and nurbs_ex11p.

This is the error I get for commit c598010

CMake Error at config/cmake/modules/MfemCmakeUtilities.cmake:123 (add_executable):
add_executable cannot create target "ex1" because another target with the
same name already exists.  The existing target is an executable created in
source directory "/home/ido/Data/mfem/mfem_laplacian_dev/examples".  See
documentation for policy CMP0002 for more details.
Call Stack (most recent call first):
miniapps/nurbs/CMakeLists.txt:24 (add_mfem_miniapp)

Yes, I agree about the issue with CMake but it is not specific to this PR. Even in Master, we currently use nurbs_ex1 for the executable name with CMake, but the sample runs in ex1.cpp says ex1 and not nurbs_ex1.  Let's modify ex1.cpp, ex1p.cpp and ex11p.cpp back to what it used to be (https://github.com/mfem/mfem/pull/586/commits/c598010352910ec1146f5c1f016b3847ac757a6b), and we will address this inconsistency later if needed.

I don't see a problem? Or understand the suggested fix, for that matter.
What is wrong with the 'nurbs_' prefix?
Perhaps it is appropriate to rename the cpp files and also add the 'nurbs_' prefix.

Commit c598010  does not work!
Hence the change back to state before c598010 which is done by commit f9235d6.
So in my mind there are 3 options:

Keep things as is, with prefix and things are working.
Remove prefix, but then it does not work.
A halfway house were the prefix is only were needed, this leads to inconsistencies though.

If we agree with the nurbs prefix option 1 then there are no inconsistencies to deal with later!
So why not opt for that option.

What happens if you go to ex1.cpp and change nurbs_ex1 to ex1 in Lines 3-11 of ex1.cpp?

Given that they are all comment lines that would "work". However, the comments would be wrong.
The resuling executable with the current CMakelists.txt is nurbs_ex1, due to the uniqueness issue,
so the current lines 3-11 are the correct, given the current executable name.
I would prevere having correct  comments. When the cmake uniqueness issue is fixed (if ever?) it would be a small fix to also change the comments in the cpp.
Even if the uniqueness isue would be fixed would it be desirable to drop the nurbs_ prefix?
I personally would not really care.
Perhaps it is considered undesirable that the ex1.cpp leads to an nurbs_ex1 executable. This could also be fixed by renaming ex1.cpp into nurbs_ex1.cpp.

@psocratis thank you for the clarification.
I will correct the make route and make sure everything has the nurbs_ prefix.

Hi @IdoAkkerman ,
This is not what I meant. The example files and the sample runs should be named without the prefix as they were before. This is consistent with all other examples throughout the code and with the make build system everything is fine. The only conflict comes from cmake, and therefore the executable files from cmake have to include the prefix.
Thanks,
Socratis

@psocratis The only other miniapp subdir that currently has an example is performance. My suggestion would be to "fix" the naming in this miniapp, and add a prefix consistently. Instead of changing the nurbs miniapps back to a halfway-house construction.

Hi @IdoAkkerman,
I don't have a problem with this. @tzanio what do you think?
Socratis

I think this is OK as it is for now. @kmittal2 I merged it for testing, but please let me know if you still have any reservations.

It looks good to me.

The internal regression testing found the following issue. Can you please address?
nurbs_ex1.cpp: In member function ‘virtual void Diffusion2Integrator::AssembleElementMatrix(const mfem::FiniteElement&, mfem::ElementTransformation&, mfem::DenseMatrix&)’:
nurbs_ex1.cpp:62:12: warning: unused variable ‘square’ [-Wunused-variable]
       bool square = (dim == spaceDim);
            ^
nurbs_ex1p.cpp: In member function ‘virtual void Diffusion2Integrator::AssembleElementMatrix(const mfem::FiniteElement&, mfem::ElementTransformation&, mfem::DenseMatrix&)’:
nurbs_ex1p.cpp:76:12: warning: unused variable ‘square’ [-Wunused-variable]
       bool square = (dim == spaceDim);
            ^

Note: the name changes in this branch cause failures in the memcheck, baseline scripts on tux.
We will resolve these once merged in master.

The internal regression testing found the following issue. Can you please address?
nurbs_ex1.cpp: In member function ‘virtual void Diffusion2Integrator::AssembleElementMatrix(const mfem::FiniteElement&, mfem::ElementTransformation&, mfem::DenseMatrix&)’:
nurbs_ex1.cpp:62:12: warning: unused variable ‘square’ [-Wunused-variable]
       bool square = (dim == spaceDim);
            ^
nurbs_ex1p.cpp: In member function ‘virtual void Diffusion2Integrator::AssembleElementMatrix(const mfem::FiniteElement&, mfem::ElementTransformation&, mfem::DenseMatrix&)’:
nurbs_ex1p.cpp:76:12: warning: unused variable ‘square’ [-Wunused-variable]
       bool square = (dim == spaceDim);
            ^

@IdoAkkerman had mentioned he will be on travel so I fixed this and pushed the changes.

Re-merged in next for testing ...

Re-merged in next for testing ...

Re-merged in next for testing...

Re-merged in next for testing...


Create bounds function to get the bound of an array/matrix/tensor

Add an optional argument dim to get bound according to column or row.

Very good job, this is a huge amount to add! Thank you a lot. I went through it and there's no errors. I thought about reviewing it with some comments but ultimately decided I was more or so commenting on style differences which isn't really important.
The only thing that looks like a possible fix is changing the percentile's input parameter from 1 to 100 instead to 0 to 1, like the other functions. so instead of doing percentile(a, 50), we can do percentile(a, 0.5) which follows suit with the other function that uses a probability interval of 0 to 1.
Thank you


fix: panic when the default config file doesn't exist

I should have used with_context() instead of context(), or when formatting the context message the program will unwrap a harmless None. Sorry that I didn't get it right the first time.

no worries, can't get everything right always on the first go :) appreciate the PR, glad you caught it!


How to use Document store?

Hi,
repo: git clone https://github.com/orbitdb/orbit-db-cli.git
how to replicate:
orbitdb create docdata docstore
orbitdb put docdata TestWordDoc.doc
the error:
Error: Unexpected token T in JSON at position 0

Hi @frudas24, thanks for issue!
The docstore is mean to be used for "documents" which in orbit-db's case mean JSON data. Take a look at the demo commands and output for an example.
After creating a document database with:
orbitdb create demo docstore

Instead of adding a file to the document store, you add a JSON object (note that the address will be different, copy it from the output of the create command):
orbitdb put /orbitdb/QmcE2HMrDTL3SFV1rcB5GaJVSJFfpFsnfTM4qK8SRScjyj/demo "{\"_id\":1,\"name\":\"frudas24 was here\"}" --indexBy name

You can then search by the contents of the field used for indexing, eg. search from all name fields:
orbitdb search /orbitdb/QmcE2HMrDTL3SFV1rcB5GaJVSJFfpFsnfTM4qK8SRScjyj/demo frudas24

Hope this helps and don't hesitate to ask  if you need more help! 👍

ok thanks for the answer.


Support organization package

Replace '/' by '%2F' in package name for api call and next router
Fix #10

@tony-go Done :)


[Enhance]  Add support for online video and local video playback, making the extension's functions and functions more extensive and powerful.

Very, very good markdown editing extension!
After using it for a while, I found that the markdown editing experience is really great!
But there is a problem that has been unable to solve? It has been unable to play online videos and local videos in markdown files in the VSCode environment.
After using the excellent markdown editing extension provided by you, I look forward to adding support for online video and local video playback, so that the functions and functions of this extension will be more extensive and powerful. With the rapid development of media industries such as YouTube in recent years, almost all markdown editors so far support streaming media playback functions such as online videos and local videos. Only VSCode does not support streaming media playback functions such as online video and local video, which is very frustrating. Hope you can implement this feature further.
Thank you so much for such an excellent markdown editing extension! 👍😜
OS: Windows10
Version: VScode 1.81
Version: 1.9.1691170218

<video controls muted preload="auto" poster="https://cdn.ly.com/oBw8dOUV.jpg" src="F:\video\79be16f6b3177b45c-1920-1080.mp4">
</video>
`

Hello! Playing a video file has no audio and cannot maximize the video
Video information: .mp4 based on H264

Windows10
Latest version 1.9.1691779450

Hello! Sometimes this version loads into the video, sometimes it doesn't.
Windows10
Latest version 1.9.1691779450

@zhouxinghong thanks for raising the issue.
Currently Runme has not much control about how videos are rendered in VS Code. However, there might be still ways we can improve the behavior. Let me bring this to the team and get back to you.

Thank you very much! 😜👏
Replenish:
Video information cannot be rendered if the  tag attribute wraps, like this:

<video controls muted preload="auto"
poster="https://cdn.u.com/image/iAgMxQog.jpg" src="https://bb.amm.en.ga/video/ga001.mp4">
</video>

@christian-bromann Great development team:
Hello! Can this video playback function be implemented in the next version?

@zhouxinghong thanks for raising the issue.
Currently Runme has not much control about how videos are rendered in VS Code. However, there might be still ways we can improve the behavior. Let me bring this to the team and get back to you.

👍👍👍👍👍

Hello! Can you give priority to implementing this video playback function? We've been looking forward to it for a long time. . .

Currently Runme has not much control about how videos are rendered in VS Code. However, there might be still ways we can improve the behavior. Let me bring this to the team and get back to you.

Hello! Can you give priority to implementing this video playback function? We've been looking forward to it for a long time. . .😂


Add support for GetAddress (getaddress.io) postcode geocoding and lookups

Thanks for this, and sorry for the delay. I'm having trouble trying this out. Every Geocoder.search request seems to return an empty array. Possible reasons:

Request URL includes v2/uk when it looks from the docs like it should be location instead.
Words in the search query seem to get concatenated, without spaces, which might not be what you intended.

@alexreisner Thank you for your comments. You are right, the documentation has moved on. We are using the API as it previously worked, so for instance:
https://api.getaddress.io/v2/uk/M14AL?api-key=<key>
{
    "Latitude": 53.479854583740234,
    "Longitude": -2.238126277923584,
    "Addresses": [
        "Post Office, Unit F4, Piccadilly Plaza, , , Manchester, ",
        "Unit F4, Piccadilly Plaza, , , , Manchester, "
    ]
}

We remove the spaces for the postcodes, I presume that was required at some point, although it does not appear to matter now.
It seems that getAddress now want you to go through a two step process to get to the Latitude and Longitude, e.g.
https://api.getaddress.io/location/M1 4AL?api-key=<key>
{
    "suggestions": [
        {
            "location": "M1 4AL, Manchester",
            "url": "get-location/p_ODg0MjgyZGRkYjQzMzNkIDgyNDA0MyA0YzRmMmFlMzIyNWQ0NmM=",
            "id": "p_ODg0MjgyZGRkYjQzMzNkIDgyNDA0MyA0YzRmMmFlMzIyNWQ0NmM="
        }
    ]
}

and then use the provided `url`/`id` value in the next query

https://api.getaddress.io/get-location/p_ODg0MjgyZGRkYjQzMzNkIDgyNDA0MyA0YzRmMmFlMzIyNWQ0NmM=?api-key=<key>
{
    "latitude": 53.479854583740234,
    "longitude": -2.238126277923584,
    "area": "",
    "town_or_city": "Manchester",
    "country": "England",
    "county": "",
    "postcode": "M1 4AL",
    "outcode": "M1"
}

I think what this means for this PR is that I should close it. I don't think our solution is appropriate for the wider community and has fallen behind the currently documented API usage. For now we would prefer to stick with the single API request to get the Latitude and Longitude. If getAddress contacts us to advise us that this will change in the future then we will re-evaluate then.
My apologies for any effort you have expended on this.
All the best,
Mark

That all makes sense, and I agree.
No worries at all. Thanks for the interest in contributing.


error with openGL when installed ubuntu 20.04

bash ./unix_launch.sh
-- Found SFML 2.5.1 in /home/mytricker/opt/Graphics_skeleton-1.0/libraries/SFML/GCC/x64/lib/cmake/SFML
-- Found SFML 2.5.1 in /home/mytricker/opt/Graphics_skeleton-1.0/libraries/SFML/GCC/x64/lib/cmake/SFML
-- Configuring done
CMake Error at CMakeLists.txt:82 (target_link_libraries):
Target "App" links to:
OpenGL::OpenGL

but the target was not found.  Possible reasons include:
* There is a typo in the target name.
* A find_package call is missing for an IMPORTED target.
* An ALIAS target is missing.

CMake Error at libraries/CMakeLists.txt:64 (target_link_libraries):
Target "Libraries" links to:
OpenGL::OpenGL

but the target was not found.  Possible reasons include:
* There is a typo in the target name.
* A find_package call is missing for an IMPORTED target.
* An ALIAS target is missing.

-- Generating done
CMake Generate step failed.


Resolve KafkaHealthCheck from container (#6035)

Description
This PR addresses a memory leak in the Aspire.Confluent.Kafka component caused by the creation of a new KafkaHealthCheck instance on each health check execution and the lack of proper disposal. The fix ensures that KafkaHealthCheck is registered as a singleton in the ServiceCollection, allowing reuse of the same instance for health checks. This change prevents unnecessary object creation and ensures proper resource management by allowing disposal of the KafkaHealthCheck instance when needed.
Fixes #6035
Checklist

Is this feature complete?

[x] Yes. Ready to ship.
[ ] No. Follow-up changes expected.

Are you including unit tests for the changes and scenario tests if relevant?

[ ] Yes
[x] No

Did you add public API?

[ ] Yes

If yes, did you have an API Review for it?

[ ] Yes
[ ] No

Did you add <remarks /> and <code /> elements on your triple slash comments?

[ ] Yes
[ ] No

[x] No

Does the change make any security assumptions or guarantees?

[ ] Yes

If yes, have you done a threat model and had a security review?

[ ] Yes
[ ] No

[x] No

Does the change require an update in our Aspire docs?

[ ] Yes

Link to aspire-docs issue:

[x] No

Microsoft Reviewers: Open in CodeFlow

@dotnet-policy-service agree

Is there any way to test this?

Do we have a similar bad pattern being followed in other integrations?

Is there any way to test this?

I don't see any possibility to test it, but if you suggest something - I can implement it.

Do we have a similar bad pattern being followed in other integrations?

I found these:

Aspire.RabbitMQ.Client: https://github.com/dotnet/aspire/blob/6c48677f422978e9b67f56803bba587e3cce3efc/src/Components/Aspire.RabbitMQ.Client/AspireRabbitMQExtensions.cs#L132
Aspire.Seq: https://github.com/dotnet/aspire/blob/6c48677f422978e9b67f56803bba587e3cce3efc/src/Components/Aspire.Seq/AspireSeqExtensions.cs#L75

Any updates?

@radical @eerhardt can we proceed with the next step?

@mitchdenny - this same issue existing in the Hosting layer:
https://github.com/dotnet/aspire/blob/84b2dee614249729589f4d2d567850dd682e2474/src/Aspire.Hosting.Kafka/KafkaBuilderExtensions.cs#L57-L64
I'm going to merge this PR, but we should get an issue written up for the Hosting side as well.


Cleanup Comments for IM

Problem

Add necessary comments to explain the code in Engine::BuildAndSendSingleReportData
else if (apReadHandler->IsReadType() && !hasMoreChunks)
{
apReadHandler->Shutdown();
}
Answer: for read, we don't expect status response for last report

Nice cleanup for post V1.X


(bug): ng-upload filelist binding

Reproduction link
https://angular-y1cpfq.stackblitz.io
Steps to reproduce
Setup an nz-upload component.
Binding it to a fileList as per the docs.
Once the page has loaded, perhaps there are some files that need to be displayed that are loaded asynchronously.
Add these images into the fileList by pushing or recreating the
What is expected?
A picture-card nz-upload component should display images that have been added into the fileList.
What is actually happening?
Binding issue. Items added manually into fileList do not appear on the images for the component.

Environment
Info

ng-zorro-antd
7.5.1

Browser
Chrome 75

According to Angular documentation，developers should not use push or splice to change the data passed to nzFileList
  ngOnInit() {
    setTimeout(() => {
      console.log("adding new image");
      this.fileList = this.fileList.concat(
        {
          uid: -2,
          name: 'xxx.png',
          status: 'done',
          url: 'https://zos.alipayobjects.com/rmsportal/jkjgkEfvpUPVyRjUImniVslZfWPnJuuZ.png'
        }
      )
    }, 4000);
  }

Yes I can confirm the binding does work when using concat. Thank you for the information.


Bank Statement Import - supported formats are ??

On this page some supported file formats are mentioned. However, on the detailed pages ('Telebanking' and 'Import and Reconcile Bank Statements')  different formats are mentioned.
As far as I'm aware, the support for MT940 has been stopped several years ago….. (I think from NAV2013 onwards)

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

ID: 2371b020-c485-474a-913d-f1756d6a76ab
Version Independent ID: c6023945-8f68-1de5-d12a-f8587b8b72ba
Content: Dutch Electronic Banking - Business Central
Content Source: business-central/LocalFunctionality/Netherlands/dutch-electronic-banking.md
Service: dynamics365-business-central
GitHub Login: @SorenGP
Microsoft Alias: sgroespe

Hello,
Thank you for your feedback! Our apologies for the very long wait for an answer!
A member of our team has at long last reviewed your feedback. The article in question was updated to refer to the data exchange framework since original component was indeed removed.


Safari/iOS build bug

Current situation:
Safari/iOS error after npm run build

Current findings:

Different index.html outputs after using a "custom" buildConfig with the same build options in plugins (using the babel-minify-webpack-plugin) section:

new BabelMinifyPlugin(
  { evaluate: false, mangle: false, propertyLiterals: false },
  { comments: false }
)

npm run build output:
hops624-index-html.txt
custombuildrc-index-html.txt
custom build config:
const baseBuildConfig = Object.assign({}, require('hops-config/configs/build'));
const BabelMinifyPlugin = require('babel-minify-webpack-plugin');

baseBuildConfig.plugins = baseBuildConfig.plugins
  .filter(x => x.constructor.name !== 'BabelMinifyPlugin')
  .concat([
    new BabelMinifyPlugin(
      { evaluate: false, mangle: false, propertyLiterals: false },
      { comments: false }
    )
  ])

module.exports = baseBuildConfig;

@MikeBild thanks for reporting this and for your earlier PR! This issue is unfortunately rather hard to reproduce... I went ahead and diffed the two HTML files you provided. I didn't actually see any differences except those caused by disabling mangling:
git diff --word-diff=color --word-diff-regex=. hops624-index-html.txt custombuildrc-index-html.txt'
In your original pull request, you only deactivated the property literal transform, but left the mangle names plugin active. So, could it be that that's the actual culprit? If so, we could try tweaking the mangling plugin config instead of disabling it altogether.
Anyway, I'd love to find out where that stray 蕄 (Han, \u8544) character is coming from. Understanding that might help us tweak the minifier plugin config. It does not appear to be in the HTML files. So, can you help me out in this regard?

Hi,
"In your original pull request, you only deactivated the property literal transform, but left the mangle names plugin active."
I deactivate mangle and propertyLiterals on both sides. (to be exact by patching node_modules/hops-config/configs/build.js). That's one of the issue. It seems to be, that the modified minifier options in build.js are not correctly applied to the "standard/base " hops build process.

This smells like an issue with multiple versions of hops-config in your project, then. Did you already try deleting your node_modules folder and your yarn.lock or package-lock.json files?

Ahh yes, good point. I figure that out.

@MikeBild is there anything we need to about this in Hops? Would you mind closing this issue otherwise?

@dmbch Clean up all dependencies made me happy. Thx!


Action adds index query param indefinitely  -> ?index&index&index&index

What version of Remix are you using?
1.6.8
Steps to Reproduce
A simple <Form method='post'> on index route adds  index query param for each action taken resulting in ?index&index&index&index... after 3697 was merged.
This can be mitigated by adding action='.', but then query params are not saved.
Expected Behavior
No index param added or added only once.
Actual Behavior
index param added for each action call.

I'm seeing the same behaviour with useSubmit in latest version update 1.6.8 https://stackblitz.com/edit/node-v3byq1?file=app/routes/index.tsx

Should be fixed by #4000, which should go out in 1.6.9 soon 👍


closes #533 Please try again on Romana

Deploy preview for landscape ready!
Built with commit da17481d6e7e077ca69c6c577ff769972e807b1b
https://deploy-preview-537--landscape.netlify.com


test: Allow user to specify which transport to use

Currently every transport is used to run each functional test.
This PR refactors to use only one transport per test case.
Also allows user to export TKTRANSPORT env variable to specify which transport to use.
Resolves #153

Needed to resolve these conflicts ran git pull --rebase upstream master .

Will need to fixup conflicts again.


Issue after we select with ngf-select

i see this error: [$parse:lval] http://errors.angularjs.org/1.5.0-rc.2/$parse/lval

Create a jsfiddle with steps to reproduce.


extend certificates with 100Y (#3422)

Pull request opened by the merge tool on behalf of #3422

Merged into branches iso3 in 7c8a2def170f26f66efe9d3f9427f456dd3d0cc2


Add View > Full Screen menu item

ref: #933 #1304
This PR adds a View > Full Screen menu item. On MacOS the shortcut is ^+⌘+F and on Win/Lunix it's F11. I needed to add some CSS tweaks for the tab appearance.
This PR doesn't deal with hiding/showing the toolbar in full screen mode. @pfrazee happy to look at setting that up in this PR or a future one.

Thanks for the PR! I just reworked the UI of Beaker, which is why I let this sit. I'll get it merged soon.

Okay, just added with 966b504617e8739cd3f48f05aabe9dbb9cc352e2
Thanks!


Allow passing buffer instead of path

I would be open to implementing this myself and sending a PR, would you mind pointing me in some direction?

In the node/src/lib.rs
(build it by running "yarn build" in node/)
This needs to be changed to
#[napi]
pub fn parse_header(path: String) -> Result<Value> {
  let file = File::open(path.clone())?;
  let mmap = unsafe { MmapOptions::new().map(&file)? };
}

Something like this:
use napi::Either;

#[napi]
pub fn parse_header(path_or_buf: Either<String, Buffer>) -> Result<Value> {

 let bytes = match path_or_buf {
    Either::A(path) => {
      let file = File::open(path.clone())?;
      let mmap = unsafe { MmapOptions::new().map(&file)? };
      BytesVariant::Mmap(mmap)
    }
    Either::B(buf) => BytesVariant::Vec(buf.into()),
  };

Might want to create a new function for this so that you dont need to copypaste it to every function.

Sorry if you are still working on this. I added it in 0.14.0:
https://github.com/LaihoE/demoparser/commit/477340a9b64cad3ec8d2401ec97465e726163581

Assume this is implemented now, re-open/new issue if something regarding this

@LaihoE Did you forget to regenerate typescript definitions?
https://github.com/LaihoE/demoparser/blob/af896248e401fe7b046d179d606f064acdc01e97/src/node/index.d.ts#L11


IAF layer is not expected to have any parameters. But tau_mem is listed as a parameter of this layer.

It seems like this is a side effect of IAF inheriting from LIF.
This seems like an unnecessary complication.
It also seems like this would increase the computational cost for IAF forward pass.
It would be useful to do some profiling/speed tests.

agreed, tau_mem is essentially infinite in IAF so as to re-use a maximum of the LIF implementation. You're right that tau_mem shouldn't be a parameter. I will look into how this can be prevented

I have a branch where I re-implemented the neuron to not have dependence on LIF. (Sort of going back).
Will have to run some bench marks on performance/speed.

great stuff. Happy to run this on the MNIST baseline benchmark I implemented the other day

Fixed with PR #34


Date and time limits

add props to min and max time
add props to min and max date

This is already possible by using the datePickerProps and timePickerProps attributes.


IBM Fix

Description
This is a simple fix on the pre_process checker and documentation to ensure that more than one immersed boundaries can be used properly. Please note that this is not a general fix. Boundaries immersed in the interface of two patches with different initial primitive variables are not allowed.
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)

Scope

[x] This PR comprises a set of related changes with a common goal

If you cannot check the above box, please split your PR into multiple PRs that each have a common goal.
How Has This Been Tested?
Adding one regular patch for the 2nd immersed circle boundary with an inflow.

[x] Two immersed 2D circle with inflow

https://github.com/MFlowCode/MFC/assets/98496194/d1e94475-736d-45f8-96b9-0c3d27047e0b
Test Configuration:

What computers and compilers did you use to test this: macOS and GCC

Checklist

[x] I have added comments for the new code
[x] I have made corresponding changes to the documentation (docs/)
[x] I ran ./mfc.sh format before committing my code
[x] New and existing tests pass locally with my changes, including with GPU capability enabled (both NVIDIA hardware with NVHPC compilers and AMD hardware with CRAY compilers) and disabled
[x] This PR does not introduce any repeated code (it follows the DRY principle)
[x] I cannot think of a way to condense this code and reduce any introduced additional line count

with this change, how can one have 2 IBs but 1 patch?

@sbryngelson ib_markers were defined inside s_assign_patch_primitive_variables previously, and that is not necessary. Ib_markers are now defined in patches. This avoids the assignment of primitive variables to the ib patches again and allows using more than one ib's with less or one regular patches.

Looks good. @anandrdbz can you review this code?

I think this looks good, one last thing that you might need to do once (and this isn't being done currently even in master), is to apply the immersed BC once at the start of the simulation. Right now for most of our cases,, this is not a problem, as the patch in which immersed boundary is placed is uniform. However, if the patch is non-uniform we could potentially run into some issues at the first time step

But this can also be done in a separate PR

But this can also be done in a separate PR

Thanks @anandrdbz. I agree that making a separate PR to attack this non-uniform issue will be easier as it will involve changes to src/simulation.


Unknown argument: save. Why --save doesn't work?

From here

--save, -s | Recompiles on save regardless of changes

In real life:

Unknown argument: save

My CLI command. Why --save doesn't work?
NODE_ENV='development' node_modules/.bin/webpack --config webpack.conf.js --debug --progress --display-depth --colors --display-chunks --watch --save

The option has been removed in #3896 but I don't know why.

removed from documentation


Better repo code for handling content objects.

The media.fileops.repo.Repo object is meant to be an interface into a repository where the files are loaded and then turned into content object instances.
The media.tools.common package contains code that iterates through the media objects and returns all movies.  Actually, it just returns all content objects, with no regard to the actual class.
https://github.com/cjcodeproj/medialibrary/blob/bffac778893ee617a6b84e01e4579ccc24d4cd89/src/media/tools/common.py#L46-L53
The repo code should be updated so it maintains an internal array of just content objects, and it should include methods that allow for grabbing specific content objects based on the type of content.

Additional notes:
The Repo() class is pretty barebones, because it just keeps a list of all the filenames; the Loader() class does the reading/object allocation work.  The proposed re-factoring means the repository class will have an array containing media objects, and a second array that has copies of all content objects.  It will also have built in methods that will return
content objects that match certain classes, such as get_movies().

Commit:
https://github.com/cjcodeproj/medialibrary/commit/3c1d10ff120c3898b23b0919c625e4d6f11072c8
Notes:

Lots of code refactoring for the proof of concept tools.
The TOOLS.md document has a cleaner example of using the Python shell to extract film information.


Clang fixes when building without -ffreestanding

PR #54638 proposes eliminating -ffreestanding when building with a C library other than the minimal C library. When building in this mode, clang finds a number of things to complain about in the code, including calls to abs with unsigned/float values and inappropriate types to printf.
This series also includes a C language conformance fix -- code using labels on declarations, which is not valid.

This depends on pr #56489

This depends on pr #56489

It's merged now

@keith-packard could you update this PR to remove the change to west.yml which already got in?

@keith-packard could you update this PR to remove the change to west.yml which already got in?

done.


[Conductor] tool points to incorrect build link

Upon reaching the VERIFY_RELEASE stage of the conductor tool, the output of the CLI points to https://ci.chromium.org/p/flutter/g/packaging/console which is the incorrect console link.
Link should be https://ci.chromium.org/p/flutter/g/stable_packaging/console.

cc @godofredoc @christopherfujino


Page should have proper, meaningful error pages

The website should return good looking, helpful error messages when encountering 403s and 404s instead of nothing. Maybe include a DDG search field or similar pre-populated with elements from the URL.
E.g. when you click on development guide which is linked from here, all you get is a blank page.

The 404 page is nice https://elementary.io/404
The 403 page is not https://elementary.io/_templates

Well development guide returns 404 header but not the nice 404 page. So there is probably more to this issue.

I am going to split off the 403 page into a separate issue, and make this just about 404 pages not loading when not in English. I'll likely need @emersion's help on this one.


Add ocpMajorVersion field

Description
Based on the k8s-netperf logic. Field name is different though. Shall we rename it?

Looks good to me apart from the existing comments.


Allow 204 as return code for /exec/:id/start

If used with the 'Detach: true' options, the /exec/:id/start endpoint returns 204 (No Content) rather than 200. Dockerode should treat this as a success rather an unhandled error.

:+1:
Will ping as soon as it is published.

Published on v2.1.3


Config to reference chunked bundle into one partial.

Hi, thanks for the plugin, I am generating a hashed bundle an would like to reference all the chunk into one .cshtml.
For example in main.cshtml I would have :
<script src="~/App/dist/npm.babel-runtime.0020b34e.js"></script>
<script src="~/App/dist/npm.add-px-to-style.76deca54.js"></script>
<script src="~/App/dist/Content.b0e5d7aa.js"></script>
<script type="text/javascript" src="/App/dist/main.047d6b4c.js"></script>

Ordering or scripts would be based on the dependency tree.
Do you have any example of how I can achieve that?

Please elaborate a bit more, why creating a single .cshtml helps you. I want to understand your needs a bit better. So we're on the same page: The main idea with the plugin is to generate asset specific partial views. Mostly, these partial views would be linked into your layout file. For more dynamic loading, I'd rather recommend webpack's code splitting.
For a solution example:
With the example you gave, you could use the following config, which would result in four generated views, which you link into your master page (or another partial view):
rules: [{
  // regex match asset filename(s)
  test: /.*\.js$/
}],

If you want to be able to adapt the name of the generated partial views, I'd recommend trying to the following config:
rules: [{
  // regex match asset filename(s)
  test: /.*\.js$/,
  output: {
   name: function (defaultName) { return defaultName.split(".")[0] }
  }
}],

Closing due to inactivity.


Responsive design

We are looking for a contributor who is willing to make the new website responsive.
NOTE: Please work on the beta branch

I am very first time here, and want to make your website responsive, let me know the way how can i do so!😅

I am very first time here, and want to make your website responsive, let me know the way how can i do so!😅

Hello:)
You can follow these steps:

Fork the repo
NOTE: Uncheck the Copy the main branch only. option
Clone it
Intall git to your machine and run these commands:

git clone https://github.com/Bashamega/WebDevTools.git
cd WebDevTools
git checkout main

Install dependencies:
Install node and npm, then run this:

npm i

Open the server by running:

npm run dev

Modifying the code then pushing to your fork repo then open a pr

@Bashamega As the issue is not assigned to anyone; I would like to work on this issue!! Can you assign me this issue?

I've started working on this issue.

Hey @Bashamega I think you should close this issue; As it is already completed.

Hey @Bashamega I think you should close this issue; As it is already completed.

I forgot sorry. Thanks


MAINT: improve doc and add alternative keyword to wilcoxon in scipy.stats

Closes #9046

+1 looks like a nice improvement!

CircleCI failure is not related:
FAILED scipy/io/matlab/tests/test_streams.py::TestZlibInputStream::test_seek

This is looking pretty good. Just a couple of comments left that you replied to - @pvanmulbregt if you're happy with the replies to your two open comments, can you resolve them?

This all looks good, all comments addressed and resolved. So merging. Thanks @chrisb83 and all reviewers!

Does anyone have access to the following articles:
Cureton, Edward E. 1967. “The Normal Approximation to the Signed-Rank Sampling Distribution When Zero Differences Are Present.” Journal of the American Statistical Association 62 (319): 1068–69. doi:10.2307/2283694.
[3] Pratt, J.W., Remarks on Zeros and Ties in the Wilcoxon Signed Rank Procedures
https://www.tandfonline.com/doi/abs/10.1080/01621459.1959.10501526
If so, could you send them to me? I would like to take a look in order to close #6805. Thanks!

I have the articles now.


WSL2 + DockerDesktop doesn't work - Remote procedure call failed.

Environment
Microsoft Windows [version 10.0.20215.1000]
Your Distribution version: Ubuntu 20.04
Whether the issue is on WSL 2 and/or WSL 1: 2
Steps to reproduce
Install WSL 2 with Ubuntu 20.04.
Since the last update offered by Windows Insider, neither wsl 2 nor docker starts. Bash.exe doesn't work too.
Before the update, everything worked.
WSL logs:
Docker crash while starting: https://hastebin.com/nayanoyebo.sql
Ubuntu crash while running & installing: https://hastebin.com/gutogihule.sql
Expected behavior
That WSL works as well as Docker or Bash.exe
Actual behavior
I receive the following message : 'Remote procedure call failed.'

This is known issue, please update windows to 20226.

Use get.docker.com install script, I think it is the better alternative than docker desktop

/dupe #5263


[fix] transmission: allow from_task with multiple servers

Motivation for changes:
More that one instance of transmission was not possible to use in plugin from_task
Detailed changes:

Save and restore preview transmission instance

Addressed issues:

Fixes #3331

Hmm. This is why we don't like __init__s or storing things on self in the plugins. I wonder if there is a better way... Perhaps it gets stored on task? task._transmission_client = Whatever()?

I suppose both of these solutions break if you do something like:
inputs:
- from_transmission: hostA
- from_transmission: hostB

Why are we even storing client instance like this? Shouldn't it just be created and closed in task_input (for from_transmission) and task_output (for regular transmission?)

I think that the main reason is to reuse the same connection if you use input and output or download (or all of them) in the same task. But now that I see it, like this you can't input from one server and output from another!
I also don't like to do this in the task start. One better way would be to start in the stages, and stor it in the task, and one each starge check if the config is the same, and if yes refuse, if not connect again.

I tried to re-trigger the checks, but something seams wrong! Locally was ok.
I changed the code to get get the client connection on every stage and store it to latter check if can be reused or make a new connection.

Thanks for this! Went with #3359 instead, as it seemed a bit simpler.


Revert "Serialize to indexdb"

Reverts captbaritone/webamp#658

FYI @durasj I reverted this. Loading webamp.org had a few problems: windows were all crammed in the top right corner (must not have tested without using saved state).
Also, the social links could not be clicked.

I'll probably wait until after the Internet Archive announcement to try merging this again.


Add a database to project

Docker compose with postgres should be added before
Add some library for migrations
JOOQ to use database from spring application
JOOQ classes should be generated automatically
Write some dummy code at layers api-service-repository to test and verify project structure
Use Test Containers for testing

I used Liquibase for these needs at taparia-jakarta

Also there could be some difficulties with generated sources directory. Maybe we need to split it into generated/jooq, generated/oapi.

The pleasant outcome will be that we will need to write only domain/model/*.kt, as api/http/message/*.ktand persistence/jdbc/entity will be generated for us automatically.

I propose such project structure:
monolith
  api/http
    message // generated
      <...>Message.kt (~)
    endpoint
      <...>Api.kt // implements generated
    error
      <...>ExceptionMapping.kt
  logic
    <...>Service.kt
   logged/<...>Service.kt
   metered/<...>Service.kt
  storage
    <...>Storage.kt
    jdbc
      PostgresConfig.kt
      HikariDataSource.kt
      LiquibaseMigration.kt
      Jdbc<...>Storage.kt
  domain/model
     <...>.kt
  config/Config.kt
  monitoring
    logging
      netty/NettyLogger.kt // implements Logger
      Logger.kt
    metrics
      prometheus // idea for future
        PrometheusCounter.kt implements Counter
      Counter.kt


Markdown  Render Question

Hi, I encountered a strange situation, This is the smallest reproducible code:

<!DOCTYPE html><script<EMAIL_ADDRESS>
``` bash
$ ls
Makefile
```

#### this is a title

``` bash
$ ls
```

This is the render effect:

Is this a normal situation?

If delete the Symbol $ in the first code block， the render effect seems normal.
<!DOCTYPE html><script<EMAIL_ADDRESS>
``` bash
ls
Makefile
```

#### this is a title

``` bash
$ ls
```

Please try this:
<!DOCTYPE html><script<EMAIL_ADDRESS>
\begin{md}
``` bash
$ ls
Makefile
```
\end{md}

#### this is a title

\begin{md}
``` bash
$ ls
```
\end{md}

See Markdown Priority Environment for more details.

Hi @LiuYinCarl, I am glad to know that you find this project useful. The "Used by" list seems to be automatically generated by GitHub. It appears to be checking the package.json file in a project to see if a project uses texme. If there is texme in the package.json file of a project, then GitHub appears to be automatically listing that project under the "Used by" section.

Seems it's a feature just for npm and pip project but not Golang. But still thanks for your outstanding work.


The scenario table takes up too much memory

Describe the bug
step("hi", function() {
  console.log("hi");
});

# test
## test 1
table:specs/5000.csv
* hi

The file specs/5000.csv is a csv file with 5000 lines.
num
1
2
...
5000

After executing this specification, the final memory usage reaches 11GBytes; of which gauge is 6G and html-report is 5G.
To Reproduce
Steps (or project) to reproduce the behavior:

Initialise a gauge project
Run the gauge command
See error

$ gauge init js
$ gauge run specs

Logs
Paste any log or error messages here

Expected behavior
A clear and concise description of what you expect to happen.
Screenshots
If applicable, add screenshots to help explain your problem.

Versions:

gauge -v

Windows 10
Gauge version: 1.4.3
Plugins
go (0.2.0)
html-report (4.1.2)
js (4.0.0)
screenshot (0.1.0)
Additional context
Add any other context about the problem here.


Broken on IOS 16

Tab bar and navbar are out of bounds.

Tab bar and navbar are out of bounds.

It would be helpful if you provided a screenshot


Control 9: Summary

Review messaging to make it clearer that title is not a primary way to label a form control

Change working to "Verify title is intended label"
Change the definition to start with:
If title attribute is serving as a label, verify the title is intended....

Updated messaging for CONTROL_9 to make it clearer title is not a preferred labeling technique
DEFINITION: 'Verify the @title@ attribute is serving as an appropriate label, and not a tooltip.',
SUMMARY:    'Verify @title@ is the label and not a tooltip',


Improve overflow errors

Is your feature request related to a problem or challenge?
In downstream crates such as Comet, we don't have a good way to catch overflow errors from DataFusion/Arrow. We have to rely on looking for certain words in error messages. For example:
match self.inner_abs_func.invoke(args) {
    Ok(result) => Ok(result),
    Err(DataFusionError::ArrowError(ArrowError::ComputeError(msg), trace)) if msg.contains("overflow") => { ... }

We then create a specific overflow exception:
CometError::ArithmeticOverflow {
    from_type: from_type.to_string(),
}

Describe the solution you'd like
It would be a better developer experience if DataFusion/Arrow had specific overflow errors. I see that Arrow already has DivideByZero so it seems logical that we could also have an Overflow or ArithmeticOverflow as well. I will file an issue in Arrow and link to it from this issue.
Describe alternatives you've considered
No response
Additional context
No response

I created a PR against arrow-rs to add this new error variant:
https://github.com/apache/arrow-rs/pull/5845


Pass new read_by_sender flag when sending messages

See https://zulip.com/api/send-message#parameter-read_by_sender and https://chat.zulip.org/#narrow/stream/378-api-design/topic/read_by_sender.20flag.20on.20send-message/near/1702407 for details.

This was actually already done in #456 (as the fix for #440). I see we didn't mention that in the #api design thread — mentioning it there would have been good.


SSECustomerKey is str, but AWS docs show using bytes

Describe the bug
the SSECustomerKey arg (for various operations) is type str, but various AWS docs very clearly show passing in a bytes. This is a binary value so I believe bytes is "more" correct.
Example docs: https://boto3.amazonaws.com/v1/documentation/api/latest/reference/services/s3.html#uploading-downloading-files-using-sse-customer-keys
sample from above link:
import boto3
import os

BUCKET = 'amzn-s3-demo-bucket'
KEY = os.urandom(32)
s3 = boto3.client('s3')

print("Uploading S3 object with SSE-C")
s3.put_object(Bucket=BUCKET,
              Key='encrypt-key',
              Body=b'foobar',
              SSECustomerKey=KEY,
              SSECustomerAlgorithm='AES256')
print("Done")

# Getting the object:
print("Getting S3 object...")
# Note how we're using the same ``KEY`` we
# created earlier.
response = s3.get_object(Bucket=BUCKET,
                         Key='encrypt-key',
                         SSECustomerKey=KEY,
                         SSECustomerAlgorithm='AES256')
print("Done, response body:")
print(response['Body'].read())

To Reproduce
import boto3
import os

key = os.urandom(32)

s3 = boto3.client('s3')

s3.get_object(Bucket='sombucket', Key='somekey' SSECustomerAlgorithm='AES256', SSECustomerKey=key)

key will have a red underline, Pyright gives me this:
Pyright: Argument of type "bytes" cannot be assigned to parameter "SSECustomerKey" of type "str" in function "head_object"
  "bytes" is not assignable to "str"

Additional context
MacOS: 14.6.1
Boto3: 1.35.74
types-boto3[essential]: 1.35.74

Hello. Thanks for the bug report!
It looks like SSECustomerKey argument is mutated by boto3. I can overrride its type to bytes, but I need to cross-check how it works.

As you see, boto3 documentation also refers to it as str in docs. E.g.: https://boto3.amazonaws.com/v1/documentation/api/latest/reference/services/s3/client/get_object.html . THisis clearly a bug, because docuemtnation is also generated using botocore shapes, same as my type annotations.
I am pretty sure there is some code in boto3 itself that mutates this argument.

it's very willing to accept a str as well, but when I have raw bytes converting it involves making an encoding decision and idk what boto3 expects.
IMO I think overriding this to be bytes | str is probably best? You're almost certainly correct about boto3 doing mutation internally.

Fixed in main.

SSECustomerKey and CopySourceSSECustomerKey arguments accept str | bytes
SSECustomerKeyMD5 and CopySourceSSECustomerKeyMD5 arguments are removed because boto3 calculates them internally

You can help me by testing unreleased changes:
# install uv: https://docs.astral.sh/uv/getting-started/installation/

# build types-boto3-s3 package in ./typings directory
uvx --from git+https://github.com/youtype/mypy_boto3_builder@main mypy_boto3_builder ./typings --product types-boto3-services -s s3 --output-type wheel

# install generated package
pip install typings/types_boto3_s3*

# run type checker on your codebase

Please test if you have time and let me know if it works as expected.

I'm not sure I did this right, but no it's still failing:
$ uvx --from git+https://github.com/youtype/mypy_boto3_builder@main mypy_boto3_builder ./vendored --product types-boto3-services -s s3 --output-type wheel
 Updated https://github.com/youtype/mypy_boto3_builder (ff90cfba)
13:03:20 mypy_boto3_builder: INFO    Generating types-boto3-services product
13:03:20 mypy_boto3_builder: WARNING boto3 is not installed, using botocore version: 1.35.74
13:03:20 mypy_boto3_builder: INFO    [1/1] Generating types-boto3-s3 1.35.74.post1
$ uv pip install vendored/types_boto3*
Resolved 1 package in 1ms
Prepared 1 package in 4ms
Uninstalled 1 package in 3ms
Installed 1 package in 1ms
 ~ types-boto3-s3==1.35.74.post1 (from file:///Users/travisjohnson/code/whelp/vendored/types_boto3_s3-1.35.74.post1-py3-none-any.whl)
$ uv run pyright main.py
/Users/travisjohnson/code/whelp/main.py
  /Users/travisjohnson/code/whelp/main.py:28:24 - error: Argument of type "bytes" cannot be assigned to parameter "SSECustomerKey" of type "str" in function "head_object"
    "bytes" is not assignable to "str" (reportArgumentType)
1 error, 0 warnings, 0 informations
$

I have no system-wide pip

Hmm, you did everything right. As I see, the pacakge has been successfully installed in your venv. However, I do not see that type checking for client.head_boject method has changed. It might be pyright cache, or you are also using boto3-stubs (please uninstall it if you do, use types-boto3 instead).
Also, you can check HeadObjectRequestRequestTypeDef in .venv/lib/python3.13/site-packages/types_boto3_s3/type_defs.pyi and make sure that SSECustomerKey: NotRequired[Union[str, bytes]]. Let me know if it is still SSECustomerKey: NotRequired[str] for some reason.
I tested the change locally, it should work. I am going to release updated package tomorrow.

my bad! I posted the issue for boto3, but I had changed the actual thing I wrote to use aiobotocore (I believe they both get their types from here, so this should fix both), when I did the simplified example it worked! 👍🏻

(actually I got this error first, but I just pip installed the py3 one and everything was golden)
$ uv pip install vendored/types_boto3*
error: Requirements contain conflicting URLs for package `types-boto3-s3`:
- vendored/types_boto3_s3-1.35.74.post1-py3-none-any.whl
- vendored/types_boto3_s3-1.35.76-py3-none-any.whl

Good! You can also generate type annotations for aiobotocore/aioboto3:
# aiobotocore
uvx --from git+https://github.com/youtype/mypy_boto3_builder@main mypy_boto3_builder ./vendored --product aiobotocore-services -s s3

pip install vendored/types_aiobotocore_s3*

Type annotatiosn for aio libraries are different, so it is not safe to use types-boto3 when working with botocore

Fixed in the latest builder release. Also, I released packages with the fix included:

mypy-boto3-s3 1.35.76
types-boto3-s3 1.35.76
types-aiobotocore-s3 2.15.2.post1

Please let me know if it works as expected now.

beautiful! Thank you!
I'm not sure it's related, but I noticed that types-aiobotocore[full] wasn't actually giving me types-aiobotocore-s3? I had to change full to all to get all the packages?

Hmm, probably docs are misleading a bit. types-aiobotocore[full] installs  types-aiobotocore-full package, shich contains type annotatiosn for all services. So, you do not need ``types-aiobotocore-s3when usingtypes-aiobotocore-full. The bugfix for this issue is included to types-aiobotocore-full 2.15.2.post1, so you can continue using types-aiobotocore[full]`.

perhaps you misunderstood? With types-aiobotocore[full] I did not get the s3 service annotations, I switched to types-aiobotocore[all] and then I did have all the annotations (including s3).

For some reason types_aiobotocore_s3/service_resource.pyi was not included to the latest types-aiobotocore-full release. I will check CI and re-release the package.

Fixed in 8.6.2 and published updated types-aiobotocore-full


Eclipse quick assist?

http://scn.sap.com/community/abap/eclipse/blog/2014/08/27/creating-a-abap-in-eclipse-plug-in-using-the-adt-sdk
http://wiki.scn.sap.com/wiki/display/ABAP/SAPlink+plugin+for+ABAP+in+Eclipse
http://scn.sap.com/docs/DOC-40894  Software Development Kit for the ABAP Development Tools
http://scn.sap.com/docs/DOC-54728  Starting with ADT SDK is easy - Pt.1
http://scn.sap.com/docs/DOC-40668  SAP NetWeaver How-To Guide: SDK for the ABAP Development Tools

classes CL_QUICKFIX_* ?

moved to separate project https://github.com/larshp/FORMfactor-Eclipse


Shared dashboard without an id

I have a shared dashboard and it has no id in its JSON.
Is this a documentation or a product bug?
As a consequence, I cannot see its subscription and resource group.
Nor I can see it in the Azure portal among all the other resources.
Can you explain?
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 9f609290-f427-4b0e-3e36-eb2535771d64
Version Independent ID: 49725d7c-b313-16db-ee8e-ebe9fad0fc81
Content: The structure of Azure dashboards - Azure portal
Content Source: articles/azure-portal/azure-portal-dashboards-structure.md
Service: azure-portal
GitHub Login: @JnHs
Microsoft Alias: jenhayes

Hello @ciufudean Did you try exporting any other Azure Dashboard and you see the same issue?
Could you please check with the person who shared the dashboard export with you?
Our SME will respond to your query further.

Yes, the same case for a 2nd dashboard.
I personally created and exported the 2 dashboards.
Lucian
On Sat, Sep 3, 2022 at 2:51 AM SatishBoddu @.***> wrote:

Hello @ciufudean https://github.com/ciufudean Did you try exporting any
other Azure Dashboard and you see the same issue?
Could you please check with the person who shared the dashboard export
with you?
Our SME will respond to your query further.
—
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/97933#issuecomment-1235985262,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AESRV6UUHCLRDYY2TUIC263V4KHIVANCNFSM6AAAAAAQDJ3REE
.
You are receiving this because you were mentioned.Message ID:
@.***>

@ciufudean, thank you for bringing it to our attention. I tested and found out that the "id" field is not available in the exported definition of dashboard from the portal. When it is imported back to Azure portal, the id is assigned by the azure platform itself.
To answer your question, only the dashboards which are in shared state reside as azure resources in the resource group. The private dashboards are not saved in resource groups. If you need to get the id (and hence the resource group) of the shared dashboards, you can either use get-azportaldashboard or az portal dashboard cli commands.
A sample is provided below for reference to get all shared dashboards and its Id.
//PowerShell:
get-azportaldashboard | % {write-output "id : $($_.id)", "Name : $($_.Tag['hidden-title'])" `n}

//az cli:
az portal dashboard list --query '[].{id:id, name:tags.\"hidden-title\"}'

Please let me know if you have any questions.

Thanks.
Then I suppose the doc needs to be corrected with respect to the id.
Another thing is that I navigate to my Azure portal home, I am looking at
all my resources and I cannot see these shared dashboards. The only one I
am seeing is one created by Application Insights.
How come?
Lucian
On Mon, Sep 5, 2022 at 12:09 PM Anurag Singh @.***>
wrote:

@ciufudean https://github.com/ciufudean, thank you for bringing it to
our attention. I tested and found out that the "id" field is not available
in the exported definition of dashboard from the portal. When it is
imported back to Azure portal, the id is assigned by the azure platform
itself.
To answer your question, only the dashboards which are in shared state
reside as azure resources in the resource group. The private dashboards are
not saved in resource groups. If you need to get the id (and hence the
resource group) of the shared dashboards, you can either use
get-azportaldashboard
https://docs.microsoft.com/en-us/powershell/module/az.portal/get-azportaldashboard?view=azps-8.2.0
or az portal dashboard
https://docs.microsoft.com/en-us/cli/azure/portal/dashboard?view=azure-cli-latest#az-portal-dashboard-show
cli commands.
A sample is provided below for reference to get all shared dashboards and
its Id.
//PowerShell:
get-azportaldashboard | % {write-output "id : $($.id)", "Name : $($.Tag['hidden-title'])" `n}
//az cli:
az portal dashboard list --query '[].{id:id, name:tags."hidden-title"}'
Please let me know if you have any questions.
—
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/97933#issuecomment-1236737645,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AESRV6QFOR7BNGYNSX3OWCTV4W2FHANCNFSM6AAAAAAQDJ3REE
.
You are receiving this because you were mentioned.Message ID:
@.***>

@ciufudean, thank you for the reply. Regarding the doc update, I will submit this feedback to the respective team for review.
Regarding your other questions of not being able to see shared dashboards, I could not understand the question. However, I will try to share some insights below, based on my limited understanding. Please share additional details (with supporting screenshots, if the information below does not help you)

On logging to Azure portal, if you are seeing the home view, as shown below, then the startup view is set to "Home". Please see this link to get details of how to change this view to dashboard view.

You can also switch between home and dashboard view by using the Menu option as shown below:

If you are in the dashboard view, but are unable to see a particular shared dashboard created, please search for "Shared Dashboards" in portal and see if you are able to see that dashboard in here. Else, you might have filters applied to select a different subscription where the dashboard was not saved. Please see this link for details.

Please let me know if you have any questions.


[graphql/telemetry] Add metrics and tracing

Description
This PR adds a number of metrics and logging.
Test plan
Mostly manual, firing up a service and a Grafana dashboard to test out the prometheus metrics.
Metrics

Logging
Query
{
  transactionBlocks(first:400) {
    nodes {
      digest
    }
  }
}

Log

Query
{
  epoch(id: -1) {
    epochId
  }
}

Log - note that for those errors that we do not add an error_code we are returning a warn log.

If your changes are not user-facing and do not break anything, you can skip the following section. Otherwise, please briefly describe what has changed under the Release Notes section.
Type of Change (Check all that apply)

[ ] protocol change
[ ] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes

@amnn The reason why in some warn/info we used [Response] or [Query] or [Cost]` is that all of these come from the logger, so one will get this: 1) first is the query, 2) is the validation, 3) is the response. As long as we know about it, it's fine, we can probably get the query id of an error, and then get all logs for that error, and then if we know the order we can deduct which is which. Visually, it's easier to parse and match the log to what part of the execution it happened.

The tags on logger messages are fine, I just don't want that to be used everywhere for the reason I mentioned.

@amnn @wlmyng thanks for the first round of reviews. I'd appreciate another review when you can. Since then, I did the following:

addressed all the suggestions
added inflight_requests metric
added the num_queries_top_level that records how many times a top level node is requested in a query, after talking to @suiwombat. In a nutshell, in a query as this one:

{
  epoch {
    epochId
  }
  transactionBlocks {
    nodes {
      digest
    }
  }
  chainIdentifier
}

we'll get
num_queries_top_level{path="chainIdentifier"} 1
num_queries_top_level{path="epoch"} 1
num_queries_top_level{path="transactionBlocks"} 1

Revisited the query_id and Uuid stuff, and got rid of the abstraction. Every query gets a unique identifier and can get accessed directly from context.
Added query_payload_too_large_size metric, which tracks how big the payloads are for queries that are not allowed to go through due to the too large payload.

The ctx still seems to be here, but not used? https://github.com/MystenLabs/sui/pull/15696#discussion_r1458123245

It's used for query id, session id, and so on.


[Feature Request] Pause Between Songs/Autoplay

What is the need and use case of this feature?
The need for this feature is when the user wants the track player to pause after every track finishes playing, while maintaining a full playlist being stored in the queue.  A potential use case for this is to provide a random wait time before playing the next track.
Describe the ideal solution
Ideally, there would be a function similar to setRepeatMode(), which allows for the user to set an option called pauseBetweenTracks or perhaps autoPlay (courtesy Chris Dosé from discord for this variable name).  Depending on if it is set, the track player will pause after a track reaches completion without starting the next track.
Describe alternatives you've considered
An alternative solution that I have had in mind is to have an event, similar to PlaybackTrackChanged, but instead fires when a track has finished playing (likely to have a name such as PlaybackTrackEnded).  However, due to how close in time the two events would be, it could be seen as redundant.
Additional context
I have attempted to use Event.PlaybackTrackChanged as a solution, however, with this method, the processing that happens before TrackPlayer.pause(); is called means that some of the next track begins playing.  For some songs which start out loudly, this is very noticeable.
How I can Help
While I have experience in both Java and in RN, I do not have much experience in Android Studio development using Java.  Therefore, if I were to assist in development, I would need onboarding assistance.  I also do not have experience in iOS programming, and as such I would not be able to help with that side of this feature.
However, I should be able to assist in testing the feature; having a workflow in my app that would rely on it.
This feature request is seconded by atefyamin#7501 on Discord

I think having an autoplay: boolean property that's passable to updateOptions would be a good idea. @dcvz @puckey thoughts?
@KfWelch , @dcvz will have the final say on if this is going to be approved to add to the lib. However, I will say the implementation would largely be on you. We'd be happy to assist where possible and there's some pretty good documentation to help guide you. If that doesn't sound doable we'll probably add this to the feature backlog if it gets approved.

I had this conversation on discord and one idea was to insert a silent audio between the tracks. I don't really like the solution but is just another option :)
Would be great if we could have  a  configurable pauseBetweenTracks property to be able to set the number of seconds the player would pause for ❤

Agreed that autoPlay (although I would make it playWhenReady: boolean) in updateOptions would be handy to have – another async call saved.
As far as the pausing between tracks is concerned, I get the feeling this is a pretty niche feature.. The broader use case isn't clear to me in any case.

Although... what is the autoPlay functionality exactly? As far as I am concerned it would be the same as the initial TrackPlayer.play() call – but if it would entail that even if the user called TrackPlayer.pause() in between, calling next would cause the track to play, I am a little hesitant due to extra implementation complexity.

Specifically on what you (DCVS) bring up on iOS playback, as I am obviously unfamiliar, I am seeing from what you say that there is a difference between pausing and stopping playback?  It makes me wonder if there are some further intricacies that could impact whether the app is killed between songs from a pause instead of a stop, though I can see that it might not care whether it is pause or stop between tracks.
For Puckey's most recent comment, autoplay: true would be exactly how the player currently functions, except for what I would hope is nothing but an if() statement to internally pause that is skipped over.  This would naturally be the default, due to it being a niche feature, as you recognize.  The false setting would do nothing more than pause the playback when a when a track ends, leaving the programmer/user to decide when to next call TrackPlayer.play().  In my mind, this would be the simplest option, and the user would still be able to tell the difference between a manual pause and an internal pause, as the manual pauses can have state variable changes attached to them.

My pr at https://github.com/doublesymmetry/react-native-track-player/pull/1713 (https://github.com/doublesymmetry/react-native-track-player/commit/5c9ca2fcbc296f7fd03583b54060f7857ddf5848 specifically) includes TrackPlayer.sleepWhenActiveTrackReachesEnd() which will pause the track when it has finished playing. Note that it is cleared after pausing or skipping to another item.

@KfWelch we just merged https://github.com/doublesymmetry/react-native-track-player/pull/1713 which has a bunch of new functionality. Please take a look at whats possible now and let me know if this ticket should be reopened based on that.

@jspizziri @puckey Why has this feature been removed?
f2dde5e (#1713)

My pr at #1713 (5c9ca2f specifically) includes TrackPlayer.sleepWhenActiveTrackReachesEnd() which will pause the track when it has finished playing. Note that it is cleared after pausing or skipping to another item.

It would useful having this sleepWhenActiveTrackReachesEnd feature to me. Now I should update my project's structure to stop track's autoplay.


Same stack nesting when using router.replace method

Reproduction
https://jsfiddle.net/parkjunghwan/g2L9mxke/66/
Steps to reproduce the bug
When using router.replace or router.push({...replace:true}), I want to ignore the stack if it is the same route.
Below is an example.

Click a row in the user list screen.
The user details screen is displayed. Click the "Edit" button.
The user editing screen is displayed. After editing, click "Save". At this time, call replace to ignore the "modified screen" stack and move to the detailed screen.
I want to move to the user list screen when I perform "Back" on the detail screen.

Expected behavior
When using replace, if the path? is the same, is it possible to maintain only one detail screen stack?
Actual behavior
If you press "Back" in number 4, the "Details" stack becomes two, and the back screen appears once more, and you have to press "Back" again to move to the list screen.
Additional information
No response

This is totally correct from the history api perspective so it's something you have to implement in userland. Note you could simply push instead of replace and the history entries would make more sense that way.


fizmo-sdl2 & fizmo-ncursesw crash with stack overflow on long filename parameter

Both variations I tested crash compiled from latest github source on Ubuntu 16.10 if given a long filename. Example: fizmo-ncursesw Really_Long_Filename_That_Will_Test_Robust_Processing_of_Length_how_long.dat or even a long file path including subdirectories.

So far I haven't been able to reproduce this. I've tried loading a file via Really_Long_Filename_That_Will_Test_Robust_Processing_of_Length_how_long_Really_Long_Filename_That_Will_Test_Robust_Processing_of_Length_how_long/Really_Long_Filename_That_Will_Test_Robust_Processing_of_Length_how_long_Really_Long_Filename_That_Will_Test_Robust_Processing_of_Length_how_long/Really_Long_Filename_That_Will_Test_Robust_Processing_of_Length_how_long_Really_Long_Filename_That_Will_Test_Robust_Processing_of_Length_how_long.z8 which has been working for both fizmo-ncursesw and fizmo-sdl2. Is it possible tht your .dat file was not a valid Z-Machine file and that the crash has been a result of issue #10?

Today I started another attempt at reproducing this issue. However, even trying on the described Ubuntu version and with various very deep directory structures – also cded all the way down – I was out of luck.
I can imagine that there's really some bug causing this behavior, but without some way to reproduce it I'll close this one for the moment. If you or anyone ever else has the same problem I'd love to hear about it. Maybe it's even possible to reproduce this with a fizmo having the --enable-tracing switch set for building so I can get the resulting tracelog.txt, which should help a lot in clearing this up.
Thanks for bringing up this issue and I hope I can fix it some other time.


cache crash handler

Fantastic! That took 20 million invocations of queueMicrotask() from an OOM error to ~10 seconds.


Add badges component

Description
Ref #211 . Add the badges component, kind of like this but more 8-bit like.
Compatibility
N/A
Caveats

I could not get delete/close icon right like this. So I'm opening this PR to maybe someone else (or maybe even me later) get this done. Because of this reason, I didn't put this component into index.html.
Another thing, about the other style of the badges like this

I'm thinking of putting it in a different class like nes-icon-badge, because from what I've researched, it's always on  top of some icon, and the style is different. I will try to code something and open an issue if that's ok. Any comments feel free.

Added is-icon class, not sure if is good enough though

:tada: This PR is included in version 2.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


WVD Agent Build version identification

The current guidance is not very clear in the following section:
https://docs.microsoft.com/en-us/azure/virtual-desktop/safe-url-list?source=docs#how-to-use-the-required-url-check-tool
The document does not state that the "URL Check tool" is found in the specific build version on the session host. There is also no guidance to identify the build version.
Here are my comments:
It is important to note that the WVDAgentURLTool.exe is located within the C:\Program Files\Microsoft RDInfra"Current Build" folder and not  "cd C:\Program Files\Microsoft RDInfra\RDAgent_1.0.2944.1200"
For example "C:\Program Files\Microsoft RDInfra\RDAgent_1.0.2944.1400>WVDAgentUrlTool.exe" is where I found the tool. I also note there were a number of build folders within the folder directory which made it difficult to find. However using the Azure portal > Windows Virtual Desktop > Host Pools > "Select Host Pool required", you will be able to see the current build version in use "WVD Agent Version"
I have made some suggested changed and inserted a screenshot showing the Agent version.
I trust this helps.
Best regards,
Ryan

@RMITBLOG : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Filterable MultiSelect loses dropdown label.

Detailed description

Is this issue related to a specific component?

Filterable MultiSelect

What did you expect to happen? What happened instead? What would you like to see changed?

It's not currently in the designkit (as far as I could see). The multi-select behaviour is problematic and different when filterable, in that it removes the dropdown label.
This is fine when the user is actively typing to filter down results, but leaves it looking confusing once some items have been selected - with the component reading: "(1x) Filter"
What I'd like to see changed: The dropdown label is returned when focus is not applied, making the component semantically readable and consistent with the non-filterable variant. When focus is applied the label could be replaced with the 'filter' message.

What version of the Carbon Design System are you using?
9

Additional information

Screenshots or code

Notes

Hi @lukefirth! 👋 Thanks so much for taking the time to make this issue 😄
Is this for the React.js implementation of this component?

Hey @joshblack thanks for getting back so soon. 👋
This is indeed! Link to the specific component in storybook

Issue moved to IBM/carbon-components-react #1143 via ZenHub