Uncaught Error: History.js Adapter has already been loaded...

Hey Devs!
Very nice plugin/library! Is there a way to get rid of the "Uncaught Error: History.js Adapter has already been loaded... ". I tried with try/catch, but that doesn't work.
I'm using http://labjs.com/ and unfortunately I can't control/check right now if a script was loaded before, so double loading can happen and then historyjs breaks the application with this uncaught error.
Thanks in advance,
Matthias

+1

seems to happen to me too

I can't see why this issue is closed !

+1

+1

+1

Unfortunately you're not able to catch this error since it's being thrown within the anonymous function that it's defined within.
Here's a workaround - simply unload History.js prior to loading it to ensure that it is able to setup.  It is a 4 years old bug now? :-O


Question: What's difference between LaTeX/KaTeX?

http://mathim.com/
LaTeX based. But is there is any difference?

The syntax that KaTeX supports is LaTeX-like, and it supports a subset of the LaTeX functions. You can see a list of functions that KaTeX supports here.
The chat you linked to looks like it uses MathJax, another library for rendering LaTeX in the browser.


Implement the 'events' command

The runtime does not currently implement the 'events' command. We may not be able to perfectly match all the information that the runc version supports, but we may be able to do a subset, and possibly some extra VM related info. Analyse and implement as necessary.
https://github.com/opencontainers/runc/blob/e775f0fba3ea329b8b766451c892c41a3d49594d/events.go
https://github.com/opencontainers/runc/blob/52454cf90807727923b375afd7d84ac8fd523959/man/runc-events.8.md

If everybody else are occupied then I could work on this feature after I finish with HA.

That works for me @dvoytik As per the Issue, I think the first stage would be some data gathering and analysis of what runc provides, what options we have to match those, and then what, if any, extra items we might be able to add.
One of the key aspects we should discuss, probably on a per-item basis, is if we measure some of the stats from outside or inside the VM/container, or both etc.
@mcastelino for any thoughts he has.
@sboeuf for any thoughts on his current plans to check we don't overlap etc.

Oh yeah great if @dvoytik wants to implement this. Looking at what runc expects, I would say that we want to monitor what is going on inside the VM. This means that from virtcontainers/runtime, we would call/poll on a new agent command getStats(containerID) for example, and then format those results to the format expected by runc.
Does that make sense ?

Let's list out what runc/docker currently extract/print and then we can have a discussion as to which should come from inside or outside of the VM.
I think a wiki page might be a suitable place to do that diagnosis, you think? I just want to make sure we get the decision right first time.
The new agent command sounds the right approach to me for things from in the VM, unless we can extract them via QMP for instance


Add changes to fix #1

So that when isLicensed is populated, it both actually gets checked, and that the criteria for checking also includes looking for a status of enabled.

Squash this down to one commit to keep the code history clean, please.


Automated cherry pick of #4661

Cherry pick of #4661 on release-5.19.

#4661: respect locale format in rendering post time

/cc  @saturninoabril

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

Trying to auto merge this PR.

Pull Request successfully merged
SHA: f8bdd5de0a2ea32b47cfa885b74afa41bf26f00b


[145] Issue #145 - October 10, 2019

To contribute to this issue, simply leave a comment here. Please also review our contributing guidelines.
The current draft for this issue in _drafts/. If you want to contribute directly, feel free to open a pull request.

https://twitter.com/harlanhaskins/status/1177307996512112640

Finally merged a patch that I’ve been poking at since November. If you’re working on a build system and you’re implementing the “-emit-module first, then build” optimization, try adding -Xfrontend -experimental-skip-non-inlinable-function-bodies
https://github.com/apple/swift/pull/20420

How is SE-0255 (implicit returns) not a violation of Swift’s design principles?

How is SE-0255 and the Implicit Return behavior of closures not a violation of Swift's design principles?

In Commonly Rejected Proposals, Implicit Returns are mentioned explicitly as favoring "terseness over readability". Implicit Returns eliminate needing to type 6 characters, at the expense of anyone but the person who wrote said code, being able to gleen over it quickly and derive meaning.

https://forums.swift.org/t/how-is-se-0255-implicit-returns-not-a-violation-of-swifts-design-principles/29267

Libdispatch moved to CMake 3.15.1

This is to just make sure that people are aware that as had been previously discussed, the slow crawl to CMake 3.15 is now underway. This should be largely transparent, though, a clean build would not hurt probably (but should not really be needed, I try to avoid them as much as possible). On Linux hosts if your system CMake is older than 3.15, build-script will checkout and build CMake as part of the build. We will then use that to build the rest of the projects. On macOS hosts, please update your CMake from https://cmake.org/download. On Windows, CMake 3.15.1 is available from cmake.org OR from Microsoft in VS2019.

https://forums.swift.org/t/libdispatch-moved-to-cmake-3-15-1/29268

Android application startup

I have ported an iOS app to Android using Vlad Gorlov's Swift Everywhere Toolchain.
The non-UI code is shared with the iOS version (about 20k lines of Swift code). The UI is written in Kotlin with a JNI bridge using a combination of my own Swift wrapper and swift-jni.
Aside from the initial grief of fixing all of the NS* bridging issues to get the code to build, the code mostly worked.

https://forums.swift.org/t/android-application-startup/29386

[pitch] NIO-based gRPC Swift

Include gRPC-Swift among the official and recommended SSWG projects.
gRPC enables high-performance open standards-based interoperability for Swift services with services written in other languages.

https://forums.swift.org/t/pitch-nio-based-grpc-swift/29396

https://forums.swift.org/t/prototype-protocol-powered-generic-trimming-searching-splitting/29415

Add another allSatisfy funtion that has parameter for empty

When you use allSatisfy function in Array, Dictionary...
You have to check empty.
Because if Array, Dictionary are empty, results of allSatisfy are true.

https://forums.swift.org/t/add-another-allsatisfy-funtion-that-has-parameter-for-empty/29421

SE-0266 — Synthesized Comparable conformance for enum types
The review of SE-0266 — Synthesized Comparable conformance for enum types begins now and runs through October 10, 2019.
https://forums.swift.org/t/se-0266-synthesized-comparable-conformance-for-enum-types/29457

https://twitter.com/dhartbit/status/1179835449180905475
Sent with GitHawk

Compositional Initialization

This proposal introduces an opt-in protocol, PropertyInitializable, which provides two new init methods:

failable init from a collections of extensible, typesafe keypath-value Property objects
non-failable init to clone from another instance, mutating select properties

The name “compositional init” means that this proposal allows the state of an object or struct to be assembled compositionally from sets of properties (including another instance). Compositional initialization allows mutations to be encapsulated in a clear, type-safe way.

https://forums.swift.org/t/compositional-initialization/29535

Swift on Server WorkGroup meeting updates
https://forums.swift.org/t/september-19-2019/29500

Starter task
SR-11574 [Compiler] Using try instead of throws should have a better diagnostic

[Discussion] gRPC Swift
gRPC Swift is a community backed implementation of the gRPC over HTTP/2 protocol built on top of SwiftNIO. It includes a runtime (GRPC), a code generator (protoc-gen-grpc-swift), an extensive suite of tests and examples.
https://forums.swift.org/t/discussion-grpc-swift/29584

https://spec.fm/podcasts/swift-unwrapped/309982

https://twitter.com/jckarter/status/1181293210775121920
Sent with GitHawk

https://twitter.com/johannesweiss/status/1181312935450284032

apple/swift-evolution:proposals/0265-offset-indexing-and-slicing.md@master  In review

The review of SE-0265: Offset-Based Access to Indices, Elements, and Slices begins now and runs through October 21st, 2019
https://forums.swift.org/t/se-0265-offset-based-access-to-indices-elements-and-slices/29596

Starter task
SR-11580 [SwiftSyntax] FloatLiteralExprSyntax and IntegerLiteralExprSyntax should have comuted properties that return there integer/float value

https://twitter.com/rockthebruno/status/1181889843539955712

ExpressibleByTupleLiteral instead of tuples conforming to protocols

As can be seen from the forum history, many people have suggested ways to make it possible to conform tuple types to protocols. It is perhaps most natural when the conformance can be synthesized, - in my personal usage it's certainly conformance to Equatable and Hashable that I've wanted to confer on my tuple typed keys.

https://forums.swift.org/t/expressiblebytupleliteral-instead-of-tuples-conforming-to-protocols/29626

Swift on the Server Work Group 3rd October meeting notes
https://forums.swift.org/t/october-3rd-2019/29628

263 in review

263 in review

The requested revisions to SE-0263 have been made to the proposal:

https://forums.swift.org/t/se-0263-review-2-add-a-string-initializer-with-access-to-uninitialized-storage/29663

A misunderstanding of the final SE-0249 behavior
https://forums.swift.org/t/a-misunderstanding-of-the-final-se-0249-behavior/29671

I would like to provide an update to the monorepo transition posts (https://forums.swift.org/t/llvm-monorepo-transition/25689 and https://forums.swift.org/t/llvm-monorepo-transition-update/27079) where I announced that Apple is working on the LLVM project monorepo transition for Swift, and posted links to the prototype monorepo that can be used with Swift.

https://forums.swift.org/t/llvm-monorepo-transition-happening-on-the-week-of-14th-18th-october/29674


Bug in safari when accessing team page

As reported by @ricardobaeta the team page does not look ok in safary

iPhone SE, Safari iOS 11.0.3

@marnovo @ricardobaeta We can do graceful degradation in Safari as a quick fix, and provide a proper fix after the release.
That way the Safari users will not see the awful circle bug.
It can already be validated in staging http://landing-staging.srcd.run/team/

If you agree, it would be merging the PR https://github.com/src-d/landing/pull/181

Approved, let's keep this open until we actually fix it.


A long error message breaks the notification popup

Title/Description
Error text doesn't fit in the notification popup.
Environment

Browser: Chrome
Environment: https://rinkeby.gnosis-safe.io/

Steps to reproduce

Change the owner policy from "2 of 2" to "1 of 2"
Execute and confirm
Simulate a failure somehow (it could be done by rejecting the tx and inserting the error I pasted below directly into the DOM tree or the popup)

Expected result
The error message should be user-readable.
Obtained result
Raw JSON is displayed and it doesn't fit in the popup. I'm not sure if this is because it's the test environment but even if so it should still be readable.
Screenshots

The full error message is:
Processing transaction error:  Error: Transaction has been reverted by the EVM:
{
  "blockHash": "0x86be5536d37904cc81707676f5ee639eb5bb0dd24d847a6404e28f1882930f0e",
  "blockNumber": 8377467,
  "contractAddress": null,
  "cumulativeGasUsed": 209164,
  "from": "0x474e5ded6b5d078163bfb8f6dba355c3aa5478c8",
  "gasUsed": 48290,
  "logsBloom": "0x00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000",
  "status": false,
  "to": "0xb3b83bf204c458b461de9b0cd2739db152b4fa5a",
  "transactionHash": "0x16ece47a499d4885d66c189ddfb45db8b0e4248caf8adbd346b228c8193db3cb",
  "transactionIndex": 2,
  "type": "0x0",
  "events": {}
}

May be a  duplication of https://app.zenhub.com/workspaces/safe-multisig---web-5ce554debb310a35b2f8b6f8/issues/gnosis/safe-react/1838

@liliya-soroka true, thank you!
Closing in favor of #1838.


Choose Images after uploaded

Hi dear,
How I choose pictures after i uploaded and get properties of picture such as height, width of picture . . .?
Thank for support

yes, I saw the onChange event can't be called.  How can I capture the picture data so that I can upload it to my API?


Use only one sigv4 file.

https://github.com/aws-solutions/amazon-marketing-cloud-uploader-from-aws/blob/development/source/amc_uploader/lib/sigv4.py
https://github.com/aws-solutions/amazon-marketing-cloud-uploader-from-aws/blob/development/source/api/chalicelib/sigv4.py

These two files are the same, resolve imports in project to use only one file.
This will resolve SonarQube duplicate code issue.

If you can resolve this issue, then resolve this as well -> https://github.com/aws-solutions/amazon-marketing-cloud-uploader-from-aws/issues/106


use SVG for icons

Description
I think it is better to use svg sprites for icons instead of of web fonts. it has many advantages discussed on a number of blog posts, plus it is more semantic.

This library provides no CSS.  Please open an issue on https://github.com/Semantic-Org/Semantic-UI.  Note, this issue was already opened as well: https://github.com/Semantic-Org/Semantic-UI/issues/1240


A11y_SQLAzureDataStudio_DeleteAzureAccount_DeleteAzureAccount_ScreenReader:The dialog is not announced when opened.

“Check out Accessibility Insights! - Identify accessibility bugs before check-in and make bug fixing faster and easier.”
GitHubTags:#A11y_SQLAzureDataStudioBenchmark;#A11yMAS;#A11ySev2;#A11yTCS;#SQL Azure Data Studio;#Benchmark;#DesktopApp;#MacOs;#MAS1.3.1;#VoiceOver;
Environment Details:
Application Name: Azure Data Studio
Version: 1.15.0-insider
Commit: 3a8b74a
Date: 2020-02-11T05:49:41.181Z
VS Code: 1.42.0
Electron: 7.1.11
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Darwin x64 19.3.0
Mac OS:10.15.3
Additional Details:
Mac OS:10.15.3
MAS References:MAS1.3.1
Screen reader: Voiceover.
Repro Steps:

Launch Azure Data Studio application.
Connect to server with windows credentials.
Navigate to Connections from left menu.
Select Azure, navigate to + button & hit enter.
Launch voice over .
Navigate to X button on any account in right window & hit enter.

Actual:
When the dialog opens the screen reader just announces the button in focus "No".
Expected:
The dialog should be announced when opened followed by the button.
User Impact:
Screen reader users will not know what "no" refers to if the name of the dialog is not announced prior.
Attachment for referance:

unfortunately we don't control this, the ability of openning this dialog is provided by vscode, please open an issue in vscode repo. this can be repro'ed by going to vscode and navigate to extensions view, select an extension and in the detail page, click on the repository link

Verified the bug and does not repro.
GithubTags:#Closed;

GitHubTags:#SQLADS-MAC-Feb2020;


feat(contracts): Merkle Tree library for priority queue migration

What ❔
Incremental Merkle Tree library (modified version of OZ's) and an item range inclusion check method.
Why ❔
Preparation for priority queue migration.
Checklist

[ ] PR title corresponds to the body of PR (we generate changelog entries from PRs).
[ ] Tests for the changes have been added / updated.
[ ] Documentation comments have been added / updated.

Closing in favor of #546


unimplemented: opx.gprinter

As from https://archive.org/download/3-libjune-05/3LIBJUNE05.iso/library%2Fepocmisc%2Fatmos5%2Fatmos522.SIS
According to http://www.ericlindsay.com/epoc/mopx5.htm:

Graphics printing, especially vector, table and charts, with development kit (originally for Plan 5), from Andy Clarkson. Version specific? www.twiddlebit.com

Unlikely this will ever be supported since I can't find any source or documentation for the API, nor does it seem to be that widely-used.

Well this made me smile:

John McAleely maintained a more comprehensive treatment of OPX than I do, and it would be a good starting place for more information (I found it after I got started).

That stance definitely makes sense though. The long-list of OPXs is likely to be a bit of a killer here?


dnx beta detection, truthiness check in lodash._any()

Unless I change this line https://github.com/OmniSharp/omnisharp-atom/blob/master/lib/omnisharp-atom/features/command-runner.ts#L178 to be an explicit return true or false, using beta7 and beta8, the . gets added to the command to execute.
if (any(['beta3', 'beta4', 'beta5', 'beta6'], x => runtime.indexOf(x) > -1)) {

willing to do a quick PR?
I can give you a quick rundown of how to setup your dev environment if you need.

Published to apm as v0.24.11


lastIndexOf doc looks wrong

lastIndexOf_.lastIndexOf(array, value, [fromIndex])
Returns the index of the last occurrence of value in the array, or -1 if value is not present. Pass fromIndex to start your search at a given index.
The behavior and the docs to match.  lastIndexOf with a fromIndex doesn't even make sense.
 log( _.lastIndexOf( [ 0, 1, 2, 3, 4 ], 2, 0 ) ) //=> -1
 log( _.lastIndexOf( [ 0, 1, 2, 3, 4 ], 2, 1 ) ) //=> -1
 log( _.lastIndexOf( [ 0, 1, 2, 3, 4 ], 2, 2 ) ) //=>  2
 log( _.lastIndexOf( [ 0, 1, 2, 3, 4 ], 2, 3 ) ) //=>  2
 log( _.lastIndexOf( [ 0, 1, 2, 3, 4 ], 2, 4 ) ) //=>  2

The behaviour matches the documentation though could be worded better. The [fromIndex] indicates the index the search will begin at and then decrements from that value.
If you want to, please submit a pull request with better wording

What am I missing? if you have an array
[0,1,2,3,4,5,6,7,8,9]

If you want the last index of 6, what does it matter where you start your search from, it will always be the last - unless you start at 7 of course.
The behavior looks like fromIndex should be upToIndex, or stopSearchingIndex
Look at my example here
 log( _.lastIndexOf( [ 0, 1, 2, 3, 4 ], 2, 0 ) ) //=> -1

Why is this returning -1 if the search is starting at 0?
I think the search is stopping at 0, which is why when you set the upToIndex to 2, you get your first hit
log( _.lastIndexOf( [ 0, 1, 2, 3, 4 ], 2, 2 ) ) //=>  2

I think we mostly summarized mozilla's documentation, again feel free to submit a commit with changes clarifying the docs
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/lastIndexOf

The lastIndexOf() method returns the last index at which a given element can be found in the array, or -1 if it is not present. The array is searched BACKWARDS, starting at fromIndex.
Forgot the backwards part, that explains why its so confusing.


Make it more obvious that webchat users are guests

Currently, people join the irc all the time asking php questions under the name fig123 leading people to believe that it's a single person asking hit and run questions :P
This changes the guest names to fig_guest123.

:+1:


Incorporate model with views

The goal is to get a prototype of the game which works to play in offline single-player mode

PR #15


Unable to load gptq models

Error while loading llama 8b gptq
GPTQ without g_idx
379
380 else:
382     return ext_c.make_q_matrix(w["qweight"],
383                                none_tensor,
384                                none_tensor,

(...)
393                                temp_dq,
394                                max_dq_rows)
RuntimeError: q_weight and gptq_qzeros have incompatible shapes
Please help.

Sorry I overlooked this. There is currently no support for 8-bit GPTQ models. Only 4-bit is supported. There will be more modes added at some point.

close


SDK 14 FragmentActivity testing issues

I still am not able to get my tests running correctly with the latests sdk release. I have the impression that only activities extending FragmentActivity cause problems.
2 Issues:

...Caused by: java.lang.NoClassDefFoundError: com.myapp.ui.MyFragmentActivity" causing the tests to not even start (ActivityInstrumentationTestCase2)
When calling getActivity() I get this: java.lang.RuntimeException: Unable to resolve activity for: Intent { act=android.intent.action.MAIN flg=0x10000000 cmp=com.myapp.ui/.MyFragmentMapActivity }

Both of these activities work perfectly fine outside of testing.

Hi,
I have the same issue with my app crashing on smartphone after upgrading to ADT 14.
I get no issue with Honeycomb with actionbar displaying properly
Here is the crash  Obenkyo beeing my app and "main" beein my main FragmentActivity:
10-22 17:28:56.343: E/AndroidRuntime(13904): Caused by: java.lang.ClassNotFoundException: com.Obenkyo.Main in loader dalvik.system.PathClassLoader[/mnt/asec/com.Obenkyo-2/pkg.apk]
10-22 17:28:56.343: E/AndroidRuntime(13904): 	at dalvik.system.PathClassLoader.findClass(PathClassLoader.java:240)
10-22 17:28:56.343: E/AndroidRuntime(13904): 	at java.lang.ClassLoader.loadClass(ClassLoader.java:551)
I have tried to update to the very last version and to reload the library without success.
Also, I have tried to compile the "shakespeare" sample working as expected on both smartphone and tablet. I will have to add features to this template to see where it could go wrong.
Regards.

Real problem is java.lang.ClassNotFoundException: com.Obenkyo.Main
It is looking for Main under com.Obenkyo packages.

"Main" beeing my Fragment activity class in my com.Obenkyo Package, I wonder why it is ... :/

Will also try to reproduce this starting with a simple app

This was an issue on my side, wrong package name in test constructor and library deployed twice in app/tests apk. Atomusk I don't think your issue is related to mine, might be better to create a new issue.

You are right my issue is not related. I have to investigate further. I will open another ticket if I found an issue (sorry for the mistake)

Hmm... I somehow disabled notifications for this issue. Didn't know that was possible if you were the repo owner. In any case, glad to see you guys managed to work out the issue!


Increase unit test coverage

Since we've enabled code coverage, we should also plan how to improve it. Define which parts/modules of the application (back/frontend) would benefit the most from unit testing, and schedule some time to do it as we go into maintenance mode on the project.
Start here: https://codecov.io/gh/Streetwise/streetwise-app
See also #32

Since we use postgres on prod and CI, we should use postgres for local development and testing as well.
I have noticed that tests that pass locally, often end up failing on CI. We can have docker-compose script that sets up the database for a new developer.
Thoughts @loleg

It's a tough call. I like the convenience and speed of a local SQLite db. The beauty of using Sqlalchemy, besides it's expressiveness, is support for multiple db's. I'd say the necessity of buttoning things down is the reason we have CI in the first place. I would not force developers to install Postgres for no good reason, and you're welcome to use it (in Docker or otherwise) if you want to.


Add "Open with Spyder" entry to Windows File Explorer's context menu

From pierre.raybaut on 2012-03-11T08:12:28Z
This feature has been requested in issue #466 .
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=971

From pierre.raybaut on 2012-03-18T14:14:28Z
Labels: -Type-Enhancement Type-Enh

From pierre.raybaut on 2012-03-18T13:55:42Z
Labels: -Milestone-v2.2 MS-v2.2

From pierre.raybaut on 2012-03-18T13:44:55Z
Labels: -Milestone-Release2.2 Milestone-v2.2

From pierre.raybaut on 2012-09-23T04:33:25Z
Blockedon: spyderlib:466

From pierre.raybaut on 2013-03-16T15:25:09Z
This issue was updated by revision 00ba20a9cf0a .
Please test this carefully before the final v2.2.0 release and mark this issue as Verified.
Status: Fixed

From pierre.raybaut on 2013-03-18T08:53:26Z
Could you take a look at the corresponding registry key to see what's remaining exactly ? (and export this fraction of your registry tree)

From<EMAIL_ADDRESS>on 2013-03-18T08:21:10Z
I have tested this using the procedure in comment 6. The install portion works correctly. The right-click menu gets updated as desired. However, the uninstall is not successful at removing "Edit with Spyder" from the context menu.

From pierre.raybaut on 2013-03-17T01:58:49Z
How to test this new feature:

build Spyder's installer: python setup.py build bdist_wininst
install Spyder (run the installer)
check if right-click menu on Python file shows an entry "Edit with Spyder"
uninstall Spyder
check if the "Edit with Spyder" has been removed from context menu as expected

From pierre.raybaut on 2013-03-18T08:55:14Z
Here are the exact locations to take a look at:
HKEY_CURRENT_USER\Software\Classes\Python.File\shell
HKEY_CURRENT_USER\Software\Classes\Python.NoConFile\shell

From<EMAIL_ADDRESS>on 2013-03-18T08:56:42Z
All of the registry keys seem intact. I also just found this Python bug report :( http://bugs.python.org/issue13276 It may be that the "remove" section of the post install script is not even being called.

From ccordoba12 on 2013-04-07T08:33:09Z
Ok, I will stop the final release until we solve this issue.
I'm not sure if I understood you correctly: by patching Python you mean patching its source code? before compiling it? If that's the case, I don't think that's the way to go because it won't work for people that don't use Python(x,y).
So I'd be in favor of your second solution, if it could be developed as a monkey-patch that could live in our tree.

From ccordoba12 on 2013-04-06T20:21:43Z
Labels: Cat-Miscelleneous

From pierre.raybaut on 2013-04-07T05:20:34Z
@-Carlos: This needs to be taken care of before v2.2.0 final release.
I confirm that the distutils bdist_wininst installer is partially broken: the post-install script is not executed with the '-remove' command line argument when uninstalling the package ( http://bugs.python.org/issue13276).Furthermore , the bdist_msi installer is also broken: first, it does not follow the same logic as bdist_wininst (no '-install' command line option when running the post-install script when installing the package), second the post-install script is not executed at all when uninstalling the package ( http://bugs.python.org/issue15797 ).
I must say that I'm quite disappointed in Python development team... these bugs seem quite important but were ignored since at least a year and a half for the first one, and the feature has never been implemented for the second one...
We have two solutions:

patch Python to make the distutils bdist_wininst installer work
patch Python to make the MSI package work (in this case, this could even be a monkey-patch to be included in Spyder source repo)

Status: Started
Labels: -Priority-Medium Priority-Critical

From ccordoba12 on 2013-04-08T14:48:37Z
Pierre, I know you are quite busy so you could tell more concretely what you have in mind and I could try to develop a solution according to that.
As I told you, right now I don't understand too well what we should do about this issue.

From<EMAIL_ADDRESS>on 2013-04-09T12:37:01Z
@-Pierre: I have successfully applied the patch to Python suggested here: http://bugs.python.org/issue13276 It actually doesn't modify Python itself but instead modifies the installer program that gets embedded in the installer executable we create. The required patched wininst-9.0.exe is attached. This should be placed into this directory on Windows:
C:\Python27\Lib\distutils\command
I'd try to build the installer myself, but I'm just not familiar enough with that process to be confident in getting it right. If you'll build the installer I can certainly test it.
Cc: pierre.raybaut
Attachment: wininst-9.0.exe

From<EMAIL_ADDRESS>on 2013-04-09T12:54:51Z
Okay, I can verify that, after replacing wininst-9.0.exe as described in comment 15 and building Spyder's Windows installer as described in comment 6, everything works. I repeated the Spyder install and uninstall process twice, and I can confirm that the "Edit with Spyder" context menu entry gets installed and removed properly now.

From ccordoba12 on 2013-04-09T13:15:48Z
Great news Jed! and thanks for taking care of the issue so quickly.
I think we should add the mentioned patch to our tree (outside of spyderlib of course, i.e. next to create_app.py) and create a new wiki page describing how to apply it to generate the right installer. What do you think?
Or would it be easier to just add your wininst-9.0.exe? I just would like to ensure that any of us can easily generate new installers when needed.

From<EMAIL_ADDRESS>on 2013-04-09T12:46:06Z
Sorry, just remembered comment #6 above. I'll try it out myself :).

From<EMAIL_ADDRESS>on 2013-04-09T13:37:04Z
... and attached is the 64-bit version, wininst-9.0-amd64.exe, which would be required to build installers for 64-bit Python 2.7.
@-Carlos: We're actually patching the C source to distutils which comes with the Python source. Then the wininst-9.0 components must be built form that source using Visual Studio 2008. Once built, you have to copy them into your Python lib directories so that distutils can grab them and stuff them into your installer. So it's not really something that everyone will be able to build on their own. Maybe a wiki page that describes how distutils is currently broken, provides links to download the .exe files I've built, and instructions on where to put them would be better.
Attachment: wininst-9.0-amd64.exe

From<EMAIL_ADDRESS>on 2013-04-09T20:53:27Z
I created a wiki page here to describe how to patch distutils before building the Windows installer: https://code.google.com/p/spyderlib/wiki/PatchingDistutils

From ccordoba12 on 2013-04-24T09:11:15Z
Thanks a lot!

From<EMAIL_ADDRESS>on 2013-04-24T09:02:52Z
Yes, I can generate installers.
Cc<EMAIL_ADDRESS>

From ccordoba12 on 2013-04-24T08:59:16Z
I think we can close this issue now. I just added the command needed to build the installer to the wiki page (first one of comment #6) as the last step in the process.
Jed, could you be in charge of generating the installers once 2.2 final is released? I'll take care of the source release plus our Mac app.
Status: Fixed


Add Documentation For New CLI Commands

This PR adds to the documentation, the new commands added to the agenta CLI in this dependent PR #1114

[x] agenta variant add
[ ] agenta config pull
[ ] agenta config update

Resolves #1118

thanks @devgenix ! Please note that this should be merged to docs_v2 branch and not main

@mmabrouk
From this issue #1120, the new commands will be added to both docs/advanced_guides/using_agenta_from_cli.mdx and docs/developer_guides/cli?

thanks @devgenix ! Please note that this should be merged to docs_v2 branch and not main

@mmabrouk From this issue #1120, the new commands will be added to both docs/advanced_guides/using_agenta_from_cli.mdx and docs/developer_guides/cli?

yes, however not all the commands in all the details. the using cli is a short howto guide while the developer_guides is a reference (read about the difference here https://diataxis.fr/)


make env.volumes only change package path when used

This fixes windows usage.
It also tries to fix windows builds when using env.volumes by making windows compatible
paths for env.volumes
not using env.volumes will now also mount the project in /project again
Example docker invocation on windows
with env.volumes
docker.EXE run
--userns "host"
-v "G:\\workspace\\cross_testing\\hello.txt:/mnt/g/workspace/cross_testing/hello.txt"
-e "HELLO=/mnt/g/workspace/cross_testing/hello.txt"
-e "PKG_CONFIG_ALLOW_CROSS=1"
--rm -e "XARGO_HOME=/xargo"
-e "CARGO_HOME=/cargo"
-e "CARGO_TARGET_DIR=/target"
-e "USER=Emil"
-e "CROSS_RUNNER="
-v "C:\\Users\\Emil\\.xargo:/xargo:Z"
-v "C:\\Users\\Emil\\.cargo:/cargo:Z"
-v "/cargo/bin"
-v "G:\\workspace\\cross_testing\\foo:/mnt/g/workspac/cross_testing/foo:Z"
-v "C:\\Users\\Emil\\.rustup\\toolchains\\nightly-x86_64-unknown-linux-gnu:/rust:Z,ro"
-v "G:\\workspace\\cross_testing\\foo\\target:/target:Z"
-w "/mnt/g/workspace/cross_testing/foo"
-i -t "ghcr.io/cross-rs/aarch64-unknown-linux-gnu:edge"
sh
-c "PATH=$PATH:/rust/bin cargo run -vv --target aarch64-unknown-linux-gnu"

without
docker.EXE run
--userns host
-e "PKG_CONFIG_ALLOW_CROSS=1"
--rm
-e "XARGO_HOME=/xargo"
-e "CARGO_HOME=/cargo"
-e "CARGO_TARGET_DIR=/target"
-e "USER=Emil"
-e "CROSS_RUNNER="
-v "C:\\Users\\Emil\\.xargo:/xargo:Z"
-v "C:\\Users\\Emil\\.cargo:/cargo:Z"
-v "/cargo/bin"
-v "G:\\workspace\\cross_testing\\foo:/project:Z"
-v "C:\\Users\\Emil\\.rustup\\toolchains\\nightly-x86_64-unknown-linux-gnu:/rust:Z,ro"
-v "G:\\workspace\\small dev space\\cross_testing\\foo\\target:/target:Z"
-w "/project"
-i  -t "ghcr.io/cross-rs/aarch64-unknown-linux-gnu:edge"
sh
-c "PATH=$PATH:/rust/bin cargo run -vv --target aarch64-unknown-linux-gnu"

I have not tested this with env.volumes yet

bors try --target x86

This should be feature complete now 🎉

has anyone been able to test this and give this a r+ @cross-rs/maintainers <3

ping @cross-rs/maintainers
It would be good to have this included for 0.3.0

I'm not using Windows so I cannot test this, but it looks good so far.

not using env.volumes will now also mount the project in /project again

Does this have any advantage or is this needed to fix Windows support?

not using env.volumes will now also mount the project in /project again

Does this have any advantage or is this needed to fix Windows support?

No advantage, except for not leaking paths in the general case, like what was done before.
bors r=reitermarkus


⛏ Write a test to check whether we can create/update an object with File Upload

💭 Introduction:
We want a test to check whether an attacker can create/update entity with File upload.
🎯 Requirements:

Filters - API with atleast one parameter as an input in GET query parameter or JSON body parameter accepting Files, having header for Files like “Content-Type” : “multipart/form-data” or “Content-type” : “application/octet-stream” or other types of headers used for File upload

Execute - It should replace the value with

Changing the content-type header and sending the same file as identified in the original request
Keeping the header and uploading some other file in Body
Not sending any file and keeping the header same
removing the header and then sending any file, or the same original file

Validation - If the application responds with a exception trace, it is a vulnerability.

✅ Task summary:

[ ]  Ask to be assigned to the issue.
[ ]  Wait to be assigned. We will try to assign in less than 2 hours.
[ ]  Signup for [Akto]
[ ]  Fork the [tests-library] repository, create a new branch and commit the yaml file which will be called in your test.
[ ]  Submit both the PR here.

📚 Reading
Find 100+ examples of YAML tests [here]
🙋🏼♂ Questions:
If you have questions, need any help, or just want to hang out, make sure to join us on our [Discord server].

Hey @arjun-akto, I have gone through the test-library repo and example YAML test and I feel I can contribute to this issue by writing the above mentioned test. Can you please assign this issue to me?

Hey @arjun-akto I want to work on this, could you please assign me ?

Hi @falconcode16 , @heysagnik . I have assigned the issue to you. Please feel free to connect us on our Discord server for any doubts.


can not download extentions and update

Type: Bug

it shows "Error while fetching extensions. XHR failed.
clicking on update showing tunnel error

VS Code version: Code 1.75.1 (441438abd1ac652551dbe4d408dfcec8a499b8bf, 2023-02-08T21:32:34.589Z)
OS version: Windows_NT x64 10.0.22621
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-1065G7 CPU @ 1.30GHz (8 x 1498)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
7.69GB (2.65GB free)

Process Argv
--crash-reporter-id cf6798e1-f3c9-4ba2-a7d8-cc44c35b7faf

Screen Reader
no

VM
0%

Extensions: none

Can you please provide following logs while you reproduce the error

F1 > Open View... Shared
F1 > Toggle Shared Process > Console

will you please elaborate it. It will be better if you tell me step by step

Reproduce the issue
Press F1 > Type Open View... > Select Shared - This opened Shared log
Press F1 > Type Toggle Shared Process > Console - Share the content in the console

Reproduce the issue
Press F1 > Type Open View... > Select Shared - This opened Shared log
Press F1 > Type Toggle Shared Process > Console - Share the content in the console

whenever I click on extension inside vs code it shows me error "error while fetching extension. XHR failed".
after this when I press 'F1'` it redirects me to new tab in microsoft edge.
I am not able to find shared log

Ok, instead of pressing F1 , Please do following:

Click on Gear icon on the bottom left and Click Command Pallette

here is content of console.
sharedProcessMain.js:37   ERR RequestService#request (browser) - error https://az764295.vo.msecnd.net/extensions/marketplace.json Error: XHR failed
at I.onerror (sharedProcessMain.js:16:77412)
az764295.vo.msecnd.net/extensions/marketplace.json:1          Failed to load resource: net::ERR_TUNNEL_CONNECTION_FAILED
sharedProcessMain.js:37  INFO Starting worker process with pid 13968 (type: fileWatcher, window: 1).
sharedProcess.js:4 [uncaught exception]: Error: connect ETIMEDOUT <IP_ADDRESS>:443
a @ sharedProcess.js:4
sharedProcess.js:4 Error: connect ETIMEDOUT <IP_ADDRESS>:443
at __node_internal_captureLargerStackTrace (node:internal/errors:464:5)
at __node_internal_exceptionWithHostPort (node:internal/errors:642:12)
at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1157:16)
a @ sharedProcess.js:4
sharedProcessMain.js:37  INFO Worker process with pid 13968 terminated normally (type: fileWatcher, window: 1).
sharedProcessMain.js:37  INFO Starting worker process with pid 9964 (type: fileWatcher, window: 1).
DevTools failed to load source map: Could not load content for https://ticino.blob.core.windows.net/sourcemaps/441438abd1ac652551dbe4d408dfcec8a499b8bf/core/vs/base/parts/sandbox/electron-browser/preload.js.map: Connection error: net::ERR_TUNNEL_CONNECTION_FAILED
DevTools failed to load source map: Could not load content for https://ticino.blob.core.windows.net/sourcemaps/441438abd1ac652551dbe4d408dfcec8a499b8bf/core/vs/code/electron-browser/sharedProcess/sharedProcess.js.map: Connection error: net::ERR_TUNNEL_CONNECTION_FAILED
DevTools failed to load source map: Could not load content for https://ticino.blob.core.windows.net/sourcemaps/441438abd1ac652551dbe4d408dfcec8a499b8bf/core/vs/code/electron-browser/sharedProcess/sharedProcessMain.js.map: Connection error: net::ERR_TUNNEL_CONNECTION_FAILED
DevTools failed to load source map: Could not load content for<EMAIL_ADDRESS>Connection error: net::ERR_TUNNEL_CONNECTION_FAILED
DevTools failed to load source map: Could not load content for<EMAIL_ADDRESS>Connection error: net::ERR_TUNNEL_CONNECTION_FAILED
DevTools failed to load source map: Could not load content for<EMAIL_ADDRESS>Connection error: net::ERR_TUNNEL_CONNECTION_FAILED
DevTools failed to load source map: Could not load content for<EMAIL_ADDRESS>Connection error: net::ERR_TUNNEL_CONNECTION_FAILED
DevTools failed to load source map: Could not parse content for https://ticino.blob.core.windows.net/sourcemaps/441438abd1ac652551dbe4d408dfcec8a499b8bf/core/vs/base/worker/workerMain.js.map: Unexpected end of JSON input
DevTools failed to load source map: Could not parse content for https://ticino.blob.core.windows.net/sourcemaps/441438abd1ac652551dbe4d408dfcec8a499b8bf/core/vs/platform/sharedProcess/electron-browser/sharedProcessWorkerMain.js.map: Unexpected end of JSON input

ERR_TUNNEL_CONNECTION_FAILED - seems proxy related.

ERR_TUNNEL_CONNECTION_FAILED - seems proxy related.

what should I do now?

Could you start VS Code with --log-net-log=netlog.json, retry the failing feature and then post the created netlog.json here?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Improve Clojure Syntax support

Clojure language use too much symbols and by having the same color for symbols and vars make it harder for the eyes
before

after

You should not modify the json file, see https://github.com/sainnhe/gruvbox-material-vscode#contributing


Parse error when using img with ngfSrc directive

Copying the simple example from your readme.md, I am trying to display an image thumbnail:

The directive is picking this up as I can log that its link function is triggered, but as soon as I try to upload a file I get an error:
Error: [$parse:syntax] http://errors.angularjs.org/1.3.15/$parse/syntax?p0=*&p1=not%20a%20primary%20expression&p2=7&p3=image%2F*&p4=*     at Error (native)     at http://dev.local/vendor_dev/angular/angular.min.js:6:417     at ib.throwError (http://dev.local/vendor_dev/angular/angular.min.js:191:165)     at ib.primary (http://dev.local/vendor_dev/angular/angular.min.js:190:419)     at ib.unary (http://dev.local/vendor_dev/angular/angular.min.js:197:478)     at ib.multiplicative (http://dev.local/vendor_dev/angular/angular.min.js:197:273)     at ib.additive (http://dev.local/vendor_dev/angular/angular.min.js:197:63)     at ib.relational (http://dev.local/vendor_dev/angular/angular.min.js:196:435)     at ib.equality (http://dev.local/vendor_dev/angular/angular.min.js:196:291)     at ib.logicalAND (http://dev.local/vendor_dev/angular/angular.min.js:196:167)
This error does not occur if I remove the thumbnail image tag.
I am using the ngf-drop  area only:

Drag images here

File Drag/Drop is not supported for this browser

ngf-default-src="'/thumb.jpg'"

Ah, of course. Thanks!


feat: Adding connection code to label-studio

Currently, it has 4 commands one is start_label_studio, export_results, make_local_storage and dir_to_tasks.
start_label_studio does the following:

takes a video and extracts frames from it
samples the frames based on the users choice
creates a pre-configured Label Studio project
Adds the data as tasks to Label Studio
export_results:
Allows the user to get the output in desired folder (only JSON format is supported by Label-studio)

make_local_storage:
makes a REST API call to changes the default behaviour of Label Studio to allow local data to be used for tasks.
dir_to_tasks:
adds the data in a directory as a tasks.

cc @gaurav274 @jiashenC @pchunduri6 @kaushikravichandran @Anirudh58


feat: samples for Spanner PostgreSQL JDBC

Adds samples for using the Cloud Spanner JDBC driver with databases in the PostgreSQL dialect.

cc @ansh0l @jin-jj

cc @jin-jj .

Overall, LGTM

@averikitsch Could we get an owner's review from you?

@eaball35 Would you mind merging this PR (I don't have permission in this repo to merge anything)

@eaball35 @Shabirmean @ansh0l Does any of you have write access to this repo? If so, could any of you merge it? I don't have write access, so I cannot merge this PR.


Vulnerable for SQL injection

There is no escaping applied at every query making it vulnerable to SQL injection. Could also use prepared statements.
https://github.com/miguelavaqrod/bolt-newsletter-subscription/blob/master/Extension.php#L42

Hi!
I do apologize, but I was unable to stay in contact.
Going to check it...

That seems solve with the last PR #4 .
@miguelavaqrod Could you create a release for Bolt v3 ?

OMG...
@nbehier I do apologize for this big delay in answering you.
I checked that you forked the repo and fixed those issues.
I will merge your fixes in this repo so people can update their code if needed.
Thanks a lot.


internal compiler error on atomic_xchg::()

Consider the following program:
// test.rs

#![feature(core_intrinsics)]

pub trait Foo {}

pub struct Bar;

impl Foo for Bar {}

pub fn main() {
    let b = Box::new(Bar) as Box<Foo>;
    let mut x: *const Foo = &*b;
    let y: *const Foo = &*b;
    unsafe {
        ::std::intrinsics::atomic_xchg(&mut x, y);
    }
}

When I try to compile test.rs with rustc, I get a monomorphization error:
$ rustc test.rs
error[E0511]: invalid monomorphization of `atomic_xchg` intrinsic: expected basic integer type, found `*const Foo`
  --> test.rs:16:9
   |
16 |         ::std::intrinsics::atomic_xchg(&mut x, y);
   |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

error: aborting due to previous error(s)

When I run test.rs with miri, I expect to see a similar error. Instead, I get an "internal compiler error":
$ cargo run --bin miri -- test.rs
    Finished dev [unoptimized + debuginfo] target(s) in 0.0 secs
     Running `target/debug/miri test.rs`
error: internal compiler error: src/eval_context.rs:1143: value_to_primval can't work with fat pointers

thread 'main' panicked at 'Box<Any>', /checkout/src/librustc_errors/lib.rs:480

Similar to #181. If the rustc typechecker caught these errors then there would be no problem.

I'd consider this a dupe. Let's keep it all in the other issue.


Display all one-shots

I'd like to be able to see all one-shots, not just the ones local to the scene. I need my Wilhelm Scream at the drop of a hat.
(Yeah, I'm going to make a macro for it. thanks for that)

Hi!
Do you use the CSV synchronization method?
Unfortunately, last time I checked the CSV did not include global one-shots.
The API method does load global one-shots.
For now there are two steps to do:

I need to investigate the CSV content again to see if its really missing globals.
If yes, then I should write a guide how to create global macro anyway. It is possible but its inconvenient:
For now, shortened version:
You need to go to Master Panel, and click "Show Remote Control links".
Now you can click your Wilhelm Scream, it should give you a link like:

https://syrinscape.com/online/frontend-api/elements/2357/play/?auth_token=<your_token>

Next, you copy the ID, in that case, 2357.
And paste it in your macro:
game.syrinscape.playElement(2357)

Unfortunately, I checked CSV, and while there is Wilhelm scream, it's one linked to mood, not a global one.

Closed: 0.3.0 uses API which fetches global one-shots properly.


How to autoplay youtube video?

So, you're experiencing a problem?
If you're not sure how to use the lightbox, or are experiencing a problem with it, please ask on StackOverflow - you will most definitely get a faster response time and it leaves this GitHub issues for actual bugs and not support questions.
If you really do think you have a bug, please:

[http://ashleydw.github.io/lightbox/#videos ] Provide a link to your site or a full example

[Chrome, windows 10 ] Browser name and version

In your site also autoplay of youtube not working.
http://ashleydw.github.io/lightbox/#videos

yes, looking for solution for that too. autoplay=1 is not working.

autoplay=1 on the link seems to work for me in Chrome OK.


bug issue in context manager both in actions and probes files resolved

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

The http probes function always throws true even when the target machine is down..
Popen().returncode will always give None inside context manager. Instead we can go with Popen().wait() - this will return actual return code.
I think the same issue is there in actions.py file also.. the issue is more prominent in probes.py file since we use this in steady state hypothesis.
Kindly let me know if you need more clarification.. thanks

Attached a code snippet for reference

Hi team,
Kindly review and approve my PR.


Jumpy RGB video...

Required Info

Camera Model
SR300

Firmware Version

Operating System & Version
Mac OS X 10.12.3

Kernel Version (Linux Only)

Build System
XCode and Qt Creator

I just updated my librealsense libraries, thinking this might help, but to no avail.  I am having really jumpy RGB video from my SR300.  I can grad depth and IR video as smooth as silk, but the RGB video looks like its dropping frames.  My code is grabbing video at 30 fps, but the visual appearance of the video looks like something is happening in the drivers where frames of video seem to freeze up.

I've seen this issue before on the OSX/libusb backend with SR300 before. If I remember correctly, you can try reducing the number of buffers from 64 to something more reasonable (2-16?). Requires a recompile: https://github.com/IntelRealSense/librealsense/blob/master/src/libuvc/libuvc_internal.h#L217


Runtime control of hidden options

Is there a way to control the hidden attribute of a command line option at runtime?  The use case I'm working with requires that an option should only be available to the user when executing the application on a specific operating system.  For example:
private static final boolean isDarwin = Utils.isOSType(OSType.Darwin);

@CommandLine.Option(names = {"-o", "--open"},
    description = "On OSX open the finder to the data dir that was downloaded.",
    showDefaultValue = CommandLine.Help.Visibility.ALWAYS,
    hidden = !isDarwin) // hide this option on all OS's other than Darwin (OSX)
boolean open = false;

The above example would be ideal, although the compiler complains that the value for the hidden attribute must be a constant expression.  I'm guessing it has to do with the way the annotation processor generates the code at compile time.
Since the above isn't supported, can I get at the CommandLine.Option object and toggle the hidden attribute at runtime, let's say like in the constructor of my object?
Any feedback is welcome.  Thanks in advance!

Changing the model is currently not easy.
I’m looking into whether it’s possible to change the rendering of the usage help message so that hidden options are shown. I’ll post a recipe when I have something.

I had a look and I need to make some changes to the CommandLine.Help class and its components to facilitate this. Your timing is good: this will require some API changes, so it's good to do this in a major release like the upcoming version 4.0.
My goal is to change things so that you can customize like this:
import picocli.CommandLine;
import picocli.CommandLine.Command;
import picocli.CommandLine.Help;
import picocli.CommandLine.Help.ColorScheme;
import picocli.CommandLine.IHelpFactory;
import picocli.CommandLine.Model.CommandSpec;
import picocli.CommandLine.Model.OptionSpec;
import picocli.CommandLine.Option;

@Command(name = "show-hidden-options")
public class ShowHiddenOptions {
    @Option(names = "--visible") String visible;
    @Option(names = "--hidden", hidden = true) String hidden;

    public static void main(String[] args) {
        final boolean IS_MAC_OS = System.getProperty("os.name").toLowerCase().contains("mac");

        CommandLine cmd = new CommandLine(new ShowHiddenOptions());
        cmd.setHelpFactory(new IHelpFactory() {
            @Override
            public Help create(CommandSpec commandSpec, ColorScheme colorScheme) {
                return new Help(commandSpec, colorScheme) {
                    @Override
                    protected boolean isHidden(OptionSpec option) {
                        return option.hidden() && !IS_MAC_OS;
                    }
                };
            }
        });

        cmd.usage(System.out);
    }
}

I hope to have something for you later this week.

Sorry for the delay.
I took a stab at implementing this in the Help API but I started to have doubts that this is really the right way to go.
Allowing applications to change the model is a more general and more powerful solution.
At the moment OptionSpec instances are immutable, and I would like to keep it that way, but the CommandSpec is not immutable. So, I am thinking to add the ability to replace an OptionSpec.
Something like this:
CommandLine cmd = new CommandLine(new MyApp());

// replace the old "--open" option with a different one that is not hidden on Darwin
CommandSpec spec = cmd.getCommandSpec();
OptionSpec old = spec.findOption("--open");
OptionSpec newOpenOption = OptionSpec.builder(old).hidden(!isDarwin);
spec.remove(old);
spec.add(newOpenOption);

cmd.execute(args);

I added a CommandSpec.remove(ArgSpec) method that makes it possible to do thing like in the above example: remove option, make a copy with a different attribute, then add the copy back.
See also this test.
Can you verify?

Is there a way to accomplish the same but with an ArgSpec that is part of an ArgGroup?
When removing the ArgSpec from the CommandSpec:
public CommandSpec remove(ArgSpec arg) {
  if (arg.group() != null) {
    throw new UnsupportedOperationException("Cannot remove ArgSpec that is part of an ArgGroup");
  }
...

As far as I can see the ArgGroup is immutable as well.
Is there any workaround?
Thank you in advance.

@JPedroBorges Currently there’s no way to modify argument groups. Can you raise a separate ticket for this?


StateHasChanged Explained (vs base.InvokeAsync(StateHasChanged)

[EDIT by guardrex to add the metadata. We'll get an update from the product unit on this after 5.0 reaches GA. Everyone is 🏃😅 on 5.0 at the moment (9/24).]
The differences between StateHasChanged() and base.InvokeAsync(StateHasChanged); is never clearly explained.
A topic on when a developer should invoke state has changed (via either method) as well has when is best to use each option would be helpful.
I understand that one handles things asynchronously, but are there times when it is better to have on method over the other? And what is being changed automatically vs us needing to call the method manually?
Location:
Blazor --> Web Apps --> Components / State Management / "New Item"
Abstract:
This topic will cover how State is managed in a Component, when it automatically updates, when a developer may want to manually update, and what options are available to the developer for updating the state.
Potential Outline
1.0 - Definitions: Defining State in Blazor Components.
2.0 - How State is Updated
2.1 - Automatic Updates
2.2 - Manual Update
2.2.1 - Example Scenarios when you will want to manually update.
3.0 - Manual Update Options
3.1 - StateHasUpdated(); Explained
3.1.1 - Examples
3.2 - base.InvokeAsync(StateHasChanged); Explained
3.2.1 - Examples

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

ID: a970fb72-3dd0-d173-585d-2ccdd7923f81
Version Independent ID: 4871f4f7-9cf6-ac7a-0f21-c65d34bd1119
Content: ASP.NET Core Blazor lifecycle
Content Source: aspnetcore/blazor/components/lifecycle.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

Hello @kevingates ...
It's covered lightly in a spots where it's relevant to make the call, typically for a render when Blazor isn't aware that it should update the UI. For example:

Invoke component methods externally to update state
TabSet example (cascading parameters)
Invoke JavaScript functions from .NET methods

... but yes to possibly covering it further. The good spots for more coverage are probably either or both of:

Lifecycle topic at State changes.
State Management as you suggested. I think 🤔 ... hope :pray: ... that we can place coverage in the existing topic as opposed to creating a new topic. I don't think very extensive coverage is required .. maybe. :smile:

I'll mark this for triage on the Blazor topic for engineering to review. Stand-by tho if it takes a while to get an answer ... we're very close to the holidays 🎁⛄, so I think it's likely that they won't remark on this until we get into 2020.

Let's add more information about the StateHasChanged in the Lifecycle page: https://docs.microsoft.com/en-us/aspnet/core/blazor/lifecycle?view=aspnetcore-3.1#state-changes

@guardrex I've written the following suggested documentation to cover this topic. Do you think this is suitable, and where do you think it should be added? I would suggest adding it as a new page called "Rendering" within Blazor's "Fundamendals" category.
If you want I can convert this into an actual PR for this, but if that's the sort of thing you can do in minutes yourself (having decided really where it should go) then that sounds good to me.

When components render
All components must render when they are first added to the component hierarchy by their parent component. This is the only time that a component strictly must render.
A component may choose to render at any other time according to its own logic and conventions.
Conventions for ComponentBase
By default, .razor components inherit from ComponentBase. This base class contains logic to trigger re-rendering at the following times:

After applying an updated set of parameters from the parent component
After applying an updated value for a cascading parameter
After being notified of an event and invoking one of its own event handlers
After a call to its own StateHasChanged() method

Components inherited from ComponentBase will skip the re-render (but not the initial render) if any of the following are true:

All of the parameter values are of known immutable primitive types (int, string, DateTime, etc.) and have not changed since the previous set of parameters
The component's own ShouldRender() method returns false

Controlling the rendering flow
In most cases, the ComponentBase conventions mean that the correct subset of components will re-render after an event, without developers needing to write any manual logic to tell the framework which components to re-render. The overall effect of the conventions is that the component receiving an event will itself re-render, which recursively triggers re-rendering of descendant components whose parameter values may have changed.
When to call StateHasChanged()
The StateHasChanged() method on ComponentBase lets you trigger a render at any time. However you should be careful not to call this unnecessarily (which is a common mistake) because that would impose unnecessary extra rendering costs.
You should not need to call StateHasChanged when:

Routinely handling events, whether synchonrously or asynchronously, since ComponentBase triggers a render in those cases anyway
Implementing typical lifecycle logic, such as OnInitialized or OnParametersSetAsync, whether synchonrously or asynchronously, since ComponentBase triggers a render in those cases anyway

However, it can make sense to call StateHasChanged() in the following cases.
When an asynchronous handler involves multiple asynchronous phases
Consider the following "counter" component, which updates the count four times on each click:
@page "/counter"

Current count: @currentCount

<button class="btn btn-primary" @onclick="IncrementCount">Click me</button>

@code {
    private int currentCount = 0;

    private async Task IncrementCount()
    {
        currentCount++;
        // Renders here automatically

        await Task.Delay(1000);
        currentCount++;
        StateHasChanged();

        await Task.Delay(1000);
        currentCount++;
        StateHasChanged();

        await Task.Delay(1000);
        currentCount++;
        // Renders here automatically
    }
}

The way that tasks are defined in .NET means that a receiver of the Task can only observe its final completion, not any intermediate asynchronous states. So, ComponentBase can only know about triggering re-rendering when you first return the Task and when that Task finally completes. It cannot know to re-render at other intermediate points. So if you do want to re-render at intermediate points, use StateHasChanged().
When receiving a call from something external to the Blazor rendering and event handling system
ComponentBase only knows about its own lifecycle methods and Blazor-triggered events. It doesn't know about anything else that can trigger your code.
For example, if you have a custom data store that raises C# events, then those C# events are unknown to Blazor. As another example, consider the following "counter" component that uses System.Timers.Timer to update the count at a regular interval:
@page "/counter"
@using System.Timers
@implements IDisposable

Current count: @currentCount

@code {
    private int currentCount = 0;
    private Timer timer = new Timer(1000);

    protected override void OnInitialized()
    {
        timer.Elapsed += (sender, eventArgs) => OnTimerCallback();
        timer.Start();
    }

    void OnTimerCallback()
    {
        _ = InvokeAsync(() =>
        {
            currentCount++;
            StateHasChanged();
        });
    }

    void IDisposable.Dispose() => timer.Dispose();
}

In this example, OnTimerCallback has to call StateHasChanged() for the UI to be updated, because otherwise Blazor doesn't know anything is happening. OnTimerCallback runs outside of any Blazor-managed rendering flow or event notificaiton.
Similarly, because this callback is invoked outside Blazor's synchronization context, it's necessary to wrap the logic in InvokeAsync to move it onto the renderer's synchronization context. StateHasChanged() can only be called from the renderer's synchronization context and will throw an exception otherwise. This is equivalent to marshalling to the UI thread in other UI frameworks.
To render component outside the subtree that is re-rendered by a particular event
Your UI might involve dispatching an event to one component, changing some state, and needing to re-render a completely different component that is not a descendant of the one receiving the event.
One way of doing this is to have some "state management" class, perhaps as a DI service, injected into multiple components. When one component calls a method on the state manager, the state manager may raise a C# event that is then received by an independent component.
Since these C# events are outside the Blazor rendering pipeline, you would need to call StateHasChanged() on other components you wish to render in response to your state manager's events.
This is very similar to the previous case with System.Timers.Timer. However, since your execution call stack typically remains on the renderer's synchronization context, you will not normally need to use InvokeAsync() in this case. You would only need to use InvokeAsync() if your logic does something to escape from the synchronization context, such as calling ContinueWith on a Task, or awaiting a Task with ConfigureAwait(false).

@guardrex I've written the following suggested documentation to cover this topic. Do you think this is suitable, and where do you think it should be added? I would suggest adding it as a new page called "Rendering" within Blazor's "Fundamendals" category.
If you want I can convert this into an actual PR for this, but if that's the sort of thing you can do in minutes yourself (having decided really where it should go) then that sounds good to me.

When components render
All components must render when they are first added to the component hierarchy by their parent component. This is the only time that a component strictly must render.
A component may choose to render at any other time according to its own logic and conventions.
Conventions for ComponentBase
By default, .razor components inherit from ComponentBase. This base class contains logic to trigger re-rendering at the following times:

After applying an updated set of parameters from the parent component
After applying an updated value for a cascading parameter
After being notified of an event and invoking one of its own event handlers
After a call to its own StateHasChanged() method

Components inherited from ComponentBase will skip the re-render (but not the initial render) if any of the following are true:

All of the parameter values are of known immutable primitive types (int, string, DateTime, etc.) and have not changed since the previous set of parameters
The component's own ShouldRender() method returns false

Controlling the rendering flow
In most cases, the ComponentBase conventions mean that the correct subset of components will re-render after an event, without developers needing to write any manual logic to tell the framework which components to re-render. The overall effect of the conventions is that the component receiving an event will itself re-render, which recursively triggers re-rendering of descendant components whose parameter values may have changed.
When to call StateHasChanged()
The StateHasChanged() method on ComponentBase lets you trigger a render at any time. However you should be careful not to call this unnecessarily (which is a common mistake) because that would impose unnecessary extra rendering costs.
You should not need to call StateHasChanged when:

Routinely handling events, whether synchonrously or asynchronously, since ComponentBase triggers a render in those cases anyway
Implementing typical lifecycle logic, such as OnInitialized or OnParametersSetAsync, whether synchonrously or asynchronously, since ComponentBase triggers a render in those cases anyway

However, it can make sense to call StateHasChanged() in the following cases.
When an asynchronous handler involves multiple asynchronous phases
Consider the following "counter" component, which updates the count four times on each click:
@page "/counter"

Current count: @currentCount

<button class="btn btn-primary" @onclick="IncrementCount">Click me</button>

@code {
    private int currentCount = 0;

    private async Task IncrementCount()
    {
        currentCount++;
        // Renders here automatically

        await Task.Delay(1000);
        currentCount++;
        StateHasChanged();

        await Task.Delay(1000);
        currentCount++;
        StateHasChanged();

        await Task.Delay(1000);
        currentCount++;
        // Renders here automatically
    }
}

The way that tasks are defined in .NET means that a receiver of the Task can only observe its final completion, not any intermediate asynchronous states. So, ComponentBase can only know about triggering re-rendering when you first return the Task and when that Task finally completes. It cannot know to re-render at other intermediate points. So if you do want to re-render at intermediate points, use StateHasChanged().
When receiving a call from something external to the Blazor rendering and event handling system
ComponentBase only knows about its own lifecycle methods and Blazor-triggered events. It doesn't know about anything else that can trigger your code.
For example, if you have a custom data store that raises C# events, then those C# events are unknown to Blazor. As another example, consider the following "counter" component that uses System.Timers.Timer to update the count at a regular interval:
@page "/counter"
@using System.Timers
@implements IDisposable

Current count: @currentCount

@code {
    private int currentCount = 0;
    private Timer timer = new Timer(1000);

    protected override void OnInitialized()
    {
        timer.Elapsed += (sender, eventArgs) => OnTimerCallback();
        timer.Start();
    }

    void OnTimerCallback()
    {
        _ = InvokeAsync(() =>
        {
            currentCount++;
            StateHasChanged();
        });
    }

    void IDisposable.Dispose() => timer.Dispose();
}

In this example, OnTimerCallback has to call StateHasChanged() for the UI to be updated, because otherwise Blazor doesn't know anything is happening. OnTimerCallback runs outside of any Blazor-managed rendering flow or event notificaiton.
Similarly, because this callback is invoked outside Blazor's synchronization context, it's necessary to wrap the logic in InvokeAsync to move it onto the renderer's synchronization context. StateHasChanged() can only be called from the renderer's synchronization context and will throw an exception otherwise. This is equivalent to marshalling to the UI thread in other UI frameworks.
To render component outside the subtree that is re-rendered by a particular event
Your UI might involve dispatching an event to one component, changing some state, and needing to re-render a completely different component that is not a descendant of the one receiving the event.
One way of doing this is to have some "state management" class, perhaps as a DI service, injected into multiple components. When one component calls a method on the state manager, the state manager may raise a C# event that is then received by an independent component.
Since these C# events are outside the Blazor rendering pipeline, you would need to call StateHasChanged() on other components you wish to render in response to your state manager's events.
This is very similar to the previous case with System.Timers.Timer. However, since your execution call stack typically remains on the renderer's synchronization context, you will not normally need to use InvokeAsync() in this case. You would only need to use InvokeAsync() if your logic does something to escape from the synchronization context, such as calling ContinueWith on a Task, or awaiting a Task with ConfigureAwait(false).

adding it as a new page called "Rendering" within Blazor's "Fundamendals" category

We've been classifying "fundamentals" subjects as more like 'fundamental to any app' (e.g., logging, config, DI, environments, startup ... etc.). Are you ok if this goes into the Components node of topics somewhere under the Overview ...
https://docs.microsoft.com/aspnet/core/blazor/components/

convert this into an actual PR

No worries ... I'll take care of it and ping u on the PR later.

adding it as a new page called "Rendering" within Blazor's "Fundamendals" category

We've been classifying "fundamentals" subjects as more like 'fundamental to any app' (e.g., logging, config, DI, environments, startup ... etc.). Are you ok if this goes into the Components node of topics somewhere under the Overview ...
https://docs.microsoft.com/aspnet/core/blazor/components/

convert this into an actual PR

No worries ... I'll take care of it and ping u on the PR later.

Yes, please place it wherever you feel is best! Thanks for making the PR.

Yes, please place it wherever you feel is best! Thanks for making the PR.

@SteveSandersonMS : is it possible for you to address when to use await base.InvokeAsync(StateHasChanged); versus StateHasChanged()? I don't see that mentioned in the documentation. It looks like you were getting to that at the end maybe but nothing was explicitly mentioned.
That sound: wow, thank you for this documentation. It's very help! :)

@SteveSandersonMS : is it possible for you to address when to use await base.InvokeAsync(StateHasChanged); versus StateHasChanged()? I don't see that mentioned in the documentation. It looks like you were getting to that at the end maybe but nothing was explicitly mentioned.
That sound: wow, thank you for this documentation. It's very help! :)

@SteveSandersonMS, I'll have this up and ready to merge shortly. There are only grammar nits, so I'm not planning on pinging u for review. I'll wait to merge the PR until tomorrow (Wednesday) morning to see if you respond to @kevingates and supply additional text for the topic to cover that.

@SteveSandersonMS, I'll have this up and ready to merge shortly. There are only grammar nits, so I'm not planning on pinging u for review. I'll wait to merge the PR until tomorrow (Wednesday) morning to see if you respond to @kevingates and supply additional text for the topic to cover that.

is it possible for you to address when to use await base.InvokeAsync(StateHasChanged); versus StateHasChanged()?

I'm not sure where the base. prefix is coming from. Is that mentioned somewhere else in docs? If so we should change that.
The InvokeAsync method is on the base class. Prefixing the invocation with base. doesn't achieve anything. It's the same as just calling InvokeAsync. The docs I wrote do discuss when InvokeAsync is needed.

is it possible for you to address when to use await base.InvokeAsync(StateHasChanged); versus StateHasChanged()?

I'm not sure where the base. prefix is coming from. Is that mentioned somewhere else in docs? If so we should change that.
The InvokeAsync method is on the base class. Prefixing the invocation with base. doesn't achieve anything. It's the same as just calling InvokeAsync. The docs I wrote do discuss when InvokeAsync is needed.

It's not anywhere in the docs.

It's not anywhere in the docs.

I guess I don't see it anymore either. Note, this issue was originally posted December 2019. I remember finding that somewhere, but perhaps it was removed / edited. If it isn't relevant now (and not currently listed in documentation) then don't worry about.
I appreciation the clarification.

I guess I don't see it anymore either. Note, this issue was originally posted December 2019. I remember finding that somewhere, but perhaps it was removed / edited. If it isn't relevant now (and not currently listed in documentation) then don't worry about.
I appreciation the clarification.

Yes, that's possible. The docs have churned a lot since then.

Yes, that's possible. The docs have churned a lot since then.


remove unused files

just remove unused logo images and fix typosLiveURL: https://smartfaq-clean-repo.surge.sh

@kantord  quick review :)

Has conflicts :(


Package Update User is not supported/ not working

Please check http://stackoverflow.com/questions/28847090/laravel-sentry-update-the-user It seems that your package has an error there.
It says that user->save is an undefined method. The problem is that it cannot find the user by ID and is instead returning all the users.
I had programmed everything with your package but will have to switch to the standard Auth.

Feel free to submit a PR with a fix.

I do not have a fix for it. That's why I was asking. You are basically
blowing up my masters thesis, I was really hoping to get some help with it.
Can you help me? I do not understand the inside of your code.
On Wed, Mar 4, 2015 at 7:57 PM, Bruno Gaspar<EMAIL_ADDRESS>wrote:

Feel free to submit a PR with a fix.
—
Reply to this email directly or view it on GitHub
https://github.com/cartalyst/sentry/issues/515#issuecomment-77284972.

@seiccn my suggestion is for you to debug your code first, maybe do dd($user); and see what it returns and start from there.
You can look here too since you say that the problem is probably there.
Sorry if i can't be of better assistance, but i don't have time to setup an environment with Sentry at the moment to try to replicate your issue, sorry.


changes re prefixOrder

https://github.com/atextor/turtle-formatter/?tab=readme-ov-file#nice-and-configurable-formatting says

(1) Prefixes are sorted by common, then custom.

https://atextor.de/owl-cli/main/snapshot/usage.html#write-command says

--prefixOrder Sort order for prefixes (if they appear in a model) default [rdf, rdfs, xsd, owl]

But take https://www.entsoe.eu/Documents/CIM_documents/Grid_Model_CIM/IEC61970-600-2_CGMES_3_0_1_ApplicationProfiles.zip
eg this ontology v3.0/RDFSEd2Beta/IEC61970-600-2_CGMES_3_0_0_RDFS_501Ed2CD_EQ.rdf .
Both documents above agree, and the output conforms; the rest of the prefixes are sorted alphabetically.
But I question the usability of the result.
In the result below, I marked the "custom" prefixes with ###:
@prefix rdf: <http://www.w3.org/1999/02/22-rdf-syntax-ns#> .
@prefix rdfs: <http://www.w3.org/2000/01/rdf-schema#> .
@prefix xsd: <http://www.w3.org/2001/XMLSchema#> .
@prefix owl: <http://www.w3.org/2002/07/owl#> .
@prefix cim: <http://iec.ch/TC57/CIM100#> . ###
@prefix cims: <http://iec.ch/TC57/1999/rdf-schema-extensions-19990926#> . ###
@prefix dcat: <http://www.w3.org/ns/dcat#> .
@prefix dct: <http://purl.org/dc/terms/> .
@prefix eq: <http://iec.ch/TC57/ns/CIM/CoreEquipment-EU#> . ###
@prefix eu: <http://iec.ch/TC57/CIM100-European#> . ###
@prefix skos: <http://www.w3.org/2004/02/skos/core#> .

You see that the interesting (custom) prefixes are spread all over the place.
This is IMHO worse than alphabetically sorting all prefixes.
(That ontology doesn't reuse dcat, dct, skos: it just uses them for metadata.)
So my proposal is this:

rename prefixOrder to externalPrefixes
put externalPrefixes last since the interesting stuff should come first.
put a blank line before them
make the default value empty since what should be considered "external" (or "standard") is IMHO ontology-dependent.

Hi @VladimirAlexiev,
thanks for bringing this up. I agree with you that this features needs to be better configurable. However I don't agree with you on which prefixes are "interesting" or where they should go; I personally prefer standard prefixes at the top, so reading prefixes from top to bottom always starts with the same things across different files. That's the reason I chose those prefixes as the default value.
The idea of prefixOrder therefore is to choose exactly what order you want, including both "standard" and "external" prefixes, so it does not only apply to the latter. The default value just happens to include "standard" prefixes.
By using a prefixOrder of e.g. cim, cims, eq, eu, rdf, rdfs, xsd, owl, dcat, dct, skos, you should be able to achieve what you want.
Allowing (configurable) blank lines to be inserted at certain places in the prefixes list is a nice idea.

But see also the linked issue.
That's why I called them externalPrefixes


Tooltip to unvote show "Dislike" instead of "Unlike"

When i vote for a post and then bring my mouse over the like icon there will be a tooltip "Dislike":

This is wrong, there should be a tooltip "Unlike" instead, "Dislike" should be used for downvote only.

Fixed in: https://github.com/busyorg/busy/pull/952


No columns in label preview/printer

Describe the bug
When I print a long set of labels that require "columning" - these columns do not show up/get formatted at all
To Reproduce
Steps to reproduce the behavior:

When I  generate a lot of labels that should get "columned" (e.g. 100 QR codes)
Then I click preview/print
See one long column instead of a specified number of columns

Specs:

Result (no columns):

Expected behavior
Should get columns
Environment (please identify where you experience this bug:
*** One of ***

Sandbox - Sandworm
Firefox

@tmcelrath I think resolved ... maybe?

Not in production? Works in Sandworm I think.


FortiClient VPN Package update failure (FortiClient ZTNA replaced FortClient VPN Only client)

Brief description of your issue
We use WAU (Winget Auto Updater) with GPO deployed and configured via Intune to our machines, policy orginally was to autoupdate all apps (Blanket policy) without a whitelist of blacklist, we have had to blacklist the ID Fortinet.FortiClientVPN this morning a MSI was run to update Forticlient from Winget and replaced the VPN only client with Forticlient's ZTNA client across the board, the annoying part is apart of the install is to trigger the update client side on first launch so all clients are prompted with an Admin Prompt which cannot be avoided until we uninstall ZTNA and install VPN Only.
Logs:

Beginning a Windows Installer transaction: C:\WINDOWS\TEMP\WinGet\defaultState\Fortinet.FortiClientVPN.<IP_ADDRESS>3\extracted\FortiClient.msi. Client Process Id: 24532.

Starting session 0 - 2024-07-16T20:34:37.009371500Z.

Application 'C:\Program Files\Fortinet\FortiClient\FortiTray.exe' (pid 3804) cannot be restarted - Application SID does not match Conductor SID..

Machine restart is required.

Starting session 1 - 2024-07-16T20:34:38.487256600Z.

Application 'C:\Program Files\Fortinet\FortiClient\FortiTray.exe' (pid 3804) cannot be restarted - Application SID does not match Conductor SID..

Application or service 'FortiClient SSLVPN daemon' could not be shut down.

Application or service 'FortiClient Logging daemon' could not be shut down.

Application or service 'FortiClient Settings Service' could not be shut down.

Application or service 'FortiClient VPN Service Scheduler' could not be shut down.

Windows Installer requires a system restart. Product Name: FortiClient VPN. Product Version: <IP_ADDRESS>5. Product Language: 1033. Manufacturer: Fortinet Technologies Inc. Type of System Restart: 1. Reason for Restart: 2.

The operation completed successfully.

Product: FortiClient -- Installation completed successfully.

Ending a Windows Installer transaction: C:\WINDOWS\TEMP\WinGet\defaultState\Fortinet.FortiClientVPN.<IP_ADDRESS>3\extracted\FortiClient.msi. Client Process Id: 24532.

Steps to reproduce
FortiClient VPN is installed via a Win32 Intune install package with a .bat file that installs the VPN only client and configures the VPN Profile to registry for all users.
WinGet WAU GPO Config deployed via Intune defines all apps to update at Login (now set a blacklist to block FortiClient VPN)
On login all users Fortclient.exe when launches prompts to finalise the update with elevated privileges and forces a reboot.
On reboot or relaunch FortiClient VPN is replaced with FortClient (ZTNA)
Expected behavior
We expect a Winget Update of the package to not replace the product with a different product, Yes you can connect with VPN using the ZTNA product, but it also displays a licensing error to the end users and informs them that "Free VPN will stop functioning on the 16th of August" which is problematic when we do not use EMS or ZTNA at this site.
Actual behavior
Winget List details the installed version before Winget Update:
Name                                       Id                                         Version          Available Source
FortiClient VPN                        Fortinet.FortiClientVPN        <IP_ADDRESS>8     winget
Repo Search returns the following
Name                                       Id                                         Version          Available Source
FortiClient VPN                         Fortinet.FortiClientVPN       <IP_ADDRESS>3      winget
what is returned after Winget Updates the Package
Name               Id                                                                                Version           Available Source
FortiClient        ARP\Machine\X64{079B00DA-23ED-4F29-AED8-7137…   <IP_ADDRESS>3
Environment
winget --info
Windows Package Manager v1.8.1911
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.19045.4713
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.23.1911.0

Winget Directories
-----------------------------------------------------------------------------------------------------------------------
Logs                               %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\Diagâ€¦
User Settings                      %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\settâ€¦
Portable Links Directory (User)    %LOCALAPPDATA%\Microsoft\WinGet\Links
Portable Links Directory (Machine) C:\Program Files\WinGet\Links
Portable Package Root (User)       %LOCALAPPDATA%\Microsoft\WinGet\Packages
Portable Package Root              C:\Program Files\WinGet\Packages
Portable Package Root (x86)        C:\Program Files (x86)\WinGet\Packages
Installer Downloads                %USERPROFILE%\OneDrive\Downloads

Links
---------------------------------------------------------------------------
Privacy Statement   https://aka.ms/winget-privacy
License Agreement   https://aka.ms/winget-license
Third Party Notices https://aka.ms/winget-3rdPartyNotice
Homepage            https://aka.ms/winget
Windows Store Terms https://www.microsoft.com/en-us/storedocs/terms-of-sale

Admin Setting                             State
--------------------------------------------------
LocalManifestFiles                        Disabled
BypassCertificatePinningForMicrosoftStore Disabled
InstallerHashOverride                     Disabled
LocalArchiveMalwareScanOverride           Disabled
ProxyCommandLineOptions                   Disabled
DefaultProxy                              Disabled

Winget AutoUpdater (Deployed via Windows Store (NEW)) - Intune
Winget AutoUpdater GPO is deployed via Intune custom ADMX Templates
Winget Autoupdater Config as per below

Enable Automatic Updates       - Enabled
Updates at Logon               - Enabled
Notification Level             - Enabled
Enable Start Menu Shortcuts    - Enabled
Updates Interval               - Enabled
Run WAU on metered connection  - Enabled
Activate WAU GPO Management    - Enabled

(NEW) Application GPO Blacklist - Enabled
   - Fortinet.FortiClientVPN

See comment here https://github.com/microsoft/winget-pkgs/issues/163137#issuecomment-2232123003

I found a rehosted version on this repo: https://dl.partian.co/FortiClient/Windows/7.2.4/FortiClientVPNSetup_<IP_ADDRESS>2_x64.exe
And a slightly older version on this education website's repo: http://uberupdate.lvusd.org/FortiClient/Windows/FortiClientVPNSetup_<IP_ADDRESS>4_x64.exe
I hope this helps somewhat.

Here is another, but hosted on another education institution's repo: https://download.lawr.ucdavis.edu/pub/Win/Fortinet/FortiClientVPNSetup_<IP_ADDRESS>0_x64.exe

I was unable to communicate with Fortinet support, so I am just bruteforcing the filestore url now.

OMG

@y2k04 here is the latest version <IP_ADDRESS>8. How can we add it to winget?
https://cunextgenus-my.sharepoint.com/:u:/g/personal/duncan_pereira_cunextgen_com/ESpjMS9fujlPo82VQ6qnmhoB5PrvfyZx0-Zy2sxKaZdltw?e=jfipgN

Like the moderators said in an older post about updating FortiClientVPN, just rehost it!
https://github.com/y2k04/forticlient-vpn-setup/releases/tag/<IP_ADDRESS>8

Added most of the versions I found to
https://github.com/y2k04/forticlient-vpn-setup

We unfortunately had this issue also.
Is it going to be reviewed how an installer for a different product was picked up as an upgrade to this product?
@denelon
https://github.com/microsoft/winget-pkgs/pull/163199
Thank you


Cannot pass keyword to custom recipe

Hello,
I think I sort of get the basic gist of how to create my own custom recipe as suggested in the new tutorial. However, I cannot seem to pass the keyword arguments down. That is
Let says I have
function plot!(p::Plot(MDSimulation); markersize = 1)
         simulation = to_value(p[1])
        scatter!(simulation.positions, color = :blue, markersize = markersize )
#more stuff
end

I seem that the value of markersize is always set to 1 eventhough I change value of the keyword. Now if I naively implement
function visualize(simulation::MDSimulation; markersize = 1)
   scatter!(simulation.positions, color = :blue, markersize = markersize )
   #more stuff
end

everything works fine. I have two questions

How can I make the kwarg pass down to the function body?
Is it necessary to use the syntax suggested in the tutorial? Can I just use the naive version?

2 Is it necessary to use the syntax suggested in the tutorial? Can I just use the naive version?

Yes, you need to follow the syntax in the tutorial!


Library not loaded

Environment

OS: MacOS BigSur
scrcpy version: 1.17.1
installation method: brew
device model: POT-LX1
Android version: Q

Describe the bug
Can't run at all. I tried brew update and brew cleanup and reinstalling. Too dumb to understand.
dyld: Library not loaded: /usr/local/opt/libogg/lib/libogg.0.dylib
  Referenced from: /usr/local/opt/ffmpeg/lib/libavformat.58.dylib
  Reason: image not found
Abort trap: 6

I don't know, you should report/ask to homebrew.

Hi, you should uninstall and reinstall brew and after that install again scrcpy
https://stackoverflow.com/questions/50657490/ffmpeg-macos-install-reason-image-not-found/50657591

backup brew packages (optional) brew bundle
uninstall brew ruby -e "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/master/uninstall)"
install brew again /usr/bin/ruby -e "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/master/install)"
restore the bundle brew bundle (if u did the backup)

Or can try scrcpy for macOS.


urdf_launch: 0.1.0-1 in 'iron/distribution.yaml' [bloom]

Increasing version of package(s) in repository urdf_launch to 0.1.0-1:

upstream repository: https://github.com/MetroRobots/urdf_launch.git
release repository: https://github.com/ros2-gbp/urdf_launch-release.git
distro file: iron/distribution.yaml
bloom version: 0.11.2
previous version for package: null

urdf_launch
* Initial Package
* Contributors: David V. Lu!!

Holding for Iron sync


Handle async result with null completion

If a listener method completes its async result with a null
value, do not attempt to send the result.
cherry-pick to 2.1.x

... and cherry-picked to 2.1.x as https://github.com/spring-projects/spring-amqp/commit/6a872ad447cad602f59cffaa36594b3814615f4b after fixing conflicts.


Analysis error comparing value in generic function: The operator '<' isn't defined for the type 'Object'. Try defining the operator '<'.

Dart SDK version: 2.15.1 (stable) on windows_x64
I have a class that takes an object with some settings. I need the ability to override some settings with differing logic only in specific cases and have come up with a solution that the analyzer throws an error for. Here is a stripped-down version which demonstrates the problem:
typedef SettingOrFunc = int? Function<T>(ContainsValue<T> tag);

class ContainsValue<T> {
  final T val;
  ContainsValue(this.val);
}

class UsesSetting {
  final SettingOrFunc? setting;
  UsesSetting({this.setting});

  /// Do stuff with wrappedValue here,
  /// for my project, if wrappedValue is/returns null,
  /// default values are used otherwise uses the wrappedValue's return.
}

void main(List<String> args) {
  UsesSetting(
    setting: <num>(t) => t.val == 50 ? 0 : null, // is fine
  );
  UsesSetting(
    setting: <num>(t) => t.val < 50 ? 0 : null, // throws analysis error:
                                                // The operator '<' isn't defined for the type 'Object'.
                                                // Try defining the operator '<'. dart(undefined_operator)
  );
}

Which throws the error (when run - or just highlighting from the analyzer):
main.dart:22:37: Error: The operator '<' isn't defined for the class 'Object?'.
 - 'Object' is from 'dart:core'.
Try correcting the operator to an existing operator, or defining a '<' operator.
    wrappedValue: <num>(t) => t.val < 50 ? 0 : null,  // throws static analysis error:

At first, I thought this may have been because Dart is unsound and the value is within an object, but then slightly modified to just use the generic:
typedef SettingOrFunc = int? Function<T>(T val);

class UsesSetting {
  final SettingOrFunc? setting;
  UsesSetting({this.setting});

  /// Do stuff with setting here,
  /// for my project, if wrappedValue is/returns null,
  /// default values are used otherwise uses the wrappedValue's return.
}

void main(List<String> args) {
  UsesSetting(
    setting: <num>(t) => t == 50 ? 0 : null, // is fine
  );
  UsesSetting(
    setting: <num>(t) => t < 50 ? 0 : null, // throws analysis error:
                                            // The operator '<' isn't defined for the type 'Object'.
                                            // Try defining the operator '<'. dart(undefined_operator)
  );
}

which throws (basically the same):
main.dart:22:32: Error: The operator '<' isn't defined for the class 'Object?'.
 - 'Object' is from 'dart:core'.
Try correcting the operator to an existing operator, or defining a '<' operator.
    setting: <num>(t) => t.val < 50 ? 0 : null, // throws analysis error:

My belief is this shouldn't be throwing an error. If I highlight over the value (in the first example) or use code-completion inside the function body the analysis tools know the proper type. When not using the typedef it also has the same error.

Interestingly just found that this throws an error:
  UsesSetting(
    wrappedValue: <num>(t) => (t.val as num) < 50 ? 0 : null,
  );

But this is fine:
  UsesSetting(
    wrappedValue: <double>(t) => (t.val as num) < 50 ? 0 : null,
  );

And upon further testing, confirmed that the error only occurs when <T> is exactly the same type as as ___.

Note that <num>(t) => (t.val as num) < 50 ? 0 : null introduces a type variable whose name is num (which is extremely unlikely to be helpful), so you probably meant to write something else (perhaps num was intended to be the name of an existing type, rather than the name that declares a new type variable).
When that function literal is passed to the UsesSetting constructor, it is given a context type of SettingOrFunc? which means int? Function<T>(ContainsValue<T>)?, which is the reason why the actual argument must be a generic function.
Perhaps you meant to write something like this?:
typedef SettingOrFunc<T> = int? Function(ContainsValue<T> tag);

class ContainsValue<T> {
  final T val;
  ContainsValue(this.val);
}

class UsesSetting<T> {
  final SettingOrFunc<T>? setting;
  UsesSetting({this.setting});

  /// Do stuff with wrappedValue here,
  /// for my project, if wrappedValue is/returns null,
  /// default values are used otherwise uses the wrappedValue's return.
}

void main(List<String> args) {
  UsesSetting<num>(
    setting: (t) => t.val == 50 ? 0 : null, // is fine
  );
  UsesSetting<num>(
    setting: (t) => t.val < 50 ? 0 : null,
  );
}

Hmm, when I try the examples above, I see a slightly different behavior than what was reported:

The errors do not come from the analyzer, they come from the front end.  The analyzer reports no errors at all (!)
In every case I've tried, the error message clearly mentions the top type Object?, not Object.  So I don't think the error message needs to be tweaked.  @idraper do you have an example where the error message mentions Object rather than Object??  I see your issue mentions Object, but since I only see Object?, I'm assuming that was a typo.

The fact that the front end and the analyzer disagree about whether there is an error is definitely bad.  I've opened a separate issue for it: https://github.com/dart-lang/sdk/issues/48955

Funny, here's a screenshot from DartPad where the analyzer reports an error based on 'Object'

.

Yes, I also expected for t.val to be resolved to the bound of <num>, i.e. Object?.
This change will switch everything to TypeSystem.resolveToBound(), which is also fixed there.
https://dart-review.googlesource.com/c/sdk/+/243702
I will need to run this through google3 before landing.
And so, we will report:
error: The operator '<' can't be unconditionally invoked because the receiver can be 'null'. (unchecked_use_of_nullable_value at [my_test] bin/test3.dart:22)

introduces a type variable whose name is num

Ah - I definitely missed that. This helps me at least understand why the error is being thrown. I thought it was assigning num to the value of T - definitely an oversight.

Perhaps you meant to write something like this?

That was my intended behavior, however, I was specifically trying to avoid having the class generic but as I'm experimenting I'm not finding a way to do it :/. Might just bite the bullet and redesign some other stuff or maybe try something slightly different. Regardless - not related to this issue.

However, it is somewhat surprising that the error message seems to imply that the type of t.val is Object rather than an actual top type (like Object? or dynamic).

Personally, I agree - in hindsight, that's what led to my confusion in the first place.

do you have an example where the error message mentions Object rather than Object?? I see your issue mentions Object, but since I only see Object?, I'm assuming that was a typo.

Checked and confirmed not a typo, same as @eernstg , I only see Object in the message NOT Object?. I also do see the error in the anaylzer (not just visual).