GetPositionAsync never returns, timesout or raises exception on first call

Bug Information
Version Number of Plugin: <IP_ADDRESS>
Device Tested On: iPhone 5S iOS11.3
Simulator Tested On: N/A
Version of VS:  2017
Version of Xamarin: <IP_ADDRESS>7
Versions of other things you are using: Xcode 9.3
Steps to reproduce the behaviour
Run the application for the first time under debug.
Request position using .GetPositionAsync()
Confirm the pop-up to allow permission for locations
.GetPositionAsync() never returns, timesout, or throws an exception.
Close app, launch again, no need for pop-up as permissions already granted, works fine
Expected Behavior
GetPositionAsync() should either turn a location, timeout or throw an exception.
Have tried adding a timeout of 10s (Timespan)  to the .GetPositionAsync() call and/or a cancellation token - still never returns
Actual Behavior
.GetPositionAsync() never returns
Code snippet
private async Task<bool> GetCurrentLocation()
        {
            var locator = CrossGeolocator.Current;
            locator.DesiredAccuracy = 100;
            if (!CrossGeolocator.IsSupported)
                return false;
            if (!CrossGeolocator.Current.IsGeolocationAvailable)
                return false;
            try
            {
                var position = await locator.GetPositionAsync(); // This never returns

                latitude = position.Latitude;
                longitude = position.Longitude;
                return true;
            }
            catch (Exception e)
            {
                string reason = e.Message;
                return false;
            }
        }

plist
<key>NSLocationWhenInUseUsageDescription</key> 	<string>You are about to use location!</string> 	<key>NSLocationAlwaysAndWhenInUseUsageDescription</key> 	<string>You are about to use location!</string> 	<key>UIBackgroundModes</key> 	<array> 		<string>location</string> 		<string>bluetooth-central</string> 	</array>
Screenshots
N/A

Have you read through: https://jamesmontemagno.github.io/GeolocatorPlugin/GettingStarted.html specifically the iOS section on proper plist setup.

Hi,
Yes, read, followed to the letter. It still hangs.
G.

I can confirm that issue as well.

I can confirm as well.  Seems to be an issue building with 11.3 SDK.  Was working fine when I was building with SDK 10.2.  However, now that I upgraded my Mac(build machine) to Sierra High (Mac OS 10.13).  I cannot go back to 10.2 SDK.  Well, I can install the SDK but get several build issues. Not important in this case though.  About to test with 11.0.  However, I can assume it will not work though.  Appears to be the changes to 11+.  Possibly permissions.  Just as stevenhawkin I followed the permission document correctly. Will let you know my findings after I try building with SDK 11.

I also have this one on an app/code that worked fine in the past.

@devapalanisamy Thanks for the suggestions, doesn't seem to fix it for me though :/

Your info plist looks inforrect.
Simply Add:
NSLocationWhenInUseUsageDescription
This app needs access to location when open.
Background Updates
Only implement this and add these properites if you need background updates for your application. Most likely you will not. Adding this also has direct impact on permissions and prompts to the user. Please be very careful when adding this information.
Inside of your info.plist you must enable Background Modes/UIBackgroundModes for location updates. Here is a full guide. Your info.plist should contain something like this:
UIBackgroundModes

location

In addition to NSLocationWhenInUseUsageDescription you are required to add NSLocationAlwaysAndWhenInUseUsageDescription keys in your app’s Info.plist file. (If your app supports iOS 10 and earlier, the NSLocationAlwaysUsageDescription key is also required.) If those keys are not present, authorization requests fail immediately.
NSLocationAlwaysUsageDescription
This app needs access to location when in the background.
NSLocationAlwaysAndWhenInUseUsageDescription
This app needs access to location when open and in the background.

Also, if you are using the simulator make sure you actually set the location in it first
If you are only using it while your app is running ONLY add the first permission.

I was facing the same issue for some Android device.
Working Device = Samsung,
Not Working Device = Motorola(MotoG)
Below is my code.
var locator = CrossGeolocator.Current;
locator.DesiredAccuracy = 10;
var _position = await locator.GetPositionAsync(TimeSpan.FromSeconds(10));

I had the same problem.
I was using xamarin live player and doesnt works.
var locator = CrossGeolocator.Current;
locator.DesiredAccuracy = 50;
var _position = await locator.GetPositionAsync(TimeSpan.FromSeconds(60));

Same problem here with android, using live player, android emulator or real device, it always crashs and 2 days before everything was working fine.

I was facing the same issue on Android, but it was caused my not awaiting an interface call to the method calling: var locator = CrossGeolocator.Current;

I'm also facing this issue right now. Was anything done to address it?


Provide APIs for load balance to be used outside of reverse proxy

What should we add or change to make your life better?
Provide a service in order to have the same load balancing policy for a cluster outside reverse proxy middleware.
public interface ILoadBalanceService
{
    DestinationConfig? PickDestination(string cluster);
}

Why is this important to you?
We are using YARP with one cluster and multiple destinations. Most requests are being proxied to the cluster but we also have some routes that need some logic and don't exactly need to be proxied but to make a http call to the cluster. We don't need to send HttpContext.Request to the cluster and also don't want the response to be sent in HttpContext.Response. We only need the destination address of the cluster.

It seems odd that we'd only do this for load balancing and not other proxy features like session affinity, passive health checks, retry, etc..
There have been a few issues asking about getting access to the clusters from outside the main pipeline, like https://github.com/microsoft/reverse-proxy/issues/1159. IClusterChangeListener works but is cumbersome. Once you had the cluster, most of the load balancing algorithms are trivial.
Note the concurrency counter is internal so you wouldn't be able to reliably track the load added by these extra requests.
https://github.com/microsoft/reverse-proxy/blob/81134198615b1d898683466b0b6fef3b62c93074/src/ReverseProxy/Model/DestinationState.cs#L60

There have been a few issues asking about getting access to the clusters from outside the main pipeline, like #1159. IClusterChangeListener works but is cumbersome. Once you had the cluster, most of the load balancing algorithms are trivial.

How about adding an interface like IClusterProvider and ProxyConfigManager could implement it by returning the clusters dictionary?

It seems odd that we'd only do this for load balancing and not other proxy features like session affinity, passive health checks, retry, etc..
Note the concurrency counter is internal so you wouldn't be able to reliably track the load added by these extra requests.

I guess it would be nice to be able to leverage all YARP features outside the main pipeline. What do you think about adding this?
Task<HttpResponse> SendAsync(HttpRequest request, string cluster);

This method could first make sure that HttpRequest has an relative path so it could be added to the base address from destination. I think it could also handle session affinity, passive health checks and also load balancing.

Triage: This is not a direction we plan to evolve the design. Unless we hear from more users, we won't do anything here ... closing for now.

@Tratcher Right now all load balancing, session affinity and health check are coupled to HttpContext. In order to prevent possible future breaking changes do you think it worth decoupled them from HttpContext now in case in future there's a plan to provide them outside main pipeline?

That coupling was intentional to enable extensibility based on request fields. Also, I don't see how session affinity would work without that coupling, it requires access to the request and response even for its normal scenarios.
We are considering what it would look like for features like load balancing and health checks to exist independently from YARP, maybe as their own library or HttpClient layer, but we don't anticipate an existing model like YARP to directly consume those.

That coupling was intentional to enable extensibility based on request fields. Also, I don't see how session affinity would work without that coupling, it requires access to the request and response even for its normal scenarios.

I was thinking to use some types like RequestTransformContext but without its HttpContext property!

We are considering what it would look like for features like load balancing and health checks to exist independently from YARP, maybe as their own library or HttpClient layer, but we don't anticipate an existing model like YARP to directly consume those.

I understand, thanks for the explanation.

There have been a few issues asking about getting access to the clusters from outside the main pipeline, like #1159. IClusterChangeListener works but is cumbersome. Once you had the cluster, most of the load balancing algorithms are trivial.

How about adding an interface like IClusterProvider and ProxyConfigManager could implement it by returning the clusters dictionary?

AFAIK ProxyConfigManager is internal, so is not easily accessible. Having a mechanism off the ProxyFeature to be able to get a read-only snapshot of the current clusters would be very useful. (hint)

It seems odd that we'd only do this for load balancing and not other proxy features like session affinity, passive health checks, retry, etc..
Note the concurrency counter is internal so you wouldn't be able to reliably track the load added by these extra requests.

I guess it would be nice to be able to leverage all YARP features outside the main pipeline. What do you think about adding this?
Task<HttpResponse> SendAsync(HttpRequest request, string cluster);

This method could first make sure that HttpRequest has an relative path so it could be added to the base address from destination. I think it could also handle session affinity, passive health checks and also load balancing.

As Karel said above, you can always use HttpClient to make a request to yourself, which would then be proxied to the appropriate destination - in a service meshy like way.

We are considering what it would look like for features like load balancing and health checks to exist independently from YARP, maybe as their own library or HttpClient layer, but we don't anticipate an existing model like YARP to directly consume those.

I understand, thanks for the explanation.

This is intended for service to service calls within a deployment. Rather than going through a service mesh, you'd configure a version of HttpClient with the equivalent of routes and clusters, and it would then be able to do that load balancing etc within the service, saving a hop and latency.

Re-opening for 1.x consideration. There are scenarios such as #1414 where there is a desire to be able to choose which cluster to route a request to. The other is the ability to resolve a cluster to a destination (or httpclient based on clusters config).

Some of this will be fixed by #126. Adding to backlog for external use of load balancing features.


Scheduling messages

A function where you can schedule a message to be sent at a later date/time

implemented in #52


Back button from Notifications screen is missing

On the Notification list screen, the back button is missing. There should be a back button that takes the user back to the Home screen.

Device:
iPhone SE, Build 457

resolved by: https://github.com/hyperledger/aries-mobile-agent-react-native/pull/507


Suggestions - Magic Item bonuses / Temporary adjustments

I really love that the sheet can now level up AC/PD/MD and the like--it's great to see that I won't have to remember to do that!
One thing I'd like to see, though, if possible: allowance for magic item bonuses and temporary bonuses  to AC/PD/MD.
I think it could work by just putting in a couple extra boxes underneath the AC/PD/MD section for each column: one for "Magic Item Bonus" and one for "Temporary Bonus."
So, and forgive me for not having a clue about tables and doing it that way, it might be like this:

With total being the addition of Base AC, Level, the appropriate median stat mod, Magic Item bonus, and Temporary Bonus. (Level and stat mod not shown above as they'd be pulled from elsewhere on the sheet. The above used a level 5 character with 0 median stat.)
The arrangement works vertically in the example above, but could just as easily go horizontally if you preferred. Vertically I figured the Total on top would work well to make it the most prominent on the sheet for reference.
This seems like it would be generally useful and not add too much to the sheet.
As a side note...it might be good to also have slots for magic weapon / temp bonuses to melee attacks, ranged attacks, and magic spells. I do see the +misc by the basic attacks, but it'd be cool to have a slot labeled specifically for magic bonuses that could be easily (and understandably) referenced by any macro. I guess you can use the misc slots for that, but I picture those more for random things that come up rather than roughly permanent bonuses.

(one side note that I didn't see until now--looks like the MD total does go to the right place, I think, but the tooltip reads AC rather than MD.)

I added item bonus fields to abilities, basic attacks, defenses and initiative.


Update examples to use new Tokio

The old tokio-core crate is deprecated in favour of the tokio crate,
which also provides the new multithreaded Tokio runtime. Although h2
is generic over the runtime, the examples do depend on tokio.
This branch update the example code to use the new tokio library
rather than tokio-core.
For the most part, this was pretty trivial --- simply replacing
handle.spawn(...) with tokio::spawn(...) and core.run(...) with
tokio::run(...). There were a couple of cases where error and item
types from spawned futures had to be mapped to (), (). Alternatively,
this could have been avoided by using the single-threaded runtime and
calling block_on instead to await the value of those futures, but I
thought it was better to reduce the amount of tokio-specific code in the
examples.
Signed-off-by: Eliza Weisman<EMAIL_ADDRESS>

Looks like the reason CI is failing is because there are some doctests for examples in the RustDoc that also use tokio-core, and this branch removed the dependency. I'll update those as well.


PP-5391 add refund events model

WHAT YOU DID

introduce refund event classes
refactor eventdetails package

We introduced parentResourceExternalId which is the charge's externalId. This is going to be the link on Ledger's side to the corresponding payment.
with @mrlumbu

I'm confident that this change does not require further review and meets the secure coding requirements of GOV.UK Pay


shell: fix tab key functionality.

Fixing commit: #7ad59c84c84e8b7ea3c0438ff01c8a6343e65812
Signed-off-by: Jakub Rzeszutko<EMAIL_ADDRESS>

Codecov Report

Merging #12152 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master   #12152      +/-   ##
==========================================
- Coverage   48.05%   48.05%   -0.01%
==========================================
  Files         284      284
  Lines       43368    43369       +1
  Branches    10397    10397
==========================================
  Hits        20840    20840
- Misses      18388    18389       +1
  Partials     4140     4140

Impacted Files
Coverage Δ

subsys/shell/shell.c
22.2% <0%> (-0.04%)
:arrow_down:

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 9fe81e9...0e88668. Read the comment docs.


Move the 'Comparisons' code into a separate repository.

As part of our attempt to slim down the core adam module into its minimal set of classes, and also as preparation for re-writing some of the Picard-style metrics (this is relevant to the NCI Cloud Pilot project), we should move out all the core code and commands which compute read- and variant-level metrics from the adam-core module into a separate module and repository.
This repository (qc-metrics) has already been created.  This issue is for deleting the code (in org.bdgenomics.adam.metrics), the ComparisonTraversalEngine class, all their test suites, and the two commands (FindReads and CompareADAM) which depend on them.
All of this will be re-created in the qc-metrics repository.

+1

Will you move the variant concordance work into qc-metrics as well?

Where is that?

The big ones that I'm aware of are:

https://github.com/bigdatagenomics/adam/blob/master/adam-cli/src/main/scala/org/bdgenomics/adam/cli/GenotypeConcordance.scala
https://github.com/bigdatagenomics/adam/blob/master/adam-core/src/main/scala/org/bdgenomics/adam/rdd/variation/VariationRDDFunctions.scala

Yeah, I'll move those.  One sec.  (I'm building up a corresponding branch in qc-metrics as I go, of course.)

Sounds awesome; looking forward to it!

While I'm working on this (and I've got kid duty today, so it's intermittent) can you open up an issue or two, for any other major chunks of code like this that you think need to be moved to another repo too?

YGI, will do after brunch.

Thx.


On peer registration form cursor is jumping to first text box

If user wants to input password or peer name cursor is jumping to first textbox and user sholud make double click to stay on necessary textbox.

Fixed in master.


Update check for unset override_allele_scheme to include empty string

Description of feature
Currently, the parameter override_allele_scheme is considered unset only if it has a value of null. See for example:
https://github.com/phac-nml/mikrokondo/blob/c32ce856fbac0e017f7b1d6111a12feaf59eaff5/subworkflows/local/locidex.nf#L26-L30
Within IRIDA Next, I wish to configure allele schemes so they are set using drop-down values like below:

I wish to have an option Automated selection from allele database, which will be used to indicate that the override_allele_scheme should be skipped and automated database selection should be performed. This is the equivalent of setting override_allele_scheme = null.
However, in IRIDA Next, you cannot override parameters and set to null, they must be strings. Hence, I would like both override_allele_scheme = null and override_allele_scheme = "" to be treated identically.

I there may be some additional changes to some control flow logic in the subworkflows/local/locidex.nf but I think replacing null with "" would work out well!
I can start work on this tommorow if you would like. Would this have to be rolled out as part of a patch or do you see it being released in the next minor release?

Thanks. Should be a patch, but doesn't have to be immediate.

fixed in PR 109


Add marine equipment specialist document

New specialist publisher for Marine Equipment Approved Recommendations by Maritime Coastguard Agency.

LGTM, great work 😊


image size

Hi Andy!
Could you please give some insights about doubling the input size? Zoom image by 2,  go through network and then upsample by 16 to recover the resolution. Why simply do network forward and upsample by 32?

@mike112223 I am not sure, but maybe because it's less interpolation when you upsample by 16 rather than by 32, so you do not "assume" alot of information.


feat: add support for Schema::increments

This will allow migrations, jobs and other database related features to work properly.
#174 needs to be merged first.
Closes #157

Thanks for reviewing! @tyabu12


avoid in-function graph imports in lazy.py

See docs.
Note that GRAPH=1 is not covered in the CI tests.

Changes made in tinygrad/:
------------------------------------------------------------
files                             insertions       deletions
------------------------------------------------------------
tinygrad/graph.py                          6               4
tinygrad/lazy.py                           1               2
------------------------------------------------------------
total                                      7               6
------------------------------------------------------------
lines added in the tinygrad folder: 1

Oh duh. Removed the reference. I believe this still makes sense.

Should https://github.com/tinygrad/tinygrad/issues/1653 be closed now that this has been merged?

No, that's a different thing


add ability to use RoundRobinDnsAddressResolverGroup in Connection manager.

now is java 1.6 compatible

Thank you!


corrected config publish command

Getting InvalidArgumentException while executing php artisan publish:config vinkla/instagram
[InvalidArgumentException]
There are no commands defined in the "publish" namespace.

http://laravel.com/docs/4.2/packages#package-configuration

Thanks for you contribution.


🔄 Reload project on all created/deleted files during watch

This addresses #183 - creating or deleting markdown files now reloads the entire project, adding/removing pages for these files. Previously this full reload only happened when the _toc.yml file was updated or a new bib file was added.
This likely introduces some inefficiencies - we are reloading everything when we really only need to reload the project structure... but I suspect in practice, things are fast enough to not worry - this still happens infrequently compared to page edits. Something to keep an eye on, though.
One other small change is renaming the triggerReload argument to serverReload, hopefully eliminate some confusion with page vs. project vs. server reload.

Looks great!


Configuration cache should support composite builds

The configuration cache doesn't support composite builds yet. This issue is about adding that support.
Expected Behavior
The configuration cache supports composite builds.
Current Behavior
See https://docs.gradle.org/nightly/userguide/configuration_cache.html#not_yet_implemented

I don't suppose there is any target milestone for this yet?  Is there anything we of the community can do to help?  :)  Do you know generally how this should be implemented?

This will ship in 6.8


[DNM] coprocessor: Rewrite RPN function framework and macro

What have you changed? (mandatory)
Rewrite RPN function macro and framework.
Ugly things are put into the macro. Logic code becomes more clear.
Need to add more docs.
Will split it to small parts so that people can review.
What are the type of the changes? (mandatory)

Engineering (engineering change which doesn't change any feature or fix any issue)

How has this PR been tested? (mandatory)
make dev
Does this PR affect documentation (docs) or release note? (mandatory)
No
Does this PR affect tidb-ansible update? (mandatory)
No

/run-all-tests

/bench

/run-all-tests

/bench


[Runtime] Check for demangling failure when looking up prespecialized metadata.

_buildDemanglingForGenericType can fail, so we need to check for NULL.

@swift-ci please smoke test


Can't use BMM commands in Paper 1.19.3

[10:09:37 ERROR]: Caught previously unhandled exception :
[10:09:37 ERROR]: Paper Async Command Builder Thread Pool - 5
java.lang.NoSuchMethodError: 'boolean net.minecraft.commands.CommandListenerWrapper.k()'
    at de.miraculixx.bmm.MarkerCommand.mainCommand$lambda$38$lambda$37$lambda$36$lambda$31(MarkerCommand.kt:111) ~[bmm-paper-1.3.0.jar:?]
    at com.mojang.brigadier.tree.CommandNode.canUse(CommandNode.java:79) ~[paper-1.19.3.jar:git-Paper-446]
    at net.minecraft.commands.Commands.fillUsableCommands(Commands.java:477) ~[?:?]
    at net.minecraft.commands.Commands.fillUsableCommands(Commands.java:512) ~[?:?]
    at net.minecraft.commands.Commands.fillUsableCommands(Commands.java:512) ~[?:?]
    at net.minecraft.commands.Commands.sendAsync(Commands.java:429) ~[?:?]
    at net.minecraft.commands.Commands.lambda$sendCommands$5(Commands.java:408) ~[?:?]
    at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136) ~[?:?]
    at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635) ~[?:?]
    at java.lang.Thread.run(Thread.java:833) ~[?:?]

This also has the side effect of bugging out the entire autocompletion, for other commands too.

That happens, if you use the non Legacy Paper Version of BMM in versions older then 1.19

Fixed in latest version for 1.18.x - 1.19.x


zq: Pass null ticker to (*MuxOutput).Pull(..)

We currently pass a 10s ticker to (*MuxOutput).Pull(..) in zq and tests. This doesn't seem necessary, but changing that to a nil ticker breaks some tests. Investigate and fix.

(this was discussed in a PR comment thread: https://github.com/brimsec/zq/pull/549#discussion_r407103358)


wip: sdk->sdk-core -- wrapper package

why? 🤔

we closing this or what's good?

At the very least, if we decide to do this we'll do it from a fresh branch


[IOS] If app was bundled via Fastlane, it crash on startup

Question Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Question Subject
Question about build-issues with fastlane
Question Description
I run my fastlane and all working like expected. At least the App is uploaded to the Apple-Store and I'm able to download it via Testflight.
But if I start the App, it crashed with an error like:

You attempted to use a firebase module that's not installed natively on your iOS project by calling firebase.crashlytics(). Ensure you have the required Firebase iOS SDK pod for this module included in your Podfile, in this instance confirm you've added „pod 'undefined'“ to your Podfile

If I bundle the App via XCode it works at my local device... even If i do archive in XCode and deploy it to the app-store, load it from there and run the app... all working well.
But only If I use Fastlane to deploy, the app crash immediately after startup.
Here is the cut from my fastfile:

  lane :deploy_internal do |options|
    git_pull
    ensure_git_status_clean
    get_certificates           # invokes cert - must be installed locally in your mac-keychain
    get_provisioning_profile   # invokes sigh - must be installed locally in your mac-keychain

    if options[:version] == "minor" || options[:version] == "major"
      puts options[:version]
      bumpVersion(version: options[:version])
    else
      bumpVersion(version: "patch")
    end
	increment_build_number(
      build_number: Time.now.strftime("%Y%m%d%H%M"),
      xcodeproj: ENV["XCODEPROJ"]
  	)
	build_app(
	  scheme: "MyNiceApp",
	  workspace: ENV["XCWORKSPACE"],
	  include_symbols: true,
	  include_bitcode: true
	)
    upload_to_testflight(skip_waiting_for_build_processing: true)
    refresh_dsyms
    commit_version_bump(message: 'Bump build IOS', xcodeproj: ENV["XCODEPROJ"])
    push_to_git_remote
  end
end

BTW: If I bundle on XCode, I use the same Scheme: MyNiceApp.
Might the issue depend on build_app, or upload_to_testflight parameters?

What output are you getting when running your lane?
Please also see what @fastlane-bot posted.

I experienced a similar issue where I would archive and distribute to the app store with xcode and it worked fine, but distributing through fastlane would result in the app crashing during startup.
I have a watchOS companion app and as a result I have 2 separate schemes: one for the iPhone app and the Watch app. I would normally use the watchOS scheme to archive and distribute, however when I initialized fastlane in the project, it chose the iphone scheme to archive and distribute, which had a problem.
So check your Fastfile and look at what scheme it is using:
build_app(workspace: "MyApp.xcworkspace", scheme: "MyApp")
... and try to Archive and Distribute with that scheme through xcode and that version might crash the same way as the fastlane version.
So you can either change your Fastfile to point to the right scheme, or fix the scheme that Fastfile chose, but at least this will eliminate fastlane as a variable 😅

Very interesting scenario  @mrvisser thanks for sharing. Any clue why building & archiving for watchOS is the right choice here? Is this still the case as of Xcode 12? 🤔

I'm not sure which is the "right" scheme for archiving an iOS app with a watch companion, they both appear to do the same thing when it comes to archiving.
I just actually had a misconfiguration in my iPhone app scheme that caused an issue. Once I knew it wasn't fastlane at fault I managed to track down the issue in my iPhone app scheme (the Archive task in the scheme had "Debug" build configuration instead of "Release").

Got it! Thanks for the explanation @mrvisser 😊 that makes sense!


can you add relation api ?

it is a great job for accessing thingsboard, i really appreciated you. Now i need init many asset and device by api,but i found there is no relation in your code. can you add relation api ? Thanks!

Hey, hope it is not late, but relation can be added like this:
relations = {"from": {"id": from_entity_id,
"entityType": from_entity_type},
"type": "Contains", "to": {"entityType": to_entity_type,
"id": to_entity_id}}
relations_r = requests.post('http://<IP_ADDRESS>:8080/api/relation', json=relations, headers=headers)
headers should be the usual part as is in the code.


Fix striped for tables. Fixes #279.

Proposed changes

Makes striped work for tables again, which is a fix for #279. Bug is described within the issue.
Screenshots (if appropriate) or codepen:

Types of changes

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

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

@Smankusors Force pushed with proper commit message :)

@Smankusors Makes sense. I just restored the old rules, but it makes sense to reduce nesting :) Applied your suggestion.

@Smankusors fixed commit messages to make commit lint happy again 😓

thanks for your contribution!
also, I invited you as the member of materializecss. Welcome 🎉


Ignore unexpected blob id in replication

Summary
Add a configuration to ignore unexpected blob id in replication thread. It's safe to do so because these blob ids don't belong to the partition anyway, we should just ignore it.
Test
no tests added

Codecov Report
Attention: Patch coverage is 0% with 10 lines in your changes missing coverage. Please review.

Project coverage is 18.48%. Comparing base (52ba813) to head (c9090da).
Report is 133 commits behind head on master.

Files with missing lines
Patch %
Lines

...va/com/github/ambry/replication/ReplicaThread.java
0.00%
7 Missing :warning:

...ava/com/github/ambry/config/ReplicationConfig.java
0.00%
2 Missing :warning:

...m/github/ambry/replication/ReplicationMetrics.java
0.00%
1 Missing :warning:

:exclamation:  There is a different number of reports uploaded between BASE (52ba813) and HEAD (c9090da). Click for more details.
HEAD has 2 uploads less than BASE

Flag
BASE (52ba813)
HEAD (c9090da)

3
1

Additional details and impacted files
@@              Coverage Diff              @@
##             master    #2932       +/-   ##
=============================================
- Coverage     64.24%   18.48%   -45.77%
+ Complexity    10398     2947     -7451
=============================================
  Files           840      865       +25
  Lines         71755    73666     +1911
  Branches       8611     8863      +252
=============================================
- Hits          46099    13617    -32482
- Misses        23004    58776    +35772
+ Partials       2652     1273     -1379

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Timestamp missing in the .on event emitter

I'm trying to use the  logger.on('logging', function (transport, level, msg, meta)){..} event emitter to fire a socket.io event, however the timestamp that I have defined in my transport is not available in the emitter, only the message itself is passed as an argument. How can I get it?
Thanks.

@softelos did you ever figure this out?

Have the same issue. Using logger.stream().on('log', callback) works but this relogs the previously added logs.


Mixing spaces and tabs

Hi @jo-mueller, looking into https://github.com/conda-forge/napari-stl-exporter-feedstock/pull/4 and updating the patch it seems that tabs and spaces are mixed in the file. Just letting you know so you can fix in a future release.

PS: I have an extension on the browser called "Refined GitHub" which adds some nice stuff, like seeing tabs and spaces on github files.
Cheers!

Hi @goanpeca thanks a bunch for spotting this and also for the tip with the browser extension, it's very helpful :) I think this was introduced here PR #32 , I wasn't aware that inconsistent tabs in the config files would lead to trouble during the azure build process - I probably made this mistake in some other repos as well!

I wasn't aware that inconsistent tabs in the config files would lead to trouble during the azure build process - I probably made this mistake in some other repos as well!

In general it should not be an issue, but it just makes the creation of the patch files a bit more convoluted :)
It is just nice to be consistent ;-)


Schematic for Angular Material Dialog

Feature Description
Provide a brief summary of the feature you would like to see.
I find the setup process of components to be quite redundant in some cases. An example is creating a dialog. I see that angular material has schematics. It would be nice if there was something for a dialog:
ng generate @angular/material:dialog <component-name>
This would generate a component like so:
import { Component, Inject, OnInit } from '@angular/core';
import { MAT_DIALOG_DATA, MatDialogRef } from '@angular/material';

@Component({
  selector: 'app-aggregate-dialog',
  templateUrl: './aggregate-dialog.component.html',
  styleUrls: ['./aggregate-dialog.component.scss']
})
export class AggregateDialogComponent implements OnInit {
  private _dialogRef: MatDialogRef<AggregateDialogComponent>;
  constructor(dialogRef: MatDialogRef<AggregateDialogComponent>, @Inject(MAT_DIALOG_DATA) data: any) {
    this._dialogRef = dialogRef;
  }

  ngOnInit() {
  }

}

This would also mean the dialog component would be added to the entryComponents in the module.
Use Case
Describe the use case(s) that the proposed feature would enable.
For dialog-heavy applications, setting up dialogs require quite a few steps. It would be nice if this was a little more streamlined. Given that (I believe) if you want to use an Angular Material dialog, it requires it's own component I think it would be useful for the proper (or common) references to be added when generated. It sounds like the entryComponents step won't be needed in v9, so that is nice.

For the record, the step about adding it to entryComponents won't be necessary with Ivy.

@crisbeto For the record, that is already mentioned in the original request.


Revendor g/g v1.89.

References https://github.com/metal-stack/os-metal-extension/pull/49.
The additional files are added at one place now by the os-extension.

has still github.com/gardener/gardener v1.87.4 in go.mod?


escaping json in $ref when converting spec to graphql schema

One of the types in my swagger schema is as follows:
$ref": "#/definitions/Get Item Subtypes by Type for Company"
The error I'm getting is:
Error: Definition Get%20Item%20Subtypes%20by%20Type%20for%20Company was not found in schema
Is this not being excaped correctly? Is there a manual way to resove this?

JSON References are URIs and should always be URI-encoded, per the spec.   However, if you don't URI-encode your references, then json-schema-ref-parser will try to automatically encode and decode them for you as needed.  This is not bulletproof though, so I suggest that you always URI-encode your references yourself.
I created a simple reproduction of your example that demonstrates both a non-encoded reference and a properly-encoded reference.  In my example, json-schema-ref-parser was able to successfully handle even the non-encoded reference.  Not sure why it's not working in your case, but if you encode your references it should work

Thank you for this @BigstickCarpet !


Update the DSpaceObject model and services

Make the changes to the data model and services to use the new map structure for metadata (see https://jira.duraspace.org/browse/DS-3907)

Closing. Fixed by https://github.com/DSpace/dspace-angular/pull/347  See also https://jira.lyrasis.org/browse/DS-4107


Post to output DB when sink returns 409

This is to solve the problem with database latency. When database write returns timeout, it doesn't means that the database write has failed. In the next retry, the duplicate detections would kicks in which will make the earlier components to not write to its output db.
Example:
aggregator returns 403 (Even though the write didn't fail).
retry happens, and aggregator returns 409 to accumulator.
accumulator, meter, and collector would see 409 and doesn't write to its db.
Fix: when sink returns 409, treats it as 201.

Coverage decreased (-4.5%) to 86.802% when pulling e66bb5427f813bea8443f835310bdbd7f3d0139d on KRuelY:duplicates into 0e9a649ed27c6418af470c9145182773989fdc79 on cloudfoundry-incubator:master.


Whether the shape-indexed features are used in the fine MLP cascade?

As described in the brief description that "Landmark prediction is replaced by bounding box regression", dose that mean the shape-indexed features are not actually used in the fine MLP cascade as described in the referenced paper?

@hobinson The shape-indexed feature is not used in the open source version.


KOGITO-2710 Correct Jenkinsfile drools

https://issues.redhat.com/browse/KOGITO-2710
Many thanks for submitting your Pull Request :heart:!
Please make sure that your PR meets the following requirements:

[x] You have read the contributors guide
[x] Pull Request title is properly formatted: KOGITO-XYZ Subject
[ ] Pull Request title contains the target branch if not targeting master: [0.9.x] KOGITO-XYZ Subject
[x] Pull Request contains link to the JIRA issue
[x] Pull Request contains link to any dependent or related Pull Request
[x] Pull Request contains description of the issue
[x] Pull Request does not include fixes for issues other than the main ticket

waiting for a full run of the pipeline before merging

@cristianonicolai @evacchi Good to go
https://rhba-jenkins.rhev-ci-vms.eng.rdu2.redhat.com/job/KIE/job/kogito/job/custom-kogito-drools-snapshot/95/console


conda-forge

https://conda-forge.org/docs/maintainer/adding_pkgs/

https://github.com/conda-forge/staged-recipes/pull/28397

scipy-stubs can now be installed through the conda-forge channel, with e.g.
conda install conda-forge::scipy-stubs

See https://anaconda.org/conda-forge/scipy-stubs

Thanks for your help on this, @lucascolley!


dBeaver 6.1.4 Can't Show List of Tables/Views in Database Navigator

System information:
MacOS 10.14.5
dBeaver 6.1.4
Connection specification:

Teradata 16.20

Describe the problem you're observing:
After upgrading to 6.1.4, when I attempt to view the database objects in the Navigator pane, it results in the spinning beachball from which the only way out is to Force Quit dBeaver.
Steps to reproduce, if exist:
Every time I attempt to expand the Connection to see Tables/Views

Include any warning/errors/backtraces from the logs

What version of Teradat driver do you use?
In 6.1.1+ we have upgraded bundled Java version to 11. This may cause problems in old JDBC drivers.
If you use default preconfigured driver (15.0) - please try to download fresh driver from https://downloads.teradata.com/download/connectivity/jdbc-driver

I'm using the 16.20 Teradata driver, but will check for a newer version.  I had no issues like this with 6.1.3.

I decided to try downloading the installer and re-installing from there.  The issue is now resolved for me.  Not sure why updating via Homebrew would have resulted in this issue.


Unclear when viz is actually ready

Sorry to post this here but I don't have a Twitter account to contact you with.
My use case is that I want to apply a filter as soon as I can after the viz has loaded.
I assumed I could use the onCustomViewLoaded callback to call a function that does something similar to your "Add a Filter" example using the useRef to pass a vizRef to TableauEmbed. However, when that callback fires, vizRef.current is undefined.
Is there another callback I should be using?
My current code looks like this:
const addUserFilter = (userId) => {
  if (vizRef && vizRef.current) {
    const viz = vizRef.current
    viz.addFilter('user_id', userId)
  }
}

It helps if I don't gate the TableauEmbed off in a conditional. Sorry for the noise.

No worries! I haven't checked Twitter in ages -- should probably change that. Glad you figured it out.
Is there anything that could be added to the readme to help others who might fall into a similar issue?

I think the difference between addFilter and applyFilterAsync is confusing in the Tableau docs. It's not your library's fault. It talks about using addFilter before initialization but in the React scenario, it's a bit unclear when that would actually occur. The fact that it simply rerenders the viz if you use it afterwards is a clue about what's happening. But, it's not something that might be immediately apparent.
That said, if it would be possible to have a VizFilter component that I could drop inside the TableauEmbed block, that would be really cool. Something like this:
<TableauEmbed url='https://tableau.com'>
  <VizFilter field='username' value={username} />
</TableauEmbed>

That would initialize the filters as written. Then, I could still have the more advanced applyFilterAsync functions for "interactive" filtering if I want them. Anything that mirrors the React components more closely to the "WebComponent" pattern in the Tableau documentation seems like it would be easier for users to make sense of. Well, at least this user. :)


Consider to revert dynamic grow from scroll when reopening

Testing #191496
I think that growing quick chat when you scroll is an interesting idea, but I would reset the size back to how it was when you reopen quick chat because its harder to manage the size to make it small again. You cannot just scroll to get back where you were.

Ahhh I struggle with this one because it might be that you wanna:

scroll back to a former question
switch to your editor
switch back to that former question

which you get with the current behavior. You can get back to the original size if you double click the sash... but that's maybe not obvious.

cc @Tyriar would love your thoughts.

Actually had forgotten that a double click on the sash quickly restores the size, so maybe its not as bad...

I think we'd want some other way to reset as double clicking the sash isn't widely known. Do we reset the height when the question session times out? If so I think it's fine as is.

I guess something like 30 seconds before revert would work alright, either that or some button shows up which lets the user go back to auto.

What are some verification steps for this issue?
Here's what I see so far:

If I ask a question, it starts from the top and starts growing downwards again
If I close the chat and then reopen it a minute later, the answer is still there, and the chat is still expanded. If I ask another question, it starts from the top again, but that's true even if I don't wait 30 seconds
If I manually move the sash, it seems like it's just stuck at that size

Verification steps:

Ask a question like reverse a linked list in TS
Get a long answer:

scroll down to make it bigger

hide it and show it again quickly. It should still look like screenshot 2
hide it and wait 1min and show it again. It should look like screenshot 1


[Wave Collect][QBO] Fix the type mistakes made in the Import main page PR

Fix the type mistakes made in this PR
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01b4b1d75427bcd4d6
Upwork Job ID:<PHONE_NUMBER>909917184
Last Price Increase: 2024-04-12

Doesn't look like @parasharrajat reviewed this, so unassigning. @hayata-suenaga feel free to close it when it's deployed.

#40196 deployed to staging. Once it hits prod, we'll close. 👍

waiting for prod deployment


feat: implement ry openqasm generation

Issue #, if available:
Description of changes:
Implement OpenQASM generation for IR gate (+ demo bug)
Testing done:
tox
Merge Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.
General

[X] I have read the CONTRIBUTING doc
[X] I used the commit message format described in CONTRIBUTING
[X] I have updated any necessary documentation, including READMEs and API docs (if appropriate)

Tests

[X] I have added tests that prove my fix is effective or that my feature works (if appropriate)
[X] I have checked that my tests are not configured for a specific region or account (if appropriate)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

PR was for demo purposes. Closing


React UI: tags support

Added tags support to new UI
Now you can add tags to annotation with button on the sidebar

And edit it with objects panel like other objects

TODO:

[x] Fix filters
[x] Fix Ctrl+C N shortcuts
[x] Add color changing
[x] Fix objects hiding
[ ]  add tags drawing to cvat-canvas

Hi, some found issues:

Probably we should not display buttons "to background", "to foreground" at all for tags.
Filter doesn't work
Changing color for a tag doesn't work
N shortcut to create new tag doesn't work
Ctrl+C, Ctrl+B shortcuts don't work
Hide/show all button doesn't work correctly (try to create tag and hide/show all)
Left/Right arrow buttons (search with a filter) don't work correctly if there is a tag on a found frame (I mean next or previous)

Changing color for a tag doesn't work

Tags are not displayed on canvas so maybe no color changer is needed

@bsekachev fixed issues, can you make review again?

Sure.
A couple of issues with copy:

Ctrl + V works only once after Ctrl + C. Is it expected?
It seems to be attributes aren't copied from source tag.

Hello, can you give an estimation of when the tags would be merged? One week? One month? more?
Thank you,
Best regards

@vinorth05, thank you for your interest in CVAT project. I guess this PR will be finished and merged in few days


not should return a boolean when the argument is a value (not function)

currently not does not accept values as arguments, but if it did
const myObj = { a: 1 }
not('a' in myObj) // false

done in https://github.com/a-synchronous/rubico/pull/211/files


Can you do a Coding Challenge?

I want you to create like circle packing but not with circles. Do that with line.
A line should being drawn. it will be going straight but not straightly (line can only rotate to 0 - 180 degrees). It should keep going until edges (or a color different pixel). It should turn to a angle when the line met edges (or a color different pixel)

@shiffman Hey! Can you create Fluid Simulation in p5.js?


Unable to bind to 'element.mousedown' event when clicking elements. Unable to listen for CMD/CTRL key.

I'm wondering if you can provide advice here.
I am trying to listen to CTRL-click / CMD-click when you click on an element, such as a task.
I tried

  modeler.on('element.click', (event) => {
    const element = event.element;

    if (event.originalEvent.metaKey || event.originalEvent.ctrlKey) {
      // do something
    } else {
      // do something else
    }
  });

But it appears that for whatever reason, when you CMD/CTRL click on an element, it does not trigger the 'element.click' event at all.
I tried disabling the keyboard bindings but that did not help.
I also tried with

  modeler.on('element.mousedown', (event) => {
    console.log('element.mousedown.event', event);
    console.log('element.mousedown.event.originalEvent', event.originalEvent);

    const element = event.element;

    if (event.originalEvent.metaKey || event.originalEvent.ctrlKey) {
      console.log('element.cmd-mousedown.element', element);
      console.log('element.cmd-mousedown.element.incoming', element.incoming);
      console.log('element.cmd-mousedown.element.outgoing', element.outgoing);
    }
  });

But it's a similar issue, when I click on an element, such as a task, it does not trigger the 'element.mousedown'
event at all, with or without modifier keys pressed down
I was able to achieve a workaround by listening for the ALT key instead:
  modeler.on('element.click', (event) => {
    const element = event.element;

    if (event.originalEvent.altKey) {
      // do something
    } else {
      // do something else
    }
  });

The above works.. but I was really hoping to use the CMD/CTRL key instead of the ALT key.
Do you know why the above is not working with the CMD/CTRL key? I believe this should work..
Many thanks!!

I also tried it and found that it does work as you said.
But the problem is that bpmn.js doesn't support such use, and it's hard for me to make changes here.


Set default link address size to 8 bytes

Hi !
https://github.com/zephyrproject-rtos/zephyr/blob/580d789cd0e923124cf81425a93e567631ba32ce/include/zephyr/net/net_linkaddr.h#L37
I have own out-of-tree device drivers and can't define your own ones:
/** Maximum length of the link address */
#ifdef CONFIG_NET_L2_IEEE802154
#define NET_LINK_ADDR_MAX_LENGTH 8
#else
#ifdef CONFIG_NET_L2_PPP
#define NET_LINK_ADDR_MAX_LENGTH 8
#else
#define NET_LINK_ADDR_MAX_LENGTH 6
#endif
#endif
But I need NET_LINK_ADDR_MAX_LENGTH 8 bytes
Can default LINK address size to be set to 8 bytes or have independant configuration for avoid binding with limited set of L2 drivers.

I would not change the default one, but you could propose a patch that allows application to define one. For example something like
diff --git a/include/zephyr/net/net_linkaddr.h b/include/zephyr/net/net_linkaddr.h
index 9e21e7dc9f..857df8f148 100644
--- a/include/zephyr/net/net_linkaddr.h
+++ b/include/zephyr/net/net_linkaddr.h
@@ -34,9 +34,11 @@ extern "C" {
 #ifdef CONFIG_NET_L2_PPP
 #define NET_LINK_ADDR_MAX_LENGTH 8
 #else
+#ifndef NET_LINK_ADDR_MAX_LENGTH
 #define NET_LINK_ADDR_MAX_LENGTH 6
 #endif
 #endif
+#endif

 /**
  * Type of the link address. This indicates the network technology that this

Yes I have own patch as well. But I'm implementing an out-of-tree driver/l2 radio driver CONFIG_NET_L2_XXX and looks like
need to modify each zephyr release because the link address reserved max space is hard coded to the lowest value.
Have it sense to improve configurability at some moment?

If the patch I wrote few comments above is not enough, just propose a PR that adds the Kconfig option you want.

Overall idea is able to have support for several network interfaces with different length of link addresses.
But length is used in hardcoded way https://elixir.bootlin.com/zephyr/v3.3.0/source/include/zephyr/net/net_linkaddr.h#L96
And some common solution need if several interfaces ( at list 3) with different lengths need to be used.

Overall idea is able to have support for several network interfaces with different length of link addresses.
But length is used in hardcoded way https://elixir.bootlin.com/zephyr/v3.3.0/source/include/zephyr/net/net_linkaddr.h#L96
And some common solution need if several interfaces ( at list 3) with different lengths need to be used.


Check for data-interval on the first slide of carousel (#31716)

When starting a cycle for a carousel, it only checks for a default interval, and not an interval defined on the slide element via data props. This adds a check in before creating the interval to move to the next slide.
Fixes https://github.com/twbs/bootstrap/issues/31716

Let me know if you want the patch for V4 too.. https://github.com/twbs/bootstrap/pull/31820

We handle backports ourselves usually by cherry picking. And I'd prefer if we did the same here too. Only the tests need to be adapted, but we'll see after this lands.

We handle backports ourselves usually by cherry picking. And I'd prefer if we did the same here too. Only the tests need to be adapted, but we'll see after this lands.

Ok. FYI this change includes using SelectorEngine, V4 doesn't.

Hmm, true. I guess your v4-dev PR is fine then; I can change the commit message when merging that later.

@mitchellbryson thanks for the PRs! Pretty solid contributions. :)
One last thing before merging: can you please update (or make a new ones) https://codepen.io/Johann-S/pen/zYqeNNO?editors=1010 to print the duration of the first slider too?

@XhmikosR thanks!
Sure, here you go: https://codepen.io/mitchellbryson/pen/poyXXMd

Maybe I'm missing something but I'm getting inconsistent/wrong results. Isn't the second slider's duration supposed to be ~2s? I'm getting ~10s.
https://codepen.io/XhmikosR/pen/yLOdmgE?editors=1111

@XhmikosR  Yep same here, it's not working with transitions. Looking into this now.

The last commit fixes the issue with transitions by setting _activeElement to nextElement at the end of slide() but not after the transition is complete - I'm not familiar enough to know if this is going to be a problem or not so let me know.
A better approach might be to track nextElement on the instance too, but that would require more of a rewrite and I wanted to keep this PR simple until told otherwise.

I checked again with https://codepen.io/XhmikosR/pen/yLOdmgE?editors=1011 and it seems to work right :)

@mitchellbryson I don't suppose you can update the tests so the previous regression is caught? If I didn't try with the Codepen, the patch would have been wrong.

@XhmikosR yep, I think that's a very good idea :)

_slide() no longer deals with updating the _config.interval, it only updates the _activeElement (just like it does with updating the active indicators through _setActiveIndicatorElement()) and now only cycle() updates the _config.interval, based on the current _activeElement.
_updateInterval() now uses an already set _activeElement, or it finds one (first run).
Updates to the tests reflect that next() is no longer dealing with intervals, only cycle() is.

@XhmikosR thinks LGTM here 👍

I'd still like tests to include the previous failed attempt which wasn't caught :/

@XhmikosR is https://github.com/twbs/bootstrap/pull/31818/files#diff-5eae769087ca4f58ffb64ddef2d4b0e1R877 not sufficient? I added the 2nd cycle() call (which was missing before).

@mitchellbryson oh I didn't see that patch, sorry. I think it should be covered now, indeed. @Johann-S please confirm that this covers all cases.

I updated the pen to cover to show the 2nd bugfix working too: https://codepen.io/mitchellbryson/pen/ZEWgBEN?editors=1011

I can confirm that tests fail propery on main BTW :)
Chrome Headless 85.0.4183.121 (Windows 10) Carousel cycle should get interval from data attribute on the active item element FAILED
        Error: Expected 1814 to equal 7.
            at <Jasmine>
            at UserContext.<anonymous> (js/tests/unit/js/tests/unit/carousel.spec.js:898:41 <- js/tests/unit/carousel.spec.js:16886:43)
            at <Jasmine>
        Error: Expected 1814 to equal 9385.
            at <Jasmine>
            at UserContext.<anonymous> (js/tests/unit/js/tests/unit/carousel.spec.js:903:41 <- js/tests/unit/carousel.spec.js:16889:43)
            at <Jasmine>
Chrome Headless 85.0.4183.121 (Windows 10) Carousel cycle should get interval from data attribute on the active item element FAILED
        Error: Expected 1814 to equal 7.
            at <Jasmine>
            at UserContext.<anonymous> (js/tests/unit/js/tests/unit/carousel.spec.js:898:41 <- js/tests/unit/carousel.spec.js:16886:43)
            at <Jasmine>
        Error: Expected 1814 to equal 9385.
            at <Jasmine>
            at UserContext.<anonymous> (js/tests/unit/js/tests/unit/carousel.spec.js:903:41 <- js/tests/unit/carousel.spec.js:16889:43)
            at <Jasmine>
........................................
Chrome Headless 85.0.4183.121 (Windows 10) Carousel next should update the active element to the next item before sliding FAILED
        Error: Expected null to equal
...
.
            at <Jasmine>
            at UserContext.<anonymous> (js/tests/unit/js/tests/unit/carousel.spec.js:656:39 <- js/tests/unit/carousel.spec.js:16779:41)
            at <Jasmine>
Chrome Headless 85.0.4183.121 (Windows 10) Carousel next should update the active element to the next item before sliding FAILED
        Error: Expected null to equal
...
.
            at <Jasmine>
            at UserContext.<anonymous> (js/tests/unit/js/tests/unit/carousel.spec.js:656:39 <- js/tests/unit/carousel.spec.js:16779:41)
            at <Jasmine>
...
 I'd like to get this patch into alpha3 and #31820 assuming you reopen it and adapt it after this one is merged.


Stableswap subgraph

Description

Type of change

[ ] Docs change / dependency upgrade
[ ] Configuration / tooling changes
[ ] Refactoring
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Requires changes in customer code

High-level change(s) description - from the user's perspective

Related Issue(s)
Fixes
Related pull request(s)

subgraph are deployed yet but merging this for future updates


Fix order discount recalculation

This PR adds missing recalulation of the order.undiscounted_total model field. Also it removes duplicated logic from tax/calculate_checkout_shipping which is already done in manager.
Fixes #11002
Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-6391 Simple taxes


Implementation Of Queue.cpp

This is a CPP code regarding Implementation of queue.

ok I am doing it

Mam, the required changes are made please check


Region activations are not sample-accurate

Preferably the region activations should allow to be evaluated at precise frames.
This issue concerns / will concern at least

cc
bend
velocity
channel aftertouch
poly aftertouch

One way to ensure this without introducing too much complexity is just to buffer all "messages" before the renderBlock function is called, sort them appropriately, and perform their associated actions afterwards.
To be clear just in case, it's not that the current version is not "sample accurate", but it assumes that CC and note events are sent sorted for the purpose of region activation (or note stealing for that matter).

Let me try to explain it differently:

if sfizz is requested a cycle 1 second
an activation message on->off arrives at 0.5 second within the cycle

then we have the following expectations, which will not be met in current sfizz

the note-on arrived at time 0.25 must not activate the region;
the note-on arrived at 0.75 does.

Yes unless the received sequence is ordered, i.e.

Note on at 0.25
Activation message at 0.5
Note on at 0.75

API updated in #771


Developer demo loads node worker and breaks

        fetch('https://caffeine.js.org/js/squeakjs/tether/handler.js')
          .then((handlerSourceResult) => {handlerSourceResult.text()
            .then((handlerSource) => {
/*  HERE  */  fetch('https://caffeine.js.org/node/squeak-worker.js')
                .then((virtualMachineSourceResult) => {virtualMachineSourceResult.text()
                  .then((virtualMachineSource) => {
                    window.caffeineWorker = new Worker(URL.createObjectURL(new Blob([virtualMachineSource])))
                    window.caffeineWorker.onmessage = (new Function(handlerSource))()
                    window.caffeineWorker.postMessage('start')})})})})

Trying to run the demo, it fails with

ReferenceError: require is not defined

I think that's because it's trying to load the node worker instead of the squeakjs worker.

Thanks! Fixed in ac1dcc3.


Groovy - "unzip:  cannot find zipfile directory in one of ..."

I've been using sdkman for a while with no problems until now:
Downloading: groovy 2.4.5

  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
  0     0    0     0    0     0      0      0 --:--:--  0:00:02 --:--:--     0
  0 34.7M    0 37744    0     0    663      0 15:14:41  0:00:56 15:13:45   282curl: (18) transfer closed with 36345309 bytes remaining to read
  End-of-central-directory signature not found.  Either this file is not
  a zipfile, or it constitutes one disk of a multi-part archive.  In the
  latter case the central directory and zipfile comment will be found on
  the last disk(s) of this archive.
unzip:  cannot find zipfile directory in one of /Users/willisio/.sdkman/archives/groovy-2.4.5.zip or
        /Users/willisio/.sdkman/archives/groovy-2.4.5.zip.zip, and cannot find /Users/willisio/.sdkman/archives/groovy-2.4.5.zip.ZIP, period.

Stop! The archive was corrupt and has been removed! Please try installing again.

Gradle works with no problem.

We do not host any of the binaries ourselves; all we do is redirect curl to where vendors have published them. In the case of Groovy, it would be on Bintray.
If for some or other reason the binary becomes unavailable, we won't be able to download it and you will see a message as above. If this problem persists, please don't hesitiate to let us know on our Gitter user-issues channel.


Crashing on startup

crash.txt
and
2021_12_05_log.txt
thanks a ton!

And again with this crash. I don't understand what's going on.
 I know that the HX25 grenade launcher makes this happen, among other things.

Fixed! Please redownload the latest version.


Fixes Windows CI: prepend '/c/Program Files/Git/bin'

Cherry-Picked from
https://github.com/facebook/reason/commit/d3822b78b9a8c37594d3d5bff640f9159af0e298:
Fixes Windows CI: prepend '/c/Program Files/Git/bin' (#2611)

Fixes Windows CI: prepend '/c/Program Files/Git/bin'

Bash@3 started failing after a recent change related to WSL
https://github.com/actions/virtual-environments/pull/1081
https://github.com/actions/virtual-environments/pull/1276

Fix slashes

Pick bash from env

closes #73

Thank you!


add netlify.toml and runtime.txt

Codecov Report

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

@@           Coverage Diff           @@
##           master      #29   +/-   ##
=======================================
  Coverage   94.97%   94.97%
=======================================
  Files          18       18
  Lines         717      717
=======================================
  Hits          681      681
  Misses         36       36

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 0f57737...4b4a740. Read the comment docs.


REPEATER isn't working?

I've been using this project for the last few days and it helped me alot.
I was needing something to do like a "Mesh" network, using the ESP32 to receive a package and then send it again to the gateway.
This code has an "REPEATER" option, but It seens like it doesnt do nothing... am I correct?

Yes you are correct.
I am still working on this idea but since there was no interest I have not yet implemented  nor tested its functionality 100%.
Maarten
Verstuurd vanaf mijn iPad
Op 13 jul. 2020 om 13:35 heeft Antonio Emilio <notifications<EMAIL_ADDRESS>het volgende geschreven:
I've been using this project for the last few days and it helped me alot.
I was needing something to do like a "Mesh" network, using the ESP32 to receive a package and then send it again to the gateway.
This code has an "REPEATER" option, but It seens like it doesnt do nothing... am I correct?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fthings4u%2FESP-1ch-Gateway%2Fissues%2F40&data=02|01||6b077c4f20b94c3bdaa908d82720dce8|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637302369376785295&sdata=f7oCfeLubF505Bhiya7Vj34MVwDyimSAMx9o9Xn%2FIl0%3D&reserved=0, or unsubscribehttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABMSRHGP3XKM3T4NZBV7G5LR3LWQRANCNFSM4OYMQGZQ&data=02|01||6b077c4f20b94c3bdaa908d82720dce8|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637302369376795282&sdata=CPNYkONs2q0O8%2B3BXsfbiVEKtE1rH0ue01wmYuC6hwM%3D&reserved=0.

Yes you are correct. I am still working on this idea but since there was no interest I have not yet implemented nor tested its functionality 100%. Maarten Verstuurd vanaf mijn iPad Op 13 jul. 2020 om 13:35 heeft Antonio Emilio <notifications<EMAIL_ADDRESS>het volgende geschreven: I've been using this project for the last few days and it helped me alot. I was needing something to do like a "Mesh" network, using the ESP32 to receive a package and then send it again to the gateway. This code has an "REPEATER" option, but It seens like it doesnt do nothing... am I correct? — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fthings4u%2FESP-1ch-Gateway%2Fissues%2F40&data=02|01||6b077c4f20b94c3bdaa908d82720dce8|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637302369376785295&sdata=f7oCfeLubF505Bhiya7Vj34MVwDyimSAMx9o9Xn%2FIl0%3D&reserved=0, or unsubscribehttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABMSRHGP3XKM3T4NZBV7G5LR3LWQRANCNFSM4OYMQGZQ&data=02|01||6b077c4f20b94c3bdaa908d82720dce8|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637302369376795282&sdata=CPNYkONs2q0O8%2B3BXsfbiVEKtE1rH0ue01wmYuC6hwM%3D&reserved=0.

I got it. I made some adaptations, integrating another code with this one and it worked.
I think it would be a good feature...

@antonio-emilio  Could you share the modified code?
@platenspeler thank you for such a library but why in the repeater mode it still requires the SSID(even when i set the _LOCALSERVER=0)? i would like to test the Repeater functionality in a place where no wifi signal or ethernet exists.


[Fix-8746][UI] Rectify the issue with affecting the original node's data when editing the form of the copied node.

Purpose of the pull request
This PR will close #8746 .

Brief change log

Verify this pull request
This change added tests and can be verified as follows:

Manually verified the change by testing locally.
The test results are as follows:

After copying a node:

When editing the form of the new task node, it seems like:

Now the original node data won't be affected, it seems like:

Codecov Report

Merging #8791 (cad5682) into dev (97a0eed) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                dev    #8791   +/-   ##
=========================================
  Coverage     45.30%   45.30%
  Complexity     4076     4076
=========================================
  Files           702      702
  Lines         27110    27110
  Branches       2897     2897
=========================================
  Hits          12283    12283
  Misses        13674    13674
  Partials       1153     1153

Impacted Files
Coverage Δ

...dolphinscheduler/remote/future/ResponseFuture.java
81.96% <0.00%> (-1.64%)
:arrow_down:

...e/dolphinscheduler/remote/NettyRemotingClient.java
53.52% <0.00%> (+0.70%)
:arrow_up:

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 97a0eed...cad5682. Read the comment docs.


Implement linking planned states to events

This PR adds functionality, which allow the user to:

see conflicting planned states for specified event,
automatically inform the events' manager about conflicting planned states when a new event has been planned,
link planned states to the event,
see states linked to the event,
get detail of the event with detail of linked planned states included,
modify set of linked planned states for events (set, link additional) and
unlink separate or all linked states from the event.

Looking at EventsService now, why exactly do you need global locking when plannig events? You don't have to deal with any collisions between events planning. Remove it.

This was prepared from the #11 for linking planned states, but in between these PRs, we have decided events will be planned separately from planning states, so all of the collisions have been already solved by states planning service. I will remove these.


chore: update the build scripts of the examples

Checklist

[ ] run make test
[x] PR title must follow conventional-commit-standard
[ ] tests and/or benchmarks are included
[ ] documentation is changed or added
[x] commit message and code follows the Code of conduct

Did you check the docs and READMEs of the examples?

Yep


docs: Add example for recognized prefix

Resolves: #176

There appear to be merge conflicts. Perhaps merge with master locally and push a commit? @tituschewxj


fix: use loop to get value . On Get() and Contains()

The get and contains functions, which get the first element of the linked list directly, now get the value by traversing the list

Codecov Report
Patch coverage: 50.00% and project coverage change: +0.17% :tada:

Comparison is base (0b6a00b) 80.17% compared to head (253d1e1) 80.35%.
Report is 28 commits behind head on main.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #124      +/-   ##
==========================================
+ Coverage   80.17%   80.35%   +0.17%
==========================================
  Files          63       64       +1
  Lines        6463     6616     +153
==========================================
+ Hits         5182     5316     +134
- Misses        992      998       +6
- Partials      289      302      +13

Files Changed
Coverage Δ

datastructure/hashmap/hashmap.go
87.75% <50.00%> (-6.50%)
:arrow_down:

... and 8 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

已合并


[FEATURE] Request for bumping up urllib3 version in previous opensearch-py functions

Is your feature request related to a problem?
An issue was discovered in urllib3 before 1.26.5: when provided with a URL containing many @ characters in the authority component, the authority regular expression exhibits catastrophic backtracking, causing a denial of service if a URL were passed as a parameter or redirected to via an HTTP redirect. This request is for opensearch-benchmark.
What solution would you like?
Bumping up the urllib3 version from >=1.21.1 to >= 1.26.5 in previous releases of opensearch-py specifically 2.2.0 and 1.0.0, which are used by opensearch-benchmark versions 1.1 and 1.0.
What alternatives have you considered?
I could hardcode it in the setup.py in opensearch-benchmarks so that it installs the correct version after it installs opensearch-py
https://github.com/opensearch-project/opensearch-benchmark/blob/5a99b0770ab3a0df4145e76f3cacb95dd8118073/setup.py#L61
Do you have any additional context?
Not really. This is just a request, and the issue can be handled easily if denied.

If https://github.com/opensearch-project/opensearch-py/pull/518 goes through, can't you bump the version of opensearch-py you're depending on in opensearch-benchmark?

https://github.com/opensearch-project/opensearch-benchmark/issues/378

Hello @AkshathRaghav and @IanHoang, I've merged the changes to the main branch. Now, we have a few more steps to complete:

Similar changes should be merged into other branches needed by opensearch-benchmarks.
Please note: Check the noxfile.py in the corresponding branches to confirm the Python versions it uses and ensure
they are supported by urllib3 1.26.5.

Release patch versions for the corresponding branches.

Similar changes should be merged into other branches needed by opensearch-benchmarks.
Please note: Check the noxfile.py in the corresponding branches to confirm the Python versions it uses and ensure
they are supported by urllib3 1.26.5.

I assume you mean changing old versions of opensearch-py? I would propose that's not necessary since dependent projects can bump the version they depend on.

Release patch versions for the corresponding branches.

I assume you mean branches of opensearch-py?

@wbeckler @saimedhi To clarify, based on releasing guide and since OSB is using opensearch-py 2.2.0, we would cherry-pick the commit (#518) from main onto 2.X and put out a PR with a label patch and backport?

Hello @IanHoang, after a thoughtful discussion with wbeckler, we've decided to release a new opensearch-py version that incorporates the changes from issue #518. Following the release, please consider updating opensearch-benchmark to use this latest opensearch-py version. We believe this approach is the most effective, and we kindly request your comment on the issue, expressing your desire to release opensearch-py 2.3.2.

Thanks @saimedhi and @wbeckler!


Console Error is thrown when Configure Checks Web view is opened.

Describe the bug
When the Configure Checks webview is opened for the first time on a project , the console errors are thrown
To Reproduce
Steps to reproduce the behavior:

Open zzz6 project using the Open scripture editor form the hamburger menu
Open Configure Checks from the webview menu on zzz6 project
Observe the console errors.

Expected behavior
Should not have any errors in console

This issue is handled in #1159

I have verified the fix. No console errors are thrown.


EXC_BAD_ACCESS at Session

[x] I've read, understood, and done my best to follow the *CONTRIBUTING guidelines.

What did you do?
Create MoyaProvider init with initializing default Session.
The project tested from clean state, previously removing Derived Data.
What did you expect to happen?
The same flow as was in previous version, v4.9.1.
What happened instead?
Getting the error: Thread 1: EXC_BAD_ACCESS (code=1, address=0xfffffffffffffff8),
at the Session.swift:313 line, regarding access to the convertible, in DataRequest creation.
Alamofire Environment
Alamofire version: 5.0.4
Xcode version: Version 11.3 (11C29)
Swift version: 5.1.3
Platform(s) running Alamofire: iOS 10.3
macOS version running Xcode: 10.15.3 (19D76)

In order to properly investigate a crash like this, we need a few things. These include:

The crashing call stack from your crash report.
Code snippets of your usage relevant to the crash report.
Ideally, a sample project which replicates the issue.

Without those we're investigating in the dark. My brief research has shown that crashes at address=0xfffffffffffffff8 are usually due, at least in part, to compiler bugs or other issues in the language.

Very strange, I got this error when I use Alamofire as dependency for an SDK, and cocoapods installs it via .podspec.

To me that definitely points to some sort of compilation issue. If you build the same thing using SPM, do you see the same issue?

Another possibility would be #3051 if you're using a free developer account.

If I install dependency and use it directly in the application, there is no any real time crash error with it.
Only in the case, when I use this through SDK. As I've noticed the bug wasn't not exist with version 4.9.1.

Another possibility would be #3051 if you're using a free developer account.
Not my case.

I have this same issue, exc_bad_access in Session. It occurred after I moved AlamofireImage and Alamofire off Carthage and into their own separate frameworks, built with release configuration for distribution and as static libraries.

Hi @michaeleisel, did you try with v5.0.5?

Was there a fix for this issue?

It can be, but in my case, I've updated other details also, like RxSwift for Moya etc.

You're saying that, at one point, it fixed the issue, but then you made other changes that caused the error to surface again? What specifically changed up to 5.0.5 that helped this issue?

There are no changes in 5.0.5 that would affect this. It's most likely some sort of build or linking issue that affects your projects specifically. You aren't trying to build with library stability or anything are you? And if anyone can make a small project that reproduces this issue that'd be great.

I am indeed building with library stability, so that this static library can be used across dev machines

Alamofire doesn't support library stability and likely never will due to the restrictions it places on library changes. It's generally unsafe to turn on library stability for libraries that don't support it, and it isn't generally recommended unless you're shipping your own binary products. If you are shipping binary products, all of your dependencies must also support library stability mode or you can run into issues like this. If you want to share static binaries locally, you don't need library stability, you just need to ensure your machines are using the same version of the Swift compiler.

I see. If there was some way to fail fast and alert the dev about this, that'd be great. My impression was that there weren't really caveats like this, and I imagine a lot of other devs might be surprised by this as well. Also, +1 to reconsidering that stance on module stability. You can see that module stability is important to lots of devs, and that the workaround is a major tradeoff, from the comments starting at https://github.com/Carthage/Carthage/pull/2902#issuecomment-574692640

There is no tooling to detect breaking changes, which is one of the reasons Alamofire won't be supporting the mode. There are a host of drawbacks to enabling library evolution mode, as outlined in this blog post. Like I said, unless you're shipping a binary library to the public, you don't need to enable library evolution, and Apple doesn't recommend it. ABI stability is enough for CI and other local build system to produce static artifacts that are compatible across machines, as long as the compiler version is the same.

You're saying that, at one point, it fixed the issue, but then you made other changes that caused the error to surface again? What specifically changed up to 5.0.5 that helped this issue?

but then you made other changes that caused the error
Where had I said that?

@michaeleisel

You're saying that, at one point, it fixed the issue, but then you made other changes that caused the error to surface again? What specifically changed up to 5.0.5 that helped this issue?

Where had I said that?

@dimpiax disregard

@jshier it makes a lot of things about tooling more difficult. For example, build systems can't cache the Alamofire library as a build artifact, unless they cache it for each compiler version. Or you could limit people to just one compiler version, e.g. Swift 5.1 which seems to be in Xcode 11.0-11.3. But then people start using Xcode 11.4, or some future beta to make sure that their app works on it, and you have problems.

Yes, it would be up to the individual projects to manage their build artifacts and process, including Xcode versions. That's a choice made by those projects. At least in that case there will be a nice build time error saying that the static products were built by a different version of the Swift compiler, making it much easier to see and fix the issue than the library evolution crashes reported here.

I think we've discovered the cause of the issue, so I'm closing this issue.

@jshier could you provide the cause?

@dimpiax It seems likely that your SDK was compiled with library evolution mode enabled, which Alamofire doesn't support and can cause crashes if a non-resilient change is made to the library.


Kernel doesn't start (Docker/Stack installation)

Hi, I'm using Docker with stack, I followed the instruction for installation, everything was OK but when I start Jupyter, the kernel doesn't seem to boot ... here is the message I get ...
thanks for your help because I really don't know what to do ....
kondor@devcom:~/dev/containers/IHaskell$ stack exec jupyter -- notebook --NotebookApp.token='' notebooks
[W 15:18:33.933 NotebookApp] All authentication is disabled.  Anyone who can connect to this server will be able to run code.
[I 15:18:33.936 NotebookApp] Serving notebooks from local directory: /home/kondor/dev/containers/IHaskell/notebooks
[I 15:18:33.937 NotebookApp] The Jupyter Notebook is running at:
[I 15:18:33.937 NotebookApp] http://localhost:8888/
[I 15:18:33.937 NotebookApp] Use Control-C to stop this server and shut down all kernels (twice to skip confirmation).
[W 15:18:33.940 NotebookApp] No web browser found: could not locate runnable browser.
[W 15:18:47.258 NotebookApp] 404 GET /api/kernels/c23e7590-40db-44c5-8abd-c342d53b7ad9/channels?session_id=cb73a4f4cc72437a828f5c851b585a31 (<IP_ADDRESS>): Kernel does not exist: c23e7590-40db-44c5-8abd-c342d53b7ad9
[W 15:18:47.315 NotebookApp] 404 GET /api/kernels/c23e7590-40db-44c5-8abd-c342d53b7ad9/channels?session_id=cb73a4f4cc72437a828f5c851b585a31 (<IP_ADDRESS>) 75.13ms referer=None
[I 15:18:47.316 NotebookApp] 302 GET / (<IP_ADDRESS>) 70.04ms
[W 15:18:50.349 NotebookApp] Notebook Conjugate Gradient.ipynb is not trusted
[I 15:18:51.225 NotebookApp] Kernel started: e80b6ee9-03bb-48e3-b61a-f6cd758c4cb3
[W 15:18:51.255 NotebookApp] 404 GET /nbextensions/widgets/notebook/js/extension.js?v=20200217151833 (<IP_ADDRESS>) 3.80ms referer=http://localhost:8888/notebooks/Conjugate%20Gradient.ipynb
This command must be run on host OS (not in a Docker container).
ihaskell: readCreateProcess: stack "exec" "env" (exit 1): failed
[I 15:18:54.223 NotebookApp] KernelRestarter: restarting kernel (1/5), new random ports
This command must be run on host OS (not in a Docker container).
ihaskell: readCreateProcess: stack "exec" "env" (exit 1): failed
[I 15:18:57.235 NotebookApp] KernelRestarter: restarting kernel (2/5), new random ports
This command must be run on host OS (not in a Docker container).
ihaskell: readCreateProcess: stack "exec" "env" (exit 1): failed
[I 15:19:00.250 NotebookApp] KernelRestarter: restarting kernel (3/5), new random ports
This command must be run on host OS (not in a Docker container).
ihaskell: readCreateProcess: stack "exec" "env" (exit 1): failed
[I 15:19:03.267 NotebookApp] KernelRestarter: restarting kernel (4/5), new random ports
This command must be run on host OS (not in a Docker container).
ihaskell: readCreateProcess: stack "exec" "env" (exit 1): failed
[W 15:19:06.283 NotebookApp] KernelRestarter: restart failed
[W 15:19:06.283 NotebookApp] Kernel e80b6ee9-03bb-48e3-b61a-f6cd758c4cb3 died, removing from map.

The pb seem to come from
ihaskell readCreateProcess stack "exec" "env" (exit 1) failed

??

I get the message "This command must be run on host OS (not in a Docker container)."
when I try to do "stack exec env" inside the container ... but when I use the command specified in the documentation i.e.
stack --docker exec jupyter -- notebook --NotebookApp.token='' notebooks
it doesn't work "see my first message"

To be clear, what happens when you run
stack --docker exec jupyter -- notebook --NotebookApp.token='' notebooks

?
Your first message omits the --docker parameter.

If you just want a Docker container with IHaskell pre-installed, you might consider https://github.com/jamesdbrock/ihaskell-notebook

Closing this, please reopen if you're still having issues.


Good Job!!

    Good Job!!

I will suggest you make your input more controlled to accept only positive integers
Originally posted by @j-tee in https://github.com/LIBERCOSOFT/leaderboard/pull/2#discussion_r994277193

Hi @LIBERCOSOFT , Good Job with your project 👍,
However, I'm suggesting that it would be better to have a use of exporting and importing of modules in your project.


[Feature Request] Field Addons / Inline Forms

We'd like to be able to add icons / glyphs / etc. in input-group-addons to the forms as described here:
ASF Issue 273.
If we implemented this for -datepicker and submitted a pull request, would the maintainers be amenable to merging it?  The goal is to be able to attach an icon to the field like so:

I don't see why not.

Uploaded #18

#18 has been merged

Thank you sir.  Any thoughts on when the next release might occur, so that bower users could get this patch?

Not sure when, we have a lot of stuff going on at work. We'll get to it asap.
On Wed, Apr 1, 2015 at 5:05 PM, Erik Dykema<EMAIL_ADDRESS>wrote:

Thank you sir.  Any thoughts on when the next release might occur, so that bower users could get this patch?
Reply to this email directly or view it on GitHub:
https://github.com/Textalk/angular-schema-form-datepicker/issues/17#issuecomment-88516853


[Feature] Easily get pixels coordinates from highlight event on line charts

What problem does this feature solve?
I was working on a label position switch from top to bottom based on the highlighted point coordinates — in order to avoid the label to overflow the chart area. I tried different things, like tweaking the tooltip configuration since its position property provides many information ...But the coordinates are the mouse's and not the highlighted point.
Unless I didn't find the right way to do this, I had to first combine the seriesIndex and dataIndex with the getOption method, fetch the value and use it along with seriesIndex with the convertToPixel method to finally get the coordinates. It's not obvious and it feels like something is missing.
What does the proposed API look like?
Currently, the information provided by the highlight event on line charts is the following, it's very tiny...
{
  "type": "highlight",
  "escapeConnect": true,
  "batch": [
    {
      "seriesIndex": 0,
      "dataIndexInside": 24,
      "dataIndex": 24,
      "type": "highlight",
      "escapeConnect": true,
      "notBlur": true,
      "batch": null
    }
  ]
}

Is it possible to add the value here so we can skip the getOption part? Or even better, having a dynamic positioning of labels in the chart config ? Something similar to the tooltip's ^^

Here's the code snippet I ended up with ftw
  highlightPortfolio(event: ECElementEvent & { batch: LabelLayoutOptionCallbackParams[]}) {
    const { seriesIndex = 0, dataIndex = 0 } = event.batch[0];
    const height = this.chartInstance.getHeight();
    const options = this.chartInstance.getOption();
    const { value } = (options['series'] as any)[seriesIndex].data[dataIndex];
    const pixels = this.chartInstance.convertToPixel({ seriesIndex }, value as number[]);

    const isLow = pixels[1] > height / 2;

    this.chartInstance.setOption({
      series: {
        emphasis: {
          label: {
            position: isLow ? 'top' : 'bottom',
            offset: isLow ? [0, -5] : [0, 5],
          },
        },
      },
    });
  }


Start up probelm

After the plugin is installed and configured, one extra restart is needed. I don't know why right now :(

Finally fixed!


New version: StandardNotes.StandardNotes version 3.143.0

Previous version: 3.142.8
Created with Komac v0.9.0
Microsoft Reviewers: Open in CodeFlow

/azp run


feat: add  multi-sign docker

Codecov Report

Merging #94 (84ab480) into main (472d691) will decrease coverage by 3.16%.
The diff coverage is n/a.

:exclamation: Current head 84ab480 differs from pull request most recent head 1a8ad76. Consider uploading reports for the commit 1a8ad76 to get more accurate results

@@            Coverage Diff             @@
##             main      #94      +/-   ##
==========================================
- Coverage   41.26%   38.10%   -3.17%
==========================================
  Files          23       17       -6
  Lines        1958     1732     -226
  Branches       48        0      -48
==========================================
- Hits          808      660     -148
+ Misses       1150     1072      -78

Impacted Files
Coverage Δ

...th-contracts/contracts/libraries/MultisigUtils.sol

...les/eth-contracts/contracts/libraries/SafeMath.sol

...es/eth-contracts/contracts/libraries/SafeERC20.sol

...ules/eth-contracts/contracts/libraries/Address.sol

...in-modules/eth-contracts/contracts/ForceBridge.sol

...ain-modules/eth-contracts/contracts/test/ERC20.sol

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 472d691...1a8ad76. Read the comment docs.


Unable to read older pytorch pickle files

It seems candle's pickle support currently assumes the newer pickle format torch switched to some time ago. (At least, I'm assuming this is the case). This is a slight issue as many of the .pth files I'd like to be able to load using candle are using this older format.
Specifically, I'm getting a Zip(InvalidArchive("Could not find central directory end")) when trying to use pickle::read_all on many older .pth files. Newer ones seem to work fine, but those are also unusable due to the load key issue I've mentioned previously.
My hunch is that this is due to torch switching how they pickle their path files. I remember at some point torch.save had an option to use the newer zip format, so I'm guessing these old ones are using something other than zip to store the pickle data, and thus the zip library can't read it properly. I'm not sure how to confirm this though.
I've added support for .pth reading to my candle ESRGAN repo, where this can be pretty easily recreated. One of these model files should trigger it, as they are older files:

https://openmodeldb.info/models/4x-BS-Deviance
https://openmodeldb.info/models/4x-FSDedither
https://openmodeldb.info/models/4x-DigiPaint
https://openmodeldb.info/models/4x-Manga109Attempt

If you need a smaller repro, I can try to provide one when I have time tomorrow.

Did a little digging, and here is the legacy model loading code in pytorch, for reference

Thanks for reporting this, I don't think the issue is within the zip crate as the files appear not to be in the zip format but rather just be a stream of serialized pickle. E.g. the following manages to read the start of the file (but fails with some torch error that I haven't investigated).
import pickle
filename = '4x_Deviance_60000G.pth'

def persistent_load(persid):
    print(persid)

with open(filename, 'rb') as fobj:
    p = pickle.Unpickler(fobj)
    p.persistent_load = persistent_load

    while True:
        data = p.load()
        print(data)

In principle, we should be able to tweak the candle_core::pickle module to try to read this format but it's tricky to know how hard that will end up being without giving it a try (as we don't support the whole space of pickle files at all). As this format seems deprecated compared to the new zip based approach, I would be tempted not to spend time supporting it but if you want to give it a try, certainly feel free.


Add missing Leafdoc template files

Fix https://github.com/Leaflet/Leaflet/issues/4556
The destructor.hbs file was listed in Leafdoc so I've also added it, but it's actually not used by any Leaflet comment yet. I can remove it if we decide it's really useless.

Let's leave it for now. The real fix is to make Leafdoc not load templates if they are not needed.


update CONTRIBUTING.md

I would like to add following things in contributiong.md
1.Difference between GIT and GITHUB
2.How to clone,fork repository
3.How to create a branch and then use git push to push to repo
4.Create a PR
5.Squash commits in a single issue into one
6, Updating the forked and local repo as the updations are made in the upstream

Hi @khareyash05 can you assign me this I Can work on this if it is still available

Any updates about this?

I thought adding best practices could help a little more to new comers

Updated the contributing.md


[Feature Request] Essence of Delirium weapon calculation

Hi !
First of all, big thanks for great tool ! really have been enjoying this tool. Would like to know if there is already calculations for delirium weapons on tool somewhere or could you add one for it when you have time and energy :) ?

Not yet. I was putting it off since I didn't think essences would be staying. I need to find the time to work on it; I'm not sure how easy it will be to support.

Wouldn't you just handle it like bleed and show it separately since it's a non-stacking dot?

Yeah, more or less. Now that 1.3.0 is out I should be able to spare the time to implement it. We'll see.

The next update will add support for Decay.


Why doesn't worker support cloud native way

This is just an idea.
Why doesn't st2 support cloud native way.
Works run like kubernetes jobs, pack can be a docker image.

Related talk from our very own @dzimine: https://www.youtube.com/watch?v=GApuwM9tz-Q

Related talk from our very own @dzimine: https://www.youtube.com/watch?v=GApuwM9tz-Q


Add support for RSA-OAEP 256 (#1293)

This PR adds support for RSA-OAEP-256, as mentioned in RFC 7518 (https://datatracker.ietf.org/doc/html/rfc7518#section-4.1) and fixes Issue 1293

@microsoft-github-policy-service agree

@brentschmaltz can you review when you have time?

I removed all the ifdefs since they were effectively "> net45", and no longer needed

@karoberts sorry to get to this so late, but we would like to take this.
We can build on your work OR you can rebase on top of dev.

@karoberts sorry to get to this so late, but we would like to take this. We can build on your work OR you can rebase on top of dev.

Sounds great!  I have rebased on top of latest dev.

@karoberts Awesome, we are trying to get this into IdentityModel 8 that is coordinated with .net 9.

@karoberts Thanks.


preview with converted files

This Pull Request is used for previewing the project with all documents included

no longer required. see oi-wiki/oiwiki-bootstrap


What happens if classification_errors are all ones?

Does this not mean that best_error is one, and so the log function returns -inf?  What happens then?
Thank you for your work on this repo!  Great stuff.

This was incorrect.


openlibrary search prototype

Search query field and button in toolbar.
Search against openlibrary
render result listview

(skipping openplatform search for now)

using openlibrary for bibliographic data for now.

Noting that the openlibrary api search is marked as "experimental" . and it is a bit sluggish  right now

Improve list view here : #17

look into search performance : #18


Cleaning/moving folders & renaming

Created new jupyter notebook in VS code. So I deleted unused colab notebook. Now we have cleaned notebook to work further. I moved my new cleaned airB&B jupyter notebook in notebooks folder.

The data set looks much nicer than the raw data. Should be good to work with.