Expose build mode in environment of asset transformer processes

In service of https://github.com/flutter/flutter/issues/143348
When invoking a package to transform an asset, we set FLUTTER_BUILD_MODE to the CLI name of the build mode being used. Inspired by https://github.com/flutter/flutter/issues/101077#issuecomment-1890379501:

Do transformers know whether they get executed in debug or release mode? I kinda imagine that being useful. Ex: There's a transformer that optimizes the file size of images. Depending on the amount and size of the images, that could take a significant amount of time. Therefore, I might want to only execute it in release builds.

Note for the reviewer: the interesting part of this change can be found in the commit set environment variable to build mode when running asset transformer…. The rest of the change is updating call sites with a new argument.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] I followed the breaking change policy and added Data Driven Fixes where supported.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

I do think a better solution would be to allow users to define separate asset configurations per build mode. For purely demonstrative purposes, consider this example fictional pubspec:
flutter:
  assets:
    - path: assets/hello.png
      buildModes: ["debug"]
    - path: assets/hello.png
      buildModes: ["profile", "release"]
      transformers:
        - package: optimizer
          args: --quality 95

However, this would be 1) harder to implement and maintain 2) would call for a bunch of design discussion. I think this PR is still useful as a stop gap. I imagine, for example, an image optimization transformer could provide an option like "--release-only" which tells the transformer to just copy the file as-is when not running in release mode.
flutter:
  assets:
    - path: assets/hello.png
      transformers:
        - package: optimizer
          args: --release-only --quality 95

Reason for revert: compilation issue has turned the tree red

Reason for revert: compilation issue has turned the tree red

For posterity can you link a failing a build?

Reason for revert: compilation issue has turned the tree red

For posterity can you link a failing a build? (This is something I always check when reviewing re-lands)

https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket/8753729103951007409/+/u/flutter_config_--clear-features/stdout
Building flutter tool...
Resolving dependencies...
Got dependencies.
flutter/packages/flutter_tools/lib/src/bundle_builder.dart:177:61: Error: Required named parameter 'buildMode' must be provided.
  final AssetTransformer assetTransformer = AssetTransformer(
                                                            ^
flutter/packages/flutter_tools/lib/src/build_system/tools/asset_transformer.dart:22:3: Context: Found this candidate, but the arguments don't match.
  AssetTransformer({

@CaseyHillers is it possible to get the presubmit google testing run for this PR? I see the check passed but it failed postsubmit

@CaseyHillers is it possible to get the presubmit google testing run for this PR? I see the check passed but it failed postsubmit

The PR branch was stale—it branched off master before https://github.com/flutter/flutter/pull/144734 landed. That PR introduced a new AssetTransformer call site, while this PR modified the signature of AssetTransformer. I'm assuming this is why the checks initially passed.

The PR branch was stale—it branched off master before #144734 landed. That PR introduced a new AssetTransformer call site, while this PR modified the signature of AssetTransformer. I'm assuming this is why the checks initially passed.

Interesting. Thanks for the explanation @andrewkolos!


Add support for x-authentication-scheme and Bearer auth scheme in apiKey security definition

Description
As seen in issue #124, many users require support for JWT, specifically using the Bearer authentication scheme.
Using JWT seems like a very common scenario, as are various non-Basic authentication schemes that use the Bearer authentication scheme as basis.
A hackish solution was offered in that thread, but the resultant experience isn't very comfortable and requires you to edit the Connexion source files (hence it is a problem to be based on the latest version of connexion).
I suggest a vendor-specific setting, x-authentication-scheme, in the security definition might suffice:
securityDefinitions:
  jwt:
    type: apiKey
    name: Authorization
    in: header
    x-authentication-scheme: Bearer

This will cause the API key set in the Authorize dialog to be treated as a JWT token, hence when it is part of a Authorization header the prefix "Bearer " will be appended.

Related issue #406


Split the readme

The readme file is already huge, I'd suggest to keep only reasonable info in the readme and split other parts with examples into the small md files. What do you think, @stefanjudis ?

Love that! Great idea! :+1:
Am 21.12.2015 9:47 nachm. schrieb "Uladzimir Havenchyk" <
<EMAIL_ADDRESS>
The readme file is already huge, I'd suggest to keep only reasonable info
in the readme and split other parts with examples into the small md files
What do you think, @stefanjudis https://github.com/stefanjudis ?
—
Reply to this email directly or view it on GitHub
https://github.com/loopline-systems/electron-builder/issues/101.


Translation "dom-elements" in Turkish.

Hi,
Please review the document. Let me know about the possible improvements or the unintended misleading contexts.

Hello @ozcanzaferayan,
Thanks for your inputs. I adapted the required changes. And pushed the latest branch.


Same sample in Windows 10 IoT Core and c#

Dear our current project it is developed 100% on C#language and Windows 10 IoT Core.
Any one has migrated current project to C# language.
Any alert about run this way?
Thanks
Kind regards

Here is an official sample by the windows team that does exactly that: https://github.com/microsoft/Windows-iotcore-samples/blob/1fb6eb4eaf5e488ab4ab544fe7b2abceab7e8afe/Samples/EdgeModules/README.md


New KataCoda Scenario - Migration Lab

Following the framework of our KubeVirt Migration Lab I have created a Katacoda scenario that uses the two nodes of a Katacoda 1.18 k8s cluster to perform a live migration of a Cirros VM while running a netcat-in-a-while-loop web "server".
Reviewers may try before merge at: My Katacoda Profile

Hi, while @mazzystr is out, I wonder if @codificat @iranzo would care to take a look, review, and possibly approve?
CC @jobbler @aglitke

/lgtm
/approve

@cwilkers Nice Job!  Claim your prize on the email list.  When you do I will follow up with a tweet.

Tweet sent .... https://twitter.com/kubevirt/status/1432460980567482373


Mvp

Merge branch mvp to main
add minimum viable capabilities (file system backend)

Get instruction
Create instruction
Update instruction
Delete instruction

Unit test passing ✅

MVP achieved packaged on pypi


VR in viro core

Hi guys, how can I set a HelloWorld VR scene by using viro core instead of viro react?
Can I seek for advices?

Hi @HLneoh ,
Have you seen our Hello World projects from our Getting Started page here -> https://virocore.viromedia.com/docs/getting-started ? If you download the HelloWorld projects here, you'll see we have three samples - virocore-helloworld-vr, virocore-helloworld-ar & virocore-helloworld-scene. You can see how to set and use a VR scene in ViroActivityVR in there.
Thanks,

Hi @manbod, thanks for the comment.
How can I replace the "HELLO WORLD" text with an image (jpg/png)?
So that when I click on the image, I can enter a new VR scene. Is it possible to make it?


Update audio element

Related to #409

Thanks Christian 🙇
Updated the post and going to squash and merge now.


Fix empty marker issues

This PR addresses a family of related problems we were seeing when depending on particular packages.
Based on what I saw in the codebase, there is a concept of an "empty" marker. However there were a few places in the code that didn't handle these empty markers properly.
Here's an example pyproject.toml where we were hitting the issue:
[tool.poetry]
name = "example_project"
version = "1.0.0"
description = ""
authors = [""]

[tool.poetry.dependencies]
python = ">=2.7, != 3.0.*, != 3.1.*, != 3.2.*, != 3.3.*"

[tool.poetry.dev-dependencies]
pytest = "^3.0.6"
pylama = "^7.6"
pylint = { version = "^2.2", python = ">= 3.4" }
pytest-cov = "^2.6"
black = { version = "18.9b0", python = ">= 3.6" }
sphinx = { version = "^2.0", python = ">= 3.5" }

[build-system]
requires = ["poetry>=0.12"]
build-backend = "poetry.masonry.api"

Doing poetry install once to generate the poetry.lock and then running poetry install again will reproduce the issue:
$ poetry install
Installing dependencies from lock file

[InvalidRequirement]
Invalid requirement, parse error at "'; <empty'"

install [--no-dev] [--dry-run] [-E|--extras EXTRAS] [--develop DEVELOP]

Also I believe this fixes #978 which seems to be hitting a similar problem.
Pull Request Check List
This is just a reminder about the most common mistakes. Please make sure that you tick all appropriate boxes.  But please read our contribution guide at least once, it will save you unnecessary review cycles!

[x] Added tests for changed code.
[ ] Updated documentation for changed code.

Note: If your Pull Request introduces a new feature or changes the current behavior, it should be based
on the develop branch. If it's a bug fix or only a documentation update, it should be based on the master branch.
If you have any questions to any of the points above, just submit and ask!  This checklist is here to help you, not to deter you from contributing!

@Nitori- I've been looking into this some more, and from what I understand, what happens here is the following. When poetry resolves the dependencies here, it will find that pylint depends on isort, which depends on backports.functools-lru-cache; python<3.2. After resolution, poetry builds a dependency graph (I think it's actually building a tree here, but might be wrong). While building this graph, the marker for each package in each branch it occurs is derived by intersecting the declared markers with the markers of its parent in the dependency graph (this way, marker get propagated down to transitive dependencies). This is where the <empty> marker can get generated. In this case, the marker for pylint is python>=3.4 (as declared in pyproject.toml), but the marker for backports.functools-lru-cache is python<3.2 - the intersection of the two is empty, because the backports.functools-lru-cache is not needed (at least in that branch of the dependency graph)!
In the next step derives the appropriate markers and installation order for each package by flattening that graph. If a package occurs in multiple branches of the graph, the markers get unioned, and thus a previously <empty> marker can turn into a valid one. In your example, this doesn't happen though, and so the <empty> marker ends up in poetry.lock.
@sdispater I'm wondering, if the above explanation is correct, should packages with an <empty> marker just be filtered out at this point? Or should this issue be fixed upstream, so that the solver doesn't even include the package in the solution?

@sdispater @stephsamson @brycedrennan It would be great to get some feedback on my comment above - I'm happy to work on that, but would need some guidance to avoid wasting time.

OK so I realized that this fix might have been simpler than I understood it to be. If I just parse <empty> as an EmptyConstraint() the bug seems to go away.


"GET https://cdn.webrtc-experiment.com/gif-recorder.js net::ERR_NAME_NOT_RESOLVED"

It looks like your CDN is no longer accessible.
What happened ? Is there a way to download the cdn content ?

The files you mentioned can be found in this directory:

https://github.com/muaz-khan/RecordRTC/tree/master/libs

All the files from CDN are open-sourced. At least in this repo: https://github.com/muaz-khan/WebRTC-Experiment
PS. DNS (A/CName propagation) is having issues. I'm working on fixing it.

Thank you, I used the cdn sources so my app can still record the next time the cdn has a problem.
You were quick fixing it ;)


Feat: RECORD_AUDIO Permission shouldn't be required

Hi. What if we use camera preview only for taking photos.
Is it possible to dynamicaly check and request RECORD_AUDIO permission even if client calls startRecordVideo function?

Ya. This needs to be removed. This is a huge issue. It's really screwed me with a recent deployment as I didn't realize this permission got added and it's most definitely not needed just to take a picture.

We will fix it thanks for the feedback

@qliqdev did you set disableAudio to true when you call the plugin?

We published a fix for Android, it was missing on android please try 6.5.0

We published a fix for Android, it was missing on android please try 6.5.0

Thx a lot


Add Contributing section to README

I also committed some things along the way. Here's the commit list with comments:

Add config.allow-plugins to composer.json

When I ran composer install, I got:
composer/package-versions-deprecated contains a Composer plugin which is currently not in your allow-plugins config. See https://getcomposer.org/allow-plugins
Do you trust "composer/package-versions-deprecated" to execute code and wish to enable it now? (writes "allow-plugins" to composer.json) [y,n,d,?]    y

Run phpcs
Fix check issue: InvalidNullableReturnType

I ran composer run-script check -- --alter --issues=InvalidNullableReturnType and included everything but getExecution(): ?WorkflowExecution, since it should either throw or return a non-null value (suppressed below). Let me know if I should take any of the remaining changes out!

Add Contributing section to README

I think I'm missing a step, since vendor/bin/phpunit --testsuite=Functional --testdox --verbose results in errors like:
 ✘ Cancelled  1 ms
   ┐
   ├ Error: Class "Grpc\ChannelCredentials" not found
   │
   ╵ /Users/me/gh/sdk-php/src/Client/GRPC/BaseClient.php:57
   ╵ /Users/me/gh/sdk-php/tests/Functional/Client/ClientTestCase.php:34
   ╵ /Users/me/gh/sdk-php/tests/Functional/Client/UntypedWorkflowStubTestCase.php:168
   ╵ /Users/me/gh/sdk-php/vendor/phpunit/phpunit/phpunit:98
   ┴

 ✘ Terminated  1 ms
   ┐
   ├ Error: Class "Grpc\ChannelCredentials" not found
   │
   ╵ /Users/me/gh/sdk-php/src/Client/GRPC/BaseClient.php:57
   ╵ /Users/me/gh/sdk-php/tests/Functional/Client/ClientTestCase.php:34
   ╵ /Users/me/gh/sdk-php/tests/Functional/Client/UntypedWorkflowStubTestCase.php:185
   ╵ /Users/me/gh/sdk-php/vendor/phpunit/phpunit/phpunit:98
   ┴

ERRORS!
Tests: 71, Assertions: 167, Errors: 42, Failures: 1.

Suppress InvalidNullableReturnType Psalm issue for getExecution

composer run-script check no longer logs:
------------------------------
Psalm can automatically fix 1 of these issues.
Run Psalm again with
--alter --issues=InvalidNullableReturnType --dry-run
to see what it can fix.
------------------------------

Hi @lorensr Thank you for the request! 👍 But could you please separate it into several small requests: each containing only one kind of changes? One for docs, one for Composer, and so on...


⚠ Turnos 24x7 has degraded performance

In 664f948, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 2482 ms

Resolved: Turnos 24x7 performance has improved in 51b047b.


feat: resizeable window

Hopefully fixes #261 and once a release is generated should address #243

Sorry for the pre-commit fixes too at the same time it was complaining. Just focus on the css changes and try running it locally.

I already tested it out, but I saw some issues along the way so I decided to add the fixes here. I also updated the prettier configuration but currently have no idea how to resolve the pre-commit consistency check action errors.

I will take a look after work @roldanjr. There are also a few visual issues I've found that need slight tweaks to.

I may make the pre-commit one less strict, its just a nice one for some of the basic stuff to clean things up and enforce certain rules as it can hook into stuff like prettier.
But the way you can run it is pre-release run and it'll run over your files with changes.
Or run pre-release run --all-files to go over all files.

need to also check this out to find whats going on there.
Seems originally the window was too thin to notice the visual bug but when it goes smaller than a certain amount suddenly it snaps to look like this.

I also need to fix up the settings window to allow that to scale vertically properly.
A lot of these changes we are making for resizing should also assist with mobile support due to varying screen sizes :D

I will need to look tomorrow.

Ive also solved #311 to allow us to tinker with the UI easier.

Just a few small tweaks. I believe the last one is modifying this to be css based shortening.

Though I am doing that now.

Finished the final part I wanted @roldanjr. The issue I had was i was trying to mess with the layout a lot and then forgot the timer was a display: grid; type and that was what was making the size not act as I was expecting.
I've updated it to use flexboxes instead so it resizes a little more cleanly especially with the css ellipsis hiding.

As long as the compact mode isn't broken this pr is ready to merge. I'll check it tomorrow as it's past midnight right now.

The compact mode layout has been broken. Already pushed the fix for that.
I also add restrictions to users from resizing the window during the compact mode.
You can merge this @sekwah41 after you check my updates.

the minimum size the  main app can go to is too small now

Found the issue. It's a side effect of making the window resizeable after setting the minimum size.

The only thing I may say is due to how it was working before. The compact mode may still have some issues on certain operating systems when the fullscreen break is used. Though we can investigate that when the reports come in and on which os.
Though ill try to do a quick test on windows and mac once they have built.

I just got this error coming out of full screen break.

windows after a full iteration and going back to compact mode

Was just playing a game of league and realised we are not cancelling the global listener to the escape key.

Confirmed works on mac and Linux. Though windows doesn't properly resize fully. Will try listening to the fullscreen event later.

Why are there so many whitespace changes?

Oh I see, prettier config was updated. Any chance we can keep that in a separate pull request?

Thats my bad partially i told precommit to run over all the files to clean it up.

You can ignore the whitespace in the github ui.

Whitespaces should be sorted @gillisandrew and will be a different pr.

I've done testing on mac, windows and Linux with the resizing issues and it seems to be fully sorted now. Had to fix a series of annoying behaviours and mostly windows was the problem.


Test pr

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@bfdbea9). Click here to learn what that means.
The diff coverage is 87.17%.

@@            Coverage Diff            @@
##             master       #8   +/-   ##
=========================================
  Coverage          ?   49.86%
=========================================
  Files             ?       90
  Lines             ?     4550
  Branches          ?        0
=========================================
  Hits              ?     2269
  Misses            ?     2244
  Partials          ?       37

Impacted Files
Coverage Δ

smallCargo.go
100% <100%> (ø)

ships.go
100% <100%> (ø)

baseShip.go
100% <100%> (ø)

ogame.go
39.78% <61.53%> (ø)

Continue to review full report at Codecov.

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


[KYUUBI #3605][Subtask][Authz] WIP: extend PrivilegeObject andAccessResource to support catalog

Why are the changes needed?

to close #3605.

extend PrivilegeObject,AccessRequest, AccessResource and RangerAccessRequest to support catalog

How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before make a pull request

Codecov Report

Merging #3606 (54c3395) into master (e80a7db) will increase coverage by 0.16%.
The diff coverage is 82.75%.

:exclamation: Current head 54c3395 differs from pull request most recent head b1de7fc. Consider uploading reports for the commit b1de7fc to get more accurate results

@@             Coverage Diff              @@
##             master    #3606      +/-   ##
============================================
+ Coverage     51.75%   51.92%   +0.16%
  Complexity       13       13
============================================
  Files           483      484       +1
  Lines         27025    27110      +85
  Branches       3779     3785       +6
============================================
+ Hits          13987    14076      +89
+ Misses        11671    11667       -4
  Partials       1367     1367

Impacted Files
Coverage Δ

...in/spark/authz/ranger/SparkRangerAdminPlugin.scala
61.11% <45.45%> (-1.39%)
:arrow_down:

...n/spark/authz/ranger/FilteredShowObjectsExec.scala
42.10% <50.00%> (ø)

...k/authz/ranger/RuleReplaceShowObjectCommands.scala
62.90% <75.00%> (ø)

.../plugin/spark/authz/ranger/RuleAuthorization.scala
81.39% <87.50%> (+2.44%)
:arrow_up:

...ubi/plugin/spark/authz/ranger/AccessResource.scala
85.71% <90.90%> (+1.09%)
:arrow_up:

...he/kyuubi/plugin/spark/authz/PrivilegeObject.scala
85.71% <100.00%> (+2.38%)
:arrow_up:

.../kyuubi/plugin/spark/authz/PrivilegesBuilder.scala
69.78% <100.00%> (+0.46%)
:arrow_up:

...uthz/ranger/RuleApplyRowFilterAndDataMasking.scala
92.45% <100.00%> (+1.97%)
:arrow_up:

.../apache/kyuubi/plugin/spark/authz/v2Commands.scala
90.95% <100.00%> (+1.26%)
:arrow_up:

...yuubi/server/metadata/jdbc/JDBCMetadataStore.scala
89.27% <0.00%> (-0.70%)
:arrow_down:

... and 9 more

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

Sorry to see the unnarrowed gap between the current implementation and the community's vision and requirements. Have to suspend this PR again after the first implementation with 3 times of reimplementations in 2 months, due to the impact of unexpected refactoring in Authz'core, the interrupted order of subtasks for multi-catalog, and the limitation of my personal capability of designing with exhausted efforts.


support optional proxy url

we're using a proxy url to get around CORS, would be really cool if we could pass the proxy url in via the options object, to be used for the fetch request. the response should retain the original url.
i'd be happy to work on a PR for this if its an acceptable idea.

please do submit a PR


Update Terracotta for Postgres Integration

Update when Terracotta Master (which has Postgres ingegration for backend DB) is released.
This also means we could drop MySQL DB

Closed by #182


xmrig: 6.12.2 -> 6.14.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/xmrig/versions.
meta.description for xmrig is: "Monero (XMR) CPU miner"
meta.homepage for xmrig is: "https://github.com/xmrig/xmrig"
meta.changelog for xmrig is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

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

/nix/store/gbxlki7ybq4psy8vyq800hf8r02hyn9n-xmrig-6.14.0/bin/xmrig passed the binary check.

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

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

found 6.14.0 with grep in /nix/store/gbxlki7ybq4psy8vyq800hf8r02hyn9n-xmrig-6.14.0

found 6.14.0 in filename of file in /nix/store/gbxlki7ybq4psy8vyq800hf8r02hyn9n-xmrig-6.14.0

directory tree listing: https://gist.github.com/2715e43c27051104f2d3f6fad7a7d047

du listing: https://gist.github.com/3dd9e8075a7a0e1cd068e5499e39aff5

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

2 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
0 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
0 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
xmrig

Instructions to test this update (click to expand)

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

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A xmrig https://github.com/r-ryantm/nixpkgs/archive/9fcaece16a92b2471de20d9cc224d88b25868d2c.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/gbxlki7ybq4psy8vyq800hf8r02hyn9n-xmrig-6.14.0
ls -la /nix/store/gbxlki7ybq4psy8vyq800hf8r02hyn9n-xmrig-6.14.0/bin

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

1 package built:

xmrig

Maintainer pings
cc @fpletz @kim0 for testing.

Result of nixpkgs-review pr 134018 at 9fcaece1 run on x86_64-linux 1

1 package built successfully:

xmrig

2 suggestions:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/applications/misc/xmrig/default.nix:26:3:
   |
26 |   installPhase = ''
   |   ^

warning: missing-patch-comment
Consider adding a comment explaining the purpose of this patch on the line preceeding.
Near pkgs/applications/misc/xmrig/default.nix:21:15:
   |
21 |   patches = [ ./donate-level.patch ];
   |               ^


[1.2.0] Cannot read from measurement if its name is starting with an underscore.

Bug report
Hey,
in  InfluxDB version 1.2.0 it is not possible to read from measurements starting with an underscore. If the name of a measurement starts with "_" the response is always empty. In a previous version (e.g. 1.0.1), the measurements could be queried as expected.
System info: Windows 7 / InfluxDB 1.2.0
Steps to reproduce:

Create a measurement starting with an underscore. E.g.: _data
Reading from the measurement. E.g.: SELECT * FROM _data

Expected behavior: The data contained in the measurement
Actual behavior: An empty response

Thanks for the report @mariusoe.
Confirmed:
 // ⇒  influxd-v1.1.0

> insert _hello value=1
> select * from _hello
name: _hello
time                value
----                -----
1488198319116228104 1

// ⇒  influxd-v1.2.1-rc5

> insert _hello value=1
> select * from "_hello"
> select * from _hello
> select * from "db".."_hello"
> show measurements
name: measurements
name
----
_hello
cpu

>

Possible dupe of #7961.

Have the same issue, is there a way to workaround this issue?  I have a bunch of measurements that start with '_'.

Fixed via #8069


Fix issues with DragOverlay measuring

This PR follows-up on #408 and #426
It fixes a number of issues with the measuring strategy for the DragOverlay. The main change is that we need to subtract the transform applied on the DragOverlay node when measuring it, otherwise the returned measurement includes the transforms, which means that the next time we compute the collision rect (by adding the transform to the drag overlay position), the newly calculated collision rect has double transforms, which is incorrect.
@wmain can you test this PR against your branch and confirm that it fixes the issues you were seeing with the original solution?

I converted this PR to draft because I still need to make a couple tweaks, but it should be good enough for you to give it a try

Looks good on my branch!

Good to hear. Out of curiosity, are you positioning the drag overlay child with transform or margin / padding / top / left?
I'm a bit worried about the amount of complexity it adds to try and support measuring the drag overlay child using getBoundingClientRect() (because it is transformed) and am debating shifting to the getViewRect(https://github.com/clauderic/dnd-kit/blob/master/packages/core/src/utilities/rect/getRect.ts#L92-L110) method instead, which ignores transforms.

Good to hear. Out of curiosity, are you positioning the drag overlay child with transform or margin / padding / top / left?
I'm a bit worried about the amount of complexity it adds to try and support measuring the drag overlay child using getBoundingClientRect() (because it is transformed) and am debating shifting to the getViewRect(https://github.com/clauderic/dnd-kit/blob/master/packages/core/src/utilities/rect/getRect.ts#L92-L110) method instead, which ignores transforms.

We add display: flex, alignItems: center, and justifyContent: center to the DragOverlay styles.

That counts towards offsetTop / offsetLeft so would still be supported (https://developer.mozilla.org/en-US/docs/Web/API/HTMLElement/offsetTop)

For my curiosity's sake, which step of computation are you considering shifting? Shouldn't the final rect include transformations and therefore make getBoundingClientRect appropriate?

There are a lot of timing-related concerns with regards to measuring elements that have transforms applied. It requires elaborate orchestration to make sure you measure an element at the appropriate point in time, either before or after transforms are updated.
You can see issues with the current strategy if you navigate to the Multiple containers story and sort with the keyboard. The timing of when the DragOverlay node is measured conflicts with the collision detection logic (there's a race condition between the two).
https://user-images.githubusercontent.com/1416436/129971595-fabfa2ea-530b-4c8e-84ad-9f18e5373271.mp4
These issues aren't impossible to solve, but I'd rather keep the code simple and reliable for now and improve on it later.
https://user-images.githubusercontent.com/1416436/129971617-9ea15449-4a1d-441d-a6d2-20e868f1d162.mp4


Add function to remove a local branch

Description
Inspired by Laravel php artisan migrate --pretend, that prints SQL queries instead of running them on the database.
I have added a new flag --dry-run to git trim, that prints branches going to be trimmed instead of actually deleting them.
This will solve the problem of not knowing what branches are going to be impacted, fearing to delete unwanted branches.
Changes

Added --dry-run flag to git trim option.
Added remove_local_branch function.
Sorted flags by alphabetical order.

@mohammed-elhaouari, thanks. I'll clean this up shortly as I just pushed some changes to master.

@jasonmccreary great. Good luck mate.

@jasonmccreary I have resolved old merge conflicts. Left for you to do the cleanup.

Again, I have more changes inbound. However, if you're going to work on it, please make the option dry-run inline with other common commands.

@jasonmccreary I have resolved all conflicts. And resolved the points raised. I have also sorted flags in alphabetical order. Though,  I think dry-run and reset flags should be the last options. When you run git trim -h it makes sense to see them last in the last, it gives visual feedback that these 2 options are secondary options.
I don't know, but I feel that flags should not be sorted alphabetically

Vs

@mohammed-elhaouari, let's keep it alphabetical.
Also, please do not make formatting changes to the code.

@jasonmccreary removed all the automatic formatting applied by my IDE.

@mohammed-elhaouari, awesome. Thanks for your continued work on this!


decrease chunk size

When load time frame 10 min, get error that size of chunk should be less than 5 month


Support for Premium File Shares

Hi Service Team,
Working with a customer, they highlighted that this article is a bit confusing and does not reference any of the premium file share information, suggesting we add an additional section regarding creating premium file shares with Storage Explorer. Is this something that could be accomplished easily?
Thanks,
Isaac

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

ID: c99092d0-6ace-7927-d405-320f13cb5935
Version Independent ID: d142f557-8bf8-fcbe-82ef-99302e9c6712
Content: Quickstart for managing Azure file shares using Azure Storage Explorer
Content Source: articles/storage/files/storage-how-to-use-files-storage-explorer.md
Service: storage
Sub-service: files
GitHub Login: @roygara
Microsoft Alias: rogarana

@mrisaac Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

@mrisaac Thanks for reaching out, sorry for the delay. I went ahead and tested creating premium file shares with the storage explorer. The process is virtually identical, there are no differences whatsoever. However, you cannot modify the quota size of shares when creating with storage explorer and they only create in the default 100 GiB size, as opposed to the max of 100 TiB. For this reason, I don't think it makes sense to emphasize or highlight premium file share creation with the storage explorer. You would need to use one of the other methods to resize the shares anyway.
Accordingly, I'm going to close this out, please feel free to comment and reopen this if you've any further questions.
#please-close

Side note: I went ahead and tested storage explorer with standard file shares,. You cannot modify the quota for them either but it at least creates them at 5120 GB. Which is the max size unless you're using the large file shares feature (in which case the max size is also 100 TiB).


[lwm2m] add w/r firmware update info API.

now support TOCKEN_INFO and FW_DOWNLOAD_INFO type, others type can be added later.

OK


componentDidMount is not triggered on inner components

export default class Example extends Component {
  constructor (props) {
    super(props)
    this.state = {
      loaded: false
    }
  }

  componentDidMount () {
    this.setState({
      loaded: true
    })
  }

  render () {
    let { loaded } = this.state
    return loaded
      ?
💕

      :
Loading...

  }
}

Everything stays in loaded false state probably because componentDidMount never gets triggered, my folder structure is

pages -> index.js
components -> example.js

Running on latest version of next.js
Is this behavior intended?

It's not intended, and in fact Next.js should not be affecting that, can you provide a repo to reproduce it? I think something else it's wrong 🤔

I'll re-open when there's an example to reproduce.

Thanks, I'm trying to make the example as minimal code as possible 😢
But basically im using dotenv and also next-routes with some getInitialProps maybe its too much stuff in the mix??


Re-generate unsigned vs signed benchmarks

Previous ones failed with division by zero for some data types.

LGTM


Refactoring Geometry

I would like to implement the following modification in multibody/geometry.hpp

Remove classes DistanceResults and CollisionResults and directly work with FCL native types. For distance results, possibly add the 3 member functions as static functions (although I do not think they are vital). Keep operator== outside of the classes.

Remove NONE from the enum.

Move all the definitions before struct GeometryModel in a geometry-fwd.hpp

move inner/outer objects from GeomModel to GeomData. Stop automatically feeling them at model construction time. Possibly fill them automatically at data construction time, or explicitly with a dedicated method.

rename all xxx_yyy as xxxYyy, except oMg_fcl

It was suggested by @stonneau to rather store in data a CollisionObject that a fcl::Transform, which makes the call to fcl computations easier.

Remove ref to GeomModel from GeomData. Consequently change the signature of the algorithms using GeomData, to explicitly add GeomModel as an argument.

Remove methods computeAllCollisions and computeAllDistance, as they are redundant with the algorithm geometry.hpp

Possibly also remove computeCollision and computeDistance, and put the two as external functions in algorithm/geometry. Change their internal process to avoid (stack) memory allocation of a Collision/DistanceResult object.

Remove displayCollisionPair.

Add a PairIndex type (like JointIndex and GeomIndex) to make arguments of some methods more explicit.

Opinion, in particular from @fvalenza, welcome. Please help.

@jcarpent agreed on 6. I start implementing 6 that I need for hpp::pinocchio

1/ If we work directly with fcl type, we lose the info concerning what are the collision pair ( the two GeomIndex) involved in this particular result.
2/ No particular opinion.  It was to force user to understand what is loaded by specifying it explicitly ( i.e using the GeometryModel as a CollisionModel or a VisualModel)
3/ Agreed. Some structs are even binded in python, so we could even put them in their own file may be?
4/ If we want to add/remove outerobjects or innerobjects on the fly, it may be better. Do we need to consider updated the list of collision pairs then ?
6/ As previsouly stated, it's the tradeoff of redundant information (the pointer on the collisiongeometry is both in GeometryModel and GeometryData then) vs easier calls to fcl computations. I personnally prefer storing the CollisionGeometry in the Model (as it is constant) and the transform in the Data. Just my personnal pov.
7/ Agreed, some GeomData methods work with the number of geometry objects. We need to propagate this information in GeomData if it' still needed after rework ( see point 7/8 particularly)
5/8/9/10/11/ OK

Thx.

We loose it directly, but we can easily recover it in the corresponding position of the collisionPairs vector, right?
geomData.collision_results[i] ==corresponds to==> geomModel.collisionPairs[i]

I think this is rather a design problem. By adding NONE, you check the choice of the user at run time (in DEBUG only, with an assert), while by not having it in the enum, the choice is checked at compil time, as it becomes a mandatory (not optional) argument of the parsing methods.

I dont think the naming is very important. I would tend to prefer to limit the number of header files, and name it "-fwd" makes it clear that it is of lesser importance. Maybe -fwd is not the good suffix though.

I would propose to have inner/outer lists as protected object, that can be accessed only with read-only method.

As part of the #190 , i am doing point 3. I may PR only this point or point 3 and the issue together this afternoon.

Thanks for your help.
Then lets decide the name of the include. I think fwd is not so good, as it is not exactly forward declarations.
I suggest fcl.hpp (ie pinocchio/multibody/fcl.hpp). It should contains all the fcl #include that we are needing in Pinocchio, and the wrapping, typedef and correspond #ifdef of the fcl classes.

I pas this issue from question to allocated. @fvalenza does 3 first (de-assign yourself when done). If somebody wants to do the rest, assign yourself, or assign me if nobody else volunteer.

I did not manage to work on that topic during August, so I remove the label.

Pull-request #302 already provides the following changes:

removes class CollisionResult,
memory allocation in computeCollision.

Outer objects are not used in HPP (I have to double check this assertion). I think inner objects make sense in GeometryModel.
I agree with points 1, 2, 3, 5, 10, 11. I have no opinion for point 6, 8. 7 implies that the use of GeomData is slightly more complex.

solved by #314


ISPN-11430 Align microprofile metrics and config artifact versions wi…

…th Wildfly latest
https://issues.redhat.com/browse/ISPN-11430

seems like I broke the wf modules tests

seems like I broke the wf modules tests

Playing the long game  I like it. The wf modules are longer in master, so you should be good after rebasing :slightly_smiling_face:

Rebased


clone command failing in case of multiple branches

@netrajpatel before merge this code we need to test this fix on this branch itself


Can still add to multi-value inputs if removeOnly is set after initialization

If removeOnly is enabled via setOptions after a multi-value selectivity instance has been initialized, you can still add items by entering text in the input field.
See this codepen.

Thanks for the report!


Fix minion listing

Make concept of cache that expires after ttl into generic util
Use this to not health check minions more than needed
Also health check in parallel if needed

There are better fixes possible, but this should make it tolerable until those fixes appear.
Closes #2973

cc @satnam6502 @brendandburns

The last two commits should remove a bottleneck for pod listing, too.

/cc @ddysher

LGTM except nits. Can we squash some commits?

Nit fixed, squashed down the improvement commits-- I think the three left are good logical divisions.


API: Change behavior of scaleshift function

The scaleshift function currently divides by the amplitudes and applies negative delays to the waveforms. Change this to the expected behavior, which multiplies the waveforms by the amplitudes and applies positive delays to the waveforms.

Fixed in 9a2768a55c99b1ccf0cf049cb539ac180c7fe0d1


Remaining cleanup after migration

[ ]  Page history (like Edit this Page) - linking to git history on github
[ ] Work out all redirects and implement them in _config.yml
[ ] Front page - still needs some work (e.g. to link to core sections)
[ ] Also should we auto-expand menu (may not be obvious to users)
[x] Tweak tagline - cf current community site ...
[x] Point to main database - people may have ended up here and wonder where the data is!
[x] sorting the main menu to have a sensible ordering
[ ] content tweaks (e.g. i think we thought the help section could do with some refactoring)
[ ] ensure RSS feeds work
[ ] generate sitemap https://github.com/jekyll/jekyll-sitemap

@danfowler i'd also add:

sorting the main menu to have a sensible ordering
content tweaks (e.g. i think we thought the help section could do with some refactoring)

Added.

Menu

Home

Get Involved

OpenSpending Next
Events

Help and Guides

Resources

EU stuff - great submenu

Meta

About

Blog

All we will do remaining is sort out redirects.

@rgrp Was working on this just now, and noticed that the old blog post permalink format was something like:
http://community.openspending.org/2014/03/hackathon-odd14-barcelona/
(As seen here: https://twitter.com/openspending/status/449576966329339904)
Whereas the default new one would be:
http://community.openspending.org/blog/2014/03/28/hackathon-odd14-barcelona/
Should we keep the old format?  The main (weird) thing would be losing the /blog/ prefix for post permalinks.

Hm, in trying to switch to a permalink style other than 'pretty' for blog posts, I ran into this issue: https://github.com/jekyll/jekyll/issues/2691
Basically, we would lose the 'pretty' permalink for pages (trailing slash) with no clear way to re-enable aside from manually renaming the files, pushing into directories, etc.

I'm happy with subdomain blog - we will just have to do more redirect work. Happy for you to call it.

Cool. Will add redirects for old posts.

Posts redirected.


Rate limited when visiting browse page

Going to https://mailarchive.ietf.org/ and then clicking browse with an empty cache seemed to result in being blocked temporarily due to rate limiting. The status page seems to indicate the service is functioning well, is this intended behavior?
I noticed my browser is doing at least two requests for every page due to favicon.ico returning a 404.

Since diagnosing this is likely to involve address details, lets shift this to email at<EMAIL_ADDRESS>please?


[8.x] Improved error handling for key:generate

The key:generate command would say that it had set the APP_KEY when it had not set it. This could happen if your .env file does not contain APP_KEY= or if for some reason file_put_contents() failed.
Fixes #20719 by checking if file_put_contents() was able to write the file else displaying an error message. If APP_KEY= is not present in the .env file it will write it to the end of the file.
If you do not want it to append to the end of the .env file then that part can be removed but I think having an error message when the key is not set is important.

Please send this to 8.x. Master is 9.x.


fix(response): fix response status code as not same as real upstream …

…status when using kong_buffered_http

Summary
When kong uses kong_buffered_http and upstream timeout, the retire status code need to return 504, not 502.

Checklist

[ ] The Pull Request has tests
[ ] There's an entry in the CHANGELOG
[ ] There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Full changelog

[Implement ...]

Issue reference

FTI-5320

Could you add a test so we make sure this issue never comes back?

Need a change log entry for it.


Content Collection SSR code snippet has type errors + fix

https://docs.astro.build/en/guides/content-collections/#building-for-server-output-ssr
the suggested code snippet has type errors as slug and entry can be undefined and the guide doesn't mention handling them or if this expected behavior. The resulting ergo feels a bit off since typescript seems to be a significant part of the Content Collection Guide yet is completely neglected here. (assuming I didn't miss anything, or course)
---
import { getEntryBySlug } from 'astro:content';
// 1. Get the slug from the incoming server request
const { slug } = Astro.params;
// 2. Query for the entry directly using the request slug
const entry = await getEntryBySlug('blog', slug);
// 3. (Optional) render the entry to HTML in the template
const { Content } = await entry.render();
---

{entry.data.title}

<Content />

possible solutions
note/warning
the simplest solution would be to add a note or warning below the snippet, letting devs know to handle the type errors accordingly assuming Astro has no opinions on how to handle the errors themselves.
e.g.

⚠ if you're using Typescript slug can be undefined if ..., and entry can undefined if ..., so handle with care.

improved code snippet
this is the better solution imo.
---
import { getEntryBySlug } from 'astro:content';
// 1. Get the slug from the incoming server request
const { slug } = Astro.params;
// 2. check if the slug identifier matches the filename
if (slug === undefined) {
    throw new Error("slug/filename mismatch");
}
// 3. Query for the entry directly using the request slug
const entry = await getEntryBySlug('blog', slug);
// 4. redirect if the entry does not exist
if (entry === undefined) {
    return Astro.redirect("/404");
}
// 5. (Optional) render the entry to HTML in the template
const { Content } = await entry.render();
---

{entry.data.title}

<Content />

I agree we’d want to go the improved code snippet route on this one! We can nitpick what should be inside the if blocks when something is undefined but would you want to create a PR to get the ball rolling?


Recogize more truecolor terminals properly

There are some ways to detect truecolor terminals that tcell is not using, if the README is currently correct.
Please see this document for more ways to detect truecolor. I will quote the important sections below.

Since ncurses-6.0-20180121, terminfo began to support the 24-bit True Color capability under the name of "RGB". You need to use the "setaf" and "setab" commands to set the foreground and background respectively.

Out of all the options I mention, this seems like it could be the most reliable. I am not sure how widespread support for this is though, and whether it depends on ncurses or not.

There will be no reliable way to detect the "RGB" flag until the new release of terminfo/ncurses. S-Lang author added a check for $COLORTERM containing either "truecolor" or "24bit" (case sensitive). In addition, VTE, Konsole and iTerm2 set this variable to "truecolor". It has been in VTE for a while and but is relatively new, being still git-only in Konsole and iTerm2).

This is obviously not a reliable method, and is not forwarded via sudo, SSH etc. However, whenever it errs, it errs on the safe side. It does not advertise support when it is actually supported. [...] Checking $COLORTERM is recommended though since it will lead to a more seamless desktop experience where only one variable needs to be set. This would be system-wide so that the user would not need to set it separately for each app.

The $COLORTERM variable is why I originally made this issue. It is very easy to add support for, solves my personal terminal concerns (I use Guake), and is a growing standard, due to support from terminal libraries like VTE. I could make a PR to support this, although I would appreciate some guidance/direction.
The document mentions another method:

A more reliable method in an interactive program which can read terminal responses, and one that is transparent to things like sudo, SSH, etc.. is to simply try setting a truecolor value and then query the terminal to ask what color it currently has. If the response replies the same color that was set then it indicates truecolor is supported.

[...] Here we ask to set the background color to RGB(1,2,3) - an unlikely default choice - and request the value that we just set.

While potentially being more accurate, I don't see this as a viable solution for tcell, as it would be comparatively slow, and require changing visual things like the background on application startup. Maybe changing text color (without printing text yet) is possible?

In fact the README is probably a bit stale in this regard.  We do respect $COLORTERM (on UNIX / Linux terminals), and a number of the terminals have builtin support for True Color automatically.  Also, the "mkinfo" utility checks for the RGB property and if it finds it, it assumes that Xterm compatible 24-bit color escapes will work.
(I didn't try to hack the code to grok the the details of that escape because that would break other things in terminfo which currently assume no data type larger than a byte.)
At any rate, only the README needs to be updated.

Btw, we also will recognize a "Tc" entry in terminfo as indicating support for 24-bit color.

We look for this in $COLORTERM:
switch os.Getenv("COLORTERM") {
case "truecolor", "24bit", "24-bit":
	addtruecolor = true
}

We do respect $COLORTERM (on UNIX / Linux terminals)

Yay! Great to hear, thanks.

a number of the terminals have builtin support for True Color automatically

My understanding (and reasoning behind creating this issue) was that even if that were true, tcell wouldn't use truecolor on those terminals, unless they included some indication in their $TERM.
Somewhat off-topic but to be honest I'm confused why there is a .Truecolor option for tcell.Color. Why not always use truecolor versions of colors on terminals that support it, and don't use it on ones that don't? For my application, I'm stuck calling .Truecolor on all my colors that I know won't be part of the 256 xterm ones.

The reason for .Truecolor is that some colors are referenced by a "theme" -- i.e. the color is part of the color palette used by XTerm etc.  So the question is what is more important: accurate (direct) color, or honoring choices made by the user.  The answer varies depending on the use -- when rendering an image you probably want accurate colors, but if you're just highlighting text in an editor, you probably want to stick to the palette choices made by the user.
If you know you're allocating a color by RGB, then use the NewRGBColor() or NewHexColor() to allocate the color.  Those colors are direct color, and will strictly honor the RGB settings.
Note that the way we distinguish is by the setting of a high order bit in the color.  If you just pick some random 24-bit color value, it probably won't go well since it won't be something that sits outside the palette.
(To think about why this matters, consider the value 0 -- or #000000 -- is this really "black", or is it the theme value for black.    Well it matters quite a bit which -- so we need an extra bit to indicate that we really want #000000 black, not just the themed value for black.


Add alias for show_migrations

Whenever I want to show migrations I end up running ./manage.py show_migrations because the other show* management commands contain an underscore.  This patch add the alias show_migrations so that the command works with and without the underscore.
Thank you!

There are no other management commands provided by Django that start with show_*. Perhaps you're using a third-party app that is providing one with that prefix?
Additional aliases like this serve no purpose and will confuse everyone else. You can add this to your own project if you desire it.

Just FYI here's one way I add aliases to my projects: https://github.com/shangxiao/stupid-django-tricks/tree/master/management_command_alias
I get tired of either typing the full showmigrations or pressing tab and waiting for it respond after a couple of seconds so I have an alias sm setup.


Initializing the Plugin

When I first started up the widget you can see that there is no default item for sort items.

@ianblair44 Do we have any default view for this type of situation when we do not have any data in the plugin? Which default item are we referring here?

The default value for sort items should be "manual". It shouldn't just show a blank box.

Please review it.

Design layout is not visible.

For design, it should default to the first layout on each one. It shouldn't be a blank box.

@ianblair44  Yes! Ian, @sandeepchhapola is  working on it. Let me review it.


feat: replace kpartx with losetup

losetup supports --partscan option since 2012 and could be used instead of kpartx to map image partitions to loopback devices. Because most of the distributions come with util-linux preinstalled (in fact probably all of them, but I'm not sure how to verify that), this will remove unnecessary dependency.

I will work on that feature


Kwxm/contributing/branch names

Added some suggestions for naming branches systematically.

You might want to use git commit --amend in future. Or you could rebase your branch to squash the commits. I'll probably merge this with a squash, so no need to do it this time :)


added modulo functionality for Bigint % int (where int < base)

Ignore the failing build.
I'll go over all pull requests this weekend. Thanks!

Yes, I'll do that.

done.


BlElement inspector fails to redraw after minimising

After inspecting the result of an example that returns a BlElement, e.g.
BlGridLayoutExamples new exampleCellFitContent

and minimising and restoring the window, the Live view fails to be redrawn, leaving a grey window.
Steps to reproduce:

Inspect the results of the above code
Ensure the Live tab is selected.
Minimise the window
Restore the window

The Live tab should be blank grey.
@girba I could only reproduce this now in Morphic windows.  In SDL2 windows it appears to work correctly (I can't check Glutin windows).  Can you confirm?
Thanks,
Alistair

I cannot reproduce this using Morphic on Mac.

Does it happen in SDL2 or Glutin?

No

Hmm, so when you reported the issue to me with GToolkitImageExamples>>gridForDirectory:, how did you reproduce it?

The logic is the following: when user closes a Morphic window with Bloc space in it we stop that Bloc space. (space close).
Unfortunately, I couldn’t find a way to distinguish between minimization of a morphic window and removal of a Morph from the world because in both cases Morph>>#delete is sent.

I believe there isn't much to do about this one. Another reason to focus on the native windows :)


consider making wrapper scripts fail "early" and upon any "undefined variable" via adding set -eu

singularity ships a number of shell scripts, but unfortunately none of them is "paranoid enough" to enforce that they exit as soon as error occurs (in a pipe or a compound command) or some undefined variable is used.  Not sure which are invoked/used while running root-suided sexec, but I think it could be a good "stability" and possibly "security" measure if behavior of those scripts would become more "deterministic" in some sense, by forcing the assumptions that all errored executions are failures (unless specifically handled) and that all variables must be defined after some point in their execution life.

I'm not sure I exactly follow... Is this something you could help with or give me some specific examples to help me see exactly what you are thinking of?
Thanks!

Oh sure... will do later today
Cheers
On July 13, 2016 2:28:37 PM EDT, "Gregory M. Kurtzer"<EMAIL_ADDRESS>wrote:

I'm not sure I exactly follow... Is this something you could help with
or give me some specific examples to help me see exactly what you are
thinking of?
Thanks!

You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/gmkurtzer/singularity/issues/134#issuecomment-232444923

I think this is pretty much done at this point. I have set -u in functions, and set -e in bootstrap'ing (and fixed the bugs associated with doing that). Do you feel there is more I should do here, or am I clear to close this issue?
Thanks!

since indeed set-ings get inherited and now present in main functions, I guess set -u would do its role in other scripts.  But you think it is not worthwhile of similar -e  "protection" all throughout the scripts?

Implementing -e across the board had a lot of unintended consequences, for example I am currently trying to catch errors and print a decent notification. With -e set, it would just stop because the controlling shell would exit before it could print a notification/error/warning.
I figured though that some things like the bootstrapping should indeed quit on error. Thoughts?

could you point me to specific lines?  it is just a matter I guess again of explicit statements/assessment of result of execution of the line in question, e.g.
$> bash /tmp/1.sh
/bin/ls: cannot access 'alskjdf': No such file or directory
ls failed. too bad

$> cat /tmp/1.sh
#!/bin/bash

set -e
set -u

if ! /bin/ls alskjdf; then
   echo "ls failed. too bad"
   exit 1
fi

echo "the end"

I found that the test.sh was just stop at random places when -e was set, and it made debugging very difficult. If you want to see for yourself add 'set -e' right above where it sources the functions, and then run test.sh. Here is my output:
 + singularity mount -h                                         (retval=0) OK
 + singularity mount --help                                     (retval=0) OK
 + singularity help bogus                                       [gmk@centos7-x64 singularity]$

I am currently using set -u and set -e in various places in the shell code so I'm gonna close this. If for any reason it needs to be reopened, comment on it and I'll reopen.
Thanks!


[1438] CSV Import Handles CR Line Endings [master]

Resolves #1438

Imported CSVs can now contain CR or CFLR line endings

Merging based on review of #1483 - @esoterik can you add this to the v6.0.0 CHANGELOG?


Pages did not save.

Today spent 2 hours building some custom monsters for my game coming up tomorrow and saved them. I left the page and went back to look them over and Homebrewery seems to have no record of me ever making the page. I tried selecting Recent Brews and it did nothing. No recently edited or view pages. The only page that opens up is the homepage. I tried making a new page and wrote some random things on it. I saved it and tried the same thing again and again nothing.

Check your user page.

I have been trying to get to my user page but every time I click my
username it brings me back to this:
The Homebrewery
Welcome traveler from an antique land. Please sit and tell us of what you
have seen. The unheard of monsters, who slither and bite. Tell us of the
wondrous items and and artifacts you have found, their mysteries yet to be
unlocked. Of the vexing vocations and surprising skills you have seen.
Let me know if I am doing this wrong.
On Tue, Oct 2, 2018 at 5:02 PM RKuerten<EMAIL_ADDRESS>wrote:

Check your user page.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stolksdorf/homebrewery/issues/766#issuecomment-426469954,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ApxJWCRZorVyWUtfwcamMTlKugC-Wtybks5ug_6VgaJpZM4XE6fn
.

--
In the desert
I saw a creature, naked, bestial,
Who, squatting upon the ground,
Held his heart in his hands,
And ate of it.

If you really saved the brew, you can access it at http://homebrewery.naturalcrit.com/user/your-user-name
Substitute your-user-name for the one you chose for the site.

I'm experiencing the same issue. After I click my username at the top, it brings me back to the main page with the demo. The one I'm working on is still open in a different tab to not lose it, and seems as if its 'auto saving' but doesnt show up on the 2nd tab under recent either.

I'm using Google Chrome.
Here's a few screenshots with descriptions
https://imgur.com/a/6HEygpA

Well, from the info you provide, I believe the site can't handle usernames with special characters. Not sure, but I don't see any reason for this behavior.
@WorriorGuild, does your username have any special characters (something that's not a letter or number)?

No special characters.
On Sat, Oct 6, 2018, 11:06 AM RKuerten<EMAIL_ADDRESS>wrote:

Well, from the info you provide, I believe the site can't handle usernames
with special characters. Not sure, but I don't see any reason for this
behavior.
@WorriorGuild https://github.com/WorriorGuild, does your username have
any special characters (something that's not a letter or number)?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/stolksdorf/homebrewery/issues/766#issuecomment-427594674,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ApxJWFmksWz4PetWxXfUUZE0h2fod8Tfks5uiPEZgaJpZM4XE6fn
.


credit

#include <cs50.h>
#include <stdio.h>
//the last part works ok when separately, I will submit an example as credit1
void print_result(int n);
int main(void)
{
long number;
printf("Number: ");
number = get_long_long();

//getting separate digits, then (check)sum, then - above - valid or not

int a=0, b=0, c=0, d=0, e=0, f=0, g=0, h=0, i=0, j=0, k=0, l=0, m=0, n=0, o=0, p=0;
int b1=0, b2=0, d1=0, d2=0, f1=0, f2=0, h1=0, h2=0, j1=0, j2=0, l1=0, l2=0, n1=0, n2=0, p1=0, p2=0;
int count = 0;
int sum = 0;

if (number>0)
{
    a=number%10;
    number/=10;
    count++;

}

if (number>0)
{
    b=number%10;
    number/=10;
    count++;
    b*=2;
    if (b>9)
    {
        b1=b%10;
        b2=b/10;
        b = b1 + b2;
    }
}

if (number>0)
{
    c=number%10;
    number/=10;
    count++;

}

if (number>0)
{
    d=number%10;
    number/=10;
    count++;
    d*=2;
    if (d>9)
    {
        d1=d%10;
        d2=d/10;
        d = d1 + d2;
    }
}

if (number>0)
{
    e=number%10;
    number/=10;
    count++;

}

if (number>0)
{
    f=number%10;
    number/=10;
    count++;
    f*=2;
    if (f>9)
    {
        f1=f%10;
        f2=f/10;
        f = f1 + f2;
    }
}

if (number>0)
{
    g=number%10;
    number/=10;
    count++;
}

if (number>0)
{
    h=number%10;
    number/=10;
    count++;
    h*=2;
    if (h>9)
    {
        h1=h%10;
        h2=h/10;
        h = h1 + h2;
    }
}

if (number>0)
{
    i=number%10;
    number/=10;
    count++;
}

if (number>0)
{
    j=number%10;
    number/=10;
    count++;
    j*=2;
    if (j>9)
    {
        j1=j%10;
        j2=j/10;
        j = j1 + j2;
    }
}

if (number>0)
{
    k=number%10;
    number/=10;
    count++;
}
if (number>0)
{
    l=number%10;
    number/=10;
    count++;
    l*=2;
    if (l>9)
    {
        l1=l%10;
        l2=l/10;
        l = l1 + l2;
    }

}

if (number>0)
{
    m=number%10;
    number/=10;
    count++;
}

if (number>0)
{
    n=number%10;
    number/=10;
    count++;
    n*=2;
    if (n>9)
    {
        n1=n%10;
        n2=n/10;
        n = n1 + n2;
    }
}

if (number>0)
{
    o=number%10;
    number/=10;
    count++;
}

if (number>0)
{
    p=number%10;
    number/=10;
    count++;
    p*=2;
    if (p>9)
    {
        p1=p%10;
        p2=p/10;
        p = p1 + p2;
    }
}

//validate checksum and print card type
//PROBLEM HERE. Initially I had printf for every output (and it works in a separate file where I assign values to input parameters rather than use calculated values as here). when I substituted printf's below with result = and 1 printf in the end, nothing much happened. The program informs about invalid results and once recognized a 16-digit visa card.
bool valid;
string result = "";

if(sum%10 == 0) valid = true;
else valid = false;

if(!valid) result = "INVALID\n";
else
{
    if(count==15 && o==3)
    {
        if(n==4 || n==7) result = "AMEX\n";
    }
    else if (count == 16 && p == 4) result = "VISA\n";
    else if (count == 13 && m == 4) result = "VISA\n";
    else if (count == 16 && p == 5)
    {
        if (o>0 && o<6) result = "MASTERCARD\n";
    }
    else result = "INVALID\n";
}

printf("%s", result);

}

problem in the end of the code, when giving result what type of card it is. separately works ok.

resolved. problem was with p and n

You had two problems with your code. First, in each of your 16 if statement you didn't add the result to your sum variable thus it remained zero.
Second, in digits where you multiply by 2 and then split the digit into two halves you set the resultant of these operations to the original value of the digit thus overwriting its original value which resulted in the false checking of the card type. What I did is I kept the value of the digit as is and declared a new variable new_n in case of n and applied all of the operations to that new variable to avoid messing with the original digit n. This should be done in all of the variables were we multiply and then split and then add but it will be ample to do it in n and p only cause these are the only two digits that are involved in the checking process at the end of the code.
There was also no need for void print_result(int n) function.

Here's the modified working version of your code with comments where changes were added :
#include <cs50.h>
#include <stdio.h>

int main(void)
{

long number;

printf("Number: ");
number = get_long_long();

//getting separate digits, then (check)sum, then - above - valid or not

int a=0, b=0, c=0, d=0, e=0, f=0, g=0, h=0, i=0, j=0, k=0, l=0, m=0, n=0, o=0, p=0;
int b1=0, b2=0, d1=0, d2=0, f1=0, f2=0, h1=0, h2=0, j1=0, j2=0, l1=0, l2=0, n1=0, n2=0, p1=0, p2=0;
int count = 0;
int sum = 0;

if (number>0)
{
    a=number%10;
    number/=10;
    count++;
    sum+=a;
    //incremented the value of sum by a
}

if (number>0)
{
    b=number%10;
    number/=10;
    count++;
    b*=2;
    if (b>9)
    {
        b1=b%10;
        b2=b/10;
        b = b1 + b2;
    }
    sum+=b;
    //incremented the value of sum by b [NOTE : We messed up the actual value of the digit in b here]
}

if (number>0)
{
    c=number%10;
    number/=10;
    count++;
    sum+=c;
}

if (number>0)
{
    d=number%10;
    number/=10;
    count++;
    d*=2;
    if (d>9)
    {
        d1=d%10;
        d2=d/10;
        d = d1 + d2;
    }
    sum+=d;
}

if (number>0)
{
    e=number%10;
    number/=10;
    count++;
    sum+=e;
}

if (number>0)
{
    f=number%10;
    number/=10;
    count++;
    f*=2;
    if (f>9)
    {
        f1=f%10;
        f2=f/10;
        f = f1 + f2;
    }
    sum+=f;
}

if (number>0)
{
    g=number%10;
    number/=10;
    count++;
    sum+=g;
}

if (number>0)
{
    h=number%10;
    number/=10;
    count++;
    h*=2;
    if (h>9)
    {
        h1=h%10;
        h2=h/10;
        h = h1 + h2;
    }
    sum+=h;
}

if (number>0)
{
    i=number%10;
    number/=10;
    count++;
    sum+=i;
}

if (number>0)
{
    j=number%10;
    number/=10;
    count++;
    j*=2;
    if (j>9)
    {
        j1=j%10;
        j2=j/10;
        j = j1 + j2;
    }
    sum+=j;
}

if (number>0)
{
    k=number%10;
    number/=10;
    count++;
    sum+=k;
}
if (number>0)
{
    l=number%10;
    number/=10;
    count++;
    l*=2;
    if (l>9)
    {
        l1=l%10;
        l2=l/10;
        l = l1 + l2;
    }
    sum+=l;
}

if (number>0)
{
    m=number%10;
    number/=10;
    count++;
    sum+=m;
}

if (number>0)
{
    n=number%10;
    number/=10;
    count++;
    int new_n=n*2;
    if (new_n>9)
    {
        n1=new_n%10;
        n2=new_n/10;
        new_n = n1 + n2;
    }
    sum+=new_n;
    //Since we'll be needing n in our checking, I kept the original value as is and created a new variable for the multiplication and mod operations
}

if (number>0)
{
    o=number%10;
    number/=10;
    count++;
    sum+=o;
}

if (number>0)
{
    p=number%10;
    number/=10;
    count++;
    int new_p=p*2;
    if (new_p>9)
    {
        p1=new_p%10;
        p2=new_p/10;
        new_p = p1 + p2;
    }
    sum+=new_p;
}

//validate checksum and print card type

bool valid;
string result = "";

if(sum%10 == 0) valid = true;
else valid = false;

if(!valid) result = "INVALID\n";
else
{
    if(count==15 && o==3)
    {
        if(n==4 || n==7) result = "AMEX\n";
    }
    else if (count == 16 && p == 4) result = "VISA\n";
    else if (count == 13 && m == 4) result = "VISA\n";
    else if (count == 16 && p == 5)
    {
        if (o>0 && o<6) result = "MASTERCARD\n";
    }
    else result = "INVALID\n";
}
printf("%s", result);
}

thanks. I lost this line while copy-pasting and found this myself, when checking what sum is calculated (the default 0 was a very convenient variant for the program :D)
sum = a + b + c + d + e + f + g + h + i + j + k + l + m + nx + o + px;
the code is working and is accepted by the bot. however, when I typed<PHONE_NUMBER>4598345, the program "identified" it as "VISA", but it's 17 digits! I need to write a restriction for the length of the card number. the count stops working at 16.
Done!
Improved part of code:
`if (number>0)
{
p=number%10;
number/=10;
count++;
//to use in checksum
px=p*2;
if (px>9)
{
p1=px%10;
p2=px/10;
px = p1 + p2;
}
}
if (number>0)
{
    count++;
}

sum = a + b + c + d + e + f + g + h + i + j + k + l + m + nx + o + px;

`
Thanks for taking part!

and it appears you are the person who developed the validating mechanism I used. Thanks again!

@ElenaErratic Any time ;)


test: templates tests

Describe your changes
Issue ticket number and link
Checklist before requesting a review

[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] If it is a core feature, I have added thorough tests.

Codecov Report
Base: 70.28% // Head: 73.89% // Increases project coverage by +3.61% :tada:

Coverage data is based on head (2613db4) compared to base (1de5844).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #63      +/-   ##
==========================================
+ Coverage   70.28%   73.89%   +3.61%
==========================================
  Files          56       56
  Lines        2699     2697       -2
==========================================
+ Hits         1897     1993      +96
+ Misses        802      704      -98

Impacted Files
Coverage Δ

src/wanna/cli/__main__.py
82.14% <0.00%> (+82.14%)
:arrow_up:

src/wanna/cli/version.py
39.13% <0.00%> (+39.13%)
:arrow_up:

src/wanna/cli/plugins/pipeline_plugin.py
50.00% <0.00%> (+50.00%)
:arrow_up:

src/wanna/cli/plugins/tensorboard_plugin.py
58.62% <0.00%> (+58.62%)
:arrow_up:

src/wanna/cli/plugins/components_plugin.py
90.90% <0.00%> (+90.90%)
:arrow_up:

src/wanna/cli/plugins/runner.py
100.00% <0.00%> (+100.00%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


iOS SDK notification handling package

This PR refactors the notification handling code for iOS into a package bundled with the cocoapods dependency.
C-Breez PR https://github.com/breez/c-breez/pull/803
breez-sdk-swift PR https://github.com/breez/breez-sdk-swift/pull/12
TODO:

[x]  Integrate RedeemSwap notifications

The package is used to define a notification service and tasks needed to be executed for each notification type. The package has a ResourceHelper to load resources via resource ID in the bundle Info.plist, falling back to a default resource when one is not defined by the application. A KeychainHelper to read secured values from the Keychain.
The application can simply extend SDKNotificationService then implement getConnectRequest:
class NotificationService: SDKNotificationService {
    private let accountMnemonic: String = "account_mnemonic"
    private let accountApiKey: String = "account_api_key"

    override init() {
        let logsDir = FileManager
            .default.containerURL(forSecurityApplicationGroupIdentifier: accessGroup)!.appendingPathComponent("logs")
        let extensionLogFile = logsDir.appendingPathComponent("\(Date().timeIntervalSince1970).ios-extension.log")

        super.init()
        self.logger.setup(level: .debug, showThreadName: true, showLevel: true, showFileNames: true, showLineNumbers: true, writeToFile: extensionLogFile.path)
    }

    override func getConnectRequest() -> ConnectRequest? {
        guard let apiKey = KeychainHelper.shared.getFlutterString(accessGroup: accessGroup, key: accountApiKey) else {
            logger.error("API key not found")
            return nil
        }
        logger.verbose("API_KEY: \(apiKey)")
        var config = defaultConfig(envType: EnvironmentType.production, apiKey: apiKey,
                                   nodeConfig: NodeConfig.greenlight(
                                    config: GreenlightNodeConfig(partnerCredentials: nil, inviteCode: nil)))
        config.workingDir = FileManager
            .default.containerURL(forSecurityApplicationGroupIdentifier: accessGroup)!
            .absoluteString

        // Construct the seed
        guard let mnemonic = KeychainHelper.shared.getFlutterString(accessGroup: accessGroup, key: accountMnemonic) else {
            logger.error("Mnemonic not found")
            return nil
        }
        guard let seed = try? mnemonicToSeed(phrase: mnemonic) else {
            logger.error("Invalid seed")
            return nil
        }
        return ConnectRequest(config: config, seed: seed)
    }
}

Closed in favor of #834


Movement broken after 1st capture

After capturing a single pawn, pieces can no longer move. The sequence of moves shown in the readme GIF is no longer possible.
This seems to have been triggered by the upgrade to Bevy 0.4 (324b6dcbcb5f4a3d18ea22df6b4d6b9db4faf970), as the commit just before (22f65da1bc602e817552d336a5d88bb45b4e317f) works fine.
Using Windows 10 64-bit.

After the first capture, when I select a new piece, this line:
https://github.com/guimcaballero/bevy_chess/blob/6f72668f5377841d673f0a10921091b339d8c7b4/src/board.rs#L199
get_mut returns Err(NoSuchEntity), whereas it usually returns something like Ok((86v0, Piece { color: Black, piece_type: Pawn, x: 6, y: 3 }))

Thanks for the error report!
It looks like the SelectedPiece resource wasn't being set correctly to None for some weird reason. Moving the resetting code to be inside the event fixed the issue.

Thanks for the error report!
It looks like the SelectedPiece resource wasn't being set correctly to None for some weird reason. Moving the resetting code to be inside the event fixed the issue.


How to use BotFramework-WebChat without azure subscription?

I have a bot created using composer. I am able to use it in the emulator but now I want to render it on a webpage. I found BotFramework-WebChat, but it is using an Azure subscription. I want to do it without an azure subscription.

Hi @ukab23
The Microsoft Bot Framework team prefers that how to questions be submitted on Stack Overflow. The official Bot Framework Github repos are the preferred platform for submitting bug fixes and feature requests.

You can look into offline directline. It's not an official bot framework solution, and hasn't been updated in some time, but I'm assuming it should still work.


Es cache

This should trigger a travis build test for us to look at.  Also, this feels PR worthy.

Unfortunately I think we disabled Travis builds for all but master.

merged externally


Add "0Ho" package id to the force:package:installed:list command

Is your feature request related to a problem? Please describe.
Most, if not all, of package related commands show packages as the "0Ho" second generation package id.  Based on comment from Kevin Cripps of Salesforce, the "0Ho" id is the package id that is scoped to the DevHub.  Some packages are third-party in nature and would not have an "0Ho" id, but it would be helpful when processing CI related use cases to see the "0Ho" id in addition to the "033" package id.  This will allow us to better reconcile when the package version may have branch attributes and reconcile if a package dependency is "non-pinned".
What are you trying to do
I am trying to reconcile the output of force:package:installed:list command to the output of the force:package:version:list command.  The latter only deals with "0Ho" ids while the former deals with "033" record ids
Describe the solution you'd like
Change the force:package:installed:list JSON output to add an additional JSON attribute to show the "0Ho" record id ....when it is available.
Describe alternatives you've considered
n/a

@sfdc-db-gmail can you please take a look at this issue? I think it is a packaging bug.

@ImJohnMDaniel - I have added this to the product backlog.
@clairebianchi

Is this bug is fixed? Can I able to get the 0Ho id? If not, Can you please let me know how I can get the 0Ho id? I'm trying to update the package version which has all the necessary details in the config json. However, when I try to create a package version, it reports as: The provided package ID '0HoXXX' is invalid. Am I missing anything? Below is my command:
sfdx force:package:version:create --package "" -a Test-0.5 -n <IP_ADDRESS> -d force-app
--definitionfile config/project-scratch-def.json --installationkey test1234 --wait 20
Thanks in advance!

Is this bug fixed? Can I able to get the 0Ho id? If not, Can you please let me know how I can get the 0Ho id? I'm trying to update the package version which has all the necessary details in the config json. However, when I try to create a package version, it reports as: The provided package ID '0HoXXX' is invalid. Am I missing anything? Below is my command:
sfdx force📦version:create --package "" -a Test-0.5 -n <IP_ADDRESS> -d force-app
--definitionfile config/project-scratch-def.json --installationkey test1234 --wait 20
Thanks in advance!

Hi PadhmaPriya, did you find out how to get this 0ho Id with one of the sfdx force:package command ? I'm stuck at the same point as you ..

Since this feature seems to be still open:
Has anybody an idea where to find the "0Ho"-ID of a managed package?
Especially I am looking for the CPQ 0Ho Id (033A0000000PgEfIAK).


Issue 320: api.helper.get.visible_emails_pre change: addition of the …

Issue 320: api.helper.get.visible_emails_pre change: addition of the ioss parameter, removed en lang dependency for tabs, updated category logic

@Fatal-Errol: Can you test if this patch solves your problem?

Sorry guys, it was incorrect pull request.

Lets continue discussion at initial issue: https://github.com/KartikTalwar/gmail.js/issues/320


Key not present in Dictionary: "p"

When I call  TmuxNavigatePrevious command I get notification (noice) with error
09:14:35 msg_show E716: Key not present in Dictionary: "p"
  Error  09:14:46 msg_show.emsg Error detected while processing function 45_TmuxAwareNavigate:
The command still takes me to the previous pane so it works okay. It doesn't happen with other commands

Hmm, it looks like the g:tmux_navigator_no_wrap setting throws an error when navigating from Vim->tmux via the previous command (relevant lines). It's not obvious to me how to resolve here, but wanted to note.


Empty quote.

If I try adding a quote, it posts an empty one. There are tags, but no body.

Thanks, I see the issue and should have a fix out for it soon

Should be fixed in 0.0.5