SetLogger should delete previously set custom logger.

As specified in the doc comment for SetLogger, "the logger becomes the property of the logging module and should not be deleted by the caller".
Not only should the LogDestination delete a custom logger in its destructor, but it should also delete a previous logger when another logger is passed to SetLogger().

Codecov Report

Merging #853 (c0fc870) into master (278ed96) will increase coverage by 3.06%.
The diff coverage is 90.00%.

@@            Coverage Diff             @@
##           master     #853      +/-   ##
==========================================
+ Coverage   73.17%   76.24%   +3.06%
==========================================
  Files          17       14       -3
  Lines        3277     2917     -360
==========================================
- Hits         2398     2224     -174
+ Misses        879      693     -186

Impacted Files
Coverage Δ

src/logging.cc
75.52% <90.00%> (+0.85%)
:arrow_up:

src/utilities.cc
65.57% <0.00%> (-2.31%)
:arrow_down:

src/base/mutex.h
100.00% <0.00%> (ø)

src/windows/dirent.h

src/windows/port.h

src/windows/port.cc

src/demangle.cc
97.59% <0.00%> (+0.11%)
:arrow_up:

src/signalhandler.cc
6.75% <0.00%> (+0.30%)
:arrow_up:

src/vlog_is_on.cc
70.31% <0.00%> (+0.31%)
:arrow_up:

src/glog/logging.h.in
80.00% <0.00%> (+0.54%)
:arrow_up:

... and 2 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Underscore in config for cmakePath seems to immediately terminate the string.

Brief Issue Summary
On a linux system, I need to build for Windows. I am using the MinGW toolchain, and that requires me to use a CMake executable built specifically for it. The CMake executable happens to have the wonderful path "/usr/bin/x86_64-w64-mingw32-cmake". The configure UI seems to verify the path fine, however upon trying to configure I get the error " Bad CMake executable: "x86_" "
Of course, I have placed a symlink in the .vscode folder in my project dir, and just linked to there, but this is still a pretty frustrating issue.
CMake Tools Diagnostics
N/A, CMake doesn't run.

Debug Log
N/A, CMake doesn't run.

Additional Information
No response

Update: I only just now realized that the MinGW executable is a wrapper script that just runs regular cmake with a bunch of new defines, but unless there's something else I'm very obviously missing, the problem still stands. It seems like it's unable to parse underscores.

@Gavator234 Thanks for notifying us of this issue. We will add it to the backlog and investigate it at our soonest bandwidth.


Improve diff usage?

ZAProxy is using a cleaner diff command than we are.  Investigate and possibly improve.
      - name: Compare the expected and actual dist/ directories
        run: |
          if [ "$(git diff --ignore-space-at-eol dist/ | wc -l)" -gt "0" ]; then
            echo "Detected uncommitted changes after build.  See status below:"
            git diff
            exit 1

https://github.com/zaproxy/action-full-scan/blob/765e58613da1d2acd26babd97b680b4b69bf8669/.github/workflows/check-dist.yml#L37

Diff usage long since revisited and improved!


ntr cia launcher doesn't work after leave friend list

as I mentioned in the title, also open ntr cfw early and go on your friends list and then leave the ntr cfw simply no longer displays the menu and try to open again gives the same error.
error details:
open process failed: d9001818
current firmware not supported.
kversion: 02320100
bootntr failed
it works normal if I did not go on the friends list, but if I open the friends list ntr launcher only works if I leave the emunand and open emunand again. tested with rxtools 3.02be emunand 9.9.0-26u, sysnand 4.2.0-9u :)

It seems home menu was restarted for ensuring enough memory to run friend list, at this moment this issue may not be solved.


Investigate which db.Migrator methods need to be implemented

Background
It is impossible to tell at the moment which methods in the interface that db.Migrator tries to implement actually need to contain an implementation. It seems some of them are also not implemented in esvector. Others are stubbed out in esvector (just returning nil, but no error).
Goals

[ ] find out if any of the currently not-implemented methods are actually required
[ ] if yes, create issues for the missing methods
[ ] if no, delete the missing methods from the interface and cleanup in the associated usecase.

cc @laura-ham This is related to what we talked about yesterday: You mentioned schema-related API methods that possibly still exist and can't be used. This issue has the same underlying problem - only that its affect are the respective methods on the standalone and es packages.
Either way we should clean this up as part of either the v0.23.0 or the v1.0.0 release.


Fix log numbber incorrect on pulsar metadata

Modifications
Fix log numbber incorrect on pulsar metadata
Documentation
Check the box below and label this PR (if you have committer privilege).
Need to update docs?

[x] no-need-doc
Simple changes, no need doc

/pulsarbot run-failure-checks


Git branch dropdown clips branch names

Lapce Version
Version 0.2.6
System information
MacOS Monterey 12.3
Describe the bug
The git branch dropdown is hiding the branch name. See the attached screenshot below.
Additional information
Screenshot

What is hidden here?

What is hidden here?

Some branches are not showing full name. Is this dropdown a fixed width?

Yes, that's expected, as we don't have many dynamic UI components

Fixed since we outsourced branch selection to palette


Transaction issue

I have sent tongue coin from my kucoin account to my wallet the transaction is confirmed on the ton scan but it's not deposited to the wallet I tried to contact the support team but nobody answers I try to refresh it many times it's not working the money is still not in my wallet can anybody help
Transaction number below
MO4eHC39A8XS/8dvZJ4qJtUMAOWnUF7T34Z+71pmju8=

103443375

It is done

Meni tonlarimni qaytarin

I have sent tongue coin from my kucoin account to my wallet the transaction is confirmed on the ton scan but it's not deposited to the wallet I tried to contact the support team but nobody answers I try to refresh it many times it's not working the money is still not in my wallet can anybody help
Transaction number below
EQD5mxRgCuRNLxKxeOjG6r14iSroLF5FtomPnet-sgP5xNJb
103443375

https://github.com/toncenter/tonweb/issues/154#issuecomment-2146687535

You have sent Xempire coins to a Binance wallet
UQD5mxRgCuRNLxKxeOjG6r14iSroLF5FtomPnet-sgP5xI-e
من محفظتي TON
UQBF4GwGJ6tvTQqmQqooKjeEY942iK5sFLVxXULMnlPFvjsO
ولم يتم وصول العملات

Sent on 24 Oct, 23:54
Transaction Number bd923086

@aborakan777 Thanks for posting the issue.
Your query would be best dealt with by the support team.
For the fastest resolution please see the link below to our dedicated support line:
Support : Tonkeeper Help Center
Note: Click on the live chat icon at the bottom corner of the page to start a conversation


implementation for /api/search_subreddits

Hey!
Maybe I missed it but I couldn't find an implementation for /api/search_subreddits Is that exist?
Thanks!
UPDATE
I cloned and added this method myself so let me know if you guys want PR.

There isn't a function for it at the moment. Definitely add a PR if you already fixed it :)
For the future, we do have snoowrap.oauthRequest() for situations where there is no implementation yet or you need more flexibility. So no need to fork it and go that route unless you're creating a PR


[thread-cert] fix case 5.3.6 issue #5909

delete the second trickle time check to fix issue #5909  since there may be another advertisement with router costs set {1,0,1} which will cause the sniff time difference value larger than 3 seconds


Fix integration test README codeblock.

The codeblock cargo test -p minimint <test-name> should be cargo test -p minimint-tests <test-name>.

Codecov Report

Merging #433 (48ebbe7) into master (3606a20) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #433   +/-   ##
=======================================
  Coverage   74.52%   74.52%
=======================================
  Files          83       83
  Lines       10515    10515
=======================================
  Hits         7836     7836
  Misses       2679     2679

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Add deploy step to pull request pipile

In case when renovate suggests new dependecy from build files, such as gradle or gradle wrapper version, the pull request pipeline can be positive but it does not granty the successfully deploy to maven centrall.
Soulition:
Add deploy to maven centall from pull request pipeline. Version template {version}-short_sha, short_sha of build commit.

Implemented


Add searchform partial and function to replace default WordPress functionality

Inspired by this thread:
https://discourse.roots.io/t/custom-search-form/9079
The WordPress searchform, and the ability to customize it, is basic WP functionality that Sage unintentionally breaks. This PR predictably replaces the WP default template with a Blade partial, much like the the comments partials we already ship, letting users use get_search_form() as normal.

I'm not super attached to it, I just cut it down because I felt like search_form_ was going to lead to some very long variables names.

Could just use search ?

Would it make more sense to put the $sf_* stuff in the app controller? Out of the box, Sage seems slightly more oriented towards using that model. Though I think the docs introduce filters as the easy way, the code itself emphasizes the controller more, and I think most discussion on Discourse focuses on that too.
No strong feelings, just raising the question. Other than that, LGTM. Thanks all!

My reason for the filter approach is that it's the "native" approach—Controller is a 3rd party package. IMO if we offer a "native" way to do a thing then our most basic implementation should use that implementation, not depend on a 3rd party package.

That makes sense to me. TBH I thought it was odd that Sage uses controller out of the box and not the data filters (to the extent it uses either) for the same reason. Could just be because App::title() wouldn't fit in a data filter.

Need to fix indenting to pass phpcs. Line 87 has 1 space, should be 0; the rest should use 4-space indents rather than 2.

So, yay?


When calling PCCurrentUser.leaveRoom(id:completionHandler:) a crash occurs

What?
When PCCurrentUser.leaveRoom(id:completionHandler:) is called it causes a recursive crash (see attached screenshot)

Environment

PusherChatkit version: 0.7.0
Xcode version: 9.3
Swift version: 4.1
Platform(s) running PusherChatkit: iOS + Simulator

CC @hamchapman

This seems to be caused by two methods having the same signature which is causing the recursion and ultimately the crash.
The private method should likely be renamed.

Ah, good catch. I think it's even simpler than that: https://github.com/pusher/chatkit-swift/blob/9c249cc883c7bebe512454c31b4e33dc33761f89/Sources/PCCurrentUser.swift#L407
That function is directly calling itself. The first parameter should be roomId, not just id.
I'll push out a new version with that fix in it today.

Just released version 0.7.1 that fixes this, and includes the fix that was in #71
Thanks for all the reports and PR @steve228uk !


NFT "Add to Offer" option

Expected UX coming from reference GUI Wallet.
Current UX: User selects Offers -> Create Offer -> +NFT -> Realizes they need launcher id -> ... -> Offer is created. Quite tedious.
Additionally, an NFT lookup option/dropdown in create offer dialogue would be nice

Good idea, this is a related issue which will make the Offering assets a dropdown instead of a manual entry:
https://github.com/xch-dev/sage/issues/118

Multiselect makes this feature really useful


Yatin: Watch wanelo.co

Yatin requests the watch of the watch_keyword wanelo\.co. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

wanelo\.co has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Rejected by Spevacus in Charcoal HQ. Reason: 'Appears to be collateral. Spam link was in the user's profile.'


f.lux not working on ubuntu 14.04

I'm trying to use f.lux with the GUI, but just doesn't work. Followed these steps: http://ubuntuhandbook.org/index.php/2016/03/install-f-lux-in-ubuntu-16-04/
Whenever I click on a preview color temperature, the GUI app freezes for a couple of seconds. Nothing else happens. What command should I run to attach here the debug?

Downgrading to nvidia-340 worked for me. I believe it's a graphic card's driver issue.

Had a same situation on 16.04 LTS

same with 16.10 LTS

You could try running xflux manually and see if you get an error:
xflux -l 0 -g 0 -nofork

You'll want to stop any existing xflux processes before doing this.
It's likely you're running into an upstream issue (https://forum.justgetflux.com/topic/2373/sorry-we-only-support-8-and-10-bit-displays-right-now) that is often reported here (https://github.com/xflux-gui/xflux-gui/issues/27).

Yes, it shows the displays error:
--------
Welcome to xflux (f.lux for X)
This will only work if you're running X on console.

Found 1 screen.
Sorry, we only support 8 and 10-bit displays right now.
david@Phenom-Ubuntu:~$

ATM i'm alternatively using redshift, although the GUI is not working, so I have to run it manually.

@EnforcerZhukov @ntc2 same with me - "Sorry, we only support 8 and 10-bit displays right now.". Redshift 1.10 works, even the gui (redshift-gtk) version. Made a switch,

Duplicate of #27. We should probably change fluxgui to detect this problem and refer the user to Issue #27.


AMQPError is private

Thank you for your work on AMQP bindings for Rust! I'm looking forward to using these in a project soon.
I noticed (while trying to set up error-chain) that the AMQPError and AMQPResult type are never exported from the amqp crate. This makes it difficult to store them inside other error types and propagate them up the call chain.
This could be fixed by adding pub use amqp_error::AMQPError; to your lib.rs file. I'd be happy to prepare a PR if you're interested.

This is actually fixed in debd11195a74aa576a58aabfb1577de8ddd04a03, but it hasn't been released to crates.io yet.

It is released in v0.0.20.

Thank you!


Support for Safari

Hey @jswanner. Really appreciate the work you've done here! Just wanted to know if there are any plans to create/support a version of this extension for Safari?

Hey @dhruvit-r, thanks for the kind words. Great question regarding Safari; I've been using Safari more lately and so have run into occasions when I wanted to use this extension there as well. I haven't put any effort into that yet, but it is something I've thought about.
The first hurdle I would need to get over is installing Xcode, which for whatever reason is always something I avoid for as long as I can.

@jswanner Totally understandable. I don't have enough bandwidth at this point, but I can work on and support a fork that is specifically for Safari at some point of time in the future. Would that be something you're comfortable with?

@dhruvit-r yeah, if you are willing to take that on, I'd be comfortable with that.

@dhruvit-r any progress yet? I'm also looking forward to use DFWP on Safari.

@cmaring I created a very "crude" version that I personally use, but procrastinated on putting in the "finishing touches" and  publishing/pushing it :)
I've attached the version I use. It's probably outdated and it's not super polished, but it gets the job done (I really only have a couple websites that bug me, and this takes care of those).

Pushed source code here: https://github.com/dhruvit-r/DontFuckWithPaste.

Thanks. Just got it pulled and  installed and it does get the job done.


Add Circle and Sphere shapes

Added these to my fork and wanted the pass them along

Nice!  Thank you.


Forced Chat Command

Have you searched for similar requests? 🕵♀

[X] I've searched open and closed feature requests.

Describe the Feature in Detail 📝
Kohl's Admin Infinite has a chat command used to fake chat messages from a user.
This would be a fun command for trolling, but since it can be heavily abused it should be restricted to the creator role.
Current Situation/Workaround 🔄
No response
Expected Benefits 🚀
No response

GOOD, i'd love for this to be a thing


App crashes on Proceed button of Send screen:

https://github.com/bithyve/bitcoin-keeper/assets/62699947/9d0bc9eb-c5c8-4832-b52a-5ef6c230639d

App crashes when we click on proceed button without selecting any wallet/vault or entering any address.

Verified this issue on keeper app v 1.2.0(319)


no password necessary to move service

Probably since we changed auth to Fastly root scope tokens, no password is necessary to change service ownership.
The VCL tool still asks for password to perform certain operations, like create on service on a customer ID or change the service ownership.

Fixed in https://github.com/stephenbasile/vcl_cli/pull/14


Allow configuring alias language for contributed snippets

Motivation: https://github.com/vuejs/vetur/issues/718
vue-html is a language similar to html but with a few additions. However, Vetur users would like to use html snippets contributed by extensions inside vue-html.
For users' own snippets, they can achieve this by copy-pasting their snippets between html and vue-html. However, is there anyway to make contributed html snippets available to vue-html too?

Use xyz.code-snippets-files when adding/contributing snippets. Those allow for a scope attribute which allows to target more than one language per snippet

This is not resolved yet -- for user-defined snippets they could add scope, however for extension contributed extension there is no easy way to do that. I suggest something like emmet.syntaxProfile that allow users to mark snippets for certain language to be available for additional languages, such as
{
  "snippest.syntaxProfile": {
    "html": ["vue-html", "handlebars", "razor", "jinja"]
  }
}

Has this been implemented yet? This would dramatically increase vscode's usefullness considering the amount of file extensions that allow for mixed languages and languages that themselves are simply preprocessors or templating engines for some other language. Extensions for such languages rarely implement any features for the base language.

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!


Whois deprecation - migrate to RDAP

The whois "protocol" is being deprecated in favour of RDAP. Would there be interest in looking to adding RDAP functionality to this repo?
The migration seems like it will take a few years since it's based on registrar contract renewals. And a number of registrars have said they'll end whois services after a period of operating both - but again this will be on a rolling schedule. So the solution seems to be to try both to answer queries - specifically try RDAP and if it fails, try whois.
Alternatively there could be an rdap specific plugin. The issue is the overlap period where for some registrars will only support whois, some will support both and some will only support rdap. There's an argument that the name is confusing, but I suspect the term "whois" will refer to the data regardless of the protocol.
There's an existing Go RDAP library that should make it relatively easy to do.

@lyda Thanks for raising this issue and PR!
It feels like RDAP is separate enough to be its own plugin, based on naming for searchability, under, etc. If a domain doesn't support RDAP yet, the WHOIS plugin will still be available for use.
@johnsmyth - Any thoughts on a separate plugin vs. a separate table in this repo, e.g., rdap_domain?

Im not very familiar with the details - how compatible are the responses?  Depending on how much they vary, it might not really be useful to capture them in a single table anyway?

Oh, and if it helps, these are the corresponding Go types:

rdap.Domain: https://pkg.go.dev/github.com/openrdap/rdap#Domain
whoisparser.WhoisInfo: https://pkg.go.dev/github.com/likexian/whois-parser#WhoisInfo

As an aside, I suspect people will pronounce RDAP "whois" for a long time. Just like the save icon is still a 3.5" floppy disk and we still talk about "albums" with relation to music. In terms of language, I think the name of this plugin long after nothing listens to port 43.

@lyda @cbruno10 IMO, a new plugin would be a better solution considering we have all means to access the APIs. In future after the deprecation comes through, it would become difficult to rename/deprecate this plugin and create a new one for RDAP.

I think I agree with @lyda that folks will continue to refer to it as whois even when it is technically RDAP, so i dont mind adding support for RDAP in this plugin.  We do something similar for RSS - the RSS plugin supports both RSS and Atom.

@lyda @johnsmyth That's a good point, in the future if we ever want to have an RDAP plugin, we could create it then and deprecate the table in this plugin (if required).
@lyda Please let us know when https://github.com/turbot/steampipe-plugin-whois/pull/36 is ready for review (or if it already is) and we'll have a pass at it, thanks!

Little busy this week but will get to it again Saturday morning Irish time. I think the PR is mostly there but would like to review it and add comments on my decisions.
Also need to see if there are columns we could get from rdap that are missing from that WhoisInfo (sp?) type. At the very least I can document them.

Thanks @lyda , no rush of course! Looking forward to adding RDAP support soon 😃

OK, looked through it and the PR is ready. I have some thoughts. Might want to add a few columns - maybe an rdap_server and some of the rdap fields.


fix: Add missing volume mount to transformer container when using modelcars

Adding the missing mount of the emptyDir volume that holds the symbolic link to the model data in the model sidecar to a transformer container, if it is configured.

What this PR does / why we need it:
This PR fixes a bug when using transformer with modelcars in the serving runtime
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
No issue yet (since the fix is obvious), happy to open one if needed.
Type of changes
Please delete options that are not relevant.

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

Feature/Issue validation/testing:
See the included unit test, that also tests that the proper volume mount is added
when no transformer is used
Checklist:

[X] Have you added unit/e2e tests that prove your fix is effective or that this feature works?
[X] Has code been commented, particularly in hard-to-understand areas?
[ ] Have you made corresponding changes to the documentation?

Release note:

NONE

/retest

Not sure why this test is failing, this PR doesn't touch anything in this area.
@terrytangyuan @yuzisun would it be possible to merge this fix before the next release ? It actually fixes a severe issue when running transformers with modelcar.

yes we should
/assign @yuzisun

/ok-to-test
/retest

/lgtm
/approve


use original app bundle identifier

Reviewer: Mia
Asana: https://app.asana.com/0/0/409779663324551/f
CC:
Description:
Update the app's bundle id so that it can replace the old app.
Steps to test this PR:

Install the original app
Favourite a story
Perform a search and favourite it
Install this version of the app - it should replace the original one
Confirm favourites are now bookmarks
Confirm Share from Safari to Duck Duck Go works

Reviewer Checklist:

[ ] Ensure the PR solves the problem
[ ] Review every line of code
[ ] Ensure the PR does no harm by testing the changes thoroughly
[ ] Get help if you're uncomfortable with any of the above!
[ ] Determine if there are any quick wins that improve the implementation

PR DRI Checklist:

[ ] Get advice or leverage existing code
[ ] Agree on technical approach with reviewer (if the changes are nuanced)
[ ] Ensure that there is a testing strategy (and documented non-automated tests)
[ ] Ensure there is a documented monitoring strategy (if necessary)
[ ] Consider systems implications (Database connections, Grafana stats, CPU)

Looks good!


[Workflows] Add reviewdog workflow and PR template

Add a PR template
Add a reviewdog template in CI

@okdas Changed the goal of this PR so please take a look again when you can.

@okdas Friendly bump on this please

I'm bypassing E2E tests introduced in #103 because these are just github workflow.


[ffi] Add support for abi-specific structs

Sometimes structs are defined differently on different platforms. (Some fields might be commented out on some platforms.)
A workaround is to have a different struct for the different platforms, but that cascades into everything that is using that struct (function signatures, struct fields, and pointers).
We could introduce an AbiSpecificStruct that exposes all the different fields on all platforms in one api.
@AbiSpecificStructMapping({
  Abi.linuxX64: MyStructLinux(),
  Abi.windowsX64: MyStructWindows(),
  // ...
})
class MyStruct extends AbiSpecificStruct {
  external int a;
  external int b;
}

This would be the struct/union counterpart of https://github.com/dart-lang/sdk/issues/42563.
edit: We would probably make an API similar to https://api.dart.dev/stable/2.17.0/dart-ffi/AbiSpecificInteger-class.html with annotations instead of types.

@dcharkes That's pretty neat. What happens if accessing field external int b on Windows, in this example? A compiler error, I trust?

Hey guys, I'm looking to use this package https://pub.dev/packages/steamworks on both Windows and MacOS. This issue is blocking the package to work properly.
I've been making a breakthrough RPG with Flutter for the past 2 years and about to start the last stretch of production to ship a demo and then release. This package will be used to send Steam Achievement to Steam API.
Is there any way I can help on this issue? Can you point me to the right resource so I can try to work on it?

Hi @LeoSandbox nice to meet you!
We welcome contributions! ❤
The first step is to build a Dart SDK locally:

https://github.com/dart-lang/sdk/blob/main/docs/Building.md
https://github.com/dart-lang/sdk/blob/main/CONTRIBUTING.md

The best starting point would be to look at the commits that introduced ABI-specific integers:

https://dart-review.googlesource.com/c/sdk/+/221501
https://dart-review.googlesource.com/c/sdk/+/221821

The analyzer is a different implementation than the common compiler frontend (CFE).
To run the analyzer and CFE from source to be able to run the debugger in vscode:
			{
				"name": "dart analyzer.dart",
				"type": "dart",
				"request": "launch",
				"program": "pkg/analyzer_cli/bin/analyzer.dart",
				"args": [
					"tests/ffi/static_checks/vmspecific_static_checks_array_test.dart",
				],
				"toolArgs": [],
				"enableAsserts": true,
				"cwd": "${workspaceFolder}",
			},
			{
				"name": "dart gen_kernel.dart",
				"type": "dart",
				"request": "launch",
				"program": "pkg/vm/bin/gen_kernel.dart",
				"args": [
					"--enable-experiment=records",
					"--platform=${workspaceFolder}/xcodebuild/ReleaseARM64/vm_platform_strong.dill",
					"pkg/vm/testcases/transformations/ffi/address_of_struct_element.dart",
				],
				"toolArgs": [],
				"enableAsserts": true,
				"cwd": "${workspaceFolder}",
			},

You might need to adjust the build folder to out/ instead of xcodebuild/ if you're not on MacOS.
The use of the API could look something like the following:
@AbiSpecificStructMapping({
  Abi.linuxX64: MyStructLinux(),
  Abi.windowsX64: MyStructWindows(),
  // ...
})
final class MyStruct extends AbiSpecificStruct {
  // No annotation, the native types are per ABI
  external int a;

  // The ABI-specific struct should have the superset of all fields of the struct per ABI.
  external int b;

  // The ABI-specific struct should have the superset of all fields of the struct per ABI.
  external int c;
}

final class MyStructLinux extends Struct {
  @Int8()
  external int a;

  @Int8()
  external int b;
}

final class MyStructWindows extends Struct {
  @Int64()
  external int a;

  @Int64()
  external int c;
}

I expect a PR to implement this feature to roughly have the following changes:

[ ] A new class AbiSpecificStruct (and AbiSpecificUnion) in sdk/lib/ffi/abi_specific.dart

[ ] With a long doc-comment including examples

[ ] An implementation for static errors

[ ] The ABI-specific struct must have the superset of all fields of the structs mentioned in the annotation
[ ] The ABI-specific struct must have a Dart type compatible with the fields of the structs mentioned in the annotation (e.g. Windows cannot have @Double() external double a; while Linux has @Int8() external int a;).
[ ] The annotation must only mention other structs if it's a struct.
[ ] The error messages are defined in pkg/analyzer/messages.yaml and pkg/front_end/messages.yaml. You'll need to run the code generators mentioned in those files to get the Dart methods to use in the analyzer and CFE to report the errors.

[ ] Tests for static errors (both for CFE and analyzer) in tests/ffi/static_checks/vmspecific_static_checks_abi_specific_struct_test.dart
[ ] An implementation in the CFE in pkg/vm/lib/transformations/ffi/*.dart. I believe AbiSpecificNativeTypeCfe can be mostly reused to deal with structs/unions besides only integers. Basically the idea is that the external Foo x is transformed to a getter and setter with the real body that has a big switch based on ABI in it.
[ ] Unit test cases for the CFE similar to pkg/vm/testcases/transformations/ffi/abi_specific_int.dart. They show the transformed code before it is send to the backend.
[ ] Integration test cases that exercise the code in tests/ffi/*_test.dart

[ ] The best way to do this is probably to extend the test generator in tests/ffi/function_callbacks_structs_by_value_generated_test.dart to be able to generate tests for AbiSpecificStructs. Then you don't have to write tests by hand.

[ ] The most tricky part will most likely be the backend implementation for passing AbiSpecificStructs by value in FFI calls and callbacks (Pointer.fromFunction and NativeCallables). The implementation there would be in runtime/vm/compiler/frontend/* See the linked PR for what was changed for AbiSpecificInteger.

You'll have to touch many parts of the Dart compiler for this feature (similar to https://dart-review.googlesource.com/c/sdk/+/221501). However, the feature should be somewhat straightforward to implement due to its similarity to the AbiSpecificIntegers. Please feel free to ping me or this issue with any questions!


Editor: Add "Share via Gist/Pastebin/hastebin..." feature

Sharing code is a good thing...
This feature would allow users to upload the code online, and share it faster.¹

GitHub Gist API
About. Permanent raw link: https://gist.github.com/{gist_user}/{gist_id}/raw/{file_name}

Pastebin API
About. See also Scraping API.

Hastebin API
About. Raw link: http://hastebin.com/raw/{key}

¹ I would recommend to prompt the user to confirm their action, just in case.

We can't implement any such feature that requires the user to log in, because there is no way to hide the password from any server admin running malicious lua code.

can't implement any such feature that requires the user to log in

What the **** are you talking about?
GitHub Gist, Pastebin and hastebin... all allow public/anonymous uploads - no login/account is required.
What password?! Well there is API key.

I only meant that I am pretty sure that hastebin allows anonymous pastes, and that I wasn't sure about the others.

I'm pretty sure both pastebin or github just require api keys to which on github I know you can restrict, not sure about pastebin.

How about OAuth2?

Ok and then how do you get the oauth key out of the browser and into the lua code so that we can upload the code using it?

Moreover, there are "private" options for Gist and Pastebin.
GitHub calls it secret Gist. Pastebin calls it unlisted paste.
I am not sure about hastebin.
You simply never share the link if you want it "private". See, no user accounts required, etc.

The manual parts are required to keep a level of discretion/control by the end user. ShareX does something similar, where you have to copy-paste the OAuth key to the appropriate textbox for whatever integration.

A simple anonymous "Upload to Pastebin" plus a couple of other well-established services is fine

Yes. This. No need for user accounts/logins, passwords, etc... I have never mentioned to do the whole thing with user accounts, and so on.
Ok, let's just do public and private/anonymous (secret/unlisted) sharing, that does not require user accounts/passwords.. It is a simple HTTP (POST) request.
So... Focus on it. API key.
If wire comes with builtin API key, chances are, Pastebin will block it due too many requests or something... Well, I think it takes just five players to keep spamming (POSTing) HTTP requests repeatedly (with same API key) for it to be blocked by Pastebin after a couple of minutes. I am not sure about Gist; never tried it..
There could also be a daily/weekly/monthly limit or something.
It is all on the client-side, nothing is networked to the server-side. So, the question is, how do you save API key on the client, without server being able to see it? Because, it would be very annoying to have to enter API key every single time. I have an idea, here it is:
When the user clicks Upload to Gist/Pastebin/hastebin button, it would check if there is a file key.txt, if this is the first time the user clicked it - it will return false. So, the user is prompted to enter API key, and their secret phrase (API key would be encrypted with user's secret phrase). Next time the user clicks Upload to Gist/Pastebin/hastebin button, key.txt file will exist, so it would return true, and this time it would only ask a user to enter their secret phrase.
This way, only "infected"/bad servers could reach to client's file system, and even if they do, it will be encrypted, so if they doesn't know the secret phrase, they could try brute-force technique, but nothing can stop "bad" hackers....
So, what do you think of that approach?

If it requires an API key inputted manually then forget it.

Gmod does have javascript integration, so I guess that could work, but that can be intercepted by admin abusers.

Correction: Anything can be intercepted only if wire has been infected, such as a client sending such information to the server and then to admin abuser...
Let's not worry about that, because that's again the story about players' trust the server they are playing on...
So, without API key. Okay. @oldmud0 has already explained it a bit. Now I will try to round it.
When the user clicks Upload to Gist/Pastebin/hastebin button, it would copy the code to clipboard, open Steam web browser in overlay, and automatically navigate to the wanted service (see gui.OpenURL function). But, there is a problem with that, Steam overlay can be enabled/disabled by the user via Steam settings. So, instead of using Steam overlay, create a new window/frame with DHTML on it, and use JavaScript integration to automate it as much as possible. This way, a user can be presented with website interface, allowing them to set any options offered by services such as Gist/Pastebin/hastebin. Minimizing the number of clicks the user has to do, right now I think it can be minimized down to only 2 clicks.
What are your thoughts about this approach?

If the text editor had a useful integration point for third-party addons (eg. calling a hook and passing in the menu bar as an action), this could easily be achieved by a third-party addon. It would feel like serious feature creep for Wiremod to be responsible for dealing with API keys or manipulating third-party websites via Javascript injection.

If the text editor had a useful integration point for third-party addons

It sounds like Editor was not designed and constructed with extensibility in mind.
So, new issue?

It would feel like serious feature creep for Wiremod to be responsible...

Yeah, I agree..


Users should be able to "Like" other users answers

After reading somebody's else answer you might want to just give a "thumbs up" to it as you also think it is the right one, it may be helpful for the author to accept or trust it is the right answer to try.

Definitely something to consider. Thanks for suggesting!

Anybody working on this? I would like to get this one!

@BobbyBorisov nope, feel free.
Let's nail down the terminology first. I think "Like" is a good word for what we're trying to achieve here. Looking around I found the following library which we could use? https://github.com/cybercog/laravel-likeable

Hey @driesvints , I think 'Like' is better than other words like 'Favorite' and etc as well. I got free time today and i decided to implement this feature. Unfortunately i saw the likeable package when i was halfway through the implementation and tests(It is just one trait). I submitted the first version of it and i can do a [WIP] pull-request in order you to see it. What is left is to write comments and to style the button/thumbs up or thumbs down.

Thank you for looking and working into this, "Like" sounds good to me
@bobbyborisov. We really needed this feature.
On Sun, Dec 3, 2017, 12:48 PM BobbyBorisov<EMAIL_ADDRESS>wrote:

Hey @driesvints https://github.com/driesvints , I think 'Like' is
better than other words like 'Favorite' and etc as well. I got free time
today and i decided to implement this feature. Unfortunately i saw the
likeable package when i was halfway through the implementation and
tests(It is just one trait). I submitted the first version of it and i can
do a [WIP] pull-request in order you to see it. What is left is to write
comments and to style the button/thumbs up or thumbs down.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/laravelio/portal/issues/334#issuecomment-348813221,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ASp8H0hnpJ9PzPJ_AHJ9owt4YUsI8qTxks5s8wj4gaJpZM4Qt7C8
.

@BobbyBorisov yeah, just "liking" a post is fine. No need for it to go negative. Submit it and let's see if we continue from your PR or if we're going to use the package :)

This is now merged and deployed.


nixos/pulseaudio: remove broken support for 32bit on 64bit systems

Motivation for this change
In #154276, alsa-lib was updated and old patches removed. One of those patches, from 2014, allowed alsa-lib to recognize a libs field in /etc/asound.conf, and the pulseaudio module leveraged this to optionally add support for running both 32bit and 64bit apps on 64bit platforms.
With the patch removed, alsa-lib is unable to parse the /etc/asound.conf generated by the pulseaudio module (regardless whether the user has opted into 32bit support), and alsa utils such as alsactl, alsamixer, speaker-test, are broken. E.g.
$ alsactl monitor default
alsa-lib control.c:1464:(snd_ctl_open_conf) Unknown field libs
Cannot open ctl default

Once the issue was identified, it was reasoned that the 32bit support is too niche a use-case to justify patching a system as complex as alsa, especially given the availability of pipewire.
I removed the support32Bit option from pulseaudio and changed the generated config to use syntax recognized by alsa-lib. I then removed several references to this option from other modules (steam, jack, pipewire).
Note: I rebuilt my system against this branch and confirmed that alsa works as expected for my use-cases. I have not tested steam, jack, or pipewire.
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@L-as You mentioned,

The only scenario I can think of where this is still relevant is Steam with proprietary old 32-bit games that use ALSA directly, but you can just switch to PipeWire for that! Perhaps that's just something we should note in the release notes for the next NixOS release.

I'm not sure what action the notes should suggest to affected users. Could you advise?

I also have this issue in current master (not pulseaudio but pipewire/alsa.enable = true)
with this patch, I check speaker-test alsamixer and some steam games(Hollow Knight and Oxygen Not included) and works fine

... Now I'm confused about what hardware.pulseaudio.support32Bit did? I just rebased #153916 on master and commented out support32Bit but the 32 bit portion of the test still works...

Can someone try this with a 32-bit game?

Tested this branch on my system and it fixed pipewire support for ALSA within Mixxx.

Thanks for working on this!
Small note: this also drops (similarly broken) 32bit alsa support for jack and pipewire (removing the libs entries from those asoundrc's as well), perhaps update the PR title?

Small note: this also drops (similarly broken) 32bit alsa support for jack and pipewire (removing the libs entries from those asoundrc's as well), perhaps update the PR title?

@dtzWill That's a good point, but I'm considering changing the direction of this PR to fix the 32bit support (re-adding the alsa-lib patch that was removed) and deprecate the option rather than removing it right away. (see https://github.com/NixOS/nixpkgs/pull/157352#discussion_r795285011)
I'll update the PR title one way or the other though :+1:

Given that it's not clear what alternative to suggest to users who currently rely on the 32bit support, I think the best thing is to fix what's currently broken, and leave the deprecation/removal of 32bit support to someone knowledgeable enough to compose appropriate deprecation warnings (including suggested config changes for affected users).
So I've created an alternative PR that confines itself to fixing what's currently broken: #157631

Closing this in favor of #157631


[Bug]: connectivity_plus: build application not working when plugin is installed

Platform
Android 8
Plugin
connectivity_plus
Version
3.0.6
Flutter SDK
3.7.12
Steps to reproduce

Create new app with: flutter create
Install connectivity_plus with: flutter pub add connectivity_plus
Build app and get error

Code Sample
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        // This is the theme of your application.
        //
        // Try running your application with "flutter run". You'll see the
        // application has a blue toolbar. Then, without quitting the app, try
        // changing the primarySwatch below to Colors.green and then invoke
        // "hot reload" (press "r" in the console where you ran "flutter run",
        // or simply save your changes to "hot reload" in a Flutter IDE).
        // Notice that the counter didn't reset back to zero; the application
        // is not restarted.
        primarySwatch: Colors.pink,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key, required this.title});

  // This widget is the home page of your application. It is stateful, meaning
  // that it has a State object (defined below) that contains fields that affect
  // how it looks.

  // This class is the configuration for the state. It holds the values (in this
  // case the title) provided by the parent (in this case the App widget) and
  // used by the build method of the State. Fields in a Widget subclass are
  // always marked "final".

  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  int _counter = 0;

  void _incrementCounter() {
    setState(() {
      // This call to setState tells the Flutter framework that something has
      // changed in this State, which causes it to rerun the build method below
      // so that the display can reflect the updated values. If we changed
      // _counter without calling setState(), then the build method would not be
      // called again, and so nothing would appear to happen.
      _counter++;
    });
  }

  @override
  Widget build(BuildContext context) {
    // This method is rerun every time setState is called, for instance as done
    // by the _incrementCounter method above.
    //
    // The Flutter framework has been optimized to make rerunning build methods
    // fast, so that you can just rebuild anything that needs updating rather
    // than having to individually change instances of widgets.
    return Scaffold(
      appBar: AppBar(
        // Here we take the value from the MyHomePage object that was created by
        // the App.build method, and use it to set our appbar title.
        title: Text(widget.title),
      ),
      body: Center(
        // Center is a layout widget. It takes a single child and positions it
        // in the middle of the parent.
        child: Column(
          // Column is also a layout widget. It takes a list of children and
          // arranges them vertically. By default, it sizes itself to fit its
          // children horizontally, and tries to be as tall as its parent.
          //
          // Invoke "debug painting" (press "p" in the console, choose the
          // "Toggle Debug Paint" action from the Flutter Inspector in Android
          // Studio, or the "Toggle Debug Paint" command in Visual Studio Code)
          // to see the wireframe for each widget.
          //
          // Column has various properties to control how it sizes itself and
          // how it positions its children. Here we use mainAxisAlignment to
          // center the children vertically; the main axis here is the vertical
          // axis because Columns are vertical (the cross axis would be
          // horizontal).
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            const Text(
              'You have pushed the button this many times:',
            ),
            Text(
              '$_counter',
              style: Theme.of(context).textTheme.headlineMedium,
            ),
          ],
        ),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: _incrementCounter,
        tooltip: 'Increment',
        child: const Icon(Icons.add),
      ), // This trailing comma makes auto-formatting nicer for build methods.
    );
  }
}

Logs
Launching lib\main.dart on Android SDK built for x86 in debug mode...
main.dart:1

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':connectivity_plus:parseDebugLocalResources'.
> Could not resolve all files for configuration ':connectivity_plus:androidApis'.
   > Failed to transform android.jar to match attributes {artifactType=android-platform-attr, org.gradle.libraryelements=jar, org.gradle.usage=java-runtime}.
      > Execution failed for PlatformAttrTransform: C:\Users\USER\AppData\Local\Android\sdk\platforms\android-31\android.jar.
         > C:\Users\USER\AppData\Local\Android\sdk\platforms\android-31\android.jar

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 12s
Exception: Gradle task assembleDebug failed with exit code 1
Exited (sigterm)

Flutter Doctor
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.7.12, on Microsoft Windows [Version 10.0.19044.2846], locale en-US)
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 34.0.0-rc3)
[√] Chrome - develop for the web
[√] Visual Studio - develop for Windows (Visual Studio Community 2022 17.5.3)
[√] Android Studio (version 2022.2)
[√] VS Code (version 1.78.0)
[√] Connected device (4 available)
[√] HTTP Host Availability

Checklist before submitting a bug

[X] I Google'd a solution and I couldn't find it
[X] I searched on StackOverflow for a solution and I couldn't find it
[X] I read the README.md file of the plugin
[X] I'm using the latest version of the plugin
[X] All dependencies are up to date with flutter pub upgrade
[X] I did a flutter clean
[X] I tried running the example project

You need to check which JDK version you use in the project where you are trying to add the plugin.

update new version 4.0.0

https://github.com/fluttercommunity/plus_plugins/issues/1672

Sorry for the late reply.
I have solved the issue.
I put the details below, it might help someone else:

I'm using Android Studio Flamingo | 2022.2.1 and the embedded SDKs so I think it's 17 or 18.

Solution:

I have uninstall all Android SDK Build Tools (34, 31, 30) and install again and it worked.

Thanks a lot for providing further feedback 👍🏻

I tried all above solutions it's not worked still am getting the below issue
Multiple build operations failed.     Could not resolve all files for configuration ':connectivity_plus:androidApis'.     Could not resolve all files for configuration ':flutter_inappwebview:androidApis'.     Could not resolve all files for configuration ':flutter_secure_storage:androidApis'.     Could not resolve all files for configuration ':fluttertoast:androidApis'. Could not resolve all files for configuration ':connectivity_plus:androidApis'. Failed to transform android.jar to match attributes {artifactType=android-mockable-jar, org.gradle.libraryelements=jar, org.gradle.usage=java-runtime, returnDefaultValues=false}. Execution failed for MockableJarTransform: /Users/*****/Android/sdk/platforms/android-33/android.jar. Cannot create mockable android.jar /*****/Library/Android/sdk/platforms/android-33/android.jar


Add Docstrings to Code base

Pull Request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[X] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[X] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: #3
What is the new behavior?

Docstrings added for each function of the CLI

Does this introduce a breaking change?

[ ] Yes
[X] No

Other information

@all-contributors please add @Gonmeso for code


Speed Increase

Hello, I have made some changes to increase the performance of the calculations:

Refer to HyperEdges by their position/index rather than passing around SmallVecs
Move ".contains()" calls to the setup, and cache results
Minimise recalculation of "equation_lhs()" in the gradient as much as possible.
Calculate the cluster probabilities in two halves, only one of which needs to be recalculated at a time.

This brought the surface_code calculation down to 2 seconds on my computer.

It might be possible to improve the solution accuracy by using logits as the solver parameters to smoothly clamp the probabilities to between 0 and 1. I haven't tried it here, but it is a common way to avoid negative values in the solution if they are not wanted.

Thank you for providing so much guidance and assistance; it has truly helped me a lot.

Hi, No worries! Sorry for responding so slow, work got very busy. There were a couple of points I wanted to handover before I forget:

Nonlinear conjugate gradient requires a line search that nearly runs to completion (c2 = 0.05 rather than 0.9) to be efficient. This has to do with how conjugate gradient method differs from regular gradient decent. I had forgotten this when I first looked at
I was wrong when I initially said that I didn't expect poor conditioning. Given how much LBFGS and BFGS outperform NCG it is clear that having access to a hessian or an approximate hessian is very beneficial for this type of problem.

You clearly work on interesting things, so feel free to let me know if you'd like a hand with similar work in rust. All the best.


xtask: make testing cwd independent

The only non-trivial thing in this PR should be how 'log_path' is specified in the CLI. I propose handling it in the following way: if 'log_path' is relative, it will be made absolute relative to the root project_path and not to where the command is actually called. Absolute paths would not been changed
closes #232

Current dependencies on/for this PR:

main

PR #247   👈

This stack of pull requests is managed by Graphite.

Merged the wrong PR, oops.


catch pylint exception messages and continue the flow execution

Addressing this issue https://github.com/Netflix/metaflow/issues/292

@jasonge27 - @romain-intel's #294 is ready to be merged which includes this fix. Can you please verify that #294 addresses #292 and if so, close this PR?

Sounds good! Just took a look and seems like #294 has the same change regarding the issue. Closing this PR.


Added Shell Sort Algorithm

Related Issue

Shell Sort (#1047)

Shell sort is a generalized version of the insertion sort algorithm. It first sorts elements that are far apart from each other and successively reduces the interval between the elements to be sorted.
Proposed Changes

Added ShellSort.cpp
Made corresponding changes in the README.

Checklist

[X] ✅ My code follows the code style of this project.
[X] 📝 My change requires a change to the README.
[X] 🎀 I have updated the README accordingly.
[X] 👀 I have read the CONTRIBUTING document.
[X] ✨ I have added tests to cover my changes.
[X] 🚩 All new and existing tests passed.
[X] 🌟 ed the repo

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Output Screenshots

Hi @Lakhankumawat @siddhi-244 kindly review my PR and let me know if any change is required

Your contribution is good but your pr will be on hold if the earlier prs are inactive .

closed #1047

Your contribution is good but your pr will be on hold and closed if the earlier prs are inactive , your pr will be reopened reviewed and merged .

@Lakhankumawat I'm sorry the other ShellSort PR wasn't merged and it was opened 1 month ago so I had missed out on that.
Just a trivial question.
Since I have made a rather detailed README and noticed that the other PR hasn't, any chances the other PR wouldn't get merged, and mine would be reopened and merged instead? Or is it closed permanently?
Please let me know.


Great tool but Facebook images are washed out on phones but fine on computers.  Any way to fix this issue?

I've been using this to upload 3D drone photos to Facebook and I love it.  Thank you for the tool!  Unfortunately, the 3D Facebook photos appear washed out when viewing them on both of my Galaxy phones.  This is with both the Facebook app and Chrome browser via phone.  On my Windows 11 computer, they look great in both Chrome and Edge.  I've been creating the depth map with the res101 model, using the defaults.  Am I doing something wrong?  Is there a workaround?  Thanks.

An example is here in case the image doesn't post.  PC on the left, phone on the right.
https://photos.app.goo.gl/x4g41rxNqdummuNp8
I also tried https://github.com/Extraltodeus/depthmap2mask with a different model and have the same results.

Interesting, I got three different results on my Pixel 5a from the app, chrome, and chrome incognito.
https://discord.gg/swVmDx7Q


Use standard library importlib.resources

We previously used importlib-resources as a backport of the standard library importlib.resources to earlier versions of Python. We now use >=Python 3.9, so we can replace all uses with the standard library importlib.resources instead.

No idea why this doesn't work, and I can't reproduce the test failures locally (possibly due to Python 3.9 vs Python 3.10) so I'm shelving this for now.


add support for cstring, seq[char] and openArray[char] as input string

Although I would like to support cstring, seq[char] and openArray[char], I do feel like that it makes code and documentation more complex. I can't think of good cases where you would not have a string already or couldn't get one easily.
Do you have a good situation where this matters?

non-mutable string parameter shouldn't even be allowed when openArray[char] exists

Although I would like to support cstring, seq[char] and openArray[char], I do feel like that it makes code and documentation more complex. I can't think of good cases where you would not have a string already or couldn't get one easily.
Do you have a good situation where this matters?

cstring is convenient in JS backend has it avoid to convert the whole input to String (utf16 -> utf8)
I found it also interesting when getting cstring from ffi linked libs yet in UTF8 (like sqlite).  It avoid a copy of the whole cstring to nim string.
seq[char] and openArray[char] is coming from the fact I'm using a libsodium binding. after decryption I have either a seq[byte] or an openArray[byte].  Casting those to seq[char] or openArray[char] is safe when I know by the application that it is a utf8 sequence and avoid a copy to the Nim String.  I don't think it is safe to cast a seq[byte] to nim string even if I see it in lot of libs because, up to my knowledge, there is no guaranty the underlying C structs will remain compatible.

Sorry closing this. I think it makes the code a bit hard to read.


Create a external table with 'location' command success while elasticsearch-hadoop do not support 'location' function

Previously I thought that was my mistake, so I raised a question hear : https://discuss.elastic.co/t/to-map-hive-table-into-es/51900
But after I searched for days, I learnt that one can only use 'insert' command to add data through elasticsearch-hadoop.
This would be a problem when you wanted to sqoop data from mysql to HDFS.
For instance, there is a external Hive table that created with 'location' command, so it is mapped to a HDFS directory. Every time you use sqoop add more data to the HDFS directory, the table increase itself. However, as the elasticsearch-hadoop do not support 'location' mapping, when I create a external Hive table stored by 'org.elasticsearch.hadoop.hive.EsStorageHandler', every time I want to add any data, I have to create a Hive table, and use 'insert' command to manually add data.
As today, there is tons of data producing everyday, it is necessary to make tools auto incremental.

Based on the resolved conversation from here I'm moving to close this ticket. The LOCATION keyword on Hive's external table DDL assumes that the external table is loaded from HDFS. This is not always the case in the event of using a custom storage handler. Since the data is being queried directly from Elasticsearch, the Elasticsearch-Hadoop Hive integration does not leverage the LOCATION functionality on external table definitions.


Try disabling filters for autocomplete.

Maybe this will improve autocomplete latency a little?

coverage: 93.484% (+0.2%) from 93.244%
when pulling fead24f6b4db7d86e0a8a957cca401c81807d4cd on tweak-autocompleter
into 064139786ac84c5199ac37d0a1aadd0c09a2f851 on main.


Unable to play video with ffpyplayer on windows

Software Versions

Python: 3.7.7
OS: Windows 10 Home
Kivy: 1.11.1
Kivy installation method: pip

Describe the bug
When trying to play a video with the Videoplayer on windows and ffpyplayer  for video rendering, I get the following error when hitting play:
[ERROR][ffpyplayer] output.mp4: Protocol not found

I then tried to enable ffmpeg as the video renderer, but then I get the following error:
 AttributeError: module 'ffmpeg' has no attribute 'FFVideo'

The video runs well when I play it on my system, or when I use the ffmpeg system installation on it, so I'm not sure what could be the problem
Expected behavior
I expected the video to play correctly
To Reproduce
This is basic Video app, not sure it would be useful to provide code, I think it's more someting installation related
Code and Logs and screenshots
Same thing as previous

try this first:
python -m pip uninstall kivy.deps.gstreamer ffmpeg-python

I had the same problem. News on how to fixed? [ERROR ] [ffpyplayer ]video.mp4: Protocol not found

I had the same problem. News on how to fixed? [ERROR ] [ffpyplayer ]video.mp4: Protocol not found

in my case, I solved the problem by removing the videoplayers which I do not need

Do you have a colon in the filename? What is the exact full filename of the video?

Duplicate of https://github.com/kivy/kivy/issues/6964.

python -m pip uninstall kivy.deps.gstreamer ffmpeg-python

This worked for my Windows 10. Thank you.

Hope if I uninstall kivy deps.gstreamer,I won't gave bugs with kivy?

I uninstalled ffpyplayer and installed ffmpeg. It worked in my case


[WIP - DO NOT MERGE] Reactify

TODO:

[x] Home page
[ ] Resources pages

Connection flow

[x] Web USB flashing
[ ] Bluetooth connection logic
[ ] Connection states and hooks
[ ] Bluetooth and usb compatibility hook (also sub in where it is currently mocked in dialogs)

Connection flow dialogs

[x]  What will you need
[x]  Connect Cable
[x]  WebUsb Flashing Tutorial
[x]  Manual Flashing Tutorial
[x]  Connect Battery
[x]  Enter Bluetooth Pattern
[x]  Connect Bluetooth Tutorial
[x] WebUsb Choose microbit
[x] Connecting microbits (radio)
[x] Connecting bluetooth
[x] Flashing in progress
[x] Try again web usb
[x] Try again bluetooth
[x] Bad firmware error
[x] Unsupported microbit

Others

[ ] Add data page
[ ] Train model page
[ ] Test model page
[ ] Replace support links where there is TODO

I think we should take this opportunity to decide how best to share the WebUSB-related code with Python and see if we incorporate Web Bluetooth in a similar way.


Facing issue while installing sfdx on Ubuntu 17.10

I am facing attached error when installing sfdx.

Any help appreciated.

Do you have node installed?
On Dec 16, 2017 10:09 AM, "RajendraSinghNagar"<EMAIL_ADDRESS>wrote:
I am facing attached error when installing sfdx.
[image: screenshot from 2017-12-16 20-32-11]
https://user-images.githubusercontent.com/3901703/34071740-1c54ea1e-e2a1-11e7-881f-ab704db70ab3.png
Any help appreciated.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/forcedotcom/sfdx-simple/issues/33, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABNaPKJQ8qu7l6ElBRJIZLFejVlzQG7oks5tA90NgaJpZM4REXDk
.

Well, Yes Node 9.3 was installed in my machine.
I don't know how, but sfdx commands are working now. May be system restart worked :P.
Regards,
Raj


Safety check to prevent crash: status in updateTitle could be undefined

I encountered a condition with my Chromecast when this safety check is needed. Have not yet debugged further, but I thought I send a quick patch right away.

Thanks!


[Feature Request] Automatic Backtesting

Hey Hey,
i am missing a feature, that automatically calculates the best strategy for an asset pair over a timeframe.
Explain:
User gathers a database of 2 weeks, and has made backtests on it with a strategy.
It would be a nice feature to have an option to find a better combination of indicator values for this startegy by gekko calculatin trough it by it self.
With DEMA indicators for example. you let the automatic backtester calculate the best candlesize, best shot value, best long value.
The haasbot has a similar feature from my understanding, but i never personally used it.
Of course this will not be the best everlasting strategy then, but every once in a while you will have to make new calculations that may have been more profitable in the last 2 weeks or what ever timeframe you specify.

Agreed completely. I discussed a possible way to do this here: https://github.com/askmike/gekko/issues/831
And here's an implementation:
https://github.com/askmike/gekko/issues/840

Yes! Calculating the "best strategy" is a very hard problem:

first you have to define what best means:

least risky trades
most profit overall
combination of both (eg. sharpe)
least amount of trades that lost money
least exposure
a combination of all the above

than you need to figure out a way to come up with versions to test

GA (see links above)
brute force
AI/ML (at this point you probably ditch TA altogether)

For this reason there is no one size fits all backtest optimizer.
Though @thegamecat already coded a solution that implements GA, see #767.
Note that whichever route you go, the backtester needs to be fast. The backtester is currently very slow. See this comment to why it's slow and this comment for my ideas on how to make it faster.

A 1m candle 28 day window takes about 3-4 seconds with native non talib indicators.
I'm doing some thinking about how the ga could be used in real time to evaluate possibilities but I don't think it's the right tool on its own, if at all.

Yes! Calculating the "best strategy" is a very hard problem:

first you have to define what best means:
least risky trades
most profit overall
combination of both (eg. sharpe)
least amount of trades that lost money
least exposure

yes you are right. this will have to be defined, but where does the backtest tell me those things today?
For me the best is the one that is the highest away from the buy and hold.
Why not keep it simple?

@decoye you are describing a GA but a GA has limitations. It optimises for that specific scenario. What you want is machine learning to spot a scenario then execute the GA optimised strategy for that scenario.

hmm, i would be happy with an automated process, that tests from a range to a range.
lets put up a scenario here.
lets see if we could have been profitable in the last 2 weeks with a simple dema strategy.
how i now would do it is to import the range of two weeks of the exchange in question.
and then make backtests on it. long demo is 5 short dema is 10 time is 2h, hmm damn thats not profitable against buy and hold.
okay, next one long is 6 short is 10 still 2h, still not good. on and on and on and on.
once i find a lead i write it down and carry on the test to maybe find a more profitable one.
this is brute force by plain food work. that can be done easily by gekko. i think.


Fix 'Use $(...) notation instead of legacy backticked ....' issue in several places

CodeFactor found an issue: Use $(...) notation instead of legacy backticked ....

patch merged


Need help

Hi. It's me again :)
K, i used command $ python rnn_tf.py and some hours ago it was finished. And what should i do now?
Should i use command $ python rnn_tf.py saved/model.ckpt "The " to generate my text?
Will it generate a new file at "saved" directory ? How can i find text file?
Or mb i need to use another prefix? I didn't understand it yet :( Can u explain me a little, please?
Thx a lot and sorry for giving u some work :)

Did you see the output of the program at the end of training (it should output the first example of generated text).
If you use the command with saved/model.ckpt, it will use the trained model to generate new text. The trained model is saved automatically in that folder after you run the program in training mode. :)

Yeah. I saw, thx :)
But there's one more question: if i want to use another textfile to train with another language, should i delete model.ckpt or delete some other files. Or all files (language, characters for new language) will be rewrite and work correctly?

Well, model.ckpt will get overwritten with the new training anyway. Just replace the line of code where the text file is loaded with your own text file. :)

Okay. If i'll try to set something like 50000 batches - will it be too much or everything will be ok? :)

Awesome. :)

And what about using 2 or more texts to train the model?
For example:
There's a writer and soon he'll die (holly shit!), but he want to train this programm to continue his job. He wants to train this programm over 100+ texts for learning his "writing style".
Can we do something like that?

Just concatenate the text files / copy all their contents in the same file / modify the code to load all the text files you want and concatenate them in the same string used for training. ;)

Thanks!


Azure Functions Deployment Misconfiguration

Describe the bug
Deploying a JS functions app to Azure does not currently work. Several settings are missing or misconfigured - see "Expected behavior" section below.
Version
Built and run from main today (ddc2a6784b362bf54f62a65f48886eb2eb8688f1).
Browser

[ ] Electron distribution
[ ] Chrome
[ ] Safari
[ ] Firefox
[x] Edge

OS

[x] macOS
[ ] Windows
[ ] Ubuntu

To Reproduce
Steps to reproduce the behavior:

Generate a new JS Blank bot using the Azure Functions variant
Publish bot to Azure

Expected behavior

FUNCTIONS_WORKER_RUNTIME should be node instead of js
AzureWebJobsStorage is missing, should be a connection string for the Storage Account used to deploy the function, this leads to warnings about auto-scaling
WEBSITE_NODE_DEFAULT_VERSION should possibly be set?

@joshgummersall

I am happy to set WEBSITE_NODE_DEFAULT_VERSION, but I have not idea what value to use.  Is there a way to determine this from the JS runtime?
Can you tell me more about AzureWebJobsStorage? Is this meant to be a connection string to Cosmos DB or some other storage?
FUNCTIONS_WORKER_RUNTIME we pull the runtime from the adaptive runtime key (e.g. adaptive-runtime-node-functions), so if it is being set to js then I think we have a bug in the runtime key value we are using.  @benbrown How is the runtime key set in the JS templates?

It's possible that the node default version is not required, though I think the default is something like Node v10 which is very old. https://github.com/projectkudu/kudu/wiki/Node-versioning explains the setting, the SDK is going to EOL v10 support at some point and it isn't supported by the Node maintainers anymore so it should like be 12+, ideally 14+.
Docs for WebJobStorage: https://docs.microsoft.com/en-us/azure/azure-functions/functions-app-settings#azurewebjobsstorage, this is a setting that is automatically configured when you create a function app in the Azure portal
I used the runtime key that @benbrown suggested so hopefully it's not too tricky to add support for aliases or something. That key is set in the generator code


Random Timeout in Request/Reply

Hi everybody! Could someone please help me with this https://play.golang.org/p/cfAAERFCer? I get timeout error randomly - it's a simple request-reply sample & I'm running nats with gnatsd.

Hi @dc0d, you are getting a nats: timeout because there are no Subscribers ready to reply by the time the requests are sent.
You might get better results by calling Flush after you Subscribe in the receiver side...
        _, err = nc.Subscribe(subj, func(msg *nats.Msg) {
                cnt++
                replyMsg := fmt.Sprintf("from %v reply to request: %s", agentId, msg.Data)
                nc.Publish(msg.Reply, []byte(replyMsg))
        })
        err = nc.Flush()
        if err != nil {
                fmt.Println(err)
        }

but practically it is about making sure that there is at least a receiver by the time the requests are sent, in terms of this snippet a small wait before the requests are sent would help for example:
        go rpl(1)
        <-time.After(100 * time.Millisecond)

        go req(1)
        go req(2)
        go req(3)


Results minival vs minival_emb

Hi,
I was wondering which of the two values (minival & minival_emb) is reported in the paper.
Thank you.

Hi @avinashsai, we use minival. The metrics with suffix _emb is the intermediate retrieval results from the contrastive scores.


Buffers remain open when work buffer is closed

If I open a file to edit, NNP is activated.
If I then close this file, I'm left with the empty side buffers created by NNP.
I'm not sure how hard this is, but it be great if these where the last buffers open, that they'd close themselves.

oh this is weird, normally NNP should trigger a disable if you close of the main buffer (left, curr or right)
I'm not able to reproduce locally, could you provide some steps?

Ah indeed it stays open when using BufferClose, should work well with :q

Hey @BerkeleyTrue, which command/key do you use to kill the buffer? It would be great for me to properly reproduce

Apologies, I'm just using :q

Apologies, I'm just using :q

Made a lot of tests and with the above option it should disable it (hopefully!)


multithread safe uWS::Loop::defaultLoop()

When I try to create two app object in two thread, the Loops in each thread is confused, then program crashed. code snippet is:
void threadhttp1()
{
uWS::App app.get("/",[](auto resp, auto req){resp->end("hello.world");})
.listen(10086,[](auto token){}).run();
}
void threadhttp2()
{
uWS::App app.get("/",[](auto resp, auto* req){resp->end("hello.world");})
.listen(10087,[](auto* token){}).run();
}
thread th1(threadhttp1);
thread th2(threadhttp2);
th1.join();
th2.join();**
It's crash on uv's timer mini-heap with access null pointer at most of time. Finally I found the function uWS::Loop::defaultLoop() is not thread safe.

I don't know, you can check for yourself

Add an mutex lock guard in defaultLoop() has performance impact, so I try make sure multithread safe at first call uWS::Loop::defaultLoop() in each thread function, we call the defaultLoop() with a mutex lock guard as the first  call. After that, it's work!
std::mutex g_lock;
void threadhttp1()
{
{std::lock_guardstd::mutex lock(g_lock); uWS::Loop::defaultLoop();}
uWS::App app.get("/",[](auto resp, auto* req){resp->end("hello.world");})
.listen(10086,[](auto* token){}).run();
}
void threadhttp2()
{
{std::lock_guardstd::mutex lock(g_lock); uWS::Loop::defaultLoop();}
WS::App app.get("/",[](auto resp, auto* req){resp->end("hello.world");})
.listen(10087,[](auto* token){}).run();
}
thread th1(threadhttp1);
thread th2(threadhttp2);
th1.join();
th2.join();

Good find, it's now fixed inside defaultLoop


chore: Deprecate unused relation classes

This is cruft from Lean 3 core that was never cleaned up

bors merge

I agree with everything else, but are we sure IsLinearOrder α r is never going to come up?

Given that it hasn't in 7 years of mathlib existing...


Updated the dependency on the object-path component to fix a security…

… issue.

Pull Request Test Coverage Report for Build 150

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 137:
0.0%

Covered Lines:
296

Relevant Lines:
296

💛  - Coveralls


Suggested feature: Return all countries

Hi,
This would do me excellent if a function was implement to return all country codes.
Ste,

Hi,
You mean an array of country codes ? such as ['FR', 'EN', ...] ?

Yes, that would be great.
Thank you
On Mon, Sep 7, 2015 at 1:36 PM, Romain Sertelon<EMAIL_ADDRESS>wrote:

Hi,
You mean an array of country codes ? such as ['FR', 'EN', ...] ?
—
Reply to this email directly or view it on GitHub
https://github.com/rsertelon/iso-3166-country-codes-angular/issues/18#issuecomment-138285683
.

Will do ASAP ;)

I think we can simply use this?
ISO3166.codeToCountry


Remove version: taers232c.GAMADV-XTD3 version 6.70.00

This version is older than what has been set in max-versions-to-keep by the publisher.
Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/taers232c/GAMADV-XTD3/releases/download/v6.70.00/gamadv-xtd3-6.70.00-windows-x86_64.msi
OK
(Automated message - build 910)


problem on large xef file

I recorded .xef file about 180 GB. When the program render on 74800 frames, the error appeared and stop running.

Sorry I have not tried to record such a large file. Maybe it is the maximum frame it can save for the provided API or something...


Remove Application

As an application operator I would like to be able to run wego application remove [app] to remove gitops control of that app. By default this should leave workloads running but remove the gitops management, so I can stop gitops automation for an application, and remove it from the repository — but leave the workloads running in the current state.
--delete-workload flag proposed to remove all the workloads under management as well.

As an application operator I would like to be able to run wego application remove [app] to remove gitops control of that app.  By default this should leave workloads running but remove the gitops management.

Clean up UX so people don't get confused between remove and delete.

How about delete and remove-automation or deautomate  ? unautomate ?

https://github.com/fluxcd/flux2/discussions/763 has some more information.

Remove the app always, and always require the --force flag.

There is no indication (that I have found) in this story what the --force flag is supposed to do -- only that it is always required. If it means, "also remove the workloads from the cluster", then I feel it is poorly named. If it means something else, please describe its function.
Also, it is hard to interpret this sentence:
As an application operator I would like to be able to run wego application remove [app] to should  not leave workloads running but remove the gitops management, so I can clean up after tests.

In any case, I am currently unable to determine what is being requested here which is why I moved it back to "Refine"

Let's change it to --remove-workload
By default it would just remove the automation, and because that is hard, the --remove-workload flag is (for now) always required.

@jrryjcksn and @rokshana-b does this need a QA label?

This is open because we don't yet support remove via PR.

Implementation notes:

Remove the entry from the user/kustomization for a given cluster
Remove the apps/<my app> directory from the gitops repo

This is the work plan I have in mind for this ticket. Let me know if I should consider something different.

In the case where we want to use a PR to avoid the app being removed from k8s, I would like to highlight that if we have a PR(in the app config repo) in which one commit contains prune:false and another commit has the changes related to removing the app, it will leave the app running because flux syncs all commits in order. Syncing prune:false first, will make the app remain alive and the second commit will remove the automation of that app. But if this PR gets "squash and merged" it won't work (Maybe we want to specify this in the docs).

@josecordaz Let's scope this down to not include the --remove-workload flag for now. We can figure out how to do remove-workload later.

no problem

Application can be removed with and without PR


Player info struct is not correct for older demos.

On some demos the deathscsv command produces output like.
, ,<weapon>, <headshot>
Instead of the expected
<attacker_name>, <victim_name>, <weapon>, <headshot>
A demo file that presents this issue is found in #21. This is due to the format of player_info_t changing. Luckily, it looks like that struct is meant to be versioned. The player_info_t that works for the demo file in #21 looks like this
struct player_info_t
{
	// version for future compatibility
	uint32			version;
	// network xuid
	uint32			xuid;
	// scoreboard information
	char			name[ 32 ];
        ...  <the rest is the same, I think> ...

Note: this is many bytes shorter than the currently defined player_info_t
The first 4 bytes of player_info_t seems to always be a version. The rest of the structure changes depending on that version. The parsing code should be updated to act differently on different versions and should raise an error when an unknown version is encountered.

Actually not so sure if that is a version for GOTV player it always seems to be 0...hmm, I wonder if we could use the network protocol version from the header or something instead

You can use that as a switch, yes. It's basically the version number of CS:GO.

So the only difference between demo version is player_info_t uses uint32 in the older demos and uint46 presently?

@kaimallea The character array for the name is also sized differently.
@moritzuehling So it's always safe to use the first 4 bytes as a version number? It's been a few months since I have looked at any of this but I seem to remember getting some strange numbers. Also if some of the versions are 8 bytes (https://github.com/csgo-data/demoinfogo-linux/blob/552a2b3170ac5ab68bc6e049ef0dc0816f9543b0/src/demofiledump.h#L66) and some are 4 bytes it seems like that could be a point of issue?

@taylorfinnell ah, I see -- MAX_PLAYER_NAME_LENGTH is now 128. Next question is ofc about the version numbers. I'll wait on @moritzuehling response :smiley:


Add a graph diff tool

I want to be able to do
% seafoam examples/diff/added-truffle-profile-before.bgv.gz:4 render --diff examples/diff/added-truffle-profile-after.bgv.gz:4

And see nodes that haven't changed greyed out, removed nodes in red, added nodes in green, like a source code diff.
Example graph and start of implementation in https://github.com/Shopify/seafoam/tree/diff.

Trying not to have speculative issues.


JSX Fragment <></> Support

When I use the new React JSX Fragment syntax <></> I get

[standard] Strings must use singlequote. (quotes)

Is there plans to support this newer syntax?

I realize now this is probably a babel issue.


Fix ingress api version selection

Turns out using  .Capabilities.APIVersions.Has "networking.k8s.io/v1” is flawed because while the api version is already available in 1.15 but does not contain the Ingress resource.
Using .Capabilities.APIVersions.Has "networking.k8s.io/v1/Ingress” is only supported with helm >= helm 2.15 and we are still using helm 2.11 in k- controlplanes.
So we have to go by kubernetes version instead to decide if to use networking.k8s.io/v1 for ingress resources.

Please bump the chart version. :blush:

Done!


Revert "add -L to get stat for symlink"

This reverts commit 9a5d74983c4f23ba3efcfae602b3133491a5cee2.
We will tackle this commit separately with symlinking. The idea that Victoria and Alex raised it the right one, we have to remodel it in the next minor update.

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
GitHub Users Who Are Not Authorized To Contribute
The following GitHub users do not appear to have signed a CLA:

@arlimus

Please sign the CLA here.


How to get current zoomScale?

On iOS we can look for the scrollView onScroll event to get current zoomScale.
How can I get it for for this module on Android?

I add a listener for scale changing. Giving a function to the prop onScaleChange will give you something like this. The example has an update.
{
  "focusY":<PHONE_NUMBER>96875,
  "focusX":<PHONE_NUMBER>871094,
  "scaleFactor": 1.0006426572799683
}

Awesome, thanks!

For those who are new to this, note that arguments above are found in e.NativeEvent.scaleFactor, ...
              onScaleChange={(e) => {
                console.log(e.nativeEvent);
              }}/>

Not work for iOS?


Cleanup relative path in compile elements to have a link

without the link, VS will show all relative path elements in the root of the
project explorer. adding link will organize the view and also avoid a conflic
with the same file names

anyone know why the different platforms build checks is not enabled on this PR?

Jenkins is down temporarily.

@dotnet-bot test this please

LGTM. Thank you for cleaning this up!


Require jittered o/d points to be a minimum distance apart

For #1

@Robinlovelace, here ya go!

:fire:


GPU Utilization zero when evaluating GGUFs through lm-evaluation

i'm enabling gpu-acceleration during installation  as suggested in readme:
CMAKE_ARGS="-DLLAMA_CUBLAS=on" FORCE_CMAKE=1 pip install 'llama-cpp-python[server]'
Then activating the local server by:
python3 -m llama_cpp.server --model models/7B/llama-model.gguf --n_gpu_layers -1
and then finally using lm-evaluation for evaluating:
lm_eval --model gguf --model_args base_url=http://localhost:8000 --tasks multimedqa
....this just runs evaluation without using any GPU

On 1xa100 - runpod

the same, just use CPU running, but occupied about 500MB GPU-memory

the same, any solution?


⚠ Bofrost.lu has degraded performance

In bee27c6, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1416 ms

Resolved: Bofrost.lu performance has improved in cbb6a93 after 7 minutes.


Roll Engine from 323033b1459d to 74774ea0b36d (1 revision)

https://github.com/flutter/engine/compare/323033b1459d...74774ea0b36d
2021-05-19<EMAIL_ADDRESS>Roll Dart SDK from 67be110b5ba8 to 3a8c551ec08e (9 revisions) (flutter/engine#26258)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md

Commit queue failed; closing this roll.


PoC for cdk8s-plus-21

Research ways to support multiple versions of k8s

After a few days of research, sharing conclusions.
Problem Statement
We are currently unable to add support for features that were introduced in any spec above 1.17.0.
This is because adding such support in the current library will cause consumers to be exposed to features that will not work on their k8s 17 deployment.
This means we have to provide a separate library for each spec version (which is why named this library with -17).
Problem is that if we want to add support for features that exist in multiple specs, we would have to duplicate code across all libraries - which can become an overwhelming maintenance burden.
Proposed Solutions
Following are a few ideas we tried.
Extension
Since k8s specs are backwards compatible, this lends itself well to have later versions of resources extend earlier versions of the same resource:
import * as kplus17 from 'cdk8s-plus-17'

export class Service extends kplus17.Service {}

This presents a few problems:

Enums cannot be extended
Static classes cannot be extended
Spec is not forward-compatible. This means that we cannot invoke the 21 constructor with properties from 17, since some properties might have changed optionality (went from required to optional).

Delegation to Latest
Implement all the logic in the library corresponding to the latest spec, and change earlier spec libraries to delegate to it.
import * as kplus21 from 'cdk8s-plus-21'

export class Service {

}

This presents a few problems:

Cannot delegate implementations that accept a concrete type since all types are foreign.
Since we cannot extend the interfaces and classes, we would have to implement continuous code generation of the public API available in each library.

Codegen View
The idea here, which wasn't actually attempted is to have the latest spec library be fully aware of all previous spec versions, essentially as if it were the only library and is meant to serve all spec versions (failing on invalid properties based on the k8s version).
We would continuously duplicate this library across all spec versions, but codegenerate a different view (i.e public API) for each of them, corresponding to the spec.
This presents a major implementation challenge that we are not sure is worth it.
Manual
Instead of trying to come up with a magical mechanism to solve this, simply bite the bullet and accept the maintenance burden of supporting multiple packages. This essentially means treating each library as if it was it own version line, and perform backports and forward-merges as necessary, just like when maintaining two major versions of the same library.
Chosen Solution
Manual was choses because this is the one at the least unknowns and we know is going to work. We also decided to limit the number of spec versions we will support to the 3 latest versions.


Cannot find module '../cordova/platform_metadata'

cordova plugin add cordova-plugin-photo-library --variable PHOTO_LIBRARY_USAGE_DESCRIPTION="To choose photos" --save
Installing "cordova-plugin-photo-library" for ios
Installing "cordova-plugin-file" for ios
Installing "cordova-plugin-compat" for ios
Installing "cordova-plugin-add-swift-support" for ios
Adding cordova-plugin-photo-library to package.json
Saved plugin info for "cordova-plugin-photo-library" to config.xml
(node:61839) UnhandledPromiseRejectionWarning: Error: Cannot find module '../cordova/platform_metadata'
    at Function.Module._resolveFilename (module.js:543:15)
    at Function.Module._load (module.js:470:25)
    at Module.require (module.js:593:17)
    at require (internal/module.js:11:18)
    at Context.requireCordovaModule (/usr/local/lib/node_modules/cordova/node_modules/cordova-lib/src/hooks/Context.js:77:12)
    at module.exports (/Users/ptkdev/cordova/meingifs/plugins/cordova-plugin-add-swift-support/src/add-swift-support.js:23:34)

@ptkdev This was a bug in cordova-plugin-add-swift-support, just update it to latest version (1.7.1)
cordova plugin rm cordova-plugin-add-swift-support
cordova plugin add cordova-plugin-add-swift-support

Thanks!

I had this problem with swift plugin version now I upgrade it to 1.7.2.
install swift plugin before cordova-plugin-photo-library because this plugin is using 1.6.0
@bwdrop Thanks!

If you already installed cordova-plugin-photo-library then you can just force remove cordova-plugin-add-swift-support and then add it back. This should fix the problem.
cordova plugin rm cordova-plugin-add-swift-support --force    cordova plugin add cordova-plugin-add-swift-support
and for ionic
ionic cordova plugin rm cordova-plugin-add-swift-support --force   ionic  cordova plugin add cordova-plugin-add-swift-support

I love you man

thanks @omeasraf

To make this fix more permanent you will have to add cordova-plugin-add-swift-support before cordova-plugin-photo-library in your config.xml just like so:
...
    <plugin name="cordova-plugin-add-swift-support" spec="1.7.2" />
    <plugin name="cordova-plugin-photo-library" spec="2.2.1">
        <variable name="PHOTO_LIBRARY_USAGE_DESCRIPTION" value="" />
    </plugin>
...

I'm not into the plugin development, but couldn't  this be fixed by simply bumping the dependency version in the plugin.xml?

I have got the same error after adding geo fence plugin
swift plugin got installed while geofencing is installing...can someone help me!Thanks in advance


Direct Attachment without conversion into byte or passing URL

I have used trello api for node, that api is also proving ability to directly attach attachments without converting file/image into bytes and without passing url.
One of the important functionality that this code is missing is to directly attach the attachment into trello cards without converting it into bytes from desktop.
I want to open that attachment in the same tab after adding attachment.
Just like this

The API docs say to upload as a Multipart Form Data, which is what I'm doing. In .Net, this requires that I supply it with a byte array.  I'm not sure what else I could be doing here.

I created a short test for this:
var list = _factory.List(TrelloIds.ListId);
var card = await list.Cards.Add("attachment test");

var fileData = File.ReadAllBytes(@"c:\users\gregs\desktop\images.jpg");
var attachment = await card.Attachments.Add(fileData, "test", ct);

It works just fine: https://trello.com/c/s3frWHY4

After spending the whole day, I figure out the way to add attachment without converting, the file into byte array. With this feature one can view the attachment on trello board instead of downloading and then converting the byte array into image.
I made some changes in the code, one of the major change in WebApiClient File and tested the code on my trell account, It was working fine. However, when I tried to push my branch into this repository, I am getting the access denied message

Here is the snippet for the some modification in WebApiClient file.

I was getting the access denied message

Yes, you have to fork the repo and create a pull request from your fork.
While those changes work for local files (specifically only PNG images), they may not work for other URIs or contents.
Lastly, please don't post screenshots of code; just post the code.  Text in images cannot be copied.

I'm not entirely sure what's not working for you with the existing code and the code snippet I posted.  You posted a screenshot of an open image in Trello, but I don't know what you're getting.  As I mentioned, it's all working fine for me and in my integration test suite which runs on Appveyor.


python3Packages.vt-py: 0.8.0 -> 0.9.0

Motivation for this change
Update to latest upstream release 0.9.0
Change log: https://github.com/VirusTotal/vt-py/releases/tag/0.9.0
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

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

3 packages built:

ioccheck
python38Packages.vt-py
python39Packages.vt-py


Develop automated workflow for downloading corrected transcripts

Desired workflow:

[ ] run a task to get a list of GUIDs that have the completed status (or something equivalent)
[ ] run a script to hit the aapb transcript endpoint for a specific list of GUIDs (I'd like to be able to do this for the GUIDs I get from the first task, but also be able to add any other ones I want, so the two tasks don't have to feed directly into each other)
[ ] use script Kevin wrote to upload new transcript to americanarchive.org S3 that moves the current transcript and then adds the new one, so it doesn't overwrite the previous transcript, in case we want it in the future.
[ ] submit batch metadata updates to the transcript status for each GUID to set status to Correct.
[ ] reindex affected GUIDs on AAPB.

This work is done, but needs to be released. It adds a new boolean field :released to the Transcript model, that marks whether a completed transcript has been synced to AAPB, and adds a ruby wrapper for Kevin's transcript versioning script. The user must ssh into fixitplus, and then run workflow.rb. This will generate the transcript files for all unreleased correct transcripts, and then push them into AAPB's transcript folder on S3.


Use numeric test for multipleOf

Slightly more elegant than hardcoding an absolute precision

Thank you for your contribution!


VnV Plan 5.1: Table 1 Brackets

All the items in the F column are {()} and then you have another column named {S(p,r)}. I'm not sure why you have those double brackets. If F is supposed to be a set {(),(),...} then why are there no inputs with more than one item?

I appreciate your comment, but in section 5.1 the goal of test cases is to measure run-time based on the number of joints. So the other variables are held the same to avoid their effect. Otherwise, the external force can be a set of different forces. but in this section, there is only one force for all of the test cases