Bluez and dbus dependency on Mac OS / Crossplatform Ideas

Here you are using blurz, inside dependencies, which has the dependency of dbus-1, not being included in Mac OS ( 10.14.6 )
....., output: Output { status: ExitStatus(ExitStatus(256)), stdout: "", stderr: "Package dbus-1 was not found in the pkg-config search path.\nPerhaps you should add the directory containing `dbus-1.pc\'\nto the PKG_CONFIG_PATH environment variable\nNo package \'dbus-1\' found\nPackage dbus-1 was not found in the pkg-config search path.\n.......

warning: build failed, waiting for other jobs to finish...
error: build failed

https://github.com/iot2tangle/Streams-ble-gateway/blob/4169b54adf320206ad5d1b974dffd553b5f55607/Cargo.lock#L217
https://github.com/iot2tangle/Streams-ble-gateway/blob/4169b54adf320206ad5d1b974dffd553b5f55607/Cargo.toml#L20
"Fix" of that issue was just running brew install dbus
followed by brew services start dbus to avoid following error:
dbus[11416]: Dynamic session lookup supported but failed: launchd did not provide a socket path, verify that org.freedesktop.dbus-session.plist is loaded!
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: D-Bus error

And even after that, there were further errors running the GW on mac - as i had predicted.
Makes sense, because dbus and especially bluez for bluetooth connections is for Linux kernel BLE communication only.
See here:
https://stackoverflow.com/a/63136987
❯ brew services start dbus
==> Successfully started `dbus` (label: org.freedesktop.dbus-session)
❯ cargo run --release --bin scan
    Finished release [optimized] target(s) in 0.47s
     Running `target/release/scan`
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: D-Bus error: Failed to connect to socket /usr/local/var/run/dbus/system_bus_socket: No such file or directory (org.freedesktop.DBus.Error.FileNotFound)', bin/scan.rs:10:59

To scale and being able to run the Gateway crossplatform, we have multiple options.

Detect running OS at runtime, switching the library to be used ( will fail, as compilation will fail as described above )
Remove the dependency of blurz completely, and using another RUST BLE implementation, that coveres those cases
( On OSX, you have to use the native interface CoreBluetooth | IOBluetooth | bluetoothd  )

Checking the crates for CoreBluetooth https://crates.io/keywords/CoreBluetooth gives us 3 options, with one being very active in development, worth a look at: https://crates.io/crates/btleplug
Quote:
btleplug is a Rust BLE library, support Windows 10, macOS, Linux, and possibly iOS. It is currently made up of parts of other abandoned projects with a goal of building a fully cross platform proof of concept.

Our goal is the bring in some of the outstanding PRs from other projects, expand the platform support, and possibly make the API surface more ergonomic for being a truly cross-platform library.

That btleplug library seems promising, having support for windows, android, mac os, linux, even iOS but has some limitations for each platforms especially async:

Another options would by some python wrapper accessiong Mac OS BLE, parsing output of blueutil for mac os (https://github.com/toy/blueutil) or like noble library for node.js solved, connecting via XpcConnection to the native Mac OS bluetoothd daemon service.
https://github.com/noble/noble/blob/master/lib/mac/highsierra.js#L18
--- or creating platform specific repos / build configs ála make and ./configure.
Just giving some feedback and ideas to solve those issues, to be more flexible to more endusers.
Maybe someone knows right away, which BLE functions are essential, and check that embedded image, if it would be an option trying to replace blurz with btleplug in some development branch.

Looking at this repo, it is not usable as is, but i like the abstraction of a generic interface, so the high-level code does not need to care about the underlaying implementation. Thinking further, looking at the core lib for streams, we could also create some core library abstracted, to have one unified BLE lib working crossplatform.
https://github.com/iot2tangle/Streams-ble-gateway/blob/4169b54adf320206ad5d1b974dffd553b5f55607/Cargo.toml#L16


Controls not working

video controls are not working.
Are they disabled? if yes how to enable them?

Hi can you give me more details? or demo page?

closing this issue since no feedback as been provided


chore: fingerlie-test2 to 0.0.1

chore: Promote fingerlie-test2 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


[elastalert-server]Supports SSL connection of elasticsearch

Please refer to the correspondence of johnsusek / elastalert-server.
https://github.com/johnsusek/elastalert-server/commit/22acdb8067e33ae91f19125f54fe65fd881aa17a
https://github.com/johnsusek/elastalert-server/commit/34b856acf6f2de2b660ff58e3f473e481a9ad25f
https://github.com/johnsusek/elastalert-server/commit/82b1c1375bab5c71fa0d1de5bd14e97be583f762
https://github.com/johnsusek/elastalert-server/commit/1aa9a64d458be2cd3d256b3ed1713ec0b0c2db0e#diff-215965a40aa0cceb2ab7a5a2e8ca76489b7d347a77d9531b33e7ecc1c436059c

elastalert itself supports ssl connection for elastic serach, but elatalert-server does not support ssl connection when connecting to elastic serach.

https://github.com/Karql/elastalert/issues/13


Additional steps to Manual installation of the dependencies of EOS

Hello to all EOS enthusiast, if you try to manually install the dependencies of @eosio with a clean install Ubuntu 16.10, while following the instructions of the Manual installation of the dependencies you will get these errors:

Thanks to @deaddy and his article: First Install & Run of EOS on a Blank Ubuntu 16 we just need to a a few steps before the sudo apt-get update such as:

sudo nano /etc/apt/sources.list

Taken from this address: http://releases.llvm.org/download.html add these lines:
deb http://apt.llvm.org/xenial/ llvm-toolchain-xenial-4.0 main
deb-src http://apt.llvm.org/xenial/ llvm-toolchain-xenial-4.0 main

Save and Exit

Then after the addition, success!

And that's my suggestion for improvement.  Let's make EOS the best it can be.
Posted on Utopian.io -  Rewarding Open Source Contributors

yes that seems like the issue I was experiencing, I wish I knew at the time.
Though I just used the 17.10 and all worked fine there (as clang/cmake defaults are just right), lot less issues on that side (and all built, running, working fine).


Confusing analysis warning for cloud provider namespaces

When I run istioctl x analyze --use-kube using my own build of HEAD I get three warnings:
Warn [IST0102](Namespace/ibm-cert-store) The namespace is not enabled for Istio injection. Run 'kubectl label namespace ibm-cert-store istio-injection=enabled' to enable it, or 'kubectl label namespace ibm-cert-store istio-injection=disabled' to explicitly mark it as not needing injection
Warn [IST0102](Namespace/ibm-observe) The namespace is not enabled for Istio injection. Run 'kubectl label namespace ibm-observe istio-injection=enabled' to enable it, or 'kubectl label namespace ibm-observe istio-injection=disabled' to explicitly mark it as not needing injection
Warn [IST0102](Namespace/ibm-system) The namespace is not enabled for Istio injection. Run 'kubectl label namespace ibm-system istio-injection=enabled' to enable it, or 'kubectl label namespace ibm-system istio-injection=disabled' to explicitly mark it as not needing injection

I wondered why I saw these for ibm- namespaces, but not other namespaces, and found galley/pkg/config/analysis/analyzers/injection/injection.go and found Galley considers kube- and istio- system namespaces with a TODO to make this configurable.
I ask that ibm- be added to that list.  Even better: make this configurable as promised so that other cloud providers don't also get these warnings.

cc @davidebbo

It's a tricky problem as there is probably no reliable way to determine whether something is a "system namespace".
So yes, making it configurable would be good, although at the same time we want it to have good defaults. i.e. I wouldn't want everyone to have to specify that kube-system should be excluded.
Short term, I'm good with just adding "ibm-" to the excluded prefix list, until we come up with a proper way of making this configurable.
/cc @sushicw

https://github.com/istio/istio/pull/17869 should greatly mitigate this, since it modifies istioctl analyze to analyze one namespace at a time.
We can probably update the injection analyzer to remove those hardcoded exceptions now.

Leaving this issue open for now to track the need to generally ignore system namespaces, even when we want to analyze all namespaces.


[SPARK-25313][SQL][FOLLOW-UP][BACKPORT-2.3] Fix InsertIntoHiveDirCommand output schema in Parquet issue

What changes were proposed in this pull request?
Backport https://github.com/apache/spark/pull/22359 to branch-2.3.
How was this patch tested?
unit tests

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/testing-k8s-prb-make-spark-distribution-unified/2993/
Test PASSed.

Test build #95910 has started for PR 22387 at commit a7b857c.

LGTM

Test build #95910 has finished for PR 22387 at commit a7b857c.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/95910/
Test FAILed.

Test build #95916 has finished for PR 22387 at commit a7b857c.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/95916/
Test FAILed.

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/testing-k8s-prb-make-spark-distribution-unified/3001/
Test PASSed.

Test build #95921 has started for PR 22387 at commit a7b857c.

Test build #95921 has finished for PR 22387 at commit a7b857c.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/95921/
Test FAILed.

retest this please

Test build #95925 has finished for PR 22387 at commit a7b857c.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/95925/
Test FAILed.

retest 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/testing-k8s-prb-make-spark-distribution-unified/3009/
Test PASSed.

Test build #95933 has started for PR 22387 at commit a7b857c.

Merged to branch-2.3.


Circle ci orb

Turns out, there is a CircleCI Orb that does at least as much as Travis did: https://circleci.com/orbs/registry/orb/broadinstitute/bits-puppet
I haven't tested the publish part, for obvious reason. That's one less CI tool to learn and master...

Possible improvement include testing for puppet 6 in parallel. But there are no known breaking syntax changes, and Debian 10 comes wih puppet 5 anyway. So... low priority


Update deploying-bots.md

Updated documentation for deploying bots in junction with https://github.com/zulip/python-zulip-api/pull/832

Thanks! Please go through our guide on submitting a pull request to learn how to present your proposed changes to Zulip, so that your work can be reviewed.

updated.

Unfortunately, this PR still does not meet the guidelines linked above.

@mpagler can you resubmit this with a new commit message? The change is helpful, but it looks like you've deleted your fork, so I can't reopen it.

oh? i deleted it, because i thought it was because of "main branch".
I try to get to it this week.

opened new request... hopefully now correct (only via github interface)
https://github.com/zulip/zulip/pull/32793


[test] Fix metric name in references for CPMD and VASP checks

Codecov Report

Merging #2496 (93b8f80) into master (75e07b4) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2496   +/-   ##
=======================================
  Coverage   85.87%   85.87%
=======================================
  Files          58       58
  Lines       10775    10775
=======================================
  Hits         9253     9253
  Misses       1522     1522

Continue to review full report at Codecov.

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


Some more server protocols

Hi,
maybe you want to add the following protocols to the README for EnemyTerritory / RtCW master server:
14 RTCWCOOP
57 RTCW 1.0
60 RTCW 1.4x
61 ioRTCW
82 ET 3.00 (inofficial)
83 ET 2.56
84 ET 2.60(b)
Cheers

Thanks for the list of protocols but I think I will keep only the latest main protocol for each game in the main readme.
If you want you could add wiki article with this list.


Do not call update() for ProgressNofities

 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.


Performance optimization for AnalyzeSolutionGenerator

Description

WorkspaceBuilderHelper.BuildIncremental - if a project reference was already built and added to a workspace, use the existing ProjectAnalysisResult instead of building again.

Supplemental testing
BenchmarkDotNet results for CodeAnalyzerByLanguage.AnalyzeSolutionGenerator method execution

BenchmarkDotNet=v0.13.2, OS=Windows 10 (10.0.20348.887)
Intel Xeon Platinum 8375C CPU 2.90GHz, 1 CPU, 8 logical and 4 physical cores
.NET SDK=6.0.400
  [Host]     : .NET 6.0.8 (6.0.822.36306), X64 RyuJIT AVX2
  Job-WHKMPK : .NET 6.0.8 (6.0.822.36306), X64 RyuJIT AVX2

LaunchCount=1  RunStrategy=Monitoring

Method
Mean
Error
StdDev
Ratio
RatioSD
Gen0
Gen1
Gen2
Allocated
Alloc Ratio

AnalyzeSolutionGenerator (Before changes)
141.83 s
3.001 s
1.985 s
3.55
0.05
202000.0000
89000.0000
1000.0000
5.73 GB
1.17

AnalyzeSolutionGenerator (After changes)
62.38 s
0.619 s
0.409 s
1.59
0.02
169000.0000
77000.0000
1000.0000
4.85 GB
0.99

Additional context
Benchmarking code (in my fork; not in this PR) -

Before changes
After changes

Unit tests -
10 unit tests were failing on the main branch before my changes. No additional tests are failing after my changes.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thank you for submitting this change


[Feature] support more ways of alert

Now,the alert module only support email，It is  necessary to support more ways.I strongly recommend DolphinScheduler  have a standard interface
to help users to realize various alert.the CGI is a good idea, DS callback the CGI  developed by users.

You can take a look at https://github.com/apache/incubator-dolphinscheduler/issues/3049

please see #3049


documentation does not work as explained

Documentation seems to indicate the following parameters are not required and can be blank but each requires a value if it is passed into powershell.
would it be more clear to use defaults
$AzureSSISEdition = "Standard"
$AzureSSISLicenseType = "LicenseIncluded"
for "$SetupScriptContainerSasUri" inlcude a link to URL to help set this. https://social.msdn.microsoft.com/Forums/en-US/3f20f72a-7ff6-4cf8-869c-413cc1b11a90/your-azuressis-ir-has-failed-to-start-due-to-custom-setup-failure?forum=AzureDataFactory and specify to not include the parameter if not setting. I tried using blank and $null values, Neither worked. For some reason if the parameter is not included that works.
$SetupScriptContainerSasUri = ""
btw: I am running Azure 5.1.2
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: e8473dce-d69d-625d-797a-912f3e9984ce
Version Independent ID: c00dcda3-3080-e98d-d3a4-a6cd26081304
Content: Provision the Azure-SSIS Integration Runtime with PowerShell
Content Source: articles/data-factory/tutorial-deploy-ssis-packages-azure-powershell.md
Service: data-factory
GitHub Login: @swinarko
Microsoft Alias: sawinark

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

Thanks for reporting these errors, we will fix them accordingly.

#please-close


Fixed --export-secret, set all hashsums to SKIP in PKGBUILD for git

Hey, so I just saw this. GitHub is very bad about getting emails out.
Let me test this on a few systems first. I'm fairly certain you need --export-secret-keys, which is only a stub in GPG's local storage if you're using a smartcard.

Nope - that's the reason you can store a URL on OpenPGP cards, so you can download the public key easily on a strangers computer (or if you use them as part of building security, seen exactly one company do that). Either way, using --export-secret-subkeys will export an unprotected secret key for non-smartcard users (the hook works - with some modifications - for those users too)

I ended up taking a different approach to this since gpg 2.1.15 has started behaving oddly on my system when I bought a new Yubikey NEO. It refused to allow --export-secret-keys to work. So we are now attempting to export the secret key, and if it returns nothing (egrep '.+' fails) it will use --export.
Unfortunately, they still have not deprecated the stub functionality entirely - keys will still show up in --list-secret-keys even when gpg does not in fact have the secret key on hand. This is incorrect behavior IMHO - gpg --export-secret-keys should work for anything that shows up in --list-secret-keys, but should output a public key block with some flag set indicating a smartcard stub.
Until gpg is able to easily discern between local and smartcard-hosted secret keys the behavior will remain the same for now, including (unfortunately) the storage of unnecessary subkeys.


Enable to specify the destination database and schema for artifacts

Solves https://github.com/elementary-data/elementary/issues/98
Overview of the implementation
We introduce new two dbt variables to determine the destination database and schema for artifacts, instead of using the database and schema of the first node whose package_name is elementary.

Hi @yu-iskw ! great to see your contributions, thanks so much for doing this. See my comment in the original issue. Waiting to hear from you before diving in to this PR, thanks!

@oravi Thank you for the comment. I totally misunderstood the current implementation. We might still have a point to discuss, but the PR is no longer required. So, let's close. Thank you very much.


DRILL-4531: Add a Drill customized rule for pushing filter past aggre…

…gate

+1.  Are there other rules that also may be getting applied with mixed convention (Calcite logical and Drill logical) ?  I am surprised that we don't encountered more such cases..

We did have other rules getting applied mixed convention. For DRILL-3855, I'll fix push filter/project past setop rules. I'll open another JIRA to go through all the other rules.
Even with the mixed convention, the issue reported in this jira did not always happen for every query. That's probably why we did not encountered more such cases before.

I think there is another bug here. DrillAggregateRel should assert when a none convention is passed in. If you add that assertion where do we fail? We should fix that

In general, this pattern should fail earlier. There should be an AbstractConverter between these rels. We should probably make sure that a rels assert their convention to avoid this type of bug earlier.

Adding assertion to DrillAggregateRel did not show new failure when using the default FilterAggregateTransposeRule. In the rule, copy() method is used to create new instance of aggregate. So, the aggregate created by this rule should not violate the convention check.
The patten of mixed rels does happen and it's the expected thing, since Drill's logical planning is dealing with two convention: NONE and LOGICAL. So, at any time, the RelSet would have RelSubSets for both conventions. It's normal to see a LogicalFilter whose input is a RelSubset with a DrillAggregate as a member.
The problem is not we should not have such mixed patten. In stead, the problem is that the Rule should not match such pattern. If the rule matches such mixed Rel, then looks like it could cause problem. With the default rule, the trace of running 50 seconds shows FilterAggregateTransposeRule is fired abnormally high (it could be even high if it does not timeout)
1055 [FilterAggregateTransposeRule]
45 [DrillPushProjectPastFilterRule]
38 [ProjectMergeRule:force_mode]
32 [DrillReduceExpressionsRule(Filter)]
30 [ReduceExpressionsRule_Project]
30 [DrillProjectRule]
26 [DrillAggregateRule]
If we add the restriction, then the new rule was just fired coupled of times.

Your expectation about mixed conventions is inconsistent with my own. My expectation: a RelSubset should only have a single convention (in fact it should have only a single trait definition and all children should have satisfying trait definitions). I would expect the following:
(1) Prels should only have PHYSICAL
(2) Drels should only have LOGICAL
(3) Calcite Logical Rels should only have NONE
My previous statement mistook the convention of the DrillAggregateRel in the below. I thought it said NONE.
[rel#217:LogicalFilter.NONE.ANY([]).[], rel#6972:DrillAggregateRel.LOGICAL.ANY([]).]
It seems like the actual problem is that LogicalFilter is allowing an input of LOGICAL. I don't believe that LOGICAL should satisfy NONE. It seems that LogicalFilter should assert that its child is of Convention NONE. (In other words, a LogicalFilter shouldn't have Convention transformation properties.) Otherwise, we'll see a large number of invalid (but infinite) plans.

I agree that your expectation for RelSubset makes sense to me. However, for now it does not happen that way. The following is the trace for the query which went through planning with this customized rule (I removed some rels).
Set#1, type: RecordType(BIGINT custkey, ANY custAddress)
...
Set#2, type: (DrillRecordRow[, l_orderkey, l_partkey, l_linenumber])
rel#273:Subset#2.LOGICAL.ANY([]).[], best=rel#442, importance=0.31381059609000006
rel#275:AbstractConverter.LOGICAL.ANY([])., rowcount=100.0, cumulative cost={inf}
rel#442:DrillScanRel.LOGICAL.ANY([]).[](table=[cp, tpch/lineitem.parquet],groupscan=ParquetGroupScan [entries=[ReadEntryWithPath [path=classpath:/tpch/lineitem.parquet]], selectionRoot=classpath:/tpch/lineitem.parquet, numFiles=1, usedMetadataFile=false, columns=[*]]), rowcount=60175.0, cumulative cost={60175.0 rows, 6.0175E8 cpu, 0.0 io, 0.0 network, 0.0 memory}
Set#3, type: (DrillRecordRow[, l_orderkey, l_partkey, l_linenumber])
rel#82:Subset#3.NONE.ANY([]).[], best=null, importance=0.3874204890000001
rel#345:LogicalFilter.NONE.ANY([]).[[]](input=rel#273:Subset#2.LOGICAL.ANY([]).[],condition=AND(>=($1, 20160101), <=($2, 20160301), OR(=($2, 1), =($2, 2), =($2, 5), =($2, 6)))), rowcount=6.25, cumulative cost={inf}
rel#345:LogicalFilter has a child rel#273 with LOGICAL convention.
As another example, for the following query:
Select n_name, n_nationkey from cp.tpch/nation.parquet where n_nationkey > 5
The trace:
Set#0, type: (DrillRecordRow[, n_nationkey, n_name])
...
Set#1, type: (DrillRecordRow[, n_nationkey, n_name])
rel#21:Subset#1.NONE.ANY([]).[], best=null, importance=0.81
rel#20:LogicalFilter.NONE.ANY([]).[](input=rel#19:Subset#0.ENUMERABLE.ANY([]).[],condition=>($1, 5)), rowcount=50.0, cumulative cost={inf}
rel#37:LogicalFilter.NONE.ANY([]).[[]](input=rel#19:Subset#0.ENUMERABLE.ANY([]).[],condition=>($1, 5)), rowcount=50.0, cumulative cost={inf}
rel#35:Subset#1.LOGICAL.ANY([]).[], best=rel#60, importance=0.81
rel#60:DrillFilterRel.LOGICAL.ANY([]).[](input=rel#57:Subset#0.LOGICAL.ANY([]).[],condition=>($1, 5)), rowcount=50.0, cumulative cost={125.0 rows, 250600.0 cpu, 0.0 io, 0.0 network, 0.0 memory}
Set#2, type: RecordType(ANY n_name, ANY n_nationkey)
rel#23:Subset#2.NONE.ANY([]).[], best=null, importance=0.9
rel#53:LogicalProject.NONE.ANY([])., rowcount=50.0, cumulative cost={inf}
Again, rel#53:LogicalProject has a child rel#35 whose convention is LOGICAL.
I think the reason that we have such mixed rels is we have different kinds of rules, used in a single Volcano planning phase.

Rule matchs base class Filter/Project, etc only.
Rule matches LogicalFilter/LogicalProject, etc
Rule uses copy() method to generate a new Rel
Rule  uses RelFactory to generate a new Rel.
convent rule, which convert from Calcite logical (NONE/Enumerable) to Drill logical (LOGICAL)

For instance, ProjectMergeRule, which matches base Project, yet uses default RelFactory, will match both LogicalProject and DrillProject, but produce LogicalProject as outcome. That will cause the mixed rels.
2 things we may consider to fix this:

Separate the convent rules from the other transformation rules. Apply convert rule first, then transformation rule match DrillLogical only. That's similar to what other system (hive) is doing.
go through every rule we use, and we need make sure the convention of input and ouptput of a transformation rule should be same, except for the convert rule.

The above 2 things would take some considerably effort, though.

Yeah, you're right and thanks for reviewing fully. We can shelve for now but this definitely seems like something we should get to.
+1 on the patch.

Thanks for the discussion, @jacques-n . I opened DRILL-4541 to track the issues we discussed. I'll work on it and see if I can find a solution.


did sems portal data change?

Hi, I know this isn't maintained anymore, but maybe someone else is experiencing the same problem.
As of April 1st 2023 I am not getting any uploads to pvoutput.org anymore.
Checking the log is see lots of these errors:
May 02 11:50:14 odroid gw2pvo[22388]: INFO     Normal, 2810.0 W now, 6.9 kWh today, 34268.1 kWh all time, 246.5 V grid, 561.6 V PV
May 02 11:50:19 odroid gw2pvo[22388]: ERROR    Expecting value: line 2 column 1 (char 1)
May 02 11:55:17 odroid gw2pvo[22388]: INFO     Normal, 2815.0 W now, 7.2 kWh today, 34268.4 kWh all time, 246.8 V grid, 561.2 V PV
May 02 11:55:23 odroid gw2pvo[22388]: ERROR    Expecting value: line 2 column 1 (char 1)

It seems to be reading data correctly, but then something fails.
Did GoodWe change anything in the data that is expected by this script?
Maybe anyone with more python knowledge than myself knows what to do here.
Cheers

It is still working for me.

May 02 13:26:26 lplex gw2pvo[1020]: INFO     Normal, 807.0 W now, 4.1 kWh today, 10592.4 kWh all time, 238.4 V grid, 558.6 V PV
May 02 13:31:26 lplex gw2pvo[1020]: INFO     Normal, 994.0 W now, 4.2 kWh today, 10592.5 kWh all time, 238.2 V grid, 572.9 V PV
May 02 13:36:26 lplex gw2pvo[1020]: INFO     Normal, 1048.0 W now, 4.3 kWh today, 10592.6 kWh all time, 238.9 V grid, 572.3 V PV
May 02 13:41:26 lplex gw2pvo[1020]: INFO     Normal, 1603.0 W now, 4.3 kWh today, 10592.6 kWh all time, 240.6 V grid, 574.1 V PV
May 02 13:46:27 lplex gw2pvo[1020]: INFO     Normal, 1017.0 W now, 4.5 kWh today, 10592.8 kWh all time, 237.9 V grid, 557.6 V PV
May 02 13:51:26 lplex gw2pvo[1020]: INFO     Normal, 760.0 W now, 4.5 kWh today, 10592.8 kWh all time, 233.4 V grid, 533.6 V PV
May 02 13:56:26 lplex gw2pvo[1020]: INFO     Normal, 843.0 W now, 4.6 kWh today, 10592.9 kWh all time, 236.5 V grid, 561.8 V PV
May 02 14:01:26 lplex gw2pvo[1020]: INFO     Normal, 419.0 W now, 4.7 kWh today, 10593.0 kWh all time, 235.3 V grid, 537.9 V PV
May 02 14:06:26 lplex gw2pvo[1020]: INFO     Normal, 1332.0 W now, 4.7 kWh today, 10593.0 kWh all time, 237.2 V grid, 581.5 V PV
May 02 14:11:26 lplex gw2pvo[1020]: INFO     Normal, 361.0 W now, 4.8 kWh today, 10593.1 kWh all time, 235.3 V grid, 524.4 V PV

It's working for me also
On Tue, May 2, 2023, 1:45 PM Geoff Lamb @.***> wrote:

May 02 13:26:26 lplex gw2pvo[1020]: INFO Normal, 807.0 W now, 4.1 kWh
today, 10592.4 kWh all time, 238.4 V grid, 558.6 V PV
May 02 13:31:26 lplex gw2pvo[1020]: INFO Normal, 994.0 W now, 4.2 kWh
today, 10592.5 kWh all time, 238.2 V grid, 572.9 V PV
May 02 13:36:26 lplex gw2pvo[1020]: INFO Normal, 1048.0 W now, 4.3 kWh
today, 10592.6 kWh all time, 238.9 V grid, 572.3 V PV
May 02 13:41:26 lplex gw2pvo[1020]: INFO Normal, 1603.0 W now, 4.3 kWh
today, 10592.6 kWh all time, 240.6 V grid, 574.1 V PV
May 02 13:46:27 lplex gw2pvo[1020]: INFO Normal, 1017.0 W now, 4.5 kWh
today, 10592.8 kWh all time, 237.9 V grid, 557.6 V PV
May 02 13:51:26 lplex gw2pvo[1020]: INFO Normal, 760.0 W now, 4.5 kWh
today, 10592.8 kWh all time, 233.4 V grid, 533.6 V PV
May 02 13:56:26 lplex gw2pvo[1020]: INFO Normal, 843.0 W now, 4.6 kWh
today, 10592.9 kWh all time, 236.5 V grid, 561.8 V PV
May 02 14:01:26 lplex gw2pvo[1020]: INFO Normal, 419.0 W now, 4.7 kWh
today, 10593.0 kWh all time, 235.3 V grid, 537.9 V PV
May 02 14:06:26 lplex gw2pvo[1020]: INFO Normal, 1332.0 W now, 4.7 kWh
today, 10593.0 kWh all time, 237.2 V grid, 581.5 V PV
May 02 14:11:26 lplex gw2pvo[1020]: INFO Normal, 361.0 W now, 4.8 kWh
today, 10593.1 kWh all time, 235.3 V grid, 524.4 V PV
—
Reply to this email directly, view it on GitHub
https://github.com/markruys/gw2pvo/issues/73#issuecomment-1530850505,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACQUEIVT2ERXFCNRB35Z633XECC75ANCNFSM6AAAAAAXSQBBMU
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

hmmm, that's odd...
restarted but still seeing the same behaviour.

Got it, forgot to also update the new api key in the service file and removing the darksky api.


[FLOC-4006] Make sure old files don't pollute upload documentation.

https://clusterhq.atlassian.net/browse/FLOC-4006

:heavy_check_mark: thanks for improving this. It looks to work fine, and the published docs for the branch look unchanged.
Given that this is green, I'm going to merge it and watch Jenkins.


Not getting any pushes on my Mac desktop anymore

I have pushbullet app on my phone and sidneys pb-for-dekstop on my Mac and my Ubuntu desktop.
For some reason I no longer receiving any notifications on my MacBook installation. Tried resetting the app configuration, reinstalling from scratch, muting/unmuting.
Unfortunately nothing works.
Embedded test notification works fine, so it's not the notification permission problem. But I don't get any synced notifications from my phone and the ones I intentionally send from my phone to the Mac.
Could anyone pls advise where else to look to dig deeper?
App Version
v8.9.5
Installation Type
Setup
Operating System
Mac OS Mojave 10.14.3

Upgraded to 8.10.1 - the issue is still there.
I also can confirm, that I can send notifications from pb-for-dekstop to mobile client, SMS mirroring also works. Just not getting notifications on Mac desktop.

I also just received notification from connected IFTT service on my Mac...

Same

I'm also dealing with this same issue. I'm on version 8.12 and I've repeatedly resent configuration and reconnected to my android phone by no dice. Running Mojave.

Same here on Ubuntu 18.10

Not getting notifications on Windows 10 either

The issue still exists.
I can send pushes from Mac to other devices (Android, iOS, Windows) but Mac can't receive any.
Have tried reconnecting to PushBullet, reset configuration and reinstall but to no avail.

Same here Ubuntu 19.04

Same issue on ElementryOS

Same on Pop OS 19.04

Still seems to be an issue in v9.0.9.

All right, so the issue seems to come from generateNotificationImage() method of push.js.
The method seems to be trying to generate an icon for the notification from various image sources (chat url, sender account image, etc..) and returns whichever gives back a valid image first in a fallback sequence.
However, when it's trying to generate the chat icon, it looks for the open chat room by email with window.pb.targets.by_email(). This returns a null object, as a chat room by this email doesn't exist.
It then tries to get the image_url property of our null object, which then throws an error, stopping the notification from showing all together.
Relevant code:
https://github.com/sidneys/pb-for-desktop/blob/6f076b4999a29059d921568ff584c11008852c72/app/scripts/renderer/pushbullet/push.js#L285-L288

@btaens,
Thanks for the commit! Unfortunately though, it didn't fix the issue for me. I changed the code to use your commit and re-built it for MacOS, but unfortunately, it wasn't the magic bullet :(

Not only MacOS related (please change issue title and labels)
Also don't get any notifications on my ARCH Linux (Manjaro)

Update: The fix from @btaens help's me on ARCH Linux. Using the Android app and typing something in "push", either directed to all or just my PC, will create a notification on linux.
But using the test notification button in the Android app doesn't create a notification on linux.
Instead I'm receiving a notification on android.
Debugging the push.js turns out that testIfPushIsIgnored is returning true, because the push object doesn't contains a active property.
https://github.com/sidneys/pb-for-desktop/blob/f042bbbaed8c6fcdd12ec86b6f46fa237a2a920c/app/scripts/renderer/pushbullet/push.js#L832-L834
Either the android app creates an invalid structure, or the electron app handles the test notification not properly.
Maybe @sidneys can help identifying the problem?
Some self-crafted notification, which works with https://github.com/sidneys/pb-for-desktop/pull/107
{
  "active": true,
  "iden": "xxx",
  "created":<PHONE_NUMBER>.72517,
  "modified":<PHONE_NUMBER>.758848,
  "type": "note",
  "dismissed": false,
  "guid": "5c61133c-af06-4a78-ab55-ad1008fa47b4",
  "direction": "self",
  "sender_iden": "xxx",
  "sender_email": "xxx",
  "sender_email_normalized": "xxx",
  "sender_name": "GodLesZ Blubb",
  "receiver_iden": "xxx",
  "receiver_email": "xxx",
  "receiver_email_normalized": "xxx",
  "source_device_iden": "xxx",
  "awake_app_guids": [
    "xxx"
  ],
  "body": "Test an alle"
}

And here a test notification using the Android app "test" button in the "mirror" tab, which triggers isIgnored
{
  "actions": [
    {
      "label": "Fertig",
      "trigger_key": "com.pushbullet.android_3_null_Fertig"
    }
  ],
  "application_name": "Pushbullet",
  "body": "Wenn Sie dies an Ihrem PC sehen, funktioniert die Benachrichtigungsweiterleitung!",
  "client_version": 349,
  "dismissible": true,
  "icon": "some valid base64 hash",
  "notification_id": "3",
  "package_name": "com.pushbullet.android"
  "source_device_iden": "xxx",
  "source_user_iden": "xxx",
  "title": "Testbenachrichtigung",
  "type": "mirror"
}

Aaaand during writing this answer, here his again a notification from an Android game app, which triggers isIgnored with true and is therefore not shown on my linux:
{
  "application_name": "Jurassic Tribes",
  "body": "Eine Sammeltruppe ist ist zurückgekehrt",
  "client_version": 349,
  "dismissible": true,
  "icon": "some valid base64 hash",
  "notification_id": "-261202488",
  "package_name": "com.games37.jc",
  "source_device_iden": "xxx",
  "source_user_iden": "xxx",
  "title": "Jurassic Tribes",
  "type": "mirror"
}

I'm not sure this project is still supported by the developer... my issue has been created for 23 days and no response yet.

I'm not sure this project is still supported by the developer... my issue has been created for 23 days and no response yet.

Yea but he released 9.5.0 just 24 days ago, so it's still active.

Yeah, true. I guess I see this as a fundamental bug w/ the project because of the fact that receiving notifications on PC is the main purpose to use it and I've received 0 support in the past month. I know I'm not paying for it and I'm not owed anything, but I'd think this would be a top priority to fix for usability.

I'm unable to receive notifications on Linux Mint 19.2, and it sounds like it may be related to this issue.

My IFTTT applets push notifications to me just fine
Notifications aren't being mirrored (including SMS notifications)
I don't receive pushes that I send from my phone
Universal copy/paste works fine

@vuther
@GodLesZ
@shoguevara
Thanks for all your efforts, comments and documentation. The issue will be addressed asap, there was simply a lack of time. A new version of the project will be released today.

@vuther
@GodLesZ
@shoguevara
Thanks for all your efforts, comments and documentation. Due to a lack of time, work on the issue backlog had been delayed for some time.
The issue was addressed in the new release (with great help via @btaens pull requests) and is available here:
https://github.com/sidneys/pb-for-desktop/releases
Please validate whether the bug still occurs.
Cheers,
S

I'm unable to receive notifications on Linux Mint 19.2, and it sounds like it may be related to this issue.
* My IFTTT applets push notifications to me just fine

* Notifications aren't being mirrored (including SMS notifications)

* I don't receive pushes that I send from my phone

* Universal copy/paste works fine

@chetcuti
Thanks for the report. Does the issue still persist in  version 10.2.0?
Kind Regards, S

@sidneys, I just installed the most recent version and it seems to have fixed the issues I was having. Notifications are now mirroring and I'm receiving pushes on my desktop that I sent from my phone.
Unfortunately I've actually had to switch away from PB for Desktop though. Since the notifications aren't persistent I've had to switch to the browse extension, which does have persistent notifications. I would prefer to use the actual app but I was missing too many notifications because I rarely look down at my taskbar to see if there are new notifications.


batch normalization

I am a little bit confused by the batch normalization implementation here: everytime I run bn a self.bn(x) is used, where a new bn layer is created, i.e., bn_module = nn.BatchNorm1d(x.size()[1]).cuda(). Will this fail to train the parameters in the bn as it is created incrementally?

I also feel confused about this... It seems that as x.size(1) (that is, maximum graph size of a batch) varies from time to time, the size of BatchNorm cannot be fixed. However, I still think this usage of BatchNorm problematic. Hoping to get more explanation about this!

Hi,
Thanks for pointing out. I think batch norm is quite confusing for GNNs and what you said make sense. I pushed the new version with bn having trainable parameters registered. It is confusing not just because of the size of graph, but also that I'm not sure if 2d batch norm or 1d make more sense, but there's also the problem that there is no alignment between nodes of different graphs, and we cannot do normalization along the axis of nodes.
Performance-wise I don't see a difference. I'm still working on improving batch norm in general for GNNs.
Thanks again for raising this issue!
Rex


Please tag versions

Please create versions in github, so the project can work nicely with composer

@cbschuld please tag a release.

+1

+1 @cbschuld That's an easy one... Please just tag the repo, which will help a lot!

@cbschuld Is there any likelihood of tagging a release? Even something like 0.1?

+1 иначе приходится прописывать "minimum-stability": "dev

+1

What is the problem? To create a tag, you need only 1-2 minutes

+1

@PATROMO @visavi @Krknv @visavi @misilot @sbuckpesch @adrazich @calmdev @misilot - understood 100%... I have not had a lot of cycles to take care of this until now.  I'm going to be making a lot of changes over the next week or so.  Sorry for the massive radio silence on the project; I am back into it!

v 1.9.1. was tagged but I'll work on some of the issues and get this more up-to-date and will be releasing other versions soon


Add command line positional args for both upload and dss to the HCA documentation page.

Hi,
It may be a good idea to add the positional arguments for both upload and dss to the HCA documentation page.

Does this issue close with the work done on dcp-cli #146?


Allow state change from props

There is no way to access the default state:
state = {
        expanded: false,
        truncated: false
    }

It would be nice to be able to change this from the props. I am building an app where I would like to collapse the expanded text when clicking on a new element. I have local a local state called 'longdescription' which is toggled by onClick, but even if I reset it at re-render it doesn't collapse the long text:
  <ShowMoreText
             lines={ 2 }
             more='more'
             less='less'
             anchorClass='more-btn'
             onClick={ togglelongDescription }
    >
             <p>
                       { pressAreaDescription }
             </p>
    </ShowMoreText>

Thank you for the recommendation.
The requested functionality (added new prop expanded) could be found in<EMAIL_ADDRESS>Best Regards
Zdravko

Thanks for the prompt response and integration!
Now I get an error when trying to use the new prop:
checkPropTypes.js:20 Warning: Failed prop type: ShowMoreText: prop type expandedis invalid; it must be a function, usually from theprop-typespackage, but receivedundefined.
I think there's a typo on the PropTypes definition, instead of
PropTypes.boolean
it should be
PropTypes.bool
based on https://github.com/facebook/prop-types
thanks!

The bug fix is applied.
Thank you.


RoR group project: set up your Kanban board

Description

Add link to the project
We are two team members in our group

Team member 1 - @mumo-esther
Team member 2 - @veronica365

Hi, @veronica365 & @mumo-esther 👋,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Status: Invalid Code Review Request

You have submitted a project, that loads a 404. Kindly ensure that the project and/or Kanban board is not private
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me @brytebee in your question so I can receive the notification.
Use @brytebee on slack for faster responses.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.

Hi, @veronica365 & @mumo-esther 👋,
You've done a great job, go ahead and start the project

##Highlights

Task are distributed equally  ✅
All column are created in the board(todo, progress, review, Done) ✅
All tasks from the requirements are added in Kanban board  ✅


Inconsistent pattern and prompt

The definition of <\END> tag in the TEXT_DELIMITER_PATTERN_MUTATION is inconsistent with <END> tag in the meta_sample_template:
TEXT_DELIMITER_PATTERN_MUTATION = r"(?s)(?<=<START>)(.*?)(?=</END>)"

meta_sample_template: |
  You are given a task description and a prompt instruction and different styles known as meta prompts:
  [Task Description]: {task_description}
  [Meta Prompt]: {meta_prompts}
  Now you need to generate {num_variations} variations of following Instruction adaptively mixing meta prompt while keeping similar semantic meaning.
  Make sure to wrap each generated prompt with <START> and <END>
  [Prompt Instruction]: {prompt_instruction}
  [Generated Prompts]:

Maybe  TEXT_DELIMITER_PATTERN_MUTATION should be changed to <END> like in other places

Hi, thanks for your comments.
Please let us know in case you face an error while running the code. To the best of our knowledge, the current setting seems to be working fine

The problem occurs in promptwizard/glue/promptopt/techniques/critique_n_refine/core_logic.py in line 114:
matches = re.findall(DatasetSpecificProcessing.TEXT_DELIMITER_PATTERN_MUTATION, generated_mutated_prompt)

matches become empty list, because generated_mutated_prompt icnludes <END> tag instead of <\END> tag in the TEXT_DELIMITER_PATTERN_MUTATION
It was experienced by running demos/scenarios/dataset_scenarios_demo.ipynb:
Scenario 1 : We have no training data , but we also don't want in-context examples in final prompt with OpenAI gpt-4o-mini model


upgrade: drivelist to v5.0.25

This version contains various improvements on the drive detection system
mainly on Windows. The improvements can be summarized as follows:

Drivelist no longer spawns a .bat script to perform its job. This
means that a lot of spawning related issues are now fixed

Drivelist doesn't fetch drive sizes from WMI anymore, which was known
to incorrectly report a very small size for certain drives, causing
the famous "0.0 GB" Windows issues (see
https://github.com/resin-io-modules/drivelist/issues/142)

Change-Type: patch
Changelog-Entry: Implement Windows drive detection using C++
Changelog-Entry: Fix various Windows .bat spawning issues
Changelog-Entry: Fix 0.0 GB Windows drive detection issues
Fixes: https://github.com/resin-io/etcher/issues/1108
Fixes: https://github.com/resin-io/etcher/issues/1054
Fixes: https://github.com/resin-io/etcher/issues/995
Fixes: https://github.com/resin-io/etcher/issues/1483
Fixes: https://github.com/resin-io/etcher/issues/1142
Signed-off-by: Juan Cruz Viotti<EMAIL_ADDRESS>

This commit still references a drivelist unmerged branch. I'll update it once the new version is released to npm.

This will also include https://github.com/resin-io-modules/drivelist/pull/191 if I'm not mistaken, fixing https://github.com/resin-io/etcher/issues/1571 – @jviotti do you want to add that to the commit message as well?

Oh, right, I'll add it.

This PR has now been updated to point to a version at npm.

See https://github.com/resin-io-modules/drivelist/commit/bcf8d7477050aecd720eb58a31031f8843d5dc70

For anybody affected by any of the issues that this bug closed, you can test out the following builds of Etcher:

Windows x64: https://resin-nightly-downloads.s3.amazonaws.com/etcher/2017-07-12/1.0.0%2B77c60b9/Etcher-1.0.0%2B77c60b9-win32-x64.exe
Windows x86: https://resin-nightly-downloads.s3.amazonaws.com/etcher/2017-07-12/1.0.0%2B77c60b9/Etcher-1.0.0%2B77c60b9-win32-x86.exe
OSX: https://resin-nightly-downloads.s3.amazonaws.com/etcher/2017-07-13/1.0.0%2B77c60b9/Etcher-1.0.0%2B77c60b9-darwin-x64.dmg
Linux x64: https://resin-nightly-downloads.s3.amazonaws.com/etcher/2017-07-13/1.0.0%2B77c60b9/etcher-1.0.0%2B77c60b9-linux-x64.zip
Linux x86: https://resin-nightly-downloads.s3.amazonaws.com/etcher/2017-07-13/1.0.0%2B77c60b9/etcher-1.0.0%2B77c60b9-linux-x86.zip

Hello! So I'm having the same issue, when starting up Etcher, the error saying "Couldn't scan the drives" is showing up.
I tested the build posted above for x64 bit architecture as I have Windows 10 Pro N x64 bit, it installed but the same issue came up.
I read before you might believe would be the antivirus, so I disabled Windows Defender and firewall, same issue.

@DarkWolfx Maybe you're also being affected by https://github.com/resin-io-modules/drivelist/issues/198 ?
If you could jump over there and help us to diagnose the issue that would be great thank you :)


Wrong answers for flatMap

val xs = List(1,2,3)
xs flatMap {x => List(x, x*2)}

gives
List(1, 2, 3)

but
xs map {x =>List(x, x*2)} flatten

gives
List(1, 2, 2, 4, 3, 6)

Confirming that too. The only displayed results correspond to the first line entered, so for
val xs = List(1,2,3)
xs flatMap {x => List(x, x*2)}

you get the result of val xs = List(1,2,3), which can be a bit surprising (and needs to be fixed, which I'll do soon).

This is now fixed:


vuetify+nuxt+vue-macros combination of components has stopped working

Hello,
recently, the combination of vuetify+nuxt(-module)+vue-macros stopped working.
If I remove Vue macros from the module, it seems to work. I'm not sure if I'm in the right place with the bug, but maybe just as a heads up that the bug exists.
Here is the Stackblitz for it:
https://stackblitz.com/edit/nuxt-starter-5xyynj?file=nuxt.config.ts
P.S.:
If I try to implement the whole thing without the vuetify-nuxt-module, but rather via the plugin, I get the error:
Cannot start nuxt:  Vuetify plugin must be loaded after the vue plugin
at configResolved (node_modules/vite-plugin-vuetify/dist/index.cjs:29:15)
at node_modules/vite/dist/node/chunks/dep-BaOMuo4I.js:66327:67
at Array.map ()
at resolveConfig (node_modules/vite/dist/node/chunks/dep-BaOMuo4I.js:66327:53)
at async _createServer (node_modules/vite/dist/node/chunks/dep-BaOMuo4I.js:62687:18)
at async buildClient (node_modules/@nuxt/vite-builder/dist/shared/vite-builder.CMaRZVaO.mjs:745:24)
at async withLogs (node_modules/@nuxt/vite-builder/dist/shared/vite-builder.CMaRZVaO.mjs:1835:3)
at async bundle (node_modules/@nuxt/vite-builder/dist/shared/vite-builder.CMaRZVaO.mjs:1826:3)
at async bundle (node_modules/nuxt/dist/index.mjs:5888:5)
at async build (node_modules/nuxt/dist/index.mjs:5763:3)

looks like something wrong in @vue-macros/nuxt, I'm going to try to fix it

@vue-macros/nuxt removing vite:vue plugin https://github.com/vue-macros/vue-macros/blob/main/packages/nuxt/src/index.ts#L41-L42, I'll send you a module to fix the problem and maybe detect @vue-macros/nuxt module here to move the internal plugins.

@AndreyKindin check my comment on vue macros issue

@AndreyKindin check https://stackblitz.com/edit/nuxt-starter-jq6cno?file=nuxt.config.ts,app.vue,package.json,.nuxt%2Fcomponents.d.ts again
It is using the pkg-pr-new from this PR https://github.com/vuetifyjs/nuxt-module/pull/276 and it is working since there is no dependency on vite:vue  plugin => moved to Nuxt auto import
I need to add directove to Nuxt and we can remove custom Vuetify Vite plugin here: rn only used for directives ignoring any component or lab component auto import

The reason is that Vue Macros will remove vite:vue plugin from the original plugin list and add it back into an array.
before: [a, 'vite:vue', b]
after: [a, [b, 'vite:vue', c, d], e]
now: [a, b, 'vite:vue', c, d, e]
This way it won't affect other plugins. Successfully fixed at https://stackblitz.com/edit/nuxt-starter-itabzd.

thx @sxzz ❤


In-app Billing Vulnerability

im using the v 3.0.2 version and i got this message in my google play console
Security alert
Your app is using an incorrect implementation of in-app billing. Please see this Google Help Center     article for more information.
Affects APK version 10.

https://support.google.com/faqs/answer/7054270?hl=en
is this vulnerabily was fixed in the latest release?
cdt,

I have the same problem. I also tried to rebuild the app with SDK 23 but it didn't help.
I tried to use latest stable version (4.0.0) but it has a min SDK 6.0 as a requirements while latest stable TI SDK is 5.3.1.GA.

I have the same problem.
Message I got in developer console:
"Alerta de seguridad
Tu aplicación incluye una implementación no segura del controlador WebViewClient.onReceivedSslError. En concreto, la implementación ignora todos los errores de validación de certificados SSL, por lo que tu aplicación será vulnerable a los ataques "man-in-the-middle". Un atacante podría modificar el contenido de la vista WebView afectada, leer los datos que se hayan transmitido (como las credenciales de inicio de sesión) y ejecutar código en la aplicación con JavaScript."

I do not see this error and I am using 3.0.2, building with 5.4.0 GA (Was using 5.3.1 before this with no error either). What Android SDK's are you targeting? My min is 14, target is 23.

I used the latest version latest stable version (4.0.0) with titanium sdk 5.3.0 and still giving the same error.
Please fix this. It's urgent.

Are any of you working with subscriptions or consumable products? That's the only two things my app doesn't use and I'm not seeing this error.

This is the inapp billing error and I am using subscription product. Please give the solution.
It says like this- https://support.google.com/faqs/answer/7054270

Hello Please help me regarding this issue. It's urgent.

DO NOT use v4.0.0. It says Compiled to work with Titanium SDK v6.0.0+. You are using 5.3.0.
Use https://github.com/appcelerator-archive/ti.inappbilling/releases/tag/v3.0.2

Also, if you have errors, please provide logs and reproducible steps. Without those, it's very hard to help or assist.

On building the project there is no error but on uploading to play store and when it publish Google send the error says like invalid inappbilling. please go through this link.
https://support.google.com/faqs/answer/7054270

I m also facing the same problem, I mentioned to the google and they mentioned me:-
Hi Prateek,
Thanks for contacting Google Play Developer Support!
I see that you have received an alert regarding your app(s) using an unsafe implementation of Google Play’s in-app billing service.
Firstly, please note that Google Play is not currently enforcing on apps with this unsafe implementation. These alerts are being sent to developers on an informational basis only. No action will be taken against your apps or your developer account if you choose not to act upon them.
Please also note that if your app(s) does not contain any active in-app products you may simply ignore this warning.
However, we do encourage developers to address these security concerns as soon as possible if they are indeed present in your app's APK. Invoking the in-app billing service without setting a target package for the intent can enable a malicious package to bypass the Play store billing system and access items that have not been purchased.
If your app appears to be abiding by all practices described in the help center article linked below, and you still see this alert in the Developer Console, you may also need to ensure that your app(s) is using the newest version of Google Play Services.
https://support.google.com/faqs/answer/7054270?hl=en
Please be aware that if you are using IabHelper, you must still use the most-recent SDK even if the manual implementation appears to be correct as per our guidelines linked above. You can get the most recent SDK of IabHelper here:
https://github.com/googlesamples/android-play-billing/
If these vulnerabilities are coming from a third party development tool or library, please work with them to address the security concerns.
To verify that the vulnerability is addressed, please submit the updated version of your app to the Developer Console and wait 5 hours. If the app hasn’t been updated correctly, the warning remain.
I hope this helps! If you have any further questions, please let me know. I'm happy to help.
Regards,
Alex
Google Play Developer Support

@Geek-Fish im using consumable one
I even compiled the module with the latest version of the SDK and the latest version of the file IabHelper.java and i got every time the same warning message.
i'm using the 5.4.0.GA sdk minSdkVersion="15" android:targetSdkVersion="23"
how can i add the latest version of google-play-services ?

I believe someone will have to download the latest SDK and compare it with
the module to make any changes to the JS code, then recompile. I will look
at this eventually, but can't just yet.
On Sun, Aug 28, 2016, 3:37 PM fadaytak<EMAIL_ADDRESS>wrote:

@Geek-Fish https://github.com/Geek-Fish im using consumable one
I even compiled the module with the latest version of the SDK and the
latest version of the file IabHelper.java and i got every time the same
warning message.
i'm using the 5.4.0.GA sdk minSdkVersion="15"
android:targetSdkVersion="23"
how can i add the latest version of google-play-services ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/appcelerator-archive/ti.inappbilling/issues/18#issuecomment-242994658,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIN8zNxzK9RySG5y2qgQbcMPxZT3t7vTks5qkeOAgaJpZM4Je6uk
.

how can i get the latest version of "google-play-services.jar"
with the latest sdk manager we got other file format but not a jar file
can any one publish the latest google-play-services.jar 9.0.4 to compile the module with it.

Appcelerator makes the jar file when you compile the module. It might not
be as easy as that though. I'm sure the module is going to need modified
further than just getting the new SDK and recompiling.
On Tue, Aug 30, 2016, 9:37 AM fadaytak<EMAIL_ADDRESS>wrote:

how can i get the latest version of "google-play-services.jar"
with the latest sdk manager we got other file format but not a jar file
can any one publish the latest google-play-services.jar 9.0.4 to compile
the module with it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/appcelerator-archive/ti.inappbilling/issues/18#issuecomment-243440895,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIN8zG6t0EyPthqBm5XQPYmsA_tziRsNks5qlDIpgaJpZM4Je6uk
.

Hello Team,
Is there any update on the above point(Google security alert).

If needed, (which seems to be the case), please proceed to create and open a JIRA ticket here.

It's not giving the module link.


"Index -1 out of bounds for length 33" randomly when placing train tracks

Describe the Bug
Happens when you place a train track randomly
Reproduction Steps

place a bunch of train tracks
eventually placing one of them will crash

Expected Result
for it to work
Screenshots and Videos
No response
Crash Report or Log
https://mclo.gs/JPBR2Jk & https://mclo.gs/GvFpvwI
Operating System
Windows 11
Mod Version
0.5.1d
Minecraft Version
1.20.1
Other Mods
continuity-3.0.0-beta.4+1.20.1.jar
create-sodium-fix-0.5.1-d-build.5+mc1.20.1.jar
createaddition-fabric+1.20.1-0.9.0.jar
dynamic-fps-3.2.1+minecraft-1.20.0.jar
entityculling-fabric-1.6.2-mc1.20.1.jar
essential_1-2-3_fabric_1-20-1.jar
ferritecore-6.0.1-fabric.jar
ImmediatelyFast-1.2.7+1.20.2.jar
immersive-portals-3.3.9-mc1.20.1-fabric.jar
indium-1.0.27+mc1.20.1.jar
lambdynamiclights-2.3.2+1.20.1.jar
lazydfu-0.1.3.jar
lithium-fabric-mc1.20.1-0.11.2.jar
mixintrace-1.1.1+1.17.jar
modmenu-7.2.2.jar
puzzle-1.5.2+1.20.jar
qfapi-7.4.0_qsl-6.1.2_fapi-0.90.0_mc-1.20.1.jar
reeses_sodium_options-1.6.5+mc1.20.1-build.95.jar
replaymod-1.20.1-2.6.14.jar
sodium-extra-0.5.1+mc1.20.1-build.112.jar
sodium-fabric-mc1.20.1-0.5.3.jar
soundphysics-fabric-1.20.1-1.2.1.jar
starlight-1.1.2+fabric.dbc156f.jar
Steam_Rails-1.5.1+fabric-mc1.20.1.jar
vivecraft-1.20.1-1.1.7-fabric.jar
worldedit-mod-7.2.15.jar
Additional Context
No response

use create fabric.


Team 1 - Project 2

Hi CDL team,
Please find a Project - 2 branch with our contribution to Week 2 Cohort Project.
Cheers,
Team 1

Well done, thanks for an excellent project


Mentorship Programme - Hero

Create the Hero component with props for heading, text and background colour.
Create a reusable component for the background colour.

I'm not sure, I don't think I ever had the opportunity to talk to @adeola2310. Yes I can split, I don't understand the value of making the body text a component - I thought you just fetch the text to the page? And the JSX will just be

I'm not sure, I don't think I ever had the opportunity to talk to @adeola2310. Yes I can split, I don't understand the value of making the body text a component - I thought you just fetch the text to the page? And the JSX will just be <Typography>

I think in this case it is simply a typography you're right 👍 so it sits on the page itself

@adeola2310 feel free to work on this task or reassign yourself to another one.


Windows keybase.exe launch/retry

If we can't connect, we'll try to launch the service(s).
Testing with build https://ci.keyba.se/job/windows-installer/783/

lets not request reviews from individuals. just tag @keybase/react-hackers so we can all see it

@keybase/react-hackers


Use concurrency group for deployment

It seems this could be useful for us as well if we merge multiple PRs: https://github.com/SciML/SciMLBenchmarks.jl/pull/454

@staticfloat Do you think that's a good idea?

Yes. It needs a resign of the hashes, I’ll do that later, once I’m at the computer

Just a small reminder @staticfloat, in case you have access to your computer.

Thank you!


The problem with popups when component is re-rendered

I have a problem when component is re-render because of the assignment of cords to url. By that, my popups flash on the map as many times as re-render occurs. Before using this component, everything was perfectly fine

Same here

In addition, another problem occurs when two markers have the same position. Then when the component is re-rendered, the pop-ups are closed.

I had a vaguely related issue but I solved by using memo on my component that showed the pop up.

I had a vaguely related issue but I solved by using memo on my component that showed the pop up.

Can you pass here code snippet how it looks like in your case?

Sorry I cannot give my code sample but effectively on the return statement for the component that showed the pop up I wrapped the the result with memo.

Im trying to use, but i have the problem 'NotFoundError: Failed to execute 'removeChild' on 'Node': The node to be removed is not a child of this node.' when im clicking on popups ...
Can you help?

Hi, I've been spending a lot of time today trying to find out why my code was causing flickering, unwanted rerenders ...until i found this open issue.
In case it helps someone, here is what i implemented, thanks to @nottud comment above
const MyMainComponent = ({offers}) => {
    return (
        <MarkerClusterGroup
            chunkedLoading
            showCoverageOnHover={false}
            maxClusterRadius={40}
        >
            {offers.map((offer) => (
                <MyMarker key={offer.reference} position={[offer.latitude, offer.longitude]} offer={offer} />
            ))}
         </MarkerClusterGroup>
    )
}

const MyMarker = ({position, offer}: MapMarkerProps) => {
    const markerIcon = L.divIcon({
        html: `

        `,
        className: '',
        iconSize: L.point(40, 40)
    });

    // weird hack, marker clustered causes a lot of redenders when having popups
    return useMemo(() => (
        <Marker position={position} icon={markerIcon}>
            <Popup
                maxWidth={260}
                closeButton={false}
                offset={L.point(0, -20)}
            >
                <OfferCard
                    offer={offer}
                    variant='map-popup'
                />
            </Popup>
        </Marker>
    ), [offer]);
}

The useMemo seems to have resolved all the issues i had.


Added a new welcome text to the top of the intro page

Hi!!
 Nicely done 👯♂ !!


Docker build error (licences)

When trying to build droidbot with docker using:
git clone https://github.com/lynnlyc/droidbot.git
docker build -t honeynet/droidbot droidbot

Gives the folowing licence issues:

...
November 20, 2015
Do you accept the license 'android-sdk-license-c81a61d9' [y/n]: n
...
June 2014.
Do you accept the license 'android-sdk-preview-license-d099d938' [y/n]: n
...
Do you accept the license 'mips-android-sysimage-license-acdab23e' [y/n]: n
...
Error: Target id is not valid. Use 'android list targets' to get the target ids.
The command '/bin/sh -c expect /build/accept-licenses "android update sdk --no-ui --all --filter platform-tool,system-image,android-16" "android-sdk-license-5be876d5" &&     echo "\n"| android create avd -n droidbox -t 1 -d 2' returned a non-zero code: 1

If i alter /droidbot/docker/accept-licences to accept all licences i get the folowing error (set line 20 to y):

...
Installing Archives:
Preparing to install archives
Downloading Android SDK Platform-tools, revision 23.1
Installing Android SDK Platform-tools, revision 23.1
Stopping ADB server failed (code -1).
Installed Android SDK Platform-tools, revision 23.199%)
Downloading Android SDK Platform-tools, revision 24 rc2
Installing Android SDK Platform-tools, revision 24 rc2
Stopping ADB server succeeded.
Installed Android SDK Platform-tools, revision 24 rc299%)
Downloading SDK Platform Android 4.1.2, API 16, revision 5
Installing SDK Platform Android 4.1.2, API 16, revision 5
Installed SDK Platform Android 4.1.2, API 16, revision 596%)
Skipping 'Android TV ARM EABI v7a System Image, Android API 23, revision 3'; it depends on 'SDK Platform Android 6.0, API 23, revision 3' which was not installed.
Skipping 'Android TV Intel x86 Atom System Image, Android API 23, revision 3'; it depends on 'SDK Platform Android 6.0, API 23, revision 3' which was not installed.
Skipping 'Android Wear ARM EABI v7a System Image, Android API 23, revision 3'; it depends on 'SDK Platform Android 6.0, API 23, revision 3' which was not installed.
Skipping 'Android Wear Intel x86 Atom System Image, Android API 23, revision 3'; it depends on 'SDK Platform Android 6.0, API 23, revision 3' which was not installed.
Skipping 'ARM EABI v7a System Image, Android API 23, revision 3'; it depends on 'SDK Platform Android 6.0, API 23, revision 3' which was not installed.
Skipping 'Intel x86 Atom_64 System Image, Android API 23, revision 8'; it depends on 'SDK Platform Android 6.0, API 23, revision 3' which was not installed.
Skipping 'Intel x86 Atom System Image, Android API 23, revision 8'; it depends on 'SDK Platform Android 6.0, API 23, revision 3' which was not installed.
Skipping 'Google APIs ARM EABI v7a System Image, Android API 23, revision 7'; it depends on 'SDK Platform Android 6.0, API 23, revision 3' which was not installed.
Skipping 'Google APIs Intel x86 Atom_64 System Image, Android API 23, revision 12'; it depends on 'SDK Platform Android 6.0, API 23, revision 3' which was not installed.
Skipping 'Google APIs Intel x86 Atom System Image, Android API 23, revision 12'; it depends on 'SDK Platform Android 6.0, API 23, revision 3' which was not installed.
Skipping 'Android TV Intel x86 Atom System Image, Android API N, revision 2'; it depends on 'SDK Platform Android N Preview, revision 2' which was not installed.
Skipping 'Intel x86 Atom_64 System Image, Android API N, revision 2'; it depends on 'SDK Platform Android N Preview, revision 2' which was not installed.
Skipping 'Intel x86 Atom System Image, Android API N, revision 2'; it depends on 'SDK Platform Android N Preview, revision 2' which was not installed.
Skipping 'Android TV ARM EABI v7a System Image, Android API 22, revision 1'; it depends on 'SDK Platform Android 5.1.1, API 22, revision 2' which was not installed.
Skipping 'Android TV Intel x86 Atom System Image, Android API 22, revision 1'; it depends on 'SDK Platform Android 5.1.1, API 22, revision 2' which was not installed.
Skipping 'Android Wear ARM EABI v7a System Image, Android API 22, revision 7'; it depends on 'SDK Platform Android 5.1.1, API 22, revision 2' which was not installed.
Skipping 'Android Wear Intel x86 Atom System Image, Android API 22, revision 7'; it depends on 'SDK Platform Android 5.1.1, API 22, revision 2' which was not installed.
Skipping 'ARM EABI v7a System Image, Android API 22, revision 1'; it depends on 'SDK Platform Android 5.1.1, API 22, revision 2' which was not installed.
Skipping 'Intel x86 Atom_64 System Image, Android API 22, revision 4'; it depends on 'SDK Platform Android 5.1.1, API 22, revision 2' which was not installed.
Skipping 'Intel x86 Atom System Image, Android API 22, revision 4'; it depends on 'SDK Platform Android 5.1.1, API 22, revision 2' which was not installed.
Skipping 'Google APIs ARM EABI v7a System Image, Android API 22, revision 1'; it depends on 'SDK Platform Android 5.1.1, API 22, revision 2' which was not installed.
Skipping 'Google APIs Intel x86 Atom_64 System Image, Android API 22, revision 4'; it depends on 'SDK Platform Android 5.1.1, API 22, revision 2' which was not installed.
Skipping 'Google APIs Intel x86 Atom System Image, Android API 22, revision 4'; it depends on 'SDK Platform Android 5.1.1, API 22, revision 2' which was not installed.
Skipping 'Android TV ARM EABI v7a System Image, Android API 21, revision 3'; it depends on 'SDK Platform Android 5.0.1, API 21, revision 2' which was not installed.
Skipping 'Android TV Intel x86 Atom System Image, Android API 21, revision 3'; it depends on 'SDK Platform Android 5.0.1, API 21, revision 2' which was not installed.
Skipping 'Android Wear ARM EABI v7a System Image, Android API 21, revision 3'; it depends on 'SDK Platform Android 5.0.1, API 21, revision 2' which was not installed.
Skipping 'Android Wear Intel x86 Atom System Image, Android API 21, revision 3'; it depends on 'SDK Platform Android 5.0.1, API 21, revision 2' which was not installed.
Skipping 'ARM EABI v7a System Image, Android API 21, revision 3'; it depends on 'SDK Platform Android 5.0.1, API 21, revision 2' which was not installed.
Skipping 'Intel x86 Atom_64 System Image, Android API 21, revision 3'; it depends on 'SDK Platform Android 5.0.1, API 21, revision 2' which was not installed.
Skipping 'Intel x86 Atom System Image, Android API 21, revision 3'; it depends on 'SDK Platform Android 5.0.1, API 21, revision 2' which was not installed.
Skipping 'Google APIs ARM EABI v7a System Image, Android API 21, revision 10'; it depends on 'SDK Platform Android 5.0.1, API 21, revision 2' which was not installed.
Skipping 'Google APIs Intel x86 Atom_64 System Image, Android API 21, revision 10'; it depends on 'SDK Platform Android 5.0.1, API 21, revision 2' which was not installed.
Skipping 'Google APIs Intel x86 Atom System Image, Android API 21, revision 10'; it depends on 'SDK Platform Android 5.0.1, API 21, revision 2' which was not installed.
Skipping 'Android Wear ARM EABI v7a System Image, Android API 20, revision 4'; it depends on 'SDK Platform Android 4.4W.2, API 20, revision 2' which was not installed.
Skipping 'Android Wear Intel x86 Atom System Image, Android API 20, revision 4'; it depends on 'SDK Platform Android 4.4W.2, API 20, revision 2' which was not installed.
Skipping 'ARM EABI v7a System Image, Android API 19, revision 3'; it depends on 'SDK Platform Android 4.4.2, API 19, revision 4' which was not installed.
Skipping 'Intel x86 Atom System Image, Android API 19, revision 3'; it depends on 'SDK Platform Android 4.4.2, API 19, revision 4' which was not installed.
Skipping 'ARM EABI v7a System Image, Android API 18, revision 3'; it depends on 'SDK Platform Android 4.3.1, API 18, revision 3' which was not installed.
Skipping 'Intel x86 Atom System Image, Android API 18, revision 2'; it depends on 'SDK Platform Android 4.3.1, API 18, revision 3' which was not installed.
Skipping 'ARM EABI v7a System Image, Android API 17, revision 3'; it depends on 'SDK Platform Android 4.2.2, API 17, revision 3' which was not installed.
Skipping 'Intel x86 Atom System Image, Android API 17, revision 2'; it depends on 'SDK Platform Android 4.2.2, API 17, revision 3' which was not installed.
Skipping 'MIPS System Image, Android API 17, revision 1'; it depends on 'SDK Platform Android 4.2.2, API 17, revision 3' which was not installed.
Downloading ARM EABI v7a System Image, Android API 16, revision 4
Installing ARM EABI v7a System Image, Android API 16, revision 4
Installed ARM EABI v7a System Image, Android API 16, revision 499%)
Downloading Intel x86 Atom System Image, Android API 16, revision 2
Installing Intel x86 Atom System Image, Android API 16, revision 2
Installed Intel x86 Atom System Image, Android API 16, revision 299%)
Downloading MIPS System Image, Android API 16, revision 1
Installing MIPS System Image, Android API 16, revision 1
Installed MIPS System Image, Android API 16, revision 199%)
Skipping 'ARM EABI v7a System Image, Android API 15, revision 3'; it depends on 'SDK Platform Android 4.0.3, API 15, revision 5' which was not installed.
Skipping 'Intel x86 Atom System Image, Android API 15, revision 2'; it depends on 'SDK Platform Android 4.0.3, API 15, revision 5' which was not installed.
Skipping 'MIPS System Image, Android API 15, revision 1'; it depends on 'SDK Platform Android 4.0.3, API 15, revision 5' which was not installed.
Skipping 'ARM EABI v7a System Image, Android API 14, revision 2'; it depends on 'SDK Platform Android 4.0, API 14, revision 4' which was not installed.
Skipping 'Intel x86 Atom System Image, Android API 10, revision 3'; it depends on 'SDK Platform Android 2.3.3, API 10, revision 2' which was not installed.
Stopping ADB server succeeded.
Starting ADB server succeeded.
Done. 6 packages installed.
Valid ABIs: default/armeabi-v7a, default/mips, default/x86
Error: This platform has more than one ABI. Please specify one using --abi.
The command '/bin/sh -c expect /build/accept-licenses "android update sdk --no-ui --all --filter platform-tool,system-image,android-16" "android-sdk-license-5be876d5" &&     echo "\n"| android create avd -n droidbox -t 1 -d 2' returned a non-zero code: 1

I'm using Linux (4.5.1)
I opted to build it myself because i want to get the logcat file together with the output of droidbox when examining a apk (and this was not possible otherwise).

Note that this has been fixed in the upstream docker image at https://github.com/aikinci/droidbox
It is the fact that each new sdk version has a different license identifier.

Alright thanks, when i try to build the upstream docker image i got the following error message:

...
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
Reading package lists...
Building dependency tree...
Package openjdk-7-jre-headless is not available, but is referred to by another package.
This may mean that the package is missing, has been obsoleted, or
is only available from another source
E: Package 'openjdk-7-jre-headless' has no installation candidate
... returned a non-zero code: 100

But since this is an upstream issue i will close this one.


Moved showing/hiding sidebar to toggleSidebar function

Issue: #187

Great work! Thank you @JLambertazzo .


Get full inputs/outputs from execution data

TL;DR
Fixes Input/Output view when using GCS.
Type

[ ] Bug Fix
[x] Feature
[ ] Plugin

Are all requirements met?

[x] Code completed
[x] Smoke tested
[ ] Unit tests added
[ ] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
flyteadmin doesn't know how to return public URLs for GCS objects. It returns URL in format gs://... that doesn't work.  This fixes Input/Output view when using GCS for Storage by using new fullInputs/fullOutputs fields in getExecutionData proto. Change is backward-compatible, if full inputs/outputs aren't present, it's going to fallback on the existing mechanism.
I had to take https://github.com/lyft/flyteconsole/commit/6a3b80f175b357bc81fd5f7fea31c8e89d13cd0d and few other commits because I needed new fields in flyteidl.
Tracking Issue
https://github.com/lyft/flyte/issues/444
Follow-up issue
NA

cc @kumare3

@kanterov today you showed that you know typescript 😜. But why not a separate PR

Codecov Report

Merging #92 into master will increase coverage by 0.16%.
The diff coverage is 72.82%.

@@            Coverage Diff             @@
##           master      #92      +/-   ##
==========================================
+ Coverage   66.16%   66.33%   +0.16%
==========================================
  Files         368      370       +2
  Lines        5894     5977      +83
  Branches      910      942      +32
==========================================
+ Hits         3900     3965      +65
- Misses       1994     2012      +18

Impacted Files
Coverage Δ

...Details/ExecutionNodeDetails/OutputNodeDetails.tsx
0.00% <0.00%> (ø)

.../Executions/ExecutionDetails/NodeExecutionData.tsx
0.00% <0.00%> (ø)

...ponents/Executions/ExecutionInputsOutputsModal.tsx
34.61% <0.00%> (-4.52%)
:arrow_down:

src/components/Executions/Tables/constants.ts
100.00% <ø> (ø)

src/components/Executions/Tables/styles.ts
100.00% <ø> (ø)

src/components/common/styles.ts
100.00% <ø> (ø)

src/models/Common/constants.ts
90.90% <ø> (ø)

...xecutions/ExecutionDetails/NodeExecutionInputs.tsx
46.15% <33.33%> (-8.40%)
:arrow_down:

...ecutions/ExecutionDetails/NodeExecutionOutputs.tsx
46.15% <33.33%> (-8.40%)
:arrow_down:

...components/Executions/NodeExecutionCacheStatus.tsx
89.36% <89.36%> (ø)

... and 10 more

Continue to review full report at Codecov.

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

@kanterov Thanks for doing this!
The other PR should be merged today and you can rebase off of master after that. Or you could just update the version of flyteidl in package.json and run yarn to update the lock file instead of pulling in all of the other changes.
Would you mind writing/updating tests for the components that you changed to make sure they use the new data when it's available and fall back to the old data otherwise?

@schottra done. Some non-code related builds checks are failing, it seems I need to configure something for them to work. Do I need to fix those?

@kanterov I think the build/push actions are expected to fail, since they run from a fork and you won't have permissions to push to github/dockerhub. I should update those to not run on forked repositories.
However, the CLA check needs to be addressed before we merge. Would you mind clicking through on that and signing the contributor agreement if you haven't already?
If you have, then the email address associated with your commits also needs to be the one associated with your github account, so that the check can match them.

@kanterov If you rebase on latest master, I've updated the Github actions, so two of those failed checks should now pass.

@schottra sorry it took some time, but now everything is done :) please take a look again

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


Document comments in --batch-file

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[ ] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests
[ ] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Bug fix
[x] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
Document ability to have comment lines in --batch-file files.
See also #4307 and #8315.

awsome I thought of this a long while ago but never implemented it


Remove unassigned items

Is there any formatting that could hide the unassigned items? My list doesn't allow unassigned items so it's pointless and confusing to users.

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

ID: 33dfff5b-2b9e-94c7-ffa2-73719f49fb20
Version Independent ID: 33dfff5b-2b9e-94c7-ffa2-73719f49fb20
Content: Format board view to customize SharePoint
Content Source: docs/declarative-customization/view-board-formatting.md
Product: sharepoint
GitHub Login: @VesaJuvonen
Microsoft Alias: vesaj

@RadiuomOxide Thanks for this. Currently there isn't any way to do this via formatter or any view level configuration. But this is a valid feedback. Will discuss this internally and track the request in our backlog.
Thanks!

I am facing the same issue - would love to get rid of the "Unassigned items" bucket.

This issue has been put to Microsoft many times over the years but they've never done anything about it. I really wish they would as "unassigned items" is so annoying when you don't need it (which is most of the time!!!)

@neilx1979 not sure if you are referring to the same issue. Board View was relased a few months ago, and the ask here is to be able to hide that.
Can you please elaborate the issue that has been raised so many time over the years?

The principal is the same as when you get "Unassigned items" in a grouped list.
People have been asking for that to be removed since 2017.

The principal isn't quite the same @neilx1979 as in a grouped list the "Unassigned Items" should only appear if there are unassigned items. In the Board View it's there even if there are none. Get your point though there should be an option to stop grouping by it in grouped views also.

If somebody will work on it, it may be worth checking if it's related to more general issue that when the view is filtered (not that there are no items with the value, but filtered) the column in the view stays in (0)

Any Microsoft follow-up on this topic? What's the point of sharing pain point if there is no update...

Agreed that the ability to remove the 'Unassigned items' bucket from the board view would be great. Otherwise, the view is not very useful, based on how much screen space it takes up.

Same for me. Any progress?

Waiting for Microsoft to add this and many other fundamental options, in the meantime I use this CSS to hide it:
div[data-bucket-id='unassigned'] {display: none;}

just stumbled upon this. @By-Pixel's solution is unfortunately unavailable to users without admin rights (or where admin has restricted the possibility to modify CSS). If anyone has found a similar solution using the JSON formatter field, that would be very useful. In the meanwhile: please Microsoft add the possibility to remove this bucket!

Any update on this Microsoft?

Any updates?

Waiting for Microsoft to add this and many other fundamental options, in the meantime I use this CSS to hide it: div[data-bucket-id='unassigned'] {display: none;}

Hi all, I know it's been a while now but how can I try this option? I don't understand much of json...tried to include it on json of advanced formating view but didn't work. Please, can someone explain where and how should I add this?
thanks!!

@alinebgf it's not JSON, it's CSS, and it needs to be added to a manifest file, for which you need to be granted administrative rights. I don't, hence it's not a viable solution.

@lucamanu ohh ok then... thanks for clarifying! I would never guess it hahaha
Microsoft should really address this, so anoying...

Please, @shagra-ms, fix this. It's really annoying and makes the board view so much less attractive to use.

Thank you everyone for the feedback. I apologise for not being able to get back earlier with an update. The team has been super busy, which didn't allow me to come back with a meaningful update.
Update on the issue - we are expecting to roll this functionality out this quarter. I hope that addresses your pain points.
Thanks!

@shagra-ms
Do you have an update on this?
I still see no option to hide/delete it.

@shagra-ms The bucket "Unassigned" still shows and I can't find an option to hide it. Was this actually rolled out in Q4/2023?

There is a option on buckets that you can hide/show when not needed.

here the bucket option:

@aaclage Thanks for that. Seems you have an ellipses menu left of the 'plus' icon in the top right corner of each bucket. We don't seem to have that (yet?) in our tenant (tenant in switzerland region).

@justanyusernameplease i have a tenant with activation that allow preview features.
you can also see the same behavior in Lists online https://lists.live.com/, the feature looks to be there just need to be release globally.

Will this become available from within sharepoint? Currently only use this when in Microsoft Lists web app. Once viewed in sharepoint page unassigned is shown again.

Will this become available from within sharepoint? Currently only use this when in Microsoft Lists web app. Once viewed in sharepoint page unassigned is shown again.
Then once the sharepoint page is viewed again that view is corrupted.

I have the same issue

Not available for me yet. Tried from the app, from different browsers. Incredibly annoying.

I have the same issue. Without dots to configure that.

@rolaseba this isn't expected.
Can you share the exact repro?

Adding my voice to ask for a solution here. I don't have the three dots to configure buckets on the board.

Dear Mr. @joejekpenyong,
Can you validate if you have the new List UI, you can see in the view bar like sample below as new List UI


unable to find scope SymbolPolicy::Scope for Symbol

i'm getting
unable to find scope `SymbolPolicy::Scope` for `Symbol`

after adding
authorize :dashboard, :show?

to my controller. if i remove it, app works normally.
i'm not using after_acion :verify_authorized, as someone suggested on (#193).
if i use a policy that does not exist, liek
authorize :asd, :show?

i get
unable to find policy `AsdPolicy` for `:asd`

it also will not work with any other symbol for which a policy exist.
i'm on rails (<IP_ADDRESS>) and pundit (1.1.0).

I just noticed that ApplicationPolicy#show?, which DashboardPolicy was inheriting was calling scope(). We can pretend this never happened and forget my existence thx


Dev Tools: We should have a way to inject the real executor into the docker-compose environment.

Currently, our docker-compose builds a fake executor which is a mock k8 service.  This is useful for testing pure armada features but if you want to submit an actual pod or interact with Kubernetes, you are blocked.
The code we need to update is the makefile:
https://github.com/G-Research/armada/blob/master/makefile#L575
We should allow a dev to build either a fake-executor or a normal executor.

Hey team! Please add your planning poker estimate with Zenhub @dejanzele @jayofdoom @kannon92 @richscott @suprjinx

i have an idea that the docker-compose environment could just use golang:alpine images and run "go run cmd/executor" .. etc for each service (the same command we use in native, more or less)


Pass through screenProps to nested navigators

Motivation
This formalizes passing screenProps to nested navigator screens by passing through the screenProps parameter to screen component in addition to pulling them out to props individually.
Prior to the change, this was accomplished by doing something like the following on the parent navigator, so that the screenProps would be available in nested navigator's screens:
<AppNavigator
  screenProps={{
    exampleScreenProp: 'exampleScreenPropValue',
    screenProps: {
      exampleScreenProp: 'exampleScreenPropValue',
      // add additional levels for number of nested navigators
    },
  }}
/>

Open to suggestions, changes, etc. If accepted, a follow-up could change the behavior to only pass through the screenProps prop into screen component (along with navigator) instead of pulling them into separate params on the component (remove {...screenProps}).
Test Plan

Run NavigationPlayground example on both Android and iOS and verify that the "exampleScreenPropValue" is displayed on the nested navigator's screens.
Integrate into own app and verify that the screenProps hack mentioned above can now be removed.

One other option would be to only pass the non-splatted version if Component is a navigator, though that may require type introspection or forking the behavior further up the stack.

I don't like this very much

My gut says that making the breaking change is probably the best option since it is the cleanest, but I also agree that avoiding breaking changes is probably for the better.

I agree that making it a breaking change is the best option. Probably we can just include a warning saying that the behaviour has changed (with a prop to disabled the warning) and remove the warning in future.

I agree that making it a breaking change is the best option. Probably we can just include a warning saying that the behaviour has changed (with a prop to disabled the warning) and remove the warning in future.

Sounds good, I'll put up a commit soon.

Closing in favor of #241


Update to CoffeeScript v1.9.3

First of all, thanks for an awesome product. I've started using it in my latest Visual Studio extension https://visualstudiogallery.msdn.microsoft.com/3b329021-cd7a-4a01-86fc-714c2d05bb6c and it rocks!!
Is there a chance to get support for the latest CoffeeScript compiler version 1.9.3?

Hey! I didn't know anyone was still using this, let alone for a new project. To be honest, I've not used it myself for years (which is why it's still at CoffeeScript version 1.3.1).
I'll have a look to see if I can update it. It involves resurrecting an old VM, so it it's a bit of work.
Out of curiosity; Have you looked at other methods of compiling CoffeeScript? I'm surprised that this old project is the one you like best. How did CoffeeScript compilation work for Web Essentials?

I've compiled using Windows Script Host and Node.js earlier. Web Essentials uses node.js.
So far, your project has been the easiest to work with. Spinning up a node.js server adds complexity and more moving parts that can (and will) fail. Would you recommend a different approach?

I don't have a better approach at the moment. And if it works, it works!
I've update the nuget package to CoffeeScript 1.9.3: http://www.nuget.org/packages/CoffeeSharp/
The small stuff I tried it on seems to work. Do let me know if something is broken.


5.2.1 Value "MaxFailedAttempts" causes incorrect "Failed" report

Because the extension attribute is using grep to search for "Failed", it also triggers irrespective of the value in 5.2.1. This is because the line also contains "maxFailedAttempts".

This might be solved by simply using the following in the EA:
echo "<result>$(grep -c "Failed;" "$auditfile")</result>"
instead of
echo "<result>$(grep -c "Failed" "$auditfile")</result>"


Tutorial "Cherry Pickr" step serial numbers are different from other tutorials

Describe the bug
Tutorial "Cherry Pickr" step serial numbers are different from other tutorials
Steps to reproduce the behavior

Navigate to https://arcade.makecode.com/lessons/cherry-pickr
Observe the tutorial serial numbers

Expect behavior
Tutorial "Cherry Pickr" step serial numbers should be same as other tutorials

Actual behavior
Tutorial "Cherry Pickr" step serial numbers are different from other tutorials

Additional context

OS: Windows(this bug repro on each browser)
Machine build: 17763.1.amd64fre.rs5_release.180914-1434
arcade version: 0.12.12
Microsoft MakeCode version: 5.17.25

This is not used as a tutorial but is always viewed as a document. Hey @jwunderl, this was one of the first walkthrough things done early on. Is this still useful?

this was done by the high school interns, so we wanted to keep it (also discussed in https://www.linkedin.com/feed/update/urn:li:activity:6542447817380757504?fbclid=IwAR3TVFY5EuXhWImBXKiKAmwepW4ClgZClQORmiWq6rYsYyoEqmC1udLHIhM). I do think the approach is useful for showing how to build up more complicated games in discrete chunks

Ah, ok.

This is not a tutorial. It's a "Lesson" so it doesn't observe the tutorial step labeling style.


Improve accuracy of CalcTextWidth if selected font has kerning pairs

I discovered that if a font has kerning pairs and the text you wish to get the width of contains one (or more) of those pairs the value of CalcTextWidth will be off for each pair in the text. I couple You's or To's and things get really bad really quickly (the cursor and highlighted text should be directly adjacent to the previous Y):

See discussion here where the suggestion was made to switch to using DrawText with DT_CALCRECT.
I made the change and now things look good again:

In the process of investigating I also discovered that despite the documentation, GdiDrawText does not return:
     * @return {Array<number>}
     *     index | meaning

     *     [0] left   (DT_CALCRECT)

     *     [1] top    (DT_CALCRECT)

     *     [2] right  (DT_CALCRECT)

     *     [3] bottom (DT_CALCRECT)

     *     [4] characters drawn

even if DT_CALCRECT flag is set. I had no idea how you wanted to handle that, so I left it as is.

@marc2k3 CalcTextWidth will be affected in JSP as well, although over there at least GdiDrawText does not claim to return any values :)

This new code doesn't like large strings/being called repeatedly as is done by the EstimateLineWrap function. It freezes fb2k indefinitely until you kill it with task manager. Try the last.fm bio sample.

Yeah, I'm seeing the same thing. For some reason I've never been able to attach to FSM's dll's to debug exactly what's going wrong. I can understand it being slow, but doesn't make sense why it would lock up FB.
Simple solution would be to have EstimateLineWraps's recursive functions continue to always call GetTextExtentPoint32 I suppose.

Should be safe now as it only calls DrawText from CalcTextWidth, and in all other cases uses the less accurate version which should be perfectly fine.

This is buggy if the text contains ampersands. You need to combine DT_PREFIX with DT_CALCRECT when using DrawText.

D'oh. Should have caught that. Also added a check to skip this block if text.length is 1 or less. Kerning pairs don't matter if you can't have a kerning pair.

@kbuffington thanks for your contribution!
There are a few things that need to be changed before it can be merged though:

This is a breaking change, hence it must be moved under a separate argument (e. g. use_exact or smth) and should not be enabled by default.
This new argument must be added to docs.
DT_SINGLELINE should be added to DrawText call.

@kbuffington can you fix the PR, plz? So that I could include it in the release (which I'm planning to ship soon).

Yeah, I'll try to make some time for this. I'm not completely sure what you meant by: "DT_SINGLELINE should be added to DrawText call." but I haven't looked at the code in a month so maybe it'll make more sense when I do.

DT_SINGLELINE single line causes method to ignore new lines (i.e. \n), this way it would work the same as GetTextExtentPoint32

After looking at the code I understand what you were referencing and will add that.

Believe I've addressed everything, retested with new optional parameters and it all seems good.

Awesome, thanks!


Add Lee Algorithm: Shortest path in a maze

I propose to add Lee Algorithm which is use to find the shortest path in a maze.
Real Life Applications:

Maze Game
Solving Rubik's Cube faster
Map Technology
Social Network

I will add proper description, will update the readme in Graph's README.
The code will be in C++ & C
Please assign me this. @anjul2000 @plazzy99 @nakul-19

Don't do it in C

Don't do it in C

alright

I would like to contribute to the above problem in java as I am a participant of GSSOC'21. Kindly assign this to me.


datastore - ExampleTransaction_getOrCreate

https://github.com/GoogleCloudPlatform/golang-samples/blob/master/datastore/snippets/snippet_test.go#L612
The linked example is supposed to check if an entity exists, and if it does not exist create it. It doesn't seem to actually verify if the entity exists between the get and create.

That happens with the err != datastore.ErrNoSuchEntity check.


Memory leak

I created the following script to call a stored procedure on a remote mssql server (not under my control) using data from an http request. It's leaking memory with each request/query. Any suggestions?

var odbc = require("odbc");
var db = new odbc.Database();

db.open("DSN=fiscal;UID=<user>;PWD=<password>", function(err)
{
    if (err) {
        console.log(err.message);
        return;
    }

    require('http').createServer(function (req, res) {

        res.end();

        var parts = require('url').parse(req.url, true);
        var matches = parts.pathname.match('^/([a-zA-Z_]+)/$');

        if (matches) {

            var procedure = matches[1];
            var p = parts.query;

            var query = "declare @outcome int;"
                + " exec " + procedure + " " + p.bsID + ", " + p.ticketid + ", " + p.amount + ", @outcome OUT;"
                + " select @outcome";

            db.query(query, function(err, rows, moreResultSets)
            {
                if (err) {
                    console.log(err.message);
                } else {
                    console.log(rows);
                }
            });

        } else {
            console.log('Unable to parse request');
        }

    }).listen(8082, "<IP_ADDRESS>");
});

process.on('uncaughtException', function (err) {
	console.log('uncaughtException:' + err);
	console.log(err.stack);
});

I am using node 0.6.7 on CentOS5.

@Nibbler999, Thanks for reporting this, excellent find. I think I may have found the issue. There is an un-closed scope in Database::New()
If you want to try to fix this yourself before I push out a fix, change
Handle<Value> Database::New(const Arguments& args) {
  HandleScope scope;
  Database* dbo = new Database();
  dbo->Wrap(args.This());
  return args.This();
}

to
Handle<Value> Database::New(const Arguments& args) {
  HandleScope scope;
  Database* dbo = new Database();
  dbo->Wrap(args.This());
  scope.Close(Undefined());
  return args.This();
}

which is in src/Database.cpp on line 59 ( https://github.com/w1nk/node-odbc/blob/master/src/Database.cpp#L59 )

Scratch that, I messed up my test case. That is not the solution, sorry.

@w1nk committed the fix for this.
I will leave this open for a few days to hopefully hear back positive or negative results from @Nibbler999.

Awesome, thanks nibbler :).