Update tag value types to support int, float, bool

We support these other types as tag values, so let's update the type definition to allow it.
[skip changeset] as it's an improvement on a not yet released feature.

Merging because I want to release this with the new package version.


Add an option to save with Unix line endings by default

There should be an option on Application Settings.

Yes it will be available on next version.

This feature has added on v1.9


Implemented j guide pages

Desktop
https://www.loom.com/share/d27e2806a72c449687361b8b1aaf299e
Mobile
https://www.loom.com/share/4c4d0b1bfb5d4aba8070af0f8c05a4d3

@Oluwa-nifemi please fix merge conflict


Missing methods to handle errors on apple pay.

Is your feature request related to a problem? Please describe.
I am unable to find any methods to cancel apple pay in case of an error. If we pass anything other than proper clientSecret to confirmApplePayPayment the whole app crashes. Am I missing
Describe the solution you'd like
A new method like cancelApplePayPrompt should be enough.

To show an error in the apple pay sheet you need to use the callback handler, e.g. https://stripe.dev/stripe-react-native/api-reference/interfaces/useapplepayprops.html#onshippingcontactselected
There is no way to close the Apple Pay sheet programmatically.

tipsi-stripe managed to do it on their library. May be we could apply the same logic here?

This seems to be how they are doing it.
RCT_EXPORT_METHOD(cancelApplePayRequest:(RCTPromiseResolveBlock)resolve
                  rejecter:(RCTPromiseRejectBlock)reject) {
    if (applePayCompletion) {
        promiseResolver = resolve;
        [self resolveApplePayCompletion:PKPaymentAuthorizationStatusFailure];
    } else {
        resolve(nil);
    }
}


Pls support Elan Touchscreen

I have an Elan Touchscreen and it works fine with 10.11.6 and 10.12.1 12B5657
Pls support
https://youtu.be/1FbQMclR82Y
BTW: very promising ....
Pls make it fully function
https://youtu.be/lso3yaW1k3E

You may need UPDD's driver + UPDD Gestures, will cost you a lot though. I'm using my Ativ tab 7 with macOS with touch and all that stuff and it's working fine :)

Yet touch-base has evaluation driver, work greet with my Yoga 15

We "support" only official Mac devices that don't come with a Touch Bar. For unsupported devices, you're on your own. We just can't support this devices since we don't own them and can't test for them and don't even know what's broken.
Even though we don't support these devices, we're willing to include fixes for these devices when submitted through a PR, as long as they don't break functionality on supported devices.

The touchscreen work exactly as if the real touch bar.

OK, so then there is no issue?

I haven't retried iTunes again. But if the Touchbar is oded as is default on real Mac, I can test


Feature/#93 nodes with metadata

Description
Expose graph data to fragment result
Motivation and Context

In order to enable chrome extension to print graph data, the data must be exposed to fragment result
Screenshots (if appropriate)
Upgrade notes (if appropriate)

Types of changes

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

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

I hereby agree to the terms of the Knot.x Contributor License Agreement.

We decided to reimplement this functionality to not change engine interfaces.


Poor performance when refreshing kafka topics

Hello,
Yesterday we upgraded aiven terraform provider from version v1.1.3 to v1.2.3 and after that we observed 7 times longer freshness time for our HCL code. I can see that majority of it is spend on refreshing kafka topics.
Are You aware of this performance issue? Can it be fixed in some way?

Hello,
It is the first time when we here about this issue and perhaps it is related to a new Kafka Topic Cache layer that was implemented in v1.2.1. Can you please share more details about your Kafka Topics set up, for example, how many Kafka clusters and topics do you have?

Hello,
I can confirm this behavior by upgrading 1.0.19 -> 1.2.3
time terraform plan command with both versions:
1.0.19
real    0m26.103s
user    0m8.782s
sys     0m1.788s
1.2.3
real    2m26.216s
user    0m9.015s
sys     0m1.806s
My terraform code consists of 36 aiven_service_user, 108 aiven_kafka_acl, 52 aiven_kafka_topic and some negligible resources.
But the terraform plan seems to hang on some aiven_kafka_topic resources but not all that is weird. Like 5-10s on some line "aiven_kafka_topic (...) Refreshing state..."

perhaps it is related to a new Kafka Topic Cache layer that was implemented in v1.2.1.
I have just tested upgrade in 1.2.0 -> no problem, but with 1.2.1 -> problem. Your guess is right @ivan-savciuc

Improvement is available in https://github.com/aiven/terraform-provider-aiven/releases/tag/v1.2.4

I confirm performance issue is gone in 1.2.4, thanks! 👍


Add migration to bring Postgres in parity with SQLAlchemy

this migration & related code sqlalchemy model changes should bring our postgres database schema in sync with the schema defined in sqlalchemy.  we need this so that we can drop our in-house migration framework and switch to alembic, which can automatically generate migrations for basic schema changes and enforce correctness.
there's quite a bit of funny business in here, so i'll add comments to explain individual changes.  please review carefully!
i still need to test with sqlite better, so there might be some hiccups with that.
there are some risks associated with rolling out the contents of this pr to customers, because we're trusting that several of the newly-non-nullable columns are actually that way, i.e. they don't actually contain null values.  as far as i can see in our dbs, this is the case, but it's hard to be sure for customers since postgres did not enforce these constraints thus far.

with @neutralino1's help, i tried testing this against local sqlite... and we have a problem.  basically all of the ALTER TABLE statements are unsupported by sqlite (the same is true for the indexes, but those can at least be dropped and recreated fairly easily).  which means we can't actually get to parity unless we do the ugly table rename, create the new schema, and then copy the data from old into new.  that also means we'll need to add some way to detect that the db is sqlite in our current migration framework.
i'm not sure i see any path forward other than biting the bullet and doing the above.  @tscurtu @augray @neutralino1?

there are some risks associated with rolling out the contents of this pr to customers, because we're trusting that several of the newly-non-nullable columns are actually that way, i.e. they don't actually contain null values. as far as i can see in our dbs, this is the case, but it's hard to be sure for customers since postgres did not enforce these constraints thus far.

Our first on-prem customer quite likely has some old rows for which some of those columns actually do contain nulls. Our own dev DBs have all been re-created from scratch more recently than their DB. So perhaps we need to have part of the migration script populate the rows with nulls using default values.


Fails to load value of custom property to int attribute

Definition:
<cf:list formId="web-icon" rowId="1">
    <cf:setFieldAsCustomProperty fieldName="fileId" type="file" />
    <img:favicon fileId="cf:custom" />
    <web:getProperty name="cf:custom" />
</cf:list>

Error:
09:23:58	WARNING: [512] Missing required attribute! [fileId]. On line 499 in file ..../app/scripts/php/libs/DefaultPhp.class.php.

Attributes set from custom property lacks type checking.
If this will be problem, we should wrap evaluated attribute value to some function which will do the check.


Optimization generates style warnings by @media print queries and removes style definitions.

I've implemented a website that has multiple themes. It is possible to switch between the themes without reloading the app. For this each theme has an own style file where the 3rd party dependencies are imported wrapped by a class to use their SCSS variables and to style the elements (e.g. the buttons) without overwriting each statement of the 3rd party style files where a color is already set. In addition, there are specific styles of the website divided in several files (base.scss, print.scss, ...). All of them are imported in a single file named portal.scss that is included in the angunar.json. The imports are wrapped by a style statement to create a higher priority of the website styles than the styles of the 3rd party libraries without using !important.
Everything works fine using ng serve (no errors, no warnings). But building or serving the app with --prod generates several warnings that a unexpected bracket is at line xyz.
I've created a small example to reproduce the behaviour. You can check out the example and run it:
When the optimization flag is disabled ("optimization": false) in the angular.json the app starts without any warning. I opened the non-optimized style file and jumped to the lines where an unexpected bracket should be. In each case there is a closing bracket of an @media print statement.
🌍  Your Environment
Angular Version:

     _                      _                 ____ _     ___
    / \   _ __   __ _ _   _| | __ _ _ __     / ___| |   |_ _|
   / △ \ | '_ \ / _` | | | | |/ _` | '__|   | |   | |    | |
  / ___ \| | | | (_| | |_| | | (_| | |      | |___| |___ | |
 /_/   \_\_| |_|\__, |\__,_|_|\__,_|_|       \____|_____|___|
                |___/

Angular CLI: 7.3.9
Node: 10.15.3
OS: darwin x64
Angular: 7.2.15
... animations, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.13.9
@angular-devkit/build-angular     0.13.9
@angular-devkit/build-optimizer   0.13.9
@angular-devkit/build-webpack     0.13.9
@angular-devkit/core              7.3.9
@angular-devkit/schematics        7.3.9
@angular/cli                      7.3.9
@ngtools/webpack                  7.3.9
@schematics/angular               7.3.9
@schematics/update                0.13.9
rxjs                              6.3.3
typescript                        3.2.4
webpack                           4.29.0

Thank you very much. 👏 Since I moved the @page part outside my part works. :-)
Now, I have to check bootstrap and other 3rd party dependencies because there are warnings, too.


Unexpected Keyboard Interupt

Bug Description
The program exit on its own without click Ctrl + C.
Expected behavior
Keep running
Steps to Reproduce
hyperglass start
https://gist.github.com/YuetAu/416e0522bd70d798649e220f7795fbb4
Local Configurations
hyperglass.yaml
devices.yaml
commands.yaml is blank
Logs
https://gist.github.com/YuetAu/416e0522bd70d798649e220f7795fbb4
Possible Solution
I have tried edit the code and found out it exited because of SystemExit instead of KeyboardInterupt. But I can't extract why or which bit of code causes the error.
Environment
Server

Metric
Value

hyperglass Version
1.0.0-beta.76

hyperglass Path
/etc/hyperglass

Python Version
3.6.8

Platform Info
Linux-5.10.13-1.el7.elrepo.x86_64-x86_64-with-centos-7.9.2009-Core

CPU Info
Intel(R) Xeon(R) CPU E5-2690 0 @ 2.90GHz

Logical Cores
8

Physical Cores
8

Processor Speed
2.9GHz

Total Memory
6.23 GB

Memory Utilization
89.6%

Total Disk Space
102.69 GB

Disk Utilization
23.1%

I had this problem, i fix it rebooting the machine. I don't know what causes it, but i think maybe be an error with multiple active sessions.

Hey there,
I'm not sure I fully understand the issue - are you saying you get the Stopping hyperglass due to keyboard interrupt. error out of nowhere, even if you did not issue a keyboard interrupt (e.g. CtrlC)? If so, that's quite strange. The only thing I can think of is that Gunicorn does raise a SystemExit exception when it encounters an error.
I'm working on a beta.77 release tonight to address some random bugs, I'm throwing in an extra logging statement to print the exception raised during hyperglass start before re-raising. Once it's up, give that a try and see if you get any more information about the cause.

Yes, the problem is as mention and rebooting fix the problem. I am sorry I didn't do that before submitting a bug report.


"Commercial Marketplace portal" link in the first section doesn't work, because there is a trailing period

The "Commercial Marketplace portal" link in the first section doesn't work, because there is a trailing period inside the markdown for the url, and so it redirects to bing.com.

You can view your offers in the Commercial Marketplace portal in Partner Center.

This link works- https://go.microsoft.com/fwlink/?linkid=2165935
This link does not- https://go.microsoft.com/fwlink/?linkid=2165935.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: bd1fa2c6-830a-51d2-41d8-8e71ff701a7b
Version Independent ID: 9914c993-6d1e-1c2d-67c8-80ac349f6cfa
Content: Update an existing offer - Microsoft commercial marketplace
Content Source: articles/marketplace/update-existing-offer.md
Service: marketplace
Sub-service: partnercenter-marketplace-publisher
GitHub Login: @trkeya
Microsoft Alias: trkeya

Thanks for bringing this to our attention. We will investigate it further and update you shortly.

@alexsmithMSFT - Thanks again for bringing this to our attention. We have updated the Doc and it will be live once its merged.  We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


please add the English change log (deepl)

Please, it helps. There are very few Chinese people here
By the way, the English readme is good

Added. Thanks for the suggestion.


Toyota: 2021+ RAV4 Prime

Minor support pieces only, bulk of the work is in opendbc.

[ ] commaai/opendbc#1287
[ ] commaai/panda#2042
[x] Suppress from CARS.md and shop

Additional work that's teed-up for after this PR:
jyoung8607/opendbc#8 (more RAV4 Prime fingerprints)
jyoung8607/opendbc#9 (4th gen Toyota Sienna with SecOC)

Additional work that's teed-up for after this PR:
jyoung8607/opendbc#8 (more RAV4 Prime fingerprints) jyoung8607/opendbc#9 (4th gen Toyota Sienna with SecOC)

@jyoung8607
is there any update on the Toyota Sienna 21+
if you need help a contract will help you

on Github -Toyota: 2021-23 RAV4 Prime it only described REV4 not sienna
However, I already tried and it's not working on Sienna.
Issue


Vendor gorilla/mux using gvt, not godep

I was having problems using multiple tools (godep and gvt) when using this in Kurma. So, I decided it was easier just to fix everything up. Plus, nobody likes rewriting import paths.
@wallyqs @krobertson @mbhinder

Also running Continuum-side unit tests just to verify.

Newer version of gorilla? LGTM on green on Continuum side.

Also, yeah, newer version of Gorilla. But this package is used only in unit tests, so I wasn't too concerned about that. Unit tests are green Continuum-side.


chore Improve error msg about fe name format

run buildall

_ No description provided. _

decription add a msg example maybe better


more card actions

[x] MoveToList
[x] MoveToPos
[x] Archive
[x] SendToBoard

LGTM other than the minor questions/comments. Can you answer them pls? Thanks for your contribution!

Please ignore the failing tests. They are fixed in another PR.
Also, @VojtechVitek, let me set the API key and token in the Travis repository, or perhaps you can set them yourself, with your account, by just joining the board: https://trello.com/b/iZVEfBeQ/go-trello-test-board
It's very annoying, but I am writing tests considering that the Board will be left the same way it was before the test suite was run. For example, if during the test you add a card, on a subsequent test case you have to delete that same card. I can't think of a better way to do it. @Luzifer what do you think of this approach?
It's dangerous that someone can create a test that doesn't meet this criteria and submit a PR, the test will be run by Travis and the board will be messed up.

@fiatjaf can't we create new board for each test?

That's a good idea, but I think this would configure an abusive usage, since Trello boards can't be deleted.


fix: transient errors for s3 artifacts: Fixes #7349

When connection problems occur while loading artifacts, the workflow init step exits and the workflow fails. This is a regression.
We propose a fix in this PR for issue: #7349.
Fixes #7349
In this PR we add an extra check for Transient errors every time the S3 Transient errors are checked.
Don't bother creating a PR until you've done this:

[x] Run make pre-commit -B to fix codegen, lint, and commit message problems.

Create your PR as a draft.

Your PR needs to pass the required checks before it can be approved. If the check is not required (e.g. E2E tests) it
does not need to pass.
Once required tests have passed, you can make it "Ready for review".
Say how how you tested your changes. If you changed the UI, attach screenshots.

Tips:

If changes were requested, and you've made them, then dismiss the review to get it looked at again.
Add you organization to USERS.md if you like.
You can ask for help!

I found a problem in my local test setup. That is fixed now. Sorry for the many fix: codestyle commits. I'm going to squash the previous (codestyle) commits run the tests again and force push them again.

@terrytangyuan would you be willing to own reviewing this?

In case you want me to rebase onto master and force-push this to deal with the lint error, just let me know.

Yes please fix the lint errors

Hey all,
I have an issue currently with a s3 backend for workflow artifacts. If I run a workflow I'm able to view the pod logs for a few minutes before I get a 500 Internal Server Error. Is this related? Using Argo Workflows 3.2.4


godot 4.x : The parameter type of the OptionButton.ItemSelected has not changed, the complier needs long instead of integer.

Tested versions
godot 4.x
System information
windows 10
Issue description
I notice this issue of  The parameter type of the OptionButton.ItemSelected has not changed, the complier needs long instead of integer.  At least that's the case with C# in the documentation. and if there's no problem with the doc on GDScript, can you add a C#-relater type description
Steps to reproduce
OptionButton opButton = GetNode("...");
opButton.ItemSelected += OnItemSelected;
......
private void OnItemSelected(long Index)
{
// ......
}
Minimal reproduction project (MRP)
this may be an imperfection of the doc,and there should be no need for minimal replication projects.

Duplicate of https://github.com/godotengine/godot/issues/65311


Offset and limit parameters are switched around

https://github.com/ergoplatform/ergo-appkit/blob/580452799fb5d7dfeeb1bda28803042928f16a3a/lib-impl/src/main/java/org/ergoplatform/appkit/impl/NodeDataSourceImpl.java#L221

If you provide a PR we can merge it very fast :)

If you provide a PR we can merge it very fast :)

morphic already fixed it, using a snapshot with the fix

So this could get closed?


RFC: Add "process" labels

We currently use "status/5-cherry-pick" and "status/6-cherry-picked" labels in our workflow, but these labels aren't described anywhere and probably should not have a "status/" prefix, as they're not directly related to the review process.
During the recent evaluation of the 1.11 release, we discussed cases where PR's that were opened directly in the bump/release branch were not merged back to "master" (e.g. release notes), so I added some labels for that as well.
Finally, documentation updates can take place between releases, and require cherry-picking to the docs branch. Given that those cherry picks often aren't related to changes in the release, there's no need to cherry-pick them to the bump/release-branch, so to prevent confusion, I suggest to have a separate set of labels for those.
This PR is just to open the discussion, I'm not really tied to the naming used, or the location where these labels are described; suggestions welcome!

ping @docker/core-engine-maintainers @mlaventure @vieux

I don't have enough experience with the cherry-picking or merging back to master. I think the new process labels do make it clear the direction that the patch is going.

LGTM this will help quite alot in getting the docs team caught up on updates to the other products
@icecrime would you be the person to create the labels in the other repositories?

Design LGTM


Question about NCECriterion. py

https://github.com/HobbitLong/CMC/blob/58d06e9a82f7fea2e4af0a251726e9c6bf67c7c9/NCE/NCECriterion.py#L30
I think the purpose of using NCE is to avoid expensive summation over entire vector in softmax. But in your implementation, there is still summation over entire log_D0 which confused me. I'll appreciate it if you explain this.
I'm new to this field, and hope you point out my misunderstanding if there is.

@Abyssaledge , first the summation is outside the log, I guess you mean for softmax-ce, the summation is inside the log?
For the summation, see the m in Eq 11. in the paper

@HobbitLong  Thanks for your reply. I didn't make my point clearly.  What confused me is that why your implementation is more efficient than infoNCE in CPC?  I believe NCE tries to avoid summing all the negative samples  by using Monte Carlo, because summation over so many negative samples is expensive.
But your code still sums up all the negative samples:log_D0.view(-1, 1).sum(0), where log_D0 contains "similarity" with all the negatives smaples.
我的理解是NCE通过避免对所有负样本求和来降低计算量，但是log_D0这个tensor中元素的数量和负样本数一样，那对它求和不是就不能起到降级计算量的目的了吗？是不是我对NCE或者是计算量的理解有误？刚入坑CV，见笑了。。。

@HobbitLong  I seem to figure out why I'm confused. The k and N in Eq 1. and Eq 9. respectively in your paper mean the size of memory bank？ Does the m in Eq.11 mean the number of noise samples sampled from memory bank？ I might mixed them up.


gitea on MacOS X (homebrew build) unable to resolve tailscale *.ts.net names

Description
My Gitea instance on MacOS X (homebrew build) is unable to resolve tailscale *.ts.net names. I wanted to make internal hooks all reachable via the tailscale private net only, and allthough  I am able to curl the hook URL successfully on the same host, the gitea binary build is not able to resolve that address.
Gitea Version
1.17.3
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
Delivery: Post "https://myhost.tailXXXXX.ts.net/giteahook": dial tcp: lookup myhost.tailXXXXX.ts.net on <IP_ADDRESS>:53: no such host
Screenshots
Delivery: Post "https://myhost.tailXXXXX.ts.net/giteahook": dial tcp: lookup myhost.tailXXXXX.ts.net on <IP_ADDRESS>:53: no such host
The IP number is the IP number of my DSL router, but I have also tailscale with magic DNS running, and all other services appear to pick name resolution via the OS X native DNS resolver to map the tailscale network.
Git Version
2.24.3 (Apple Git-128)
Operating System
MacOS X Catalina
How are you running Gitea?
I am running the homebrew build of gitea as a server via a launchd plist file.
Database
PostgreSQL

Check if other golang applications can resolve, I imagine this is not an issue related to gite application code.

So my question here: would it be possible to make the build homebrew uses cgo or do I have to build it myself?

The build homebrew uses is built with CGO so that sqlite works, although netdns is overridden the tag netgo is set.
Not that it likely matters, but for debugging sake are you using an M1/2 device or an amd64 mac?

Good news, I reached out to tailscale and they provided some insight. This is due to an interaction between go and macOS, but will be fixed in go 1.20. Depending on your timelines you may need to build Gitea with the alternate options until that version of go is released. I'll close this ticket now as there is nothing we can do except wait for the new go version to release.
Source: https://twitter.com/bradfitz/status/1600364606022889473


Fix social link styling

What does this PR do and why?
Fixes regression to the Discord link.
Also Archive a project
Closes https://github.com/jbx-protocol/juice-interface/issues/1515
Closes https://github.com/jbx-protocol/juice-interface/issues/1517
Screenshots or screen recordings

before
after

Acceptance checklist

[x] I have evaluated the Approval Guidelines for this PR.
[x] I have tested this PR in all supported browsers.
[x] I have tested this PR in dark mode and light mode (if applicable).

@johnnyd-eth can you pls review

Clean LGTM


Signal Export Generates Portfolio Targets for Non-Tradable Securities

Expected Behavior
Generate PortfolioTarget for a portfolio with Index.
Actual Behavior
Since Index is not supported (because it's not tradable), we get the following message:
Index security type is not supported by Collective2. Allowed security types: [Equity,Forex,Option,Future,FutureOption,Crypto,CryptoFuture,Cfd,IndexOption]

Potential Solution
Add
if (!security.IsTradable) continue;

to GetPortfolioTargets
Reproducing the Problem
Add [TestCase(SecurityType.Index, "SPX", 68)] to SignalExportManagerGetsCorrectPortfolioTargetArray.
Checklist

[x] I have completely filled out this template
[x] I have confirmed that this issue exists on the current master branch
[x] I have confirmed that this is not a duplicate issue by searching issues
[x] I have provided detailed steps to reproduce the issue

Reopen since it has been observed with LEAN v<IP_ADDRESS>.16780:

Code to reproduce the issue: c7ebc1
(it«t not reproducible in backtest)


update ms.topic=overview for what-is-azure-powershell.md

Update metadata per https://dev.azure.com/ceapex/CPS/_workitems/edit/557382

docs-conceptual\azps-6.6.0\what-is-azure-powershell.md
docs-conceptual\azps-7.1.0\what-is-azure-powershell.md
docs-conceptual\azps-7.2.0\what-is-azure-powershell.md

Docs Build status updates of commit 8d3dee9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-conceptual/azps-6.6.0/what-is-azure-powershell.md
:white_check_mark:Succeeded
View (azps-6.6.0)

docs-conceptual/azps-7.1.0/what-is-azure-powershell.md
:white_check_mark:Succeeded
View (azps-7.1.0)

docs-conceptual/azps-7.2.0/what-is-azure-powershell.md
:white_check_mark:Succeeded
View (azps-7.2.0)

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


New approach to multiple GPUs

Adding the dual GPU mode was an extensive undertaking, and is downright funky to work with. That mode will be abolished, and instead, built-in support for multiple GPU Meter skins will be added, similar to the 4.0 update of SilverAzide's gadgets. Users will be able to go load these variants, configure the HWiNFO IDs, and be ready to go.
All settings in the GPU Meter settings skin will only affect the "GPU0" GPU Meter. The other three GPU Meters will have their own settings files, which have to be manually edited to change. Instructions on changing the variables will be included in those files.
To-do

[x] Delete dual GPU variant
[x] Move GPU Meter into "GPU0" subfolder
[x] Change path references in stylesheet
[x] Update settings system code to stop using meter groups instead of meter names
[x] Copy GPU Meter into "GPU1", "GPU2", and "GPU3" subfolders
[x] Copy GpuSettings.inc file into four distinct files
[x] Change variable file references for GPU Meter variants
[x] Add variant settings files to settings backup script

Done!


feat(ledger): Add ledgerService interface

Signed-off-by: DaevMithran<EMAIL_ADDRESS>Question

instead of indyLedgers. Can we change it to ledgers : { indy: [ ], cosmos: [ ] } .etc

Thanks for this PR @DaevMithran. Left a suggestion, as I think we can get this into 0.2.x (we would like to get it in before 0.3.0).
Also it seems the tests are failing. Could you take a look?

@TimoGlastra I beleive the failing test cases are due to the previous commits. I'll create another pull request for v0.2.3 then. I'll remove the breaking change

is there any update on this?


Dropdown source attribute

Warning: Failed prop type: Invalid prop label of type object supplied to Dropdown, expected string.
in Dropdown (created by ThemedDropdown)
in ThemedDropdown (created by ProfileInformationForm)
and i've already passed an array of objects with keys: value:string, label:string
even if pass the array of the example =>
countries = [     { value: 'EN-gb', label: 'England' },     { value: 'ES-es', label: 'Spain'},     { value: 'TH-th', label: 'Thailand', disabled: true },     { value: 'EN-en', label: 'USA'}   ]; it's the same story
Thank you

my bad... i solved the problem.


Fix Jest Tests

Fix Jest Tests
Summary: For some reason, the current jest command is running on /dist/ test files, which fails to parse the transpiled js. For example:
 FAIL  dist/__tests__/parsers/hf/hf.test.js
  HuggingFaceTextGeneration ModelParser
    ✕ uses HuggingFace API token from environment variable if it exists
    ✕ serializing params to config prompt
    ✕ serialize callbacks
    ✕ deserializing config prompt to params
    ✕ deserialize callbacks
    ✕ run prompt, non-streaming
    ✕ run prompt, streaming
    ✕ run callbacks

  ● HuggingFaceTextGeneration ModelParser › uses HuggingFace API token from environment variable if it exists

    TypeError: mockedHfInference.mockClear is not a function

      102 | describe("HuggingFaceTextGeneration ModelParser", () => {
      103 |     beforeEach(() => {
    > 104 |         mockedHfInference.mockClear();
          |                           ^
      105 |         mockGetApiKeyFromEnv.mockClear();
      106 |     });
      107 |     test("uses HuggingFace API token from environment variable if it exists", () => __awaiter(void 0, void 0, void 0, function* () {

      at Object.<anonymous> (dist/__tests__/parsers/hf/hf.test.js:104:27)

To fix, just explicitly tell jest to skip dist and node_modules tests.
Test Plan:
(aiconfig) ryanholinshead@Ryans-MacBook-Pro typescript % yarn test
yarn run v1.22.19
$ jest --runInBand
watchman warning:  Recrawled this watch 83 times, most recently because:
MustScanSubDirs UserDroppedTo resolve, please review the information on
https://facebook.github.io/watchman/docs/troubleshooting.html#recrawl
To clear this warning, run:
`watchman watch-del '/Users/ryanholinshead/Projects/aiconfig' ; watchman watch-project '/Users/ryanholinshead/Projects/aiconfig'`

(node:90548) [DEP0040] DeprecationWarning: The `punycode` module is deprecated. Please use a userland alternative instead.
(Use `node --trace-deprecation ...` to show where the warning was created)
 PASS  __tests__/parsers/hf/hf.test.ts
  HuggingFaceTextGeneration ModelParser
    ✓ uses HuggingFace API token from environment variable if it exists (9 ms)
    ✓ serializing params to config prompt (4 ms)
    ✓ serialize callbacks (1 ms)
    ✓ deserializing config prompt to params (2 ms)
    ✓ deserialize callbacks (1 ms)
    ✓ run prompt, non-streaming (3 ms)
    ✓ run prompt, streaming (2 ms)
    ✓ run callbacks (1 ms)

 PASS  __tests__/parsers/palm-text/palm.test.ts
  PaLM Text ModelParser
    ✓ serializing params to config prompt (2 ms)
    ✓ deserializing params to config (1 ms)
    ✓ run prompt, non-streaming (80 ms)

 PASS  __tests__/config.test.ts
  Loading an AIConfig
    ✓ loading a basic chatgpt query config (2 ms)
    ✓ loading a prompt chain (2 ms)
    ✓ deserialize and re-serialize a prompt chain (1 ms)
    ✓ serialize a prompt chain with different settings (1 ms)

 PASS  __tests__/testProgramaticallyCreateConfig.ts
  test Get Global Settings
    ✓ Retrieving global setting from AIConfig with 1 model
  ExtractOverrideSettings function
    ✓ Should return initial settings when no global settings are defined
    ✓ Should return an override when initial settings differ from global settings
    ✓ Should return empty override when global settings match initial settings
    ✓ Should return empty override when Global settings defined and initial settings are empty

 PASS  __tests__/testSave.ts
  AIConfigRuntime save()
    ✓ saves the config and checks if the config json doesn't have key filePath (3 ms)

Test Suites: 5 passed, 5 total
Tests:       21 passed, 21 total
Snapshots:   0 total
Time:        2.069 s
Ran all test suites.

Oh sick, thanks a lot!


Operation timed out after 10000 milliseconds with 0 bytes received

Hello Pytesttest team,
Thanks for your help!
The failure symptom is that "Operation timed out after 10000 milliseconds with 0 bytes received". Is there any way to wait for the response from server?
+++++++++++++++LOG++++++++++++++
ERROR:Test Failed: Create new domain after getting the access token URL=thisIsUrl Group=Account HTTP Status Code: None
ERROR:Test Failure, failure type: Curl Exception, Reason: Curl Exception: (28, 'Operation timed out after 10000 milliseconds with 0 bytes received')
ERROR:Validator/Error details:Traceback (most recent call last):
File "/usr/lib/python2.6/site-packages/pyresttest/resttest.py", line 351, in run_test
curl.perform()  # Run the actual call
error: (28, 'Operation timed out after 10000 milliseconds with 0 bytes received')
+++++++++++END OF LOG+++++++++++
Thanks,
Danh

@danghoaidanh, if the actual call takes more time than the default timeout,
you can increase the timeout value as follows:

config:

testset: "Quickstart app tests"
timeout: 600000

I hope this is what you were looking for!!
Regards,
Pradeep. S
On Wed, 23 Mar 2016 at 09:19 danghoaidanh<EMAIL_ADDRESS>wrote:

Hello Pytesttest team,
Thanks for your help!
The failure symptom is that "Operation timed out after 10000 milliseconds
with 0 bytes received". Is there any way to wait for the response from
server?
+++++++++++++++LOG++++++++++++++
ERROR:Test Failed: Create new domain after getting the access token
URL=thisIsUrl Group=Account HTTP Status Code: None
ERROR:Test Failure, failure type: Curl Exception, Reason: Curl Exception:
(28, 'Operation timed out after 10000 milliseconds with 0 bytes received')
ERROR:Validator/Error details:Traceback (most recent call last):
File "/usr/lib/python2.6/site-packages/pyresttest/resttest.py", line 351,
in run_test
curl.perform() # Run the actual call
error: (28, 'Operation timed out after 10000 milliseconds with 0 bytes
received')
+++++++++++END OF LOG+++++++++++
Thanks,
Danh
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/svanoort/pyresttest/issues/181

It works thanks!

Thank you, @spradeepv !


Fix TypeScript libdef; add CI script to validate

Adds a check-tsd script to validate the TypeScript libdef
Adds check-tsd to Travis
Fixes related type errors

Fixes #63

Pull Request Test Coverage Report for Build 140

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 139:
0.0%

Covered Lines:
362

Relevant Lines:
362

💛  - Coveralls


ec2_on_demand

Hey,
The ec2_on_demand_standard_count metric gives me back the number of instances I have, and the awsquota_ec2_on_demand_standard_count_limit give me back the number of CPUs I can use.  I would like to get the number of CPUs I'm using.

+1 to the above issue


WIP - Simplify add new locale

GUS: https://gus.lightning.force.com/lightning/r/ADM_Work__c/a07EE000008uxKNYAY/view
Description
This is an attempt to simplify the process for adding new locales to the PWA Kit as the existing process requires 4 steps (including running 2 separate CLI commands in extract-messages and compile-messages).
An attempt was made to remove the need to maintain a separate list of default messages in the _app component for the locale selector. however, it does not appear to be removable since Babel requires a statically analyzable object to be passed into defineMessages (more context is available in this stackoverflow).
Types of Changes

[ ] Bug fix (non-breaking change that fixes an issue)
[ ] New feature (non-breaking change that adds functionality)
[X] Documentation update
[ ] Breaking change (could cause existing functionality to not work as expected)

Breaking changes include:

Removing a public function or component or prop
Adding a required argument to a function
Changing the data type of a function parameter or return value
Adding a new peer dependency to package.json

Changes

Add defaultMessages for all locales supported by RefArchGlobal
Combine extract-messages and compile-messages into a single command

How to Test-Drive This PR

(step1)

Checklists

General

[ ] Changes are covered by test cases
[ ] CHANGELOG.md updated with a short description of changes (not required for documentation updates)

Accessibility Compliance
You must check off all items in one of the follow two lists:

[X] There are no changes to UI

or...

[ ] Changes were tested with a Screen Reader (iOS VoiceOver or Android Talkback) and had no issues
[ ] Changes comply with WCAG 2.0 guidelines levels A and AA
[ ] Changes to common UI patterns and interactions comply with WAI-ARIA best practices

Localization

[ ] Changes include a UI text update in the Retail React App (which requires translation)

Closing this in favor of https://github.com/SalesforceCommerceCloud/pwa-kit/pull/160


Bad variable name leading code example to fail.

Array destructuring [fadeAdmin] instead of [fadeAnim] in the SnackPlayer code example.
Renaming to fadeAnim in the example.

closing because of dupe


GD-577: Parameterized tests not counted in total count

The used GdUnit4 version
4.3.4 (Latest Release)
The used Godot version
v4.3.stable
Operating System
Windows 10.0.22631 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3060 Laptop GPU (NVIDIA; <IP_ADDRESS>99) - AMD Ryzen 7 5800H with Radeon Graphics (16 Threads)
Describe the bug
Note: I'm using C#, haven't tested to see if this bug also happens in GDScript
Parameterized tests are not included into "total_count", causing the progress bar value to be a bit misleading. The fix can either be to only show total test value (i.e. 34:34), only show total test + parameterized (i.e. 85:85), or should parameterized tests in a separate section (i.e. 34:34 (85:85)).

Sidenote, I think the progress bar delimiter should be changed from ":" to "/", it represents progress much more clearly. Just my opinion, though.

Steps to Reproduce

Create parameterized tests in C#.
Run tests.
See progress bar.

Minimal reproduction project
No response

Hi, can you verify this issue with the latest release v4.4.0 (master branch), I did some refactorings in this area and this issue should be resolved with v4.4.0

Tested with master branch, using gdUnit4.api v.4.3.1, and ran into a different issue where only GDScript tests were being counted, probably because I was using the 4.3.1 C# api since I couldn't find the newer version on nuget. If you release an rc version I can check again with that.

But I did confirm parameterized tests in GDScript doesn't have this issue, in both 4.4.0 and 4.3.1.

Ok seams we have an issue here, thanks for reporting.
With GdUnit4.4 I introduced the flaky test handling and changed the way how to communicate counters to the status bar.
I need to re-investigate the changes to find a solution it works for GDScript and existing C# API.

@poohcom1 if you develop your C# using external IDE, I suggest installing the gdunit4-test-adaper.
This allows you to direct develop and debug tests in your IDE.
check out https://github.com/MikeSchulze/gdUnit4Net/blob/master/testadapter/README.md


⛳ swift for tensorflow example

Thanks @johndpope. Just checking you haven’t missed the build setting docs in
https://github.com/yonaskolb/XcodeGen/blob/master/Docs/Usage.md and
https://github.com/yonaskolb/XcodeGen/blob/master/Docs/ProjectSpec.md


Problems with v1.7.0-dev1

Version of the custom_component

Version v1.7.0-dev1
Describe the bug
The start and stop of charging is sometimes not done.
Suspecting problems with the debounce implementation in #129

Seems like the problems I had with v1.7.0-dev1 was related to my charger. Have fixed my charger issue, and have not been able to recreate the issue I had before.

OK, now I have been able to recreate the issue. In some cases, the charger is not turned off when the target SOC is reached.

This issue should now be fixed by #136  and is included in v1.7.0.


Unable to customise theme breakpoints

[x] The issue is present in the latest release.
[ ] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

No longer able to customise theme breakpoints.
Expected Behavior 🤔

Be able to import the BreakpointOptions type from core and use it to pass in custom breakpoints to theme.
Steps to Reproduce 🕹
import { BreakpointsOptions } from "@material-ui/core/styles/createBreakpoints";

const getBreakpoints = (theme: CustomerUITheme): BreakpointsOptions => ({
    values: theme.breakpoints,
});

shows error Cannot find module '@material-ui/core/styles/createBreakpoints
there doesn't appear to be any error passing breakpoints into the createTheme function, and i believe createBreakpoints has moved to the system package.
Do we need to install @material-ui/system to be able to use the BreakpointsOptions type?

@robphoenix thanks for the report. Looks like I re-exported only the typings which were previously exported from @material-ui/core/styles. I would suggest we re-export these typings directly from here:
index 67019cb6e5..64c2eaf38d 100644
--- a/packages/material-ui/src/styles/index.d.ts
+++ b/packages/material-ui/src/styles/index.d.ts
@@ -35,6 +35,9 @@ export {
   Direction,
   Breakpoint,
   BreakpointOverrides,
+  BreakpointValues,
+  Breakpoints,
+  BreakpointsOptions,
   CreateMUIStyled,
   CSSObject,
 } from '@material-ui/system';

In this case you would be able to do:
-import { BreakpointsOptions } from "@material-ui/core/styles/createBreakpoints";
+import { BreakpointsOptions } from "@material-ui/core/styles";

 const getBreakpoints = (theme: CustomerUITheme): BreakpointsOptions => ({
     values: theme.breakpoints,
 });

@eps1lon @oliviertassinari should we still support exports from modules like: @material-ui/core/styles/createBreakpoints? I thought the longest path we support is three, for example till @material-ui/core/styles. Should I list all breaking changes related to these moved files?

should we still support exports from modules like: @material-ui/core/styles/createBreakpoints?

We can for internal usage. They just aren't useable for the public but we probably should make that happen as well.

We can for internal usage. They just aren't useable for the public but we probably should make that happen as well.

I thought that we should support up to level 3, otherwise we are making public the folder structure that we have internally, which with changes like this will break 😕 Also not supporting levels bigger than 3 will allow us to have private types as well.

The proposed patch looks great. I would only personally not make BreakpointValues public, and even inline it: for consistency with the other keys:
diff --git a/packages/material-ui-system/src/createTheme/createBreakpoints.d.ts b/packages/material-ui-system/src/createTheme/createBreakpoints.d.ts
index 6389985a74..3a9544acb7 100644
--- a/packages/material-ui-system/src/createTheme/createBreakpoints.d.ts
+++ b/packages/material-ui-system/src/createTheme/createBreakpoints.d.ts
@@ -6,12 +6,11 @@ export type Breakpoint = OverridableStringUnion<
   'xs' | 'sm' | 'md' | 'lg' | 'xl',
   BreakpointOverrides
 >;
-export type BreakpointValues = { [key in Breakpoint]: number };
 export const keys: Breakpoint[];

 export interface Breakpoints {
   keys: Breakpoint[];
-  values: BreakpointValues;
+  values: { [key in Breakpoint]: number };
   up: (key: Breakpoint | number) => string;
   down: (key: Breakpoint | number) => string;
   between: (start: Breakpoint | number, end: Breakpoint | number) => string;

Regarding the side questions. @material-ui/core/styles/createBreakpoints was never allowed. With #26254 we can enforce to make them unreachable.

I thought that we should support up to level 3, otherwise we are making public the folder structure that we have internally, which with changes like this will break confused

I wanted to say that we should make it public via public path. So re-export it from something at level 2 or 3. Not tell people to export from level >=4

I wanted to say that we should make it public via public path. So re-export it from something at level 2 or 3. Not tell people to export from level >=4

Got it, I misunderstood your comment.


Features for Team Bay Area

I will work on the features in this order.

[ ] Implement binary (debian) releases. Follow these steps: http://wiki.ros.org/bloom/Tutorials/FirstTimeRelease

[ ] unit and integration testing. I found a few unit tests and I think no integration tests. Integration tests could be done together with the data from https://ternaris.com/marv-robotics/demo/.
@amc-nu do you have couple of minutes to show me how much of tests you guys have and where they are?

[ ] check if ROS computational graph has chaged (e.g. if topic names have changed)
[ ] message definition has changed (should be warning when you start the system)
[ ] node crashes because the image has wrong encoding
[ ] YOLO2 https://github.com/CPFL/Autoware/tree/master/ros/src/computing/perception/detection/packages/cv_tracker/nodes,  Euclidean Clustering for LiDAR,
https://github.com/CPFL/Autoware/tree/master/ros/src/computing/perception/detection/packages/lidar_tracker/nodes, range_fusion => check functional performance against annotated set of data:
https://github.com/CPFL/Autoware/tree/master/ros/src/util/packages/kitti_pkg (publisher), dataset: http://www.cvlibs.net/datasets/kitti/raw_data.php

[ ] Integration of Gazebo 8.0.0
ROS kinetic uses Gazebo 7.x.x series. There have been significant advances in Gazebo 8.0 and OSRF also has a car simulation:

https://www.osrfoundation.org/simulated-car-demo/
https://discourse.ros.org/t/gazebo8-ros-lunar-wrappers-available/2328

[ ] Add more text to https://github.com/CPFL/Autoware-Manuals/blob/master/en/Autoware_UsersManual_v1.1.pdf

[ ] Porting to ROS 2.0
@amc-nu in which branch do you currently do porting to 2.0?

[ ] Use PClint and lint the code. We could use  http://google-styleguide.googlecode.com/svn/trunk/cpplint/  and http://clang.llvm.org/docs/ClangFormat.html

[ ] Docker - is there a docker for Autoware?

[ ] Allow storage of setting in run time manager. @ckirksey3 said he can work on that.

[ ] Fix CMakeLists files. See for instance https://github.com/CPFL/Autoware/pull/655/commits/ea10395f52f1ebeec9edfc64632d890405836f81. It needs QTCore dependency even though the package does not even use it.

@amc-nu could we quickly discuss later if these sound OK and if I should create sub-features in my own fork or here?

@bobjohnson2040: would you like to help too?


Some API additions in preparation for 0.3.0

This adds ohmd_status values for error handling, ohmd_string_description/ohmd_gets to get the library shaders, ohmd_list_geti and device class and flags, and the ohmd_control_hint/type that describe controller properties.

Commits looks great, merged


need reuse connection

I need to do benchmark for mqtt broker test. But when using netpoll client tcp connection to create mqtt connection, netpoll can't reuse same tcp connection to create multiple mqtt connection.
(mqtt lib: https://github.com/eclipse/paho.golang)
Using java netty can satisfy my need.
So can netpoll add this feature or is there another way to do this?

“can't reuse same tcp connection to create multiple mqtt connection.”
it depends on your usage. Are you using netpoll dialer API?

yes, i'm using client DialConnection api.
need eventloop is client side.

Netpoll dont control the reuse connection logic, it should reuse by yourself.


Replace hard-coded links/text with siteConfig variables

Is this a bug report?
No.
It's a PR proposal/RFC.
Have you read the Contributing Guidelines on issues?
Yes
Environment
(OS, Node, npm, yarn)
OS: Windows 8.1
Node: v6.10.3
npm: 3.10.10
Steps to Reproduce

Create new project utilizing docusaurus-init.
Set siteConfig properties such as copyright: or  baseUrl:.
Serve docs.
Check content of footer and help.html.
(alternatively) Look inside files such as website/pages/en/help.js or website/core/footer.js (among others).

Expected Behavior
Copyright inside footer and help link should be set to properties set in siteConfig.js.
Actual Behavior
Copyright inside footer and help link are set to hard-coded defaults inside website/pages/en/help.js and website/core/footer.js.
Proposal
Replace the hard-coded links and text blocks with variables accessing siteConfig (i e this.props.config.copyright) by default.
This way new users won't be confused when the properties exposed in siteConfig.js aren't set on their personal builds.

I think I would actually consider this on the border of feature request and bug 😄
Would you like to submit a PR for this?

@Happy-Ferret All right. I consider this issue claimed then 😄

Unless there's something I haven't considered (code style? Additional occurrences of hard-coded values one could easily get rid of?), I'll create the PR later today.

FYI. My current implementation replaces the editUrl inside users.html, too.
Figured we could just base this on siteConfig.repoUrl + /edit/master/website/siteConfig.js.
Although in the future it might make sense to modify the siteConfig API and retarget editUrl: so it can be applied to more than just docs (I'm thinking an array would be nice).


Support Kaia units

Proposed changes

Remove submodules for conformance-test and integration-test
Support kei, Gkei, KAIA units
-- Added functions : fromKei, toKei, convertFromKei, convertToKei

Types of changes
Please put an x in the boxes related to your change.

[ ] Bugfix
[x] New feature or enhancement
[ ] Others

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the CONTRIBUTING GUIDELINES doc
[x] I have signed the CLA
[ ] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Related issues

N/A

Further comments

N/A

I have read the CLA Document and I hereby sign the CLA


Standalone quote

Adding a single " causes the rest of the file to be highlighted as if I were creating a quote/string. This can happen when using the character as an abbreviation for "inches", as an example. The proper behavior is to treat an escaped \" as a single entity, not the start of a quote.

Thank you for your report, it is very much appreciated.
I will try to find a solution to this problem.
Please give me time to verify.
Thank you in advance.


feat: 2024 05 19/20 - [merged]

Merges feat-2024-05-19 -> development
#2 #7

mentioned in commit 2c81007c0ae73586fdaabe9387cc625486dee8f4

mentioned in commit 191244ed40f8feacf8885f0eb9fff5b32ef91252

added 10m of time spent

mentioned in commit 5273b58afb383843de6da3faca552fe143fff8eb

added 2 commits
2c81007c - ci(docker): always build on dev branch5273b58a - fix(ui): Ensure navigation menu entry highlighted for sub items
Compare with previous version

mentioned in commit e8ab3a0aa50e0c6e22ffd126c93c3ad39ef16fd5

mentioned in commit 7e3651f2b376fce2740f125c33df07354cbfcae2

added 2 commits
7afaa951 - chore: add link for kbbdf40d95 - chore: add link for playbook
Compare with previous version

mentioned in commit bdf40d952e7bb95b9da6e814292bfc11a2b1e82b

added 2h of time spent

added 2 commits
191244ed - refactor(template): content_title can be rendered in basee8ab3a0a - chore: add skeleton for information app
Compare with previous version

added 2 commits
c52fd080 - feat(api): API accept computer inventory7e3651f2 - feat(ui): Show inventory details if they exist
Compare with previous version

mentioned in commit 7afaa951d3b113396833bfa05380e382f8543bbf

added 1 commit
90de0cc8 - docs: add inventory details
Compare with previous version

mentioned in commit 6ac8e025b06906ca87afac30d0b94134d314166b

added 1 commit
6b5acc0d - docs: add inventory details
Compare with previous version

mentioned in commit c52fd0802ed2395fa3a62ec41ec0297b0bca373e

added 1 commit
6ac8e025 - test(api): add placeholder tests for inventory
Compare with previous version

added 1 commit
e93ce07d - feat(ui): Show inventory details if they exist
Compare with previous version

mentioned in commit 6b5acc0d575706d30a41d26f51dfe7d6f7bdf945

mentioned in commit e93ce07d887b3d3151cb6714df27fd5cf9fdcd67

mentioned in commit 90de0cc8c989c643eca0e71dfd7245ddb88e981c

mentioned in commit cad2bfe7daeb4fdecf6233d32911f7b83fe313fb

mentioned in commit dec2942996073746f78463b69156e67c8d879b72

mentioned in commit 725e6b8c922e0c1a6fd9d96a0f2581a8aad4a737

added 2 commits
fb041f77 - feat(device): Record inventory date and show as part of detailscad2bfe7 - chore(ui): add issue link for manufacturer and model
Compare with previous version

mentioned in commit fb041f77ebb9b9b44b0ebbed955252e22d3ee4dc

mentioned in commit 8e0df948d5006976981eb2ac3918c38fe74aff14

mentioned in commit 4562e921e99466785bd85d7b73652038becddd46

mentioned in commit 4d5f229fc737606bf272b0ccea6390ad2737dae1

mentioned in commit 5dadc3fe98e3317c83305b73c1f9763617f486a8

added 13h 33m of time spent

added 6 commits
8e0df948 - feat(core): notes model added to core725e6b8c - feat(itam): Ability to add notes on devices4d5f229f - feat(itam): Ability to add notes to operating systemsdec29429 - feat(itam): Ability to add notes to software4df25575 - chore(core): remove serial_number field from notes model4562e921 - test(core): place holder tests for notes model
Compare with previous version

added 14m of time spent

mentioned in commit 4df25575e8456a4f537f710d05a615482638c6b6

added 1 commit
5dadc3fe - docs(core): Add details about model notes
Compare with previous version

added 1 commit
7ae7ffae - fix: correct typo in notes templates
Compare with previous version

mentioned in commit 7ae7ffaef46cc27a2bae9acdcfa70dff80a05f7a

added 4m of time spent

mentioned in commit 5bf2e03c9f9b1fab805ef083a95d1df0a28ce948


Clear token if received blockwise message doesn't have one

[x] I confirm this contribution is my own and I agree to license it with Apache 2.0.
[x] I confirm the moderators may change the PR before merging it in.
Summary of changes
Clear token if received message doesn't have one. This fix prevents situation where the response is sending token even when it's not required anymore.

What about if the server is adding a token for the block2 GET?
Will we use the new token then, or will we still use the original token from subscription GET?


Create Dockerfile

Environment

Python version: N/A
Nautobot version: 1.0.0b3

Proposed Functionality
Dockerfile for deployments

Use Case
Container based deployment and development

Database Changes
None

External Dependencies
Docker

Should be assigned to @nniehoff

Can you please add some more context about the why here?

Is this different than #195?

It is the issue for 195.


global_goals problem

actions = torch.randn(num_scenes, 2)*6
cpu_actions = nn.Sigmoid()(actions).numpy()
global_goals = [[int(action[0] * local_w),
int(action[1] * local_h)]
for action in cpu_actions]
global_goals = [[min(x, int(local_w - 1)),
min(y, int(local_h - 1))]
for x, y in global_goals]
"""""""""""""""""""""""""""""""""""""""""""""""""""""""""""""
I saw this code in main.py, and I was confused. I understand that the global_goals variable should store the global goal, the decision made in the previous step. But the global_goals variable I see in main.py is all randomly selected. Why?

Hi, this part of code is only for initializing the global goal in the beginning.
To update the goal, please checkout line 727 of main.py.


getUserMedia with advanced constraints is broken

Browsers and versions affected
Chrome 55 (at least, not tested with others)
Description
When calling getUserMedia with advanced constraints consisting in an object with multiple properties (f.e. width and height) adapter.js converts it to optional constraints and it fails because it requires multiple properties to come in separate constraints.
Steps to reproduce
Run from the command line:
navigator.mediaDevices.getUserMedia({"audio":true,"video":{"advanced":[{"width":{"min":1280,"max":1280}, "height":{"min":720,"max":720}}]}}).then(console.log, console.log)
Expected results
MediaStream logged
Actual results
Error, invalid constraints
Workaround
Pass advanced constraints with multiple properties as separate constraints:
navigator.mediaDevices.getUserMedia({"audio":true,"video":{"advanced":[{"width":{"min":1280,"max":1280}}, {"height":{"min":720,"max":720}}]}}).then(console.log, console.log)

@jan-ivar  Do you know why are we moving advanced constrains to optional constrains?  It breaks my app if I use adapter.js because of that.      Do we really need that change?

paging @jan-ivar

@ggarber The bug here is adapter shouldn't be messing with constraints at all in Chrome 55. That polyfill may have been useful long ago in older versions.
I see a couple of PRs (https://github.com/webrtc/adapter/pull/336 and https://github.com/webrtc/adapter/pull/371 before it) have tried to remove it, but seem to both have run into test errors and probably distractions.
And not rhetorical: What are you using advanced constraints for?

Should I make a small PR in the mean time just to leave the advanced constraints untouched?
@jan-ivar We are using advanced constraints to set min/max&ideal resolution and framerate.   Shouldn't we be using them?  Any advice on the usage of them is more than welcomed.

I'm in the same boat here. Have users reporting a framerate of 3 when we are setting ideal to 12 (I have confirmed it). I believe it's this issue. Happy to help debug if necessary.
Nick

@ggarber Use new constraints:
video: {
  width: {min: 1024, ideal: 1280, max: 1920},
  height: {min: 776, ideal: 720, max: 1080}
}

@ngauthier Does it work without adapter?

@jan-ivar I haven't tried it without. I'll give it a shot when I'm back to work Tuesday. Thanks!

@jan-ivar We found Chrome does not support ideal, so we try first to get an exact value by setting {min, max} to the same thing (I guess Chrome didn't support exact either, this may have changed or maybe we just went along with min, max because they worked), then we ask for ideal in the next constraint set so that at least Firefox can try to get the best match, and Chrome ignores it but will fallback to whatever its default is.

@voltrevo Works fine for me in Chrome 55. Can we see your source?

@jan-ivar yeah I found yesterday that exact works now in chrome too. The important one that's missing is ideal.

@jan-ivar I'm on a bus atm but can make a snippet for you soon if you like.

@voltrevo Just remove the width constraint and you'll see ideal constraints work too (plain values are ideal by default).

@jan-ivar Chrome doesn't seem to treat them the same, e.g. https://jsfiddle.net/7yevnvu3/ - you get 640x480. But if you specify width: {exact: 1280} you get 1280x720.

@voltrevo I can reproduce it. I think you've found a Chrome bug! Please file a bug on Chrome. With adapter.js it wfm (1280x720), without it doesn't (640x480). Cc @alvestrand.

We're working on ideal, aiming for M58. At the moment we accept and ignore it.

Ok so maybe we shouldn't remove the constraints polyfill just yet then. @fippo FYI ^
I'll provide a PR here then.

@jan-ivar Yeah I probably should've done that earlier :-P, new Chrome bug here.

Chrome bug assigned and made dependent on "implementing the fix" bug.
Should we close this? I don't see an adapter action that can be taken.

@alvestrand The OP is a bug in the constraints polyfill: It doesn't support multiple constraints per entry in the advanced array. I can fix that, or we can yell at people using advanced. I'm happy to do the latter.

I saw that Chrome consider the advanced array option in "reverse" order, starting from the last.
Firefox, instead, works as per W3C Specification:

The ordering of advanced ConstraintSets is significant. In the preceding example it is impossible to satisfy both the 1920x1280 ConstraintSet and the 4x3 aspect ratio ConstraintSet ....

Is that a chrome bug? are you considering this difference?

Can you comment, @guidou?

Chrome 59 has support for spec-compliant video constraints.
For audio, standard syntax is accepted, but ideal is ignored and processing is not according to spec. We are aiming at adding spec-compliant audio constraints processing  in 60.

@ggarber is it ok if we close this as a dupe of #553? Send complaints my way if not :-)

I'm having issues with passing any constraints at all in chrome v 59.0.3071.125 on my S8. If any constraints are passed the browser crashes.
Works well on every other device and chrome version I have tried even v 59.0.3071.115 on a S6.
I can post my actual code if needed but the jsfiddle posted above by @jan-ivar crashes it as well.
Is this an open issue? or is there a work around? I have a project that goes live next week that I am relying on this to work.

@Hoopstr42 is the problem you had still an issue? If so, please file a chromium bug at http://crbug.com


Build Failed while Creating dist Files

Overview
Unable to build python package using python -m build
Steps to Reproduce

Install build if not already installed: python -m pip install --upgrade build
Run build commands as python -m build

Error Traceback
PS E:\Visual Crossing\VisualCrossing> python -m build
* Creating virtualenv isolated environment...
* Installing packages in isolated environment... (setuptools >= 40.8.0, wheel)
* Getting dependencies for sdist...
C:\Users\debma\AppData\Local\Temp\build-env-kcmhic5p\lib\site-packages\setuptools\dist.py:487: UserWarning: Normalizing '0.0.1-beta' to '0.0.1b0'
  warnings.warn(tmpl.format(**locals()))
running egg_info
creating VisualCrossing.egg-info
writing VisualCrossing.egg-info\PKG-INFO
writing dependency_links to VisualCrossing.egg-info\dependency_links.txt
writing top-level names to VisualCrossing.egg-info\top_level.txt
writing manifest file 'VisualCrossing.egg-info\SOURCES.txt'
reading manifest file 'VisualCrossing.egg-info\SOURCES.txt'
reading manifest template 'MANIFEST.in'
warning: manifest_maker: MANIFEST.in, line 1: unknown action 'inlude'

warning: manifest_maker: MANIFEST.in, line 7: unknown action 'requirements.txt'

adding license file 'LICENSE'
writing manifest file 'VisualCrossing.egg-info\SOURCES.txt'
* Building sdist...
C:\Users\debma\AppData\Local\Temp\build-env-kcmhic5p\lib\site-packages\setuptools\dist.py:487: UserWarning: Normalizing '0.0.1-beta' to '0.0.1b0'
  warnings.warn(tmpl.format(**locals()))
running sdist
running egg_info
writing VisualCrossing.egg-info\PKG-INFO
writing dependency_links to VisualCrossing.egg-info\dependency_links.txt
writing top-level names to VisualCrossing.egg-info\top_level.txt
reading manifest file 'VisualCrossing.egg-info\SOURCES.txt'
reading manifest template 'MANIFEST.in'
warning: manifest_maker: MANIFEST.in, line 1: unknown action 'inlude'

warning: manifest_maker: MANIFEST.in, line 7: unknown action 'requirements.txt'

adding license file 'LICENSE'
writing manifest file 'VisualCrossing.egg-info\SOURCES.txt'
running check
creating VisualCrossing-0.0.1b0
creating VisualCrossing-0.0.1b0\VisualCrossing
creating VisualCrossing-0.0.1b0\VisualCrossing.egg-info
copying files to VisualCrossing-0.0.1b0...
copying CHANGELOG.md -> VisualCrossing-0.0.1b0
copying LICENSE -> VisualCrossing-0.0.1b0
copying MANIFEST.in -> VisualCrossing-0.0.1b0
copying README.md -> VisualCrossing-0.0.1b0
copying setup.cfg -> VisualCrossing-0.0.1b0
copying setup.py -> VisualCrossing-0.0.1b0
copying VisualCrossing\__init__.py -> VisualCrossing-0.0.1b0\VisualCrossing
copying VisualCrossing\api.py -> VisualCrossing-0.0.1b0\VisualCrossing
copying VisualCrossing\errors.py -> VisualCrossing-0.0.1b0\VisualCrossing
copying VisualCrossing.egg-info\PKG-INFO -> VisualCrossing-0.0.1b0\VisualCrossing.egg-info
copying VisualCrossing.egg-info\SOURCES.txt -> VisualCrossing-0.0.1b0\VisualCrossing.egg-info
copying VisualCrossing.egg-info\dependency_links.txt -> VisualCrossing-0.0.1b0\VisualCrossing.egg-info
copying VisualCrossing.egg-info\top_level.txt -> VisualCrossing-0.0.1b0\VisualCrossing.egg-info
Writing VisualCrossing-0.0.1b0\setup.cfg
Creating tar archive
removing 'VisualCrossing-0.0.1b0' (and everything under it)
* Building wheel from sdist
* Creating virtualenv isolated environment...
* Installing packages in isolated environment... (setuptools >= 40.8.0, wheel)
* Getting dependencies for wheel...
Traceback (most recent call last):
  File "setup.py", line 20, in <module>
    VERSION = open(VERSIONFILE, "rt").read() # always read as str()
FileNotFoundError: [Errno 2] No such file or directory: 'static\\VERSION'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "D:\Anaconda3\lib\site-packages\pep517\in_process\_in_process.py", line 349, in <module>
    main()
  File "D:\Anaconda3\lib\site-packages\pep517\in_process\_in_process.py", line 331, in main
    json_out['return_val'] = hook(**hook_input['kwargs'])
  File "D:\Anaconda3\lib\site-packages\pep517\in_process\_in_process.py", line 117, in get_requires_for_build_wheel
    return hook(config_settings)
  File "C:\Users\debma\AppData\Local\Temp\build-env-b5i0_cm7\lib\site-packages\setuptools\build_meta.py", line 154, in get_requires_for_build_wheel
    return self._get_build_requires(
  File "C:\Users\debma\AppData\Local\Temp\build-env-b5i0_cm7\lib\site-packages\setuptools\build_meta.py", line 135, in _get_build_requires
    self.run_setup()
  File "C:\Users\debma\AppData\Local\Temp\build-env-b5i0_cm7\lib\site-packages\setuptools\build_meta.py", line 258, in run_setup
    super(_BuildMetaLegacyBackend,
  File "C:\Users\debma\AppData\Local\Temp\build-env-b5i0_cm7\lib\site-packages\setuptools\build_meta.py", line 150, in run_setup
    exec(compile(code, __file__, 'exec'), locals())
  File "setup.py", line 22, in <module>
    raise RuntimeError("unable to run setuptools")
RuntimeError: unable to run setuptools

ERROR Backend subproccess exited when trying to invoke get_requires_for_build_wheel

Context

[x] Fix MANIFEST.in : keyword include is missing at Line-12
[x] Understand why static/VERSION is not understood

Python Version : 3.8.3
Operating System : Microsoft Windows 10 Home Single Language (10.0.19043 N/A Build 19043)

Fixed MANIFEST.in and FileNotFoundError on 7f060b41f8b25bbc6a7c4ae8bab88c7db846bf3a..c8facd0304245c394301fef747878bb74149fb0b, but build failed as:
PS E:\Visual Crossing\VisualCrossing> python -m build
* Creating virtualenv isolated environment...
* Installing packages in isolated environment... (setuptools >= 40.8.0, wheel)
* Getting dependencies for sdist...
Traceback (most recent call last):
  File "D:\Anaconda3\lib\site-packages\pep517\in_process\_in_process.py", line 349, in <module>
    main()
  File "D:\Anaconda3\lib\site-packages\pep517\in_process\_in_process.py", line 331, in main
    json_out['return_val'] = hook(**hook_input['kwargs'])
  File "D:\Anaconda3\lib\site-packages\pep517\in_process\_in_process.py", line 284, in get_requires_for_build_sdist
    return hook(config_settings)
  File "C:\Users\debma\AppData\Local\Temp\build-env-3e62nveb\lib\site-packages\setuptools\build_meta.py", line 159, in get_requires_for_build_sdist
    return self._get_build_requires(config_settings, requirements=[])
  File "C:\Users\debma\AppData\Local\Temp\build-env-3e62nveb\lib\site-packages\setuptools\build_meta.py", line 135, in _get_build_requires
    self.run_setup()
  File "C:\Users\debma\AppData\Local\Temp\build-env-3e62nveb\lib\site-packages\setuptools\build_meta.py", line 258, in run_setup
    super(_BuildMetaLegacyBackend,
  File "C:\Users\debma\AppData\Local\Temp\build-env-3e62nveb\lib\site-packages\setuptools\build_meta.py", line 150, in run_setup
    exec(compile(code, __file__, 'exec'), locals())
  File "setup.py", line 17, in <module>
    from VisualCrossing import (
  File "E:\Visual Crossing\VisualCrossing\VisualCrossing\__init__.py", line 20, in <module>
    from .api import API
  File "E:\Visual Crossing\VisualCrossing\VisualCrossing\api.py", line 5, in <module>
    import requests
ModuleNotFoundError: No module named 'requests'

ERROR Backend subproccess exited when trying to invoke get_requires_for_build_sdist

Check:

[ ] is requests installed?
[ ] is correct environment selected?
[ ] does code works with direct-imports?

Using twine instead to publish your PyPI package as follows:
dPramanik@mint~:$ pip install twine
dPramanik@mint~:$ python setup.py sdist bdist_wheel # creates file in dist
dPramanik@mint~:$ twine check dist/* # check your package

Once you are satisfied with build and test you can now upload the package to PyPI. For this, first you can also test-upload, or directly publish with twine upload dist/*. More information is available here.
Note: Do check package version number, as lib\site-packages\setuptools\dist.py:458: UserWarning: Normalizing '0.0.1-beta' to '0.0.1b0', thus might create problem for documentation and  versioning (also check pypa/setuptools#308).


Specify differences between componentUpdated and update

IMHO the change still doesn't tell the difference between componentUpdated and update. Nevertheless, the whole idea may be better explained with a note instead of an inline, as it involves two hooks instead of only componentUpdated itself. I'd rewrite what Evan said in the channel into something like this:

Though pretty much similar, update and componentUpdated has a subtle difference: When update is called on an element, there’s no guarantee whether its children have also been updated. On the other hand, when componentUpdated is called, the children would have been updated as well. Hence, if your directive relies on its children being already updated, use componentUpdated instead of update.

It's a bit wordy, though. Maybe you guys can shorten it somehow.

@posva @phanan I just made some tweaks to try to simplify the language. What do you think?

it's better. I'll still add the tip that @phanan said

@posva Do you think the tip is still necessary with the bolding? I tried to capture the content of the tip there.

@chrisvfritz No, sorry, it's fine like that. If people find it unclear we can add the tip later on

Sounds good. 🙂  Thanks for making this happen!


Duplicate router created when using custom router

Describe the Bug
In the browser runtime, generouted unconditionally instantiates a browser router: https://github.com/oedotme/generouted/blob/3dde75f9af243b7c3bf256e5c5966bba3c7ef716/packages/generouted/src/react-router.tsx#L49
This causes event listeners (in particular for the history events) to be registered for this router, even if it is not ever used because the user is using the routes export directly to make their own router.
As a result, loader functions will be fired twice per browser history push/pop navigation (and probably in other instances as well). The router created by generouted is listening to events and responding by kicking off loader functions, even though it is never actually used.
Generouted Version
1.19.6
Your Example Website or App or Reproduction
https://stackblitz.com/edit/sb1-cllb2d?file=src%2Fpages%2Findex.tsx
Note that this app is using the routes (lowercase) export from @generouted/react-router to build its own BrowserRouter to use, and is not using the Routes export.
Steps to Reproduce the Bug or Issue

Navigate using a link
Navigate back using browser buttons
Observe that the loader count went up by 2 and that in the console, the loader fired twice in a row for the same navigation

Expected Behavior
The loader should only have been executed once (by the router created by the user)
Screenshots or Videos
No response
Platform

OS: macOS
Browser: Chrome 129, Firefox 131

Additional context
No response

@ethanwu10 Thanks for reporting this issue.
I just pushed a release v1.19.7 to run createBrowserRouter(...) internally only when using the <Routes /> export.
You can continue to use the routes export the same way to create a custom router with createBrowserRouter.
Thanks!


kerblam run needs not copy the dockerfile

kerblam run copies the dockerfile in /data/dockerfiles/ in the root of the repo as Dockerfile, then runs docker build, docker run, etc...
Docker supports an -f flag to specify which dockerfile to use.
The Executor could be changed to just accept an env: PathBuf (or similar) and just use the path as the dockerfile (instead of needing a FileMover).
Makes cleanup easier too.

Potentially fixed in deb18a37e9ad441af24c749069ee3518436dfa4a but needs additional checks.

I checked and this should be already fixed. Closing.


Added test for polygon from array of points

Coverage remained the same at 98.028% when pulling 404aadfe7bc9342d52fc6ea2b3122cc20c6de202 on dragosprotung:testSolidPolygonFromArrayOfPoints into f354743fc4b91d6f10e73127f45458a0f99c2efc on creof:master.

Coverage remained the same at 98.028% when pulling 404aadfe7bc9342d52fc6ea2b3122cc20c6de202 on dragosprotung:testSolidPolygonFromArrayOfPoints into f354743fc4b91d6f10e73127f45458a0f99c2efc on creof:master.

Coverage remained the same at 98.028% when pulling 404aadfe7bc9342d52fc6ea2b3122cc20c6de202 on dragosprotung:testSolidPolygonFromArrayOfPoints into f354743fc4b91d6f10e73127f45458a0f99c2efc on creof:master.

Coverage remained the same at 98.028% when pulling 404aadfe7bc9342d52fc6ea2b3122cc20c6de202 on dragosprotung:testSolidPolygonFromArrayOfPoints into f354743fc4b91d6f10e73127f45458a0f99c2efc on creof:master.


Fix a Teaching Tip Crash in unpackaged apps

Move Package::Current() to a try block

@StephenLPeters can you add the "Fixes #NNN" thing to the PR description?


preset: static has been broken since 0.3.5

open solid-start basic example.
change app.config to:

import { defineConfig } from "@solidjs/start/config";

export default defineConfig({
  ssr: true,
  server: {

    prerender: {
      crawlLinks: true
    },
    preset: 'static'
  }
});

build

┌───────────────────────────────────┐
│ ⚙  Preparing app for static...    │
└───────────────────────────────────┘
✔ Generated public .output/public                                                                                vinxi 1:31:02 AM
ℹ Initializing prerenderer                                                                                vinxi 1:31:02 AM
ℹ Building Nitro Server (preset: nitro-prerender)                                                                    vinxi 1:31:02 AM
✔ Nitro Server built                                                                                vinxi 1:31:02 AM
✔ You can preview this build using npx serve .output/public                                                          vinxi 1:31:02 AM

 ERROR  Should be provided by compiler                                                                                1:31:02 AM

  at .vinxi/prerender/index.mjs:2780:7
  at ModuleJob.run (node:internal/modules/esm/module_job:218:25)
  at async ModuleLoader.import (node:internal/modules/esm/loader:323:24)
  at async prerender (/Users/admin/repos/solid-start/node_modules/.pnpm/nitropack@2.9.1/node_modules/nitropack/dist/chunks/prerender.mjs:57:26)
  at async createBuild (/Users/admin/repos/solid-start/node_modules/.pnpm/vinxi@0.3.5_debug@4.3.4_preact@10.19.3/node_modules/vinxi/lib/build.js:311:2)
  at async Object.run (/Users/admin/repos/solid-start/node_modules/.pnpm/vinxi@0.3.5_debug@4.3.4_preact@10.19.3/node_modules/vinxi/bin/cli.mjs:225:5)
  at async runCommand (/Users/admin/repos/solid-start/node_modules/.pnpm/citty@0.1.6/node_modules/citty/dist/index.mjs:316:16)
  at async runCommand (/Users/admin/repos/solid-start/node_modules/.pnpm/citty@0.1.6/node_modules/citty/dist/index.mjs:307:11)
  at async runMain (/Users/admin/repos/solid-start/node_modules/.pnpm/citty@0.1.6/node_modules/citty/dist/index.mjs:445:7)

 ERROR  Should be provided by compiler                                                                                1:31:02 AM

 ELIFECYCLE  Command failed with exit code 1.

the build would complete without error up to and including vinxi 0.3.4

I tried this build with vinxi 0.3.8, works fine. Can you check again and see if the errors persists with 0.3.8

This kind of errors sometimes happens if there is multiple versions of vinxi in your project (maybe in the pnpm directory). But this looks like you are using vinxi 0.3.5, give 0.3.8 a try and get back to me

You're right, if I do clone an example with pnpm create solid, it works fine, so it's only if I try to run the examples directly from the solid-start repo, even if i bump vinxi both inside the example and in repo root.


fix: intrainingui checks appid

Necessary for multi-cl apps
Get bot info takes appId

:tada: This PR is included in version 0.275.8 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Avoid using the default system Locale and printing to System.out in production code

Avoid using the default system locale, use Locale.ENGLISH instead:

In String.format(), replace with StringUtils.safeFormat() in the whole codebase: 90% changes of this PR.
String.toLowerCase() and toUpperCase(), replace with StringUtils.toLowerCase() and toUpperCase() accordingly.
System.out.printf(), in test code
Misc

This is not just stylistic change, it may actually lead to errors if Druid-running servers have non-English locale, String.toLowerCase() Javadoc gives a scary example.

Don't use System.out, e.printStackTrace() and similar in production code, replace with logging. Except for a few classes in druid-services, like io.druid.cli.Main.
Replace new IOException(String.format(...)) with new IOE(...), and similarly with ISE, IAE and UOE where possible.

This PR doesn't enforce prohibition of using the default system locale and System.out, because I want to do that with this plugin: https://github.com/policeman-tools/forbidden-apis, and it also forbids using the default timezone, for what I want to create a separate PR.

Tests in error:

  QueryGranularityTest.testDeadLock:815 »  test timed out after 10000 millisecon...


Ported IPC wings from SimpleStation14

About the PR

Added wing marking for IPCs
Changelog

🆑

add: Added IPC wings

i wonder which character this is meant to be for

i wonder which character this is meant to be for

IPCs, no?

i wonder which character this is meant to be for

IPCs, no? Is the code saying otherwise? XD

V1 ultrakill


Uncaught SyntaxError: Failed to execute 'querySelectorAll' on 'Element': 'path.link[data-target=-...

[Enter steps to reproduce below:]

...
...

Atom Version: 1.0.3
System: Unknown Windows Version
Thrown From: atom-typescript package, v5.2.4
Stack Trace
Uncaught SyntaxError: Failed to execute 'querySelectorAll' on 'Element': 'path.link[data-target=------------TypeScriptCommon-autoGenerated-GeneratedApiServicesAndClasses-d-ts]' is not a valid selector.
At C:\Users\Mikhailiuk\.atom\packages\atom-typescript\node_modules\d3\d3.js:0

Error: Failed to execute 'querySelectorAll' on 'Element': 'path.link[data-target=------------TypeScriptCommon-autoGenerated-GeneratedApiServicesAndClasses-d-ts]' is not a valid selector.
    at Error (native)
    at d3_selectAll (C:\Users\Mikhailiuk\.atom\packages\atom-typescript\node_modules\d3\d3.js:564:14)
    at SVGGElement.<anonymous> (C:\Users\Mikhailiuk\.atom\packages\atom-typescript\node_modules\d3\d3.js:620:14)
    at Array.d3_selectionPrototype.selectAll (C:\Users\Mikhailiuk\.atom\packages\atom-typescript\node_modules\d3\d3.js:611:55)
    at SVGPathElement.<anonymous> (C:\Users\Mikhailiuk\.atom\packages\atom-typescript\dist\main\atom\views\dependencyView.js:247:43)
    at C:\Users\Mikhailiuk\.atom\packages\atom-typescript\node_modules\d3\d3.js:962:16
    at d3_selection_each (C:\Users\Mikhailiuk\.atom\packages\atom-typescript\node_modules\d3\d3.js:968:30)
    at Array.d3_selectionPrototype.each (C:\Users\Mikhailiuk\.atom\packages\atom-typescript\node_modules\d3\d3.js:961:12)
    at updateNodeTransparencies (C:\Users\Mikhailiuk\.atom\packages\atom-typescript\dist\main\atom\views\dependencyView.js:233:15)
    at onNodeMouseOver (C:\Users\Mikhailiuk\.atom\packages\atom-typescript\dist\main\atom\views\dependencyView.js:205:9)

Commands
     -2:13.1.0 core:move-down (atom-text-editor.editor.is-focused)
     -2:10.9.0 core:move-left (atom-text-editor.editor.is-focused)
     -2:10.4.0 core:move-right (atom-text-editor.editor.is-focused)
     -2:09.8.0 core:move-left (atom-text-editor.editor.is-focused)
 68x -2:09.3.0 core:move-right (atom-text-editor.editor.is-focused)
     -1:46.6.0 core:save (atom-text-editor.editor.is-focused)
     -1:46.5.0 linter:lint (atom-text-editor.editor.is-focused)
     -0:45.2.0 core:save (atom-text-editor.editor.is-focused)
     -0:45.2.0 linter:lint (atom-text-editor.editor.is-focused)
     -0:44.1.0 core:save (atom-text-editor.editor.is-focused)
     -0:44.1.0 linter:lint (atom-text-editor.editor.is-focused)
     -0:42.4.0 command-palette:toggle (atom-text-editor.editor.is-focused)
 14x -0:38.6.0 core:move-up (atom-text-editor.editor.mini.is-focused)
  5x -0:35.6.0 core:move-down (atom-text-editor.editor.mini.is-focused)
     -0:34.7.0 core:confirm (atom-text-editor.editor.mini.is-focused)
     -0:34.7.0 typescript:dependency-view (atom-text-editor.editor)

Config
{
  "core": {
    "ignoredNames": [
      "bower_components node_packages"
    ]
  },
  "atom-typescript": {}
}

Installed Packages
# User
.bin, vundefined
atom-typescript, v5.2.4
linter, v1.2.4

# Dev
No dev packages

@basarat, do you want any additional information?
Thank you!

do you want any additional information

Yes please. Quick test : Dependency view worked fine on the project I am working on

Can we have a repro :rose:

I'm sorry. I cant provide you my repo:(
But i observed my project and think that problem is the i have ts project which relates to another ts project in another dir. I catch this error only when i hover mouse pinter to node which represents module from another project. You can see that exception in selector which contains long relative paths.
if you want - i'll create any open repo
Thank you!

Hello, @basarat !
I created simple example which reproduce this issue.
unpack this zip then open  topDir/testProject/ folder in atom. navigate to app.tsx open typescript dependency view. Then mouse pointer hover to one from elements.
Thank you!

5.2.9
@stepancar thanks for the repro :rose: !


In empty state diagram elements are not displayed in variables tab

Describe the bug
While there is no input/output variables and we Click anywhere outside in the process without selecting any element clicking on the process dropdown does not show any elements present in the process until we click on any element itself
Steps to reproduce

Create a process diagram that has no input/output variables
Click anywhere outside in the process without selecting any element
Open the variables tab
click on the dropdown beside the process name
No elements in the diagram is displayed
Click on any element and now clicking on the dropdown displays the element beneath the process. This is not the case when there is input/output variables are available in the process

https://github.com/user-attachments/assets/1ea3ecf3-17e6-4703-8011-8202d10017ca
Expected behavior
Display elements in the process even in empty state on clicking the dropdown
Environment

OS: [MacOS 10.2]
Camunda Modeler Version: [5.27.0]
Execution Platform: [Camunda 8]

Additional context
No response

The way it's implemented right now is to display only the elements which read or write variables.

Does this notion change anything regarding your report?

Hey @barmac just to re-clarify what you mentioned above ,

display only the elements which read or write variables.

means there is no empty state if only when we have process variables added to the element . On that note the empty state msg makes sense. Thanks for clarifying !!

Thanks, I will then close the issue.


Compiling Tailwind with SASS, Gulp & package.json

I wasn't really sure where the best place to ask this question is as I'm working on customising this set-up which relates to a specific CMS but it's the Tailwind/Gulp part I'm having trouble with.
I've used Tailwind and Gulp before but this time I'm moving all the configuration into a package.json as described in the Craft scaffolding project link above. So I think it's in there that I'm going wrong.
What's happening is that the output CSS is including normalize.css at the top (which is expected, see the code below), and also correctly compiling my SASS code, but it's printing out
@tailwind preflight;
@tailwind components;

etc. instead of outputting the CSS.
So with a SCSS file that looks like
@charset "UTF-8";
@tailwind preflight;
@tailwind components;

 @import
  "components/mixins",
  …other scss files…
;
@tailwind utilities;

the CSS output looks like
/* Normalize.css code */

@tailwind preflight;
@tailwind components;

/* Compiled SCSS code */

@tailwind utilities;

My Tailwind file is exactly the same as https://github.com/nystudio107/craft/blob/master/tailwind.js and my Gulpfile is pretty much the same as https://github.com/nystudio107/craft/blob/master/gulpfile.js except that I've removed the Vue components, critical CSS, favicon and fontello functions which I'm not using.
My package.json file looks like
    {
        "paths": {
            "src": {
                "base": "./src/",
                "css": "",
                "img": "./src/img/",
                "js": "./src/js/",
                "scss": "./src/scss/"
            },
            "dist": {
                "base": "./public_html/",
                "css": "./public_html/assets/css/",
                "js": "./public_html/assets/js/",
                "fonts": "./public_html/assets/fonts/",
                "img": "./public_html/assets/img/"
            },
            "build": {
                "base": "./build/",
                "css": "./build/css/",
                "js": "./build/js/",
                "html": "./build/html/",
                "img": "./build/img/"
            },
            "tailwindcss": {
                "src": "./src/scss/master.scss", // not sure about this line
                "conf": "./tailwind.js"
            },
            "scss": [{
                "src": "./src/scss/master.scss" // or this one
            }],
            "templates": "./public_html/site/templates/"
        },
        "vars": {
            "siteCssName": "site.combined.min.css",
            "scssName": "master.scss", // or this one
            "cssName": "master.css" // or this one
        },
        "globs": {
            "distCss": [
                "./node_modules/normalize.css/normalize.css",
                "./build/css/*.css"
            ]
        },
    }

Everything else is pretty much the same as https://github.com/nystudio107/craft/blob/master/package.json The difference between my setup and the Craft scaffolding one is that I'm using src/scss/ instead of src/css/.
Edit: just realised that it's not completely compiling the SCSS code properly because any @apply or @screen statements are being printed as written.

I use this same setup. Did you remove dependencies & devDependencies from package.json? I believe when you fire ./nys-setup after the initial installation it goes in and adds all the necessary node modules, include tailwind.

It turns out the problem was this:
 "tailwindcss": {
     "src": "./src/scss/master.scss",
  }

should've been this:
 "tailwindcss": {
     "src": "./build/css/master.css",
  }

although even with that @tailwind components is still being printed out and the output isn't minifed. But at least the rest of the Tailwind styles are now being included.

Progress!


Add AddTab to publically exposed interface.

Resolves issue #2350, Just a single addition
Draft till I get the changes made to the changelog.

To explain the force push: I realized I hadnt rebased to main yet.


GraphExecutor: Fix wild pointer assign when input and output are reshape

If reshape in our relay expression, like reshape + matmul expr, when we use set_input_zero_copy or set_output_zero_copy，the result goes wrong.

Could you please add correspondent unit tests?

Could you please add correspondent unit tests?
ok

@echuraev , cc: @masahi
Hi, can we please merge this? current set_output_zero_copy doesn't work without these changes when we have a (reshape) nop at the end of a graph (it prevents to forward the pointer of the actual output area through the nop) :pray:
Thank you!

oh also @vinx13 was involved sorry, forgot to mention

@corehalt The PR needs to resolve the conflict. Feel free to take over this PR and create a new one.

Yes it's fine.


Neato Botvac D7 Latest Firmware (4.3.1) Causes Component Not To Setup

Home Assistant release with the issue:

All after this firmware update
Last working Home Assistant release (if known):
None after this firmware update
Operating environment (Hass.io/Docker/Windows/etc.):

All environments impacted
Component/platform:

neato:
https://www.home-assistant.io/components/neato
Description of problem:
After updating to firmware 4.3.1 HA will fail to load as neato has changed the cleaning service name.  This issue is a problem with the dependency and it is fixed in this PR: https://github.com/stianaske/pybotvac/pull/28
Once it is merged and a new version of the dependency is pushed to pypi I will submit a PR to correct this.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
neato:
  username: username
  password: password

Traceback (if applicable):
Error during setup of component neato
Traceback (most recent call last):
  File "/home/pi/home-assistant/homeassistant/setup.py", line 148, in _async_setup_component
    component.setup, hass, processed_config)  # type: ignore
  File "/usr/local/lib/python3.6/concurrent/futures/thread.py", line 56, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/home/pi/home-assistant/homeassistant/components/neato.py", line 154, in setup
    hass.data[NEATO_LOGIN] = NeatoHub(hass, config[DOMAIN], Account)
  File "/home/pi/home-assistant/homeassistant/components/neato.py", line 178, in __init__
    self._hass.data[NEATO_ROBOTS] = self.my_neato.robots
  File "/home/pi/home-assistant/lib/python3.6/site-packages/pybotvac/account.py", line 63, in robots
    self.refresh_robots()
  File "/home/pi/home-assistant/lib/python3.6/site-packages/pybotvac/account.py", line 111, in refresh_robots
    endpoint=robot['nucleo_url']))
  File "/home/pi/home-assistant/lib/python3.6/site-packages/pybotvac/robot.py", line 42, in __init__
    raise UnsupportedDevice("Version " + self.service_version + " of service houseCleaning is not known")
pybotvac.robot.UnsupportedDevice: Version basic-4 of service houseCleaning is not known

Additional information:

Same issue, same log. Just to confirm:
Error during setup of component neato Traceback (most recent call last):   File "/usr/local/lib/python3.6/site-packages/homeassistant/setup.py", line 148, in _async_setup_component     component.setup, hass, processed_config)  # type: ignore   File "/usr/local/lib/python3.6/concurrent/futures/thread.py", line 56, in run     result = self.fn(*self.args, **self.kwargs)   File "/usr/local/lib/python3.6/site-packages/homeassistant/components/neato.py", line 103, in setup     hass.data[NEATO_LOGIN] = NeatoHub(hass, config[DOMAIN], Account)   File "/usr/local/lib/python3.6/site-packages/homeassistant/components/neato.py", line 127, in __init__     self._hass.data[NEATO_ROBOTS] = self.my_neato.robots   File "/usr/local/lib/python3.6/site-packages/pybotvac/account.py", line 64, in robots     self.refresh_robots()   File "/usr/local/lib/python3.6/site-packages/pybotvac/account.py", line 112, in refresh_robots     endpoint=robot['nucleo_url']))   File "/usr/local/lib/python3.6/site-packages/pybotvac/robot.py", line 42, in __init__     raise UnsupportedDevice("Version " + self.service_version + " of service houseCleaning is not known") pybotvac.robot.UnsupportedDevice: Version basic-4 of service houseCleaning is not known

@rafuz so you don't get stuck, I have updated my custom component and personal repo with the D7 code and latest changes: https://github.com/dshokouhi/hass-neato-custom-component
to revert back you just need to remove the files and restart

Got my D7 yesterday and was curious about HASS integration, and I'm happy to say your extra commits did the trick for me!
Thank you!

Hi,
Im new to home assistant. Where can i add this code for Neato? Im using HA for docker/qnap.
B/rgs Mads

Hi mjuhldk,
Instructions are included with the github link
[(https://github.com/dshokouhi/hass-neato-custom-component)]
kind regards,
Ruben

Got this error:
Mon Dec 10 2018 12:08:04 GMT+0100 (Centraleuropæisk normaltid)
Error during setup of component neato
Traceback (most recent call last):
File "/usr/src/app/homeassistant/setup.py", line 148, in
_async_setup_component
component.setup, hass, processed_config)  # type: ignore
File "/usr/local/lib/python3.6/concurrent/futures/thread.py", line 56, in run
result = self.fn(*self.args, **self.kwargs)
File "/config/custom_components/neato.py", line 155, in setup
hass.data[NEATO_LOGIN] = NeatoHub(hass, config[DOMAIN], Account)
File "/config/custom_components/neato.py", line 179, in init
self._hass.data[NEATO_ROBOTS] = self.my_neato.robots
File "/config/deps/lib/python3.6/site-packages/pybotvac/account.py",
line 63, in robots
self.refresh_robots()
File "/config/deps/lib/python3.6/site-packages/pybotvac/account.py",
line 109, in refresh_robots
endpoint=robot['nucleo_url']))
File "/config/deps/lib/python3.6/site-packages/pybotvac/robot.py",
line 41, in init
if self.service_version not in SUPPORTED_SERVICES:
File "/config/deps/lib/python3.6/site-packages/pybotvac/robot.py",
line 213, in service_version
return self.available_services['houseCleaning']
File "/config/deps/lib/python3.6/site-packages/pybotvac/robot.py",
line 209, in available_services
return self.state['availableServices']
File "/config/deps/lib/python3.6/site-packages/pybotvac/robot.py",
line 205, in state
return self.get_robot_state().json()
File "/config/deps/lib/python3.6/site-packages/pybotvac/robot.py",
line 163, in get_robot_state
return self._message({'reqId': "1", 'cmd': "getRobotState"})
File "/config/deps/lib/python3.6/site-packages/pybotvac/robot.py",
line 60, in _message
response.raise_for_status()
File "/usr/local/lib/python3.6/site-packages/requests/models.py",
line 940, in raise_for_status
raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 404 Client Error: Not Found for url:
https://nucleo.neatocloud.com/vendors/neato/robots/OPS35518-30451178BEA2/messages
Den man. 10. dec. 2018 kl. 11.01 skrev rudeq<EMAIL_ADDRESS>
Hi mjuhldk,
Instructions are included with the github link
[(https://github.com/dshokouhi/hass-neato-custom-component)]
kind regards,
Ruben
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/issues/19044#issuecomment-445757742,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Arot-XyZUzo1Hk0Q-exomMK1gMU4f0Fgks5u3jDigaJpZM4ZDEPC
.

Hi.
A reset of the botvac fixed the issue.
Thanks
Regards mads
man. 10. dec. 2018 kl. 16.30 skrev Daniel Shokouhi<EMAIL_ADDRESS>
:

@mjuhldk https://github.com/mjuhldk that error usually means the botvac
is either not connected to wifi or it needs to be restarted. Can you check
the status of the botvac in the neato app and make sure you can still
control it? Restarting the botvac usually fixes that error.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/issues/19044#issuecomment-445856297,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Arot-Zx0iuN3ooCKrJRsUJBgDPEt_efKks5u3n4FgaJpZM4ZDEPC
.

When this fix will be in home assistant?

@rafuz we need pybotvac to release a new pypi package since the PR was merged, then I can submit a PR to bump the library.

Dear,
https://github.com/stianaske/pybotvac/commit/1d39638ae0c0081d266e7ad74e0bf89d9971442b
seems to be fixed.
Could you fix in ha?
Thanks