HDDS-3593. Set a configurable maximum limit of available pipeline

What changes were proposed in this pull request?
Set a configurable maximum limit of available pipeline,
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-3593
How was this patch tested?
A new cluster will not create pipeline more than confg  autoCreateLimit,

@xiaoyuyao  Thanks for review my PR.
ozone.scm.ratis.pipeline.limit is different with ozone.scm.pipeline.creation.auto.limit.

ozone.scm.ratis.pipeline.limit is the total limit, if achieve this limit, no more pipeline would be created.
ozone.scm.pipeline.creation.auto.limit is a limitation for BackgroundPipelineCreator, before BackgroundPipelineCreator want to create a pipeline periodically, it will check if the count of current available pipeline achieve the ozone.scm.pipeline.creation.auto.limit, if achieved already, BackgroundPipelineCreator will skip this loop.

Pipeline limit is better to define along with datanode number. ozone.scm.ratis.pipeline.limit is mainly used in test and please be very careful to set it in distributed cluster since it's static and doesn't consider cluster expansion or shrinkage.
Also talk with baolong offline regarding this change.
@maobaolong @xiaoyuyao

@timmylicheng  @xiaoyuyao Thanks for your review and the suggestion. I change the limit to a rate which consider healthy datanode number. PTAL.

Thanks @timmylicheng and @maobaolong for the clarification. If we really want to push a ratio/factor based on the number of live datanodes, can we move the logic down into RatisPipelineProvider with other limit check? This way, we don't need to introduce NodeManager to BackgroundPipelineCreator.

Thanks @xiaoyuyao for your suggestion, i if put the limit check to RatisPipelineProvider, all the ratis pipeline creation will be limit, my key point is limit the pipeline creation from BackgroundPipelineCreator, and a bigger limit check for other pipeline creation, for example, the putkey rpc call request a unknown pipeline, like 4 replication, if it cannot find the unknown type and replication factor pipeline, it will create pipeline sync, without the limit for BackgroundPipelineCreator, there must be no quota for the unknown type or replication factor pipeline.

Thanks @xiaoyuyao for your suggestion, i if put the limit check to RatisPipelineProvider, all the ratis pipeline creation will be limit, my key point is limit the pipeline creation from BackgroundPipelineCreator, and a bigger limit check for other pipeline creation, for example, the putkey rpc call request a unknown pipeline, like 4 replication, if it cannot find the unknown type and replication factor pipeline, it will create pipeline sync, without the limit for BackgroundPipelineCreator, there must be no quota for the unknown type or replication factor pipeline.

@maobaolong You still have FACTOR to help you classify different behaviors. RatisPipelineProvider is the best place if you would complex logic for different Ratis pipelines. If you just want to skip creation, you can take a look at skipCreation function in BackgroundPipelineCreator. But I would you suggest to have a complete strategy to allocate different factor's pipelines.
At the end of the day, your purpose is to support multiple Ratis pipeline with different factors. This would include preallocation of pipelines, resource split and potential pipeline eviction. It may be more than  just stop creating pipelines to leave quota to other pipelines with different factors. It's hard to rely on configs to know the best setup of pipeline quota. And it's almost impossible to maintain to balance of pipeline quota if you consider cluster expansion or shrinkage.

/pending see @timmylicheng comments


Nimbus complains of eth_subscribe missing method when using websockets

Nimbus encounters this error when trying to use Besu as an EL client.
WRN 2022-05-17 20:30:30.819-04:00 Eth1 chain monitoring failure, restarting  topics="eth1" err="subscribeForBlockHeaders(p.web3, blockHeaderHandler, errorHandler) failed 3 times. Last error: {\"code\":-32604,\"message\":\"Method not enabled\"}"

Changing nimbus to use http instead of ws alleviates this issue.  It is re-enabled in the current kiln-dev-auth branch.


retry on nonce too low

as seen below, when there's contention on the smart contract because of the short block times we are more likely to see this, we just need to retry in the client
2023-07-17 12:19:02,606;ResourceProvider;🟠 post resource offer
Exception in thread Thread-1 (platformListener):
Traceback (most recent call last):
  File "/usr/lib/python3.10/threading.py", line 1016, in _bootstrap_inner
    self.run()
  File "/usr/lib/python3.10/threading.py", line 953, in run
    self._target(*self._args, **self._kwargs)
  File "/app/modicum/ResourceProvider.py", line 303, in platformListener
    self.postDefaultOffer()
  File "/app/modicum/ResourceProvider.py", line 90, in postDefaultOffer
    self.postOffer({"request": "post",
  File "/app/modicum/ResourceProvider.py", line 78, in postOffer
    ).transact({
  File "/usr/local/lib/python3.10/dist-packages/web3/contract/contract.py", line 479, in transact
    return transact_with_contract_function(
  File "/usr/local/lib/python3.10/dist-packages/web3/contract/utils.py", line 172, in transact_with_contract_function
    txn_hash = w3.eth.send_transaction(transact_transaction)
  File "/usr/local/lib/python3.10/dist-packages/web3/eth/eth.py", line 362, in send_transaction
    return self._send_transaction(transaction)
  File "/usr/local/lib/python3.10/dist-packages/web3/module.py", line 68, in caller
    result = w3.manager.request_blocking(
  File "/usr/local/lib/python3.10/dist-packages/web3/manager.py", line 232, in request_blocking
    return self.formatted_response(
  File "/usr/local/lib/python3.10/dist-packages/web3/manager.py", line 205, in formatted_response
    raise ValueError(response["error"])
ValueError: {'code': -32000, 'message': 'nonce too low: next nonce 120, tx nonce 119'}

fixed in f51092ea026832ffca951b40745ce356ea1a0cbc

didn't work. nonce was baked into raw transaction and retrying that is pointless


[TS] Support style on ColumnSet

Automatically generated task. Any updates to the title and body will be overwritten. Do NOT modify the labels.
TS implementation of #2031

Looks like this is already implemented, closing.


Cannot run in backgroup with nohup or systemd

nohup sfz . &

will abort immediately .

Thank for you report. Would release v0.6.0 soon.

v0.6.0 released!
https://github.com/weihanglo/sfz/releases/tag/v0.6.0

thanks


Cron Job Configuration not saving

What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):

Create new cron job
Enter "1" for each of the settings in Job configuration
Save
Click the new Workload

Result:

All say "n/a"

Other details that may be helpful:
Expected result: they should say 1
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): 2.1.1
Installation option (single install/HA):
sibngle install

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported):
Machine type (cloud/VM/metal) and specifications (CPU/memory):
Kubernetes version (use kubectl version):

(paste the output here)

Docker (use docker info):

(paste the output here)

I cannot reproduce this issue in v2.1.1
Is there any step I missed?

Click Deploy
Select Run on a cron schedule
Input 1 for Completions, Parallelism, Back Off Limit and Active Deadline Seconds
Click Save
Go to cronjob detail page to check the value of Completions, Parallelism, Back Off Limit and Active Deadline Seconds

v2.3.6 same behavior, even editing yml, values are not preserved

Got the same issue in 2.3.5. Steps to reproduce:

Create a CronJob without changing anything in Job Configuration section.
Edit the created CronJob and set some fields in Job Configuration (parallelism for example), click Save.
Open the CronJob details or edit again - values set on step 2 are not saved.

If any of Job Configuration values est upon CronJob creation, then they are editable further on.
Also, if parallelism (and perhaps any other relevant field) is changed in yaml (.spec.jobTemplate.spec.parallelism) then the problem goes away.
@loganhz could you test this scenario and reopen this issue?

Still experiencing this in 2.3.8.


Development: Improve client test coverage for exercise header components

Checklist
General

[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Client

[x] I followed the coding and design guidelines.
[x] I added multiple integration tests (Jest) related to the features (with a high test coverage), while following the test guidelines.

Motivation and Context
The components in app/exercises/shared/exercise-headers were not tested before.
Review Progress
Code Review

[x] Review 1
[ ] Review 2

Test Coverage

File
Branch
Statement

difficulty-badge.component.ts
90%
96%

header-exercise-page-with-details.component.ts
100%
100%

header-participation-page.component.ts
100%
100%

included-in-score-badge.component.ts
96.55%
88.88%

Thank you for this PR! Great work


lightningd: SQLITE version mismatch: compiled 3031001, now 3035005

Installed using,
https://github.com/fiatjaf/trustedcoin#extra-how-to-bootstrap-a-lightning-node-from-scratch-without-bitcoin-core-on-ubuntu-amd64
After a server restart, now getting lightningd: SQLITE version mismatch: compiled 3031001, now 3035005
Found this, https://github.com/ElementsProject/lightning/issues/1497
Do I need to recompile trustedcoin_linux_amd64?
Whats the safest way to get it running?

Sorry, I didn't see this a million years ago when it was created.
But that is not related to trustedcoin.


Reviewed the readme and made updates

Updated a few things in the README.

some links were old, I found a way to change the openshift links to
track the latest release
tried to state where OpenShift is required
other fixes that may help users understand what the policies do

Refs:

https://github.com/open-cluster-management/backlog/issues/17105

Signed-off-by: Gus Parvin<EMAIL_ADDRESS>

@dockerymick Thanks for reviewing.  I think I've made all of the requested updates.  Please check for any more issues when you get a chance.  Thanks again!


feat: add more deprovisioning logs

Description
Adds more visibility regarding consolidation decisions/
How was this change tested?
Deployed to EKS.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Sample of New logs

karpenter-764d9bf778-jfgtd controller 2022-12-06T17:56:57.849Z	INFO	controller.deprovisioning	node ip-192-168-134-177.us-west-2.compute.internal can't be consolidated, can't replace with a cheaper node	{"commit": "5d20fe2"}

karpenter-764d9bf778-jfgtd controller 2022-12-06T17:59:19.151Z	INFO	controller.deprovisioning	node ip-192-168-134-177.us-west-2.compute.internal can't be consolidated, pdb default/inflate-pdb prevents pod evictions	{"commit": "5d20fe2"}

karpenter-ff5c78494-mkqdg controller 2022-12-06T18:40:01.118Z	INFO	controller.provisioner	pod default/inflate-anti-self-55894c5d8b-522jd has a preferred Anti-Affinity which can prevent consolidation	{"commit": "5d20fe2"}
karpenter-ff5c78494-mkqdg controller 2022-12-06T18:40:01.118Z	INFO	controller.provisioner	pod default/inflate-anti-self-55894c5d8b-wtkmt has a preferred Anti-Affinity which can prevent consolidation	{"commit": "5d20fe2"}

Sample of new Events
Events:
  Type     Reason                   Age                From             Message
  ----     ------                   ----               ----             -------
  Normal   Unconsolidatable         66s                karpenter        pdb default/inflate-pdb prevents pod evictions

Merge conflicts resolved.

@ellistarn I was debating on removing the logs from this and just leaving the events, WDYT?

@ellistarn I was debating on removing the logs from this and just leaving the events, WDYT?

I'd really love an abstraction = event+log for major things that we're doing. I've heard customers asking for events recently.

@ellistarn I was debating on removing the logs from this and just leaving the events, WDYT?

I'd really love an abstraction = event+log for major things that we're doing. I've heard customers asking for events recently.

The reason I had second thoughts here are these are things we are not doing for the most part (e.g. not consolidating node XYZ due to a PDB).  They'll repeat every hour, which is fine for an event, but I thought it might be too noisy for logs.

I wonder if it gives the wrong impression if we only do it every hour, since we're checking much more frequently. Definitely agree that we don't want to spam on this.

Removed the logging and changed time so we get events every 15 minutes.  Process should now be "Wonder why that node isn't consolidated ->  kubectl describe node node-name" and observe events indicating why.


Create MANIFEST.in

Add include_package_data=True to setup.py

Bringing over changes that Yushi had made


REQUEST - Any plans for Kali on Crouton to be Intel friendly?

I love every single thing about Crouton. It's fabulous and makes me like Linux again -- notably finding the cli bonus add-on makes me really, really happy.
I did have one small request --
I've tried to install Kali via Crouton for my Chromebook Pixel 2, but unfortunately, I think Kali at this point is AMD-only. Any chance of rerolling this to be intel-friendly as well? I'd do it myself, but I'm not particularly helpful when it comes to programming and I'd probably break things.
Danke!
--Carey

I don't think crouton kali is intel-unfriendly. I got a crouton kali install on my Acer C710 with Intel Celleron running for 3 months now. I don't use it very often, but when I do, it seems to run fine.
If there are no major changes in the last 3 months I don't see any reason why crouton kali is intel-unfriendly.... or did I miss something?

I suspect you're misinterpreting "amd64".  Kali works fine on all platforms.

Got it. It managed to work on the second go. Sorry I didn't close it.
On Sat, Aug 1, 2015, 21:15 David Schneider<EMAIL_ADDRESS>wrote:

I suspect you're misinterpreting "amd64
https://en.wikipedia.org/wiki/X86-64". Kali works fine on all platforms.
—
Reply to this email directly or view it on GitHub
https://github.com/dnschneid/crouton/issues/1886#issuecomment-126949743.


chore(deps): update ci dependencies

This PR contains the following updates:

Package
Type
Update
Change

common-dev-assets

digest
f2e1cf2 -> f0e8ba6

terraform-ibm-modules/common-pipeline-assets
action
minor
v1.20.0 -> v1.21.0

Release Notes

terraform-ibm-modules/common-pipeline-assets (terraform-ibm-modules/common-pipeline-assets)
v1.21.0
Compare Source
Features

Release pipeline now creates GIT issue and slack alert on failure (#636) (3a51e77)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

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


Genuinely unplayable on steam deck

I don't know if it's because I have other mods active, didnt extract the mods correctly, but here's a list of issues I found
Pausing freezes the game all the time.
Switch to another style has the chance to softlock enemies, and I can't attack either
Sometimes when I go into battle the enemies re frozen, and I can't attack.

Note: The solution is to actually install with Ryu Mod Manager, place the New Mods and Heat Actions into the mods folder, and the mod manager should handle everything from there.
I have had a few annoying experiences with RMM before, so I didn't think that would work.


Add test location provider

Closes https://github.com/microsoft/vscode-python-internalbacklog/issues/238
Adds a test location provider that uses the old test location script. After the Test UI work, I will do a follow up PR where it will use Language Server when available, and use the script only when language server is not available.
@kimadeline I also cleaned up the .eslintignore in this PR.

@kimadeline @karrtikr Can i get a review on this? This is currently blocking my next PR.


[SPARK-9385] [PYSPARK] Enable PEP8 but disable installing pylint.

Instead of disabling all python style check, we should enable PEP8. So, this PR just comments out the part installing pylint.

Merged build triggered.

Merged build started.

Test build #38596 has started for   PR 7704 at commit 0056359.

Alright, I saw
========================================================================
Running Python style checks
========================================================================
PEP8 checks passed.

in the jenkins output. I am merging it to master.

Test build #38596 has finished for   PR 7704 at commit 0056359.

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

Merged build finished. Test FAILed.


proposal: Function evaluation in if with variable assignment should be done after checking the conditions

Consider the following code:
func main() {
    if err := foo(); false && err == nil {
      // ...
    }
}

func foo() error {
    return nil
}

Here, foo should not be evaluated because condition will result in false irrespective of the value of err. This applies to similar case with condition:
err := foo(); true || err == nil

Here the condition will be evaluated to true in all the cases, so function evaluation is not required at all.
I tried to achieve with different syntax like this:
func main() {
    if false && (err := foo(); err == nil) {
      // ...
    }
}

But it's not allowed in Go. I don't think there is even a way to achieve this currently.
Also, this behaviour will be inline with existing behaviour of if statement without variable assignment. Consider the following snippet:
func main() {
    if false && foo() {
        // ...
    }
}

func foo() bool {
    return true
}

Here, because of false as an initial condition, the foo won't be evaluated at all.
This might save a few lines of code and time to evaluate the function if the other condition(s) can determine the result of the combined condition.
Let me know if this examples/explanation is confusing or I am supposed to provide more details. Thanks!

This change is not backwards compatible, so I don't think we can do it as proposed.
It's easy enough to do currently. Your first example:
if false {
    if err := foo(); err == nil {
    }
}

It just requires more than one if statement.


KRKPD-572: wrap in if statement

Type of change

[ ] Bugfix

[ ] Feature

[ ] New bidder adapter

[ ] Code style update (formatting, local variables)

[x] Refactoring (no functional changes, no api changes)

[ ] Build related changes

[ ] CI related changes

[ ] Does this change affect user-facing APIs or examples documented on http://prebid.org?

[ ] Other

Description of change

Other information

Test passing without any spec changes.


Generate CSV files from a Google Docs spreadsheet

Is there any tool or guidance on creating a set of CSV files from a GDocs spreadsheet such as the suggested template?
https://docs.google.com/spreadsheets/d/1qkysLj5MAAvaysuac_9H0hOvTAb1aJEMhUVJAECmj2c/edit#gid=595985385

I've updated the Wiki article with more detailed instructions. Closing this now. Thanks for this contribution!
Here's the updated article: https://github.com/codeforamerica/ohana-api/wiki/Automatically-generate-CSV-files-from-an-OpenReferral-Google-spreadsheet


Fix #1117 and clarify about UI refresh.

Addresses #1117. Also contains clarifies that the Apps/Explorer might need a refresh to before upgrades show/are usable, as I got quite confused about seeing version 102 of the blockchain, but still getting 500 pUnits as ExistentialDeposit, when I should have gotten 100. Refreshing fixed that.

Thanks for the contribution @tifrel . I added some minor edits to the write-up section. Will leave this PR open and merge once I update the solution, which is now out-dated and may confuse folks, as you mentioned 😃. Cheers!

@sacha-l do you have a new solution in play? If not, I added a note to warn, and for now this should be good to go, followed by #1123 (that will need to rebase on this)

@sacha-l do you have a new solution in play? If not, I added a note to warn, and for now this should be good to go, followed by #1123 (that will need to rebase on this)

All updated now - better have the solution with the latest node template available :)


Consider Switching from Highcharts to D3.js

To avoid possible licensing issues in the case of a "pro" or "hosted" version in the future.

Closing this, as I moved to Chartist.js with things starting to finish up around a78e92a436ce78eca71a345b5d49396b3fa72766.


add postrenderers patch prevent etcd-renaming and use local chart with secretStringData

Prefixing of etcd PVCs
This change adds JSON-Patches to reset the renaming of the breaking change by YAKE 1.86.
The prefix "virtual-garden-" will be removed from the PVCs.
This change can be dropped, if we changed the names of our PVCs (quiet long downtime) or we deploy etcd differently.
Please refer to our documentation: https://hedgedoc.ske.eu01.stackit.cloud/W4zDhnvqQDqnfYrkQ04BSQ
Breaking change on etcd-backup-restore v0.19
Additionally we needed a workaround to pass the env vars to our etcd backup. Due to the breaking change of etcd-backup-restore v0.19, we had to find a way to pass the credentials to the helm chart without using our way via ENV vars. We introduced the secretStringData as a method to pass it via in ske-base to this helm chart. Additionally, we moved the upstream chart to the local chart, because it isn't used in any way.
This change can be dropped, whenever it is fixed upstream (by us).

/lgtm
please also update the PR description

/approve


Update RunCommand operation output as valid json object

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[ ] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[ ] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[ ] If applicable, the PR references the bug/issue that it fixes.
[ ] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[ ] I have read the contribution guidelines.
[ ] My spec meets the review criteria:

[ ] The spec conforms to the Swagger 2.0 specification.
[ ] The spec follows the guidelines described in the Swagger checklist document.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Automation for azure-sdk-for-python

Automation for azure-sdk-for-go

This is to address incidents reported the PowerShell run command returns empty result. This is due to the fact that operation output was reported as array instead of valid application/json object.  CRP changes are deployed and now the operation output is {value:[...]} instead of just array [...]
The fix is from api version 2018-04-01

Automation for azure-sdk-for-node

Automation for azure-sdk-for-java

Automation for azure-sdk-for-ruby

@AutorestCI regenerate azure-sdk-for-python

Encountered an unknown error
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/dist-packages/azure_devtools/ci_tools/github_tools.py", line 32, in exception_to_github
    yield context
  File "/usr/local/lib/python3.6/dist-packages/azure_devtools/ci_tools/bot_framework.py", line 105, in manage_comment
    response = getattr(self.handler, orderstr)(webhook_data.issue, *split_text)
  File "/usr/local/lib/python3.6/dist-packages/swaggertosdk/restapi/restbot.py", line 54, in regenerate
    return self.rebuild(issue, repotag)
  File "/usr/local/lib/python3.6/dist-packages/swaggertosdk/restapi/restbot.py", line 36, in rebuild
    self.sdk_default_base
  File "/usr/local/lib/python3.6/dist-packages/swaggertosdk/restapi/github_handler.py", line 151, in rest_pr_management
    sdk_tag=sdk_tag
  File "/usr/local/lib/python3.6/dist-packages/swaggertosdk/SwaggerToSdkNewCLI.py", line 243, in generate_sdk_from_git_object
    with manage_git_folder(gh_token, Path(temp_dir) / Path("rest"), branched_rest_api_id) as restapi_git_folder, \
  File "/usr/lib/python3.6/contextlib.py", line 81, in __enter__
    return next(self.gen)
  File "/usr/local/lib/python3.6/dist-packages/azure_devtools/ci_tools/github_tools.py", line 267, in manage_git_folder
    clone_to_path(gh_token, temp_dir, split_git_id[0], branch_or_commit=branch, pr_number=pr_number)
  File "/usr/local/lib/python3.6/dist-packages/azure_devtools/ci_tools/github_tools.py", line 198, in clone_to_path
    _git_clone_to_path(https_authenticated_url, folder, branch_or_commit)
  File "/usr/local/lib/python3.6/dist-packages/azure_devtools/ci_tools/git_tools.py", line 81, in clone_to_path
    repo.git.checkout(branch_or_commit)
  File "/usr/local/lib/python3.6/dist-packages/git/cmd.py", line 548, in <lambda>
    return lambda *args, **kwargs: self._call_process(name, *args, **kwargs)
  File "/usr/local/lib/python3.6/dist-packages/git/cmd.py", line 1014, in _call_process
    return self.execute(call, **exec_kwargs)
  File "/usr/local/lib/python3.6/dist-packages/git/cmd.py", line 825, in execute
    raise GitCommandError(command, status, stderr_value, stdout_value)
git.exc.GitCommandError: Cmd('git') failed due to: exit code(128)
  cmdline: git checkout f88c260d04c6fad2d9080322d3164f621632a6d4
  stderr: 'fatal: reference is not a tree: f88c260d04c6fad2d9080322d3164f621632a6d4'

@AutorestCI regenerate azure-sdk-for-python

Not related to this PR, but the link here is invalid - https://github.com/koralski/azure-rest-api-specs/blob/e3d9ca01eb41c555b03705a38e58173de2196194/specification/compute/resource-manager/Microsoft.Compute/stable/2018-04-01/examples/VirtualMachineRunCommandGet.json#L30.
Thought you might want to take a look.


Add ability to tag cells in Jupyter notebook

Discussed in https://github.com/microsoft/vscode-python/discussions/16598

Originally posted by scottsmith1474 July  1, 2021
Love the new jupyter notebook interface, would love to be able to tag cells like you can in jupyter lab. Table of contents would seal the deal as well.
Thanks!

Hi @scottsmith1474, you can find the table of contents in the File Explorer side bar in the Outline tab

@scotthsmith1474 If you'd like to show support for the other request you have raised, I believe it is this one here: https://github.com/microsoft/vscode/issues/103519


Fix for firmware checkout issue on travis

This is stopping some of the travis builds to fail.

The arduous pull request (in the end) fixes up Travis to make a fully-working test-before merge system for us.


New Rule: create vscode plugins

Rule details
vscode plugins
Related ECMAScript feature
No response
What type of rule is this?
Other feature
Which codebase the rule should be implemented?

Participation

[ ] I am willing to submit a pull request to implement this rule.

Additional comments
No response

Backlog of release v3.0.0

free to take

In progress by @nemmip


Please consider fixing issues detected by repository checker

Notification from ioBroker Check and Service Bot
Dear adapter developer,
I'm the ioBroker Check and Service Bot. I'm an automated tool processing routine tasks for the ioBroker infrastructure. I have recently checked the repository for your adapter gree-hvac for common errors and appropiate suggestions to keep this adapter up to date.
Please see the result of the check below.
ioBroker.gree-hvac
   -

:thumbsup: No errors found
WARNINGS:

[ ] :eyes: [W130] Many "common.news" found in io-package.json. Repositorybuilder will truncate at 7 news. Please remove old news.

Please review issues reported and consider fixing them as soon as appropiate.
Warnings reported by repository checker should be reviewed. While some warnings can be considered as a suggestion and be ignored due to good reasons or a dedicated decision of the developer, most warnings should be fixed as soon as appropiate.
Feel free to contact me (@iobroker-bot) if you have any questions or feel that an issue is incorrectly flagged.
And THANKS A LOT for maintaining this adapter from me and all users.
Let's work together for the best user experience.
your
ioBroker Check and Service Bot
@mcm1957 for evidence

Fixed

Thanks for processing and closing this issue.
I will recheck for new issues later. So it's no problem if this issue has been closed premature.
You may start a new check at any time by adding the following comment to this issue:
@iobroker-bot recheck
Please note that I (and the server at GitHub) have always plenty of work to do. So it may last up to 30 minutes until you see a reaction. I will drop a comment here as soon as I start processing.
Feel free to contact me (@iobroker-bot) if you have any questions or feel that an issue is incorrectly flagged.  Let's work together for the best user experience.
your
ioBroker Check and Service Bot


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-yml-archive-non-versioning-archive/pull/380 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-yml-archive-non-versioning-archive.git

Docs Build status updates of commit 480ab04:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

docs/TOC.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-3.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-3.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


convolve_boundary_option renamed to boundary_option

Description
boundary_option moved to numeric/algorithm.hpp

References
closes #387

Tasklist

[x] Ensure all CI builds pass
[ ] Review and approve

I will extend support for new boundary_options for extend_boundary  in a separate PR


Change document.querySelectorAll(query)[0] to querySelectorquery

Update document.querySelectorAll element selectors to document.querySelector
Resolves #50

Other parts of the code will need to be updated where these are called or there isn't too much point I'm afraid


Typo: "oher"

In the first “Syllable structure” section (why are there two?) “oher” should be “other”.

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

ID: afaf5060-fd0d-2b4c-7555-e3d81ccafe24
Version Independent ID: 06c7f355-4cd9-4d0c-188b-4d65d4a5b406
Content: Creating and Supporting OpenType Fonts for Tibetan Script - Typography
Content Source: typographydocs/script-development/tibetan.md
Product: windows
GitHub Login: @mijacobs
Microsoft Alias: mijacobs

Closing as Fixed.


testutils.py failure

While executing BLISS test suite, which partly rely on silx test suite for Flint, it happened that
_outcome was None:
self = <test_image_plot.TestImagePlot testMethod=test_display_image>

    def _currentTestSucceeded(self):
        if hasattr(self, '_outcome'):
            # For Python >= 3.4
            result = self.defaultTestResult()  # these 2 methods have no side effects
>           self._feedErrorsToResult(result, self._outcome.errors)
E           AttributeError: 'NoneType' object has no attribute 'errors'

../../miniconda3/envs/bliss/lib/python3.7/site-packages/silx/gui/utils/testutils.py:158: AttributeError

(maybe more for @vallsv ?)

You think it is about Python version incompatibility?

Yes obviously

It looks like it can happen when all the tests was skipped.


If an nonexistent input is accessed in an output, no error is printed

It simply stops execution of the output code without printing an error. For example:
from shiny import App, render, ui

app_ui = ui.page_fluid(
    ui.input_slider("n", "N", 0, 100, 20),
    ui.output_text_verbatim("txt"),
)

def server(input, output, session):
    @output
    @render.text
    def txt():
        return f"n*2 is {input.foo() * 2}"

app = App(app_ui, server)

Live example

One problem this causes in practice is that it can be hard to diagnose when the code accidentally accesses a nonexistent input value -- because it's a silent exception, nothing happens, not even a warning or error message.

Should this return MISSING_TYPE instead? Then an error should be thrown when MISSING_TYPE * 2 is attempted

A couple of ideas that came up in a discussion today.
From @schloerke:
If input.n isn't set, it could either:

Throw an exception
Return None

One issue with both of these approaches is that it may show a flash of red text briefly in the browser, for cases where the input does not exist right away, like with dynamic UI. Currently the app below does not have a red flash, but with both of the approaches above, it would:
from shiny.express import input, render, ui

@render.ui
def foo():
    return ui.input_slider("n", "N", 0, 100, 20)

@render.text
def txt():
    return f"n*2 is {input.n() * 2}"

Another idea, from @cpsievert:
It could continue to raise a silent exception, but a new kind of silent exception where it is logged to the console but not shown in the browser. That way the developer would still be aware of the issue, but the end user would never see a red flash. We could also add more detail to error message telling the developer how to make it go away.

Yeah, this one has been annoying me too, in addition to the nonexistent issue, the same kind of thing happens if there's an error while doing some reactive calculation.
If the issue is just that the input isn't set, I think returning None would be fine, the user can handle that as they wish, it's not unusual to check a value's existence in other types of program.
If a reactive calculation (reactive.calc, setting a reactive.value) fails for any reason including not handling an input being None I think it should throw the exception.


[test] Update WCOW uVM and vSMB, and HostProcess functional tests

Update and un-skip WCOW uVM and container tests as well as WCOW vSMB tests.
Add:

WCOW uVM benchmarks
LCOW boot file tests that verify kernel boot args and dmesg logs
WCOW Host Process tests, including dedicated tests for setting username, and verifying hostname and volume mounts.

Unified container and uVM lifecycle, exec, and IO tests for LCOW and WCOW.
Moved tests from:

lcow_bench_test.go to uvm_bench_test.go
lcow_container_test.go to container_test.go
lcow_test.go to lcow_uvm_test.go and uvm_test.go

Fix bug where removing a direct-mapped vSMB share failes.
Run (non-virtualization/uVM) functional tests within CI.
Starting Host Process containers requires SYSTEM to create a process with a specified token, so use PsExec.exe (from sysutils) to run tests.
PR is split into three commits: the first does preparatory reorg work, specifically for LCOW-only tests that are generalized to work on both LCOW and WCOW in the last subsequent commit.

Seems like there are a bunch of changes combined in this PR, would it be possible to split it into multiple commits so that it is easier to review?

rebased to resolve merge conflicts (update test/go.[mod|sum])


Change pancake recipe

To not use crafting table

This has been done.

The current recipe is still a crafting table recipe just done in the mixer it seems. I think a better solution would be an assembly recipe with dough

Any recipe that involves a phisical bucket should probably be converted to a pouring or assembly recipe


Applying viewShadowColor property ignores borderRadius

When setting viewShadowColor property to ImageView it ignores borderRadius property.

The problem, I think, is related to TiUIView. Try to use the same properties on the native Ti.UI.ImageView and you will see the same behavior.


[SECURITY] Prevent rhostname array overflow

This is an automatically generated security fix for a vulnerability detected in your code which is a variant of CVE-2020-8597.
You can read in more detail about this vulnerability in CERT Advisory VU#782301.
The vulnerability occurs because, given that vallen was checked to be less than len, it can never be the case that vallen >= len + sizeof(rhostname). Therefore, rhostname never gets trimmed and the rhostname array may overflow.
While this PR was generated for your project automatically, the supporting analysis was performed and verified by the GitHub Security Lab.
The original finding was reported by Ilja Van Sprundel from IOActive.
The proposed patch was developed by Paul Mackerras (paulusmack) from the Samba project in
paulusmack/ppp@8d7970b#diff-b7f5f2404cf3f5c09b1f8ad9364bb340.
The original vulnerability got assigned CVE-2020-8597 which has a CVSS v3.1 Base Score of 9.8/10.
Source
The source code that generated and submitted this PR is based on JLLeitschuh/bulk-security-pr-generator.
Opting-Out
This bot will respect the ROBOTS.txt format. If you'd like to opt-out of any future automated security vulnerability fixes like this, please consider adding a file called
.github/GH-ROBOTS.txt to your repository with the line:
'''
User-agent: GSL/bulk-security-pr-generator
Disallow: *
'''
Alternatively, if this project is no longer actively maintained, consider archiving the repository.
CLA Requirements
This section is only relevant if your project requires contributors to sign a Contributor License Agreement (CLA) for external contributions.
It is unlikely that we will be able to directly sign CLAs. However, all contributed commits are already automatically signed-off.

The meaning of a signoff depends on the project, but it typically certifies that committer has the rights to submit this work under the same license and agrees to a Developer Certificate of Origin
(see https://developercertificate.org/ for more information).
- Git Commit Signoff documentation

If signing your organization's CLA is a strict-requirement for merging this contribution, please feel free to close this PR.

Codecov Report

Merging #516 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #516      +/-   ##
==========================================
- Coverage   54.16%   54.16%   -0.01%
==========================================
  Files         329      329
  Lines       57161    57161
==========================================
- Hits        30964    30962       -2
- Misses      26197    26199       +2

Impacted Files
Coverage Δ

src/lib/profiles/security/WeaveCert.cpp
88.5% <0%> (-0.46%)
:arrow_down:

Continue to review full report at Codecov.

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

This has already been fixed identically in lwip upstream (see http://git.savannah.nongnu.org/cgit/lwip.git/tree/src/netif/ppp/eap.c#n1426). However, OpenWeave doesn't use PPP or EAP in LwIP, so this is a no-op for OpenWeave one way or another.


Add StrykersAerospaceandIVAs

Closes #2654

Looks good, merging with my thanks for this PR and all the others today.


Don't hardcode console path

Console path is hardcoded in the EntityConfigBundle. Since Sensio Distribution 4.0 the Symfony 3 directory structure is available (even in Symfony <3.0). However this hardcoding makes it a bit harder to use bin/console. I would suggest introducing a parameter which can be overriden in the app config.
Also, I am not sure if EntityConfigBundle is the right place for such a service (command executor). Wouldn't DistributionBundle be better?

Hi @sagikazarmark
Thanks for pointing this issue, we will investigate what we can do with the console path and how to join both command executors: from EntityConfigBundle and from InstallerBundle.

Hi there,
The new release just introduced a third hardcode way to run commands:
https://github.com/orocrm/platform/blame/master/src/Oro/Bundle/EntityExtendBundle/OroEntityExtendBundle.php#L107


Discussion on Community Management

Description
A divergent discussion in #18931 surfaced insufficient support in the discuss forums which is an important issue.  @StevenJokes and @qqaatw brought up a valid point that it's vital to the community.
Here I'd like to tap on that discussion and explore how our community can better help new users engage by making sure their inquiries are heard. Topics in this discussion include: what channels to use for different types of discussion, how to surface unanswered questions more easily, how to build mechanisms to have users help each other out.

My 0.02, it will be better to have a unified place to discuss all MXNet related issues. There are multiple forums but I don't think it's not active, including dev@, JIRA.
I think GitHub will be a good choice for the interface of MXNet and their user/developers.

I don't think GitHub will be the only one place.
Please look the mxnet issues' numbers. Too many.
It's hard to search directly in google too.
Keep only bugs to report in github.

We need a discussion that it's easy to search and solve some specific problems.

Frequent Asked Questions is a good try.
BUT!!

404

--
And you should list
<EMAIL_ADDRESS>in README.md.
There is no wonder that no one reply to you.
I don't know, and I believe thatmany people don't know the place.

@szha
Why do you tell them to ask in discuss.mxnet.io?
But no reply to them these months.
Being open is not in words, but in action.
Being friendly is not be "nice guy" and polite, but treat their question carefully.
Being a leader is not "open some topics" to let them just make some questions no one will reply , but answer their questions , join their discussions to understand what we need to do next , and invite them to create more interesting things together.

I'm clarifying the communication channel usage in #18992


GnuPlot without PDF terminal

When support for PDF terminal is missing in GnuPlot, a PDF with an error message should be created (currently an exception appears). The file should contain a procedure, how to correctly recompile GnuPlot using SMBL or how to replace it with binaries downloaded from the official website.

Create FAQ answer for this issue

Split plotting into more GP scripts:

Plotting PDF
Plotting PNG

When error appears during plotting PDF, replace final PDF by an "error PDF file" with description how to solve it. This file should be part of the RNFtools Python package.


Shebang first

This PR moves the Shebang strategy above Filename:
STRATEGIES = [
  Linguist::Strategy::Modeline,
  Linguist::Shebang,
  Linguist::Strategy::Filename,
  Linguist::Heuristics,
  Linguist::Classifier
]

IMO Modeline should still be before Shebang as it's an explicit statement of content by the user. This PR fixes https://github.com/github/linguist/issues/2081 and https://github.com/github/linguist/issues/1991
@pchaigno said in https://github.com/github/linguist/issues/1991:

It would require a bit of work since currently the first strategy is also generating the list of candidates but it could surely help solve this type of issue.

I don't believe this is an issue as neither the Modeline or Shebang strategies rely upon 'incoming' languages like the Heuristics does. As the Filename strategy still precedes the Heuristics we should be OK.
/ cc @pchaigno @bkeepers

Also, https://github.com/github/linguist/commit/88e79cd3a8bc7749783693a35ea4409a4709303a adds some fixtures that failed previously to this.

I don't believe this is an issue as neither the Modeline or Shebang strategies rely upon 'incoming' languages like the Heuristics does. As the Filename strategy still precedes the Heuristics we should be OK.

I forgot we don't need candidates for the Shebang strategy when I wrote that.
:+1: to this PR!

This is now live on github.com.


[Bug] Onboard wallet modal ios chrome when trying to connect

Current Behavior
when i connect wallet in IOS 17 using chrome.

Expected Behavior
No response
Steps To Reproduce
No response
What package is effected by this issue?
@web3-onboard/core
Is this a build or a runtime issue?
Build
Package Version
2.21.0
Node Version
16.16.0
What browsers are you seeing the problem on?
Chrome
Relevant log output
No response
Anything else?
No response
Sanity Check

[X] If this is a build issue, I have included my build config. If this is a runtime issue, I have included reproduction steps and/or a Minimal, Reproducible Example.

Thanks for the report. We'll take a look in to this.


[Key Vault Administration] Add support for service version 7.4-preview.1 and settings APIs

[x] Add support for preview service version and new settings APIs.
[ ] Add tests and recordings for KeyVaultSettingsClient and KeyVaultSettingsAsyncClient.
[ ] Re-record all tests for the Key Vault Administration package for 7.4-preview.1.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-security-keyvault-administration


fix typos and wording

bors try

bors try-

bors r+

bors r+

bors r+

bors r+
please work...


Add support for zabbix sender

Please add support for zabbix sender since it usually comes in a different package (both under Debian and RedHat)

As PR is merged, I close this issue.


Update open-policy-agent/opa to 0.8.2

Current version in release-1.0 contains a transitive dependency on an old github.com/dchest/siphash which doesn't have a license. The new version of siphash is needed for updated license.

Merged


Playground : Adding a wrapper div to support alignment (left, center, right) for Youtube Video. Adding exportDom function to the FigmaNode.

PLayground : I added exportDom to FigmaNode, and modified little bit the existing exportDom in Youtube Node to support alignment (left, center, right), because we could align it in the editor, but it was not reflected in the HTML output.

Can you add this to the DecoratorBlockNode instead which is the node which wraps these nodes? Something like this on that node should work:
 exportDOM(): DOMExportOutput {
     const element = document.createElement('div');
     element.style.textAlign = this.__format;
     return {element};
 }

Closing this for the moment due to inactivity.


Failure in connecting to ecovacs

Hi all,
I can't seem to get this one to work my config is as follows
    {
        "email"<EMAIL_ADDRESS>        "password": "xxxxx",
        "countryCode": "AU",
        "refreshTimer": 0,
        "publishBipSwitch": false,
        "publishSwitch": false,
        "publishFan": false,
        "publishMotionDetector": true,
        "publishAutoSwitch": true,
        "publishEdgeSwitch": false,
        "publishSpotSwitch": false,
        "publishAreaSwitchesAsSeparateDevices": false,
        "showInfoLogs": false,
        "platform": "DeebotEcovacs"
    }
],

The error I am getting is

I have found that the AU region doesn't work... Changed to FR and it is now online!


Command 'meteor npm install ionic-scripts --save' gets this error

Hello, I am following the tutotial of Whatssap Project with Meteor and Ionic(http://www.angular-meteor.com/tutorials/whatsapp/meteor/bootstrapping), but at the moment when I run the command 'meteor npm install ionic-scripts --save' I get this bug:
+-- UNMET PEER DEPENDENCY<EMAIL_ADDRESS>+-- UNMET PEER DEPENDENCY<EMAIL_ADDRESS>+-- UNMET PEER DEPENDENCY<EMAIL_ADDRESS>+-- UNMET PEER DEPENDENCY<EMAIL_ADDRESS>`--<EMAIL_ADDRESS>I don´t know if someone can help me, I will really appreciate it.
Thanks

@juliocesar12 'Peer Dependencies' are dependencies whose existence are crucial otherwise the dependent package won't work, and they need to be loaded directly from your project!  To solve the issue, simply run the following:
$ npm install angular
$ npm install angular-animate
$ npm install angular-sanitize
$ npm install angular-ui-router

I'm pretty certain that everything will work fine now, but just to make sure just give me your approval so I can close this issue :-)

I will also update the tutorial as well, in the previous versions of npm peer dependencies were installed automatically but now this feature is no more.


Added prop to remove links in date numbers in month view

Fixes this: https://github.com/intljusticemission/react-big-calendar/issues/125
Not sure how the best prop/argument schema should be, so I'm happy to take suggestions.

I'll go ahead and close this since it's been a while. Thanks for doing this though @manutenfruits! And if you want to update the PR to use a component or render prop instead of a boolean, just re-open! :)

I am also facing same issue using react big calendar after rendering calendar each day number rendering as anchor tag i need to convert to button tag please let me know is there is props or any custom way to add button on each date on calendar.


Implement push diagnostics in terms of DocumentDiagnosticsRequest

Instead of listening for document updates sent from sourcekitd and sending a PublishDiagnosticsNotification based on the sourcekitd notification, wait for a little while and then execute an internal DocumentDiagnosticsRequest to load diagnostics and send them to the client.
This has two advantages:

It unifies the two diagnostic implementations
It removes the need to keep track of semantic diagnostics in SwiftLanguageServer
It gets us one step closed to opening and editing documents in syntactic_only mode. The only thing that is left now are semantic tokens.

@swift-ci Please test

@swift-ci Please test Linux

@swift-ci Please test Windows

@swift-ci Please test

@swift-ci Please test

@swift-ci Please test Windows


refactor(accordion): Fix data-testId on main files

Summary
Structural Element (Base, Gene, DNA, Chromosome or Cell)
Shortcut ticket: sh-224676
Right now, data-testid is sometimes included in the index file of components and is then always present whenever that component is used. It should only be used in the test file. We need to remove it from index files and add it to test files where needed.
Checklist

[ ] Default Story in Storybook
[ ] LivePreview Story in Storybook
[ ] Test Story in Storybook
[ ] Tests written
[ ] Variables from defaultTheme.ts used wherever possible
[ ] If updating an existing component, depreciate flag has been used where necessary
[ ] Chromatic build verified by @chanzuckerberg/sds-design

looks great! thank you masoud. I wonder if we could add this as a lint rule, like data-testid is only allowed in certain file types

Hmm, that sounds like a good idea. I'll look into it :D
Thank you @bethbertozzi 👍🏻


import dask_ml.cluster raises a ModuleNotFoundError from sklearn

$ pip install dask-ml
$ python
>>> import dask_ml.cluster
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/peter/.virtualenvs/DASK/lib/python3.7/site-packages/dask_ml/cluster/__init__.py", line 3, in <module>
    from .k_means import KMeans  # noqa
  File "/Users/peter/.virtualenvs/DASK/lib/python3.7/site-packages/dask_ml/cluster/k_means.py", line 23, in <module>
    from ._compat import _k_init
  File "/Users/peter/.virtualenvs/DASK/lib/python3.7/site-packages/dask_ml/cluster/_compat.py", line 4, in <module>
    from sklearn.cluster._k_means import _k_init
ModuleNotFoundError: No module named 'sklearn.cluster._k_means'
>>>

It looks like this issue is related to the latest version of scikit-learn deprecating that module, because installing with the previous version of scikit-learn works.
$ pip install dask-ml==1.1.1 scikit-learn==0.21.*
$ python
>>> import dask_ml.cluster
>>>

Does it work with dask-ml master? I restructured a few things to avoid
warnings, but I don't recall if they've been released yet.
On Fri, Jan 3, 2020 at 2:37 PM Peter Morrow<EMAIL_ADDRESS>wrote:

$ pip install dask-ml
$ python

import dask_ml.cluster
Traceback (most recent call last):
File "", line 1, in
File "/Users/peter/.virtualenvs/DASK/lib/python3.7/site-packages/dask_ml/cluster/init.py", line 3, in
from .k_means import KMeans  # noqa
File "/Users/peter/.virtualenvs/DASK/lib/python3.7/site-packages/dask_ml/cluster/k_means.py", line 23, in
from ._compat import _k_init
File "/Users/peter/.virtualenvs/DASK/lib/python3.7/site-packages/dask_ml/cluster/_compat.py", line 4, in
from sklearn.cluster._k_means import _k_init
ModuleNotFoundError: No module named 'sklearn.cluster._k_means'

It looks like this issue is related to the latest version of scikit-learn
deprecating that module, because installing with the previous version of
scikit-learn works.
$ pip install dask-ml==1.1.1 scikit-learn==0.21.*
$ python

import dask_ml.cluster

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask-ml/issues/597?email_source=notifications&email_token=AAKAOIUVOHYYM62MWMZDK3DQ36OYFA5CNFSM4KCRU6JKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4ID6MC4A,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKAOIT2EKLWWHT2N2WFIUDQ36OYFANCNFSM4KCRU6JA
.

Doesn't looks like it. I tried installing master and got the same issue.
$ pip install git+https://github.com/dask/dask-ml.git
$ pip freeze | grep scikit-learn
scikit-learn==0.22.1

I think it has to do with this dependency -changing it from scikit-learn>=0.21 to scikit-learn==0.21 might do the trick.

We don't want to pin an exact version of scikit-learn. We'd rather update
things to be compatible.
I would gladly accept a PR to fix things, otherwise I'll be able to look at
it in a few days.
On Fri, Jan 3, 2020 at 2:52 PM Peter Morrow<EMAIL_ADDRESS>wrote:

Doesn't looks like it. I tried installing master and got the same issue.
$ pip install git+https://github.com/dask/dask-ml.git
$ pip freeze | grep scikit-learn
scikit-learn==0.22.1
I think it has to do with this dependency
https://github.com/dask/dask-ml/blob/master/setup.py#L19 -changing it
from scikit-learn>=0.21 to scikit-learn==0.21 might do the trick.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask-ml/issues/597?email_source=notifications&email_token=AAKAOIR2D7Q4OWJAS3FXQNDQ36QRTA5CNFSM4KCRU6JKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEICBY6Y#issuecomment-570694779,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKAOIWE5LYFZ22OAGS3ZTTQ36QRTANCNFSM4KCRU6JA
.

I can take a stab at a PR.
In the dask_ml.cluster._compat module there is this import logic:
if SK_022:
    from sklearn.cluster._k_means import _k_init
else:
    from sklearn.cluster.k_means_ import _k_init

Changing it to always import _k_init from sklean.cluster.k_means_ doesn't raise a ModuleNotFoundError but yields a FutureWarning.

The sklearn.cluster.k_means_ module is  deprecated in version 0.22 and will be removed in version 0.24. The corresponding classes / functions should instead be imported from sklearn.cluster. Anything that cannot be imported from sklearn.cluster is now part of the private API.

I could submit a PR to always import from sklearn.cluster.k_means_, but that's probably not a good long term solution.
Or I could look into changing the dask_ml.cluster.k_means.init_pp function's implementation to use scikit-learn's public api. Not sure what this entails but I assume that you're using _k_init from the private scikit-learn API because there weren't any good options in the public API.
What approach do you think is best right now?

Thanks for looking into this.
Just importing from sklearn.cluster.k_means_ likely won't pass CI. That
will cause warnings which will cause the tests to fail.
I would need to look more closely to say for sure. If init_pp is available
via the public API that'd be best.
On Fri, Jan 3, 2020 at 3:31 PM Peter Morrow<EMAIL_ADDRESS>wrote:

I can take a stab at a PR.
In the dask_ml.cluster._compat module there is this import logic:
if SK_022:
from sklearn.cluster._k_means import k_init
else:
from sklearn.cluster.k_means import _k_init
Changing it to always import k_init from sklean.cluster.k_means doesn't
raise a ModuleNotFoundError but yields a FutureWarning.
The sklearn.cluster.k_means_ module is deprecated in version 0.22 and will
be removed in version 0.24. The corresponding classes / functions should
instead be imported from sklearn.cluster. Anything that cannot be imported
from sklearn.cluster is now part of the private API.
I could submit a PR to always import from sklearn.cluster.k_means_, but
that's probably not a good long term solution.
Or I could look into changing the dask_ml.cluster.k_means.init_pp
function's implementation to use scikit-learn's public api. Not sure what
this entails but I assume that you're using _k_init from the private
scikit-learn API because there weren't any good options in the public API.
What approach do you think is best right now?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask-ml/issues/597?email_source=notifications&email_token=AAKAOISQEM6UAWOU4THDRJLQ36VD3A5CNFSM4KCRU6JKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEICED5Y#issuecomment-570704375,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKAOIQIXLGEHAU5Y4SKZHTQ36VD3ANCNFSM4KCRU6JA
.

Looking into this now.

<ipython-input-48-d46ea042c026> in <module>
     12 import umap.plot
     13 import matplotlib.pyplot as plt
---> 14 import dask_ml.cluster

/project/libs/lib/python3.6/site-packages/dask_ml/cluster/__init__.py in <module>
      1 """Unsupervised Clustering Algorithms"""
      2
----> 3 from .k_means import KMeans  # noqa
      4 from .spectral import SpectralClustering  # noqa

/project/libs/lib/python3.6/site-packages/dask_ml/cluster/k_means.py in <module>
     20 )
     21 from ..utils import _timed, _timer, check_array, row_norms
---> 22 from ._compat import _k_init
     23
     24 import numba  # isort:skip (see https://github.com/dask/dask-ml/pull/577)

/project/libs/lib/python3.6/site-packages/dask_ml/cluster/_compat.py in <module>
----> 1 from sklearn.cluster._kmeans import _k_init
      2
      3 __all__ = ["_k_init"]

ImportError: cannot import name '_k_init'

I am using python 3.6, dask-ml-1.7.0 and scikit-learn-0.23.2


Fix top page of the web site

The top page of deep-cell.ucsd.edu is broken now and it should be fixed.

Fixed by replacing index.html.


Fix trezor implementation to work with python-trezor 0.11.0

python-trezor 0.11.0 changed a bunch of their api stuff, so this updates the trezor implementation to work with 0.11.0. The minimum trezor library version is also set in setup.py.

Merging this since travis built with an earlier version of HWI indicating that the implementation was fixed.


[stdlib] Fixes to doc. for formIndex of Indexable

Documentation for the blueprint and the implementation of formIndex(_:offsetBy:limitedBy:): added description for third parameter limit, which was previously omitted.
The description is copied from the analogous (non-inout mutating) index(_:offsetBy:limitedBy:) method.

@swift-ci Please smoke test and merge


OS and MySQL upgrade - take 2

OS upgrade - doc fixes
First cut of Rocky Linux 9 provisioning script
Add docker, docker-compose, and vim-enhanced
Add licenses, document install script
Update docs for MySQL 8 too

Resolves #34


Maybe a wrong explanation video in the form builder documentation?

This is  a bug
Prerequisites

[x] Are you running the latest version?
[x] Are you reporting to the correct repository?
[x] Did you check the documentation?
[x] Did you perform a cursory search?

Description
Clicking in the short video in the docs, shows me how to edit a permission, not a form. Is that correct?
Steps to Reproduce
In the Form Builder documentation, click on the short video at the begining
Expected behavior
A video setting up a form, creating a new one or editing an already existing one.
Actual behavior
Displays a video setting up a permission.

Oh, I see the confusion.. for most of the packages we have short presentation videos where we only show some of the capabilities.
Maybe at some point we will have video tutorials with coding steps but I don't expect this to happen too soon.
If you browse through the app you will see different implementation scenarios and you can also peek  in the code to see how's implemented. Together with the existing docs and the support on github you should be able to build something without too much effort.

This issue was more related to let you know if such video was a mistake or not.
But, as you say, I think I can find out the way to code new forms.
Thanks!


[CoreCLR/CoreFX] Runtime / RID Support update

Is there a reason why this is titled "coreclr/corefx", which are out of support? Or did you mean runtime?

for .NET 8.0 and .NET 9.0

Given that we're excluding 6.0, runtime does not need to make any manual RID additions starting with .NET 8.0.


Add doctest to function "dynamically"

The following shows up when running pytest --doctest-modules -v (or using pycharm to run doctests):
def f(x):
   """
   Increments by one.
   >>> f(2)
   3
   """
   return x + 1

But the following gives me the docs (with the example "doctest") just fine, but it is not included in the actual testing.
f = lambda x: x + 1
f.__doc__ = """
Increments by one.
>>> f(2)
3
"""

Posted to https://stackoverflow.com/questions/66360760/how-can-i-add-doctests-dynamically-in-python


Add a page for titler example

I started putting a page for titiler. I thought I would open the pr to be on the same page about a few details that will be better handled now than later.

I assumed that carbonplan/map example and titiler example would live on separate pages. but maybe we want them to live on the same page so users can toggle between them?
I put titiler example under /zarr-v2/titiler I am not sure if this is the best place, though.
I used MapboxGL v2, which requires Mapbox token for local dev. (This will need to be saved in Vercel environment too.) It is nice to see the variety of projections that MapboxGL v2 offers. But I can switch it to Mapboxgl v1 (or Maplibre? I haven't tried it yet, but I expect it to be similar to MapboxGL1) if an open-source tool is preferred.

Ah, I see. Yeah, we might not need it; let's add it if needed.


Subscriptions

Please add support of GraphQL real-time subscriptions (updates).

I'm a web developer of https://gdezerno.ru progressive web app (PWA) built using web components (Polymer) on front-end and Firebase on back-end.
I like Firebase, because data is synced across all clients in realtime. But Firebase has some downsides: it doesn't support complex database queries, it can not be deployed on my own server.
Now I'm researching the possibility of migrating back-end from Firebase to Prisma (https://prisma.io). It uses OpenCRUD GraphQL API (https://www.prisma.io/features/graphql-api/).
See https://blog.graph.cool/graphql-vs-firebase-496498546142 for more info.
But I'm worrying about the integration of Polymer with GraphQL. Now I use PolymerFire (https://webcomponents.org/element/firebase/polymerfire) to integrate Polymer with Firebase.
But if I migrate, I will need a replacement of PolymerFire webcomponents.
I plan to use GraphQL client-side subscription on front-end and GraphQL server-side subscription instead of Cloud Functions for Firebase database events triggers on back-end.

It would be up to the app author to set up and bundle subscriptions-transport-ws. Our components would just expose APIs on the client as web component base classes.

Should I use ApolloQuery class for subscriptions?

@bennypowers Could you please show (small code sample) how to use your base classes with subscriptions?   Should I use apollo-link-ws too?

I haven't tried yet. If you already have apollo-link-ws set up in your client build, try sending a subscription query through an ApolloQuery element. NO clue if it will work and i suspect it wont

@bennypowers

NO clue if it will work and i suspect it wont

This is not good. Subscriptions is one of the most important features why developers use Apollo Client (alongside with cache).

Did you give it a try? I'm curious to know what happened.

It's not possible to use subscriptions in ApolloQuery due to Error: Must contain a query definition. error.
It can be dirty fixed by changing client.watchQuery to client.subscribe in https://github.com/bennypowers/lit-apollo/blob/master/apollo-query.js#L279.
A few months ago, I added subscriptions (along with file uploads) support to polymer-apollo-client.
PTAL here to see it in action.
What about ApolloSubscription class and <apollo-subscription> custom element similar to my <graphql-subscription>? :wink:

Yeah looks like now's the time to start on that implementation. If what you say is correct, it may be as straightforward as extending ApolloQuery and implementing subscribe method as above. I'll look into this shortly.

@FluorescentHallucinogen  see https://warm-lowlands-46449.herokuapp.com/


Improve the resolution of graphics for Retina screens

WIP.
SVG bundling and inlining now work. Only major piece left is the popup yellow shield/pointer thing.
Using CSS filters for drop shadows, which generally work with -webkit- prefix and is under consideration for next version of IE, but the effect is slight enough to not worth polyfilling.

Popup is done.
All that's left is the sawtooth image, but that's not going to be noticeable so won't fix.


flutter doctor tips error

[!] Android Studio (version 4.1.0)
• Android Studio at D:\Android\Android Studio
X Flutter plugin not installed; this adds Flutter specific functionality.
X Dart plugin not installed; this adds Dart specific functionality.
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

Hi @gionchat
This is not an error but a warning that does not stops you from using Flutter. It is a known issue, you can just ignore it for now.
Thank you for filling this.


flask command should auto-detect module

Even when installed as a package, it can be useful in complex situations to have a wsgi.py file for setting the app up to run. The flask command should detect this file if no FLASK_APP is specified at all.
Additionally, it's common to name quick demo apps app.py, so that should be detected too.
From discussion in #2375.

is wsgi.py even a common name? when using it in a wsgi container i think it's more common to name it something.wsgi (after all, it's not referenced as a python module but rather using a fileystem path)

The problem is that something is not standard, so there's no way to autodetect it. One strategy could be to also support importing something.wsgi if something is specified before trying create_app.

Django defines the application in a wsgi.py module.


Implement shipping service (business layer)

Unit tests can come later when the repository tests are finished. But you can try and execute the function that are defined by the repository interface.
https://github.com/TokoBapak/mock-shipping-provider/blob/413bb3e6abb6362ced4b4bdc42173c1c82f6065d/business/shipping/shipping.go#L17-L32
Interface definition:
https://github.com/TokoBapak/mock-shipping-provider/blob/68058d64cc9b8f7db73478fd0a1392055c5c8d45/business/shipping.go#L11-L24
The background worker on the Create function is a simple goroutine that is managed by a time.Sleep. So basically it goes like this:
func Create(ctx context.Context, request business.CreateRequest) (business.CreateResponse, error) {
  var orderHistory = []OrderHistory{
    {
      Status: primitive.StatusOrderPlaced,
      Timestamp: time.Now(),
    },
    {
      Status: primitive.StatusPickupPending,
      Timestamp: time.Now().Add(time.Minute),
    },
    {
      Status: primitive.StatusPickedUp,
      Timestamp: time.Now().Add(time.Hour * 2),
   }, // and so on forth
  }

  // Insert the order history to the database
  go func() {
    for i, entry := range orderHistory {
      if entry.Status == primitive.StatusOrderPlaced {
        continue
      }

      time.Sleep(entry.Timestamp.Sub(orderHistory[i-1].Timestamp))
      // Send webhook
    }
  }()
}

The example above can be wrong. Please test your function accordingly.

btw, I think this return should be non-array
status-history
'cuz. when we see the response sample of the StatusHistory and the Struct of it. its just history that given the array to the response.
{
    "reference_number": "string",
    "air_waybill": "string",
    "history": [
        {
            "status_code": 1,
            "status_description": "ORDER_PLACED",
            "timestamp": "2023-01-01T00:00:00Z",
            "note": "string"
        }
    ]
}

I found this issue while I tried to handle the presentation-layer.

btw, I think this return should be non-array status-history
'cuz. when we see the response sample of the StatusHistory and the Struct of it. its just history that given the array to the response.
{
    "reference_number": "string",
    "air_waybill": "string",
    "history": [
        {
            "status_code": 1,
            "status_description": "ORDER_PLACED",
            "timestamp": "2023-01-01T00:00:00Z",
            "note": "string"
        }
    ]
}

I found this issue while I tried to handle the presentation-layer.

Oh you're right. You can change that one.

I think the shipping.Estimate() should return an array of EstimateResult with included Provider, because in the presentation we expect array of Estimation to be returned as response
https://github.com/TokoBapak/mock-shipping-provider/blob/68058d64cc9b8f7db73478fd0a1392055c5c8d45/presentation/schema/estimate_response.go#L10

we've already discussed this btw. but yes, you're right

we've already discussed this btw. but yes, you're right

you are discuss about StatusHistory 😅

nope, we discuss it on private chat yesterday.

Again, should we change the request method for /status-history into POST?
Because the StatusRequest struct in the package business requires some fields
https://github.com/TokoBapak/mock-shipping-provider/blob/68058d64cc9b8f7db73478fd0a1392055c5c8d45/business/shipping.go#L65

so, what's the StatusRequest used for?
Where can we get the required data? is it from the query parameters?


DOC: add a docstring generator and stubs

This adds a python file _add_newdocs.py. It has placeholder docstrings for all the functions in the c-extension module. These will be turned into C strings in a FAST_NUMPY_LOOPS.h header file, which will be picked up by compilation.

I will go ahead and merge this. We can iterate on the idea (it is a bit hacky right now) and on the docstring content


ETH GAS LIMIT TOO HIGH

Hello. I made a hashlips minting dapp but the ethereum gas limit is set WAY too high. Like double compared to other projects. WHERE IN THE DAPP CAN I CHANGE THIS??????

Did you launch the dapp on github?  If so, how did you do it and I will tell you how to change the eth to matic or lower the gas.  My Gas was over $5000 eth and ended up after trying a few times, to .0005 matic.  Just need to get the dapp up and running properly.  So, did you get the dapp to work on github or just local:3000?

can I know what the most suitable gas limit to set?


Usage of rich text box in voyager

Version information

Laravel: v5.4
Voyager: v2.0
PHP: 7.0
Database: [MySQL] [5.1.73]

Description

when i use rich text box in voyager admin panel, it is correct but in front end side it appear as symbol. how can solve this?
Screenshots
If applicable, add screenshots to help explain your problem.

Make sure you're using the correct interpolation on your blade file:

{!! $variable !!}

That's exactly the problem.

Make sure you're using the correct interpolation on your blade file:

{!! $variable !!}

Tank you very much


Suggestion - Add the libheif binary files to the wheel when building

Instead of having to specify them every package that uses it, include the .dll files in the wheel itself.
For example just need to add the line: data_files=[('bin', [str(x.absolute()) for x in pathlib.Path(lib_dirs[0]).glob("*.dll")])], to the setup( section of setup.py
Then in cyheifloader\__init__.py change the line to libheif_path = os.getenv('LIBHEIF_PATH', os.path.join(sys.prefix, 'bin'))
That way when you run python setup.py bdist_wheel, the wheel file itself could house the required files itself and be truly distributable.
Currently untested on Linux, just Windows.

Great idea and thanks for the PR. Added a couple of comments to the PR.

Thanks again for adding the ability to package the dependencies on Windows.


Support for Google Application Default Credentials

Description
This change provides the support for Application Default Credentials for the repository-gcs plugin to support the snapshot api requests.
The solution uses the existent google auth library. The google client is instantiated being wrapped in a doPrivileged block i.e using SocketAccess class to avoid Access Denied errors and to connect to google cloud services.
Related Issues
Resolves #8375
Check List

[x] New functionality includes testing.

[x] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff
[x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Thanks for creating this pull request - I'll take another look after tests have been added.

fyi, I have been engaged in some other critical issues which did not allow me to work on this PR. Rest assured I have this inlined after the critical fixes and will work on the test cases.

What has been added: Pull the main branch changes and make couple of code neat changes

Hi @peternied and @dblock, I have added the tests, when you have some time please have a look, thanks

❌ Gradle check result for 773184b: FAILURE

https://github.com/opensearch-project/OpenSearch/issues/10006
https://github.com/opensearch-project/OpenSearch/issues/9499

@peternied want to re-CR
@fahadshamiinsta unfortunately for flaky tests it's amend, force push, rince and repeat until it's green :( you can fix a flaky test while it's rebuilding 😅

Hi @fahadshamiinsta, is this being worked upon? Pls free to reach out to maintainers for further reviews.

❌ Gradle check result for e4583a4: FAILURE
Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

https://github.com/opensearch-project/OpenSearch/issues/10006

❌ Gradle check result for 6fb32ef: FAILURE

https://github.com/opensearch-project/OpenSearch/issues/5176

❌ Gradle check result for 7cf868a: FAILURE

https://github.com/opensearch-project/OpenSearch/issues/10558

UNSTABLE

https://github.com/opensearch-project/OpenSearch/issues/7980

FAILURE

https://github.com/opensearch-project/OpenSearch/issues/10006

Close! I have some naming asks, I think the way the classes and methods are named is incorrect.
Iterate to green please!
Is there a doc on how to use this? Do we need to write one?

@dblock please have another look, I have referenced the flaky tests to the PR and provide the link of the document in the class comment. I had to do extra commits to trigger the ci checks (not sure if there is another way)

@fahadshamiinsta you will need to manually backport this if you want it in the next 2.x release (2.12)


Fix ssl_session_reuse to compile on macOS and FreeBSD

[approve ci autest]

[approve ci clang-analyzer]

Cherry-picked to v9.0.x branch.

This has merge conflicts for 8.1.x, if anyone wants this for 8.1.x, please mark this as Project 8.x again, and make a PR with a cherry-pick -x against the 8.1.x branch.


bump: pythonPackages.simanneal, fastjet, herwig

Motivation for this change
Bump some of the packages I maintain.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build python2Packages.simanneal python3Packages.simanneal fastjet herwig

Unexpected error: command failed with exit code 1 on x86_64-linux (full log)
Attempted: python2Packages.simanneal, python3Packages.simanneal, fastjet, herwig
Partial log (click to expand)
============================= test session starts ==============================
platform linux -- Python 3.6.6, pytest-3.7.4, py-1.5.4, pluggy-0.7.1
rootdir: /build/source, inifile:
collected 4 items

tests/test_anneal.py ....                                                [100%]

=========================== 4 passed in 1.87 seconds ===========================
pytestcachePhase
error: build of '/nix/store/ldsgagpg8ww885a5ga08br8a08931sq3-herwig-7.1.4.drv', '/nix/store/lnj74dig2snxa1cggkxdjwhc82hi7z6g-fastjet-3.3.2.drv' failed

Unexpected error: command failed with exit code 1 on x86_64-linux (full log)
Attempted: fastjet, herwig
Partial log (click to expand)
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:--  0:02:11 --:--:--     0curl: (7) Failed to connect to fastjet.fr port 80: Connection timed out
error: cannot download fastjet-3.3.2.tar.gz from any mirror
builder for '/nix/store/q00m2nn5hmkm6xdz0apcbprxpilsdm8j-fastjet-3.3.2.tar.gz.drv' failed with exit code 1
cannot build derivation '/nix/store/lnj74dig2snxa1cggkxdjwhc82hi7z6g-fastjet-3.3.2.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/zkh99xqzqdw78kmv61f9gdpq5h9h74pr-rivet-2.6.1.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/rh0cv711zp5a3lpz9f9s6fpvr9q66b5l-thepeg-2.1.4.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/ldsgagpg8ww885a5ga08br8a08931sq3-herwig-7.1.4.drv': 2 dependencies couldn't be built
error: build of '/nix/store/ldsgagpg8ww885a5ga08br8a08931sq3-herwig-7.1.4.drv', '/nix/store/lnj74dig2snxa1cggkxdjwhc82hi7z6g-fastjet-3.3.2.drv' failed

Failure on aarch64-linux (full log)
Attempted: python2Packages.simanneal, python3Packages.simanneal, fastjet, herwig
Partial log (click to expand)
f951: Warning: command line option '-Wformat-security' is valid for C/C++/ObjC/ObjC++ but not for Fortran
f951: Warning: command line option '-Wformat-security' is valid for C/C++/ObjC/ObjC++ but not for Fortran
clooptools.cc: In function 'int {anonymous}::start_redirection(std::__cxx11::string)':
clooptools.cc:111:12: warning: ignoring return value of 'FILE* freopen(const char*, const char*, FILE*)', declared with attribute warn_unused_result [-Wunused-result]
     freopen(logfilename.c_str(), "a", stdout);
     ~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
make[1]: Leaving directory '/build/Herwig-7.1.4/Looptools'
make: *** [Makefile:520: all-recursive] Error 1
builder for '/nix/store/7cv9isf6nwk5h1p88blbh9c5yq4v58xs-herwig-7.1.4.drv' failed with exit code 2
error: build of '/nix/store/7cv9isf6nwk5h1p88blbh9c5yq4v58xs-herwig-7.1.4.drv' failed

Failure on aarch64-linux (full log)
Attempted: fastjet, herwig
Partial log (click to expand)
f951: Warning: command line option '-Wformat=1' is valid for C/C++/ObjC/ObjC++ but not for Fortran
f951: Warning: command line option '-Wformat-security' is valid for C/C++/ObjC/ObjC++ but not for Fortran
clooptools.cc: In function 'int {anonymous}::start_redirection(std::__cxx11::string)':
clooptools.cc:111:12: warning: ignoring return value of 'FILE* freopen(const char*, const char*, FILE*)', declared with attribute warn_unused_result [-Wunused-result]
     freopen(logfilename.c_str(), "a", stdout);
     ~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
make[1]: Leaving directory '/build/Herwig-7.1.4/Looptools'
make: *** [Makefile:520: all-recursive] Error 1
builder for '/nix/store/7cv9isf6nwk5h1p88blbh9c5yq4v58xs-herwig-7.1.4.drv' failed with exit code 2
error: build of '/nix/store/7cv9isf6nwk5h1p88blbh9c5yq4v58xs-herwig-7.1.4.drv' failed

Success on x86_64-darwin (full log)
Attempted: fastjet, herwig
Partial log (click to expand)
make[2]: Nothing to be done for 'install-data-am'.
make[2]: Leaving directory '/private/tmp/nix-build-herwig-7.1.4.drv-0/Herwig-7.1.4'
make[1]: Leaving directory '/private/tmp/nix-build-herwig-7.1.4.drv-0/Herwig-7.1.4'
post-installation fixup
strip is /nix/store/g5r4apl0za012ffs6ladinwa5w0m1l3k-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/aywgqhm47ddkp4jdga5gfn5xswjxz2ya-herwig-7.1.4/lib  /nix/store/aywgqhm47ddkp4jdga5gfn5xswjxz2ya-herwig-7.1.4/bin
patching script interpreter paths in /nix/store/aywgqhm47ddkp4jdga5gfn5xswjxz2ya-herwig-7.1.4
/nix/store/aywgqhm47ddkp4jdga5gfn5xswjxz2ya-herwig-7.1.4/bin/herwig-config: interpreter directive changed from " /usr/bin/env bash" to "/nix/store/n9hba031gjky8hpjgx9fnlaxhidyzxbz-bash-4.4-p23/bin/bash"
/nix/store/kxr2f4dx1vahjc9n27lpmzsagihkbjhf-fastjet-3.3.2
/nix/store/aywgqhm47ddkp4jdga5gfn5xswjxz2ya-herwig-7.1.4

Success on x86_64-darwin (full log)
Attempted: python2Packages.simanneal, python3Packages.simanneal, fastjet, herwig
Partial log (click to expand)
make[1]: Leaving directory '/private/tmp/nix-build-herwig-7.1.4.drv-0/Herwig-7.1.4'
post-installation fixup
strip is /nix/store/g5r4apl0za012ffs6ladinwa5w0m1l3k-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/aywgqhm47ddkp4jdga5gfn5xswjxz2ya-herwig-7.1.4/lib  /nix/store/aywgqhm47ddkp4jdga5gfn5xswjxz2ya-herwig-7.1.4/bin
patching script interpreter paths in /nix/store/aywgqhm47ddkp4jdga5gfn5xswjxz2ya-herwig-7.1.4
/nix/store/aywgqhm47ddkp4jdga5gfn5xswjxz2ya-herwig-7.1.4/bin/herwig-config: interpreter directive changed from " /usr/bin/env bash" to "/nix/store/n9hba031gjky8hpjgx9fnlaxhidyzxbz-bash-4.4-p23/bin/bash"
/nix/store/8irc0mkf65hrbpd9wm24bdggnbk9cxnm-python2.7-simanneal-0.4.2
/nix/store/i9nlf0p3qb4jgbnzv29bnh9hklsxrgik-python3.6-simanneal-0.4.2
/nix/store/kxr2f4dx1vahjc9n27lpmzsagihkbjhf-fastjet-3.3.2
/nix/store/aywgqhm47ddkp4jdga5gfn5xswjxz2ya-herwig-7.1.4

@GrahamcOfBorg build python2Packages.simanneal python3Packages.simanneal fastjet herwig

@GrahamcOfBorg build python2Packages.simanneal python3Packages.simanneal fastjet herwig

@grahamc, there is something wrong with the bot. Is it down?

Unexpected error: command failed with exit code 1 on aarch64-linux (full log)
Attempted: python2Packages.simanneal, python3Packages.simanneal, fastjet, herwig
Partial log (click to expand)
building '/nix/store/jgyaw8lb59k43dl06swa1fl6j4xb0fdi-texlive-chronosys-1.2.drv'...
building '/nix/store/545llzjqyphy1sxzv7zi006db6b6p9l3-texlive-cmextra-2018.drv'...
building '/nix/store/mvb7zkzrrsfvnanqbv02qvr13s2zpdwp-texlive-courier-2018.drv'...
building '/nix/store/pfmh2rfnv0n0l7ms5rrx2i5y810fqi20-texlive-docbytex-2018.drv'...
building '/nix/store/xqqhridbp72kkaxpcpn9gdrqvdn359k9-texlive-dvipdfmx-2018.drv'...
building '/nix/store/3wvawgbhy6ykzwmxzvqfc0gj6r2v73w9-texlive-ec-1.0.drv'...
building '/nix/store/spw28nqcskxg4nl65mb6hzfbs7r4kvi0-texlive-encxvlna-1.1.drv'...
building '/nix/store/j08zmlhw12y4mk6lvvpwmbbcfc3y008f-texlive-epsf-dvipdfmx-2014.drv'...
building '/nix/store/vbhz39iph5x5kc1cvpz6f0p5hkzv7w3p-texlive-euro-ce-3.0b.drv'...
error: all build users are currently in use; consider creating additional users and adding them to the 'nixbld' group

Unexpected error: command failed with exit code 1 on aarch64-linux (full log)
Attempted: python2Packages.simanneal, python3Packages.simanneal, fastjet, herwig
Partial log (click to expand)
building '/nix/store/iha2kpfj67z14pbiqn55dihl0q07xh7d-texlive-dowith-r0.32.drv'...
building '/nix/store/s3fyn262m6awpdizbdv0p4klhxzlcdcm-texlive-dsptricks-1.0.drv'...
building '/nix/store/jwj27y16gf9lh1vhn3njvvjdm7vwdi7l-texlive-dvips-2018.drv'...
building '/nix/store/30872ypdd8w418bdz5zi8lz8ma9ms9l4-texlive-eijkhout-2018.drv'...
building '/nix/store/nwqrf8ldfz1nwj6f40s2rpdjwq054ff2-texlive-enctex-2018.drv'...
building '/nix/store/g18d1qkl5as214lfc20fs4bsaz37j9s5-texlive-epigram-2018.drv'...
building '/nix/store/j3bjic66mwnh9ffz91g26b53b3xf4r1i-texlive-etex-2018.drv'...
building '/nix/store/3h3nxy6i4dcza5nn3ymanxjmmi4q754g-texlive-etex-pkg-2.7.drv'...
building '/nix/store/61gah93rwfgznwm7z71axbniiz1zdswv-texlive-euro-1.1.drv'...
error: all build users are currently in use; consider creating additional users and adding them to the 'nixbld' group

Unexpected error: command failed with exit code 1 on x86_64-linux (full log)
Attempted: python2Packages.simanneal, python3Packages.simanneal, fastjet, herwig
Partial log (click to expand)
copying path '/nix/store/lmbk9q370nall4pbahrm0hnaikpg8r3f-texlive-bin-2018-doc' from 'https://cache.nixos.org'...
copying path '/nix/store/crvv8sq56a2qqlrd5gqsxrx4x74rmnmm-texlive-core-big.bin-2018-luatex' from 'https://cache.nixos.org'...
copying path '/nix/store/l46xml3a26k3gv6wj9f8aiwvmsbzfaxw-texlive-core-big.bin-2018-metafont' from 'https://cache.nixos.org'...
copying path '/nix/store/2m71836aw0k94p0rvf6z7pa1w4zzwswh-texlive-xdvi.bin-2018' from 'https://cache.nixos.org'...
copying path '/nix/store/q478pwvq4cz28f386b9dgkk65bwkcwmn-yoda-1.7.3' from 'https://cache.nixos.org'...
copying path '/nix/store/23aiwc0f9visq83bpi5vshp3sx7rhlr7-texlive-combined-2018' from 'https://cache.nixos.org'...
cannot build derivation '/nix/store/0dp7ci525pbdv30glrqa5mg1bq0yjq5g-rivet-2.6.1.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/1zii6q5410pwidns7kk7qxzd7vsjflhn-thepeg-2.1.4.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/8jyc44mip42zb1gik1p85bmnm2xvzmr7-herwig-7.1.4.drv': 2 dependencies couldn't be built
error: build of '/nix/store/8jyc44mip42zb1gik1p85bmnm2xvzmr7-herwig-7.1.4.drv', '/nix/store/lnj74dig2snxa1cggkxdjwhc82hi7z6g-fastjet-3.3.2.drv' failed

Unexpected error: command failed with exit code 1 on x86_64-linux (full log)
Attempted: python2Packages.simanneal, python3Packages.simanneal, fastjet, herwig
Partial log (click to expand)
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:--  0:02:08 --:--:--     0curl: (7) Failed to connect to fastjet.fr port 80: Connection timed out
error: cannot download fastjet-3.3.2.tar.gz from any mirror
builder for '/nix/store/q00m2nn5hmkm6xdz0apcbprxpilsdm8j-fastjet-3.3.2.tar.gz.drv' failed with exit code 1
cannot build derivation '/nix/store/lnj74dig2snxa1cggkxdjwhc82hi7z6g-fastjet-3.3.2.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/0dp7ci525pbdv30glrqa5mg1bq0yjq5g-rivet-2.6.1.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/1zii6q5410pwidns7kk7qxzd7vsjflhn-thepeg-2.1.4.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/8jyc44mip42zb1gik1p85bmnm2xvzmr7-herwig-7.1.4.drv': 2 dependencies couldn't be built
error: build of '/nix/store/8jyc44mip42zb1gik1p85bmnm2xvzmr7-herwig-7.1.4.drv', '/nix/store/lnj74dig2snxa1cggkxdjwhc82hi7z6g-fastjet-3.3.2.drv' failed

Success on x86_64-darwin (full log)
Attempted: python2Packages.simanneal, python3Packages.simanneal, fastjet, herwig
Partial log (click to expand)
make[1]: Leaving directory '/private/tmp/nix-build-herwig-7.1.4.drv-0/Herwig-7.1.4'
post-installation fixup
strip is /nix/store/g5r4apl0za012ffs6ladinwa5w0m1l3k-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/yqb6k1by6y25mjm41n91s8kkj5r9j055-herwig-7.1.4/lib  /nix/store/yqb6k1by6y25mjm41n91s8kkj5r9j055-herwig-7.1.4/bin
patching script interpreter paths in /nix/store/yqb6k1by6y25mjm41n91s8kkj5r9j055-herwig-7.1.4
/nix/store/yqb6k1by6y25mjm41n91s8kkj5r9j055-herwig-7.1.4/bin/herwig-config: interpreter directive changed from " /usr/bin/env bash" to "/nix/store/n9hba031gjky8hpjgx9fnlaxhidyzxbz-bash-4.4-p23/bin/bash"
/nix/store/8irc0mkf65hrbpd9wm24bdggnbk9cxnm-python2.7-simanneal-0.4.2
/nix/store/i9nlf0p3qb4jgbnzv29bnh9hklsxrgik-python3.6-simanneal-0.4.2
/nix/store/kxr2f4dx1vahjc9n27lpmzsagihkbjhf-fastjet-3.3.2
/nix/store/yqb6k1by6y25mjm41n91s8kkj5r9j055-herwig-7.1.4

Success on x86_64-darwin (full log)
Attempted: python2Packages.simanneal, python3Packages.simanneal, fastjet, herwig
Partial log (click to expand)
make[1]: Leaving directory '/private/tmp/nix-build-herwig-7.1.4.drv-0/Herwig-7.1.4'
post-installation fixup
strip is /nix/store/g5r4apl0za012ffs6ladinwa5w0m1l3k-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/yqb6k1by6y25mjm41n91s8kkj5r9j055-herwig-7.1.4/lib  /nix/store/yqb6k1by6y25mjm41n91s8kkj5r9j055-herwig-7.1.4/bin
patching script interpreter paths in /nix/store/yqb6k1by6y25mjm41n91s8kkj5r9j055-herwig-7.1.4
/nix/store/yqb6k1by6y25mjm41n91s8kkj5r9j055-herwig-7.1.4/bin/herwig-config: interpreter directive changed from " /usr/bin/env bash" to "/nix/store/n9hba031gjky8hpjgx9fnlaxhidyzxbz-bash-4.4-p23/bin/bash"
/nix/store/8irc0mkf65hrbpd9wm24bdggnbk9cxnm-python2.7-simanneal-0.4.2
/nix/store/i9nlf0p3qb4jgbnzv29bnh9hklsxrgik-python3.6-simanneal-0.4.2
/nix/store/kxr2f4dx1vahjc9n27lpmzsagihkbjhf-fastjet-3.3.2
/nix/store/yqb6k1by6y25mjm41n91s8kkj5r9j055-herwig-7.1.4

@GrahamcOfBorg build python2Packages.simanneal python3Packages.simanneal fastjet herwig

Failure on aarch64-linux (full log)
Attempted: python2Packages.simanneal, python3Packages.simanneal, fastjet, herwig
Partial log (click to expand)
f951: Warning: command line option '-Wformat=1' is valid for C/C++/ObjC/ObjC++ but not for Fortran
f951: Warning: command line option '-Wformat-security' is valid for C/C++/ObjC/ObjC++ but not for Fortran
clooptools.cc: In function 'int {anonymous}::start_redirection(std::__cxx11::string)':
clooptools.cc:111:12: warning: ignoring return value of 'FILE* freopen(const char*, const char*, FILE*)', declared with attribute warn_unused_result [-Wunused-result]
     freopen(logfilename.c_str(), "a", stdout);
     ~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
make[1]: Leaving directory '/build/Herwig-7.1.4/Looptools'
make: *** [Makefile:520: all-recursive] Error 1
builder for '/nix/store/cxaxm0vql8jmays1nn1wpwap89986n74-herwig-7.1.4.drv' failed with exit code 2
error: build of '/nix/store/cxaxm0vql8jmays1nn1wpwap89986n74-herwig-7.1.4.drv' failed

Success on x86_64-darwin (full log)
Attempted: python2Packages.simanneal, python3Packages.simanneal, fastjet, herwig
Partial log (click to expand)
make[1]: Leaving directory '/private/tmp/nix-build-herwig-7.1.4.drv-0/Herwig-7.1.4'
post-installation fixup
strip is /nix/store/rk3p99jbj4idl3dkkg1rxja5gz0qgs98-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/3i094ma6fd31qn5ab1l5qr2jgqg9ackc-herwig-7.1.4/lib  /nix/store/3i094ma6fd31qn5ab1l5qr2jgqg9ackc-herwig-7.1.4/bin
patching script interpreter paths in /nix/store/3i094ma6fd31qn5ab1l5qr2jgqg9ackc-herwig-7.1.4
/nix/store/3i094ma6fd31qn5ab1l5qr2jgqg9ackc-herwig-7.1.4/bin/herwig-config: interpreter directive changed from " /usr/bin/env bash" to "/nix/store/ixr5kls084hdlrs1wcy4c1q9fyw3rxfb-bash-4.4-p23/bin/bash"
/nix/store/53d2fhfmpcz9x2694k2m5pp4mhy3n85d-python2.7-simanneal-0.4.2
/nix/store/qhdm147lzg9aqqk7nlr6laa8b4jm1ygj-python3.7-simanneal-0.4.2
/nix/store/yb8chh51m9m4h0r2qnjswcd9kxb1h4hg-fastjet-3.3.2
/nix/store/3i094ma6fd31qn5ab1l5qr2jgqg9ackc-herwig-7.1.4

Timed out, unknown build status on x86_64-linux (full log)
Attempted: python2Packages.simanneal, python3Packages.simanneal, fastjet, herwig
Partial log (click to expand)
Making all in Builtin
make[3]: Entering directory '/build/Herwig-7.1.4/MatrixElement/Matchbox/Builtin'
Making all in Amplitudes
make[4]: Entering directory '/build/Herwig-7.1.4/MatrixElement/Matchbox/Builtin/Amplitudes'
Concatenating .cc files into Amplitudes__all.cc
make  all-am
make[5]: Entering directory '/build/Herwig-7.1.4/MatrixElement/Matchbox/Builtin/Amplitudes'
  CXX      Amplitudes__all.lo
building of '/nix/store/v3w08dzxyw64wxkwp3l7fx0syrn5n9rd-herwig-7.1.4.drv' timed out after 3600 seconds
error: build of '/nix/store/v3w08dzxyw64wxkwp3l7fx0syrn5n9rd-herwig-7.1.4.drv' failed

Is this ready for review?

@Mic92 Yes, it is!

Is herwig is compatible with aarch64? If not it is better to disable it there.

@Mic92 I don't think anyone will run it on ARM in the near future.

Success on x86_64-darwin (full log)
Attempted: fastjet, herwig
Partial log (click to expand)
/nix/store/yb8chh51m9m4h0r2qnjswcd9kxb1h4hg-fastjet-3.3.2
/nix/store/3i094ma6fd31qn5ab1l5qr2jgqg9ackc-herwig-7.1.4

Success on aarch64-linux (full log)
Attempted: fastjet
The following builds were skipped because they don't evaluate on aarch64-linux: herwig
Partial log (click to expand)
a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowBroken = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowBroken = true; }
to ~/.config/nixpkgs/config.nix.

/nix/store/4l70sdmx7b8pfc4l6fzcgszqcv8j4r0d-fastjet-3.3.2

Unexpected error: command failed with exit code 1 on x86_64-linux (full log)
Attempted: fastjet, herwig
Partial log (click to expand)
copying path '/nix/store/y5lm8ydw96hy1bzrgj97bh9yrs8k9q5w-texlive-bin-2018' from 'https://cache.nixos.org'...
copying path '/nix/store/hbm20pgc3kpy6lxsc2r36ggyx7n9l0in-texlive-core-big.bin-2018-luatex' from 'https://cache.nixos.org'...
copying path '/nix/store/iidhswrjadz7xfd1zciczkdwgyqgbn6k-texlive-core-big.bin-2018-metafont' from 'https://cache.nixos.org'...
copying path '/nix/store/pbrakxr3wlbq19q9z7qgib443z8svwzr-texlive-xdvi.bin-2018' from 'https://cache.nixos.org'...
copying path '/nix/store/ngi5p4zq4xa4hr9a0bf3v8kz2mxw4dxw-texlive-combined-2018' from 'https://cache.nixos.org'...
cannot link '/nix/store/.links/1zyw0yjzvmaaz5fydi16ybvwshd1nh0yv8q80gs0frix12gbibmk' to '/nix/store/ngi5p4zq4xa4hr9a0bf3v8kz2mxw4dxw-texlive-combined-2018/bin/psbook': No space left on device
cannot build derivation '/nix/store/jhhgbnrpnxv31lww89jv3c1v52klsd1r-rivet-2.6.1.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/3gs0578fcr3jykiady5h1582xz0js8dc-thepeg-2.1.4.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/v3w08dzxyw64wxkwp3l7fx0syrn5n9rd-herwig-7.1.4.drv': 2 dependencies couldn't be built
error: build of '/nix/store/v3w08dzxyw64wxkwp3l7fx0syrn5n9rd-herwig-7.1.4.drv', '/nix/store/xn5fi4cy631kn0yc3wcqzf21vk698nin-fastjet-3.3.2.drv' failed


Make Position2D components editable in selection panel

Related

Part of #8299

What
This adds a simple single line edit for position 2D. There is no speed_fn because I think in most applications UI coordinates are probably good unit (for example in graphs).
https://github.com/user-attachments/assets/2fad00c4-8e68-40fe-a474-32fa5806814e

oh interesting, so actually it preserves the style better than what's there in vec.rs
It's just that for edit boxes we so far didn't want to spend so much horiztonal space 🤔

oops sorry mouse slipped and clicked the wrong button