Use cargo-check in cargo maker

Cargo just got check command merged, which is what we wanted. It's essentially like -Zno-trans, so it should work significantly faster for the syntax checks. I will open the PR changing the maker after nightly version has been released, so that people who use nightly compiler could get it running now. For the actual merge i think that we should wait until this feature hits stable, and it's supposed to be introduced in 1.15, so in approximately 7 weeks. For more information please refer to this PR.

Would love to see this 👍  Is there a PR I can check out, @fuine?

@hpurmann sorry, I haven't got enough time lately. Currently i can see that there is an ongoing WIP PR to use the json output: #942. What's more is that the check command has not been introduced in the 1.15 version, but is currently in beta, so we need to wait 7 weeks for its stabilization. After #942 gets resolved then I will open said PR.

Ok great, looking forward to that. Thank you for your work!

We'll probably want to monitor https://github.com/rust-lang/cargo/issues/3431 for when we'll be able to also check tests. I know for me at least that's a very important feature.

cargo-check is now in rust stable. Tests are still not compiled, though.

I'm going to take a closer look at this issue over the weekend, and hopefully open a new PR.

It looks like there are two things that are relevant to this issue:

should neomake opt into using cargo-check without test support, or should it stick to the current manner of calling cargo test? cargo-check is significantly faster, but on the other hand some people might want to compile their tests as well.
I wouldn't like to create conflicts with #1123/#942. @blueyed @JelteF can you provide a rough estimation of when that PR could possibly land? To be fair the only change would modify one line, but I don't want to cause more troubles, seeing as that PR consists of quite a lot of commits.

With regards to your second point #924 is mergeble now as far as I can tell
and works great. The #1123 PR requires more work, especially rewriting of
the tests.
The best way for this issue to be fixed right now in my opinion is by
giving the user the choice between a cargo full and a cargo check maker.
This can simply be done by copying the cargo maker definition and changing
the command. The only thing I'm not sure about is which one to have as the
default. I'd vote for check as the speed really makes it much nicer.
On Fri, Mar 17, 2017, 00:04 Łukasz Neumann<EMAIL_ADDRESS>wrote:

It looks like there are two things that are relevant to this issue:

should neomake opt into using cargo-check without test support, or
should it stick to the current manner of calling cargo test?
cargo-check is significantly faster, but on the other hand some people
might want to compile their tests as well.
I wouldn't like to create conflicts with #1123
https://github.com/neomake/neomake/pull/1123/#942
https://github.com/neomake/neomake/pull/942. @blueyed
https://github.com/blueyed @JelteF https://github.com/JelteF can
you provide a rough estimation of when that PR could possibly land? To be
fair the only change would modify one line, but I don't want to cause more
troubles, seeing as that PR consists of quite a lot of commits.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/neomake/neomake/issues/843#issuecomment-287218668,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABG8JsZ4UGqX9gSJsTE3WPV301InawXWks5rmb_hgaJpZM4LNRe3
.

So do you think that cargo-check should be opted into in the current makers and you would port that over to the json PRs, or do you want to first land the json support and only after that should we switch to cargo-check?
I would also like to know what does @jonhoo think about cargo-check being the default, as he was the main voice of concern regarding tests support (not that I am against checking tests, but I think that we also should use cargo-check by default, and users which need to see the test results could use the current checker).

I'm very torn. I would strongly prefer moving to cargo check for the speed benefits (after all, this is why I currently use timeout 2s cargo test --no-run as my command), but giving up having tests checked seems unfortunate.

Actually, seems https://github.com/rust-lang/cargo/pull/3731 has landed, so cargo check --all should be available now. I think that's what we should use.

Eeeexcept apparently --all doesn't check tests yet (https://github.com/rust-lang/cargo/issues/3431#issuecomment-280708927) :(
I think my vote is for using cargo check --all, and then we'll eventually get checking of tests as well when that lands. Users like me who care deeply about the tests can keep the old command for now.

Please keep in mind that the --all flag is currently in beta, so we would need to wait for another 6 weeks to use that flag.

@fuine sure. I vote cargo check now, then cargo check --all in six weeks.

#1123 has been merged, so now a pull request can be created to change the command to cargo check and maybe have a second maker for cargo build.

maybe have a second maker for cargo build

Or document how to change/configure the args.

I like the documentation idea, but you should remember that in order for the current rustc/cargo makers to work they need the --message-format=json argument to be in args. Can we ask users to always remember about that argument, or is there any clever trick to hide that argument away from the user that I am not seeing?

@fuine
It would be possible for the maker to have a fn property, which gets used as constructor, and could generate the args, i.e. add --message-format=json there always.
But when documenting it, it could just be included there - in the end it's the same: you either have to lookup and copy'n'paste a settings name to configure this behavior, or define it altogether.  I see that there are several other args though..

'args': ['test', '--no-run', '--message-format=json', '--quiet'],

So if 'test' can be replaced with check or build there, then a setting like g:neomake_rust_cargo_method could be used directly, or otherwise change the args accordingly.
Related: https://github.com/neomake/neomake/pull/1003.

I did :PlugUpdate and now I get:
Neomake: [3.-:-] project maker not found: cargo

My config:
autocmd! BufWritePost,BufEnter *.rs Neomake! cargo

Any ideas?

Maybe just a more descriptive error, that the cargo maker is now the
default rust file maker. So that the regular rule to run neomake on save is
now enough.
On Sat, Apr 8, 2017, 09:58 Daniel Hahler<EMAIL_ADDRESS>wrote:

Remove the !
@JelteF https://github.com/JelteF
Could/should we have some redirection or message for existing users?
We could have the old project maker behave like a file maker?!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/neomake/neomake/issues/843#issuecomment-292702346,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABG8JkSWldjioty0VTgcOE4e6gHdgMP7ks5rtz44gaJpZM4LNRe3
.

That works. But :cc, :cn etc. don't.

Actually... it works for warnings, but not for errors...

@dpc
Sound like you should for a new issue?!

'args': ['test', '--no-run', '--message-format=json', '--quiet'],

So if 'test' can be replaced with check or build there, then a setting like g:neomake_rust_cargo_method could be used directly, or otherwise change the args accordingly.
Related: #1003.

@blueyed
The problem here is that for the test maker we'd want to use test --no-run arguments, whereas for the check maker we should probably settle on the single check (obviously both options should use the --message-format=json --quiet flags).  Please note that with the 1.17 version of Rust (scheduled 27.04) we should get the --all flag for cargo-check, which sadly does not support tests just yet.
To sum it up, this is how two argument arrays should look like for test and check respectively:

'args': ['test', '--no-run', '--message-format=json', '--quiet'],
'args': ['check', '--message-format=json', '--quiet'],

@fuine
Sounds like a config var for the maker makes sense then, e.g. neomake_rust_cargo_mode?!
It should use neomake#GetSetting (or wait for #1156), so that it can be configured on several levels.

I actually have the same issue as @dpc with quick-errors no longer working after this change..

@jonhoo
You mean after #1123?
You commented there already, but this is different?!
Can you provide steps to reproduce it?

@dpc @jonhoo
Location lists are used now.

@blueyed I'm not sure how to use GetSetting in the maker, as I couldn't find any example of its usage outside of neomake.vim. I wrote a working prototype using a simple
let maker_command = get(g:, 'neomake_rust_cargo_command', ['check']), because similar approach is used in the javac maker. Do you want me to open a PR with this approach or would you give me some pointers in here, as I don't know if this is acceptable?

@fuine
Please create a PR. It is fine like that, but should be a string, and not a list probably?!
You could also look at b: additionally.

@blueyed ahh, it's the switch from quickfix to location lists that tripped me up. thanks!

The PR for this issue has been merge. At the moment cargo maker will by default invoke cargo-check command. You can configure the exact command, along with its args via the g:neomake_rust_cargo_command and b:neomake_rust_cargo_command lists (e.g. to check everything, including test modules, put let g:neomake_rust_cargo_command = ['test', '--no-run'] in your .vimrc file).

@fuine we may want to change the default to cargo check --tests now that rust-lang/cargo#3431 has seen some progress.

@jonhoo it looks like this is not supported in the stable release and as such I don't want to push things too rapidly. I think that this PR is what we want in the end, but until it lands in the stable I don't think neomake should change its current behaviour. Please note that you can locally set g:neomake_rust_cargo_command to what you prefer.

Can we switch to using cargo check --tests by now?
(via https://github.com/neomake/neomake/issues/847#issuecomment-366076436)

I think this should really be cargo check --all --all-targets.

Agreed with @jonhoo, that looks like sensible default. Do you want me to open a PR for this?

@fuine
Yes, please.


fix/ci: changing node version to 7.9.0

Coverage remained the same at 83.189% when pulling 8de525d8054312507c06d12c1c9e42d38169340a on safesurfer:ci-fix into fe6afd10b33a228915701ccc4af786cbdc08d0a9 on maidsafe:master.

Coverage remained the same at 83.189% when pulling 8de525d8054312507c06d12c1c9e42d38169340a on safesurfer:ci-fix into fe6afd10b33a228915701ccc4af786cbdc08d0a9 on maidsafe:master.


Use the messagetxt option to make yaya easier to localize

I didn't write LoadMassageFromTxt because I'm not familiar with the file IO with charset
free to change what I write&thanks

I think I forgot @ponapalt .....
草

I'm tired so I just took a rest.
Note : please fix typo in another PR : massage => message

Did you mean this?
https://github.com/Taromati2/yaya-shiori/commit/7097a94ece94640a7b33390f2594ca63f374f105
I don't know exactly what Japanese culture is like in this, but in my place, if you are tired, you just need to say to others, "Hey, I'm a little tired. Maybe I'll go on holiday for a week or play games at home for a few days", and they will understand. So I'm a bit confusing when you don't respond for a few days.
And if you have set SSP as a private git repository, maybe I can help.
Of course, if you don't want to, don't force yourself, I can understand

I've fixed typo.
https://github.com/ponapalt/yaya-shiori/commit/e1d4ad1da794552524b2e7b29a930eebc180cc04
I'm sorry to make you worry. I'll be back to normal after a few more days off.


Revert "Mandatory SIL linker pass"

Reverts apple/swift#15930
PR 15930 was only smoke tested. It's been blocking full PR testing.
SIL verification failed: external declarations of SILFunctions with shared visibility is not allowed: SingleFunction || !hasSharedVisibility(RefF->getLinkage()) || RefF->hasForeignBody()
Verifying instruction:
->   // function_ref specialized FixedWidthInteger.init(truncatingIfNeeded:)
%928 = function_ref @$Ss17FixedWidthIntegerPsE18truncatingIfNeededxqd___tcs06BinaryC0Rd__lufCqd__s6UInt64VXMTA2FRszsADRd__r__lIetMiyd_Tpq5 : $@convention(method) <τ_0_0 where τ_0_0 == UInt64><τ_1_0 where τ_1_0 : BinaryInteger> (@in τ_1_0, @thick UInt64.Type) -> UInt64 // user: %929
%929 = apply %928(%926, %29) : $@convention(method) <τ_0_0 where τ_0_0 == UInt64><τ_1_0 where τ_1_0 : BinaryInteger> (@in τ_1_0, @thick UInt64.Type) -> UInt64 // user: %930
In function:
// globalinit_33_F417D752D2C4E9330E1C700411CE0C6A_func15
sil private @globalinit_33_F417D752D2C4E9330E1C700411CE0C6A_func15 : $@convention(c) () -> ()
Building Dispatch for WatchSimulator.

@swift-ci test.

Build failed
Swift Test OS X Platform
Git Sha - 39de8c7aedae3258a8f2226d60e4f122a40f47c3

@swift-ci smoke test OS X Platform.

@slavapestov I'm reverting multiple PR's to try to get PR testing working again.


Make environment variable searches case-insensitive.

In some cases the environment variable names may have a different case
from what is expected, and envVars.Contains() will fail. So we store it
in a case-insensitive string dictionary and use .ContainsKey() instead.

Ah! I didn't know that was possible... looks good to me though! :)


unknown_version test fails on Android

I'm getting the following error when running the unknown_version test on Android.
[200/288] quiche_device_test_src_lib#tests::unknown_version: FAILED (118ms)
STACKTRACE:
---- tests::unknown_version stderr ----
thread 'main' panicked at 'assertion failed: (left == right)
left: Err(TlsFail),
right: Err(UnknownVersion)', external/rust/crates/quiche/src/lib.rs:4833:9
Any idea why I'm seeing TlsFail instead of UnknownVersion?
My temporary workaround is to ignore the test, but I would like to figure out what's wrong: See: https://android-review.googlesource.com/c/platform/external/rust/crates/quiche/+/1523319/10/src/lib.rs#4825

I think it might be because you are using a newer BoringSSL version, see https://github.com/cloudflare/quiche/pull/826/commits/ecfacccb15ed7f2083001482b7f7dfb5d366ac48#diff-b1a35a68f14e696205874893c07fd24fdb88882b47c23cc0e0c80a30c7d53759R5239

Thank you!


chore: bdd-ghfjxy-import-1583644773 to 0.0.1

chore: Promote bdd-ghfjxy-import-1583644773 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


gon.rabl makes test fail

When I have gon.rabl as: 'sites' in my controller my functional test fails.
It says: NoMethodError: undefined method `controller_path' for nil:NilClass

Hello, thank you for issue. Can you please provide simple app with failing test? Or at least example code of failing test. Thank you in advance

There is a sample app : http://s000.tinyupload.com/index.php?file_id=51122105514226965471

Great, downloading

I'm having the same issue with jbuilder. I assume the controller is defined slightly different when mocked out during functional tests in rails.
Did you ever reproduce the issue?


Explain that a step considered "complete" when exit code is 0

I hope I understand the "step completeness" correctly. If not, please explain what's a "complete" step.
Thanks!

Thanks @koresar for taking the time to submit a PR… those docs about step completeness are definitely confusing. We'll get this fixed up in #76 for you!


Allow specifying alt override in title

Problem:
In my index.md in the frontmatter where I put the title of my landing page, I have some html that renders a hand waving...

But then the html string gets rendered in the chrome tab

It's be nice to specify the actual alt text for the title in this case so the tab can have a sane text representation in the browser

Oh that helped a little but didn't perfectly fix it... I wasn't really worried about having the gif in the tab anyways, just the title would be fine

oh, do you have a <title></title> tag anywhere?


HBASE-24376 MergeNormalizer is merging non-adjacent regions and causi…

…ng region overlaps/holes.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 35s
Docker mode activated.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 57s
the patch passed

+1 :green_heart:
compile
1m 15s
the patch passed

+1 :green_heart:
javac
1m 15s
the patch passed

+1 :green_heart:
shadedjars
6m 40s
patch has no errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 45s
hbase-server in the patch failed.

_ Other Tests _

+1 :green_heart:
unit
191m 30s
hbase-server in the patch passed.

221m 55s

Subsystem
Report/Notes

Docker
Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1734/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1734

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux ec9419367e9f 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 696894e2cc

Default Java
2020-01-14

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1734/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-server.txt

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1734/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1734/1/testReport/

Max. process+thread count
3146 (vs. ulimit of 12500)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1734/1/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m  0s
Docker mode activated.

-1 :x:
docker
2m 41s
Docker failed to build yetus/hbase:023c6fe56b.

Subsystem
Report/Notes

GITHUB PR
https://github.com/apache/hbase/pull/1734

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1734/2/console

versions
git=2.17.1

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 37s
Docker mode activated.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 54s
the patch passed

+1 :green_heart:
compile
1m 13s
the patch passed

+1 :green_heart:
javac
1m 13s
the patch passed

+1 :green_heart:
shadedjars
6m 40s
patch has no errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 42s
hbase-server in the patch failed.

_ Other Tests _

+1 :green_heart:
unit
191m 12s
hbase-server in the patch passed.

223m 35s

Subsystem
Report/Notes

Docker
Client=19.03.9 Server=19.03.9 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1734/3/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1734

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 2c077a12651d 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 0dae377f53

Default Java
2020-01-14

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1734/3/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-server.txt

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1734/3/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1734/3/testReport/

Max. process+thread count
2754 (vs. ulimit of 12500)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1734/3/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

@virajjasani Yeah, the test is not flakey as it is mostly from testRegionNormalizationMergeOnCluster(), and that one never shows up in the flakey list. I will add more comments in tableRegions.sort(RegionInfo.COMPARATOR). Thanks for the review.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 49s
Docker mode activated.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m  0s
the patch passed

+1 :green_heart:
compile
1m  3s
the patch passed

+1 :green_heart:
javac
1m  3s
the patch passed

+1 :green_heart:
shadedjars
5m 46s
patch has no errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 37s
hbase-server in the patch failed.

 Thanks.

Two very Important finds here, thank you @huaxiangsun !

Thanks for the reviews!


Adding the plugin to an existing gradle project

Failed to apply plugin [id 'net.vivin.gradle-semantic-build-versioning']

org.gradle.api.internal.project.DefaultProject_Decorated cannot be cast to org.gradle.api.initialization.Settings

logs:
Caused by: org.gradle.api.internal.plugins.PluginApplicationException: Failed to apply plugin [id 'net.vivin.gradle-semantic-build-versioning']
at org.gradle.api.internal.plugins.DefaultPluginManager.doApply(DefaultPluginManager.java:156)
at org.gradle.api.internal.plugins.DefaultPluginManager.apply(DefaultPluginManager.java:113)
at org.gradle.api.internal.plugins.DefaultObjectConfigurationAction.applyType(DefaultObjectConfigurationAction.java:113)
at org.gradle.api.internal.plugins.DefaultObjectConfigurationAction.access$200(DefaultObjectConfigurationAction.java:36)
at org.gradle.api.internal.plugins.DefaultObjectConfigurationAction$3.run(DefaultObjectConfigurationAction.java:80)
at org.gradle.api.internal.plugins.DefaultObjectConfigurationAction.execute(DefaultObjectConfigurationAction.java:136)
at org.gradle.api.internal.project.AbstractPluginAware.apply(AbstractPluginAware.java:44)
at org.gradle.api.internal.project.ProjectScript.apply(ProjectScript.java:34)
at org.gradle.api.Script$apply.callCurrent(Unknown Source)
at build_ow8z3b7o1ol9633aj9ki9csm.run(D:\swa-parts-transfer\build.gradle:3)
at org.gradle.groovy.scripts.internal.DefaultScriptRunnerFactory$ScriptRunnerImpl.run(DefaultScriptRunnerFactory.java:90)
... 62 more
Caused by: java.lang.ClassCastException: org.gradle.api.internal.project.DefaultProject_Decorated cannot be cast to org.gradle.api.initialization.Settings
at net.vivin.gradle.versioning.SemanticBuildVersioningPlugin.apply(SemanticBuildVersioningPlugin.groovy)
at org.gradle.api.internal.plugins.ImperativeOnlyPluginApplicator.applyImperative(ImperativeOnlyPluginApplicator.java:35)
at org.gradle.api.internal.plugins.RuleBasedPluginApplicator.applyImperative(RuleBasedPluginApplicator.java:43)
at org.gradle.api.internal.plugins.DefaultPluginManager.doApply(DefaultPluginManager.java:140)

Check out README. This is a settings plugin and so should be used within settings.gradle; not build.gradle.

Hi Vivin,
The snippet I pasted was from settings. Grade.
Sincerely,
On Oct 12, 2017 4:22 PM, "Vivin Paliath"<EMAIL_ADDRESS>wrote:

Check out README
https://github.com/vivin/gradle-semantic-build-versioning/blob/master/README.md.
This is a settings plugin and so should be used within settings.gradle;
not build.gradle.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/vivin/gradle-semantic-build-versioning/issues/82#issuecomment-336254834,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQeu6kt4tjwBcozp1lKVsrOAdNN0GqRiks5srnUIgaJpZM4P3fCB
.

@BadrJbai Can you paste your settings.gradle please?

Hi Vivin,
I sent you an email.
Many thanks,
On Thu, Oct 12, 2017 at 4:54 PM, Vivin Paliath<EMAIL_ADDRESS>wrote:

@BadrJbai https://github.com/badrjbai Can you paste your settings.gradle
please?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/vivin/gradle-semantic-build-versioning/issues/82#issuecomment-336263899,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQeu6s3q1WKgVxAB2goD16Z6Zv8xYzUuks5srnyIgaJpZM4P3fCB
.

--
*Badr Jbaï, University of Ottawa, Computer **Engineering Graduate Student
. *

@BadrJbai
Nothing in the settings.gradle you sent especially stands out. Although, I did notice that you had buildscript block with the plugin repository and classpath dependency inside a gradle.allprojects build. Can you try pulling that out to the top level of your settings.gradle?  Perhaps putting that inside gradle.allprojects is affecting it somehow.
The error message suggests that the plugin is being applied to a project, so something is causing that to happen.

@BadrJbai Did that work?

Hey Vivin,
Something cam up at work. i have not got back it hopefully this week at
work. I  will get back to you if the plugin integration worked with my
project.
Cheers,
On Sun, Oct 15, 2017 at 4:59 PM, Vivin Paliath<EMAIL_ADDRESS>wrote:

@BadrJbai https://github.com/badrjbai Did that work?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/vivin/gradle-semantic-build-versioning/issues/82#issuecomment-336741156,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQeu6lHUtqb6iSXzUtI4JlTL3zyeN-wKks5ssnIugaJpZM4P3fCB
.

--
*Badr Jbaï, University of Ottawa, Computer **Engineering Graduate Student
. *

Hi Vivin,
Our Gradle wrapper is using 3.5. Do you think we have an issue there?
Many thanks,
On Oct 15, 2017 5:06 PM, "Badr Jbaï"<EMAIL_ADDRESS>wrote:

Hey Vivin,
Something cam up at work. i have not got back it hopefully this week at
work. I  will get back to you if the plugin integration worked with my
project.
Cheers,
On Sun, Oct 15, 2017 at 4:59 PM, Vivin Paliath<EMAIL_ADDRESS>wrote:

@BadrJbai https://github.com/badrjbai Did that work?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/vivin/gradle-semantic-build-versioning/issues/82#issuecomment-336741156,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQeu6lHUtqb6iSXzUtI4JlTL3zyeN-wKks5ssnIugaJpZM4P3fCB
.

--
*Badr Jbaï, University of Ottawa, Computer **Engineering Graduate
Student . *

@BadrJbai Not sure... I don't think there should be a problem... Did you try what I suggested above to see if that fixes it?

Working on it tomorrow and next week ! I will keep you posted my friend 😊
Manny thanks!
On Oct 19, 2017 5:23 PM, "Vivin Paliath"<EMAIL_ADDRESS>wrote:

@BadrJbai https://github.com/badrjbai Not sure... I don't think there
should be a problem... Did you try what I suggested above to see if that
fixes it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/vivin/gradle-semantic-build-versioning/issues/82#issuecomment-338042188,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQeu6px1ctFG3bZyf14OZ7EzTvaLf32hks5st73ZgaJpZM4P3fCB
.


[DOCFIX] Update K8s docs

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8557/
Test FAILed.

Tests in error:
alluxio.client.fs.FreeAndDeleteIntegrationTest.deleteDir(alluxio.client.fs.FreeAndDeleteIntegrationTest)
  Run 1: FreeAndDeleteIntegrationTest.deleteDir:145 » TestTimedOut test timed out after...
  Run 2: FreeAndDeleteIntegrationTest.deleteDir:145 » TestTimedOut test timed out after...

alluxio.server.ft.ZookeeperFailureIntegrationTest.zkConnectionPolicy_Session(alluxio.server.ft.ZookeeperFailureIntegrationTest)
  Run 1: ZookeeperFailureIntegrationTest.zkConnectionPolicy_Session:139 » Timeout Timed...
  Run 2: ZookeeperFailureIntegrationTest.zkConnectionPolicy_Session:139 » Timeout Timed...

alluxio.server.ft.ZookeeperFailureIntegrationTest.zkConnectionPolicy_Standard(alluxio.server.ft.ZookeeperFailureIntegrationTest)
  Run 1: ZookeeperFailureIntegrationTest.zkConnectionPolicy_Standard:119 » Timeout Time...
  Run 2: ZookeeperFailureIntegrationTest.zkConnectionPolicy_Standard:119 » Timeout Time...

alluxio.server.ft.journal.ufs.UfsJournalIntegrationTest.file(alluxio.server.ft.journal.ufs.UfsJournalIntegrationTest)
  Run 1: UfsJournalIntegrationTest.file:328->fileTestUtil:332->createFsMasterFromJournal:603 » Runtime
  Run 2: UfsJournalIntegrationTest.file:328->fileTestUtil:332->createFsMasterFromJournal:603 » Runtime

alluxio-bot, test this please.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8568/
Test PASSed.

Is this WIP?

@jiacheliu3 yes


Add the Cohere client name to cohere requests

Adding the client name to Cohere to support debugging and tracing of requests

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

Thanks for signing the CLA! I'll get it merged once the test are fixed.

Thanks for signing the CLA! I'll get it merged once the test are fixed.

For the tests what is the expectation for the API keys, do you have a test API key?

For the tests what is the expectation for the API keys, do you have a test API key?

I think the failures are due to the lints? We run a nightly test with our CI API key, so this PR will be fine to merge as long as the changes were tested locally (with a valid API).


update from pinned nightly rustc to unpinned stable

switch rust-toolchain from pinned nightly to unpinned stable
remove nightly features from lib.rs
drain_filter: use vec.retain in place of nightly drain_filter feature
variant_count:

add variant_count crate to Cargo.toml
use variant_count::VariantCount in place of nightly feature

Also:

add to gitignore: vscode, trunk, MacOS
one typo fix: "Gallery" -> "Galley"
edited readme

add language notations to all code-blocks
consistent capitalization
a few typo fixes
update nvm version reference
other minor edits

Code compiles & passes tests.

Thank you. 😊
I'm really green with Rust, so I am 100% positive that I'll make a lot of mistakes along the way. For all I know, the changes I made might alter our benchmark performance -- I haven't done any benchmarking -- or some other negative side-effect. But it felt good to make those changes. 😄
Thank you for the positive feedback.

The rust code is crazy efficient (but only in release mode) so not too worried about performance there. I tried two different serialization strategies, but they were about the same. Only note though regarding performance is that if the code uses less than 1% of a core when the page is in the background you won't be throttled in the same way. Really important to make the game run well for something like an idle game.
Btw, if you really wanna give this a go and contribute, I'll try to write down something regarding some of my design choices, just so you have a reference.

You should also know I have a vue3 game also that I worked on after this:
https://github.com/avacore1337/tower-idle
Didn't get as far with content there though, but it is typescript which is good if you like that. It also has a newer toolchain for javascript since I started on that later.

I really appreciate all the thoughts you have. Thank you. Just eager to learn and improve.


Turn off test mode for now

Application test mode needs a little bit of work to get to a polished state. I think for now we should turn off test mode in the UI and enable it in a subsequent release.
@harshach @shahsank3t

@sureshms I lean towards to shipping it. Can we make this decision next week depending on priorities if we can't get it we will hide it. Also can you list out the issues that you see right now

@harshach I am not sure about the current UX. It just adds more complexity. That said, I am going to test it and come up with concrete set of issues. If we decide to include it, I am fine. For now let's make this issue p1.

Current UX doesn't cover adding sink expected output, and also doesn't provide showing test histories btw. It only has event log related feature now. This seems a bit out of sync with what we've designed.

A few issues with test mode -
https://hortonworks.jira.com/browse/BUG-81212
https://hortonworks.jira.com/browse/BUG-81216
https://hortonworks.jira.com/browse/BUG-81218
If we plan to ship test mode we should consider addressing the above. The test mode currently does not produce any output for time based window/aggregate since it runs for a fixed duration (30 secs). So may be we should expose additional options for users control when to emit data from sources and how long to run the tests (may be by clicking start/stop/emit buttons)

We can also have an option for users to enter inputs while the tests are running and continuously show the generated outputs until the user stops the tests. (something like kafka console producer/consumer). Right now the tests are more like a batch job.

For now, we have hidden test mode from UI.
Commit id: https://github.com/hortonworks/streamline/commit/9d7fb0b9e79eaee014c3e3f3ef4bc9aa150b8044


Row height

I tried to override the default styles for row height as in docs:
@use<EMAIL_ADDRESS>with (
  $gantt-item-height: 44px
)

But it doesn't affect the chart at all.
It's impossible to override it manually also because the SVG link lines are rendered at the wrong place.

I confirm that this is a defect，we will provide row height config in version 17

Please upgrade the version to resolve the issue


Keyboad shortcut or @action to add all open files to context

Describe the need of your request
When I work on implementation of a feature/function, most of the time all relevant files end up being open. Currently I switch to Project > Open files toolbar and select all files with mouse, then I can add them to context in one go. This hugely improves the quality of the output. But having to switch to mouse is not fun.  Cmd+A includes parent folders so that's not helpful.
Proposed solution
Any of the following would be enough to enable a keyboard only flow:

An IDE action to work live cmd+A but selects only files in the "Open files" window.
A global action to add all open files to CodeGPT context.
An @Open files chat command to include all open files to chat.

Additional context
No response

Thanks for reporting!
Currently, when you include files using the '@' command, the list of files are sorted with open files appearing first, followed by the others. However, this doesn't solve the issue of including all open files in your session. I will most likely add a new sub-action under 'Files' as 'Include all open files', with an option to attach a shortcut for it.

Thanks, I'm looking forward to landing of that feature.


GUI not loading, no KBFS

So on my Surface Book (which seems to have lots of trouble with Keybase [#5215]), I now can't load the client at all. KBFS isn't starting, so I don't even have the K: drive that would be inaccessible anymore.
my log id: 8eed4a4c49681b668b1f5d1c

Upon further inspection, it looks like both of my Windows clients cannot load GUI or access KBFS.
other client log: 3d55f92809ff36a9b3ffa41c

cc: @zanderz

Looks like this is potentially an AV interaction issue.
https://virustotal.com/en/file/e638a56952be1f3f78b7f1c41fb875c9d6b93717ba4554263ec25b8afdb722d6/analysis/1493082395/
I use Bitdefender, which is one of the AVs flagging runquiet.exe.

It does look as though our launcher utility, runquiet.exe, was blocked.
Among the measures we're taking to cut down on this, we're renaming runquiet.exe to keybaserq.exe, and we're going to stop rebuilding it for every release, so hopefully the false positive reports can have more of an effect. If you would like to try the keybaserq change, you can download it here

Looks like that version (and today's definition updates in Bitdefender) stops the trojan detection in the shortcut, but still flags keybaserq. I've added an exclusion for now in Bitdefender.

thank you, that's an interesting result. We'll re-submit.

I am still having the issue with the release from zanderz from 2 days ago.  Every time I restart my computer, it won't load keybase without uninstall and reinstalling it.  I am not using any 3rd part AV application other than that built into Windows.

@jsb2092 You said the version from 9 days ago fixed this, right? https://github.com/keybase/client/issues/6687#issuecomment-298427438
And now it came back? The same fix should be in both builds. Can you please do a keybase log send? Thanks.

@zanderz Correct, it DID fix it, until I rebooted.
The log ID is: 11a20128ef0f87012647a31c

Thank you @jsb2092. I see the GUI connecting to the service at 9:44 today in this log, did it work that time or was it in disconnected state?

@zanderz Correct, that was after a reinstall.  This is a log after a reboot
f46a54966723ef893fb1941c

ah ok, @jbs2092, the service doesn't seem to be running at all in that second log. The installer logs seem to  show that the startup shortcut was added OK. Does Task Manager show any instance of keybase.exe running (lower-case)? Do you have a shortcut called KeybaseStartup in %APPDATA%\Microsoft\Windows\Start Menu\Programs\Startup ?
If keybase.exe is not running and the shortcut is present, can you try just invoking it?
Maybe our log send needs to add some of this info, but in the meantime, thanks again.

@zanderz
I have 4 instances of keybase.exe running in task manager.  KeybaseStatup is in the startup folder
This is a new log, along with some output from when I generated it.

INFO ignoring error getting keybase status: Timed out waiting for pipe '\.\pipe\kbservice\Users\jsb\AppData\Local\Keybase\keybased.sock' to come available
INFO ignoring UI logs: Timed out waiting for pipe '\.\pipe\kbservice\Users\jsb\AppData\Local\Keybase\keybased.sock' to come available

Success! Your log ID is:
3c3d4f0dd2561b09075a4c1c

@jsb2092 Are these 4 instances of Keybase.exe (capitalized), or are any of them keybase.exe (all lower case)? Did you double-click KeybaseStartup?

@zanderz sorry, I didn't fully answer previously.  They were all with an uppercase K, and clicking on the KeybaseStartup caused two lowercase k instances to start (named Keybase utility)

@jsb2092 The 3c3d4f0dd2561b09075a4c1c log still looks like lower-case keybase.exe is not running at all - I assume it was sent before you launched KeybaseStartup? If not, is the GUI connected now? If the GUI is still not connected, can you "quit" it via the tray menu and restart it with the start menu Keybase shortcut?

@zanderz yes, that was before I ran statup, and running the startup manually did fix it.


Latest Nuget Doesn't install on Xamarin.Forms Android

Winphone 8 is not supported
Please refrain from +1's, nuget library update requests (this is not an issue with the library), and requesting timelines for something.  Please also make sure you have read the FAQ section on the README

Please check all of the platforms you are having the issue on

[] iOS
[X] Android
[] UWP
[] PCL

Please state the issue to the best of your ability below
Failed when installing using npm targeting android project. I am using Xamarin.Android.Support.Design version <IP_ADDRESS>
Please copy and paste the exception details below.  If you are having a crash and this detail is missing, your issue will be closed without warning.
Unable to find a version of 'Xamarin.Android.Support.Design' that is compatible with 'Acr.UserDialogs 6.0.0 constraint: Xamarin.Android.Support.Design (>= 23.3.0 && < 24.0.0)', 'BottomNavigationBar 1.1.3 constraint: Xamarin.Android.Support.Design (>= 23.4.0)', 'Xamarin.Forms <IP_ADDRESS> constraint: Xamarin.Android.Support.Design (= 23.3.0)'.

Your issue is with bottom navigation bar (requires 23.4) and xamarin forms requires 23.3


R-18G toggle?

I can respect why r18mode excludes all other "R-18'X'" tags, but could you please add a config mode to download these too?
Alternatively, if you could add an option to download by 'list + tags', then this will solve similar requests.
Much love ❤👊

R18mode in config.ini will download only R-18* tagged images.
Check your configuration on the Pixiv website and delete your cookie value in config.ini after updating.

So I tested this again with a freshly-made account, a new zip of the latest release, corrected my view settings as above, reloaded my cookie by signing out and in, then double-checking the view settings.
I set r18mode to true, turned off processfromdb and set daylastupdated to 0, and no other changes except login u/p and the new cookie.
I picked at random this(19719856) user to test R-18G tags, and this(237010) user to test R-18? tags.
I tried with both modes 1 (by member id), and 4 (from list.txt).
No matter what I did only R-18 images were downloaded. Even if they were double tagged as R-18 and R-18'X', they downloaded, but only if they had an R-18 tag too.
Feel free to test this yourself, it takes about 20 mins to do everything 👍

I set r18mode to true, turned off processfromdb and set daylastupdated to 0, and no other changes except login u/p and the new cookie.
No matter what I did only R-18 images were downloaded.

Off course it will only download the R-18.
Like I said, if you set r18mode=True, it will only download R-18 images.
If you set r18mode=False, it will download all images (including R-18).

Except, what I'm saying is that the R-18G images ARE NOT downloaded when r18mode=True
Only the R-18 images, and not the R-18G are being downloaded in this mode. See for yourself.

Ahh, no wonder. It is no supported by pixiv
tried below query, it doesn't return any images.

oldui
http://www.pixiv.net/member_illust.php?id=19719856&tag=R-18 OR R-18G

whitecube
https://www.pixiv.net/rpc/whitecube/index.php?mode=user_new_unified&id=19719856&offset_illusts=0&offset_novels=0&limit=50&tt=*yourtoken*&tag=R-18 or R-18G

So erm, could you please add an r18gmode, or make an option to download by 'list + tag(s)' ?

Modify the option 4. You can set the tag to be downloaded from the member id list.
If no tag given/empty string, it will download all image (older behaviour)

Thanks ever so much!!

And this thing breaks my automation now...

4 - Download from list
(optional: followed by path to list and optional tag)

should be the same: PixivUtil2.py -s 4 -x or PixivUtil2.py -s 4 tag_here -x

Ah. Missed that line.
It just didn't get that it needs to use default if there's no tag... But that's fixed now.

The tag feature interacts with r18mode = true.

yep, no changes for that as I'm using it 😄

No, I think it overwrote my pre-set tag? Will test tomorrow and, if doubts remains, elaborate in separate issue.

crap copy paste error
whitecube ==> OK
https://github.com/Nandaka/PixivUtil2/blob/master/PixivBrowserFactory.py#L357
oldui ==> Not OK.
https://github.com/Nandaka/PixivUtil2/blob/master/PixivBrowserFactory.py#L376

updated on https://github.com/Nandaka/PixivUtil2/releases/tag/v20161207

How should I update my command line to make it stop asking for tags? My current line is "-s -4 -x".

Try PixivUtil2.py -s 4 tag_here -x

Do you mean it? I haven't found the "tag_here" in the code or readme. Do you mean that I should put some tag there or that any non-valid tag would make it not look for tags? I need the old behavior for now.

woops missing one argument, it is expecting the list.txt file as the first
argument, then followed by optional tag, else it will go to the "Tag : "
line and use the default list.txt location
try

PixivUtil2.py -s 4 list.txt -x
PixivUtil2.py -s 4 list.txt tag_here -x
PixivUtil2.py -s 4 tag_here -x

Ok, so it seems "-s 4 list.txt -x" worked. Thanks.


Decrypt image with Mediakey from ChatStorage.sqlite (IOS)

Hi,
i identified column ZMEDIAKEY in table ZWAMEDIAITEM in the IOS version of
Whatsapp sqlite database containing the MediaKey for image decryption.
(Ok, you could have guessed by the name of the column)
Unfortunately the content is stored in a Blob and I#m not able to use it directly.
Whatsapp-Web-Reveng is showing me the correct value. Also copying ZMEDIAKEY from
another sqlite database leads me to a working MediaKey for decryption.
Unfortunately this process is very clumsy. Would like to get it done by scrypt.
Anybody any idea how to get from
{select quote(zmediakey) from zwamediaitem;"}
X'0A2027BDE5011940BF16D599255AD8A2CA00FAF0ABE35D4BBE82984A3D1B0D73982E12204B6477490B97591CF29DE9272EFF25AF1CB32821891263B5D1BC60F910D05E7218F6FC84E7052000'
to
"mediaKey" : "J73lARlAvxbVmSVa2KLKAPrwq+NdS76CmEo9Gw1zmC4="  ????
regards
jones

I have the same problem, it's been 4 years since then. Did you somehow solve the problem?


Add a history system

Create a way to keep track of data that has been modified. Record which user modified the data and the date it was changed. Store the data of all the columns as a json packet.

The initial system seems to be working


JS/TS language settings not working, cannot change TypeScript version

Issue Type: Bug
I cannot access any Intellisense features whatsoever, the right click menu has only "change all occurrences," cut, copy, paste, and "command palette", and I cannot change the TypeScript version. When I attempt to do so, I get the following error log:
[2019-02-13 22:36:47.695] [exthost] [error] TypeError: Cannot read property 'serviceClient' of undefined
	at t.SelectTypeScriptVersionCommand.execute (c:\Users\redacted\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\typescript-language-features\dist\extension.js:1:70764)
	at e._executeContributedCommand (c:\Users\redacted\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\services\extensions\node\extensionHostProcess.js:485:959)
	at e.$executeContributedCommand (c:\Users\redacted\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\services\extensions\node\extensionHostProcess.js:486:443)
	at t._doInvokeHandler (c:\Users\redacted\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\services\extensions\node\extensionHostProcess.js:644:118)
	at t._invokeHandler (c:\Users\redacted\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\services\extensions\node\extensionHostProcess.js:643:744)
	at t._receiveRequest (c:\Users\redacted\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\services\extensions\node\extensionHostProcess.js:642:322)
	at t._receiveOneMessage (c:\Users\redacted\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\services\extensions\node\extensionHostProcess.js:641:155)
	at c:\Users\redacted\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\services\extensions\node\extensionHostProcess.js:638:960
	at c:\Users\redacted\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\services\extensions\node\extensionHostProcess.js:38:994
	at e.fire (c:\Users\redacted\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\services\extensions\node\extensionHostProcess.js:44:113)
	at a (c:\Users\redacted\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\services\extensions\node\extensionHostProcess.js:170:213)
	at Socket.f._socketDataListener (c:\Users\redacted\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\services\extensions\node\extensionHostProcess.js:170:434)
	at Socket.emit (events.js:182:13)
	at addChunk (_stream_readable.js:279:12)
	at readableAddChunk (_stream_readable.js:264:11)
	at Socket.Readable.push (_stream_readable.js:219:10)
	at Pipe.onread (net.js:636:20) typescript.selectTypeScriptVersion

I have uninstalled and reinstalled the editor, disabled all extensions, and tried the insiders edition, and none change this.
VS Code version: Code 1.31.1 (1b8e8302e405050205e69b59abb3559592bb9e60, 2019-02-12T02:20:54.427Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6700HQ CPU @ 2.60GHz (8 x 2592)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
15.83GB (9.08GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (5)
Extension|Author (truncated)|Version
---|---|---
monitrc|dru|0.0.1
dotenv|mik|1.0.1
python|ms-|2019.1.0
vscode-docker|Pet|0.5.2
wallaby-vscode|Wal|1.0.109

Closed, discovered the issue was a PATH problem as discussed in https://github.com/Microsoft/vscode/issues/17945. Closing the editor and reopening it was necessary for the editor to start working again.


[stable/grafana] Add image renderer container as chart option

Is your feature request related to a problem? Please describe.
Image rendering is now handled by an external container in Grafana 7.0. Listing the grafana-image-renderer plugin under the plugins directive in the current chart results in a crash loop with the following log output:
t=2020-06-28T11:23:27+0000 lvl=eror msg="A service failed" logger=server err="Failed to start renderer plugin: Unrecognized remote plugin message: \n\nThis usually means that the plugin is either invalid or simply\nneeds to be recompiled to support the latest protocol."

Issue links:

https://github.com/grafana/grafana/issues/25180
https://github.com/helm/charts/issues/21959

Describe the solution you'd like
A chart option to easily enable the image renderer sidecar container as describe in https://grafana.com/blog/2020/05/07/grafana-7.0-preview-new-image-renderer-plugin-to-replace-phantomjs/ which embeds the solution below into the chart. Something like sidecar.imageRenderer.enabled: true
Describe alternatives you've considered
Currently using the solution outlined below.
Additional context

In case someone else is stumbling across this and needs a quickfix with the current chart:
    extraContainers: |
      - name: renderer
        image: grafana/grafana-image-renderer:latest

    env:
      GF_RENDERING_SERVER_URL: http://localhost:8081/render
      GF_RENDERING_CALLBACK_URL: http://localhost:3000/

Originally posted by @languitar in https://github.com/helm/charts/issues/21959#issuecomment-640653320

Bumping issue

Bumping issue


Add redirect to our sites after clicking logos in the footer

The goal here is to redirect people to 1hive.org when they click the bee in the footer, and to Connext site when they click on Connext.

Howdy @crisog Could I step up on this issue ? if you haven't assigned anyone.

Howdy @crisog Could I step up on this issue? if you haven't assigned anyone.

@Harthor is working on this, sorry that I didn't assign it here right away. We have some other repositories where you can help too. If you are not on our discord yet, feel free to join! https://discord.gg/uvP7mpjPJ4

The goal here is to redirect people to 1hive.org when they click the bee in the footer, and to Connext site when they click on Connext.

Hey @crisog! Sorry for bothering again lol, but since 1hive.org goes to 1hive proposals page, better to use wiki.1hive.org (opens in a new page on click) OR 1hive.io, dunno what should be better.

Hey @vaish567, @Harthor has been having some trouble completing this issue. Are you still interested in helping? His progress is on #37.
Thanks in advance :pray:

Hey @vaish567, @Harthor had some trouble completing this issue. Are you still interested in helping? His progress is on #37.
Thanks in advance 🙏

@vaish567  If you want to look at it together, the issue has to do with the size of the images for the responsive part (phones), my discord is
tonga 🐝 #3112

@Harthor I'm not able to add you on discord. Could you send me the request? data_type#7296

@Harthor I'm not able to add you on discord. Could you send me the request? data_type#7296

I think we can add each other if we share a common server, maybe you can ping me on café chat on 1Hive? @tonga  🐝, (I can't find you either)

Hello @Harthor Please check I have sent you request so that we can work on it.

@Harthor I'm not able to add you on discord. Could you send me the request? data_type#7296

I think we can add each other if we share a common server, maybe you can ping me on café chat on 1Hive? @tonga , (I can't find you either)

Hello @Harthor Please check I have sent you request so that we can work on it.

Would be cool if someone could review the pull request, the footer doesn't look exactly the same, but pretty similar.


Implement IInstrument for DMM

[x] This contribution adheres to CONTRIBUTING.md.

What does this Pull Request accomplish?

Creates an implementation of IInstrument for NI-DMM
Adds Tests for the Create / Attach / Detach and Close

Based on the NI-Digital
Why should this Pull Request be merged?
Implements the following US AB#2533859
What testing has been done?
Auto Tests pass

There are 0 checks on this PR, presumably because of the branch conflicts.  Please ensure the checks run and the G tests pass before submitting. I'd like to do a last quick check after the conflicts are resolved, so you can mention me when that's done.

use of the branch conflicts. Please ensure the checks run and the G tests pass before submitting. I'd like to do a last quick check after the conflicts are resolved, so y

@dixonjoel

Minor: There are some inconsistencies in the capitalization of "DMM" vs "Dmm" in folder names. For the tests, the Virtual Folder is "niDmm" while the disk folder is "niDMM".
measurement-services-labview\Source\Tests\Tests.Runtime\Measurement Server\niDMM


fix: typo in menusExtensionPoint

This PR fixes #
Typos in menusExtensionPoint.ts, it should be menu items instead of submenu items;

Anything preventing this PR to be merged?


Make actions argument type optional in withRPCRedux

The second argument of withRPCRedux required an actions property so if you wanted to use any of other properties like mapStateToProps you must pass null for actions to not fail a flow check.
Also remove an unnecessary rpcId

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


Apply naming convention to output assets

PR Overview
This applies the new naming conventions to output assets and prunes some output tables in the database.

PR Checklist

[ ] Merge the most recent version of the branch you are merging into (probably dev).
[ ] All CI checks are passing. Run tests locally to debug failures
[ ] Make sure you've included good docstrings.
[ ] For major data coverage & analysis changes, run data validation tests
[ ] Include unit tests for new functions and classes.
[ ] Defensive data quality/sanity checks in analyses & data processing functions.
[ ] Update the release notes and reference reference the PR and related issues.
[ ] Do your own explanatory review of the PR to help the reviewer understand what's going on and identify issues preemptively.

Closing this in favor of creating smaller PRs to be merged into #2818.
output_asset_aggregations.ipynb is on the resolve-eia-denorm-assets branch.


Modin incompatibility with Seaborn boxplot

System information

Modin version (modin.__version__): 0.12.0+62.g3c740dbf
Python version: 3.7

Creating a boxplot using seaborn results in an error.
Source code
plt.figure(figsize=(10, 10))
sns.boxplot(y=df_1['Title'], x=df_1['Salary'],order=sorted_index_descent)
plt.ylabel('')
plt.title('Salaries for Data Science Positions')
plt.show()

Error message

@labanyamukhopadhyay could you please re-check if this still happens?

Seaborn and specifically sns.boxplot as in the example above are currently interoperable with Modin. Users will have to install seaborn from Github for its most recent updates: pip install "seaborn @ git+https://github.com/mwaskom/seaborn"


update the store names in the test files

Coverage remained the same at 84.615% when pulling baef3ac3286eac9c7e743ae39eeeda4a2817981d on implement-scorecard-feedback into 9002fc46e092102f7806f9bcf90e922d4ddbfffb on develop.

Coverage remained the same at 84.615% when pulling f531bf00fb23364e0bd32209f828ee56bf7513b5 on implement-scorecard-feedback into 9002fc46e092102f7806f9bcf90e922d4ddbfffb on develop.

Coverage remained the same at 84.615% when pulling f531bf00fb23364e0bd32209f828ee56bf7513b5 on implement-scorecard-feedback into 9002fc46e092102f7806f9bcf90e922d4ddbfffb on develop.


Add function attachInterrupt.

Usage: attachInterrupt(PA_4, blink, FALLING);

https://github.com/Microsoft/devkit-sdk/issues/789


[US] User Action Logging & Tracing

As an application configurator, I want to log and trace relevant user actions, so that I can capture user interactions and activities on data pages.

MoSCoW
Risk
Story Points

Must
High
8

Acceptance Criteria

Any relevant action done by a user is logged into a table
Messages are displayed upon success or failure of such actions

Risks

Confidential or sensitive information could be leaked

Pre-requisites

[x] Meet with the stakeholder to clarify the requirements if deemed necessary
[x] Discuss with the pod how the logging mechanisms would work and arrive to a consensus
[x] Figure out what actions should be logged and how should they be displayed

Tasks

[x] Implement user-facing logging mechanisms for user actions
[x] Log user actions with their timestamps and any errors encountered by such action
[x] Display messages or notifications for operations done (user, role, timestamp, details potentially with primary key, schema and table name)

UI Mockup

https://github.com/WillTrem/UInnovate/assets/78176331/2a423edb-1018-420f-92f8-b8be8f35fe1a

Looks great! Signing off!


Execute key exchange command on first Subutai initalization

In order to enable execution of commands on MH we need to explicitly execute
"subutai import management -t {token}"  on RH with MH one time during initializaiton of local peer.
Make sure command has succeeded otherwise retry with intervals until it succeeds

Implemented in LocalPeer. Exchange happens once on every SS initialization.


fix: correct minor typo in word_tokenize

This pull request address 2425. It simply refactor the docstring of preserve_line parameter in word_tokenize function to be more meaningful.

Thanks @12mohaned


codestyle: Annotate the keylime tenant tool and 2 dependencies

This PR annotates the keylime tenant tool and 2 dependencies.

This really is just a draft for now...

/packit test

/packit retest-failed

/packit retest-failed


Columnstore Index causes unnecessarily-specific Scaffold-DB output.

After uprading all our components to version 3.1-preview3 we noticed scaffold-db started picking up a previously ignored table which utilizes an SQL Server columnstore index[1]. The table contains a column of type nvarchar(max) which, when scaffolded, results in a model that won't load[2] in Sqlite. The generated code includes a .HasColumnType("varbinary(max)") line as part of the column's configuration, causing the Sqlite provider to throw an exception for it doesn't recognize that type.
This seems to also happen on columns of type varchar and varbinary. It only happens if the length is max and only if there is a columnstore index in place.
[1] - it is clustered so the table doesn't have a conventional PK, meaning prior to 3.1 it was considered a query type and wasn't picked up (my guess).
[2] - We create in-memory Sqlite instances for tests by using the DatabaseFacade.GenerateCreateScript() method.
Steps to reproduce
Create a table with an array-based column of length max with a CSI:
CREATE TABLE [TestTable] (
	[ID] [int] IDENTITY(1,1),
	[Body] [nvarchar](max) NOT NULL
)
GO

CREATE CLUSTERED COLUMNSTORE INDEX cci_Test ON [TestTable]
GO

Running scaffold-db against this DB creates a PrimaryContext.cs with the following config:
modelBuilder.Entity<TestTable>(entity =>
{
    entity.HasNoKey();

    entity.ToTable("TestTable");

    entity.HasIndex(e => new { e.ID, e.Body })
        .HasName("cci_Test");

    entity.Property(e => e.Body)
        .IsRequired()
        .HasColumnType("nvarchar(max)");

    entity.Property(e => e.ID).ValueGeneratedOnAdd();
});

This breaks with Sqlite which uses the TEXT type. Changing the Body column to nvarchar(255) modifies the relevant part of the config to
entity.Property(e => e.Body)
     .IsRequired()
     .HasMaxLength(255);

Which works fine with Sqlite. Similarly, removing the cci_Test index results in the following (valid) config:
entity.Property(e => e.Body).IsRequired();

I would've expected to see this same result from the initial example - which would seemingly make it work correctly with both SQL Server and Sqlite.
Further technical details
EF Core version: 3.1-preview3
Database provider: Microsoft.EntityFrameworkCore.SqlServer, Microsoft.EntityFrameworkCore.Sqlite
Target framework: net461
Operating system: Win10
IDE: Visual Studio 2019 16.3

Scaffolding is provider specific, so you cannot expect a scaffolded SQL Server model to work with SQLite, if it does it is pure luck!

While I see what you mean, this directly breaks the possibility of testing code that uses reverse-engineering to keep the DB model up-to-date, when using Sqlite as the testing back-end, as suggested here. As the first article implies, this would not be an issue if or when #831 is implemented.
All of this is secondary to the issue at hand which naively seems to be that the scaffold tool has trouble recognizing column types consistently. For a known type, I would expect the tool to use only one of the following formats:

explicitly specify its type and size using HasColumnType("<type>(<length>)")
use type defaults and known methods such as HasMaxLength(<length>)

The examples I gave suggest this is not the case. Additionally, I would not expect the presence of an index or the changing of a column size to affect the type of any column as seen by the tool.

The issue here seem to be nvarchar(max) is being scaffolded in fluent API even though it should be default. ColumnStore index is red-herring.
@ixtreon - As @ErikEJ said, scaffolding DbContext for 1 provider and using it for another may work in some scenario but it is certainly not a supported scenario.

CREATE TABLE [TestTable] (
[ID] [int] IDENTITY(1,1),
[Body] nvarchar NOT NULL
)
GO
CREATE CLUSTERED COLUMNSTORE INDEX cci_Test ON [TestTable]
GO

The script you provided fails with error
The statement failed. Column 'Body' has a data type that cannot participate in a columnstore index. Omit column 'Body'.
If I use something other than nvarchar(max) like nvarchar(100) then scaffolded code contains
                entity.Property(e => e.Body)
                    .IsRequired()
                    .HasMaxLength(100);

Please provide a runnable project which demonstrate the issue you are seeing.

@ixtreon With regard to testing we have an issue to update the documentation. The trade-off here is between ease of setting up a test database against how closely that database matches what the production database will. For example, if the target is SQL Azure, then:

Testing against the in-memory database is sufficient for simple updates and queries, but behavior specific to relational databases may be different or missing.
Testing against a SQLite in-memory database will bring in more of the relational-specific behaviors and hence be closer to how SQL Azure behaves.
Testing with SQL Server locally will be very close to how SQL Azure works, but there can still be differences.
Testing against SQL Azure will ensure the database behaves the same as in production.

@smitpatel Using SQL Server Dev 64-bit, v14.0.2027.2 the following SQL script outputs "Commands completed successfully" in SSMS; the table is created and a CCI is present:
CREATE TABLE [TestTable] (
	[ID] [int] IDENTITY(1,1),
	[Body] [nvarchar](max) NOT NULL
)
GO

CREATE CLUSTERED COLUMNSTORE INDEX cci_Test ON [TestTable]
GO

This is slightly different from the SQL you've run which contains [Body] nvarchar NOT NULL (the max is omitted). Both versions seem to work on my SQL Server instance and create a table with a corresponding CCI. The second one creates a column of type nvarchar(1) though which shouldn't trigger the issue from what I've observed.
@ajcvickers I am aware the two providers are different. This was the reason to choose Sqlite over in-memory for testing. I am afraid I'm not catching your broader point though.

@ixtreon - I copy-pasted your SQL only. (it was not nvarchar(1), just github's formatting failed)
I still hit the error

It requires SqlServer 2017. We will investigate on our end further.

@AndriySvyryd verified that output posted above is correct.
Column Body is part of an index. According to SqlServerTypeMappingSource indexes have limited max length. 450 for unicode string hence nvarchar(450). Across all SqlServer versions supported by EF Core, some of them requires such constraint and disallow creating index on column with datatype nvarchar(max) (As in above for SqlServer 2016). Which means that the type we encountered for this model in database is non-default hence we have to scaffold it. Since it is max size, we need to use HasColumnType rather than using HaxMaxLength.
While the invariant that string column participating in index must be limited size may not be true across all versions. Cost of making typeMapping version specific is high. The downside is, code need to specify column type explicitly which is not too bad.
Closing as by-design


'RIGHT-TO-LEFT OVERRIDE' (U+202E) not displayed properly

Describe the bug
It seems that YaTA strips/does not render the unicode character 'RIGHT-TO-LEFT OVERRIDE' (U+202E) correctly.
To Reproduce
On Twitch vanilla chat:

Insert !pts rhymu8354 99999 into the chatbox
Position the cursor before the first number (between the u and 8)
Insert the unicode character U+202E

On at least Ubuntu Firefox, you can hold down CTRL+Shift, then type u202e, then let go of CTRL+Shift
Assuming GitHub doesn't mangle, it should then look like this: !pts rhymu8354 99999

Send

Expected behavior
Display the message as:

!pts rhymu8354 99999

Actual behavior
The message is being displayed as:

!pts rhymu8354 99999

The right one is the vanilla twitch chat, the left one is the streamers chat, which claims to use YaTA. See it in the VOD: 0192, 3h53m42s. Note that in the chat-replay, it seems it is also displayed incorrectly, but it did display like in the screenshot when I sent it. (Maybe it was displayed like that just on my end?)
Environment
I am on Ubuntu 18.04 with Firefox 67.0 (64-bit), using Twitch vanilla chat. Try asking @rhymu8354 for their information. I guess at least Windows 10 on Chrome.
Note: I don't use YaTA, so I am not sure if there is an setting about this behaviour. If there is, then I am sorry.

Thank you for the detailed report, @Neui !  @HiDeoo is correct; I don't use YaTA in my overlay.  Maybe I should?  The overlay I currently use is from StreamElements.  I may replace this with cmgriffing's "chatterlay" service when that's ready.  I'll let Chris know about this so he can test that his overlay doesn't have this issue.

I let Chris know in his stream; not sure if he will fix it or not.

Thanks for confirming :)

I don't use YaTA in my overlay. Maybe I should?

It's nice to see people using YaTA altho I would admit, I'm not sure a chat overlay would be a good use case with all the UI baked into it, most of the tools would be useless in there, etc.
On the contrary, if you take a tool like the one @cmgriffing is building, it's from the ground up built properly for this usage, with overlays in mind, with tons of features dedicated to that specific usage so I think it's a no-brainer better and more qualitative choice.


MODCLUSTER-485 Move build to Travis CI and create matrices for JDK/pr…

…oper Tomcat builds
https://issues.jboss.org/browse/MODCLUSTER-485

Triggering build using a merge of 87e12c6591e092510738d592687a7b5ee2e6b73d on branch master:
Private: https://jenkins.mw.lab.eng.bos.redhat.com/hudson/job/mod_cluster-1.3.x-pull-player-executor/

Build 171 outcome was SUCCESS using a merge of 87e12c6591e092510738d592687a7b5ee2e6b73d on branch master:
Private: https://jenkins.mw.lab.eng.bos.redhat.com/hudson/job/mod_cluster-1.3.x-pull-player-executor/171
Public: https://hudson.jboss.org/hudson/job/mod_cluster-1.3.x-pull-player-executor/171

Lets give it one more thought -- the common module won't compile against new Tomcat  libs.


Raise an exception when trying to subscribe to an unregister event

Fixes #3
@solnic @mensfeld I added the feature request to the library. Hope it helps 😄

@solnic any update on this one?

unregistered implies that the event was previously registered but has been removed, is this the case or is it the case for events that have never been registered? If it's the latter I'd go for something like NoEventRegisteredError and similar messages

@solnic @mensfeld @AMHOL I applied the changes. 😄

@solnic any update on this one?

Ups I accidentally close it 😄

Uff...

I think it would also be helpful to raise an exception when trying to publish an event that doesn't exist. It's easy to make a typo & waste time looking for the problem.

@matugm that is a good idea. I've already wasted a lot of time because of that ;)

This would help a lot if it will be merged and released:)

@drqCode looks like @GustavoCaso went M.I.A so somebody could pick it up, update, address issues that were mentioned in the review, and open a new PR.

@GustavoCaso I don't have permissions to push into your fork, so I'm closing that issue and wil reopen with your code from the origin.


Improve visually impaired functionality

Here's a review we got from a german user that inspired us to think about this issue:
"the app is not barrier-free for blind and visually impaired people. Blind and visually impaired people usually work with VoiceOver. Many fields and buttons in the app as well as some of the text cannot be read under VoiceOver. Apple gives blind people and disabled people the opportunity to use many apps via VoiceOver. Unfortunately, this app is not or only partially possible with this app. Hence only one star! As soon as the app is completely accessible, this will be checked again."
Brainstorming ideas to help:

Buttons missing VoiceOver
Infinite scrolling disable VoiceOver
Navigate into and out of pages for categories. Need to let user know location
VoiceOver voice needs to speak differently from vocable voice

Lauren took a preliminary look at the app:
The most important thing I got from looking into this is that there should be some easy-to-access mechanism for the user to pause/stop the audio within the app itself. VoiceOver can be quickly disabled on iPhones. Having a quick and easy way to disable the Vocable voice gives the user control over what voice they want for the situation. If they need the Vocable app the read out texxt, they can turn off VO. If they need VO to read out the text, they can tell Vocable to not read out anything.
Here's some info. It's web focused, but the same principle applies! https://www.w3.org/WAI/WCAG21/Understanding/audio-control.html


Add USB 3.0 support

Should support USB 2.0 backwards compatible.
Support transfer of raw frames rather than MJPEG stream.
Take a look at https://github.com/mossmann/daisho/tree/master/sw/fpga/common/usb3
Daisho USB Core

This is an open source USB 3.0 core.  It can be used to implement a USB device
on an FPGA with a USB 3.0 transceiver such as the TUSB1310A.  It can also
operate in USB 2.0 mode with a ULPI transceiver.

This core is being developed for the Altera Cyclone IV.  Operation on other
FPGA platforms may be possible with some work.

Planned functionality includes device (not host) operation in USB 2.0 and 3.0
(including SuperSpeed) modes.

The core is part of Project Daisho but may be used on other platforms.

This issue was moved to timvideos/HDMI2USB-misoc-firmware#159


feat: port Data.Set.Pointwise.Basic

[x] depends on: https://github.com/leanprover/std4/pull/83
[x] depends on: https://github.com/JLimperg/aesop/pull/38
[ ] depends on: #1447

This PR/issue depends on:

leanprover/std4#83
JLimperg/aesop#38
leanprover-community/mathlib4#1447
By Dependent Issues (🤖). Happy coding!

This PR/issue depends on:

leanprover/std4#83
JLimperg/aesop#38
leanprover-community/mathlib4#1447
By Dependent Issues (🤖). Happy coding!

Thanks :tada:
bors merge


Monument record submission button addition and changes

Current:

Change to
Three buttons:

create new monument sheet
upload existing monument sheet
close

From Sean:

I like the simplicity, but surveyor may confuse new and existing monument with new and existing record sheet.

Please change:

new to new sheet
existing to existing sheet

validated


python38Packages.pytools: 2021.2.7 -> 2021.2.8

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python38Packages.pytools is: "Miscellaneous Python lifesavers."
meta.homepage for python38Packages.pytools is: "https://github.com/inducer/pytools/"
meta.changelog for python38Packages.pytools is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 2021.2.8 with grep in /nix/store/ipirilf2b2w4jffn8kw417wmvcv2fwa7-python3.8-pytools-2021.2.8

found 2021.2.8 in filename of file in /nix/store/ipirilf2b2w4jffn8kw417wmvcv2fwa7-python3.8-pytools-2021.2.8

directory tree listing: https://gist.github.com/b19cf5e83f3009770607718b161f1151

du listing: https://gist.github.com/1970c4bb46e726b5e21f5db6b7cb97e0

Rebuild report (if merged into master) (click to expand)

74 total rebuild path(s)

19 package rebuild(s)

19 x86_64-linux rebuild(s)
19 i686-linux rebuild(s)
18 x86_64-darwin rebuild(s)
18 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
python38Packages.cgen
python38Packages.genpy
python38Packages.loo-py
python38Packages.pycuda
python38Packages.pymbolic
python38Packages.pyopencl
python38Packages.pytools
python38Packages.reikna
python38Packages.sasmodels
python39Packages.cgen
python39Packages.genpy
python39Packages.loo-py
python39Packages.pycuda
python39Packages.pymbolic
python39Packages.pyopencl
python39Packages.pytools
python39Packages.reikna
python39Packages.sasmodels
xpraWithNvenc

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/ipirilf2b2w4jffn8kw417wmvcv2fwa7-python3.8-pytools-2021.2.8 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python38Packages.pytools https://github.com/r-ryantm/nixpkgs/archive/1bdde2bfb91fa67cc6ff4520a756b494123b68dd.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ipirilf2b2w4jffn8kw417wmvcv2fwa7-python3.8-pytools-2021.2.8
ls -la /nix/store/ipirilf2b2w4jffn8kw417wmvcv2fwa7-python3.8-pytools-2021.2.8/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

19 packages built:

python38Packages.cgen
python38Packages.genpy
python38Packages.loo-py
python38Packages.pycuda
python38Packages.pymbolic
python38Packages.pyopencl
python38Packages.pytools
python38Packages.reikna
python38Packages.sasmodels
python39Packages.cgen
python39Packages.genpy
python39Packages.loo-py
python39Packages.pycuda
python39Packages.pymbolic
python39Packages.pyopencl
python39Packages.pytools
python39Packages.reikna
python39Packages.sasmodels
xpraWithNvenc

Maintainer pings
cc @artuuge for testing.

Result of nixpkgs-review pr 135391 at 1bdde2bf run on aarch64-linux 1

2 packages marked as broken and skipped:

python38Packages.pycuda
python39Packages.pycuda

16 packages failed to build:

python38Packages.cgen
python38Packages.genpy
python38Packages.loo-py
python38Packages.pymbolic
python38Packages.pyopencl
python38Packages.pytools
python38Packages.reikna
python38Packages.sasmodels
python39Packages.cgen
python39Packages.genpy
python39Packages.loo-py
python39Packages.pymbolic
python39Packages.pyopencl
python39Packages.pytools
python39Packages.reikna
python39Packages.sasmodels

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement. Does something look off? Please file an issue or reach out on IRC.


Cherrypick: 9a5c448a31bacc08e73fcae4636094f9b6e2be6a

Commit:

llvm/llvm-project@9a5c448a31bacc08e73fcae4636094f9b6e2be6a

Why is this important or safe?
Trivial crash fix on common code path.

Merged in llvm/llvm-project@2378a6e4f136d574c294da72f2dde0225ba05e89


Search router

Search Route
Changed

profiles.test.js
searchRouter.js
package-lock.json
package.json

New

searchModel.js

Features

new search functionality

will now accept a title, category name, zip code and address in a query to filter search results
if no zip or address is given, will default to profile's address for sorting
gives distance to item in meters

Issues

no issue currently, but with a larger item amount, pagination of results would be best

You can ignore my comments, just doing a critique with the TPLs.
BTW very nice PR description and commits. It's a nice sized PR and easy to follow.


fix: write cache ID to file

Env variables are tricky and don't work

:tada: This PR is included in version 2.3.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Tweak metadata in setup.py for pypi.

Since the setup.cfg description config proved ineffective, this PR should include deleting that file, as the only purpose of setup.cfg is to try and get the README onto pypi.

I want to leave that in there - multiple tutorials indicated that that should work, so maybe it just isn't an immediate thing?  Other projects also have better looking things in their sidebar, and I'm not sure how they do it, so I think it just might take a bit more research down the road to get that looking nice.


Storage blocks issue

All storage blocks' harvest/mining level set as 3, that at many cases different from same ore's harvest/mining level
Here is the screenshots with silver as the example

(i already checked: it isn't a hwyla's/cimtb's bug, btw)

Yeah I am aware of this!
I havent had time to test out what mining levels I should use for them yet, hence why all of them have assigned the same placeholder value of 3.
This is what I am currently working on, and I will push an update for this later this week.

This has been implemented in the upcoming 0.9.6 release. Thank you for raising this issue!


Add DateTimePicker

Project URL
https://github.com/itsmeichigo/DateTimePicker
Description
A nicer iOS UI component for picking date and time
Checklist

[x] Only one project/change is in this pull request
[x] Addition in chronological order (bottom of category)
[x] Supports iOS 8 or later
[x] Supports Swift 3
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

          1 Error

  :no_entry_sign:
  Found 2 link issues

          1 Warning

  :warning:
  Too many changes (when adding, please keep it to one project per Pull Request)

Link issues by awesome_bot

Line
Status
Link

1914
301
https://github.com/movielala/VideoSplashKit redirects tohttps://github.com/mojilala/VideoSplashKit

2158
301
https://www.reddit.com/r/iosprogramming redirects tohttps://www.reddit.com/r/iOSProgramming/

  Generated by :no_entry_sign: danger

@naeemshaikh90  The project states iOS 9 and later  (not iOS 8.0+) in its README.md.

@azawawi What do you think about this? #1198

Thanks! 😉


Troubles with uPrintSE

Hi!
I am having troubles with the uPrintSE chips. I have two main issues.

I can't decrypt and print out the information on the chip. I read out the chip via putty and tried to encrypt the information on it and it would give me an error (see image).

BUT, when I do the same thing with an uPrint chip (not SE), it works perfectly fine and it will print out the information (see image below)

So, I kinda had the feeling that this is an uPrint SE specific issue. I'm confused why it works fine with an uPrint chip but not with an SE. Anyone any ideas?

Issue: If I overwrite my uPrint SE chip, the printer will not recognize the material. It says "invalid carrier".
I used this to generate the new hex codes and then copied it on the chip.
./stratasys-cli.py eeprom --machine-type uprintse --eeprom-uid b61014d006cec1b3 --serial-number 7070.0 --material-name P430_IVR --manufacturing-lot 7070 --manufacturing-date "2001-01-01 01:01:01" --use-date "2002-02-02 02:02:02" --initial-material 40.0 --current-material 40.0 --key-fragment<PHONE_NUMBER>414141 --version 1 --signature MAYA -o three_percent.bin

Here is the UID and the information on chip:
b3 c1 ce 06 d0 14 10 b6
e7 6d 89 74 99 43 53 72 04 50 3d 84 c4 27 2b 62 5b e3 65 36 86 53 e3 c8 10 28 38 f4 45 d5 3f 0c f1 35 ac 94 bd bc c8 12 cf 48 39 87 fa 0a 6e 25 cf 48 39 87 fa 0a 6e 25 80 b8 88 20 b5 95 72 87 42 c4 00 00 00 00 c6 53 55 aa 55 47 9e e8 4e be 6d 0f 00 00 00 00 00 00 c8 3a 9a 08 26 d2 ea 84 8f bc 9d c9 01 00 00 00 53 54 52 41 54 41 53 59 53 02 34 f1 de 56 45 bd 32 46 60 31 21 a5 c8 fa

Please try to use "--signatrue STRATASYS" for the creatoin of the new chip. AFAIK I've had issues when using others.
Reading the SE: Same failure here. Maybe the ID of the SE (6B2A268B5ED3374A) is not correct. When you open the https://github.com/bvanheu/stratasys/blob/master/stratasys/machine.py you can see that it is the same as "ktype" (6B2A268B5ED3374A)

Did this ever get answered?
I'm having the same issue.

Yes, machine ID is the same for all uprint models. Please choose uprint for decrypt

have you actually confirmed this?

sorrx, I think a postet a wront info. The IDs are probably not the same.
I can't read a SE dump with the uprint machine ID. And the SE ID seems to be wrong.

So this machine id '6B2A268B5ED3374A' works for Uprint but not Uprint/SE?
I can update the code to reflect this.

No, the ID F3A91DBE6B0B2255 works for uprint(+) and Hp DesignJet.
The ID for SE(+) is not confirmed, AFAIK.

Another thin bvanheu,
how do you have found out the machine-IDs of the other printers?

256FF: they were given to me by email.
See this commit: 71320cdbdeeb08009371f5ccfebe5af85ba5d7a2

Has this issue been addressed?  I also have an SE(+), but I'm unable to decrypt the dump.
Is there anything I can do on my end to help solve this?  I could send dump files or mail physical cartridges if that makes anything easier.

Does the EEPROM UID play into the encryption?
To start, I'd like to see if I can "clone" one of my full chips to an empty one.  I know the printer keeps track of serial numbers, but I'd like to see if I can write SOMETHING to my cartridge that the printer will recognize.
Right now, I'm using @256FF 's cartridge writer (https://github.com/256FF/CartridgeWriter) and I commented out all the safety checks (like checksum validation) to see if I could write the same encrypted data to a new chip.  Currently, this does not seem to work, but I'll post updates as I go along.  Any tips or suggestions would be greatly appreciated :)

@thekakester I'll happy to help you in this task. My last uPrint chip have gone ReadOnly (forgot to refill it before printing for a customer yesterday....) And I have only uPrintSE chips available that I can't use because I can't rewrite them :/
So basically, my printer is useless right now and I need it ^^ I'll do anything including this bruteforce to advance on this SE machine code

Hey silvood,
Afaik you have my email, right? Contact me and I can send you one or two uprint chips.

From: Silvood<EMAIL_ADDRESS>Sent: Tuesday, November 28, 2017 9:35:34 AM
To: bvanheu/stratasys
Cc: Odin; Mention
Subject: Re: [bvanheu/stratasys] Troubles with uPrintSE (#52)
@thekakesterhttps://github.com/thekakester I'll happy to help you in this task. My last uPrint chip have gone ReadOnly (forgot to refill it before printing for a customer yesterday....) And I have only uPrintSE chips available that I can't use because I can't rewrite them :/
So basically, my printer is useless right now and I need it ^^ I'll do anything including this bruteforce to advance on this SE machine code
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/bvanheu/stratasys/issues/52#issuecomment-347450711, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AT5AfUHZGfw9VT8sSN9nBT7I2O5XXXT8ks5s68XVgaJpZM4OOcRB.

@256FF , thanks for the offer, but I was able to find one with a friend (stratasys resseller), he asked one of his customer to give me one ^^

Hello everyone,
I'm also interested in re-using cartridges for uprint se plus so I'll share my knowledge and maybe you'll help me in further experiments. So what everybody already knows is that uprint se uses DS2432 EEPROM that is SHA-1 encrypted and one need NDA with MaximIntegrated to get information about decrypting it. But what I did managed to do is to make a backup of EEPROM through UART interface of the printer and also successfully restore it to the same EEPROM but after some changes(printed 1% out of it). Though 3d printer recognized it as corrupted after I restored previous backup. After I restored actual backup printer recognized it again, which means that uprintse saves eeproms he worked with on HDD too. So the next logical step is to get to HDD, create backup of it and try to find where it keeps data about eeproms. The problem is that I haven't found any videos or manuals how to open uprintse printer and where to search for HDD. Does anyone have teardown experience?

Nordeal, the hard drive is in the rear electronics bay.  Remove the top panel (3 screws on the top rear), then the side panels (3 screws each, also on the rear), remove the filament tubes from the Y-connectors, then remove the 3 screws at the top of the electronics bay (bottom panel on the rear).  The electronics bay door should swing down, and you should be able to see the drive in the upper left of the bay.
I believe the EEPROM data is stored in a file called system.dat, and I seem to recall that it keeps a second copy somewhere else on the system that also needs to be cleared out if you want to re-use EEPROMs.

HaveBlueXST, thank you for quick answer. With your help I managed to find HDD, clone it and make some changes. I removed system.dat and set rc.local to remove this file on each start. Saved eeprom state, printed 1%, returned saved state, restarted printer and it worked! The printer said he has 1% more material as before restart. So I've left it to print a big part overnight. In the morning there was 38% material left. Backed up current eeprom state, written previous, restarted printer and on the restart the printer did something he never did before - unloaded material and after that it said that material bay is empty, eeprom read-only, couldn't even return it to previous state, I also tried to get a bit more information from printer using logs, but it didn't create log files neither in /var/log nor in /etc. Attempt to start sshd also failed, though I added "/etc/init.d/sshd start" to rc.local and access to ssh port in iptables

Hi,
I have Uprint SE.  Can you help me how to do it ? I want to hack eeprom chips and i don't know how ?

Hi,
There are two options for you

Edit the Linux of your printer to delete the cache files where the eeprom ID's are stored at startup. Dump the full eeprom of your chips. Make sure you don't print lower than 2%. Rewrite the eeprom and reboot. The chip is full again.

Convert the printer to a non SE one. Than there is a program from silvood to flash the eeprom through rs232 with a new serial number so that you don't need to clear the flash file.

Kind regards
Von: altug19
Gesendet: Dienstag, 28. Mai, 11:36
Betreff: Re: [bvanheu/stratatools] Troubles with uPrintSE (#52)
An: bvanheu/stratatools
Cc: Odin, Mention
Hi,
I have Uprint SE.  Can you help me how to do it ? I want to hack eeprom chips and i don't know how ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/bvanheu/stratatools/issues/52?email_source=notifications&email_token=AE7EA7O5SUWL5LCE5PS3RRLPXT4KDA5CNFSM4DRZYRA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWLRVRQ#issuecomment-496442054, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AE7EA7JB42P5XVSASEH3M7TPXT4KDANCNFSM4DRZYRAQ.

Hi, I have same issue as @altug19 , and I have a question.
I have uprint SE, not uprint.
If I could get the uprint chip, I can rewrite the uprint chip and use the chip for uprint SE?
Best regards,

I'll close this bug because the following ID can be used for UprintSE 09FBD4B61FC0B327.
See https://github.com/bvanheu/stratatools/commit/41accb5cc35dfcbb94298694bc5f60aaaa82307c

Hey silvood, Afaik you have my email, right? Contact me and I can send you one or two uprint chips.
…
________________________________ From: Silvood<EMAIL_ADDRESS>Sent: Tuesday, November 28, 2017 9:35:34 AM To: bvanheu/stratasys Cc: Odin; Mention Subject: Re: [bvanheu/stratasys] Troubles with uPrintSE (#52) @thekakesterhttps://github.com/thekakester I'll happy to help you in this task. My last uPrint chip have gone ReadOnly (forgot to refill it before printing for a customer yesterday....) And I have only uPrintSE chips available that I can't use because I can't rewrite them :/ So basically, my printer is useless right now and I need it ^^ I'll do anything including this bruteforce to advance on this SE machine code — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub<#52 (comment)>, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AT5AfUHZGfw9VT8sSN9nBT7I2O5XXXT8ks5s68XVgaJpZM4OOcRB.

Hi Oden,
My uprint chips also went read only... would you be able to help? :) Not sure what to do.

hi, I have Uprint SE andi used cartridge writer program from riverfrontam. I used on model filament and worked well. Then , my support material was at %1 percent and i used read button and then pushed decrypt button and filled the support material but i forgot to change serial number part. Then after  i load the support printer said cartridge is empty. I think machine has logs for serial numbers and didn't accept the %100 support material after changing. Do you have any suggestions for me ?

hi, I have Uprint SE andi used cartridge writer program from riverfrontam. I used on model filament and worked well. Then , my support material was at %1 percent and i used read button and then pushed decrypt button and filled the support material but i forgot to change serial number part. Then after i load the support printer said cartridge is empty. I think machine has logs for serial numbers and didn't accept the %100 support material after changing. Do you have any suggestions for me ?

Did you solved your problem? If not, please send me your crypted data and I will send the new data back to you.

hi, I have Uprint SE andi used cartridge writer program from riverfrontam. I used on model filament and worked well. Then , my support material was at %1 percent and i used read button and then pushed decrypt button and filled the support material but i forgot to change serial number part. Then after i load the support printer said cartridge is empty. I think machine has logs for serial numbers and didn't accept the %100 support material after changing. Do you have any suggestions for me ?

Did you solved your problem? If not, please send me your crypted data and I will send the new data back to you.

**hi, i faced the same problem. Machine logs the serial number for sure. Please watch this video and follow the instructions; https://www.youtube.com/watch?v=_YLUCKgAaJk&ab_channel=ScrapTechTips.
you should back up the cartridge filament datas before the procedure. you have two possibilities in front of you. One is eeprom on your support filament is on read only mode or not. I have my old cartridge data for the support material for back up purpose. I can send the datas for you and you can write the data on your own eeprom. You see the procedure on the video how you should do that. The datas on my other computer please write your email here and i will send the datas for your testing.      **

hi, I have Uprint SE andi used cartridge writer program from riverfrontam. I used on model filament and worked well. Then , my support material was at %1 percent and i used read button and then pushed decrypt button and filled the support material but i forgot to change serial number part. Then after i load the support printer said cartridge is empty. I think machine has logs for serial numbers and didn't accept the %100 support material after changing. Do you have any suggestions for me ?

Did you solved your problem? If not, please send me your crypted data and I will send the new data back to you.

yes i solved the problem thank you very much


[Flogo Web UI] Activity list not updated after installing activity

Current behavior (how does the issue manifest): After installing an activity using the "install new activity" button, the list isn't automatically refreshed, and the only way to see the new activity is by refreshing the browser window (or exiting the app)
Expected behavior: Installing a new activity should refresh the list automatically
Minimal steps to reproduce the problem (not required if feature enhancement):

Create a new app (or open an existing one)
Install a new activity from GitHub using the "install new activity" button

Please tell us about your environment (Operating system, docker version, browser & web ui version, etc): Chrome browser (Version 69.0.3497.100 (Official Build) (64-bit)), Docker (18.06.1-ce), Flogo Web UI (0.5.6)
Flogo version (CLI & contrib/lib. If unknown, leave empty or state unknown): 0.5.6

@retgits We noticed the same issue and fixed it in v0.5.6. Could you please share the activity contribution url which you are trying to install?

It happens for any activity that I want to install, and I've tried about 20 of them (you can almost pick any of the ones I have in my GitHub repository: https://github.com/retgits/flogo-components)

Just to give an update for people seeing the same issue. Thanks to a debug session with @apandurangi and @fcastill, we found a workaround for this issue. In the dialog window where you paste the URL, simply remove the https:// part (so make it a proper go import statement, like github.com/retgits/flogo-components/activity/null) and it will work.

Fixed in v0.5.7


New rule: Extra properties lead to lying types

This is unsettling:
type Foo = { readonly foo: string };
type Bar = { readonly foo: string, readonly bar?: () => unknown };
// This won't compile
// const foo: Foo = { foo: "foo", bar: "bar" };
// But this will
const thing = { foo: "foo", bar: "bar" };
const foo: Foo = thing;
// Uh oh...
const bar: Bar = foo;
if (bar.bar !== undefined) {
    // Runtime explosion
    // [ERR]: Executed JavaScript Failed:
    // [ERR]: bar.bar is not a function
    bar.bar();
}

It’s hard to point at precisely which line is “wrong”.
We may want a new rule that forbids:
const foo: Foo = thing;

We can think of it as taking the tsc error reported by
const foo: Foo = { foo: "foo", bar: "bar" };

and expanding its scope beyond just object literals.

Or is
const bar: Bar = foo;

the thing we want to flag as an error?
If the destination type has an optional property and the source type has no such property (optional or otherwise), there is no way for us to know whether an incompatible property with the same name exists at runtime.
And, importantly, you can't just blame this on unsafe type assertions, or incorrect *.d.ts typings or any shenanigans like that.


ionic 3 ion select with search bar

how to add search bar in ion select in ionic 3 such as how to search country in list of country in ion select

Hello! Thanks for opening an issue with us! As this seems like more of a support question I will urge that you ask this question on our forum or on our slack channel. Thanks for using Ionic!