[HOLD for payment 2023-08-01] [$1000] Close account offline - Close account is enabled by ENTER and no close account text

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Issue found when executing PR https://github.com/Expensify/App/pull/19328
Action Performed:

Go to staging.new..expensify.com
Go offline
Go toSettings > Security > Close account
Enter email address
Press ENTER on keyboard
Reconnect network

Expected Result:
In Step 5, Close account CTA is disabled and cannot be accessed by keyboard
In Step 6, after reconnecting network, close account message shows up in login screen
Actual Result:
In Step 5, Close account CTA is disabled but it can be accessed by keyboard
In Step 6, after reconnecting network, close account message does not show up in login screen
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[x] Android / Chrome
[ ] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/0e4e2dbd-4eb9-4985-a3ef-05fe5bf30d0c
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0137e6674165abf18a
Upwork Job ID:<PHONE_NUMBER>891503616
Last Price Increase: 2023-07-12

Possibly introduced by this PR?

Possibly introduced by https://github.com/Expensify/App/pull/19328?

Yes, possibly! Otherwise, here is the original issue where we refactored the Close Account page to use a form: https://github.com/Expensify/App/issues/11341.
I'll comment on the recent PR to see if it's a regression.

Alright, sounds like there's consensus that we need to audit the behavior across all pages and fix the root cause. Adding the external label.

Waiting for proposals.

Hey @eVoloshchak - since we haven't gotten any other proposals yet, could you please review the proposal from @hoangzinh to see if that will work here? Thanks!

Hey @eVoloshchak - have you had a chance to review the proposals we've gotten so far?

@eVoloshchak can you please take a look at the proposals we've gotten so far?

I agree with @samh-nl on root cause for this one

The FormSubmit component does not evaluate the prop enabledWhenOffline and the current online/offline status when submitting forms

However, I think we should proceed with @hoangzinh's proposal. All of the other proposals will disable form validation (local front-end validation) when enabledWhenOffline=false and you're offline
🎀👀🎀 C+ reviewed!
cc: @sakluger

PR was merged and deployed to staging last week, should deploy to prod today or tomorrow.

Summarizing payouts for this issue:
Contributor: @hoangzinh $1500
Contributor+: @eVoloshchak $1500 (payable after BZ checklist is completed)
Above payments include efficiency bonus 🎉
Upwork job: https://www.upwork.com/jobs/~0137e6674165abf18a

Oh yeah - @eVoloshchak could you please finish the BZ checklist so that your payment can be approved? Thanks!

Not seeing a request for this one yet in NewDot. Let me know if I'm missing something.

The PR that introduced the bug has been identified. Link to the PR: I can't pinpoint the exact PR that caused this. The first PR to use enabledWhenOffline prop was https://github.com/Expensify/App/pull/10395, but it's completely unrelated to this and was adding this prop to use in a single specific case. I'd say this is something that wasn't implemented initially, not a regression per se
The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: N/A
A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion: I don't think this is necessary, in general, keyboard navigation is out of scope, with the exception of this specific case (confirming something by pressing space/enter)
Determine if we should create a regression test for this bug.
Is it easy to test for this bug? Yes
Is the bug related to an important user flow? Yes
Is it an impactful bug? No
While this specific case isn't impactful, the bug was present on all forms throughout the app, so we might want to add a regression test

Regression Test Proposal

Go to Settings > Security > Close account
Disconnect from network
Enter your default contact method
While contact method input is still focused, press ENTER on keyboard
Verify that there is no close account confirmation modal shown
Connect back to the network
While contact method input is focused, press ENTER on keyboard
Verify that close account confirmation modal is shown

Do we agree 👍 or 👎

Requested the payment via NewDot

Reviewed details for @eVoloshchak. These details are accurate based on summary from Business Reviewer and are now approved for payment in NewDot.

Created the regression test issue: https://github.com/Expensify/Expensify/issues/305598. We're all set here, thanks everyone!


upgrade reqwest to 0.9

I am attempting to upgrade this client to ACMEv02 (in a fork, since this appears to be abandoned).
I've merged this in over in my repo to the main branch in this commit: https://github.com/dacut/acme-client/commit/a18e8ec87e73d4654556944f18f8709686166de8


I am getting version mismatch error

The current Dart SDK version is 2.1.0-dev.5.0.flutter-a2eb050044.
Because app depends on geolocation >=0.1.1 which requires SDK version <2.0.0, version solving failed.

This means the geolocation plugin needs to be updated. Please raise the issue in the geolocation repository.


Update continuous-integration-faqs.md per CI-14248

CI-14248
Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.
Description

Please describe your changes: CI-14248
Jira/GitHub Issue numbers (if any): CI-14248
Preview links/images (Internal contributors only): __________________

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged:

[ ] Successful preview build.
[ ] Code owner review.
[ ] No merge conflicts.
[ ] Release notes/new features docs: Feature/version released to at least one prod environment.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://670837d62e4f8c555034842a--harness-developer.netlify.app

Resolved via https://github.com/harness/developer-hub/pull/8521


variable future.utils.PY3 -- is time bomb

issue same as in six-project: https://bitbucket.org/gutworth/six/issues/22/variable-sixpy3-is-time-bomb ..
(same as in six-project you may mark this issue as wontfix . but , ok i warned about this issue )

see this file as a good pattern -- https://github.com/python/cpython/blob/master/Lib/asyncio/compat.py

Following that logic, six.PY2 should be PY2 = sys.version_info[0] >= 2 ?? :)
Pretty sure this can just be closed.

Following that logic, six.PY2 should be PY2 = sys.version_info[0] >= 2 ??

yes. exactly.
but variable just PY2 for six/python-future is not needed -- if variable PY3 exists.

Pretty sure this can just be closed.

of course you can. when time-bomb will explode -- I just will reopen this issue-ticket :-)

I've adjusted some of the conditionals, but this is not sufficient for Py4.0 support.  We'll cross that bridge once it's laid down...


[Bug] App Crash: IllegalArgumentException

What I did before

played around with the "remove background" feature
I used some of the possible actions, including to revert them, before trying another one
at the end I decided to go back to the main activity, with not applying any of the changes

Device: SM-G973F (samsung - beyond1), SDK: 31 (12), App: 2.4.3-alpha01 (98)
java.lang.IllegalArgumentException: width(-63) must be >= 0
at l2.p.z(SourceFile:31)
at z.r.a(SourceFile:69)
at m0.c.S0(SourceFile:18)
at z.t.p(SourceFile:499)
at s.v0.p(SourceFile:359)
at x1.f0.d(SourceFile:74)
at z1.p.a(SourceFile:51)
at r.m.i(SourceFile:44)
at r.m.o(SourceFile:476)
at x1.z.h(SourceFile:8)
at z1.p.a(SourceFile:83)
at r.m.i(SourceFile:10)
at r.m.o(SourceFile:489)
at x1.z.h(SourceFile:8)
at z1.p.a(SourceFile:83)
at k1.q0.h(SourceFile:1)
at z1.p.a(SourceFile:83)
at v.f0.h(SourceFile:99)
at z1.p.a(SourceFile:83)
at z1.f0.c(SourceFile:26)
at l2.p.K(SourceFile:59)
at c1.y.b(SourceFile:48)
at c1.z.c(SourceFile:162)
at z1.i1.a(SourceFile:1)
at z1.d0.o0(SourceFile:139)
at z1.d0.a(SourceFile:119)
at v.g1.a(SourceFile:394)
at v.e1.d(SourceFile:32)
at z1.p.a(SourceFile:51)
at z1.f0.c(SourceFile:26)
at l2.p.K(SourceFile:59)
at c1.y.b(SourceFile:48)
at c1.z.c(SourceFile:162)
at z1.i1.a(SourceFile:1)
at z1.d0.o0(SourceFile:139)
at z1.d0.a(SourceFile:119)
at v.g1.a(SourceFile:394)
at v.e1.d(SourceFile:32)
at z1.p.a(SourceFile:51)
at v.f0.h(SourceFile:99)
at z1.p.a(SourceFile:83)
at z1.f0.c(SourceFile:26)
at l2.p.K(SourceFile:59)
at c1.y.b(SourceFile:48)
at c1.z.c(SourceFile:162)
at z1.i1.a(SourceFile:1)
at z1.d0.o0(SourceFile:139)
at z1.d0.a(SourceFile:119)
at v.t.d(SourceFile:83)
at z1.p.a(SourceFile:51)
at z1.f0.c(SourceFile:26)
at l2.p.K(SourceFile:59)
at c1.y.b(SourceFile:48)
at c1.z.c(SourceFile:162)
at z1.i1.a(SourceFile:1)
at z1.d0.o0(SourceFile:139)
at z1.d0.a(SourceFile:119)
at v.t.d(SourceFile:83)
at z1.p.a(SourceFile:51)
at z1.f0.c(SourceFile:26)
at l2.p.K(SourceFile:59)
at c1.y.b(SourceFile:48)
at c1.z.c(SourceFile:162)
at z1.i1.a(SourceFile:1)
at z1.d0.o0(SourceFile:139)
at z1.d0.a(SourceFile:119)
at v.g1.a(SourceFile:394)
at v.e1.d(SourceFile:32)
at z1.p.a(SourceFile:51)
at z1.f0.c(SourceFile:26)
at l2.p.K(SourceFile:59)
at c1.y.b(SourceFile:48)
at c1.z.c(SourceFile:162)
at z1.i1.a(SourceFile:1)
at z1.d0.o0(SourceFile:139)
at z1.d0.a(SourceFile:119)
at p.x.d(SourceFile:368)
at z1.p.a(SourceFile:51)
at p.z.a(SourceFile:11)
at p.z.o(SourceFile:28)
at x1.z.h(SourceFile:8)
at z1.p.a(SourceFile:83)
at p.v0.h(SourceFile:48)
at z1.p.a(SourceFile:83)
at k1.q0.h(SourceFile:1)
at z1.p.a(SourceFile:83)
at p.j.o(SourceFile:9)
at x1.z.h(SourceFile:8)
at z1.p.a(SourceFile:83)
at z1.f0.c(SourceFile:26)
at l2.p.K(SourceFile:59)
at c1.y.b(SourceFile:48)
at c1.z.c(SourceFile:162)
at z1.i1.a(SourceFile:1)
at z1.d0.o0(SourceFile:139)
at z1.d0.a(SourceFile:119)
at p.s.d(SourceFile:84)
at z1.p.a(SourceFile:51)
at p.z.a(SourceFile:88)
at p.z.o(SourceFile:15)
at x1.z.h(SourceFile:8)
at z1.p.a(SourceFile:83)
at z1.f0.c(SourceFile:26)
at l2.p.K(SourceFile:59)
at c1.y.b(SourceFile:48)
at c1.z.c(SourceFile:162)
at z1.i1.a(SourceFile:1)
at z1.d0.o0(SourceFile:139)
at z1.d0.a(SourceFile:119)
at z1.e0.a(SourceFile:34)
at v.t.d(SourceFile:83)
at z1.q.a(SourceFile:38)
at k1.q0.h(SourceFile:1)
at z1.v.a(SourceFile:11)
at v.f0.h(SourceFile:99)
at z1.v.a(SourceFile:11)
at q.k0.b(SourceFile:34)
at q.k0.c(SourceFile:52)
at l2.p.K(SourceFile:59)
at c1.y.b(SourceFile:48)
at c1.z.c(SourceFile:162)
at z1.i1.a(SourceFile:1)
at z1.e0.q0(SourceFile:107)
at z1.e0.a(SourceFile:106)
at x1.g1.d(SourceFile:84)
at z1.q.a(SourceFile:38)
at q.k0.b(SourceFile:34)
at q.k0.c(SourceFile:52)
at l2.p.K(SourceFile:59)
at c1.y.b(SourceFile:48)
at c1.z.c(SourceFile:162)
at z1.i1.a(SourceFile:1)
at z1.e0.q0(SourceFile:107)
at z1.l0.c(SourceFile:18)
at z1.l0.l(SourceFile:1)
at z1.l0.i(SourceFile:57)
at androidx.compose.ui.platform.AndroidComposeView.onMeasure(SourceFile:79)
at android.view.View.measure(View.java:27129)
at androidx.compose.ui.platform.a.g(SourceFile:1)
at androidx.compose.ui.platform.a.onMeasure(Unknown Source:3)
at android.view.View.measure(View.java:27129)
at android.view.ViewGroup.measureChildWithMargins(ViewGroup.java:7980)
at android.widget.FrameLayout.onMeasure(FrameLayout.java:197)
at androidx.appcompat.widget.ContentFrameLayout.onMeasure(Unknown Source:159)
at android.view.View.measure(View.java:27129)
at android.view.ViewGroup.measureChildWithMargins(ViewGroup.java:7980)
at android.widget.LinearLayout.measureChildBeforeLayout(LinearLayout.java:1552)
at android.widget.LinearLayout.measureVertical(LinearLayout.java:842)
at android.widget.LinearLayout.onMeasure(LinearLayout.java:721)
at android.view.View.measure(View.java:27129)
at android.view.ViewGroup.measureChildWithMargins(ViewGroup.java:7980)
at android.widget.FrameLayout.onMeasure(FrameLayout.java:197)
at android.view.View.measure(View.java:27129)
at android.view.ViewGroup.measureChildWithMargins(ViewGroup.java:7980)
at android.widget.LinearLayout.measureChildBeforeLayout(LinearLayout.java:1552)
at android.widget.LinearLayout.measureVertical(LinearLayout.java:842)
at android.widget.LinearLayout.onMeasure(LinearLayout.java:721)
at android.view.View.measure(View.java:27129)
at android.view.ViewGroup.measureChildWithMargins(ViewGroup.java:7980)
at android.widget.FrameLayout.onMeasure(FrameLayout.java:197)
at com.android.internal.policy.DecorView.onMeasure(DecorView.java:1278)
at android.view.View.measure(View.java:27129)
at android.view.ViewRootImpl.performMeasure(ViewRootImpl.java:4541)
at android.view.ViewRootImpl.performTraversals(ViewRootImpl.java:3972)
at android.view.ViewRootImpl.doTraversal(ViewRootImpl.java:2924)
at android.view.ViewRootImpl$TraversalRunnable.run(ViewRootImpl.java:10513)
at android.view.Choreographer$CallbackRecord.run(Choreographer.java:1108)
at android.view.Choreographer.doCallbacks(Choreographer.java:866)
at android.view.Choreographer.doFrame(Choreographer.java:797)
at android.view.Choreographer$FrameDisplayEventReceiver.run(Choreographer.java:1092)
at android.os.Handler.handleCallback(Handler.java:938)
at android.os.Handler.dispatchMessage(Handler.java:99)
at android.os.Looper.loopOnce(Looper.java:226)
at android.os.Looper.loop(Looper.java:313)
at android.app.ActivityThread.main(ActivityThread.java:8751)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:571)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1135)

maybe fixed


Overlap in Mobile View

Hi there need some fixation in mobile view
Please assign this issue to me

Ill assign You the Task.
complete #64 #62 #25  also.

I have made the pr
Please review it


Create product page

Task Description
Create product page. Product page should clearly show product name, description, price and other relevant information.
Technical Considerations
Product page should use new app directory of Next.js 13 for simplified data fetching.
Resource
https://beta.nextjs.org/docs/routing/defining-routes

Next.js 13 app dir is not yet been supported by tRPC (the backend)

I think this issue has been resolved, closing it


Upgrade Kotlin to version 1.3.50

Please have a look at the individual commit messages for the details.

Merged, ignoring unrelated test error.


New release w/o chrono dep

Can we get a new release with #61? I'd really love for my downstream CI to stop yelling at me about the CVE.

I'm a bit annoyed by the fact that this causes these warnings. yasna does not use the offending API of the time crate, but it does/did use the fact that chrono supports leap seconds. Lacking leap second support can introduce actual bugs to programs. So the CVE warning pushes yasna from one solution that has no issues to another solution that does have issues, worsening security not helping with it.
Maybe we should just do our time handling ourselves, idk.

Bump?

+1 bump on release

Resolved now. See https://github.com/est31/rcgen/pull/66#issuecomment-1024929716 for why it took so long.


LanguageServer  Fails to start on Mac M1

Type: Bug
The language server fails to start whenever I open VSCode. Following tips on other posts I uninstalled extensions, and also deleted from my extension folder and keep getting the same results. I suspect it has to do with the M1 architecture since I have to point my dotnet to the x64 version for general use.
From the C# Log
Dotnet path: /Users/jwallace/Library/Application Support/Code - Insiders/User/globalStorage/ms-dotnettools.vscode-dotnet-runtime/.dotnet/7.0.8/dotnet
Activating C# + C# Dev Kit + C# IntelliCode...
Unhandled exception. System.IO.FileLoadException: Could not load file or assembly 'Microsoft.CodeAnalysis.LanguageServer, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'.
[Error - 6:14:58 AM] Server initialization failed.

Could you clarify what this means though?
I suspect it has to do with the M1 architecture since I have to point my dotnet to the x64 version for general use.

Since the extension is using an arm64 version, I don't think it will work with an x64 dotnet.  However it should be downloading and running the arm64 version of the dotnet runtime (from here Dotnet path: /Users/jwallace/Library/Application Support/Code - Insiders/User/globalStorage/ms-dotnettools.vscode-dotnet-runtime/.dotnet/7.0.8/dotnet

I was running from ~/Downloads, so I decided to start clean

I uninstalled from ~/Downloads
I downloaded latest VSCode insiders (1.80.0), put in Applications folder
I installed only C# Dev Kit insiders
Restart VSCode
Open C# Project

Now in the C# log I only see this. Does this mean it's running?
Dotnet path: /Users/.../Library/Application Support/Code - Insiders/User/globalStorage/ms-dotnettools.vscode-dotnet-runtime/.dotnet/7.0.8/dotnet
Activating C# + C# Dev Kit + C# IntelliCode...
info: LanguageServerHost[0]
      Starting server...

When opening the Solution Explorer it says A compatible.NET SDK was not found and in the Projects Log it says
2023-06-30 06:43:31.661 [info] Unhandled exception. System.IO.FileLoadException: Could not load file or assembly 'Microsoft.VisualStudio.ProjectSystem.Server.BuildHost, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'.
[createdump] thread_get_state(55f09) FAILED (os/kern) invalid argument (4)
[createdump] Failure took 9ms

2023-06-30 06:43:33.625 [info] Project system initialization finished. 0 project(s) are loaded, and 5 failed to load.

@jimAtLoyal Currently you will need to have matching architecture for the SDK and extensions as the extensions need to load components out of the installed SDK. If it is necessary to stay on x64, you could install an x64 build of VS Code. Otherwise, you will need to install an arm64 build of the .NET SDK.

I have both x64 and arm64 of .NET SDK 7 installed.

Now in the C# log I only see this. Does this mean it's running?

Yup looks like the C# lsp server is running fine now.
But based on the other logs you got from devkit, devkit is failing to load the project.

When opening the Solution Explorer it says A compatible.NET SDK was not found and in the Projects Log it says

Is the project you're trying to load also targeting .net 7?  Or do you have a global.json pointing it at a different SDK?
Devkit could also be getting confused by the presence of both the x64 and arm64 version of the sdk, but I'm not familiar enough with their side to know if that could trip them up.  Tagging @lifengl and @tmeschter for the devkit errors.

Since the LanguageServer is working, should this be closed and the Project issue reported elsewhere?
No global.json TFM is net7.0
> /usr/local/share/dotnet/x64/dotnet --version
7.0.305
> /usr/local/share/dotnet/dotnet --version
7.0.305

> dotnet sdk  check
.NET SDKs:
Version      Status
----------------------------------------
6.0.408      Patch 6.0.411 is available.
7.0.201      Patch 7.0.203 is available.
7.0.305      Up to date.

Try out the newest .NET SDK features with .NET 8.0.100-preview.5.23303.2.

.NET Runtimes:
Name                          Version      Status
---------------------------------------------------------------------
Microsoft.AspNetCore.App      6.0.16       Patch 6.0.19 is available.
Microsoft.NETCore.App         6.0.16       Patch 6.0.19 is available.
Microsoft.AspNetCore.App      7.0.3        Patch 7.0.8 is available.
Microsoft.NETCore.App         7.0.3        Patch 7.0.8 is available.
Microsoft.AspNetCore.App      7.0.8        Up to date.

I happened to run code --version this morning, and even though I'm on an M1, it is say x64, if that makes any difference.
> code --version
1.80.0-insider
7406a9bdfca18f82a3977ba5eac908d41df7ccac
x64

I've noticed that the universal VS Code install runs as x64 under Rosetta. It is a very interesting configuration you have. It might be worth a shot simply installing the apple silicon build of VS Code. https://code.visualstudio.com/download

Still can't find a compatible SDK when running the arm version
> code --version
1.80.0-insider
660393deaaa6d1996740ff4880f1bad43768c814
arm64

Sorry for all the back and forth on finding a solution. This shows we should do more testing with non-standard configurations.

I have to install the x64 version and alias dotnet to /usr/local/share/dotnet/x64/dotnet not the arm64.

Now that you have the arm64 VS Code and arm64 builds of the extensions installed. This seems to be the last bit that might be confusing things. Does it work if you remove the x64 dotnet alias?

I am running arm64 versions of .net 7 and vs code and get the same error.

same issue here

same issue here

same here

here some more from the C# dev kit output in vs code
Starting Open a solution...
Starting Open a solution with environment service...
Starting Clear environment...
Starting Spawn .NET server...
.NET server STDERR: Failed to load /Users/jhunter/Library/Application Support/Code/User/globalStorage/ms-dotnettools.vscode-dotnet-runtime/.dotnet/7.0.9/host/fxr/7.0.9/libhostfxr.dylib, error: dlopen(/Users/jhunter/Library/Application Support/Code/User/globalStorage/ms-dotnettools.vscode-dotnet-runtime/.dotnet/7.0.9/host/fxr/7.0.9/libhostfxr.dylib, 0x0001): tried: '/Users/jhunter/Library/Application Support/Code/User/globalStorage/ms-dotnettools.vscode-dotnet-runtime/.dotnet/7.0.9/host/fxr/7.0.9/libhostfxr.dylib' (mach-o file, but is an incompatible architecture (have 'x86_64', need 'arm64')), '/System/Volumes/Preboot/Cryptexes/OS/Users/jhunter/Library/Application Support/Code/User/globalStorage/ms-dotnettools.vscode-dotnet-runtime/.dotnet/7.0.9/host/fxr/7.0.9/libhostfxr.dylib' (no such file), '/Users/jhunter/Library/Application Support/Code/User/globalStorage/ms-dotnettools.vscode-dotnet-runtime/.dotnet/7.0.9/host/fxr/7.0.9/libhostfxr.dylib' (mach-o file, but is an incompatible architecture (have 'x86_64', need 'arm64'))
The library libhostfxr.dylib was found, but loading it from /Users/jhunter/Library/Application Support/Code/User/globalStorage/ms-dotnettools.vscode-dotnet-runtime/.dotnet/7.0.9/host/fxr/7.0.9/libhostfxr.dylib failed
  - Installing .NET prerequisites might help resolve this problem.
     https://go.microsoft.com/fwlink/?linkid=2063366

Dotnet path: /Users/haowang/Library/Application Support/Code/User/globalStorage/ms-dotnettools.vscode-dotnet-runtime/.dotnet/7.0.10/dotnet
Activating C# standalone...
Unhandled exception. System.IO.FileLoadException: Could not load file or assembly 'Microsoft.CodeAnalysis.LanguageServer, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'.
[Error - 2:12:52 PM] Server initialization failed.
Message: Pending response rejected since connection got disposed
Code: -32097
[Info  - 2:12:52 PM] Connection to server got closed. Server will restart.
true
[Error - 2:12:52 PM] Microsoft.CodeAnalysis.LanguageServer client: couldn't create connection to server.
Message: Pending response rejected since connection got disposed
Code: -32097
Dotnet path: /Users/haowang/Library/Application Support/Code/User/globalStorage/ms-dotnettools.vscode-dotnet-runtime/.dotnet/7.0.10/dotnet
Activating C# standalone...
Unhandled exception. System.IO.FileLoadException: Could not load file or assembly 'Microsoft.CodeAnalysis.LanguageServer, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'.
[Error - 2:12:53 PM] Server initialization failed.
Message: Pending response rejected since connection got disposed
Code: -32097
[Info  - 2:12:53 PM] Connection to server got closed. Server will restart.
true
[Error - 2:12:53 PM] Microsoft.CodeAnalysis.LanguageServer client: couldn't create connection to server.
Message: Pending response rejected since connection got disposed
Code: -32097
Dotnet path: /Users/haowang/Library/Application Support/Code/User/globalStorage/ms-dotnettools.vscode-dotnet-runtime/.dotnet/7.0.10/dotnet
Activating C# standalone...
Unhandled exception. System.IO.FileLoadException: Could not load file or assembly 'Microsoft.CodeAnalysis.LanguageServer, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'.
[Error - 2:12:53 PM] Server initialization failed.
Message: Pending response rejected since connection got disposed
Code: -32097
[Info  - 2:12:53 PM] Connection to server got closed. Server will restart.
true
[Error - 2:12:53 PM] Microsoft.CodeAnalysis.LanguageServer client: couldn't create connection to server.
Message: Pending response rejected since connection got disposed
Code: -32097
[Error - 2:12:53 PM] Restarting server failed
Message: Pending response rejected since connection got disposed
Code: -32097
Dotnet path: /Users/haowang/Library/Application Support/Code/User/globalStorage/ms-dotnettools.vscode-dotnet-runtime/.dotnet/7.0.10/dotnet
Activating C# standalone...
Unhandled exception. System.IO.FileLoadException: Could not load file or assembly 'Microsoft.CodeAnalysis.LanguageServer, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'.
[Error - 2:12:53 PM] Server initialization failed.
Message: Pending response rejected since connection got disposed
Code: -32097
[Info  - 2:12:53 PM] Connection to server got closed. Server will restart.
true
[Error - 2:12:53 PM] Microsoft.CodeAnalysis.LanguageServer client: couldn't create connection to server.
Message: Pending response rejected since connection got disposed
Code: -32097
[Error - 2:12:53 PM] Restarting server failed
Message: Pending response rejected since connection got disposed
Code: -32097
Dotnet path: /Users/haowang/Library/Application Support/Code/User/globalStorage/ms-dotnettools.vscode-dotnet-runtime/.dotnet/7.0.10/dotnet
Activating C# standalone...
Unhandled exception. System.IO.FileLoadException: Could not load file or assembly 'Microsoft.CodeAnalysis.LanguageServer, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'.
[Error - 2:12:53 PM] Server initialization failed.
Message: Pending response rejected since connection got disposed
Code: -32097
[Error - 2:12:53 PM] The Microsoft.CodeAnalysis.LanguageServer server crashed 5 times in the last 3 minutes. The server will not be restarted. See the output for more information.
[Error - 2:12:53 PM] Microsoft.CodeAnalysis.LanguageServer client: couldn't create connection to server.
Message: Pending response rejected since connection got disposed
Code: -32097
[Error - 2:12:53 PM] Restarting server failed
Message: Pending response rejected since connection got disposed
Code: -32097

same issue, but I have .net 5 project run x84 version sdk.

We've made a few fixes in this area (for example loading dotnet from the path when available) and do have one more mac related fixes on the way.
To anyone encountering this issue - would you be able to try the latest versions of C# (and devkit if applicable)?  If it still is not working, please attach the C# output window logs - there will hopefully be some additional log info there we can use.
Additionally - could you let me know if dotnet has been installed via homebrew?

installed C# Dev kit 0.5.147 version, i got message from project window
2023-10-04 16:41:57.630 [info] Unhandled exception. System.IO.FileLoadException: Could not load file or assembly 'Microsoft.VisualStudio.ProjectSystem.Server.BuildHost, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'.

2023-10-04 16:41:57.650 [info] .NET SDK for X64 CPU may have been installed on an Apple computer with Apple CPU, which does not work with C# Dev Kit. Please uninstall it and reinstall SDK for ARM 64 platform before using C# Dev Kit. A reboot might be necessary after installing the new SDK.
2023-10-04 16:41:57.652 [info] [createdump] thread_get_state(322f) FAILED (os/kern) invalid argument (4)
[createdump] Failure took 29ms

and C# window output
[stderr] Unhandled exception. [stderr] System.IO.FileLoadException: Could not load file or assembly 'Microsoft.CodeAnalysis.LanguageServer, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'.
[Error - 5:12:37 PM] Microsoft.CodeAnalysis.LanguageServer client: couldn't create connection to server.
Error: Timeout. Named pipe information not received from server.
	at N.<anonymous> (/Users/wenming/.vscode/extensions/ms-dotnettools.csharp-2.4.4-darwin-arm64/dist/extension.js:2:1441424)
	at Generator.next (<anonymous>)
	at s (/Users/wenming/.vscode/extensions/ms-dotnettools.csharp-2.4.4-darwin-arm64/dist/extension.js:2:1431992)

dotnet info
.NET SDK:
 Version:   8.0.100-rc.1.23463.5
 Commit:    e7f4de8816

Runtime Environment:
 OS Name:     Mac OS X
 OS Version:  13.5
 OS Platform: Darwin
 RID:         osx-x64
 Base Path:   /usr/local/share/dotnet/x64/sdk/8.0.100-rc.1.23463.5/

.NET workloads installed:
There are no installed workloads to display.

Host:
  Version:      8.0.0-rc.1.23419.4
  Architecture: x64
  Commit:       92959931a3
  RID:          osx-x64

.NET SDKs installed:
  5.0.407 [/usr/local/share/dotnet/x64/sdk]
  6.0.202 [/usr/local/share/dotnet/x64/sdk]
  6.0.411 [/usr/local/share/dotnet/x64/sdk]
  7.0.401 [/usr/local/share/dotnet/x64/sdk]
  8.0.100-rc.1.23463.5 [/usr/local/share/dotnet/x64/sdk]

.NET runtimes installed:
  Microsoft.AspNetCore.App 5.0.16 [/usr/local/share/dotnet/x64/shared/Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 6.0.4 [/usr/local/share/dotnet/x64/shared/Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 6.0.19 [/usr/local/share/dotnet/x64/shared/Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 7.0.11 [/usr/local/share/dotnet/x64/shared/Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 8.0.0-rc.1.23421.29 [/usr/local/share/dotnet/x64/shared/Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 5.0.16 [/usr/local/share/dotnet/x64/shared/Microsoft.NETCore.App]
  Microsoft.NETCore.App 6.0.4 [/usr/local/share/dotnet/x64/shared/Microsoft.NETCore.App]
  Microsoft.NETCore.App 6.0.19 [/usr/local/share/dotnet/x64/shared/Microsoft.NETCore.App]
  Microsoft.NETCore.App 7.0.11 [/usr/local/share/dotnet/x64/shared/Microsoft.NETCore.App]
  Microsoft.NETCore.App 8.0.0-rc.1.23419.4 [/usr/local/share/dotnet/x64/shared/Microsoft.NETCore.App]

my mac was M1 chip, macOS 13.5.1

Now that it is GA, any update? I re-enabled it today on my M1 and still get errors.

@jimAtLoyal this is still on my radar, but we've been busy tackling a few high hitting issues.  It looks like the error you're getting is a bit different now (we've changed a bunch of things in relation to process launch and dotnet discovery).  I've not been able to reproduce this specific issue on my M1 (though I've encountered others in relation to project loading with both arm64 and x64 dotnet installed).
That error means that the client failed to connect to the process - generally that happens when the process fails to start, but that doesn't appear to be happening in this case.  A couple requests

If you haven't already, set dotnet.server.trace to Trace
You can also provide a (user writeable) path in dotnet.server.crashDumpPath which will capture a dump if the process crashes (not sure if it is).  If it does, please email it to me (listed in my github profile) and reference this issue.
When it hits that, is the server process running?  Running ps -ef | grep Microsoft.CodeAnalysis.LanguageServer should show you something if its running.

One more thing - can you share the output of where dotnet and include the original path if they are symlinks (via readlink <path from where>)?

I set the three values you mentioned in my VSCode settings.
    "dotnet.server.trace": "Trace",
    "dotnet.server.crashDumpPath": "/var/folders/wt/48syr7hn5qs3qw_vbs0gl3l00000gn/T/vscodeump", # TMPDIR/vscodedump
    "dotnet.server.startTimeout": 30000

> which dotnet
/usr/local/share/dotnet/x64/dotnet

I restarted and got this in the log, but nothing in the dump folder. There is an invalid argument message. Quite a few mainly empty files get created in $TMPDIR
2023-10-10 07:19:46.659 [info] Unhandled exception. System.IO.FileLoadException: Could not load file or assembly 'Microsoft.VisualStudio.ProjectSystem.Server.BuildHost, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'.
[createdump] thread_get_state(172e6) FAILED (os/kern) invalid argument (4)
[createdump] Failure took 8ms

2023-10-10 07:19:46.971 [info] .NET SDK for X64 CPU may have been installed on an Apple computer with Apple CPU, which does not work with C# Dev Kit. Please uninstall it and reinstall SDK for ARM 64 platform before using C# Dev Kit. A reboot might be necessary after installing the new SDK.
2023-10-10 07:19:49.046 [info] Project system initialization finished. 0 project(s) are loaded, and 9 failed to load.

And ps
ps -ef | grep Microsoft.CodeAnalysis.LanguageServer
  501 15996 15900   0  7:19AM ??         0:05.32 /Users/me/Library/Application Support/Code - Insiders/User/globalStorage/ms-dotnettools.vscode-dotnet-runtime/.dotnet/7.0.11~arm64/dotnet /Users/me/.vscode-insiders/extensions/ms-dotnettools.csharp-2.5.30-darwin-arm64/.roslyn/Microsoft.CodeAnalysis.LanguageServer.dll --logLevel Trace --starredCompletionComponentPath /Users/me/.vscode-insiders/extensions/ms-dotnettools.vscodeintellicode-csharp-0.1.26-darwin-arm64/components/starred-suggestions/node_modules/@vsintellicode/starred-suggestions-csharp --sharedDependencies<EMAIL_ADDRESS>--extension<EMAIL_ADDRESS>--sessionId 482b3a75-5560-444e-9130-757c49e00c861696936782289 --telemetryLevel all --extensionLogDirectory /Users/me/Library/Application Support/Code - Insiders/logs/20231010T071941/window1/exthost/ms-dotnettools.csharp

So this is where I'm at

The dotnet on the path is the x64 version of dotnet.  The server therefore falls back to the dotnet runtime download because its arm64.
For some reason - the server is crashing when running against the arm64 runtime it downloaded.  I'm not sure why this is happening - I haven't been able to reproduce this.  It might be a corrupt installation.
Collecting the crash dump appears to be failing.  Pretty sure this is because dotnet-dump is running using the x64 dotnet on the path - and dotnet-dump does not support collecting dumps on the Rosetta emulation (see https://github.com/dotnet/runtime/issues/90938)

I have two things you can try.  The first is to reorder which SDK architectures are on your path to ensure that VSCode starts with the $PATH pointing to the arm64 version of dotnet.  First make sure VSCode is entirely closed (not just minimized).  Then, edit your shell profile to ensure the arm64 version is ahead of the x64 version of dotnet.  I was able to do this via editing my .zshrc file (since I'm on a zsh shell) to look something like:
export PATH="/usr/local/share/dotnet:/usr/local/share/x64:$PATH"

This ensures that the arm64 dotnet installation is seen on the path before the x64 version.  My where output looks kind of weird after with multiple of the same thing in the path, but importantly the arm64 path is at the top
dabarbet@Davids-MacBook-Pro ~ % where dotnet
/usr/local/share/dotnet/dotnet
/usr/local/share/dotnet/x64/dotnet
/usr/local/share/dotnet/x64/dotnet
/usr/local/share/dotnet/dotnet
/usr/local/share/dotnet/x64/dotnet

Then running source ~/.zshrc and code <my repo> should cause the server to pick up the arm64 version of dotnet and you'll see this line in the output window which points to the arm64 dotnet.
Dotnet path: /usr/local/share/dotnet/dotnet

If that doesn't help at all, then I would try the following

Completely remove all existing dotnet installations (if you installed via homebrew, uninstall that as well).  Basically you want /usr/local/share/dotnet to not exist, and remove anything configuring a dotnet path from your environment variables in your profile.  See also https://learn.microsoft.com/en-us/dotnet/core/install/remove-runtime-sdk-versions?pivots=os-macos
At this point dotnet --info and where dotnet should not work
Delete everything in /Users/<User>/Library/Application Support/Code - Insiders/User/globalStorage/ms-dotnettools.vscode-dotnet-runtime/
Install only the arm64 dotnet SDK and check if you are able to open a project successfully with VSCode.
Install an x64 SDK (if you need it), and verify that you're able to open a project successfully.

Thanks. Setting the path before launching code fixed it! "It's always easy when you know the answer"
And during this process I found by upgrading the Confluent NuGet, I no longer need the x64 version.


Page being "type": "page" and having a sidebar nontheless

I have an index file that is of the "type": "page", meaning it has no sidebar, which is the desired behavior for the index file.
I have a navbar item ("documentation") that also has the "type": "page" because that is what's necessary to have it on the navbar.
When the user opens said Documentation by clicking this navbar link, he needs to be able to see the sidebar, though, to be able to navigate the documentation. Setting the "sidebar" to true in the "theme" object in _meta.json for the documentation file does not work.
{
  "index": {
    "title": "About",
    "type": "page",
    "display": "hidden"
  },
  "documentation": {
    "title": "Documentation",
    "type": "page",
    "theme": {
      "breadcrumb": true,
      "footer": true,
      "sidebar": true,
      "toc": true,
      "pagination": true
    }
  }
}

Any solution to this? This has to be possible, right?

@gianlucajahn  Can you provide a link to an example?

Please provide a reproduction

closing due to silence


IAuthorizationPolicy Evaluate() called after ServiceAuthorizationManager CheckAccessCore(): why?

The operationContext.ServiceSecurityContext.AuthorizationContext.ClaimSets is populated at the time the enumerable is "touched" by mean of a foreach of a GetEnumerator.
This is good to know that the call to the registered IAuthorizationPolicy's is not done prior to entering the CheckAccessCore() method but is delayed to the time the Claimset colleciton is enumerated.
Thank to this post -> https://social.msdn.microsoft.com/Forums/en-US/455bf97a-cc80-4c3b-82cd-82d1a2ef3b0b/iauthorizationpolicy-evaluate-called-after-serviceauthorizationmanager-checkaccesscore-why?forum=netfxnetcom

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

ID: 671f4c42-b796-3cd1-058f-b23e5043e582
Version Independent ID: d146f4e0-61ed-3594-600d-76cd4ef1e657
Content: How to: Create a Custom Authorization Manager for a Service
Content Source: docs/framework/wcf/extending/how-to-create-a-custom-authorization-manager-for-a-service.md
Product: dotnet-framework
Technology: dotnet-wcf
GitHub Login: @HongGit
Microsoft Alias: wcfsrvt

Hello. I'm trying to close the loop on these older issues. I'm having a hard time understanding what specific part of the documentation you're referring to and what your suggestion is to fix it. Can you help me understand a bit more?

@HongGit  Perhaps you or your team can handle this?

@mconnew?

Sent email

Sorry for the slow response. As you'll see, this isn't a quick 3 line answer so I needed to find the time to write it up.
Let's start by going over what the purpose of the IAuthorizationPolicy.Evaluate method is. It enables:

Adding more authorization policies
Add properties to the EvaluationContext
Add Claim's to the EvaluationContext

So the reason why you would want the Evaluate method to be called is because you require either of 2 or 3 in your code somewhere (1 just effectively results in more of 2 and 3 happening). The results of these are exposed by the class AuthorizationContext. This means the timing of the call to IAuthorizationPolicy.Evaluate only needs to happen before you need access any properties on AuthorizationContext.
The AuthorizationContext is lazily created so unless you get the AuthorizationContext, you won't cause the Evaluate methods to be called. The timing of calling IAuthorizationPolicy.Evaluate is dependent on when the AuthorizationContext is needed. If you look at the code for ServiceAuthorizationManager.CheckAccessCore here, you can see that the steps are basically:

Call GetAuthorizationPolicies to get collection of IAuthorizationPolicy instances.
Create a new instance of ServiceSecurityContext holding the authorization policies.
Set the ServiceSecurityContext on the property operationContext.IncomingMessageProperties.Security.ServiceSecurityContext

The thing to note is that it's a collection of IAuthorizationPolicy instances that were passed to the constructor of ServiceSecurityContext, not an AuthorizationContext or EvaluationContext. ServiceSecurityContext will lazily create an AuthorizationContext using the passed policies when the AuthorizationContext property is accessed. It's during the creation of the AuthorizationContext that an EvaluationContext is created and the Evaluate method is called on the IAuthorizationPolicy instances. If you don't need to check the AuthorizationContext  in the CheckAccessCore method, there's no need to do the extra work of instantiating one. An example scenario where this might be the case is if you are checking for the presence of a specific HTTP header in CheckAccessCore so don't need to access anything on ServiceSecurityContext.
If you don't access ServiceSecurityContext.AuthorizationContext in your code (or any of the properties which use it such as PrimaryIdentity), it's possible the authorization policies are never evaluated. There are some code paths in WCF which will cause the AuthorizationContext to be executed. For example, the method which calls ServiceAuthorizationManager.CheckAccess has code which will accesses ServiceSecurityContext.AuthorizationContext if security auditing is enabled for successful service authorization.
So basically if you need to access the claims from AuthorizationContext, then get them via the property operationContext.IncomingMessageProperties.Security.ServiceSecurityContext.AuthorizationContext.ClaimSets and the authorization policies will be evaluated and will generate any necessary claims.
As the policies are evaluated at the point you first need the output of them being evaluated, it shouldn't matter when they are evaluated as the results will be available when you need them. Because you seem to care about when it happens, it makes me concerned that you might be relying on a side effect of calling the Evaluate method. If that's that case, it's a code smell and you are probably misusing the feature and likely need to achieve what you want a different way. If you don't want to go down that path and just want an easy way to trigger calling Evaluate, just get the ServiceSecurityContext.AuthorizationContext property at the point you expect it to be done (e.g. in the CheckAccessCore method). There's no negative side effects for WCF of triggering this earlier than you need, it's just CPU cycles we're optimistically trying to avoid if not necessary.


Proposing changing created_at format to include seconds, to fix library order

Closes https://github.com/sanger/traction-ui/issues/1993
Changes proposed in this pull request

The idea is to change the format of the created_at data, returned to the Traction UI from the Traction Service, so that it includes the seconds yyyy-mm-dd hh:mm:ss. This is to help solve the issue where two libraries are created within the same minute but the ordering is not predictably by the latest, resulting in barcodes being swapped.
If this does solve the problem, maybe updating timestamps through out the application to include seconds could prevent similar issues for other resources. (Changing from .to_fs(:us) to .to_fs(:db)). I also can’t seem to find any documentation on  .to_fs(:us), so not sure if that format is something we’re specifying?
The new format would be e.g. 2024-10-11 13:04:47, which is different to the old format 2024/10/11 12:50. We would need to check with the users if this is okay, or possibly reformat it in the front end to match the old, or, if we do decide to update them all to include seconds, maybe them all being consistent with the new format is okay

PacBio Libraries Page:
Before

After

Instructions for Reviewers

Start up Traction Service and UI
From PacBio Sample page, create two libraries within the same minute
Go to PacBio Library page, and they should be ordered by default with the most recent at the top
The UI should show the seconds.

Just thinking about this a bit further based on Steve's message in the traction slack - If we are bulk creating libraries they could be created in the same second which may cause issues as well. Not sure if thats worth considering here or not.

Just thinking about this a bit further based on Steve's message in the traction slack - If we are planning on bulk creating libraries they could be created in the same second which may cause issues as well. Not sure if thats worth considering here or not.

It is worth considering. This is partly why I wanted to use the id in reverse order.


How can I make my Topshelf service stop, when a certain type of exception happens?

If your service's Start method throws an exception, the service will fail to start.
Once the service is running, if an unhandled exception is thrown, the service will stop, and report it as a crash to the service control manager. This will allow any service restart behavior configured in the services configuration to take over as it was an abnormal exit.
If you need to Stop your service programatically, use the HostControl method Stop. The HostControl interface is passed to your service class's Start method when your class implements the ServiceControl interface. If you are using the delegate-based WhenStart... methods, well, switch to ServiceControl.

In my case, it doesn't responses when an AggregateException is thrown from Parallel.ForEach loop. Service status is running but service seems stuck and nothing happen.
Even AppDomain.CurrentDomain.UnhandledException didn't take care as well as OS's Application event is not logged.

A code example of what you're seeing (as a Gist, etc.) really helps to understand the situation and the unexpected behavior. If you could post something that I could use to reproduce the issue, that would be very helpful.
All of the stock sample services show throwing exceptions, etc. and they work properly, so you may have encountered a situation that is unexpected.

While testing a custom logger (using HostLogger.UseLogger) I realized that Unhandled Exceptions thrown from other threads do not seem to be catched. The service seems to just continue without any notification that anything went wrong. Exceptions thrown from the main thread seems to be catched fine.

How are you generating your threads? Tasks, by default, I believe capture exceptions. If the exception doesn't bubble up to the app domain, there's no way for it be caught there.

Task.Run(() => {
// your thrown exception here won't stop the service
});

new Thread(() => {
// your exception here will stop the service
}).Start();

Fast forward to 2018, TopShelf has HostControl.Stop(TopshelfExitCode). https://github.com/Topshelf/Topshelf/commit/fc9bff9e99947fbdc8a889f286cebae6fe08702a
I chose to use this to stop the service host with TopshelfExitCode.UnhandledServiceException anytime I hit an unhandled exception in my async run thread. It does work to stop the windows service, and generate an Error windows system event log. The comment on the commit indicates that the recovery should trigger.

This allows services to signal recovery states to the Windows SCM

My service is set to recovery option " restart the service". However, it does not cause the recovery options to trigger.


Ref 18 miss PDF link

To maintain a consistent style with other references,
I suggest add the pdf link of Reference 18
(Evolution strategies as a scalable alternative to reinforcement learning):
https://arxiv.org/pdf/1703.03864

Thanks, not sure how we missed that!


Managed Service Provider Request for Bids

Sent out RFP to 3 companies. Have been answering questions and providing more information as needed to keep conversation moving.

Heard from Chris about end of service. Ardiah out as possible vendor. Suggested Rock Bound, will probably have to go back to them after exhausting smaller businesses.


Solve #2

Okay I wanted to have different pull request for each issue, but the readme commit keeps popping up with it. I think I should have made the branch from a different commit.
Let me know if you want me to do it all again

Okay this commit should do. Sorry again for the hiccup commit on the README on this pull request, I am still trying to get the hang of git.

@VladKappa cool


Preserving images when writing to output

First of all, this project is great, I was able to read in an Excel file with tons of complex formatting and write it back out with all the formatting intact, with only few things missing.
The images are missing.

Looks like you guys have this one reported:
https://github.com/guyonroche/exceljs/issues/34


Fix nightly perf test

Description
Changes proposed in this pull request:

Remove dependency to gcloud

Related issue(s)

Helm is probably still missing: https://status.build.kyma-project.io/view/gs/kyma-prow-logs/logs/nightly-istio-performance-test-gcp/1676018101751123968#1:build-log.txt%3A273

I missed the Helm error. With the dependency to Helm I don't know if we are not better off using the e2e-gcloud image.


Revert "Skip services with clustername node-admin"

Reverts vespa-engine/vespa#8020

https://github.com/vespa-engine/vespa/pull/8025 instead


Add S3 buckets for mysql_xtrabackups

This creates buckets to allow streaming MySQL backups.
Please note a bucket has already been deployed in the test environment.

:+1:


Implement tokens

Is your feature request related to a problem? Please describe.
N/A
Describe the solution you'd like
Would like a "token" resource that implements

post /users/{username}/tokens
get /users/{username}/tokens
delete /users/{username}/tokens/{token}

Describe alternatives you've considered
None.
Additional context
N/A

I am willing to try to implement this or to test it if someone else implements it. I am unsure how to do this since go-sdk/gitea doesn't seem to support it. ATM I'm thinking of implementing as, basically, a resource wrapper around the necessary curl commands.

implemented via c72fcb3a49cdc2d64e17d4b1361d010efb3f9f7c and released in v0.10.0


Enable navigationPreload in the service worker

Hey folks! I've been looking at the Squoosh service worker code as an inspiration as I've been building Wormhole. Thanks for open sourcing this – it's quite helpful!
One question – is there a reason why you haven't enabled the navigationPreload option? I've found that it causes extra network requests even for cached content and was wondering if perhaps that's why you've not enabled it here.
Apologies if this isn't the correct forum for this question. Thanks again!

Navigation preload is only useful if you need content from the network as part of a navigation.
For instance, an app like Twitter may do a navigation preload of the main timeline JSON, so that request can start even while the service worker is still spinning up.
With Squoosh we don't need any network data as part of our render - it's entirely cached, so there's nothing to preload as part of navigation preload.
(Closing this, but I'm happy to continue discussing)

@jakearchibald Thanks for the response!

an app like Twitter may do a navigation preload of the main timeline JSON

Can you actually select which requests get preloaded with navigation preload? It seemed to me like the API was all-or-nothing. Of course, you can avoid using the preloadResponse  but won't the HTTP requests be made anyway?

Can you actually select which requests get preloaded with navigation preload?

Nah, but you can set a header to go along with the request. That means the server can return something different for nav-preload requests. I guess it could return an empty response in places where you know you won't use the response.


Support Fallback Sink

for reference - https://github.com/numaproj/numaflow-go/pull/120

fixed #49


zoom-us killed by SIGILL

Describe the bug
I notice zoom-us from nixpkgs-unstable gets killed by SIGILL shortly after the application window is shown. This does not occur when running zoom-us from NixOS 21.11.
A strace ends with the following output:
clone(child_stack=NULL, flags=CLONE_CHILD_CLEARTID|CLONE_CHILD_SETTID|SIGCHLD, child_tidptr=0x7f3f8f5caa10) = 2145086
write(2, "Start subprocess: /nix/store/h78js36whvrbgcaz5nrmr0i9byxppaad-zoom-<IP_ADDRESS>45/opt/zoom/zoom sucessfully,  process pid: 2145086 \n", 129) = 129
wait4(-1, 0x7ffc19443d5c, 0, NULL)      = 2145086
--- SIGCHLD {si_signo=SIGCHLD, si_code=CLD_KILLED, si_pid=2145086, si_uid=1000, si_status=SIGILL, si_utime=47, si_stime=53} ---
write(2, "zoom was exited due to a handled signal: 4 \n", 44) = 44
write(2, "ZoomLauncher exit.\n", 19)    = 19

Steps To Reproduce
Observe that the zoom process is killed by SIGILL shortly after the application window is shown.
NIXPKGS_ALLOW_UNFREE=1 nix run --impure nixpkgs\#zoom-us

Expected behavior
The Zoom application proceeds to run without crashing.
Notify maintainers

@danbst @tadfisher  @doronbehar
Metadata
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.15.41, NixOS, 22.05 (Quokka)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.8.1`
 - channels(root): `"nixos-21.11, nixos-unstable"`
 - channels(seb): `"home-manager, unstable"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

@doronbehar I can‘t reproduce this issue, at least I get into a video call and to making faces at myself for a minute without issues. I am running zoom 5.10.4, on x86_64-linux with kernel 5.17.9.

From what I can muster, SIGILL is caught while in libcef.so. It looks like libcef.so is part of the binary distribution of Zoom and not built as part of nixpkgs? But I figure if @tomjnixon can run it with no crash it’s more likely an application bug rather than some ABI issue?
Core was generated by `/nix/store/h78js36whvrbgcaz5nrmr0i9byxppaad-zoom-<IP_ADDRESS>45/opt/zoom/.zoom '.
Program terminated with signal SIGILL, Illegal instruction.
#0  0x00007fc33e3c75a5 in ?? () from /nix/store/h78js36whvrbgcaz5nrmr0i9byxppaad-zoom-<IP_ADDRESS>45/opt/zoom/cef/libcef.so
[Current thread is 1 (Thread 0x7fc30bfff640 (LWP 3134979))]
>>> bt
#0  0x00007fc33e3c75a5 in ?? () from /nix/store/h78js36whvrbgcaz5nrmr0i9byxppaad-zoom-<IP_ADDRESS>45/opt/zoom/cef/libcef.so
#1  0x00007fc30bffd498 in ?? ()
#2  0x00007fc30bffd490 in ?? ()
#3  0x00007fc3441fdaa0 in ?? () from /nix/store/bvy2z17rzlvkx2sj7fy99ajm853yv898-glibc-2.34-210/lib/ld-linux-x86-64.so.2
#4  0x00007fc335d093a4 in dlsym_doit () from /nix/store/bvy2z17rzlvkx2sj7fy99ajm853yv898-glibc-2.34-210/lib/libc.so.6
#5  0x00007fc335dd1d15 in _dl_catch_exception () from /nix/store/bvy2z17rzlvkx2sj7fy99ajm853yv898-glibc-2.34-210/lib/libc.so.6

I now notice that one Zoom invocation results in two coredumps. The other one is a SIGSYS arising from libc. I’m not really versed in such dark magics, but FWIW:
Core was generated by `/nix/store/h78js36whvrbgcaz5nrmr0i9byxppaad-zoom-<IP_ADDRESS>45/opt/zoom/.zoom --t'.
Program terminated with signal SIGSYS, Bad system call.
#0  0x00007fc6ebef8be9 in clone3 () from /nix/store/bvy2z17rzlvkx2sj7fy99ajm853yv898-glibc-2.34-210/lib/libc.so.6
>>> bt
#0  0x00007fc6ebef8be9 in clone3 () from /nix/store/bvy2z17rzlvkx2sj7fy99ajm853yv898-glibc-2.34-210/lib/libc.so.6
#1  0x00007fc6ebef8c4d in __clone_internal () from /nix/store/bvy2z17rzlvkx2sj7fy99ajm853yv898-glibc-2.34-210/lib/libc.so.6
#2  0x00007fc6ebe75cb5 in create_thread () from /nix/store/bvy2z17rzlvkx2sj7fy99ajm853yv898-glibc-2.34-210/lib/libc.so.6
#3  0x00007fc6ebe76701 in<EMAIL_ADDRESS>() from /nix/store/bvy2z17rzlvkx2sj7fy99ajm853yv898-glibc-2.34-210/lib/libc.so.6
#4  0x00007fc6f45bf051 in ?? () from /nix/store/h78js36whvrbgcaz5nrmr0i9byxppaad-zoom-<IP_ADDRESS>45/opt/zoom/cef/libcef.so

I really don't expect to be able to debug this, but could you post the output of info proc mapping and bt (and maybe disassemble if the output is short, particularly around the => marker) from gdb on the SIGILL core?
To me this really feels like a zoom problem. You don't happen to be using an ancient or unusual processor do you?

It looks like version 5.10.6 (3192) is out for linux now, so it might be worth trying that too.

Cheers, that seems to resolve the problem.

You don't happen to be using an ancient or unusual processor do you?

Not really, it’s an i9-10885H.
In case you’re still curious for whatever reason, I’ve attached the information copied from gdb. Otherwise I’d say this issue is obsoleted by the new Zoom release. (NB disassemble just said “No function contains program counter for selected frame.”)
backtrace.txt
procmap.txt

my zoom is dying too, I got it working for now with zoom --no-sandbox

In case you’re still curious for whatever reason, I’ve attached the information copied from gdb. Otherwise I’d say this issue is obsoleted by the new Zoom release. (NB disassemble just said “No function contains program counter for selected frame.”)
backtrace.txt procmap.txt

Thanks, that's pretty interesting; something jumps into the middle of an instruction. Good that updating it works anyway.


changes made in json

These changes provide structure, improve project maintainability, and make it easier for collaborators or others to understand and work with the project.

@codingblasts where is the issue for this


Bing GitHub Action Alive

1. Describe the bug

https://github.com/apache/spark/pull/32096
Spark seems to have it fixed
2. Environments
2.1 Versions

Kyuubi

[ ] 1.1.0
[ ] 1.0.3
[ ] 0.8.x and earlier

Spark

[ ] 3.1.x
[ ] 3.0.x
[ ] 2.4.x and earlier

2.2 Kyuubi Configurations
2.3 Spark Configurations
3. Steps to reproduce
3.1 Operations
3.2 Expected behavior

3.3 Actual behavior
4. Additional context
4.1 Log of the server
4.2 Log of the corresponding engine

cc @turboFei


Trouble importing NonprofitOperationsToolkitFlows <IP_ADDRESS>

Hi.
I've been trying to import NonprofitOperationsTookitFlows <IP_ADDRESS> and get an error for a missing component - new_sharedcommondataserviceforapps_ea70f which is a Connection Reference required by Planning Period - calculate Weekdays.
Thanks.
Randy

Hi @vernius28939, can you please try to import in make.powerapps.com instead of the old classic interface? We believe that is what causes this error. Thank you!

Hi @vernius28939, can you please try to import in make.powerapps.com instead of the old classic interface? We believe that is what causes this error. Thank you!

Hi @klbyrd.
I tried importing using make.powerapps.com and still got an error.
Thanks.

Hi @klbyrd.
I tried importing using make.powerapps.com and still got an error.
Thanks.

@vernius28939 we have updated the flow. That should resolve your issue. https://github.com/microsoft/Industry-Accelerator-Nonprofit/releases/download/v<IP_ADDRESS>/NonprofitOperationsToolkitFlows_1_1_3_6_managed.zip

@vernius28939 we have updated the flow. That should resolve your issue. https://github.com/microsoft/Industry-Accelerator-Nonprofit/releases/download/v<IP_ADDRESS>/NonprofitOperationsToolkitFlows_1_1_3_6_managed.zip


Javascript lexer adds a line

Not sure if the problem is in Hugo or Chrome, but the following will end up being a two line code snippet:
var foo = "bar";

@kaushalmodi you have any experience with this?

It seems to be a problem with the Javascript lexer. Change it to Go and it behaves OK.

It seems to be a problem with the Javascript lexer. Change it to Go and it behaves OK.

Yes, I was going to ask what language that was. I have not seen issue; just tested with emacs-lisp and works OK.


UrlSync: Support browser history steps, remove singleton

Adds new function to SceneObjectUrlSyncHandler named shouldCreateHistoryStep that enable scene apps to control when new history steps should be created due to state changes
Adds new feature to UrlSyncManager to sync initial scene state to URL, this is important as we need this initial URL state to go back to after a say a time range state
remove getUrlSyncManager and remove the singleton nature of UrlSyncManager (needed for other reasons https://github.com/grafana/scenes/pull/841)

To not make this a big new behavior I am adding new options to control it

updateUrlOnInit
createBrowserHistorySteps (only SceneTimeRange is taking advantage of this)

can be set on SceneApp or UrlSyncContextProvider / useUrlSync
Question is should they be inverted and so the default state is enabled (ie updateUrlOnInit => disableUpdateUrlonInit,   createBrowserHistorySteps => disableBrowserHistorySteps), so it's more opt-out than opt-in
Release notes
getUrlSyncManager is no longer exported as UrlSyncManager is now no longer global singleton but local to the UrlSyncContextProvider.
If you called getUrlSyncManager().getUrlState that util function is available via the exported object sceneUtils.

@ivanortegaalba pushed some basic docs, we had zero docs on url syncing :)


[1.5.0-rc.5] Failed to load templateUrl path when enabling hmr

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

Versions.

Angular CLI: 1.5.0-rc.6
Node: 8.6.0
OS: linux x64
Angular: 5.0.0-rc.7
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router

@angular/cli: 1.5.0-rc.6
@angular-devkit/build-optimizer: 0.0.31
@angular-devkit/core: 0.0.20
@angular-devkit/schematics: 0.0.34
@ngtools/json-schema: 1.1.0
@ngtools/webpack: 1.8.0-rc.6
@schematics/angular: 0.0.48
typescript: 2.5.3
webpack: 3.8.1

Repro steps.

https://github.com/aitboudad/ng-cli-19994
ng serve --hmr -e=hmr

The log given by the failure.

@aitboudad heya, can you try doing ng serve --hmr -e=hmr --no-aot? With Angular 5 + CLI 1.5 serve defaults to AOT, and I suspect HMR doesn't work with AOT.

@filipesilva it's work with --no-aot when using hmr, I suggest to turn off aot as default

@filipesilva for the build part ng build --prod --build-optimizer -sm produce a large bundle size that contains @angular/compiler!!

I'll make a PR to add this information to the HMR docs.
I think you're getting @angular/compiler in your production bundle because of the import { createNewHosts } from '@angularclass/hmr'; import. To compile updated components on the fly it must be using the compiler.

another issue I've got when using lazy load route:
ERROR in ./src/app/child/child.module.ts
Module build failed: Error: ..../child/child.module.ts is not part of the compilation. Please make sure it is in your tsconfig via the 'files' or 'include' property.
    at AngularCompilerPlugin.getCompiledFile (node_modules/@ngtools/webpack/src/angular_compiler_plugin.js:693:23)
    at plugin.done.then (node_modules/@ngtools/webpack/src/loader.js:467:39)
    at <anonymous>
 @ ./src/$$_lazy_route_resource lazy
<EMAIL_ADDRESS> @ ./src/main.ts
 @ multi webpack-dev-server/client?http://<IP_ADDRESS>:9096/ webpack/hot/dev-server ./src/main.ts

That one seems very similar to what's reported in #8207, but I haven't been able to reproduce it yet. Can you tell me exactly how to reproduce it please? Also would like to know if it only happens with HMR on.

I think you're getting @angular/compiler in your production bundle because of the import { createNewHosts } from '@angularclass/hmr'; import. To compile updated components on the fly it must be using the compiler.

It's not hmr but on how we call bootstrapModule try with:
const bootstrap = () => platformBrowserDynamic().bootstrapModule(AppModule);
bootstrap();

and you'll noticed that the @angular/compiler is part of main bundle.

Thanks a lot for setting up that repro, it really helps to debug what's going on 👍

I'm trying your repo and this is what I get:

ng serve: builds correctly but fails at runtime with the following error:

GET http://localhost:4200/ui-select/ui-select.html 404 (Not Found)
Unhandled Promise rejection: Failed to load ../ui-select/ui-select.html ; Zone: <root> ; Task: Promise.then ; Value: Failed to load ../ui-select/ui-select.html undefined

ng serve --no-aot: fails to build with the following error:

ERROR in ./src/app/home/home.module.ts
Module build failed: Error: D:\sandbox\ng-cli-19994\src\app\home\home.module.ts is not part of the compilation. Please make sure it is in your tsconfig via the 'files' or 'include' property.
    at AngularCompilerPlugin.getCompiledFile (D:\sandbox\ng-cli-19994\node_modules\@ngtools\webpack\src\angular_compiler_plugin.js:693:23)
    at plugin.done.then (D:\sandbox\ng-cli-19994\node_modules\@ngtools\webpack\src\loader.js:467:39)
 @ ./src/$$_lazy_route_resource lazy
<EMAIL_ADDRESS> @ ./src/main.ts
 @ multi webpack-dev-server/client?http://<IP_ADDRESS>:0 ./src/main.ts

ng serve --hmr -e=hmr --no-aot: same as ng serve --no-aot

Investigating further.

I also tried your repro but with CLI 1.4.9, Angular packages at 4.4.6 and TypeScript at 2.3.2:

ng serve --aot: builds correctly and does not fail at runtime
ng serve --no-aot: same as above
ng serve --hmr -e=hmr --no-aot: same as ng serve --aot: same as above
ng serve --hmr -e=hmr --no-aot: same as ng serve --no-aot: same as above

So all of these worked before, but are not working with 1.5. This is a bug and will be fixed.

I'm working on these errors (and others) in https://github.com/angular/angular-cli/pull/8215.
It looks like there's some problem in the bootstrap replacement as you mentioned.
// doesn't work
const bootstrap = () => platformBrowserDynamic().bootstrapModule(AppModule);

// works
function bootstrap () {
  return platformBrowserDynamic().bootstrapModule(AppModule);
}

@filipesilva related to your comments here https://github.com/angular/angular-cli/issues/8207#issuecomment-340177756 and in this issue, what is the issue? I have tried, with or without aot and with or without hmr. no option works for me.
Is it a cli issue or angular's?

@asadsahi it is a CLI issue. Have you tried https://github.com/angular/angular-cli/issues/8234#issuecomment-340202362? (replacing the bootstrap)

@filipesilva with angular 5.0.0-rc.8 & angular-cli 1.5.0-rc.6 tried following:
aot + hmr = doesn't work
no aot + no hmr = doesn't work
I have tried with both bootstrap options:
const bootstrap = () => platformBrowserDynamic().bootstrapModule(AppModule);
function bootstrap () {
return platformBrowserDynamic().bootstrapModule(AppModule);
}
getting these errors:

[0]  @ multi webpack-dev-server/client?http://<IP_ADDRESS>:0 ./src/main.ts
[0] ERROR in ./src/app/+profile/profile.module.ts
[0] Module build failed: Error: c:\Users\Asad\ngnode-fullstack\src\app\+profile\profile.module.ts is not part of the compilation. Please make sure it is in your tsconfig via the 'files' or 'include' property.
[0]     at AngularCompilerPlugin.getCompiledFile (c:\Users\Asad\ngnode-fullstack\node_modules\@ngtools\webpack\src\angular_compiler_plugin.js:693:23)
[0]     at plugin.done.then (c:\Users\Asad\ngnode-fullstack\node_modules\@ngtools\webpack\src\loader.js:467:39)
[0]     at <anonymous>
[0]  @ ./src/$$_lazy_route_resource lazy
[0]<EMAIL_ADDRESS>[0]  @ ./src/main.ts
[0]  @ multi webpack-dev-server/client?http://<IP_ADDRESS>:0 ./src/main.ts
[0] ERROR in ./src/app/+register/register.module.ts
[0] Module build failed: Error: c:\Users\Asad\ngnode-fullstack\src\app\+register\register.module.ts is not part of the compilation. Please make sure it is in your tsconfig via the 'files' or 'include' property.

I can confirm that (locally) the fixes I have in https://github.com/angular/angular-cli/pull/8215 together with changing the bootstrap as I mention in https://github.com/angular/angular-cli/issues/8234#issuecomment-340202362 make all the scenarios that previously worked in 1.4.9 also work in 1.5.0 (https://github.com/angular/angular-cli/issues/8234#issuecomment-340186316).
I'm still working out the kinks out of that PR but it should be good to go for the next release. Thanks for bearing with me while I debug this.

Heya, pushed a new commit into https://github.com/angular/angular-cli/pull/8215 that should take care of the bootstrap problem.
@asadsahi the problems you're seeing should also be fixed in that PR.

@filipesilva should the fix be in angular-cli v1.5.0-rc.7 ? can't see any difference.
Also with cli v1.5.0+ is following bootstrap a permanent change?
function bootstrap() {
return platformBrowserDynamic().bootstrapModule(AppModule);
}

@asadsahi it's not merged yet, but should be merged for the next release. You should be able to use const bootstrap = () => platformBrowserDynamic().bootstrapModule(AppModule); when that PR is merged.

@aitboudad @asadsahi the fix for this is coming out as<EMAIL_ADDRESS>Please try it together with<EMAIL_ADDRESS>(and other Angular packages) and let me know if you're still seeing this.

@filipesilva initial step seems to be faster BUT unfortunatly  when I edit file the recompilation takes still:
on 0% = 30 seconds like before then the rest in 10seconds
Angular CLI: 1.5.0-rc.8
Node: 6.11.0
OS: win32 x64
Angular: 5.0.0-rc.8
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router

@angular/cli: 1.5.0-rc.8
@angular-devkit/build-optimizer: 0.0.31
@angular-devkit/core: 0.0.20
@angular-devkit/schematics: 0.0.34
@ngtools/json-schema: 1.1.0
@ngtools/webpack: 1.8.0-rc.8
@schematics/angular: 0.0.49
typescript: 2.5.3
webpack: 3.8.1

@filipesilva I can confirm that everything is working now :+1: , the only remark I have is the incremental update is a little bit slow for AOT mode

@istiti @aitboudad with CLI 1.5 and Angular 5, ng serve will default to using AOT. It should be much faster than before, but it seems it still isn't very fast in your projects. You can turn it off with --no-aot but I would also like to make it faster if I can.
Would you be interested in submitting a CPU profile showing where the time is spent on your rebuilds? I can use that to figure out bottlenecks. I put detailed instructions on how you can do that here: https://github.com/angular/angular-cli/issues/8259


Stateful model support

What does this PR do?

Support to convert and run stateful models.
TODO

[ ] Command line interface update
[ ] Testing on a wider range of topologies

Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Besides the first two items described in TODO we should revise Seq2seq models as well and make decoding part stateful as well: https://github.com/huggingface/optimum-intel/blob/main/optimum/intel/openvino/modeling_seq2seq.py#L412

@AlexKoff88, is it going to just cover more model architectures to be stateful without providing extra value to those architectures that have already been enabled in this PR? Can we do that in a separate PR? So we will introduce stateful capabilities gradually providing necessary API changes in this first PR?

Besides the first two items described in TODO we should revise Seq2seq models as well and make decoding part stateful as well: https://github.com/huggingface/optimum-intel/blob/main/optimum/intel/openvino/modeling_seq2seq.py#L412

@AlexKoff88, is it going to just cover more model architectures to be stateful without providing extra value to those architectures that have already been enabled in this PR? Can we do that in a separate PR? So we will introduce stateful capabilities gradually providing necessary API changes in this first PR?

I think we can do it in a separate PR when reworking Seq2seq models and getting rid of two decoder sub-models.

@echarlaix, I cannot push changes to https://github.com/huggingface/optimum-intel/pull/493, so I've continued development in my original PR with all changes from that PR. Could you approve workflows to allow checks running here?

This PR is completely superseded by https://github.com/huggingface/optimum-intel/pull/493.


Records  in table are always insert empty in the table

Hi We  are planing to upgrade the alssql  version  from .2 to 1.7.3.  We are using the browser version of alasql and download  from<EMAIL_ADDRESS>Got this link from official site (https://github.com/AlaSQL/alasql) and include in the our webpage. But with latest records  in table are always insert empty in the table.
We are trying to create the table 'BCPartners' and then insert the data into JSON formate. We can see the records set but each column has blank value.This is a blocker for us so need help ASAP
Below are the sample code and more details
-----------
Sample HTML  page
---------------------
<html>

<head></head>

<body>
    <script language="javascript" src="scripts/3rdParty/alasql.min.js"></script>
    <script language="javascript" src="test.js"></script>
    "CREATE TABLE IF NOT EXISTS [BCPartners] ( RecordNum INTEGER PRIMARY KEY AUTOINCREMENT , [TABLE_NAME] TEXT DEFAULT \"\", [BC] TEXT DEFAULT \"\")" =
    <script>
        var data = [

          	 {TABLE_NAME: "S_CON_ADDR", BC: "Pharma Address Inter Table - CE,CUT Address for Acâ€¦,Pharma Pick Address,CUT Address Inter Table - CE"},
          	 {TABLE_NAME: "S_PROD_INT", BC: "Pharma Internal Product,Internal Product,FINS Security,Pharma Sample Lot for Picklist"},
          	{TABLE_NAME: "S_LST_OF_VAL", BC: "List Of Values,PickList Generic,List Of Values Chiâ€¦ Of Values Dynamic,PickList Hierarchical Sub-Area"},
                   {TABLE_NAME: "S_EMP_PRDINT", BC: "Pharma Personal Products List,Pharma Personal Sampâ€¦Items List,Pharma Personal Promotional Items List"},
          	 {TABLE_NAME: "S_ADDR_PER", BC: "CUT Address,CUT Address - CE,Pharma Address"},
          	 {TABLE_NAME: "S_LIT", BC: "Admin Sales Tool,Admin Sales Tool Thumbnail"},
          	{TABLE_NAME: "S_EVT_ACT", BC: "Pharma Professional Call - Mobile,Pharma Account Call,Pharma Professional Call,Action"},
          	 {TABLE_NAME: "S_MSG_PLAN", BC: "LS Messaging Plans BC,LS Admin Messaging Plans BC"},
          	 {TABLE_NAME: "S_ACT_PRDINT", BC: "LS Pharma Call Sample Dropped Rollup,LS Pharma Proâ€¦motional Items Dropped,Pharma Call Sample Dropped"},

          ]

          function showResult(x){
              document.getElementById('result').textContent = JSON.stringify(x,  null, '\t');
          }

          CreateDB();
    </script>

    <xmp id="result"></xmp>
</body>

</html>

---------- Sample JS (test.js) code ---------
function CreateDB() {
	debugger
	var inmemDB = new alasql.Database("alasql_db");
	window._memDB = inmemDB;
	var
		query = "CREATE TABLE IF NOT EXISTS [BCPartners] ( RecordNum INTEGER PRIMARY KEY
	AUTOINCREMENT, [TABLE_NAME] TEXT DEFAULT\ "\", [BC] TEXT DEFAULT \"\")"
	inmemDB.exec(query);;
	inmemDB.exec("insert into [BCPartners] select * from ?
			",[data]);
}

Can I work on this issue

@harish343 Absolutely

how can I find this file scripts/3rdParty/alasql.min.js

Use<EMAIL_ADDRESS>instead so you dont get the minified version.

plz provide documentation and which database I can use with alasql

how can I check records table


DO NOT MERGE - integration: Use the brand new dind image.

We now have dockerswarm/dind built using https://github.com/aluzzardi/dind:

Much more lightweight.
Exists in every version since Docker 1.0.0.

Depends on #700 as it cannot run on aufs.
Signed-off-by: Andrea Luzzardi<EMAIL_ADDRESS>

image affinity seems to fail with this PR and on an unrelated note, docker cp is stuck on both master and this PR

@aluzzardi I wonder what you changed since the last image was able to run on aufs /cc @jfrazelle

@vieux not really sure but:
1/ It wasn't supposed to work: https://github.com/docker/docker/blob/master/daemon/graphdriver/aufs/aufs.go#L48
2/ @jfrazelle, correct me if I'm wrong, but we run jenkins on debian therefore no aufs anwyay?

there is aufs, we test all graph drivers, prs are all aufs
On Mon, May 4, 2015 at 11:47 AM, Andrea Luzzardi<EMAIL_ADDRESS>wrote:

@vieux https://github.com/vieux not really sure but:
1/ It wasn't supposed to work:
https://github.com/docker/docker/blob/master/daemon/graphdriver/aufs/aufs.go#L48
2/ @jfrazelle https://github.com/jfrazelle, correct me if I'm wrong,
but we run jenkins on debian therefore no aufs anwyay?
—
Reply to this email directly or view it on GitHub
https://github.com/docker/swarm/pull/703#issuecomment-98810553.


Jenkins: Concurrent processes fail when updating crates.io index simultaneously

Currnetly holding up:

#298

Example build failure:

Jenkins build 31

One way to fix it is to have dedicated $HOME for each of four workers.

This isn't really a problem anymore as we've switched to travis (at least for the time being).


Update the tutorial code to be compatible with windows crate version 0.35.0

Please update the documentation so that the code written in the tutorial is compatible with latest windows crate version 0.35.0. Currently using windows::build! macro gives an error saying build! macro not found. Even when I used windows::core::build! it still failed.

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

ID: 9a621c9b-3a97-3d49-4de9-b309a491a75b
Version Independent ID: 492ab689-9940-02ee-2f38-bafcd1dc9ad2
Content: RSS reader tutorial (Rust for Windows with VS Code)
Content Source: hub/dev-environment/rust/rss-reader-rust-for-windows.md
Product: dev-environment
Technology: windows-rust
GitHub Login: @stevewhims
Microsoft Alias: stwhi

Thanks for the heads up, @varunkamra. I've created a work item for myself (on my backlog) to take a look at this. That work item contains a link to this GitHub issue, so I'll go ahead and close this issue for now (so that I'm not tracking the work in two places), and I'll reopen this issue when I get to my work item.
Thanks again!
Steve

Sounds good, thank you! Although, would you mind telling me how to fix this issue? I haven’t been able to figure this out myself. Does build macro not exist anymore? I tried using windows::core::build!() but that did not work with 0.35.0 version so I had to use the 0.9.0 as mentioned in the doc to get it working.

It's been 2 months, looks like the sample code has not been updated as we are now on v .37, throws error mentioned in this thread:
Compiling bindings v0.1.0 (C:\Users\jayson.knight\Source\repos\samples\rust\rss_reader\bindings)
error[E0433]: failed to resolve: could not find build in windows
--> bindings\build.rs:3:14
|
3 |     windows::build!(
|              ^^^^^ could not find build in windows
For more information about this error, try rustc --explain E0433.
error: could not compile bindings due to previous error

For whoever's following along on this issue:
windows-rs now no longer requires you to have a separate bindings crate, instead just using it directly, and enabling the features you need. To find out what features are needed for a function or type, you can find it in the crate docs (link), and it tells you what features need to be enabled to have access to that function.

What @mbartlett21 said, but also note that the tutorial's result is present in the samples for the windows crate - latest version, 0.39, has a tiny difference (there are new string macros in the crate).
I'd love to see the tutorial itself updated - it was the first thing I found when looking for a way to make a windows application with Rust, and the narrative structure can be helpful. I also understand this is a dynamic package that will probably change faster than the tutorial can be updated.
Therefore, maybe I suggest editing the tutorial to make it clear that it's for a specific version of the windows crate, and why? I personally would love it updated to .39, because the removal of the bindings sub-project makes the thing far cleaner, and constitutes a considerable difference from "past practice" - but even as is, it's a useful tutorial, as long as people don't get punished for doing the "reasonable by default" thing of updating the patch level.
Mind you, the patch level "guarantees" don't really apply for 0.something version, but I don't think it's weird that people often forget that.


Engage Weekly - March 14th, 2022

This is a weekly public meeting to discuss the engagement across different areas of the Postman API Network, coordinating and planning around the advocacy, education, and community building work occurring within Postman Open Technologies.
Agenda:

Administrative - Taking care of administrative aspects of running this meeting.

Intake

Github
Slack - #api-network-partnerships #api-network-wins
Airtable -- Just for master index.

Public - This meeting will be open to the network, partners, and general public.

Trending - What teams, APIs, and collections are trending lately on the network.

Google Maps
Roaring
ArGIS

Priorities - What priorities requests have come in from internal and external stakeholders.

Pipedrive - Kin will work process and hand over to Arlemi.
Zoho Sign - Arlemi is working with.
Zoho CRM - Write a blog post.
Plaid - Write a blog post.
Huwawei - What is next for Huwaei?
Twitter - Meeting with this week to discuss their OpenAPI journey.
Github - Have a meeting coming up with Github to discuss their Postman collections / workspace.
Paypal - Discussions about their workspace possibilities.
Zoom
Axis Bank
Grey Noise (Website) (API)
Ory (Website) (Postman Team)
Dolby (Team) - New API coming GA in March - Music mastering. Guest blogging.
Render (API) (Workspace?)
John Deere (API)
Razorpay (Workspace - Engagement conversation.
Docker
Autodesk
Infobip
WhatsApp
Backblaze
Salt Security - Road Show.
Tyk (Workspace) - We have blog post and other storytelling occurring.

Campaigns - What are the areas we are focusing on with campaigns and components.

Anything?

Opportunities - Identifying where the opportunities are for focusing our attention.

Web3
Kroger - experimental!
Discord - Relationships with them.
API-First Index
Stripe DevChats

Blueprints - Formalizing the guidance we provide as part of this meetings.

Network Engagement - The numbers that matter.

Please add any agenda items you’d like to discuss below, and we will meet weekly, and push forward projects, productions, stories, collections, and tools to support our work.
Meeting:

March 14th, 2022 - 8:30 AM PST
Zoom - Ask for access below.

Any big companies that aren't listed here? That should be?

Robotics - https://developer.softbankrobotics.com/

Webflow - https://developers.webflow.com


Models class not generated with KAPT2

I've been using Kotlin 1.0.5 but also tried with 1.0.4 just to be sure - in both cases Requery does not generate any classes.
Parts of my Gradle build file:
apply plugin: 'kotlin-android'
apply plugin: 'kotlin-android-extensions'
apply plugin: 'kotlin-kapt'

kapt {
  generateStubs = true
}

// ...

dependencies {
  // ...
  compile 'com.google.dagger:dagger:2.8'
  kapt 'com.google.dagger:dagger-compiler:2.8'

  compile "io.requery:requery:$requery"
  compile "io.requery:requery-android:$requery"
  compile "io.requery:requery-kotlin:$requery"
  kapt "io.requery:requery-processor:$requery"
}

I see stuff generated for Dagger 2 in  build/generated/source/kapt2 but nothing for Requery.
Update
Aha, because I had apply plugin: 'kotlin-kapt', I had been using kapt2 instead of kapt. So, it means Requery is not working with kapt2.
Any change kapt 2 will be supported? It's a real shame to move to older stuff. I know there is kapt 3 already on the way...

Kapt2 is currently not supported. Beware, it generates something in Kotlin 1.0.6 EAP but it's bogus (no compile time errors but it fails at runtime when working with EntityStore). Stick with kapt for now.

See #296. I looked at kapt2 and that appears to simply not process anything. I'm not sure if people are using it successfully with other annotation processing libraries but there are no apparent issues with the requery processor. kapt2 never even invokes the requery processor so it actually never runs.

I've been using kapt2 successfully with Dagger 2. Had absolutely zero problems.


Game crashes and scene becomes corrupted when you set a button min size bigger than it's container

I get the following error:
 drivers/unix/net_socket_posix.cpp:199 - Socket error: 104
It basically makes my project to crash and makes my scene to be corrupted.

Steps to Reproduce:
Basically I simply was making a calculator layout and in the last step when I inputed the = final button, I set my minimal size to 120 while it's parent was 60, then hit play and the game crashes.

OS
Lubuntu last version.

Minimal project
Calculator.zip

EDIT:
Can confirm the bug happens, and after testing It seems these steps cause it to happen.

Add HSeparator as the first element in a VBoxContainer
Set the VBoxContainer min size to 60x 60y
Set the min size to 60x 60y
Add buttons of also the same 60x 60y size after the HSeparator
Hit play

Scene becomes corrupted.

I don't think it has anything to do with the min size. I could not reproduce it at all no matter what I tried to do with min size, even following your steps exactly.
Looking at the TSCN file the error is happening because one of the nodes has the name "" which I'm guessing is breaking the scene loader. If you open the TSCN file and manually change the name to "/" is works again and can be opened. However inside Godot you can't set the name of a node to "/" because / is an illegal character for node names. The character "" should probably be an illegal too since it breaks the TSCN.

I don't think it has anything to do with the min size. I could not reproduce it at all no matter what I tried to do with min size, even following your steps exactly.
Looking at the TSCN file the error is happening because one of the nodes has the name "" which I'm guessing is breaking the scene loader. If you open the TSCN file and manually change the name to "/" is works again and can be opened. However inside Godot you can't set the name of a node to "/" because / is an illegal character for node names. The character "" should probably be an illegal too since it breaks the TSCN.

Oh, I remember setting up \ as the node name, in any case it seems \ is an allowed name and this could be the cause of the crash of the scene.
Maybe godot should warn about "" as node name.

I guess the problem is because it treats "\" as a string starting with an escaped " character, while it should be "\\" for things to work properly.
Serialization:
https://github.com/godotengine/godot/blob/b9dc2e7e4dd38621ef0dd07d208ffc5468954adb/scene/resources/resource_format_text.cpp#L1715
Deserialization:
https://github.com/godotengine/godot/blob/b9dc2e7e4dd38621ef0dd07d208ffc5468954adb/scene/resources/resource_format_text.cpp#L206-L208
https://github.com/godotengine/godot/blob/b9dc2e7e4dd38621ef0dd07d208ffc5468954adb/scene/resources/resource_format_text.cpp#L274
https://github.com/godotengine/godot/blob/b9dc2e7e4dd38621ef0dd07d208ffc5468954adb/core/variant_parser.cpp#L1458-L1460
https://github.com/godotengine/godot/blob/b9dc2e7e4dd38621ef0dd07d208ffc5468954adb/core/variant_parser.cpp#L214
Seems like the deserialization code works fine, but the serialization code could use String.c_escape, like it does when saving groups a bit later. Note that the same probably applies to saving parent and owner.


Upgraded api-gateway-request-validation to 1.2.0

:+1:

@cristianconstantin pls update the readme with the newer version of the module as well

@cristianconstantin ping

@ddragosd: I just updated the README.


getPushRegistrationId

Hey Everyone,
putting in notifications, and setting up android it wants to use
getPushRegistrationId(), this function takes a call back. I am struggling as to what function to call within? is there a specific FCM fucntion that returns a value. Any help clarify from the docs would be helpful.

returns a promise
could either in async function use await i.e
let token = await getPushRegistrationId()

Keep in mind I believe this function is looking for fcm token on Mixpanel. So unless you have retrieved it elsewhere and then setPushRegistrationId(token) would not expect a value to be returned

ok thanks that makes sense. so is the setPushRegistrationId(token) expanding that long string from google.
example :
token = 'efklewnfreklwth93u43218509u321043u94u10932'
setPushRegistrationId(token)

No its not the server token.
For android you need an FCM token for IOS you need an apns token
The token is specific to the device. You are associating the device with the user in mixpanel so that when mixpanel sends a notification it reaches the device.
IOS example with react native firebase.
https://github.com/davodesign84/react-native-mixpanel/issues/234

cool thanks, the thing i am stuggling with for android is what is the function i have to use to return the FCM Token.


Monterey uses port 5000

Describe the bug
Mac OS X Monterey is using port 5000 for Control Center.  Due to that, Bulwark will not run as that port is being used.
Problem Locations
lsof -nP -iTCP -sTCP:LISTEN | grep 5000
ControlCe 67072    20u  IPv4 0xbbce7d9a96544dc7      0t0  TCP *:5000 (LISTEN)
ControlCe 67072   21u  IPv6 0xbbce7d9a7cc4670f      0t0  TCP *:5000 (LISTEN)Steps to Reproduce
Steps to reproduce the behavior:

Attempt to start bulwark.
Notice how the start fails due to the port being in use.

Expected behavior
Expected bulwark to start.
Screenshots
N/A
Desktop (please complete the following information):

Mac OS X Monterey

Changing port from 5000 to 4500 for production.  Keep in mind the port is hardcoded in environment.prod.ts and that was left as is for now. It should be dynamic or pull from a variable in the future.


Updated Spanish translation

Thanks for opening this pull request @moucho.
Could you confirm this change is definitely okay? I can't speak Spanish at all but the original text looks like it's closer to the EN original text of 'Set Date'.

I did this change because in Spanish the literal translation doesn't fit on one line, so it doesn't look right on screen.
My reasoning for changing it it's because I think it's clear enough, on each line there is a header for the filter option so a link with the 'date' string should be used to set the date.
Just my two cents :)

Message From Masao Soseki

From: Marcos<EMAIL_ADDRESS>Sent: Sunday, May 20, 2018 8:14:54 PM
To: BookStackApp/BookStack
Cc: Subscribed
Subject: Re: [BookStackApp/BookStack] Updated Spanish translation (#846)
I did this change because in Spanish the literal translation doesn't fit on one line, so it doesn't look right on screen.
My reasoning for changing it it's because I think it's clear enough, on each line there is a header for the filter option so a link with the 'date' string should be used to set the date.
Just my two cents :)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/BookStackApp/BookStack/pull/846#issuecomment-390473550, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ADRhrE6T7lRoBaSlDBQxIfOuv6l0N2Kpks5t0VAugaJpZM4UD_CM.

?
Message From Masao Soseki

From: Marcos<EMAIL_ADDRESS>Sent: Sunday, May 20, 2018 8:14:54 PM
To: BookStackApp/BookStack
Cc: Subscribed
Subject: Re: [BookStackApp/BookStack] Updated Spanish translation (#846)
I did this change because in Spanish the literal translation doesn't fit on one line, so it doesn't look right on screen.
My reasoning for changing it it's because I think it's clear enough, on each line there is a header for the filter option so a link with the 'date' string should be used to set the date.
Just my two cents :)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/BookStackApp/BookStack/pull/846#issuecomment-390473550, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ADRhrE6T7lRoBaSlDBQxIfOuv6l0N2Kpks5t0VAugaJpZM4UD_CM.


Report page 🍟

Description
##It looks aweful
The report page looks totally cramped and neither it gives out a good UX nor does it looks nice on the first impression
I want to have a clean neat report page !!
You can refer to this design if you want ⤵

Also make sure the X icon on the top right looks better , currently looks as if we have pressed it from the sides.
The color scheme can be the same, the fonts too.
The fields can be the same as the previous one.
Screenshots

Additional information
No response

Hi @IAmTamal ✋ Can I work on this?

I would love to work on this!!

I would love to work on this!!

Awesome, go ahead !!

Hi I am a newbie and I clone it but could not find the Report page to edit?

Hi , look at the report button in the footer , that's how the page is acessed.
And as of code it is in Footer.jsx

Also I am not being able to run the code here what I did
npm install and I got this
npm WARN config global --global, --local are deprecated. Use --location=global instead.
npm WARN ERESOLVE overriding peer dependency
npm ERR! code ERESOLVE
npm ERR! ERESOLVE could not resolve
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/typescript
npm ERR!   peerOptional<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   node_modules/react-intl/node_modules/@formatjs/intl
npm ERR<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!     node_modules/react-intl
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peerOptional<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/react-intl
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Conflicting peer dependency<EMAIL_ADDRESS>npm ERR! node_modules/typescript
npm ERR!   peerOptional<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   node_modules/react-intl
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR! See C:\Users\Ripple\AppData\Local\npm-cache\eresolve-report.txt for a full report.
npm ERR!      C:\Users\Ripple\AppData\Local\npm-cache_logs\2022-06-30T06_19_04_119Z-debug-0.log
PS C:\Users\Ripple\Desktop\zulip\zulip-mobile> npm start
npm WARN config global --global, --local are deprecated. Use --location=global instead.

<EMAIL_ADDRESS>start
react-native start

Hi @IAmTamal  , would like to ask you that can we use material-ui icons for top right cross button?

Also I am not being able to run the code here what I did
npm install and I got this
npm WARN config global --global, --local are deprecated. Use --location=global instead.
npm WARN ERESOLVE overriding peer dependency
npm ERR! code ERESOLVE
npm ERR! ERESOLVE could not resolve
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/typescript
npm ERR!   peerOptional<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   node_modules/react-intl/node_modules/@formatjs/intl
npm ERR<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!     node_modules/react-intl
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peerOptional<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/react-intl
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Conflicting peer dependency<EMAIL_ADDRESS>npm ERR! node_modules/typescript
npm ERR!   peerOptional<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   node_modules/react-intl
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR! See C:\Users\Ripple\AppData\Local\npm-cache\eresolve-report.txt for a full report.
npm ERR!      C:\Users\Ripple\AppData\Local\npm-cache_logs\2022-06-30T06_19_04_119Z-debug-0.log
PS C:\Users\Ripple\Desktop\zulip\zulip-mobile> npm start
npm WARN config global --global, --local are deprecated. Use --location=global instead.

<EMAIL_ADDRESS>start
react-native start

May I ask you that where did u do npm install i.e in which respected folder?

@prabhpreet-code zulip\zulip-mobile>

@NishanBhandai You need to clone it first in any directory in your computer by method git clone and then cd Milan i.e the name of the project after that cd src folder then here do npm install then npm start ... it will redirect you to the website at your localhost.

Hi @IAmTamal , would like to ask you that can we use material-ui icons for top right cross button?

Yes !!

2022-06-30.22-31-21.mp4
@IAmTamal could you please have a look at this? Is it okay? any changes to be made?

Looks perfect Prabhpreet , feel free to make a PR.
Will be merged tomorrow 🚀

Great!! thanks, Tamal...


toolset props change to fix msvs always out of date bug with clang-cl.

When building projects using the clang-cl toolset integration visual studio always things the projects are out of date.
This is due to clang-cl not generating intermediate pdb files. It silently ignores the command line option specifying the name of intermediate pdb files and puts debug data in the .obj files.
adding a tag to the toolset props compile section to clear that command line argument keeps MSVS from looking for the file and assuming project is out of date.
additional change eliminates an annoying ignored command line argument warning.

And here I was wondering why it always rebuilt, thanks for the fix!


git merge

When you select git merge, it would give you a list of all your branches just like when "change branch" except when you select it that branch gets merged into your current one.
(Also after we get that basic support maybe more complex one to deal with merging conflicts might be something add. I.e. http://sjl.bitbucket.org/threesome.vim)

added this in, on pull request here: https://github.com/kemayo/sublime-text-2-git/pull/71

merged in and working


Fix syntax hilighting of clojure code

after https://github.com/clojure/tools.reader/blob/b62ce9964a9770578b5f2179a2166cbfc956dfe8/src/main/clojure/clojure/tools/reader.clj#L36 the highlighter seems to thing it is in a string literal for a large chunk of the rest of the file (everything is orange)

Hi @hiredman, thanks for reporting this - the issue is being tracked over here: https://github.com/atom/language-clojure/issues/11