do_not_package_if_src_was_modified spurious errors

do_not_package_if_src_was_modified is failing on macos CI with:
Expected: execs
    but: exited with exit code: 0

Cargo is not able to always detect a modification during packaging on low-resolution filesystems like HFS if the package is extracted and build.rs runs within 1 second.
This is caused by #6257.  Previously the timestamps of the original files were in the past, but now the timestamps are all set to the extraction time.
Some options to fix:

Change tar to optionally set timestamps, but ignore errors.
Disable do_not_package_if_src_was_modified on macos.
Change run_verify to use a better method for detecting modifications.
Revert #6257. Cargo now requires setting mtimes ever since #6484.
a. Note: I cannot reproduce #6238. I tried creating various FAT partitions on Windows 8 and Windows 10, and they all seemed to be able to set file times just fine.  How can I repro it?

Any thoughts/preferences?
I get about 20% failure locally.  Seen recently at

https://github.com/rust-lang/cargo/pull/6395#issuecomment-469054607 (log no longer available).
https://travis-ci.org/ehuss/cargo/jobs/501876478

#6395 (comment) (log no longer available).

https://github.com/travis-ci/travis-ci/issues/2993 logs available via email request

I'd be fine with either reverting or ignoring on osx, it's not super critical this test runs everywhere. That we require the ability to set mtime is a good point, although the main tarbal extraction point, $CARGO_HOME, is more likely to be in a "different and weird filesystem" than the build directory which may mean we should just ignore the test.

Verification is done in target/package, not CARGO_HOME.

@khionu Can you give me instructions for setting up a filesystem that reproduces #6238?  The change from #6257 is causing some CI problems, and I want to better understand what motivated the change. A few months ago, Cargo switched to requiring the ability to set mtimes, so I'm thinking unless there is a strong motivation for such filesystems, reverting #6257 might be the best option.  Supporting filesystems that can't adjust mtimes may take a fair bit of effort at this stage.


[SPARK-30070][SQL] Support datetimes predicate - overlaps

What changes were proposed in this pull request?
 * The operator `OVERLAPS` determines whether or not two chronological periods overlap in time. A
 * chronological period is specified by a pair of datetimes (starting and ending).
 *
 * If the length of the period is greater than 0, then the period consists of all points of time
 * greater than or equal to the lower endpoint, and less than the upper endpoint,
 * a.k.a [lower, upper).
 *
 * If the length of the period is equal to 0, then the period consists of a single point in time,
 * the lower endpoint, a.k.a [lower, lower].
 *
 * Two periods overlap if they have at least one point in common.

postgres=# select (cast(a as timestamp), cast(b as timestamp)) overlaps (cast(c as timestamp), cast(d as timestamp)) from (values
 ('2011-11-11', '2011-11-11', '2011-11-11', '2011-11-11'),
 ('2011-11-10', '2011-11-11', '2011-11-11', '2011-11-12'),
 ('2011-11-11', '2011-11-10', '2011-11-11', '2011-11-12'),
 ('2011-11-11', '2011-11-10', '2011-11-12', '2011-11-11'),
 ('2011-11-10', '2011-11-11', '2011-11-12', '2011-11-13'),
 ('2011-11-10', '2011-11-20', '2011-11-11', '2011-11-19'),
 ('2011-11-11', '2011-11-19', '2011-11-10', '2011-11-20')) t(a,b,c,d);
 overlaps
----------
 t
 f
 f
 f
 f
 t
 t
(7 rows)

Checked PostgreSQL, it support datetimes in pairs but without support for datetime + interval, so I decide we support the former and leave the latter to a followup.
Why are the changes needed?
ANSI support FeatureID: F053
Better PostgreSQL feature parity too.
Does this PR introduce any user-facing change?
Yes, add a datetime predicate.
How was this patch tested?

add ut

Test build #114574 has started for PR 26702 at commit fffcecd.

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/SparkPullRequestBuilder-K8s/19402/
Test PASSed.

Test build #114574 has finished for PR 26702 at commit fffcecd.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class UnresolvedDateTimeOverlaps(

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

retest this please

Test build #114583 has started for PR 26702 at commit fffcecd.

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/SparkPullRequestBuilder-K8s/19411/
Test PASSed.

Test build #114583 has finished for PR 26702 at commit fffcecd.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class UnresolvedDateTimeOverlaps(

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

The behaivour of the other systems (e.g., oracle and mysql) is the same with the pg one above? Can you check?

Test build #114595 has started for PR 26702 at commit 83ad8e4.

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/SparkPullRequestBuilder-K8s/19423/
Test PASSed.

Hi @maropu, the overlaps's behavior  in the SQL standard is very clear, and PostgreSQL follows it exactly.  So here we do exactly the same.
In Oracle, it is also supported but undocumented.
Not supported by MySQL.

I'm not faimilar with this syntax though, to regard the range as [lower, upper) is the same among them? Which the description below does come from, the pg doc or the SQL definition?
 /**
 * The operator `OVERLAPS` determines whether or not two chronological periods overlap in time. A
 * chronological period is specified by a pair of datetimes (starting and ending) or as a
 * starting datetime and an interval(resolved by analyzer to datetimes pairs early).
 *
 * If the length of the period is greater than 0, then the period consists of all points of time
 * greater than or equal to the lower endpoint, and less than the upper endpoint,
 * a.k.a [lower, upper).
 *
 * If the length of the period is equal to 0, then the period consists of a single point in time,
 * the lower endpoint, a.k.a [lower, lower].
 *
 * Two periods overlap if they have at least one point in common.
 *
 */

The comments above are based on the SQL Standard.
Here is the pg's description

This expression yields true when two time periods (defined by their endpoints) overlap, false when they do not overlap. The endpoints can be specified as pairs of dates, times, or time stamps; or as a date, time, or time stamp followed by an interval. When a pair of values is provided, either the start or the end can be written first; OVERLAPS automatically takes the earlier value of the pair as the start. Each time period is considered to represent the half-open interval start <= time < end, unless start and end are equal in which case it represents that single time instant. This means for instance that two time periods with only an endpoint in common do not overlap.
from link:
https://www.postgresql.org/docs/current/functions-datetime.html

Test build #114601 has started for PR 26702 at commit f4963f4.

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/SparkPullRequestBuilder-K8s/19429/
Test PASSed.

Test build #114595 has finished for PR 26702 at commit 83ad8e4.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/114595/
Test PASSed.

Test build #114601 has finished for PR 26702 at commit f4963f4.

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

retest this please

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

Test build #114613 has started for PR 26702 at commit f4963f4.

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/SparkPullRequestBuilder-K8s/19440/
Test PASSed.

Test build #114613 has finished for PR 26702 at commit f4963f4.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/114613/
Test PASSed.

cc @cloud-fan @gatorsmile, thanks for reviewing this.

Test build #114750 has started for PR 26702 at commit 3b39ec1.

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/SparkPullRequestBuilder-K8s/19572/
Test PASSed.

Test build #114750 has finished for PR 26702 at commit 3b39ec1.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/114750/
Test PASSed.


Code Chunk and Code Coverage Report show nothing after load covertege via PublishCodeCoverageResults

I use PublishCodeCoverageResults to publish code coverage report into tfs
But Code Chunk and Code Coverage reports are empty

What tool are you using to generate the CC? Can you paste the logs from the Publish CC task? Thanks!

I generate code coverage via https://github.com/mmanela/chutzpah (for typescript) or OpenCover for .NET
And convert into Cobertura
TFS Show me lines and brances in summary.
But Excel reports Code Chunk and Code Coverage show nothing
2016-06-17T09:51:45.0300285Z Executing the powershell script: c:\****\tasks\PublishCodeCoverageResults\1.0.3\PublishCodeCoverageResults.ps1
2016-06-17T09:51:45.8082990Z ##[debug]Entering PublishCodeCoverage.ps1
2016-06-17T09:51:45.8082990Z ##[debug]codeCoverageTool = Cobertura
2016-06-17T09:51:45.8092755Z ##[debug]summaryFileLocation = c:\*****\coverage-report.html.xml
2016-06-17T09:51:45.8092755Z ##[debug]reportDirectory =
2016-06-17T09:51:45.8102520Z ##[debug]additionalCodeCoverageFiles =
2016-06-17T09:51:45.8288055Z ##[debug]Loading module from path 'C:\****\agent\worker\Modules\Microsoft.TeamFoundation.DistributedTask.Task.CodeCoverage\Microsoft.TeamFoundation.DistributedTask.Task.CodeCoverage.dll'.
2016-06-17T09:51:45.8444295Z ##[debug]Importing cmdlet 'Enable-CodeCoverage'.
2016-06-17T09:51:45.8444295Z ##[debug]Importing cmdlet 'Publish-CodeCoverage'.
2016-06-17T09:51:45.8864190Z Starting 'Publish-CodeCoverage' cmdlet...
2016-06-17T09:51:46.3102200Z ##[debug]Fetched BuildId '4499'.
2016-06-17T09:51:46.3492800Z Publishing coverage summary data to TFS server.
2016-06-17T09:51:46.3824810Z ##[debug]PublishCoverageSummary with following parameters
2016-06-17T09:51:46.3824810Z ##[debug] project ******
2016-06-17T09:51:46.3834575Z ##[debug] buildId 4499
2016-06-17T09:51:46.3834575Z ##[debug] Lines- 1774 of 5471 covered.
2016-06-17T09:51:46.3844340Z ##[debug] Branches- 0 of 0 covered.
2016-06-17T09:51:47.8364895Z Publishing additional files to TFS server.
2016-06-17T09:51:47.8384425Z ##[debug]Creating build artifacts with following parameters
2016-06-17T09:51:47.8384425Z ##[debug] project ******
2016-06-17T09:51:47.8394190Z ##[debug] buildId 4499
2016-06-17T09:51:47.8394190Z ##[debug] type Container
2016-06-17T09:51:47.8394190Z ##[debug] name Code Coverage Report_4499
2016-06-17T09:51:47.8394190Z ##[debug] fileContainerPath C:\Users\****\AppData\Local\Temp\Code Coverage Report_4499
2016-06-17T09:51:47.8403955Z ##[debug] browsable False
2016-06-17T09:51:48.4009065Z Max Concurrent Uploads 4, Max Creators 1
2016-06-17T09:51:48.4516845Z Found 1 files to upload.
2016-06-17T09:51:48.4565670Z Files found locally 1,
2016-06-17T09:51:48.4565670Z Files evaluated 0,
2016-06-17T09:51:48.4565670Z Files left to evaluate 1.,
2016-06-17T09:51:48.4575435Z Files created without upload 0,
2016-06-17T09:51:48.4575435Z Files uploaded 0
2016-06-17T09:51:48.4575435Z Files left to process 1
2016-06-17T09:51:48.4575435Z ---------------------------
2016-06-17T09:51:50.4593685Z Created 0 files without uploading content. Total files processed 1
2016-06-17T09:51:50.4603450Z Uploaded artifact 'C:\Users\****\AppData\Local\Temp\Code Coverage Report_4499' to container folder 'Code Coverage Report_4499' of build 4499.
2016-06-17T09:51:52.2629640Z Associated artifact 94 with build 4499
2016-06-17T09:51:52.2658935Z ##[debug]Leaving script PublishCodeCoverage.ps1

Sorry, but it sounds like the results are being uploaded to the server fine. Are you then downloading them and opening in Excel? What is shown if the just open the HTML file locally? Thanks!

I talk about this reports https://msdn.microsoft.com/en-us/library/ms244661.aspx
And data in this tables:
https://msdn.microsoft.com/en-us/library/ee620645.aspx
https://msdn.microsoft.com/en-us/library/ee620639.aspx

There are two fields report directory and additionalCoverageFiles in the task. You can use them to specify files to upload to server. From the logs you have not specified them.

Please let us know if Ashwini's suggestion does not work.

I don't need upload reports and additionalCoverageFiles like artifacts.
I talk about TFS tables:
[TFS_Warehouse].[dbo].[FactRunCoverage]
[TFS_Warehouse].[dbo].[FactBuildCoverage]
And TFS Analisys


Add a launch file and/or documentation for writing a ROS map

This should be documented and/or turned into an example:
https://github.com/googlecartographer/cartographer_ros/issues/553#issuecomment-338669056

cartographer_pbstream_to_ros_map is now the simplest way to do it and seems to be used by people. Documentation improvements are always welcome though :)


Can we have an optional act_as model

I have a database that has two different behavior. In one scenario, I am saving content directly into that table and I do not want to have its entries as a part of the actable table. But for another scenario I want it to act as child table to actable. Does this gem provide support for this kind of behavior?

Closing due to inactivity. Feel free to reopen if this is still an issue!


(WIP) Shade cassandra-all-4.0.10 dep of cql module

What this PR does:
Tries to shade "cassandra-all" dependency of cql module to see if we can avoid exposing it to persistence backends.
Which issue(s) this PR fixes:
N/A
Checklist

[x] Changes manually tested
[ ] Automated Tests added/updated
[ ] Documentation added/updated
[x] CLA Signed: DataStax CLA

Superceded by actual PR #2683 (against 2.1 branch); closing this one.


Weird behavior with file browser when directory has seperate root directory

1. What version of Caddy are you using (caddy -version)?
0.11.0
2. What are you trying to do?
Have a directory with a different root, and be able to use look through the files with browse
3. What is your entire Caddyfile?
www.sunsetstation13.com sunsetstation13.com {
        proxy / https://sunsetstation.github.io
        proxy /sunset https://sunsetstation13.com/parsed-logs/sunset
}

www.sunsetstation13.com/parsed-logs sunsetstation13.com/parsed-logs {
        root /var/www/sunsetstation13.com/stats/parsed-logs
        browse
        gzip
}

4.  What did you expect to see?
Clicking on the sunset folder in sunsetstation13.com/parsed-logs should take me to sunsetstation13.com/parsed-logs/sunset
7. What did you see instead (give full error messages and/or log)?
Clicking on the sunset folder takes me to sunsetstation13.com/sunset
8. How can someone who is starting from scratch reproduce the bug as minimally as possible?
Literally copy the caddyfile config above and strip the proxy /sunset https://sunsetstation13.com/parsed-logs/sunset (current workaround i'm using)

Ah, I remember this one. It's not really a bug; Caddy is technically doing what you told it to do. Append your site definitions with a / forward slash, so www.sunsetstation13.com/parsed-logs/ sunsetstation13.com/parsed-logs/ and then it will know that there is a directory delineation there. The path in the address part of the Caddyfile is technically a prefix, not bounded by directory structure, since it might not be serving a directory structure.

Also, @mholt, this is completely unrelated, but how do I serve already gzipped .txt files and letting the browser un-zip them?

this is completely unrelated, but how do I serve already gzipped .txt files and letting the browser un-zip them, like what you'd get with nginx and gzip_static on;?

Don't use the gzip directive, then add this header to the response: Content-Encoding: gzip

@mholt Yeah but I can't find a way to apply headers exclusively to .gz files

and it breaks browse, @mholt


Why can't DGPs have zero-valued multipliers?

This is probably an ignorant question, so please feel free to direct me to a more appropriate venue to ask it.
I'm working on a problem that involves linear combinations of nonnegative variables, which are collected in a matrix for convenience. Sometimes I get a coefficient that is zero, but never less than zero, and the majority of the coefficients for these variables are positive.
When taking a linear combination, a zero coefficient is the same as simply leaving that term out of the sum. For example, if a[0] == 0 but a[k] > 0 for all 0 < k < len(a), then y = a.T @ x is equivalent to z = a[1:].T @ x[1:]; however, y.is_dgp() == False while z.is_dgp() == True.
Is there a reason that we cannot have zero-valued constants in affine transformations in DGPs? Is there some way we could build in a feature to accept DGPs with such constants, provided, for example, that no term involving a vector x is equal to zero?

It's a fine question, thanks for asking it.
You're of course right that, say, the value of 0*x + y is the same as y. However, as you noticed, the former expression won't parse as DGP, but the latter will.
The reason for this is that the DGP grammar requires that all expressions to be positive, since only positive expressions can be log-log convex (and the DGP grammar is a rule for combining log-log convex expressions). Because we make this assumption, CVXPY can safely take the logs of all constants in the programs when compiling a DGP problem to a conic (DCP) problem behind-the-scenes (this log transformation is necessary).
It might be possible to add special logic that strips out zero-valued subexpressions before running the Dgp2Dcp reduction. We already have one special case for expressions of the form x + 0, where 0 is the literal numeric constant 0.
I can tag this issue as a feature request, but I can't promise that it will get implemented soon.

@r-zip you may be interested in the idea of wraps, as illustrated here. You can assert symbolic properties of expressions by creating new wraps. Of course you're bypassing the DCP checking in some sense and might run into failure modes like what Akshay described where you're taking the log of zero. But for a sophisticated user it might be worth exploring.


rustPlatform.cargoSetupHook: improve cargoHash instructions

Description of changes
Currently cargo-setup-hook instructs the builder upon cargoSha256 or cargoHash being out-of-date compared to the Cargo.lock file.
The instructions can be simplified a bit, because nowadays it is fine to keep a hash empty, instead of filling it with<PHONE_NUMBER>000000000000000000000000000000000000000000.
Nix nowadays outputs SRI hashes, which should usually be placed in cargoHash instead of cargoSha256, but the instructions are still only referring to cargoSha256.
Lastly, the output of Nix doesn't include got: sha256:  anymore, as it now outputs got: sha256-XXXXXXXXX=. It would be nice to make it clear that the trailing = is important as well.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

This needs to go to staging.

Done :+1:

Also: can you use the PR description in your commit description(s)?

Done :+1:

Lastly, nit, but I think it would make sense to use cargoSetupHook or rustPlatform.cargoSetupHook (the former has been used in previous commits, the latter is used for other members of rustPlatform -- the latter is probably preferable I'd say), but definitely not cargo-setup-hook, as we tend to use attribute set names in commit messages when applicable (OfBorg does some things based on the attribute set name in a commit message -- even if they wouldn't apply here, I'd argue we shouldn't break convention).

I have used rustPlatform.cargoSetupHook instead of cargo-setup-hook :+1:

You've also pinged a lot of people during the rebase, which we can't unping.
Please reopen this.


Remove Docile from package

Ref #43. Seems like Docile is now redundant.

Thanks!

Would you mind tagging a new release too please?
On Thursday 11 August 2016, john verzani<EMAIL_ADDRESS>wrote:

Thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/JuliaLang/Roots.jl/pull/44#issuecomment-239156623,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIrgQWP5D4cTgxOi9a0BOpceijUvf1hZks5qex_ggaJpZM4Jh7Pd
.


User with Add Permission unable to Add

Started with fresh permission enabled djnango-cms in my local dev.
From SU admin panel added a new User (Page) (say: testuser) and granted him 'Add', 'Change', 'Delete' Permission on Page.

You have no permission on this page (Expected: Page Link with ADD, and CHANGE enabled)
Re-Login as SU
Add new Page, and go to its Page Permission and explicitle add the 'testuser' Grant him permission on all "Page & Descendant".
Re-Login as testuser
Page Link appeared but only with 'Change' enabled (Add Not Enabled)
Go to Page, Now You can Add, Modify in that root page or Add Child, but you still can't delete any thing.
i'm new to Django-CMS, so please correct me, if this functionality is correct or i'm doing something wrong, Also What's the significant of 'DELETE' permission or 'can_delete' permission, if i'm not getting that enabled.

Verification: using the steps provided above, I can replicate the issue.
Needs looking into, or clarification on why it's occuring.

Alright. I think I've narrowed down the problem: the admin help_text for Global Page Permissions says, about which Sites a user can modify, 'If none selected, user haves granted permissions to all sites.', however, that's not true, because of has_page_add_permissions, where the GlobalPagePermission queryset filters out valid permissions which aren't in this site (sites__in=[site]). This is true of has_page_change_permissions too, though that query pops exists() on the end, which is a  slightly saner result.
For the purposes of verification: Proceed with the ticket's outlined steps (to confirm the existance of the problem), then go and give the testuser access to sites explicitly. If @kvarun31 could confirm whether he had any sites selected, it would go a long way to confirming the problem area as the one I've described.


Input_number?

Is there a reason why input numbers are not accepted? I would like to use it to set variables on my touch interface...

Second that!


Hide UI

This revives an old commit which added functionality to hide the UI. Now the "hide" button works again.
Fixes https://github.com/webaverse/app/issues/3261

Works fine. Just these minor issues:
1-When logged in, the address doesn't goes away while we hide the UI
https://user-images.githubusercontent.com/35926530/179811729-f5f67dbb-1a3b-4c19-b813-fc8b63eb2914.mp4
2- This little ctrl+h makes the UI look out of sync. Sometimes it isn't even visible due to background. Can we remove it? or either ad it in to the hide button someway

@alisaad673 I have fixed these. I think that we should pull it in but maybe make a ticket to add a toast to notify the user that they can press ctrl+h to show UI again. For now it's probably only power users anyways.

Sure. Will make one.


[Bug] [SQL] SQL Task Node Custom Parameters run failed

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
when i using Custom Parameters of SQL Task Node with OUT type, sql query result is more than one row, in downstream sql using the parameters failed, if upstream sql result is one row, it work fine.
failed work flow:

work flow success

What you expected to happen
SQL Task Custom parameters work fine
How to reproduce
add two sql task node, in first node query select and add custom parameter OUT type, in donwstream sql using the parameters
Anything else
No response
Version
3.0.0-beta-1
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

here is the log:
[LOG-PATH]: /data/apache-dolphinscheduler-3.0.0-beta-1-bin/worker-server/logs/20220610/5803827748480_10-155-217.log, [HOST]:  Host{address='xxxx:1234', ip='xxxx', port=1234} [INFO] 2022-06-10 08:46:55.800 +0800 [taskAppId=TASK-20220610-5803827748480_10-155-217] TaskLogLogger-class org.apache.dolphinscheduler.plugin.task.sql.SqlTask:[120] - Full sql parameters: SqlParameters{type='MYSQL', datasource=1, sql='select name from test.userinfo1 where name in (${name})', sqlType=0, sendEmail=null, displayRows=10, limit=0, segmentSeparator=, udfs='null', showType='null', connParams='null', groupId='0', title='null', preStatements=[], postStatements=[]} [INFO] 2022-06-10 08:46:55.801 +0800 [taskAppId=TASK-20220610-5803827748480_10-155-217] TaskLogLogger-class org.apache.dolphinscheduler.plugin.task.sql.SqlTask:[121] - sql type : MYSQL, datasource : 1, sql : select name from test.userinfo1 where name in (${name}) , localParams : [],udfs : null,showType : null,connParams : null,varPool : [Property{prop='name', direct=IN, type=LIST, value='["lisi","zhangsan"]'}] ,query max result limit  0 [INFO] 2022-06-10 08:46:55.802 +0800 [taskAppId=TASK-20220610-5803827748480_10-155-217] TaskLogLogger-class org.apache.dolphinscheduler.plugin.task.sql.SqlTask:[91] - setSqlParamsMap: Property with paramName: name put in sqlParamsMap of content  select name from test.userinfo1 where name in (${name}) successfully. [INFO] 2022-06-10 08:46:55.802 +0800 [taskAppId=TASK-20220610-5803827748480_10-155-217] TaskLogLogger-class org.apache.dolphinscheduler.plugin.task.sql.SqlTask:[387] - after replace sql , preparing :  select name from test.userinfo1 where name in (?) [INFO] 2022-06-10 08:46:55.802 +0800 [taskAppId=TASK-20220610-5803827748480_10-155-217] TaskLogLogger-class org.apache.dolphinscheduler.plugin.task.sql.SqlTask:[396] - Sql Params are replaced sql , parameters:["lisi","zhangsan"](LIST) [INFO] 2022-06-10 08:46:55.802 +0800 [taskAppId=TASK-20220610-5803827748480_10-155-217] TaskLogLogger-class org.apache.dolphinscheduler.plugin.task.sql.SqlTask:[465] - can't find udf function resource [INFO] 2022-06-10 08:46:55.804 +0800 [taskAppId=TASK-20220610-5803827748480_10-155-217] TaskLogLogger-class org.apache.dolphinscheduler.plugin.task.sql.SqlTask:[369] - prepare statement replace sql : HikariProxyPreparedStatement@1139720980 wrapping com.mysql.cj.jdbc.ClientPreparedStatement:  select name from test.userinfo1 where name in (** NOT SPECIFIED **)  [INFO] 2022-06-10 08:46:55.805 +0800 [taskAppId=TASK-20220610-5803827748480_10-155-217] TaskLogLogger-class org.apache.dolphinscheduler.plugin.task.sql.SqlTask:[297] - main statement execute query, for sql:  select name from test.userinfo1 where name in (?) [ERROR] 2022-06-10 08:46:55.805 +0800 [taskAppId=TASK-20220610-5803827748480_10-155-217] TaskLogLogger-class org.apache.dolphinscheduler.plugin.task.sql.SqlTask:[213] - execute sql error: No value specified for parameter 1 [ERROR] 2022-06-10 08:46:55.805 +0800 [taskAppId=TASK-20220610-5803827748480_10-155-217] TaskLogLogger-class org.apache.dolphinscheduler.plugin.task.sql.SqlTask:[164] - sql task error: java.sql.SQLException: No value specified for parameter 1

I can try to solve the problem.

I can try to solve the problem.

Hi @stalary sorry for reply late, I had assigned to you.

Are you still work on this issue? @stalary

Are you still work on this issue? @stalary

I'm still doing it. You can do it first if you're in a hurry.

It looks like that SqlTask missed of processing parameters with type DataType.LIST
could we resolve this issue together？@stalary @zhongjiajie

It looks like that SqlTask missed of processing parameters with type DataType.LIST
could we resolve this issue together？@stalary @zhongjiajie

You can do with @stalary if he do not mind

Are you still work on this issue? @stalary

I'm still doing it. You can do it first if you're in a hurry.

If you still doing with this, please go ahead, we may consider releases in version 3.0.1, WDTY @caishunfeng

It looks like that SqlTask missed of processing parameters with type DataType.LIST could we resolve this issue together？@stalary @zhongjiajie

Do you have any ideas now that I'm still in debug

Sql Params are replaced sql , parameters:"x1","x2", I think we should replace [] to ()

I will modify ParameterUtils.setInParameter.

It looks like that SqlTask missed of processing parameters with type DataType.LIST could we resolve this issue together？@stalary @zhongjiajie

Do you have any ideas now that I'm still in debug

Transform parameter of type LIST to VARCHAR；
Format sql statement with condition IN.

It looks like that SqlTask missed of processing parameters with type DataType.LIST could we resolve this issue together？@stalary @zhongjiajie

Do you have any ideas now that I'm still in debug

Transform parameter of type LIST to VARCHAR；
Format sql statement with condition IN.

I have submitted the code, you can help review it~
https://github.com/apache/dolphinscheduler/pull/11285


Launch Screen, Home Screen, Consent and Login

added launch and home screen modifications

added the baseline survey to the app


Setup issue and PR templates

With this PR I have finally taken the time to fix our PR template and write other issue templates.
Once they are accepted here, I will file PRs in all our repos as well so please discuss the templates here.
(I added all of you so this gets some visibility and possibly discussion going)
closes #17

I do not like the current help wanted template. The about text does not really make sense and the template content is not actually a template but an information for the issue poster that should rather be moved to the readme.
I would prefer to make the help wanted template similar to the one for bugs:

Describe the problem
Describe steps to reproduce
Describe context


feature support array type

Proposed changes
Issue Number: close #xxx

remove AbstractDataType
support generice on function signature
support array and map type in type coercion and type check
enable array type in Nereids
add element_at and element_slice syntax in Nereids parser
remove FROM from nonReserved keyword list

TODO:

support lambda expression
add below array function

array_apply
array_concat
array_filter
array_sortby
array_exists
array_first_index
array_last_index
array_count
array_shuffle shuffle
array_pushfront
array_pushback
array_repeat
array_zip
reverse
concat_ws
split_by_string
bitmap_from_array
bitmap_to_array
multi_search_all_positions
multi_match_any
tokenize

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 46.25 seconds
stream load tsv:          511 seconds loaded<PHONE_NUMBER>9 Bytes, about 139 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          66 seconds loaded<PHONE_NUMBER> Bytes, about 15 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.4 seconds inserted 10000000 Rows, about 340K ops/s
storage size:<PHONE_NUMBER>4 Bytes

run buildall

run buildall

run buildall

run buildall

run buildall

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 46.79 seconds
stream load tsv:          545 seconds loaded<PHONE_NUMBER>9 Bytes, about 130 MB/s
stream load json:         21 seconds loaded<PHONE_NUMBER> Bytes, about 107 MB/s
stream load orc:          66 seconds loaded<PHONE_NUMBER> Bytes, about 15 MB/s
stream load parquet:          32 seconds loaded 861443392 Bytes, about 25 MB/s
insert into select:          29.3 seconds inserted 10000000 Rows, about 341K ops/s
storage size:<PHONE_NUMBER>5 Bytes

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 47.82 seconds
stream load tsv:          538 seconds loaded<PHONE_NUMBER>9 Bytes, about 132 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          30 seconds loaded 861443392 Bytes, about 27 MB/s
insert into select:          29.7 seconds inserted 10000000 Rows, about 336K ops/s
storage size:<PHONE_NUMBER>8 Bytes

run buildall


[Apple Notes] Image file name conflict results in "(error reading attachment)"

When importing from Apple Notes, "(error reading attachment)" may occur.
I suspect that this is related to a naming conflict of images when they're placed in the same attachments folder. I have a dozen or so images named "pasted graphic.png" (you can search this in the Apple Notes search field), and they are all coming up with that error.
I'm not sure if the solution here is to rename the filename (i.e. "pasted graphic-01.png") during import?

This also appears to affect other attachments with duplicate names, such as html.

Same here, with .TIFF, amongst others

Same here,
I have a myriad of PDF documents that I just scanned, so they are called "Scanned documents" and it failed to import many of them correctly when they are on the same note.
Also, when fixing this, keep in mind that "filenames" in notes can have characters that are not valid filenames (like / or \n), and those notes work on iOS and, they don't work on macOS "sometimes"  (at least last time I tried, they might've fixed it in the last months, it was still broken around November 2023)

Appears to be fixed in #310

Great, thank you for the update!


use zerolog.LevelWriter interface for performance

If we log something (e.g. as local log level got set to debug) but we only want to send events for Error to sentry, each event is parsed to check for the log level.
As there is the LevelWriter interface as zerolog suggests for multiple outputs here. That way we might skip parsing the event if the log level is disabled.
This PR implements this.
For reference I implemented benchmarks as well. This are the results on my device
# make benchmarks
[...]
pkg: github.com/archdx/zerolog-sentry
BenchmarkParseLogEvent-8                          272346              3806 ns/op            5136 B/op         14 allocs/op
BenchmarkParseLogEvent_DisabledLevel-8            253910              4014 ns/op            5136 B/op         14 allocs/op
BenchmarkWriteLogEvent-8                           65269             21795 ns/op           12586 B/op        109 allocs/op
BenchmarkWriteLogEvent_Disabled-8               20960934             57.01 ns/op               0 B/op          0 allocs/op
BenchmarkWriteLogLevelEvent-8                      68061             16895 ns/op           12618 B/op        109 allocs/op
BenchmarkWriteLogLevelEvent_DisabledLevel-8    167343889             6.876 ns/op               0 B/op          0 allocs/op
PASS
ok      github.com/archdx/zerolog-sentry        9.532s

Codecov Report

Merging #9 (fde250b) into master (22731d7) will decrease coverage by 5.73%.
The diff coverage is 18.51%.

@@            Coverage Diff             @@
##           master       #9      +/-   ##
==========================================
- Coverage   53.17%   47.44%   -5.73%
==========================================
  Files           1        1
  Lines         126      137      +11
==========================================
- Hits           67       65       -2
- Misses         51       67      +16
+ Partials        8        5       -3

Impacted Files
Coverage Δ

writer.go
47.44% <18.51%> (-5.73%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Do some dreaming about a roadmap

Still relevant?

Building up the command line tool to support custom environments:
Something like lorri init python


this._extend is not a function

What am I doing wrong here?  I've included jpeg_camera_with_dependencies.js.

JS file:
$(function() {
    if (!window.JpegCamera) {
        alert("Neither getUserMedia nor Flash are available!");
    }
    else {
        var takeSnapshotButton = document.getElementById('takePhoto');
        takeSnapshotButton.addEventListener("click", function(e) {

        }, false);

        // either HTML5 or Flash are available
        var camera = JpegCamera("#camera", {}).ready(function () {
            camera.show_stream();

            var snapshot = camera.capture().show()
        }).error(function () {
            alert("Camera access was declined.");
        });
    }
});

I'm sorry, but I cannot provide this kind of debugging support. Try posting your question on StackOverflow.

+1 in rails 5.1.2

If you have similar problem, fix it by adding "new" keyword before JpegCamera as it's a class and not a function.
$(function() {
    if (!window.JpegCamera) {
        alert("Neither getUserMedia nor Flash are available!");
    }
    else {
        // either HTML5 or Flash are available
        var camera = new JpegCamera("#camera", {}).ready(function () {
            camera.show_stream();

            var snapshot = camera.capture().show()
        }).error(function () {
            alert("Camera access was declined.");
        });
    }
});

This problem happened because of a mistake in the documentations.
https://amw.github.io/jpeg_camera/doc/class/JpegCamera.html
Please fix it. Thanks


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dependencies-1.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 1220

Questions ?
Please refer to the Backport tool documentation


Release v0.2.0

We added quite a few contributions in since the last release.

[ ] Release and test a candidate for 0.2.0 release.
[ ] Release v0.2.0

/assign
/cc @levikobi @mlavacca @robscott @shaneutt only FYI


Endpoint for The icelandic football status table.

LGTM other from those two comments! :dart:

This looks good to me, it seems that @MiniGod has taken a peek at this :eyes: and the tests are passing so I'm merging this!

Thanks for this @alexanderbsig! :tada:

:cake: for me  :smile:
When will it be live?
Do I add the documentation for apis.is myself?

It auto deploys when all tests are passing. However some data endpoints are broken and need to be fixed.

Allright sweet :smile:  what about the documentation for apis.is?

@benediktvaldez can help you out there

Awesome stuff! Not sure how to proceed myself, since this is v1 stuff, and as stated in apis-is/docs#6 I have no idea where that is located..
@alexanderbsig I'll let you know when I know more! :rabbit:

TBH it is a bit strange setup since we did not go through with the beta thing. The current "live" docs are located in this repo on the gh-pages branch


Added Mode Toggler

Related Issue
#560
Description
Made the mode toggler with SVG images that changes based on the mode.
Type of PR

[ ] Bug fix
[x] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
Uploading Screen Recording 2024-07-19 201325.mp4…
Checklist:

[x] I have performed a self-review of my code
[x] I have read and followed the Contribution Guidelines.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.

Additional context:
This pr is under GSSoC 2024. Please label it with appropriate labels.

@CSEdgeOff please review and merge this pr

@VijaySamant4368 Add the screenshots of the outputs in different views

After that small change, the Screenshots would be as follows:
At full screen

Width 1500px

Width 1000px

Width 500px

Width 300px

Although making it responsive for even smaller screen width is unnecessary, I can make it responsive, but I would have to change the other parts of the source-code.

@Durgesh-Vaigandla, please kindly check.

@VijaySamant4368 in mobile view incorporate the button inside the navbar I mean if the nav is open the button would display

@Durgesh-Vaigandla, do you mean that I have to add the Toggle-Button in the drop down menu?
"If the user press the three lines(hamburger) button, then only the Toggle Mode should show" Is this what you mean?

@VijaySamant4368 yeah exactly

@Durgesh-Vaigandla it may take some time.

Should I make it such that it is outside till the width is say 500px, but if it is less than that, it goes inside.
As the nav-bar would have a lot of empty space.
@Durgesh-Vaigandla

@Durgesh-Vaigandla will these suffice?
For width less than 1100px, it is still same
The nav-Bar

Inside Hamburger

Pressing the 'Toggle Mode' toggles the mode and closes the hamburger.

@Durgesh-Vaigandla please label this as GSSoC Level 3

@Durgesh-Vaigandla does these changes look good, or is there any other specification that I should do?

@Durgesh-Vaigandla @CSEdgeOff please checkout this pr

@Durgesh-Vaigandla @CSEdgeOff  please kindly checkout this pr


OS problems with testthat::expect_snapshot()

In one of these expectations, I got an error on my laptop - thinking the expectation was not up to date. However, this is an OS problem I think.
Here you can see that all tests pass on mac-os and fail on others.
This test is disabled this for now & needs an update. Solution would probably be rounding the coefs and do an approx comparison with the snapshot.
Relevant commit: ca3ced79

Hmh, that's odd.  I don't know how to see which test(s) failed in the GitHub diagnostics (I only see which OSes it failed on?)...
Two other points though:

The tests all pass for me at e8e9de50a4 when I run devtools::test() on my Ubuntu 20.04 laptop.
The change at 31f6a5843f15 fails the "twofold results with bootstraps haven't changed" test for me -- I keep getting the old values.

Do we know whether this is the test that's failing on the other OSes?
As you say @sinanpl, if the test is only off by a tiny amount we could change the tolerance (though it looks like we might need to use testthat::expect_snapshot_value() for that, or just round the numbers ourselves).  However if the difference is larger, or if it's the problem I mention above about the difference between e8e9de50a4 and 31f6a5843f15, then I'm not sure how to debug this further?
FWIW, here's the full bootstrap distribution I get when I run that test interactively (and, for reference, the summary, which is what fails the test):

Click to expand long results
> twofold$bootstraps$dists$gaps
$gap
    estimate
1  0.1970309
2  0.1885666
3  0.1881771
4  0.1932397
5  0.2206101
6  0.1911830
7  0.1966584
8  0.1512249
9  0.2399630
10 0.2123248

$pct_gap
     estimate
1  0.07290854
2  0.07089820
3  0.06964412
4  0.07205452
5  0.08146208
6  0.06936620
7  0.07221689
8  0.05679332
9  0.08833806
10 0.07876546

$EY_a
   estimate
1  2.702439
2  2.659682
3  2.701981
4  2.681854
5  2.708132
6  2.756141
7  2.723163
8  2.662723
9  2.716417
10 2.695659

$EY_b
   estimate
1  2.505408
2  2.471115
3  2.513804
4  2.488615
5  2.487522
6  2.564958
7  2.526505
8  2.511498
9  2.476454
10 2.483334

> summary(twofold)
Oaxaca Blinder Decomposition model
----------------------------------
Type: twofold
Formula: ln_real_wage ~ age + education | female
Data: chicago_long

Descriptives
            n    %n mean(ln_real_wage)
female==0 412 57.9%                2.7
female==1 300 42.1%                2.5

Gap: 0.21
% Diff: 7.62%
              coefficient   % of gap           se          2.5%         97.5%
explained           -0.03     -14.1% 2.038143e-02 -7.192541e-02 -1.736619e-02
unexplained          0.23     114.1% 2.090135e-02  2.049169e-01  2.636727e-01
unexplained_a        0.00      -0.0% 8.519047e-16 -1.898993e-15  7.903563e-16
unexplained_b        0.23     114.1% 2.090135e-02  2.049169e-01  2.636727e-01

The analysis I'm working on now is only using threefold results, so this possible error in the twofold results aren't a super high priority for me at the moment...  But of course @sinanpl let me know if I can help check anything for you on this?

Whoops, wrong bootstraps.  Here are (I think) the ones that are going wrong in that test:

Click to expand
> twofold$bootstraps$dists$overall
$explained
      estimate
1  -0.01685624
2  -0.02217687
3  -0.07595069
4  -0.05076081
5  -0.01912269
6  -0.05806056
7  -0.04147327
8  -0.05200049
9  -0.02214232
10 -0.02316519

$unexplained
    estimate
1  0.2138871
2  0.2107435
3  0.2641278
4  0.2440005
5  0.2397328
6  0.2492436
7  0.2381317
8  0.2032254
9  0.2621053
10 0.2354900

$unexplained_a
        estimate
1  -9.704694e-16
2  -1.996233e-15
3  -7.096103e-16
4  -7.945034e-16
5   1.100899e-15
6  -2.792905e-16
7  -1.351946e-15
8  -4.536302e-16
9  -1.564057e-15
10 -3.400058e-16

$unexplained_b
    estimate
1  0.2138871
2  0.2107435
3  0.2641278
4  0.2440005
5  0.2397328
6  0.2492436
7  0.2381317
8  0.2032254
9  0.2621053
10 0.2354900

I don't know how to see which test(s) failed in the GitHub diagnostics (I only see which OSes it failed on?)...

Ah, never mind, apparently I was blocking the JS that did this.  So I can see the results now.  I may have to revisit the rest of my comment in light of that then lol.

Okay, that's really strange -- it looks like that one line of unexplained_a bootstraps changes considerably across all the OSes (including, it seems, mine and yours @sinanpl)...  That makes me think that something's going wrong with that calculation beyond just the floating point tolerance.  I'll have a look.

Okay, looking at it more closely I'm coming back @sinanpl to your point of view that this in fact a rounding/tolerance issue: I forgot that the estimates that all look very different are in fact extremely close to zero and so only differ after the first nine decimal places or so.  (In fact, I get different results when I run the test interactively compared to when I run it with devtools::test().
So I'm glad that it doesn't look like an error in the calculation code.  But like I say above, unfortunately it doesn't look like testthat::snapshot() has a tolerance option, so we'll have to get a little cleverer to get this test to run again.

#48 solves this with rounding


Center data within points thumbnail

Description
This is the minimum amount of work I could do to fix the bug described in #1450, where the points thumbnail did not center the data as desired/expected.
The bug existed because Points._update_thumbnail fills out a 32x32 image regardless of the shape of the points layer's data extent. By contrast, ImageBase._update_thumbnail returns an image where one of the dimensions should be 32 and the other will be in [1, 32] depending on the anisotropy of the shape of the image.
The fix calculates the non-isotropic thumbnail_shape in Points._update_thumbnail in the same way that Image._update_thumbnail does. Padding and centering happens in Layer.thumbnail.setter. Using the same example in the bug description, this seems to work.

There is some opportunity for code sharing between the different implementations of update_thumbnail, though I haven't attempted that here yet. I also believe there are similar bugs to this one for Vectors, Tracks, and maybe Shapes (note that Surface does not have a thumbnail).
Do we want to expand this fix to handle those, or should I open a separate issue(s) with examples?
In the long term, I'm wondering if we can reuse vispy to render thumbnails (e.g. using offscreen rendering) to avoid maintaining different rendering pipelines in Layer._update_thumbnail. I think that might be an overall easier and more maintainable solution.
Type of change

[x] Bug-fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Based on the code added, I think this should be a safe bug fix because the coordinate/thumbnail indexing is clipped to be in bounds.
References
Closes #1450.
How has this been tested?

[x] example: added a test to test_points.py to cover this case.
[ ] example: all existing tests pass with my change

Final checklist:

[x] My PR is the minimum possible work for the desired functionality
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[ ] If I included new strings, I have used trans. to make them localizable.
For more information see our translations guide.

@jni : I assigned this to you as you opened the original issue, but feel free to add/reassign anyone.

Do we want to expand this fix to handle those, or should I open a separate issue(s) with examples?
In the long term, I'm wondering if we can reuse vispy to render thumbnails (e.g. using offscreen rendering) to avoid maintaining different rendering pipelines in Layer._update_thumbnail. I think that might be an overall easier and more maintainable solution.

I've opened #3409 to track the same problem with shape layer, but did not open an issue about letting vispy to the thumbnail. Do you want to do it ?

Thanks @Carreau.
Can I update #3409 to clarify that Vectors and maybe/probably Tracks are affected by the same issue? Or should I open a separate issue for each layer?
My feeling is that we should probably come up with a general (and less fractured/fragile) implementation for _update_thumbnail that would fix all these issues on all layers, and maybe add a thumbnail for Surface too. Reusing our vispy layer implementations (that we need anyway) to do that seems like one potentially good way to do that, but there could be others.

Yes, feel free to update all you want.
On Wed, Sep 22, 2021, 11:59 Andy Sweet @.***> wrote:

Thanks @Carreau https://github.com/Carreau.
Can I update #3409 https://github.com/napari/napari/issues/3409 to
clarify that Vectors and maybe/probably Tracks are affected by the same
issue? Or should I open a separate issue for each layer?
My feeling is that we should probably come up with a general (and less
fractured/fragile) implementation for _update_thumbnail that would fix
all these issues on all layers, and maybe add a thumbnail for Surface
too. Reusing our vispy layer implementations (that we need anyway) to do
that seems like one potentially good way to do that, but there could be
others.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/napari/napari/pull/3406#issuecomment-925218732, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AACR5T53TZDIJMXFZGAYQCDUDIRSTANCNFSM5EPSNHJQ
.


Identity "Generation" is unused

Entities have a "generation" associated with them but it is always set to 1 since nothing passes a generation in to the constructor. https://github.com/Byteron/HypEcs/blob/2cc54cd794e598d03d7b8e63364006a9baf268bd/src/Identity.cs#L27-L32
Inside Despawn() we queue up the "unused" id without incrementing the generation https://github.com/Byteron/HypEcs/blob/2cc54cd794e598d03d7b8e63364006a9baf268bd/src/Archetypes.cs#L75
and then inside Spawn() we pop that id without incrementing the generation https://github.com/Byteron/HypEcs/blob/2cc54cd794e598d03d7b8e63364006a9baf268bd/src/Archetypes.cs#L37
Additionally, functions like IsAlive() do not check the generation at all https://github.com/Byteron/HypEcs/blob/2cc54cd794e598d03d7b8e63364006a9baf268bd/src/Archetypes.cs#L283-L287
It's easy to reproduce this issue since the despawned ids are used immediately
var entity = world.Spawn().Id();
world.Despawn(entity);
var anotherEntity = world.Spawn().Id();
Assert.IsFalse(entity == anotherEntity); //entity == anotherEntity will return true
Assert.IsTrue(world.IsAlive(anotherEntity));
Assert.IsFalse(world.IsAlive(entity)); // This will of course also be true unexpectedly

The same issue exists in RelEcs https://github.com/Byteron/RelEcs/issues/39

This PR fixes the issue:
https://github.com/Byteron/HypEcs/pull/5


setting warning 'Matches multiple schemas when only one must validate.'

settings.json:
"stylusSupremacy.sortProperties": "grouped",
file: 'file:///Users/sldobri/Library/Application%20Support/Code/User/settings.json'
severity: 'Warning'
message: 'Matches multiple schemas when only one must validate.'
at: '130,37'
source: ''

Thank you for reporting the issue.
Please spare me 1-3 days for investigation and patching.

Would you mind sharing sample Stylus files? Or, this issue may happen regardless of Stylus code.

the extension works fine like settings show, but the warning is in option selected.

do you see the erro show only in viscose setting when select "grouped"

Sorry, I was terribly busy, but I will check this out on Monday. Please bear with me.

It appears that JSON schema for Stylus Supremacy settings was written incorrectly. I will patch this tomorrow.

The extension version 2.0.0 is out, which this issue has been addressed.

Cheers!

{
"version": "0.2.0",
"configurations": [
{
"name": "Debug Android",
"program": "${workspaceRoot}/.vscode/App.js",
"type": "reactnative",
"request": "launch",
"platform": "android",
"sourceMaps": true
}
]

}
why second curley bracket shown matches multiple schemas when only one must validate. how to resolve it in VS Code


Escaped symbols ([, $, etc.) should be rendered as regular text

Escaped symbols (\[, \$, etc.) are meant to be regular text.
The problem is that they currently appear in the same muted color as special characters ([, $, *, etc.).
They should be rendered in the same color as regular text, so as to be more legible (especially in dark mode).
Thanks!

OK, here is the solution to this issue:
.cm-s-obsidian span.cm-hmd-escape-char {
  color: var(--text-normal);
}


New Sample - Search Movies

Q
A

Bug fix?
no

New feature?
no

New sample?
yes

Solution Details

Q
A

Solution requires premium license
yes

Solution uses experimental features (see licensing guidelines)
no

Which connectors are used?
OMDb Connector

Solution uses on-premises connectors?
no

Solution uses custom connectors?
yes

Layout
Phone

What's in this Pull Request?
This is a new sample.
Sample Summary: This sample allows users to search for movies using any keyword and/or the year of release

Hi @aaryan2134 -
Thanks for the sample! Nice to see a sample that uses the Independent Publisher Connector in there!
Cheers,
Daniel

Thank you!


Not able to search for topic tag in the News schema

I'm not able to search for topic tags in the News template, but I can search for country tags.

LGTM :thumbsup:


Update redjubjub to 0.5 (or next crates.io release)

Motivation
In https://github.com/ZcashFoundation/zebra/pull/2351 we are temporarily pointing to a commit in the main branch of redjubjub. We should point to the next release after it's published.
Specifications
N/A
Designs
N/A
Solution
Update the redjubjub to its next release.
Alternatives
Keep pointing to the commit, but that should be avoided
Related Work
Leftover from #2351

Duplicate of #2952.


update composer  bootstrap4

please update required config of composer "yiisoft/yii2-bootstrap4": "~1.0@dev", with "~2.0"

Done, ty:
https://github.com/simialbi/yii2-widget-datetimepicker/releases/tag/2.4.0


Adds distribution using builds

This pull requests closes #65, closes #64, closes #62, closes #67, closes #41:

Makes takeout have 0 dependencies.
Avoids global conflits
This pull request already contains a pre-built version with the tag v1.0.1, so you just have to:

# Merge this pull request, and then locally:
git checkout main
git pull
git tag v1.0.1
git push --tags

Note: For next versions, before tagging a new version, you need to perform : ./takeout app:build. 🚀

Thanks so much @nunomaduro!


move lz4 to a separate package

lz4 support is already optional and can be handled with the current extension system. As mentioned https://github.com/asdf-format/asdf/pull/1570#issuecomment-1611051442 moving this to a separate package (such as asdf-compressors) would only require a minor dependency change for users:
pip install asdf-compressors[lz4]
instead of
pip install asdf[lz4]
This has a few benefits:

simplifying the core library
making the use of this optional compressor explicit in written file (by having lz4 handled by an extension, an entry in 'history.extensions' will be created containing version information about the extension that supplied the lz4 compressor)
this external package provides a concrete example of how to provide a custom block compressor to asdf
this package can be expanded to include other compression algorithms (like lz4 frame and zstd as proposed for asdf in https://github.com/asdf-format/asdf/pull/1571 and https://github.com/asdf-format/asdf/pull/1570).

A few folks are out on vacation but I'd love to hear thoughts from @eslavich and @perrygreenfield (when available).

Not sure what the naming conventions are for asdf related packages, but we might consider other names like asdf-compression or asdf-compress. Maybe asdf-compressors implies that the decompressors are missing :)

Just pinging as a reminder...

Thanks! I expect this will be at least another month. Much of the work is going into prepping for asdf 3.0 which is nearing completion. See the 3.0 roadmap if you're curious:
https://github.com/asdf-format/asdf/wiki/3.0-Roadmap
I did list the compression updates on the general roadmap (note this is generally not used but is currently up-to-date):
https://github.com/asdf-format/asdf/wiki/Roadmap


Non-existent apt source configured on Raspbian

Cookbook version
6.1.1
Chef-client version
13.3.42
Platform Details
Raspbian
Scenario:
On Debian platform family, an apt source file nginx.list is created with http://nginx.org/packages/#{node['platform']}. This results as http://nginx.org/packages/raspbian on Raspbian, however it does not exists (http://nginx.org/packages/debian works just fine)
Steps to Reproduce:
Run this cookbook on Raspberry PI with Raspbian
Expected Result:
Sucessful chef-client run with proper apt source.
Actual Result:
The chef-client fails when running apt: source http://nginx.org/packages/raspbian does not exists.

A workaround is to add the following attribute in your cookbook:
default['nginx']['upstream_repository'] = node['platform'] == 'raspbian' ? 'http://nginx.org/packages/debian' : default['nginx']['upstream_repository']
I'll try and make a pull request to fix this

@PierreBeucher We currently only support platforms that we can test within Test Kitchen either using Vagrant or Dokken images. This allows us to have full CI for each PR and is critical to not just supporting a platform at a point in time, but maintaining that support over time. We don't have any way to continue testing Raspbian support and thus I can't accept a new platform for this cookbook. You can either handle the repo setup in a wrapper cookbook or set the attributes directly.


Selection and ObservableCollection issue

Reproduction steps (i used ControlCatalog):

Add a "Delete" button to the TabControl page:

<Button Command="{Binding DeleteTab}" Content="Delete"/>

Change List to ObservableCollection:

private class PageViewModel : ReactiveObject
{
    private ObservableCollection<TabItemViewModel> _tabs;

    private Dock _tabPlacement;

    public ObservableCollection<TabItemViewModel> Tabs
    {
        get => _tabs;
        set => this.RaiseAndSetIfChanged(ref _tabs, value);
    }

    public Dock TabPlacement
    {
        get { return _tabPlacement; }
        set { this.RaiseAndSetIfChanged(ref _tabPlacement, value); }
    }

    private void DeleteTab()
    {
        Tabs.RemoveAt(0);
    }
}

Launch the application, click on the button and this will happen:

I'm guessing this was broken after the global changes related to the SelectionModel, since TabControl hasn't changed
Used the most recent preview of Avalonia

Thanks for the excellent repro! On it now.

Perhaps this information can be useful.
If you delete any item before the selection, shows the previous item, not the selected one before deletion.
The deleted item is displayed here:

And here is the previous one:

When removing the last item, it highlights the first one (I don't think this will be critical).


fluentd build, deploy and config for sending app errors to discord

Description
PR contains the Openshift templates and instructions for build/deploy of a Fluentd container.
The fluentd container receives logs from a fluent-bit sidecar on a CDOGS node application and outputs error notifications to the Showcase Team's Discord channel.
This is part of a rough proof of concept for monitoring.
PR for CDOGS sidecar:
See wiki for more info.
Types of changes

New feature (non-breaking change which adds functionality)
Documentation (non-breaking change with enhancements to documentation)

Checklist

[x] I have read the CONTRIBUTING doc
[ ] I have checked that unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have updated the OpenAPI 3.0 v*.api-spec.yaml documentation (if appropriate)
[ ] I have added necessary documentation (if appropriate)

Further comments
For demonstration purposes, notifications are triggered on 4** response codes (rather than 5**).

@jujaga after some struggles with /bin/bash script, I've fixed the liveness and readiness probes.
the ruby processes count should be 1 (the demonset runs 2)
see latest commit.
so those checks are now working.


[CHALLENGE SUBMISSION] Challenge 17

Please provide the following information for your submission:
What is your name? (First, Last)
Deepak Dhami
Where is your GitHub Repo ?(With your challenge solution)
https://github.com/DexterPOSH/25dayofserverless/tree/master/day17IoT
What Challenge is this for? (Challenge number between 1 and 25)
17
(Optional) Anything else we should know? e.g., language used, location, blog post?
Azure functions in Python, input binding is the eventhub for the Azure IoT Hub.
It parses the data and does quick validation and then sends out tweet using Twython py package.
Funny, when people noticed these automated tweets, while I was testing the API.

From all of us on the Cloud Advocacy team at Microsoft, thank you for creating this issue!
Have a Happy Holiday season!
Feel free to share your solution on Twitter with the tag #25DaysOfServerless


Incorrect SQL query generated by dropIndex in migration

What steps will reproduce the problem?

In database PortgreSQL create schema myproject

In schema create table mytable

In table create fields id:PKint and category_id:int

create index on field category_id with name myproject_categoryid_index

Write migration with next code

$this->dropIndex(‘myproject_categoryid_index’, 'myproject.mytable');

run php yii migrate

What is the expected result?
must be generated SQL with schema
so it must be like this
DROP INDEX myproject.myproject_categoryid_index RESTRICT
and migration executed succesful
What do you get instead?
$this->dropIndex(‘myproject_categoryid_index’, 'myproject.mytable');
this line of code generate SQL:
DROP INDEX myproject_categoryid_index
without schema!
and when executed get error:
 > drop index myproject_categoryid_index on myproject.mytable ...Exception: SQLSTATE[42704]: Undefined object: 7 ОШИБКА:  индекс "myproject_categoryid_index" не существует
Additional info

Q
A

Yii version
<IP_ADDRESS>?

PHP version
7.2

Operating system
Debian

But in the error trace the schema exists! The problem is a nonexistent index on table. You check

Don't work
$this->dropIndex(‘myproject_categoryid_index’, 'myproject.mytable');
Work
Yii::$app->db->createCommand(‘DROP INDEX myproject.myproject_categoryid_index RESTRICT')->execute();

Because -
$this->dropIndex(‘myproject_categoryid_index’, 'myproject.mytable');
this line of code generate SQL:
DROP INDEX myproject_categoryid_index

it use default 'public' schema in Postgre SQl but in my case i use custom 'myproject' schema


Provider: Salesforce Commerce Cloud

Provider: Salesforce Commerce Cloud
Ecomerce Provider: https://www.salesforce.com/products/commerce-cloud/ecommerce/
Assigned to: @cellmanguney

Hi @okbel Yes I can take the Salesforce Commerce Cloud implementation part!

Hi @okbel Yes I can take the Salesforce Commerce Cloud implementation part!

Lovely!! Thank you.

We need these (utilities)[https://github.com/vercel/commerce/tree/agnostic#structure]
Submit a PR adding a folder under /framework that exposes those functions.
Once agnostic branch is merged, everybody will be able to use this provider.
Let me know if you have any questions. Also reacheable at Twitter @okbel.

Lovely!! Thank you.

We need these (utilities)[https://github.com/vercel/commerce/tree/agnostic#structure]
Submit a PR adding a folder under /framework that exposes those functions.
Once agnostic branch is merged, everybody will be able to use this provider.
Let me know if you have any questions. Also reacheable at Twitter @okbel.

Hi @okbel, @cellmanguney
Do we have already any progress in the scope of SFCC provider integration? If this can be possible, I would like to join the integration efforts and contribute there

Hi @delros
I have started the integration roughly, set a on-demand Sandbox with commerce API ready with ShopAPI user. Which can connect now.
I couldn't find a time for deep dive yet ), any effort would be more than welcome.

@cellmanguney ok, that's sounds good enough :) let's start and try to collaborate then. Can we have short call or chat? Where can I contact you?

Hi @cellmanguney, do you need any help with this? We will be starting a new project soon and I don't want to reinvent the wheel :)

@cellmanguney , @delros Did you have a chance to sync up on this? I am from same company with @Trimud here and we have experience in sfcc headless refapps dev would be great to have SFCC provider in Next Commerce as well. Please let us know how we can contribute and what is current status on your side.

hi @MarkiyanPyts , @Trimud
That would be great if you can participate the development, so far I couldn't make time to dive in, you can take the lead.
Also I'm fine to take part of code-reviews and fix some parts.
do you have access to Vercel Slack  ? if not @okbel can we invite @MarkiyanPyts @Trimud @delros to Vercel Slack?
We can create a channel there and keep in touch more and follow up all the process, maybe have some meetings.

Hello, @cellmanguney thanks for answering,
No we don't have access to slack yet but would be great to have access to be able to better follow what's going on,
Then, let's see where we are,
short status meeting would be good to see what is the current Repo branches and so on

@Trimud
I will try to send invite to Slack, if not probably @okbel can fix that.
Meanwhile let's make a hangout call for plan. I will send you an email (the one at your github landing page)
so we can continue from there

Hi @cellmanguney and @okbel ,
I would be happy to contribute as well. Can I also join to the team/slack channel etc?

Absolutely, can you hand me your emails through Twitter DMs? @okbel @MarkiyanPyts @ugurtemiz @Trimud

Hi @okbel I hope you have received my message in Twitter?

Hi @okbel do you have any updates?

@okbel ,  @cellmanguney  Is there already a timeline that you have in mind?

We're actively working on this but if there's any available contributor who wants to help ping me. @okbel.

Hi @okbel
I believe I have pinged you several times already but I didn't get anything back from you :)

@trimud ;) check your twitter messages


Shelly BLU Door/Window, Button presses other than single normal press doesn't register

Describe the bug
When trying to trigger anything other than normal "button press" like double, triple, long, long double, or long triple, home assistant never get any other event than the normal "press" using a "Shelly BLU Door/Window" sensor. Doesn't matter how long you hold the button, how long/quick you press the button repeatedly or how many times. it never generates any event other than normal "press"
To Reproduce

Press the button on a "Shelly BLU Door/Window" sensor in any combination of presses or holding times
Listen to the "bthome_ble_event" in home assistant
Notice that only the single button press come thru (if you hold or press quickly it does NOT come thru as a normal single press, it literally just doesn't register)

What firmware is installed on the device?
For Home Assistant issues please create an issue here: https://github.com/home-assistant/core/issues/new?assignees=&labels=&projects=&template=bug_report.yml as it helps tracking the version and getting logs.

What firmware is installed on the device? For Home Assistant issues please create an issue here: https://github.com/home-assistant/core/issues/new?assignees=&labels=&projects=&template=bug_report.yml as it helps tracking the version and getting logs.

The sensors firmware is: 1.0.20
Home assistant is newest stable version
And i do not believe its a problem with home assistant since BTHome doesn't create the events for all the other press types, so i am quite sure its with BTHome and not homeassistant

As explained above, tracking issues and the relevant versions is easier in Home assistant issues, it doesn't matter if the issue is related to the library or Home Assistant. In most of the libraries developed only for Home assistant we even block creating an issue for the library.
I overlooked at first sight that your report is for Shelly BLU Door, I thought it is for the Shelly BLU button. As far as I remember shelly BLU Door only supports single press events, I will check it an report back.

As explained above, tracking issues and the relevant versions is easier in Home assistant issues, it doesn't matter if the issue is related to the library or Home Assistant. In most of the libraries developed only for Home assistant we even block creating an issue for the library.
I overlooked at first sight that your report is for Shelly BLU Door, I thought it is for the Shelly BLU button. As far as I remember shelly BLU Door only supports single press events, I will check it an report back.

When in automation, it shows the options to trigger from Shelly BLU Door with the different press types, so if it not supported it would be nice if it didnt show those options :)

I have validated with the Shelly BLE app that the device only sends single press events. Currently there is no way to know which events are supported by device, we had an issue about it https://github.com/home-assistant/core/issues/116245 but did not find a solution.


Guidelines/conventions for algo/UDP descriptions

This came up while reviewing a couple of description fixup PRs by @rahmandawibowo-vito :

#49
#50
#54
#56
#57

It looks we're currently quite inconsistent with how algorithm and/or descriptions are handled:

"eurac_pv_farm_detection":

the algorithm has a description at  algorithm_catalog/eurac_pv_farm_detection/eurac_pv_farm_detection_description.md\
UDP has another descroption at openeo_udp/eurac_pv_farm_detection/README.md, which is copied (by generate.py) into openeo_udp/eurac_pv_farm_detection/eurac_pv_farm_detection.json
#57 tries to fix this by duplicating the decsription from algorithm_catalog/eurac_pv_farm_detection/eurac_pv_farm_detection_description.md to openeo_udp/eurac_pv_farm_detection/README.md . Note that that readme references an image pv_ml_output.png which is not duplicated, so the latter readme is kind of broken. The image could also be duplicated of course, but duplication like this is of course not ideal and will lead to long term maintainability problems.

"sentinel1_stats": same kind of problem as "eurac_pv_farm_detection"
"max_ndvi_composite" uses different solution: the UDP generate.py script explicitly links to the description markdown file from the algorithm description algorithm_catalog/max_ndvi_composite_files/max_ndvi_composite_description.md. Which eliminates the duplication problem, but instead causes a weird coupling issue: the UDP depends on a resource from the algorithm folder, while the algorithm itself actually doesn't directly use this resource.

tl;dr I think there is kind of confusion about whether what is described (algorithm vs UDP) and where it should be described (algorithm or UDP related folders).

apparently @rahmandawibowo-vito  raised basically the same concern at the same time in #61


enable icount on arc qemu

additional commits on top of #26646

original PR approved + timeout fixes for CI

original PR approved + timeout fixes for CI


Exit editor with non-zero return code if --build-solutions fails

While writing some automation for building my C#-based game with Godot Mono, I noticed that when using --build-solutions via CLI,  failed C# builds don't affect the exit code. So this reduces the ability to automate the build process of a game, because to know if the game failed, you would have to try to search for failure messages in the output.
I have another version of this change for 3.x which I tested using the builds from GitHub Actions. The change is identical in the two branches.
Where godot-mono is the official release of 3.5.1:
godot-mono  --build-solutions --no-window --verbose --path ~/Desktop/MyGame/ || echo "failed with code $?"
# [...]
ERROR: An EditorPlugin build callback failed.
   at: call_build (editor/editor_node.cpp:5532)
ERROR: Command line option --build-solutions was passed, but the build callback failed. Aborting.
# [...]
Orphan StringName: SplashScreenFinish
Orphan StringName: Invoke
StringName: 4 unclaimed string names at exit.
# it did not print 'failed with code'

And this is  my 3.x branch build
./godot.x11.opt.tools.64.mono  --build-solutions --no-window --verbose --path ~/Desktop/MyGame/ || echo "failed with code $?"
# [...]
ERROR: An EditorPlugin build callback failed.
   at: call_build (editor/editor_node.cpp:5532)
ERROR: Command line option --build-solutions was passed, but the build callback failed. Aborting.
# [...]
Orphan StringName: Invoke
StringName: 4 unclaimed string names at exit.
failed with code 1
# exit code was set to 1

I wanted to also fail --export and --export debug commands if this is thrown in ExportPlugin.cs
throw new InvalidOperationException("Failed to build project.");
Since that also exits 0 for me. But I wonder if any exception thrown in that file should be failing the build? Not sure where to influence the return code during --export if ExportPlugin.cs fails.
Thanks to Calinou in the contributors chat for the tips
Edit: force pushed to get rid of merge commit

Thanks! And congrats for your first merged Godot contribution :tada:


Adding Directionality into MCF

for graph 3 and 7, I was expected the same edges to be on the output, but that did not happen. *reminder to see why later

with the updates to the code, graph 3 and 7 are still not the same

Let's make a v0.1 release before merging this so that this version of MCF can become v0.2. That will make it easier for the Dockerfile in SPRAS to refer to versioned releases of this code.

Ready for code review

Ready for code review again


Update authentication.md

Fix broken link to 2FA
Description of the change:
2FA page link at the bottom of https://sendgrid.com/docs/for-developers/sending-email/authentication/ was missing a second forward slash after https causing it to attempt to navigate to https://sendgrid.com/sendgrid.com/docs/ui/account-and-settings/two-factor-authentication/
Reason for the change:
Broken link
Link to original source:

Closes #

Hello @seanmateer,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX


Add response phase

Purpose
Allow plugins to use the response phase. This provides feature parity with other PDKs like Go.

 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.

Thank you @danielpoonwj !


Dear PyGui showcase

Hey,
We noticed that you made a great app with Dear PyGui and I just wanted to let you know that it is now featured on the Dear PyGui showcase page.
https://github.com/hoffstadt/DearPyGui/wiki/Dear-PyGui-Showcase#dearbagplayer
Do let us know if you make more apps with Dear PyGui and thanks for building this cool looking app!

Have you considered posting the app on r/python, r/robotics or other subreddit? I'm sure people would upvote that!

@DataExplorerUser Thank you so much for putting this project on the Dear PyGui showcase page! I really love the way Dear PyGui creates widgets, callbacks and handlers which is pretty convenient. Will consider posting stuff in the future, thanks for your suggestion!

You are more than welcome. I have passed on your appreciation to the core developers. If you have suggestions for improvements of Dear PyGui, e.g. features that are missing, be sure to file an issue on GitHub. And if have ideas where DPG could be applied and promoted, let us know!

Will do!

@s95050937 I have created a new repo to store demo rosbag files for testing DearBagPlayer. See #9 or the new README for more details. Cheers!


Viewscreen

Discussed in https://github.com/Thorium-Sim/thorium-nova/discussions/41 and https://github.com/Thorium-Sim/thorium-nova/discussions/320
This issue encapsulates all of the features needed for the first iteration of ship's viewscreen. This isn't meant to be all-inclusive, just enough to get started.

[x] #421
[x] #415
[x] #416
[x] #417
[x] #420
[x] #418
[ ] Viewscreen - Create a system for displaying overlays on the viewscreen.
[ ] Viewscreen - Create a card for the crew to control what is displayed on the viewscreen.
[ ] Viewscreen - Create a widget that allows any crew member to view the viewscreen on their computer.

In part of design for this. One of the frustrating things when participating in a bridge sim is that the Flight Director has full control of the viewscreen. This is addressing that issue certainly, but the controls for what is on the viewscreen should be in full control of the crew.

Yes, crew should be able to affect what's on the view screen. The challenge is that the view screen is one of the primary ways the crew gets information about stuff they should be focusing on. So there's probably going to be a good bit of automation, where the simulation overrides the crew's view screen choices to show information that's crucial to the mission.
This could take many forms, including a notification indicating the crew to choose to adjust their view screen to the view that the simulation wants.
How all that shakes out is TBD, but you're on the right track. Not necessarily full control, but crew does have control.

If there were alerts guiding people to the correct thing, that could also help. I'm basing this off of StarCraft. Almost the entire time you have full control of where you are looking at. There are alerts and buttons that pop up that can direct your attention to something and on occasion forces you to look at a specific thing.
With Nova being more crew driven with a flight enhancer for those who use it, having the ability to give maximum control to crew would also help immersion.

Thinking through it, I imagine that several stations can send information to the main viewscreen. For example, if there are interesting readings from operations, they can show it onscreen for everyone to see. The pilot can show the outboard cameras so planets can be seen and much more. It becomes a collective space where the captain can direct what they want to see at the moment. This helps fulfill the fantasy of the captain giving the order "on screen" and they can have everyone view what is happening.
This also let's the captain talk to individuals for information that doesn't need the full attention of everyone.
An example of this was when I was last at CMSC. I was first officer and I triaged information for the captain where possible because there is a ton of information coming in all the time. I could speak with different officers at their station and if anything needed a decision from the captain I could bring it up and the captain could have it put on the viewscreen.
Another consideration is that for flights without a director, every station will need access to the viewscreen individually which is something we can set up as a base line.
This is getting into conversations of what the "standard" ship size should be and while it is a blast to play with 8-10 people. A feasible amount for at home play would be 3-4. But that conversation is best held elsewhere and another time, right now we are just getting functionality coded.

I was curious what choices we would go for. Starting with small and large configuration is a good place for sure and most of those questions can't really be answered until we get some prototype and start testing.
The only other thing I want to make clear, since part of the design is for people to play in their homes or even over the internet, it is expected that most players will only have one monitor available to them so having a viewscreen button available to people is certainly a good thing to keep in mind. But we'll leave most of that conversation for when more parts are starting to come together, as I might be wrong altogether.


[oneDNN-aarch64] Update threadpool scheduler to make local thread act…

…ivate and deactivate tp
Resolves segmentation fault in https://github.com/tensorflow/tensorflow/pull/58071/
Signed-off-by: cfRod<EMAIL_ADDRESS>

@penpornk

I noticed that the Arm CI failed because two patches "//third_party/mkl_dnn:onednn_reorder_padded.patch",  and "//third_party/mkl_dnn:onednn_reorder_update.patch", are not on upstream TensorFlow. I will updated the patch

@cfRod @elfringham Do you know why the ARM CI is skipped here? (This was after I had explicitly clicked "Rerun all jobs" on the CI page.) Should I create a duplicate PR just to test? Thank you!

@penpornk The ARM_CI job was actually scheduled twice, once for each label added 'kokoro:force-run' and 'ready to pull'. The one that shows here is for 'ready to pull' which it is set to ignore. The ARM_CI run for 'kokoro:force-run' is, as I write this, still running. https://github.com/tensorflow/tensorflow/actions/runs/3937186516.
It is unfortunate that you cannot see both ARM_CI jobs in this PR's status box.

@elfringham Thank you very much for the explanation!


Docs: Add matcher examples

This PR resolves #41 . Adding examples to matchers of:

[x] Assertion
[x] ArrayAssertion
[x] BooleanAssertion
[x] FunctionAssertion
[x] DateAssertion
[x] ErrorAssertion
[x] NumberAssertion
[x] ObjectAssertion
[x] PromiseAssertion
[x] StringAssertion

Thank you for reviewing my PR @JoseLion ! I added a commit with your suggestions.


HOTFIX: Fix tag non hashable

Fixes #1131
Developers certificate of origin

[X] I certify that this contribution is covered by the MIT license as defined in our LICENSE and adheres to the Developer Certificate of Origin.

@apayne97 @kaminow could I grab a review here?


update node-pre-gyp version to ^0.14.0

Fixes #1264 - Installing with --target= results with "Unsupported target version" for targets >= 11

@mceIdo, I see one core issue with upgrading to this release of this package: Node 4 and 5 are still supported for Clang-3.5 in the travis config file. The minimum version that this upgrade supports is 6, so that would need to be one of the first things that needs to be resolved.

please review this pull request
i need latest node-pre-gyp to install prebuilt binaries from local path due to corporate firewall issues.

The problem is: kewde, the maintainer, tried upgrading to that version only to have it break something. He never documented what happened but did close the PR he tried setting up. I don’t know enough about how all of this works yet to be able to know what the issue was but do know it’s currently not viable yet.
Could you test this out, @mrsimb? I’ll get around it to myself once able. Just working extra to make sure the Covid-19 pandemic doesn’t make me homeless if my city goes under lockdown.

The real problem atm is that every pull request is failing because some of appveyor ci jobs always fail.
These jobs try to download binaries for electron but the server it downloads from sends 403 error.
I think someone with access to ci config should just disable these jobs so we could continue to receive updates of this library. People who depend on binaries build by these jobs will have to build them from source.

The real problem atm is that every pull request is failing because some of appveyor ci jobs always fail.
These jobs try to download binaries for electron but the server it downloads from sends 403 error.
I think someone with access to ci config should just disable these jobs so we could continue to receive updates of this library.

There might be other issues with it too outside of the tests failing. I’m not a full maintainer but am here to assist with getting things working properly. I just don’t have full permissions yet and the main maintainers are trying to get things going as quickly as they can within their means. Thanks for your patience and we’ll get this sorted out as quickly as we’re able.

Would it be possible to get this change reviewed/fixed soon ?

It should be noted that node-pre-gyp fixed a double run error so this upgrade would be awesome as otherwise it makes installing this project normally (npm install) impossible for me if a remote source is not available.
Ideally, I'd love to see this backported to the 4.2.0 release series as well.


Composite build fails when dependencies are locked and global dependency substitution rules are disabled

Current Behavior
./gradlew build fails for a composite build when dependencies are locked and global dependency substitution rules are disabled.
When the root build has a dependency on the included build, gradle writes the dependency to the lockfile (if dependency substitution is disabled). But when reading the lockfile it seems to filter out this dependency (even though dependency substitution is disabled).
Thus the build fails with the following error:
> Could not resolve all files for configuration ':compileClasspath'.
   > Resolved 'at.ctrlbreak:library1:1.0' which is not part of the dependency lock state

Expected Behavior
The composite build should not fail
Context (optional)
This seems to happen because when loading the lockfile in org.gradle.internal.locking.DefaultDependencyLockingProvider#loadLockState(java.lang.String) the dependency is not added to the result. The root cause seems to be that org.gradle.internal.locking.DefaultDependencyLockingProvider#isSubstitutedInComposite ignores the fact that dependency substitution is disabled and returns true.
I found a workaround/hack that fixes the build (at least with gradle 8.7):
Defining a dependency substitution for the included build that just substitutes an imaginary module with the library1 project ('includeBuild("library1") { dependencySubstitution { substitute(module("foo:bar")).using(project(":")) } }') makes the build work.
This seems to work because in org.gradle.composite.internal.IncludedBuildDependencySubstitutionsBuilder#build(org.gradle.internal.build.IncludedBuildState) the available modules are not added to the context if the substitution rules may add project dependency (which is achieved by adding the substitution of the imaginary module). In that case the subsitution rule actions are registered instead of adding the available modules.
Not sure how long this workaround will work, as I'm not sure it this behavior is intended or a bug on it's own that might be fixed any time.
Steps to Reproduce
Assume the following setup:

The root project (call it reproducer-project) has an included build (call it library1).
Reproducer-project has a dependency on library1 but global dependency substitution is disabled for all configurations in reproducer-project (configurations.all { resolutionStrategy.useGlobalDependencySubstitutionRules = false }).
Dependency locking is activated for all configurations in reproducer-project (configurations.all { resolutionStrategy.activateDependencyLocking() }).
The lockfile for reproducer-project has been written by executing ./gradlew dependencies --write-locks
Library1 has been published to a repository that is configured in reproducer-project

When trying to build the project now by calling (./gradlew build) the build fails because it resolves library1 with (according to the error message) is not part of the dependency lock state. When having a look at the gradle.lockfile one can however see that it actually is part of the lockfile.
A project demonstrating the bug has been prepared at https://github.com/strguntbr/gradle-composite-build-bug
Gradle version
8.7
Build scan URL (optional)
No response
Your Environment (optional)
No response

The issue is in the backlog of the relevant team and is prioritized by them.

Related:

https://github.com/gradle/gradle/issues/16254


mlb not working, new mlb.tv api?

I think mlb is using a new api as EplusTV wont play any mlb stream as of today.  I found this over here for mlbserver if that helps:
https://github.com/tonywagner/mlbserver/issues/44
statusText: 'Unauthorized',
headers: AxiosHeaders {
  'content-type': 'application/json; charset=utf-8',
  'transfer-encoding': 'chunked',
  connection: 'close',
  date: 'Fri, 19 Jul 2024 23:12:05 GMT',
  'x-dss-edge': 'auth.bad-partner',
  'www-authenticate': 'Bearer code="access-token.invalid", description="auth.bad-partner"',
  'x-request-id': '6d7bbbd6c1396bc61ae094eae07b4ed2',
  'x-bamtech-region': 'us-east-1',
  'cache-control': 'no-store',
  'x-cache': 'Error from cloudfront',

This is the same issue I have too. Mlb.tv Kodi addon just recently fixed this issue but not eplustv yet.

Should be fixed in v2.1.13 - that being said you have to delete your entries.db and schedule.db files before running the update


feat: E11215009 30oct exercise

E11215009,Aiali: https://github.com/raymundlin/threathunting202309/blob/main/E11215009%2CAilali.yaml
E11215006,Celeste: https://github.com/raymundlin/threathunting202309/blob/main/E11215006%2CCeleste.yml
Checks

[ ] Your YAML file has required fields as listed in Specs
[ ] All checks have passed yaml-lint passed
[ ] You agree to contribute your work under MIT license

E11215009,Aiali: https://github.com/raymundlin/threathunting202309/blob/main/E11215009%2CAilali.yaml
E11215006,Celeste: https://github.com/raymundlin/threathunting202309/blob/main/E11215006%2CCeleste.yml

WhoIS, E done


Expected parent IDs to be set when ordering by parent ID.

Hi Prisma Team! My Prisma Client just crashed. This is the report:
Versions

Name
Version

Node
v16.13.1

OS
windows

Prisma Client
4.9.0

Query Engine
ceb5c99003b99c9ee2c1d2e618e359c14aef2ea5

Database
mongodb

Logs
prisma:tryLoadEnv Environment variables loaded from C:\Users\k\Desktop\imessage-gql\backend\.env
prisma:tryLoadEnv Environment variables loaded from C:\Users\k\Desktop\imessage-gql\backend\.env
prisma:client dirname C:\Users\k\Desktop\imessage-gql\backend\node_modules\.prisma\client
prisma:client relativePath ..\..\..\src\prisma
prisma:client cwd C:\Users\k\Desktop\imessage-gql\backend\src\prisma
prisma:client clientVersion 4.9.0
prisma:client clientEngineType library
prisma:client:libraryEngine internalSetup
prisma:client:libraryEngine:loader Searching for Query Engine Library in C:\Users\k\Desktop\imessage-gql\backend\node_modules\.prisma\client
prisma:client:libraryEngine:loader loadEngine using C:\Users\k\Desktop\imessage-gql\backend\node_modules\.prisma\client\query_engine-windows.dll.node
prisma:client:libraryEngine library starting
prisma:client:libraryEngine library started
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true

Client Snippet
// PLEASE FILL YOUR CODE SNIPPET HERE

Schema
// PLEASE ADD YOUR SCHEMA HERE IF POSSIBLE

Prisma Engine Query
{"X":{}}

Hi, I don't see any errors in this report, can you send more informations about your problem please ?

Also, I see this is likely a duplicate issue, so I'm going to close this in favor of https://github.com/prisma/prisma/issues/19122. We encourage you to share any additional information in there. Thanks!


jython: 2.7.1 -> 2.7.2b2

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/jython/versions.
meta.description for jython is: '"Python interpreter written in Java"'.
meta.homepage for jython is: '"https://jython.org/"

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

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

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

found 2.7.2b2 with grep in /nix/store/612bs3hz4lzawzfrkkvjk2xb36v0mbmc-jython-2.7.2b2

directory tree listing: https://gist.github.com/07c4e473449c0f61acf5b16e32ad774e

du listing: https://gist.github.com/dec6e8c1b23bb12c4e1a9c959b4df3bd

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

3 total rebuild path(s)
1 package rebuild(s)
1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
jython

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/612bs3hz4lzawzfrkkvjk2xb36v0mbmc-jython-2.7.2b2 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A jython https://github.com/r-ryantm/nixpkgs/archive/5bd1df5c6074c9d3e4d4136dea853b35f8390596.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/612bs3hz4lzawzfrkkvjk2xb36v0mbmc-jython-2.7.2b2
ls -la /nix/store/612bs3hz4lzawzfrkkvjk2xb36v0mbmc-jython-2.7.2b2/bin

Closing because it's a beta release