Inappropriate error message in AsString

Using the propery AsString on the return value of Eval may result in the following message:

Can only convert numeric arrays of length 1 to double.

This is confusing, as I do not request a double.
I had this issue when I was receiving a list of strings and had to use AsStrings in stead.

perhaps a legitimate bug, then.  unfortunately I am no longer maintaining this project.   hope someone else can help!


bookshop css tag not allowing the use of @extend in component styles

The current use of the bookshop tag inside the main scss stylesheet wraps all the bookshop styles inside a media query. This means that extends cannot be used inside bookshop component styles, which makes templating a bit more difficult.

Hi @rdano89 👋
Thanks for reaching out. For context, that media query is there so that the component browser can strip Bookshop styles out of the built CSS, and inject them back onto the page for local development.
Do you have a code sample of the use-case you're trying to achieve? (And whether you're using Jekyll or Eleventy)

Ah, I presume Jekyll due to the bookshop tag reference, and I can see the issue I believe you're referencing here: extending-selectors-from-within-media-queries-with-sass
That is a little bit of a pain — I tend to use mixins over extends so it isn't a problem I had run into yet. I'll have a further think on how we can achieve the namespacing we require without interfering in this workflow 🤔

I am using Jekyll. For this case I am basically trying to expand the template that i currently use so that it works in the booskhop framework.
Basically I have all my sass partials in ./site/assets/css, which are imported in the main scss file (./site/assets/site.scss) before the {% bookshop %} tag. Since I have some utility classes, i'd like to pass them to bookshop components via extends.
A very basic example:
I created a link bookshop component. This component receives params like a url, a label and a open_in_new_tab and generates an anchor link.
Then I have a header nav, for example. And I want to use it but applying custom styles (like a padding). For the sake of usability i'd like to use my .px-md (horizontal padding = [ some value], and I would just do it by extending it inside the header-nav.scss file, so it is only applied to the links inside the header-nav block. If I ever change the .px-md class, then styles would be automatically updated.

Gotcha. I'll investigate this week whether this can be solved.
The only immediate workarounds that come to mind would be:

Using mixins instead of extends, or
Moving all SCSS files into the shared/jekyll folder in Bookshop and using the bookshop-sass CLI to compile all styles, skipping Jekyll entirely. The CLI compiles all styles before wrapping them in the media query, so would somewhat skirt the problem.

Sorry that these are both a bit of a hassle, I'll keep you posted on whether I can track down a solution with the current workflow.


[JBEAP-20177] [RESTEASY-2681] merged changes from keycloak into code base

Issue: https://issues.redhat.com/browse/JBEAP-20177
Upstream Issue: https://issues.redhat.com/browse/RESTEASY-2681
Upstream PR: https://github.com/resteasy/Resteasy/pull/2499
Backport for the RESTEASY-2681 in the 3.11/EAP_7.3 branch. Just the upstream PR but the JWSInput constructors are not modified, the old constructor signatures are maintained. The idea is avoiding further issues and the modifications needed in the calls.
@asoldano @rsearls @ronsigal Please take a look when you have time. Test pass OK and I have executed one round of tests in the jboss-eap7 branch too.

This looks OK to me


New

Trying to run some of your mini projects but I cannot find the areas to clone the script

Trying to run some of your mini projects but I cannot find the areas to clone the script

https://briancoyner.github.io/articles/2013-06-05-git-sparse-checkout/

Trying to run some of your mini projects but I cannot find the areas to clone the script

Follow the steps mentioned in [https://github.com/Python-World/python-mini-projects#readme]. First, you need to Fork this project to your repository then clone it to your local machine.


add-main-page-visuals

Our Main page has several visuals: an image and icons.
The latest I'd like to use as icon buttons.
Here I plan to discuss and gather ideas, links, suggestions about where to find these stuff, which one to choose, and how to implement.

I found an awesome post on using SVG icons, here @mar-bi
Really neat, here's the main part:
import React from 'react'

const Icon = props => {
  const styles = {
    svg: {
      display: 'inline-block',
      verticalAlign: 'middle'
    },
    path: {
      fill: props.color
    }
  }

  return (
    <svg
      style={styles.svg}
      width={`${props.size}px`}
      height={`${props.size}px`}
      viewBox="0 0 1024 1024"
    >
      <path style={styles.path} d={props.icon} />
    </svg>
  )
}

Icon.defaultProps = {
  size: 16
}

export default Icon

Then call it <Icon icon={ICONS.EMAIL} size={64} /> you'll need to set up some constants and get the SVGs as well, natch 🙃

That's cool. :+1:  I'm going to check it right now.
Also I walked around this library - React Icons.
It has a whole bunch of stuff inside.

Include popular icons in your React projects easly with react-icons, which utilizes ES6 imports that allows you to include only the icons that your project is using.
Currently supported icons are:

Material Design Icons by Google https://www.google.com/design/icons/ (licence: CC-BY 4.0)
Font Awesome by Dave Gandy - http://fontawesome.io (licence: SIL OFL 1.1)
Typicons by Stephen Hutchings - http://typicons.com (licence: CC BY-SA)
Github Octicons icons by Github https://octicons.github.com/ (licence: SIL OFL 1.1

The usage is very simple: install - import -use
npm install react-icons --save

import FaBeer from 'react-icons/fa/beer';

class Question extends React.Component {
    render() {
        return <h3> Lets go for a <FaBeer />? </h3>
    }
}

One of Gatsby starters contains it. That's how I found it. :smile:

I found React Icons really annoying to drill down to find the exact icon you wanted to use, it was a while since I used it though, I like the option I linked because you had so much control over it

That's great!
Thank you for the review. :+1:  I was afraid that this React Icon can be unhandy.
Let's follow your option. :small_airplane:

Ok, cool 😎
One thing to bear in mind ould be that React Icons is all of:
Material Design Icons by Google https://www.google.com/design/icons/ (licence: CC-BY 4.0)
Font Awesome by Dave Gandy - http://fontawesome.io (licence: SIL OFL 1.1)
Typicons by Stephen Hutchings - http://typicons.com (licence: CC BY-SA)
Github Octicons icons by Github https://octicons.github.com/ (licence: SIL OFL 1.1

Whilst the icons we would want to use will be a fraction of them all 🤔

We need to extract 5+ icons from all this pool. :no_mouth:
However, this pool does exists. There are free icons with normal license and no attribution, I think.
A half of the task for searching icons is done. :smile:

I need the same library for images :laughing:
Is there any React Images? :thinking:

@spences10, I read yesterday this really very awesome post you suggested. :+1:
I like the way how we can deal with .svg icons very much. It is quite easy and lightweight - we will add only icons we need.
The author also suggests a great resource for icons - IcoMoon.
I picked up some icons from there. I will show you what I've chosen. :smile:

Yeah icomoon is where I got all the fa icons
Which ones have you chosen?

These 17 icons. We need to choose from them 5 for the main page.
Basically, the icons are for jobs, learn, support, share, community.

Cool, ok, so we can go ahead and make constants.js with all the icon svg in there and create a icons.js for the component?

Right.
Let's create them.

@mar-bi  I presume you have all the SVGs you need now?

Yes, you are right. :+1:
I have the SVG's for the main page.
I picked up a bit more than the page needs. I'm going to create an issue and present all them there giving a chance for all of us to choose the best icons.
Of course, I'd like to handle constants.js and icons.js :sunglasses:

As we have got our nice icons and an impressive image for Main Hero, I close this issue.
The addition of icons goes to #131 .


YAML/JSON support

It seems tera cannot handle YAML like
audit-events:
  sso.auth.success:
    description: Authentification success

Result
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Error { kind: Msg("Failed to render '__tera_one_off'"), source: Some(Error { kind: Msg("Forloop containers have to be an ident or a function call (tried to iterate on 'Math(MathExpr { lhs: Expr { val: Ident(\"audit\"), negated: false, filters: [] }, rhs: Expr { val: Ident(\"events\"), negated: false, filters: [] }, operator: Sub })')"), source: None }) }', src/main.rs:64:66

Remove - from audit-events and you will get
Thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Error { kind: Msg("Failed to render '__tera_one_off'"), source: Some(Error { kind: Msg("Tried to iterate using key value on variable `auditevents`, but it is missing a key"), source: None }) }', src/main.rs:64:66

Can one iterate over YAML map?

The issue comes from tera itself. It would be best opening the issue here.
I will push a workaround to your issue in a few min.

I added your use case and a workaround here.


DefaultParameters looks bug-prone.

I think DefaultParameters tend to be unintentionally mutated. Because DefaultParameters is a pointer, following code will pollute it.
params := winrm.DefaultParameters
params.Timeout = timeout // ⚠ Ouch! winrm.DefaultParameters is also changed!

You can see such codes even in README.md.
https://github.com/masterzen/winrm/blob/e811dad5ac77d4d981dc85fd43a587843274bca0/README.md#L139-L140
I also put a real-world example here.
https://github.com/hashicorp/terraform/blob/8f7744da0959334f461c18ccf15f8b19d8c09fe6/internal/communicator/winrm/communicator.go#L63-L64

I suggest to add something like NewDefaultParameters() and add // Deprecated into DefaultParameters.


Wallet refactor

This PR moves the wallet and bitcoindevkit library code from PadawanWalletApplication to a Wallet object that gets instantiated upon boot in the now almost empty PadawanWalletApplication.

Another suboptimal aspect of the Wallet object as currently implemented is how it saves to shared preferences. Because the Wallet doesn't have access to SharedPreferences directly, you need to pass a SharedPreferences.Editor as an argument to the createWallet() method, which uses saveWallet()`.
I'm looking into potentially better ways to do this.

Thanks for the feedback!


armin kissing

Description
Armin
(Optional:) Please add any files, screenshots, or other information here.
No response
(Required) What is this issue most closely related to? Select one.
Choose One
Internal issue ID
29fef14b-be32-4cbd-8dcc-25887594b0b7
Paper URL
https://arxiv.org/html/2409.19946v1
Browser
Safari/604.1
Device Type
iOS

armin kissing

armin kissing


[1.2.0] Features from #68

This pull request finishes issue #68.

Coverage decreased (-0.21%) to 89.87% when pulling dbb8ef7c3222a68c7642ae16489d527122f05af4 on CozyLife:1.2.0 into 73a5fabad6ed2c9c8b7515f6cce1f547ac29d897 on alecsammon:master.

Coverage decreased (-0.21%) to 89.87% when pulling dbb8ef7c3222a68c7642ae16489d527122f05af4 on CozyLife:1.2.0 into 73a5fabad6ed2c9c8b7515f6cce1f547ac29d897 on alecsammon:master.

Coverage decreased (-0.21%) to 89.87% when pulling d46de77c584a0df758d182a549280ede7d602b0f on CozyLife:1.2.0 into 73a5fabad6ed2c9c8b7515f6cce1f547ac29d897 on alecsammon:master.


feat(datepicker): implement comparison and overlap ranges

Adds the ability to render a comparison range in the date range picker. When the comparison overlaps with the primary range, the overlapping dates are shown in a separate "overlap" range.
Some examples for reference:

I've reworked it based on the feedback @mmalerba

I noticed a couple visual bugs:

Notice the extra green glow on the right side of the solid green circle:
This is when I have selected 13 and was hovering on 6. Notice the extra pale blue selection, and the fact that the circle is blue instead of green:

Yeah that's annoying, but it's a side effect of the changes which add the gap between the calendar rows. The problem is that the element isn't perfect circular anymore so in some cases it can end up sticking out. We might have to re-evaluate how we do the row gap.
That one's a little tricky to resolve with the current approach. I'll see if I can address it when I rework it to use gradients.

Did some hacking in chrome devtools and this seemed to fix the first issue for LTR:
.mat-calendar-body-cell::before, .mat-calendar-body-cell::after {
  left: 0;
  right: 0;
  // width: unspecified;
}

.mat-calendar-body-range-start:not(.mat-calendar-body-in-comparison-range)::before, .mat-calendar-body-range-start.mat-calendar-body-comparison-start::before, .mat-calendar-body-range-start::after, .mat-calendar-body-comparison-start:not(.mat-calendar-body-in-range)::before, .mat-calendar-body-comparison-start::after {
  left: 5%;
  border-top-left-radius: 999px;
  border-bottom-left-radius: 999px;
  // width: unspecified;
}

.mat-calendar-body-range-end:not(.mat-calendar-body-in-comparison-range)::before, .mat-calendar-body-range-end.mat-calendar-body-comparison-end::before, .mat-calendar-body-range-end::after, .mat-calendar-body-range-start.mat-calendar-body-comparison-end::before, .mat-calendar-body-comparison-end:not(.mat-calendar-body-in-range)::before, .mat-calendar-body-comparison-end:not(.mat-calendar-body-range-start)::after {
  border-top-right-radius: 999px;
  border-bottom-right-radius: 999px;
  right: 5%;
  // width: unspecified;
}

Would still need to be updated for RTL (which is currently broken with and without these adjustments).
It might just be worth revisiting this whole approach though, as it seems very complicated and brittle


Return &'static str instead of &str in Symbol functions

As far as I can tell, clingo internally does not free allocated strings ( taken from here ). On one hand this can be an issue with long running applications, on the other hand this would mean all functions in this library that return string slices could correctly return &static str.
Besides being correct this would have the benefit of allowing consumers of the library to store the slices without first allocating a string.
PS:
This has the additional benefit of stating up front what the memory behavior of the library is going to be. Because strings were bound to the lifetime of symbols, which themselves were only ids, my first worry was that there would be a soundness bug somewhere 😅

This does not hold for all strings (but most). Strings associated with symbols are internalized. Other strings might be but it is better not to make the assumption. For example it is better to assume that the lifetime of strings in the AST (the strings here are internalized) or TheoryData (the strings here are not internalized because they come from clasp) is tied to the lifetime of the corresponding owning objects. Just drop me a message for cases were you are not sure or need more info.

Yes, you're correct in that this doesn't hold for all &strs. The two functions where I'm certain that it is the case are Symbol::string and Symbol::name, and those are also the ones most interesting for me, as I'm writing conversion logic in the form of Symbol -> T (slightly related to https://github.com/potassco/clingo_derive/issues/3 ) :)

Ah, I completely missed the second part of that conversation in https://github.com/potassco/clingo/issues/186
I guess my comment can be disregarded 😅

commit f823e0c0706c70165c6037bb08a6fe8686e1bd51 to wip branch
The following functions now return &'static str for internalized strings.
Location::begin_file()
Location::end_file()
Signature::name()
Symbol::name()
Symbol::string()


Unknown action type: ClearUserGlobalPointerValue when deserializing database

Happens for databases where the global pointer value was cleared

Fixed in 4.2.6413-dev


[PY] feat: change rate limited action to http error action

Linked issues
closes: #632
Details
Change rate limited action to http error action
Change details

Change rate limited action to http error action
Add test case for the action

Attestation Checklist

[x] My code follows the style guidelines of this project

I have checked for/fixed spelling, linting, and other errors

I have commented my code for clarity

I have made corresponding changes to the documentation (we use TypeDoc to document our code)

My changes generate no new warnings

I have added tests that validates my changes, and provides sufficient test coverage. I have tested with:

Local testing
E2E testing in Teams

New and existing unit tests pass locally with my changes

Additional information

Feel free to add other relevant information below

@aacebo looks like only you have merge perms for this branch :)

@aacebo looks like only you have merge perms for this branch :)

thats strange, maybe because im a codeowner?


tf.image.resize_image running on CPU when I use tf.float16

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
yes

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
OpenSUSE 12.1

TensorFlow installed from (source or binary):
Source

TensorFlow version (use command below):
1.1

Python version:
3.5

CUDA/cuDNN version:
8.0

GPU model and memory:
Tesla M40, 11GB

Exact command to reproduce:
tf.image.resize_image(input with float16, shape)

Describe the problem
Tried to run inference using float16 but I saw a significant inference slow down comparing with float32. I did some further study and found it's running on CPU, which blocks the speed up of using float16.

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Bazel version
Mobile device

Is this still an issue?

Closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


style(favicon): Changed the favicon

Removed the default angular favicon to the rx icon. Issue #293 can be closed with this PR.

Codecov Report

Merging #294 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #294   +/-   ##
=======================================
  Coverage   79.62%   79.62%
=======================================
  Files          17       17
  Lines         216      216
  Branches        9        9
=======================================
  Hits          172      172
  Misses         42       42
  Partials        2        2

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 9c8c064...e63cfe6. Read the comment docs.


Integrate New phase colors

#1761

Updated event template files with new colors
Added test cases for color_with_opacity
Updated resource forecast UI to take color intensity as user input
Updated resource forecast API to return/render colors based on color intensity value
Downgraded openpyxl to 3.0.10 due to an error in parsing certain color codes from the excel files.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

:exclamation: No coverage uploaded for pull request base (develop@e5956c0). Click here to learn what that means.

Additional details and impacted files
@@            Coverage Diff             @@
##             develop    #1836   +/-   ##
==========================================
  Coverage           ?   68.25%
==========================================
  Files              ?      253
  Lines              ?     8621
  Branches           ?        0
==========================================
  Hits               ?     5884
  Misses             ?     2737
  Partials           ?        0

Flag
Coverage Δ

epictrack-api
68.25% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


serve(aot): with Ivy, does not detect changes to files

🐞 Bug report
Command
- [x] serve

Is this a regression?
Yes? I didn't use AOT for dev mode serving previously because it wasn't as performant as it is now in Ivy.
Description
The Ivy Guide suggests enabling "aot": true for dev mode builds.
What I’m finding during that experience is that while the rebuilds are super fast in Dev mode (aot: true), the Angular CLI often doesn’t pick up changes to SCSS and HTML. I need to constantly re-run ng serve over and over again to see my changes. It’s not the kind of DX that we’re looking for.
Changing back to aot: false works around this issue.
🔬 Minimal Reproduction

https://github.com/angular-hispano/angular-hispano/tree/enableIvy
npm start
Make changes to files like the following and see that the build reports

Date: 2019-06-09T02:07:26.153Z - Hash: fd4823b00365cd497714 - Time: 192ms
7 unchanged chunks
ℹ ｢wdm｣: Compiled successfully.

But the changes aren't reflected in the browser.
Change
https://github.com/angular-hispano/angular-hispano/blob/18c79aaf9c075feea4bdff7c3e3f39e08f88db2e/projects/web/src/app/landing/landing.component.html#L30-L31
to
<section id="call-to-action">

Change
https://github.com/angular-hispano/angular-hispano/blob/d5be179b0b7a428c0a9225862df33caafb330ce8/projects/web/src/app/landing/landing.component.scss#L10-L12
to
    &#call-to-action {
      display: flex;
      flex-flow: row wrap;
      padding-top: 24px;
    }

🔥 Exception or Error
None
🌍 Your Environment

Angular CLI: 8.0.2
Node: 12.3.0
OS: darwin x64
Angular: 8.1.0-next.1
... animations, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.800.2
@angular-devkit/build-angular     0.800.2
@angular-devkit/build-optimizer   0.800.2
@angular-devkit/build-webpack     0.800.2
@angular-devkit/core              8.0.2
@angular-devkit/schematics        8.0.2
@angular/cdk                      8.0.0
@angular/cli                      8.0.2
@angular/fire                     5.2.1
@angular/material                 8.0.0
@ngtools/webpack                  8.0.2
@schematics/angular               8.0.2
@schematics/update                0.800.2
rxjs                              6.5.2
typescript                        3.4.5
webpack                           4.30.0

Anything else relevant?
Related to https://github.com/angular/angular-cli/issues/6347
Browser: Chrome Stable Version 75.0.3770.80 (Official Build) (64-bit)
OS: macOS Mojave
Package Manager: NPM

From debugging the linked repository, it appears that this line in the Ivy AOT compiler is causing the component file to be skipped: https://github.com/angular/angular/blob/master/packages/compiler-cli/src/ngtsc/program.ts#L350
The root cause may be that that the template and style dependencies of a component are not accounted for in the incremental compilation checks.


Use poetry

coverage: 54.1%. remained the same
when pulling 360a1bc8b51f6c2298a482967b0d4ed88df345bf on use_poetry
into 033e56b7382ce0daa9fdfc6d3f4edbb718f44dbb on main.


Support Select placeholder in Blazor

🧹 Tech Debt
Supporting the placeholder feature in Blazor has implementation challenges related to how we need to not only expose the Selected property for users to set on a ListOption, but we also need to be able to handle changing the Selected state based on the value changing on the Select.
Currently, the NimbleListOption Razor component supplies a Selected property, but it's not even being leveraged in the template, meaning that the Selected property currently on ListOption is basically broken/useless.
The current version of Microsoft's Fluent UI Blazor library seems to have addressed these problems (albeit in a seemingly goofy way). Their FluenOption now binds the selected attribute of the web component to the Selected property on the Razor component as expected, and then also manages toggling that state via a handler that gets called not only when the option is clicked, but also when the keydown handler runs for the owner component.

Removed tech debt label because this has client-facing impact so it's hard to prioritize against the backlog of internal dev-facing work.


columnar support for Arrow tables

Detect Arrow tables and use as much of the direct access to the columns as we can—first and foremost, by not materializing the data on mark.initialize, and by routing a string accessor to getChild.
We don't add apache-arrow as a dependency (which means detection is done with duck typing of the methods we use… we could reinforce this a bit if needed, but I think that's fine).
The story is a bit complicated in the group transform (and maybe other places?) because we're actually making a new output data which currently uses take and map to create a new dataset that "looks like" the original data array.
In the Arrow table case, we might want take to be a "filtered" table, but I don't think it exists (API reference). We return instead an array of Row objects (which are Proxy objects into the columns); it's probably the best memory-wise, even though I don't like the looks of it. Anyway they're easy to convert to regular objects by writing ({...d}).
For a more thorough investigation of the places where we assume that values are arrays, I ran all the unit tests by replacing the data by an Arrow table (in Mark and facet data). This resulted in 25 "changed" snapshots (see diff); all of them are, it seems, only due to dates that change during the conversion to arrow. None of them were crashes.
I still need to investigate why the dates are modified (I'm thinking that may be because Arrow coerces them to Date32<day> — nope, they are DateMillisecond).
closes #191
cc: @jheer

The issue with dates can be reduced to this (which is independent of Plot, ):
import * as Arrow from "apache-arrow";
const data = [{date: new Date(1950, 1, 2)}]
console.log(data, [...Arrow.tableFromJSON(data)].map(d=> ({...d})));

// [ { date: 1950-02-01T23:00:00.000Z } ] [ { date: 1950-03-23T16:02:47.296Z } ]
you can see that the date is off by a lot. Am I missing something, should I open an issue on https://github.com/apache/arrow @domoritz?

Hmm, we’ve had success reading Date values in DuckDB transferred in Arrow format. So I’d be sure to check if this is an encoding problem (Date to Arrow) or a decoding issue (Arrow to Date) first. Some encoders in Arrow JS (eg for Decimal) are known to be broken. I sometimes have had to use DuckDB or pyarrow to generate Arrow bytes for testing.
On a related note, in Mosaic we special case Timestamp types as Arrow JS returns those as numbers; we then instantiate Date objects ourselves.

The internal data is like this:
 Data {
    type: DateMillisecond [Date] { typeId: 8, unit: 1 },
    children: [],
    dictionary: undefined,
    offset: 0,
    length: 1,
    _nullCount: 0,
    stride: 2,
    values: Int32Array(16) [
      -1314529784,
      -146,
      0,
     …

The date is encoded on the two first 32bit numbers, which I decode manually to (-146*(2**32)) -<PHONE_NUMBER> = -628563600000 which is my initial date.
So it's apparently the decoding that fails.

The date is encoded on the two first 32bit numbers, which I decode manually to (-146*(2**32)) -<PHONE_NUMBER> = -628563600000 which is my initial date.
So it's apparently the decoding that fails.

I think DuckDB produces either Date32 or Timestamp values, so I haven't tripped over this yet! Thanks for documenting it.

I've reported the Date issue at https://github.com/apache/arrow/issues/40718. I think it is orthogonal to this PR, since I can get the same error with Plot#main and an arrow table—though it does not explain all the 25 differences :(

Tested with https://github.com/apache/arrow/pull/40725 everything works smoothly (except for test "mark data parallel to facet data triggers a warning" which is not relevant). Thanks for the super quick fix @trxcllnt and @domoritz!

In the PR that fixes the Date bug (https://github.com/apache/arrow/pull/40725), @domoritz also changes the .get(i) value accessor into .at(i), which means that everywhere we use (array)[i] vs (arrow vector).get(i), we could now unify with (array or vector).at(i). This is probably the change that will help us the most.
An example of a (custom) data transform that breaks with arrow tables is here. It uses data.flatMap, which does not exist on the "fake array". We could add it easily.
(I don't think we need to rush this, we should probably wait for 40725 to land.)

Yeah, compatibility with native arrays was my main goal with supporting at. I'd be supportive of adding map and flatMap.
The next arrow release is in April so we could try to get the change in there (releases are ~ every three months). However, you probably don't want to rely on the latest arrow library being used so having a stop gap until the new library is common makes sense.

continued in (and superseded by) #2115


Refactor secrets to enable detecting missing secrets at the start of a run.

This is kind of undoing #68. Some highlights:

Objects can only access the secrets they list in their get_used_secrets method.
There is no longer a global SECRETS dict. All secrets are passed explicitly to objects.

I like where you're going with this, but I'm struggling with this usage a bit:
_API_KEY_SECRET = UseSecret(
    scope="together",
    key="api_key",
    instructions="See https://api.together.xyz/settings/api-keys",
    required=True,

[...]

    def get_used_secrets(self) -> Sequence[UseSecret]:
        return [_API_KEY_SECRET]

    def load(self, secrets: SecretValues) -> None:
        self.api_key = secrets.get_required("together", "api_key")

This is a fair bit of machinery, and it feels a bit duplicative to me. I take it you ended with this so that if something needs multiple keys, all the errors will happen at once?
I guess our target user here is the more serious sort of developer, and that the extra work for that person is saving difficulty for the less sophisticated users, so that seems like a good trade to me.

I take it you ended up with this so that if something needs multiple keys, all the errors will happen at once?

I did not have that in mind when designing, but it is definitely a benefit. My main guiding principle was to force objects to explicitly report what they need in a way that is hard to circumvent.
For example, we could instead monitor load for MissingSecretValue exceptions. However, this isn't guaranteed because the SUT/Annotator could use load to hold a reference to secrets and then only call get_required in a later function. In the current setup storing secrets is fine, because we've already verified the required secrets are present.
If we want to move toward a "best effort" style of detecting used secrets, I think there are refactors we could do with less work on SUT/Annotator owners.

Ah, ok. That makes sense. I generally favor supportive approaches over controlling ones, so normally I'd go for a "best effort" approach over "force". But given that this helps support our end users, I say let's go for it and see how SUT and Annotator makers like it.

For posterity, an alternate approach would be have the SUT and annotators just imperatively load the secrets, rather than asking for them. This is a more pythonic approach (ask forgiveness, not permission).
To use the HuggingFaceClient example:
    def load_secrets(self, secrets: Secrets) -> None:
        self.hugging_face_token = secret.get(
            scope="hugging_face",
            key="token",
            required=False,
            instructions="You can create tokens at https://huggingface.co/settings/tokens.",
        )

    def setup(self) -> None:
        self.model = AutoModelForCausalLM.from_pretrained(
            self.model_path,
            trust_remote_code=True,
            token=self.hugging_face_token,
            **self.hg_kwargs,
        ).to(self.device)
        self.wrapped_tokenizer = create_tokenizer(
            self.model_path, self.hugging_face_token, **self.hg_kwargs
        )

Where Secrets.get() looks like:
class Secrets:
    def get(scope: str, key: str, required: bool, instructions: str) -> str:
        secret = self._get_secret(scope, key)
        if required and secret is None:
            raise SecretNotFoundError(scope, key, instructions)
        return secret

Then the runner can call all load_secrets() methods before the run:
secret_not_found_errors = []
for secret_requester in annotators + [sut]:
    try:
        secret_requester.load_secrets(secrets)
    except SecretNotFoundError as error:
        secret_not_found_errors.append(error)

raise MissingSecretsError(collate_missing_secrets(secret_not_found_errors))

For posterity, an alternate approach would be have the SUT and annotators just imperatively load the secrets, rather than asking for them

That has two possible issues:

In your example, imagine HuggingFaceClient's load_secrets function just did self.secrets = secrets, then calls self.secrets.get in setup. That is perfectly valid code, but we won't detect failures until setup, which is outside the collate_missing_secrets call.
If an object has 3 get calls in load_secrets, we'll only get one SecretNotFoundError.


[Bug] sp_helpme - Case [name] in all_objects ORDER BY

Describe the bug
On case sensitive collations, I receive the error message below returned when running sp_helpme without an parameters.
Msg 209, Level 16, State 1, Line 17
Ambiguous column name 'name'.

This is caused within the ORDER BY declaration on the first SET @SQLString. [name] should be [Name].
To Reproduce
Run sp_helpme from a CS collation.
Expected behavior
I expect to a return of all the objects.
Versions(please complete the following information):

OS: N/A
SQL Server: SQL Server 2014 ( any CS collation )
SSMS: v18

Additional context
I'm working on the pull request now.

Thanks again for the bug & fix!


PlatformIO error for String operators: more than one operator "+" matches these operands

Using PlatformIO it compiles successfully, but String operators indicates error for any String operators due to MB_String:
Code:
  String str1 = "string1";
  String str2 = "They are a simple Strings " + str1;

Error message:
more than one operator "+" matches these operands
function "operator+(const StringSumHelper &lhs, const String &rhs)" (declared at line 185 of "C:\Users\x\.platformio\packages\framework-arduinoespressif32\cores\esp32\WString.h")
function "operator+(MB_String &&lhs, const MB_String &rhs)" (declared at line 2048 of "C:\Users\detre\.platformio\packages\framework-arduinoespressif32\libraries\ESP-Mail-Client-master\src\extras\MB_String.h")
operand types are: const char [27] + String
function "operator+(MB_String &&lhs, char rhs)" does not match because argument #2 does not match parameter
...

Platformio.ini (no Global lib copy):
[platformio]
default_envs = upload

[env:upload]
platform = espressif32
board = esp32dev
framework = arduino
monitor_speed = 115200
monitor_filters = direct, esp32_exception_decoder
build_flags =
    -mfix-esp32-psram-cache-issue

lib_deps =
  mobizt/ESP Mail Client

build_type = release
upload_port = COM6
upload_speed = 921600

I used "disable error squiggles" to make errors disappear. I have a rather large code, where it gives tons of errors. Recommendation?

It's VSCode issue not library issue (just IntelliSense syntax error) and not PlatformIO issue and it is related to Strung class instead of MB_String.
You can search PlatformIO community forum or search google and for this issue.

It is not only me. Concerning IntelliSense I found this, also related to MB_String:
https://community.platformio.org/t/false-problem-in-intellisense-in-vscode-more-than-one-operator-matches-these-operands/32347/3
I can't find any resolution to this. Could you help?

I can't remember that this issue existed on ESP32 or ESP8266 PIO platforms.
For me it gone for a while after the PlatformIO and the Arduino core updated but can't remember when.
You should update both PIO and Arduino Core.

I am still struggling with this. Removing ESP-Mail-Client solves these errors in my project. So it is the library does something.
MB_String.h:879:56: error: call of overloaded 'appendNum(int&)' is ambiguous
MB_String.h:881:57: error: call of overloaded 'appendNum(short unsigned int&)' is ambiguous
MB_String.h:889:53: error: call of overloaded 'appendNum(bool&)' is ambiguous
MB_String.h:1530:17: error: ambiguous overload for 'operator+=' (operand types are 'MB_String' and 'int')
MB_Time.h:298:13: error: ambiguous overload for 'operator+=' (operand types are 'MB_String' and 'int')
etc...

I don't have any problem with that as library developed in PlatformIO and intensively tested in many platforms.
It looks like your PlatformIO installation problem. Please try to uninstall and reinstall PlatformIO and the platform SDK.

I reinstalled it. Even tried Arduino as EspIdf component. Seems IDF is more strict in C++ error checking. Same errors. In total 20-30, some examples:
MB_String.h[119]: expected nested-name-specifier before 'enable_if'
MB_String.h[553]: call of overloaded 'appendNum(bool&)' is ambiguous
MB_String.h[781]: call of overloaded 'appendNum(bool&)' is ambiguous
...
MB_String.h[2037]:optimization attribute on 'MB_String operator+(const MB_String&, const MB_String&)' follows definition but the attribute doesn't match [-Wattributes]
...

The overload candidates for most of them:
.pio/libdeps/esp32dev/ESP Mail Client/src/extras/MB_Time.h:298:13: error: ambiguous overload for 'operator+=' (operand types are 'MB_String' and 'int')
       tzStr += 0;
       ~~~~~~^~~~
In file included from .pio/libdeps/esp32dev/ESP Mail Client/src/./extras/MB_FS.h:43,
                 from .pio/libdeps/esp32dev/ESP Mail Client/src/extras/RFC2047.h:13,
                 from .pio/libdeps/esp32dev/ESP Mail Client/src/ESP_Mail_Client.h:39,
                 from include/email.h:3,
                 from include/menus\menu_debug.h:10,
                 from include/menus\menu_main.h:10,
                 from src/main.cpp:12:
.pio/libdeps/esp32dev/ESP Mail Client/src/./extras/MB_String.h:692:16: note: candidate: 'MB_String& MB_String::operator+=(StringSumHelper)'
     MB_String &operator+=(StringSumHelper rval)
                ^~~~~~~~
.pio/libdeps/esp32dev/ESP Mail Client/src/./extras/MB_String.h:700:16: note: candidate: 'MB_String& MB_String::operator+=(const __FlashStringHelper*)'
     MB_String &operator+=(const __FlashStringHelper *str)
                ^~~~~~~~
.pio/libdeps/esp32dev/ESP Mail Client/src/./extras/MB_String.h:728:16: note: candidate: 'MB_String& MB_String::operator+=(const string&)'
     MB_String &operator+=(const std::string &rhs)
                ^~~~~~~~
.pio/libdeps/esp32dev/ESP Mail Client/src/./extras/MB_String.h:734:16: note: candidate: 'MB_String& MB_String::operator+=(const String&)'
     MB_String &operator+=(const String &rhs)
                ^~~~~~~~
.pio/libdeps/esp32dev/ESP Mail Client/src/./extras/MB_String.h:753:16: note: candidate: 'MB_String& MB_String::operator+=(const MB_String&)'
     MB_String &operator+=(const MB_String &rhs)
                ^~~~~~~~
.pio/libdeps/esp32dev/ESP Mail Client/src/./extras/MB_String.h:759:16: note: candidate: 'MB_String& MB_String::operator+=(const char*)'
     MB_String &operator+=(const char *cstr)
                ^~~~~~~~
.pio/libdeps/esp32dev/ESP Mail Client/src/./extras/MB_String.h:773:16: note: candidate: 'MB_String& MB_String::operator+=(char)'
     MB_String &operator+=(char cstr)
                ^~~~~~~~
.pio/libdeps/esp32dev/ESP Mail Client/src/./extras/MB_String.h:779:16: note: candidate: 'MB_String& MB_String::operator+=(bool)'
     MB_String &operator+=(bool value)
                ^~~~~~~~
.pio/libdeps/esp32dev/ESP Mail Client/src/./extras/MB_String.h:791:16: note: candidate: 'MB_String& MB_String::operator+=(mb_string::MB_StringPtr)'
     MB_String &operator+=(MB_StringPtr ptr)

I will say this for the last time and will limit the conversation on this user error.
I don't have this such MB_String compilation error from intensively test in 5 years for AVR, STM32, SAMD, Teensy, ESP8266, ESP32 and RaspberryPi Pico on PlatformIO and Arduino IDE (full error and warnings compile level)
The library has no error when compile with the CI test too.
If you know how to debug the information from the compiler errors especially "ambiguous", you will know why.
The different sources path of the same headers "MB_String,h" in this case from different location leads this compilation error.
The library examples compilation must be ok. If you still get the error when just compiling the library examples, your IDE cache will be the problem.
The library and IDE installation should not be in the sandbox, virtual folder, OS strict location or cloud storage folder (OneDrive, Google Drive).

If you don't understand what I mean for "ambiguous"
Please look at the following folders from the compilation errors.
.pio/libdeps/esp32dev/ESP Mail Client/src/extras
.pio/libdeps/esp32dev/ESP Mail Client/src/./extras
How you create the later one (.pio/libdeps/esp32dev/ESP Mail Client/src/./extras) as it is does not created by PIO?


Using attr_accessible in the controller

Rails supports attr_accessible in the controller, which imo, is where it belongs.
What's the best way to get inherited_resources working with attr_accessible?

There isn't afaik. Notice inherited resources has a deprecation notice though, if you are using it and interested in maintaining it, please let me know. :)

man, i just found inherited_resources like two weeks ago and it's already deprecated.  I can't keep up. :)

Closing this one


I need to put heading attributes in sequential order in the html and css

I put the heading elements in order


Adding Natalie WA to GTN contributors list

Update CONTRIBUTORS.yaml
--> Please remove all this text before submitting your PR, it's just a checklist to help you :) <--

Give your pull request a good title

e.g. "Fix typo in ansible-galaxy tutorial" or "Add new transcriptomics tutorial covering a new sequencing technology"

Describe your changes in detail, list anything you still need some help with or things that are still TODO
Check that your images are allowed to be re-hosted by the GTN!
Still working on it? Make it a Draft pull request

Once it is ready, choose Ready for Review
Then the automated tests will run and we will know to review and merge it

@teresa-m you can now add Natalie to Galaxy Academy event with id natalie-wa

@shiltemann thanks I will add Natalie.


Compile list of financial functions needed

What are the missing calculations we need to get this to 1.0?

keleven

This could be a starter list
http://www.excelfunctions.net/Excel-Financial-Functions.html


support constraint(s)

PR progress checklist (to be filled in by reviewers)

[ ] Changes to documentation are appropriate (or tick if not required)
[ ] Changes to tests are appropriate (or tick if not required)
[ ] Reviews completed

What type of PR is this?

Primary type

[ ] [build]    Changes related to the build system
[ ] [chore]    Changes to the build process or auxiliary tools and libraries such as documentation generation
[ ] [ci]       Changes to the continuous integration configuration
[x] [feat]     A new feature
[ ] [fix]      A bug fix
[ ] [perf]     A code change that improves performance
[ ] [refactor] A code change that neither fixes a bug nor adds a feature
[ ] [revert]   A change used to revert a previous commit
[ ] [style]    Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc.)

Secondary type

[ ] [docs]     Documentation changes
[ ] [test]     Adding missing or correcting existing tests

Does this PR introduce a BREAKING CHANGE?

No.
Related issues and/or pull requests

N/A
Describe the changes you're proposing

This adds support for the constraint from and constraints from keywords as documented in ntpd.conf(8).
Pillar / config required to test the proposed changes

openntpd:
  constraint:
    - <IP_ADDRESS>
  constraints:
    - 'www.google.com'

This results in:
constraint from <IP_ADDRESS>
constraints from www.google.com

Debug log showing how the proposed changes work

Documentation checklist

[x] Updated the README (e.g. Available states).
[x] Updated pillar.example.

Testing checklist

[ ] Included in Kitchen (i.e. under state_top).
[ ] Covered by new/existing tests (e.g. InSpec, Serverspec, etc.).
[ ] Updated the relevant test pillar.

Additional context

@jasperla Apologies for the delayed response.
@n-rodriguez Would you be able to review this PR?

@myii LGTM

Merged -- thanks @jasperla and @n-rodriguez!

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


Sync health check

What is wrong?
With the new plugin architecture we end up with loose coupling.  I had a long running sync where the discovery service stopped responding (no diagnosis).  This meant my client stopped establishing new connections.
How can it be fixed
We should implement some kind of health check in the main eth1 syncing trinity process (and really any of the main apps we write) that validates that all of the necessary components are running/responding and crashes the application if they are unresponsive.

This has become more important imho based on my work on #536 .  The services that plugin runs are critical to the health of the node and if they crash the trinity node should really probably crash with it.

Maybe DaemonPlugin to follow along our naming from BaseService?
Or maybe this should just apply to all plugins: if any one of them dies, it takes down Trinity with it.

I think this makes a lot of sense!
A simple implementation might be for the main process to run a watchdog: main_endpoint.subscribe(FeedTheWatchdog, self.feed_watchdog). All processes that we consider important run a coro which sends something like endpoint.broadcast(FeedTheWatchdog("discovery")) every x seconds. If the watchdog ever notices that it's stopped receiving messages from any one service it brings trinity down. This would detect things like the discovery process failing to start and the discovery process crashing or stalling it's event loop, but it wouldn't detect some other critical coro failing and leaving behind just the coro which feeds the watchdog.

Watchdog is probably a good starting point.  I'm thinking for this to be robust, we should eventually be able to declare something like:
The discovery service is healthy if:

The response to event X satisfies this health_check_fn(response)
...

Basically, registering some set of periodic checks that run which can have arbitrary logic.  These could start out as hard coded/.


feat(sbb-journey-header): component refactoring

Preflight Checklist

[x] I have read the Contributing Guidelines for this project.
[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the pull request tracker for a Pull Request (PR) that matches the one I want to submit, without success.

Issue
This PR Closes #1240
Pull request checklist
Please check if your PR fulfills the following requirements:

[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been reviewed and added / updated if needed (for bug fixes / features)

See Review Guidelines for more information what is checked during review process.
Changes
Changes in this pull request:

component refactoring

Browsers
I tested the build on the following browsers:

[x] Firefox Desktop
[x] Chrome Desktop
[x] Edge Desktop
[x] Safari Desktop
[ ] Chrome Mobile
[ ] Safari Mobile

Screen readers
I tested the build on the following browsers:

[x] JAWS Firefox Desktop
[x] JAWS Chrome Desktop
[x] NVDA Firefox Desktop
[x] NVDA Chrome Desktop
[x] VoiceOver Safari Desktop
[x] VoiceOver Chrome Desktop
[ ] VoiceOver Safari Mobile
[ ] Android Accessibility Suite Chrome Mobile

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[x] Other (please describe): component refactoring

Does this introduce a breaking change?

[x] Yes
[ ] No

appearance prop has been removed;
isRoundTrip prop has been renamed to roundTrip;
markup prop has been renamed to level and values has been changed;
negative prop has been added;
size has now m and l as values instead of 5 and 4.

Other information

Codecov Report

Merging #1539 (6af588d) into master (ab917b0) will decrease coverage by 1.35%.
The diff coverage is 62.27%.

@@            Coverage Diff             @@
##           master    #1539      +/-   ##
==========================================
- Coverage   54.85%   53.50%   -1.35%
==========================================
  Files          49       84      +35
  Lines        1659     3568    +1909
  Branches      406      984     +578
==========================================
+ Hits          910     1909     +999
- Misses        671     1524     +853
- Partials       78      135      +57

Impacted Files
Coverage Δ

...mponents/sbb-accordion-item/sbb-accordion-item.tsx
0.00% <0.00%> (-37.21%)
:arrow_down:

src/components/sbb-clock/sbb-clock.tsx
0.00% <0.00%> (ø)

src/components/sbb-link-list/sbb-link-list.tsx
84.00% <ø> (-4.24%)
:arrow_down:

src/components/sbb-link/sbb-link.tsx
59.57% <ø> (-31.86%)
:arrow_down:

src/components/sbb-logo/sbb-logo.tsx
0.00% <ø> (ø)

src/components/sbb-menu-action/sbb-menu-action.tsx
61.90% <ø> (ø)

src/components/sbb-menu/sbb-menu.tsx
26.54% <ø> (ø)

...ts/sbb-navigation-action/sbb-navigation-action.tsx
71.42% <ø> (ø)

...onents/sbb-navigation-list/sbb-navigation-list.tsx
84.00% <ø> (ø)

...mponents/sbb-checkbox-group/sbb-checkbox-group.tsx
32.00% <32.00%> (ø)

... and 68 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


cmd/cover: inconsistent NumStmt with prometheus

What version of Go are you using (go version)?

$ go version
go version go1.16.2 linux/amd64

Does this issue reproduce with the latest release?
It is indeed the latest release
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/root/.cache/go-build"
GOENV="/root/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/root/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/root/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/root/.go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/root/.go/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="go1.16.2"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/dev/null"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build2943966716=/tmp/go-build -gno-record-gcc-switches"

What did you do?
git clone https://github.com/prometheus/prometheus
cd prometheus/promql/parser
go test -coverprofile=wrong.out
go tool cover -html wrong.out
What did you expect to see?
Get the HTML coverage report
What did you see instead?
Error message cover: inconsistent NumStmt: changed from 2 to 1
The wrong.out contains indeed (running grep generated_parser.y.go wrong.out | grep 178
github.com/prometheus/prometheus/promql/parser/generated_parser.y.go:178.0,180.0 2 1
github.com/prometheus/prometheus/promql/parser/generated_parser.y.go:179.0,178.0 2 1
github.com/prometheus/prometheus/promql/parser/generated_parser.y.go:176.0,178.0 1 1
github.com/prometheus/prometheus/promql/parser/generated_parser.y.go:178.0,180.0 1 1

I found this by modifying the covermessage with adding %#+v about other fields

It still does not work on my laptop.

OS: macOS Catalina 10.15.5 19F101 x86_64 / Intel Chip
Go version: 1.18.3

Any update on this issue?

I encountered the same problem with the latest Go 1.20. Unfortunately, it's in a proprietary software, but also applies to a generated file -- in our case by Ragel. I'll try to find reproducer for this.

friendly ping @golang/runtime ?

@neild the line directive strikes again it seems (messing up line numbers between the generated file and the generating file)

It seems that //line directives inserted into generated parser by goyacc result in invalid entries in coverprofile.
The workaround could be to disable line directives via -l flag, see https://github.com/zalando/skipper/pull/2450.


Microsoft Edge could not be found - Unable to launch a file

Environment (please complete the following information):

OS: MacOS v12.14 - Monterey
Extension version: 2.1.0

Describe the bug:

When trying to launch - even the simplest configuration - vsCode raises the error

🛑
Microsoft Edge could not be found. Ensure you have installed Microsoft Edge and that you have selected 'default' or the appropriate version of Microsoft Edge in the extension settings panel.

even this configuration fails:
    {
      "type": "msedge",
      "name": "Launch Microsoft Edge",
      "request": "launch",
      "file": "${workspaceFolder}/.vscode/blank.html",
    },

while changing msedge by firefox DOES work
Repro steps:

Install the extension
Click on the activity panel
Configure launch.json
change the URL
Launch Instance
FAILS

Expected behavior:

I expect a new Edge instance to run my blank.html (firefox is able to launch it)
this is the file I'm trying to open:
<!DOCTYPE html>
<html lang="en">
  <head>
    <meta charset="utf-8" />
    <title>example</title>
    <meta name="viewport" content="width=device-width, initial-scale=1" />
  </head>
  <body></body>
</html>

Additional context:

The extension seems to be installed just fine.:

VIDEO
https://user-images.githubusercontent.com/2211052/181476029-61e27943-450c-4023-b557-591efe7407e6.mov
In this video:

I have configured the simplest launch possible.
try to run it with msedge -> fails 🔴
try to run it with pwa-msedge -> fails 🔴
change to firefox -> OK ✅

Hi there, did you download and install Edge? On a Mac, it isn't available by default, you need to get it at:
https://www.microsoft.com/en-us/edge
This extension needs Edge to work, it doesn't support any other browsers. Sorry if that wasn't clear. Firefox will open the file, but none of the communication between browser and VS Code will work.

Also <3 for your incredibly detailed issue post!

ok this is embarrassing 😭 .
Works like a charm, thanks !


Remove fixed seed for test_l1_loss and test_l2_loss

Description
ubuntu@ip-172-31-20-16:~/mxnet/tests/python/unittest$ MXNET_TEST_COUNT=10000  nosetests test_loss.py:test_l1_loss; MXNET_TEST_COUNT=10000  nosetests ../gpu/test_operator_gpu.py:test
_l1_loss;
/usr/local/lib/python3.5/dist-packages/nose/util.py:453: DeprecationWarning: inspect.getargspec() is deprecated, use inspect.signature() instead
  inspect.getargspec(func)
[INFO] Setting module np/mx/python random seeds, use MXNET_MODULE_SEED=27178258 to reproduce.
^C[INFO] 2454 of 10000: Setting test np/mx/python random seeds, use MXNET_TEST_SEED=1464611288 to reproduce.

----------------------------------------------------------------------
Ran 1 test in 2414.984s

OK
/usr/local/lib/python3.5/dist-packages/nose/util.py:453: DeprecationWarning: inspect.getargspec() is deprecated, use inspect.signature() instead
  inspect.getargspec(func)
[INFO] Setting module np/mx/python random seeds, use MXNET_MODULE_SEED=376845522 to reproduce.
[INFO] 9113 of 10000: Setting test np/mx/python random seeds, use MXNET_TEST_SEED=1915136543 to reproduce.

----------------------------------------------------------------------
Ran 1 test in 8659.136s

OK

ubuntu@ip-172-31-20-16:~/mxnet/tests/python/unittest$ MXNET_TEST_COUNT=10000  nosetests test_loss.py:test_l2_loss
/usr/local/lib/python3.5/dist-packages/nose/util.py:453: DeprecationWarning: inspect.getargspec() is deprecated, use inspect.signature() instead
  inspect.getargspec(func)
[INFO] Setting module np/mx/python random seeds, use MXNET_MODULE_SEED=1323890397 to reproduce.
[INFO] 2890 of 10000: Setting test np/mx/python random seeds, use MXNET_TEST_SEED=227084913 to reproduce.

----------------------------------------------------------------------
Ran 1 test in 4019.911s

Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[ ] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[ ] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

@marcoabreu

There's still an open review

@marcoabreu replied in https://github.com/apache/incubator-mxnet/pull/11995#discussion_r207601830

Please merge yourself when PR is ready

@haojin2 It's a minor thing here but please don't delete others' comment... See the last section at https://cwiki.apache.org/confluence/display/MXNET/Development+Process


Unable to start properly

When starting with 1.10.0 or 1.10.2 I get a gray screen with message:
"A server with the specified hostname could not be found." with a reload button. Pressing reload a few times will cause a crash.

This seems to have resolved itself. Perhaps just coincidental that it lined up with an upgrade to the player. Possibly network issues on my end. Closing.


NetGear: Add variable colorspace frames support to Frame Compression

Detailed Description

NetGear currently only supports BGR colorspace frames with its Frame Compression but rejects any other colorspaces(always defaults to BGR). This issue will add support for additional colorspaces.
Context

NetGear API only supports BGR colorspace as of now and doesn't let users to manually alter it when its Frame Compression mode is enabled. But the underline simplejpeg library do support specifying input frames colorspace with its encode_jpeg and decode_jpeg methods. So this issue will track addition of this support for letting user directly control input frames colorspace easily.
Your Current Environment

VidGear version:  0.2.2
Branch:  development
Python version:  latest
Operating System and version: all

Successfully resolved and merged in commit https://github.com/abhiTronix/vidgear/commit/2a377007f4dbc06f9d84a9b7ff6ed081417c6618


.submodules file contains paths which are relative to itself (not the .git root)

Mmmm, maybe I am mis-reading this, but it looks like the old way is correct?
The submodule file should contain file paths relative to git root.

There were old versions of cabal-extras that behaved relative to the project name. :older_man:

Makes sense, ok, killing this


Add Argo app status to search collector

Related Issue: https://github.com/open-cluster-management/backlog/issues/11353
Description of changes

Added the Argo app status to the search collector

The search query now return the status of the Argo app:

/hold so that its not automatically merged with a review.


chore(zero): Switch xxhash to pure js version.

I did some quick benchmarking and though this is much slower than the wasm version, it's in the neighborhood of 4-5x not 10x, so that's nice. Also hashing is just so fast it's not likely to be the biggest of our problems for ast hashing or inserting optimistic data. Let's unblock people from using Zero in env's like RN first and optimize this if it profiles show it is a problem later.

@natew do you have a way to test this locally w/o me doing a build to ensure it fixes things for you.

@darkgnotic do I need to do anything other than bump PROTOCOL_VERSION?

@darkgnotic FYI I don't think invalidation.ts was being used, so removed.

we don't even use xxhash to hash anything besides the AST.
export function rowIDHash(id: RowID): string { is dead code. Check it out: https://github.com/rocicorp/mono/pull/3441

export function rowIDHash(id: RowID): string { is also dead code. So we don't use xxhash in that many places. Just toPrimaryKeyString? which appears to only be used in crud mutations.

It's also used when inserting entities into replicache when processing pokes https://github.com/rocicorp/mono/blob/ab69c6cc8f0cca9d568bdc2372bf555319cd9a33/packages/zero-client/src/client/zero-poke-handler.ts#L317
this is probably the most performance sensitive usage.

@darkgnotic do I need to do anything other than bump PROTOCOL_VERSION?

That should be it.
Good catch on needing to bump the PROTOCOL_VERSION. Even though the structure doesn't change, all of the query's hashes will.  👍


Release/1.1.0

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

:exclamation: No coverage uploaded for pull request base (main@77504a9). Click here to learn what that means.

Additional details and impacted files
@@          Coverage Diff           @@
##             main    #165   +/-   ##
======================================
  Coverage        ?   9.26%
======================================
  Files           ?      11
  Lines           ?     637
  Branches        ?       0
======================================
  Hits            ?      59
  Misses          ?     578
  Partials        ?       0

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Tests with the release image in local environment were successful


redirect writes headers always

Its look suspecious. The code below will redirect:
    public function actionIndex()
    {
        $this->redirect("/xxx");
    }

Method documentation says that:
     * You can use it in an action by returning the [[Response]] directly:
     *

if response was not returned, redirect should not be

It modifies the Response object, if you do not reset it after calling redirect() the response is sent with the headers set by the redirect method.


⚠ Bofrost.de has degraded performance

In 7147e7c, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1481 ms

Resolved: Bofrost.de performance has improved in 7063c3d after 311 days, 3 hours, 15 minutes.


TWEAK: Update the default config Insight servers for BTC and LTC.

Make config use openbazaar.org Insight hosts by default.

Coverage remained the same at 33.728% when pulling b290b1bcbb03c62211eeeac00d48a2513674060f on TS_multiwallet into ac1f54b104f74ee195d9ed470e763ba45c9e6589 on multiwallet.

@tyler-smith I assume we should wait until the Blockbook research shakes out before merging this?

Yeah. I made this PR right before we started investigating BB. I'll just reopen it later.


Add class to anchor

Is there a way to get the properties dialog for anchors? Would been nice to add classes to anchors.

Hi @Rverm, the inspector bar doesn't display inline tags as is (e.g tags within a paragraph or other textbox). You currently need to add the style manually by viewing the code tab in the properties dialog for the parent element and adding the class against the tag in the HTML.
Can you describe your use case? If you're looking to insert a button styled link for example then creating a tool to do this might be a simple option.

@anthonyjb yes, looking for button styled links as I use multiple/different styles for links. Thanks!

Do you want the link/button to be inside the paragraph or sit at the same level as the paragraph, e.g:

foo

Bar

or

foo

Bar

In the paragraph would be nice.

Hi @Rverm sorry for the late reply, there are a number of ways to approach this, I think the nicest approach for your user's is a custom tool (though it does feel a little overkill for the situation). The following code mocks up a Button link class you could use as a basis (it's heavily based on the link tool - just cut down some what):
class ContentTools.Tools.Button extends ContentTools.Tools.Bold

    # Insert a button link.

    ContentTools.ToolShelf.stow(@, 'button')

    @label = 'Button'
    @icon = 'link' # Or set a custom one
    @tagName = 'a'

    @getAttr: (attrName, element, selection) ->
        # Get an attribute for the element and selection

        # Find the first character in the selected text that has an `a` tag
        # and return the named attributes value.
        [from, to] = selection.get()
        selectedContent = element.content.slice(from, to)
        for c in selectedContent.characters
            if not c.hasTags('a')
                continue

            for tag in c.tags()
                if tag.name() == 'a'
                    return tag.attr(attrName)

        return ''

    @apply: (element, selection, callback) ->
        applied = false

        # If the selection is collapsed then we need to select the entire
        # entire link.
        if selection.isCollapsed()

            # Find the bounds of the link
            characters = element.content.characters
            starts = selection.get(0)[0]
            ends = starts

            while starts > 0 and characters[starts - 1].hasTags('a')
                starts -= 1

            while ends < characters.length and characters[ends].hasTags('a')
                ends += 1

            # Select the link in full
            selection = new ContentSelect.Range(starts, ends)
            selection.select(element.domElement())

        # Text elements
        element.storeState()

        # Add a fake selection wrapper to the selected text so that it
        # appears to be selected when the focus is lost by the element.
        selectTag = new HTMLString.Tag('span', {'class': 'ct--puesdo-select'})
        [from, to] = selection.get()
        element.content = element.content.format(from, to, selectTag)
        element.updateInnerHTML()

        # Measure a rectangle of the content selected so we can position the
        # dialog centrally.
        domElement = element.domElement()
        measureSpan = domElement.getElementsByClassName('ct--puesdo-select')
        rect = measureSpan[0].getBoundingClientRect()

        # Set-up the dialog
        app = ContentTools.EditorApp.get()

        # Modal
        modal = new ContentTools.ModalUI(transparent=true, allowScrolling=true)

        # When the modal is clicked on the dialog should close
        modal.addEventListener 'click', () ->
            @unmount()
            dialog.hide()

            if element.content
                # Remove the fake selection from the element
                element.content = element.content.unformat(from, to, selectTag)
                element.updateInnerHTML()

                # Restore the selection
                element.restoreState()

            callback(applied)

        # Dialog
        dialog = new ContentTools.LinkDialog(
            @getAttr('href', element, selection),
            @getAttr('target', element, selection)
            )

        # Get the scroll position required for the dialog
        [scrollX, scrollY] = ContentTools.getScrollPosition()

        dialog.position([
            rect.left + (rect.width / 2) + scrollX,
            rect.top + (rect.height / 2) + scrollY
            ])

        dialog.addEventListener 'save', (ev) ->
            detail = ev.detail()
            applied = true

            # Clear any existing button link
            element.content = element.content.unformat(from, to, 'a')

            # If specified add the new button link
            if detail.href

                # Set the CSS class for styling the button link
                detail['class'] = 'button'

                a = new HTMLString.Tag('a', detail)
                element.content = element.content.format(from, to, a)
                element.content.optimize()

            element.updateInnerHTML()

            # Make sure the element is marked as tainted
            element.taint()

            # Close the modal and dialog
            modal.dispatchEvent(modal.createEvent('click'))

        app.attach(modal)
        app.attach(dialog)
        modal.show()
        dialog.show()

You can find more a detailed explanation on how to add tools to your toolbox in the Adding new tool tutorial, if you have time it's not a bad idea to read through this if you're making custom tools.
If your not using CoffeeScript (which I recommend if you're creating custom tools but it's not required), then the code above be converted to JS as a starting point here http://js2coffee.thomaskalka.de/#coffee2js though the output cans seem a little cryptic at first.

@andyjbgm Great, going to take a closer look. Thanks!

@Rverm let me know how you get on :)

Any update about this ? In the bootstrap era, it would be really useful to be able to open the popup dialog for link element to add some classes or custom attributes, without having to put them in a block level element to edit code directly (things you cannot explain to a non-technical user).


Add custom Control

If I see correctly the .js main file I declare custom controls like that:
...,
customControls : [
                L.mapbox.geocoderControl('foursquare.map-ikj05elx'),
                L.control.locate({
                                    setView: true,
                                    locateOptions: {maxZoom: 10}
                })
],...

Which I would traditionally declare like:
 map.addControl( L.mapbox.geocoderControl('foursquare.map-ikj05elx') )
       .addControl( L.control.locate({
                                    setView: true,
                                    locateOptions: {maxZoom: 10} })
        )

However it doesn't work. Do I do something wrong?

Old issue, code has changed a lot since this, closing. Please reply if you think this should be reopened. Thanks.

This should be reopened
https://github.com/tombatossals/angular-leaflet-directive/issues/929

I worked this out by commenting the following lines.
if (!isValidControlType(controlType)) {
$log.error(errorHeader + ' Invalid control type: ' + controlType + '.');
return;
}
OR may be just add "custom" in controlTypes array around line 284.


Issue/2706 cleanup docs

Submission Checklist

[ ] Run unit tests: ./runTests.py src/test/unit
[ ] Run cpplint: make cpplint
[x] Declare copyright holder and open-source license: see below

Summary
Removes directory src/docs and all make tasks for documentation.
Intended Effect
Keep all user-facing documentation in repo docs.  This simplifies build and testing requirements for this repo.
How to Verify
Check makefile tasks.  Check that there are no longer make tasks make manual, make docs, make clean-manual, make clean-docs
Copyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company):
Columbia University
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

I think Bob should probably review this - I don't know anything about the docs.

@seantalts and @bob-carpenter - this needs to go in for 2.19 release.

@bob-carpenter or @VMatthijs or @seantalts please review so this can go in for 2.19
Bob and Matthijs - everything that was here has been captured to docs - just checked.

I thought we had decided not to do this at some point? How is the Stan reference going to stay in sync with the code?

the doxygen stuff is still there.
only the markdown files are gone.

What about the function reference?

the functions reference is in the docs repo - this is a consequence of putting all docs online.
do we want to roll the docs back into the mono repo?  totally doable...  docs urls become mc-stan.org/stan/.... instead of mc-stan.org/docs

I don't know, to me it seems like it would be easier to have the actual functions reference and anything else that's likely to be really sensitive to the version number in the repo w/ the code. I think we could still have a docs repo that just hosts stuff online? I don't know. Like what's the plan for releases if they stay in a different repo?

(the release plan is extremely important as there is a release tomorrow and I'm performing it and I have no idea what to do about docs now).

don't understand distinction between functions reference and other manuals.
for the release, we need to generate the 2_19 docs from the docs repo.
we can bundle the pdfs for the 2_19 docs with the release.
the srcs in the docs repo are up-to-date and ready to go for 2.19.  in particular, syntax for affine transform is there.
running the docs python build script with MAJOR/MINOR args 2 19 will generate pdfs.
once the release is out, we need to then go into the website - mc-stan.github.io and update the documentation page.
happy to help with all of these tasks.

I mean the functions manual and Stan language reference are different from the "user's guide" or whatever we're calling all of the sections on modeling, right? I think they might tie more closely to the version of Stan than the user's guide does. An example might be that we don't want to confuse someone running 2.16 into thinking they have access to "for each" loops, or something similar. I don't know how important that is. @bob-carpenter do you have a sense?
My goals here are to 1) minimize incoming support requests 2) make it easy to tie a version to anything we release and 3) make releasing easier as time goes on, not harder. In mostly that order, I think. I'll also acknowledge that I don't have that good of a sense for how the docs are typically used, especially now that we're also releasing an HTML version.

the problem is that there are cross-references between the manuals.
at one point it was all the same LaTeX doc.
when we went to markdown things were split into 3 documents.
the 3 docs should be in the same repo - they share styling, bibliography, and again, cross-refs.

Do you think you could add a Jenkins stage that runs the script on the right repo when we merge to master on Stan?

OK, will try.

On Mar 17, 2019, at 6:13 PM, seantalts<EMAIL_ADDRESS>wrote:
I mean the functions manual and Stan language reference are different from the "user's guide" or whatever we're calling all of the sections on modeling, right? I think they might tie more closely to the version of Stan than the user's guide does.

That's right.
The problem with tying the docs into the rest of the code is
that they now require not only R, but pandoc installed in
order to build them.

An example might be that we don't want to confuse someone running 2.16 into thinking they have access to "for each" loops, or something similar. I don't know how important that is. @bob-carpenter do you have a sense?

I thought Mitzi had some plan for release versioning
of the docs on the web site separate from the releases.

My goals here are to 1) minimize incoming support requests 2) make it easy to tie a version to anything we release and 3) make releasing easier as time goes on, not harder. In mostly that order, I think. I'll also acknowledge that I don't have that good of a sense for how the docs are typically used, especially now that we're also releasing an HTML version.

I agree with your three priorities, but somewhere in the
mix needs to be (?) make contributing code easier, not harder.

On Mar 17, 2019, at 6:19 PM, Mitzi Morris<EMAIL_ADDRESS>wrote:

the problem is that there are cross-references between the manuals.

I think detailed cross-references should be eliminated and replaced
with references to the top-level doc with no direct link other
than to the page of documentation on the web site.
The reference manual should be self-contained.
The functions reference can make a top-level citation of the reference
manual.
These two need to be kept up to date with releases.
The user's guide can make a top-level citation of the reference manual
or names of chapters.  This should ideally be kept up with
releases, but it's not critical.
The methodology "book" should be pulled out from the user's
guide.  It can make top-level citations to other things.
This one does not need to be kept up to date with new
releases and there's even an advantage in keeping it lagging
a bit for users who haven't installed new versions.

Bob

The problem with tying the docs into the rest of the code is
that they now require not only R, but pandoc installed in
order to build them.

Why is this a problem for tying them into the code? For Jenkins, I can dockerize the build now that I learned how to do this in a better way (that doesn't involve circleCI). That might also be an easy way for developers to make the docs, actually. Are there other issues involved there?
I agree and/or defer to you on everything else you've said.
I know there is also the URL issue - I don't personally mind if the Stan user's guide is at /stan instead of /docs, though... and same for cmdstan at /cmdstan.

On Mar 19, 2019, at 4:43 AM, seantalts<EMAIL_ADDRESS>wrote:
The problem with tying the docs into the rest of the code is
that they now require not only R, but pandoc installed in
order to build them.
Why is this a problem for tying them into the code? For Jenkins, I can dockerize the build now that I learned how to do this in a better way (that doesn't involve circleCI). That might also be an easy way for developers to make the docs, actually. Are there other issues involved there?

Sure, if you can make it work, that'd be fine by me.
If we go back to requiring devs to keep the doc up to date with each PR, it'll require everyone to install R and pandoc.  No problem for me, as I have them installed.
Is the users guide going to move, too?  If so, that should wait until after the users guide and "book" are pulled apart.  We needed to keep the book in its own repo because it's not going to be kept up to date and Andrew's going to be messing with it in ways that violate all of our testing, coding style, etc., and we don't want that affecting the main stan-dev/stan repo.

I agree and/or defer to you on everything else you've said.
I know there is also the URL issue - I don't personally mind if the Stan user's guide is at /stan instead of /docs, though... and same for cmdstan at /cmdstan.

I don't care either, but I do think if things move, we need to forward appropriately forever, or links go stale.

If we go back to requiring devs to keep the doc up to date with each PR, it'll require everyone to install R and pandoc.

True, but that requirement can be independent of where the documentation lives, right? Though I think that having that requirement + putting it in a separate repo is operationally difficult. If we drop the update manual requirement (but who takes over?) and the manual release can be somehow automated I'd be just about as happy with that even if it lives in a different repo.

Is the users guide going to move, too?

That's in docs already right? That location makes sense to me.


Pseudo double data rate

Routers are not actually double data rate, but on the negative clock cycle the valid signal is set false in order to make the receive function sensitivity okay. That's bad... probably

We will leave it as such for now. Not highly relevant, router works quite well that way. Maybe write a remark in the user manual


Stack implementation using singly Linked List.

💥 Proposal
(All Basic operation of stack using singly Linked List .)
Push
Pop
Peek
Empty checker function.
Have you read the Contributing Guidelines on Pull Requests?
(yes.)

I would like to do this in c++

I would like to do this in c++.

@Rishikeshrajrxl  This issue is already exist 399

In #399 will be implementing using Array data structure.
But here  I will be implementing it using Linked List.

@Rishikeshrajrxl We cannot make too much variety of issues of the same Algorithm.
Better to look on some different issues.
Thanks !

ok


Fix CheckBox and CheckButton not using icon_max_width

quick attempt. follows original rendering behavior in terms of where things are drawn if the other currently not drawn icons are larger.
fixes #91697

I don't think there's anything major i have to do, this seems like it's fine and everything else related to fitting the icons (e.g. different aspect ratios between all the state textures) seems to work the same as the old behavior (icons are drawn to the top left corner of the max size of all the icons), they're just now all able to be hard capped to a specific width.

Might be worth documenting this in icon_max_width description. Something like "If the button has additional icons (e.g. [CheckBox], they will also be limited."

Added KoBeWi's doc suggestion.

Thanks! And congrats for your first merged Godot contribution :tada:


Update go.mod dependencies to address vulnerabilities

What this PR does / why we need it:
We need to update a few dependencies to ensure that we are able to remove from the go.mod the github.com/hashicorp/vault >= 1.10.0, < 1.10.3 to address vulnerabilities.

Which issue(s) this PR fixes:
Special notes for your reviewer:
Steps to reproduce
Check https://github.com/replicatedhq/kURL/security/dependabot
Does this PR introduce a user-facing change?
NONE
Does this PR require documentation?
NONE

Hi @emosbaugh,
I reverted the change to upgrade the rock (so we can achieve the goal without be blocked by). So could we move with this one?


sydney.exceptions.CreateConversationException: Failed to create conversation, received status: 404

Getting this error when using Sydney-Client with Python3.10 and Python3.11 on Ubuntu 22.04.03 and Debian 12
I know the _U cookie is correct so I'm unsure why im getting this error.
Please help.
Traceback (most recent call last):
File "/home/xxxx/testing.py", line 24, in
asyncio.run(main())
File "/usr/lib/python3.10/asyncio/runners.py", line 44, in run
return loop.run_until_complete(main)
File "/usr/lib/python3.10/asyncio/base_events.py", line 649, in run_until_complete
return future.result()
File "/home/xxxx/testing.py", line 7, in main
async with SydneyClient(bing_u_cookie="x", style="precise") as sydney:
File "/usr/local/lib/python3.10/dist-packages/sydney/sydney.py", line 93, in aenter
await self.start_conversation()
File "/usr/local/lib/python3.10/dist-packages/sydney/sydney.py", line 427, in start_conversation
raise CreateConversationException(
sydney.exceptions.CreateConversationException: Failed to create conversation, received status: 404
Unclosed client session
client_session: <aiohttp.client.ClientSession object at 0x7f494528a320>

Hi @tiernan1979 I believe I have seen this error before a few times when running the test suite. Does it keep happening even when you rerun your code?

Hi Vsakkas, I have just tried again this morning and it appears to be working.. Potentially while trying to automate the login via Selenium they blocked me potentially.
Thanks
Grant

That seems like a likely cause, Bing servers don't like when we do too many requests at once!
Let me know if this error persists later on, also, automating the login is a cool project to work on :)


wrong naming of Zinc Dust

I have fixed this issue days ago in the repo and just simply didn't release it.


Documenting TODOs

There are several TODOs scattered around the code base such as:
https://github.com/microsoft/opentelemetry-exporters-python/blob/8aac369664eafa82e8ed77d7d69067ef8cebf9a3/azure_monitor/src/azure_monitor/trace.py#L125
Could there be a string detailing what needs to be done after the TODO or can issues be made so that these can be taken care of?

Yes this is a good suggestion. @reyang can you help here?

@lzchen, most likely not due to other priorities.

Fixed with [#23]


Handle write of v9 paks

Added simple rewrite test for v8b.
Added simple rewrite test for v9.
Write frozen_index for v9 compatibility.

Working on v11 write support.


not able to set up the code

Hi,
I'm trying to set up vscode locally but getting the following error.
npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\Program Files (x86)\nodejs\node.exe" "C:\Program Files (x8
6)\nodejs\node_modules\npm\bin\npm-cli.js" "--prefix" "extensions/vscode-ap
i-tests/" "install" "extensions/vscode-api-tests/"
npm ERR! node v4.2.2
npm ERR! npm  v2.14.7
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>postinstall: node ./node_modules/vscode/bin/ins tall
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script 'node ./node_mo
dules/vscode/bin/install'.
npm ERR! This is most likely a problem with the vscode-api-tests package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node ./node_modules/vscode/bin/install
npm ERR! You can get their info via:
npm ERR!     npm owner ls vscode-api-tests
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     E:\vs-code\vscode-master\npm-debug.log
npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\Program Files (x86)\nodejs\node.exe" "C:\Program Files (x8
6)\nodejs\node_modules\npm\bin\npm-cli.js" "install"
npm ERR! node v4.2.2
npm ERR! npm  v2.14.7
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>postinstall: npm --prefix extensions/vscode-api-tests/ in stall extensions/vscode-api-tests/ && npm --prefix extensions/json/ install exte nsions/json/ && npm --prefix extensions/typescript/ install extensions/typescrip t/
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script 'npm --prefix extensions/
vscode-api-tests/ install extensions/vscode-api-tests/ && npm --prefix extension
s/json/ install extensions/json/ && npm --prefix extensions/typescript/ install
extensions/typescript/'.
npm ERR! This is most likely a problem with the Code package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     npm --prefix extensions/vscode-api-tests/ install extensions/vscode
-api-tests/ && npm --prefix extensions/json/ install extensions/json/ && npm --p
refix extensions/typescript/ install extensions/typescript/
npm ERR! You can get their info via:
npm ERR!     npm owner ls Code
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     E:\vs-code\vscode-master\npm-debug.log
Python 2.7 environment path as well as visual studio 2013 path has been set.
When I execute ./scripts/npm.bat install command it installs some of the components but while testing it is throwing the above error.
Please help me to resolve this issue.
Thanks,

@bpasero any idea?

@Sujay-shetty can you attach the npm debug log?

Hi @bpasero,
Please find the npm logs below.
304 info<EMAIL_ADDRESS>Failed to exec postinstall script
305 verbose stack Error<EMAIL_ADDRESS>postinstall: npm --prefix extensions/vscode-api-tests/ install extensions/vscode-api-tests/ && npm --prefix extensions/json/ install extensions/json/ && npm --prefix extensions/typescript/ install extensions/typescript/
305 verbose stack Exit status 1
305 verbose stack     at EventEmitter. (C:\Program Files (x86)\nodejs\node_modules\npm\lib\utils\lifecycle.js:214:16)
305 verbose stack     at emitTwo (events.js:87:13)
305 verbose stack     at EventEmitter.emit (events.js:172:7)
305 verbose stack     at ChildProcess. (C:\Program Files (x86)\nodejs\node_modules\npm\lib\utils\spawn.js:24:14)
305 verbose stack     at emitTwo (events.js:87:13)
305 verbose stack     at ChildProcess.emit (events.js:172:7)
305 verbose stack     at maybeClose (internal/child_process.js:818:16)
305 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:211:5)
306 verbose pkgid<EMAIL_ADDRESS>307 verbose cwd E:\vs-code\vscode-master
308 error Windows_NT 6.1.7601
309 error argv "C:\Program Files (x86)\nodejs\node.exe" "C:\Program Files (x86)\nodejs\node_modules\npm\bin\npm-cli.js" "install"
310 error node v4.2.2
311 error npm  v2.14.7
312 error code ELIFECYCLE
313 error<EMAIL_ADDRESS>postinstall: npm --prefix extensions/vscode-api-tests/ install extensions/vscode-api-tests/ && npm --prefix extensions/json/ install extensions/json/ && npm --prefix extensions/typescript/ install extensions/typescript/
313 error Exit status 1
314 error Failed at the<EMAIL_ADDRESS>postinstall script 'npm --prefix extensions/vscode-api-tests/ install extensions/vscode-api-tests/ && npm --prefix extensions/json/ install extensions/json/ && npm --prefix extensions/typescript/ install extensions/typescript/'.
314 error This is most likely a problem with the Code package,
314 error not with npm itself.
314 error Tell the author that this fails on your system:
314 error     npm --prefix extensions/vscode-api-tests/ install extensions/vscode-api-tests/ && npm --prefix extensions/json/ install extensions/json/ && npm --prefix extensions/typescript/ install extensions/typescript/
314 error You can get their info via:
314 error     npm owner ls Code
314 error There is likely additional logging output above.
315 verbose exit [ 1, true ]

@joaomoreno I can reproduce on Win 10 with npm 3, but not with npm 2. I think npm 3 might be more picky about failing postinstall tasks.
It fails here for me:
npm ERR<EMAIL_ADDRESS>compile: tsc -p .
npm ERR! Exit status 2

We delivered a fix to our build scripts. The NPM postinstall hooks no longer trigger the compilation of the JSON extension. This is now done in our regular build script (gulp watch).
I tested it with npm 3.


Phase 1 EKYC UX Fixes: Add an obvious option to close out of the final screen and start a new eKYC flow. This option should take the user to the home screen.

The current EKYC hub app (see above) has a "start new" button, but I think we should remove it from this screen and just allow people to log in and out from the main page.  The ekyc that pops up after selecting the ekyc option should have a "start new" option at the top.  The style of the top of this modal should just be flat white with a grey line separating the navigation from the page body instead of the dark blue header that shows up.  We should still allow the user to exit from this page as well.

For the scope of this ticket, only address the short term fix, but I have included long term fix images below as well below for reference.

@michelleinez Short-term fix has been implemented. Should we close this and create a separate ticket for the long-term fix?


setup.py

after: sudo python setup.py install
/usr/local/lib/python3.7/dist-packages/setuptools_scm/integration.py:38: RuntimeWarning:
ERROR: setuptools==40.8.0 is used in combination with setuptools_scm>=6.x
Your build configuration is incomplete and previously worked by accident!
This happens as setuptools is unable to replace itself when a activated build dependency
requires a more recent setuptools version
(it does not respect "setuptools>X" in setup_requires).
setuptools>=31 is required for setup.cfg metadata support
setuptools>=42 is required for pyproject.toml configuration support
Suggested workarounds if applicable:

preinstalling build dependencies like setuptools_scm before running setup.py
installing setuptools_scm using the system package manager to ensure consistency
migrating from the deprecated setup_requires mechanism to pep517/518
and using a pyproject.toml to declare build dependencies
which are reliably pre-installed before running the build tools

"""
Traceback (most recent call last):
File "setup.py", line 61, in
py_modules=["adafruit_ltr390"],
File "/usr/lib/python3/dist-packages/setuptools/init.py", line 145, in setup
return distutils.core.setup(**attrs)
File "/usr/lib/python3.7/distutils/core.py", line 108, in setup
_setup_distribution = dist = klass(attrs)
File "/usr/lib/python3/dist-packages/setuptools/dist.py", line 447, in init
k: v for k, v in attrs.items()
File "/usr/lib/python3.7/distutils/dist.py", line 292, in init
self.finalize_options()
File "/usr/lib/python3/dist-packages/setuptools/dist.py", line 739, in finalize_options
ep.load()(self, ep.name, value)
File "/usr/local/lib/python3.7/dist-packages/setuptools_scm/integration.py", line 65, in version_keyword
dist.metadata.version = _get_version(config)
File "/usr/local/lib/python3.7/dist-packages/setuptools_scm/init.py", line 177, in _get_version
parsed_version = _do_parse(config)
File "/usr/local/lib/python3.7/dist-packages/setuptools_scm/init.py", line 143, in do_parse
"use git+https://github.com/user/proj.git#egg=proj" % config.absolute_root
LookupError: setuptools-scm was unable to detect version for '/home/pi/LTR390'.
Make sure you're either building from a fully intact git repository or PyPI tarballs. Most other sources (such as GitHub's tarballs, a git checkout without the .git folder) don't contain the necessary metadata and will not work.
For example, if you're using pip, instead of https://github.com/user/proj/archive/master.zip use git+https://github.com/user/proj.git#egg=proj
do i have to edit the setup.py file ? What iam missing here?

ah okay fixed it with: python -m pip install pyproject.toml
that was new for me!


Error while using ensemble mode - "cannot import name 'nets_537227KB' from 'lib_v5'"

Describe the bug
I am trying to use the ensemble HP2 model with Save all outputs enabled. When start conversion, the first two models run successfully and I get the ouputs files. However get the error in the title when trying to process the 3rd model.
&&&&&&&&&&&&&&&&&&&&&&&&  START copy paste &&&&&&&&&&&&&&&&&
C:\XXXXX\ultimatevocalremovergui-master\VocalRemover.py:92: DeprecationWarning: ANTIALIAS is deprecated and will be removed in Pillow 10 (2023-07-01). Use Resampling.LANCZOS instead.
img = img.resize((size[0], int(size[0] * ratio)), Image.ANTIALIAS)
Model Parameters: lib_v5/modelparams/4band_44100.json
C:\XXXXXX\Python\lib\site-packages\librosa\core\audio.py:162: UserWarning: PySoundFile failed. Trying audioread instead.
warnings.warn("PySoundFile failed. Trying audioread instead.")
100%|██████████████████████████████████████████████████████████████████████████████████| 95/95 [12:58<00:00,  8.20s/it]
Model Parameters: lib_v5/modelparams/4band_44100.json
100%|██████████████████████████████████████████████████████████████████████████████████| 95/95 [13:10<00:00,  8.32s/it]
File "C:\XXXXXX\ultimatevocalremovergui-master\inference_v5_ensemble.py", line 280, in main
from lib_v5 import nets_537227KB as nets
ImportError cannot import name 'nets_537227KB' from 'lib_v5' (unknown location)
Traceback Error: "  File "C:\XXXXXX\ultimatevocalremovergui-master\inference_v5_ensemble.py", line 280, in main
from lib_v5 import nets_537227KB as nets
"
ImportError: "cannot import name 'nets_537227KB' from 'lib_v5' (unknown location)"
File: C:/XXXXXX/YTDL/YTDownloads/Vethala-Potta-MassTamilan.com.mp3
Please contact the creator and attach a screenshot of this error with the file and settings that caused it!
&&&&&&&&&&&&&&&&&&&&&&&& END copy paste &&&&&&&&&&&&&&&&&
To Reproduce
Steps to reproduce the behavior:

Selected settings: '...'
Conversion Method : Ensemble
Windows Size : 512
Choose Ensemble : HP 2 models
Save all outputs :enabled
See error
"cannot import name 'nets_537227KB' from 'lib_v5'"

Expected behavior
I expect the ensemble model to complete the splitting operations
Screenshots
If applicable, add screenshots to help explain your problem and annotate the images.
https://imgur.com/a/uEV4Jdw
Desktop (please complete the following information):

OS: Win 11

Additional context
Add any other context about the problem here.

Update: I was successfully able to run the ensemble option with HP1 models

same issue here

This issue has been addressed. Please update to version 5.2.0 using the Windows installer provided on the main page.


Update content to fix refer links

Fix broken links.

This change is

Please hold off on this PR until @pwittrock chimes in. My guess it that we intend to include both api-reference and resources-reference in the docs. For now, we have resources-reference redirected to api-reference.

@kairen Thanks for your PR. We are planning to add the resources-reference section to the docs soon. So these links don't need to be changed. For now, they are being redirected. Also, I'm working on updating the old user-guide links across the entire doc set.