For object type C, type parameter T and mapped type { [K in keyof T & C]: any }, keyof C isn't assignable to K

TypeScript Version: 2.5.2
Code
export interface ConfirmationComponentProps {
  title?: string,
}
function propTypes<OProps extends {}>() : {[K in keyof (OProps & ConfirmationComponentProps)]?: any } {
  return {
    title: null,
  };
};

Expected behavior:
Compile with no errors.
Actual behavior:
Fails with: { title: null }is not assignable to {[K in keyof (OProps & ConfirmationComponentProps)]?: any }`
The problem disappears if I remove the intersection type & and just use ConfirmationComponentProps; or if I replace OProps with a concrete type like:
function propTypes<OProps extends {}>() : {[K in keyof {x: number} & ConfirmationComponentProps]?: any } {
  return {
    title: null,
  };
};

OProps could be any type, in which case, it could have properties that you're simply not satisfying here. Any time that you use a type parameter that's only used in an output position, that probably isn't very correct.
Could you ellaborate what you're trying to achieve?

@DanielRosenwasser  Yes, of course.
The example was just a simplification of the problem.
My case is that I'm doing a react component decorator (HOC); that is a function with type:
type withConfirmation<OProps> =  (c: Component<OProps>) => Component<OProps & ConfirmationModalProps>

So, it returns a new component that receives the original props (OProps) and new ones.
The problem is when I implement the decorator, there I define a new React Component (the wrapper) that expects both properties; and try to define propTypes for it:
interface ConfirmationComponentProps {
   title: string;
}

class ConfirmationComponent extends React.Component<ConfirmationComponentProps & OProps > {
   static propTypes: {
      title: PropTypes.string.;
   }
  ....
}

now, typings for react, define that propTypes has type:
type ValidationMap<T> = {[K in keyof T]?: Validator<T> };

where T are the props, in my case ConfirmationComponentProps & OProps.
As you can see, is not that I'm trying to do something fishy myself. It's just a combination of what i believe two valid usecases. On one side, having a decorator that expands the properties you may pass to the component, on the other side trying to define the propTypes of such a component.
propTypes you may argue is not necessary when you type your components with typescript, and probably will remove then later. But I'm performing a migration from js to typescript, so  I wanted to leave them for now.
Hope it helps clarifying the need!

Is this also the same issue with trying to use the new inferred in type guards with generics?
function foo<T>(x: T) {
    if ("a" in x) {
        x.a // Property 'a' does not exist on type 'T'.
    }
}

Or is this something else?

@mcortesi An even more simplified view of the problem is assignment of the string literal "title" to K extends keyof (OProps & ConfirmationComponentProps):
function propTypes<OProps extends {}, K extends keyof (OProps & ConfirmationComponentProps)>(
    k: K, t: 'title'): K {
    k = t
    return k
};

Unfortunately, since K extends keyof (OProps & ConfirmationComponentProps), it may be a smaller union -- with fewer properties. Here are a couple of calls that show off why k = t is incorrect:
var n: never = propTypes<{}, never>('title', null as never) // 'n' now has the value 'title'!
var a: 'a' = propTypes<{ a: number }, 'a'>('title', 'a') // 'a' now has the value 'title'!

I'll see if I can think of a workaround.

But in the original issue, it's [K in keyof X]: ..., which should span all keys, not K extends keyof X, where it can be fewer.

Good point. But K in ... is modelled as K extends ... most places in the compiler -- it's represented internally as a TypeParameter. So it would need special-case code to behave differently, which is probably not there.

Hi! @sandersn thanks for the reply.
I agree with @jcalz since the example use in in every other example I've used it expands to all the keys. like
type Readonly<T> = {
    readonly [P in keyof T]: T[P];
}

implies all keys of T must be present.
It's probably a special case, as you mention.
Nevertheless, it worths nothing that if I don't use the intersection types (&), it works:
function propTypes2() : {[K in keyof (ConfirmationComponentProps)]?: any } {
  return {
    title: null,
  };
};

An idea here, is that it might be related to the fact that, since OProps is unkwown by propTypes, keyof (OProps & ConfirmationComponentProps) means I know only the keys of ConfirmationComponentProps, but not of OProps. So it's impossible for the function to create a object with those keys. In my example it should be possible since I'm using {[K in keyof (OProps & ConfirmationComponentProps)]?: any } with the ?, so It's a partial view of that.
But it's kinda messy.
Don't know if that helps!

@jcalz regarding your in type guard question: narrowing only removes types from a union, and if ("a" in x) is only treated as narrowing, not as an assertion. Contrast its behaviour to instanceof, which does act as an assertion for classes with an inheritance relationship.
It's a separate issue. The first question that I think needs to be resolved is what to do when you have concrete types:
interface A { a }
interface B extends A { b }
function f(a: A) {
    if ("b" in a) {
        a // what is the type of a? A? A & { b } ? never?
    }
}
declare let a: A
declare let b: B
f(a)
f(b)

So should I file it separately or not bother?

@jcalz Yes, it's a good feature request. Might be impossible, but we should at least have an issue for people to explore the design space.

I have a scenario that I think is caused by this issue, I've simplified it to the following:
interface A {
  prop: string;
}

declare let partialA: Partial<A>;
declare let a: A;

if (partialA.prop !== undefined) {
/*
    TS2322: Type 'Partial<A>' is not assignable to type 'A'.
  Types of property 'prop' are incompatible.
    Type 'string | undefined' is not assignable to type 'string'.
      Type 'undefined' is not assignable to type 'string'
*/
  a = partialA;
  a;
}

Possibly related...
// Extract keys whose values match a condition
type Filter<T, Cond, U extends keyof T = keyof T> = {
  [K in U]: T[K] extends Cond ? K : never
}[U]

// Extract method names from an object type
type Methods<T> = Filter<T, (...args: any[]) => any>

interface Events {
  foo(): void
}

class Foo<T, U = T & Events> {
  foo(arg: Methods<U>): void { }
  bar(arg: keyof U): void { }
  tau(arg: keyof (T & Events)): void { }
  test() {
    this.foo('foo') // [error] "foo" is not assignable
    this.bar('foo') // [error] "foo" is not assignable
    this.tau('foo') // OK
  }
}

Playground link: https://goo.gl/t88tRg

Was this magically fixed somewhere? I can't reproduce the issue anymore.  Playground link


Commands

"buf" and "temp" are now static char arrays, and not dynamically allocated arrays.
Added a /nick command, which allows to set a nickname (instead of just using a IP address). The default nickname of a client is his IP address. When someone changes his nickname, all the other clients are informed.
Added a /list command to get the list of connected clients (warning: the buffer containing the list of users' nicknames could overflow if there are too many clients connected to the server ; the number of clients should be limited accordingly).

Hey, thank you for your PR.
I like the general idea of having commands, but there are some reasons why I'll not merge your PR:
General preferences for schwatz:

Even though I'm really thankful for any help, I prefer to implement this on my own for learning purposes, I would prefer any recommendations or fixes about my implementation
I would like to implement encryption before thinking of commands (because everybody should chat securely before being able to change a nickname)

Implementation details of the PR:

Commands are also sent to every user as messages (I guess there should be a simple 'continue' after the strcmp's)
'buf' and 'temp' should both not be stack allocated because it causes issues with random gibberish
The '/nick' command doesn't check if somebody already has that nickname
As you've already commented, too much clients on a '/list' command could cause a buffer overflow
There is no check to indicate if a command doesn't exist
Not generally an issue but it might be better to check for (at least some) issues on the client-side before sending a request to the server in the first place
I would most definitely implement it so that the client checks for which command was typed and just sends a flag to the server (for example an enum, where every command has a specific ID), that way the server won't have to handle annoying strcmp's, which are generally to avoid but also computationally expensive

As this is (from what your github profile looks like) your first PR, don't feel like you did a terrible job. I will mainly not merge your PR because of personal preferences about what the project should look like.

No problem. It is indeed my first pull request. Sorry to have bothered you.

You didn't bother me, I've just thought of a different implementation. It's not your problem that I nowhere specified how I want the implementation to be like. Please don't stop contributing to projects in general, open source is heavily needed and just great. I would simply recommend to work on things that have been specified, such as for example issues by users.


Add rolling aggregates to SArray

Overall, this looks good to me. My only suggestion is to change the name of rolling_apply in the C++ side and the cython side to something else to future proof when we have a more generic rolling_apply.


Add row unpacked income_statement_ferc1 table

We developed some tools for unpacking FERC Form 1 tables that are organized by row number (where each row should really be a column in a database table) for some client work, and this allows us to make a bunch of financial tables much cleaner and easier to use.
To make this happen we need to:

[x] Compile a row map for the f1_income_stmnt table
[ ] Compile a row map for the f1_incm_stmt_2 table (2nd half of the form... in another table?!)
[ ] Write a new pudl.extract.ferc1.income_statement() function.
[ ] Write a new pudl.transform.ferc1.income_statement() function.
[ ] Compile a tabular data resource specification for the new income_statement_ferc1 table.
[ ] Debug or document any data issues that come up during the ETL process.

All of these tables were addressed in the XBRL FERC 1 refactor. See #1574


tabindex doesn't work on md-checkbox

Bug, feature request, or proposal:
I am not able to disable tabbing to a md-checkbox.
What is the expected behavior?
<md-checkbox tabIndex="-1"></md-checkbox> should not be skipped over when tabbing through a form.
What is the current behavior?
The checkbox receives focus when tabbing through the form. In addition, there isn't any visual indication that the checkbox has focus, so it's difficult to tell where I am in the form unless I press the spacebar to toggle that particular entry.
What are the steps to reproduce?
http://plnkr.co/edit/0P5w3xAmYWhx4ZrEoWUU?p=preview
What is the use-case or motivation for changing an existing behavior?
I have checkbox that I want on the form, but not to be tabbed to.
Which versions of Angular, Material, OS, browsers are affected?
Tested on Angular 2.4.8, Angular/Material 2.0.0-beta.2, Chrome
Is there anything else we should know?
That more or less covers it.

You mentioned in the issue that you are using tabIndex in camelcase. But in your Plunkr it's actually just tabindex.
For those components, just using tabIndex in camelcase should do the trick.
http://plnkr.co/edit/4ScdvxMz8q2Q5RC6bILh?p=preview

Whoops..I had the wrong title down...it's actually md-checkbox that tabindex doesn't work. I also corrected the example to have a lowercase tabindex

WRT the focus indication, it is being tracked via https://github.com/angular/material2/issues/3102.

@DevVersion So I just came back to this and see that it does indeed work with the camelcase tabIndex. Is that the plan to keep it like that? I'm glad that it works, but given that the spec for tabindex is all lowercase, I wonder if that will cause additional confusion (beyond just myself).

Not sure. I see that it can cause confusion here, but I think the tabIndex is in camel-cause because when using an input binding, you are actually setting the native tabIndex property.
That's why for native inputs you can also do
<input [tabIndex]="2">

Not sure if we should support both.

@tinayuangao are you able to confirm if this is an issue?

It's fixed by #3403. Now checkbox has focus indicator and [tabIndex]="-1" works.


Add dtest for CASSANDRA-13053

See title.

+1, very readable

Test looks good - if the plan goes forward as on CASSANDRA-13053 to fix 2.2+, this should be gated to run on only 2.2+.


style: Update CartSummary component to match design

Impact: minor
Type: *style
The CartSummary component was providing incorrect font sizes and spacing, and it did not match the design. This update corrects the font size and spacing to match the design:
Design: https://zpl.io/V0ypYWR

🚀 Preview deployed
Built with commit e02dea89cbc975066e78f76482256779155d41d2
https://deploy-preview-398--stoic-hodgkin-c0179e.netlify.com

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


MAINTAINERS: Add tomi-font as modem subsys collaborator

Add tomi-font as modem subsystem collaborator

Unfortunately I cannot add myself as a reviewer. But here is my check mark: :white_check_mark:

@aescolar Completely forgot about contributor nomination
Contributor nomination created #69761


Filled in more fields of the TACLS netkan file.

Resubmitted #597
Is that better?

I'm afraid that if you set to depend on TACLS-Config, and that is set to depend on Realism Overhaul config, and that downloads Real Solar Systems and a lot of other mods, I think that the end result is not the expected one.


Update vendored Kubernetes to v1.29.0

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Updating the rc to the final tag.
Which issue(s) this PR fixes:

None
Special notes for your reviewer:
None
Does this PR introduce a user-facing change?

None

/lgtm


immutable (beta): SWRGlobalState.get(cache) is undefined when using SSG

Bug report
Description / Observed Behavior
I encounter this error on build when using useSWRImmutable on a Server-Side Generated page (getStaticProps)
const { data } = useSWRImmutable(key, fetcher)

TypeError: Cannot read property '0' of undefined

This code seems to be the cause:
https://github.com/vercel/swr/blob/master/src/use-swr.ts#L46
const [
  EVENT_REVALIDATORS,
  STATE_UPDATERS,
  MUTATION_TS,
  MUTATION_END_TS,
  CONCURRENT_PROMISES,
  CONCURRENT_PROMISES_TS
] = SWRGlobalState.get(cache) as GlobalState

SWRGlobalState.get(cache) seems to be undefined in this case
Additional Context
<EMAIL_ADDRESS>

@hazae41 Thanks for reporting and using the beta version of swr 🙏


methods assume internal structure that might not be available

Hi, I get the following error when the history is set to [] and subsequently (after a loadSession event) we run clearMessages(), then we get the following error:
Error: TypeError: Cannot read properties of undefined (reading 'innerContainer')
    at t.hideRoleElements (deepChat.bundle.js:1:22554)
    at Wi.addInnerContainerElements (deepChat.bundle.js:1:79629)
    at Wi.createMessageElements (deepChat.bundle.js:1:79057)
    at Wi.createNewMessageElement (deepChat.bundle.js:1:78745)
    at Wi.createAndAppendNewMessageElement (deepChat.bundle.js:1:78069)
    at Wi.addIntroductoryMessage (deepChat.bundle.js:1:194035)
    at Wi.clearMessages (deepChat.bundle.js:1:198408)
    at index.html:209:36

Hi @Morriz.
I have been trying to reproduce the error but can't seem to trigger it.
There is a chance that the dev version of Deep Chat may have fixed your issue. Could you try either deep-chat-dev or deep-chat-react-dev packages version 9.0.204 to see if they fix the issue for you. These packages are used exactly the same as the core ones except their names are different.
If the issue is still occurring, could you share some more details of the configuration you are using for Deep Chat or how you are using it.
Thankyou!

Tnx again for long reply. I have stopped using chatElementRef.clearMessages in my scenario and used chatElementRef.history = [] instead, which helped :)


linux: Completely separate local containers from proot containers

Make it possible to use linux_local_conatiner without proot. proot is broken on travis-ci so we can't use it there at all. This necessitated changes and refactoring in a few places, but we tightened up our configuration of apt.conf on DpkgLocal so that most accesses to dpkg-related state are made out of the 'containerised' environment instead of the root directory.
This at least makes it possible to install and run packages on both Ubuntu Trusty and Precise.

Coverage increased (+10.8%) to 66.515% when pulling ef272ae247ab474b8ea88d4c704584b5222da897 on completely-separate-local-containers into c7057b984f474524854f9f119c128194b509aefc on master.

Coverage increased (+16.6%) to 72.323% when pulling ddbd626e4b1e25014773f3d82923339851ca6cf5 on completely-separate-local-containers into c7057b984f474524854f9f119c128194b509aefc on master.

Coverage increased (+16.6%) to 72.323% when pulling ddbd626e4b1e25014773f3d82923339851ca6cf5 on completely-separate-local-containers into c7057b984f474524854f9f119c128194b509aefc on master.

Coverage increased (+10.8%) to 66.515% when pulling<PHONE_NUMBER>dd66b402c436c483a8abe0150349fb on completely-separate-local-containers into c7057b984f474524854f9f119c128194b509aefc on master.


Base fields for Euclidean triples

Some of the Euclidean Belyi maps have incorrect base fields. These maps could are defined over QQ, as is clear from the defining equations, but their given base fields are number fields of degree > 1. (All the sixth cyclotomic field, I think.) Some examples:
http://beta.lmfdb.org/Belyi/3T1/[3%2C3%2C3]/3/3/3/g1/a
http://beta.lmfdb.org/Belyi/8T13/[6%2C3%2C2]/62/3311/2222/g0/a
http://beta.lmfdb.org/Belyi/9T11/[6%2C3%2C2]/621/333/22221/g0/a

This one, too:
http://beta.lmfdb.org/Belyi/9T9/[4%2C4%2C2]/441/441/22221/g0/a


feat(cursor): add cursor pointer to scss utils

If it needs a pointer, it can almost always be an anchor element. This is not needed


Unexpected value from nativeGetEnabledTags: 0

Hi,
I tried emulator and physical device(android) to run my app but results are same, it is "Unexpected value from nativeGetEnabledTags: 0". I changed body: in the scaffold as a Text widget then it worked.
I decided to change my "ProductList" codes with my course codes. After that, same results were happened again.
Thank you.

Hi @Saroglu
can you please provide your flutter doctor -v ,
your flutter run --verbose
and your pubspec.yaml
Thank you

Hi @Saroglu
Without additional information, we are unfortunately not sure how to resolve this issue. We are
therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the
bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution


ModuleNotFoundError: No module named 'chat_with_retrieval'     in chat_with_documents.py

When I run in the console: streamlit run chat_with_retrieval/chat_with_documents.py
i got this error messge: ModuleNotFoundError: No module named 'chat_with_retrieval'

Hi @WilsonAmadeoPerez! That's a new one. Could you please let me know your langchain version? pip show langchain?
Thanks!
Ben

Can you try and check what happens when you add a init.py file to the chat_with_retrieval directory? It can be an empty python script

@WilsonAmadeoPerez @CSalle's advice is much better, please ignore mine.

Thanks @CSalle and @benman1 for your answers:
1.the langchain version is: Version: 0.1.12
2. I added the the init.py file to the chat_with_retrieval directory. The error message is still: ModuleNotFoundError: No module named 'chat_with_retrieval'

Thanks a lot, @CSalle!


fix the description and the code

Additionally, the ovs-ctl script allows starting / stopping the daemons
individually using specific options.  To start just the ovsdb-server::
$ export PATH=$PATH:/usr/local/share/openvswitch/scripts
$ ovs-ctl --no-ovsdb-server start

Likewise, to start just the ovs-vswitchd::
$ export PATH=$PATH:/usr/local/share/openvswitch/scripts
$ ovs-ctl --no-ovs-vswitchd start

This patch looks wrong to me.  It doesn't make sense to use --no-ovsdb-server to start the OVSDB server.


Build dmd on Solaris

Fixes the followinf issues:

add missing } in root/port.c
add Solaris/SunOS as known OS to test/Makefile
add ini files for Solaris

LGTM.

Auto-merge toggled on


Remove boundListener

Following up from https://github.com/pillarjs/path-to-regexp/issues/333, it appears some users are passing Express.js instances directly to this library. Express.js has a method called bind (from node.js require('http').METHODS) so this is not the native method. For a very long time this has basically been a noop because it does a return this, but recently an implicit string coercion was removed for all the app[METHODS] methods, so this line now causes an error.
Opening to start a discussion since issues are disabled.

Express shadows native fn.bind() on request listeners? That is surprising! I can imagine that might cause a few problems.
This logic here can't safely be removed unfortunately - as described in the comment there, we need to ensure that within the 'request' event listener, this refers to the server you're listening to (and conceptually here that's httpolyglot, not the otherwise-invisible internal server instances). Dropping that will break various listener patterns elsewhere.
Should be an easy fix though - can you update this to use Function.prototype.bind instead, with a comment to explain why?
(Looks like CI is failing due to unrelated type changes elsewhere - I'll sort that shortly, don't worry about it)

@pimterry Updated. For any express server this line has not worked to bind a function and just returned this. It's probably not too widespread in other frameworks, directly generating methods from http.METHODS feels like something that should be avoided.

Thanks @blakeembrey! Merged & released as v2.2.2.


feat(Page): add timestamp to Request and Response events

To be able to know when requests exactly have happened during the network timeline you need a timestamp. Here it is. Timestamp is null when intercepted, this is not available in the Devtools protocol.

The use-case is to align the received list of Network requests with a timestamp to identify in which order the came in. I'd like to have all the timing information Chromium provides through the Devtools protocol like https://developers.google.com/web/tools/chrome-devtools/network-performance/reference#requests but could not find it, so for now the timestamp is good.

@aslushnikov I have updated the PR with the changes you proposed. The failing Travis build is by an error which I assume did not create: Error: read ECONNRESET and only on Node.js 7. How can I retrigger the build or is this a bug?

@aslushnikov any news on this?

@aslushnikov could you have a check on this and perhaps initiate a review? Just curious when I can use this or what needs to change?

@joeyvandijk Sorry for the long response. I'm on vacation and didn't find a way to set status on github.
Regarding the API shape, I had some time to think and I ended up loving the timing namespace.
Here's the suggestion:
request.timing.start() // monotonic time of request creation in renderer
request.timing.issued() // monotonic time of request been sent out to the server
request.timing.headersReceived() // monotonic time of response headers received
request.timing.finished() // monotonic time of when we got all the response in
request.timing.failed() // monotonic time of when we got the failure

Some of the timings are available in Network.ResourceTiming object, some are dispatched with network events (e.g. loadingFinished / loadingFailed events).
Let me know what you think.
@paulirish: let me know if there are any concerns with this.

Why not request.timing.metrics() to get and obj of everything? We can add new properties in later and keep the API smaller. Also akin to page.metrics() and the webs timing apis.
Having a method for each of the metrics feels overkill.

@joeyvandijk still interested in doing it? If not do you mind if I take over? Really need this feature :)

Go forward and implement further while for my use-case I will also get all
timings on client-side with window.performance. 👍
On Fri, 30 Mar 2018 at 14:58, Rafael Ugolini<EMAIL_ADDRESS>wrote:

@joeyvandijk https://github.com/joeyvandijk still interested in doing
it? If not do you mind if I take over? Really need this feature :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GoogleChrome/puppeteer/pull/2015#issuecomment-377518844,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADOSZLntMwka1harVhhRKa4vaTitXTBSks5tjivbgaJpZM4SC6Pj
.

Why not request.timing.metrics() to get and obj of everything? We can add new properties in later and keep the API smaller. Also more consistency with page.metrics() and other web timing apis.

Let's do request.metrics() then, to be even more consistent with page.metrics()

Some thoughts on naming..
timing typically means time durations, rather than timestamps. So the earlier proposal would be awkwardly mixing these.
Timestamps are a little easier to maintain forward compatibility for, as diff duration spans become interesting over time. (appcache, service worker, etc). And if you're only representing timings, it becomes unclear what their sequence is, and how they add up. We've observed this challenge when implementing HAR.
HAR spec
The HAR spec reports timings rather than timestamps. Their names: blocked dns connect send wait receive ssl.
Resource timing spec
The resource timing spec reports timestamps. Their names:  workerStart redirectStart redirectEnd fetchStart domainLookupStart domainLookupEnd connectStart connectEnd secureConnectionStart requestStart responseStart responseEnd
chrome devtools

@aslushnikov's earlier proposal:
request.timing.start() // monotonic time of request creation in renderer
request.timing.issued() // monotonic time of request been sent out to the server
request.timing.headersReceived() // monotonic time of response headers received
request.timing.finished() // monotonic time of when we got all the response in
request.timing.failed() // monotonic time of when we got the failure

So I think the proposal is a good start. But a few notes:

DevTools uses started to indicate the request was issued, so changing "started" in here is probably confusing.
DevTools considers the timestamp of failure to be the end time of the request
The proposal doesn't offer any timing details within the following:

DNS lookup
TCP handshake
TLS handshake
service worker handling
redirects
proxy negotiation

That said, I think it's fine to punt including those details until later.
Here's a revised proposal:
request.timing.enqueued() // monotonic time of request creation in renderer
request.timing.sent() // monotonic time of when the (final post-redirect) request was sent out to the server
request.timing.headersReceived() // monotonic time of response headers received
request.timing.finished() // monotonic time of when the response completed (or when the request failed)

This is costing too much time and while I really appreciate the input/feedback, it provides me coding the approach 3 different times. I did not mind the first approach, understood the 2nd approach and approve the 3rd approach, but what if I finish? While it be generic enough? If someone will follow-up, please do so @rafaelugolini but for my use-case I can also use window.performance.
Perhaps in the (near) future I will need this approach and will implement as described by @paulirish . Sorry and thanks for the feedback, but this is not helping my use-case.


Small replay output improvements

WHY are these changes introduced?
Small improvements to replay output based on slack conversation.
WHAT is this pull request doing?

Adds a leading + when the instruction count difference is >0.
Adds an empty line after a replay output to make it easier to tell when the rebuild from watch is happening.

How to test your changes?
Run replay with watch i.e.
pnpm shopify app function replay --path /Users/lopert/code/log-streaming/app/log-streamer/extensions/product-discount

and change the function to see it rebuild / display stats
Post-release steps

Measuring impact
How do we know this change was effective? Please choose one:

[X] n/a - this doesn't need measurement, e.g. a linting rule or a bug-fix
[ ] Existing analytics will cater for this addition
[ ] PR includes analytics changes to measure impact

Checklist

[ ] I've considered possible cross-platform impacts (Mac, Linux, Windows)
[ ] I've considered possible documentation changes

Thank you @lopert!


Console not loading on Local host

I installed using Windows 2022 server. I didn't get any errors but when I browse to local host on edge or chrome, I am not able to see a console. I followed the steps on the read me and I didn't see anything else I should have done. I do have IIS on the local host machine so I can see that it is running just nothing on port 3000.

Well i'm a little confused, if you followed our readme why is IIS involve, what role did you want IIS to play here? Fabric operations console has it's own webserver (node.js + express). Where you able to use our scripts on the reademe to start a network and the console? Does docker ps show several containers running?

It ended up being the port used to deploy but I got it up thank you!

cool!


Submit Template

the submit template is no longer in the list of options under the ellipsis as specified in #2 above

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

ID: 3bf8d0d2-3a6f-c3f1-494d-b0ea73c22f9b
Version Independent ID: 41bc35b2-5a4d-7406-1ae7-27b292033fcd
Content: Submit a flow template - Microsoft Flow
Content Source: flow-docs/publish-a-template.md
Service: flow
GitHub Login: @sunaysv
Microsoft Alias: sunayv

@JerryCRMI Thanks for bringing this item to our attention. It's now fixed: https://docs.microsoft.com/en-us/flow/publish-a-template.


Install new CRDs during upgrade

helm upgrade --install currently skips installing all CRDs during upgrade, despite having a SkipCRDs option.
This PR updates the code so that we install new CRDs during upgrade, which allows an updated chart to provide a new CRD.
This is done in the same order as it would take place on an install
What this PR does / why we need it:
This closes #11969 and helps in ref #6581
Currently, if you install a chart, Helm will install new CRDs. If you upgrade to a chart which has new CRDs, Helm won't install them.
This is despite:

https://helm.sh/docs/chart_best_practices/custom_resource_definitions/ implying that new CRDs will be installed on upgrade
helm upgrade taking a SkipCRDs flag, which currently doesn't do anything (and claims: "if set, no CRDs will be installed when an upgrade is performed with install flag enabled. By default, CRDs are installed if not already present, when an upgrade is performed with install flag enabled"). This change makes that text accurate.

Special notes for your reviewer:

Note, in theory this might not be regarded as backwards compatible, because helm upgrade <foo> will now install CRDs. However, if the user didn't want CRDs installed, they could have passed --skip-crds.

Although this is flagged as a large change, the bulk of the change is just moving stuff from install.go to actions.go, so that it can be used in upgrade.go.

If applicable:

[ ] this PR contains documentation
[ ] this PR contains unit tests
[ ] this PR has been tested for backwards compatibility

Hi all, it would be really useful to have this PR approved after testing as "Install new CRDs during upgrade" is frequent request from our client using our Helm Chart. Thanks

The link you provide explicitly says that it does not supports upgrades in Some caveats (and explanations)

There is no support at this time for upgrading or deleting CRDs using Helm.

I think this PR is interesting but it should be guarded by as switch (--force-upgrade-cdrs ?) and disabled by default for backward compatibility.

@jouve
Thanks for the feedback

The link you provide explicitly says that it does not supports upgrades in Some caveats (and explanations)

There is no support at this time for upgrading or deleting CRDs using Helm.

I think this PR is interesting but it should be guarded by as switch (--force-upgrade-cdrs ?) and disabled by default for backward compatibility.

This PR doesn't change that - it doesn't upgrade or delete CRDs, it merely adds new ones.
The existing flags on helm already suggests this will be done, but prior to this change, this is not the case.
From helm upgrade --help.

--skip-crds                                  if set, no CRDs will be installed when an upgrade is performed with install flag enabled. By default, CRDs are installed if not already present, when an upgrade is performed with install flag enabled

As such, I don't think a new command line option is sensible.
If a user wants to preserve the existing behaviour, they can run helm upgrade --install --skip-crds. If they want the new behaviour they can run helm upgrade --install.
We could add --crds, as a no-op (or worse, --no-skip-crds 🤮 ), and leave the current, bugged, behaviour, and fix the options text, but I'm not convinced this is helpful to anyone.


MultiSelect, closeOnSelect:false - Jumps to top of list each selection

When using a select with mode of multiple and closeOnSelect as false, the drop down jumps to the top of the list with each selection.  This requires the user to scroll back down to where they were in the list to find the next selection.  Search works well; however, you need to know what you're searching for.  Sometimes, as a user, you don't know what choices your list contains.
Here is a jsFiddle:
http://jsfiddle.net/WP97R/1/

Hmm. I am still getting this issue in 3.5.1 -- It is very frustrating for my client to have to scroll back down to where they were after each selection.

Reopening this pending further debugging.

Confirmed as fixed in 3.5.2 again.

This Bug is happening again with version 4.0.3
http://jsfiddle.net/BonDoQ/rzvemnqy/
Problem:
https://github.com/select2/select2/blob/master/src/js/select2/results.js#L279
It always highlight the  firstSelectedItem / firstItem
Solution:
check this condition before trigger that function would prevent the behaviour for that case
if(!(self.options.get('multiple') && !self.options.get('closeOnSelect'))) {
  self.highlightFirstItem();
}

I solved this with the following:
var scrollTop;
$('#mySelect2').on("select2:selecting", function( event ){
    var $pr = $( '#'+event.params.args.data._resultId ).parent();
    scrollTop = $pr.prop('scrollTop');
});
$('#mySelect2').on("select2:select", function( event ){
    var $pr = $( '#'+event.params.data._resultId ).parent();
    $pr.prop('scrollTop', scrollTop );
});

Perhaps not the most elegant solution, but essentially we listen for the selecting event and then grab the current scrollTop of the selectable options panel. On the actual selection, we reset the panel's scrollTop back to its original. This happens so fast that you should see no jump in the control window. Only test with Select2 v. 4.x.
The nice thing about this solution is that you don't have to hack the component or include anything in your config functions.

I am able to resolve this locally by commenting out self.highlightFirstItem in the results.js file. However, this is not a great fix, because whenever the node_modules are installed the change will be overwritten. I suggest have this feature be set to a boolean, so that users can easily turn it on or off. The list we are using for our clients is long, and jumping to the top of the list doesn't improve the UX. What is a better workaround? I've tried the DOM manipulation but haven't been able to lock it down. Thanks!
`  container.on('select', function () {
if (!container.isOpen()) {
return;
}
  self.setClasses();
  self.highlightFirstItem();
});`

Could noy find a better way than this...
`  var scrollTopSl2;
$(document).on('select2:selecting', 'select[multiple]', function(event){
scrollTopSl2 = $( '#'+event.params.args.data._resultId ).closest('#select2-'+event.target.id+'-results').prop('scrollTop');

});
$(document).on('select2:select', 'select[multiple]', function(event){
 $( '#'+event.params.data._resultId ).closest('#select2-'+event.target.id+'-results').prop('scrollTop', scrollTopSl2 );

});
$(document).on('select2:unselecting', 'select[multiple]', function(event){
scrollTopSl2 = $( '#'+event.params.args.data._resultId ).closest('#select2-'+event.target.id+'-results').prop('scrollTop')

});
$(document).on('select2:unselect', 'select[multiple]', function(event){
 $( '#'+event.params.data._resultId ).closest('#select2-'+event.target.id+'-results').prop('scrollTop', scrollTopSl2 );

}); `

Seems like this is happening in the current version.

plus 1 to that, happening for me too.  Just done an NPM update to make sure I have the latest build (April 4th 2017) and still no diff.

@jackson-sandland's solution worked for me as well. I also agree that it's poor UX behavior as I can't see any upsides to this behavior.

Any progress on this issue? Was it ever corrected in master branch?
Thanks!

Doesn't appear so, for me, I just wrapped my selects in custom components and managed the datatype marshaling myself.

@kevin-brown Can we get my #4869 PR merged so we can get a fix for this on master?

I'm eagerly awaiting as well, as I use the CDN hosted source, so I can't fix it. Thanks!

Reopening this issue, as apparently it is still happening in 4.0.x.

Actually, I am going to close this issue because there is a newer issue, #4417, which seems to have some insight as to what's causing this issue in 4.0.x.

Find this part of the select2.min.js and delete it.
,f=this.$results.scrollTop()+(e-d),g=e-d;f-=2*a.outerHeight(!1),c<=2?this.$results.scrollTop(0):(g>this.$results.outerHeight()||g<0)&&this.$results.scrollTop(f)

Please continue discussion in #4417.


Exception in thread "Excess Enum Table Reader" java.lang.OutOfMemoryError: unable to create new native thread

Hi team,
When I start the blueflood,the log will show "Exception in thread "Excess Enum Table Reader" java.lang.OutOfMemoryError: unable to create new native thread"
I have tried to modify -Xms，-Xmx，-Xss,and
proc/sys/kernel/pid_max，
/proc/sys/kernel/thread-max，
max_user_process（ulimit -u），
/proc/sys/vm/max_map_count。
but the error  still exist.
thanks

We removed that thread, as we removed support for enums recently. If you pick the latest, you should not see that error.

thanks


CB-21467 extends the topology to populate ranger and ranger UI URLs

CB-21467 extends the topology to populate ranger and ranger UI URLs when the ranger is running on the master node. In the case of EDL, we moved the Ranger from the GATEWAY node to the MASTER node. But the salt searched it in the GATEWAY host, which means it's not loading the ranger URLs into the cdp-proxy topology;
Tested.
The Datalake Ranger UI loaded.
The Ranger API call was successful.
The content of  /var/lib/knox/gateway/data/deployments/cdp-proxy.topo.18795723142/META-INF/topology.xml
  <service>
      <role>RANGER</role>
      <url>https://irokolya-env-04-18-001-gateway0.irokolya.xcu2-8y8x.wl.cloudera.site:6182</url>
      <url>https://irokolya-env-04-18-001-gateway1.irokolya.xcu2-8y8x.wl.cloudera.site:6182</url>
      <param>
         <name>replayBufferSize</name>
         <value>128</value>
      </param>
   </service>
   <service>
      <role>RANGERUI</role>
      <url>https://irokolya-env-04-18-001-gateway0.irokolya.xcu2-8y8x.wl.cloudera.site:6182</url>
      <url>https://irokolya-env-04-18-001-gateway1.irokolya.xcu2-8y8x.wl.cloudera.site:6182</url>
      <param>
         <name>replayBufferSize</name>
         <value>128</value>
      </param>
   </service>

@kkalvagadda1 @RokolyA94 do you still want this cahnge?


Create kg121

https://kzsh121.dnepredu.com/
https://www.education.ua/schools/10037/

@KiryllBlokhin lib/domains/education/ukr/kg121: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

@KiryllBlokhin Thank you for your patience while waiting for a review and processing your request. Pull request merged.
Please start requesting the licenses in about an hour, to let the changes to propagate through our system.


Single-Plot only generates few points

The file given here:
https://github.com/hz-b/RAY-X/issues/94
only generates a two points when ran with -p

that's how it looks for me.

I get only one point, as far as I can see (this is on a6d3e3d4, the single-plot branch doesn't output anything)

This might be a bug in the tracing :
As we're emulating RAY-UI : We need to look for rays that crossed the longest length 1234 etc
Here's an output of test_multi.rml
Left : extraParam, Right: amount
[            Plotter.cpp:157] 0:5840
[            Plotter.cpp:157] 1:4594
[            Plotter.cpp:157] 2:53862
[            Plotter.cpp:157] 3:62308
[            Plotter.cpp:157] 4:53617
[            Plotter.cpp:157] 5:4420
[            Plotter.cpp:157] 6:1107039
[            Plotter.cpp:157] 54:1
[            Plotter.cpp:157] 61:49366
[            Plotter.cpp:157] 62:187286
[            Plotter.cpp:157] 63:234534
[            Plotter.cpp:157] 64:187468
[            Plotter.cpp:157] 65:49660
[            Plotter.cpp:157] 212:1
[            Plotter.cpp:157] 454:1
[            Plotter.cpp:157] 612:2
[            Plotter.cpp:157] 654:1

You can see that the ones that meet 6 (ImagePlane) with the longest path are not that much!!

Here's a trace when looking only for the Imageplane being last as in 6XXXXX is accepeted

However this doesn't look like RAY-UI

The fix also breaks David's beamline which used to be correct when emulating the "sequential tracing" in plots

As we're emulating RAY-UI : We need to look for rays that crossed the longest length 1234 etc

Can you explain what you mean with that?
I'm under the impression "single-plot" should render all rays ending in the ImagePlane, not only those taking the longest path to it, right?

Fixed in f48389a921b4ce6a2b7b92e3b71537e5307893aa . Use LONGEST_PATH if you want to revert back to old footprints


awaiting IObservable

This isn't really an akavache issue per se - I'm just curious how usage of akavache intersects with the advice to have library methods call ConfigureAwait(false).  It appears that the extension methods don't provide for this.  Am I missing something ?

If you want to do that, you can call someMethod.ToTask().ConfigureAwait(false)

Much appreciated - missed that.


Replace Airtable

Approvals
Things Airtable does for us
Critical

[ ] Data Source intake via form: https://airtable.com/shrJafakrcmTxHU2i
[ ] Code-free bulk Data Source intake via csv import: https://airtable.com/shrbnadIQyefJhI1D
[ ] Publicly view database without auth: https://airtable.com/shrUAtA8qYasEaepI/tblx8XaKnFTphWNQM
[ ] CSV export without auth

We could require auth, but that's a barrier to access and requires a documented reason

[x] API access: https://airtable.com/app473MWXVJVaD7Es/api/docs
[ ] One-click Data Source approval (who approved it, when it was approved), replaced by Retool workflow
[ ] #21
[ ] Data Requests: form here, replaced by Requests foirm i
[ ] Data Volunteers: form here, replaced by Data Sources App users

Nice to have

[ ] Some edit history (probably limited)
[ ] Spreadsheet-like UI for editing rows (using CSV editors, WRGL, DoltHub, etc instead is probably fine)

@josh-chamberlain Does this tie into #1 ? If there's interest in migrating from Airtable to Postgres and building a service around it, I'd be happy to look into this.
Sorry to double-dip the issue response if that's the case, just looking for some clarification on this one.

@DouglasKrouth yes, but that issue is a bit out of date. We do have an API live. Maybe it's best for you to be in touch with @mbodeantor to see where exactly would be best for you to pitch in. Lots of good work going on, lots of areas in need of help with no one active.

@josh-chamberlain I have my guess as to some of the reasons, but just to make sure I'm fully in the know, what are our primary motivations for replacing Airtable? What are the disadvantages of using Airtable we're trying to rectify?

@maxachis definitely, updated above

@josh-chamberlain Considering we're now at the point where we've got our Retool apps set up in varying states, what's next to do on this?

@maxachis well, there's the front end view for new source—other than that, these things can happen in the future! Before I close this issue, I'll create issues for those other back end pieces so that you don't run out of stuff to do 🙉


Days Field after hours or a way to go Beyond pass 23:59 like 139 hr: 59 min.....

Feature request
I am using angular timepicker for duration selection and i know that i can use date picker and time picker together to do that i want to do it in following ways.
Is there way to select a unlimited hours instead of limiting it to 23:59 or can you add a day input field that increments when the counter reaches 23:59..
Or is there any way i can do these on my side..
http://keith-wood.name/timeentry.html -> format -> Unlimited Hours. some thing like these
Thank you

What you're talking about is effectively a duration field, and not a time field.  Our time picker was not designed to be used for duration.  I'm not suggesting that this wouldn't be a useful feature, but it IS one that has not been requested before (to my knowledge) - certainly not requested enough for me to remember.
This is not going to be a very high priority for us and you'd probably be better off, taking our timepicker code and tweaking it as you see fit.  If you do that, you might consider open-sourcing that component in your own repo.
I'm going to close this for now until/unless one of the other devs would like to reopen.


Reload page on admin/mod creation/deletition

In my opinion the usersEdit page should be reloaded when a new mod/admin is created / deleted, such that the new users shows up in the table immediately after creation. With suspension and activation this is already the case and should be a minor fix. Maybe this has something to do with the difference between http PUT and POST.

In my option the table should refresh, since the data-source array is updated on that event...

It's true that the array is updated, the mat-table however only loads it's data from the specified dataSource (= array) synchronously (e.g on page load), unless you implement complicated subscription logic, and manually call a function on every possible update (add, delete, etc.). That's why I think reloading the page would be a more elagant why of solving this, especially if the user is prompted with a confirmation message before.


Tree displays zero items whenever the unit system changes

When the unit system changes, the tree view displays zero items. The issue resolves itself after switching to a different type of tree (such as from Models to Categories). This error is occurring in @itwin/tree-widget-react v2.0.2
Step 1:

Step 2:

Step 3:

Issue was in @itwin/presentation-components package. Published<EMAIL_ADDRESS>with a fix,


Documentation: Reference Extauth Implementation

Talking with @plombardi89 on Slack earlier, he requested to submit an issue for creating example documentation for implementing auth-service with an external authentication provider. In our case, an article outlining setup with Auth0 would be excellent. If we can get someone to help us set this up, we'd be happy to blog about it. :)

We've added this as part of the Pro docs!


fix: correctly set line caps

Correctly set the capability flags in geodLineInitInt which was previously ignoring the passed in caps Mask.

@tidwall its been a while so just checking in to see if you could apply these PR?

You got it. Thanks!


Chinese character encoding

When I use needle to post a Chinese character, and get incorrect codes on server.

What options are you using to send the data? Example code might help.

Thanks for your answer,I have solved the problem by encode the data before send.

And my question like this problem : https://github.com/tomas/needle/issues/97


_WD_UpdateDriver() - $bFlag64 = Default handling

https://github.com/Danp2/WebDriver/pull/139#issuecomment-1021198135
If $bFlag64 = Default then update should perform checking which version is used.

in case you aproove my PR then before commit
;                  $bFlag64     - [optional] Install 64bit version? Default is False

should be chagned


Notification on TV ?

Is there a way to send notification to the TV ?. I want to show a notification when someone is at the door, sort of a pop up ?

No, but u can open a browser page on the TV, or tell it to play a video. I know at least one user that wanted to send the smart doorbell feed to the TV when someone was at the door, not sure if he ever succeeded.

No, but u can open a browser page on the TV, or tell it to play a video. I know at least one user that wanted to send the smart doorbell feed to the TV when someone was at the door, not sure if he ever succeeded.

I was able to pay mp3 file from a local web server for that but was looking
for notifications as it is available in LG
On Fri., Jan. 1, 2021, 5:00 a.m. Alexandru Branza<EMAIL_ADDRESS>wrote:

No, but u can open a browser page on the TV, or tell it to play a video. I
know at least one user that wanted to send the smart doorbell feed to the
TV when someone was at the door, not sure if he ever succeeded.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jaruba/ha-samsungtv-tizen/issues/110#issuecomment-753295650,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AR7FX57OTRZEFGDGNFX6IY3SXWMMFANCNFSM4VP37T6A
.

I was able to pay mp3 file from a local web server for that but was looking
for notifications as it is available in LG
On Fri., Jan. 1, 2021, 5:00 a.m. Alexandru Branza<EMAIL_ADDRESS>wrote:

No, but u can open a browser page on the TV, or tell it to play a video. I
know at least one user that wanted to send the smart doorbell feed to the
TV when someone was at the door, not sure if he ever succeeded.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jaruba/ha-samsungtv-tizen/issues/110#issuecomment-753295650,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AR7FX57OTRZEFGDGNFX6IY3SXWMMFANCNFSM4VP37T6A
.


fix actions names

following up #9

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
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 54.006%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
182

Relevant Lines:
337

💛  - Coveralls


Remove Pivotal references from OSS login server style

We should cleanup the pivotal references from the login server html where it is not part of the pivotal l&f, e.g. https://github.com/cloudfoundry/uaa/blob/master/login/src/main/resources/templates/web/access_confirmation.html

8fe8e6f1c6fc3c0bbf60d561f9fcc81301a14e16


UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'isArray' of undefined

Calling web3.eth.getTransactionReceipit always throw that error

Versions higher than<EMAIL_ADDRESS>does have the same issue, use<EMAIL_ADDRESS>for now could fix this.

Fixed in #138


Implement additional data for listbox option

Depends on

#564
#568
#566

Why?
The listbox option should be extended to support displaying additional data.
Design
Figma link.
Acceptance criteria

[ ] a badge can be shown
[ ] a tag can be shown
[ ] a suffix text can be displayed

Out of scope

keyboard shortcut
consider implementing a dedicated component for keyboard shortcuts (e.g. with a prop to switch between windows and mac style)

Definition of Done

[ ] example usage added to apps/alpha-test-app/src/views/HomeView.vue
[ ] covered by component tests
[ ] screenshot tests are updated
[ ] follow-up tickets were created if necessary
[ ] updated version + documentation is deployed
[ ] Storybook can show the feature
[ ] Set task to In Approval

Approval

add link to deployed storybook
add link to deployed docs
[ ] approved by designer @jannick-ux
[ ] approved by dev add dev tag

Implementation details
add details if necessary, e.g.,

suffix text should support string values (not only keyboard shortcuts)
for flexibility, use slots

Info: started in feat/742-additional-select-data, but aborted. there are plans to refactor flyout contents to be slot based. this impacts the way this feature is implemented greatly.


Display info is doubled in the neon2-wrapper

when the display info checkbox is active, the info box is doubled within Rodan. This may be a compatibility issue.

fixed in neon2-wrapper


Add a footer to the website

Is your feature request related to a problem? Please describe.
There should be a footer with copyright and icons and tags
Describe the solution you'd like
I would like to add the footer

Please assign this issue under LGMSOC21
@yashsehgal hgal @ItsMeIshan Ishan @nicoolel oolel


Cluster state is not the same in Cluster Management vs Home page.

Rancher Server Setup

Rancher version: 2.6-head commit id: e93a53c
Installation option (Docker install/Helm Chart): docker install

Information about the Cluster

Kubernetes version: v1.21.10+rke2r1
Cluster Type (Local/Downstream): Downstream DO Node driver cluster

User Information

What is the role of the user logged in? (Admin/Cluster Owner/Cluster Member/Project Owner/Project Member/Custom) admin

To reproduce:

Dpeloy an rke2 Do node driver cluster

when the cluster comes up Active, take a snapshot on the cluster

Restore to this snapshot

due to an existing bug, the cluster will be stuck in Provisioning state with an error in the CLuster Management page

But on the home page this cluster is seen in Active state and is misleading

I'll take a look at this one on Monday, if we're showing the status of the provisioning cluster and the management cluster as reported by the backend I'll transfer to rancher/rancher

I couldn't get a snapshots to be created on my older rancher instance (due to https://github.com/rancher/rancher/pull/36707) so brought up a new one (v2.6-head, 4df2214).
On the home page we show the cluster type management.cattle.io.cluster and in cluster management we show provisioning.cattle.io.cluster. When restoring the snapshot it looked like the status text for both types matched throughout the restore. Frustratingly the restore succeeded and did not enter an error state.
I edited the code such that the snapshot name was broken and then saw the issue.
The text on screen for the cluster manager cluster list comes from provisioning.cattle.io.cluster metadata.state.message and contained the error (etcdsnapshots.rke.cattle.io \"rc-on-demand-rc-pool1-6f5b29c5-mpkhh-1646664901-localborked\" not found)
The text on on screen for the home page cluster list comes from management.cattle.io.cluster metadata.state.message. The state is isn't in error and the message reports ready.
"state": {
"error": false,
"message": "Resource is Ready",
"name": "active",
"transitioning": false
},

There's two approaches to resolve this

This is working as expected, the provisioning cluster shows the state of provisioning, where the last provision failed. The cluster is still in a working state so the mgmt cluster reports it as active
The state of the mgmt cluster should mirror the provisioning state, regardless of it's 'working' state. This would be require backend changes to fix (the ui just shows what's reported).

At this time I'd recommend we go with number 1, keeping the working and provisioning state separate.

Closing as per - https://github.com/rancher/dashboard/issues/5255#issuecomment-1060804086


make link field required in Data object

done
https://github.com/bridge2ai/model-card-schema/commit/6aade186871e170685f6280f85d9785e45f76aff


Update vendored dependencies

Coverage decreased (-0.03%) to 93.676% when pulling 9c655ed33a93b97b859ec2547b377020447088fb on vendoring into a88ee29755df14d6051fe8c00d525ba2f6f7e6d9 on master.


updated bw tool run with subcommands ignores commandline driver flag values

It appears that the updated bw tool does not correctly parse or pass the arguments/flags for the bolt database(and perhaps other drivers) when invoked with a subcommand(ie 'bql','server',...). The default values are used regardless, with the result that the BWBOLT driver and associated configuration options are inaccessible via the commandline.
I added two printfs for *boltDBPath and *driverName beneath the flag.Parse() in main() to demonstrate the flag/variable values. Adding '=' between flag and value can alter parsing behavior, but does not have any effect on driver flag values.
If the defaults are modified in the source code, for example with driverName to 'BOLT' and boltDBPath to 'bar.db' the bolt driver and database appear to work and persist correctly with the subcommands.
examples:
no subcommand, correctly parsing values,'foo.db' is created if it does not exist, 'foo.db' seems to be also parsed as a command
$ ./bw -bolt_db_path foo.db -driver BWBOLT
boltDBPath: foo.db
driverName: BWBOLT
command "foo.db" not recognized. Usage:
...

with '=', missing command correctly identified
./bw -bolt_db_path=foo.db -driver=BWBOLT
boltDBPath: foo.db
driverName: BWBOLT
missing command. Usage:
...

subcommand 'bql', incorrectly using default values
$ ./bw bql -bolt_db_path foo.db -driver BWBOLT
boltDBPath:
driverName: VOLATILE
Welcome to BadWolf vCli (0.8.1-dev)
Using driver "VOLATILE". Type quit; to exit
Session started at 2017-08-30 12:14:50.032730347 +0000 UTC m=+0.011496867
...

subcommand 'server', incorrectly using default values, port incorrectly parsed
$ ./bw server 8080 -bolt_db_path foo.db -driver BWBOLT
boltDBPath:
driverName: VOLATILE
2017/08/30 12:15:22 [2017-08-30 12:15:22.883815494 +0000 UTC m=+0.006301636] Invalid port number "BWBOLT"; strconv.Atoi: parsing "BWBOLT": invalid syntax
...

subcommand 'server', incorrectly using default values, port correctly parsed but positionally ambiguous
$ ./bw server -bolt_db_path foo.db -driver BWBOLT 8080
boltDBPath:
driverName: VOLATILE
2017/08/30 12:16:04 [2017-08-30 12:16:04.878077288 +0000 UTC m=+0.006620474] Starting server at port 8080
...

Thanks for the report. I will take a look soon.

Mmh, this  seems to be the result of how the Go flag library works together with the CLI. It assumes the flags are the first thing you pass before you select any command. The fix may require also some fixing on the original code for command dispatching. Will dig a bit more tomorrow.

https://github.com/google/badwolf/commit/1495d07013b3ce7ff6abfea27586399674012af5
I updated some of the command processing to use the default args extracted by the flag package. One thing to keep in mind is the flags are global (it does not use FlagSets), hence the flags are expected between the bw and its commands. That means your examples 3, 4, and 5 will not work since the flag package will consider them as arguments.

Thanks for the changes and explanation. Driver flag tests were successful with the 'bql' and 'server' commands. I used the term 'subcommand' in my initial issue, but here have substituted 'command' as you have used in your comments.
From your informative comments, this is my understanding of the correct way to use bw with flags and a command:
bw  <flag(s)>  <command>  <command argument(s)>

Would it be possible to update the bw usage message to reflect the necessary order? Or add a note/example on flag position? I would be happy to open another issue(perhaps 'bw cli help behavior')  to discuss this and other cli messages if it is useful.
For others who may read this, the correctly running versions of my examples 3,4,5 are:
$ ./bw -driver BWBOLT -bolt_db_path foo.db bql

$ ./bw -driver BWBOLT -bolt_db_path foo.db server 8080

https://github.com/google/badwolf/blob/master/docs/command_line_tool.md#usage
Mentioned that flags should appear before he command you want to run. I have reworded it a bit and also ugdated the driver README. Please see:
https://github.com/google/badwolf/commit/c6e955b7b1f0274cce5e508681d2c2a77fb6f876
https://github.com/xllora/bwdrivers/commit/4743aee734536fe1a3fafb66b12f5a1034ea5d3a


Searching for files and folders shows nothing, whether I use activation command or not "?".

Microsoft PowerToys version
0.76.1
Installation method
PowerToys auto-update
Running as admin
None
Area(s) with issue?
PowerToys Run
Steps to reproduce
Install the latest version and simply search for it
✔ Expected Behavior
Just as good as before restoring to 0.76
❌ Actual Behavior
No response
Other Software
No response

/bugreport

Thank you so much for your confirmation.
Log attachment is as follows.
PowerToys Run.zip

Uploading installationFolderStructure.txt…

Hi, I fixed the problem after reinstalling Edge, thanks


Follow semver rules?

Don't be afraid to start with 1.0.0 and bump to 2.0.0 if things changes.
You will save our lives: imagine us when seeing 0.10 and 0.11, consider it as a minor release and then you realise that there is a breaking change behind that.

@XVincentX Totally agree. I think we are actually close to being ready to do 1.0.0. I'll work on a new roadmap to get us there soon.
Please continue to push if we don't do it soon :)

Great, thanks.
By the way, in my opinion, if it's public, then it MUST follow the semver rules. That means: 1.0.0

@XVincentX I mostly agree here. I like to think of pre-1.0.0 as a stage where you're getting it to a stable release, because OSS is inherently public even before things are stable. Before 1.0.0, use at your own risk! But now we are close to having a stable API (and we may be there). So to me, 1.0.0 says this is our first stable API and we will be very careful with our changes, even non-breaking changes.

Closing this in favor of #83

@XVincentX From Semver rules,

Major version zero (0.y.z) is for initial development. Anything may change at any time. The public API should not be considered stable.

So, we are still following server rules.


Set the weights of a single layer

When using an imbalanced dataset, it is sometimes useful to set the biases of the softmax layer to the log of the class frequency. However, in the current implementation, we either have to set all the weights manually, or use the random initialization. Is it in the plan to make this feature available?
I know that it is possible to set the biases to the log values in pylearn2, but it could also be set by sknn.

We don't have support for customization of the initialization yet, but it's a common feature we should support. Do you have any thoughts about the API?

In pylearn2, there is the parameter in the constructor of the softmax layer:
 init_bias_target_marginals : dataset
        Take the probability distribution of the targets into account to
        intelligently initialize biases.

I do not have another use case, so I guess this would be relatively simple. I do not know if other backend support this, so maybe it could be done in sknn directly. As far as I know, this is relatively straightforward.


Sticky element is bigger than the viewport

Requested Update
A way to scroll the sticky element if it's bigger than the viewporr
Why Is This Update Needed?
Currently if the sticky element is bigger than the viewport, you have to scroll all the parent to see the remaining content.
Are There Examples Of This Requested Update Elsewhere?
http://leafo.net/sticky-kit/
on the section Scrollable Sticky Element
Not really sure if this is accomplished with sticky bottom.

Managed to do a pen using the possition bottom missed the
// setup bottom sticky at the beggining.
Example for anyone interested


Pedestrian Dataset-Label --2--> EventData

Hi,
I'm interested in using your DVS-based 'Pedestrian Detection Dataset' for my current research. But unfortunately I have some trouble to map the label annotations to the original data.
According to my understanding of the data the dataset consists out of a total of 12 parts (the available *.aedat files).
According to the published details:

All recordings last about 30s, with slight variations in length.

Opening the AEdat-files in the AEViewer from the jaer-project reports other lengths for the recordings:
File 1			approx. 71s
File 2			approx. 58s
File 3			approx. 24s
File 4			approx. 255s
File 5			approx. 39s
File 6			approx. 61s
File 7			approx. 61s
File 8			approx. 95s
File 9			approx. 26s
File 10			approx. 79s
File 11			approx. 39s
File 12			approx. 44s

A large part of pedestrian detection raw data are converted to 4670 frame images through SAE encoding method with frame interval of 20ms. In our experiment, all these images have been labelled via annotation tool -labelImg.

Even when I calculate with an average of 30s per file, I assume a total of approx. 12 * 30s = 360s = 360000ms / 20ms = 18000 frames.
How did you select the subset of 4670 images from the dataset for the annotations?
The available XML label-information refers to one image in the SAE-encoding. But unfortunately it is (at least for me, now) not possible to map this image name to one aedat-file and the corredsponding 20ms window wihtin this file.
So, it is not possible to use the annotations for another encoding or even for a complete other approach?
I tried to map the published SAE-frames (by the way: 'Pedestrian frame.rar' contains about 700 more images than 'Pedestrian label.rar' contains Label-XML-files?) back to the AEdat-files by doing:

Generating own SAE-Frames with the in the repository provided 'sae.py' with a time-window of 20ms
and comparing the md5-hash from my own generated frames to the provided frames in 'Pedestrian frame'

Unfortunately this seems not to work because of an unkown post-processing on the provided frames. I assume some morphological post-processing to remove some noise.
Example for self-converted:

Example for provided:

@tbolten
hi,
I see your issue when I encountered the same question about the selection of the 4670 images (actually 4668 images with xml labels). Have you solved this problem?
Appreciate your reply.

@shirleyatgithub
No, unfortunately I could not resolve the problem.


[Relay] Reduce plevel of conv2d winograd implementation on cuda

Fix issue https://discuss.tvm.ai/t/performance-has-been-too-slow-since-the-tvm-update/5865

I think it's better to wait until the commit author issue is resolved https://discuss.tvm.ai/t/github-issue-the-commit-author-is-wrong-since-today/5880/


Wheel propagation not always working for bottom or right when using zoom (at least in Chrome)

Really sorry, but I don't know how to use GitHub rules. Anyway, I found an issue and made a fix to it, and thought I would share it.
The issue is that if you set the wheel propagation option to true, the wheel event might still be "stopped" although you scrolled to the end of the element. This happens at least when site based zoom is applied (assume it to happen with element zoom as well).
The reason why it doesn't work is that the "shouldPreventDefault" check uses scrollTop + offsetHeight === scrollHeight -logic. However, scrollTop or scrollLeft is not always an integer (if zoomed). It seems that for my case (scrollTop + offsetHeight) is usually a little bit more than the rounded integer, but sometimes a little bit less. To handle all this I changed two lines (725 and 728): take a Math.ceil of the scroll value and change === to >= . Now works for me - and will also still fully scroll to the bottom or right (due to how perfect-scrollbar handles "shouldPreventDefault" after the check).
MASTER BRANCH LINES:
725:  element.scrollTop + element.offsetHeight === element.scrollHeight;
728:  element.scrollLeft + element.offsetWidth === element.offsetWidth;
WHEEL PROPRAGATION ZOOM FIX LINES:
725: Math.ceil(element.scrollTop) + element.offsetHeight >= element.scrollHeight;
728: Math.ceil(element.scrollLeft) + element.offsetWidth >= element.scrollWidth;

+1 to add it in next version...


Add Python

We should add a Python tab for these.

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

ID: d0a7a652-1a73-79a8-515e-5d1cc1de47fb
Version Independent ID: b1668900-477b-c2f6-9a06-33171bd911ab
Content: How bots for Microsoft Teams work - Bot Service
Content Source: articles/v4sdk/bot-builder-basics-teams.md
Service: bot-service
GitHub Login: @WashingtonKayaker
Microsoft Alias: kamrani

@clearab The fix is ready, pending your review (of PR 2162).

The fix will go live with the next push from master to live.


how to remove buttons and timer from notification panel

how to remove previously,next buttons and timer from notification panel,i just want to have a pause and stop button.

Sorry @Kasiits  There is not right now any method to customized notification. we could add it as a feature.


Lsmallwood/colormap render

@khajra please review the new colormap raster renderer sample

cc @ldanzinger - I think this already matches the new format

thanks @ldanzinger - i've fixed that up


Add security impact to the "awaiting acceptance" section

Might want to link to where the policies can be found.

@afeld what you think about that ^^?

LGTM


Remove py35-djmaster test environment

Failing on Travis (example) with:

Downloading archive: https://s3.amazonaws.com/travis-python-archives/binaries/ubuntu/14.04/x86_64/python-3.5.tar.bz2
$ curl -sSf -o python-3.5.tar.bz2 ${archive_url}
curl: (56) SSL read: error:00000000:lib(0):func(0):reason(0), errno 104
Unable to download 3.5 archive. The archive may not exist. Please consider a different version.

It looks like they removed the package so it can't be used any more.

Are you sure this wasn't due to a transient issue? The py35-dj21 envrionment seems to be building fine. Unfortunately, I've found Travis to have the intermittent failure you posted. Can you try rebuilding it?

You're right, it was transient, the rebuild on #1108 passed for the environment.


Merge C and C++ lists, and add splint, oclint, clang-tidy and flawfinder

C can usually be analysed by the C++ tools... I also added oclint to this list, but maybe it belongs elsewhere. What's your opinion about "multi-language" tools? Maybe they should be repeated, so that people only interested in a single language can find all tools.

Awesome. Thanks! :bowtie:


Timx 291 orchestration

Purpose and background context
Update the application's orchestration to streamline the code and improve maintainability
How can a reviewer manually see the effects of these changes?
Run a JSON and an XML transform to ensure that records are still transformed successfully after the orchestration refactor:
pipenv run transform -o output/gismit.json -i  tests/fixtures/aardvark/aardvark_record_all_fields.jsonl -s gismit

pipenv run transform -i tests/fixtures/dspace/dspace_dim_records.xml -o output/dspace-transformed-records.json -s whoas

Includes new or updated dependencies?
NO
Changes expectations for external applications?
NO
What are the relevant tickets?

https://mitlibraries.atlassian.net/browse/TIMX-291

Developer

[x] All new ENV is documented in README
[x] All new ENV has been added to staging and production environments
[x] All related Jira tickets are linked in commit message(s)
[x] Stakeholder approval has been confirmed (or is not needed)

Code Reviewer(s)

[ ] The commit message is clear and follows our guidelines (not just this PR message)
[ ] There are appropriate tests covering any new functionality
[ ] The provided documentation is sufficient for understanding any new functionality introduced
[ ] Any manual tests have been performed and verified
[ ] New dependencies are appropriate or there were no changes

@ghukill @jonavellecuerdo I'm making this PR official, thank you both so very much for all of the discussion!

@ehanson8 The additional updates are looking good to me! Waiting for response re: ordering (had a follow-up question for @ghukill ) and think one change request still needs to be implemented.

@ghukill @jonavellecuerdo Changes pushed!