Fix typo in BlockChangeFlag JavaDoc ("phsyics" => "physics")

Such small fixes should probably be reported there: https://github.com/SpongePowered/SpongeAPI/issues/1000


Fix lighthouse bug #136 (Play2's anorm can't work on postgresql)

This patch works around postgresql's behaviour to return the empty string on getTableName().
It calls getBaseTableName() instead if the ResultSetMetaData is a PGResultSetMetaData.
Disadvantage:

adds a dependency on the postgresql JDBC driver

(bug report: https://play.lighthouseapp.com/projects/82401/tickets/136-play2s-anorm-cant-work-on-postgresql)

Guillaume Bort implemented a workaround, basically using the same approach, but using structural typing, so he didn't need to depend on the postgresql jdbc driver.
Therefore this patch is obsolete now.


StatsD and DogStatsD does not understand the meter metrics

This is related to https://github.com/petabridge/phobos-issues/issues/15
It seems like StatsD removed their support for Meter and DogStatsD doesn't support it either.
Instead, DogStatsD converts Meter into their own Rate metric, which is similar in function to the AppMetrics and StatsD Count metric.

4.2.0-preview.0.33 pushed


adds maven wrapper

Adding maven wrapper from https://github.com/takari/maven-wrapper eradicates users to setup maven in local environment and can start developing or contributing directly
@xscript @yungez @aovechki @ZhijunZhao

@rajadilipkolli,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Codecov Report

Merging #74 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #74   +/-   ##
=======================================
  Coverage   52.06%   52.06%
=======================================
  Files          35       35
  Lines         557      557
  Branches       47       47
=======================================
  Hits          290      290
  Misses        244      244
  Partials       23       23

Continue to review full report at Codecov.

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

rebased with master

any update on this?

@yungez Fixed comments and rebased code with master, Please review


riscv: telink:tl3218x: add ble optimize.

add optimize for ble.
low down the ramcost.

can not set the cache size to 8k , otherwise the tl3218x compile will fail for ram not enough . the lighting-app will open the ota, factorydata and shell , without shell is enough , but with shell will exceed 1312bytes .


[IGNORE] Update dashboard migration logic

Due to the breaking changes in https://github.com/perses/perses/pull/1032, we need to update the dashboard migration and validation logic. Specifically, remove queries from each plugin and add it to the panel level.
Merging this branch to eunicorn/define-queries-in-panel-spec

Thanks for tackling this! As expected this datamodel breaking change broke the migration unit tests, given the time difference I guess I have time to open a PR to your branch for this ;)

Thanks for tackling this! As expected this datamodel breaking change broke the migration unit tests, given the time difference I guess I have time to open a PR to your branch for this ;)
EDIT actually it's quite more than just unit tests, 'have to change a whole part of the schemas load & validation logic - but for the best! Not sure I'll be able to finish today but anyway I'm on it.

Thank you so much! Let me know if there's anything i can do to help!

Not sure I'll be able to finish today but anyway I'm on it.

One thought @eunicorn, I would love to do a v0.27.0 release before we officially merge the PRs that introduce the breaking queries change (there are a bunch of bugfixes in flight that will be good to get published), I'm planning to take another look at those PRs soon too

@AntoineThebaud do you mind approving this? thanks!


Nested elements with transitions: possible to not trigger all?

Is it possible to remove a block from the document that uses a transition without also triggering all the transitions inside that block? For example:
  {{#if foo}}

       Welcome to my cool section
       {{#if bar}}

         Here is a single piece of that section

       {{/if}}

  {{/if}}

If foo becomes false, that entire block will be removed and trigger the outro, as expected. But as it fades, I can see the bar block transitioning away with its own slide outro at the same time. Ideally, only the outermost transition would happen.
This example is overly simplified, but as as my project has become more complex, I've ended up with more and more of these situations -- like a list of things which can appear or disappear as a whole, with individual list items that can also appear and disappear on their own.

I also find that the transitions should only apply to the root element being removed. I don't feel it makes sense...
The current behavior doesn't allow me to create a Collapse component in a clean way, for example.

Closing this as it falls under the umbrella of #1853, which is an ongoing (albeit hibernating) discussion.


add ts_hour to ac wiring loss line

To test:

Run a subhourly simulation
Confirm that the AC wiring losses match the input (1% is default)

Unchecking the "save all output variables over lifetime" option is recommended if testing with a one minute weather file.
Fixes https://github.com/NREL/SAM/issues/1082

Push build on ubuntu failed due to error connecting to cmake. We can either re-run the tests or trust the results of the mac push and pull request builds.

Push build on ubuntu failed due to error connecting to cmake. We can either re-run the tests or trust the results of the mac push and pull request builds.

All tests seem to be passing now!


Changes to templates not reflected in local development environment

I am attempting to contribute a simple change to the HTML templates of this project, but I am getting stuck in the testing phase because my changes are not reflected in my local instance of Miniflux.
For example, when I change template/html/common/layout.html to have <title>Foo</title> instead of the actual <title> tag, then run make run and open localhost:8080, I still get a page with the title "Unread Items (10) - Miniflux." Same thing with make build then ./miniflux-darwin-amd64.
From what I can tell, common.go is being updated with my changes to layout.html, so I think go generate is running successfully.
go version yields go version go1.10.3 darwin/amd64. I am using MacOS 10.13.5. I have tested with Firefox and Chrome.
What am I doing wrong?

Hum, it should work If you see your changes in the generated file. There is no magic here.

I will make a fresh clone of the directory and try again.

I figured it out. The problem came up because I had cloned my fork of the project to go/src/github.com/adamvig/miniflux, but all of the packages have absolute package names, for example github.com/miniflux/miniflux/cli.
This meant that even though I was running make run in the repository of my fork, since I also had go/src/github.com/miniflux/miniflux cloned, the build process was using the source files from that repository instead.
It could be useful to update the documentation to reflect this.
The correct way (as far as I can tell) to contribute to this project is:

fork the project on GitHub
clone the original project to go/src/github.com/miniflux/miniflux
change the remotes of that repository to point to your fork


Dont work

i turned on the miner on my computer but it doesn't work

What GPU do you have?

amd ryzen 5 2600 six-core processor 3.40 ghz

As your GPU?

oh sorry rx 590

RX 590 8GB? Also, people have been reporting issues with OpenCL with the newest AMD drivers. I might have to find some workaround for this if it's even possible.

hmm ok ty


Error precompiling

I can ]add Casacore (v 0.2.0) in Julia 1.8.3 , but when it tries to precompile it gives this error:
julia> import Pkg; Pkg.precompile()
Precompiling project...
  ✗ Casacore
  0 dependencies successfully precompiled in 1 seconds. 381 already precompiled.

ERROR: The following 1 direct dependency failed to precompile:

Casacore [72bb1062-94f5-49fa-bb69-94b615203ad9]

Failed to precompile Casacore [72bb1062-94f5-49fa-bb69-94b615203ad9] to /Users/davidm/.julia/compiled/v1.8/Casacore/jl_pJbmQl.
ERROR: LoadError: UndefVarError: libcasacorecxx not defined
Stacktrace:
 [1] top-level scope
   @ ~/.julia/packages/Casacore/Xzq3F/src/LibCasacore.jl:7
 [2] include(mod::Module, _path::String)
   @ Base ./Base.jl:419
 [3] include(x::String)
   @ Casacore ~/.julia/packages/Casacore/Xzq3F/src/Casacore.jl:1
 [4] top-level scope
   @ ~/.julia/packages/Casacore/Xzq3F/src/Casacore.jl:3
 [5] include
   @ ./Base.jl:419 [inlined]
 [6] include_package_for_output(pkg::Base.PkgId, input::String, depot_path::Vector{String}, dl_load_path::Vector{String}, load_path::Vector{String}, concrete_deps::Vector{Pair{Base.PkgId, UInt64}}, source::Nothing)
   @ Base ./loading.jl:1554
 [7] top-level scope
   @ stdin:1
in expression starting at /Users/davidm/.julia/packages/Casacore/Xzq3F/src/LibCasacore.jl:1
in expression starting at /Users/davidm/.julia/packages/Casacore/Xzq3F/src/Casacore.jl:1
in expression starting at stdin:1
Stacktrace:
 [1] pkgerror(msg::String)
   @ Pkg.Types /Applications/Julia-1.8.app/Contents/Resources/julia/share/julia/stdlib/v1.8/Pkg/src/Types.jl:67
 [2] precompile(ctx::Pkg.Types.Context, pkgs::Vector{String}; internal_call::Bool, strict::Bool, warn_loaded::Bool, already_instantiated::Bool, kwargs::Base.Pairs{Symbol, Union{}, Tuple{}, NamedTuple{(), Tuple{}}})
   @ Pkg.API /Applications/Julia-1.8.app/Contents/Resources/julia/share/julia/stdlib/v1.8/Pkg/src/API.jl:1432
 [3] precompile
   @ /Applications/Julia-1.8.app/Contents/Resources/julia/share/julia/stdlib/v1.8/Pkg/src/API.jl:1063 [inlined]
 [4] #precompile#225
   @ /Applications/Julia-1.8.app/Contents/Resources/julia/share/julia/stdlib/v1.8/Pkg/src/API.jl:1062 [inlined]
 [5] precompile (repeats 2 times)
   @ /Applications/Julia-1.8.app/Contents/Resources/julia/share/julia/stdlib/v1.8/Pkg/src/API.jl:1062 [inlined]
 [6] top-level scope
   @ REPL[26]:1

Here is my version info:
julia> versioninfo()
Julia Version 1.8.3
Commit 0434deb161e (2022-11-14 20:14 UTC)
Platform Info:
  OS: macOS (arm64-apple-darwin21.3.0)
  CPU: 8 × Apple M2
  WORD_SIZE: 64
  LIBM: libopenlibm
  LLVM: libLLVM-13.0.1 (ORCJIT, apple-m1)
  Threads: 1 on 4 virtual cores

Is this due to lack of M2 support in casacore itself?  If it can't (yet) work, it would at least be nice to get a friendlier error message.

Hi David, thanks for the bug report.
The error you're seeing is because the casacorecxx binary for your system is missing. Now, I haven't tested this on an ARM system but from the build instructions I'm reading for casacore_jll and casacorecxx_jll, it seems like we should be building for a system if:

it's not windows
it's not freebsd
and the libc is glibc (not musl)

So I think (??) actually this should work on MacOS/M{1,2}. I should get a chance this weekend to test properly on that hardware.

After a little investigation, it turns out the requirement libc == glibc essentially blocked macOS builds of casacorecxx_jll.
@david-macmahon I have posted a PR to enable macOS builds, and once this is accepted, I'll ask if you could update casacorecxx_jll and run the test suite to see how things go.

@david-macmahon The package has been rebuilt. From the Pkg REPL, run pkg> up Casacore and then run pkg> test Casacore and let me know if the test suite passes on macOS aarch64.

Thanks! It passed the tests on my MacBook Air M2 laptop (see above for versioninfo() output).  Here's what I got:
     Testing Running tests...
Test Summary: | Pass  Broken  Total  Time
Casacore.jl   |  188       1    189  8.2s
     Testing Casacore tests passed

Great news!


⚠ SceneTime has degraded performance

In 8e88b88, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1616 ms

Resolved: SceneTime performance has improved in 12e46d5 after 8 minutes.


Experimental expansion (non-stale edition)

Evidently, pull requests can become stale & auto-close on this repository. Didn't know that, so didn't think to keep this thing updated periodically. Whoops. Anyhow, I added some new things today just so this isn't a flat repost of the last PR, but as a refresh:

RawKeys to partial struct to spread across multiple files for readability, ensure non-breaking implementation
Exclusively use low-level hooks, WorkInBackground checked in dedicated method & dynamically toggleable
Low level hook for mouse; scroll wheel & XButton 2 support as a result
String loading method improved, now testable in LogRawInput

New Additions:

RuntimeInitializeOnLoadMethod added to check if runInBackground is enabled; if not, flash a warning and enable for the current playsession
RawKey variables now readonly fields & built in static constructor, ensuring they're only created once
Some code cleanup (Better null check for dictionary & removed non low-level methods)

My bad, the job wasn't configured properly; should be fixed now.

I've checked the changes and can see a lot of useful stuff, but unfortunately I won't be able to maintain all that. The single low-level hook however is definitely something I'd like to keep, as it'll both simplify the solution and add support for more inputs. If you can limit the PR to just that (while keeping the old keys enum), I'll gladly accept it.

Trimming down to the single LL hook while keeping the enum format should be manageable. Expanding beyond that is looking more like something better suited for another project entirely, so I'll keep that in mind when proposing any future changes


List of all HEx colors?

Are Hex codes of all possible colours(from using randomColor) available somewhere?

No need. I checked the code of the repo

No worries, glad you found them!


Add other types of triggers rather than binary switch

I have few temperature sensors around my house. I have smoke detectors, yes, but if temperature raises to 60 degree Celsius, I may be certain that there is some fire. I want alarm to be triggered with those conditions too.
Possible to add more complex conditions to sensors?
If i add automation and trigger the built in HA alarm, will it work? Will your alarm be triggered?

Duplicate of https://github.com/nielsfaber/alarmo/issues/184


Fix slashing conditions

What is wrong?
https://github.com/ethereum/eth2.0-specs/pull/373
How can it be fixed
Fix eth.beacon.helpers.is_double_vote and eth.beacon.helpers.is_surround_vote as https://github.com/ethereum/eth2.0-specs/pull/373.

I can work on this!


Make layouting usable independent of other plotting features

Description
I really like the proplot package! For me, by far the most important feature is the extremely powerful and flexibel way of aranging subplots in a figure by using units like inches instead of always having to compute in figure coordinates which change whenever any part of the figure changes.
However, I also have to use other plotting packages, most notably seaborn, which rely on many properties of matplotlib axes and plot objects. This is the reason why right now very often I can't use proplot although I would love to use it at least to layout my figure, or why at least I'm very hesitant to use proplot: changes are, some results down the line I have to port everything back to standard matplotlib just to be able to use some statistical plotting function that I need.
This brings me to my feature request: It would be absolutely awesome, if it would be possible to use pplt.subplots to create a figure and the subplots, but have all other features of proplot disabled or set to standard matplotlib behaviour (for example, by having subplots return standard matplotlib objects and have all config changes disabled). Then using proplot would be less of an all-or-nothing decision.

Really glad you find proplot useful!
This is definitely valid, but after thinking about it I'm leaning toward the alternative: inviting users to annoy me with 1000 bug reports, and we'll slowly plug away at them over time :). ProPlot is "supposed" to be mainly a superset of matplotlib except in a few very limited cases (e.g. legend accepting lists of handles as a single positional argument). However, it does not entirely live up to that goal. Fixing bugs that crop up with external packages would be a win-win:

It would make the learning curve easier. If a developer writes matplotlib code that's incompatible with proplot, a new proplot user can do it too. This would help minimize backwards-incompatible changes and issue relevant warnings/gentle fallback behavior wherever incompatibilities appear (I try to do this already, but it's not systematic).
(As you said) It would let users invest in other packages, and take the pressure off of me to keep adding features. For example, while proplot overlaps with seaborn in several areas due to differing philosophies/design choices (e.g. "palettes" vs. colormaps/cycles and constructor functions), it would be silly of me to start replicating all of seaborn's plotting commands/behaviors in proplot, beyond the most basic stuff (e.g. kernel density plots, multicolor histograms). I'd like to support seaborn (maybe even include seaborn examples in the docs some day), but it's just a matter of limited time.

Also, by far the largest, most sweeping incompatibilities are related to geometry stuff, while the plotting commands can be close to 100% matplotlib-compatible if they just have more time to mature. So this would not necessarily solve all integration problems. And of course, this would require some pretty severe refactoring.
So, I'll close this issue for now and invite the bug reports instead, but let me know if you disagree with any of these points. Also, if you haven't yet, you should upgrade to v0.9 as it fixes some very basic/annoying seaborn integration issues (suppresses colormap re-registration warnings, fixes fatal bug related to "swarm plots" -- see #272).

Also, if the scope of the incompatibilities is larger/more intractable than I thought, we can definitely reopen.

@lukelbd I see your point. In this case, I'll happily open issues as I come across them :).


Add webhooks for MoonPay and Ramp

Description
Add Webhook support for Moonpay and Ramp.
This PR includes adding tx hashes of those two providers to Firebase.
Remaining action items related to this:

Add Transak webhooks (blocked on staging not working for them)
Add Simplex event integration (they don't have webhooks and it depends on having their API integration working)
Add Analytics events for all providers and send them to Segment/BigQuery.

Other changes
N/A
Tested
Only on Alfajores. It requires real cash in to test on mainnet and the providers are not available on my country.
Related issues

Fixes #25

Backwards compatibility
N/A

Hey @gnardini I have verified this issue using Alfajores 1.14.0(58) build for iOS & observed following:
Observations:

For Moonpay:

It is observed that, user get "Sorry we can't process action at this time" message displayed on "staging.moonpay.com" page after tapping on "Continue" button. Hence we are blocked to proceed as of now. Attachment

For Ramp:

It is observed that, user able to change country to "United States" from any random country even US is not supported message displayed at Ramp page.

Also it is observed that, "STATE IS REQUIRED" message displayed even user entered valid state name.
Attachment: RampIssues.mp4

Can you please take look & help us to verify this issue
Verified On Devices: iPhone SE (14.2)
Note: For now we are not able to see latest Alfajores build for Android. Will update you when we will get build for same.
Thanks..!!

@Celoqa directions for working with test transactions on staging with Ramp - the transaction succeeds on Ramps side: Attachment

Hey @MuckT Thank you so much for giving detail information with attachment:)
I have tested this scenario on Alfajores Build V1.14.2(61) for iOS & Alfajores Build V1.14.1  for Android.

Yes now we are able to complete transaction using Ramp provider & able to get Transaction Receipt after completion of transaction successfully.

Attachment:
RampSucceedTransactionProcess.wmv
RampReceipt.jpg
Verified On Devices: iPhone SE (14.2), Redmi Note 8 (9.0)
Thanks..!!


Unable to create a cluster in vSphere if the dc name consists of only numbers

What happened:
If the vSphere datacenter name specified in VSphereDatacenterConfig contains only numbers, then during generation of the VSphereMachineTemplate an integer value is substituted in spec.template.spec.datacenter (even if the value is a string in VSphereDatacenterConfig). This results in an error while applying the new resources of this kind.
2022-09-17T12:57:07.798+0300    V6      Executing command       {"cmd": "/usr/local/bin/docker exec -i eksa_1663408305631319000 kubectl apply -f - --namespace eksa-system --kubeconfig eks-system/generated/eks-system.kind.kubeconfig"}
2022-09-17T12:57:08.194+0300    V5      Error happened during retry     {"error": "executing apply: Error from server (Invalid): error when creating \"STDIN\": VSphereMachineTemplate.infrastructure.cluster.x-k8s.io \"eks-system-control-plane-template-1663408479593\" is invalid: spec.template.spec.datacenter: Invalid value: \"integer\": spec.template.spec.datacenter in body must be of type string: \"integer\"\nError from server (Invalid): error when creating \"STDIN\": VSphereMachineTemplate.infrastructure.cluster.x-k8s.io \"eks-system-etcd-template-1663408479593\" is invalid: spec.template.spec.datacenter: Invalid value: \"integer\": spec.template.spec.datacenter in body must be of type string: \"integer\"\nError from server (Invalid): error when creating \"STDIN\": VSphereMachineTemplate.infrastructure.cluster.x-k8s.io \"eks-system-md-0-1663408479597\" is invalid: spec.template.spec.datacenter: Invalid value: \"integer\": spec.template.spec.datacenter in body must be of type string: \"integer\"\n", "retries": 28}

What you expected to happen:
When generating a resource manifest of kind VSphereMachineTemplate, always convert the spec.template.spec.datacenter value to a string.
How to reproduce it (as minimally and precisely as possible):
Name the datacenter in vSphere like 1234 and try to create a new initial-cluster.
Anything else we need to know?:

Closing in favor of #3382 getting merged


Set VLAN Priority on dhcp6c packets

Add VLAN tag to outgoing dhcp6c packets. Option setting in WAN dhcp6c section.

Removed fixed 'wan' and used the existing routine to get the info required.

Finally back in France where this functionality is needed with Orange FTTP service
Works like a charm

Good stuff.
We still need to get the modified dhcp6c client sorted too, that will finally end the problems with Orange France.

Indeed if we can get the modified dhcp6c client into the main build along with the modified dhclient client then Orange fttp users in France will able to upgrade via the GUI and not need manually replace the files each time
The two modified clients are very stable as my firewall has been up without issues for 3 months

I have issues a PR for dhcp6c changes on hrs-allbsd/wide-dhcpv6
Add the ability to use RAW parameters in the config PR #8

In an effort to improve the tracking of changes and bug/feature requests, we have decided to require an entry on the pfSense Redmine issue tracker associated with every pull request, and likewise the Redmine entry should also have a link back to the pull request.
If you could, please associate this PR to a Redmine issue either by locating an existing issue at https://redmine.pfsense.org or by creating a new issue. Add a link to the Redmine issue that points to this PR, and also add a link on the PR to the Redmine issue.
For more information, see https://doc.pfsense.org/index.php/Submitting_a_Pull_Request_via_Github
Thanks!

Redmine 8200

Redmine #8200

Redmine Issue: https://redmine.pfsense.org/issues/8200


Fire behaves wrongly

Fire can only be lit on the top faces of solid blocks, as opposed to being able to be lit on any face of a flammable block. Any pre-existing fire that isn't on the top face of a solid block will extinguish quickly.
Versions:
mc 1.19.3
ql 17.8, 18.1-beta.25
qfapi 5.0.0-alpha.5, 5.0.0-alpha.6
qsl 4.0.0-beta.2
fapi 0.68.1, 0.69.1

Quilted Fabric API 5.0.0-alpha.7 has updated to QSL 4.0.0-beta.3 now (and for real this time :P)
This will fix the issue


Generic RealtekCardReader

Hi,
I'm having a big problems with my USB3.0 card reader not being recognized.
My Laptop is Thinkpad T480
The reader is defined as: 0x0bda - 0x0316
I tried to edit the kext but it still doesn't get recognized
I also tried kext Generic, but to no avail.
how can i fix?
Thanks

Your card reader is not supported and will not be supported by this driver. In fact, it implements the UAS protocol, so in theory it should be natively supported by Apple’s USB driver if your USB ports are mapped properly. Please seek help on forums.


[BUG] New Drakes are not implamented

Describe the bug
If one of the Teams slain an Hextech Drake (maybe Chemtank aswell, atm disabled) and you activated the Objective Pop Ups (Spawn and/or Kill) it won't appear.
After a short View in the Folders its missing in the Path: Frontend\ingame\images\scoreboardPopUps\Dragon
btw nice Tool! Keep going!

They will be in the next version.
Temporary solution: https://github.com/floh22/LeagueBroadcast/issues/89


Now the unit id can be accessed on the server when responding a request

Now the unit id can be accessed on the server when responding a request by the call function. This is important for implementing gateways as one tcp or tls server can be a gateway for multiple modbus rtu servers.

Thanks. Yes, that's the case. SlaveRequest can be used to get the unit id.


Request: Following redirect URLs

Some of my pocket bookmarks are saved as t.co URLs from Twitter. So it's pretty difficult to find out what these URLs are at first glance without visiting them.
Would it be worth us making a HEAD request to the URL, checking for a redirect, and a "Location" header, then working with the forwarded URL instead?
I can probably tackle this problem if desired

Did you figure out a solution to this on your own? I'm curious as to how you ended up solving it.

Not as of yet, I have a hacky solution - but I was planning on implementing it properly and making a PR.
Not sure if many people encounter the same problem/it's worth solving though, hence the close.


"Regarding some issues with pretrained weights."

Hello! Dear author, it's a pleasure to read your excellent and meaningful work. However, I've noticed some issues when loading pretrained parameters during training.

mmsegmentation :
The model and loaded state dict do not match exactly
"Thank you for taking the time to reply amidst your busy schedule, I appreciate it."

Oh, by the way! Author, are the weight files open-sourced on your homepage for BiFormer_mm? Can they be directly loaded and used?


Serverpod Not Signing in using Google on Android

I have followed @Isakdl 's Medium Blog, Official Docs, and looked at all Open/Closed Issues, but Google Sign in is just not working. Please Note that I am only using Android.
To Reproduce
What I am doing -

I have added   serverpod_auth_server: 1.2.6 in server's pubspec, pod.webServer.addRoute(auth.RouteGoogleSignIn(), '/googlesignin'); in server.dart and ran serverpod generate, applied migrations and ran the web server.
I have defined modules: serverpod_auth:  nickname: auth in generator.yaml
I have added serverpod_auth_client: 1.2.6 in Client's pubspec
I have added serverpod_auth_shared_flutter: 1.2.6 and serverpod_auth_google_flutter: 1.2.6 in Flutter App's pubspec.
I have created a project in GCP and enabled People API
I have created Client ID for Web Application - Defined Authorized JavaScript origins as http://localhost:8082 and http://localhost:49660. Defined Authorized redirect URIs as http://localhost:8082, http://localhost:49660, http://localhost:8082/googlesignin.
I have downloaded the Web Application JSON and stored it as server/config/google_client_secret.json.
I have created Client ID for Android and defined SHA-1 certificate fingerprint. I have downloaded the Json file and stored it as talkinbird_flutter/android/app/google-services.json.
I am testing on emulator so my PC and my AVD both has the same IP.
I have tried using my own IP Address and <IP_ADDRESS> - Using my own IP Address takes me to OAuth Consent Screen and then give me error, Using <IP_ADDRESS> straight away gives me error after clicking on my gmail id.
I have tried using both Android's Client ID and Web Application's Client ID in SignInWithGoogleButton and found out that the Web Application one takes me to OAuth Consent Screen so I belive its the right one to use.
I have tried running  './gradlew clean', flutter clean and flutter pub cache repair and Invalidate Cache and Restart in Android Studio multiple times in between trying various solutions as mentioned above.

main.dart -
late SessionManager sessionManager;
late Client client;

void main() async {
  WidgetsFlutterBinding.ensureInitialized();

  const ipAddress = <My IP Address>;
  // const ipAddress = '<IP_ADDRESS>';

  client = Client(
    'http://$ipAddress:8080/',
    authenticationKeyManager: FlutterAuthenticationKeyManager(),
  )..connectivityMonitor = FlutterConnectivityMonitor();

  sessionManager = SessionManager(
    caller: client.modules.auth,
  );
  await sessionManager.initialize();

  runApp(const TalkinBird());
}

Sign In Screen -
import 'package:flutter/material.dart';
import 'package:serverpod_auth_google_flutter/serverpod_auth_google_flutter.dart';
import 'package:talkinbird_flutter/screens/user_details_ui.dart';

import '../main.dart';

const _serverClientId = "<Web Application's Client ID>";

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

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Center(
        child: Dialog(
          child: Container(
            width: 260,
            padding: const EdgeInsets.all(16),
            child: Column(
              mainAxisSize: MainAxisSize.min,
              crossAxisAlignment: CrossAxisAlignment.stretch,
              children: [
                SignInWithGoogleButton(
                  onSignedIn: () {
                    print('Signed in with Google!');
                    Navigator.of(context).push(MaterialPageRoute(
                      builder: (context) => const UserDetailsUI(),
                    ));
                  },
                  caller: client.modules.auth,
                  serverClientId: _clientId,
                  redirectUri: Uri.parse('http://localhost:8082/googlesignin'),
                ),
              ],
            ),
          ),
        ),
      ),
    );
  }
}

Error Logs right after the App is started on Android Emulator -
Syncing files to device sdk gphone x86...

Serverpod versions

CLI Version: 1.2.6
Library version: 1.2.6

Platform information
I am on a Windows Machine, using Android Studio and working on Pixel 8 API 30 Android Emulator.
I have defined org.gradle.jvmargs=-XX:MaxHeapSize=256m -Xmx2536m in gradle.properties and multidex support is enabled.
In my app/gradle - compileSdk is 34, ndkVersion "25.1.8937393", minSdkVersion is flutter.minSdkVersion, targetSdkVersion is flutter.targetSdkVersion
Here is the flutter doctor's outopt -
PS C:\talkinbird\talkinbird_flutter> flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.19.4, on Microsoft Windows [Version 10.0.22621.3296], locale en-IN)
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[√] Chrome - develop for the web
[√] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.7.6)
[√] Android Studio (version 2023.2)
[√] Connected device (4 available)
[√] Network resources

• No issues found!

Possible issues I see,
In the signin button you pass _clientid serverClientId and you are giving the clientId parameter. Also it seems that there is an error in the connection between the server and the client. I am not familiar enough with the android emulator and how it handles networking. You might need to investigate this further. But if passing the <IP_ADDRESS> ip address is valid you might also need to add this to the allowed hosts and redirect URLs in your gcp project.

According to documentation, clientID is not used in Android and I am passing the right arguement, I have named it clientID because that's what it's named in GCP.
It's clearly mentioned in the example directory and in some issues/ stackoverflow links that you have to use the IP address of your system.
<IP_ADDRESS> doesn't work. Also, if I go ahead and try to add http://<IP_ADDRESS>:8082/ in redirect URL, it doesn't work. Adding http://:8082/ also doesn't work. GCP Console says "Invalid Origin: must end with a public top-level domain (such as .com or .org)" which doesn't seem to be the case with localhost urls.
I also tried testing on real physical device on the same IP as my system and that doesn't work either.

I have read up a bit on the android emulator here and it tells me the ip address of the emulator is <IP_ADDRESS> instead of <IP_ADDRESS>. I actually think you got further in the process when you used the computer's local ip address.  I hope this resolves that issue. I don't know how it will behave with regards to the GCP allowed origins for which I will need more information.
If my information of the <IP_ADDRESS>/24 address space is outdated can you show me when it has changed.
Then I think if the <IP_ADDRESS> ip address doesn't work try again with the local ip and send logs of both the flutter and serverpod application

Read my Point No. 10 at the top level comment to see the difference between using real IP address and 10.0.02 . Even if it works at <IP_ADDRESS>, it doesnt resolve my issue since the app is intend to work on real device.
These are the places its mentioned to use a real IP for android. These images are from Part 1 and Part 2 of the article mentioned in docs.

You haven't read my reply thoroughly. I indicate that <IP_ADDRESS> is not in the IP address space of the android emulator. <IP_ADDRESS>/24 is with the default gateway at <IP_ADDRESS> which is connected to your desktop. Il put them below eachother:
<IP_ADDRESS> (the one you are currently using)
<IP_ADDRESS> (the one you should be using according to the documentation I found)
Either way I now suggest you use the actual ip-address of the computer (the Ip-address on your local network) instead of the <IP_ADDRESS> as it more closely resembles the real world use case. This should resolve the http time out errors. They were probably caused by the IP you gave not being connected to an actual device in the virtual network.
To resolve the other issues we need more data. I will need the logs of your flutter application and your serverpod backend when you make the calls.


try to find a datetime type to match postgres SQL timestamp type

I create a Struct with a variable named "created_at", corresponding to a column "created_at" (data type: timestamp) in postgres SQL.
however, I couldn't compile it in rust.
I used the exmaple "serverfn_sqlx", just changed the databese and Struct .
The original "serverfn_sqlx" example works well.
BTW,  it would be better to have an example showing 2 sections on the webpage: (like CRUD with javascript)
top section--- insert a new record into a table inside SQL database.
bottom section --- showing table records, automatically referesh after a new record being inserted into the table. (create a resource ? )
I could use server action to get table records, but don't know how to implement 'TableDataProvider'.
(I'm a non-IT guy)
Thanks.
Yingwei

Well that is one of several possibilities to create a CRUD interface.
What error do you get when you try to compile?
Also when you really don't know programming then I wouldn't recommend using Rust as your first programming language. It is awesome but has a steep initial learning curve.
Maybe start with Python or JavaScript instead.

Thanks for your advice, I'll think about Python.


Update restify to v5

Is this likely to be merged any time soon? It's just that is is breaking the tests for https://github.com/kwhitley/apicache.

Thanks!

Thanks @svozza & @zephrax!!! One step closer to solving the Node 7.7+ and Restify conflict in apicache... getting so much (rightful) flak on not having a release out yet!


Publish to package manager

It would be great if we could just install this from something like apt-get.
I would like to do the following steps:

sudo apt-get install wait-for-it
wait-for-it www.google.com:80 -- echo "google is up"

If you would like to volunteer to package wait-for-it.sh, that would be great -- in the meantime, you can always use the raw file link https://raw.githubusercontent.com/vishnubob/wait-for-it/master/wait-for-it.sh in your deployment scripts.

That is good advice as long as you dont change the parameters to the script on master.  Eventually publishing it to a package manager will give you greater flexibility without breaking things for your (very thankful) consumers.
I will setup my project to pull from that link.  I wont be able to setup the packaging for a bit, so I will leave this open for anyone for now.  I will pick it up once I get some time.

Well, even with a package manager, you would have to introduce the notion of a release. Like saying that today's version of master is v1.0.0. Then you could create a package (rpm, deb, ...) for v1.0.0.
How do you know when to create v1.0.1 (or should it be v1.1.0?) - someone will have to decide when to call something a release. Since this repo currently does not use the concept of versions/releases, it's going to be hard to map that to a package manager.

Would be great to have it in apk too: http://wiki.alpinelinux.org/wiki/Creating_an_Alpine_package

Hey guys.
Debian Developper here.
Unless one of you guys wanted to do it, I'll package wait-for-it for Debian and publish it next week. Maybe tuesday.

Oh well. I had time today.
I have uploaded wait-for-it to Debian. It should be available in a couple of days.

https://tracker.debian.org/pkg/wait-for-it

Nice!
On Tue, Aug 2, 2016 at 1:00 PM Alexandre Viau<EMAIL_ADDRESS>wrote:

https://tracker.debian.org/pkg/wait-for-it
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/vishnubob/wait-for-it/issues/3#issuecomment-236988442,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AASIT2qzba2UA4BlwkMDvGigNzYGVs-8ks5qb4WygaJpZM4H7Cpa
.

Will the package be available in Jessie at some point? or is it impossible or is it impossible to include it?

It could, but it would would never be in the main repositories, it would be in jessie-backports. However, I don't intend to spend time doing that.

This is a neat tool, has anyone tried to get it into EPEL?

sudo apt update && sudo apt install wait-for-it worked for me.

sudo apt update && sudo apt install wait-for-it worked for me.

That's on Debian systems. I'm looking for Fedora/CentOS.


  Doing keyshare reqs doesn't fix them, which is weird.  EW should be fishing the megolm sessions out of my online backup, too, so it sounds like something bad is happening here specific to SS.
Close this issue when: We have a test case which sends E2EE messages and then logs into SS proxy the first time to verify key backup works.

We have this, 1 year later.. via complemet-crypto.


Search bar colors in light/dark mode

Are you experiencing an issue with...
shields.io
🐞 Description
I'm seeing some inconsistencies with the coloring of elements in light/dark mode.
Light mode:

Dark mode:

For reference, this is what the https://docusaurus.io/ website search bar looks like.
Light mode:

Dark mode:

The search icon (magnifying glass) in the search field is a dark color in light mode but doesn't switch to a lighter color in dark mode.

The placeholder text in light mode seems a little too faint to be properly readable. It looks fine in the screenshot above but take a look for yourself and tell me what you think!

I think this is really an issue with https://github.com/easyops-cn/docusaurus-search-local
You can reproduce the same behaviour on https://easyops-cn.github.io/docusaurus-search-local/
I think the relevant upstream issue for the icon is https://github.com/easyops-cn/docusaurus-search-local/issues/336


[Base] Accept callbacks in componentsProps.*

The componentsProps fields currently accept plain objects, that are spread to corresponding slots' props:
<InputUnstyled
  componentsProps={{
    root: { className: 'custom-root' },
    input: { className: 'custom-input' },
  }}
/>

This approach makes it harder to apply slots' props conditionally, based on the state of the unstyled component. The only way it's possible right now, is by providing a custom slot component with the logic that determines the props to set.
This is unnecessarily verbose. A much simpler way (originally proposed in https://github.com/mui/material-ui/pull/30700#discussion_r788279436) would be to accept functions in componentsProps' fields. Such a function would be called with ownerState as the parameter:
<InputUnstyled
  componentsProps={{
    root: ({ focused }) => ({ className: focused ? 'custom-focus' : '' }),
    input: ({ error }) => ({ 'data-error': error }),
  }}
/>

cc @flaviendelangle

The only way it's possible right now, is by providing a custom slot component with the logic that determines the props to set.

For context, the assumption in https://github.com/mui/material-ui/issues/21453 is that it would be enough. But I agree that for using the current API with utility classes (e.g. Tailwind CSS), it becomes very challenging. In https://headlessui.com/react/tabs they use a render function to solve this problem:
        <Tab as={Fragment}>
          {({ selected }) => (
            <button
              className={
                selected ? 'bg-blue-500 text-white' : 'bg-white text-black'
              }
            >
              Tab 1
            </button>
          )}
        </Tab>

We have on different problem solved by the callbacks in componentsProps.* which is the mobile / desktop props on the pickers.
By the way @alexfauquette we did not implement the callback at the same level
<DatePicker
  componentsProps={{
    actionBar: {
      // The actions will be the same between desktop and mobile
      actions: ['clear'],

      // The actions will be different between desktop and mobile
      actions: (variant) => (variant === 'desktop' ? [] : ['clear']),
    },
  }}
/>

In https://headlessui.com/react/tabs they use a render function to solve this problem

It's not possible to modify the Tab's props this way. This could only work for composite components, so it's not really the same problem we're solving with componentsProps

I wonder because the problem we solve seems to be all about class names. I mean, the concrete problems we refer to in this GitHub issue so far are https://github.com/mui/material-ui/pull/30700#pullrequestreview-857544962 and https://headlessui.com/react/tabs, all about the class attribute.

I've seen requests from the community to enable componentsProps on Material UI components and the motivation was not customization of classes (#33018). I wouldn't drop componentsProps altogether. I agree that using classes, as you proposed, would be simpler and require less code. I would not replace componentsProps with classes, though. We could think of adding the classes prop if it the community thinks it would be beneficial. The downside of this is that we'd have yet another way of customization that we'll need to support.

I'm closing this issue as we've got all the existing Base components covered. Let's discuss other changes in a new issue if needed.


Update promises.json

Updated the year prefix to 17: for both nodes inside "basics".

Sorry about the confusion. But the basics nodes have steps 1. 2. and so on.
And articles, talks and videos that are note in basics node may have a year prefix attached.
I do think that it is a bit unclear on our side though and we will need to make it more clear. Perhaps you have an idea how we can do it. 🙂
Basics should have steps because we want learning to be linear. You learn this first, then learn this, covering any knowledge dependencies on your way. And the year prefixes just add context to articles and blogs because it's good to know if an article/video is not outdated.

Thanks Nikita for the explanation.
So, the article inside basics currently labelled 1. and the video tutorial playlist that I added (originally with 2.) are both covering the basics. They just are done in different formats and with a slightly different approach.
So, how do you decide on the numbering then? Does it matter?
How long does it take for another item to appear?  Eg: The JS Promises video playlist that I added yesterday.
Thanks,
Steve

From: Nikita Voloboev<EMAIL_ADDRESS>Sent: Wednesday, September 13, 2017 9:02:33 PM
To: learn-anything/maps
Cc: Steve Griffith; Author
Subject: Re: [learn-anything/maps] Update promises.json (#1202)
Sorry about the confusion. But the basics nodes have steps 1. 2. and so on.
And articles, talks and videos that are note in basics node may have a year prefix attached.
I do think that it is a bit unclear on our side though and we will need to make it more clear. Perhaps you have an idea how we can do it. 🙂
Basics should have steps because we want learning to be linear. You learn this first, then learn this, covering any knowledge dependencies on your way. And the year prefixes just add context to articles and blogs because it's good to know if an article/video is not outdated.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/learn-anything/maps/pull/1202#issuecomment-329338900, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABJQyK2XcKLdy5ENbd2w9LKB0n8cNwCEks5siHsogaJpZM4PW6Ro.

The numbering is decided by the community. The idea is that step 2. needs some knowledge from what was covered in 1. steps. In your case, it falls under 1. too as it explains promises too same as article linked there but in a different (video) approach and the explanation was really good so it was added.
The Javascript Promises map is updated. 🙂

Thanks!

From: Nikita Voloboev<EMAIL_ADDRESS>Sent: Thursday, September 14, 2017 8:53:31 AM
To: learn-anything/maps
Cc: Steve Griffith; Author
Subject: Re: [learn-anything/maps] Update promises.json (#1202)
The numbering is decided by the community. The idea is that step 2. needs some knowledge from what was covered in 1. steps. In your case, it falls under 1. too as it explains promises too same as article linked there but in a different (video) approach and the explanation was really good so it was added.
The Javascript Promiseshttps://learn-anything.xyz/programming/programming-languages/javascript/promises map is updated. 🙂
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/learn-anything/maps/pull/1202#issuecomment-329472387, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABJQyA9g6lDZLRS7uwNzUUwbl7dZ0ojQks5siSHLgaJpZM4PW6Ro.


Very long documents are not captured

If the number of tr exceeds 3000, the canvas data is returned as "data :,".
And an empty image is added. Can anyone help?
this is jsfiddle url.
https://jsfiddle.net/ahmax/4goc5616/

i have same problem!


Need to separate private key wallets and HD wallets.

This has been discussed in issues #221, #516, #396. I'm not sure why the wallet still allows mixing of private keys and HD child accounts.
Right now if I make an HD wallet with a passphrase from scratch, I can create child accounts. When I go to create a child account, it gives me the choice to enter a private key or create an account deterministically. There should ONLY be the option for creating an account deterministically. There should NOT be an option for importing a private key at this part.
On the sign up screen, where a person is given the option to make a phase phrase, restore a passphrase, or login with Ledger, there is also at the bottom a chance for people to create a private key wallet. That is the only place in the wallet IMO where a person should be entering a private key.

When I'm in a wallet that was started by a passphrase and I'm making a child account, there should be no option to choose if I'm going to make the child account deterministically or via private key. It should only be via deterministically.

mixing private keys and deterministic keys in one profile means that when a person backs up their passphrase, any accounts that were added to that profile not with the passphrase are not getting recreated when that passphrase is reimported on a new machine. It seems like common sense but many users don't get how HD wallets work. They think that when they import a private key in that wallet, it is now protected with the passphrase they have. In the past people have then trashed their private keys and only kept their passphrase. Yes.... I know that sounds silly but it has caused lost funds and many complaints before.

When somebody is in a private key wallet that was made via the the intended "import private key" option shown below which is in the create new profile section, there needs to be further discussion.

The point of discussion is as follows.  Once a person has imported a private key and now they are in their dashboard and go to accounts and create a new account, should the child accounts ONLY come from importing additional private keys, or should they be created from ONLY creating HD child accounts from the original imported private key?

I, for one, have absolutely no idea what a "High Definition" wallet is, or what is stored in a mnemonic phrase. I have played with the wallet and didn't understand the different "kinds" of accounts there are. That needs some serious documentation work.

Hierarchical Deterministic Wallet (HD Wallet) is a where all the private keys in a private key are derived deterministically from a random set of words which is often in this industry 12-24 words.
In the start of blockchain, almost everyone managed profiles by creating random private keys. This was problematic as a person had to manage and record and back up lots and lots of private yes. At some point most of the industry moved over to HD wallets. The benefit of an HD wallet is it allows "one key to rule them all" and storing the key as a set of words instead of 64 character hexadecimal as 12 words has a lot less user error writing it down and backing up then 64 characters.
By mixing a profile with an HD passphrase and a string of characters now means a person needs to back up both separately, but we have no clean and easy way to back up both at the same time, nor have I ever seen any wallet pull that off. It definitely can't be put in a QR code like we are doing now. This leads to people thinking they have backed up accounts and they haven't when the two systems are mixed.

@jabo38 could you verify current behavior?

Right now hierarchical deterministic wallets allow to add a custom private key address. As described in this issue, this is not a good idea. Reopen this issue?

Right now hierarchical deterministic wallets allow to add a custom private key address. As described in this issue, this is not a good idea. Reopen this issue?


fix function signature

Buffer should be a better type for get_raw_bodys.

Codecov Report

Merging #162 (d74863c) into master (2977f80) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #162   +/-   ##
=======================================
  Coverage   89.24%   89.24%
=======================================
  Files           7        7
  Lines         186      186
  Branches       20       20
=======================================
  Hits          166      166
  Misses         20       20

Flag
Coverage Δ

node-12.x
89.24% <ø> (ø)

node-14.x
89.24% <ø> (ø)

node-16.x
89.24% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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


Added capability to add arguments to exchanges

I needed to add arguments to RabbitMQ exchanges, so I added capability to the module to handle the arguments.

could you please document this new parameter in the README, and then squash the commits down to one.

documented and squashed

Hi There
Now that #328 is merged, can you please confim if you still need this? If so, please perform a rebase.
Thanks!


Feature upgrade qt 5 12 version

https://www.appveyor.com/docs/windows-images-software/#qt

Please use imperative mood in your commit summaries.


Is GPU processing a prerequisite?

Hi. First of all, thanks a lot for your work.
I am about to try your scripts and I wanted to now if I could run everything without using GPUs. I have a server with 40 CPUs. Will it run?

Yes it runs on CPU, especially if you already have a model and just want to do predictions. But I think you will save on your electricity bill by renting a machine with GPU on Amazon with spot price (as low as $0.08/yr) and saving the model after every iteration to S3 (so you can continue your work if you happen to lose the machine because of spot price changes.)
If you are using Theano you can configure it to use CPU for example:
import os
os.environ['THEANO_FLAGS'] = 'device=cpu,floatX=float32'

Hello @udibr, I've reading about AWS using and I have decided to do what you said. Is there any special AMI with pre-installed dependencies that you would recommend to run your project?
Thanks!

go to "Community AMIs and search for CUDA

thanks for answering @udibr  Do you mean I should search in "shared AMIs"? i just want to be sure (I don't see the word 'Community' there)

https://console.aws.amazon.com/ec2/v2/home?region=us-east-1#LaunchInstanceWizard:

Just in case anyone runs across this issue AWS has launched a new Deep Learning AMI with everything you need to get started running this project.

https://aws.amazon.com/blogs/ai/the-aws-deep-learning-ami-now-with-ubuntu/


test: test deploy

Checklist

[ ] Fix linting errors
[ ] Tests have been added / updated (or snapshots)

Change information

Codecov Report

Merging #173 (b6c621a) into dev (f38aa0e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev     #173   +/-   ##
=======================================
  Coverage   96.43%   96.43%
=======================================
  Files         206      206
  Lines        4654     4654
  Branches      968      968
=======================================
  Hits         4488     4488
  Misses        164      164
  Partials        2        2

Continue to review full report at Codecov.

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


Add dbc package for melodic

Hi all, I already solved it temporarily with a .rosinstall file. But I thought I'd add it to rosdistro so others can benefit as well.
I was under the assumption rosdep could handle source-installs.
So if that's not the case I don't need this PR. And I won't debian-release other peoples packages 😄

I'll leave the name-suggestion request open though, these are still valid suggestions to change to a more descriptive name.


middleware close implementation (improvement)

Hi!
In my project, I need to be able to close connection to clients from the server in runtime.
Technically, I need to do something like
everyClient(function(client) {
   client.end("disconnect");
});

and clearInterval. So no more active connections and no leaking timers any more.
Would you accept such pull request?

I'd be open to something like this, is this the same as https://github.com/glenjamin/webpack-hot-middleware/issues/176 ?
What scenario prompted you to need this?

Yes, looks like what I need.
I need to add and remove webpack build configuration in runtime. So I can stop webpackDevMiddleware watch changes using close(), but can't stop hotMiddleware correctly.
Removing plugin handlers does not make sense for me, but I think it is good to have feature as well.


[Event Request] Need event in function WhseLinesExist cudeunit 5777

Add please event OnBeforeSetSourceFilter in function WhseLinesExist
procedure WhseLinesExist(SourceType: Integer; SourceSubType: Option; SourceNo: Code[20]; SourceLineNo: Integer; SourceSublineNo: Integer; SourceQty: Decimal): Boolean
var
    WhseRcptLine: Record "Warehouse Receipt Line";
    WhseShptLine: Record "Warehouse Shipment Line";
    WhseManagement: Codeunit "Whse. Management";
begin
    if not WhseRcptLine.ReadPermission then
        exit;
    if ((SourceType = DATABASE::"Purchase Line") and (SourceSubType = 1) and (SourceQty >= 0)) or
       ((SourceType = DATABASE::"Purchase Line") and (SourceSubType = 5) and (SourceQty < 0)) or
       ((SourceType = DATABASE::"Sales Line") and (SourceSubType = 1) and (SourceQty < 0)) or
       ((SourceType = DATABASE::"Sales Line") and (SourceSubType = 5) and (SourceQty >= 0)) or
       ((SourceType = DATABASE::"Transfer Line") and (SourceSubType = 1))
    then begin
        WhseManagement.SetSourceFilterForWhseRcptLine(WhseRcptLine, SourceType, SourceSubType, SourceNo, SourceLineNo, true);
        if not WhseRcptLine.IsEmpty then begin
            TableCaptionValue := WhseRcptLine.TableCaption;
            exit(true);
        end;
    end;

    if ((SourceType = DATABASE::"Purchase Line") and (SourceSubType = 1) and (SourceQty < 0)) or
       ((SourceType = DATABASE::"Purchase Line") and (SourceSubType = 5) and (SourceQty >= 0)) or
       ((SourceType = DATABASE::"Sales Line") and (SourceSubType = 1) and (SourceQty >= 0)) or
       ((SourceType = DATABASE::"Sales Line") and (SourceSubType = 5) and (SourceQty < 0)) or
       ((SourceType = DATABASE::"Transfer Line") and (SourceSubType = 0)) or
       ((SourceType = DATABASE::"Service Line") and (SourceSubType = 1))
    then begin
        OnBeforeSetSourceFilter(SourceType,SourceSubType,SourceQty)

        WhseShptLine.SetSourceFilter(SourceType, SourceSubType, SourceNo, SourceLineNo, true);
        if not WhseShptLine.IsEmpty then begin
            TableCaptionValue := WhseShptLine.TableCaption;
            exit(true);
        end;
    end;
end;

Event:
[IntegrationEvent(false, false)]
local procedure OnBeforeSetSourceFilter(SourceType: Integer; SourceSubType: Option; SourceNo: Code[20];SourceQty: Decimal)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Allow mount method to attach to specific HTML Element rather than generically attaching to the document

What problem does this feature solve?
This allows the developer to attach a component within the context of a page, and potentially allows for more integrated testing, e.g. via puppeteer.
What does the proposed API look like?
Currently you offer this option in the API:
https://vue-test-utils.vuejs.org/api/options.html#attachtodocument
I believe it should be expanded to allow specification of where in the document it is attached.
const wrapper = mount(Component, {
  attachTo: document.getElementById('app'),
})

This is roughly equivalent to the API used by Enzyme
https://enzymejs.github.io/enzyme/docs/api/mount.html

Could you explain a bit what use cases will this solve?

Let's say your Vue app exists within the context of a larger page. Only one part of your app is Vue. Another part is just static HTML, another part is react.
Does it render correctly?
I can do this by just rendering via Vue and taking screenshots with puppeteer, but then I'm not able to use all of the convenience methods of Vue test utils.

Another use case could be to test that hydration of a SSR paged works successfully. I can't use Vue test utils, as far as I know, to hydrate markup. That could be very beneficial, and, potentially, part of this API.

Another alternative could be that it simply checks for attachToDocument to be either a boolean or HTMLElement, and in the case of the latter uses that element instead of creating a new one

This makes sense to me. There is a PR; just need to docs to be updated @jnields and we can merge it up.

fixed in #1492


[ADLA] - Adding existing Catalog ACL APIs

Adding six existing ACL APIs for Catalog -- thus this PR is to fill in the missing gap.

Since they were existing, we could not give feedback about the usage of OP to define the call.  We will work on possibly removing this and redesigning it in a new API version for Catalog, which is in progress.

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[x] If applicable, the PR references the bug/issue that it fixes.
[x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[x] My spec meets the review criteria:

[x] The spec conforms to the Swagger 2.0 specification.
[x] The spec follows the guidelines described in the Swagger checklist document.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

No modification for AutorestCI/azure-sdk-for-ruby

 https://github.com/Azure/azure-sdk-for-python/pull/1608

No modification for AutorestCI/azure-sdk-for-node


Build error on Android?

App Gradle:
buildscript {
    repositories {
        maven { url 'https://plugins.gradle.org/m2/' } // Gradle Plugin Portal
    }

    // Added for QR
    apply plugin: 'com.android.application'
    apply plugin: 'kotlin-android'

    dependencies {
        implementation "org.jetbrains.kotlin:kotlin-stdlib-jre7:$kotlin_version"
        classpath 'gradle.plugin.com.onesignal:onesignal-gradle-plugin:[0.10.2, 0.99.99]'
    }
}

Android Gradle:
buildscript {
    ext.kotlin_version = '1.2.31'

    repositories {
        google()
        jcenter()
    }

    dependencies {
        classpath 'com.android.tools.build:gradle:3.2.1'
        classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:$kotlin_version"
    }
}

My error:

What went wrong:
A problem occurred evaluating project ':app'.

Could not find method implementation() for arguments [org.jetbrains.kotlin:kotlin-stdlib-jre7:1.2.31] on object of type org.gradle.api.internal.artifacts.dsl.dependencies.DefaultDependencyHandler.

Anyone has any idea? Thanks!

there are two kinds of dependencies blocks. One is for dependencies for the build tools, that's the one in the buildscript block. The other is the dependencies for the app you're building.
implementation is only for the second kind.
The problem is caused by you adding
implementation "org.jetbrains.kotlin:kotlin-stdlib-jre7:$kotlin_version"
do the buildscript dependencies block. it should be outstide of the buildscript block.
also, you shouldn't be applying the plugins inside of the buildscript block.
I think what you want is something more like
app/build.gradle:
buildscript {
    repositories {
        maven { url 'https://plugins.gradle.org/m2/' } // Gradle Plugin Portal
    }

    dependencies {
        classpath 'gradle.plugin.com.onesignal:onesignal-gradle-plugin:[0.10.2, 0.99.99]'
    }
}
apply plugin: 'com.android.application'
apply plugin: 'kotlin-android'

dependencies {
        implementation "org.jetbrains.kotlin:kotlin-stdlib-jre7:$kotlin_version"
}

According to the above configuration, the operation still reports an error.
/build.gradle

buildscript {
    ext.kotlin_version = '1.2.31'
    repositories {
        google()
        jcenter()
    }

    dependencies {
        classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:$kotlin_version"
        classpath 'com.android.tools.build:gradle:3.2.1'
    }
}

/app/build.gradle

dependencies {
    implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk7:$kotlin_version"
    testImplementation 'junit:junit:4.12'
    androidTestImplementation 'com.android.support.test:runner:1.0.2'
    androidTestImplementation 'com.android.support.test.espresso:espresso-core:3.0.2'
}

error info

Try to update to the latest version of the package.


Works only on postgres db?

I tried making it work on another db inside the same container but I can't seem to select the extension 😕

Can you give details about how you're starting the container, and how you're configuring the extension? This should be possible to accomplish, but I'll need a few more details.

Same issue. I've got multiple Timescale databases and would like to try out pg_timeseries project.
Started the docker container as in instructions and managed to install the extensions into postgres database

Then creating a new database "test" and trying to install the extension there:

According to pg_cron docs one should schedule jobs in another database by
SELECT cron.schedule_in_database('weekly-vacuum', '0 4 * * 0', 'VACUUM', 'some_other_database');

Presently installing in other database than where pg_cron is installed is not possible?

Thanks for an interesting project!


Issue with connecting to Mongodb

Hi,
I just installed cyclotron on OSX El Capitan  and I am trying to run app.js from inside cyclotron-svc. I keep getting this error related to connecting to port 27017. I have checked using netstat that mongod is running and listening on port 27017.
Thanks in advance!
Needs latest SSL Root Certificate Authority data /Users/sriram/Documents/dashboards/cyclotron/cyclotron-svc/node_modules/ssl-root-cas/ssl-root-cas-latest.js
Loading latest certificates from https://mxr.mozilla.org/nss/source/lib/ckfw/builtins/certdata.txt?raw=1
Cyclotron running on port 8077

/Users/sriram/Documents/dashboards/cyclotron/cyclotron-svc/node_modules/mongoose/node_modules/mongodb/lib/server.js:242
        process.nextTick(function() { throw err; })
                                      ^
Error: connect ECONNREFUSED ::1:27017
    at Object.exports._errnoException (util.js:1033:11)
    at exports._exceptionWithHostPort (util.js:1056:20)
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1105:14)```

Hi @ssriram1, I think this is related to #13, with the service binding to IPv6 instead of IPv4.  Can you try making the following change at the very bottom of cyclotron-svc/app.js:
app.listen(port, '<IP_ADDRESS>', function(){
    console.log('Cyclotron running on port %d', port);
});

Hi @ssriram1, I'm looking at this again and I realized I totally misread your issue.  I saw ::1:27017 which looks like an IPv6 issue, and thought it was the same as the other issue.
Maybe localhost isn't resolving on your machine, so you can try updating the Mongo config in config/config.js with the actual IP address:
mongodb: 'mongodb://<IP_ADDRESS>:27017/cyclotron',

Does the Mongo shell connect successfully as well?


Check on multiple states in one intent

Now that we have binary_sensor support, I wanted to create intents to check if all doors/windows are closed
However, I ran into the problem that there are also covers which are doors/windows
So to check if all doors are open, we need to check for covers which are open (and technically also which are opening and closing) but also for binary_sensors which are on (all with device_class: door)
Is there a way to check on multiple states?

@synesthesiam Could this somehow be added, also see #1200 how this can help in resolving the current conflicts

I made  suggestion in https://github.com/home-assistant/intents/discussions/1200 but I haven't had time to think it through very much :slightly_smiling_face:


Mod no longer working in newest version of Among Us

Among Us update seems to have broken the mod!

(v2020.12.3)

Created pull request with working version for newest Steam release.
#15

Ok, so Among Us has just got a next new update. I'm working on rewriting class definitions yet again, I will probably make a PR for that tomorrow.

Created yet another #16 to support newest version from Steam.

Outdated. I suggest closing.
Latest version of CSC supports 2020.12.9s, but seems like the release is broken.

Now it works ig.


Address collection of too many Robjs

This has been discussed extensively: #397, and other places.
PR does two things:

Addresses changes in the latest version of libR-sys, that might have broken extendr.
Refactors FromIterator-impls for Strings and List. R cannot handle protecting more
than 10k elements at a time, but protecting one element, that contain more than 10k is no issue.
Thus intermediate objects are no longer pre-allocated into Robj, and then assigned to Strings / List resulting Robj.
Instead immediate objects are written directly into the object.

Furthermore there was an inconsistency before where Rf_unprotect was set to n, but it would
incidentally unprotect the result Robj, and leaving the first intermediate Robj.
This code might have affected performance, and thus benchmarking this should be done in a follow-up PR.
Problem is this didn't work for some cases, and thus it is more important to make it functional, than say blazingly fast.
Follow-up PR

[ ] Currently, there is no way to get an unprotected Robj from a rust-compatible type.
We should do that, as the batched allocations in FromIterator shouldn't use ownership-module
but rather rely on allocating and unprotecting its own intermediaries.
[x] Add a gc-torture test to see if our protection mechanisms work.

Context: Turns out this didn't have anything to do with extendr's ownership module: Using r-bindings directly, i.e. without extendr:
  let total_protected = 10_000 * 6;
  let dummy_string = "Jamie Foxx";
  for _p in 0..total_protected {
    unsafe {
      Rf_protect(Rf_mkCharLenCE(
        dummy_string.as_ptr() as _,
        dummy_string.len() as _,
        cetype_t::CE_UTF8,
      ));

This would break.

Please see Discord thread for reproducible examples

I would also like to change this:
https://github.com/extendr/extendr/pull/398#discussion_r847264770
I've removed it and the test that was present in
extendr\extendr-api\tests\issue-397-memory-allocation.rs
works fine with this PR.

Suggested edit:
diff --git a/extendr-api/src/ownership.rs b/extendr-api/src/ownership.rs
index 759303cc..a70a8fe0 100644
--- a/extendr-api/src/ownership.rs
+++ b/extendr-api/src/ownership.rs
@@ -158,7 +158,7 @@ impl Ownership {
     // Garbage collect the tracking structures.
     unsafe fn garbage_collect(&mut self) {
         // println!("garbage_collect {} {}", self.cur_index, self.max_index);
-        let new_size = self.cur_index * 2 + EXTRA_PRESERVATION_SIZE;
+        let new_size = self.cur_index + EXTRA_PRESERVATION_SIZE;
         let new_sexp = Rf_allocVector(VECSXP, new_size as R_xlen_t);
         R_PreserveObject(new_sexp);
         let old_sexp = self.preservation as SEXP;

I think this works because of:

Protecting an R object automatically protects all the R objects pointed to in the corresponding SEXPREC, for example all elements of a protected list are automatically protected.
https://cran.r-project.org/doc/manuals/R-exts.html#Garbage-Collection

Although there is a point at which we have to PROTECT each newly-created Robj before it gets put into the List, and I'm worried that we're not doing that.

From reading R-extss, it sounds like it might be okay to not PROTECT each object as long as we can guarantee that no memory allocations or anything that might trigger the GC happens before it gets put into the list.

I'm immediately passing the intermediate objects to a protected vector. Isn't that good enough?
Note that the resulting vector is protected by the ownership module.

I think it's okay as long as you can guarantee that the GC will not run between the point of you creating the Robj and the point of you putting it in the list. Even SET_VECTOR_ELT might somehow force GC.

Right. For the List version I need to protect, assign to the list, and unprotect(1).

Yeah, I think so? It might incur a small performance penalty but it will be safer. Also it won't be anywhere near as bad as the old way because we do it one at a time.

@multimeric I don't think I need to do that.
This line let item: Robj = v.into(); presumably invokes
impl<T> From<T> for Robj
where
    T: ToVectorValue,

Which takes the scalar, wraps it in something that is IntoIterator, and then calls .collect_robj().
This in turn calls fixed_size_collect which uses Robj::alloc_vector [extendr ownership module]
The only "bad" thing about the List's FromIterator impl is that it would be nice if it sidesteps
ownership-module as to not clog up the rust gc with something intermediary.
If you guys can suggest something now for that, then we can get in, otherwise I believe that's better suited for
a follow-up PR.

I'm still running into this issue. I'd love to be able to collect more objects from lists. This is a limiting factor in the use of extendr. :/
Is there anything I can (try) to do to help this get merged?

FWIW to prevent this issue: one cannot collect into a list but rather should build the list iteratively using List::new() and .set_elt()
See https://github.com/JosiahParry/sfconversions/commit/d3ad0f30445e9a473c1c6343300a19a416c168f4 for example of required refactor.

I'm tending to the comments here now. For what it is worth, I believe List::from_values is also safe to use.

I'm tending to the comments here now. For what it is worth, I believe List::from_values is also safe to use.

Would collecting into Vec<T> then List::from_values() cause an additional / unwanted allocations or anything that might bloat memory or slow it down?

You're very correct. The idea would be to pass an interator to from_values, thus side-stepping the Vec<T> collecting / allocation.

@Ilia-Kosenkov I've addressed the comments by @multimeric. I have also had this approved. Would you mind giving this a sanity check and merging?

Looks ok to me, but let's resolve conflicts and run latest CI to be absolutely sure

run latest CI to be absolutely sure

I think you'll probably hit this error, too.
https://github.com/extendr/extendr/pull/586#issuecomment-1637495135

I ask @Ilia-Kosenkov to kindly give this a look and please merge it if you're satisfied.


Tado fix

Description:
Updated reference to PyTodo.
Fixed Tado component - updated reference to PyTado==0.2.1 from PyTado==0.1.10.
Related issue (if applicable): fixes #
fixes #8010
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>
Example entry for configuration.yaml (if applicable):

Checklist:
If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

Hello @nvanggaard,
When attempting to inspect the commits of your pull request for CLA signature status among all authors we encountered commit(s) which were not linked to a GitHub account, thus not allowing us to determine their status(es).
The commits that are missing a linked GitHub account are the following:

abb3ee34b981e759d45a5765c62f4148e3c79772 - This commit has something that looks like an email address (nvanggaard@gmail.com). Maybe try linking that to GitHub?.
3151bb3fbf9e4da93e91c677010a64c80f260573 - This commit has something that looks like an email address (nvanggaard@gmail.com). Maybe try linking that to GitHub?.

Unfortunately, we are unable to accept this pull request until this situation is corrected.
Here are your options:

If you had an email address set for the commit that simply wasn't linked to your GitHub account you can link that email now and it will retroactively apply to your commits. The simplest way to do this is to click the link to one of the above commits and look for a blue question mark in a blue circle in the top left. Hovering over that bubble will show you what email address you used. Clicking on that button will take you to your email address settings on GitHub. Just add the email address on that page and you're all set. GitHub has more information about this option in their help center.

If you didn't use an email address at all, it was an invalid email, or it's one you can't link to your GitHub, you will need to change the authorship information of the commit and your global Git settings so this doesn't happen again going forward. GitHub provides some great instructions on how to change your authorship information in their help center.

If you only made a single commit you should be able to rungit commit --amend --author="Author Name<EMAIL_ADDRESS>
(substituting Author Name and<EMAIL_ADDRESS>for your actual information) to set the authorship information.
If you made more than one commit and the commit with the missing authorship information is not the most recent one you have two options:

You can re-create all commits missing authorship information. This is going to be the easiest solution for developers that aren't extremely confident in their Git and command line skills.
You can use this script that GitHub provides to rewrite history. Please note: this should be used only if you are very confident in your abilities and understand its impacts.

Whichever method you choose, I will come by to re-check the pull request once you push the fixes to this branch.

We apologize for this inconvenience, especially since it usually bites new contributors to Home Assistant. We hope you understand the need for us to protect ourselves and the great community we all have built legally. The best thing to come out of this is that you only need to fix this once and it benefits the entire Home Assistant and GitHub community.
Thanks, I look forward to checking this PR again soon! :heart:

Email updated

Duplicate of #8041.


Any pretrained model on  Mapillary Vistas Dataset

Hi, This repository seems very interesting  .Any pretrained model on  Mapillary Vistas Dataset  is available?
Thanks

Hello @robi56 ,
Unfortunately we did not release any pre-trained model, so as to respect their Terms of Use (see details on the dataset website if you have not visited it yet).
Our advice here is to request Mapillary data on your side and train your own model, if possible. (Sorry if it looks unsatisfying).
Regarding other datasets, that is a good point, and we can plan to check if it possible in the next days.

@robi56 Should you be interested in a pre-trained model, we can work with Mapillary to get a commercial licence for the data and sell you our pre-trained model. Do not hesitate to contact Oslandia directly by email for details if interested.

As @vpicavet answered, we can release pre-trained models for this dataset only under specific circumstances. The issue itself may be closed, do not hesitate to ask us for further details if interested.


The exporter simply doesn't work

Tried on 2.93.2 and 2.93.8 ...

How it didn't work? Any error logs?

I speculate the root issue is same as #11.
Would you mind if trying potential solutions written in https://github.com/takahirox/glTF-Blender-IO-materials-variants/issues/11#issuecomment-972424441?

Tried to use the add-on with gltf exporter 1.8.19 giving me:
gather_gltf_hook fails on <glTF-Blender-IO-materials-variants.glTF2ExportUserExtension object at 0x0000027E356D5BD0>
glTF2ExportUserExtension.gather_gltf_hook() takes 3 positional arguments but 5 were given

We have released v0.0.3. Would you try it to check if the problem is still reproducible?

Closing for now. Please reopen if the problem is still reproducible with the latest add-on.


Server disconnect menu item deleting .json server configuration

Majority of people seeing a menu item named "Disconnect" when clicking on the server in the channel list think this is to disconnect - ergo disconnect from the server and possibly remove it from the channel list until Lounge is started again or you type /server .
What actualy happens is removal of .json server configuration. This is a missmatch between what the majority of people expect to happen and can lead to loss of server configurations including NickServ passwords and more.
I suggest: Turn "Disconnect" into disconnecting you from the server and remove it from the channel list until Lounge is started again or you type /server . New menu item named "Remove server" remove .json server configuration.

This button should be something like Disconnect & Remove or some better wording, but 👎 on disconnecting until the server is started again. Users should never care about when a server restarts (even though they do now re: scrollback loading 😅).
So it should be a straightforward change assuming we agree on the wording. @xPaw, thoughts?

29.04.2017 on #thelounge IRC channel:
20:00   +xPaw   that's a poorly named button, it should be 'Remove' or something
20:00   fnutt   I cannot close a network in TheLounge without all the configuration is getting deleted?
20:00   +xPaw   send a /quit
20:00   fnutt   Delete network from config
20:00   fnutt   That's what it does.
20:00   fnutt   Disconnect should be /quit. :p
20:01   +xPaw   err, what the shit
20:03   +xPaw   well then, there's a /disconnect command a /quit command with different implementations

Oh so this is why my config is disappearing all the time. I had issues on a server where I couldn't connect to a channel because "I had already joined" and I couldn't send a message because "I hadn't joined the channel". This happens sometimes, so I hit disconnect, to reconnect. Not knowing my config has been deleted. I would prefer better wording.

Maybe it would be better keeping the config in the json, but setting a "autoconnect" to false so it doesn't reconnect, but if you do "/connect name" it'll reconnect and reset "autoconnect" to true?
I feel like maybe this would be less confusing, and more useful.
Having said that, the whole config on the server for what irc we connect to is all a bit disconnected from the frontend. We need to have a think about a way to deal with this.
Maybe renaming to "remove" is the best idea, and we can have a "reconnect" which will disconnect and reconnect rather than removing it?

Maybe it would be better keeping the config in the json, but setting a "autoconnect" to false so it doesn't reconnect, but if you do "/connect name" it'll reconnect and reset "autoconnect" to true?

That's what ZNC does, for example. Disconnecting does just that—it closes any open connection to that network and disables auto-connection until the user tells ZNC to connect to it again.

Maybe renaming to "remove" is the best idea, and we can have a "reconnect" which will disconnect and reconnect rather than removing it?

Separating the concepts of "disconnecting" and "deleting" a network would be great. (I swear this has come up before, but if so I can't find the thread.)

Fixed by #2114


Transform defaults

Description of changes
Improve default parameters for transform.
Includes

[X] Code changes
[ ] Tests
[ ] Documentation

This is probably not up-to-date anymore? https://opentsne.readthedocs.io/en/latest/api/index.html#openTSNE.TSNEEmbedding.transform

Oh yeah, thanks, great catch!


add method to generate unique token and reply with link

This PR generates a link to my site with a unique token for the user who sends /verify.
This Part just consists of generating a token. The token is not saved to the RDS backend yet.
Will add the part saving data in the RDS backend in future PRs.
To test out the PR in action please join my server: https://discord.gg/tRQrgm7D
and send the /hello and /verify commands.

hi @yesyash We can not verify something completely random. To test it out we can store the random token and then check if the new token generated is different.


Dark Mode settings

Hi
Could you please add a dark mode for the startup of the app?
You know, developers and white ain't a good combination :=)
Thanks!

The app does support dark mode, however currently it just uses the system's mode which is not available on older Android versions. In the settings a selection will be added to override this setting.

Awesome!
Yeah it probably was because I use a custom ROM and maybe detection failed but no problem :)

Will be fixed with https://github.com/Macro-Deck-App/Macro-Deck-Client-App/pull/17

♥


Fix invalid comparison of string with number

In this case, the unit is of type string. As such, "codepoint" > 0
always produces false, hence the code in question always became -1.
I found this while debugging https://crbug.com/1161083, which turns
out was already reported and fixed with
https://github.com/codemirror/CodeMirror/commit/37d7b2efceb192c94811a13b2b7b3eec4b786608

Thanks for spotting that, but the intention of that code was d19a746e, so I've pushed that instead.

Thanks for spotting that, but the intention of that code was d19a746e, so I've pushed that instead.


feat: ollama provider

Example config:
{
"OPENAI_API_KEY":"",
"SHAI_SUGGESTION_COUNT": "3",
"SHAI_API_PROVIDER": "ollama",
"OLLAMA_MODEL": "phi3.5",
"OLLAMA_API_BASE": "http://localhost:11434/v1/",
"SHAI_TEMPERATURE": "0.05"
}
New feature that integrates ollama

Awesome! Could you also mention it in the README?

And increment the version patch?

done

anything else?


How to make it immediate ?

Hi,
First of all, very nice plugin 👍
I tried changing a few vars to make the chat display messages immediately, but I'm not sure that I understand every part of it. I'm probably doing mistakes but I changed this:
       - self.actStamp[1] = true; self.actStamp[2] = false

       + self.actStamp[1] = true; self.actStamp[2] = true

So, messages display immediately but I'm having this error:
ERROR:SCRIPT: /DefChat/DefChat.gui_script:316: bad argument #1 to 'len' (string expected, got nil)
stack traceback:
	[C]: in function 'len'
	/DefChat/DefChat.gui_script:316: in function 'stamp'
	/DefChat/DefChat.gui_script:90: in function </DefChat/DefChat.gui_script:84>

Any idea on how I can make it cleaner please ?

Hey there! Thanks for reporting this. I didn't test out the instant mode with the newest version. It was an 'Oopsie' on my part. It was the cause of a missing 'else' statement, so a part of the stamping function ran when it wasn't supposed to ;). I just uploaded the fixed version.
I agree, my coding is definitely not optimal for understanding yet. Now that it's working correctly and someone's taken interest in it, I'll use the knowledge I gained over the past few months to make things more optimized.

No worries, it's already cool that you shared this.
I tried the fixed version and it's in "instant mode" like I was expecting.
But I'm not sure instant = false nor self.typeSpeed are working though, for those who want the old behaviour:
typeSpeed is delaying the entire message but not the letters one by one, and instant has no effect.

That's odd. Instant = false works on my end. You have to change the variable at the end of the buildMessage(). It will be a line as such:
self.actStamp.go = true; self.actStamp.instant = true
Honestly, I don't understand why I made this so awkward. I'll most definitely make many improvements soon. And about the typeSpeed bug, I'll have to look into why that would be.

Indeed, I was changing the wrong one. Thank your for the fix and the feedback :)
(You may close the issue.)

self.actStamp is now self.stamp_text, and the self.stamp_text.instant variable has been moved to initialization where it's more suitable.


After enable mcs env, istiod:discovery is not ready

Bug Description
istiod Deployment:
env:
- name: ENABLE_MCS_SERVICE_DISCOVERY
value: "true"
- name: ENABLE_MCS_HOST
value: "true"
- name: ENABLE_MCS_CLUSTER_LOCAL
value: "true"

pod/istiod-578cfd8657-4bc69                 0/1     Running   0          9m52s
log:
2022-12-29T02:38:02.471837Z     warn    Secure Webhook Server is not ready: Get "https://:15017/httpsReady": dial tcp :15017: connect: connection refused
2022-12-29T02:38:03.079704Z     info    klog    k8s.io/client-go@v0.25.1/tools/cache/reflector.go:169: failed to list *unstructured.Unstructured: the server could not find the requested resource
2022-12-29T02:38:03.079765Z     error   watch error in cluster Kubernetes: failed to list *unstructured.Unstructured: the server could not find the requested resource
2022-12-29T02:38:05.471404Z     warn    Secure Webhook Server is not ready: Get "https://:15017/httpsReady": dial tcp :15017: connect: connection refused
2022-12-29T02:38:08.472189Z     warn    discovery is not ready:
2022-12-29T02:38:11.472275Z     warn    discovery is not ready:
2022-12-29T02:38:14.471338Z     warn    discovery is not ready:
2022-12-29T02:38:17.472217Z     warn    discovery is not ready:
2022-12-29T02:38:20.471736Z     warn    discovery is not ready:
2022-12-29T02:38:23.471439Z     warn    discovery is not ready:
2022-12-29T02:38:26.472036Z     warn    discovery is not ready:
2022-12-29T02:38:29.471535Z     warn    discovery is not ready:

what happen?
how to solve?
Version
client version: 1.16.0
control plane version: 1.16.0

Client Version: version.Info{Major:"1", Minor:"25", GitVersion:"v1.25.4", GitCommit:"872a965c6c6526caa949f0c6ac028ef7aff3fb78", GitTreeState:"clean", BuildDate:"2022-11-09T13:36:36Z", GoVersion:"go1.19.3", Compiler:"gc", Platform:"darwin/amd64"}
Kustomize Version: v4.5.7
Server Version: version.Info{Major:"1", Minor:"25", GitVersion:"v1.25.4-gke.2100", GitCommit:"0e55ef943b612cf213ba4abfd1e6e915b9832969", GitTreeState:"clean", BuildDate:"2022-12-07T09:33:35Z", GoVersion:"go1.19.3 X:boringcrypto", Compiler:"gc", Platform:"linux/amd64"}

Additional Information
No response

Do you have the MCS CRD installed? it's not present by default or installed by Istio

Even CRD not installed, istiod should not be blocked

This is user explicitly turning on a feature though.. why would the MCS feature be turned on if mcs is not enabled?

maybe by mistake, same as gateway apis. If not installed, istio will not be blocked


vcpkg include directories incorrect

It seems that when installing using vcpkg, the headers expect that the include root is in the sail directory, but vcpkg places the includes under sail. To include something in your own project, you need
#include <sail/sail-c++/sail-c++.h>

but the sail headers includes other sail headers as
#include <sail-c++/...>

Am I missing something here? Was this project updated since it was set up for vcpkg that caused this?

Hi! Do you use CMake? With CMake, you get correct include paths. Sample project:
CMakeLists.txt
cmake_minimum_required(VERSION 3.10)

project(SAIL VERSION 1.0.0
             DESCRIPTION "sail-probe"
             LANGUAGES C CXX)

set(CMAKE_CXX_STANDARD 17)
set(CMAKE_CXX_STANDARD_REQUIRED ON)
set(CMAKE_CXX_EXTENSIONS OFF)

add_executable(sail-probe sail-probe.cpp)

find_package(SailC++ REQUIRED)
target_link_libraries(sail-probe SAIL::sail-c++)

sail-probe.cpp
#include <sail-c++/sail-c++.h>

int main(int argc, char *argv[]) {

    sail::image_reader reader;
    sail::image image;

    SAIL_TRY(reader.probe(argv[1], &image));

    return 0;
}

Compilation
mkdir build
cd build
cmake -A x64 -DCMAKE_TOOLCHAIN_FILE=D:\projects\git.vcpkg.sail\scripts\buildsystems\vcpkg.cmake -DCMAKE_BUILD_TYPE=Release ..
cmake --build . --config Release

Running
>Release\sail-probe.exe "D:/images/png/xin-gray-1bit.png"
SAIL: [D] [io_file.c:177] Opening file 'D:/images/png/xin-gray-1bit.png' in 'rb' mode
SAIL: [D] [context_private.c:696] Allocated a new thread-local context<PHONE_NUMBER>4FFF40
SAIL: [I] [context_private.c:655] Version 0.9.0
SAIL: [D] [context_private.c:54] Add 'D:\projects\sail-test\build\Release' to the DLL search paths
SAIL: [D] [context_private.c:181] SAIL_MY_CODECS_PATH environment variable is not set. Not loading codecs from it
SAIL: [D] [context_private.c:364] Enumerated codecs:
SAIL: [D] [context_private.c:369] 1. BMP [Bitmap Picture] 1.1.2
SAIL: [D] [context_private.c:369] 2. GIF [Graphics Interchange Format] 1.3.1
SAIL: [D] [context_private.c:369] 3. JPEG [Joint Photographic Experts Group] <IP_ADDRESS>
SAIL: [D] [context_private.c:369] 4. PNG [Portable Network Graphics] 1.1.3
SAIL: [D] [context_private.c:369] 5. TIFF [Tagged Image File Format] 1.0.1
SAIL: [D] [context_private.c:677] Initialized in 27 ms.
SAIL: [D] [codec_info.c:115] Read magic number: '89 50 4e 47 0d 0a 1a 0a 00 00 00 0d 49 48 44 52'
SAIL: [D] [codec_info.c:126] Found codec info: 'PNG'
SAIL: [D] [helpers.c:330] PNG: Found ICC profile 'Dot Gain 20%' 912 bytes long
SAIL: [D] [palette-c++.cpp:129] NULL pointer has been passed to sail::palette(). The object is untouched

Please re-open in case of any issues.

I am still having this issue and am trying not to use CMake, has this been fixed for Visual Studio 2022?
1>C:\vcpkg\installed\x64-windows-static\include\sail\sail\sail_junior.h(33,14): fatal  error C1083: Cannot open include file: 'sail-common/error.h': No such file or directory

It is the same wether I use x86, x64 or x64-static.

@AlexMollard  Do you use the MSVS vcpkg integration?

Yeah, I am currently using a few other libraries without issue.

Oh, I see. I didn't notice this error before as I use universal sail headers. I think you can try the following potential fixes:

Use universal includes: #include <sail-c++/sail-c++.h>. They already include sail-common/common.h with the necessary definition
Or, include sail-common/common.h before including any sail cpp headers.

There may be more errors like take. I've just pushed several fixes.
Until the fixes reach vcpkg, using universal includes is recommended.

SAIL-RC2 with the missing includes fix has been merged into vcpkg. Please update the local copy of vcpkg and install rc2. Adding VcpkgInstalledDir/include/sail to the project include path is still required if you don't use cmake.


fix: GET: /config return all configurations when running standalone

I hereby agree to the terms of the GreptimeDB CLA
What's changed and what's your intention?
Fixes #2584.
This PR makes frontend http-server

return frontend config when running distributed mode
return standalone config when running standalone mode

Checklist

[ ]  I have written the necessary rustdoc comments.
[ ]  I have added the necessary unit tests and integration tests.

Refer to a related PR or issue link (optional)
close #2584

cc @etolbakov, you might be interested in this

@NiwakaDev Sorry for being so late to review this PR. There are some conflicts that have to be resolved.

@evenyag  PTAL.
I guess that I addressed your review.


a stage manager like feature

Description of the new feature / enhancement
a feature like stage manager but for windows to better enhance the multitasking experience
Scenario when this would be used?
sometimes when doing multitasking with virtual desktop, we often forgot the windows that existed in virtual desktop. with stage manager like feature, we can see clearly a set of windows that we are working on and can easily switch and manage windows in the same workspace.

right now im using a alternative of stage manager for windows, it's quite buggy and the dragging for specific windows is not really working . i would like to see this feature implemented directly to powertoys and see many users take advantage of this feature.
Supporting information
No response

going to /dup #31874 as it has an active Github project