Infer reset

This PR implements ResetType (Reset in the serialize format) as an "abstract" reset. Things of type Reset will be inferred in a flow forward way to be either UInt<1> or AsyncReset. They are disallowed on top-level inputs and BlackBox outputs.
Currently, this pass runs after ExpandConnects because I didn't want to duplicate the work of expanding connections. I think it probably should run earlier, during ResolveAndCheck, but will require a little bit of work to do that. Whenever this pass is run, https://github.com/freechipsproject/firrtl/pull/1045 must run after it.
I'm starting this as a Draft Pull Request because it's ready for a look, but probably needs to be modified to run in ResolveAndCheck

This is pretty much done. What's missing:

[ ] Unify with https://github.com/freechipsproject/firrtl/pull/1045 (this should be run before the check in #1045)
[ ] Make this run before Dedup. Similarly to past issues with Width inference, Dedup means that you must infer the same result for two Reset-templated instances that dedup to the same Module
[ ] Add tests that deal with the Dedup stuff

Overall looks really good! It could use more documentation explaining some of the new classes and what many of the private functions do. Supposing this works prior to RemoveAccesses, we need to see a test assuming the loc of a Connect is a WSubAccess.

@azidar I have updated this by adapting some code from ExpandConnects, it now runs between ResolveAndCheck and DedupModules, it should be a fully compliant transform and thus useful to run on its own on any HighFirrtl!

Seems this Travis CI job stuck forever.

@XVilka: not sure what was going on here, but a merge conflict may gate Travis from starting.
@jackkoenig: I locally merged with master and pushed. The force push was due to one false start... Should be clean now.

Done, including support for aggregates

LGTM. Congrats @jackkoenig !!!!


Refactor to support local VMs

What does this PR do?
This PR refactors the VM creation process to better support the use case of local vms.
The actual support for local vms will be added in subsequent PRs.
Which scenarios this will impact?
Motivation
Additional Notes

It looks like the linter is failing


can't extend pool on devnet

I'm trying to extend my pool
https://explorer.devnet.grid.tf/api/v1/reservations/pools/1471
but the values are not being added even after multiple retries
note: the pool has negative ipv4us value

How were the negative ipv4us set?

Can you add the explorer response to this ticket please?

Question:

Did you mange to make a payment?
Did you get refunded ?

@DylanVerstraete
request logs and reservation result but I now keep getting 502 bad gateway randomly on explorer requests to https://explorer.devnet.grid.tf/api/v1/reservations/pools/1471 and sometimes to users api as well
2020-12-21 15:25:47.901 | DEBUG    | jumpscale.clients.explorer.explorer:log_request:26 - Request https://explorer.devnet.grid.tf/api/v1/reservations/pools method: POST body: b'{"json": "{\\"pool_id\\": 1471, \\"cus\\": 0, \\"sus\\": 144, \\"ipv4us\\": 0, \\"node_ids\\": [\\"BpTAry1Na2s1J8RAHNyDsbvaBSM3FjR4gMXEKga3UPbs\\", \\"FTothsg9ZuJubAEzZByEgQQUmkWM637x93YH1QSJM242\\", \\"2anfZwrzskXiUHPLTqH1veJAia8G6rW2eFLy5YFMa1MP\\", \\"3NAkUYqm5iPRmNAnmLfjwdreqDssvsebj4uPUt9BxFPm\\", \\"5f4QEXpTcKt9Tbngn4hvraXd2G4QVFkagabUeKpn55oc\\", \\"DJHD5cMjRkuVh4enQ6dqCAuPNUPfH2a45gLv7JDDswbT\\", \\"BBmMHCw392RLiByVEUT2ac6y6qJpbW9xj8TRdgk5rMGN\\", \\"48YmkyvfoXnXqEojpJieMfFPnCv2enEuqEJcmhvkcdAk\\", \\"3kcJhRkTDuL51HmikzWFMndWwJbJGg8dZeqG9e72c6LV\\", \\"qzuTJJVd5boi6Uyoco1WWnSgzTb7q8uN79AjBT9x9N3\\", \\"9PdutHsdDSxcKUUyDg8ovS1KWh47qLT5R9h5uoFgRUH2\\", \\"Gr8NxBLHe7yjSsnSTgTqGr7BHbyAUVPJqs8fnudEE4Sf\\", \\"HtKZ3SwHcGM7nTsm8ykRGttfodFfLkghrmD5FGypgobK\\"], \\"currencies\\": [\\"TFT\\"]}", "data_reservation": {"pool_id": 1471, "cus": 0, "sus": 144, "ipv4us": 0, "node_ids": ["BpTAry1Na2s1J8RAHNyDsbvaBSM3FjR4gMXEKga3UPbs", "FTothsg9ZuJubAEzZByEgQQUmkWM637x93YH1QSJM242", "2anfZwrzskXiUHPLTqH1veJAia8G6rW2eFLy5YFMa1MP", "3NAkUYqm5iPRmNAnmLfjwdreqDssvsebj4uPUt9BxFPm", "5f4QEXpTcKt9Tbngn4hvraXd2G4QVFkagabUeKpn55oc", "DJHD5cMjRkuVh4enQ6dqCAuPNUPfH2a45gLv7JDDswbT", "BBmMHCw392RLiByVEUT2ac6y6qJpbW9xj8TRdgk5rMGN", "48YmkyvfoXnXqEojpJieMfFPnCv2enEuqEJcmhvkcdAk", "3kcJhRkTDuL51HmikzWFMndWwJbJGg8dZeqG9e72c6LV", "qzuTJJVd5boi6Uyoco1WWnSgzTb7q8uN79AjBT9x9N3", "9PdutHsdDSxcKUUyDg8ovS1KWh47qLT5R9h5uoFgRUH2", "Gr8NxBLHe7yjSsnSTgTqGr7BHbyAUVPJqs8fnudEE4Sf", "HtKZ3SwHcGM7nTsm8ykRGttfodFfLkghrmD5FGypgobK"], "currencies": ["TFT"]}, "customer_tid": 2391, "customer_signature": "c7c11b7e4a661324e29a9349d152b50d70cfa2dd48e9e0b67e861e863bacb6a35aa8472e7278cbf73a85c53bc109e6470ef350f836c4b53e0667ebe761e4980b"}' headers: {'user-agent': 'python-requests/2.24.0', 'accept-encoding': 'gzip, deflate', 'accept': '*/*', 'connection': 'keep-alive', 'threebot-id': '2391', 'Content-Length': '1674', 'content-type': 'application/json', 'date': 'Mon, 21 Dec 2020 15:25:47 GMT', 'authorization': 'Signature keyId="2391",algorithm="ed25519",signature="H+xt6vNbfYTdi2JW0JDKAuQdjVJ9Xjmzd5yIN5T7YdhYyFZho8qcEywDPszZV2ujs+QIWF3CzPJ0KVZsfCmUDw==",created="1608564347",expires="1608564407",headers="(created) date threebot-id"'}
2020-12-21 15:25:47.902 | INFO     | jumpscale.sals.vdc.deployer:_log:734 - VDC: 24bb689a139140219a12eb0288f782c9 NAME: testca: OWNER: magidentfinal function: extend executed successfully. return PoolCreated(
  reservation_id=1498,
  escrow_information=PoolEscrow(
  address='',
  asset='',
  amount=0
)
)
2020-12-21 15:25:47.904 | INFO     | jumpscale.sals.vdc.deployer:_log:734 - VDC: 24bb689a139140219a12eb0288f782c9 NAME: testca: OWNER: magidentfinal extending pool 1471 with cus: 0, sus: 144, ipv4us: 0, reservation_info PoolCreated(
  reservation_id=1498,
  escrow_information=PoolEscrow(
  address='',
  asset='',
  amount=0
)
)

@muhamadazmy no I didn't do a payment since it is on devnet and escrow amount is returned 0

@m-motawea devnet is having issues atm, fixing. Will let you know when it's done

@m-motawea retry please. We just redeployed devnet explorer.

NOW you can test :D

I can extend the pool now. thanks


Remove dependence on old version of the qrcode-generator library and streamline the development artifacts and external API

This set of changes aims to bring this project up to date with latest upstream developments, along with proper versioning of the declared dependencies and a switch to yarn tooling.
The dependence on an outdated version of kazuhikoarase/qrcode-generator has become an impedance to further development of this utility and equivalent or better ASCII rendering functionality is now part of the upstream library, and so this PR removes its implementation from the source code of this project and sets up a dependence on the latest upstream version of qrcode-generator.
Also, this PR fixes the unit tests and makes some small but material changes to the external API.
In terms of further development, it looks valuable to incorporate the ability to set the data mode, code type, and error correction level both when calling the node API and also when invoking this utility from the command line.

@gtanner, would you please consider merging this?

@mwbrooks, would you please take a look at this pull request?

@gtanner, I hate to bug you but...


Moldova ban list delay issues

I read these as "pending a court decision" as in "not active yet" which probably means we should not treat these as active yet.
Google translate:

The execution of AAP decision no. 7/23 of 05/29/2023 according to the decision of the Chisinau Court, Rișcani seat of 07/07/2023, until the examination of the action in administrative litigation.

The execution of AAP decision no. 8/23 of 29.05.2023 according to the decision of the Chisinau District Court Rîscani of 07.07.2023, until the court decision remains final.

no response to email to<EMAIL_ADDRESS>messaged a Moldovan friend to help with interpretation.

This has been unresolved too long now. What if we just put the whole string in the sanction.reason prop? No need to lose sleep parsing this.


docs(beta): remove beta warning from header

closes #3816

Coverage increased (+0.2%) to 96.945% when pulling a72d698d2966db2547b50d0e0683fa4f135669ac on JWO719:feature/remove-beta-warning into df0ea7c78767c07a6ed839608af5a7bb4cefbde5 on ReactiveX:master.

Thanks, @JWO719!


Complete Maintainers field for E4S Product Dictionary

@sameershende @maherou  we have this page https://e4s.readthedocs.io/en/latest/E4S_Products.html which i helped create that summarize all the E4S Products in single page. This has been quite useful when we need to reach out to package maintainers and I'd like this list to be updated so facilities can reach out to the correct person. Often times the maintainers field in spack package is not the same as E4S POC simply because we may have a maintainer non-US citizen that may not have access to DOE machine so it doesn't make sense we reach out to some unknown person that may violate security protocols.
I'd like these maintainers to have access to DOE system at the minimum but these POC would also serve as a direct communication for any E4S support including

User support tickets that reach up to this project
POC for compliance with E4S community policy that would be useful for new products that get added into list using the issue form https://github.com/E4S-Project/e4s/issues/new/choose "New Package Request"
POC for debugging package builds at facility and contributing test at facility

So one option is we reach out to each teams and get the POC and populate this ourselves, or we communicate this with all the ECP teams that are E4S member products and review this list and update this if its inaccurate. The second option would sustainable if users know about this page, often times folks may leave their position go elsewhere so it should be responsibility for teams to update the list as pose to us figuring this out.
In case you forgot you may want to read https://github.com/E4S-Project/e4s/pull/19 which started this discussion and led to this page

Hi Mike,
Yes, the E4S support team should be the primary contact for these concerns and they would route it to the product team and triage and update the issue on the issue tracker.
Thanks,
- Sameer

On Feb 10, 2022, at 9:12 AM, Mike Heroux @.***> wrote:
Hi Shahzeb,
As we stand up the E4S support contract, I would prefer that the E4S support team be the primary contact for these concerns. I am not opposed to having a list of maintainers for each product, but am thinking that using the E4S support team could do a good job of handling these issues as part of the tier-2 responsibilities.
Does this make sense to you and Sameer?
Mike

From: Shahzeb Siddiqui @.>
Sent: Thursday, February 10, 2022 10:50 AM
To: E4S-Project/e4s @.>
Cc: Heroux, Michael A @.>; Mention @.>
Subject: [EXTERNAL] [E4S-Project/e4s] Complete Maintainers field for E4S Product Dictionary (Issue #45)
@sameershendehttps://github.com/sameershende @maherouhttps://github.com/maherou we have this page https://e4s.readthedocs.io/en/latest/E4S_Products.html which i helped create that summarize all the E4S Products in single page. This has been quite useful when we need to reach out to package maintainers and I'd like this list to be updated so facilities can reach out to the correct person. Often times the maintainers field in spack package is not the same as E4S POC simply because we may have a maintainer non-US citizen that may not have access to DOE machine so it doesn't make sense we reach out to some unknown person that may violate security protocols.
I'd like these maintainers to have access to DOE system at the minimum but these POC would also serve as a direct communication for any E4S support including

User support tickets that reach up to this project
POC for compliance with E4S community policy that would be useful for new products that get added into list using the issue form https://github.com/E4S-Project/e4s/issues/new/choose "New Package Request"
POC for debugging package builds at facility and contributing test at facility

So one option is we reach out to each teams and get the POC and populate this ourselves, or we communicate this with all the ECP teams that are E4S member products and review this list and update this if its inaccurate. The second option would sustainable if users know about this page, often times folks may leave their position go elsewhere so it should be responsibility for teams to update the list as pose to us figuring this out.
—
Reply to this email directly, view it on GitHubhttps://github.com/E4S-Project/e4s/issues/45, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAHZIS4FGPZMRPPKKMKXERLU2PUE3ANCNFSM5OBKZSXQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID: @.***>
—
Reply to this email directly, view it on GitHub https://github.com/E4S-Project/e4s/issues/45#issuecomment-1035183840, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAUHIOIHG6GWYZ7RVESSIBTU2PWW5ANCNFSM5OBKZSXQ.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.

So what should we do with the maintainers list in the table. Should we remove it?

Good question.  I think we still want a maintainer list, but don’t want users to directly contact a maintainer as the default first step.  The details are things we still need to work out as we ramp up on the E4S support effort.  The contract for E4S support should be in place soon.
Michael A. Heroux
+1<PHONE_NUMBER>
https://maherou.github.io

From: Shahzeb Siddiqui @.>
Sent: Thursday, February 10, 2022 1:51:31 PM
To: E4S-Project/e4s @.>
Cc: Heroux, Michael A @.>; Mention @.>
Subject: [EXTERNAL] Re: [E4S-Project/e4s] Complete Maintainers field for E4S Product Dictionary (Issue #45)
So what should we do with the maintainers list in the table. Should we remove it?
—
Reply to this email directly, view it on GitHubhttps://github.com/E4S-Project/e4s/issues/45#issuecomment-1035421176, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAHZIS5HM6HDGEXOJ2HDLYDU2QJMHANCNFSM5OBKZSXQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID: @.***>

I understand, so it seems like we need this information internally for E4S team. Whether this be outside of github in some gdrive or excel spreadsheet would be fine.

Correct.  I don’t think there is a rush to move it right now.  Let’s see how our workflows evolve.
From: Shahzeb Siddiqui @.>
Reply-To: E4S-Project/e4s @.>
Date: Thursday, February 10, 2022 at 2:11 PM
To: E4S-Project/e4s @.>
Cc: "Heroux, Michael A" @.>, Mention @.***>
Subject: [EXTERNAL] Re: [E4S-Project/e4s] Complete Maintainers field for E4S Product Dictionary (Issue #45)
I understand, so it seems like we need this information internally for E4S team. Whether this be outside of github in some gdrive or excel spreadsheet would be fine.
—
Reply to this email directly, view it on GitHubhttps://github.com/E4S-Project/e4s/issues/45#issuecomment-1035447133, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAHZIS5LM7PH4I3UK52HWIDU2QLVHANCNFSM5OBKZSXQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID: @.***>

Resolved.


Tab Components

I have gone through these following points

[] Check latest documentation: https://docs.nativebase.io/
[] Check for existing open/closed issues for a possible duplicate before creating a new issue: https://github.com/GeekyAnts/NativeBase/issues
[] Use the latest NativeBase release: https://github.com/GeekyAnts/NativeBase/releases
[] Check examples from NativeBase KitchenSink https://github.com/GeekyAnts/NativeBase-KitchenSink
[] For discussion purpose make use of NativeBase Slack: http://slack.nativebase.io/
[] For queries related to theme, check Theme Variables from Docs and live NativeBase Theme Editor http://nativebase.io/customizer/

Issue Description

node, npm, react-native, react and native-base version, expo version if used, xcode version
Expected behaviour
Actual behaviour
Steps to reproduce

Is the bug present in both iOS and Android or in any one of them?
Any other additional info which would help us debug the issue quicker.
###Important
If you want your issue to be looked at quicker, attach a snack reproducible with your issue. Makes it easier for us!

How to use tab component inside single component with his own props.? How to re - render for child component when tab is changed

Hi @HeinXtet , can you explain a little bit more?

okay . i have dynamic tabs component , but use screen for components are same class .. when tab is changed child component is not re-rendered . example.  i have  3 tab and <ChildTabComponent props={owen props eg: index or id}/> is wrapped . when slide or pressed the tab , childtab is not re-rendered . how can re-render for that?

           <Tabs               tabBarBackgroundColor={Colors.grayBackground2}               tabBarUnderlineStyle={{backgroundColor: Colors.primaryColor}}               renderTabBar={() => <ScrollableTab />}               style={{backgroundColor: Colors.grayBackground2}}>               {this.props.category!.subCategories.map((item) => (                 <Tab                   activeTabStyle={{backgroundColor: Colors.grayBackground2}}                   tabStyle={{backgroundColor: Colors.grayBackground2}}                   activeTextStyle={styles.activeTabStyle}                   heading={getLabel(item.name, localeProvider.name)}>                   <ProductList item={item} />                 </Tab>               ))}             </Tabs>  i use like this . i just want to re-render for child component  when tab is changed

Hi, please refer to the docs here, we can use onChangeTab callback for your purpose.
https://docs.nativebase.io/Components.html#tabs-def-headref

yes onChangeTap can handle in parent Component but child component didupdate is never call even change some prop

Hi @HeinXtet , can you provide minimal code reproduction on snack for the same?


Add apt GPG key keyserver receive failed: No data-Warning: apt-key is deprecated. Manage keyring files in trusted.gpg.d instead (

Can you fix the file: tasks/general_use_software.yaml :
ASK [post-installation : Add apt GPG key] *************************************
fatal: [default]: FAILED! => {"changed": true, "cmd": "apt-key adv --keyserver hkp://keyserver.ubuntu.com:80 --recv-keys CB2DE8E5", "delta": "0:00:00.697673", "end": "2023-03-02 11:06:44.712359", "msg": "non-zero return code", "rc": 2, "start": "2023-03-02 11:06:44.014686", "stderr": "Warning: apt-key is deprecated. Manage keyring files in trusted.gpg.d instead (see apt-key(8)).\ngpg: keyserver receive failed: No data", "stderr_lines": ["Warning: apt-key is deprecated. Manage keyring files in trusted.gpg.d instead (see apt-key(8)).", "gpg: keyserver receive failed: No data"], "stdout": "Executing: /tmp/apt-key-gpghome.0trD8oSkFF/gpg.1.sh --keyserver hkp://keyserver.ubuntu.com:80 --recv-keys CB2DE8E5", "stdout_lines": ["Executing: /tmp/apt-key-gpghome.0trD8oSkFF/gpg.1.sh --keyserver hkp://keyserver.ubuntu.com:80 --recv-keys CB2DE8E5"]}

Hi @borisux
I just pushed a fix for this issue as well.
Can you confirm it's working for you now?


Update PlayingOnDev.md

added the names of the client and server zip as people oftwn download the source

Is this solving an actual problem? I haven't heard of anyone downloading the sources by mistake, and the source code zips are clearly labeled as such.


Add an optional parameter similarity_score to VectorContextRetrieve…

Description
Added an optional parameter similarity_score to VectorContextRetriever for filtering the nodes retrieved
New Package?
Did I fill in the tool.llamahub section in the pyproject.toml and provide a detailed README.md for my new integration or package?

[ ] Yes
[x] No

Version Bump?
Did I bump the version in the pyproject.toml file of the package I am updating? (Except for the llama-index-core package)

[ ] Yes
[x] No

Type of Change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] 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

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Added new unit/integration tests
[ ] Added new notebook (that tests end-to-end)
[x] I stared at the code and made sure it makes sense

Suggested Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added Google Colab support for the newly added notebooks.
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] I ran make format; make lint to appease the lint gods

@nerdai I had committed the test code by mistake. I've modified the code to resolve all the issues you've mentioned in the latest commit.

@nerdai the only issue I can think of is if the vector store returns distance instead of a score.

But, should be fine for now too :)


Set thresholds for retrying describeJobs in AWS Batch integration

describeJobs API for AWS Batch has particularly low concurrency limits. This PR adds throttling and jitter to the calls we make to aid in higher resiliency.

@romain-intel Can you bless this PR?

FWIW, I'm still hitting this error when attempting to run ~300 Batch jobs concurrently.

I'm also still running into this error with 200 concurrent Batch jobs.

I'm also still running into this error with 200 concurrent Batch jobs.

I just ran into this error with only 1 concurrent Batch job. Ironically, it happened when trying to start the end task which is empty for my flow... I'm using metaflow 2.2.7


error: root struct of file 'std' has no member named 'Build'

Hi,
I'm on Pop!_OS 22.04 LTS, zig version 0.10.1 installed via snap.
Trying to run the following example
git clone https://github.com/michal-z/zig-gamedev.git
cd zig-gamedev
zig build physically_based_rendering_wgpu-run

I get this error:
$ zig build physically_based_rendering_wgpu-run
/home/ck/git/github/michal-z/zig-gamedev/build.zig:6:21: error: root struct of file 'std' has no member named 'Build'
pub fn build(b: *std.Build) void {
                 ~~~^~~~~~
referenced by:
    runBuild: /snap/zig/6352/lib/build_runner.zig:231:45
    usage__anon_5148: /snap/zig/6352/lib/build_runner.zig:242:13
    remaining reference traces hidden; use '-freference-trace' to see all reference traces

I just updated to zig version 0.11.0-dev.1575+289e8fab7 and the error messages changed:
$ zig build physically_based_rendering_wgpu-run
/home/ck/git/github/michal-z/zig-gamedev/build.zig:104:18: error: no field or member function named 'addPackage' in 'Build.CompileStep'
    zbullet_tests.addPackage(zmath.pkg);
    ~~~~~~~~~~~~~^~~~~~~~~~~
/snap/zig/6526/lib/std/Build/CompileStep.zig:1:1: note: struct declared here
const builtin = @import("builtin");
^~~~~
referenced by:
    runBuild__anon_6037: /snap/zig/6526/lib/std/Build.zig:1560:27
    usage__anon_5706: /snap/zig/6526/lib/build_runner.zig:239:20
    remaining reference traces hidden; use '-freference-trace' to see all reference traces

/home/ck/git/github/michal-z/zig-gamedev/libs/zmesh/build.zig:29:46: error: root struct of file 'Build' has no member named 'Pkg'
pub fn getPkg(dependencies: []const std.Build.Pkg) std.Build.Pkg {
                                    ~~~~~~~~~^~~~
/home/ck/git/github/michal-z/zig-gamedev/libs/zgpu/build.zig:51:46: error: root struct of file 'Build' has no member named 'Pkg'
pub fn getPkg(dependencies: []const std.Build.Pkg) std.Build.Pkg {
                                    ~~~~~~~~~^~~~
/home/ck/git/github/michal-z/zig-gamedev/samples/minimal/build.zig:44:8: error: no field or member function named 'addPackage' in 'Build.CompileStep'
    exe.addPackage(zwin32.pkg);
    ~~~^~~~~~~~~~~
/snap/zig/6526/lib/std/Build/CompileStep.zig:1:1: note: struct declared here
const builtin = @import("builtin");
^~~~~
/home/ck/git/github/michal-z/zig-gamedev/libs/zd3d12/build.zig:35:46: error: root struct of file 'Build' has no member named 'Pkg'
pub fn getPkg(dependencies: []const std.Build.Pkg) std.Build.Pkg {
                                    ~~~~~~~~~^~~~
/home/ck/git/github/michal-z/zig-gamedev/libs/zpix/build.zig:29:46: error: root struct of file 'Build' has no member named 'Pkg'
pub fn getPkg(dependencies: []const std.Build.Pkg) std.Build.Pkg {
                                    ~~~~~~~~~^~~~
/home/ck/git/github/michal-z/zig-gamedev/libs/zbullet/build.zig:3:26: error: root struct of file 'Build' has no member named 'Pkg'
pub const pkg = std.Build.Pkg{

Hi, yes there were some breaking changes in Zig build API - I'm currently adapting the code to those changes. Sorry for the inconvenience.
You could use compiler version 0.11.0-dev.1557 for now.

Most libs and samples should compile and run fine now.

Everything should work fine now with the latest Zig compiler.


FLEX-3263 ~ bug/jms template receive timeout

All is well.

This pull request makes the build fail.

All is well.


feat: MediaGallery

GitHub Issue (If applicable): closes https://github.com/unoplatform/uno/issues/16721, closes https://github.com/unoplatform/uno.toolkit.ui/issues/1147

PR Type
What kind of change does this PR introduce?

Feature

What is the current behavior?

What is the new behavior?

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Tested code with current supported SDKs
[ ] Tested the changes where applicable:

[ ] UWP
[ ] WinUI
[ ] iOS
[ ] Android
[ ] WASM
[ ] MacOS

[ ] Updated the documentation as needed:

[ ] General Doc Update
[ ] Controls Doc Update
[ ] Extensions Doc Update
[ ] controls-styles.md
[ ] lightweight-styling.md (LightWeight Styling Resource Keys)

[ ] Runtime Tests and/or UI Tests for the changes have been added (for bug fixes / features) (if applicable)
[ ] Contains NO breaking changes
[ ] Associated with an issue (GitHub or internal)
[ ] Commits must be following the Conventional Commits specification.

Other information

Internal Issue (If applicable):

@agneszitte docs added


Refactor PagedDoraWorker by injecting MetaManager and UfsManager

What changes are proposed in this pull request?

If an object is created inside PagedDoraWorker constructor, extract that creation to before the constructor and use dependency injection to inject it to the worker object. This doesn't change any creation logic, just a refactor to better adapt to dependency injection flavor.
There is a circular dependency between MetaManager and PagedDoraWorker. This change removes that cycle. Now we create one, then create the other. Before, we create one and in the construction, we let this ref escape and create the other. Some methods are either moved or changed to static.
By adapting to dependency injection, we rely on UfsManager interface instead of DoraUfsManager implementation. Some method signatures are extracted to the interface level.
A few other small refactors to get rid of some downcasts and variable scope changes. Reasons are attached in comments on this PR.

Why are the changes needed?
Improve code quality and extensibility.
Does this PR introduce any user facing changes?
No. All refactor changes are small and equivalent to existing code. So nothing should break.

alluxio-bot, merge this please

alluxio-bot, cherry-pick this to branch-3-1.1.x please


wishlist redesign

Description
Redesign wishlist page according to the new design on FE
Checklist

[ ] 🔽 My branch is up-to-date with "development" branch
[ ] ✅All tests passed locally
[ ] ✨My changes working with up-to-date admin and back-end part locally, like charm
[ ] 🔗 Link pull request to issue

Please add screenshot of console. It should be clear.


Add a linter and goimports checker

This PR adds a linter that will be used by the CI for PR checks. Closes #7

The CI was fixed and this PR passes the checks now, so putting it under review again

Rebased


APIs for terminator results that forward ownership

Add SILValue::getDefiningInstructionOrTerminator().
This allows code to handle terminator results similar to any other
instruction result. Data flow should generally handle terminator
results like any other instruction that may forward operand ownership
to its results. The fact that it is represented as a block argument is
an implementation detail that gets in the way of conceptual
simplicity.

Add APIs for terminator results that forward ownership.
Add TermInst::forwardedOperand.

Add SILArgument::forwardedTerminatorResultOperand. This API will be
moved into a proper TerminatorResult abstraction.

Remove getSingleTerminatorOperand, which could be misused because it's
not necessarilly forwarding ownership.

Remove the isTransformationTerminator API, which is not useful or well
defined.

Rewrite several instances of complex logic to handle block arguments
with the simple terminator result API. This defines away potential
bugs where we don't detect casts that perform implicit conversion.

Replace uses of the SILPhiArgument type and code that explicitly
handle block arguments. Control flow is irrelevant in these
situations. SILPhiArgument needs to be deleted ASAP. Instead, use
simple APIs like SILArgument::isTerminatorResult(). Eventually this
will be replaced by a TerminatorResult type.

@swift-ci test

@swift-ci smoke test


[clipboard-x11] fix SendDataRequest re-run task

What does the pull request do?

re-run task in SendDataRequest
What is the current behavior?

wrong dependency on _requestedFormatsTcs to create new task
What is the updated/expected behavior with this PR?

SendDataRequest stability
How was the solution implemented (if it's not obvious)?

Checklist

[ ] Added unit tests (if possible)?
[ ] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/Documentation with user documentation

Breaking changes

Obsoletions / Deprecations

Fixed issues

You can test this PR using the following package version. 11.0.999-cibuild0037073-beta. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]


After opening AppMaps from the instructions page, only one AppMap tab is functional

To reproduce:

Open the Open AppMaps instruction page
Click on one of the links to an AppMap
Then click back to the instruction page
Then click back to the AppMap

Now, only one of the tabs will work correctly. The other will be blank.

Thanks! Unfortunately, there's not much we can do about this because the rendering and panel handling is provided by the SDK.
This is hopefully fixed by https://github.com/applandinc/appmap-intellij-plugin/pull/81/commits/18c7cbebdde0b4f3c50c01db67bb144448b6d542. I'd be glad if you could test this again with the next update.

Should be fixed, please reopen if this is not working yet for you


Getting a no method named concat2 found for type rusoto_core::ByteStream in the current scope

Using rusoto_core and s3 = '0.36.0'
and trying to run

    let request = GetObjectRequest {
        bucket: String::from("my-bucket"),
        key: String::from("my-object-key"),
        ..Default::default()
    };

    let object = s3.get_object(request).sync().unwrap();
    let stream = object.body.unwrap();

    let bytes  = stream.concat2().wait().unwrap()

leads to an error as below
error[E0599]: no method named concat2 found for type rusoto_core::ByteStream in the current scope

I'm hitting this today on 0.42 of Rusoto + futures 0.3
I'm struggling to find the futures 0.3 version of concat2 that works for Rusoto. It seems concat is missing some constraints.
Its also possible that there is an easy way to read the body of an S3 GetObject call that I'm missing.


[Snyk] Security upgrade babel-plugin-module-resolver from 4.1.0 to 5.0.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `npm` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

package.json
package-lock.json

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity

499/1000  Why? Proof of Concept exploit, Recently disclosed, CVSS 6.4
Prototype Pollution SNYK-JS-JSON5-3182856
Yes
Proof of Concept

(*) Note that the real score may have changed since the PR was raised.

Commit messages

Package name: babel-plugin-module-resolver
    The new version differs by 3 commits.

ff8a445 chore(release): 5.0.0
c43e71c chore: Update dependencies and find-babel-config to fix json5 vulnerabilities (#441)
f2daf06 docs: Valid install commands when using Github copy button (#426)

See the full diff

Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Prototype Pollution

I though we originally did not use this, but we do just not for it seems, but for a log we use to access the library, it's a bit weird, but yeah I think we need to update this.
Can't really test as this is only used in prod for logs, so even it fails it's not critical, and I see no reason why it should fail.

not an emergency snyk just doesn't have a list

🚀 Deployed to staging by @sketchydroide in version: 1.2.52-1 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by @Julesssss in version: 1.2.52-4 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Michael/agora 1686 copy addresses when clicking on 0x5f4 shorten addresses UI

resolves AGORA-1686.
I implemented two different approaches, as shown in the loom. I'm more of a fan of the second approach, since it offers better visual affordance that the address is copyable, but the first approach is more consistent with the behavior of clicking the discord icon to copy it.
I'm happy to go with either, just let me know!

PR-Codex overview
The focus of this PR is to enhance user interaction by adding copy functionality to human addresses in delegate cards.
Detailed summary

Added copyable prop to DelegateProfileImage for copy functionality
Created CopyableHumanAddress component for copying addresses
Updated DelegateCard and Stack components to handle onClick events

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

I think I'm getting prettier errors which is causing the failed builds... I'll figure out how to get it such that my code is formatting correctly on save. I have a new computer so it's currently not set up 100% yet.

my only feedback is that this icon is a little thin! does hero icons have a thicker version? if not feel free to grab a thicker one as an svg here: https://www.figma.com/file/KgDS03VEYPclv3l7lJ3Sfm/Agora-Icons?type=design&node-id=181%3A128951&mode=design&t=lnzGu9Gg45a2PNSP-1
Also i know our icon system is a bit of a mess! file it under the list of design system to dos

 my only feedback is that this icon is a little thin! does hero icons have a thicker version? if not feel free to grab a thicker one as an svg here: https://www.figma.com/file/KgDS03VEYPclv3l7lJ3Sfm/Agora-Icons?type=design&node-id=181%3A128951&mode=design&t=lnzGu9Gg45a2PNSP-1
Also i know our icon system is a bit of a mess! file it under the list of design system to dos

I don't think heroicons have a thicker icon, they have a solid, filled icon set but I wasn't a big fan of the solid filled clipboard icon. I can try one of yours though, thanks for the link!


After I use reverse proxy, I can't log in via domain

After I use reverse proxy, I can't log in. I let my_IP:3000 reserve proxy huginn.my_domain and then I found that I can log in via the IP:port but when I use the huginn.my_domain to log in, the word "The change you wanted was rejected. Maybe you tried to change something you didn't have access to." is present and I can't login.
What's the problem? And how I can solve it?

You may refer to these comments
https://github.com/huginn/huginn/issues/2357#issuecomment-442313281
https://github.com/huginn/huginn/issues/2569#issuecomment-547844248

If you have this line in your reverse proxy config, REMOVE it:
add_header Referrer-Policy no-referrer;

Correct value:
#add_header Referrer-Policy no-referrer;


Don't generate dbml refs for embedded schemas

Currently for embedded schemas that reference other schemas, the generated dbml file is invalid.
Example:
This schema
typed_schema "my_schema" do
  embeds_many :embedded_schema, EmbeddedSchema, on_replace: :delete, primary_key: false do
    field(:my_field, :string)

    belongs_to(:other_schema, OtherSchema, primary_key: true, type: :binary_id)
  end
end

Generate a Ref "line" like the following (notice the empty string):
Ref: other_schema.id < "".other_schema_id
This PR changes the behaviour to ignore lines like this.

Thank you!


feat: email internationalization/translation

Email sent to users should match a new settings called "Language" that is used to determine in what language the emails should be written. This new feature added in the settings could still be different from the one currently on the website that translate the showed text on the fly.

Do you have an URL to some documentation/examples for this ?

Do you have an URL to some documentation/examples for this ?

There is a blog post about this topic here: How to use djoser signals


TFLite: use the int8 type for no_float quantization

use the int8 type for inference input and output when quantizing with int8_no_float mode

Suharsh,
Could you review this pull request?

The model signature (input/output) needs to be opted into directly via the inference_input and inference_output flags. I believe there is work in progress to introduce these flags into the V2 converter.
The reason for this is that users who are converting to a float model, will suddenly get a model that expects a different input type and output type than what is obvious, so the converter expects the input type and output type provided explicitly.
so this is working as intended for now.

Here is the issue tracking that https://github.com/tensorflow/tensorflow/issues/38285


Generated Datadog provider.tf

Hey team!
We're getting to know terraformer, for a quick spike we're using it to import some Datadog resources. Everything seems to be setup correctly and the command to import those resources seems to do its job:
terraformer import datadog --path-output="my-project" --path-pattern="{output}/" --compact \
    --resources=monitor --filter="Name=tags;Value='role:my-awesome-role'" \

The above generates a provider.tf which seems to be in conflict with the latest datadog provider published address:
terraform {
  required_providers {
    datadog = {
      version = "~> 3.0.0"
    }
  }
}

When running terraform plan we're getting the following error:
╷
│ Error: Could not load plugin
│
│
│ Plugin reinitialization required. Please run "terraform init".
│
│ Plugins are external binaries that Terraform uses to access and manipulate
│ resources. The configuration provided requires plugins which can't be located,
│ don't satisfy the version constraints, or are otherwise incompatible.
│
│ Terraform automatically discovers provider requirements from your
│ configuration, including providers used in child modules. To see the
│ requirements and constraints, run "terraform providers".
│
│ failed to instantiate provider "registry.terraform.io/hashicorp/datadog" to obtain schema: unknown provider "registry.terraform.io/hashicorp/datadog"

We're assuming the provider.tf configuration should override the source path of the module, like:
terraform {
  required_providers {
    datadog = {
      source = "datadog/datadog"
      version = "~> 3.0.0"
    }
  }
}

Adding the source field for the provider fixes our issue, would be nice tho that terraformer generated the right provider configuration.
Thoughts?
Many thanks!

+1

Yeah might be stale but still an issue I think.. :)

+1

Agreed, this is still an issue.
provider.go currently parses a file path like .terraform/providers/registry.terraform.io/datadog/datadog/3.15.1/darwin_amd64/terraform-provider-datadog_v3.15.1 to figure out the name and version. The source is not there to parse, so the fix for this would probably have to have a mapping of names to sources hard coded.


Feature Request: Download latest acquia backup instead of dumping it ourselves

Provide ability to use existing Acquia backups
When doing a ddev pull acquia an on-demand data dump is initiated, which can take a very long time. In most instances, using the backup databases that may have been created by Acquia would be sufficient. In my case, this saves an hour and a quarter.
Existing backup downloaded by default with option for on-demand dump
Here is the output from a Lando pull:
lando pull                                                                                ─╯
? Pull code from? none
? Pull database from? Stage, test (vcs: sprint4.18-certificate-functionality)
? Pull files from? none
--
Saved credentials to /var/www/.acquia/cloud_api.conf
    ✔ Destroying all current tables in local database if needed...

 ! [NOTE] This is a multisite application. Drupal will load the default site unless you've configured sites.php for this
 !        environment: https://docs.acquia.com/cloud-platform/develop/drupal/multisite/

 [INFO] Using a database backup that is 11 hours old. Backup #189250915 was created at Sun Oct 24 6:48:35 UTC 2021.

        You can view your backups here:
        https://cloud.acquia.com/a/environments/12594-15786fc5-f286-4275-abf8-af232cce1823/databases

        To generate a new backup, re-run this command with the --on-demand option.

    ✔ Importing Drupal database download
        }
    ✔ Checking db pull for expected tables...
    ✔ Pull completed successfully!

The acquia integration in soon-to-be-released release v1.18.1 downloads a gzipped database, which may fix the concerns you have, I'd appreciate it if you could try it out. It may solve the slowness issue. It still requires a dump on the server side, but the contents are brought down as a gzipped file.

@siliconmeadow could you please take a look at the acquia.yaml.example in ddev v1.18.1 and give it a try and see if it resolves your download issue?

Still hope you'll give that a try @siliconmeadow

Hi @rfay - am I right in thinking that the magic is:
drush @${project_id} sql-dump -n --alias-path=~/.drush --extra-dump=--no-tablespaces --gzip >db.sql.gz

If so, I've been successful with that in the past. I needed to change the --extra-dump to simply --extra when using Drush 8, but that's a different matter and I was considering how to handle different versions of Drush more gracefully.
Rather than doing this, what will perform better is if we download the latest db backup, as you'd saw was possible with acli, instead of creating an "on demand" one. I picked apart the Lando Acquia plugin to see what they were doing with their use of acli. Instead of the drush sql-dump command above, I've swapped it with:
acli -n pull:db @${project_id}

Which gets closer, but it seems like acli makes assumptions about the local environment and I can't see where to change that. In short, I don't see where they've specified --host localhost --user drupal drupal. Here is the verbose transcript when running acli -n pull:db my_acquia_project.dev directly in the web container:
ddev exec acli  -n -vvv pull:db rcnimain.dev                                                                                ─╯

Box Requirements Checker
========================

> Using PHP 7.4.26
> PHP is using the following php.ini file:
  /etc/php/7.4/cli/php.ini

> Checking Box requirements:
  ✔ The application requires the version "^7.3 | ^8.0" or greater.
  ✔ The application requires the extension "json".
  ✔ The package "brick/math" requires the extension "json".
  ✔ The package "guzzlehttp/guzzle" requires the extension "json".
  ✔ The package "ramsey/uuid" requires the extension "json".
  ✔ The package "zumba/amplitude-php" requires the extension "curl".

Acquia CLI version: 1.19.0
Connecting to database drupal
[notice] Command: 'which' 'mysql' [Exit: 0]
ERROR 2002 (HY000): Can't connect to local MySQL server through socket '/run/mysqld/mysqld.sock' (2)

[notice] Command: 'mysql' '--host' 'localhost' '--user' 'drupal' 'drupal' [Exit: 1]

In PullCommandBase.php line 371:

  [Acquia\Cli\Exception\AcquiaCliException]
  Unable to connect to local database using credentials mysql:://drupal:drupal@localhost/drupal. ERROR 2002 (HY000): Can't connect to local MySQL server through socket '/run/mysqld/mysqld.sock' (2)

Exception trace:
  at phar:///usr/local/bin/acli/src/Command/Pull/PullCommandBase.php:371
 Acquia\Cli\Command\Pull\PullCommandBase->connectToLocalDatabase() at phar:///usr/local/bin/acli/src/Command/Pull/PullCommandBase.php:129
 Acquia\Cli\Command\Pull\PullCommandBase->pullDatabase() at phar:///usr/local/bin/acli/src/Command/Pull/PullDatabaseCommand.php:52
 Acquia\Cli\Command\Pull\PullDatabaseCommand->execute() at phar:///usr/local/bin/acli/vendor/symfony/console/Command/Command.php:298
 Symfony\Component\Console\Command\Command->run() at phar:///usr/local/bin/acli/src/Command/CommandBase.php:428
 Acquia\Cli\Command\CommandBase->run() at phar:///usr/local/bin/acli/vendor/symfony/console/Application.php:1023
 Symfony\Component\Console\Application->doRunCommand() at phar:///usr/local/bin/acli/vendor/symfony/console/Application.php:299
 Symfony\Component\Console\Application->doRun() at phar:///usr/local/bin/acli/vendor/symfony/console/Application.php:171
 Symfony\Component\Console\Application->run() at phar:///usr/local/bin/acli/bin/acli:69
 require() at /usr/local/bin/acli:14

pull:database [--no-scripts] [--on-demand] [--no-import] [--multiple-dbs] [--] [<environmentId> [<site>]]

 [help] You can find Acquia CLI documentation at https://docs.acquia.com/acquia-cli/

        You can submit a support ticket at https://insight.acquia.com/support/tickets/new?product=p:cli

Failed to execute command acli -n -vvv pull:db rcnimain.dev: exit status 1

On an aside, the web container has version 1.14.x of acli. I'll look at your image build scripts to see how to get a newer version can be installed.

I was asking you to take a look at the Acquia example as it is and try it out for speed. Could you give it a shot? https://github.com/drud/ddev/blob/master/pkg/ddevapp/dotddev_assets/providers/acquia.yaml.example
I see you tested with acli v1.19, which had major behavior change with aliases download, which is why it's pinned at v1.14.
But I'd love to have you test it as is and compare the speed to what you were after. It's currently set up to pull a gzipped dump.
I'm wide open to improvements, and I saw that acli in a recent version had its own pull that might have some potential.

I tested exactly that feature for speed already.

Right, but I don't think you did since the recent revamp of the acquia.yaml.example, where it started to transfer gzipped versions of everything. Your feature request predated that change.

Right, but I don't think you did since the recent revamp of the acquia.yaml.example, where it started to transfer gzipped versions of everything. Your feature request predated that change. The question is whether your feature request is still relevant given the changes.

I did after the revamp of the file. Before 1.18.x was added. Within the same week of @greggmarshall's mod was added, I tried it out. I'm sorry I didn't mention it.
The way I see it is that the pull of the DB is actually not the most descriptive of names as it does pull the DB but also imports it.
The importing of the DB is fast. Doing a drush sql-dump is not near as fast as simply downloaded an already created gzipped file done from last night's backup. Which is what exec acli  -n my_acquia_proj.dev does. It's far more energy efficient.

Looking forward to an improvement. With ddev v1.18.2, it's no longer required to let ddev import the database, acli can do it or whatever. See https://ddev.readthedocs.io/en/latest/users/providers/provider-introduction/

Another benefit of using the Acquia backups would be to reduce the load on the production server, assuming that's the DB being pulled.  If you have a half dozen developers, each doing the appropriate thing of starting each feature with a fresh copy of the DB, you could be doing a half dozen pulls a day.  My last project was concerned about it enough that we used a spare environment (they had a bunch) and had a weekly cron job update it from production then the developers pulled from there.
I'm not quite sure how to extract the name or id of the latest backup.  I saw the ACLI commands above but not sure how you go from the output of the first to the second in a script.

I'm checking to see how I can specify a different host, username, pw and database with acli pull:db as it's defaulting to
[notice] Command: 'mysql' '--host' 'localhost' '--user' 'drupal' 'drupal' [Exit: 1]

I felt like I was making progress and thought my increased activity on this issue above was going in the right direction! Not so certain now.

acli doesn't seem to respect Drupal's settings.php for db connection? That would be weird. Definitely a bug report if it's happening. But as you know, every experiment you can try can be tried out first manually, line by line, inside the web container.

acli uses certain environmental variables for the db connection. See the setLocalDbHost method, for example. I've set ACLI_DB_USER, ACLI_DB_PASSWORD,  ACLI_DB_PASSWORD, ACLI_DB_NAME, ACLI_DB_HOST to the values ddev expects (see full transcript below).
This moves things on considerably, as the file _does_get downloaded from Acquia.
After the file is downloaded, acli then drops the db database.
The next step, creating the db database fails, with an error which seems strange:
[notice] Command: 'mysql' '--host' 'db' '--user' 'db' '-e' 'DROP DATABASE IF EXISTS db' [Exit: 0]
Creating new empty database db
ERROR 1049 (42000): Unknown database 'db'

[notice] Command: 'mysql' '--host' 'db' '--user' 'db' '-e' 'create database db' [Exit: 1]

As I've typed this, I just remembered that when doing the Lando db pull the database is not dropped. Instead of dropping the db, each table is dropped. I'll have to investigate that next.
Full transcript
ddev pull acquia --skip-files                                                                                ─╯
You're about to delete the current database and replace with the results of a fresh pull.
Would you like to continue? [Y/n] (yes):
Authenticating...
Saved credentials to /home/sheppardr/.acquia/cloud_api.conf
Obtaining database...

Box Requirements Checker
========================

> Using PHP 7.4.26
> PHP is using the following php.ini file:
  /etc/php/7.4/cli/php.ini

> Checking Box requirements:
  ✔ The application requires the version "^7.3 | ^8.0" or greater.
  ✔ The application requires the extension "json".
  ✔ The package "brick/math" requires the extension "json".
  ✔ The package "guzzlehttp/guzzle" requires the extension "json".
  ✔ The package "ramsey/uuid" requires the extension "json".
  ✔ The package "zumba/amplitude-php" requires the extension "curl".

Acquia CLI version: 1.14.0
[debug] Searching for an environment matching alias redacted.
[debug] Found application redacted matching alias redacted.
[debug] Found environment redacted matching redacted.
Connecting to database db
[notice] Command: 'which' 'mysql' [Exit: 0]
[notice] Command: 'mysql' '--host' 'db' '--user' 'db' 'db' [Exit: 0]
[debug] Using environment Dev redacted
[debug] Using database backup (id #redacted) generated at 2021-12-14T08:42:34+00:00

 [INFO] Using a database backup that is 13 hours old. Backup #redacted was created at Tue Dec 14 8:42:34 UTC 2021.

        You can view your backups here:
        https://cloud.acquia.com/a/environments/redacted/databases

        To generate a new backup, re-run this command with the --on-demand option.

    ⌛ Downloading Drupal database copy from the Cloud Platform...
    ✔ Downloading Drupal database copy from the Cloud Platform
    ⌛ Importing Drupal database download...

Dropping database db
[notice] Command: 'mysql' '--host' 'db' '--user' 'db' '-e' 'DROP DATABASE IF EXISTS db' [Exit: 0]
Creating new empty database db
ERROR 1049 (42000): Unknown database 'db'

[notice] Command: 'mysql' '--host' 'db' '--user' 'db' '-e' 'create database db' [Exit: 1]

In PullCommandBase.php line 465:

  [Acquia\Cli\Exception\AcquiaCliException]
  Unable to create a local database. ERROR 1049 (42000): Unknown database 'db'

Exception trace:
  at phar:///usr/local/bin/acli/src/Command/Pull/PullCommandBase.php:465
 Acquia\Cli\Command\Pull\PullCommandBase->createLocalDatabase() at phar:///usr/local/bin/acli/src/Command/Pull/PullCommandBase.php:268
 Acquia\Cli\Command\Pull\PullCommandBase->importRemoteDatabase() at phar:///usr/local/bin/acli/src/Command/Pull/PullCommandBase.php:158
 Acquia\Cli\Command\Pull\PullCommandBase->pullDatabase() at phar:///usr/local/bin/acli/src/Command/Pull/PullDatabaseCommand.php:50
 Acquia\Cli\Command\Pull\PullDatabaseCommand->execute() at phar:///usr/local/bin/acli/vendor/symfony/console/Command/Command.php:299
 Symfony\Component\Console\Command\Command->run() at phar:///usr/local/bin/acli/src/Command/CommandBase.php:414
 Acquia\Cli\Command\CommandBase->run() at phar:///usr/local/bin/acli/vendor/symfony/console/Application.php:996
 Symfony\Component\Console\Application->doRunCommand() at phar:///usr/local/bin/acli/vendor/symfony/console/Application.php:295
 Symfony\Component\Console\Application->doRun() at phar:///usr/local/bin/acli/vendor/symfony/console/Application.php:167
 Symfony\Component\Console\Application->run() at phar:///usr/local/bin/acli/bin/acli:89
 require() at /usr/local/bin/acli:14

pull:database [--no-scripts] [--on-demand] [--no-import] [--] [<environmentId> [<site>]]

 [help] You can find Acquia CLI documentation at https://docs.acquia.com/acquia-cli/

        You can submit a support ticket at https://insight.acquia.com/support/tickets/new?product=p:cli

Pull failed: Failed to exec # set -x   # You can enable bash debugging output by uncommenting
set -eu -o pipefail
ls /var/www/html/.ddev >/dev/null # This just refreshes stale NFS if possible
pushd /var/www/html/.ddev/.downloads >/dev/null
acli pull:db -n -vvv @${project_id}
 on web: exit status 1

It's using "localhost" for the host, I assume. That's the standard problem.

It's using "localhost" for the host, I assume. That's the standard problem.

I wish you'd have read my comment in more detail. I'd already established that and am on to the next thing to resolve. I despair.

I so appreciate all your work on this!
So the bottom line is... don't try to work inside the pull script until you can just do the things by hand, one by one.  Pioneer the acli command first.
But I see Access denied for user<EMAIL_ADDRESS>(using password: YES). Can you confirm that <IP_ADDRESS> is the right IP for the hostname db (ping <IP_ADDRESS> and verify that it's ddev-<project>-db? Note that if you're using a ddev version before v1.18.2 you may have a docker-compose version that uses ambiguous hostnames, and 'db' may not land at the right place.
But try to get acli to work inside the web container and when it works there, then transplant the technique into the acquia.yaml. It's too hard to experiment with things using the acquia.yaml.
It won't surprise me if you're up against an acli bug. But you should be able to demonstrate that pretty easily.

I'm also happy to do a zoom or whatever and work with you on this. We could have some fun.

Looking at this:
ERROR 1045 (28000): Access denied for user<EMAIL_ADDRESS>(using password: YES)

[notice] Command: 'mysql' '--host' 'db' '--user' 'root' 'db' [Exit: 1]

Lead me to log into the web container and using mysql directly.
ddev ssh                                                                                ─╯
sheppardr@rcnimain-web:/var/www/html/docroot$ mysql --host db --user root db
ERROR 1045 (28000): Access denied for user<EMAIL_ADDRESS>(using password: YES)

If I add the -p so that I get prompted to enter the password (root as you say) it works:
sheppardr@rcnimain-web:/var/www/html/docroot$ mysql --host db --user root -p db
Enter password:
Welcome to the MariaDB monitor.  Commands end with ; or \g.
Your MariaDB connection id is 31
Server version: 10.3.32-MariaDB-1:10.3.32+maria~focal-log mariadb.org binary distribution

Copyright (c) 2000, 2018, Oracle, MariaDB Corporation Ab and others.

Type 'help;' or '\h' for help. Type '\c' to clear the current input statement.

MariaDB [db]>

Out of curiosity, when you simply type mysql it seems to be the equivalent of:
mysql --host db --user db db

AND notice that password switch wasn't on that command. I'm wondering if there are further environmental variables for the mysql client command. Maybe there's a dot file for mysql or something?

Yes, and in latest ddev, the name of the host is ddev-<project>-db, so please use that.

I stepped sideways and renamed .my.cnf to my.cnf.tmp because I had a hunch that with acli, they hadn't made presumptions that whatever local environment you're working on would have a .my.cnf file. This moved further along:
ddev pull acquia --skip-files                                                                                ─╯
You're about to delete the current database and replace with the results of a fresh pull.
Would you like to continue? [Y/n] (yes):
Authenticating...
Saved credentials to /home/sheppardr/.acquia/cloud_api.conf
Obtaining database...

Box Requirements Checker
========================

> Using PHP 7.4.26
> PHP is using the following php.ini file:
  /etc/php/7.4/cli/php.ini

> Checking Box requirements:
  ✔ The application requires the version "^7.3 | ^8.0" or greater.
  ✔ The application requires the extension "json".
  ✔ The package "brick/math" requires the extension "json".
  ✔ The package "guzzlehttp/guzzle" requires the extension "json".
  ✔ The package "ramsey/uuid" requires the extension "json".
  ✔ The package "zumba/amplitude-php" requires the extension "curl".

Acquia CLI version: 1.14.0
[debug] Searching for an environment matching alias rcnimain.dev.
[debug] Found application 15786fc5-f286-4275-abf8-af232cce1823 matching alias rcnimain.
[debug] Found environment 12595-15786fc5-f286-4275-abf8-af232cce1823 matching dev.
Connecting to database db
[notice] Command: 'which' 'mysql' [Exit: 0]
[notice] Command: 'mysql' '--host' 'db' '--user' 'db' 'db' [Exit: 0]
[debug] Using environment Dev 12595-15786fc5-f286-4275-abf8-af232cce1823
[debug] Using database backup (id #194708111) generated at 2021-12-15T08:42:29+00:00

 [INFO] Using a database backup that is 8 hours old. Backup #194708111 was created at Wed Dec 15 8:42:29 UTC 2021.

        You can view your backups here:
        https://cloud.acquia.com/a/environments/12595-15786fc5-f286-4275-abf8-af232cce1823/databases

        To generate a new backup, re-run this command with the --on-demand option.

    ⌛ Downloading Drupal database copy from the Cloud Platform...
    ✔ Downloading Drupal database copy from the Cloud Platform
    ⌛ Importing Drupal database download...

Dropping database db
[notice] Command: 'mysql' '--host' 'db' '--user' 'db' '-e' 'DROP DATABASE IF EXISTS db' [Exit: 0]
Creating new empty database db
[notice] Command: 'mysql' '--host' 'db' '--user' 'db' '-e' 'create database db' [Exit: 0]
Importing downloaded file to database db
[debug] Importing /tmp/dev-rcnimain-rcnimaindev-2021-12-15T08:42:29+00:00.sql.gz to MySQL on local machine
[notice] Command: 'which' 'gunzip' [Exit: 0]
[notice] Command: 'which' 'pv' [Exit: 1]

 [WARNING] Please install `pv` to see progress bar

    ✔ Importing Drupal database download
[notice] Command: 'which' 'drush' [Exit: 0]
{
    "php-os": "Linux",
    "php-conf": [
        "\/etc\/php\/7.4\/cli\/php.ini"
    ],
    "drush-script": "\/usr\/local\/bin\/drush8",
    "drush-version": "8.4.8",
    "drush-temp": "\/tmp",
    "drush-conf": [],
    "drush-alias-files": [],
    "%paths": []
}

[notice] Command: 'drush' 'status' '--fields=db-status,drush-version' '--format=json' '--no-interaction' [Exit: 0]
[notice] Drush does not have an active database connection. Skipping cache:rebuild
Importing database...
Pull failed: open : no such file or directory

I wonder which file it can't find.... I can see the file in /tmp on the web container.

@rfay - I might be at the end on this one, as I'm starting to wonder if the sole project I'm working on at the moment is too edge case.

It's D7 and using Drush 8. That in itself shouldn't really be a problem.
It was set up as a multi-site and the settings files are also set up in an unconventional way.

Here is the output from ddev drush status
ddev drush status                                                                                ─╯
 Drupal version         :  7.82
 Site URI               :  local.rcni.com
 Database driver        :  mysql
 Database hostname      :  db
 Database port          :  3306
 Database username      :  db
 Database name          :  db
 PHP configuration      :  /etc/php/7.4/cli/php.ini
 PHP OS                 :  Linux
 Drush script           :  /usr/local/bin/drush8
 Drush version          :  8.4.8
 Drush temp directory   :  /tmp
 Drush configuration    :  /var/www/html/docroot/sites/default/drushrc.php /var/www/html/docroot/sites/all/drush/drushrc.php
 Drush alias files      :  /var/www/html/docroot/sites/rcn_nspace/drush/aliases.drushrc.php
 Drupal root            :  /var/www/html/docroot
 Drupal Settings File   :  sites/rcn_nspace/settings.php
 Site path              :  sites/rcn_nspace

Thanks so much for all the hard work on this. Multisite does in fact make it really hard to use the traditional technique, but now that the pull doesn't assume that ddev will do the import necessarily (acli can do it itself) that may work out. Anyway, thanks for all the hard work. If we can just figure out how acli is supposed to work we can make progress. There may be acli issues.

I like the idea, but I am thinking, perhaps rather than add specific resources like Aquia, what if they were to add some way where tools/features that are routinely added in the install/setup process could be added to some file, which would check and add these items. I'm thinking, like a script/batch file, just have a file that had a list of commands like:
pull acquia composer require drush/drush composer require drupal/admin_toolbar
Just my $.02
And another handy feature would be to add a command something like:
ddev self-update

We don't have access to Acquia any more; they promise to get it for us someday. But this will have to be postponed.

@siliconmeadow this is currently in head, brew install --HEAD drud/ddev/ddev and will be in v1.19.4 shortly, I hope it works for you. Of course you can try it without changing ddev at all, it's just a change to the acquia.example, https://github.com/drud/ddev/blob/6494172eb0f59ee3f39059e46667418939a557c8/pkg/ddevapp/dotddev_assets/providers/acquia.yaml.example#L41-L52


ENYO-3093: Fix video-slider focus bug

Issue
The VideoPlayer can lose window/spotlight focus while holding/pressing the VideoTransportSlider. At this point, the video controls can be hidden (via the autoHide job). At this point, when you attempt to tap the video, you receive an error in the console. When the VideoTransportSlider loses window/spotlight focus while being held/pressed, it never receives an up event (even when regaining focus), so the slider remains in a semi-unreleased/drag state. Clicking the video, even when the controls are hidden, attempts to fire drag events.
Fix
Issue an up event in the specific case where the slider loses focus while an unreleased downEvent exists.
Enyo-DCO-1.1-Signed-off-by: Jeremy Thomas<EMAIL_ADDRESS>

Could this change be relevant to moonstone/Slider as well?

This change could also be relevant to moonstone/Slider as a similar edge-case. However, in this case, moonstone/VideoTransportSlider overrides the moonstone/Slider spotBlur method, which is why I've settled on where I've placed this change. So we could look at creating another JIRA/PR for that use-case.

Closing PR as Jira is closed.


e2e(k8s): keep only hybrid windows scenario in e2e

Hybrid clusters (Windows + Linux) e2e tests capture if Windows nodes are functional which frees us to remove Windows-only e2e tests. This removes windows/kubernetes.json from

test/acse-conf/acse-feature-validation.json
test/acse-conf/acse-regression.json

This change is

@jchauncey would you like to note the ginkgo failure before we merge?

thanks. that node count failure has started cropping up some. Ill deflake shortly

Clarified with @anhowe that we can't remove the windows-only test because it is indirectly proving out windows masters as schedulable (which hybrid clusters don't).
Long-term, we'll incorporate all this coverage into the minimum number of clusters using the next-gen ginkgo runner.


dibs: isogram

Feel free to open this PR again if you resume the work. If you need any help, just ask!


How to contribute to google_maps_flutter and google_maps_flutter_platform_interface at the same time ?

I made a PR on google_maps_flutter. However, google_maps_flutter depends on google_maps_flutter_platform_interface and my PR needed to modify both plugins.
So I changed pubspec.yaml as follow in google_maps_flutter
dependencies:
  flutter:
    sdk: flutter
  flutter_plugin_android_lifecycle: ^1.0.0
  google_maps_flutter_platform_interface: # I changed this from ^1.0.4
    path: ../google_maps_flutter_platform_interface

Then I did the Pull Request but on of the checks says:
* Don't depend on "google_maps_flutter_platform_interface" from the path source. Use the hosted source instead. For example:

I don't understand. How am I supposed to use the hosted plugin although I am modifying it

Hi @tolotrasamuel
It seems you are asking for assistance. Please see https://flutter.dev/community for resources and asking questions like this,
you may also get some help if you post it on https://www.reddit.com/r/flutterhelp/
Please understand that GitHub issues is not for support on sending PRs into Flutter, you may get help by posting on https://discord.com/invite/N7Yshp4.
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you

@tolotrasamuel our tooling is not able to merge changes to multiple packages simultaneously, in order to contribute a change to google_maps_flutter that requires changes into google_maps_flutter_platform_interface, you'll need to have a PR to modify the platform_interface, get that merged and published, and then have another PR that modifies the plugin. (You can use this time to bump the minimal version required of the platform interface with the version you just published as well).
In my experience, it's been best to have one big PR with all the changes (like the one you already have), which is unpublishable but easy for people to download and test, and once that's reviewed, then create the two (or sometimes three) reviews that need to be merged in the right order, those should be approved quickly, given that the whole review work happened in the main (unpublishable) review.
Thanks!


Small cardinalities make everything nominal/ordinal

For example
a,b,c
1,2,3
4,5,6
7,8,9

The description is ambiguous -- please clarify.

@kanitw thoughts?

I do not understand what you mean by

if there are fewer than 5% different values

Alternatively, we could also set all numeric values to Q by default and add a popup to ask if they want to pick suspicious field as ordinal

@domoritz can you clarify my question?

interpret numbers as O only if there are fewer than 5% different values

My previous self meant that if there is less than 1 unique value per 20 tuples, then we should interpret things as ordinal. Otherwise quantitative. But I'm not sure how well this heuristic works.

Ok I think we'll go with your last proposal

This will be fixed as a part of https://github.com/uwdata/compassql/issues/88 and #182


deal with input files

Here's the high-level approach: for each input file path, we…

get filesystem metadata for the path

we bail here if the path doesn't exist or is a directory

use the metadata to generate a key to look up the file's hash
if we haven't stored the hash, generate it
use those hashes to calculate the job's hash (which informs if we need to re-run)

Based on #35; review/merge that first!

requesting both of your review since we don't have a group set up yet!


How to open MFEM mesh format in Paraview

Hello MFEM team,
I am interested in the MFEM mesh format to visualize high order discontinuous solutions and I have a question about the mesh format. On the MFEM website, it says "visualization tool natively supported by MFEM is ParaView".  But when I use Paraview to open a sample mesh file ('beam-quad.mesh' from the website), paraview does not recognize the mesh format. Is there some extra plugins I need to install on Paraview so that it could recognize the MFEM format?
Thanks a lot for your help!

If you want to use ParaView to visualize meshes and solutions from MFEM, you will need to export them to ParaView format. ParaView cannot read MFEM format (either natively or through a plugin).
In order to export your mesh to ParaView format, the following function should do it.
#include "mfem.hpp"
using namespace mfem;
int main()
{
   Mesh mesh = Mesh::LoadFromFile("../data/beam-quad.mesh");
   mesh.PrintVTU("beam_paraview");
   return 0;
}

If you also want to visualize solution fields, look at ParaViewDataCollection.

Okay, got it!
Thank you for the reply.


[virtual-wan] az network vhub: Add new subgroup route-map to support Managing virtual hub route map

This checklist is used to make sure that common guidelines for a pull request are followed.
https://github.com/Azure/azure-cli/issues/24061
Related command

az network vhub route-map
General Guidelines

[ ] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[ ] Have you run python scripts/ci/test_index.py -q locally?

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
The precondition is to put your code inside this repository and upgrade the version in the pull request but do not modify src/index.json.

virtual-wan


Sync?

Hello!
I have tried to sync a belongsToMany relationship and the filter seams to don't apply: all table (even brother rows) is cleared except for those who were sent on form
Have a nice day!

a failing test case would be great for tracking this down.

Closing out a long ago asked question. With more details or a failing test case we can revisit.


How to get all fabrics from device

CSA SDK V1.1, Use api OperationalCredentialsCluster->readFabricsAttribute just callback one fabric.
The iOS platform use Apple SDK api which has a params can filter callback, but java layer has not similar this.
is there any way can get all fabrics from devices.

Detailed process

Could you provide the kotlin code that you use to get all the fabrics. I'll replicate and investigate.

suspend fun getCommissionedFabrics(context: Context, nodeId: Long, callback: FabricsAttributeCallback){
    ChipClusters.OperationalCredentialsCluster(getConnectedDevicePointer(context, nodeId), 0).readFabricsAttribute(callback)
}


Use logstream in the duplicate_test

/kind cleanup
Changes
Change the task names to use the helper so the logs for them show up in logstream.
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[ ] Includes tests (if functionality changed/added)
[ ] Includes docs (if user facing)
[ ] Commit messages follow commit message best practices
[ ] Release notes block has been filled in or deleted (only if no user facing changes)

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task to build and release this image.

Reviewer Notes
If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.
Release Notes
NONE

/assign @ImJasonH @vdemeester @afrittoli

/lgtm


[1pt] Create a navigation menu

Create a navigation menu that allows users to open all of the pages you created.

PR#24


Fix  DisconnectAsync hangs on Windows and Android after scanning then…

… connecting with ConnectToKnownDeviceAsync #734
In the native Windows and Android adapter, it always create a new native IDevice instance when using ConnectToKnownDeviceAsync if the device has already been discovered via scanning you end up with two IDevice instances.  If the user then calls DisconnectDeviceAsync on the IDevice instance that was discovered via scanning, this issue occurs.
I DID NOT check the other platform adapters so this issue may be present in other platforms.  The solution is easy; just check the DiscoveredDevicesRegistry before creating a new Device instance.

Ohh no!  the builder must be broken..
wasn't me.. :)

Ohh no! the builder must be broken..
wasn't me.. :)

Yeah, it certainly does not look like you caused it. It happens in another PR as well. Might be due to updated tooling versions on the build host. Will try to look into that ...

Alright, I fixed the CI issues in #737 and rebased this one, so that all build checks succeed now.

Yeah that sounds good to me.  I can handle that.  I don't have a way to test on iOS and MAC so someone should probably look it over.  I also think we should add our device object to the DiscoveredDevicesRegistry if it's not already in there. Provided of-course that ConnectToKnownDeviceAsync succeeds.  This will cover us on both ends.

Yeah that sounds good to me. I can handle that.

Awesome, thanks! :+1:

I don't have a way to test on iOS and MAC so someone should probably look it over.

I can possibly take care of that part. In any case, if your generic code works well on Android and Windows, I think the probability is quite low that it will cause problems on iOS (given that it is reasonably simple).

I also think we should add our device object to the DiscoveredDevicesRegistry if it's not already in there. Provided of-course that ConnectToKnownDeviceAsync succeeds. This will cover us on both ends.

Yes, I think that makes sense.

Previous changes reverted and new proposed changes pushed.  Tested on Windows and Android.  I've also updated code to throw a DeviceConnectionException as apposed to a generic Exception when ConnectToKnownDeviceNativeAsync cannot find the device being requested.


Use latest grlc image

Latest clariah/grlc image can be used, so there is no need to build the image here.

CLARIAH/grlc#182 is now resolved. PR updated to use  clariah/grlc:dev docker image.

clariah/grlc:latest image can now be used :-)


Feature: Provide "Scheduler extender" for k8s

Some of the backends only allow a limited amount of storage per machine. Ex: DigitalOcean only allow 5 "block storage" per machine.
We should provide a "Scheduler extender" or something similar to ensure that k8s does not try to attach more storage that possible. For DO that means max 5 "block storage" per machine.
Scheduler extender seems to provide what we need.
See https://github.com/kubernetes/kubernetes/pull/41804#issuecomment-290259182 for some background about this "idea".

https://github.com/gaocegege/simple-k8s-scheduler-extender provide a basic "scheduler extender" example.
And we can properly use the existing NewMaxPDVolumeCountPredicate function.
We just need to create our own VolumeFilter but that shouldn't be that hard. (vol.FlexVolumeSource != nil and vol.FlexVolumeSource.Driver == "rexray ....). Last but not least we also need to create a scheduler policy file. Something like this https://github.com/kubernetes/kubernetes/blob/master/examples/scheduler-policy-config-with-extender.json , I think we can/should remove predicates and priorities, and it would then just use the default (???).
TODO: Does libstorage provide a API, so I can get max volume per machine somehow?

@klausenbusk libstorage does not provide that sort of metadata per machine right now.  The most you can get is the size of a volume.  This is certainly an interesting idea.

@klausenbusk libstorage does not provide that sort of metadata per machine right now. The most you can get is the size of a volume. This is certainly an interesting idea.

The only thing I need from libstorage is a API, which return the max amount of volumes per "storage providers" per machine. K8s has everything else I need, I think. Ex: For DigitalOcean it should just return 5.
I should properly open a new issue in the libstorage repo.

@klausenbusk so you simply need a way to return (for each given driver) the max size for each volume orchestrated by libStorage.  If that is the case, you can do that right now.  Take a look at the API here http://docs.libstorage.apiary.io/.  You can enumerate services, volumes, and get volume sizes.  Hope that helps.

@klausenbusk so you simply need a way to return (for each given driver) the max size for each volume orchestrated by libStorage.

I need a way to get the max attachable volume per instance/machine, that the storage provider support.
AWS for example support 40 volumes per instance, DigitalOcean 5 volumes and GCE 4-16 volumes per instance.

@klausenbusk ah ok.  The API does not return max attachable volumes per instance  for a given service right now. Please open a libstorage issue https://github.com/codedellemc/libstorage requesting that and reference this issue here for context.  Thanks!


Crowding Distance

There is an error in Crowding Distance function.
It should be values1[i-1] - values1[i+i]
and values2[i-1] - values2[i+1]

I agree

I agree.

I think it's not just this error
sorted1 = sort_by_values(front, values1[:])
sorted2 = sort_by_values(front, values2[:])
the obtained two ordinal lists must be opposite, because front is a non-dominated solution, then their distance calculations have been misplaced, and should be changed to:
def crowding_distance(values1, values2, front):     distance = [0 for i in range(0,len(front))]     sorted1 = sort_by_values(front, values1[:])     # sorted2 = sort_by_values(front, values2[:])     distance[0] =<PHONE_NUMBER>444444     distance[len(front) - 1] =<PHONE_NUMBER>444444     for k in range(1,len(front)-1):         distance[k] = distance[k]+ abs(values1[sorted1[k+1]] - values1[sorted1[k-1]])/(max(values1)-min(values1))         distance[k] = distance[k]+ abs(values2[sorted1[k+1]] - values2[sorted1[k-1]])/(max(values2)-min(values2))     return distance

I think it's not just this error
sorted1 = sort_by_values(front, values1[:])
sorted2 = sort_by_values(front, values2[:])
the obtained two ordinal lists must be opposite, because front is a non-dominated solution, then their distance calculations have been misplaced, and should be changed to:
def crowding_distance(values1, values2, front):
distance = [0 for i in range(0,len(front))]
sorted1 = sort_by_values(front, values1[:])
# sorted2 = sort_by_values(front, values2[:])
distance[0] =<PHONE_NUMBER>444444
distance[len(front) - 1] =<PHONE_NUMBER>444444
for k in range(1,len(front)-1):
distance[k] = distance[k]+ abs(values1[sorted1[k+1]] - values1[sorted1[k-1]])/(max(values1)-min(values1))
distance[k] = distance[k]+ abs(values2[sorted1[k+1]] - values2[sorted1[k-1]])/(max(values2)-min(values2))
return distance


RenderStack should handle top, right, bottom, left all being zero

Previously, we didn't set the width of the child correctly when top and bottom
(or left and right) were both specified.
Fixes #275

@eseidelGoogle

Lgtm
On Thursday, July 30, 2015, Adam Barth<EMAIL_ADDRESS>wrote:

@eseidelGoogle https://github.com/eseidelGoogle
—
Reply to this email directly or view it on GitHub
https://github.com/domokit/sky_engine/pull/373#issuecomment-126404517.


Implement a test case for #2828

I've been working on a fix, so I'll close this and re-open as a PR against upstream.


Merge Josh's fixes into ZMQ fix branch.

@tskluzac I see this merge request has been closed. Can the branch be deleted?


Archive/Delete this repo

what

This should be called something other than scaffolding
It should be something like to "monochart" or "charts/universal", "charts/declarative"

why

As scaffolding, this is a very verbose and complex chart

I think we should just archive/delete. Instead do https://github.com/cloudposse/helm-chart-scaffolding/issues/9

@osterman could you archive this repo?

Will do

See: https://github.com/cloudposse/charts/tree/master/incubator/monochart


Allow Sending INFRARED signals

Is your feature request related to a problem? Please describe.
why is it possible to send subghz signal, emulate NFC, RFID, iButton, but not infrared?
Describe the solution you'd like
Allow sending Infrared signals from App -> Archive -> Infrared signal in same way as i would send a subghz signal, or any other supported signal
Also it is possible to send IR signals with ios app but not android....
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
I asked this in Telegram group multiple times, but got a response of "maybeee, soon, why is it needed?"

I agree this would be nice.

We will add this, in the future, it is already planned

still waiting :D

The first thing I noticed when I tried the app remote control was the lack of this feature.
It would be very nice, could use the flipper as an ir blaster.

We will add this, in the future, it is already planned

well, it's been almost a year from me opening the issue, still no feature present..

This was done a long time ago, forgot to close it


Fix typo in Readme.md in batch section

Fix issue https://github.com/Dynamoid/Dynamoid/issues/84

Coverage remained the same at 95.892% when pulling 388a7db62c66f110fdeb11a5387119b718b79484 on andrykonchin:fix-typo-in-batch-example into fe2de35fd421fd84bd36d91786844fc419f5cb2d on Dynamoid:master.


Action Creators

In the change log, it says:

Remove ActionCreator since this can easily be solved with Rx as a single value stream

Can you explain what you mean by that with a very small example?

Without being specific about the library, something like:
let signal: Signal<Action, NetworkError> = Signal.networkRequest("www.fake.com").map {
    AppAction.setFakeData($0)
}
dispatch(signal) // Only works if you've conformed Signal to StreamType

Thanks! I understand now. But I think that a good old fashioned Action Creator could be very useful. Let me try to explain my reasoning:
I've always been annoyed about the lack of distinction between interface actions (i.e., as a result of tapping a button) and data actions, which might result from that interface action. Action creators seem great for this use case: an interface action-creator could spawn one or more data-oriented actions.
I really like how Redux Thunk works, because it means that the interface action is a real action that can be logged, serialised, etc...
How about a similar implementation where:
struct ActionCreator : Action {
    let closure: () -> ()
}

let actionCreatorMiddlewate = Middleware<State>().filter { getState, dispatch, action in
    if let actionCreator = action as? ActionCreator {
        actionCreators.closure()
        return true
    } else {
        return false
    }
}
}

I guess the fact that this can be easily coded with a simple middleware means it does not necessarily belong in the library, but I think it would be great to have a few simple middleware like this, the kind that everybody will use at some point or another.

Ah I see so you'd like a One to Many ActionCreator. I can see the utility of this but since the closure takes no arguments it's not really fitting as something to use in Middleware, you could just call it as a function from wherever you need to select a user.
An option that does fit in with Middleware would be something like:
/// Allows turning one action into many
public func map(_ transform: @escaping (GetState, Action) -> [Action]) -> Middleware<State> {
    return Middleware<State> { getState, dispatch, action in
        self.transform(getState, dispatch, action).flatMap {
            let actions = transform(getState, $0)
            if let action = actions.first, actions.count == 1 {
                return action
            } else {
                actions.forEach(dispatch)
                return nil
            }
        }
    }
}

@agentk I'd like to get your input on both ActionCreator and the idea of adding a map to many.

you could just call it as a function from wherever you need to select a user.

But I would like to have the "interface action" treated as a normal action that goes through the store and middleware stack (even if it does not end up reaching the reducers). If means, for example, that if I have a logging middleware, it gets automatically logged as well.

since the closure takes no arguments it's not really fitting as something to use in Middleware

I think I don't quite understand how Middleware work in ReSwift then. I got a bit confused by your example piece of code. Can you point me to examples of middleware that might help me understand them better?
But coming back to Action Creators, my example was perhaps a bit too simple. The solution the a function returning a list of Actions is that it does not handle async operations. Here is the peace of code from redux-thunk which I'm trying to replicate:
function makeASandwich(forPerson, secretSauce) {
  return {
    type: 'MAKE_SANDWICH',
    forPerson,
    secretSauce
  };
}

function apologize(fromPerson, toPerson, error) {
  return {
    type: 'APOLOGIZE',
    fromPerson,
    toPerson,
    error
  };
}

function withdrawMoney(amount) {
  return {
    type: 'WITHDRAW',
    amount
  };
}

// Even without middleware, you can dispatch an action:
store.dispatch(withdrawMoney(100));

// But what do you do when you need to start an asynchronous action,
// such as an API call, or a router transition?

// Meet thunks.
// A thunk is a function that returns a function.
// This is a thunk.

function makeASandwichWithSecretSauce(forPerson) {

  // Invert control!
  // Return a function that accepts `dispatch` so we can dispatch later.
  // Thunk middleware knows how to turn thunk async actions into actions.

  return function (dispatch) {
    return fetchSecretSauce().then(
      sauce => dispatch(makeASandwich(forPerson, sauce)),
      error => dispatch(apologize('The Sandwich Shop', forPerson, error))
    );
  };
}

// Thunk middleware lets me dispatch thunk async actions
// as if they were actions!

store.dispatch(
  makeASandwichWithSecretSauce('Me')
);

I don't like the direction that adding ActionCreator back to the project would take us in, when it is possible to do something like this:
let person = …
let signal = Signal.fetchSecretSauce().map { sauce in
    makeASandwich(for: person, condiments: [sauce])
}.flatMapError {
    Signal<Action, NoError>(apologize(from: "The Sandwich Shop", for: person, error: $0))
}
store.dispatch(signal)

You can also flatMap streams if you wanted to send multiple Actions on success or failure, like this:
let signal = Signal.networkRequest("www.fake.com").flatMap { user in
    Signal<Action, NoError>(values: LogOut(), SetUser(user), PopRoute())
}

This seems to have been addressed in the design of redux-observable, under the term of Epics
Check out https://youtu.be/AslncyG8whg?t=1370

Fluorine is also very interesting. https://github.com/philpl/fluorine


version 12.1 report 12.0 with -V switch

Minor issue but i stiil want to address it: -V report version 12.0 which should be 12.1

Yes, this was a miss.


document the support for constraints

close #290
close #301

During testing (and confirming user reports), the value and fit parameters have no effect on the forward solutions.  This needs further testing.
Marking this PR as draft for now.

The diffractometer (engine) mode is used to hold an axis at a constant value for a forward(hkl) calculation.  The Constraints value attribute:

Constant value to be used if held constant by forward mode.

This comment describes the meaning of fit, but not in the context of forward() computations.
- Crystal lattice refinement
  - with more than 2 reflections you can select which parameter must
    be fitted.

Here are the places in libhkl source where fit_get appears and this parameter might be used:
(base) prjemian@arf:~/.../picca/hkl$ git grep fit_get
gui/hkl-gui.c:          fit = hkl_parameter_fit_get(p);                         \
gui/hkl-gui.c:          gboolean fit = hkl_parameter_fit_get(p);                \
hkl.h:HKLAPI int hkl_parameter_fit_get(const HklParameter *self) HKL_ARG_NONNULL(1);
hkl/hkl-parameter.c: * hkl_parameter_fit_get:
hkl/hkl-parameter.c:int hkl_parameter_fit_get(const HklParameter *self)
tests/hkl-axis-t.c:     ok(TRUE == hkl_parameter_fit_get(axis), __func__);
tests/hkl-axis-t.c:     ok(TRUE == hkl_parameter_fit_get(axis), __func__);
tests/hkl-axis-t.c:     ok(TRUE == hkl_parameter_fit_get(copy), __func__);
tests/hkl-axis-t.c:     ok(TRUE == hkl_parameter_fit_get(copy), __func__);

Not apparent it is used anywhere in the computation of forward(hkl) solutions.

To avoid making a breaking change, by dropping this parameter altogether, consider these actions now:

deprecate its use
provide warning messages

Confirmed with Fred Picca that the fit parameter is not used when calculating angles from h,k,l:

This interface was developped for the sample parameters a/b/c/alpha/beta/gamma/ux/uy/uz in order to fit the UB martix.
The parameter interface define a fit_get/set and since the axes derived from this interface, the  functions are available, but of no use.

@mrakitin @rodolakis @strempfer @padraic-shafer -- This is ready for review now.


Sanity check on CPU bottleneck with grow_gpu_hist under Windows

Environment info
Hardware:
i7 6900K @ 4.0GHz
4 x GTX 1080
Operating System:
Windows 10 Pro (x64)
Compiler:
cl.exe (Visual Studio Community 2015)
Package used (python/R/jvm/C++):
Python 3.5.2
xgboost version used:
0.6, compiled from source with GPU support, CUB etc. are from submodules.
If installing from source, please provide

The commit hash (git rev-parse HEAD)
91e778c6db3f30f9b5b4ced7752a64881a0f15cb

Steps to reproduce
import xgboost as xgb
import numpy as np
from sklearn.datasets import make_classification

X, y = make_classification(1000000, n_features=50, random_state=7)
param = {'objective': 'binary:logistic', 'updater': 'grow_gpu_hist', 'n_gpus':1}
dtrain = xgb.DMatrix(X, y)
%time model = xgb.train(param, dtrain, 500)

Above training (using Jupyter Notebook's timing magic command, but you can use any other) takes 14.8s to run. While comparable to the 11.1s reported on a Pascal Titan X, the training is bottlenecked by CPU (95% CPU usage). If I set n_gpus to 4, the training finishes in 12.9s (virtually no speed up). As I don't have access to a Linux system with comparable specs right now, I just want a quick sanity check on whether the high CPU usage is a Windows specific issue, or is it WAD.

There is a start-up overhead for the quantisation and tree algorithm set up. How many iterations did you run?

500, as shown in the code. Or if you mean rerun the whole training process: no, following runs aren't any faster.
I ran the same code on a Amazon EC2 p2.xlarge (half of a K80 and 4 vCPU, and it barely got past 150% CPU utilization (equivalence of ~30% on Windows). So I am now fairly sure it wasn't supposed to happen.

I think there is an issue with msvc. I made a note about this previously #2279.
Someone will need to profile it to find the cause. Contribution is very welcome :)

I installed Ubuntu 16.04 on the same machine today and ran the same experiment.

OS/setting
Time (s)
CPU utilization (normalized)

Windows 10, n_gpus=1
14.8s
95%

Windows 10, n_gpus=4
12.9s
95%

Ubuntu 16.04, n_gpus=1
13.2s
20%

Ubuntu 16.04, n_gpus=4
46.5s (what happened here?)
20%

In Ubuntu, can you try adding -DCMAKE_BUILD_TYPE=Release  when calling cmake?  I think nccl is being compiled as debug.  Also, latest HEAD is smarter regarding how it handles sharding on multiple GPUs (but that doesn't explain windows vs. ubuntu behavior).  Thanks.

@DeepLearningJohnDoe Whether more gpus will be beneficial in terms of speed depends highly upon the DMA setup (i.e. which boards have full peer to peer).  You can run tensorflow that shows the DMA map or you can run the cuda samples device inquiry code and it'll tell you the peer to peer ability across all 4 GPUs.
Speed on multiple GPUs also depends upon the data and depth.   Too small data won't be any faster as xgboost is dominated by CPU time and transfer  to/from GPU time.  Too few depth and not enough work to do on GPU.  Too large depth, and lots of communication between GPUs.  Too many features, lots of communication.  etc.

I'll try -DCMAKE_BUILD_TYPE=Release come Monday.
On Windows only a specific set of cards (Everything Tesla, some Quadro, Titans up to Maxwell Titan X) has P2P access and only in TCC mode (can't drive displays); since mine are 1080s they don't have it.

-DCMAKE_BUILD_TYPE=Release does put the wall time on Ubuntu with n_gpus=4 at 8.2s, so I think nccl was being compiled as debug in my previous setting.
Still doesn't explain the Windows performance issue, though.
fwiw here's the output of p2pBandwidthLatencyTest and nvidia-smi topo -m under Ubuntu. Under Windows the non-diagonal terms of the connectivity matrix are zero as P2P can't be enabled.