Can't install matplotlib

note: This error originates from a subprocess, and is likely not a problem with pip.
ERROR: Failed building wheel for matplotlib
Running setup.py clean for matplotlib
error: subprocess-exited-with-error
× python setup.py clean did not run successfully.
│ exit code: 1
╰─> [6 lines of output]
Traceback (most recent call last):
File "", line 2, in
File "", line 34, in
File "C:\Users\sudha\AppData\Local\Temp\pip-install-v62djluw\matplotlib_c968c02102d3415eae7d405bfa2c0c3e\setup.py", line 247, in
long_description=long_description,
NameError: name 'long_description' is not defined
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
ERROR: Failed cleaning build dir for matplotlib
Failed to build matplotlib
Installing collected packages: matplotlib
Running setup.py install for matplotlib ... error
error: subprocess-exited-with-error
× Running setup.py install for matplotlib did not run successfully.
│ exit code: 1
╰─> [23 lines of output]
  Edit setup.cfg to change the build options; suppress output with --quiet.

  BUILDING MATPLOTLIB
    matplotlib: yes [3.2.0]
        python: yes [3.9.13 (tags/v3.9.13:6de2ca5, May 17 2022, 16:36:42) [MSC
                    v.1929 64 bit (AMD64)]]
      platform: yes [win32]
   sample_data: yes [installing]
         tests: no  [skipping due to configuration]
           agg: yes [installing]
         tkagg: yes [installing; run-time loading from Python Tcl/Tk]
        macosx: no  [Mac OS-X only]

  running install
  running build
  running build_py
  copying lib\matplotlib\mpl-data\matplotlibrc -> build\lib.win-amd64-3.9\matplotlib\mpl-data
  UPDATING build\lib.win-amd64-3.9\matplotlib\_version.py
  set build\lib.win-amd64-3.9\matplotlib\_version.py to '3.2.0'
  running build_ext
  building 'matplotlib.ft2font' extension
  error: Microsoft Visual C++ 14.0 or greater is required. Get it with "Microsoft C++ Build Tools": https://visualstudio.microsoft.com/visual-cpp-build-tools/
  [end of output]

note: This error originates from a subprocess, and is likely not a problem with pip.
error: legacy-install-failure
× Encountered error while trying to install package.
╰─> matplotlib
note: This is an issue with the package mentioned above, not pip.
hint: See above for output from the failure.

Try to install Microsoft C++ Build Tools should fix that issue

no solution yet? (I have microsoft C++ build tools installed)

Matplotlib isn't really an important part of the app, and I really should have considered lighter alternatives... For the meantime, if you're unable to get Matplotlib to work, just globally find the word Matplotlib in the project folder on your text-editor and remove the code part that's executed by Matplotlib (only in the dashboard pane I think)

Done


Use Gradle Property and Provider to enable lazy evaluation for jib.extraDirectories parameters

In this PR:
Enable lazy evaluation in Gradle for various jib.extraDirectories parameters (ref: discussions in #2732).
jib.extraDirectories.permissions: Map<String, String> => MapProperty<String, String>
jib.extraDirectories.paths.path.from: Path => Property<Path>
jib.extraDirectories.paths.path.into: String => Property<String>
Lazy evaluation (via a Provider) for configuring the jib.extraDirectories.paths parameter directly is also enabled. Previously a Provider was able to be passed, but gets treated as a non-provider object and resolves file paths too early.
Both use cases below are supported in this PR:
extraDirectories {
   paths = project.provider { [value1, value2] }
   permissions = project.provider { value }
}

extraDirectories {
   paths {
     path {
       from = project.provider { value }
       into = project.provider { value }
       includes = project.provider { value }
       excludes = project.provider { value }
    }
  }
  permissions = project.provider { value }
}

Todos from presubmit checks:

[x] Address code smells - convert added anonymous inner classes to lambdas
[ ] Look into why added tests are not addressing line coverage
[ ] Fix test with string filepath that will not work with windows

Hmm I’m not sure why codecov is marking the overloaded setPaths(Provider<>) and setFrom(Provider<>) as uncovered, despite the tests added in JibPluginTest.java. Will look into this!

@elefeint Thank you for the review! I’ve updated the readme and made a note in the changelog.


Move all files from the frontend-client subdir to the project root

There is no need for a frontend-client subdirectory as there is no server counterpart in this repository.
I tested this by running the refbox and sending some instructions (including a delivery). I did not encounter any issues.

This makes a lot of sense. Thanks for also updating the outdated README!

Thank you for the very quick review :)


Last CentOS 6 build

Hi, can someone help point me to the final CentOS 6 build? My company is still using CentOS 6 so I can't use the newer builds.
Thanks in advance!

Dont' remember, but it probably started failing. Cent OS 6 is not updated since 2017 and was EOL November 2020 and at that point often things fall apart due to unmaintained/removed containers and packages (You probably experience that as well since you have to use Cent OS 6). So probably around May it started failing so it was finally removed in July.
If you can get the release script to work to build Cent OS 6 releases, we're happy to accept pull requests.

You're right that it's so old that it's a pain to compile anything that can work on it, but I've worked for multiple hardware company and most of them are still on Cent/RHEL 6 which is unfortunate. I will try to find a way to compile it and report back. It would be nice if we still have the centOS 6 branch so I can get a starting point.

This was the patch that removed it ... so maybe start with the reversse patch
https://github.com/chipsalliance/verible/commit/3b9a0750398a29398c4dbd8981a2fbb457d6cd64.patch
Also paging @umarcor who can give more context. I suspect it was removed due to age.

We now also distribute statically linked binaries in the releases:
https://github.com/chipsalliance/verible/releases
Can you try if that would work on your system ?


Issue #762 Set fontFamily based on OS instead of hardcoding

@rasmus-storjohann-PG or @tomy-pg - feel free to do a review on this, looks good from my end.

@ian-lin-PG looks good to me.

I am gonna leave a few screenshots for @cherrielam to check here.

[ ] iOS teaser fontFamily should match the topic fontFamily
[ ] Android teaser fontFamily should match the topic fontFamily
[ ] iOS link fontFamily should match paragraph fontFamily
[ ] Android link fontFamily should match paragraph fontFamily


refactor: reduce duplicate code

Signed-off-by: Ye Sijun<EMAIL_ADDRESS>

@mikebrow I added two unit tests for WithUserID and WithUsername.


Please downgrade System.Collections.Immutable from 9.0.0 to 8.0.0

When running from a source generator, Any reference that is also a dependency of Microsoft.CodeAnalysis.CSharp must not be a greater version than referenced by the Microsoft.CodeAnalysis.CSharp project. There is no workaround that I could find, other than to do a complete fork of the library to remove the reference.
CS1705 Assembly 'Corvus.Json.ExtendedTypes' with identity 'Corvus.Json.ExtendedTypes, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null' uses 'System.Collections.Immutable, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' which has a higher version than referenced assembly 'System.Collections.Immutable' with identity 'System.Collections.Immutable, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'
The references in that package are maintained by the visual studio team at Microsoft, separate from the .NET team, and tend to be delayed. We have to wait for the next major version of that to come out, and they don't give out release dates from what I understand.
The Corvus source generator library sadly isn't sufficient for my needs so I require a dependency on Corvus.Json.ExtendedTypes. I was finally able to get it working when building locally without this dependency (there's nothing in it that requires System.Collections.Immutable version 9.0.0 over version 8.0.0 that I could find).
It would be great if there was at least an ExtendedTypes package maintained that is compatible with the latest version of Microsoft.CodeAnalysis.CSharp at any time.
BTW: To deal with the .NET Standard "Invalid Program" issue which occurs when running from dotnet build (but doesn't occur from Visual Studio), I actually got it to work by having the generator target both .net standard and .net 9, and mute the warning that tells you not to do that. Works like a charm so far, though it's a bit of a hack, but one less hurdle out of the way to get this pipeline into production!
  <ItemGroup Condition="'$(BuildingInsideVisualStudio)' == 'true'">
    <ProjectReference Include="..\ACRealms.Roslyn.RealmProps\ACRealms.Roslyn.RealmProps.csproj" OutputItemType="Analyzer" ReferenceOutputAssembly="false" SetTargetFramework="TargetFramework=netstandard2.0" />
  </ItemGroup>
  <ItemGroup Condition="'$(BuildingInsideVisualStudio)' != 'true'">
    <ProjectReference Include="..\ACRealms.Roslyn.RealmProps\ACRealms.Roslyn.RealmProps.csproj" OutputItemType="Analyzer" ReferenceOutputAssembly="false" SetTargetFramework="TargetFramework=net9.0" />
  </ItemGroup>

I have pushed up a new preview build which includes a new package Corvus.Json.SourceGeneratorTools
https://www.nuget.org/packages/Corvus.Json.SourceGeneratorTools/4.1.1-preview.2
This is a self contained assembly with a pure netstandard2.0 package and 8.x dependencies which specifically helps you build analysers/source generators. It embeds an analyser version of Corvus.Json.ExtendedTypes and all of the code generation scaffolding in a single package, amongst other things.
I am just working on some sample code which illustrates how to use it and I will provide a link here when that is done (probably over the weekend.)

However, prior to building this library, I am happily running the v9.x System.Text.Json in a SourceGenerator targeting netstandard2.0 - it doesn't need anything net9.0 to do so. So maybe there is another issue?

Ah! Of course.

Closed with V4.1.1


cdc: changefeed shouldn't be blocked by sync-point

Signed-off-by: qupeng<EMAIL_ADDRESS>What problem does this PR solve?
Issue Number: close #6827
Currently pause and resume a sync-point enabled changefeed can make it hang forever. The code path looks like

in cdc/owner/changefeed.initialize, a sync-point barrier (startTs) is added;
it loads checkpointTs and resolvedTs from ETCD, and generally resolvedTs is larger than checkpointTs;
in handleBarrier, barrierTs == checkpointTs is fine, but barrierTs == resolvedTs is not.
then barrier can't be advanced correctly, the changefeed hangs forever.

The root cause is is the sync-point barrier is startTs, but resolvedTs is loaded from ETCD, which can be larger than startTs.
What is changed and how it works?
Check List
Tests

Unit test

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
None

Codecov Report

Merging #6837 (b665963) into master (435053b) will increase coverage by 0.4541%.
The diff coverage is 69.6581%.

Flag
Coverage Δ

cdc
67.2865% <71.7149%> (+1.2087%)
:arrow_up:

dm
51.9973% <40.0000%> (-0.0089%)
:arrow_down:

engine
62.6695% <0.0000%> (-0.3570%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
@@               Coverage Diff                @@
##             master      #6837        +/-   ##
================================================
+ Coverage   59.4341%   59.8882%   +0.4541%
================================================
  Files           787        783         -4
  Lines         89743      89520       -223
================================================
+ Hits          53338      53612       +274
+ Misses        31702      31197       -505
- Partials       4703       4711         +8

/run-all-tests

/run-verify
/run-leak-test


pre-commit.sh doesn't work on Mac OS

Report from Joel:

When I was trying to setup helm to get a feel for it (running ./pre-commit.sh) on my MacBook Pro (13.0.1) with conda I get the following error:
ERROR: Could not find a version that satisfies the requirement torch==1.12.1+cu113 (from versions: 1.4.0, 1.5.0, 1.5.1, 1.6.0, 1.7.0, 1.7.1, 1.8.0, 1.8.1, 1.9.0, 1.9.1, 1.10.0, 1.10.1, 1.10.2, 1.11.0, 1.12.0, 1.12.1, 1.13.0)

The installation command doesn't work because the package is Linux-specific.

Specific CUDA version needed for summarization metrics.  Would be nice to make things more modular, where we don't need all libraries for all metrics to be loaded to run anything.
Solution for now: remove it.  If running summarization, do these things.


Regexes yName bug

console.log(Regexes.Parse(/\y{Name}/));
12/22/2017 9:41:49 PM: Info: danabars: BrowserConsole: /(?:[ƻǀ-ǃʔא-תװ-ײؠ-ؿف-يٮٯٱ-ۓەۮۯۺ-ۼۿܐܒ-ܯݍ-ޥޱߊ-ߪࠀ-ࠕࡀ-ࡘऄ-हऽॐक़-ॡॲ-ॷॹ-ॿঅ-ঌএঐও-নপ-রলশ-হঽৎড়ঢ়য়-ৡৰৱਅ-ਊਏਐਓ-ਨਪ-ਰਲਲ਼ਵਸ਼ਸਹਖ਼-ੜਫ਼ੲ-ੴઅ-ઍએ-ઑઓ-નપ-રલળવ-હઽૐૠૡଅ-ଌଏଐଓ-ନପ-ରଲଳଵ-ହଽଡ଼ଢ଼ୟ-ୡୱஃஅ-ஊஎ-ஐஒ-கஙச
... etc

Whats the bug?

Does that look output look correct to you?

Sure? It should match any "word" character across all unicode. I'm not sure how it represents that exactly. I guess by putting them all in a []?

Hmmmmm ok.  I filed this while debugging, and in retrospect I think this was spurious.  I was having a hard time getting some \y{Name} to match and this seemed super suspicious.  I had an incorrect memory at the time of logging it sometimes and having it look much shorter, so thought something was up for some corner case.
Somewhat dubious that giant character set is the best way to match, but seems working for now.
Sorry for confusion!  Too much debugging.  /o\

OK! I think there's better alternatives in newer chrome v_v but can't upgrade CEF atm.


Issue with "note" snippet and numbered lists

When working with @michmike on the windows docs, it was brought to my attention that when a numbered list has a "note" snippet in the middle, it will break the sequence of numbers.
If the note snippet is indented over to match the list, the note gets formatted as a single line code block. This ultimately results in none of the markdown working (like URLs).
If the note is at the beginning of the list or the end, it is 100% fine.
For today, we will deal with the correct numbers and mangled notes snippet but it would be nice to fix. I'm thinking it might have to do with the list CSS - but need some help from someone more familiar with this.
Example: https://github.com/kubernetes/website/commit/d06922552b76536142ab65ddeeb865dfde3ddd8d#diff-2b903255e1cc87bd31083e5081f0be4a
Website build example (middle of section, see note format): https://deploy-preview-13279--kubernetes-io-vnext-staging.netlify.com/docs/setup/windows/user-guide-windows-nodes/#getting-started-adding-a-windows-node-to-your-cluster

@jimangel. I played with the rendering of the note shortcode in a list. I applied a short term fix to the windows-node setup page.
The CSS is different based upon whether the initial note shortcode markup is on a line by itself. Does not seem to matter about the closing note shortcode. Here are my findings:

The CSS is

, and the link works
{{< note >}}The VNI must be set to 4096 and port 4789 for Flannel on Linux to interoperate with Flannel on Windows. Support for other VNIs is coming soon. See the VXLAN documentation for an explanation of these fields.
{{< /note >}}

The CSS is

, no links
{{< note >}}
The VNI must be set to 4096 and port 4789 for Flannel on Linux to interoperate with Flannel on Windows. Support for other VNIs is coming soon. See the VXLAN documentation for an explanation of these fields.
{{< /note >}}

@jimangel, upon further investigation 😃,
I believe this explains the note rendering:

In the case where the note shortcode opening tag is on a single line:
{{< note >}}
The VNI must be set to 4096 and port 4789 for Flannel ...
{{< /note >}}

We define a note shortcode which adds the Note** prefix followed by all of the text (.Inner) between the shortcode note markup. The text is then processed through the hugo markdownify function (black friday markdown processor?). The text that is slurped up and processed happens to contain the indentation from the list which causes the extra
 tags to surround the text. These extra HTML tags then triggers the k8s/website CSS for
 which lacks the correct highlighting for links.
Ideally, I think, we only want to capture the text for the note and not the list indentation. So, to fix, it could be possible to clean-up all spaces, tabs before the first alpha-numeric character.

Thank you @kbhawkey for your diligence on this issue including adding examples in our docs!
/close


[WIP] Automate - RFE - Added Retirement functionality for Services.

Modified methods to allow a dialog to pass retirement days and days to warn
for a service bundle and service item.
Modified spec to include retirement days

@tinaafitz
Please Review

@tinaafitz
Please Review

I made the messages consistent between Catalogbundle and Catlaloginit as discussed.
@tinaafitz
Please Review

@billfitzgerald0120
Looks good.
@mkanoor
Please review.

@billfitzgerald0120 @tinaafitz
Would this change based on PR https://github.com/ManageIQ/manageiq/pull/9985

@mkanoor @billfitzgerald0120
Yes, it would change. I have changes to make to that PR and will follow up here when it's complete.

@tinaafitz
Please review

@tinaafitz
Please review

@billfitzgerald0120 Looks good.
@gmcculloug Please review.

Bump @gmcculloug

Moved to new repo
See: https://github.com/ManageIQ/manageiq-content/pull/28


Feature/android-mute-default

The purpose of this pull request is to change the defualt behavior on android where on starting trip event we always initialize the voice instructions commands to unmuted. For our business specific logic is best to left it muted on start, and then the user can decide whether or not to change it's mute state. For the IOS app, we can programmatically assign it to mute in our implementation instead, so this is an android-only feature.

Wrong branch, use


📊 agriculture: Add indicator on food trade

Add FAOSTAT indicator on food trade balance as a function of domestic supply.
See motivation in this conversation. But note the caveats mentioned in the code.
The resulting new chart is here.

Thanks for the review @spoonerf, I'll wait for Max's feedback before merging.


Add declare variable for supporting esm format

Jira card
https://employmenthero.atlassian.net/browse/PBL-1861
Summary
Add declare variable for supporting esm format following the changes of the original repo.
Screenshot

Before
After

Vite

Webpack

Vite Build
N/A

Thanks everyone ❤


Persian Locale problem

When using Persian locale dates are not converted by Jalali Calendar and only the captions are translated.

To see dates in Jalali Calendar, you need to use the Jalaali calendar plugin.

this plugin ain't implement all methods and introduce new functions it makes others library to malfunction.
I use this module to localizing mat-datepicker and after rewriting some function to force it to use jalaali moment understood which jalaali isn't support this use case and just  support jalali calendar when we call explicit methods like jYear().


Document MXNET_LIBRARY_PATH environment variable which was not docume…

…nted explicitly.
Description
Document MXNET_LIBRARY_PATH variable.
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[x] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[x] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[x] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

@tqchen @marcoabreu

Sorry, I'm currently swamped with other tasks and unable to assist here.


Rename test modes in spark-tests.sh [skip ci]

Signed-off-by: Peixin Li<EMAIL_ADDRESS>Previously ALL IT_ONLY may confuse people since we have more independent tests that require specific test params like  iceberg, avro or regex(UTF-8). Rename test mode to DEFAULT which means tests that do not require extra conda lib
verified locally

build


Clean outdated committed bundle assets

FrameworkBundle and SensioDistributionBundle don't have installable assets anymore, but the deletion was not committed when updated them.

Thanks @stof.


chore(release): v0.2.2

:robot: I have created a release beep boop
0.2.2 (2023-03-15)
Performance Improvements

more friendly Json format in document editor (2a2b8a8)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/riccox/meilisearch-ui/releases/tag/v0.2.2 :sunflower:


Icon manager needed

On every floor there should be a different set of icons (for possible obstacles). If a user taps on one of the icons a text box should appear.

Fixed


Add Stripe.com connector

Adds in Stripe.com WDC to the community connector section

merging in stripe connector listing


stream stops abruptly in some instances

sometimes, an openai-compatible provider doesn't give a finish reason, causing the stream to stop prematurely, such as with google's gemini openai-compatible api

Thanks for pointing this out!
I never actually tested Google's OpenAI compatible API. Just did though, and I see the issue.
It indeed looks like finish_reason is always None which is definitely incorrect on their part.
I also noticed that the final chunk contains both finish_reason (correct) and finishReason (incorrect). And it looks like only finishReason contains the actual finish reason (in this case stop).
I see other issues too...like the content of each chunk is way too long. Here's some that I saw for example:
content=' carp:\n\nBartholomew "Barty" Butterfield, a taxider'
content="mist with a penchant for sherry and questionable hygiene, inherited his great-aunt Mildred's dilapidated mansion – a gothic monstrosity clinging precariously to a cliff"
content=' overlooking a churning, grey sea.  Mildred, a recluse with a rumored penchant for necromancy (and possibly tax evasion), left Barty only'

Versus with OpenAI's API they're nice and small:
content='Oh'
content=','
content=' sure'
content=','
content=' absolutely'
content=','

Their API is still in beta so hopefully we'll see these issues resolved. Otherwise I'll have to bandaid fix it in llmcord :)


python311Packages.bimmer-connected: 0.13.8 -> 0.13.9

Diff: https://github.com/bimmerconnected/bimmer_connected/compare/refs/tags/0.13.8...0.13.9
Changelog: https://github.com/bimmerconnected/bimmer_connected/releases/tag/0.13.9
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 247597 run on x86_64-linux 1

5 packages built:

home-assistant-component-tests.bmw_connected_drive
python310Packages.bimmer-connected
python310Packages.bimmer-connected.dist
python311Packages.bimmer-connected
python311Packages.bimmer-connected.dist


Addy is sorted out - now just Jeff.

My account looks good, too! 👍


Youtube does not provide 3gp any longer. That's it. All questions should be addressed to them.

Youtube does not provide 3gp any longer. That's it. All questions should be addressed to them.
Originally posted by @dstftw in https://github.com/rg3/youtube-dl/issues/20069#issuecomment-468008527

Sorry. Was created accidentally when trying to copy reference link.


ENHANCEMENT conan

Please use this template for reporting suspected bugs or requests for help.
Issue Description
Have support for Conan to load the library in CMake
Environment
Linux

tag or commit hash on which this occured
OS version: <e.g. ubuntu-16.04>
...

Minimal test code / Steps to reproduce the issue
What's the actual result?

What's the expected result?

Hello,
If you can make a PR that does not break our current normal cmake build and adds support for conan, we will accept this PR.
FYI: We have stopped using this code internally and will be archiving this repo soon too.


Urgent : Need help Reg : C# CRUD code to interact with delta lake and exception while trying out this project locally.

Hi @dennyglee , Hi @mightyshazam,
I am Chandan from Microsoft Hardware Health team. we are trying to interact with deltalake using C# for our project needs. so we are exploring this incubator project. our requirement was to create bare minimum prototype for :

Creating a delta lake table
Writing to delta lake table
Reading delta lake table
Deleting a record from a Delta Lake table

We tried out this project locally which generated deltalog and parquet files.

However we facing exception due some unreleased handle in Rust code :  May be C# Code is trying to release the handle of some resource, but the native Rust code may be still holding onto it, preventing the C# side from releasing it.

Please help take a look if possible for below points as we are blocked on this:

The resolution for above exception
Is there a possibility that The Parquet file generated as output of console application is not well formed --> Please comment
As this is pretty new code so please help add C# CRUD code  to interact with ADLS/cloud deltatables using C#

Thanks,
Chandan Kumar

@Dev4all365 can you provide some minimal code to reproduce the issue or at least a stacktrace?

Hi @mightyshazam, i am sharing Program.cs code file which i used and also stack trace file, exception started at Line:45 Program.cs. Thanks
DeltaLake.zip


Easy script

I use this script to do show warnings for TODO, WARNING, and FIXME so I don't forget about them.
KEYWORDS="TODO:|FIXME:|WARNING:|\?\?\?:|\!\!\!:" find "${SRCROOT}" \( -name "*.swift" \) -print0 | \ xargs -0 egrep --with-filename --line-number --only-matching "($KEYWORDS).*\$" | \ perl -p -e "s/($KEYWORDS)/ warning: \$1/"

Yes for our purposes we like to generate errors too depending on build configuration, and halt the build if there are errors. And we also need to be able to exclude third-party directories. And it's nice for those features to be configurable as arguments. Thanks!


Sequences of quotation marks

What processing does the Microsoft C startup code apply to sequences of quotation marks? There seems to be more going on than what is documented on this page. For example, I get
input -> argv[1]
"a""b" -> a"b
"a"""b" -> a"b
"a""""b" -> a"b
"a"""""b" -> a""b

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

ID: 767a133c-6b2a-d353-f39a-9ced2772a1ff
Version Independent ID: 03b704f2-39fc-f68a-f649-0ed46eb9282f
Content: Parsing C Command-Line Arguments
Content Source: docs/c-language/parsing-c-command-line-arguments.md
Product: visual-cpp
GitHub Login: @mikeblome
Microsoft Alias: mblome

I'm using CreateProcessA to eliminate the command interpreter. I'm sending the following string:
"""" "\\\"""
The result is a single argument with the following value:
" \"
Can you explain this? I would have expected two arguments with the values of: " and \"".
Here's the CreateProcessA call:
CreateProcessA(
            NULL,
            "printArgs.exe \"\"\"\" \"\\\\\\\"\"\"",
            ...)

And printArgs.exe is simply:
   for (int i = 0; i < argc; i++)
      printf("%s\r\n", argv[i]);

Note I don't have access to argv_parsing.cpp so would really appreciate an explaination.


pub build customization

This issue was originally filed by<EMAIL_ADDRESS>
What steps will clearly show the issue / need for enhancement?

Using a content/source directory other than 'web' is not practical at the moment.

The name 'web' is too specific to be the default/required layout standard.

There is no clear way (impossible) to specify command line arguments to pub build within pubspec.yaml

What would you like to see instead?

Make it easier to customize the directory layout.

Instead of the standard 'web' directory I think 'client', 'ui', 'app' or even just 'src' would be more generic and applicable as a standard directory name. Personally I like 'app' - it's generic and fits well with 'lib' (and it's not tied to specific medium like 'web'). It's easy to think of 'app' next to 'lib' being "the specific 'app'lication of this specific lib".

Provide the ability to specify command line arguments to pub commands inside pubspec.yaml, just like $include, $exclude, etc. Alternately, the command line arguments could be stored in a separate file, like Python's setup.py configuration with the setup.cfg file: https://docs.python.org/2/distutils/configfile.html

This issue has been moved to dart-lang/pub#1184.


⚠ Secret Website 3 has degraded performance

In 99446c6, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 26602 ms

Resolved: Secret Website 3 performance has improved in 541e4b0.


[PR to validate Build] Updates for new ProjFS API

No review needed yet

The functional tests and unit tests are green whipee!
@wilbaker this is ready for a first pass.  I'm sure there will be lots of feedback.
@cgallred I turned off 2 tests that I can turn back on once we have a version with IWriteBuffer

/azp run Windows - Full Functional Tests

@wilbaker it's all building now (wahoo!).
Ready for review.  I'd recommend grabbing a coffee and scheduling a relaxing time shot.


Transition with Scoped Slot Children only works on component leave.

Version
3.1.0-beta.3
Reproduction link
here
Steps to reproduce

Create a component with a named slot. Add a scoped variable
Add a Transition component as the slot parent.
Notice how the slider component only applies the animation on component leave.

What is expected?
Animation should work on child component enter and leave
What is actually happening?
Animation only works on component leave.

Do you mean this sfc playground ?

Do you mean this sfc playground ?

Yep, that playground.

I mean it works correctly, see that playground I provided.

See also: https://v3.vuejs.org/guide/migration/transition.html


how is caffe2's speed compare to tensorflow?

are there any experiments?

I am not aware of that right now. Internally we measure the performance by looking at the theoretical peak possible (like the scaling efficiency when you use distributed training). C2 has been doing pretty well; if you find performance degradations - especially compared to Caffe - please do let us know by sending an issue. We are usually very good to figure out perf.


Is this gem still being actively developed?

Hi,
I see there are a lot of pull requests and issues opened for this gem and none of them are addressed. People are investing their time to make this gem better and nothing happens.
In the official documentation, this gem is still mentioned as one of the development assets one can use to develop shopify themes:
https://docs.shopify.com/themes/development/getting-started/choosing-an-editor#theme-gem
I thought Shopify is a mature platform with a respectable company behind it. This lack of activity makes me think differently. Can you please either write on top of Readme that the gem is no longer maintained or continue maintaining it?
Thanks!
Viktor

Especially considering that the Shopify Theme desktop app is so woefully out of date.

Looks like themekit is the recommended tool over shopify_theme now:
https://twitter.com/j_venator/status/710027524638580736
http://themekit.cat

Bump.
Also, @Moondrop I don't know if you are using any other tooling for your development but would you happen to know how reliable the Gulp/Grunt plugins are? Example: https://github.com/mikenorthorp/gulp-shopify-upload

I used grunt shopify plugin. It seems pretty reliable and it's much more configurable than this gem (it's a grunt plugin after all).
I did experience issues while "watching" files. Sometimes grunt-watch-contrib would just think all files changed and would start re-uploading them all one-by-one. It could also be due to my setup: my dev folder is symlinked and it's also inside of a Dropbox folder.

@vfonic We've run into similar issues on both grunt and gulp. We were able to solve the trying to upload the kitchen sink problem with gulp-filter and doing a check for exactly what file was changed and relying on theme upload rather than watch.

@chryton I gave the grunt shopfiy a brief try, but I ran into issues deleting files so I stopped using it.
Been using themekit for a bit and it works great. I compile all my assets with grunt, and have another watch running themekit to handle the uploads. It's what the Shopify employees use as well. Here's a nice write up they did on getting started with it:
https://www.shopify.com/partners/blog/95401862-3-simple-steps-for-setting-up-a-local-shopify-theme-development-environment

@Moondrop Thanks for the heads up. I came across ThemeKit a bit earlier but wasn't sure if it was getting any more support than this is/was.
If Shopify does not want to maintain this any longer, they should make that warning in their readme and direct users to ThemeKit rather than having developers stumble around the tools available.


Basic questions

While I'm still coming up to speed I have a few questions about the basic functionality of Ponzu. I was originally planning to use Bolt as the CMS to store content for my app, but am now looking at Ponzu because there's no official API for Bolt. Most of my questions are related to things that were pretty easy in Bolt.
Can content items have versions? For example to restore a previous version of a specific content item, or to schedule approval and publishing of an updated version of a content item on a specific date?
Is there an easy way to manage translation of content items into different languages? E.g have a content item where I can easily access the English and French versions of it from the admin UI.
For basic API authentication to ensure that not everyone who finds the web address can view or edit content - I assume the access add-on is what I need to use?
For repeated items is there any ordering? And can they be reordered easily? E.g. I have Procedure and ProcedureSteps content types. A Procedure contains several ProcedureSteps, and the order is important. If I select related ProcedureStep items when editing a Procedure item, are they returned in API requests in the order selected? If a mistake is made when selecting them, can the order be changed?
I know you say that Ponzu is super fast - are there any benchmarks published to show just how fast? (Might be a nice advertising feature on your site :) )
Any recommended (and cheap) go hosting providers? Is Google apps always free tier the most suitable?
Thanks!

Can content items have versions? For example to restore a previous version of a specific content item, or to schedule approval and publishing of an updated version of a content item on a specific date?

Currently, no, but this would be an awesome PR.

Is there an easy way to manage translation of content items into different languages? E.g have a content item where I can easily access the English and French versions of it from the admin UI.

Also currently no, and would love to see this as a PR from the community 👍

For basic API authentication to ensure that not everyone who finds the web address can view or edit content - I assume the access add-on is what I need to use?

No one but your Admin users can edit content by default. You can implement the api.Updateable or api.Createable interfaces to enable outside requests to change/create content. There are other interfaces in the documentation (https://docs.ponzu-cms.org) which may help you as well. Yes, there is an addon available for protecting content items from outside requests w/o a JWT in the header or cookie, but it's fairly untested: https://github.com/bosssauce/access
Though, the functionality is something you could add yourself and customize to your exact needs.

For repeated items is there any ordering? And can they be reordered easily? E.g. I have Procedure and ProcedureSteps content types. A Procedure contains several ProcedureSteps, and the order is important. If I select related ProcedureStep items when editing a Procedure item, are they returned in API requests in the order selected? If a mistake is made when selecting them, can the order be changed?

Repeated items are returned through the API in the exact order you have them listed in the CMS editor.
They can be easily re-ordered through the UI, but it does require you to remove one and add it again -- the UX is not the best for re-ordering (another PR opportunity which would be great).

I know you say that Ponzu is super fast - are there any benchmarks published to show just how fast? (Might be a nice advertising feature on your site :) )

Only the net/http package is used for network request handling, so you can defer to the Go benchmarks of any popular technology publication on the topic. Additionally, since the data fetched by each API call is simply bytes on disk, mostly pre-processed and formatted for the response, there is practically no database latency you may find in other systems.

Any recommended (and cheap) go hosting providers? Is Google apps always free tier the most suitable?

There really aren't "Go hosting providers" as your ponzu project will be compiled to a single static binary. You can put this binary on any system your build step targets (see https://dave.cheney.net/2015/08/22/cross-compilation-with-go-1-5). I would recommend running a $5/mo instance on Digital Ocean using a pre-configured Debian Linux distribution. There are many other options, but it is hard to make any recommendation without knowing how you intend to deploy your project.


Update template devDependencies

Summary
Update template devDependencies before 0.64.0 release
Changelog
[JavaScript] [Changed] - Update devDependencies (@babel/core to 7.12.9, @babel/runtime to 7.12.5, @react-native-community/eslint-config to 2.0.0, babel-jest to 26.6.3, eslint to 7.14.0 and jest to 26.6.3)
Test Plan

[ ] Check babel
[ ] Check eslint
[ ] Check jest

Thanks for the PR @Bardiamist, let me try to get it in front of the right people to tell us if this is mergeable :)

@cpojer, can you help with this? I'm not sure if these deps can be bumped if they don't match the versions of the rest of the monorepo

Yes they can be updated, you should be able to import and land the diff.


Unexpected results with Stable diffusion 2.1

Hi, I tested your code with both stable diffusion 1.5 and 2.1, and reconstructed results are so different. (Stable diffusion 2.1 seems doing much worse) Here are examples with DAVIS dataset samples.

car turn
(sd2.1)

(sd1.5)

bike
(sd2.1)

(sd1.5)

bear
(sd2.1)

(sd1.5)

Could you explain if there is any reason why the stable diffusion 2.1 performs pool?
Thanks for the your great work!

Turns out we didn't use accelerate. Thanks!


What's the deal with commits?

Question
What happens when two different commits are made to the same version of a file? I've been writing some tests for the core.py file, but as of writing this comment, @gracetangg is also writing tests, and I noticed that we are both working from the much shorter version of the test_core.py file that is on the main repo. I want to keep writing tests, but I don't know if that will create conflicts with the pull requests that have not yet been committed to the main turbopy repo.

To my knowledge, if you're working on separate tests (tests for different things) then GitHub shouldn't have a large issue. It might have a complaint about line numbers being off but that's easily fixable. Usually, big problems occur when two people edit the same line of code or a file is deleted while another person edits the same file. Admins may have a better idea of how to resolve the conflicts, but that's my knowledge so far @AOstenfeld .

That's a great question. There are several things that can happen in a case like this. The main thing that we would like to avoid is having two people work on the same parts of the same file simultaneously, since that leads to duplication of effort. In the case you describe, you are both writing tests in the same file, but the tests are for different parts of core.py.
In this case, you can both make PRs from your dev branch to main. I will look at (and test) the code in your PR, and then merge. Here, I've merged @gracetangg's PR, and requested a few changes to yours. Once you've made those changes, I will approve your PR. Then, before I merge it to main, I will resolve any potential conflicts between your submission and the main branch. Once I resolve those, I can merge your PR to main.
Side note, by "resolve merge conflicts" I mean that I will look at the code, and take the lines that I want from each file. Then I make a new commit with the lines that I want, and use that when merging your contribution into main.
This is a bit hard to explain in words, so I'd be happy to do a call sometime and explain. Or we can discuss either in this afternoon's call, or the meeting on Wed morning.

@AOstenfeld if these answers help, then go ahead and close this issue.


Instructions don't apply to VS2022

There is Cloud Explorer in VS2022. How do you do this in VS2022.

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

ID: 0065e05e-1e7e-20aa-badb-dfe2d297ddd2
Version Independent ID: aab0065a-72ef-b6ea-5ac1-d85e96e9e099
Content: Troubleshoot with Visual Studio - Azure App Service
Content Source: articles/app-service/troubleshoot-dotnet-visual-studio.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

@pdegenhardt Thanks for your feedback! We will investigate and update as appropriate.

pdegenhardt, after initial review, I have assigned the issue to the content author to review further and update the document as appropriate. Thanks again for the feedback!

#reassign:msangapu-msft

#reassign:msangapu-msft

Hi @cephalin, this is .NET content.
#reassign:cephalin

#reassign:cephalin

Internal item created: https://dev.azure.com/msft-skilling/Content/_workitems/edit/288583
#please-close


💚 Set the Python environment previous than caching

Codecov Report

Merging #15 (c35a555) into main (67b98b2) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #15   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            8         8
  Lines          118       118
=========================================
  Hits           118       118

Continue to review full report at Codecov.

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


8.0.0 breaks certain requires

(Apologies, as I write this I only have a couple of minutes so I'm including a very non-minimal reproduction of a problem. If it turns out to be hard to reproduce, next week I can work on a minimal reproduction. Please ask, or even just close this issue if the non-minimal repro is too much of a problem!  But I thought it useful to have the issue in to at least help anyone else affected.)
The following repo uses rollup-plugin-commonjs 7.0.0, and works:
https://github.com/OasisDigital/angular-aot-es2015-rollup/tree/rollup-plugin-commonjs-179
npm install; ./aot-build.sh
But with rollup-plugin-commonjs is changed to 8.0.0, it fails with the following:
node_modules/core-js/modules/_shared-key.js (7:4)
5:
6: var shared = require$$0('keys')
7:   , ;
       ^
8: var _sharedKey = function(key){
9:   return shared[key] || (shared[key] = uid(key));

I think either Buble or Uglify is objecting to the syntax, and perhaps should instead accept it - but this syntax only appears with 8.0.0, and I saw there's recently been a change to the code handling rewriting of requires.
https://github.com/rollup/rollup-plugin-commonjs/commit/c8c2548a41405991f4022bd130f99c26907513ba
... so there is some chance that @Rich-Harris will simply see my snippet above and know what caused it.

Thanks! Fixed in 8.0.1

I thank you for the fix, and promise not to sin again with an overly large repro repo. At least not very often...


Add export default & modify default params

dependency for rbush (PR https://github.com/mourner/rbush/pull/79)
Ref: https://github.com/mapbox/supercluster/pull/62
**Note: Moved default params to the main quickselect function

👍 My bad.. didn't follow that thread 😄 I'll submit a change

@mourner Added main() to handle the default params, then quickselect executes recursively without calculating the default params.

Let's rename main to quickselect and quickselect to quickselectStep to keep stack traces nice.

👍 Makes sense, didn't know how you wanted to rename it, but I like it!

Done 🎉

@DenisCarriere you forgot to rename the internal quickselect call. It doesn't trigger failures but it's a perf hit.

Ops! True 🤦♂

Renamed internal quickselect


Move author info to footer

The theme has it to the side but obviously that's unnecessary

Done ✅

Done ✅


Create Post using Markdown Language

Create a simple post on Climate-Change and World Hunger using markdown language. The process the documented here 👉
https://github.com/tasnimzotder/hacktoberfest-2021-little-steps#readme
The format is like 👇
---
username: <YOUR_USERNAME>
title: <TITLE>
date: <MONTH DATE, YEAR>
cover_img: '<YOUR_IMAGE_URL>'
excerpt: <EXCERPT>
---

# <TITLE>

<CONTENT>

By opening a PR, make an awareness in this planet.

I'm a Beginner , please assign me in this project.

Sure @pritambera2000 You just open a PR according to the guideline given in the README.md file.

Markdown language reference 📝 👉 https://guides.github.com/features/mastering-markdown/


Nix Dockerfile

As a first step in CI we could build a docker image that has all the system and code dependencies pre installed. Only rebuilding the image if the definition, flake or cabal file changed.

This could also use cachix mentioned in #91

I think we can close this - it was intended for speed, but cachix has got CI into a reasonable state for now


Hide Content Only Works For Non-Proof Directives, here

Describe the bug
context
When I add the class dropdown to any directive except
````{prf:proof}
:class: dropdown
````

the directives become collapsible in the HTML output opened in a browser.
expectation
I expected the feature to work for actual proofs, too.
bug

But instead I only succeed creating collapsible remarks, corollaries, notes, lemmas, theorems, ....
Additional tests suggest the feature may not work in these two situations, either:

anywhere in the markdown file after a prf:proof directive had been used
directives that have a caption as in ````{prf:lemma} caption to be shown as Lemma 1 (caption)

I did not find any error message possibly related to this.

problem
This is a problem for people creating lectures or slides for talks because hiding proofs is often of greater interest than hiding theorems when we want to convey the concepts.
Reproduce the bug

Create a Jupyter Book making use of sphinx-proof
Add the directives

````{prf:theorem}
:class: dropdown

All natural numbers are small.
````

and
````{prf:proof}
:class: dropdown

* $1$ Is small.
* If $n$ is small then $S(n)$, the successor of $n$ is small.
* By induction, all natural numbers are small.
````

jb build book/

List your environment
Jupyter Book      : 0.13.0
External ToC      : 0.2.4
MyST-Parser       : 0.15.2
MyST-NB           : 0.13.2
Sphinx Book Theme : 0.3.3
Jupyter-Cache     : 0.4.3
NbClient          : 0.5.3

Same issue here. I think it might be because "proof" doesn't have a title line that can be made clickable? Maybe it would be useful to add an option to provide a title to the "Proof" block.

Same problem here, anyone found a way around?

Now, I'm focused on completely different topics and cannot test that now. However another dropdown issue was solved by putting the contents inside a {div}-directive. Would anynone test and tell if that helps here, too?


Consider using arrays

Instead of ILoF classes.
Also the use of ...getType() code snippets is not good object-oriented style. It means the ILoF classes are doing things that should be divided up into responsibilities of the Fruit/Bomb classes. Come see me for help restructuring that.

Implemented arrays and got rid of getType().


Y24-099 Tube Rack Banking Sequencescape API

User story
As PSD, we would like to update Sequencescape v2 API to provide Limber support for TubeRack model similar to Plates using the versioned tube rack model proposed in Reuse of TubeRacks document.
Who are the primary contacts for this story
Abdullah, Katy, Andrew
Who is the nominated tester for UAT
TBD
Acceptance criteria

[ ] Check existing labware creator (in  DPL-811) for Sequencescape API calls and replace them for tube racks.
[ ] Check TubeRacks and associated objects are retrieved by using JSONAPI includes, i.e. Tubes and RackedTube.
[ ] Add resource methods to TubeRacks and RackedTubes so that they are similar to Plate and Wells.
[ ] Add TubeRack barcode into new TubeRack poly_metadata in order to enable history tracking of physical tube rack

Dependencies
This story is blocked by the following dependencies:

#1482

References
This story has a non-blocking relationship with:

#1314
#1713

Additional context
The versioned tube rack model is researched in DPL-1011 Re-use of tube racks and the model is summarised in the Google Document DPL-1011 Re-use of tube racks
This story is part of re-implementation of DPL-811 Transfer PBMC isolations into FluidX tube racks (Banking) using the versioned tube rack model.
Story relationships are shown in the Lucidchart diagram TubeRack Stories .
It is not possible to release the implementation of this story in isolation.

A couple of queries:

Do the features that require these APIs already exist or need to created? I suspect "API Support to allow History tracking of physical tube rack.." is a feature that does not yet exist?  Would it make sense to split that feature out into its own user story so that we understand what the business need before designing this API call?
I can see "API Support to find a tube on a tube rack" would be useful / needed. Curious as to how it works at the moment and/or where this is being called from. In the sense of - could this be split out into another smaller story?

"API Support to allow History tracking of physical tube rack using poly_metadata" - a bit more detail would be good. I think what we mean here is that the tube rack barcode will be saved into poly_metadata when the tube rack is created - is that correct?

TODO: Add more detail to how the criteria can be done if possible.

spec/models/labware_creators/plate_split_to_tube_racks_spec.rb fix breaking tests - part of this story
understand / fix lots of other broken tests on PR into epic branch - part of this story
see if we can refactor the labware creator code into common tube rack includes modules - leave until Cryostor work, when new labware creator will be needed
tube rack state - currently have a method on TubeRackPresenter for state, move to model? causes bug in relatives tab code if not present - put this into the presenter story.
submission of tube racks - is this needed? currently using tube submission 'Limber-Htp - scRNA Core cDNA Prep GEM-X 5p' on the tubes, and then scan them in for cryostor creation - part of the cryostor block of stories - write it down somewhere on those. If submission is on tube rack, need a way to create a tube rack before making the submission.
driver file for Bank plate to tubes is not working 'Hamilton LRC PBMC Bank plate to LRC Bank Seq and Spare tubes CSV' - story to fix this
bed verification is not working 'Hamilton LRC PBMC Bank => LRC Bank Seq and LRC Bank Spare' - have existing story
manual transfer not working - integration suite cannot progress without bed verification OR 'Manual Transfer' - make story to fix this
tube custom metadata for rack and coordinate - needed? - this is OK, we want this to be removed - racked_tubes should replace this
Think about the 'redoing the step' scenario & re-jig the existing backlog stories for the new data model.
Check on work completion code - is it triggered, test it after manual transfer is done. Relates to tube rack state (#4) - new story
Fix integration suite - make sure written down in acceptance criteria. Create new epic branch and merge Andrew's branch in.


Added jennifer, sam, hermes

jennifer
Added to ORM/ODM Extensions section
hermes
Added to Search section
sam
Added to Task management section

[x] Tests pass (crystal spec)
[x] Description of the entry is clear and concise. There is no excessive information like
"... for Crystal programming language",
"... for Crystal",
"... written in Crystal" etc.

Done

Thx 👍


Save location where job was run

Data saved for an invocation (benchmarking run) needs to indicate which Galaxy instance the invocation is from.  The easiest solution is likely to create a directory for each Galaxy instance in the invocations directory.

The name of the cloud instance (as specified in the profile.yml file) and server IP address are saved to the metrics output files.  Output files will be sorted into directories based on the experiment they are part of rather than the server location.


[T4A5][T16-B3] Varun Gupta

Ready for review

@varung97 added comments, please close this PR after you have read them


PSMoveService stop work

[ERROR] ClientNetworkManager::stop - Problem closing the socket: 应用程序没有调用 WSAStartup，或者 WSAStartup 失败。
[ERROR] handle_server_connection_close_failed - Error disconnecting from service: 应用程序没有调用 WSAStartup，或者 WSAStartup 失败。

PSMoveService always stop working for a few seconds. and sometimes error like this.

Please follow the instructions described in the Readme file.
Getting Help
Please start with the wiki. If you can't find help with your problem then please search through the issues (especially the closed ones) to see if your problem has been addressed already. If you still find no previous mention of your problem then you have one of two options:
A) Join us in the the PSMoveService Google Group to ask your question. There are several people there who have experience debugging problems with the PSMoveService.
B) If your problem actually is a new bug and you have files to attach (logs, pictures, etc) then go ahead and create a new issue. That said, it's probably best to start with the Google Group first anyway since we can help add context before posting an issue.

I'm going through and close all issues that are troubleshooting support related for tracking purposes.


Add schemaVersion to all BaseDocuments' metadata

I wonder if we should just leave it as String; it's only going to date us and frankly this piece of metadata isn't really meant to be used client side.

On the one hand, we do target specific Foundry versions; lots of things are minor breaking changes but we have their exact shape here. Metadata is weird because it breaks the general rule of avoiding literal values so that we can help people be future compatible.
It was modeled that way primarily for Metadata.types and while that's been deprecated, coreTypes swoops in as another case of definitely needing to be as exact as possible. It feels a bit weird to say "be as exact as possible everywhere except schemaVersion".
For consistencies sake how about we make label, labelPlural, and schemaVersion all string? I believe everything else's exact value is important. e.g. collection's exact value is used in ConfiguredCollectionClassForName and Game.Data.


Is there a way to do some validation before sending queries/mutations?

My use case is I wanna check if JWT token is still valid before sending queries to server like how we are able to use middleware in the networkInterface.

Could you do this using a network interface middleware? You can do anything you want in there, and the function runs before every single request.

@sammkj the new API ( #120 ) allows you to set skip: true on a query to bypass it based on information. If you can't use the network interface middleware, this is a place you could do that.


Adding support for pre-commit framework

Adding support for pre-commit framework
Issue #, if available: #115
Description of changes: Added the required .pre-commit-hooks.yaml file to support the pre-commit framework detailed here: https://pre-commit.com/
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Hi @mtdowling, could you merge this PR, please?

+1 @mtdowling, would love if you merge this PR and cut a new release.

+1

Tip for future PRs: If you prefix the issue number with some variation of "fix", "close", or "resolve", then the issue will be closed when the PR is merged. https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue#linking-a-pull-request-to-an-issue-using-a-keyword


Error: unable to build kubernetes objects from release manifest: unable to recognize "": no matches for kind "Deployment" in version "extensions/v1beta1"

When i run bellow command
helm install phpfpm . --set mariadb.mariadbRootPassword=DB_ROOTPASSWORD,mariadb.mariadbUser=DB_USERNAME,mariadb.mariadbPassword=DB_USERPASSWORD,mariadb.mariadbDatabase=DB_NAME
Output
Error: unable to build kubernetes objects from release manifest: unable to recognize "": no matches for kind "Deployment" in version "extensions/v1beta1"
I have changed in Deployment.yaml and ingress.yaml
apiVersion: extensions/v1beta1 to apiVersion: apps/v1
Again run above command still getting error.
Error: unable to build kubernetes objects from release manifest: [unable to recognize "": no matches for kind "Deployment" in version "extensions/v1beta1", error validating "": error validating data: ValidationError(Deployment.spec): missing required field "selector" in io.k8s.api.apps.v1.DeploymentSpec]

hi @NanheKumar I was not able to reproduce it
$ helm install phpfpm  \
  --set mariadb.mariadbRootPassword=DB_ROOTPASSWORD \
  --set mariadb.mariadbUser=DB_USERNAME \
  --set mariadb.mariadbPassword=DB_USERPASSWORD \
  --set mariadb.mariadbDatabase=DB_NAME \
  .

# NAME: phpfpm
# LAST DEPLOYED: Wed Apr  1 09:38:40 2020
# NAMESPACE: darteaga
# STATUS: deployed
# REVISION: 1
# TEST SUITE: None
# NOTES:
# 1. Get the application URL by running these commands:
#   export NODE_PORT=$(kubectl get --namespace darteaga -o jsonpath="{.spec.ports[0].nodePort}" services phpfpm-php-app-nginx)
#   export NODE_IP=$(kubectl get nodes --namespace darteaga -o jsonpath="{.items[0].status.addresses[0].address}")
#   echo http://$NODE_IP:$NODE_PORT

I don't know if your ar trying to use DB_ROOTPASSWORD as environment variable, if it is the case you should use mariadb.mariadbRootPassword="$DB_ROOTPASSWORD", also for the rest.

I'm getting the same error. I'm using:
helm v3.1.2
kubernetes server v1.18.0
kubectl v1.15.5

Could you share with us your commands and configurations?

kubectl get po -A
NAMESPACE     NAME                               READY   STATUS    RESTARTS   AGE
kube-system   coredns-66bff467f8-pngmh           1/1     Running   0          60s
kube-system   coredns-66bff467f8-s84q2           1/1     Running   0          60s
kube-system   etcd-minikube                      1/1     Running   0          75s
kube-system   kindnet-krnmg                      1/1     Running   0          59s
kube-system   kube-apiserver-minikube            1/1     Running   0          75s
kube-system   kube-controller-manager-minikube   1/1     Running   0          75s
kube-system   kube-proxy-9plgr                   1/1     Running   0          59s
kube-system   kube-scheduler-minikube            1/1     Running   0          75s
kube-system   storage-provisioner                1/1     Running   1          74s

kubectl cluster-info

Kubernetes master is running at https://<IP_ADDRESS>:32768
KubeDNS is running at https://<IP_ADDRESS>:32768/api/v1/namespaces/kube-system/services/kube-dns:dns/proxy

helm install phpfpm . --set mariadb.mariadbRootPassword=mini,mariadb.mariadbUser=mini,mariadb.mariadbPassword=mini,mariadb.mariadbDatabase=mini

Error: unable to build kubernetes objects from release manifest: unable to recognize "": no matches for kind "Deployment" in version "extensions/v1beta1"

Does that help?

did you do any changes on the templates?, I'm still unable to reproduce the issue.
I cloned the repo
$ git clone git@github.com:bitnami/tutorials.git
$ cd phpfpm-k8s/helm-chart

Copy your command and execute directly
$ helm install phpfpm . --set mariadb.mariadbRootPassword=mini,mariadb.mariadbUser=mini,mariadb.mariadbPassword=mini,mariadb.mariadbDatabase=mini
# NAME: phpfpm
# LAST DEPLOYED: Mon Apr  6 07:51:59 2020
# NAMESPACE: darteaga
# STATUS: deployed
# REVISION: 1
# TEST SUITE: None
# NOTES:
# 1. Get the application URL by running these commands:
#   export NODE_PORT=$(kubectl get --namespace darteaga -o jsonpath="{.spec.ports[0].nodePort}" services phpfpm-php-app-nginx)
#   export NODE_IP=$(kubectl get nodes --namespace darteaga -o jsonpath="{.items[0].status.addresses[0].address}")
#   echo http://$NODE_IP:$NODE_PORT

I have the exact same error as @KarneAsada. But it might not be related with this tutorial, OR, the code in the tutorial is specificly written for a k8s version. Which version are you running @dani8art ?
I found also this issues in which the "no matches for kind..." error is discussed https://github.com/helm/helm/issues/6969 https://github.com/helm/helm/issues/7219
And https://github.com/helm/helm/issues/6969#issuecomment-653016307 for 1.18 specific.
$ helm version --short
v3.2.4+g0ad800e
$ kubectl version --short
Client Version: v1.18.5
Server Version: v1.18.3

Hi @ErikTrapman, @KarneAsada, @NanheKumar I've just created a PR that should fix this versioning error. Thank you for your feedback!

I get the same errors with the tutorial. I can see the merged code from the PR available, but it still looks for extensions/v1beta1
helm install phpfpm . --set mariadb.mariadbRootPassword=mini,mariadb.mariadbUser=mini,mariadb.mariadbPassword=mini,mariadb.mariadbDatabase=mini
Error: unable to build kubernetes objects from release manifest: [unable to recognize "": no matches for kind "Deployment" in version "extensions/v1beta1", error validating "": error validating data: ValidationError(Deployment.spec): missing required field "selector" in io.k8s.api.apps.v1.DeploymentSpec]

$ helm version --short
v3.2.4+g0ad800e
$ kubectl version --short
Client Version: v1.19.2
Server Version: v1.19.2

Hi, sorry I was not able to reproduce it, this is my output where the apiVersion is correct.
$ helm template phpfpm . --set mariadb.mariadbRootPassword=mini,mariadb.mariadbUser=mini,mariadb.mariadbPassword=mini,mariadb.mariadbDatabase=mini -s templates/deployment.yaml
---
# Source: php-app/templates/deployment.yaml
apiVersion: apps/v1
kind: Deployment
metadata:
  name: phpfpm-php-app-phpfpm
  labels:
    chart: "php-app-0.1.0"
spec:
  replicas: 1
  template:
    metadata:
      labels:
        app: phpfpm-php-app-phpfpm
    spec:
      containers:
      - name: phpfpm-php-app-phpfpm
        image: "USERNAME/phpfpm-app:0.1.0"
        imagePullPolicy: IfNotPresent
        ports:
        - containerPort: 9000
        resources:
            limits:
              cpu: 100m
              memory: 128Mi
            requests:
              cpu: 100m
              memory: 128Mi
        readinessProbe:
          tcpSocket:
            port: 9000
          initialDelaySeconds: 10
          timeoutSeconds: 5
---
# Source: php-app/templates/deployment.yaml
apiVersion: apps/v1
kind: Deployment
metadata:
  name: phpfpm-php-app-nginx
  labels:
    chart: "php-app-0.1.0"
spec:
  replicas: 1
  template:
    metadata:
      labels:
        app: phpfpm-php-app-nginx
    spec:
      containers:
      - name: phpfpm-php-app-nginx
        image: "bitnami/nginx:latest"
        imagePullPolicy: "IfNotPresent"
        ports:
        - name: http
          containerPort: 8080
        - name: https
          containerPort: 8443
        livenessProbe:
          httpGet:
            path: /
            port: http
          initialDelaySeconds: 30
          timeoutSeconds: 5
        readinessProbe:
          httpGet:
            path: /
            port: http
          initialDelaySeconds: 5
          timeoutSeconds: 1
        volumeMounts:
        - name: nginx-config
          mountPath: /bitnami/nginx/conf/vhosts
      volumes:
      - name: nginx-config
        configMap:
          name: phpfpm-php-app

did you pull the latest changes?

Thanks for lookign again into this.
Yes, I have the latest changes.
And when I run the above command, I also get the same result (the correct apiVersion), but when I try to deploy it, it gives back the error.
I have tried with different versions of kubectl. I have also tried both on minikube and on an GCE cluster.
I have also tried hardcoding the apiVersion: apps/v1 in the deployment.yaml and ingress.yaml files, but the error still appears, like if it is just ignored.

Hi @koyan, it seems like mariadb version was too old, and we were missing some selectors required in this new version (apiVersion: apps/v1). It should be solved with the PR.
Thank you so much for the feedback!

I get the same error, how can I install fixed KubeMQ?
Thanks!

Hi @eruiz I cannot find any KubeMQ chart in this repo, could you point me to it?

Hi @koyan, it seems like mariadb version was too old, and we were missing some selectors required in this new version (apiVersion: apps/v1). It should be solved with the PR.
Thank you so much for the feedback!

Thanks. Confirming that it all works good now.
Thanks

Sorry, I write my comment in the wrong thread. My apologies.


Issue with cmake during installation of pyscf

Error description
Running
pip install --prefer-binary pyscf
causes a "Failed building wheel for pyscf" error.
Details
I use Anaconda and created a new environment with Python 3.12 right before running the installation command.
The complete error message is:
`Collecting pyscf
Using cached pyscf-2.5.0.tar.gz (8.7 MB)
Preparing metadata (setup.py) ... done
Collecting numpy!=1.16,!=1.17,>=1.13 (from pyscf)
Using cached numpy-1.26.4-cp312-cp312-win_amd64.whl.metadata (61 kB)
Collecting scipy!=1.5.0,!=1.5.1 (from pyscf)
Using cached scipy-1.12.0-cp312-cp312-win_amd64.whl.metadata (60 kB)
Collecting h5py>=2.7 (from pyscf)
Using cached h5py-3.10.0-cp312-cp312-win_amd64.whl.metadata (2.5 kB)
Requirement already satisfied: setuptools in c:_workspace_\anaconda3\envs\pyscf_2\lib\site-packages (from pyscf) (68.2.2)
Using cached h5py-3.10.0-cp312-cp312-win_amd64.whl (2.7 MB)
Using cached numpy-1.26.4-cp312-cp312-win_amd64.whl (15.5 MB)
Using cached scipy-1.12.0-cp312-cp312-win_amd64.whl (45.8 MB)
Building wheels for collected packages: pyscf
Building wheel for pyscf (setup.py) ... error
error: subprocess-exited-with-error
× python setup.py bdist_wheel did not run successfully.
│ exit code: 1
╰─> [5 lines of output]
running bdist_wheel
running build
running build_py
cmake -SC:\Users\nauck\AppData\Local\Temp\pip-install-jxryn05a\pyscf_8e52318d5a07487ebe6c5a781f55c5a8\pyscf\lib -Bbuild\temp.win-amd64
error: command 'cmake' failed: None
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
ERROR: Failed building wheel for pyscf
Running setup.py clean for pyscf
Failed to build pyscf
ERROR: Could not build wheels for pyscf, which is required to install pyproject.toml-based projects`
I then decided to run pip install cmake and reran pip install --prefer-binary pyscf
The new error message was:
`Collecting pyscf
Using cached pyscf-2.5.0.tar.gz (8.7 MB)
Preparing metadata (setup.py) ... done
Collecting numpy!=1.16,!=1.17,>=1.13 (from pyscf)
Using cached numpy-1.26.4-cp312-cp312-win_amd64.whl.metadata (61 kB)
Collecting scipy!=1.5.0,!=1.5.1 (from pyscf)
Using cached scipy-1.12.0-cp312-cp312-win_amd64.whl.metadata (60 kB)
Collecting h5py>=2.7 (from pyscf)
Using cached h5py-3.10.0-cp312-cp312-win_amd64.whl.metadata (2.5 kB)
Requirement already satisfied: setuptools in c:_workspace_\anaconda3\envs\pyscf_2\lib\site-packages (from pyscf) (68.2.2)
Using cached h5py-3.10.0-cp312-cp312-win_amd64.whl (2.7 MB)
Using cached numpy-1.26.4-cp312-cp312-win_amd64.whl (15.5 MB)
Using cached scipy-1.12.0-cp312-cp312-win_amd64.whl (45.8 MB)
Building wheels for collected packages: pyscf
Building wheel for pyscf (setup.py) ... error
error: subprocess-exited-with-error
× python setup.py bdist_wheel did not run successfully.
│ exit code: 1
╰─> [20 lines of output]
running bdist_wheel
running build
running build_py
cmake -SC:\Users\nauck\AppData\Local\Temp\pip-install-ooxovqsa\pyscf_b72b407286f74cefb0b9b967defb486a\pyscf\lib -Bbuild\temp.win-amd64
CMake Error: CMAKE_GENERATOR was set but the specified generator doesn't exist. Using CMake default.
-- Building for: NMake Makefiles
CMake Error at CMakeLists.txt:16 (project):
Running
     'nmake' '-?'

    failed with:

     The system cannot find the file specified

  CMake Error: CMAKE_C_COMPILER not set, after EnableLanguage
  CMake Error: CMAKE_CXX_COMPILER not set, after EnableLanguage
  -- Configuring incomplete, errors occurred!
  error: command 'C:\\_Workspace_\\Anaconda3\\envs\\pyscf_2\\Scripts\\cmake.exe' failed with exit code 1
  [end of output]

note: This error originates from a subprocess, and is likely not a problem with pip.
ERROR: Failed building wheel for pyscf
Running setup.py clean for pyscf
Failed to build pyscf
ERROR: Could not build wheels for pyscf, which is required to install pyproject.toml-based projects`
It was my understanding that the installation should complete without requiring cmake at all.
This issue is very similar to #1684, but was not solved by running
pip install --prefer-binary pyscf==2.2.0.

Apparently pyscf is not supposed to work directly on windows and I will retry on linux.


Intent 9 analysis

Developer intents: Provisioning intents

Intent 9
5SED
5MEE
EXRA
EXRC
EXCA
EXSM
GMEC

9. "I can ask the operator to inform me if the optimal MEC for my application and a specific terminal changes, taking into account mobility events, connectivity, shortest network path, cost, network load, MEC platform load etc."
No
No
?
?
?
?
Yes

GMEC
Details
Comments

(to complete)
(to complete)
(to complete)

(others add as appropriate) |

the version of the API spec contributed does not support user subscriptions and notifications but 5GFF has an updated API spec which supports this and the plan is to contribute it to CAMARA as a generic module which can be use across multiple APIs

This intent i.e., intent 9 seems same as intent mention in the issue #112. Is that an error?


[Microsoft.App/managedEnvironments]: firstPartyConfiguration not supported

Resource Type
Microsoft.App/managedEnvironments
Api Version
2022-03-01
Issue Type
Missing property(s)
Other Notes
I'm working my way through this document to integrate container apps with Geneva logging
https://msazure.visualstudio.com/One/_git/EngSys-MDA-GenevaDocs?path=%2Fdocumentation%2Fgetting_started%2Fenvironments%2Fazurecontainerapps.md&version=GBmaster&_a=preview
This requires setting firstPartyConfiguration property values. This doesn't seem possible in bicep.
Bicep Repro

resource containerAppsEnvironment 'Microsoft.App/managedEnvironments@2022-04-01-preview' = {
name: name
location: location
tags: tags
properties: {
firstPartyConfiguration: {
loggingGenevaCertSan: san
loggingGcsEnvironment: genevaEndpoint
loggingGcsAccount: genevaLogAccount
loggingGcsNamespace: genevaNamespace
loggingGcsConfigVersion: '1.0'
loggingGcsCertificateName: genevaCertName
loggingGcsKeyVaultName: genevaKeyVaultName
metricMdmAccount: genevaMetricAccount
metricCertSan: san
metricKeyVaultName: genevaKeyVaultName
}
}
I can see that this property is not included in the bicep types - but I have used ARM to deploy directly.  https://github.com/Azure/bicep-types-az/blob/main/generated/app/microsoft.app/2023-04-01-preview/types.md#managedenvironmentproperties
Confirm

[X] I have read the troubleshooting guide and looked for duplicates.

So - it's not documented precisely - but when I changed the version to 2023-04-01-preview - azd up did create the right properties. Although whatever visual studio is using to determine what is supported isn't there. So - this appears to be a documentation issue only.


Colorize stack plot per month

Would that be an option I could easily configure?

you can run analyze.py with --cohortfm %Y-%m

In my project, when I run git-of-theseus-analyze  . --cohortfm %Y-%m, I only end up with two cohorts: "Code added in 2019-01" and "Code added in 2019-02".
This project started in January and we have been adding code all the time. I think everything younger than February is classified as "2019-02", otherwise the chart looks OK.
Do you have an idea what might be the problem?

weird, no idea. if you track down the issue, let me know!


Multiple link is reacting as a single select when no max property is set

The following setting is resulting in a single select, even when multiple is set to true:
project: link('Project', {
	type: 'entry',
	multiple: true,
	optional: false,
	condition: Entry.type.is('Project')
}),

Entering a max property is solving this issue, but I would expect that the max property is not required because the multiple property was set to true already.
project: link('Project', {
        type: 'entry',
        multiple: true,
        max: 10,
        optional: false,
        condition: Entry.type.is('Project')
}),

This issue still exists. In v 0.2 it's even not working with the max property anyymore:
estimate: link('Estimate', {
      type: 'entry',
      multiple: true,
      optional: true,
      condition: Entry.type.is('Estimate'),
      width: 0.5
}),

You can use alinea.link.multiple(...). I guess it would be best to remove multiple from the options and improve docs :) If you're only after entries use alinea.entry.multiple(...)

Multiple is no longer a part of the options in 0.4.0 but we do still need to update the docs


[Feature][web  datasource-postgresql-cdc]  support web add pg cdc dadasource

Purpose of this pull request

Check list

[ ] Code changed are covered with tests, or it does not need tests for reason:
[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs

fix fomater

fix


Footer improved

PR Description 📜
Please include summary related to the issue you have fixed and describe your PR in brief over here by specifying the issue number on which you were working below
I have modified the footer UI to match the wave, made it more readable and enhanced the overall look of it.
Fixes #4108

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[ ] I have added entries for my game in main README.md
[ ] I have added README.md in my folder
[ ] I have added working video of the game in README.md (optional)
[ ] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

(ignore high contrast, my system clicks screenshots brighter)

Also, Please add GSSOC label to this PR
## Thank you soo much for contributing to our repository 💗

Thank you @sau-mili ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

@kunjgit please check my PR

Thank you  @sau-mili , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


close function not closing the tdlib instance

Hello,
we found out that our app is not fully closing on desktop, when its closed with the mouse, and not like on the device where the process is terminated. so we suspect that tdlib is still running a few threads and we found the close function. But I only get this:
2022-12-11T23:44:53 Shutting down TELEports, Good Bye!
2022-12-11T23:44:53 [SEND] : QJsonObject({"@type":"close"})
[ 2][t 4][1670798693.805856704][Td.cpp:3550][#1][!Td]	Close Td in state 2

and still no shutdown happens. also no auth state closed message comes back. Any infos on thatß
thanks!

It is impossible to name the cause without full log with verbosity 4, or even 20 in some cases.

@levlam ok will try to get more detailed logging...

Ok it was trivial, our type check for the auth state returning from tdlib had a typo, it seems tdlib closes itself correctly ;)

@levlam I reopen this, can it be that close doe snot return an Ok message? Our async thread waits forever it seems for that confirmation (in my logging it would come before the shutdown logging which is triggered by auth state change)...
2022-12-14T00:30:12 debug (unknown) [SEND] : QJsonObject({"@extra":"req-4","@type":"close"})
2022-12-14T00:30:12 warning (unknown) tdlib has shut down...

No, It always returns Ok.

So is it guaranteed that Ok comes before updateAuthorizationState ?

Yes.

@levlam we found the problem, seems we had deadlocked the main thread and no signals came through to it anymore. It works now...


ContentFilter init method "DisplayIncludingApplicationsExceptingWindows" sends "excludingApplications" and vice-versa

There exists a init method to create a SCFilter which accepts a display, excludingApplications and exceptingWindows.
This package supports that methods using InitParams.DisplayExcludingApplicationsExceptingWindows but while passing the params to the actual function, the "excluding" applications are passed as "including" applications. ( and vice-versa )
Ref: https://github.com/svtlabs/screencapturekit-rs/blob/77c7b58a3602fec64d8f981fb62665f4cc984ed9/screencapturekit-sys/src/content_filter.rs#L61
This results in unexpected behaviour.
@1313 Is this the expected behaviour? If not, I can quickly create a PR to fix the issue

It is a bug/misstake. Happy to tanke a PR 💙


Ability to disable/delay automatic focus

Summary:
It would be beneficial to have another prop on  to disable or delay the automatic focus.
Specifically on iOS Safari, the page is automatically scrolled to the focused object. There does not appear to be any way to suppress this functionality, and workarounds are quirky. If movement transitions are applied to the modal, the screen scrolls to undesired locations. Being able to disable or set a specific delay on the focus would provide an easy workaround.

I second this. I'm also experiencing a problem while calling the modal from a link inside a table using React Virtualized. It seems that when virtualizing the window scroll, the active element (when firing a modal from within the table) becomes the whole grid and that also scrolls the page top to match the top of the grid. I will try to set up a working example of what I mean.
I was poking around in the react-modal source and it looks like we might be able to easily disable this feature like @jordanmack suggested. I think I will take a crack at it.

I was able to get a working example of what I was talking about with react-virtualized scroll.
https://codesandbox.io/s/El6mDVR8g
Now I understand this is might be an edge case but I think it would still be useful to allow the user to disable the focus manager on an individual basis to allow for more custom control in certain scenarios.

Hi there, I'm an FE infra engineer at @SlackHQ, and we've definitely run into this problem! We use react-modal to render popovers and tooltips, and unfortunately, it takes focus away from our main text input.
I submitted a PR #490 for this and it seems like an easy solution, just adds a prop shouldFocusAfterRender that defaults to true.
Appreciate the hard work, and any feedback.  🙏


Returning image hash after import

I think the import function should return the image hash. Otherwise it will be hard to find the image in the image list, if you have more than 10 images. In particular if you want to set an alias straight after import.
I still think that the property 'alias' should be used as a 'reserved' property on image import, so I could work with the alias(es) straight after import. I understand that this would merge two logical steps into one kind of transaction, where the second step could fail while the first step, the import, is successful. Nevertheless a "best effort" approach while setting aliases here should be good enough, if you don't want to rollback the image import. Can you please share the idea, why you don't like the alias setting during import?

actually, I believe the command line spec does define that --alias can be passed to both publish and import to set aliases at import time. It's just not been implemented yet.

Didn't see that, sorry. Will do it over the weekend if it has not been touched by then.

Looking at the specs, I reckon it is a bit confusing sometimes, so sorry for the questions.
The spec says:

Aliases are a one to one mapping between a user friendly name and an image.

But the 'image list' command as well as the implementation allows multiple aliases for an image, which makes sense otherwise it would be just a field in the database I'm assuming. So, multiple aliases for an image is the goal?
Not implemented yet, but the '--alias' arg for publish and import is for one alias only and not comma-separated list?
The spec says as well:

Aliases may contain any character but colons and slashes.

But in the example it is centos/7 and it works at the moment. Colons I understand because you are using it for remotes. I'm assuming that this is just a thing, which needs to be 'fixed' in the specs.

Sounds like the spec needs fixing :)
It's a many to one mapping (an image can have multiple aliases but a given alias will only ever point to a single image). And indeed slashes are allowed and in fact used. Colons are still illegal though :)

Forgot to close this one. I implemented this yesterday.
Keeping the issue open to also address the spec inaccuracy.


Update the Registry Viewer build pipeline to be static

Which area/kind this issue is related to?
/area registry
Issue Description
Update the Registry Viewer build pipeline to be static.

[ ] Registry Viewer creates static files
[ ] Registry serves static files

Related Issues
~Block by https://github.com/devfile/api/issues/720~ No longer blocked.
Target Date: TBA

Closing

blocked by PR devfile/devfile-web#48


Add title for the kernel selector

For #8866

Codecov Report

Merging #9024 into master will increase coverage by 0.33%.
The diff coverage is 81.81%.

@@            Coverage Diff             @@
##           master    #9024      +/-   ##
==========================================
+ Coverage   56.65%   56.98%   +0.33%
==========================================
  Files         550      550
  Lines       28959    28968       +9
  Branches     4370     4378       +8
==========================================
+ Hits        16407    16508     +101
+ Misses      11651    11566      -85
+ Partials      901      894       -7

Impacted Files
Coverage Δ

...nt/datascience/jupyter/kernels/kernelSelections.ts
98.03% <100%> (-0.04%)
:arrow_down:

src/client/common/utils/localize.ts
94.6% <100%> (ø)
:arrow_up:

...ient/datascience/jupyter/kernels/kernelSelector.ts
82.43% <80%> (-0.39%)
:arrow_down:

.../client/datascience/jupyter/jupyterConnectError.ts
75% <0%> (ø)
:arrow_up:

src/client/constants.ts
80% <0%> (ø)
:arrow_up:

src/client/common/utils/platform.ts
70.58% <0%> (ø)
:arrow_up:

src/datascience-ui/react-common/logger.ts
40% <0%> (ø)
:arrow_up:

src/client/language/characters.ts
37.14% <0%> (+2.85%)
:arrow_up:

src/datascience-ui/history-react/redux/actions.ts
9.67% <0%> (+3.22%)
:arrow_up:

src/client/common/utils/version.ts
81.48% <0%> (+3.7%)
:arrow_up:

... and 17 more

Continue to review full report at Codecov.

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


ES6 export function lint problem

Hi all
When using the exported generator functionality in ES6:
export function test() {
..
}

I'm getting a lint issue, 'test' was used before it was defined. Looking around, this looks like it was fixed in jshint version 2.9.0. Does this sound familiar? Would it be possible to upgrade the jshint version to 2.9.x?

Reinstalling gulp-jshint will update jshint to the latest version published in the 2.x branch. 2.9 has not been published to npm yet, which is why it's not installing.


Remove e2studio .settings directories from demo projects

@paulbartell - san,
Thank you for your suggestion.
After checking, I found that the files in .settings are unnecessary.


Fix circular dependency in setup.py

Installing this package in a new virtualenv fails because setup.py tries to import pyod which in turn tries to import all its dependencies (numpy is in the error message in particular). However, it's setup.py's responsibility to install numpy and it gets imported before it's installed.
Fixing this by decoupling the version string from the pyod init file. For reference, this is the error we were seeing prior to the fix:
(env_pyod) ☁  ~VIRTUAL_ENV  pip install pyod
Collecting pyod
  Downloading https://files.pythonhosted.org/packages/08/bf/05bef83df97f81a6492818e0a42034e1bcc222894fd66bd4ce66c23a8654/pyod-0.5.4.tar.gz (44kB)
    100% |████████████████████████████████| 51kB 2.3MB/s
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/private/var/folders/v3/d3h954156117rd5xdlfh8c0d081dvn/T/pip-install-G3JNoB/pyod/setup.py", line 2, in <module>
        from pyod import __version__
      File "pyod/__init__.py", line 4, in <module>
        from . import models
      File "pyod/models/__init__.py", line 2, in <module>
        from .abod import ABOD
      File "pyod/models/abod.py", line 12, in <module>
        import numpy as np
    ImportError: No module named numpy

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /private/var/folders/v3/d3h954156117rd5xdlfh8c0d081dvn/T/pip-install-G3JNoB/pyod/

Pull Request Test Coverage Report for Build 155

1 of 2 (50.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.04%) to 96.076%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pyod/version.py
0
1
0.0%

Totals

Change from base Build 154:
-0.04%

Covered Lines:
2326

Relevant Lines:
2421

💛  - Coveralls

@ucabvas Thanks for submitting a fix. I am just back from vacation and would take a look tomorrow.
Just wonder whether the fix is fine with all platforms (win/mac/unix)? especially with './pyod/version.py'. What about using os.path.join(os.path.sep, 'pyod', 'version.py') instead?


Atom 1.18.0 - Multiple Remote Push URL's

Prerequisites

[X] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: http://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: http://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
Not detecting remotes if multiple URL's are set for the same remote.
Steps to Reproduce

Add an additional push URL

git remote set-url --add --push origin git@github.com:username/repo
git remote set-url --add --push origin git@github.com:username/repo2

Verify it worked
git remote -v

origin	git@github.com:username/repo (fetch)
origin	git@github.com:username/repo (push)
origin	git@github.com:username/repo2 (push)

Now try to do a push from inside of Atom and you will see

Note: No remote detected for branch. Cannot push because there is no remote named "origin" for which to create a remote tracking branch.

Expected behavior:
It should work the same as running the command:

git push

Which will push it to both just fine.
Actual behavior:
It doesn't allow me to push at all.
Reproduces how often:
100% of the time. Interestingly enough though, if I only add a second remote url after I have already loaded in the repo, it will continue to allow me to push - until i close and open atom or add the repo again (at which point it detects changes the error comes).
Versions
1.18.0-beta0

Please reopen this on the atom/github repository.


Require schemaVersion parameter in VitessceConfig constructor

User story
Want current code to stay compatible under future library versions. In order to identify the behavior that users are expecting, we need to know the view config schema version they are expecting compatibility with.
Preferred solution
Change constructor params to be VitessceConfig(schemaVersion, name = null, description = null) where schemaVersion is required but name and description are optional.
Possible alternatives
Put the version in the class name? like VitessceConfigV2 but this does not seem elegant

Fixed by #215


Add limit value for expensify card

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/50686
PROPOSAL: https://github.com/Expensify/App/issues/50686#issuecomment-2408674752
Tests

Go to Workspace > Expensify card
Click '+ Issue card'
Set a limit to > 20000000
Save the card
Verify that: error message is displayed

[x] Verify that no errors appear in the JS console

Offline tests

same as above
QA Steps

same as above

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

https://github.com/user-attachments/assets/eefb9c9c-05de-4390-a0dd-48dbe6e4edb5

Android: mWeb Chrome

https://github.com/user-attachments/assets/17c51362-8db9-4aa7-9cb0-80b4ff1103a2

iOS: Native

https://github.com/user-attachments/assets/0f5a7dc5-20b2-4e14-a5c2-b69e833a3bc1

iOS: mWeb Safari

https://github.com/user-attachments/assets/626cc3e3-126b-4b25-a12d-a867dbf70339

MacOS: Chrome / Safari

https://github.com/user-attachments/assets/04923bfd-fb71-44ec-9554-6ccfa8c9c02b

MacOS: Desktop

https://github.com/user-attachments/assets/715df835-4256-4733-8fde-96ec05875084

I am a little stuck on the  iOS builds, will update later

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.