Do not emit cost data for azure virtual kubelets

Sample spec:
"metadata": {
"name": "virtual-kubelet-2",
"uid": "fake-id",
"resourceVersion": "164168545",
"creationTimestamp": "2021-07-20T11:59:43Z",
"labels": {
"alpha.service-controller.kubernetes.io/exclude-balancer": "true",
"beta.kubernetes.io/os": "linux",
"kubernetes.azure.com/managed": "false",
"kubernetes.azure.com/role": "agent",
"kubernetes.io/hostname": "virtual-kubelet-2",
"kubernetes.io/role": "agent",
"node-role.kubernetes.io/agent": "",
"node.kubernetes.io/exclude-from-external-load-balancers": "true",
"type": "virtual-kubelet"
}
"status": {
"capacity": {
"cpu": "10k",
"memory": "4Ti",
"pods": "5k"
},
"allocatable": {
"cpu": "10k",
"memory": "4Ti",
"pods": "5k"
} ```

The metrics exporter should just ignore these nodes when calculating cost

(related to Zendesk ticket #780)
gz#780

This was previously supported. Regression?

Yes, think so. I'll look into it.
On Mon Sep 13, 2021, 12:51 AM GMT, Webb Brown @.***>
wrote:
This was previously supported. Regression?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kubecost/cost-model/issues/932#issuecomment-917751050,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADOXCE7U2OQ3KXATXYYZG3UBVDIRANCNFSM5DYR2T6A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


prev(), next() and max-days aren't working in day view for qCalendarTasks!

Hi the prev() and next() method aren't working for the day view in qCalendarTasks also max-days,
You will find bellow the codepen for this issue.
Thank you!
https://codepen.io/saadeddineatif/pen/BaJKBNY

I will look into this as soon as I get a chance. Thanks for reporting! :heart:


Removing fs dependency as it is part of Node.js core

Apparently the 'fs' package is now part of the npm core, and is no longer required for import.
npm/npm#13743

Thanks @leog for your help :)
1 question - do you know since which version fs is part of npm core?
I asking because I'm thinking that maybe this change will require to jump to version 2.0.0 in order to also support npm 2.x.x (node 4 LTS) and not just npm 3.x.x (node 6 soon to be LTS)..
After we'll understand which version to set, I'll merge the PR and update the change log + npm publish..

I can see that fs is on npm core for a long time.. So it is OK just updating it to 1.0.2

Thanks for the help!

@kazazor just saw your comments here, different timezones as you may know...
That's correct, fs library was always part of Node.js core and there was a reference on the npm registry to it, the thing is that recently, it was removed from the registry so depending on fs explicitly from package.json will throw a not found error.


Add default icon to css

I'd like to add functionality to display default icon.
By default, defaultIcon is empty in the code.
So we can choose whether we display default icon or not.
I'm glad if you would consider to merge this functionality.
Thank you!

@SUKOHI Thanks for the PR. However, I do not quite understand why you would need this and how would you picture it to look like?

Hi, ratiw.
Thank you for quick reply.
That's because we can easily recognize which column is sortable without color like you did on codepen as an examples.
jQuery's dataTable actually has default icons.
Please see here if you don't mind.
You don't like it?

@SUKOHI Oh, I see. This is fine. Can you please re-submit the PR based on the develop branch instead?

I see. I created a new PR titled "Add default icon to css #2".
Thank you!


21 jackknife resampling

Can you update the tests?
On Thu, May 7, 2020 at 6:31 AM peterwhiteenjine<EMAIL_ADDRESS>wrote:

@peterwhiteenjine commented on this pull request.
In mcos/observation_simulator.py
https://github.com/enjine-com/mcos/pull/41#discussion_r421505449:

def init(self, mu: np.array, cov: np.array, n_observations: int):

   self.mu = mu

   self.cov = cov

   self.n_observations = n_observations

def simulate(self) -> (np.array, np.array):

   x = np.random.multivariate_normal(self.mu.flatten(), self.cov, size=self.n_observations)

   idx = np.arange(len(x))

   cov_hat = np.sum(np.cov(x[idx!=i], rowvar=False) for i in range(len(x)))/float(len(x) - 1)

True, there are still len(x) samples
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/enjine-com/mcos/pull/41#discussion_r421505449, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXTIXN6JOP4XQ2GS3RTMQ3RQKZ4NANCNFSM4MMWCYGQ
.


Update sqlalchemy requirement from >=1.3,<2.0 to >=1.3,<3.0 in /sdks/python

Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

R: @damccorm

Could you try pulling in the latest master? I'm seeing testPy38tft-113 failures from pip check which I think should be fixed on master.
tensorflow 2.13.1 has requirement typing-extensions<4.6.0,>=3.6.6, but you have typing-extensions 4.11.0.

Could you try pulling in the latest master? I'm seeing testPy38tft-113 failures from pip check which I think should be fixed on master.
tensorflow 2.13.1 has requirement typing-extensions<4.6.0,>=3.6.6, but you have typing-extensions 4.11.0.

Hey, I am seeing this failures, even after merging from the master. Not sure, what the issue is here.

@Dippatel98 merging

Fixes #26333


Support hfh rc version

otherwise the code doesn't work for hfh 0.11.0rc0
following #5237

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


dioxus-cli: 0.6.0 -> 0.6.1

v0.6.1 added auto-downloading of wasm-bindgen to the CLI, hence the new skipped tests.
dx now installs wasm-bindgen to {data_local_dir}/dioxus/wasm-bindgen/wasm-bindgen-{version}. Assuming you symlink the correct version of wasm-bindgen to this location, everything still works fine.
I'll look into wrapping/patching dx to make this more seamless.
Since it uses dirs::data_local_dir, we can't easily wrap on MacOS without overriding $HOME, which isn't ideal. Dioxus also supports reading global config from {data_local_dir}, so overriding it would break some functionality.
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[x] sandbox = true

[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

We could patch the lookup directory easily:
--- a/src/wasm_bindgen.rs
+++ b/src/wasm_bindgen.rs
@@ -233,10 +233,15 @@ impl WasmBindgen {

     /// Get the installation directory for the wasm-bindgen executable.
     async fn install_dir() -> anyhow::Result<PathBuf> {
-        let bindgen_dir = dirs::data_local_dir()
-            .expect("user should be running on a compatible operating system")
-            .join("dioxus/wasm-bindgen/");
+        let tool_dir = if let Ok(dir) = std::env::var("DIOXUS_TOOL_DIR") {
+            PathBuf::from(dir)
+        } else {
+            dirs::data_local_dir()
+                .expect("user should be running on a compatible operating system")
+                .join("dioxus/")
+        };

+        let bindgen_dir = tool_dir.join("wasm-bindgen/");
         fs::create_dir_all(&bindgen_dir).await?;

         Ok(bindgen_dir)

postFixup = ''
  mkdir -p $out/libexec/wasm-bindgen
  ln -s ${wasm-bindgen-cli}/bin/wasm-bindgen $out/libexec/wasm-bindgen/wasm-bindgen-${wasm-bindgen-cli.version}

  wrapProgram $out/bin/dx \
    --set DIOXUS_TOOL_DIR "$out/libexec"
'';

Only issue is that the error messages aren't very helpful in the event you have a version mismatch:
> dx serve --example hello_world --platform web
12:59:23 [dev] Failed to verify tooling: I/O Error: No such file or directory (os error 2)
dx will proceed, but you might run into errors later.
12:59:23 [dev] Build failed: Other(Failed to write main executable

Caused by:
    0: Failed to generate wasm-bindgen bindings
    1: No such file or directory (os error 2)

could we have the version, hash, and cargoHash be parameters that we can override, could just be me but I'm having issues using overrideAttrs

for what its worth I've got the wrapping of wasm-bindgen-cli working like this on my end
wasm-bindgen-cli = pkgs.wasm-bindgen-cli.override {
  version = "0.2.99";
  hash = "sha256-1AN2E9t/lZhbXdVznhTcniy+7ZzlaEp/gwLEAucs6EA=";
  cargoHash = "sha256-DbwAh8RJtW38LJp+J9Ht8fAROK9OabaJ85D9C/Vkve4=";
};
dioxus-cli = pkgs.dioxus-cli.overrideAttrs (drv: rec {
  version = "0.6.1";
  src = pkgs.fetchCrate {
    inherit version;
    pname = drv.pname;
    hash = "sha256-mQnSduf8SHYyUs6gHfI+JAvpRxYQA1DiMlvNofImElU=";
  };
  cargoDeps = drv.cargoDeps.overrideAttrs (lib.const {
    name = "${drv.cargoDeps.name}-vendor";
    inherit src;
    outputHash = "sha256-QiGnBoZV4GZb5MQ3K/PJxCfw0p/7qDmoE607hlGKOns=";
  });
  postFixup = if pkgs.stdenv.isDarwin then ''
    mkdir -p "$out/home/Library/Application Support/dioxus/wasm-bindgen"
    ln -s ${lib.getExe wasm-bindgen-cli} "$out/home/Library/Application Support/dioxus/wasm-bindgen/wasm-bindgen-${wasm-bindgen-cli.version}"
    wrapProgram $out/bin/dx \
      --set HOME $out/home
  '' else ''
    mkdir -p $out/share/dioxus/wasm-bindgen
    ln -s ${lib.getExe wasm-bindgen-cli} $out/share/dioxus/wasm-bindgen/wasm-bindgen-${wasm-bindgen-cli.version}
    wrapProgram $out/bin/dx \
      --set XDG_DATA_HOME $out/share
  '';
  checkFlags = drv.checkFlags ++ ["--skip=wasm_bindgen::test"];
  nativeBuildInputs = drv.nativeBuildInputs ++ [pkgs.makeBinaryWrapper];
});

where I'm using dx within another rustPlatform.buildRustPackage derivation to build a fullstack dioxus app

I have some pull requests open with Dioxus to try to make building - in particular building manually without the cli tool on nix - easier.
https://github.com/DioxusLabs/dioxus/pull/3419
https://github.com/DioxusLabs/dioxus/pull/3429
There is an issue opened saying it should be possible to have it lookup wasm-bindgen in the path instead of automatically downloading it. Nothing has come of it yet though.
https://github.com/DioxusLabs/dioxus/issues/3411

Anyone want to make this change into a PR and submit it to Dioxus?
https://github.com/NixOS/nixpkgs/pull/366178#issuecomment-2553840162

@brianmay I opened an issue around this problem: https://github.com/DioxusLabs/dioxus/issues/3417. There's a messy WIP branch that I can clean up and open as a PR.
I also started work on a wiki page documenting how to resolve the version mismatch error: https://wiki.nixos.org/wiki/Dioxus


fix(transfer): add scroll to channel selection

Description
Add scroll to transfer to savings channel selection
Linked Issues/Tasks
Closes #2173
Type of change

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

Tests

[ ] Detox test
[ ] Unit test
[x] No test

Are those all LDK channels, why some of them has 0 value in original ticket? Might it be BT unfinished orders?


Update module github.com/onsi/gomega to v1.36.1

This PR contains the following updates:

Package
Type
Update
Change

github.com/onsi/gomega
require
patch
v1.36.0 -> v1.36.1

Release Notes

onsi/gomega (github.com/onsi/gomega)
v1.36.1
Compare Source
1.36.1
Fixes

Fix https://github.com/onsi/gomega/issues/803 [1c6c112]
resolves #696: make HaveField great on pointer receivers given only a non-addressable value [4feb9d7]

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

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

This PR has been generated by Renovate Bot.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>7
Details

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 44.198%

Totals

Change from base Build<PHONE_NUMBER>1:
0.0%

Covered Lines:
438

Relevant Lines:
991

💛  - Coveralls


Fix JIS style punctuation

This PR fixes incorrect punctuation adjustment of Japanese question & exclamation mark ？ and ！.

Thanks!


HA Upgrades /admin/high-availability/upgrades.md

Outline and layout document about HA Master upgrade process and best practices for upgrading virtual machine images for instance
Again @bgrant0607 happy to help.
I am coding this into kops, and will need to document. Documenting here will be work, but not crazy. Who can I lean on for tech edit? Doing this will make kops better.

@chrislovecnm The other members of @kubernetes/kops-maintainers ?

@kubernetes/docs can I be assigned?

Since this is covered by #1733, I'm closing this issue.


Feature: Add Folder to project

VSCode Version: 1.12.1
OS Version: 16.04 LTS

I'd really like a Add Folder to Project feature just like in Atom, you can open multiple folders in the same window, and working from them. In my version of VSCode, when i use the open folder option, it opens a new instance of VSCode. Please this would a nice feature to add.

Duplicate of #396


Request: github oauth login option

It'd be really cool to be able to sign in to the UI via an oath provider like github.

Cool idea, but I don't think we need OAuth in dashboard, as it is only a management GUI.
It is not like the social media web sites, in which case the site may have many users, and you want to use OAuth to void your users having to register an account again.
The dashboard is different, it is a private website, only for you and your administrators.

Thanks for the reply and opening up the discussion!
yea the use case is to enable developers to just be able to use their github account, very useful for orgs that have github teams. In the dashboard ideally you would be able to authorize a particular github organization + team. Google oauth enables similar behavior so that you can authorize a given team within a google managed domain. That way our admins don't have to manage another account, so its more like enabling SSO via an oath provider similar to how you would traditionally use something like ldap. A couple of our appliances do this out of the box - for example https://rancher.com/.
Alternatively is there a way auth could be optional? Because then I could stick a oauth terminating proxy in front of the ui. I've done this for a couple of internal appliances that don't provide the functionality out of the box. Pretty easy to setup in a docker friendly environment -  https://github.com/bitly/oauth2_proxy.

being able to turn it off to enable the use of something like an oath proxy would be awesome


Support Interceptors from Beans

Currently, io.camunda.zeebe.spring.client.properties.ZeebeClientConfigurationProperties contains a field interceptors that is not connected to the spring context.
I would suggest to create an injection point where this field can be populated from Beans that implement this interface.

Out of curiosity: What is the use case where you need those interceptors @jonathanlukas?
Are you willing to provide a PR by chance?

I can do that. We are thinking about a way to enable ExecutionListener on Zeebe Jobs. They could be referenced as header containing a list potentially close a feature gap.

I can add a change to support this as part of https://github.com/camunda-community-hub/spring-zeebe/pull/363


[Warrior] fixes mortal strike, slam, shattering glyph

Based on PTR/Beta:
Changed mortal strike to 150% weapon damage
Changed Slam to 145%
Removed Minor Glyph of Shattering Throw  no longer exists in Cata

I pushed a change to master that generalizes how we setup the base class scaling You can update this PR with it. Warrior should refactor its base damage values to use SpellScaling * Coeff so its clearer from where the base damage comes and not just a magic number in the code


Test infra: Add NAT gateway IP addresses to base stack SSM parameters

Feature or Bugfix

Feature

Detail
The NAT Gateway IPs need to be added in one of the security groups for our CodeBuild. Right now, this is performed manually and is therefore not present in infrastructure-as-code. This PR is adding the IP addresses as SSM parameters to the base stack, so that we can use them from the CodeBuild definitions.
The SSM parameters will be the following:

/SDKPandas/VPC/PublicSubnet1/EIP
/SDKPandas/VPC/PublicSubnet2/EIP
/SDKPandas/VPC/PublicSubnet3/EIP

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: GitHubDistributedCodeBuild6-jWcl5DLmvupS
Commit ID: 9d23589e67685f8d3d9b9a8f7fc3090eade8846f
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubStandardCodeBuild8C06-llutOAimTATs
Commit ID: 9d23589e67685f8d3d9b9a8f7fc3090eade8846f
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: b782fe6d00bd53b7e37c1d77ac0b8c9af9b891f2
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Implement Authentication redirect for cookies

Today when we issue the challenge from SignalR a 401 is correctly returned with a Location response header with the redirect URI. We should issue a client side redirect to get to the login page. Here's what this looks like:
HTTP/1.1 401 Unauthorized
Location: http://localhost:57562/Account/Login?ReturnUrl=%2Fchat
Server: Kestrel
X-SourceFiles: =?UTF-8?B?QzpcVXNlcnNcZGZvd2xlclxEb2N1bWVudHNcR2l0SHViXFNpZ25hbFJcc2FtcGxlc1xDaGF0U2FtcGxlXGNoYXQ=?=
X-Powered-By: ASP.NET
Date: Sun, 27 Aug 2017 21:50:30 GMT
Content-Length: 0

We should also consider adding a client side hook to allow the client to handle this (maybe the client wants to pop up a client side UI to login rather than redirecting).

We should also consider adding a client side hook to allow the client to handle this (maybe the client wants to pop up a client side UI to login rather than redirecting).

I wouldn't like it if signalr would redirect me without having the chance to change that.

Websockets connections need to be let through when unauthorized and then immediately closed with some kind of error code (4401?)/message to let us tell the client that they need some authorization. This is because if we challenge during the websockets handshake it will error without any usable information.

No plans to do this per the triage decision


What is a navigation property?

A property is considered a navigation property if the type it points to ~can not~ cannot be mapped as a scalar type by the current database provider and …

What does it mean that a property points to a type?  The straightforward interpretation, that the value of a property is a Type, is obviously wrong.
Did you mean its type … is enumerable and the enumerated values cannot be mapped?  Because this is what it seems to be.  But even if this is the case, the reader does not know yet what cannot be mapped means; the previous chapters do not mention that only a few types can be mapped :thinking:

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

ID: 24b2b50e-35f8-9e0b-8ce5-840e1ea816e6
Version Independent ID: 9e0e6e2c-48fa-76c3-3bd9-813a6df22645
Content: Relationships - EF Core
Content Source: entity-framework/core/modeling/relationships.md
Product: entity-framework
Technology: entity-framework-core
GitHub Login: @AndriySvyryd
Microsoft Alias: avickers

@yecril71pl navigations aren't necessarily enumerable - reference navigations represent one-to-one relationships. If the property CLR type is supported by the database provider as a native sclar type, the property is mapped as a column; whereas if the type is a known, mapped entity type in the model, EF treats it as a reference navigation by default.
The phrase "cannot be mapped as a scalar type by the current database provider" seems pretty straightforward to me; it may be slightly clearer to simply say "isn't supported" instead of "cannot be mapped", but I don't think it's very significant here.

The phrase "cannot be mapped as a scalar type by the current database provider" seems pretty straightforward to me; it may be slightly clearer to simply say "isn't supported" instead of "cannot be mapped", but I don't think it's very significant here.

Of course it looks obvious to you; however, it is all Greek to a person who reads it for the first time, I am afraid.  Also, what I was missing in the text is that entity types cannot be scalar (in particular, an entity cannot be a value type, which is quite unexpected because an entity type feels like a value type for all purposes, especially when it contains no foreign keys).  I tried to find this information in the preceding chapters and I was unable to do it.

Our getting started page discusses entity classes multiple types and shows examples of class definitions for them. We may not explicitly say that entity types cannot be scalar, but there's a whole lot of introductory docs on modeling with examples of entity classes. The docs you're commenting on - relationships - isn't the first page users that new users are expected to read on their very first encounter with EF Core; it's reasonable to assume that some basic concepts have been acquired, otherwise we'd have to repeat all concepts and definitions in every single doc page.
Can you explain what your confusion/expectation here was? Were you expecting a navigation to be set up to a scalar type? What would that even mean in a relational database etc.?

Potentially helped by #1665.

Our getting started page discusses entity classes multiple types and shows examples of class definitions for them. We may not explicitly say that entity types cannot be scalar, but there's a whole lot of introductory docs on modelling with examples of entity classes.

Yes, and those examples do not show that value types can be entities but that the text around does not say that they cannot be.

Can you explain what your confusion/expectation here was? Were you expecting a navigation to be set up to a scalar type? What would that even mean in a relational database etc.?

Ugh, not exactly a scalar type like int, rather a value type like a location.  A location is not a scalar, it is a vector, but such a vector can be and would be represented as a value type in pure C# code.
I understand that value types like Uri (which is not a value type, but could as well be) can be treated both as entities and as scalars.  My scenario is: I would like to make a Uri a separate entity even if it hypothetically were declared as a value type.  And the text should say how or why not.

Maybe we could add a note in https://docs.microsoft.com/en-us/ef/core/modeling/entity-types on why value entity types are a bad idea (they cannot be tracked for changes).


windows11, connection closed by sqlserver: ('01000', '[01000] [Microsoft][ODBC SQL Server Driver][DBNETLIB]ConnectionWrite (send()). (10054) (SQLExecDirectW); [01000] [Microsoft][ODBC SQL Server Driver][DBNETLIB]General network error. Check your network documentation. (11)')

Please first make sure you have looked at:

Documentation: https://github.com/mkleehammer/pyodbc/wiki
Other issues

Environment
To diagnose, we usually need to know the following, including version numbers.  On Windows, be
sure to specify 32-bit Python or 64-bit:

Python: 3.10.9
pyodbc: 4.0.34
OS: windows11
DB: sqlserver 2019
driver: SQL Server

Issue
Connection used to keepalive in windows10 and windows server 2019, but when I use windows11, it always be closed by sqlserver in a random period, sometimes  8 minutes, sometimes 2 hours after established.
conn = pyodbc.connect('DRIVER={SQL Server};SERVER=Stock1;DATABASE=TestDB;UID=r_front;PWD=12345611')
I have already tried using ODBC Driver 17 for SQL Server but the problem still occured.
Thanks a lot for your help.

sometimes 8 minutes, sometimes 2 hours after established.

Is that the amount of time the connection spends idle, and are you connecting to the same server you were before? If so, it is unlikely to be a problem in the driver nor pyODBC, and more likely one in the network stack of the OS.
If you use ODBC Driver 17.4 for SQL Server or newer, you can try to set the KeepAlive/KeepAliveInterval options: https://learn.microsoft.com/en-us/sql/connect/odbc/windows/features-of-the-microsoft-odbc-driver-for-sql-server-on-windows?view=sql-server-ver16#microsoft-odbc-driver-174-for-sql-server-on-windows
This does not appear to be a pyODBC issue.

sometimes 8 minutes, sometimes 2 hours after established.

Is that the amount of time the connection spends idle, and are you connecting to the same server you were before? If so, it is unlikely to be a problem in the driver nor pyODBC, and more likely one in the network stack of the OS.
If you use ODBC Driver 17.4 for SQL Server or newer, you can try to set the KeepAlive/KeepAliveInterval options: https://learn.microsoft.com/en-us/sql/connect/odbc/windows/features-of-the-microsoft-odbc-driver-for-sql-server-on-windows?view=sql-server-ver16#microsoft-odbc-driver-174-for-sql-server-on-windows
This does not appear to be a pyODBC issue.

It's not the connection spends idle, I just made a loop to call the connection every minute,  and just connected to the same server. I will try ODBC Driver 18 for SQL Server, thank you.

That seems more like a network/OS issue than a driver issue.

That seems more like a network/OS issue than a driver issue.

I think so, the mstsc sometimes loss connection, too. I'll close this issue, thank's again.


Fix RCTFatalException in RCTFatal, line 133

@artsyit commented on Sun Jun 05 2016
Version: 2.5.0 (2016.06.03) | net.artsy.artsy
Stacktrace
-;RCTFatal;RCTAssert.m;133
RCTExceptionsManager;reportFatalException:stack:exceptionId:;RCTExceptionsManager.m;64
RCTModuleMethod;invokeWithBridge:module:arguments:;RCTModuleMethod.m;489
RCTBatchedBridge;_handleRequestNumber:moduleID:methodID:params:;RCTBatchedBridge.m;968
RCTBatchedBridge;handleBuffer:;RCTBatchedBridge.m;896
Reason
*** Terminating app due to uncaught exception 'RCTFatalException: Unhandled JS Exception: undefined is not an object (evaluating 'r.image.aspect_ratio')', reason: 'Unhandled JS Exception: undefined is not an object (evaluating 'r.image.asp..., stack:
i@617:2044
value@617:2186
value@617:3125
_renderValidatedComponentWithoutOwnerOrContext@293:5069
_renderValidatedComponent@293:5197
_updateRenderedComponent@293:4592
_performComponentUpdate@293:4392
updateComponent@293:3669
performUpdateIfNecessary@293:3190
performUpdateIfNecessary@273:665
r@271:570
perform@276:494
perform@276:494
perform@271:1534
P@271:1706
closeAll@276:1131
perform@276:581
batchedUpdates@270:453
s@271:760
a@290:69
enqueueSetState@290:915
setState@78:317
value@617:1678
a@258:71
o@256:391
a@256:582
f@254:159
g@254:262
i@260:102
processEventQueue@254:1382
s@304:92
handleTopLevel@304:181
_receiveRootNodeIDEvent@303:613
receiveEvent@303:714
value@198:3100
@198:1319
k@198:413
value@198:1291
'
Link to HockeyApp

https://rink.hockeyapp.net/manage/apps/37029/crash_reasons/123909040

@orta commented on Sun Jun 05 2016
From the looks of the crashes, it's these artists so far
[2016-06-05 12:31:15.685] [artist view] {
    "artist_id" = "wang-wusheng";
}

[2016-06-05 11:01:38.417] [artist view] {
    "artist_id" = "annie-leibovitz";
}

[2016-06-05 13:22:46.118] [artist view] {
    "artist_id" = "jennie-jieun-lee";
}

@alloy commented on Fri Jun 10 2016
Ok, what I think happens is that this is because the artwork slug is the same as the artist slug:

Relay always queries an object’s id if the queried type has that field. In the case of the artist that returns the artist’s slug, e.g. annie-leibovitz.
Relay assumes that the id field is part of the Node instance of the Relay Global Object Identification Specification, i.e. the id value is unique amongst any entity (e.g. artist/artwork).
Relay caches the artist result under the id key.
Relay notices a query for an artwork object with an id of annie-leibovitz and assumes it can return the cached data, which is in reality an artist object, not an artwork.
Everything goes downhill from here.

So it seems that I’ll have to get working on making metaphysics more Relay compatible earlier than anticipated.
/cc @dzucconi

@alloy commented on Mon Jun 13 2016
This patch at least fixes the crash, but it just means we’re omitting these artworks, which is clearly bad. https://github.com/artsy/emission/pull/177/commits/72d57e8d91e99177ebf30b25d02d91213e9ce3e2

Have just confirmed that with my Relay hacking this issue is resolved 🎉
Previously this work would not be fetched correctly because Relay would think it was the Artist with ID wang-wusheng:
{
                "relayID": "QXJ0d29yazp3YW5nLXd1c2hlbmc=",
                "title": "",
                "date": "1984",
                "sale_message": "Not for sale",
                "image": {
                    "_urlZizbi": "https://d32dm0rphc51dk.cloudfront.net/7V-j4AJJE3pFDZ9GbT7m6A/large.jpg",
                    "aspect_ratio": 1.69
                },
                "artist": {
                    "name": "Wang Wusheng",
                    "relayID": "QXJ0aXN0Ondhbmctd3VzaGVuZw=="
                },
                "partner": {
                    "name": "La Galerie Paris 1839"
                },
                "href": "/artwork/wang-wusheng"
            }


WTB: Confirmation before deleting job on web ui

In Chronos 3.0.2, clicking the red X button a job's actions will immediately remove the job. It would be awfully nice if it waited for a confirmation that you indeed want to remove said job.

:man_shrugging:


Set neutron_l2_population default True

Support has found that the new default value of False is causing issues with
stability in customer environments. Given this fact, we are no loger accepting
the default value.
Connects #973

:+1:

Looks to be a straight-forward backport. 👍


how to resize pc Simulators

please help me

Changing the two values here (800, 480) should be all that's needed.

thx  :)

when i ask this question before, i use the 6.0( the 6.0 is hard to resize pc Simulators  ). but now, i try to use 8.0

I'll assume the issue is solved; please let me know if you need further help.


Using embedded entities

I can't seem to figure out if it's supported to use embedded entities? In case it is, how would the DSL for assigning such a property on a record look like in an example?

Hi @NielsKSchjoedt!
Yes, we use embedded entities all the time. Let's say you have something like:
attr_accessor :items            # Array
defined on your primary model.
Then just call build_entity on each child model before assigning to items.
embedded_entities = []
children.each do |child|
  embedded_entities << child.build_entity
end
primary.items = embedded_entities
primary.save

Thanks, very helpful. So, I'm a bit unsure of the whole use of ancestors/children VS embedded entities. Our use case is this:
Consider this simple example setup:
class PrimaryModel
  include ActiveModel::Datastore

  ATTRIBUTES = %i{some_context other_context}
  attr_accessor *ATTRIBUTES

  def entity_properties
    ATTRIBUTES
  end
end

class ContextualFeatures
  include ActiveModel::Datastore

  ATTRIBUTES = %i{has_this has_that time_stamp some_description}
  attr_accessor *ATTRIBUTES

  def entity_properties
    ATTRIBUTES
  end
end

Now according to your description I can do something like this:
mp = PrimaryModel.new(
  some_context: ContextualFeatures.new(
    has_this: true,
    has_that: false,
    time_stamp: Time.zone.now,
    some_description: 'Here is some random text'
  ).build_entity,
  other_context: ContextualFeatures.new(
    has_this: false,
    has_that: false,
    time_stamp: Time.zone.now,
    some_description: 'Super important message!'
  ).build_entity
)
mp.save

This works and creates the embedded entity. Now when I wanna retrieve the PrimaryEntity back like so: mp = PrimaryEntity.find(1) the value of mp.some_context or mp.other_context is an instance of Google::Cloud::Datastore::Entity. Is there any "standard"/"build-in"/"best way" of transforming that into an instance of ContextualFeatures? Or am I doing this all wrong, should do things differently?
Also, do I understand correctly that this way of doing things I am utilising the "embedded_entities" property functionality of Firestore rather than the parent/child-thingy, which comes with e.g. the write throughput limitation of 1 write per 1 sec?
Hope it's OK I ask. I would like to contribute with some README examples, but I need to make sure I understand things right first :-)

Yes, your example is using embedded entities. With the parent/child relationship you have separate entities and you set the parent id in the key of the child entity.
For example, if you added this to your ContextualFeatures model:
def self.parent_key(id)
  CloudDatastore.dataset.key('PrimaryModel', id)
end

you could create them like this:
ContextualFeatures.new(
  has_this: false,
  has_that: false,
  time_stamp: Time.zone.now,
  some_description: 'Super important message!',
  parent_key_id: 1 # id of the parent
)

Your ContextualFeatures could then be queried using the parent_key. This is described in more detail in the README here.
Which is better? That is completely dependent on your application.

There is no "standard"/"build-in"/"best way" of transforming the embedded entities back into models. It always ends up being application specific. You can make use of the from_entity method though.


Timing error compiling hello_world with kv260_ispMipiRx_vcu_DP platform

I have built the kv260_ispMipiRx_vcu_DP platform provided in this repository and I am trying to synthesize the hello_world example from the Vitis Accel repository with this platform. However, the build fails with timing errors. This is my command line:
make sd_card TARGET=hw DEVICE=kv260_ispMipiRx_vcu_DP HOST_ARCH=aarch64 EDGE_COMMON_SW=/tools/xilinx/petalinux/2020.2/
And this is the end of the build log:
===>The following messages were generated while  Compiling (bitstream) accelerator binary: vadd Log file: /home/hamid/projects/Vitis_Accel_Examples/hello_world/_x.hw.kv260_ispMipiRx_vcu_DP/link/vivado/vpl/prj/prj.runs/impl_1/runme.log  :
ERROR: [VPL 101-2] design did not meet timing - Design failed to meet timing.
    Failed timing checks (paths):
        {kv260_ispMipiRx_vcu_DP_i/axi_ic_PS_0_S_AXI_HP1_FPD/m00_couplers/auto_ds/inst/gen_downsizer.gen_simple_downsizer.axi_downsizer_inst/USE_WRITE.write_data_inst/length_counter_1_reg[1]/C --> kv260_ispMipiRx_vcu_DP_i/axi_ic_PS_0_S_AXI_HP1_FPD/m00_couplers/m00_data_fifo/inst/gen_fifo.fifo_gen_inst/inst_fifo_gen/gaxi_full_lite.gwrite_ch.gwdch2.axi_wdch/grf.rf/gntv_or_sync_fifo.mem/gbm.gbmg.gbmga.ngecc.bmg/inst_blk_mem_gen/gnbram.gnativebmg.native_blk_mem_gen/valid.cstr/ramloop[0].ram.r/prim_noinit.ram/DEVICE_8SERIES.NO_BMM_INFO.SDP.WIDE_PRIM18.ram/ENARDEN}

    Please check the routed checkpoint (dr_routed_timing.dcp) and timing summary report (dr_timing_summary.rpt) for more information.
ERROR: [VPL 101-3] sourcing script /home/hamid/projects/Vitis_Accel_Examples/hello_world/_x.hw.kv260_ispMipiRx_vcu_DP/link/vivado/vpl/scripts/_full_write_bitstream_pre.tcl failed
ERROR: [VPL 60-773] In '/home/hamid/projects/Vitis_Accel_Examples/hello_world/_x.hw.kv260_ispMipiRx_vcu_DP/link/vivado/vpl/vivado.log', caught Tcl error:  problem implementing dynamic region, impl_1: write_bitstream ERROR, please look at the run log file '/home/hamid/projects/Vitis_Accel_Examples/hello_world/_x.hw.kv260_ispMipiRx_vcu_DP/link/vivado/vpl/prj/prj.runs/impl_1/runme.log' for more information
WARNING: [VPL 60-732] Link warning: No monitor points found for BD automation.
ERROR: [VPL 60-704] Integration error, problem implementing dynamic region, impl_1: write_bitstream ERROR, please look at the run log file '/home/hamid/projects/Vitis_Accel_Examples/hello_world/_x.hw.kv260_ispMipiRx_vcu_DP/link/vivado/vpl/prj/prj.runs/impl_1/runme.log' for more information
ERROR: [VPL 60-1328] Vpl run 'vpl' failed
ERROR: [VPL 60-806] Failed to finish platform linker
INFO: [v++ 60-1442] [16:37:14] Run run_link: Step vpl: Failed
Time (s): cpu = 00:00:09 ; elapsed = 00:17:51 . Memory (MB): peak = 1586.227 ; gain = 0.000 ; free physical = 147385 ; free virtual = 314962
ERROR: [v++ 60-661] v++ link run 'run_link' failed
ERROR: [v++ 60-626] Kernel link failed to complete
ERROR: [v++ 60-703] Failed to finish linking
INFO: [v++ 60-1653] Closing dispatch client.
Makefile:141: recipe for target 'build_dir.hw.kv260_ispMipiRx_vcu_DP/vadd.xclbin' failed
make: *** [build_dir.hw.kv260_ispMipiRx_vcu_DP/vadd.xclbin] Error 1

I have attached the timing summary, as well. Am I doing something wrong here?
dr_timing_summary.txt

The make options that you have listed don't apply to this repo so not sure what exactly you are running. Maybe this is really something that should be posted in the other git repo instead.
I'm not familiar with the accelerator that you are using but timing closure problems can be due to many different things. The examples that are shipped as part of this repo have all been vetted and close timing. So this is really not an "issue" as I can always create a design that does not meet timing on a given device for a given platform.
A few debug pointers:

Open the generated Vivado design and check what paths have timing errors and how much is it off
Make sure the accelerator meets timing post HLS synthesis for the target frequency
Tweak the HLS code if it doesn't meet timing
Try lowering the accelerator clock frequency
Reduce the logic footprint of the accelerator
Try different Vivado implementation strategies
Check the connectivity of the generated designs and if needed guide the Vitis platform linker what HP ports/interconnects to connect to

@chkohn Thank you for your reply. I am using the kv260_ispMipiRx_vcu_DP platform from this repository in the standard Vitis acceleration flow, in the same way I have used many other Vitis platforms for both Xilinx MPSoC and Alveo boards, including custom platforms. As part of the first test before using a new platform, I always synthesize Xilinx's own hello_world example, which is an extremely small and simple design, to make sure the platform works correctly, and I have never experienced timing errors with this simple design with the 10+ platforms I have used across different devices, except this one. If a platform does not meet timing even with the hello_world example, it means it won't meet timing with 99% of other designs, as well, except maybe with custom placement and routing constraints. Chances are, the timing closure of the platform is very tight with little to no slack time, resulting in timing issues with even small designs such as the one I am using.
Can you please clarify if the platforms in this repository are intended to be used with custom kernels in the Vitis acceleration flow, or if they are solely designed to be coupled with the overlays in this repo and Xilinx DPU, purely targeted for development of software applications without changing the platform and PL design?

@zohourih the only advice I can give is to look at the generated Vivado design and analyze it to see why timing is not met. If it's a small kernel, then it should definitely be able to meet timing. Some of the provided examples instantiate a DPU IP and a custom preprocessing accelerator and they meet timing. You can also compare the generated Vivado designs of one of the provided overlays with your kernel, it may give you a clue what's different.
The platforms are definitely meant to be extended with custom kernels but you may need to tweak clock frequency or settings for the vivado linker to achieve optimal results. Each platform comes with one example overlay which can be used as starting point or reference.
None of the provided platforms support DFX today.

@chkohn Yes, the design is extremely small, and that is why I am surprised it is failing to meet timing. The timing failure is happening with the 300 MHz clock and on the AXI interconnect that is automatically inserted into the design by Vitis to connect the user kernel to HP1. The slack is not too large (-0.040); however, considering the simplicity and small size of the user kernel, even such a small negative slack is quite surprising. I can imagine the designs provided by Xilinx have probably gone through a lot of manual timing tuning, which would incur a lot of burden for normal users if they have to do such thing for even simple and small designs. Nonetheless, I will try to dig deeper and see if I can get around the timing failure.

@zohourih ok, please share your results here.
@sweatharao please chime in here if you have any suggestions.

@zohourih can you take a look at this file and use it in your flow
https://github.com/Xilinx/kv260-vitis/blob/release-2020.2.2_k26/overlays/examples/smartcam/prj_conf/prj_config_1dpu

You can either reduce the clock frequency on your kernel  under the [clock] section
Maybe distribute the memory ports across different HP port under the [connectivity] section. This is likely not needed as your bandwidth requirement maybe low.
Try different implementation strategies  under [vivado] the section

@zohourih can this issue be closed?


.toMatchSnapshot propertyMatchers with expect.arrayContaining should not

🚀 Feature Proposal
Ability to create snapshot of an Array of Objects with dynamic fields
Motivation
Currently Jest allows using Property Matchers to ignore the value from the dynamic fields, but in case they're within an Array the rest of the fields are ignored
Example
  const r = {
    a: {
      b: [
        {
          createdAt: new Date(),
          id: Math.floor(Math.random() * 20),
          d: 'string'
        },
        {
          createdAt: new Date(),
          id: Math.floor(Math.random() * 20),
          d: 'second string'
        },
        {
          createdAt: new Date(),
          id: Math.floor(Math.random() * 20),
          d: 'third string'
        },
      ]
    }
  }

  test('should match the snapshot', () => {
    expect(r).toMatchSnapshot({
      a: {
        b: expect.arrayContaining([
          {
            id: expect.any(Number),
            createdAt: expect.any(Date)
          }
        ])
      }
    })
  })

The example above currently fails. It can result in the following snapshot
Object {
  "a": Object {
    "b": Array [
      Object {
        "createdAt": Any<Date>,
        "d": "string",
        "id": Any<Number>,
      },
      Object {
        "createdAt": Any<Date>,
        "d": "second string",
        "id": Any<Number>,
      },
      Object {
        "createdAt": Any<Date>,
        "d": "third string",
        "id": Any<Number>,
      },
    ],
  },
}

Pitch
Why does this feature belong in the Jest core platform?
There is no other way to make a snapshot that will pass the test twice without modifying the tested Array.

Great proposition!
It would solve these kind of problems far more gracefully than the current solutions.

Probably if isAsymmetricMatcher returns false for ArrayContaining and ObjectContaining it will solve the problem


Method AdaptiveByteBufAllocator.usedHeapMemory() does not return right value with customized FastThreadLocalThread thread

Expected behavior
The return value of method AdaptiveByteBufAllocator.usedHeapMemory() should be greater than 0.
Actual behavior
The return value is 0, as following output on my laptop:

alloc.usedHeapMemory():0, Thread[Thread-0,5,main]
alloc.usedHeapMemory():0, Thread[main,5,main]

Steps to reproduce
Run the main method:
import io.netty.buffer.AdaptiveByteBufAllocator;
import io.netty.util.concurrent.FastThreadLocalThread;
import io.netty.util.internal.ObjectUtil;

public class T20241121 {

    static class CustomizeFastThreadLocalThreadWithoutCleanupFastThreadLocals extends FastThreadLocalThread
                         implements Runnable {

        private final Runnable runnable;

        private CustomizeFastThreadLocalThreadWithoutCleanupFastThreadLocals(Runnable runnable) {
            this.runnable = ObjectUtil.checkNotNull(runnable, "runnable");
        }

        @Override
        public boolean willCleanupFastThreadLocals() {
            return false;
        }

        @Override
        public void run() {
            runnable.run(); // Without calling `FastThreadLocal.removeAll()`.
        }
    }

    public static void main(String[] args) throws InterruptedException {

        final AdaptiveByteBufAllocator[] alloc = new AdaptiveByteBufAllocator[1];

        Runnable task = new Runnable() {
            @Override
            public void run() {
                alloc[0] = new AdaptiveByteBufAllocator(false, true);
                alloc[0].heapBuffer(1024, 1024);
                System.out.println("alloc.usedHeapMemory():" + alloc[0].usedHeapMemory() + ", " + Thread.currentThread());
            }
        };

        Thread customizefastThreadLocalThread = new CustomizeFastThreadLocalThreadWithoutCleanupFastThreadLocals(task);
        customizefastThreadLocalThread.start();
        customizefastThreadLocalThread.join();
        System.out.println("alloc.usedHeapMemory():" + alloc[0].usedHeapMemory() + ", " + Thread.currentThread());
    }
}

Netty version
4.1.115.Final
JVM version (e.g. java -version)
Does not matter.
OS version (e.g. uname -a)
Does not matter.

The reason is here -> https://github.com/netty/netty/blob/4.1/buffer/src/main/java/io/netty/buffer/AdaptivePoolingAllocator.java#L151
It's to avoid Magazines leaks.
Another solution would be to use soft reference to the magazine, in case the fast thread local thread cannot clean up itself.
Another qq question:
Are you enabling -Dio.netty.allocator.useCachedMagazinesForNonEventLoopThreads?

Are you enabling -Dio.netty.allocator.useCachedMagazinesForNonEventLoopThreads=true?

Yes, I did it by setting the second param to be true:
alloc[0] = new AdaptiveByteBufAllocator(false, true);

@franz1981
There is another imperfect option, we can modify AdaptiveByteBufAllocator.usedHeapMemory(), to let the current thread be able to count its own threadLocalMagazine's memory usage, some code is added to AdaptiveByteBufAllocator.usedHeapMemory() as following:
public long usedMemory() {
        long sum = 0;
        for (Chunk chunk : centralQueue) {
            sum += chunk.capacity();
        }
        for (Magazine magazine : magazines) {
            sum += magazine.usedMemory.get();
        }
        if (liveCachedMagazines != null) {
            for (Magazine magazine : liveCachedMagazines) {
                sum += magazine.usedMemory.get();
            }
            // Newly added code begin.
            if (liveCachedMagazines.isEmpty() && null != threadLocalMagazine && threadLocalMagazine.isSet()) {
                Object mag = threadLocalMagazine.get();
                if (mag != NO_MAGAZINE) {
                    sum += ((Magazine)mag).usedMemory.get();
                }
            }
            // Newly added code end.
        }
        return sum;
    }

But the current thread still can not count other threads's threadLocalMagazine memory usage, which is not ideal.

Option 4: If FastThreadLocalThread.willCleanupFastThreadLocals(Thread.currentThread()) == false, then we will not use threadLocalMagazine, this needs to modify threadLocalMagazine's initialValue() method as following:
@Override
protected Object initialValue() {
    if (cachedMagazinesNonEventLoopThreads || ThreadExecutorMap.currentExecutor() != null) {
        if (!FastThreadLocalThread.willCleanupFastThreadLocals(Thread.currentThread())) {
            // To prevent leak, we will NOT use threadLocalMagazine.
            return NO_MAGAZINE;
        }
        Magazine mag = new Magazine(AdaptivePoolingAllocator.this, false);
        liveMagazines.add(mag);
        return mag;
    }
    return NO_MAGAZINE;
}

I think this is a better option than option 3.
@franz1981 WDYT?

Fire a PR, love it :)

OK, PR is on the way...


install with --no-sudo does not work

Helm version: 3.9.4
Kubectl version: not relevant.
Cloud provider: not relevant.
Steps to reproduce:
From the instructions at https://helm.sh/docs/intro/install/#from-script

[ $(id -u) != 0 ] && echo "I Am Not Root"  #  I Am Not Root
curl -fsSL -o get_helm.sh https://raw.githubusercontent.com/helm/helm/main/scripts/get-helm-3
chmod 700 get_helm.sh
./get_helm.sh -h  # "[--no-sudo]  ->> install without sudo
./get_helm.sh --no-sudo

Expected outcome:
Gets the appropriate "helm" binary for my OS and arch, puts it either in
cwd or in the first writable dir in $PATH.  Or reminds me I must provide
a destination path if I'm not root.
Seen outcome:
Downloading https://get.helm.sh/helm-v3.9.4-linux-amd64.tar.gz
Verifying checksum... Done.
Preparing to install helm into /usr/local/bin
cp: cannot create regular file '/usr/local/bin/helm': Permission denied
Failed to install helm with the arguments provided: --no-sudo
Accepted cli arguments are:
	[--help|-h ] ->> prints this help
	[--version|-v <desired_version>] . When not defined it fetches the latest release from GitHub
	e.g. --version v3.0.0 or -v canary
	[--no-sudo]  ->> install without sudo
	For support, go to https://github.com/helm/helm.

If I can't use this without being root, why give the option not to use sudo?
Workaround:

Get the url from the "Downloading" message
cd $(mktemp -d)
curl -L https://get.helm.sh/helm-v3.9.4-linux-amd64.tar.gz
copy ./*/helm and ./*/tiller to $HOME/.local/bin or wherever is appropriate for your workstation.

This is a repeat of issue #6040 (2019-07-18) which was closed by
automation, but the problem is still here in the most recent version.

Regarding the workaround, you could instead just supply an install dir e.g. HELM_INSTALL_DIR="$HOME/.local/bin" ./get-helm-3 --no-sudo. I'm not sure if that's the intended approach though.
@joejulian I'm keen to look into this. Would it be preferably to default to a user owned dir such as $HOME/.local/bin if --no-sudo is supplied? Or should it fail and prompt the user to supply HELM_INSTALL_DIR?

I would probably expect an install script to just be successful unless it can't. What if it parsed PATH and looked for the first writable directory if HELM_INSTALL_DIR isn't set?
If nothing else though, an actionable error message would be great.

I want to work on this

encountering this problem; eagerly awaiting #11383

I'm also running into this issue as well

The issue still persists

The issue still persists after
 ./get_helm.sh --no-sudo

Any idea how to fix?


Compile the Michelson interpreter

Closed in favor of https://github.com/foobar-land/coq-of-ocaml/pull/206 with a better branch name.


No data returned when charging system has error

Describe the bug
Fails to start when charging returns error. Noticed since yesterday when cable not connected correctly. After replacing cable restarted container and now ok.
To Reproduce:

Part connect cable
Restart docker container

Expected behavior:
start up and show
Screenshots:
If applicable, add screenshots to help explain your problem.
Version info:
The version you are using and the method you are using (Docker container/HA Addon/Cloned repo)
Logs:
Jul 06 08:58:16 volvo2mqtt [1] - INFO: Starting volvo2mqtt version v1.7.3
Jul 06 08:58:17 volvo2mqtt [1] - INFO: Vin: ['XXXXXXXXXXXXXXXXXX'] found!
Jul 06 08:58:17 volvo2mqtt [1] - INFO: Success! Battery Charge Level is supported by your vehicle.
Jul 06 08:58:17 volvo2mqtt [1] - INFO: Success! Electric Range is supported by your vehicle.
Jul 06 08:58:17 volvo2mqtt [1] - INFO: Success! Estimated Charging Time is supported by your vehicle.
Jul 06 08:58:17 volvo2mqtt [1] - INFO: Success! Charging System Status is supported by your vehicle.
Traceback (most recent call last):
File "/volvoAAOS2mqtt/./main.py", line 13, in
authorize()
File "/volvoAAOS2mqtt/volvo.py", line 52, in authorize
check_supported_endpoints()
File "/volvoAAOS2mqtt/volvo.py", line 159, in check_supported_endpoints
state = api_call(entity["url"], "GET", vin, entity["id"])
File "/volvoAAOS2mqtt/volvo.py", line 250, in api_call
return parse_api_data(data, sensor_id)
File "/volvoAAOS2mqtt/volvo.py", line 297, in parse_api_data
return charging_connection_states[data["chargingConnectionStatus"]["value"]]
KeyError: 'CONNECTION_STATUS_FAULT'

Would you please enable debug and send the logs ?

Done by verison v1.7.4! Are you able to test this change ?

Yep, updated and tested and now shows correct info, thanks!

Nice. Thanks for testing!


Firewall flagging keybase "does not have a valid digital signature"

Hardly a bug but just an FYI really - Norton Security doesn't like you.
This happened during an update.

CC: @zanderz
We will look into it right away


Updates SegmentedControl to Alpha status

@siddharthkp - it's working as expected. We removed that functionality in https://github.com/primer/react/pull/2220


Is there an equivalent to *gorm.DB.NewRecord() that was in GORM 1?

Your Questio n
In GORM 1, there was a function to check if a record was new, by seeing if its primary key was set. This does not exist anymore in GORM 2.
The document you expected this should be explained
It doesn't seem to be in the changelog of breaking changes.
Expected answer
Is there an equivalent function, or a new way of achieving this (without knowing about the actual model being checked)?

Hello,
Could you let me know your use case that need to know it is a new record or not?
There were many people thought GORM will check database to check if the record is new or not, to avoid the misunderstanding, we removed the method.
We haven't list some changes like this one that could be caught by the compiler in the release note, but let me know your use case first.
Thank you.

@jinzhu I have a use case where some code handles arbitrary data types, where the primary key is unknown. I used NewRecord() to see if the primary keys were set on the item before performing an update (since doing an update on an item with empty keys would instead create a new one).

@jinzhu I have a use case where some code handles arbitrary data types, where the primary key is unknown. I used NewRecord() to see if the primary keys were set on the item before performing an update (since doing an update on an item with empty keys would instead create a new one).

So if no primary key exist, no operation will perform?

So if no primary key exist, no operation will perform?

Or more so, the behaviour differs depending on whether or not there is a primary key present.

So if no primary key exist, no operation will perform?

Or more so, the behaviour differs depending on whether or not there is a primary key present.

usually the upsert / insert ignore could handle most cases, if this is required, maybe write a wrapper based on the API listed here https://gorm.io/docs/write_plugins.html
I didn't saw a strong requirement currently, so compare to the confusion it caused and the benefit, I prefer to not have it right now


4.08 support

On a 4.08.0+beta1 switch:
File "ast_convenience.ml", line 1:
Error: The implementation ast_convenience.ml
       does not match the interface ast_convenience.cmi:
       Values do not match:
         val has_attr : 'a -> ('a Location.loc * 'b) list -> bool
       is not included in
         val has_attr : string -> Parsetree.attributes -> bool
       File "ast_convenience.mli", line 108, characters 0-42:
         Expected declaration
       File "ast_convenience.ml", line 121, characters 4-12:
         Actual declaration

I believe this is addressed by #70.

The master branch installs fine on 4.08. Closing.


On descriptions

Thank you for your awosome work. I notice that your svit descriptions do not cover all images in vg. Do you filter out these images for some reason(low quality e.g.)? Or just not include them? Thank you!

Hi @John-Ge, thanks for your interest in this repo. For detailed description, the dataset contains around 106K instances, where around 2K instances are filtered out for better quality. For example, some answers generated by GPT-4 may tell that the information is based on the given "captions" and "descriptions". We exclude those instances from the result.


Is pinatra's routing as functional as klein.php's?
How do i use twig with pinatra?
Does pinatra scale well (for use in both small and large apps)?
Can i use routing in multiple paths?
example
/
/admin
L index.php
index.php

i want my routes to be separate for admin and for the rest of the app (and to be located in index.php and admin/index.php)
What about i18n?

I don't think so
I don't think so
actually I don't know now
Yes, it is just a library. You can use it as you want.
Yes, it is just a library.
actually I don't know now

will #3 and #6 be solved? Also, benchmarks when?


Create updated WinGet package for March 2024 GDK Update 4

Create updated WinGet package for March 2024 GDK Update 4
Microsoft Reviewers: Open in CodeFlow

@microsoft/xbox-developer-experience-toolbox for review


Bad performance when using Instancio for Entities in a data model

Describe the bug
We have a somewhat big data model, that has references between all the classes. I'm trying to create an object, that has a single reference to another object. This reference field is being set to a custom value using the set method on InstancioOfClassAPI. I suspect that instancio creates a builder, that also is able to build this reference class, which also has references to other classes. If this is the case, then this ends up in Instancio making a builder for the whole data model (+- 30 classes). This ends up in the Instancio.of(<myclass>) call taking up to around 1 second, making it not usable for my tests. (Also tried this on a DTO without references to other classes, and here it ran within a few milliseconds as expected.)
Expected result
Ideally, if a field is ignored or set, the api won't create a builder that includes this field, so that performance can be improved a lot.
Actual result
Seems like now, Instancio always creates a builder that is possible to populate all the fields, making it slow on "bigger" data models.
Additional information
Add any other context about the problem here
Sample code reproducing the issue
Instancio.of<myclass>.class).set(field("myfieldToOtherClass"), customCreatedInstance).create()

Instancio version:
2.6.0

@jur-clerkx, thank you for reporting this. In general, Instancio does not provide any performance guarantees, as long as the performance can be considered reasonable for a unit test. It's hard to quantify reasonable because that obviously depends on the amount of generated data and complexity of the data model.
Regarding the InstancioOfClassApi builder object, there's only one instance created per Instancio.create and Instancio.of. This is also true when generating collections and streams.
If you could provide a reproducible sample, that would help a lot to troubleshoot (proprietary classes can be renamed, if that's a concern).  If not, then I can think of the following workarounds to speed things up:

set default collection sizes to [min=1, max=2] via Settings
ignore() fields that are not needed by a particular test case (especially, cyclic OneToMany collection fields).

I suspect that your class probably has many cyclic relationships. Instancio will try to generate data until a cycle is detected, which is then terminated with a null value. A sample would be helpful to investigate further.

@armandino Thanks for the reply. Sadly I can't provide the code, since this is the code of a client of ours.
As you suggested, I've tried to use the ignore() method for relation we don't use. However, this didn't change anything in the performance of the creation of the object (still, 3 objects takes around 5 seconds), although the generated object indeed did not have the external field filled.
I ran the test in a profiler, and it seems that the issues is in the NodeFactory. createChildrenFromFields method. It will still create nodes for the ignored field, which in turn will cause for the whole whole data model to be processed.

@jur-clerkx can you try the latest version 2.7.0 and see if you notice any difference?

@jur-clerkx I'm going to close this issue. Please re-open if you're still having issues after upgrading to 2.7.0.

Hi @armandino,
I've retested this, with the 2.8.0 version. Performance seems to have improved slightly (3,5 seconds to generate 3 objects), but still too high to actually use.
Some context for our project. We've around 20-25 classes which are connected to each other. Most of these are 1-* relations. Using ignore() or a Settings object to reduce the collection size doesn't help here.
As you can see in the Flame Graph below from the profiler, almost all of the times is in creating the NodeFactory to create the objects. Seems to me that https://github.com/instancio/instancio/issues/428 is also related to how this part of the library functions. I think that including the selectors (e.g. set or ignore) in the first stage, because if these are used, you wouldn't need the Nodes for these fields.

@jur-clerkx thanks for the follow up. It's good to hear there was at least a minor improvement. I'd love to investigate this further, but I will need a reproducible sample to see what's going on.
If you can provide a sample with the classes renamed, that would help a lot. Or if you prefer to provide it privately, feel free to reach me directly at my GH username @ gmail.

@jur-clerkx not sure if you had a chance to try version 2.9.0. I believe it should improve the performance issue you were having. At least you should be able to speed things up now by setting withMaxDepth() to a lower number (currently, the default is 8), e.g.:
Foo foo = Instancio.of(Foo.class)
    .withMaxDepth(4)
    .create();

You also made a good point about handling ignore() during node creation (the first stage). I made changes to the implementation based on your suggestion in #439. Now using ignore() should actually speed things up. This change will be released in version 2.10.0.
I'll keep looking for other areas to make improvements. In the meantime, let me know if you run into any other issues. Thank you again for looking into this!


Feature Request: Changing audio format for the output

Hello and thank you for your script,it works without problems in normal use.I wonder if there is a way to change the audio format from "vorbis" to "aac" or "mp3" because I also use some older hardware that unfortunately does not support "vorbis".I looked into the script for a mention of 'vorbis" but did not find anything to change.
All I could find is a mention of an audio filter but I do not know where that filter resides and how I can edit it,I am a newb unfortunately.Any kind of help is appreciated.

I don't use vorbis specifically. It's possibly just the default of ffmpeg (which the script uses under the hood).
It might also be possible that your input source uses vorbis/ogg.
I guess it could be possible to change the output audio format to something else. Feel free to submit a patch. Unfortunately I won't have time to fix it in near future.

No problem,take as much time as you need,there is no reason to hurry working on the script.I think vorbis is indeed the default for ffmpeg.I will try to find a way to change the default,I will search for a solution.Thank you for your answer

for f in .$typeaudio ; do python3 source/silence_cutter.py "$f" output/$dt/"${f%.}.opus" $setdBsilence ; done
for f in .mp3 ; do python3 source/silence_cutter.py "$f" output/"${f%.}.opus" -35 ; done
it sets the output to opus or whatever you specify


Angular 2 Support

Do you guys already have an official plan for supporting Angular 2?  I'm planning to work on integrating this on an Angular 2 project.

Hi @bangonkali Angular 2 isn't on our roadmap at the current time. However, we can assist with any questions you may have during your integration.

Another vote for Angular support for the File Explorer.

@jenkinshouse noted! I'll bring it up internally.


res_class==NULL to properly get the resource

Not sure what "String" was supposed to do, my tests with xcb-xrm gives that
res_class==NULL properly gets the resource value, =="String" does not.

It was a quick rewrite so it's probably left from the xlib version.

Corrected.


PageHeader SecondaryCommands

Is it possible to change the background color on the Secondary menu?
Also is it possible no to stretch the menu entire width of the screen?

Here's the solution for you.
        <CommandBar>
            <CommandBar.CommandBarOverflowPresenterStyle>
                <Style TargetType="CommandBarOverflowPresenter">
                    <Setter Property="Background" Value="Red"/>
                </Style>
            </CommandBar.CommandBarOverflowPresenterStyle>
        </CommandBar>


dashboard doc updated

links and structure

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Activity formatting issue

I'm having this issue where my activity does not format as it should, and takes up a large amount of space. Is there anything that might be missing from my readme or the workflow that could be causing this?

Can you please share your current workflow and the config file of the recent-activity action?

Hey @DeviousLab
I have looked in your profile repo, and I have found there's no config file at all
There's where the problem comes I guess, it is required to have a config file
You can copy our example config file and put it in your profile repo, inside the .github folder (not inside .github/workflows)
Please let me know if the problem is solved after adding the config file so we can close this issue!

@DeviousLab Please check PR on your Repo which will resolve this issue.
Please reopen this issue if it's not resolved after merging the above mentioned PR.

Thank you all for your help, the issue is resolved now!


Login by email function does not work

Describe your bug here.
I just installed to give it  a try but the "Login to the network" online option does not work, I have already an account bc I normally play on pc.
Steps to reproduce this bug.
Go to online, options and try to log in.
Command Prompt/Terminal logs
No response
Have you modded the build?
Yes, at Source Code level
If you edited anything in this build, mention or summarize your changes.
Nope
What is your platform?
Windows

The platform is Android but It does not apear in the list

The controllers are broken On fnf online
Uploading Record_2024-09-22-20-54-33.mp4…

update to 0.7.8

I actually update it still broke
On Sun, 22 Sep 2024, 9:00 pm Lily, @.***> wrote:

update to 0.7.8
—
Reply to this email directly, view it on GitHub
https://github.com/MobilePorting/Funkin-Psych-Online-Mobile/issues/1#issuecomment-2366776823,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BKHZ4JRFY4YL7JXKVQLN6NTZX25QTAVCNFSM6AAAAABOURIGHCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRWG43TMOBSGM
.
You are receiving this because you commented.Message ID:
@.***>

Ok i just found out the problem, I was thinking that I have to press the "A" button to confirm the email but it seems that makes nothing... I just tryed to use the enter button on the android keyboard multiple times and I succesfully loged in after a few trys

Oh thanks the game actually reset my progress I lost my awards for no reason


Incorrect Email Validation Regex

Describe the bug
Email validation is notoriously hard as the rules are complex and cannot be directly translated into a regex.  However the email validation regex snippet in this does project is broken as to not obey some of the basic rules.

The local part should allow all of the following characters:

!#$%&'*+-/=?^_`{|}~

The local part should not be able to start or end with a . (full-stop).

Currently the domain part allows any number of non-linebreaking characters (including all special characters of any type): .+..  Domains should not have special characters.

Assuming that .+. should be \.+, the limiting of the top-level domain to 2-4 characters long prevents many (most) of the newer top-level domains.

To Reproduce
Steps to reproduce the behavior:

Use a Regex validator, paste in the regex generated by this extension
The following should be acceptable but are currently not:

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>(this currently matches due to the . matching issue).

The following should not be acceptable but currently are:

john<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

Expected behavior
As explained above
Additional context
Even with these changes this will not be a comprehensive email validator but catches the basics which matches most implementations.

Hey @jonathantwite sorry for the late reply, If I am not wrong the regex code provided by you is not passing any of the tests, and yes, you were right about it failing for all the test cases mentioned by you.
I have come up with another regex code like: /^[\w-\.]+@([\w-]+\.)+[\w-]{2,4}$/g
Right now the above expressions fails for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>Do let me know what you think

@jonathantwite any updates? 😁 thanks!


Error running cargo build, "failed to join search paths together"

I'm trying to do my first Cargo build and I'm getting an error:
$ cargo build
failed to join search paths together: path segment contains separator `:`
Does $DYLD_LIBRARY_PATH have an unterminated quote character?

$ echo $DYLD_LIBRARY_PATH

$ rustc --version
rustc 0.13.0-nightly (34d680009 2014-12-22 00:12:47 +0000)

$ cargo --version
cargo 0.0.1-pre-nightly (e11c317 2014-12-21 20:43:45 +0000)

My $DYLD_LIBRARY_PATH appears to be empty, is that causing the problem? What's supposed to be in that env var? This is a fairly fresh install of OS X, but I do have XCode installed.

I'm having a very similar issue:
Microsoft Windows [Version 6.1.7601]
Copyright (c) 2009 Microsoft Corporation.  All rights reserved.
C:\Users\chas1>cd projects\hello_world
C:\Users\chas1\projects\hello_world>\Rust\bin\cargo.exe build
failed to join search paths together: path segment contains "
Does $PATH have an unterminated quote character?
C:\Users\chas1\projects\hello_world>

I don't have unterminated quote character, they all match up as far as I can tell. I thought perhaps a backslash might have been escaping an ending one, so I deleted all trailing backslashes before quote characters.
Here's my path env var:
C:\Users\chas1\projects\hello_world>echo %path%
C:\Program Files\Common Files\Microsoft Shared\Windows Live;C:\Program Files\Windows Resource Kits\Tools;C:\oracle\prod
uct\11.2.0\client_1;C:\oracle\product\11.2.0\client_1\bin;C:\Anaconda;C:\Anaconda\Scripts;"C:\Program Files\WATCOM\BINNT
";"C:\Program Files\WATCOM\BINW";"C:\Program Files\Silverfrost\FTN95 Express";"C:\Program Files\ActiveState Komodo Edit
7";C:\Windows\system32;C:\Windows;C:\Windows\System32\Wbem;C:\Windows\System32\WindowsPowerShell\v1.0;"c:\Program Files
\Microsoft SQL Server\100\Tools\Binn";"c:\Program Files\Microsoft SQL Server\100\DTS\Binn";"C:\Program Files\Microsoft A
SP.NET\ASP.NET Web Pages\v1.0";"C:\Program Files\Microsoft SQL Server\110\Tools\Binn";C:\Python27;C:\mingw64\bin;"C:\Pro
gram Files\QuickTime\QTSystem";"C:\Program Files\R\R-3.0.2\bin\i386";"C:\Program Files\TortoiseSVN\bin";"C:\Program File
s\Git\cmd";"C:\Program Files\Windows Kits\8.1\Windows Performance Toolkit";"C:\Program Files\nodejs";"C:\Program Files\M
icrosoft SQL Server\120\Tools\Binn";"C:\Program Files\Microsoft\Web Platform Installer";"C:\Program Files\TortoiseGit\bi
n";"C:\Program Files\MongoDB 2.6 Standard\bin";C:\GitHub\mean-stack-skeleton;c:\GitHub\mean-stack-skeleton\node_modules
.bin;"C:\Program Files\Microsoft SDKs\TypeScript\1.0";"C:\Program Files\cURL\bin";C:\cURL\bin;C:\Go\bin;C:\TortoiseHg;C
:\Program Files\Microsoft SDKs\TypeScript\1.0;C:\Program Files\Windows Kits\8.1\Windows Performance Toolkit;C:\Program
Files\ChucK\bin;C:\Program Files\GtkSharp\2.12\bin;C:\Program Files\Microsoft SQL Server\110\Tools\Binn;C:\Program Fil
es\Microsoft SQL Server\110\DTS\Binn;C:\Program Files\Microsoft SQL Server\110\Tools\Binn\ManagementStudio;C:\Program
Files\Microsoft Visual Studio 10.0\Common7\IDE\PrivateAssemblies;C:\Program Files\Windows Live\Shared;C:\Program Files
TortoiseGit\bin;C:\Users\chas1.kre\packages\KRE-CLR-x<IP_ADDRESS>-beta1\bin;C:\Users\chas1.kre\bin;c:\Java\jdk1.8.0\bin;C:
\Users\chas1\AppData\Local.meteor;C:\Rust\bin;C:\Users\chas1\AppData\Roaming\npm;

Needless to say, this isn't a Mac, so it just may be some implementation error in the language...

Sorry, I didn't catch that it was broken up  mid word
like that, in many places. That's what happens when you paste from the command line into a browser, I guess.

@alexcrichton It looks like you just can't escape it at all, at least when interacting with bash: http://stackoverflow.com/questions/14661373/how-to-escape-colon-in-path-on-unix

@sfakler While that is interesting, mine is not identical to the OP's problem, merely similar, unless it is related to the colons' handling in "cargo build." Both are "failed to join search paths together", where brianloveswords's complains about colons, and mine complains of double quotes not being terminated.

Ok, I've figured out a rather inconvenient workaround.
After altering my PATH, to eliminate all the colons, and still getting the same error, then eliminating all backslashes before my semicolons (fail,) then finally eliminating the path altogether, it compiles, but then complains about a missing libgcc_s_dw2-1.dll (see link,) when I try to run the executable. I subsequently had to set the path again (sigh.) What a pain.

@charlesprogrammr does your current directory have a quote character anywhere inside of it? The error here can probably be more descriptive, yes, but the error conditions on windows is that a path has a " character in it (which is different than on unix).

No. I've never even attempted to use a quote in a directory name. It has something to do with the actual %PATH% environment variable, I think. With the variable out of the environment, it complied. I'll just use 2 command line windows, 1 with and 1 without until it gets straightened out. I haven't looked at the source code for Rust yet, I imagine it is some strange edge case, since only 2 of us seem to have reported anything about it.

Not at all, I think it has something to do with the value of my PATH environment variable. I'll just run 2 command line windows one with the variable and one without it until things get straightened out.

Could you paste the output of this program?
#![feature(core, env)]
use std::env;
fn main() {
    let var = env::var("PATH").unwrap();
    for path in env::split_paths(&var) {
        println!("{:?}", path);
    }

    let v: Vec<_> = env::split_paths(&var).collect();
    println!("{:?}", v);
    println!("{:?}", env::join_paths(v.iter()));
}

I haven't yet got it to compile, seems std::env is unstable and not included in the install.

Seconding the quotation mark (") problem on Windows 8.1.
failed to join search paths together: path segment contains `"`
Does $PATH have an unterminated quote character?

@argontus could you try running this program and gist the output?

@alexcrichton Here you go: https://gist.github.com/argontus/8b7c0f18af5c2e4208cc

@argontus could I see the source of the crate you're trying to build as well?

@alexcrichton Sure, actually it is the default hello world program that's created by issuing
cargo new hello_world --bin

fn main() {
    println!("Hello, world!");
}

The command I'm trying to run is
cargo build

as documented in the starter guide. :)

@argontus sorry I'm at a bit of a loss, I'll try to add some more error reporting to help figure this out

If you can build Cargo manually it may be helpful to build with https://github.com/rust-lang/cargo/pull/1291 and see what the output is.

@alexcrichton I'm getting stuck at installing all necessary dependencies (pkg-config etc.) under MSYS. Is there any way to build cargo on Windows without all the unix deps that are annoying to find/compile under different OS? Sorry if I'm hopeless, I haven't used make and other unix tools in years...

Oh most of the listed dependencies aren't actually necessary for windows (aka pkg-config), you could try just running ./configure and make and see what happens! I think you'll at least need to install cmake though.

@alexcrichton Right, I tweaked the configure script a bit and removed the pkg-config dependency, and I also had to change https to http in src/etc/dl-snapshot.py. Now make runs for some time, until...
 ...
 Downloading tar v0.1.10
 Downloading winapi v0.1.11
failed to join search paths together: path segment contains `"`
Does $PATH have an unterminated quote character?
make: *** [cargo-x86_64-pc-windows-gnu] Error 101

@alexcrichton I had time to test cargo on my work laptop, which has Windows 7 installed. cargo build works perfectly fine on the cargo version that comes with current Windows x64 installer (cargo 0.0.1-pre-nightly (9404539 2015-02-09 20:54:26 +0000)).
I'm starting to suspect the culprit here might be Windows 8.1, since my PATH is pretty similar on both machines, but that's just a wild guess. :)

@alexcrichton Just a last spam, sorry for noise :) I just tried to remove Rust completely from my home box (Windows 8.1), reinstalled from latest nightly x64 installer and now cargo build works.
cargo 0.0.1-pre-nightly (9404539 2015-02-09 20:54:26 +0000)

Now all is in order with my setup, at least. Yay!

Good call, argontus. I didn't think to look at nightly builds for a particular platform, I assumed the install on the homepage would be updated on an ongoing basis. My mistake was to assume. I just now uninstalled and installed rust-nightly-i686-pc-windows-gnu.exe, and I no longer have problems with %path%. I even got it to compile the PathFromEnv program
C:\Users\chas1\projects>cd PathsFromEnv

C:\Users\chas1\projects\PathsFromEnv>rustc PathsFromEnv.rs
PathsFromEnv.rs:1:12: 1:16 warning: unused or unknown feature, #[warn(unused_features)] on by default
PathsFromEnv.rs:1 #![feature(core, env)]
 ^~~~

C:\Users\chas1\projects\PathsFromEnv>dir
 Volume in drive C has no label.

Gah sorry everyone! That should have been the first question I asked...
Sounds like this is solved in more recent versions of cargo though!


$ variables as "recomendation"

Is your feature request related to a problem? Please describe.
in this snippet
# login to a server and forward to ssh key (dangerous but usefull for bastion hosts)
ssh -A <user>@<server>

I would like to give just "recommendations" for user.
This just sets it to whoami
$ user : whoami

This opens fzf and let me type something but typing something like root lets navi crash
$ user : whoami && echo

thread 'main' panicked at 'byte index<PHONE_NUMBER>3709551615 is out of bounds of ``', src/libcore/str/mod.rs:2017:9
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace.

I also tried
$ user : whoami && echo --- --print-query

but this resulted in the same error.
Describe the solution you'd like
I would like to write this
$ user : whoami && echo "root" --- --print-query

or
$ user : whoami --- --print-query --no-select-1

to use whoami and root only as recommendation, so I can type what ever I want.

In this case, if I type d<enter> what should be the expected value? d or denis.isidoro?

Good point.

fzf returns what you typed as the first line and the best result as the second one

I see, I'm not too familiar with fzf. Did I make my idea, workflow clear? Do you think this make sense? or is possible to implement?

Yes! My only question is: in your case, if you type p<enter> I don't know what to return. If p or palo.
If you say palo, what if the user really wants p as value?
If you say p, then the suggestion won't be useful.
WDYT?

Hmm not a simple question I would say. I guess it makes more sens palo if I would set <user> to peter I have to write it out anyway. The only problem arises when I want to set <user> to a value which is a substing of palo, which is  pal or pa or p.
In this case it seems obvious that should not be a problem, but in other cases
it might result in problems. I can't think of any so far though.

This may work, I guess
x="$(echo -e "$(whoami)\nroot" | fzf --print-query --expect tab,enter)"; echo "x is $x."

So when the user presses tab it will use the second line and the user presses enter it will use the first line?

Yes, unless the user hit enter without input. Then it would use the last line. Does that make sense?

Make sense, I mean it is kinda hidden in the cheat file. The user does not know of this behavior, because he can't see it. But I guess this is a good default.
So tab and enter will be interpreted like that?

Yeah, the final syntax could be:
$ user: whoami; echo root -- --allow-extra

and the code would take care of the rest.
A documentation entry could be provided.

A documentation entry could be provided.

Do you mean someone (I could do that) should write something in the README or a explanation window in fzf will be set and describes the tab and enter interpretation?

Hmm, I'm not sure I'm following. Before documenting it we need to implement it hahaha
You can't add any custom fzf option after ---.
That's why I suggest we create the --allow-extra option and add the relevant code.

I could try to implement it, tomorrow though. It's quite late where I am.

I have some time today, will work on this issue.

I'm on that, and it seems --multi and --allow-extra can't be used at the same time.


send all debug, info, and debug messages to stderr

What this Pull Request (PR) does
all messages go through stderr. only the relevant output goes through stdout
Related issues
NA
Screenshots
NA

I see that now there are conflicts, I will try to rebase and submit again.


dumping gifs not working?

Dumping gifs seems to not work, I tried running passing the -dg option with a couple of environments for over 30 mins with over 400 episodes and it never seems to output gifs
I ran these commands:
python3 coach.py -p CartPole_DQN -dg
python3 coach.py -p Breakout_A3C  -dg
here you can see output:
https://cl.ly/173A092l112H
find experiments/
experiments/
experiments/asdsad
experiments/asdsad/20_03_2018-14_29
experiments/asdsad/20_03_2018-14_29/run_dict_worker3.json
experiments/asdsad/20_03_2018-14_29/run_dict_worker0.json
experiments/asdsad/20_03_2018-14_29/run_dict_worker1.json
experiments/asdsad/20_03_2018-14_29/run_dict_worker4.json
experiments/asdsad/20_03_2018-14_29/run_dict_worker2.json
experiments/asdads
experiments/asdads/20_03_2018-14_30
experiments/asdads/20_03_2018-14_30/run_dict.json
experiments/asdads/20_03_2018-14_30/worker_20_03_2018-14_30_0.csv
experiments/sadd
experiments/sadd/20_03_2018-14_29
experiments/sadd/20_03_2018-14_29/worker_20_03_2018-14_29_0.csv
experiments/sadd/20_03_2018-14_29/run_dict.json
experiments/gifs
experiments/gifs/20_03_2018-14_36
experiments/gifs/20_03_2018-14_36/run_dict.json
experiments/gifs/20_03_2018-14_36/worker_20_03_2018-14_36_0.csv
experiments/efewf
experiments/efewf/20_03_2018-14_36
experiments/efewf/20_03_2018-14_36/run_dict.json
experiments/efewf/20_03_2018-14_36/worker_20_03_2018-14_36_0.csv

Did you changed the evaluation parameters?
If you didn't they may be set to way more then 400 episodes.

I have similar issue, dump gif option is only working for preset "Breakout_A3C". I have tried it with other presets but, it doesn't seem to be producing any gifs. I tried to change configuration.py parameters but that didn't fix anything.

@itaicaspi-intel should i add these evaluate_every_x_episodes and evaluation_episodes parameters for every preset I want the gifs to be produced in the following file ?
https://github.com/NervanaSystems/coach/blob/master/presets.py

this is solved for me as well by using the suggested values, I would suggest explicitly talking about this in the docs and giving an example preset that will save gifs more frequently.


Fix frontend simulator gke cue config

From #923 _simulator_image removed from frontend simulator definition and local deployment, but not for gke deployment.

This change is


GDScript: Fix setter being called in chains for shared types

When a type is shared (i.e. passed by reference) it doesn't need to be called in a setter chain (e.g. a.b.c = 0) since it will be updated in place.
This PR adds an instruction that jumps when the value is shared so it can be used to skip those cases and avoid redundant calls of setters. It also solves issues when assigning to sub-properties of read-only properties.
There are some duplicate issues for this that I could find:
Fixes #28720
Fixes #31245
Fixes #57793
Fixes #60796

Unfortunately this seems to break access to types that are returned by value:
var dict : Dictionary = { "A":Transform2D() }
dict["A"].origin = Vector2(1.0, 2.0)

Has no effect
Same goes for:
var tr = Transform3D()
tr.basis.x = Vector3(2.0, 0.0, 0.0)


Drop cuinj64 and accinj64 libraries

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Fixes https://github.com/conda-forge/cudatoolkit-feedstock/issues/30

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

The recipe license should not include the word "License".

For recipe:

License is not an SPDX identifier (or a custom LicenseRef) nor an SPDX license expression.

Documentation on acceptable licenses can be found here.

@conda-forge-admin, please re-render.

Are we not allowed to redistribute these? If not, we should delete all the existing packages.

Are we not allowed to redistribute these? If not, we should delete all the existing packages.

We aren't, but I think the lack of including these in the EULA is moreso surrounding them not having any real use outside of the profilers distributed by NVIDIA.
If deleting the packages requires any real effort or is problematic in any way then I'm okay with leaving them inplace for now until someone asks us to remove them. If that ask comes and they question why things weren't deleted then we can happily link to this comment where I, an NVIDIA employee, guided the community in this direction.

@jakirkham changes LGTM. Would it be possible to add a test that asserts that these libs aren't in the package?

Thanks @kkraus4. I think it's okay to delete them as they weren't published to main. ccing @conda-forge/core for thoughts.
Btw, can someone go through all the files and check that they are covered by the EULA?

Btw, can someone go through all the files and check that they are covered by the EULA?

I'm going through now.

Btw, can someone go through all the files and check that they are covered by the EULA?

I'm going through now.

Went through and the only library we're packaging that's not included in the EULA is libcusolverMg.so. This feels like an oversight in the EULA so have reached out internally before we remove the package.

I have a perhaps very silly question 😅 If NVIDIA is (obviously) supportive behind all these efforts, can't we safely ignore EULA and do whatever we think is best for the community?

I have a perhaps very silly question 😅 If NVIDIA is (obviously) supportive behind all these efforts, can't we safely ignore EULA and do whatever we think is best for the community?

Unfortunately we are still beholden to our legal department and ultimately they could demand things get removed regardless of our involvement / protest.
I'm all for us continuing to package things reasonably when there's questions regarding the EULA, but it's best to have us (NVIDIA) pre-emptively start conversations as opposed to knowingly continue against the EULA.

Going to go ahead and merge while we are still figuring out libcusolverMg.so. Also deleted all of the existing cudatoolkit packages.

Also deleted all of the existing cudatoolkit packages.

Unfortunately this caused Anaconda.org to forget all of the package details as we saw in issue ( https://github.com/conda-forge/cudatoolkit-feedstock/issues/2 ). Am going to repeat the same process of downloading the packaging manually uploading it again as described here ( https://github.com/conda-forge/cudatoolkit-feedstock/issues/2#issuecomment-708652960 ) to fix that issue.

Ok so the description is fixed if the package is main. However as soon as I move it to the test label it goes away. Not sure if there is a good way for us to fix this.

I've stuck packages under the test label for now. This seems to be the least of two evils atm. Happy to revisit if either someone can provide a better solution or we decide to move packages to the main label and still have issues.

Downloaded the 9.2 package right after CI published it to main from the merged PR ( https://github.com/conda-forge/cudatoolkit-feedstock/pull/44 ) and reuploaded it just now to make sure the UI render correctly ( https://github.com/conda-forge/cudatoolkit-feedstock/issues/2 ). Looks like it is working correctly again 🎉


mention migration, cosmetics

I was searching on what the order_column should look like (rtfm) and did not immediately (light speed) find it, so I guess putting the instructions in a more declarative manner (and with syntax highlight) might make sense. Besides, include a sample migration for existing data (#75).
The rest is cosmetics.

Looking good :) Just a few tweaks :)

I will look into it in a calm moment; currently its crazy times. Like "No time(s)".
Btw, the description of the github repo also states "An acts_as_sortable/acts_as_list replacement built for Rails 3 & 4". :D

Sure is crazy times. The whole of NZ is just stuck at home for 4 weeks. Unfortunately I can't change the description until @mixonic transfers this project over to me. :) Were you still keen to do this @mixonic?

afaics, I implemented all proposed changes. Thanks @tjwallace . Although I'd keep the procedural example commented out.

Excellent @fwolfst, thanks for your hard work! :D


Cannot require gem; pull request ready

@graysonc , I just submitted a pull request which should allow users to use the gem without using the require_relative command. This is helpful for PaaS like Heroku where the file path can vary. Please let me know if you have any questions or comments!

Merged #3. Thanks, @laneystroup