TNT-40078 - TargetDecisioningEngine + disabled integration tests

TargetDecisioningEngine - top level object that will be called by get_offers when ODD is enabled
Integration testing framework, artifacts, and models.  Everything under tests/schema/artifacts and tests/schema/models was pulled in from https://github.com/adobe/target-sdk-testing via subtree.

Integration tests will be enabled as part of upcoming DecisionProvider PR

Also, I thought you set up coverage reports with coveralls as part of a GitHub action?  I'd like to see coverage reporting in these PRs.  One of the goals/expectations of SDK development is to maintain at least 80% test coverage.  That metric should be shown somewhere in PRs automatically.
see https://wiki.corp.adobe.com/display/Target/Target+SDKs+Implementation+Guide#TargetSDKsImplementationGuide-Requirements

lions, tigers, and test files oh my! 😆
looks fine to me. I assume you will have a separate PR for the decisioning provider implementation with passing tests?

Yes - my next PR will include the DecisioningProvider and enable these test cases

lions, tigers, and test files oh my! 😆
looks fine to me. I assume you will have a separate PR for the decisioning provider implementation with passing tests?

Yes - my next PR will include the DecisioningProvider and enable these test cases

Also, I thought you set up coverage reports with coveralls as part of a GitHub action? I'd like to see coverage reporting in these PRs. One of the goals/expectations of SDK development is to maintain at least 80% test coverage. That metric should be shown somewhere in PRs automatically.
see https://wiki.corp.adobe.com/display/Target/Target+SDKs+Implementation+Guide#TargetSDKsImplementationGuide-Requirements
Here is the reporting page for the nodejs SDK for comparison https://coveralls.io/github/adobe/target-nodejs-sdk

The coverage check is being done, but for some reason the reporting into coveralls is broken.  Troubleshooting that is on my TODO list.  But the PR will still fail if it drops below 80%


Include verification of scroll tabs to prevent avatar animation

Hi Diego,
I attended the first Flutter Talks that happened in Sao Paulo where you lectured.
There I met your github and really liked the twitter clone repository, but I found a bug when I needed to scroll tabs.
In the example, we have only 4 tabs, which do not enable scrolling. In the adjustment I was testing, I needed to include more tabs and when I needed to scroll them, I noticed that the code was calculating all axes. Any scroll within Notification started the avatar animation.
To prevent animation, I include an if if checking the scroll is left-> right or right-> left.
If true, then it does not start avatar animation, keeping the header the same.
If the scroll is vertical, it starts the animation normally, keeping the initial idea of the project.
Here are videos highlighting the bug and another showing how the if solved this problem.
Bug - https://www.dropbox.com/s/fakjc3drzysqbb1/bug-scrolling-tabs.gif?dl=0
Fixed - https://www.dropbox.com/s/bh8jmbnyr0331zk/bug-fixed-scroll-tabs.gif?dl=0
I hope to have helped in this project, even just for examples. = D
Thanks

great! thanks for fix this issue :)


Building tflite with xnnpack on windows 10

hello team,
i wanted to use xnnpack with tflite on my windows 10 machine. found out that we need to build tf from source with tflite_with_xnnpack=true flag. i was not able to build successfully ,and I am not sure I am following the correct steps or not as I didn't get proper installation guide for building tf with xnnpack on windows. It would be helpful if I get any documentation or resource to build and use tflite with XNN pack

@tilakrayal ,
thank you,will try and update soon

Hi, Could you please follow this link to build the tflite with TFLITE_ENABLE_XNNPACK=ON for Windows and let us know if you still face an issue. Thanks!


Small update

Fix some typos.

@DouYishun This is good to merge once you sign the CLA. Thanks again! :-)


non-AMD releases of operator-sdk do not seem to behave the same as AMD release

Can someone check both the arm64 and ppc64le releases of operator-sdk for correctness? Specifically, I'm using 0.17.0 from here and those two arch builds fail to validate the exact same CSV that the amd release pass (and I know the CSV is OK - should pass on all arches). Note that this is attempting to validate a manifest that is using the "new bundle format" (this is why travis is not using operator-courier, but instead is using operator-sdk validate).
My question is - is this behavior correct? Or is there something wrong with the SDK on those two arches?
Here's the details:
Take a look at the Kiali operator's travis builds here. Notice one passed (that's the amd one) but two fail (the arm64 and ppc64le ones).
Here are the logs for the passing amd build
Here are the logs for the failing arm64 build
Here are the logs for the failing ppc64le build
Each build utilizes the same Makefile to download operator-sdk and run "validate" on the same CSV yaml. The Makefile downloads the SDK here - it is just this command:
curl -L https://github.com/operator-framework/operator-sdk/releases/download/v0.17.0/operator-sdk-v0.17.0-$$(uname -m)-linux-gnu > "${OUTDIR}/operator-sdk-install/operator-sdk" ;\

Looking at the three travis build logs, notice that they are all downloading the correct 0.17.0 version for the particular arch needed - the travis builds run operator-sdk version and here is what they say for each arch build:

the successful amd build: operator-sdk version: "v0.17.0", commit: "2fd7019f856cdb6f6618e2c3c80d15c3c79d1b6c", kubernetes version: "unknown", go version: "go1.13.10 linux/amd64"
the failed arm64 build: operator-sdk version: "v0.17.0", commit: "2fd7019f856cdb6f6618e2c3c80d15c3c79d1b6c", kubernetes version: "unknown", go version: "go1.13.10 linux/arm64"
the failed ppc64le build: operator-sdk version: "v0.17.0", commit: "2fd7019f856cdb6f6618e2c3c80d15c3c79d1b6c", kubernetes version: "unknown", go version: "go1.13.10 linux/ppc64le"

They are all 0.17.0 with the same commit and the go version shows the correct arch for each.
The arm and ppc builds fail to validate the CSV, but the amd build successfully validates the same CSV. Here's the CSV that is getting validated (note: travis passes in the proper env vars that get replaced via envsubst. You can see travis doing it for all three builds here, here, and here). Here, also, is the metadata/annotations.yaml file (this becomes important later).
Each travis run of "operator-sdk validate" are:

amd results in operator-sdk validate exiting with result code 0
arm64 results in operator-sdk validate exiting with result code 1
ppc64le results in operator-sdk validate exiting with result code 1

So the amd build succeeds - and take note of this log message from that run:
DEBU[0000] Found annotation "operators.operatorframework.io.bundle.mediatype.v1" with value "registry+v1"  bundle-dir=/home/travis/build/kiali/kiali-operator/manifests/kiali-ossm

So, it passes validation for that value of mediatpe.v1 of "registry+v1" found in the annotations.yaml.
But look at the log messages for the arm64 and ppc64le builds for that same line in annotations.yaml - they both fail with the same error message:
time="2020-07-10T08:23:45Z" level=fatal msg="Bundle format validation failed: Bundle validation errors: Expecting annotation \"operators.operatorframework.io.bundle.mediatype.v1\" to have value \"plain\" instead of \"registry+v1\"" bundle-dir=/home/travis/build/kiali/kiali-operator/manifests/kiali-ossm

Why does the SDK pass validation on amd arch, but fail validation on arm64 and ppc64le when the metadata is exactly the same for all three builds?

Seems like the problem could be that the arm and ppc travis VMs don't have envsubst installed in them (see https://travis-ci.com/github/kiali/kiali-operator/jobs/359634586#L247).
I was not able to reproduce this issue with v0.17.0 (and newer) on a raspberry pi 4.


4.1.2 Foreground completion handler crash

iOS 15.1.1, RNN 4.1.2, RN 0.64.1. Whenever I receive a push notification while the app is in the foreground, it crashes:

My handler code isn't doing anything out of the ordinary:
   RNNotifications.events().registerNotificationReceivedForeground((notification, completion) => {
      logging.info('[NotificationService] Notification Received - Foreground', notification)

      uiStore.setIsUnreadNotification(true);
      logging.info('[NotificationService] Notification Received - Foreground', 'setIsUnreadNotification() done');

      // Calling completion on iOS with `alert: true` will present the native iOS inApp notification.
      completion({ alert: true, sound: false, badge: false });
      logging.info('[NotificationService] Notification Received - Foreground', 'completion() done');
    })

I rolled back to 3.1.2 on a hunch (#746 mentioned crashes started after upgrading from 3.1.2) and the crashes have gone away. I'm just concerned about not being able to run up-to-date libraries.

Added note, when I accidentally downgraded to 3.5.x (Because I put ^3.1.2 in my package.json) the same crash happened. It was a different line number but still the same.

For me an issue occurred on iOS if we'd accidentally register multiple times

For me an issue occurred on iOS if we'd accidentally register multiple times

I bet that's my cause, too. I eventually did figure out I was registering multiple times, but never tried to re-upgrade to this version. Guess it's time!

For me an issue occurred on iOS if we'd accidentally register multiple times

Live saver! Thanks a bunch for this insight.


Cascading deletion strategy

Description
After https://github.com/kyma-project/istio/issues/128 is implemented Istio module deletion will not be possible unless custom resources present on cluster are manually deleted. Implement cascading deletion strategy which will delete ALL Istio resources, including customer created ones.
Reasons
Modularisation of Istio component
To discuss

Providing the trigger for cascading strategy

DoD:

[ ] provide documentation
- [ ] release notes and What's New updates for Kyma customers
[ ] provide unit tests
[ ] provide integration tests
[ ] test on production-like environment
- [ ] verify resource limits
[ ] followup issue

there's nothing to be done on module side. if deprovisioning will timeout additional process will clean up blocking resources and Istio module will be deleted


fix(accordion): panel checkbox collapsing panel on check

#2537
What I did

Added an index check to the collapse function so that if the element that called the change event is not an accordion-* element it will not cause the panel to close.

Can we fix this with an instanceof check on the change event?

Can we fix this with an instanceof check on the change event?

Good point, that would be simpler.  Went ahead and added the check & removed some of the code that was deprecated from using RTI for accordion.

@brianferry take a look at f47a340: i had in mind to check to see if the event was an instance of our change event.
I added a test for pf-switch and moved the setup code for the tests into the template and beforeEach blocks.


participant games card fixes #342

Please fill out the template below

Did you read the instructions in README ?
Yes
Solves which issue ?
Fixes #342
Give discription.
[Write Here. Be Specific]
Checklist
(To mark the checkboxes use x within the box brackets)

[x] Signed the code of conduct.

[x] Have you added yourself to the participants.

[x] Joined opencode zulip server.

@ananya-agrawal @prateek76 please review.

@anupm12 also add itch.io icon
icon link - https://itch.io/press-kit
also games card icon scaling and transition time looks good apply same in other pr

@prateek76 ok

@anupm12 don't add itch icon for now


Fix SkeletonIK 3D Scaling Bug

Some of us are aware of the 3D SkeletonIK Scaling Bug. This happens because the scale is reapplied to the skeleton. With this fix, the reapplied scale is removed. Not sure if it applies to 4.0 as there are issues with skins in that version last I checked. This fix was tested on a version around the pull request when the AspectRatioContainer node was added with scales of 0.011, 1, and 3

@Kylogias By "3D SkeletonIK Scaling Bug", are you referring to https://github.com/godotengine/godot/issues/40466?

@Kylogias By "3D SkeletonIK Scaling Bug", are you referring to #40466?

Yes, I didn't know the bug report number at the time of writing the request

The proposed fix is also included in #40774 which hasn't been merged yet because it's pending review (CC @TwistedTwigleg). So if #40774 is merged it will supersede this PR.

Superseded by #40774. Thanks for the contribution nevertheless!


'This page isn’t working' and 'HTTP500' errors

Opened a new ticket as am getting this error and have read the other threads here for a solution but have not found one that works for me.
When accessing the app I get:
    This page isn’t working
    XXXXXX.azurewebsites.net is currently unable to handle this request.
    HTTP ERROR 500

Am using the:

Hub primary connection key for the Application settings->APP SETTING NAME->Azure.IoT.IoTHub.ConnectionString value.
correct consumer group name for Application settings->APP SETTING NAME->Azure.IoT.IoTHub.ConsumerGroup value
Application settings->APP SETTING NAME->WEBSITE_NODE_DEFAULT_VERSION = 8.9.4

Amy ideas of what I can try next please?

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

ID: 681e10a1-05d3-5761-2afd-42602a7efecf
Version Independent ID: f72e21b1-3530-8554-f1f1-fc66b222a485
Content: Real-time data visualization of sensor data from your Azure IoT hub – Web Apps
Content Source: articles/iot-hub/iot-hub-live-data-visualization-in-web-apps.md
Service: iot-hub
GitHub Login: @rangv
Microsoft Alias: rangv

Thanks for the feedback! We are currently investigating and will update you shortly.

Update: deleted the original webapp. Started over using:
New git app repository and app setup.
Hub primary connection key for the Application settings->APP SETTING NAME->Azure.IoT.IoTHub.ConnectionString value.
correct consumer group name for Application settings->APP SETTING NAME->Azure.IoT.IoTHub.ConsumerGroup value
Application settings->APP SETTING NAME->WEBSITE_NODE_DEFAULT_VERSION = 8.9.4
Now app is working but with an empty graph

Update: App is working but the graph itself works very intermittently. The displayed data is wrong/corrupt even with the Pi receiving valid temp and humidity data. I will investigate some more but it is very unstable in my view.

Update: if you open the webpage (am using Chrome) in the browser it displays as a blank graph. If you leave the page open and come back to it about an hour later it does display graph information though it is not the same info as displayed within the BI console. If you refresh the same page the graph information is removed and a blank graph is again displayed.

Does not work on Apple iOS/Safari - always displays a blank graph.

OK. I get it now. Doh!

When the graph is accessed for the first time it will be blank. This is expected behaviour.
Graph app then starts to pull in data from the BI. This will take several minutes. Graph then starts to plot the data.
After 30 mins the graph will display actual live data, and will update accordingly.

Pity this small but critical behaviour was not included in the instructions above.

Thank you for all the troubleshooting steps @ddentrec :). I believe that your comments will definetely help others facing the same issue!
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Missing tag 0.15.0

Release 0.15.0 is on Hex, but the tag is missing on github. Could you please add it?

Done

Thanks!


cmd/gc: seems expand stack uneccessary

What version of Go are you using (go version)?

1.14

Does this issue reproduce with the latest release?
Yes
The problem:
run go tool compile -S a.go
the function
func a() uintptr {
    var x [1024]uintptr
    for i := 0; i < len(x); i++ {
        x[i] = uintptr(time.Now().Unix())
    }
    return x[0]
}

output as:
"".a STEXT size=239 args=0x8 locals=0x2028
	0x0000 00000 (a.go:6)	TEXT	"".a(SB), ABIInternal, $8232-8
	0x0000 00000 (a.go:6)	MOVQ	(TLS), CX
	0x0009 00009 (a.go:6)	MOVQ	16(CX), SI
	0x000d 00013 (a.go:6)	PCDATA	$0, $-2
	0x000d 00013 (a.go:6)	CMPQ	SI, $-1314
	0x0014 00020 (a.go:6)	JEQ	229
	0x001a 00026 (a.go:6)	LEAQ	896(SP), AX
	0x0022 00034 (a.go:6)	SUBQ	SI, AX
	0x0025 00037 (a.go:6)	CMPQ	AX, $9000
	0x002b 00043 (a.go:6)	JLS	229
	0x0031 00049 (a.go:6)	PCDATA	$0, $-1
	0x0031 00049 (a.go:6)	SUBQ	$8232, SP
	0x0038 00056 (a.go:6)	MOVQ	BP, 8224(SP)
	0x0040 00064 (a.go:6)	LEAQ	8224(SP), BP

I don't know why 768 more bytes is needed in the following line:
0x0025 00037 (a.go:6)	CMPQ	AX, $9000
Should't it be CMPQ	AX, $8232?
In some cases when there less than 9000 bytes but more than 8232,  stack expand is triggered unecessarily

I dont think reserving 8232 bytes for the stack will suffice because later the functiona calls  time.Now(SB)  which itself will need stack space ontop of the used 8232. time.Now  then calls other functions e.g. time.local ... . And this continues recursively. Reserving just 8232 will trigger another stack growth in the other leave functions.

Feel free to reopen the issue if you think there is a bug here and the above explanation why this is more than 8232 is wrong.


Describe Facebook Login Settings in Kitura-Credentials-Sample README

The README instructs the user to create an app in Facebook Developers, however it says nothing about specifying Facebook Login Settings , e.g. enabling Client/Web OAuth Login and specifying Valid OAuth redirect URIs

It appears Facebook Login is not required, so a sentence should be added to the README: "No need to add Facebook Login product".


Animation issue on some android phone browsers

Noty version: 3.2.0-beta
Browser: HUAWEI Mate9 browser
OS: Android 8.0
jsFiddle link (if available):

No doubt the default browser that comes with HUAWEI Mate9 is one of the worst. The phone is released very recently but comes with an outdated browser that is based on Chrome/40.
I am seeing an animation issue with Noty on this phone. It only surfaces with a webpack production build. Here is what I observed (true for both the mate9 browser and chrome/40):

using noty's animation setting, no notifications show up

changing animation to { open: "animated fadeIn", close: "animated fadeOut" } notifications show up but don't go away

changing animation to { open: null, close: null} noty appears to work correctly. Unfortunately there is no easy way to detect when to disable animation. (suggestions?)

The console is clean and prints no errors or warnings (I checked it using chrome remote debugging to the Chrome/40 instance on the phone).
Here is the snippet I used.
new Noty({
  text: "test",
  killer: "global",
  theme: "bootstrap-v4",
  timeout: 1000,
  progressBar: false,
  animation: { open: "animated fadeIn", close: "animated fadeOut" }
}

I think it's related to "animationend" event.

So, basically, we are listening to this event to complete the showing process and start the timer for the closing method while CSS animations are used. But sometimes it doesn't fire I guess. related: #434
I mean we can start the timer immediately after show() function but what if CSS animation takes 2 seconds to finish and notification has 2000ms timeout option. In this case, notification will close right after show animation finished. :/
Maybe we can try to read animation duration from CSS but I don't know. I'm open for suggestions.


annotation error

when I use SURVIVOR_ant for vcf file containing BND detected by pbsv.
for example:
the annotation for
"pbsv.BND.scaffoldA09:61716239-scaffoldA07:3695888"
is
"total_Annotations=2;overlapped_Annotations=BnaC09T0514400ZS.exon5,cds.BnaC09T0514400ZS"
the vcf file shows translocation is from A09 to A07, why the annotation shows the translocation is related to C09? and there are too many errors like this in the annotation file for BND, but I didn't find this problem in annotation file for INS、DEL、DUP.

sorry I didnt had time/resources to maintain this code base. I am currently using VCFanno.
Thanks
Fritz


fix other eval benchmarks

Some of the problems have bugs that mean programs constructed with their grammar cannot be executed or in some cases, just don't exist.

I think this problem comes from missing the while loop in the interpreter function from HerbInterpret.

I see that this is already an issue for that repository.


fix #345: Move Basic Auth into core

See #345 for the context.
This patch replaces the old-school Finagle filter BaiscallyAuthrorize with hipster Router combinator basicAuth.
Before:
val auth = new BaiscallyAuthrorize("user", "password")
Http.serve(":8081", auth.andThen(api.toService))

Now:
val auth = basicAuth("user", "password")
Http.serve(":8081", auth(api).toService)

Current coverage is 92.53%

Merging move-auth into master will change coverage by -0.09% by 3ce71b2

Powered by Codecov

Looks fine to me.
Related: I think we need a convenient way to wrap a specific set of endpoints in a filter in the setting of a coproduct router—e.g. in order to be able to take some Router[A :+: B :+: CNil], put it behind a login, and then compose it with an unfiltered Router[C :+: D :+: CNil] to get something we can serve.

@travisbrown good idea. If I understand you correctly. We need a way to convert SimpleFilter into something like Router[A] => Router[A]. Is that what you mean?

@vkostyukov Right, exactly.

I think it's ready to go.


Implement ingest source v2

Description
Implement the new ingest source that receives shard assignments from the control plane and stream documents from shards.
Shard assignments from the control plane are a hack at the moment and will be properly implemented in a follow-up PR.
How was this PR tested?
WIP

I have yet to add a bunch of unit tests, but this is ready for review.


Create Actions windows.yml

Creates an Actions yml file for Windows, tests against Ruby 2.4, 2.5, & 2.6.  Runs specs multi.
Tested running multi twice in my fork, both passed, and ruby-loco has run specs multi for a long time.
The action install current Rubies, as below. Also, updates to current MSYS2/MinGW gcc, currently 9.2.0 (Actions has gcc 7.3.0).
actions-ruby     default
Ruby 2.4.9        2.4.6
Ruby 2.5.7        2.5.5
Ruby 2.6.5        2.6.3

Ruby master has failures when run in Actions.  PR for that in the future.

Sorry about running this twice in CI, but there has been concern about Windows CI running parallel in the past.  I've run it several times in my fork, and ruby-loco has run parallel for a long time.  Hence, I believe it's stable.

This can run ruby-loco (ruby master), as it would be helpful to see issues here rather than in ruby/ruby.  Currently, the Actions equivalent to 'allow-failure' is a PITA, as the only way to determine if it occurs is to view the actual test step's log.  Hopefully, they'll propagate the failure indication to the individual job.

Sorry about running this twice in CI, but there has been concern about Windows CI running parallel in the past.  I've run it several times in my fork, and ruby-loco has run parallel for a long time.  Hence, I believe it's stable.

Seems fine.

This can run ruby-loco (ruby master), as it would be helpful to see issues here rather than in ruby/ruby.  Currently, the Actions equivalent to 'allow-failure' is a PITA, as the only way to determine if it occurs is to view the actual test step's log.  Hopefully, they'll propagate the failure indication to the individual job.

I personally don't like "allowed failured" as I rarely see their results and therefore never bother fixing them. Also, it is completely expected for ruby-master to fail these, they are only expected pass just after synchronizing specs. So I don't see the point to test ruby-master here.
In other words, let's ensure that's tested in the ruby/ruby, but not in this repo as it's expected to fail.

I don't see the point to test ruby-master here.

Ok.  Don't have strong feelings.  We could always test it on AppVeyor (allow-failure), or just remove AppVeyor testing...

Ok. Don't have strong feelings. We could always test it on AppVeyor (allow-failure), or just remove AppVeyor testing...

Yes, I plan to remove TravisCI and AppVeyor once GitHub Actions have shown they work well.
GitHub Actions is so much faster and unifies most of the setup between OS.

Cool.  Please ping me if anything goes haywire with Actions.  The 'action' here is downloading & installing current Ruby versions and updating the gcc chain...


outama

outama created by<EMAIL_ADDRESS>on 2017-07-12 07:46:15

<EMAIL_ADDRESS>replied, it's a verb from TAMA, but TAMA has a few meanings in Palauan. how would be good to translate OUTAMA into English?

<EMAIL_ADDRESS>replied, OUTAMA- Playing with marbles. At least, we just call them marbles, the
little round ones. :)
This word "ou" when added to another word, a noun, changes it to a verb.

J

<EMAIL_ADDRESS>replied, Ke kmal mesaul, Jelga)does anybody add anything to Jelga's words? or maybe some phrase examples?

<EMAIL_ADDRESS>replied, A Jelga a outama er rikl.
Yes, I really liked playing marbles with the kids from my neighborhood.
Aleks, I just remembered. There's another use for "ou" or maybe it's sort
of the same. Example: 1) meduch is knowing or being able to do something;
ou meduch is acting/pretending like he knows how or is able to. 2)
chedongel/simang is a show off; ou chedongel or ou simang is showing off 3)
medengei is knowing something or being knowledgeable about something; ou
mededengei (sometimes ou mededenger) is being unsure or having doubts
Just a few examples to show it's not always used to create an action verb
such as outama.
J

<EMAIL_ADDRESS>replied, Kom kmal mesaul!) OneLoveBelau!)))

<EMAIL_ADDRESS>replied, Is it a way to say that:
a bakery sells tama
or
a gun uses bullets
or
something else
?

<EMAIL_ADDRESS>replied, Aleks,
We could just add it to tama as a v.t. and not bother giving it a
definition...


Support reading mode

Support reading mode please.

I would also love this feature.

Without reading mode support, this plugin is pretty useless. Hopefully it can be added.


thrift proxy: support outlier

Signed-off-by: Yiheng Li<EMAIL_ADDRESS>Commit Message: thrift proxy support outlier
Additional Description: Put upstream result to outlier monitor in thrift route filter.
Risk Level: Low, no behavior change
Testing: UTs
Docs Changes: n/a
Release Notes: n/a
Platform Specific Features: n/a

The code looks good. Needs release notes (add an entry in docs/root/version_history/current.rst). I think perhaps in the minor behavior changes -- if someone has outlier detection enabled on a thrift cluster this change may change behavior for them.
Also, a note in docs/root/intro/arch_overview/upstream/outlier.rst  about outlier detection being available in thrift.

I've added the doc, but I'm not a native speaker, so please help me out if there are any problems.


Event Hierarchy Diagram Updated

I embedded a SVG diagram as a txt file.
Added a txt file and draw.io file in docs/images folder.
Please review the changes and let me know if you want anything changed thanks.

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

Hey, thank you very much. I think you exported the image as a draw.io embed instead of a normal SVG.
Instead of selecting Embed -> SVG in draw.io, can you use Export as -> SVG and then select the download button. (See images below.)

This way the SVG will have a .svg file attachment and wont contain embedded Javascript.
Sorry about the extra step!

Creating new PR


[USERSNAP] I can't verify my identity

Taken at: https://gov.tools/proposal_discussion
Fields:

Add a comment: I can't verify my identity

Browser: Chrome 129 (Windows 10)
Screen size: 1536x864
Open #136 in Usersnap
Powered by Usersnap.

This doesnt have enough details. Any screenshot attached or existing in Usersnap @l-br1 @kickloop ?

Closed as it doenst have sufficient information and is too vague.


The property 'EnableWindowTitle' cannot be found on this object

System Details

posh-git version/path: 0.7.3  C:\Program Files\WindowsPowerShell\Modules\posh-git\0.7.3
PowerShell version: 5.1.17134.48
git version: 2.14.1.windows.1
OS: Microsoft Windows NT 10.0.17134.0

Issue Description
I've had the following line in my PowerShell profile for years.
$GitPromptSettings.EnableWindowTitle = $true

After installing posh-git from the Gallery (and removing it from my User modules location), it no longer works! This appears on launching a new console.
The property 'EnableWindowTitle' cannot be found on this object. Verify that the property exists and can be set.
At C:\Users\anthony.mastrean\Documents\WindowsPowerShell\Microsoft.PowerShell_profile.ps1:17 char:1
+ $GitPromptSettings.EnableWindowTitle = $true
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (:) [], RuntimeException
    + FullyQualifiedErrorId : PropertyNotFound

Can you double-check the version of posh-git you're running?  What does Get-Module posh-git return?  There is definitely a $GitPromptSettings.EnableWindowTitle in 0.7.3 but in v1.0.0-betax, the state of the current master branch, the property has been renamed to $GitPromptSettings.WindowTitle.
Also, what output do you get if you execute $GitPromptSettings?

Yeah, I double-checked; it's very confusing 😄
PS> get-module posh-git

ModuleType Version    Name                                ExportedCommands
---------- -------    ----                                ----------------
Script     0.7.3      posh-git                            {Add-PoshGitToProfile, Add-SshKey, Enable-GitColors, Expand-GitCommand...}

I seem to have access to all the vars once the shell is loaded up.
PS> $GitPromptSettings.EnableWindowTitle
posh~git ~

So is it possible you got the PropertyNotFound error because you tried to access it before the posh-git module loaded?

When are system modules loaded? I thought before my user profile. Otherwise, what profile should I use for setting this var?

On May 22, 2018, at 6:18 PM, Keith Hill<EMAIL_ADDRESS>wrote:
So is it possible you got the PropertyNotFound error because you tried to access it before the posh-git module loaded?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Look in ~\Documents\WindowsPowerShell\profile.ps1 and make sure that you are accessing that property after the Import-Module posh-git statement.  If this is not in profile.ps1, check in ~\Documents\WindowsPowerShell\Microsoft.PowerShell_profile.ps1.

I do not have a profile.ps1, just this:
PS> ls

    Directory: C:\Users\anthony.mastrean\Documents\WindowsPowerShell

Mode                LastWriteTime         Length Name
----                -------------         ------ ----
-a----        5/22/2018   6:04 PM           1380
Microsoft.PowerShell_profile.ps1

And I do not explicitly Import-Module on any of the modules I've
installed via the Gallery (since they're installed at the machine-level,
I'm under the impression they're imported automatically... and that's how
it seems to work, although I'm not certain of the lifecycle of machine vs.
user profile).
On Tue, May 22, 2018 at 7:21 PM Keith Hill<EMAIL_ADDRESS>wrote:

Look in ~\Documents\WindowsPowerShell\profile.ps1 and make sure that you
are accessing that property after the Import-Module posh-git statement.
If this is not in profile.ps1, check in
~\Documents\WindowsPowerShell\Microsoft.PowerShell_profile.ps1.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dahlbyk/posh-git/issues/582#issuecomment-391171615,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AANS8siDrjczYlFJOZ8LHeW0PmI87y5Uks5t1J1vgaJpZM4UJdgM
.

Exports from system modules are automatically available, but they're not actually imported until you use a function from them. Since there's really nothing to "do" with posh-git other than import it, you'll want to add Import-Module posh-git to $PROFILE before you modify $GitPromptSettings.
Add-PoshGitToProfile can add it for you, but it will add the import at the end of your profile.

@AnthonyMastrean This is a profile Microsoft.PowerShell_profile.ps1 that applies only to the PowerShell console host.  You might open that file up in ISE or VSCode to see what's in it.


Jenkins deploy scripts

merged into master - I squashed the second commit as it was just fixing a few small things on the first one


MGMT: Support Service Principal Auth for AzureTest

Running of live tests on CI will require that support service principal authentication, which we currently do not. We will also need a suitable auth mechanism (which, MSAL might be) for running tests live locally.

It looks like there are already scripts inside eng/common to facilitate deploying resources that we can leverage, so we may not need this.

Closing this issue. We can create the necessary resources needed via ARM.


Required upgrade of HAL

After upgrading, something like the following starts to happen suddenly:
In file included from /home/pflykt/projects/zephyrproject/modules/hal/nordic/nrfx/mdk/nrf.h:124,
                 from /home/pflykt/projects/zephyrproject/modules/hal/nordic/nrfx/drivers/nrfx_common.h:39,
                 from /home/pflykt/projects/zephyrproject/modules/hal/nordic/nrfx/nrfx.h:36,
                 from ../../../soc/arm/nordic_nrf/nrf52/soc.h:16,
                 from /home/pflykt/projects/zephyrproject/zephyr/drivers/clock_control/nrf_power_clock.c:8:
/home/pflykt/projects/zephyrproject/zephyr/drivers/clock_control/nrf_power_clock.c: In function 'clock_event_check_and_clean':
/home/pflykt/projects/zephyrproject/modules/hal/nordic/nrfx/mdk/nrf52840.h:2826:37: error: incompatible type for argument 1 of 'nrf_clock_event_check'
 #define NRF_CLOCK                   ((NRF_CLOCK_Type*)         NRF_CLOCK_BASE)
                                     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/home/pflykt/projects/zephyrproject/zephyr/drivers/clock_control/nrf_power_clock.c:53:35: note: in expansion of macro 'NRF_CLOCK'
  bool ret = nrf_clock_event_check(NRF_CLOCK, evt) &&

After a bit of investigation, the above and other weird compile errors mean that the board HAL needs to be updated.
A much better usability solution would be to detect the HAL version in use and print out a warning that the installed and expected HAL versions differ.

@pfl just checking, have you run west update?

@pfl I assume you have run west update and that this issue covers the fact of handling this in a nicer way.

I think the idea was that west/cmake could check that modules are according to what manifest says and possibly give some information to the user if there is a mismatch so that user would know that he needs to run "west update".

@pfl I assume you have run west update and that this issue covers the fact of handling this in a nicer way.

Yes, I did run west update as a result. So this is about either documenting such a situation, or, as @jukkar commented, have west/cmake notice that the HAL is too old wrt to the code being compiled. The latter approach would be preferred.

Since this issue only covers HAL repos but the problem is in fact more general and already described here: https://github.com/zephyrproject-rtos/west/issues/333 I will close this issue in benefit of the one linked.


Update years for @ISMB and @VIS 2021 program pages

http://biovis.net/2021/program_ismb/
http://biovis.net/2021/ieeevis/

Thanks for point this out Jim. Others have fixed this now, so I can close the issue. Enjoy the conference!


Html

##User story
As a bootcamp student
I want the prework notes to be structured on a webpage
so that I can easily find and read information
##Acceptance criteria
Given a prework study guide website
When I visit the website in my browser
Then i see four boxes titles Html, Css, Git and JavaScript with associated notes listed


Meta tags for article thumbnails not working

Featured images are not properly recognized by social media, like LinkedIn / Twitter.

Will be fixed in v5.2.1


Add help target to Makefile for coolness

Total nice to have, close if not interested 😄 . We have similar things in pitch-black Makefiles, gives a chill dev experience:
I have made up missing comments (probably nonsense) and put things in what seemed to me like logical groupings 🤷♀  but obvs can arrange how you like.

@Callisto13 Sorry we missed this! I will pull this down and poke it today and give you a review. It looks great at first glance!

👍  I've resolved the merge conflicts, hopefully I have not accidentally removed anything which should be there. LMK if it all looks fine and I will merge

@Callisto13 I will merge this once these tests pass. Thanks for your contribution!


Failing in Travis (TestCafe browser)

Hello Puppeteer Team,
We're using Puppeteer as TestCafe browser provider and it turned out that Puppeteer fails to run in Travis. Please refer to the following Travis Job.
Error stack trace:
Was unable to open the browser "puppeteer:" due to error.

Error: Failed to launch chrome!
    /home/travis/build/MitocGroup/recink/node_modules/puppeteer/.local-chromium/linux-496140/chrome-linux/chrome: symbol lookup error: /home/travis/build/MitocGroup/recink/node_modules/puppeteer/.local-chromium/linux-496140/chrome-linux/chrome: undefined symbol: gtk_widget_get_scale_factor

    TROUBLESHOOTING: https://github.com/GoogleChrome/puppeteer/blob/master/docs/troubleshooting.md

        at Interface.onClose (/home/travis/build/MitocGroup/recink/node_modules/puppeteer/lib/Launcher.js:141:14)
        at emitNone (events.js:110:20)
        at Interface.emit (events.js:207:7)
        at Interface.close (readline.js:367:8)
        at Socket.onend (readline.js:147:10)
        at emitNone (events.js:110:20)
        at Socket.emit (events.js:207:7)
        at endReadableNT (_stream_readable.js:1059:12)
        at _combinedTickCallback (internal/process/next_tick.js:138:11)
        at process._tickDomainCallback (internal/process/next_tick.js:218:9)

HI @AlexanderC,
I don't remember this exact error, but we managed to make travis work for our own tests:

Check out our config
We also have to pass '--no-sandbox' flag to make it work on Travis

@aslushnikov Thanks for reply, i'm already using --no-sandbox (@see https://github.com/MitocGroup/recink/pull/61/files#diff-99fe08f7050dc241613a52c9b792495eR43)
I see you're using dist: trusty since our's dist: precise, i'll try if it affects puppeteer somehow.

Yeah, setting dist: trusty in travis.yml did the trick!
You definitely should have this specified somewhere in the README 😈
https://travis-ci.org/MitocGroup/recink/jobs/268427887

Puppeteer Team, you definitely should have this specified somewhere in the README 😈

We'll add this somewhere if there are more questions about travis.
For now, the issue per se might be good-enough.

dist: trusty really help.
But have we some way to run tests with dist: precise ?

Please reopen this issues as a docs issue and add this info to the faq

@gauntface feel free to send a PR.

Hi,
We are facing a similar issue with puppeteer not launching on Travis CI only for node 12.
P.S: It is working perfectly fine with Node 13.
Please find the following travis job for reference: https://travis-ci.org/preactjs/preact-cli/jobs/656122013
This is what we do for launching puppeteer instance.
I tried passing { args: ['--no-sandbox', '--disable-setuid-sandbox'] } to puppeteer.launch() as suggested in the comments above, but it didn't work.


Add export option to add collider to group

Per the message thread here, it would be very convenient if the road generator provided a way assign the collision mesh to specific groups. This is useful for some approaches of defining friction levels by stating which group a given collision is (e.g. having a road group could be used for max friction, while grass or anything else could be less). This is also the approach this somewhat common vehicle controller uses: https://godotengine.org/asset-library/asset/2558
Should be a quick add, since we already have a group adding system for the RoadLanes themselves.

This has now been completed! I ended up creating two new properties: one for node groups, and one for a meta value to assign. Meta value is likely a more performant operation, as the calling script just checks a string value, whereas comparing to see if an object belongs to a group requires godot to do a list lookup, so for anyone who has full control over their scripting system, it is more ideal to use an assigned meta tag instead of using groups.


translate (Gust in French = (Rafale

hello,
thank you for your work
it's possibly to translate Gust in French ?
thank you

Maybe, but since I don't speak french I might need a hint?
Is gust == rafale?

Thank you
Yes gust in french is : rafale

In italian gust is: Raffica

In Dutch it is: Windvlaag

Spanish --> Ráfaga

German --> Böen


tablesaw-bar doesn't show on Magento 2

Hi,
Added tablesaw to Magento 2 but tablesaw-bar doesn't showup ? Any ideas?

Sorry, unfortunately we don’t have the resources to provide support for integration with third party frameworks and libs. If you have a specific easy-to-debug reduced test case, I might make an exception—feel free to reopen if add a test case.


openssl 3.4.0-beta: Bad free memory behaviour

Hi,
I seen a segfaults with beta1 tracked down custom RCU usage in exit path with memory allocation wrapper.  The testcase
openssl-memory-track.c.gz uses CRYPTO_set_mem_functions() for custom functions and loads config via OPENSSL_init_crypto(). 3.3.2 passes fine, 3.4.0-beta1 shows

free_hook(0x5126e00) MAGIC 0 (expected abbacddeeffe7766)
free_hook(0x516f640) MAGIC 0 (expected abbacddeeffe7766)

which means, the pointer passed to the free function was never return by the malloc or realloc function (or the magic value was overwritten but based on the complete log the pointer value was never returned).
Throwing valgrind at it, I get:
 alloc_hook() PTR: 0x516ea60 RET: 0x516eaa0 (len: 24)
 ==114467== Invalid read of size 8
 ==114467==    at 0x10922B: free_hook (in /home/bigeasy/openssl/exp/tc)
 ==114467==    by 0x4A86D3B: free_oldmd (hashtable.c:290)
 ==114467==    by 0x4AAB1BD: ossl_synchronize_rcu (threads_pthread.c:628)
 ==114467==    by 0x4A871ED: ossl_ht_free (hashtable.c:337)
 ==114467==    by 0x4A942D8: ossl_namemap_free (core_namemap.c:543)
 ==114467==    by 0x4A942D8: ossl_namemap_free (core_namemap.c:536)
 ==114467==    by 0x4A923E0: context_deinit_objs (context.c:287)
 ==114467==    by 0x4A927FF: context_deinit (context.c:368)
 ==114467==    by 0x4A927FF: context_deinit (context.c:361)
 ==114467==    by 0x4A927FF: ossl_lib_ctx_default_deinit (context.c:407)
 ==114467==    by 0x4A96AC1: OPENSSL_cleanup (init.c:440)
 ==114467==    by 0x4A96AC1: OPENSSL_cleanup (init.c:353)
 ==114467==    by 0x4E3F1E0: __run_exit_handlers (exit.c:108)
 ==114467==    by 0x4E3F299: exit (exit.c:138)
 ==114467==    by 0x109111: main (in /home/bigeasy/openssl/exp/tc)
 ==114467==  Address 0x5125d88 is 8 bytes after an unallocated block of size 32 in arena "client"
 ==114467==
 free_hook(0x5125dc0) MAGIC 0 (expected abbacddeeffe7766)
 ==114467== Invalid read of size 8
 ==114467==    at 0x1092AB: free_hook (in /home/bigeasy/openssl/exp/tc)
 ==114467==    by 0x4A86D3B: free_oldmd (hashtable.c:290)
 ==114467==    by 0x4AAB1BD: ossl_synchronize_rcu (threads_pthread.c:628)
 ==114467==    by 0x4A871ED: ossl_ht_free (hashtable.c:337)
 ==114467==    by 0x4A942D8: ossl_namemap_free (core_namemap.c:543)
 ==114467==    by 0x4A942D8: ossl_namemap_free (core_namemap.c:536)
 ==114467==    by 0x4A923E0: context_deinit_objs (context.c:287)
 ==114467==    by 0x4A927FF: context_deinit (context.c:368)
 ==114467==    by 0x4A927FF: context_deinit (context.c:361)
 ==114467==    by 0x4A927FF: ossl_lib_ctx_default_deinit (context.c:407)
 ==114467==    by 0x4A96AC1: OPENSSL_cleanup (init.c:440)
 ==114467==    by 0x4A96AC1: OPENSSL_cleanup (init.c:353)
 ==114467==    by 0x4E3F1E0: __run_exit_handlers (exit.c:108)
 ==114467==    by 0x4E3F299: exit (exit.c:138)
 ==114467==    by 0x109111: main (in /home/bigeasy/openssl/exp/tc)
 ==114467==  Address 0x5125d80 is 0 bytes after an unallocated block of size 32 in arena "client"
 ==114467==
 free_hook() ptr 0x5125dc0 old 0x5125d80 (len: 0)
 ==114467== Invalid read of size 8
 ==114467==    at 0x109260: free_hook (in /home/bigeasy/openssl/exp/tc)
 ==114467==    by 0x4A86D3B: free_oldmd (hashtable.c:290)
 ==114467==    by 0x4AAB1BD: ossl_synchronize_rcu (threads_pthread.c:628)
 ==114467==    by 0x4A871ED: ossl_ht_free (hashtable.c:337)
 ==114467==    by 0x4A942D8: ossl_namemap_free (core_namemap.c:543)
 ==114467==    by 0x4A942D8: ossl_namemap_free (core_namemap.c:536)
 ==114467==    by 0x4A923E0: context_deinit_objs (context.c:287)
 ==114467==    by 0x4A927FF: context_deinit (context.c:368)
 ==114467==    by 0x4A927FF: context_deinit (context.c:361)
 ==114467==    by 0x4A927FF: ossl_lib_ctx_default_deinit (context.c:407)
 ==114467==    by 0x4A96AC1: OPENSSL_cleanup (init.c:440)
 ==114467==    by 0x4A96AC1: OPENSSL_cleanup (init.c:353)
 ==114467==    by 0x4E3F1E0: __run_exit_handlers (exit.c:108)
 ==114467==    by 0x4E3F299: exit (exit.c:138)
 ==114467==    by 0x109111: main (in /home/bigeasy/openssl/exp/tc)
 ==114467==  Address 0x5125d80 is 0 bytes after an unallocated block of size 32 in arena "client"
 ==114467==
 ==114467== Invalid write of size 8
 ==114467==    at 0x10927F: free_hook (in /home/bigeasy/openssl/exp/tc)
 ==114467==    by 0x4A86D3B: free_oldmd (hashtable.c:290)
 ==114467==    by 0x4AAB1BD: ossl_synchronize_rcu (threads_pthread.c:628)
 ==114467==    by 0x4A871ED: ossl_ht_free (hashtable.c:337)
 ==114467==    by 0x4A942D8: ossl_namemap_free (core_namemap.c:543)
 ==114467==    by 0x4A942D8: ossl_namemap_free (core_namemap.c:536)
 ==114467==    by 0x4A923E0: context_deinit_objs (context.c:287)
 ==114467==    by 0x4A927FF: context_deinit (context.c:368)
 ==114467==    by 0x4A927FF: context_deinit (context.c:361)
 ==114467==    by 0x4A927FF: ossl_lib_ctx_default_deinit (context.c:407)
 ==114467==    by 0x4A96AC1: OPENSSL_cleanup (init.c:440)
 ==114467==    by 0x4A96AC1: OPENSSL_cleanup (init.c:353)
 ==114467==    by 0x4E3F1E0: __run_exit_handlers (exit.c:108)
 ==114467==    by 0x4E3F299: exit (exit.c:138)
 ==114467==    by 0x109111: main (in /home/bigeasy/openssl/exp/tc)
 ==114467==  Address 0x5125d88 is 8 bytes after an unallocated block of size 32 in arena "client"
…

So it looks like the cleanup is looking at a bad pointer.
Sebastian

Fix merged. Thank you very much for testing 3.4.0-beta!


Accept a function instead of rendering all items up front

Is your feature request related to a problem? Please describe.
My application stutters when element size is very large (1000+).
Describe the solution you'd like
Instead of providing elements up front like
<VList>
    {rows.map((row, index) => (
        // Some jsx
    ))}
</VList>

I'd like to use this pattern:
<VList items={rows}>
   { (row, index) => renderRow(row, index) }
</VList>

Describe alternatives you've considered
None
Additional context
This is the pattern other libraries also use to avoid the up-front cost of loading items.

Currently it is not planned.
React element is just an object and light enough for most cases.
If React element creation costs, you can use useMemo, memo or other optimization techniques for React.
https://github.com/inokawa/virtua#is-there-any-way-to-improve-performance-further

The memoization still runs this large object creation though. Even with memoization there's an up-front cost in creating many objects for a complex UI with tens of React elements per row. In my case when there may be 10,000 elements for a user on a slow computer, the performance degradation will lead to a negative experience for my users. Even in a prod build on a fast PC there's a perceptible delay in loading the virtualized list. Up to you to mark it as unplanned but partially defeats the purpose of using a virtualization library.

I think there are 2 major usages of virtualization on browser, heavy and dynamic DOM(with live preview, rich editor, etc) with not so many rows, or relatively light DOM with many rows(10000~).
This library is more focused on the former usage but I agree with that it will have slower start up time compared to the others in the latter usage.
I'll try to add props for it. Thank you for the information!


Make samply listen on interface <IP_ADDRESS>

Samply runs on interface <IP_ADDRESS> on GCP. How can i make it run on interface <IP_ADDRESS>?
Struggling with connecting to it via browser on this port.
Aborted!
Local server listening at http://<IP_ADDRESS>:3002
Press Ctrl+C to stop.
channel 3: open failed: connect failed: Connection refused
channel 3: open failed: connect failed: Connection refused
channel 13: open failed: connect failed: Connection refused
channel 13: open failed: connect failed: Connection refused
channel 13: open failed: connect failed: Connection refused
channel 13: open failed: connect failed: Connection refused
channel 17: open failed: connect failed: Connection refused

This is not very clear from either from the help command or from the docs that you can call --help for record, load, etc commands as well.

That's useful feedback, thank you


what means data1_psize and  data2_psize ？

code have data1_psize and  data2_psize ？ is featuremap shape?   or filter shape ?

Feature Map shape.


Google Chart sometimes loads in weird

Check the order that things are loading to ensure that it is working properly.

Fixed


Refactoring of sortFontSizes()

Hey @alexdanilowicz! Can you please review this pull request ?
The intention of this pull request is to refactor the sortFontSizes() we were discussing in issue #254

Looks good to me! Great work @Bismay5467. I made a few changes to fix the linter problems and make a and b more clear what they were referencing. Pushed up my changes, then will get this merged!

Resolves #254


Android Gradle Plugin - Kotlin DSL - Can't set buildFeatures databinding

I am migrating to Kotlin dsl and I can't set buildFeatures property databinding any more. It is missing from AppExtension class. It is happening in dynamic feature module.
Steps to reproduce:

Convert to kotlin dsl for gradle file.
Try to enable databinding in android extension in a dynamic feature module.

I am on AGP 4.2.1. Android Studio Android Studio Arctic Fox | 2020.3.1 Beta 4
Here is code sample:
I have tried
android {
   buildFeatures {
     dataBinding = true
   }
}

and
buildFeatures.dataBinding = true
Error is:
Unresolved reference: dataBinding
some screenshots attached.

@AnastasiaRainMaker could you please provide a small reproducer project in an archive or as a github repo?

Google replied on their issue tracker https://issuetracker.google.com/issues/193452960


Federal Land Overlay

Current implementation treats public land as a single on/off switch: Federal/State. A better implementation would separate federal and state into two overlay layers, (a) Federal and (b) State.
See var loadFederalState = function() in map.js.
Underlying data is saved to HZA Data repo.

https://geoportal.hawaii.gov/datasets/HiStateGIS::government-land-ownership/about

@ggordn3r PR #47 implements the separate overlays ✅ .
In working with the gov-owned-land data, it appears that our og fed-state-dissolve file included DHHL and County owned lands in the fed-state overlay. DHHL is already handled, but would you like to see the County owned land as a.) its own overlay, b.) added back into either fed or state, or c.) left the way it currently is in PR #47 (not included)?


Fail to find the binary if CARGO_TARGET_DIR is set

To reproduce:

export CARGO_TARGET_DIR=/tmp/test/
cargo test

The CLI binary, usually found when CARGO_TARGET_DIR is not set, will not be found:
---- test::it_fails_without_source stdout ----
thread 'test::it_fails_without_source' panicked at 'called `Result::unwrap()` on an `Err` value: CargoError { cause: Some(NotFoundError { path: "/tmp/test/debug/subwasm" }) }', cli/src/main.rs:75:66

This is especially an issue when CARGO_TARGET_DIR is set to benefit from caching.
NOTE: I am working currently on https://gitlab.com/chevdor/subwasm if you want more context

Is this blocking you or is that workaround insufficient until then?

This is not blocking me at all for now, no worries. I disabled my tests for now but I will be happy to enable them again :)

I wonder if there is a platform or rustc specific aspect to this.
I just ran
assert_cmd $ TMPDIR=/Users/edpage/src/tmp/ CARGO_TARGET_DIR=/Users/edpage/src/tmp/ cargo test

(the TMPDIR is to workaround an IT thing)
assert_cmd on a Mac with 1.51.0 passes tests that lookup and run a binary included in assert_cmd (bin_fixture).

Either way, I expect #120 to help

I think I missed the update on which crate.  It looks like you are using assert_cmd for unit/whitebox tests (in src/), rather than integration/blackbox tests (in tests/).  I do not believe the binary is guaranteed to be compiled at that.  When not overriding the directory, I think it works just because something else had left behind an executable.
This is documented in the limitations
Going ahead and closing assuming this is the root cause.  Feel free to ask for this to be re-opened if something still needs addressing.


Download links skipped by screen reader

Actual Behaviour

The chrome screen reader skips the download links under Single Files section. This might make the links inaccessible to people using screen readers.
Expected Behaviour
Screen reader must read/announce the download links.

Steps to reproduce it

Install the chrome screen reader extension
Scan the download page.
See error

Screenshots of the issue

Would you like to work on the issue?
YES

@lmccart @limzykenneth Can I be assigned to this 🙂

@Rahulm2310 yes, assigned, thanks!

Yes


feat(Navigation):added AP logo and some important link

Fixes: #82
Please Review @dselman @DianaLease and suggest changes :)
Signed-off-by: Arteev Raina<EMAIL_ADDRESS>

@DianaLease Changes have been made :)


Add a declared license mapping for "The Apache 2.0 License"

As found in [1].
[1] https://repo1.maven.org/maven2/io/kotest/kotest-core/4.0.6/kotest-core-4.0.6.pom

Also see: https://github.com/kotest/kotest/pull/1517


invalid command -> echoo pouya

#execution when we run invalid command, it shouldnt exit the programm, instead it should return an error and prompt again.

fixed by mina


Handle case when one of the scans has only one empty granule, resulting in instance without any output file

The only error case remaining is when one of the scans has only one empty granule, resulting in instance without any output file: harmony proceeds with all other stitched files to concatenate them in concise

Originally posted by @ank1m in https://github.com/nasa/stitchee/issues/152#issuecomment-2049996406

So, @ank1m, the pseudo logic for the new desired behavior is....
if (NumberOfGranules == 1) and (TheGranule is empty):
copy TheGranule to OutputPath
return OutputPath
Correct?

For now, yes.
Two considerations:

We have to test if having all-NaN file for input would breaks CONCISE or not
If it doesn't break it, that means that output merged file would have a data layer full-of-NaNs, which seems like a strange choice (to be addressed after discussion with Harmony)


Without Extending

Is there anyway to use the library without extending base classes?
Also, out of curiosity, why didn't you use the dispatcher/listener approach over the code-injunction approach commonly used with AOP?
Thanks.

Hello sahal2080,
sorry for the very late reply.
The library currently only works by extending a base class. The idea behind the library was that you will only ever have to extend one class and all behaviours you need can be composed using it.
I'm also a bit confused about the suggested approaches you mention. We are using a dispatcher/listener approach.
Unlike proper AOP, this library doesn't use the equivalent of java agents to do its work. It uses reflection at class creation time to create a list of listeners to dispatch events to.


Switch RNG used in two qubit decomposer

Summary
This commit changes the RNG algorithm used in the
TwoQubitWeylDecomposition class from the default PCG64 to MT19937. The
MT19937 algorithm was the old algorithm we used in the class when
numpy's deprecated RandomState was being used. We've been seeing issues
where the decomposition with a fixed input varies which is unexpected
since there is a fixed seed for the RNG. The MT19937 is called out in
the documentation as having a guaranteed output for a fixed seed [1]
which should hopefully fix this issue.
Details and comments
[1] https://numpy.org/doc/stable/reference/random/bit_generators/mt19937.html

This probably isn't needed since #4835 fixes the issues we were seeing with inconsistent decompositions.


Unsoundness in the implementation of several functions

The source of unsoundness
https://github.com/hermit-os/uhyve/blob/419435763d826e4578d6d234549cce9a37b11e4c/src/linux/virtqueue.rs#L51-L54
Hi, we consider that ring_elem is unsound because it allows u8 pointer cast to arbitrary pointer types and dereference the types. It can break the alignment and the validity invariants at the same time.
To reproduce the bug
To reproduce the misalignment issue
use uhyvelib::linux::virtqueue::Vring;

fn main() {
    let mem: u8 = 0;
    let mut vr = Vring::new(&mem as *const u8);
    let em: &mut [u16; 4] = vr.ring_elem(0u16);
    println!("{:?}", em);
}

execute the code and get panic,
thread 'main' panicked at 'misaligned pointer dereference: address must be a multiple of 0x2 but is 0x7fff8b2c183b', /${HOME}/.cargo/registry/src/index.crates.io-6f17d22bba15001f/uhyve-0.2.2/src/linux/virtqueue.rs:53:18

The misaligned pointer dereference is triggered here because you cast the type aligned to 1 byte to the one aligned to 2 bytes. If this is what you plan, then you could change dereference to read_unaligned.
Here, we can also specify em as bool types. Run it can get you the results such as [true, true, true, false]. However, run with miri then you can find that mem.offset already access out-of-bound.
error: Undefined Behavior: out-of-bounds pointer arithmetic: alloc1424 has size 1, so pointer to 4 bytes starting at offset 0 is out-of-bounds
   --> /${HOME}/.rustup/toolchains/nightly-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/core/src/ptr/const_ptr.rs:466:18
    |
466 |         unsafe { intrinsics::offset(self, count) }
    |                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ out-of-bounds pointer arithmetic: alloc1424 has size 1, so pointer to 4 bytes starting at offset 0 is out-of-bounds

It means that our bool type points to invalid value actually.

We consider that there are some other misaligned pointer dereference in this package.
The source of unsoundness
First of all, in <linux::vcpu::UhyveCPU as vm::VirtualCPU>::r#continue:
https://github.com/hermit-os/uhyve/blob/92458f59930789a84effb8aa0b3dc103c35ccd09/src/linux/vcpu.rs#L360-L361
addr was cast to u32 and cause to misaligned pointer dereference!
Secondly, the methods implementated for uhyvelib::linux::virtqueue::Vring are unsound. The unsound methods include _flags, index, and advance_index. Worth noting that the usage of offset in index and advance_index cannot avoid alignment issues. It should be changed to the code such as self.mem.add(self.mem.align_offset(align_of::<u16>())).
To reproduce the bug
use uhyvelib::linux::virtqueue::Vring;

fn main() {
    let a: u8 = 1;
    let v = Vring::<u8>::new(&a as *const u8);
    // println!("{}", v._flags());
    println!("{}", v.index());
}

This code shows how to trigger undefined behavior with _flags and index. Just run the code,
thread 'main' panicked at /${HOME}/.cargo/registry/src/index.crates.io-6f17d22bba15001f/uhyve-0.2.2/src/linux/virtqueue.rs:38:13:
misaligned pointer dereference: address must be a multiple of 0x2 but is 0x7fff6343e199

Thank you so much for opening this issue. We are aware that our virtio implementation is problematic, both in the kernel and in Uhyve.
We are making plans on addressing those problems in the long term. If you would like to contribute, please tell us, so we can coordinate.
It would be helpful if this issue could be split up into two separate issues. The VcpuExit::IoOut(port, addr) might be solved separately. :)

@mkroening I would love to help fix the issue in my free time. Feel free to ping me in the future!

Sounds great! Contributions are always welcome. We have a Zulip for chatting and coordination. :)
I have also just added links to Zulip to the organizations README for better discoverability.

@mkroening Hi, I just noticed that the following two methods
https://github.com/hermit-os/uhyve/blob/fcdd579ae2260339d8111d64e77914f52da940ea/src/linux/virtio.rs#L258_L261
https://github.com/hermit-os/uhyve/blob/fcdd579ae2260339d8111d64e77914f52da940ea/src/linux/virtio.rs#L282-L286
could have the totally same misaligned pointer issues as described above. Could you explain more about why the clippy warning would be suppressed here?

Good find! That should definitely be fixed. I can only guess that this was just to silence clippy without being aware of the consequences and opening an issue.

Yup! I think it would be easy for you to fix it now 👍

I can't say how difficult it would be. I did not have a close look yet. :)
See https://github.com/hermit-os/uhyve/commit/aa5758ed9e87b516bf794e968967192a336e4331 for the introduction and more such cases.

Sorry I can't say that.
Personally speaking, I suggest to use read_unaligned rather than just dereference. It might be able to patch the code with minimum budget.

Can I have a go at this? :)

Can I have a go at this? :)

Sure!

I have the feeling, that this issue is vacant as of today :sweat_smile:


Fix _should_create_provider for module extension targets

Regressed with 88237f24f1060ad875845a2583793ec7758af733.

@thii I get this error still though (pretty sure it was there before this regression?):

Yes, that was there before this regression. I'll need to find a way to collect implicit dependencies added by swift_protoc_gen_aspect.


feat(NiFi): Top-level deployment ansible-playbooks

Which issue(s) this PR fixes:
Fixes #108
Additional comments:
To make clear that you license your contribution under
the Apache License Version 2.0, January 2004 and that you give permission to TOSIT,
you have to acknowledge this by using the following check-box.

[x] I hereby declare this contribution to be licenced under the Apache License Version 2.0, January 2004
[x] I hereby agree to grant TOSIT a copyright license to use my contributions.

Related Issues:

https://github.com/TOSIT-IO/TDP/issues/15
https://github.com/TOSIT-IO/tdp-collection-extras/issues/34

@pm-McFly thank you for your contribution to TDP!
In a fully open-source approach, all Issues/Pull Requests are fairly examined for integration.
Your PR integrates a new technology to TDP. Can you please open a new discussion for validation before any implementation. See the TDP new component proposal guidelines for details on how to organize your request.
Thank you again for your contribution to TDP.

Closing as stale


No change log for 3.16.0

These versions differ only in the presence of better compatibility with React.
The isPureReactComponent = true property has been added to the prototype of the PureComponent constructor
PureComponent.prototype.isPureReactComponent = true;

Ah yeah, sorry. That release was done a little more hastily than usual.

I moved the notes into 3.17.


Added tests and instructions to README for distributed tracing

:broken_heart: Tests Failed

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2022-07-06T15:29:54.429+0000

Duration: 10 min 40 sec

Test stats :test_tube:

Test
Results

Failed
1

Passed
0

Skipped
0

Total
1

Test errors
Expand to view the tests failures
#1 Test with defaults – co.elastic.opbeans.Default

Expand to view the error details
 null

no stacktrace

Steps errors
Expand to view the steps failures

Shell Script

Took 3 min 46 sec . View more details here
Description: make test

:robot: GitHub comments
To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.


Why Action1<? super Throwable> ? instead of Action1

Why not this method just return Action1<Throwable> ?
I don't understand it, can anybody explain it for me? Thanks!
https://github.com/JakeWharton/NotRxAndroid/blob/5c0ba7dff90aa69b936059d6c35e6c2b23370900/rxandroid/src/main/java/rx/android/util/RxLog.java#L55
  public static Action1<? super Throwable> error(final String tag, final String message) {
    checkNotNull(tag, "tag == null");
    checkNotNull(message, "message == null");

    final RxAndroidLogHook logHook = RxAndroidPlugins.getInstance().getLogHook();
    return new Action1<Throwable>() {
      @Override public void call(Throwable throwable) {
        logHook.log(Log.ERROR, tag, message, throwable);
      }
    };
  }

Probably got a little too copy/paste happy. #5 will end up fixing this.


Rename StorageMap::exists to contains_key

companion PR to https://github.com/paritytech/substrate/pull/4847

Anything else I need to adjust? Or do we just wait?

We could wait ~2 weeks to see if we have v2.0 at that point in time. Updating the node-template has a big impact on the DevHub and tutorials. We need to plan quite well every time we do it. It could impact even more this time because we restructured the node-template in master.

Ok, meaning we will come back to this if and when we update the substrate-node-template.
I'm guessing it does not make sense to open a PR for the node template right now as it will pull in quite a few changes.

Correct, there is a PR to update the template but it hasn't been merged because of the impact it will have. We need to be ready with PRs for all the docs an tutorials.

Updating the node-template has a big impact on the DevHub and tutorials.

That's totally true and totally a pain in the butt. I recommend we solve this problem. Each time we cut a new version of the node template we tag it with something like pre-v2.0-3e65111. Then when we reference it from a tutorial, the tutorial should start with cloning node template and checking out the correct version of the template.
For example the Creating Your First Substrate Chain tutorial could be updated to read like this.

Checkout the Substrate Node Template and create a branch for your work.
git clone -b pre-v2.0-3e65111 https://github.com/substrate-developer-hub/substrate-node-template
git checkout -b my-first-substrate-chain

Initialize your WebAssembly build environment
...

Happy with Joshy suggestion that we also need to tag substrate-node-template, and then refer to the node-template tag in our documentations and tutorials. Just to clarify on the nomenclature, so it is
<some-version>-<substrate git rev 7 digits>?

Just to clarify on the nomenclature, so it is <some-version>-<substrate git rev 7 digits>?

That's the nomenclature for now until 2.0 is released. When Substrate 2.0 is released we will also tag the node template v2.0. After that we'll have to see how regular releases are. If upstream releases v2.1 in a reasonable time, we'll follow them. If not we're back to pre-v3.0-<hash>.

I agree, Joshy's suggestion should be the way to add tags to use in tutorials

With rustdoc built for pre-v2.0-3e65111, and tutorials now explicitly referencing the tag they use, I don't see any reason this can't be merged.

Subsumed by #493


DDEV Complains about ports in use when Im not using Ports and when I change to new ones the next site finds default and New ports in use

Is there an existing issue for this?

[X] I have searched the existing issues

Run a Diagnostic and Paste Link Here
https://gist.github.com/asecondwill/20b1e7d7198d618d90a7ec8744260b50
Expected Behavior
Start and stop ddev using standard ports
Actual Behavior
Ddev complains that 80 and 443 are in use.
Failed to start starhr: Unable to listen on required ports, port 8080 is already in use,
It worked up until 2 weeks ago.  Yesterday I fixed one site by moving to 8443 and 8080, but now a second site is complaining that 443, 80, 8443 and 8080 are in use.  Previously I could run multiple DDEV sites on my machine.
I tried to fix by switching to colima from docker, but the problem persists.
I'm not running apache, lando mamp or anything else.  I'm on OSX Ventura.
Steps To Reproduce
Configure a site.  try to run.  Error.  Fix by changing ports.  Start a second site. Error on both standard and fallback ports.
Anything else?
Nothing else

Hi Thanks
OK.  I  completely removed & uninstalled docker desktop  and then did
brew install docker
pwered off
colima start --cpu 4 --memory 6 --disk 100 --dns=<IP_ADDRESS>
ddev start

same problem.
When I look to see what is conflicting, i get nothing
wwwroot git:stage ❯ sudo lsof -i :443 -sTCP:LISTEN                                                                                ⏎ ✹
wwwroot git:stage ❯

Could Fritzbox be causing this?  or is that really a seperate issue?

Fritzbox is worth solving, but has nothing to do with this issue.
Something is listening on those ports.  You can easily verify that something is listening by telnetting to it. telnet localhost 443 for example. Sometimes curl -I https://localhost:443 will tell you what's doing it.
But... you said you were having trouble with other ports, not 443. Please run the lsof against the ports that are actually having trouble.

I'll be happy to do a zoom call with you to help you sort this out. Let me know what times might work and what timezone you're in.

Its having trouble with 80 AND 443.  and then when i tried moving a ddev site to 8443 and 8080, that worked, for one site.  But not a second one.
Here are results from curl:
wwwroot git:stage ❯ curl -I https://localhost:443                                                                                ⏎ curl: (35) error:02FFF036:system library:func(4095):Connection reset by peer
and telnet

I'll be happy to do a zoom with you. We could do right now since you're still up, or we could do in my morning/your afternoon tomorrow. It would be easier to set this up if you ping me in Discord

I've just "fixed" it.  I had Screen time parental controls on, because I had used it to try and disable access to an api to test what happens when the api was unavailable.  Removing the restriction didnt seem to be enough so I turned off content restriction compeltely and rebooted the machine.  Now ddev will start.  The restriction was not for localhost so I didnt twig it might be interfering.  Quite embarrasing.  I thought maybe the ventura update had broken it but no, it was me all along!  sorry to have wasted your time and thank you for sticking with me and suggestions to debug.

How about that! Could you write a little more here about what caused the problem and how you solved it?


Selection fix

Proposed Change:
When timeline is placed with the left margin to the web page it breaks the calculation on the "dragend" event, so that the selection of events is broken, as the start and end time are shifted.
This pool request fixes that issue and also adds the selected time frame and rows to the callback "onInteraction" function:
onInteraction(key, selectedItems, {
              startTime,
              endTime,
              rowTopIndex,
              rowBottomIndex
});

Now users will be able to use the selected range for creating events, for example.
Change type
Put an x in the boxes that apply

[x] Bugfix
[x] New feature

Checklist

[x] I have added tests that prove my fix is effective or that my feature works
[] I have added necessary documentation (if appropriate)

@uncledent I've had a look at this PR and it seems that your change has broken the ability to select a time period from right to left.
see attached gif

@bendog i will fix that

Closed due to inactivity


Document ExecuteJsi and other JSI Cxx-style APIs

We should document these public APIs

Duplicate of #334


Support specifying prober interval / timeouts as durations

Copied from https://github.com/google/cloudprober/issues/611, originally filed by @vadimberezniker.
It would be nice if the prober config supported specifying durations as user-friendly strings.
i.e.
interval: "10m"
timeout: "15s"

instead of
interval_msec: 600000  # 10m
timeout_msec: 15000   # 15s

+1ed by @benner @manugarg

@manugarg I'd like to pick this up, could you please assign the issue to me?

@nvidyala Done. Thanks for picking this up.


_init function needs to be represented in the diagram

Description:
_init function is not supported in the diagrams
Steps to reproduce:
refer example : object_initializer.bal

Currently, we do not render the Objects. We might need to address this as well

No longer relevant for this repo


Global leaks, CookieAccessInfo, Cookie, CookieJar

I am using v1.1.2 och email-templates and running some tests using http://github.com/hapijs/lab
as a test runner.
As soon as I require the module lab starts to complain about leaks detected.
The following leaks were detected:CookieAccessInfo, Cookie, CookieJar

Please clean up.

Not sure how can this library leak objects it doesn't use?

it's some dependency of it then. I'll try to pinpoint it.
Den 17 feb 2015 23:20 skrev "Jeduan Cornejo"<EMAIL_ADDRESS>
Not sure how can this library leak objects it doesn't use?
—
Reply to this email directly or view it on GitHub
https://github.com/niftylettuce/node-email-templates/issues/112#issuecomment-74767428
.

@kmpm any update?


[Bug Report] infiniteScroll in official document is not working

Element UI version
2.9.1
OS/Browsers version
windows7/ chrome 74.0.3729.131
Vue version
2.6.10
Reproduction Link
https://codepen.io/anon/pen/mYQpoa
Steps to reproduce
Run the sample code in offical document site, or directly click the "Try it!" on the document site: https://element.eleme.io/#/en-US/component/infiniteScroll
What is Expected?
I could see the list and scroll down
What is actually happening?
Nothing show in screen and there is the error message in console:
"[Vue warn]: Error in directive infinite-scroll inserted hook: 'TypeError: Failed to execute 'observe' on 'MutationObserver': parameter 1 is not of type 'Node'.'

Any updates?

Got the same issue with infinite scroll component. Anybody can help? Thanks.

Fixed in 2.11.0

See https://github.com/ElemeFE/element/pull/16698/files#diff-27caed56f87e34abfd7561c620d9bcf2R10
just lost style="overflow:auto" on ul tag, it works in 2.9.1.

#16698


Update README and provide CONTRIBUTING.md

Updates the README.md to provide details that align with the types of info provided for xmtp-js, xmtp-ios, and xmtp-flutter.

Provides placeholder sections using snippets from xmtp-swift and xmtp-js README files to provide context for the types of info you might want to provide for xmtp-android.
Ctrl+f for "kotlin" to see my notes flagging areas that need some Kotlin-specific details! =)
Remove and add sections as you see fit! For dev preview, it may not be time to provide all of the details.

Adds a CONTRIBUTING.md file - which needs just one Kotlin-specific update.

Alright @jhaaaa I did a bunch of work on this today to get the docs ready for pre-preview incoming later this week. 🙏


[REVIEW] subinsk

GitHub Profile Link
https://github.com/subinsk

Hey 👋, Here are few changes needed to be dono to your profile

Consider removing the profile views count due it's inaccurate
Trying aligning the social media badges side by side
Maybe adding all the widgets under one section

Looks good, some suggestions

great profile picture
make more use of the bio
suggest moving "buy me a coffee to the end of the profile"
make better use of the blank space on the right
remove 1 or 3 of the pinned repos to make 4 or 2


30% GPU Load when Desktop Dimmer is on

🤷🏽♂ Current Behaviour

Task Manager shows that when I toggle Desktop Dimmer on, GPU loads jump ~30% on my high end GPU (GTX 1080ti)
I don't see any reason for why this should be normal?

When using an alternative like CareUEyes, there is no discernible impact on GPU loads

Same here, on Windows 10.


S3 - adding note about consistency around the list operation.

Well spotted, and you're absolutely right. Thanks for the contribution!


Keyboard Manager, assign alt codes to key (combo)

📝 Provide a description of the new feature
What is the expected behavior of the proposed feature?  What is the scenario this would be used?
I would be able to assign any alt code to any key combo, for instance:
I would be able to assign the key combo "ctrl + alt + o" to alt code: "alt + 155" using the Keyboard Manager in PowerToys. So I would be able to type ø (alt + 155) by pressing "ctrl + alt + o", even if there is no key available for "ø" in my keyboard layout.
If you'd like to see this feature implemented, add a 👍 reaction to this post.

Closing as duplicate: #6976


Check Command Registering API

There has been a few issues about a few of the commands in the command pallete not working, mainly the bufferlist ones.
I think this comes down to the buffer list checking if a menu is open, and if one is, it will not open the buffer list. Which is the most sensible approach, but means that the command pallete option does not work. Then, since the command is registered in Oni, it appears in the menu when it will never work from that menu.
Instead, does Oni need some way of setting a command as a keybind only? We want the command to be available everywhere, but we also don't really want it in the command pallete. That, or some way of more inteligently reacting to the command pallete, so the plugins can react to it.
Ie, I'm suggesting that for Oni.command.registerCommand, there be some optional flag to say "This command is a keybind" so it doesn't necessarily appear in the command pallete.

@CrossR I agree re. filtering out I get @TalAmuyal point re. visibility but tbh I think its confusing ux to have in effective commands and also tbh the vast majority of the commands that ive "added" to it werent intended to show up there there just wasn't a way to prevent that I think command discoverability should come through maybe an oni help section in app but the command palette should be reserved for functionality users can activate via that menu a la vscode or atom etc. (my two cents) re. potential implementation could add a flag to Command creation to check if the command should be rendered and in the menu just filter out ones which shouldn't render


Tell the author that this fails on your system:     karma start config/karma.conf.js

Vue.js version
1.0.23
Reproduction Link

Steps to reproduce
What is Expected?
What is actually happening?

Thanks for telling us the it fails on you system. Unfortunately, we can't really deduce anything else from your information. Care to provide a bit more details?


Detect use of asterisk (*) in PackageReference items of .NET projects

Let's update the ReadMe as part of this PR.

Let's update the ReadMe as part of this PR.

Done

Let's rebase this PR to get rid of the red CI.

Let's rebase this PR to get rid of the red CI.

Done


DeepExplainer:  TypeError: Fetch argument None has invalid type <class 'NoneType'>

I'm trying to make DeepExplainer work with my keras model (that has custom layers). I got the following error when calculating shap_values.
(Shap  version 0.34.0)
shap_values = e.shap_values(x_test[0][0:4])

Traceback (most recent call last):

  File "<ipython-input-76-3f4d7f395630>", line 1, in <module>
    shap_values = e.shap_values(x_test[0][0:1])

  File "C:\Users\andre\.conda\envs\tf_test\lib\site-packages\shap\explainers\deep\__init__.py", line 119, in shap_values
    return self.explainer.shap_values(X, ranked_outputs, output_rank_order)

  File "C:\Users\andre\.conda\envs\tf_test\lib\site-packages\shap\explainers\deep\deep_tf.py", line 272, in shap_values
    sample_phis = self.run(self.phi_symbolic(feature_ind), self.model_inputs, joint_input)

  File "C:\Users\andre\.conda\envs\tf_test\lib\site-packages\shap\explainers\deep\deep_tf.py", line 299, in run
    return self.session.run(out, feed_dict)

  File "C:\Users\andre\.conda\envs\tf_test\lib\site-packages\tensorflow\python\client\session.py", line 929, in run
    run_metadata_ptr)

  File "C:\Users\andre\.conda\envs\tf_test\lib\site-packages\tensorflow\python\client\session.py", line 1137, in _run
    self._graph, fetches, feed_dict_tensor, feed_handles=feed_handles)

  File "C:\Users\andre\.conda\envs\tf_test\lib\site-packages\tensorflow\python\client\session.py", line 471, in __init__
    self._fetch_mapper = _FetchMapper.for_fetch(fetches)

  File "C:\Users\andre\.conda\envs\tf_test\lib\site-packages\tensorflow\python\client\session.py", line 261, in for_fetch
    return _ListFetchMapper(fetch)

  File "C:\Users\andre\.conda\envs\tf_test\lib\site-packages\tensorflow\python\client\session.py", line 370, in __init__
    self._mappers = [_FetchMapper.for_fetch(fetch) for fetch in fetches]

  File "C:\Users\andre\.conda\envs\tf_test\lib\site-packages\tensorflow\python\client\session.py", line 370, in <listcomp>
    self._mappers = [_FetchMapper.for_fetch(fetch) for fetch in fetches]

  File "C:\Users\andre\.conda\envs\tf_test\lib\site-packages\tensorflow\python\client\session.py", line 258, in for_fetch
    type(fetch)))

TypeError: Fetch argument None has invalid type <class 'NoneType'>

DeepExplainer works when I replicate the simple MNIST example from the documentation, but not with my model. Do you have any workaround? I see that a couple of persons had the same issue in the past, but didnt get a solution. Thanks

Hey, have you found a fix for this? I'm trying to use GradientExplainer for a CNN model and I haven't been able to find a solution so far.


Clarification on results from demo.py.

Hi. I really like your work. Currently I tried to use this work for a clinical gait analysis. I want to get the 3D coordinates(with respect to the camera) of key joints of people.
I run your demo code and got a pretty good visualization result. However, I have several questions about the results.

Two .pth files slam_results.pth, tracking_results.pth and one wham_output.pkl are saved as output. Could you please explain what do they represent? My understanding now is that the 3D coordinates of key joints with respect to the camera is stored in wham_output.pkl, right?
When I looked into wham_output.pkl, I noticed four dictionaries named 0, 1, 4, 8. Their contents are pretty similar. I guess the pose_world is what I need, right? But what are the difference between 0, 1, 4, 8. Are they differ in the gap between the number of frames taken?

Hi @Xijieupenn , thank you for your interest in our work!
slam_results.pth contains the pose of the camera estimated by DPVO. This will be then used to compute camera angular velocity see this line. tracking_results.pth stores 2D keypoints, bounding boxes and image features of each individual shown in the video. Yes, wham_output.pkl is the actual prediction from WHAM, so it includes 3D human motion in both camera and the world coordinates.
When we run off-the-shelf human tracking algorithm, it detects and tracks each individual in the video. The tracking algorithm concludes that there are multiple people shown in the video (where each individual's ID is given as 0, 1, 2, ...) and we only take subjects appeared in the video for over 30 frames.


Fix Travis CI build errors for sidechainBMM branch

See sidechainBMM:
https://travis-ci.org/drivechain-project/bitcoin/branches
Some builds are completing but have failing unit tests, and a few of the builds are failing to compile.
This issue is possibly related: https://github.com/drivechain-project/bitcoin/issues/25

Closing, covered by https://github.com/drivechain-project/bitcoin/issues/42


Song's vocals won't play

Describe your bug here.
My song won't load their vocals (they are .ogg and named voices), can someone help?
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
no

I had this exact issue. the fix is to check  the song tab in the chart editor. is "has voice track" enabled? This is located under the song name. if not try enabling it and see if that fixes it

it's already on

is it Voices.ogg? (its case sensitive)

i think so

@MayoALT yes it is


copr: fix IN expr didn't handle unsigned/signed int properly

What problem does this PR solve?
Issue Number: close #9821
Problem Summary: IN expr didn't distinguish between unsigned int and signed int. So -1(bigint) equals<PHONE_NUMBER>3709551615(bigint unsigned), which is not compatible with MySQL. Here is TiDB issue.
What is changed and how it works?
What's Changed: Add a fn_meta for IN expr when first arg of IN expr is INT type. In this function, we use info which records arg's flag to tell whether two integer is really equal.
Related changes

Need to cherry-pick to the release branch

Check List
Tests

Unit test : impl_compare_in.rs: test_unsigned_signed_int()

Side effects:

performance regression: To fix this issue, we need to check argument's signed/unsigned flag. Also we use HashMap instread HashSet. So maybe little performance regression.

Release note

fix IN expr(coprocessor) didn't handle unsigned/signed int properly

/test

@skyzh PTAL

/test

/test

Sorry but I'm quite confused about this PR. I'm not sure if this condition is correct for all possible combinations of unsigned, signed int.
(*base_val >= 0)
                         || (arg0_unsigned && argi_unsigned)
                         || (!arg0_unsigned && !argi_unsigned)

Would you please explain this a little bit?
Personally, I would suggest removing generics of the newly-added compare function, and then adding more convincing tests which ensure this works.

Copied from TiDB to be compatiable with MySQL.
eg:
col_signed_int  IN (max_uint64)

In the above case. If col_signed_int is -1, result of IN expr is false. So we need to check if unsigned/signed flags.
Why we check "*base_val >= 0"
eg:
col_signed_int IN (col_unsigned_int)

In the above case. If both col_signed_int and col_usngned_int are 1, result of IN expr shoudle be true. So if *base_val is greater than zero, no need to check signed/unsigned falg.

/test

@lzmhhh123 PTAL

/merge

/label needs-cherry-pick-4.0

/label needs-cherry-pick-5.0

/run-cherry-picker


buggy movement

Recursive movement programmed in
Odd bug where code does not properly render any movement range above 4 movement
TODO: fix the bug, then implement terrain affects on movement

Movement no longer buggy
Move squares are created and freed when they are no longer needed
Units can only move once per turn
Units can only move where there is a valid movement square
Units cannot move onto other units anymore
If a selection outside of movement square is made, unit is deselected
Movement accounts for terrain (TODO: grab terrain data to interpret from file, like originally intended)
TODO: read up on gfc_input and fix input manager to be better


Tenant webhook base workspaces + frontend sync

Description
Continuation of #833 related to the myDrive refactoring & #834 for the namespace deletion.
This PR adds the capability of enforcing base workspaces to the tenant mutating webhook and synchronizes the frontend with the backend modifications. In particular:

removes mydrive references from the frontend (#833),
adds the mutation of the tenant upon login (lastLogin flag) so that the tenant operator can recreate the namespace (#834).

Notes
The Drive button on the frontend navbar is still present and will be removed upon removal of NextCloud.
How Has This Been Tested?

[x] Unit tests
[ ] Staging

/deploy-staging

/deploy-staging

/merge


feat: speed up masternodes syncing

Issue being fixed or feature implemented

I've modified the checks to not use mnsync status to check if the blockchain in synced as this can take up to an hour. Instead we check if headers == blocks (or we can check if verification > 0.99 if preferred)
Once synced we force mnsync next twice, checking each time that it's taken. Much like Dashmate does now
What was done?

Check blocks are synced
Force mnsync next twice (once for the chain, once for the governance objects)
Check the results

How Has This Been Tested?

Deployed devnet-echo to test, worked like a charm.
Breaking Changes

None
Checklist:

[x ] I have performed a self-review of my own code
[ x] I have commented my code, particularly in hard-to-understand areas
[ ] I have added or updated relevant unit/integration/functional/e2e tests
[ ] I have made corresponding changes to the documentation

For repository code-owners and collaborators only

[ ] I have assigned this pull request to a milestone

I don't know if this is a chore or a fix... you decide :')

I don't know if this is a chore or a fix... you decide :')

There is a framework that will help you to decide https://www.conventionalcommits.org/en/v1.0.0/

What were the benchmarks between before and after?

@PastaPastaPasta I didn't write down exact benchmarks, but this specific phase of the deployment will reach attempt 40-50 before continuing which means at least 40-50 minutes stuck here. This has been seen by both me and @strophy
When deploying with this PR enabled it will not wait and get past these items in a few minutes. If you want me to do exact benchmarks before merging I'm happy to do it


Double click on file works incorrectly

Open tabs with 2 files (File1.swift and File2.swift)

Double click on File1.swift in projects

Result: File2.swift tab is opened

@Kristalev Double click works incorrectly between projects:

Open several projects
Double click on file in project1
Double click on file in project2

Result: incorrect file is opened
https://user-images.githubusercontent.com/10060762/193032714-868b2e2a-08b2-4a8d-b659-b964e5ed7d7b.mov


Error with CocoaPods 1.0.0-beta6

I updated to CocoaPods Version 1.0.0.
When I try to install my pods I get this error:
Pre-downloading: GradientCircularProgress from https://github.com/keygx/GradientCircularProgress
[!] The GradientCircularProgress pod failed to validate due to 1 error:
- ERROR | attributes: Missing required attribute summary.

I updated podspec file and fixed this error.