How to tell if a user is being impersonated?

Is there any method or property inject on the current_user object to tell if they are being impersonated? Would like the ability to not do certain things in regular Ruby objects if the user is being impersonated, ex. sending events to Intercom.
Thanks,
Travis

Travis,
Take a look at devise_helpers.rb: https://github.com/rcook/user_impersonate2/blob/master/lib/user_impersonate/devise_helpers.rb: you can see that we use session[:staff_user_id] to store the current staff user's ID. You could certainly use this. Perhaps even better would be to compare current_user with current_staff_user:
if current_staff_user && current_user != current_staff_user
  # We're impersonating
else
  # We're not impersonating
end

You should obviously test this out thoroughly since I haven't had an opportunity to verify it myself.

@supercodepoet Did you manage to find a solution for this ? I would also love to disable intercom, but also tracking ... when we're impersonating a user

I'm working on this too now. The only way I found is this ugly code to disable intercom when having current_staff_user.
#config/initializers/intercom.rb
staff_user = ''
config.user.exclude_if = Proc.new do |user|
                           begin
                             staff_user = true if current_staff_user
                           rescue
                             staff_user = false
                           end
                           staff_user || !user.is_admin_of_a_store?
                         end

I've tried before
config.user.exclude_if = Proc.new { |user| current_staff_user || !user.is_admin_of_a_store? }

But current_staff_user || !user.is_admin_of_a_store? returns NameError: undefined local variable or method 'current_staff_user' for main:Object

I'm wrong current_staff_user is never accessible from the initializer. Is their a way of doing it ? :/

I finally did something like
#app/controllers/application_controller.rb
class ApplicationController < ActionController::Base
  skip_after_action :intercom_rails_auto_include, if: :current_staff_user?

  private

  def current_staff_user?
    session[:staff_user_id]
  end
end

And it's work.

@benoittgt: Did any changes related to this make it into the code? You are now an owner, so feel free to do what needs to be done to address this issue in the gem itself.


Add ModelState entries for greedy and type-matching model binders

part II of II for #2445
enable tests that #2445 was blocking
fix these and other tests expecting different ModelState entries

Supersedes #2696 and addresses comments on that PR

@rynowak please double-check this PR, commit f0ae34d and the new doc comments in ValueProviderResult in particular. I've made all changes we discussed in #2696.

:shipit:

715a0b6


Feature Request: Sorting Conversations by Top Votes?

I'd like to even have it implemented if this can be open for consideration. How about sorting conversations (on issues and PRs) by the votes they receive? Similar to StackOverflow?

Issues and PRs are not question and answers they are a conversion.
By sorting messages in this conversation you would also change the order in which they happened, which would be total confusion like on who is relying to what.

Thanks for the suggestion, but this is not something we want for Refined GitHub.


Too many deprecations warning on install

#npm install enduro -g

npm WARN deprecated<EMAIL_ADDRESS>express 2.x series is deprecated
npm WARN deprecated<EMAIL_ADDRESS>connect 1.x series is deprecated
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>graceful-fs v3.0.0 and before will fail on node releases >= v7.0. Please update to<EMAIL_ADDRESS>as soon as possible. Use '
npm ls graceful-fs' to find it in the tree.

Not sure we can do much about this. These are deeply nested:
├─┬<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ │ ├──<EMAIL_ADDRESS>deduped
│ │ └──<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ │ ├──<EMAIL_ADDRESS>│ │ ├──<EMAIL_ADDRESS>│ │ ├──<EMAIL_ADDRESS>deduped
│ │ ├──<EMAIL_ADDRESS>deduped
│ │ ├─┬<EMAIL_ADDRESS>│ │ │ ├──<EMAIL_ADDRESS>│ │ │ └──<EMAIL_ADDRESS>│ │ └─┬<EMAIL_ADDRESS>│ │   ├─┬<EMAIL_ADDRESS>

Why do you need browser-sync?
On Wed, Aug 23, 2017, 12:16 Martin Gottweis<EMAIL_ADDRESS>wrote:

Not sure we can do much about this. These are deeply nested:
├─┬<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ │ ├──<EMAIL_ADDRESS>deduped
│ │ └──<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ │ ├──<EMAIL_ADDRESS>│ │ ├──<EMAIL_ADDRESS>│ │ ├──<EMAIL_ADDRESS>deduped
│ │ ├──<EMAIL_ADDRESS>deduped
│ │ ├─┬<EMAIL_ADDRESS>│ │ │ ├──<EMAIL_ADDRESS>│ │ │ └──<EMAIL_ADDRESS>│ │ └─┬<EMAIL_ADDRESS>│ │   ├─┬<EMAIL_ADDRESS>—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Gottwik/Enduro/issues/111#issuecomment-324271333, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGFipm1fdKfo8asRP37O3yI9nz8eSM7zks5sa-3wgaJpZM4O_ANi
.

browser-sync is used for local development. It makes page reloads on content files and template changes, and reloads stylesheets and fonts on-the-fly
Sent from my Xiaomi MI MAX using FastHub


Need to Configure SMTP, Not sure where to begin.

Hello, I am trying to figure out how to configure SMTP. I am using Prometheus-operator with Grafana as the front-end along with Kubernetes and I need to set up an Alert Notification using SMTP. I am having a hard time finding any tutorials. Can anyone help me? -Thank you

So you are using Grafana for alerting? If yes, you just need to edit the configmap of grafana (grafana.ini) with the additional settings. http://docs.grafana.org/installation/configuration/#smtp
If you are using Alertmanager https://github.com/prometheus/alertmanager/issues/275


Fix subpath for local requests

Summary
When we create a Client4 struct for local HTTP requests, we construct the URL manually using the ListenAddress, but we don't take into account the server's configured subpath on the site URL. This PR uses the SiteURL to calculate the subpath for the constructed URL.

Codecov Report

Merging #295 (7144378) into master (849a6ca) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master     #295   +/-   ##
=======================================
  Coverage   25.96%   25.96%
=======================================
  Files          73       73
  Lines        4156     4156
=======================================
  Hits         1079     1079
  Misses       2970     2970
  Partials      107      107

Impacted Files
Coverage Δ

server/config/config.go
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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

@mickmister Does this PR fix https://mattermost.atlassian.net/browse/MM-40195?

@hanzei I'm not sure if the second suggestion above needs to be applied. Should we always test with a subpath in the tests?

@hanzei I'm not sure if the second suggestion above needs to be applied. Should we always test with a subpath in the tests?

I like that suggestion. Filled https://mattermost.atlassian.net/browse/MM-40287


feat(clip): add linear probe evaluation script

Adds a script to perform linear probe evaluation using the mlx.data module for data loading. Mostly a mirror of the Linear-probe evaluation script from the official CLIP repository.
References

https://github.com/ml-explore/mlx-examples/issues/959

Another thing to consider would be to have two commands in the linear_probe.py script. One that extracts features and saves them in a safe tensors file and another that does trains the logistic regression classifier given that file. The first part might be generally useful for extracting clip features for a dataset for instance.

Thanks for the addition. I would rename the script to linear_probe.py and add the eval. I am wondering if it would be nicer (since it is an example after all) to train a logistic regression model in MLX instead of using scikit-learn.

What I had in mind when submitting this PR was to showcase similar performance between the official implementation and the mlx port. Adding a mlx implementation of logistic regression seems like a nice idea but IMO it should reside in a different directory. Maybe another misc/ or core/ directory that contains implementations of various fundamental models.

Genlte ping @angeloskath


Refactor Line Merging of Equations, on Backspace.

When there is an equation at the end of a line and we are backspacing
from the line below. Instead of trying to treat the equation at the end
of a line as a special case (and delete it), we place the cursor at the end of the
equation and then let it be handled as the case of deleting a equation
within a line, thus the user presses backspace once to go up a line and
then again to delete an equation.

The changes look good, but can we not commit the build files?
Just add build/ to .gitignore

We could use https://www.npmjs.com/package/grunt-exec to run make during the web client build process, that would generate the build directory. Another option would be a githook that reminds us to run make before committing.

Closing until changes to the build process are merged.


[Event-Change Request] Table 121 - Purch. Rcpt. Line - OnInsertInvLineFromRcptLineOnBeforeCheckPurchLineReceiptNo

Dear ALAppExtensions team,
through this issue we would like to request a change of event publisher OnInsertInvLineFromRcptLineOnBeforeCheckPurchLineReceiptNo in Table 121 Purch. Rcpt. Line from:
[IntegrationEvent(false, false)]
local procedure OnInsertInvLineFromRcptLineOnBeforeCheckPurchLineReceiptNo(var PurchRcptLine: Record "Purch. Rcpt. Line"; var PurchLine: Record "Purchase Line"; var TempPurchLine: Record "Purchase Line"; var NextLineNo: Integer)
begin
end;

to:
[IntegrationEvent(false, false)]
local procedure OnInsertInvLineFromRcptLineOnBeforeCheckPurchLineReceiptNo(var PurchRcptLine: Record "Purch. Rcpt. Line"; var PurchLine: Record "Purchase Line"; var TempPurchLine: Record "Purchase Line"; var NextLineNo: Integer;var IsHandled: Boolean)
begin
end;

adding an IsHandled parameter to the event which would be implemented as follows:
procedure InsertInvLineFromRcptLine(var PurchLine: Record "Purchase Line")
var
    PurchInvHeader: Record "Purchase Header";
    PurchOrderHeader: Record "Purchase Header";
    PurchOrderLine: Record "Purchase Line";
    TempPurchLine: Record "Purchase Line" temporary;
    TransferOldExtLines: Codeunit "Transfer Old Ext. Text Lines";
    ItemTrackingMgt: Codeunit "Item Tracking Management";
    TranslationHelper: Codeunit "Translation Helper";
    PrepaymentMgt: Codeunit "Prepayment Mgt.";
    NextLineNo: Integer;
    ExtTextLine: Boolean;
    IsHandled: Boolean;
    DirectUnitCost: Decimal;

begin
    SetRange("Document No.", "Document No.");
    TempPurchLine := PurchLine;
    if PurchLine.Find('+') then
        NextLineNo := PurchLine."Line No." + 10000
    else
        NextLineNo := 10000;

    if PurchInvHeader."No." <> TempPurchLine."Document No." then
        PurchInvHeader.Get(TempPurchLine."Document Type", TempPurchLine."Document No.");

    // >>>> New >>>>>>>>>>>>>>>>
    IsHandled := false;
    OnInsertInvLineFromRcptLineOnBeforeCheckPurchLineReceiptNo(Rec, PurchLine, TempPurchLine, NextLineNo,IsHandled);

    if not IsHandled then
    // <<<< New <<<<<<<<<<<<<<<<
        if PurchLine."Receipt No." <> "Document No." then begin
            PurchLine.Init();
...
end;

Thanks in advance!

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 49479.


Russian lang added

Added Russian language

@turbotankist Wow, This is fantastic, Thanks so much for your work here. Will merge in now for the next patch release.


Test against Ruby 2.7.0

Coverage decreased (-0.009%) to 97.89% when pulling dc2338a63d858060d874c127d636635961a578d3 on tknzk:test-against-ruby-270 into 703875d59a82af924a2ebaf21737ff8a0f1306b5 on onelogin:master.


"Third-party software operators" is not clear in Platforms paper

I have read the paper https://github.com/cncf/tag-app-delivery/blob/platforms-v1alpha1/platforms-whitepaper/v1alpha1/paper.md
Thanks for that! It is quite informative.
https://github.com/cncf/tag-app-delivery/blob/945b4d4cd765780e91645882ac25a8d58749b802/platforms-whitepaper/v1alpha1/paper.md?plain=1#L143
Regarding the sentence above, I think Third-party software operators is not fully clear in the paper context. It can be elaborated with an example or defined in Glossary again.

Thanks @erkanerol, good feedback! We'll work on clarifying this.

Hi @erkanerol, in the final commit I changed that sentence to this:
Operators of third-party software can provision required dependencies like databases on demand and easily run that software.
WDYT?
The idea of the item is that folks will want to run third-party software like a CRM or ERP on the platform. This and MLOps are meant to serve as additional use cases beyond first-party development, which we believe is just the first adopter of "platforms."

@joshgav Thanks for taking care of this!
The unclear point for me is what the operators word refers to in this context. Real people or software?  The other items refer to real people like developers, managers, etc.

Thanks @erkanerol I hear you. Do you think administrators might be clearer than operators? I.e. this:
Administrators of third-party software can provision required dependencies like databases on demand and easily install and run that software.

Went ahead with the final adjustment to "Administrators". We can always update for v1.1 :laughing:!


Pull RandomSeedHash Generation Out of UtxoView

Current dependencies on/for this PR:

feature/proof-of-stake

PR #830

PR #848   👈

This stack of pull requests is managed by Graphite.


improvement(dev-ex): compatible/identical types should be aliased and reused where possible

It would dramatically improve the crate's usability if instead of defining discrete types for every nested object within a CRD, compatible/identical types were defined once and automatically reused.
For example, the HeaderMatch type is identical between GRPCRoute and HTTPRoute resources, and do not require discrete handling when writing code to handle them. Another good example would be the way that HTTPRouteFilters are a superset of GRPCRouteFilters and can therefore also be handled by the same code without special consideration (i.e. any code that handles all HTTPRouteFilter types inherently also handles all GRPCRouteFilter types).
As the crate currently is, a dev would have to write such code with the newtype pattern or work a bit of black magic to skirt the orphan rule. A much more ergonomic solution would be if types that were actually the same to simply have exported aliases where you'd expect them to be ([example]) or for types where the compatible type is effectively a subset to impl From<SubsetType> for SupersetType ([example]).
In the case of aliased types, I'd imagine we'd want something like:
//! NOTE - module structure and naming are for
//! illustrative purposes only, actual real world
//! names and structure should be assumed to carry
//! on as they currently are

pub mod shared {
    // ...

    #[derive(
        Eq,
        Clone,
        // ...
    )]
    #[serde(tag = "type", rename_all = "PascalCase")]
    pub enum HeaderMatch {
        // ...
    }

   // ...
}

pub mod grpcroute {
    // ...

    pub type GRPCHeaderMatch = super::shared::HeaderMatch;

    // ...
}

pub mod httproute {
    // ...

    pub type HTTPHeaderMatch = super::shared::HeaderMatch;

    // ...
}

and in the case of compatible types:
//! NOTE - see note in example above re: naming and structure

pub mod shared {
    // ...

    #[derive(
        Eq,
        Clone,
        // ...
    )]
    #[serde(tag = "type", rename_all = "PascalCase")]
    pub struct RequestHeaderFilter {
        // ...
    }

    #[derive(
        Eq,
        Clone,
        // ...
    )]
    #[serde(tag = "type", rename_all = "PascalCase")]
    pub struct RequestMirrorFilter {
        // ...
    }

   // ...
}

pub mod grpcroute {
    // ...

    #[serde(tag = "type", rename_all = "PascalCase")]
    pub enum GrpcRouteFilter {
        // ...

        /// ...
        #[serde(rename_all = "camelCase")]
        RequestHeaderModifier {
            request_header_modifier: super::share::RequestHeaderFilter,
        }

        // ...
    }

    impl From<GrpcRouteFilter> for super::httproute::HttpRouteFilter {
        // ...
    }
}

pub mod httproute {
    // ...

    #[serde(tag = "type", rename_all = "PascalCase")]
    pub enum HttpRouteFilter {
        // ...

        #[serde(rename_all = "camelCase")]
        RequestMirror {
            request_mirror: super::shared::RequestMirrorFilter,
        },

        /// ...
        #[serde(rename_all = "camelCase")]
        RequestHeaderModifier {
            request_header_modifier: super::share::RequestHeaderFilter,
        }

       // ...
    }
}

@the-wondersmith @kflynn and myself all sync'd about this recently. This would be a large boon to the ergonomics of using the library. There were a few options on the table, but we all agreed doing this at the CRD level in the upstream repository might be the best one because it has the additional potential to help other languages and clients in the future.
/cc @aryan9600 as this also relates to our statuses work in #37

@shaneutt @kflynn just as a general "keep alive", what are our next steps here?

(Don't worry, this has not fallen off my radar I have a reminder set for it)
I was hoping to hear from @aryan9600 (ping!) but I think the general idea of the Gateway API project providing CRD information for client generation seems generally helpful and reasonable to ask for at some level. Basically any proposal should follow the GEP process, and in this case I think it would be fair to start a discussion referencing both this issue and #37 and reasoning, and then we can continue discussion there as well as bring it up at the community meeting.
Once it's had some time for discussion, we'll see if we can gather any other allies and then ultimately find out who are going to be the champion(s) driving it forward to completion. Please feel free to start that discussion!

@the-wondersmith checking in?

@shaneutt apologies, I genuinely thought I'd opened that discussion. I've for sure done so now 😅.
Excited to get some feedback there

No worries! Let's discuss there for a bit and see what people think :saluting_face:

I think we should try to bring this topic up in a Gateway API community meeting to try to spark more interest. If no further interest can be sparked, then it might be a situation where we just ask (the Gateway API community) "is anyone opposed to us creating a GEP for this?" and see where we can go from there.

I think we should try to bring this topic up in a Gateway API community meeting {...}

Agreed. When / where is that?

https://gateway-api.sigs.k8s.io/contributing/#meetings <- it alternates, but right now: 3 weeks it's Monday at 3pm PST and one week it's Tuesday at 8am PST.

(Don't we now alternate 3PM PST and 8AM PST every other week?)

Ooop yep, Flynn is right :+1:

So we haven't gotten any feedback in upstream, so I guess it's really up to us how motivated we are, and what specific way we're motivated to do this. My thoughts at this point is that this would be ultimately more beneficial to add in kopium because I expect it will have a positive impact on many projects which use that, whereas I wonder if we do this in Gateway API if it will be there just for us for a prolonged period of time. Thoughts?

So we haven't gotten any feedback in upstream, {...}. My thoughts at this point is that this would be ultimately more beneficial to add in kopium {...}. Thoughts?

Apologies for the delay, the notification got buried 😖.
I couldn't agree more - kopium seems like exactly the right place for this, as it seems like it would be much easier to add type-reuse at the point of generation than as an after-thought.
What's a good starting point for us on that path?

@clux I think at this point we should ask your opinion on the matter:
tldr; when generating code from CRDs there can be duplicated types which are identical (e.g. re-used sub-types in the source API). We have been considering how we could influence kopium to identify these before emission, and collapse duplicates into single emitted types in the Rust code.
(more details above and in the links)

This would be a great ergonomic improvement in general to have.
My bet here would be on some extra key/info in the schema that can serves as some kind of namespaced $ref replacement. If we had that, then I imagine we could make the matching easier (cross a struct once in iteration, use this new key for naming the struct, short-circuit struct creation next occurrence to avoid producing duplicates).

NB1: It's a shame that we can't set $ref in crd schemas because they are very helpful for bigger code-generators like k8s-openapi / k8s-pb to ensure we have one instance of the structs. Upstream issue: https://github.com/kubernetes/kubernetes/issues/62872
NB2: The thing recently started doing; matching for well-known apimachinery type structs is so heavy handed that it's not really fit for this purpose / exposable (without injecting arbitrary code somewhere).

Thanks for your thoughts on this @clux we appreciate it!
In terms of priority this is probably our highest priority issue because we consider it a requirement for a v1 release, and we've heard that there's some interest from Linkerd folks in this project that hinges on getting this resolved.
This is definitely NOT a good-first-issue, however if someone is really interested in digging in (to kopium) it seems quite open for someone from the community to jump in here and take a look at what can be done so we'll mark it as help-wanted.

Potential brute-force solution, given the lack of standard signalling: https://github.com/kube-rs/kopium/issues/298

That sounds like a great first big step, thank you for writing that up @clux. :bow:
With that implemented I think we could consider this resolved as it covers a lot of the problems (cc @the-wondersmith @kflynn, let me know your thoughts on that). The remaining limitations (including de-duplicating with k8s-openapi types)  sound like a good follow ups we should be able to track separately and iterate towards.

@shaneutt I think @clux's proposal seems exactly as advertised - a big ol' hammer, but importantly a functional hammer 😅. I agree that, once implemented, it would largely cover sizable portion of the problems and provide a solid base for future iterative solutions.
I'd be interested to hear @kflynn's thoughts though, as I know he's recently been doing some cross-language type work in the emissary codebase, and so might have more "recently been in the trenches" insight than I can offer at the current moment.


[SSOC'23] Find minimum in rotated sorted array

Describe the solution you'd like
Suppose an array of length n sorted in ascending order is rotated between 1 and n times. the algorithm will return the minimum element of this sorted rotated array of unique elements. The solution will be in c++.
Additional context
Example:
Input: nums = [3,4,5,1,2]
Output: 1
Explanation: The original array was [1,2,3,4,5] rotated 3 times
kindly assign me this issue under SSOC'23.

@Kumar-laxmi please assign me this task under ssoc'23

@Kumar-laxmi please this issue to me so that i start working on this under ssoc'23 program.

I can solve this issue.
It can be solved by two approaches.
We can either sort the array so that the first element will be minimum.
Or we can linearly travel in the array and when we find an element which is smaller than its previous element, then that is the minimum element in the rotated sorted array.
Kindly assign me this issue so that I can work on this.

@kum

Describe the solution you'd like Suppose an array of length n sorted in ascending order is rotated between 1 and n times. the algorithm will return the minimum element of this sorted rotated array of unique elements. The solution will be in c++.
Additional context Example: Input: nums = [3,4,5,1,2] Output: 1 Explanation: The original array was [1,2,3,4,5] rotated 3 times
kindly assign me this issue under SSOC'23.

@Kumar-laxmi  kindly assign me this issue.


cli: Add release-stream flag

What this PR does / why we need it:
Allow to specify a release-stream like 4.14.0-0.nightly instead of a full pull-spec release-image

/approve

/lgtm


Add Semaphore for Blockchain Transactions From the same account.

An invalid nonce error will be thrown if the same account tries to write to ethereum at the same time.
We added a hacky retry in #30
We need to consider what parallel write means for the sidetree protocol.
I think its generally invalid for the same account to try and anchor 2 different files at once.
While this happens in tests, in production each node should have a unique funded hot wallet, and this sort of thing should not be happening.

Let me know your thoughts.

If the error happens only while testing, then we should modify our test setup, not accomodate the codebase for the tests. Three solutions come to mind:

Add a semaphore on the web3 instance, so that only one sidetree service can access it at the time (requires all sidetree instances to use the same web3 instance)
Sleeping a few ms after an RPC call to wait for consistency to be reached
Use different accounts for different instances of the sidetree service. That requires having a test service that manages which funded web3 accounts are used and which are not.

https://ethereum.stackexchange.com/questions/39790/concurrency-patterns-for-account-nonce
We currently support Candidate 2 in element-lib.
I think its the best because all the other ones require coordination which will not always be possible.
Most recent since the contract interface used by element is very small I think its ok to add the retry loop.
https://github.com/decentralized-identity/element/blob/development/packages/element-lib/src/adapters/blockchain/ethereum/index.js#L75


Same Results with Different Seeds (random_state) in LightGBM

Reproductive codes
import pandas as pd
import numpy as np
import lightgbm as lgb
from sklearn import datasets
iris = datasets.load_iris()
X = iris.data[:, :2]  # we only take the first two features.
y = iris.target
model_1 = lgb.LGBMClassifier(random_state = 1)
model_1.fit(X,y)
p_1 = model_1.predict_proba(X)
model_2 = lgb.LGBMClassifier(random_state = 2)
model_2.fit(X,y)
p_2 = model_2.predict_proba(X)
print(all(p_1[:,0] == p_2[:,0]))
print(all(p_1[:,1] == p_2[:,1]))
result:
True
True
Bug: Same Results with Different Seeds (random_state)
Although model_1 and model_2 are trained with different seeds (i.e., 1 and 2), both models predict the same probability for given dataset. In fact, should different models (which are trained with different seeds) generate different results?

if you don't use any random functions, like sub_feature or sub_row, the results are deterministic.

if you don't use any random functions, like sub_feature or sub_row, the results are deterministic.

Thanks for your replying. Actually, this is the reason why lightgbm generates the same result with different seeds. Now this issue should be closed.


How to Enable Extension after migrate to Isaac Lab

I tried to modify every 'orbit' keywork in this template to 'lab' including extension.toml, It will works fine as standalone project.
then I tried to enable this extension in the Isaac Sim GUI, particular I get the error of :

I noticed this repository hasn't been updated for a while especially after major migration to isaac lab has happend.
I am curious how this template's future  plan will be after the orbit migrated to isaac lab. Should I stick to this template?

We have now updated the template. Please feel free to open a new issue if you see any problems.


[feature] #2280: Produce permission events when role is granted/revoked

Signed-off-by: Shanin Roman<EMAIL_ADDRESS>

Description of the Change

Add mechanism to return multiple events from modify_* functions;
Fixed error when revoked role does not exist;
Revoke/Grant role also produce PermissionAdded/Removed events.

Issue
Closes #2280.

Benefits
More detailed events.

Possible Drawbacks
None.

/build " && ls "

/build skip_testing

Roles are also revoked when unregistering a role, so if we want to cover all such cases, we should generate token events there.

Good catch, thanks


When active, Youtube mute and volume up/down have seconds long delay

Once the extension takes control of the audio on YouTube, YouTube's volume up/down and mute functions act on the audio seconds later. It's making me entirely reliant on the extension's gain controls and my system's volume controls.

That's just how the browser works, I can't do anything about it.


Add hashcode and equals to RandomService

The absence of hashcode/equals was the reason of map growing at #883

Thanks


23436 Gökdere, Turkey

Subject: 23436 Gökdere, Turkey
Name: Ahmet İhsan Aytek
Email<EMAIL_ADDRESS>Tel:
Fax:
Fast reply reguested: No
Message:
It was cited from Geraads & Gulec (1999) that bovid is Prostrepsiceros rotundicornis for this locality. But the Gökdere locality which was
mentioned in Geraads & Gulec (1999) is not in Eskişehir but in Ankara.
Şenyürek, M. (1952). A Study of the Pontian Fauna of Gökdere (Elmadağı), South-East of Ankara. Belleten 16(64): 449-492.


chaintip fixes #8

@chaintip fixes #8

If this pull request is merged to fix issue #8 first, @adangert will claim the 0.00396228 BCH| ~   5.14 USD bounty and all future tips to the issue.

@chaintip fixes #8

Nice work @adangert! Your pull request has been merged and the bounty of 0.00396228 BCH| ~   5.14 USD has been sent to your wallet.


add amplify studio blog

https://aws.amazon.com/jp/blogs/news/aws-amplify-studio-figma-to-fullstack-react-app-with-minimal-programming/

thank you for approved


C-Card Alt Tags & Accessibility: removed alt contents

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

#sign-off apexprod

@cmcclister Hi - What is the reason for remove the alt text?

@cmcclister Can you connect to the originator of this request?

FYI, I believe the ALT text is being removed because it is duplicated in the text that follows, so the user experience to a screen reader user is to say the same text twice in a row. @cmcclister should be able to explain where the request came in though.

@MikeWasson @bennage @DuncanmaMSFT alt tags with values are breaking accessibility when used in the C-Cards, per @adkinn

OK, thanks!


Xeno won't execute lots of scirpts

So I'm executing a fling script but

It shows this, please fix it and make it be able to execute and scripts that's not supported like moonsec or others

the error is on the script lil bro

no the issue is with the xeno enviroment code, the userdata is broken, the script works on any other executor like solara but not on xeno, it got an userdata problem i fixed it my self

Thanks LuauDev! I’ll make sure to remember this


[hotfix] Prevent empty playerId or spectatorId in create methods.

Added tests.

Codecov Report

Merging #13 into master will increase coverage by 0.13%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #13      +/-   ##
==========================================
+ Coverage   91.47%   91.60%   +0.13%
==========================================
  Files           2        2
  Lines         129      131       +2
  Branches       39       39
==========================================
+ Hits          118      120       +2
  Misses         11       11

Impacted Files
Coverage Δ

src/server/datastore/ephemeral.ts
89.52% <100.00%> (+0.20%)
:arrow_up:

Continue to review full report at Codecov.

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


Fix build issue in c++14 for condition_variable

This fix fixes the issue raised in #12017 where condition_variable_any has to be used in c++14.
This fix defines ConditionVariableForMutex as std::condition_variable_any in c++14 and std::condition_variable otherwise.
The fix is verified with
bazel build -s --config=opt --cxxopt=-std=c++14 ...

This fix fixes #12017.
Signed-off-by: Yong Tang<EMAIL_ADDRESS>

Can one of the admins verify this patch?

@tensorflow-jenkins Test this please

@tensorflow-jenkins test this please

This message was created automatically by mail delivery software.
A message that you sent could not be delivered to one or more of its
recipients. This is a temporary error. The following address(es) deferred:
<EMAIL_ADDRESS>Domain mazecreator.com has exceeded the max emails per hour (27/25 (108%)) allowed.  Message will be reattempted later
------- This is a copy of the message, including all the headers. ------
Received: from github-smtp2-ext7.iad.github.net ([<IP_ADDRESS>]:36921 helo=github-smtp2a-ext-cp1-prd.iad.github.net)
by server2.lowesthostingrates.com with esmtps (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384:256)
(Exim 4.89)
(envelope-from<EMAIL_ADDRESS>id 1deqzR-0001hV-Vg
for<EMAIL_ADDRESS>Mon, 07 Aug 2017 17:59:44 -0500
Date: Mon, 07 Aug 2017 16:10:07 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com;
s=pf2014; t=1502147407;
bh=2aZ/WyRSvWnUPS4jcNY7PWof7l7vVce+6y5FIIdMiCQ=;
h=From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID:
List-Archive:List-Post:List-Unsubscribe:From;
b=NKzYnkkfMaovDTeH5AQ+zdovYzsCNosLfgN2ol76RuhIUEFwpYeKr7l8VVmin3AKS
KxP0WJEcAzQHfP+RMIlN/Rq79I5vspz9J3evXalbNcEallRedMt6bB9VhspS71QnRU
4wDZM1TBCcMnRDvPGtA4RDKhJ/Nm4OgMX/kmpzMM=
From: Rasmus Munk Larsen<EMAIL_ADDRESS>Reply-To: tensorflow/tensorflow<EMAIL_ADDRESS>To: tensorflow/tensorflow<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>In-Reply-To<EMAIL_ADDRESS>References<EMAIL_ADDRESS>Subject: Re: [tensorflow/tensorflow] Fix build issue in c++14 for
condition_variable (#12027)
Mime-Version: 1.0
Content-Type: multipart/alternative;
boundary="--==_mimepart_5988f34f91a7d_c3b3fccd759bc38387836";
charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: rmlarsen
X-GitHub-Recipient: Mazecreator
X-GitHub-Reason: subscribed
List-ID: tensorflow/tensorflow <tensorflow.tensorflow.github.com>
List-Archive: https://github.com/tensorflow/tensorflow
List-Post<EMAIL_ADDRESS>List-Unsubscribe<EMAIL_ADDRESS>https://github.com/notifications/unsubscribe/ARjzoDEka7ONoea0l5kGet6XwPWeIeeOks5sV5lPgaJpZM4OtQ1m
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address<EMAIL_ADDRESS>X-Spam-Status: No, score=
X-Spam-Score:
X-Spam-Bar:
X-Ham-Report:
X-Spam-Flag: NO
----==_mimepart_5988f34f91a7d_c3b3fccd759bc38387836
Content-Type: text/plain;
charset=UTF-8
Content-Transfer-Encoding: 7bit
@tensorflow-jenkins test this please
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/tensorflow/tensorflow/pull/12027#issuecomment-320804718
----==_mimepart_5988f34f91a7d_c3b3fccd759bc38387836
Content-Type: text/html;
charset=UTF-8
Content-Transfer-Encoding: 7bit
@tensorflow-jenkins test this please
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or mute the thread.

----==_mimepart_5988f34f91a7d_c3b3fccd759bc38387836--


Add CloudWatch Agent

🚀 Feature Proposal
All OpenVPN instances should be sending all their local logs to CloudWatch via the CloudWatch Agent.
Motivation
All instances should be doing this, and even more so for the OpenVPN instance since it sits on the edge.

Issue resolved via #13 and cisagov/openvpn-packer#7.


how to inherit from this 'inherited' theme?

Hi, I'd like to make use of the grav inheriance feature. But inheriting this theme doesn't seem to work. Also, "class Mytheme extends quark-open-publishing" doesn't seem to work as class name doesn't allow hyphens.

Hi @lustyffh , this is all setup in the Open Publishing Skeleton with a pre-built 'mytheme' , available at https://getgrav.org/downloads/skeletons
Please let me know if you can get things going from there etc.

Closing this, but please feel free to re-open.

@paulhibbitts Thanks a lot! As to the class Mytheme extends - the key was to type in theme name omitting hyphens - something not mentioned explicitly in Theme Inheritance documentation. Also, one shouldn't forget to copy all theme configurations in user/config/themes to the new theme file.


Short method names not working

Describe the bug
I have a class KeeperDialogManager.cs with a public method Emote(string character, string emote) that I want to call from a script. KeeperDialogManager is an autoload and the global checkbox is checked. However, inside a dialogue script, the function does not get called properly if I just write eg Emote("player", "happy"), it only works if I write the full KeeperDialogManager.Emote("player", "happy"). Do you know why this might be?
Affected version

Dialogue Manager version: 2.35.0
Godot version: v4.2.1.stable.mono.official [b09f793f5]

To Reproduce
Steps to reproduce the behavior:

Call function without class name

Expected behavior
Function should call properly
Screenshots

Wait hold on, I was just checking the global checkbox in the autoload settings. Though this from the documentation appears to be a different checkbox? Do you know where i could find this? It isn't in the Project settings > Dialogue Manager panel

It's in the Dialogue Settings window that you can open from this button:

You can also add using KeeperDialogManager to the top of a dialogue file and then shortcuts work the same way.

Thanks for the help, didn't realize that settings panel existed.

So I checked the box here, the shortcut works if I put using KeeperDialogManager at the top like you said, but without it I get an error saying the function was unable to be found. I double checked that KeeperDialogManager is being autoloaded and is global. Any ideas?

Hold on, it just started working as expected now. Maybe I just needed to restart the editor or something. Thanks for the help in any case.


How should we categorize stories (technically)?

Given the taxonomy of stories we'll need, does it make sense to create individual categories in the database or to use a tagging system?
https://github.com/thecarebot/carebot/wiki/Story-taxonomy
Requirement is that every story needs to be tagged across each of the 4 key categories.

@livlab Are we still going to categorize stories? The wiki page is gone/moved.


Cocoapod?

It would be great if this library was published as a Cocoapod.

Thanks for creating the podspec. Merged and closed.


Connectivity issues can result in episodes being marked as read.

Describe the bug
Anytime is not distinguishing between an episode stopping because it has come to the end or as a result of a connection issue or drop out. If you have mark played episodes as read enabled in settings, this can result in an episode being marked as played when it hasn't.
To Reproduce
Steps to reproduce the behaviour:

Enable mark played episodes as read
Begin streaming an episode.
Enable airplane mode and wait for buffer to run out.

Expected behaviour
If, when streaming, an episode stops due to a connectivity error the current play position should be recorded and the episode should not be marked as played.

Hi @amugofjava ,
I would like to work on that issue.
Would it be ok for you ?

Yes, I did mean that @Chralu.

Hi,
I tried to replicate this bug on the simulator (iphone) but failed: the player remains waiting for connection and does not go into time-out. At that stage, the pause button does not respond to the command.
Closed and reopened the application, the episode used as a test, still remains to be read.
I also ran the test on an Android device and the behavior is identical.

Hi @mrkrash,
Thanks for checking this out. It looks like this issue has changed since it was logged. Trying this now on iOS, the playback goes into loading state when the buffer is exhausted and stays like that, with the spinner spinning - it doesn't seem to ever time out, which is not good. It's better on Android in that it does eventually timeout, but once the connection is available again it can take two taps on the play button to get it playing again.

Hi @amugofjava,
I tried to better understand why it does not go into timed out and it seems that the problem is in just_audio library. The latter does not change state despite the lack of connection. There also seems to be an open issue about it.
I'll try to see if there is a way to recover the connection timeout status and, if so, pause the player.

Hi @mrkrash,
Thank you for diving into this. Could you please add a link to the open just_audio issue - I would be interested in taking a look (there are a lot of open issues and couldn't spot this one)?
Thank you.

Sure, https://github.com/ryanheise/just_audio/issues/117


Different colours on different windows

Using DBeaver 22.0.1 on Win 10, I tend to open up a few different windows with all tabs in 1 window for a specific DB connection, useful to help prevent doing stuff on the wrong DB. But what would be even more useful is a way to colour-code each window. Doesn't need to be an automatic colour-allocation based on DB connection as that would need to be per tab, but come to think of it that might be useful as well...? But even just a toolbar button to set background or windows header bar colour for that window, but that window alone, would be great. Not sure if this is even possible in Windows though, unless using completely customised "skin" type system, a la Winamp and similar.
Thanks all for maintaining this superb and essential tool.

Hello @longj0n
Maybe ability to create custom connections type with the coloring can help you?
You find it in the Connection settings -> General -> Edit connection types.
Usually, our users use Production connection type to mark important databases.
And you can create your own connection type.

Hi Anastasiya, thank you so much, yes this is of course exactly what I
need to prevent nasty accidents!
I should have known this was already a feature but didn't realise
where to find it.
Kind regards,
Jon
Quoting Anastasiya @.***>:

Hello @longj0n
Maybe ability to create custom connections type with the coloring
can help you?
You find it in the Connection settings -> General -> Edit connection types.
Usually, our users use Production connection type to mark important
databases.
And you can create your own connection type.
--
Reply to this email directly or view it on GitHub:
https://github.com/dbeaver/dbeaver/issues/16114#issuecomment-1091743483
You are receiving this because you were mentioned.
Message ID: @.***>

Thanks for the answer @longj0n


Support for mio 0.7?

Hi! I was taking a stab at upgrading a crate of mine that uses mio 0.6 and mio-extras 2.0 to mio 0.7, which made significant API changes (cf blog post linked to by the mio CHANGELOG). An immediate change is that Evented has been replaced by Source; I haven't looked too deeply at whether the other changes impact mio-extras more severely.
I realize this crate is in maintenance mode and this may be beyond the scope of what you were hoping to do. Any interest in pursuing this? Or would you be open to a PR moving to mio 0.7? I assume that would require a major version bump on mio-extras to 3.0.

The removal of Registration and SetReadiness looks like a much more painful change.

I'm unlikely to do this myself; but if you - or anyone else - would like to contribute a PR then I'd absolutely be open to merging and publishing.

@jgallagher let me know if you need help with this. I don't use this crate but would like to see winit to be upgraded to mio 0.7...

@kaimast I'm not currently working on this. I'm not exactly sure what to do given the discussion on tokio-rs/mio#1289. For my crate I might be able to use the suggestion there and drop mio-extras altogether (and embed the timeouts into my polling loop), although I'd rather not do that. But given the restriction of a single Waker per Poll, I don't know how you'd implement mio-extra's timers which have no such restriction.

You're right. I don't think this is feasible without extensively changing mio-extra's API. Might be easier to just remove winit's dependency on mio-extra at this point.

I recently built mio-misc as an alternative. Feedback is appreciated.

I'm basically not expecting to do anything much with mio-extras myself and haven't yet looked at your code, but:

if you want to submit a pull request, to get your code into this project, I expect I'd be likely to accept it

that's doubly true if you'd be willing for me to give you commit permissions on this repository, and indicate that you're expecting to stick around and fix issues if needed

or if you prefer to publish mio-misc as an independent project, I'd gladly accept a pull request that directed people towards it

yeah, 0.7 definitely lowered mio qualilty. I'm trying to move to 0.7 but gave up due to lack of timers as well. Looked quickly @ what it would take to pull this crate up to 0.7 but yeah, MIO architecture changed drastically on 0.7 and not for the better.

I've been looking through the crates that are still using Mio v0.6, which by the way is no longer supported, and found this crate/issue. I realise that we've made a number of breaking changes in Mio v0.7 to make the API smaller and that meant that Mio-extras doesn't work with Mio v0.7/v0.8.
The unfortunate truth is that Mio isn't really the right abstraction for a timer or channel implementation. Mio (now) focusses on I/O, dropping everything unrelated to that. Depending on the use case you're already looking pretty similar to a complete (Futures) runtime.
As more concrete solution to move forward:

For channel you can use a pipe to emulate Mio events, it's something I do for example here: https://github.com/Thomasdezeeuw/heph/blob/34a8adc74c32572c8b56d6fef3053161f32953c3/rt/src/channel.rs.
The timers a more tricky to recreate. For Linux their is always timerfd(2) and kqueue has it's own timer support, but neither can beat an in user space implementation when it comes to performance.

More concrete recommendations I can't give without know more of the usage, but I'm always willing to help, just have to ask (ping me on GitHub or Mio's Discord channel).

yeah, 0.7 definitely lowered mio qualilty. I'm trying to move to 0.7 but gave up due to lack of timers as well. Looked quickly @ what it would take to pull this crate up to 0.7 but yeah, MIO architecture changed drastically on 0.7 and not for the better.

@przygienda can you elaborate on this?

While I'd be happy to merge pull requests here that made life easier for folk, I'd also be more than happy to see crates move on to recent versions of mio and drop their mio-extras dependency altogether.
To that end I'd suggest that a good line of attack would be to work through https://crates.io/crates/mio-extras/reverse_dependencies, and figure out what is stopping those crates from getting current.

Currently the newest mio version is 0.8


Sending/Receiving IP Packets Without Root Privilege

Expected behavior
Send and receive IP packets (i.e. ICMP Request/Response) without needing to acquire root privilege on macOS or Linux. It would be more user-friendly if swift-nio could allow user to choose between different socketType (.raw or .datagram) to run under various permission and privilege to send and receive IP packets.
Actual behavior
Using NIORawSocketBootstrap in swift-nio requires the caller to execute under sudo command. Otherwise, Operation not permitted (errno: 1) will be thrown.
Steps to reproduce
Run the following code snippet will result in the following error:
socket(domain:type:protocolSubtype:): Operation not permitted (errno: 1)
import Foundation
import NIO
import NIOCore
import NIOPosix

@main
struct Test {
    public static func main() async {
        let group = MultiThreadedEventLoopGroup(numberOfThreads: 2)
        let client = NIORawSocketBootstrap(group: group)
        let channel = try! await client.connect(host: "google.com", ipProtocol: .icmp).get()
    }
}

SwiftNIO version/commit hash
SwiftNIO 2.58.0
System & version information
Please provide at the very least your operating system and Swift version
information.
macOS Version: Version 13.2.1
platform: MacBook Pro 15 (Intel-based)
swift-driver version: 1.75.2 Apple Swift version 5.8 (swiftlang-<IP_ADDRESS>.2 clang-14<IP_ADDRESS>.100)
Target: x86_64-apple-macosx13.0

Thanks for this! This is a reasonable feature request.
Can you try to investigate a few things for me? Firstly, can you try creating the SOCK_DGRAM/IPPROTO_ICMP socket yourself and then using withBoundSocket to create a DatagramChannel? I'm curious if this gets you enough functionality to get going.

Creating DatagramChannel through socket(AF_INET, SOCK_DGRAM, IPPROTO_ICMP)  and withBoundSocket eliminate raw socket permission issue, although I haven't tried sending anything through the channel. It seems to get things move forward. :). Writing C with Swift is definitely not pleasant.

I think that's the short-term alternative. If it actually works (that is, if the I/O functions for your use-case) then it's a small enhancement to enable datagram channel to do this with a Swift API.

Great! Look forward to seeing official implementation coming to swift-nio in the future!

Take a look at #2516 and see if that's useful for you.

Thanks for adding the support this fast! I replaced the socket(AF_INET, SOCK_DGRAM, IPPROTO_ICMP) with the new implementation. The channel is able to send and receive ICMP packets, as expected. This is hugely helpful! Thanks


How the training set is rendered for Objaverse

Hi, amazing work, and it's nice to see an open-sourced version of LRM!
I wonder how the camera and object scale are defined when rendering the Objaverse dataset.
Based on your code, the camera is at a distance of 2, but can you confirm that?
How is the scene normalized?
Thanks

Hi, the objects are rescaled to a box of [-1, 1], and the camera distances are randomly sampled from [1.9, 2.6] when rendering Objaverse. Once trained, we set a fixed camera distance of 2 for inference.

I'm also curious about the used intrinsics of dataset rendering, for example, fx and fy.
Why the default focal length is set to 384 for 512 resolution?
https://github.com/3DTopia/OpenLRM/blob/8dd348f275c3a00a9b915ee64ee069fa1884b416/lrm/inferrer.py#L91
Thanks!

Also curious about the intrinsics. If I want to render camera distance of 1.0, how should I adjust the intrinsics?

Hi @chenguolin @cwchenwang , apologize for the late reply.
The focal lengths here are set in accordance with the rendered Objaverse images. fx and fy are normalized by image size before sending to model, which means all you need to do is to keep fx/w=0.75 for this version.


LazyLoading enable/disable support per UnitOfWork scope for EF6.x

closed with https://github.com/stoveproject/Stove/commit/cb16c8ead9f0cfb357cb2cbf45ff100cbf7147d7


Installation with pip

Running pip search keras-rl suggests that the package is pip-installable, but when I try to do it I get a "No matching distribution found" error. Is it supposed to work? If not, it would be very convenient.

No matching distribution should mean the package is available but not for your system. What OS and version of python?

Oh! I didn't see any restrictions in setup.py, and I run on Python 2.7 as indicated in the readme. My OS is OSX 10.10.

Since the only release is a release candidate, I though it could be that the --pre switch is needed, but I tried it and it didn't work either. Anyway, as there have been significant changes since then I've installed form head with pip install git+https://github.com/matthiasplappert/keras-rl, and the installation went fine.

It wouldn't be an issue with setup.py. When you upload to PyPI, you have a lot of choices of what to build and upload, and maybe the upload didn't have a universal wheel.
I uploaded some packages not too long ago. If you look at the bottom, you'll see the list of files includes a universal wheel and a source distribution.
https://pypi.python.org/pypi/keras-adversarial/0.0.3
https://pypi.python.org/pypi/keras-tqdm/1.0.7
Whereas in keras-rl, I don't see those files listed.
https://pypi.python.org/pypi/keras-rl/0.1
My guess is the build wasn't fully uploaded to PyPI. I assume the PyPI version will be fixed as the project progresses. Probably better to use the development version for now since there are a lot of updates.

I see! So there's an entry, but nothing uploaded. Well, it makes sense to use the development version, yes. I'll stick to the git installation. Thanks!

I've just published version 0.2.0 to pypi: https://pypi.python.org/pypi/keras-rl/0.2.0
You can now use pip install keras-rl to install the package.


include

@yang-g

Please do not send spam PRs to get a Hacktoberfest shirt. This is against the Hacktoberfest rules.


Support upgrade node k8s version for GKE

rancher/rancher:2.1.7
Users can only upgrade GKE master k8s version in Rancher UI.
Users cannot upgrade node k8s version of GKE in Rancher UI

Link to https://github.com/rancher/rancher/issues/18966

If can be fixed / low risk should get in v2.2 otherwise we'll need to add a warning in the UI explaining that we do not currently support upgrade node k8s version post-upgrade of rancher and that only the master k8s version will be updated.

@davidnuzik
Just tried to upgrade node version in GKE.
upgrading node version means delete and recreate nodes.
It's kind of a dangerous operation.
Maybe we need to think about the UX a bit more.
So I would like to leave it as-is. I will add a warning in UI.
Could you provide the position and the content for the warning message, please?
Maybe we need to file another issue instead of using this one for warning message?

@loganhz I will create a separate issue for the UI warning message and suggest content for it.
I understand the concern regarding upgrading nodes.


Added Information about ReadWriteOnce Image Service and iscsid

Added iscsid information within CSI-Documentation and additional install path for running Image Service with Nutanix Volumes ReadWriteOnce

Thanks @wolfganghuse ! Adding @tuxtof and @vnephologist for their eyes on this review.

@wolfganghuse @tuxtof note that I have a few more updates to this fork before we merge.

i think it will be better to merge this pull request on a separated branch of the official repo
so we can all work on it without commit on wolfgang repo :-D

@vnephologist i think it will be better to merge this pull request on a separated branch of the official repo
so we can all work on it without commit on @wolfganghuse repo :-D

Yeah, sure.  Had originally intended only a few updates, but got carried away. ;)


Inserts fail after 16 items in attached code sample

To reproduce
Run the following code more than 16 times – e.g., if called index.js: (node index.js):
const { nSQL } = require('nano-sql')

async function start () {
  const connectionResult = await nSQL('hellodb')
    .model([
      {key: 'id', type: 'int', props: ['pk', 'ai']},
      {key: 'date', type: 'string'},
      {key: 'description', type: 'string'},
      {key: 'done', type: 'int'}
    ])
    .config({
      mode: 'PERM'
    })
    .connect()

  const addResult = await Promise.all([
    nSQL('hellodb').query('upsert', { date: new Date(), description: 'First item', done: 0 }).exec(),
    nSQL('hellodb').query('upsert', { date: new Date(), description: 'Second item', done: 1 }).exec(),
    nSQL('hellodb').query('upsert', { date: new Date(), description: 'Third item', done: 0 }).exec()
  ])

  console.log(addResult)

  const selectResult = await nSQL('hellodb').query('select').where(['done', '>', 0]).exec()

  console.log(selectResult)
  console.log(selectResult.length)
}

start()

What should happen
The length of the results from the select query should increase steadily.
What actually happens
New inserts begin to fail after 16 copies of the second item have been inserted. (Inserts continue to work for the first and third items.)
Also, the generated IDs are rather odd. I’m not sure if this is for the lexicographical sorting that LevelDB uses:
[{ date: 'Mon May 21 2018 15:42:06 GMT+0100 (IST)',
    description: 'Second item',
    done: 1,
    id: 2 },
{ date: 'Mon May 21 2018 15:42:25 GMT+0100 (IST)',
    description: 'Second item',
    done: 1,
    id: 32 },
{ date: 'Mon May 21 2018 15:42:30 GMT+0100 (IST)',
    description: 'Second item',
    done: 1,
    id: 332 },
…
{ date: 'Mon May 21 2018 15:42:44 GMT+0100 (IST)',
    description: 'Second item',
    done: 1,
    id:<PHONE_NUMBER>333332 }]

Note: I was initially doing this in a “transaction” block and seeing the same results but refactored to remove the “transaction” based on https://github.com/ClickSimply/Nano-SQL/issues/35#issuecomment-384108551 to rule that out as a factor.

Hey thanks for the bug report, this was related to how LevelDB was handling the numerical primary keys.
I've patched the issue, if you npm install from github you'll get v1.6.2 with the fix.  I'll likely push 1.6.2 to NPM later today.
Best,
Scott

@ClickSimply Thanks, Scott :) Just did a quick review of the commit and verified on my end. Fixed in c8a9814ed5c5853daa2b0fa96bef9267044d36f5. Closing.


[Gen 3] Random Battles

Adds scripts and movepools for Gen 3 Random Battles. Only fully evolved pokemon will appear, so only they got movepools. Also activates tests for gen3 and 4 randbats (since 4 was not turned on earlier).
Smogon Thread: http://www.smogon.com/forums/threads/random-battle.3600389/
Testable on http://spacialgaze.psim.us/

I think Wobuffet should be incapable of getting Leftovers due to Gen3 mirror match Wobbuffet mechanics - in fact Leftovers Wobbuffet was banned from official Nintendo tournaments due to likelihood of an infinite loop.


Updateing the CI workflows

I realized that not all workflows are using the poetry.lock as dependency management (publish workflows installing mkdocs via pip). So I worked through it and also optimized it for cached installation.
On https://github.com/MaKaNu/boxsup-pytorch/blob/development/.github/workflows/code_quality_checks.yml you can inspect my newest installation of it. On first glance it looks like more boilerplate code, but instead reduces boilerplate and runner runtime significant. Also makes it simple to run different checks in parallel jobs. The main trick: it uses my composition action https://github.com/MaKaNu/poetry-setup-composite-action/

Hm I see you reference .venv here. Does that mean you must commit your virtual environment to GitHub?


Include Auto Trait Implementations in API listings (and diffs)

This is important, because it is easy to accidentally break semver compatibility with subtle auto trait impl changes. Read https://predr.ag/blog/toward-fearless-cargo-update/ for example.
I considered not showing these by default, and instead adding a --with-auto-trait-implementations as an opt-in, but I think that becomes too complicated. Especially for CI uses cases where you would have to pass to many and too long flags to fully list and diff an API.
So instead, this PR always shows them. If it end up becoming too noisy, my plan is to get rid of --with-blanket-implementations and instead making the default output complete, and then add an opt-in to not include blanket and auto trait impls, called something like --terse or --simplified.
This is taking one step towards solving #36 and #39 and I think #151.

I think this is ready for review. I would like to compare with rustdoc HTML some more, and also probably write some more tests, but I have already done some iterations of this code, and consider it to be in quite good shape already.
Rendering Auto Trait implies is an easy special case, because they never have associated items, so they can always be rendered on one row. To render all kinds of impls, we need to think through how to render associated items. With indentation, maybe?

I have quite carefully compared new output with HTML output, and they are identical as far as I can tell. Merging.


Do not allow unchecking metrics that are needed for combined measurement

User can define a combined metric as name = channel_1_mean-channel_2_mean and uncheck channel_1_mean. This would result in not saving the combined metric name since the channel_1_mean column is not available.
Set measurements dialogue should detect which metrics are needed for the combined measurements and should prevent unchecking (checked, greyed out checkbox).

Done starting from v1.2.4rc35


Added clippy(plus fix warning), fmt, audit and ran cargo fmt.

Added the following:

cargo-audit
cargo fmt
cargo fmt
Added a single dyn for recent compiler/clippy warning.

Might need to update the README for a minimum version of Rust.

closing in favor of #9


Created README.md for my profile.

Your Username
rahulnegi20
What Changes you had made?
I added a folder of my user name and added a README,md there!

@rahulnegi20 it's fine


Invert tlsverify default in API

Fixes #1929
Signed-off-by: Jhon Honce<EMAIL_ADDRESS>

LGTM

/lgtm

bot, retest this please

/approve

/lgtm

nice job @jwhonce


Fix null pointer exception(#1427)

Test failing, does it pass for you locally?

Test failing, does it pass for you locally?

Not tested locally because I don't have a gradle environment. But why is such a simple modification wrong? I haven't seen that the error is related to my modification. I'll test it after I install the environment.

No need to install Gradle, just run the wrapper at the root of the project

Not tested locally because I don't have a gradle environment. But why is such a simple modification wrong? I haven't seen that the error is related to my modification. I'll test it after I install the environment.

Well good question, I admit that Gradle does an amazing job burying any useful information in the sea of output it produces, but after wasting some good 5m I managed to locate the failure:
com.netflix.eureka.util.StatusUtilTest > testGetStatusInfoUnsetHealth FAILED
    java.lang.AssertionError: Excpected NPE to be thrown when healthy threshold is not set
        at org.junit.Assert.fail(Assert.java:88)
        at com.netflix.eureka.util.StatusUtilTest.testGetStatusInfoUnsetHealth(StatusUtilTest.java:52)

Looks like there's a test asserting the existing behavior in eureka-core/src/test/java/com/netflix/eureka/util/StatusUtilTest.java method testGetStatusInfoUnsetHealth.
And apparently
eureka-core/src/main/java/com/netflix/eureka/util/StatusUtil.java does:
        // Only set the healthy flag if a threshold has been configured.
        if (peerEurekaNodes.getMinNumberOfAvailablePeers() > -1) {
            builder.isHealthy(upReplicasCount >= peerEurekaNodes.getMinNumberOfAvailablePeers());
        }

So there's a dependency on the existing behavior.
I'm inclined to decline the change based on this, unless you provide more information on how this affects any real use case / how to trigger this NPE.


failed to connect to server [<IP_ADDRESS>:27017] on first connect [MongoError: connect ECONNREFUSED <IP_ADDRESS>:27017]

Repro steps:

Action: cosmosDB.attachDatabaseAccount
Error type: Error
Error Message: failed to connect to server [<IP_ADDRESS>:27017] on first connect [MongoError: connect ECONNREFUSED <IP_ADDRESS>:27017]
Version: 0.8.0
OS: linux

Closing due to lack of response.


Pre-Selected value not working

Seems like I cannot establish the Pre-selected options even if I placed an object already on the reference model

I saw the same thing.  I found a work-around if you set the extra-setting 'idProperty' it will then work.
(using 2.0.0-beta.10)

I saw the same thing. I found a work-around if you set the extra-setting 'idProperty' it will then work.
(using 2.0.0-beta.10).

@davebednarski thanks your solution worked for me.

Hey guys,
Same issue with same version. Only the label "1 Checked" is correct.
If setting
idProperty: ""
It justs show all element selected. Whereas the model hold the good element selected.
Any idea?
I'll back out to version 1... this beta might not be ready for production.
Cheers

@maxblax it solved the issue for me when I used value "id" for idProperty.

I had the same issue and solved by atting
idProperty: 'id'
to the extra-settings.
This is a quite important issue, I think it should be mentioned in the documentation.