HIVE-23571:  Add ValidWriteIdList to SharedCache.TableWrapper

What changes were proposed in this pull request?

Adding ValidWriteIdList to cached store. Inorder to check whether data store in cache is stale or not
Why are the changes needed?

Adding ValidWriteIdList field to table wrapper in Cache Store.
Does this PR introduce any user-facing change?

No
How was this patch tested?

Test case added

@kishendas Can you please review this PR mean while I will add UT in this PR.

@kishendas Could you please review this PR.

@kgyrtkirk Could please review the PR?

@sankarh This PR is about adding validWriteIdList as part of cached store.
Current status

HIVE-23931 - Send ValidWriteIdList and tableId to get_*_constraints HMS APIs (open)
HIVE-23820: [HS2] Send tableId in request for get_table_request API (merged)
HIVE-23571: Add ValidWriteIdList to SharedCache.TableWrapper (open)
HIVE-23821: Send tableId in request for all the new HMS get_partition APIs (merged)

Once above PR get merged. I will raise a PR to add tableId and writedId check to all the methods and there corresponding test cases.


BR_TrackAtMouseCursor is really slow

time=reaper.time_precise()
local track=reaper.BR_TrackAtMouseCursor()
time_used=reaper.time_precise()-time
It needs more than 0.5s.

Use the native API:
https://www.extremraym.com/cloud/reascript-doc/#GetTrackFromPoint


feat: embed pedersen params

uses go:embed to embed the pedersen params and improves the ergonomics of the library. we might consider moving Curve to a curve package so the api can be e.g. curve.Sign(...)

I was thinking of ways to init the curve cleanly. This is awesome!


Replace redirect links of labels-annotations-taints

/docs/reference/kubernetes-api/labels-annotations-taints/ is redirected to /docs/reference/labels-annotations-taints/.
This PR replaces redirect links of labels-annotations-taints with the direct links.

/approve


When trying to compile C++ code I get error

 CompilerResultsLog: ERROR: Could not find definition for module 'UnrealEnginePython' (referenced via default plugins -> UnrealEnginePython.uplugin) Any fix?

It looks like you are trying to build from a binary distribution of the plugin. You need its sources

Oh I thought hitting compile would've compiled my C++ component for my blueprint, not the plugin, sorry.

Running C:/Epic Games/UE_4.18/Engine/Binaries/DotNET/UnrealBuildTool.exe  -projectfiles -project="C:/Users/myuser/Documents/Unreal Projects/MyProject8/MyProject8.uproject" -game -rocket -progress Discovering modules, targets and source code for project... ERROR: Could not find definition for module 'UnrealEnginePython' (referenced via default plugins -> UnrealEnginePython.uplugin) This happens while trying to refresh Visual Studio project too

That error means you are trying to package the project but you do not have copied the UnrealEnginePython sources into the the Plugins directory. If you do not want to use the Python plugin for the runtime, just edit its UnrealEnginePython.uplugin file and change the type of the module 'UnrealEnginePython' from 'Runtime' to 'Editor'

Oh I was just downloading the release, sor

 ERROR: Unable to instantiate module 'UnrealEnginePython': System.Exception: Unable to find Python installation
I get this now?

read here about sources installation https://github.com/20tab/UnrealEnginePython/blob/master/README.md#installation-from-sources-on-windows-64-bit

It wasn't able to find python, sent python to C:\ and renamed to Python26 and it works now. Adding some more paths to line 22 of UnrealEnginePython.Build.cs would fix it (e.g C:\Users\userhere\AppData\Local\Programs\Python\Python36 ) to it


allow reedline ctrl+o to take editor arguments

relevant: https://github.com/nushell/nushell/pull/8105
allows ctrl+o keybinding to take command arguments into account

OK based on https://github.com/nushell/nushell/issues/8714 this will break on Windows with the omnipresence of spaces in filepaths as we still need to separate flags from the command.


Android - Update to 2.2.3

Overview
Currently React Native's local cli is a bit behind in its android gradle plugin version. This PR is an attempt to update the local cli, to allow for better support moving forward.

Updates the gradle plugin version to 2.2.3
Updates the gradle wrapper to 2.14.1
Uses the all for the project wrapper to include sources for API completion

Test plan (required)

Perform all required steps here: https://github.com/facebook/react-native/tree/master/react-native-cli

Run the local npm tests and e2e tests (no longer available)
Test the local cli by using Sinopia

Make sure tests pass on both Travis and Circle CI.
TO NOTE: In a previous issue (https://github.com/facebook/react-native/issues/11500) I was able to update to 2.2.3 comfortably, however there may be other issue I am not aware of. This PR is intended to start discussion on what it will take to update.

Did 'react-native run-android' work OK for you? Did you use a device or an emulator?
I'm curious of a new version makes the "cannot upload binary to device" issue (https://github.com/facebook/react-native/issues/2720) worse.
But sticking to an old version is weird, let's try to upgrade. Merging.

@mkonicek was this merged successfully?
I haven't seen it in any recent releases, and github shows:

Closed with unmerged commits
This pull request is closed, but the jpshelley:Android-UpdateGradlePlugin branch has unmerged commits.

@facebook-github-bot @mkonicek bump ^^ Says its been merged but haven't seen it in any releases.


Weird benchmark results when adding multiversion to rawloader

Thanks for creating this. I've been hoping for something like this for a while now. I'm the author of rawloader:
https://github.com/pedrocr/rawloader/
The raw file decoders have a bunch of hot loops that I know can be sped up with new processor features from compiling with target-cpu=native. I've been doing some performance work in rawloader and have a stable benchmark I can use for any file. I tried adding multiversion to the NEF format used by Nikon cameras and saw these numbers for the average decode time:

Version
Time (in ms)

Master
322.9

Master with target-cpu=native
300.4 (-7%)

Master with only multiversion dependency
310.9 (-4%)

Master with [x86|x86_64]+avx annotation in NEF
317.3 (-2%)

Master with x86_64+avx+avx2+bmi1+bmi2+fma+lzcnt+xsave annotation in NEF
283.9 (-12%)

So, initially I wasn't able to replicate the target-cpu=native result, probably because I needed to enable more features. But what is odd is that just adding multiversion as a dependency speeds things up and then actually adding versioning makes things slower. Finally I did a target with most features the CPU should have and got a result that was significantly better than target-cpu=native which shouldn't be possible at all.
Any ideas on what could be happening here? Maybe it's just the binaries being particularly sensitive to memory layout but I'm also wondering what else multiversion could be doing here.

I'm guessing it either has something to do with memory layout, or just an inconsistent benchmark.  I wouldn't expect enabling AVX to do much, after looking at the code, since it looks like integer math and AVX mostly adds floating point instructions.  It is somewhat odd that using multiversion is faster than target-cpu=native, and that may also be due to an inconsistent benchmark.  Simply adding a dependency on multiversion may not even change the layout (since it's unused).  I think it's possible CPU or disk cache is playing a role in the faster runs.

I'll do more benchmarks to try and be sure but it seems quite stable. I've since tested other formats and it seems more reasonable in those cases.

This is most likely from alignment issues. I played around with some #[repr(align(N))] declarations in the key struct and there's a sweet spot at 32 that creates almost the same speedup as just adding the new dependency. So the new dependency is probably changing the layout enough for this to happen and it's nothing more complex.
The difference between target-cpu=native and enabling all the features may also be alignment stuff helping with vectorization. Or it may be that enabling the features but not the the target CPU is changing the LLVM code generation in ways that happen to be beneficial to this code.
Having seen the macro expansion these explanations all seem fair so I'll close this now.


tiller grpc message size opt

We have a lot of releases and in our cluster helm ls fails yet:
✗ helm ls
Error: trying to send message larger than max (30334253 vs. 20971520)
✗ helm ls -m 10
	next: api-gateway
NAME                 	REVISION	UPDATED                 	STATUS  	CHART           	NAMESPACE
.... <list of releases>

We need to make maxMsgSize configurable.
Fixes #3322

CLA signed

This handles server-side, but what about client-side as well?

See https://github.com/kubernetes/helm/pull/3782

fixed in https://github.com/helm/helm/pull/3782


Support canary events

Filter out canary (heartbeat) events by default by checking if meta.domain == "canary", per Data Engineering and Event Platform Team at T266798. Canary events can be re-enabled by passing enableCanary when creating the stream.
Will be released as 2.1.0 when finished.
To-do:

[x] Add option to re-enable
[ ] Document canary event behavior
[x] Utilize canary events as part of tests
Not-so-possible since canary events are only fired once every hour; much longer than the CI timeout.

Given that canary events will start appearing by next month, it might also be worth backporting this to 2.0.x and 1.0.x tags so that no bumps to package.json would be required for anyone on those versions. I'll think about this more tomorrow.

Given that canary events will start appearing by next month

FWIW, canary events are already present in many of the available streams.  The ones that will have them newly enabled are the ones listed in the ticket: mostly ones that were created before 2020.

Most of the streams covered by this library are part of the list (only exception being mediawiki.revision-tags-change), so it's more or less the same for this specific library. This is probably still something worth backporting to older versions.

Based on what I understood from wikitech:Data Engineering/Systems/Hadoop Event Ingestion Lifecycle#Canary Events, the meta.stream field remains intact and unchanged by ProduceCanaryEvents. The following lines are what fires the events for any client using this library:
// Emit event.
this.emit( data.meta.stream, data, event );
// Emit event to aliases of event stream.
if ( WikimediaEventStreamAliasesKey[ data.meta.stream ] ) {
    for ( const alias of WikimediaEventStreamAliasesKey[ data.meta.stream ] ) {
	this.emit( alias, data, event );
    }
}

which means if meta.stream remains unchanged, canary events will still be received by dependents. TL;DR: The way the code is written right now won't protect current users of this library from receiving canary events when they are enabled. So a backport (even one that only drops canary events and doesn't include an option to enable them) might help in getting this to users who use an older version of this library. This does mean everyone will be required to update when the events become enabled, but I've set the old versions as deprecated on npm anyway, so that was already a given from long ago.
Will now be merging this in a bit!


docs: correct required app permissions in setup description

The linked guide to setup GH apps for authentication is missing the checks permission. Without that the plugin is not able to report back to GH. This changes explicitly states this in the setup description.

[x] Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
[x] Ensure that the pull request title represents the desired changelog entry
[x] Please describe what you did
[ ] Link to relevant issues in GitHub or Jira
[ ] Link to relevant pull requests, esp. upstream and downstream changes
[ ] Ensure you have provided tests - that demonstrates feature works or fixes the issue

See also https://github.com/jenkinsci/github-branch-source-plugin/pull/746.


Feature Request: Add A Duplicate Button

Plz add a DUPLICATE button to reduce duplicate typing.
Make it easier to continue adding snippets.

Yup, will do, thanks for the suggestion.


Sound / music ?

Hi,
This looks awesome. Congrats!
Are you planning to integrate audio/sound generation in any way? I'm intrigued by the idea of mapping the points into sound or music. Any suggestion?
Thanks!

Yes! Combining visuals + sounds using the same approach of Point/Space/Form would be amazing.
At the very least we should support mapping amplitude and frequency. Potentially also offer algorithms based on music theory.
Any other suggestions or use cases you have in mind?

Yes, amplitude and frequency are mandatory. I'm the creator of tonal music theory library, and some kind of connection between both projects would be really nice. I'll think about it.
Meanwhile, do you have a plan to develop an audio module for "pts" or you're delegating that to the community?

Hi @williamngan,
I would love to work with you on this, but I am afraid (as the delay in my answers reveals) that I am too busy to put time and energy into this development (the situation may change in a couple of months).
Anyway, a Signal class that provides some kind of continuous data that can be mapped to things like amplitude or frequency would be wonderful (frequency can be used not only to control pitch, but also other kind of sound modulations).
I think this continuous signal can be converted to discrete values (like note pitch) using external libraries like tonal to produce more "musical" or "melodic" effects. But I think is ok to have this kind of transformations outside pts itself.
With this in mind, I think that if you build this kind of Signal class I would be able to produce (with help) some examples of usage with Web Audio or some libraries like Tone.js and/or Gibberish
Thanks for your work!

Sounds like a good plan, @danigb . I'll try to get started on it soon, and would love your feedbacks on it. Let's keep this thread open for discussions.
Totally understood that you might be busy (same here actually, Pts is my midnight project 😅).


Adding DeFi Oracles to the FTSO list

Please modify the filename to all lowercase and only modify your own entries.

The image file must be renamed to all lowercase, not just its references.


Python dependencies

Currently, we have dependencies on

numpy
scipy
matplotlib
pandas
(maybe I forgot some?)

Should the makefile test this and suggest their installation?

The difficulty here is that these are not strict dependencies since the analysis of the results can be done without Python.
I think suggesting the installation in the makefile is a good compromise


SIGUSR1 and other debugging mechanisms should be documented

On UNIX you can send SIGUSR1 to an Elvish process to force it to produce a Go backtrace of all goroutines. That information isn't particularly useful to anyone who isn't familiar with Go and the Elvish source code but the mechanism should still be documented. Along with any other mechanisms useful for debugging Elvish behavior. For the moment it's probably sufficient to mention SIGUSR1 in the CONTRIBUTING.md document.

As I mentioned recently on Gitter/IRC/IM yesterday the -log (and -logprefix) options also need to be documented. However, my inclination is to replace both options with a -debug option that provides a bit more flexibility. Especially with respect to filtering which messages are emitted and optionally including a stack backtrace for selected messages.


two-fer: implementation of two-fer exercise in Kotlin

In Gitter it was recommended to me by @Nobbz to also change the hello-world exercise to be updated to the new style of hello-world, (which simply asks the user to return the string "hello world!"), instead of the current implementation that requires string manipulation. This then allows string manipulation to be introduced to the user via the two-fer exercise that is implemented in this PR.

Hi! Thanks for this contribution!
I'll add review comments inline in a second, but I also wanted to post some information relevant to the hello-world update you are tackling. Here are the PRs where we updated hello-world in the Java track: https://github.com/exercism/java/pull/663 and https://github.com/exercism/java/pull/666. They should give you some pointers as to what updates may need to happen to supporting files in the Kotlin track. Feel free to reach out if you have any questions!

Hey @stkent, I believe my latest commit has completed your requested changes? If there is something I am missing or more changes required please let me know. Thanks! :+1:

Looking at this tomorrow!


Include zlib only if the opts.compress flag is true

This repository is causing issue with getting hyperdht to work on Mobile and on Pear as they both do not support node:zlib, which is required only in the case if compression flag is passed in the options.

Closing this PR and moving to my own namespace


Release checklist cep18 v1.1

[ ] (DEV) Close or defer all tickets in the epics (code complete/scope closure) - code freeze
[ ]  (DEV) Create release candidate branch (branching, tagging)
[ ]  (Business Owner) Identify destination & distribution targets (communications, announcements)
[ ]  (SRE) Ensure SRE pipelines are in place
[ ]  (DEV) unit & Integration tests have passed
[ ]  (DEV) Create readme in git repo (scope closure)
[ ]  (DEV) ensure changelog is in place
[ ]  (SRE) acceptance tests in a production environment has passed
[ ]  (DEV) Documentation updates are in place (docs team)

@RBYGLLS can we save this list in confluence and close this "issue" ?

Ok I work on it


Which dnlib version does ConfuserEx depends?

I used the newest version 3.2, but when compile, there is an error in InjectHelper.cs : can not find class ImportResolver

The release/2.0 branch depends on Version 3.2 of dnlib. The master branch depends on a custom version of dnlib you can find here.

thanks


Switch to using PackageLicenseExpression

By using license expression it will show up correctly on nuget.org and helps license validators/checkers to know what is the exact license type.

Thanks for that @lahma... this attribute is new for me


feat: Html converter package

Feature #1100

Feel free to vote the package name

Are you sure we want to introduce md here?

It would be nice only having logic for html and delta without use of md

Are you sure we want to introduce md here?

Not sure what you mean, I already added the README.md of the package

By md I mean markdown

It would be nice only having logic for html and delta without use of md

Yes, they can easily without the README.md I also updated the example
It only have option to convert the delta to HTML, then convert the HTML back to quill delta and load it, which can emulate the process faster

Is it like delta <-> mark down <-> html

By md I mean markdown

Currently, there is no built-in way to convert HTML to Delta without building it from scratch, so I use this solution as a workaround, it's from a contributor in GitHub
Of course we will update this package and do this from scratch or do it in better way but for now I think we should focus on flutter_quill and flutter_quill_extensions

Ok it does feel hacky but sure as long as it works

hacky

The logic for converting Delta to HTML is fine
The logic for converting the HTML back to Delta is not ready and we will change it in the future


2FA login not working

On Windows
$ forge login

What is your Laravel Forge email?:
 ><EMAIL_ADDRESS>
 What is your password?:
 >

Logging in...

 What is your 2FA token?:
 > 981182

In Crawler.php line 886:

  The current node list is empty.

Same here on Mac with PHP 7.4.16


i18n generation failure kills maintenance all job

[2018-10-10 02:33:32,229] WARNING in logger: Saving to shared_web/translations/da/LC_MESSAGES/messages.po
Traceback (most recent call last):
  File "run.py", line 95, in <module>
    run()
  File "run.py", line 33, in run
    task(sys.argv)
  File "run.py", line 67, in task
    run_all_tasks(module)
  File "run.py", line 93, in run_all_tasks
    s.run() # type: ignore
  File "/home/discord/decksite/maintenance/update_translations.py", line 31, in run
    local_file=pofile, filters='not_fuzzy')
  File "/home/discord/.local/lib/python3.6/site-packages/poeditor/client.py", line 442, in export
    "filters - filter results by {}".format(self.FILTER_BY))
poeditor.client.POEditorArgsException

I don't repro this on local because of
[2018-10-09 14:17:40,039] WARNING in logger: Missing poeditor.com API key

but I'm worried this is blocking actual needed jobs on prod?
I don't want to copy the key local and run it in case it screws something up.


something wrong

Hello,
I followed instructions , but all i see is "Subscribing...." message under form when trying to subscribe

This happens if the action url does not start with http:// (but only when testing from a locally opened file in the browser).
To fix, you need to make sure the action url starts with http://
I've updated the readme and instructions to be more clear about this.
Thanks for reporting it.


IE nav button position

I also noticed an issue with nav button position in IE 9/10/11. The button position is low compared to the "flat" setting and generates vertical scroll bars. I'm using the below CSS to tame it for now. I'll try and investigate to see if there is a better solution and get some screen shots as well.
@media all and (-ms-high-contrast:none)
 {
 .flipster__button { top: 15%; margin-top: -2em; } /* IE10 */
 *::-ms-backdrop, .flipster__button { top: 15% !important; margin-top: -2em; } /* IE11 */
 }

Thanks for the feedback!
Strange issue, not sure what's causing it. I haven't done a full browser test yet, but as I dig in there I'll take a closer look at this.

This repository is going into very-low-maintenance mode and declaring backlog bankruptcy


fix: product carousel on mobile in the horizontal position

Changes

closes #783

Checklist

[x] I followed contributing guidelines

@akrajna I overrode breakpoints for carousel, now it has 2 items just on vertical mobile.

works great


How to change the height of Bottom Navigation Bar

Make sure these boxes are checked before submitting your issue - thank you!

[x] Check in the issue tracker whether
this issue wasn't already reported.
[ ] If it's a bug report check that clear steps to reproduce,
stacktrace, expected behaviour and other needed details are provided.
[x] Feature requests contain a description of the feature
you're requesting as well as a brief scenario explaining
under what circumstances this might be useful.

Is there anyway to change the height of the bottom navigation bar and resize the text on the "Next" and "Back" buttons?

Hi @Sutheres,
You can do that by setting a custom style on the view as described here: https://github.com/stepstone-tech/android-material-stepper#custom-styling
So you would need to override the style of the navigation bar to change its height and button styles to change font sizes.

@zawadz88 Thanks! I was able to change the height of the navigation bar and the size of the text. However, I am having trouble trying to override the size of the chevron in the next button. Is there an attribute in ms_nextNavigationButtonStyle that would allow me to use a drawable of my own or of a different size?

You could change the drawable by specifying a different drawable in the adaper as described here: https://github.com/stepstone-tech/android-material-stepper/blob/master/README.md#changing-button-labels--compound-drawables-per-step


Fixing issues after Testing

Remove signed key
Fix search issue
Fix section heading position

1.Removed signed key ,
2.Fixed search issue
3.Fixed issue of section and chapter heading


Feat: Implement Max.Redirects & Accepted Status Codes

This PR implements options in each monitor for:

The maximum number of redirects to follow,
The ranges of status codes to accept.

I'm a bit torn with the variable naming, since I want to follow the existing naming conventions, but true camel case should be maxRedirects.
multiselect support for Vue 3 has not hit stable yet, but functionality looks fine to me.
Also, sorry for bundling a bit of styling change, but I got really triggered by the lack of spacing 😂

Appreciate with the work!
The only problem is I think in this way, it cannot cover @Hyurt's use case. https://github.com/louislam/uptime-kuma/issues/11#issuecomment-890578686

Currently the user can select a mix of preset ranges and individual codes, so "200-299", "401", "402", "403" would work.
I guess the alternative would be to give user a text box and let them enter arbitrary ranges, but I'm not sure if having to handle parsing or client-side validation is worth the effort.

Currently the user can select a mix of preset ranges and individual codes, so "200-299", "401", "402", "403" would work.
I guess the alternative would be to give user a text box and let them enter arbitrary ranges, but I'm not sure if having to handle parsing or client-side validation is worth the effort.

Nice, I do not checkout yet, I thought it accepts range only.🤣

Updated the color to match the theme.
Btw, I updated sqlite3, you may need to run "npm install" again.


Protractor, jasmine : How to print current page URL on console

My requirement is to print Current page url on console.
I tried with console.log('URL is= '+driver.browser.getCurrentUrl());
Getting below error on console
Failed: driver is not defined

Hi there!
Your question is better suited for StackOverflow or Gitter. Please ask a question there with the 'protractor' tag or post in the Gitter Channel to get help.
From the the getting help section of the README:

Please ask usage and debugging questions on StackOverflow (use the "protractor" tag) or in the Angular discussion group. (Please do not ask support questions here on Github.)

Thanks!


skip cashing for exceptions

resolves #215

well, confused about what "being_calculated" stands for...

@shaypal5, confused about what "being_calculated" stands for...

For non-local cores, like MongoDB, and long calculations, and to allow cross-machine (basically cross-kernel) caching, this supports the case where the same code calls the same decorated function with the same arguments (lets say a=3, b=5) on two different kernels/machine, 1 second apart (for example), for a calculation that lasts 10 minutes.
The first calls create a new entry for the cache key derived for a=3, b=5, with being_calculated == True in the entry, then waits for the function to return (when it will return, it will cache the result in the existing entry and update being_calculated == False). After 1 second, the same decorated function will get called with the same arguments on some other machine; it will search the MongoDB and find out an entry already exists for this argument set, and will see being_calculated == True, and will thus wait for the existing calculation (function call) to finish without triggering a second (redundant) call to the wrapped function.
Kapich?

@shaypal5 made two dataclass refactoring #237 #239 to make my path to better understand the codebase

@shaypal5 seems to be fixed now if you could check it


Release 3.0.0

Once #228 is merged.
Unless there's anything showstopping we need in? I was hoping to get #144 in, but haven't got a plan on that yet :sweat:

Waiting (possibly) on #144.

I'd like to introduce #238 in 3.0.0 as well. :tada:

Agreed.
--
Ash Furrow
iOS Developer, Author
http://ashfurrow.com

On Oct 3, 2015, at 1:39 AM, Alexander Schuch<EMAIL_ADDRESS>wrote:
I'd like to introduce #238 in 3.0.0 as well.
—
Reply to this email directly or view it on GitHub.

Just gonna tidy up the docs around plugins, and we should be good to go on this.

Fixed in https://github.com/Moya/Moya/commit/d61003dd3c5e4bea18d69622a9038daa0aae7064 and pushed to CocoaPods trunk: https://raw.githubusercontent.com/CocoaPods/Specs/9186a2cd0f90f5cc20fa56d8a01f93bdeb1480a0/Specs/Moya/3.0.0/Moya.podspec.json


Add workarounds for https://github.com/quarkusio/quarkus/issues/4407

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/camel-quarkus-pr/215/Failed Tests: 2camel-quarkus-pr/org.apache.camel.quarkus:camel-quarkus-integration-test-platform-http: 2org.apache.camel.quarkus.component.http.server.it.PlatformHttpIT.basicorg.apache.camel.quarkus.component.http.server.it.PlatformHttpIT.testRegistrySetUp

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/camel-quarkus-pr/219/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/camel-quarkus-pr/218/Build result: FAILURE[...truncated 788.02 KB...]    at java.lang.reflect.Method.invoke (Method.java:498)    at hudson.maven.Maven3Builder.call (Maven3Builder.java:139)    at hudson.maven.Maven3Builder.call (Maven3Builder.java:70)    at hudson.remoting.UserRequest.perform (UserRequest.java:212)    at hudson.remoting.UserRequest.perform (UserRequest.java:54)    at hudson.remoting.Request$2.run (Request.java:369)    at hudson.remoting.InterceptingExecutorService$1.call (InterceptingExecutorService.java:72)    at java.util.concurrent.FutureTask.run (FutureTask.java:266)    at java.util.concurrent.ThreadPoolExecutor.runWorker (ThreadPoolExecutor.java:1149)    at java.util.concurrent.ThreadPoolExecutor$Worker.run (ThreadPoolExecutor.java:624)    at java.lang.Thread.run (Thread.java:748)[ERROR] [ERROR] Re-run Maven using the -X switch to enable full debug logging.[ERROR] [ERROR] For more information about the errors and possible solutions, please read the following articles:[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException[ERROR] [ERROR] After correcting the problems, you can resume the build with the command[ERROR]   mvn  -rf :camel-quarkus-integration-test-salesforcechannel stoppedAdding one-line test results to commit status...Setting status of 9a79c0b1781f80782ebe6b5c5584984d0546fa56 to FAILURE with url https://builds.apache.org/job/camel-quarkus-pr/218/ and message: 'FAILURE 73 tests run, 4 skipped, 0 failed.'Using context: default


Linux: undefined reference to `terminal_font8'

bearlibterminal/source/terminal.d:232: undefined reference to `terminal_font8'

It works fine for me.  What version of bearlibterminal do you have, and what version of the wrapper?  How are you building?

I am use wrapper 1.5.2 and also tried ~master. Linking with .so from ./lib/ and also tried library binary from the official site.

It looks like #1 - probably, changes in the upstream

Yes, except that it's still there in upstream.  If you comment out void terminal_font8(const char*); and void font(string name) { terminal_font8(toStringz(name)); } does it work?  Are you building with dub or manually?

does it work?

Yes

Are you building with dub or manually?

With dub

Problem was on my side, sorry (since time of  #1 my /usr/local/lib contained old version of library binary)
Excuse for troubling!

Alright, cool.  I noticed that you did a static import of blt, which is mostly unnecessary since there's already a pseudo-namespace in a static struct.  If you want it to be called BT, how about import BearLibTerminal: BT = terminal?  Then you can just BT.open(), BT.read(), etc., without having to type terminal every time.

True. Thanks, I am will change it


Pcap is not working on OpenVZ correctly

Hello, Graeme!
How are you?
I found new bug in your awesome library. Could you help me with it?
sudo ./find_pcap_devices
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0x1 addr=0x0 pc=0x43b188]

goroutine 1 [running]:
code.google.com/p/gopacket/pcap.sockaddr_to_IP(0x0, 0x0, 0x0, 0x0, 0x0, 0x0)
    /home/sergey/golang/src/code.google.com/p/gopacket/pcap/pcap.go:539 +0x58
code.google.com/p/gopacket/pcap.findalladdresses(0x2be1430, 0xc2090d63f0, 0x0, 0x1)
    /home/sergey/golang/src/code.google.com/p/gopacket/pcap/pcap.go:525 +0xd5
code.google.com/p/gopacket/pcap.FindAllDevs(0xc2080c60b0, 0x3, 0x3, 0x0, 0x0)
    /home/sergey/golang/src/code.google.com/p/gopacket/pcap/pcap.go:511 +0x343
main.main()
    /home/sergey/golang/find_pcap_devices.go:9 +0x2a

goroutine 17 [syscall, locked to thread]:
runtime.goexit()
    /usr/local/go/src/runtime/asm_amd64.s:2232 +0x1

We are running gopacket inside of OpenVZ container which uses venet0 as main interface.
package main

import (
    "code.google.com/p/gopacket/pcap"
    "fmt"
)

func main() {
    all_devices, err := pcap.FindAllDevs()
    if err != nil {
        panic(err)
    }
    for _, device := range all_devices {
        fmt.Println(device.Name)
    }
}

Hey, Pavel,
I can sure try :)
First off, would you mind switching from "code.google.com/p/gopacket" to "
github.com/google/gopacket"?  We moved the code about a month ago, so we
need to make sure you're at the most recent version.
Once you've done that, please run the check again and send me the output?
Thanks!
--Graeme
On Wed, May 6, 2015 at 6:57 AM, Pavel Odintsov<EMAIL_ADDRESS>wrote:

Hello, Graeme!
How are you?
I found new bug in your awesome library. Could you help me with it?
sudo ./find_pcap_devices
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0x1 addr=0x0 pc=0x43b188]
goroutine 1 [running]:code.google.com/p/gopacket/pcap.sockaddr_to_IP(0x0, 0x0, 0x0, 0x0, 0x0, 0x0)
/home/sergey/golang/src/code.google.com/p/gopacket/pcap/pcap.go:539 +0x58code.google.com/p/gopacket/pcap.findalladdresses(0x2be1430, 0xc2090d63f0, 0x0, 0x1)
/home/sergey/golang/src/code.google.com/p/gopacket/pcap/pcap.go:525 +0xd5code.google.com/p/gopacket/pcap.FindAllDevs(0xc2080c60b0, 0x3, 0x3, 0x0, 0x0)
/home/sergey/golang/src/code.google.com/p/gopacket/pcap/pcap.go:511 +0x343main.main()
/home/sergey/golang/find_pcap_devices.go:9 +0x2a
goroutine 17 [syscall, locked to thread]:runtime.goexit()
/usr/local/go/src/runtime/asm_amd64.s:2232 +0x1
We are running gopacket inside of OpenVZ container which uses venet0 as
main interface.
package main
import (
"code.google.com/p/gopacket/pcap"
"fmt"
)
func main() {
all_devices, err := pcap.FindAllDevs()
if err != nil {
panic(err)
}
for _, device := range all_devices {
fmt.Println(device.Name)
}
}
—
Reply to this email directly or view it on GitHub
https://github.com/google/gopacket/issues/77.

Hello!
With "github.com/google/gopacket/pcap" it have same problem.
On real hardware work good, but in OpenVZ container -
sudo ./find_pcap_devices
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0x1 addr=0x0 pc=0x452788]

goroutine 1 [running]:
github.com/google/gopacket/pcap.sockaddr_to_IP(0x0, 0x0, 0x0, 0x0, 0x0, 0x0)
	/home/sergey/golang/src/github.com/google/gopacket/pcap/pcap.go:540 +0x58
github.com/google/gopacket/pcap.findalladdresses(0x1f37430, 0xc2090d83f0, 0x0, 0x1)
	/home/sergey/golang/src/github.com/google/gopacket/pcap/pcap.go:526 +0xd5
github.com/google/gopacket/pcap.FindAllDevs(0xc2080c60b0, 0x3, 0x3, 0x0, 0x0)
	/home/sergey/golang/src/github.com/google/gopacket/pcap/pcap.go:512 +0x343
main.main()
	/home/sergey/golang/find_pcap_devices.go:10 +0x2a

goroutine 17 [syscall, locked to thread]:
runtime.goexit()
	/usr/local/go/src/runtime/asm_amd64.s:2232 +0x1

Could you patch in the following diff and rerun?:
--- pcap/pcap.go        2015-05-06 09:52:39.133588166 -0600
+++ pcap/pcap.go        2015-05-06 09:56:41.882089786 -0600
@@ -523,6 +523,7 @@
for curaddr := addresses; curaddr != nil; curaddr =
(*_Ctype_struct_pcap_addr)(curaddr.next) {
var a InterfaceAddress
var err error

          fmt.Printf("XXX ADDR: %#v PTR: %p\n", curaddr, curaddr.addr)
          if a.IP, err =

sockaddr_to_IP((*syscall.RawSockaddr)(unsafe.Pointer(curaddr.addr))); err
!= nil {
continue
}
On Wed, May 6, 2015 at 9:47 AM, Sergei Mamonov<EMAIL_ADDRESS>wrote:

Hello!
With "github.com/google/gopacket/pcap" it have same problem.
On real hardware work good, but in OpenVZ container -
sudo ./find_pcap_devices
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0x1 addr=0x0 pc=0x452788]
goroutine 1 [running]:github.com/google/gopacket/pcap.sockaddr_to_IP(0x0, 0x0, 0x0, 0x0, 0x0, 0x0)
/home/sergey/golang/src/github.com/google/gopacket/pcap/pcap.go:540 +0x58github.com/google/gopacket/pcap.findalladdresses(0x1f37430, 0xc2090d83f0, 0x0, 0x1)
/home/sergey/golang/src/github.com/google/gopacket/pcap/pcap.go:526 +0xd5github.com/google/gopacket/pcap.FindAllDevs(0xc2080c60b0, 0x3, 0x3, 0x0, 0x0)
/home/sergey/golang/src/github.com/google/gopacket/pcap/pcap.go:512 +0x343
main.main()
/home/sergey/golang/find_pcap_devices.go:10 +0x2a
goroutine 17 [syscall, locked to thread]:
runtime.goexit()
/usr/local/go/src/runtime/asm_amd64.s:2232 +0x1
—
Reply to this email directly or view it on GitHub
https://github.com/google/gopacket/issues/77#issuecomment-99519233.

Output with this patch -
sudo ./find_pcap_devices
XXX ADDR: &pcap._Ctype_struct_pcap_addr{next:(*pcap._Ctype_struct_pcap_addr)(0x1f3c4d0), addr:(*pcap._Ctype_struct_sockaddr)(nil), netmask:(*pcap._Ctype_struct_sockaddr)(nil), broadaddr:(*pcap._Ctype_struct_sockaddr)(nil), dstaddr:(*pcap._Ctype_struct_sockaddr)(nil)} PTR: 0x0
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0x1 addr=0x0 pc=0x452a88]

goroutine 1 [running]:
github.com/google/gopacket/pcap.sockaddr_to_IP(0x0, 0x0, 0x0, 0x0, 0x0, 0x0)
	/home/sergey/golang/src/github.com/google/gopacket/pcap/pcap.go:542 +0x58
github.com/google/gopacket/pcap.findalladdresses(0x1f3c430, 0xc2090d23f0, 0x0, 0x1)
	/home/sergey/golang/src/github.com/google/gopacket/pcap/pcap.go:528 +0x1c2
github.com/google/gopacket/pcap.FindAllDevs(0xc2080c40b0, 0x3, 0x3, 0x0, 0x0)
	/home/sergey/golang/src/github.com/google/gopacket/pcap/pcap.go:513 +0x430
main.main()
	/home/sergey/golang/find_pcap_devices.go:10 +0x2a

goroutine 17 [syscall, locked to thread]:
runtime.goexit()
	/usr/local/go/src/runtime/asm_amd64.s:2232 +0x1

Okay, weird, it appears that sometimes we get back a linked list with
elements with nil addr.  The attached patch should fix... could you confirm
that it does?  If so, I'll send up a pull request to the master branch.
On Wed, May 6, 2015 at 10:12 AM, Sergei Mamonov<EMAIL_ADDRESS>wrote:

Output with this patch -
sudo ./find_pcap_devices
XXX ADDR: &pcap._Ctype_struct_pcap_addr{next:(*pcap._Ctype_struct_pcap_addr)(0x1f3c4d0), addr:(*pcap._Ctype_struct_sockaddr)(nil), netmask:(*pcap._Ctype_struct_sockaddr)(nil), broadaddr:(*pcap._Ctype_struct_sockaddr)(nil), dstaddr:(*pcap._Ctype_struct_sockaddr)(nil)} PTR: 0x0
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0x1 addr=0x0 pc=0x452a88]
goroutine 1 [running]:github.com/google/gopacket/pcap.sockaddr_to_IP(0x0, 0x0, 0x0, 0x0, 0x0, 0x0)
/home/sergey/golang/src/github.com/google/gopacket/pcap/pcap.go:542 +0x58github.com/google/gopacket/pcap.findalladdresses(0x1f3c430, 0xc2090d23f0, 0x0, 0x1)
/home/sergey/golang/src/github.com/google/gopacket/pcap/pcap.go:528 +0x1c2github.com/google/gopacket/pcap.FindAllDevs(0xc2080c40b0, 0x3, 0x3, 0x0, 0x0)
/home/sergey/golang/src/github.com/google/gopacket/pcap/pcap.go:513 +0x430
main.main()
/home/sergey/golang/find_pcap_devices.go:10 +0x2a
goroutine 17 [syscall, locked to thread]:
runtime.goexit()
/usr/local/go/src/runtime/asm_amd64.s:2232 +0x1
—
Reply to this email directly or view it on GitHub
https://github.com/google/gopacket/issues/77#issuecomment-99525924.

Sorry, but I not see patch.

--- pcap/pcap.go        2015-05-06 09:52:39.133588166 -0600
+++ pcap/pcap.go        2015-05-06 10:18:24.737503358 -0600
@@ -521,11 +521,21 @@
// TODO - make it support more than IPv4 and IPv6?
retval = make([]InterfaceAddress, 0, 1)
for curaddr := addresses; curaddr != nil; curaddr =
(*_Ctype_struct_pcap_addr)(curaddr.next) {

          // Strangely, it appears that in some cases, we get a pcap

address back from

          // pcap_findalldevs with a nil .addr.  It appears that we

can skip over

          // these.

          if curaddr.addr == nil {

                  continue

          }
          var a InterfaceAddress
          var err error
          if a.IP, err =

sockaddr_to_IP((*syscall.RawSockaddr)(unsafe.Pointer(curaddr.addr))); err
!= nil {
continue
}

          // To be safe, we'll also check for netmask.

          if curaddr.netmask == nil {

                  continue

          }
          if a.Netmask, err =

sockaddr_to_IP((*syscall.RawSockaddr)(unsafe.Pointer(curaddr.netmask)));
err != nil {
// If we got an IP address but we can't get a
netmask, just return the IP
// address.
On Wed, May 6, 2015 at 10:41 AM, Sergei Mamonov<EMAIL_ADDRESS>wrote:

Sorry, but I not see patch.
—
Reply to this email directly or view it on GitHub
https://github.com/google/gopacket/issues/77#issuecomment-99532712.

It work now!
sudo ./find_pcap_devices
venet0
any
lo
Thanks a lot!

https://github.com/google/gopacket/pull/78
Please take a look at that pull and comment on it if it looks like it will
work for you.
Thanks for the report!
--Graeme
On Wed, May 6, 2015 at 10:58 AM, Sergei Mamonov<EMAIL_ADDRESS>wrote:

It work now!
sudo ./find_pcap_devices
venet0
any
lo
Thanks a lot!
—
Reply to this email directly or view it on GitHub
https://github.com/google/gopacket/issues/77#issuecomment-99537485.

I believe this is now fixed.


Update to indexByObj functionality

The existing indexByObj functionality depends on the image/photo object to have a unique id field.  This is not documented in the ReadMe but this change will allow the function to work without the "id" field.

The ID field is generated by the directive if there is none as you can see here. That's why it's not mentioned in the README. But you're right if an id field is there, its expected to be unique across given images. Which is a sensible assumption IMHO.

@abuechler, made the requested changes and updated PR.


Windows Project (win+p) alternative

Description of the new feature / enhancement
Win+P has been around since Windows XP. An in all these years it hasn't changed, while multiple monitor setups have became more mainstream. For example, I have 3 displays connected to my PC and TV (through receiver), all with different capabilities (VRR, HDR, 144Hz, 120Hz,...)
We need a utility that will let you create profiles for different situations, so you can set up system for work / play / multimedia with 1 click. For example:
Profile 1 (gaming): only main display (42 inch oled tv) enabled with HDR and VRR, but resolution lowered from 4k to 3840x1600 for widescreen gaming, everything else off.
Profile 2 (work) main display (HDR off) 4k resolution enabled, 2x side displays enabled, TV off
Profile 3 (TV) only TV on (65 inch oled), if possible enable Dolby Atmos or any other spacial sound.
Ideally profile could be selected with single click or similar to current functionality (ctrl+alt+p) to switch between profiles.
I have given up on MS updating Windows Project to modern times.
Scenario when this would be used?
Any system with multiple monitors/TVs that needs fine control of display resolution/features/... and multiple different profiles.
Supporting information
No response

/dup #1052


Added  Parkinson Disease Detection

Description
added Parkinson Disease Detection using machine learning approach
Fixes #768
Type of change

[ ] Added a new machine learning frameworks, libraries or software.
[ y ] Documentation update

Checklist:

[ X] My code follows the style guidelines of this project
[ X] I have performed a self-review of my own code
[ X] I have commented my code, particularly in hard-to-understand areas
[ X] I have made corresponding changes to the documentation
[ X] My changes generate no new warnings

you have reached limit on this repo and code harbour 1 label will be removed form the codeharbour to normalise


Renault can be removed from the warning list I guess

https://en.media.renaultgroup.com/news/renault-industrial-activities-in-russia-are-suspended-9c81-989c5.html

done


can addCollaborator but not removeCollaborator

https://github.com/webaverse/contracts/blob/99b56bde896d98aaa8d63e077678dc486133a857/ethereum/WebaverseERC721.sol#L217
addCollaborator works, but removeCollaborator returns

execution reverted: you are not a collaborator

They both check for require(isCollaborator(hash, msg.sender), "you are not a collaborator");

Should be fixed now.


How to change font of editor?

To make it easier for us to help you, please include as much useful information as possible.
Version
Latest
Additional context
I'm asking about the font style of toolbar not the editing area

Change it using "css".
For example:
toolbar:
.sun-editor .se-toolbar {
}
tooltip:
.sun-editor .se-tooltip .se-tooltip-inner .se-tooltip-text {
font-family:  ..
}


Get-Device does not implement service_id or d2c_only

/client/{client_id}/device API call supports service_id and d2c_only parameters.  Not reflected in Get-Device.

The July 2024 API release added a /device endpoint which as of this comment does not include these properties. Unclear if they will remain or be expanded. Pending this issue during the development cycle.


Material property

Fix a bug where the local operators always used a diffusion coefficient of 1
Make discontinuous problem harder to solve

Codecov Report

Merging #73 into master will decrease coverage by 0.31%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master      #73      +/-   ##
==========================================
- Coverage   88.73%   88.41%   -0.32%
==========================================
  Files          25       25
  Lines        1340     1347       +7
==========================================
+ Hits         1189     1191       +2
- Misses        151      156       +5

Impacted Files
Coverage Δ

tests/test_hierarchy_helpers.hpp
60% <33.33%> (-3.02%)
: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 698ad6c...91499ff. Read the comment docs.

ok done


docs: improve the layout of mobile screen

PR Checklist

[x] Fix linting errors
[x] Label has been added

Change information
resolved #6

Maybe remove the shadow from that menu unless you've scrolled? Just a suggestion

Maybe remove the shadow from that menu unless you've scrolled? Just a suggestion

I also perfer the style without shadow, I will improve it. 🤒


[FancyZones] Set correct corner preference when snapping windows on Windows 11

Summary of the Pull Request
What is this about:
When snapping a window on Windows 11 the app retains the rounded corners. This does not happen when using native Windows 11 snap.
What is included in the PR:
Working example where apps get square borders when snapped and rounded when un-snapped.
How does someone test / validate:

Run Powertoys on Windows 11
Open Notepad (notice rounded corners)
Snap Notepad to a zone
Notepad should now have square corners
Unsnap Notepad
Notepad should now have rounded corners again

Quality Checklist

[x] Linked issue: #11991, #12787
[x] Communication: I've discussed this with core contributors in the issue.
[ ] Binaries: Any new files are added to WXS / YML

[x] No new binaries

Hi @SeraphimaZ
I took a swing at seeing if I could recreate the corner preference behavior native snap has on Windows 11.
The code works! 🎉 But it's probably not up to standards and I might have missed better places to put it.
Feel free to alter this PR or use it as inspiration. I'm not a C++ developer, so my skills here are fairly limited 😅

@hellatore I added the in progress labels to the issues.

@SeraphimaZ Done! :)

Ready for merge 🥳🎉

Ready for merge 🥳🎉

We'll wait with merging until the hotfix is out (probably next monday). Thank you for this awesome PR

Thanks a lot for the contribution!


Shader materials are not imported by the headless build

Godot version:
3.2.1 headless build
OS/device including version:
Debian 10. Probably every Linux/BSD
Issue description:
Attempted to export the project in the CI/CD. We don't have the .import directory pushed to VC, as this is optional according to the docs.
Therefore before the game can be exported, some files need to be imported. This works fine for most files, but shader materials in escn files are ignored.
For comparison, I've included both files in a zip: (Opening in a normal text editor is enough to see that the headless import lacks the shader)
models.zip
It seems like shader materials are just stripped from the imported file in the headless build, as the same model works with a spatial material.
Steps to reproduce:
Extract the attached sample project. Export the project with godot --export "Test" test and run it with "./test". The material should be correctly loaded.
Now, rm -r .import/ to force a reimport. Export the project with godot_headless --export "Test" test_headless and run it. The material should be missing.
Minimal reproduction project:
headless_shader_import.zip
Edit: fixed project zip

This happened to me, it's weird because it worked on 3.4 and stopped working on 3.5


Added node run list UI and cypress test cases.

Signed-off-by: Himanshi Chhabra<EMAIL_ADDRESS>:nut_and_bolt: Description: What code changed, and why?

Add Edit/Update node run list  UI to update the node run list.
Add cypress test case for the run list table, edit the run list, change environment, and change tab.
Add Edit/Update node run list  UI on node list dropdown page.

:chains: Related Resources
#4919
:+1: Definition of Done

Added UI for run list tab.
Added UI for edit run list.
Added cypress test cases for edit.
Added dropdown for edit run list on node list page.

:athletic_shoe: How to Build and Test the Change
PFA Video
STEP 1
inside the hab studio

[default:/src:0]# build components/automate-ui-devproxy/
[default:/src:0]# start_automate_ui_background
[default:/src:0]# start_all_services

STEP 2
open new window
go to automate UI path

$ cd components/automate-ui
and run the command

npm run serve:hab

To add data https://github.com/chef/automate/blob/master/dev-docs/adding-data/adding_test_data.md#adding-data-to-infra-views
Steps for runlist tab:

Go To  Infrastructure -> Chef Servers -> click server name-> click on org name -> Nodes.
Click on particular node -> run list tab -> will show list of run list if particular node has it -> Edit button -> to edit the runlist and update.

Steps node dropdown for edit runlist:

Go To  Infrastructure -> Chef Servers -> click server name-> click on org name -> Nodes.
Click on  ... on particular node -> click Edit run list -> to edit the runlist and update.

For cypress Build
https://github.com/chef/automate/tree/master/e2e
:white_check_mark: Checklist
All PRs must tick these:

[x] I have read the CONTRIBUTING document.
[x] All commits signed-off for the Developer Certification of Origin.

With occasional exceptions, all PRs from Progress employees must tick these:

[x] Is the code clear? (complicated code or lots of comments--subdivide and use well-named methods, meaningful variable names, etc.)
[x] Consistency checked? (user notifications, user prompts, visual patterns, code patterns, variable names)
[x] Repeated code blocks eliminated? (adapt and reuse existing components, blocks, functions, etc.)
[x] Spelling, grammar, typos checked? (at a minimum use make spell in any component directory)
[x] Code well-formatted? (indents, line breaks, etc. improve rather than hinder readability)

All PRs from Progress employees should tick these if appropriate:

[x] Tests added/updated? (all new code needs new tests)
[ ] Docs added/updated? (all customer-facing changes)

Please add a note next to any checkbox above if you are NOT ticking it.
:camera: Screenshots, if applicable

Node run list tab:
https://user-images.githubusercontent.com/61003053/119030439-82dbb780-b9c7-11eb-98e7-a7ba692741de.mp4

Edit run list on the dropdown of node list page:

https://user-images.githubusercontent.com/61003053/119534363-6fa06180-bda4-11eb-9964-09203b85a0bb.mp4

If the node has an environment which shows error:

Hi Himanshi - Do I need to rebuild anything to be able to get some run lists?  I get the "Cannot Load environment" modal on all of the nodes.

Hi Himanshi - Do I need to rebuild anything to be able to get some run lists? I get the "Cannot Load environment" modal on all of the nodes.

@SEAjamieD I have checked on my local it's showing properly at my end.

Hi Himanshi - Do I need to rebuild anything to be able to get some run lists? I get the "Cannot Load environment" modal on all of the nodes.

Hi @SEAjamieD, No we don't have to rebuild anything as we have also tested this on Vinay's machine and we fetch the branch and start the npm and it's showing a run list.
Could you share a snapshot or video so that we can check if UI is properly loaded or not?

'+' (Plus icon position is not align with below icons)

Hi @himanshi-chhabra  some observations :

When we clicked on Expand All and Collapse All, then table columns changes it's position towards left and right.
Details tab top and left padding space not matching with Run list tab.


[Bug] Waypoint removal with the right click/spacebar when no waypoints have been established

This is potentially a non-game breaking bug with the drag ruler, as the solution is generally easy and with no lasting effects. The behavior apparently arises when one hits the 'delete waypoint' key when no waypoints have been established, resulting in a 'ghost' token in your current location and the actual token remaining in the original space. Only the original space's token can be moved.
Steps to reproduce:

Have a token in the canvas and start dragging it using the Drag Ruler module.
Attempt to delete a waypoint using right-click or spacebar while dragging it.
Notice how a duplicate token graphic and a ghostly token remain in the canvas.
To clear the bug, move the token from the original position to any other place.

I believe this could be circumvented by 'gating' the 'remove waypoint' function behind a check for established waypoints. If no waypoints have been placed, either cancel the dragging, or make the downkey capture have no effect.

I cannot manage to reproduce the issue. Do you use a browser (if so, which?) or the electron Application? Do you have other Modules installed? If so, does the problem still occur if you disable all other modules except Drag Ruler?

Upon checking the console, it seems that the bug itself comes from the TerrainLayer module and not this one, as it attempts to trigger an event function which does not exist.
10:32:22.381 Uncaught TypeError: event.stopPropagation is not a function
    _onDragLeftCancel http://<IP_ADDRESS>:35411/modules/TerrainLayer/terrain-main.js:44
    deleteWaypoint http://<IP_ADDRESS>:35411/modules/drag-ruler/src/main.js:237
    onRulerMoveToken http://<IP_ADDRESS>:35411/modules/drag-ruler/src/main.js:205
    moveToken http://<IP_ADDRESS>:35411/modules/drag-ruler/src/main.js:72
    _onSpace http://<IP_ADDRESS>:35411/scripts/foundry.js:2949
    _handleKeys http://<IP_ADDRESS>:35411/scripts/foundry.js:2805
    _handleKeys http://<IP_ADDRESS>:35411/modules/drag-ruler/src/main.js:110
    _handleKeys http://<IP_ADDRESS>:35411/modules/rangefinder/src/main.js:22
    _onKeyDown http://<IP_ADDRESS>:35411/scripts/foundry.js:2761
terrain-main.js:44:9

You may close this as that's something that needs to be addressed by another author.

Judging by the log you posted this is a incompatibility between Drag Ruler and Terrain Layer. So it's neither fully the fault of Drag Ruler nor Terrain Layer and the fix might actually require me to change code in Drag Ruler. I'm keeping this issue open until that incompatibility is resolved.
Are you using the original TerrainLayer module or the fork by ironmonk88?

I'm actually using the fork provided by ironmonk88.

This module conflict has been solved by the other module in the meantime.


test

test issue

test comment

testing locally

testing locally

Stats & Info for DataCap Allocation
Multisig Notary address

f01858410

Client address

f1bstbq5bi72kyovhh7zoo2f6l6uivsjz4ey5dnqq

Last two approvers

kernelogic & cryptowhizzard

Rule to calculate the allocation request amount

400% of weekly dc amount requested

DataCap allocation requested

400TiB

Total DataCap granted for client so far

350TiB

Datacap to be granted to reach the total amount requested by the client (5 PiB)

4.65PiB

Stats

Number of deals
Number of storage providers
Previous DC Allocated
Top provider
Remaining DC

6575
26
200TiB
18.26
42.28TiB

Originally posted by @large-datacap-requests[bot] in https://github.com/filecoin-project/filecoin-plus-large-datasets/issues/917#issuecomment-1315037140


Enhancement: Configurable Title Color for Markdown Popup Windows

Description:
Addresses issue #2: Customizable Colors for Markdown Popup Titles.
Changes Made:

Added a configurable option to set the color of the title in Markdown popup windows.
Implemented the highlight group MarkdownHeadersTitle, which will allow users to change the color of the title interactively. Example: highlight MarkdownHeadersTitle guifg=<color>.
Implemented the highlight group MarkdownHeadersBorder, which will allow users to change the color of the border interactively. Example: highlight MarkdownHeadersBorder guifg=<color>.

TODO:

[X] Implement a way to change the color of the title. (ac1cd9e)
[X] Implement a way to change the color of the border. (a2975fa)
[X] Update documentation about plugin customization and configuration. (e931295 and b12688f)

Screenshots:

Title
Border

It was already possible to change the guifg and guibg of the text within the window. The highlight group: MarkdownHeadersWindow, is responsible for this. For example, highlight MarkdownHeadersWindow guifg=#61afef should change the text color to blue.

I think this PR is ready to merge! But I'll wait another day or so, just in case I think of something related to this.

I think this PR is ready to merge! But I'll wait another day or so, just in case I think of something related to this.


GEM: An error in the filename does not set exit status correctly

A justprep include file missing message does not set the exit status correctly.  It should be set to 1 so that the follow-on execution of just is abored.  This is true in the RubyGem version.  Check the Crystal version to verify exit status is set correctly there.

fixed it


Add support for Child Mappings

This handles the following scenario from the Knockout Mapping tutorial:
var mapping = {
    'children': {
        key: function(data) {
            return ko.utils.unwrapObservable(data.id);
        }
    }
}

With the current version, this mapping cannot be compiled under Typescript 1.6.  My change allows this to be compiled.  I've updated the test to force a type in order to demonstrate the compile error if the change is removed.
I'm aware that the indexed type is not properly typed (using any), but I'm unsure if this is possible with TS or not.

knockout.mapping/knockout.mapping.d.ts
to author(@borisyankov). could you review this PR?
:+1: or :-1:?
check list

[ ] pass the Travic-CI test?

Hello and thank you for your contribution!
Due to an excessively long queue of pull requests that have become stale, we are "declaring bankruptcy" and closing all PRs opened before May 1, 2016. If you'd still like to merge this code in, please open a new PR that has been merged and rebased with the master branch.
Going forward, we are committing to review or merge all PRs on a regular basis so this bankruptcy will not occur again. We apologize for the incovenience and hope you will continue to contribute to DefinitelyTyped in the future.
Thanks
The TypeScript and DefinitelyTyped teams


Gradle build: cound not find com.github.johnrengelman.shadow 2.0.4

./gradlew
Downloading https://services.gradle.org/distributions/gradle-4.9-all.zip
............................................................................................................
Welcome to Gradle 4.9!
Here are the highlights of this release:

Experimental APIs for creating and configuring tasks lazily
Pass arguments to JavaExec via CLI
Auxiliary publication dependency support for multi-project builds
Improved dependency insight report

For more details see https://docs.gradle.org/4.9/release-notes.html
Starting a Gradle Daemon (subsequent builds will be faster)
FAILURE: Build failed with an exception.

What went wrong:
Error resolving plugin [id: 'com.github.johnrengelman.shadow', version: '2.0.4', apply: false]

Could not resolve all dependencies for configuration 'detachedConfiguration1'.
Could not determine artifacts for com.github.johnrengelman.shadow:com.github.johnrengelman.shadow.gradle.plugin:2.0.4
Could not get resource 'https://plugins.gradle.org/m2/com/github/johnrengelman/shadow/com.github.johnrengelman.shadow.gradle.plugin/2.0.4/com.github.johnrengelman.shadow.gradle.plugin-2.0.4.jar'.
Could not HEAD 'https://plugins.gradle.org/m2/com/github/johnrengelman/shadow/com.github.johnrengelman.shadow.gradle.plugin/2.0.4/com.github.johnrengelman.shadow.gradle.plugin-2.0.4.jar'.
Read timed out

Try running the build again. Sometimes things fail to fetch from maven central.

Thanks. Worked 3rd time.

Seems that sometimes plugins.gradle.org is not available. Once cached locally then it tends to be OK.


Integrate fancy plugin to make nicer town_or_city and country selectors

@sebgrebe @Manoj-nathwani prioritizing after the launch, though feel free to adjust as you feel is appropriate


codelens missing frequently

Issue Type: Bug
Codelens are missing (hidden) when splitting the editors or scrolling the editor up and down. The view zone is placed in the editor correctly but the commands are missing

VS Code version: Code - Insiders 1.44.0-insider (efd7548df65de8e775015b67863190e6bd4934fc, 2020-03-13T06:01:51.669Z)
OS version: Darwin x64 19.2.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i9-9980HK CPU @ 2.40GHz (16 x 2400)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 2

Memory (System)
32.00GB (0.19GB free)

Process Argv
-psn_0_41727945

Screen Reader
no

VM
0%

Extensions (14)

Extension
Author (truncated)
Version

comment-tagged-templates
bie
0.3.0

github-markdown-preview
bie
0.0.2

markdown-emoji
bie
0.0.9

markdown-preview-github-styles
bie
0.1.6

gitlens
eam
10.2.1

vscode-pull-request-github-insiders
Git
2020.3.18182

vscode-language-pack-zh-hans
MS-
1.43.7

remote-ssh-edit-nightly
ms-
2020.3.13860

remote-ssh-nightly
ms-
2020.3.13860

js-debug-nightly
ms-
2020.2.2917

debugger-for-chrome
msj
4.12.6

kuskus-extensions-pack
ros
1.0.1

kuskus-kusto-language-server
ros
1.0.1

kuskus-kusto-syntax-highlighting
ros
1.1.8

(4 theme extensions excluded)

Can you inspect the dom when this happens? There should be a content widget and view zone, looks like one is missing

Actually they are both there but the content is empty.

Can you check the TSServer log? This looks like a case in which we receive the (empty) code lenses but fail (or are super slow) to resolve them, e.g. only the resolving puts the text here... Adding @mjbvz to check if you have seen something like this.
I haven't noticed this but my code lens selfhosting is with the complexity metrics extensions which hints towards an extension issue

We treat resolving the code lenses as a low priority operation, so other actions such as suggest can interrupt it. Also, resolve can take a little time in large projects like VS Code
@rebornix Do the code lenses ever appear if you let the editor sit there for a moment?

@mjbvz it doesn't, unless I make changes to the content again. Sometimes they come back just after a long period, but sometimes they never come back.
It will be helpful if one of you can tell me how to troubleshoot when the codelens are off positioned

tslog which might match what @mjbvz described, references is cancelled.
Info 24023[16:21:43.941] response:
    {"seq":0,"type":"response","command":"references","request_seq":7655,"success":true,"body":{"refs":[{"file":"/Users/penlv/code/vscode/src/vs/workbench/api/browser/mainThreadNotebook.ts","start":{"line":43,"offset":20},"end":{"line":43,"offset":26},"contextStart":{"line":43,"offset":3},"contextEnd":{"line":43,"offset":48},"lineText":"\t\tprivate readonly _proxy: ExtHostNotebookShape,","isWriteAccess":true,"isDefinition":true},{"file":"/Users/penlv/code/vscode/src/vs/workbench/api/browser/mainThreadNotebook.ts","start":{"line":75,"offset":38},"end":{"line":75,"offset":44},"lineText":"\t\tlet deleteExtHostCell = await this._proxy.$deleteCell(this.viewType, uri, index);","isWriteAccess":false,"isDefinition":false},{"file":"/Users/penlv/code/vscode/src/vs/workbench/api/browser/mainThreadNotebook.ts","start":{"line":85,"offset":24},"end":{"line":85,"offset":30},"lineText":"\t\tlet ret = await this._proxy.$saveNotebook(this.viewType, this.uri);","isWriteAccess":false,"isDefinition":false}],"symbolName":"_proxy","symbolStartOffset":20,"symbolDisplayString":"(property) MainThreadNotebookDocument._proxy: ExtHostNotebookShape"}}
Info 24024[16:21:43.941] event:
    {"seq":0,"type":"event","event":"syntaxDiag","body":{"file":"/Users/penlv/code/vscode/src/vs/workbench/contrib/notebook/browser/notebookEditorInput.ts","diagnostics":[]}}
Info 24025[16:21:43.951] event:
    {"seq":0,"type":"event","event":"semanticDiag","body":{"file":"/Users/penlv/code/vscode/src/vs/workbench/contrib/notebook/browser/notebookEditorInput.ts","diagnostics":[]}}
Info 24026[16:21:43.951] request:
    {"seq":7656,"type":"request","command":"references","arguments":{"file":"/Users/penlv/code/vscode/src/vs/workbench/api/browser/mainThreadNotebook.ts","line":44,"offset":10}}
Info 24027[16:21:44.93] response:
    {"seq":0,"type":"response","command":"references","request_seq":7656,"success":true,"body":{"canceled":true}}
Info 24028[16:21:44.93] event:
    {"seq":0,"type":"event","event":"suggestionDiag","body":{"file":"/Users/penlv/code/vscode/src/vs/workbench/contrib/notebook/browser/notebookEditorInput.ts","diagnostics":[{"start":{"line":115,"offset":13},"end":{"line":115,"offset":18},"text":"'group' is declared but its value is never read.","code":6133,"category":"suggestion","reportsUnnecessary":true},{"start":{"line":115,"offset":37},"end":{"line":115,"offset":44},"text":"'options' is declared but its value is never read.","code":6133,"category":"suggestion","reportsUnnecessary":true}]}}
Info 24029[16:21:44.94] event:
    {"seq":0,"type":"event","event":"requestCompleted","body":{"request_seq":7644}}

I ran into this issue less often than before, a couple of times in last few weeks. The codelens come into view slower than before, not sure if we have any update in TS side @mjbvz .
Occasionally I saw empty code lens and after debugging into this a bit, I found that the lens we received have empty title and command, which will end up with an empty span
https://github.com/microsoft/vscode/blob/16a06751fe9da1a82c7791674e74db87ebd9dd10/src/vs/editor/contrib/codelens/codelensWidget.ts#L86-L103

Nothing has changed on the TS side within the last month or so.
We compute the references code lens with low priority requests against TS server. This means any user action will go ahead of these request. I wonder if semantic coloring is now causing these requests to be more delayed than they were previously? @rebornix Can you try disabling semantic highlighting just to check this

I haven't seen this in a while? Did anyone else? A while back I did merge a PR that fixed some css issue which also could have caused this.


Testing coverage

Hi, I've been using your task to build and run the unit tests of my .NET Framework 4.7 application.
Everything runs smoothly, also the code coverage.
What I do is passing this parameter to the task:
/collect:"Code Coverage;Format=Cobertura"
Now I am in the process of migrating the application to .NET7 and while everything works fine for the building part of the pipeline, I cannot get the coverage anymore. My test projects reference both Microsoft.CodeCoverage and coverlet.collector but the parameter is no longer recognized, and I get the following error:
Unable to find a data collector with friendly name 'Code Coverage'
I already did some unsuccessful tests, but before going on I wanted to check if there is any incompatibility with your task and the coverage on .NET (not Framework) projects.
Thank you.

This is not really 'my task', but a standard task from Microsoft to which i have only changed the same so I can publish it.
It should be able to do everything the normal task can. But you may need to upgrade your agent to a more recent version.

Thanks a lot for the quick reply, I'll give it a try with an update!

I'm guessing you may want to configure the "xplat code coverage"  provider for net 7.

I tried that indeed, but I get the same message
Unable to find a data collector with friendly name 'XPlat Code Coverage'

Does it work when you try this on Azure DevOps with a similar configuration? I feel it must be an issue with the underlying task, nog my liberated version of it.


fix: Fix flaky FilesTest

What was done?

use fuel test runner
use TemporaryFolder

Test Plan

How do we know the code works?

All workflows should finish without any error (especially ubuntu-workflow)

@pawelpasterz lets remove this test completely as we discussed.

@pawelpasterz lets remove this test completely as we discussed.


Docker build throwing tar extract error

[+] Building 5.9s (14/17)
 => [internal] load build definition from Dockerfile.cpu                                                                            0.0s
 => => transferring dockerfile: 1.02kB                                                                                0.0s
 => [internal] load .dockerignore                                                                                0.0s
 => => transferring context: 2B                                                                                0.0s
 => [internal] load metadata for docker.io/library/ubuntu:20.04                                                                     5.3s
 => [internal] load build context                                                                                0.0s
 => => transferring context: 7.20kB                                                                                0.0s
 => [ 1/13] FROM docker.io/library/ubuntu:20.04@sha256:bea6d19168bbfd6af8d77c2cc3c572114eb5d113e6f422573c93cb605a0e2ffb             0.0s
 => CACHED [ 2/13] RUN apt-get update -qq     && apt-get install -y --no-install-recommends       wget       build-essential        0.0s
 => CACHED [ 3/13] RUN wget --no-check-certificate -O miniconda.sh https://repo.anaconda.com/miniconda/Miniconda3-py37_4.8.3-Linux  0.0s
 => CACHED [ 4/13] WORKDIR /home                                                                                0.0s
 => CACHED [ 5/13] COPY download_model.sh download_model.sh                                                                         0.0s
 => CACHED [ 6/13] RUN ./download_model.sh                                                                                0.0s
 => CACHED [ 7/13] RUN conda install pytorch==1.4.0 torchvision==0.5.0 cpuonly -c pytorch                                           0.0s
 => CACHED [ 8/13] COPY requirements.txt requirements.txt                                                                           0.0s
 => CACHED [ 9/13] RUN pip install -r requirements.txt                                                                              0.0s
 => ERROR [10/13] RUN tar -xvf m39v1.tar && rm -f m39v1.tar                                                                         0.5s
------
 > [10/13] RUN tar -xvf m39v1.tar && rm -f m39v1.tar:
#14 0.491 tar: This does not look like a tar archive
#14 0.491 tar: Skipping to next header
#14 0.491 tar: Exiting with failure status due to previous errors
------
executor failed running [/bin/sh -c tar -xvf m39v1.tar && rm -f m39v1.tar]: exit code: 2

Thanks for the issue, @dakshvar22 will get back to you about it soon!
You may find help in the docs and the forum, too 🤗

Hi! Can you check if m39v1.tar was downloaded successfully? Looks like that step didn't execute properly. You could also delete the docker cache and try building the image from scratch.
Out of curiosity: Why do you want to build the docker image? Can't you just reuse the docker image that's hosted online and run it?

Hi @dakshvar22  I saw m39v1.tar was not in the repo and it had added into gitignore. Where to download the file?

Out of curiosity: Why do you want to build the docker image? Can't you just reuse the docker image that's hosted online and run it?

I guess he want to add more feature?

@fifiteen82726 m39v1.tar is downloaded by COPY download_model.sh download_model.sh
RUN ./download_model.sh. That's why it has been ignored. During building docker image, the downloading was failed. Theafore it was not a tar file. To work around this download the file manually, in Dockerfile, comment out COPY download_model.sh download_model.sh RUN ./download_model.sh, add  COPY m39v1.tar ./ instead.
PS: Refer download_model.sh for download link


Implement source maps for computed properties

var v = {
    ["hello"]() {
        debugger;
    }
}

This is from tests\cases\conformance\es6\computedProperties\computedPropertyNamesSourceMap2_ES5.ts.
Right now we have unmapped spans in the emitted source map, so the debugger will jump into the .js file every time it encounters an unmapped span.

@JsonFreeman it looks like things are currently being generated correctly. Let's figure out whether this is still an issue when you return.

The final _a still looks unmapped in the picture you gave.

Ah I see what you'e saying. Let me get on that.

Should be working correctly in latest.


Sauce Connect is broken

I've setup the gem. I have one spec file with two tests running on Windows 8. When I run rake sauce:spec I see 4 jobs on sauce labs. Two jobs are on windows with the correct name, pass status, and no commands, it's an empty selenium session. The other two jobs are unamed job running on linux , no pass status, and those contain the selenium commands.
Unless I missed something obvious in the config, it seems like the sauce connect feature is broken.

I've created an open source example which reproduces the issue.

In debugging the code it's apparent that watir-webdriver isn't supported. It looks like Sauce::Selenium2 is hard coded to be a Selenium::WebDriver and not Watir::Browser which prevents using the page-object gem in watir mode.


Slows when large amount of data

I have tried with following options but still performance is low with 3000 to 4000 rows including child nodes attached screenshot error occurs
$('.tree').treegrid({
initialState: 'collapsed', gridview: true, loadonce: false,
rownumbers: false,
autoRowHeight: false,
nowrap: true,
frozenColumns: null,
fit: true });

please refer to #11


Bump black to 22.6.0 in pre-commit config

Should fix issue in black

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
170

Relevant Lines:
181

💛  - Coveralls


[simple-romp, README] Using as an module in 'video' mode

According to the README i need to use the following code for using it as an module in 'image' mode:
#!/usr/bin/env python3

import romp

settings = romp.main.default_settings
romp_model = romp.ROMP(settings)
outputs = romp_model(cv2.imread('path/to/image.jpg'))

But if iam using it in the 'video' mode, i need to do the following:
#!/usr/bin/env python3

import romp

settings = romp.main.default_settings
settings.mode = "video"
romp_model = romp.ROMP(settings)
...
outputs = romp_model(frame_image)

According to here: https://github.com/Arthur151/ROMP/blob/4a5fecd140a3c54866c3f7e040bf0ac19d810190/simple_romp/romp/main.py#L189
My question:
If ROMP get's in both cases ('video' and 'image' mode) only an single image as input, then why i need to change the mode from image to video?

@Arthur151 Not yet tested but if i want to use romp as an module in 'webcam' mode and using it's outputs in my script:
How do i do that?

Hi, Vivian
Yes, I think you have find the correct answer.
Setting the mode to video would allow users to use temporal optimization functions.
To use ROMP in 'webcam' mode without rendering, just for estimating SMPL parameters from webcam frame sequence, you can set up like this.
    settings = argparse.Namespace(GPU=gpu, center_thresh=0.2, input=None, mode='webcam', onnx=False,
                                  model_path=os.path.join(model_path, 'ROMP.pkl'), smpl_path=os.path.join(model_path, 'smpl_packed_info.pth'),
                                  smooth_coeff=3.0, temporal_optimize=True,
                                  show_largest=False, calc_smpl=False, render_mesh=False, show=False, show_items='mesh,tracking', renderer='sim3dr')
    romp = ROMP(settings)


Make highlight & selection handling more dynamic

This change makes the way components are highlighted & selected a lot more dynamic. Before, only components implemented using either UITableViewCell or UICollectionViewCell would be able to be have highlight & selection appearances (because we simply forwarded those events from the wrapper cell).
With this change, a HUBComponentWithSelectionState protocol is introduced, making any component able to respond to selection state changes. Selection & highlighting is now also done through a custom gesture recognizer, instead of attempting to hack UIKit’s touch handling code (turned out to be a mistake).
The end goal of this change is to:

Enable any component to be highlighted & selected.
Simplify the implementation of selection handling.
Enable components to have custom gesture recognizers.

@spotify/objc-dev

@Goos Addressed your comments. Want to have a second look? 🙂

Current coverage is 93.09% (diff: 53.27%)

Merging #80 into master will decrease coverage by 0.86%

@@             master        #80   diff @@
==========================================
  Files            57         57
  Lines          3875       3869     -6
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           3641       3602    -39
- Misses          234        267    +33
  Partials          0          0

Powered by Codecov. Last update d0ace1d...ed2caba

@Goos Fixed some bugs and added backwards compatibility with UITableViewCell and UICollectionViewCell. Mind taking another look just to make sure things still look good? 😁

Turns out @Goos is on vacation, so can anyone else from @spotify/objc-dev have a look please? 😄

👍

@krris Can't respond to your comment with your question (since GitHub has marked it as "outdated"). But the 0.2 value comes from experimenting and measuring how UIKit delays its highlighting & selection mechanisms, and finding a value that didn't make the UI feel slow, but yet avoided flashing of accidental highlights.


Port bindings in container list

Will be useful to show port bindings in container list, with a link, to open external port in browser.
When I run containers that exposes ports (most of the times is that way) often I have to look around to get the external port that docker has attached the containers exposed ports,  and put it in the browser to point to it. I think will be very useful to get a link in ui-for-docker, in container list, for easily access that mappings.  What do you think?

That's a pretty common use-case for me as well.

@alambike this is available in portainer which is an alternative to UIFD, please have a loot at https://github.com/portainer/portainer


Horrendously makes PowerDevice connect to the closest Provider

fixes #297
Don't merge this

Do we need a PowerConsumerComponent, just like we have a PowerProviderComponent?
Should there be a minimum distance required or let's not bother for now? If so, what should it be?

So this causes a null exception when the server shuts down. I'm not sure why

My understanding of power code is very limited. Maybe @clusterfack wants to take a look?
Side question: Is it correct to use _range like I'm doing with WorldPosition?


Failure detection on 3839 readout.

At some point i posted here about the weird hiccups i had with this sensor, i have now noticed that it will always output 3839 on error.
This can also be confirmed from other posts, like this:
https://forum.arduino.cc/t/analogwrite-causes-i2c-encoder-lockup/674080/13
"Sadly, after a couple of minutes running, the encoder stops communicating, and returns the value 3839, which is the same as you would see if you unplugged it."
So every time the encoder fails to get a reading, it outputs 3839.
For me, it simply occasionally outputs 3839. And this causes these weird hiccups in the linear reading.
It would be good if there was a possibility to filter for this, mainly in my case this is an issue in the linear output. As on other cases i can filter this out easily.
Even though i expect this is a problem on my wiring/power filtering, as im running two of these and the other has 0 of these hiccups. But still, it might be wort while having a possibility of filtering this one readout out.

Will think over it what could be done.

Here the 3839 is mentioned:

https://github.com/RobTillaart/AS5600/issues/6

Yes, it seems to be some sort of default output in case of an error.
In my case, if i stream readings from the sensor.. it simply outputs this randomly amongst the correct readings. So everything works well, but there are just random fails.
I think its either my wiring, or noisy power. Or the fact that i have several I2C sensors connected to this same port.
And the other AS5600 i have in this system, has no other devices connected to the same port. They do all share the same power source.
But i already successfully filtered this reading out in one use case where i'm just reading the raw output, and it worked perfectly. Simply ignoring the reading if its 3839. But as i'm also using the linear output of your library, where i cant filter it out at least not easily. And as i have two of these sensors, the other is more important as its closed loop motor control where i cant use any filtering as it might cause me to miss rotations or something like that.
The other is just a menu/adjustment dial, but this one was hopping around because of the random error reading.

some notes:
3839 is binary 1110 1111 1111 so almost all 1's as if there is no signal
@cranefist
Can you change the AS5600.cpp file and replace the existing readReg2() by this one?
As I do not get the 3839 reproduced, so I want to have a look at the low level I2C.
Need to improve error handling here I guess. That does not solve it right away but allows one to catch it (maybe).
uint16_t AS5600::readReg2(uint8_t reg)
{
  _wire->beginTransmission(_address);
  _wire->write(reg);
  _error = _wire->endTransmission();

  if (_error != 0)
  {
    Serial.print("\nreadReg2 (1): ");
    Serial.println(_error);
  }

  uint8_t n = _wire->requestFrom(_address, (uint8_t)2);
  if (n != 2)
  {
    Serial.print("\nreadReg2 (n): ");
    Serial.println(n);
  }
  uint16_t _data = _wire->read();
  _data <<= 8;
  _data += _wire->read();
  return _data;
}

@cranefist
Created a develop branch + PR with a first level error handling/detection.

https://github.com/RobTillaart/AS5600/tree/develop

With the function int lastError() one can see if the low level I2C functions failed.
The 3839 error might be caused by such failure (assumption).
If you have time, please give the develop branch a try.

I will try it out, i will report back in few days.... my project is quite complex for testing stuff.
I was running the I2C at 400khz, and dropping it to 100khz lowered the number of errors a lot, as i had several devices on this one port. The other AS5600 seems to run just fine at 400khz, on its own I2C line.

OK,

I was running the I2C at 400khz, and dropping it to 100khz lowered the number of errors a lot, as i had several devices on this one port.

Sounds like noise?
Can you see the signal with an oscilloscope?

I will try it out, i will report back in few days.... my project is quite complex for testing stuff.

Any progress?

@cranefist
I merged the develop branch into the master and released version 0.6.0
It adds (experimental) error checking, mainly for I2C, so you can investigate if there is a relation between I2C and 3839 readout.
I could not reproduce it.
For now I close this issue as there are no new insights, feel free to reopen when needed.
Anyway thanks for reporting the issue.


Add control_file to import_brd function as done below. Currently it does not have ability to select onl

Description of the feature
add a contol_file to import_brd:  this would enable selective net import rather than entire board.
Steps for implementing the feature
  def import_brd(
        self, input_file, aedb_path=None, control_file=None, set_as_active=True, close_active_project=False
    ):  # pragma: no cover
        """Import a board file into HFSS 3D Layout and assign the stackup from an XML file if present.

        Parameters
        ----------
        input_file : str
            Full path to the board file.
        aedb_path : str, optional
            Full path to the AEDB file.
        set_as_active : bool, optional
            Whether to set the board file as active. The default is ``True``.
        close_active_project : bool, optional
            Whether to close the active project after loading the board file.
            The default is ''False``.
        Returns
        -------
        bool
            ``True`` when successful, ``False`` when failed.

        References
        ----------

        >>> oModule.ImportExtracta
        """
        return self._import_cad(input_file, "brd", aedb_path, control_file, set_as_active, close_active_project)

Useful links and references
No response

@vishak-venkatraman I'll check if it is supported by anstranslator and if so I'll implement in next released

@maxcapodi78 this is done ? can we close the issue ?


SharePoint native mobile app (iOS / Android) detection & deep linking

Category

[x] Question
[ ] Typo
[ ] Bug
[ ] Additional article idea

Is there a way to detect if the user is using the native mobile app for SharePoint? My SPFx WebPart needs to behave slightly differently in a mobile app than in the browser. Also, is there a way to deep link directly to a user profile in the mobile app?
Thanks

This should be possible with user-agent inspection, but we'll confirm the right model in a day or two if there's additional ways to do this.

hold on there.... I can use spfx webparts in the sharepoint mobile client? i don't need to use powerapps if i want mobile? I had no idea. doubt many others did!
Sent from my iPad

On Mar 15, 2018, at 12:16 PM, Vesa Juvonen<EMAIL_ADDRESS>wrote:
This should be possible with user-agent inspection, but we'll confirm the right model in a day or two if there's additional ways to do this.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@VesaJuvonen i had looked at the user agent already but it seemd a bit hacky.
Mobile App (iOS):
Mozilla/5.0 (iPhone; CPU iPhone OS 11_2_6 like Mac OS X) AppleWebKit/604.5.6 (KHTML, like Gecko) Mobile/15D100

Safari (iOS)
Mozilla/5.0 (iPhone; CPU iPhone OS 11_2_6 like Mac OS X) AppleWebKit/604.5.6 (KHTML, like Gecko) Version/11.0 Mobile/15D100 Safari/604.1

The only difference is the missing Safari part. Haven't tested Android yet.

Hi @VesaJuvonen,
Do you have any information about this case?
Is the user agent the only way to detect that we are on the Sharepoint app?
I tried to watch the ribbon content with document.getElementById('SuiteNavPlaceHolder').innerHTML === "", but when it takes times to load on browser, we find ourselves in the same case that we are on the Sharepoint App or on the browser.
Waiting for your feedback.

Hi,
I'm also using some elements in DOM as workaround but seems a little bit hacky, A JavaScript environment/context variable would be great, I have been trying to find a clean way to identify if user is using SharePoint App without success, I subscribed to the thread, hope this get answered some day.

Hi!
This may be a bit offtopic, however, does anyone knows if besides detecting if user is accessing the site from the Mobile App, is there any way to obtain certain phone context information, as current Wifi ESSID or stuff like that? thanks

Currently navigator.userAgent for SharePoint iOS native app contains `SharePoint for iOS'

Seems that @AJIXuMuK answered on this and looks like a proper way to deal with this. If question is still open, please submit a new issue. Closing this as anwered.


Can't count number off statements with a specific acction in AWS policy documents

Description :
When trying to ensure that a AWS policy (Bucket policy) has at least one statement with a DENY action terraform-compliance will skip the test for non-compliant resources and not shoe any errors
To Reproduce

<Either a sample terraform code, or your terraform plan file if it doesn't have any confidential information>

provider "aws" {
  region  = "eu-west-1"
  profile = "cr-mmerck"
}

resource "aws_s3_bucket" "b" {
  bucket = "my_test_bucket"
}

resource "aws_s3_bucket" "c" {
  bucket = "my_test_bucket_noncompliant"
}

resource "aws_s3_bucket_policy" "b" {
  bucket = "${aws_s3_bucket.b.id}"

  policy = <<POLICY
{
  "Version": "2012-10-17",
  "Id": "Bucket Policy",
  "Statement": [
      {
          "Sid": "Only allow SSL/TLS access",
          "Effect": "Deny",
          "Principal": "*",
          "Action": "s3:*",
          "Resource": "arn:aws:s3:::/*",
          "Condition": {
              "Bool": {
                  "aws:SecureTransport": "false"
              }
          }
      }
  ]
}
POLICY
}

resource "aws_s3_bucket_policy" "c" {
  bucket = "${aws_s3_bucket.c.id}"

  policy = <<POLICY2
{
  "Version": "2012-10-17",
  "Id": "Bucket Policy",
  "Statement": [
      {
          "Sid": "Only allow SSL/TLS access",
          "Effect": "Allow",
          "Principal": "*",
          "Action": "s3:*",
          "Resource": "arn:aws:s3:::/*"
      }
  ]
}
POLICY2
}

terraform-compliance -f features -p out.plan

Scenario: Bucket policy must have at least one deny statement
Given I have aws_s3_bucket_policy defined
When it contains policy
And it contains Statement
SKIPPING: Can not find Deny defined in target terraform plan.
And its Effect is Deny
And I count them
Then its value must be greater and equal to 1

<Your feature/scenario/steps>

Scenario: Bucket policy must have at least one deny statement
    Given I have aws_s3_bucket_policy defined
    When it contains policy
    And it contains Statement
    And its Effect is Deny
    And I count them
    Then its value must be greater and equal to 1

Expected behavior :
The test should fail for policies with only allow statements. As a policy can have ALLOW and DENY statements I can't test if the action is DENY, as then all policies which also have an ALLOW statement would fail
Tested versions :

<terraform-compliance version (terraform-compliance -v)>
1.0.51
<terraform version (terraform -v)>
Terraform v0.12.12
<python runtime version, if running as a python package (python --version)>
Python 3.7.4

Additional context
I would like to test if S3 buckets enforce encryption in transit. There are several issues with this as it needs to b enforced with a bucket policy and it is not possible to test if bucket policies are attached to each bucket. But at least I would like to write a test that if a bucket policy is defined it has the correct condition to enforce encryption in transit.

Hi @merckm-cr,
Your tests are passing because you are using greater and equal in your step. So one Statement is filtered with Effect: Deny and since 1 >= 1  is true, the tests are passing.
Correct tests should use greater than instead of greater and equal to
  Scenario: Bucket policy must have at least one deny statement
    Given I have aws_s3_bucket_policy defined
    When it contains policy
    And it contains Statement
    And its Effect is Deny
    And I count them
    Then its value must be greater than 1

There was also a problem on the error messages which is fixed by #176. It will be released as 1.0.53 soon.

No this is not the issue.
I'm talking only of the second policy which contains a statement with effect ALLOW but no statement with effect DENY. The policy can have multiple statement with different effects, however I do want to enforce that at least one has a deny action.
When running the test I Gert the following output:
Scenario: Bucket policy must have at least one deny statement
    Given I have aws_s3_bucket_policy defined
    When it contains policy
    And it contains Statement
SKIPPING: Can not find Deny defined in target terraform plan.
    And its Effect is Deny
    And I count them
    Then its value must be greater to 1

It does not even come to the count statement as it does not find and statement with action DENY and thinks the scenario is not applicable to the resource. However I need to fail the test if there are now deny statements.
I can't use a condition like " And its Effect is not Deny" as the policy may have several statements and any allow statement would then fail the test. This should however be legal as long as the policy gas also at least one DENY statement.

Very interesting, I am testing the same HCL with 1.0.51 with the same Python and terraform version and the result it just as expected.
    Scenario: Bucket policy must have at least one deny statement
        Given I have aws_s3_bucket_policy defined
        When it contains policy
        And it contains Statement
        And its Effect is Deny
        And I count them
        Then its value must be greater to 1
          Failure: for None on None. 1 is not more than 1.

Can't reproduce the problem. Can you send your plan output please ?

When I removed aws_s3_bucket-policy.b from the main.tf and just test ed with aws_s3_bucket_policy.c in your example, I reproduced the problem.
    Scenario: Bucket policy must have at least one deny statement
        Given I have aws_s3_bucket_policy defined
        When it contains policy
        And it contains Statement
	SKIPPING: Can not find Deny defined in target terraform plan.
        And its Effect is Deny
        And I count them
        Then its value must be greater to 1

An example HCL for the cases above ;
resource "aws_s3_bucket_policy" "b" {
  bucket = "${aws_s3_bucket.b.id}"

  policy = <<POLICY
{
  "Version": "2012-10-17",
  "Id": "Bucket Policy",
  "Statement": [
      {
          "Sid": "Only allow SSL/TLS access",
          "Effect": "Deny",
          "Principal": "*",
          "Action": "s3:*",
          "Resource": "arn:aws:s3:::/*",
          "Condition": {
              "Bool": {
                  "aws:SecureTransport": "false"
              }
          }
      },
      {
          "Sid": "Only allow SSL/TLS access",
          "Effect": "Allow",
          "Principal": "*",
          "Action": "s3:*",
          "Resource": "arn:aws:s3:::/*"
      }
  ]
}
POLICY
}```

Yes, this is exactly the positiv (compliant case) The negative case would be a policy with 2 "Allow" statements an no "Deny" statement.

'''hcl
resource "aws_s3_bucket_policy" "b" {
bucket = "${aws_s3_bucket.b.id}"
policy = <<POLICY
{
"Version": "2012-10-17",
"Id": "Bucket Policy",
"Statement": [
{
"Sid": "Allow S3 access",
"Effect": "Allow",
"Principal": "",
"Action": "s3:",
"Resource": "arn:aws:s3:::/",
},
{
"Sid": "Allow EC2 actions",
"Effect": "Allow",
"Principal": "",
"Action": "ec2:",
"Resource": ""
}
]
}
POLICY
}
'''

Yep, and the result for this HCL is ;
    Scenario: Bucket policy must have at least one deny statement
        Given I have aws_s3_bucket_policy defined
        When it contains policy
        And it contains Statement
        And it contains Effect
        Then its value must contain Deny
          AssertionError: Effect property in aws_s3_bucket_policy did not contain Deny, it is set to ['Allow', 'Allow']

Ok thanks, I think I get it now.

Great :tada: Closing the issue then, please do not hesitate to re-open it if the issue still occurs.


Add dynamic build vars for web's index.html

Locally we run dev server and API on separate hosts. In prod, these are the same and also should be computed at runtime (window.location.*) to account for localhost aliases.
See https://vitejs.dev/guide/env-and-mode#html-env-replacement
(they can also be overridden with an env file with a .local suffix).

LGTM