Drawing Points/Vertices?

I hunted around to see what might be possible here, but couldn't find a good answer up to now. The etoViewport/Eto.OpenTK code allows for points to be overlaid on the underlying polygons (using a separate draw). Is there a way to reproduce this in Veldrid?

https://github.com/ItEndsWithTens/Eto.Veldrid/pull/7
Can't figure out how to make large points

From what I'm reading looking into this, it's not a feature supported reliably by anything except OpenGL. Vulkan supports it if the hardware has certain features, D3D11 doesn't do it at all, and I'm not sure about Metal at the moment.
Veldrid therefore has no built in support for doing it, but there are ways around the limitation, I think.

If one is only going to use OpenGL, or Vulkan with the appropriate hardware features, you can do it through the shaders. In src/shaders/VertexColor-vertex.450.glsl, just below the gl_Position assignment, add e.g.

gl_PointSize = 4.0f;

This only draws squares, but you can adjust the size that way for APIs that support it.

Write a geometry shader to emit polygons for each point you want to render. I haven't even begun studying such shaders, however, so I can't offer advice on them just yet. I do know driver support for geometry shaders is required, and not all hardware offers that, namely many mobile devices. If your target is desktop, though, I suspect you could rely on it being available.

Make your own "point" by creating a square or circle polygon, or cube, sphere, or really whatever you'd like, and use geometry instancing to place copies of it at the appropriate vertex positions. Instancing is another feature I've never worked with. I imagine trying to keep instances the right size in screen space would be its own challenge, since instances would be placed at the actual vertex locations, which means 3D points further from the camera would appear smaller.

Despite my lack of experience, I think those three are arranged in order of increasing cross-backend support, device compatibility, and difficulty. But take that with a grain of salt! I could easily be wrong.


Protocol Specification: IGM Specification V1.3

Description
Prepare the first draft of the specification for the IGM (Infrastructure Governance Model) project. This includes outlining the scope, objectives, key features, and technical requirements. The specification should serve as a foundational document for further discussions and refinements.
Goals

[ ] Define the scope and objectives of the IGM project.
[ ] Outline key features and functionalities to be included in the specification.
[ ] Draft the technical requirements and any dependencies.

Expected Outcome
A well-documented first draft of the IGM specification that outlines the scope, objectives, key features, and technical requirements, ready for review and refinement.
Acceptance Criteria

[ ] The specification draft includes all relevant sections (scope, objectives, features, technical requirements).
[ ] The draft is clear, structured, and ready for review by stakeholders.
[ ] Feedback from initial reviews is incorporated into the document for further iterations.

Mockups / Wireframes
NA
Product Name
IGM
Domain
Infrastructure Governance
Tech Skills Needed

Technical Documentation
Requirement Analysis
Specification Drafting

Complexity
Medium
Category
Documentation
Sub Category
Specification Drafting

Created the first draft of the IGM spec: https://github.com/rajaneeshk90/protocol-specifications-rk/tree/IGM
Still need to do a lot of fine tuning. Removed various compilation errors that I was facing. A compilable version is handy now.

We will update the details in the #287 and will get it reviewed with Beckn one core working group. Post that we will update the core protocol specifications.
This ticket will be picked up once the review is completed.
@faizmagic @rajaneeshk90 @viraj89


Write further unit tests for DefaultAbly

This builds on the tests added for DefaultAbly in #878, as part of #869. It does the following:

adds further tests for connect
changes some of the existing tests for connect to reflect specified client library behaviour
adds tests for updatePresenceData, disconnect, startConnection, stopConnection, and sendRawLocation

I opened the following issues as a result of working on these tests:

#908
#909
#912

Just pushed an update which fixes some mistakes in the Given / When / Then comments for the disconnect and updatePresenceData test cases.


[Eclipse] Copyright Header

We need to include a copyright header in every source file, see
https://www.eclipse.org/projects/handbook/#ip-copyright-headers
/********************************************************************************
 * Copyright (c) {year} {owner}[ and others]
 *
 * This program and the accompanying materials are made available under the
 * terms of the Apache License, Version 2.0 which is available at
 * https://www.apache.org/licenses/LICENSE-2.0
 *
 * SPDX-License-Identifier: Apache-2.0
 *
 * Contributors: 4
 *   {name} - {description}
 ********************************************************************************/

 Dominik Pinsel<EMAIL_ADDRESS>Daimler TSS GmbH, legal info/Impressum

I can do this in a one-shot commit with Intellij.

Is the initial copyright holder Microsoft Corporation?
Should the Contributors section be initially left blank or should it be:
Contributors: 1
Microsoft Corporation - initial commit

that would result in the following header:
/*
 * Copyright (c) 2021 Microsoft Corporation
 *
 *  This program and the accompanying materials are made available under the
 *  terms of the Apache License, Version 2.0 which is available at
 *  https://www.apache.org/licenses/LICENSE-2.0
 *
 *  SPDX-License-Identifier: Apache-2.0
 *
 *  Contributors: 1
 *       Microsoft Corporation - initial commit
 *
 */

and don't miss to adjust the LICENSE file :-)

Some small modifications (year and contributor description):
/*

Copyright (c) 2020, 2021 Microsoft Corporation [and others]

This program and the accompanying materials are made available under the
terms of the Apache License, Version 2.0 which is available at
https://www.apache.org/licenses/LICENSE-2.0

SPDX-License-Identifier: Apache-2.0

Contributors:

  Microsoft Corporation - initial API and implementation

*/

Also, [and others] is not added at first but as soon as it is modified, it is appended

solved by #29


feat: add dummy yanga build stage

Codecov Report
Attention: 15 lines in your changes are missing coverage. Please review.

Comparison is base (6c4e080) 68.43% compared to head (abe3e17) 69.34%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main       #8      +/-   ##
==========================================
+ Coverage   68.43%   69.34%   +0.91%
==========================================
  Files          22       21       -1
  Lines         735      734       -1
  Branches      114      114
==========================================
+ Hits          503      509       +6
+ Misses        222      215       -7
  Partials       10       10

Files
Coverage Δ

src/yanga/ybuild/stages.py
55.55% <46.15%> (-5.32%)
:arrow_down:

src/yanga/commands/build.py
0.00% <0.00%> (ø)

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


[Feature Request] Boolean

First and foremost I <3 excalidraw and it's name. FOSS for the win!
The single thing I am missing is the option to combine forms, if it is not all the boolean operators it would be really nice to be able to combine forms in a new one and have the option to change the combined outline or something.
Love!

Thanks for the great feedback!
Could you please elaborate a bit about  "option to combine forms"? It's completely clear what you mean by that.

Dear Marcel,Please refer to:https://help.figma.com/hc/en-us/articles/360039957534-Boolean-operationsKind regardsGerrit Sent from out‘n‘about. On May 7, 2024, at 23:42, Marcel Mraz @.***> wrote:
Thanks for the great feedback!
Could you please elaborate a bit about  "option to combine forms"? It's completely clear what you mean by that.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

I see what you mean now. Unsure if we would like to pollute the editor with these operations, though it's an interesting suggestion.
Another possible option is implementing a "fill tool", which should allow you to perform such operations (and many more) on your own.
I.e. in the case of an intersection, it would mean filling a transparent colour inside the intersection.

Yeah, or the old-school MS paint fill tool. It's sort of weird with transparent color as it's not a traditional color, but one we could say here filling the intersection with white color, considering the background is white as well, the result would be the same.


Bug: 'Insert' statement turns into 'Select' statement in some scenarios

var sql = require('sql')
var model = sql.define({
  name: 'test',
  columns: ['c1', 'c2']
})

console.log(model.insert([]).toQuery())
/*
This should throw an error as I am trying to insert nothing.
Clearly a mistake on the programmer's side

Instead, it fails silently:
-> { text: 'SELECT * FROM "test"', values: [] }

This means that the programmer won't be alerted to the mistake.
Because the query will not throw an error.
*/

This scenario should throw an error and alert the programmer. In any case, it should not turn into a select

:+1: This is still an issue.

This is not a bug, but a feature https://github.com/brianc/node-sql/pull/203  :)

Harmless? I disagree. What if I have billions of rows in that table? SELECT * is not harmless.

This is extremely difficult to debug. Can you explain why isn't the correct behavior to throw an error?

https://github.com/brianc/node-sql/pull/197 explains it.
Well, not really - its just backward-compatible behavior. The correct backward-incompatible behavior would be to not generate a query.


Components returned from "styled" have no backing instance, preventing refs from being set

For example,
class Parent extends Component {
  render() {
    return (

        // ChildComponent is a styletron component.
        // this.childComponent is undefined
        <ChildComponent ref={(c) => { this.childComponent =  c; }}

    );
  }
}

Yeah, styled is currently returning a stateless functional component which has no backing instance. It needs to be changed so it returns a full React component.

It appears that styletron is still returning stateless functional components from the styled method...
I am unable to set ref={} on the returned components.

lostpebble , exactly that is why I stopped using it and switched to styled-components's styled() instead.
Styletron breaks my connect() and custom decorators.

@lostpebble
Styled components are inherently stateless and can't have any methods. However, you may want to set refs on the underlying elements to get access to the DOM elements to call DOM methods, etc. You can do this with the innerRef prop.
For example: https://codesandbox.io/s/lOERlQW0V
import React, { Component } from "react";
import { render } from "react-dom";
import { styled, StyletronProvider } from "styletron-react";
import Styletron from "styletron-client";

const Foo = styled("input", {
  color: "blue",
  padding: "12px",
  borderStyle: "solid",
  borderWith: "1px",
  borderColor: "blue",
  ":focus": {
    color: "red",
    borderColor: "red"
  }
});

class Bar extends Component {
  constructor() {
    super();
    this.handleClick = this.handleClick.bind(this);
  }

  handleClick(e) {
    this.input.focus();
  }

  render() {
    return (

          <Foo
            placeholder="i am an input"
            innerRef={a => {
              this.input = a;
            }}
          />

        <button onClick={this.handleClick}>focus the input</button>

    );
  }
}

const App = () =>
  ;

render(<App />, document.getElementById("root"));

@rtsao Thanks, that does the trick! Just couldn't find reference to it anywhere in the docs.
I guess it is a standard syntax for wrapper components? But still would be good to have an indication somewhere as working with ref is quite a common occurrence.

When using "innerRef" a proxyComponent is returned, not the actual DOM element. Advice please.


Teams Usage Report

Hello guys,
I'm trying to get a report on equipment usage, but the data I download doesn't have much to do with what can be seen on the Web.
I would like to be able to download the same data that appears in the Teams admin center chart.
Regards!
Diego Campos

@diecamtab thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


Error, while creating a new workout in debug-mode

I want to contribute to this project and followed the "how to get started"-instructions described in the readme-file. Im still getting this error in heroes.dart, when i. e. creating a new workout in debug-mode. When trying the same thing without debug-mode than it works perfectly.

Does anyone know how to deal with that or had the same problem?

Yeah it's because the floating action button used in the cards needs a unique heroKey, but right now this isn't supplied. A solution would be to give them all unique keys, but personally I would just make the floating action button custom, because it's a really easy widget to make and would give us more customisation in the future.

Okay, so then could I work on this feature? I make a custom floating action button.
The custom floating action button should not have the heroTag-property, shouldn't it? (but all the other features + extra features for the future, when needed)

I don't have the right to assign you to this but I'd say go for it. Yeah the button shouldn't have a herotag seeing we won't be using it. Things to note tho when making a custom FAB. Don't forget the inkwell effect a FAB has and don't forget the shadow. The rest should just be straightforward a round container with an icon in the middle and a color.

@MhouneyLH If you would like to add extra features, I would change the inkwell animation to something else on IOS, because generally IOS doesn't have the inkwell effect.

we can set the heroTag for them to be null for now and then work on costumising them in the future. so that the issue be gone for now, and work on finding other issues. i guess

@0RaMsY0  Don't think that is gonna work because they are all null right now. I think it uses a default key then, but seeing as they all use this default key the error occurs. Seeing as this isn't a breaking issue but just a error you get in the console, I believe we can take the time to fix it properly.

i have faced the same issue a couple of minutes ago i changed the heroTag for the floatingActionBar that save the workout into null, that means as you said they use the same heroTag so if when i changed one to null it worked


spiffs build artifacts

i am using spiffs and it seems that these files are created at build time
    extras/spiffs/mkspiffs/mkspiffs
    extras/spiffs/mkspiffs/mkspiffs.o
    extras/spiffs/mkspiffs/spiffs_cache.o
    extras/spiffs/mkspiffs/spiffs_check.o
    extras/spiffs/mkspiffs/spiffs_gc.o
    extras/spiffs/mkspiffs/spiffs_hydrogen.o
    extras/spiffs/mkspiffs/spiffs_nucleus.o

i wonder if this is expected or not. in any case, the .o should be excluded from .gitignore.
having said that this won't solve the issue caused by the 1st file in this list. any guidance on how to handle this properly ?

Hi @phlemoine,
Yes. mkspiffs is currently built in the source directory. I was thinking about placing object files in the application build directory but didn't implement that at the time.
I will try to move mkspiffs objects and executable into the application's build directory.


Methods to get the data for a given torque point from the server

This introduces a function on the torqueLayer of the form
getDataForPoint:function(x,y,callback, maxNo, tolerance)
where

x and y are the coordinates of the click point on the layer
callback is of the form function(rows)
maxNo is the maximum number of data points to return from the query
tolerance is a method to add a buffer of x pixels to the click. This is because its tricky to click precisely on a point in torque at low zoom levels

@javisantana I think I understand the quadtree logic but it might be worth reviewing my query at https://github.com/CartoDB/torque/blob/bi_provider_get_point_data/lib/torque/provider/filterableJson.js#L439 to make sure I am doing this right.

I think the logic is right, it can be improved, a pixel in a tile at zoom X is a tile at X + 8 so if you have:
tx, ty, tz <- tile coords
x, y <- pixel coordinates inside the tile [0, 255]
the tile at zoom level + 8 will be -> tx + x , ty + y, tz + 8.
Anyways, I think is good enough, at zoom levels where the user can client there will be no so many points


Overhaul Logging

GUIMain.log(String) needs to be overhauled to give a better idea of what is going wrong, in what void, in what class, etc.
I think the Logger class has something like this, will investigate.

The most simple thing we can do is just take all the "GUIMain.log(e.getMessage())" and expand them to say what exception is being thrown where.


removeByTag isn't executed if no rule id is present in the rule

There's a check for (rule->actionset->id != NULL) that doesn't check for target exceptions in case a rule has no id. This must be a copy-paste from the code checking exceptions by id.
This PR removes that check.
A little enhancement could be to remove the "id=NULL" from the debug log, but this wasn't done here.

Sorry, this was for v2


Restore causes crash on HTC One M7, 5.0.2, Sense 6

The app is linked to my google account. I recently change rom to 5.0.2 on my HTC and now the app won't restore any texts. It says logging in, then calculating, then crashes and says "Unfortunately SMS Backup+ has stopped"
The sms_backup_plus.log shows nothing about restoring
Below is an adb logcat export. It starts to fail at line 531.
https://gist.github.com/samuelrhaynes/513463d912598c516d71#file-smsbackup-crash
Hope you can help

I have the same problem but with backing up whatsapp texts. I know they aren't natively supported now the app backs up call logs and SMS just fine but crashes soon after.

the crash is inside the framework code, nothing i can do about it. might be a HTC issue.


fix(ui): upload with has many bulk select not working with postgres

Fixes https://github.com/payloadcms/payload/issues/7921

After that fix now i have the following error:
This relationship field has the following invalid relationships: 766 0, 767 1
when trying to choose images from existing


Support case-insensitive log searching

UCP #203
What?
Support Case Insensitive Log Searching
How
By modifying regexp patterns with an (?i) ignore-case flag before sending SearchLogRequest to gRPC service.
Compatibility

[x] Golang regexp package, which uses re2 underneath the hood, supports inline i flag
[x] Rust regex crate also supports i flag

Screenshot

@Deardrops
Also upload a screenshot. PTAL.

Could you also provide a screen shot for the TiKV?

@breeswish Here you are.
TiDB

TiKV

@all-contributors please add @weihanglo   for code


Update MgramsevaAdapterDemandConsumer.java

since update event has all the demanddetails, we are removing the demanddetails that were not updated from the list and passing only the updated ones as part of the event to ifix


Genitalia Size Genes behave wrong for growing up Children

Reported in #8, it seems that Genes changing genitalia size are not correctly applied to children.
Aimed solution:
Check on entering adult-life-stage, re-trigger genitalia size genes if necessary.

Issue seems to come from whenever a pawn changes size the updatesize function is triggered and resizes the genitalia/breasts/anus.

https://gitgud.io/Ed86/rjw/-/blob/master/1.4/Source/Hediffs/Hediff_PartBaseNatural.cs
https://gitgud.io/Ed86/rjw/-/blob/master/1.4/Source/Comps/CompRJWHediffBodyPart.cs
We could make sizechanging genes work through a harmonypatch and just make a prefix or transpiler. We could make the genes work like a multiplier, like larger size breast means breast are two times as large than they would have been normally.


Incomplete UTF-8 support in Android App (Devanagari Script)

Internationalisation with resource bundle works fine for Spanish, but languages like Hindi/Marathi does not work.
Non native-image (JVM):

GluonFX Android build:

@jperedadnr
Actually if you look at the title bar of your screenshot, the desired titlebar is achieved ( Stream-Pi क्लाइंट)
But the main content of the javafx window is what is broken. So this looks more like javafx unable to select the appropriate font rather than bad encoding ...

The title bar of the window is actually managed by the system, so that explains why it works.
Recently I fixed probably this very same issue with this PR: https://github.com/openjdk/jfx/pull/399
Not sure if you can verify that those two languages are supported properly by Java/JavaFX on macOS?

The title bar of the window is actually managed by the system, so that explains why it works.
Recently I fixed probably this very same issue with this PR: openjdk/jfx#399
Not sure if you can verify that those two languages are supported properly by Java/JavaFX on macOS?

The thing is that it works perfectly fine with normal JVM + linux/windows. and is just looking like something specific to MacOS. The myanmar script and hindi (devanagari) script is not the same.
I dont have a MacOS system with me. I will ask some more people in the streampi community with macos to test it out

I've run a simple test case with strings from your properties files:
    @Override
    public void start(final Stage primaryStage) {
        final Label btn = new Label("सर्वर होस्टनाम/आईपी");
//        final Label btn = new Label("सर्वर पोर्ट का मूल्य पूर्णांक होना चाहिए।");
        StackPane root = new StackPane(btn);
        primaryStage.setScene(new Scene(root, 300, 250));
        primaryStage.show();
    }

    public static void main(String[] args) {
        System.setProperty("prism.debugfonts", "true");
        launch(args);
    }

And that works fine for me on macOS.
The output log is interesting:
[MacFontFinder] Name=System Font Bold
        Family=.AppleSystemUIFont
        File=/System/Library/Fonts/SFNS.ttf
        Fallback font= Kohinoor Devanagari Regular slot=6
 ...

So I haven't set any font for that label, so the System one is applied, and the font finder finds the correct font.
Back to Stream-Pi client, I see that you use  Roboto font. So I've removed that, and now the System font gets applied and the i18n panel works perfectly fine.

I had actually removed "Roboto" as the default font completely from Stream-Pi. However, I forgot to remove the CSS part where -fx-font-family: Roboto was still written.
This is probably why the Server worked just fine, because I removed all roboto entirely from them,
I will try testing it again on multiple platforms with Stream-Pi using system font, and report back asap


version 1.3.0 does not return the right dynamic type

In version 1.3.0
IQueryable queryResult = result.AutoQueryable(queryString) returns IQuerable of objects that are of type objectRealTypeName + long seemingly random string.
DataObjects.DomainObjects.v2.ExpenseOutcomeBrowseDO73390d5deeee3603d91efe6bc2b21610c9d73f8f556755dcc7b16ee102d25adc
Casting these objects to their needed type is not possible as the type is incorrect.
Is this a bug or has something changed? Reverted back to 1.1.0 for the moment.
I am using AutoQuerable  on .Net Core 2.0 project.

Hi jnUser,
Do you have a repo with an example to reproduce ?
Do you have the same issue with 1.1.0 & EF core 2?
Thanks.

@jnUser
I did some tests, and I got the random string too. But that's normal.
Version 1.1.0 worked in the same way except in one case: when the query string is empty.
AQ generates a new type dynamically so as not to keep unnecessary properties in the json (or other) generated.
I planned in the roadmap to leave the choice between auto generated type and base type.
I'll ping you when this feature becomes available

#30

@jnUser
AQ 1.4.0 is out, you can now specify you want to use base type in attribute or profile:
var res = result.AutoQueryable(queryString, new AutoQueryableProfile { UseBaseType = true })

Fixed by feat #30


feature/GEODE-11-Defining lucene index

Added a definedIndexMap field in LuceneServiceImpl to store the uninitialized lucene indexes
Added a field "status" to the list and describe lucene gfsh commands to differentiate between defined and initialized indexes.
Added a dunit test for the create index gfsh command to verify if the indexes created through gfsh are stored in definedIndexMap.

This change is mixing internal classes in the public API. You're adding getDefinedIndexes to the public API, but it returns LuceneIndexCreationProfile objects which is an internal class. Looking at LuceneIndexCreationProfile, I think these methods probably don't belong on LuceneService. Maybe put these methods on InternalLuceneService?
I think setFieldAnalyzerStrings is not a good name, because it's not actually setting anything on the object. Same with setFieldAnalyzerStringsFromProfile.
The boolean "status" field in in LuceneIndexDetails is confusing - what does true or false mean? Maybe it should be named initialized?

+1


Return nothing when there is no lsp-progress

Fix: a literal '0' appears after filename on older versions of vim-lsp.
0c8164b1b3c8017770296320a67a164b3c2aad39 added support for showing lsp
progress, but didn't always explicitly return a value. Vim uses 0 as a
return value when none are supplied, so when used with an older version
of vim-lsp that doesn't have the progress feature a 0 appeared in the
statusline.

Demonstration of how the bug manifested:

The 0 after [No Name] is the bug.

Demonstration of how the bug manifested:

The 0 after [No Name] is the bug.

thanks!

thanks!


xorm reverse

I am trying to use xorm reverse but it does not connect my mysql database. Does it not support ip addresses ? I can ping the address without problems.
I tried:
xorm reverse mysql user:password@tcp(<IP_ADDRESS>:3306)/database templates/goxorm
But it gives an error:
-bash: syntax error near unexpected token ('`
I also tried:
xorm reverse mysql<EMAIL_ADDRESS>templates/goxorm
But that gives another error:
2017/03/26 13:37:11 [Error] reverse.go:176 Default addr for network '<IP_ADDRESS>' unknown

maybe you can try
xorm reverse mysql "user:password@tcp(<IP_ADDRESS>:3306)/database" $GOPATH/src/github.com/go-xorm/cmd/xorm/templates/goxorm

Great ! The "string" did the trick. Thank you !


Wiki for "Raspberry Pi 'Dog Bone' Stackable Case" has broken link

There are two links: the eBay link is dead, and "case" links to an active Amazon listing

  Raspberry Pi 'Dog Bone' Stackable
   Case

Though it's not super clear, the code/content for the site is over in https://github.com/geerlingguy/drupal-for-kubernetes/issues/36 — so closing this issue and moving to there. Thanks for spotting this and filing the issue!


bug: Calling refreshSession on local dev environment breaks session

Bug report
Calling supabase.auth.refreshSession on local dev env breaks session.
I've added a runnable minimal example here. It runs on node but the exact same happens on the browser.
Describe the bug
After signing in with supabase.auth.signIn the supabase.auth.token obj is set in the localStorage as expected.
My use-case manipulates the user_metadata in the backend and I need to refresh it in the front-end after the calls succeed.
Calling getUser(session.access_token) returns the same obj as present in the supabase.auth.token.currentSession.user, without the updates.
The only way I found for getting the updated user obj is to call supabase.auth.refreshSession. This works on the deployed version of the app. However, on local (supabase start), calling refreshSession issues TOKEN_REFRESHED event containing the session param as an empty obj {}. The strange thing is that the supabase.auth.token remains in the localStorage untouched.
To Reproduce
Steps to reproduce the behavior, please provide code snippets or a repository:

Start a development supabase server with supabase start
Sign in with supabase.auth.signIn
Verify that supabase.auth.token obj is set in the localStorage
Call supabase.auth.refreshSession
Verify that the session is broken by watching it with supabase.auth.onAuthChange
Verify that subsequent calls to supabase.auth.session returns {}

Expected behavior
Should update the session/user with the most up-to-date server data?
System information

OS: Ubuntu Linux 5.11.0-46-generic 51~20.04.1-Ubuntu
Version of supabase-js: 1.30.6
Version of Node.js: v16.13.2

Additional context

runnable minimal example

Hey @naripok thanks for reporting this, a couple of questions:

For me, calling await supabase.auth.api.getUser(supabase.auth.session().access_token); works as expected to get the up to date user object, including the up to date user_medata, you can check out my example below. Are you saying this doesn't work for you?

For me, calling await supabase.auth.refreshSession() does correctly update the session in localStorage:
 and also the TOKEN_REFRESHED event payload and newSession payload is correctly updated:

Why did you choose to reproduce this in a Node environment? Here localStorage won't work.

So I'm not able to reproduce this in the browser environment. Can you provide some recheck with the example I posted below and reopen with a sreenrecording? Thanks.
<script src="https://cdn.jsdelivr.net/npm/@supabase/supabase-js"></script>

<script>
  function initSupabase() {
    const supabaseAdminClient = window.supabase.createClient(
      "http://localhost:54321",
      "eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJzdXBhYmFzZS1kZW1vIiwicm9sZSI6InNlcnZpY2Vfcm9sZSJ9.vI9obAHOGyVVKa3pD--kJlyxp-Z2zV9UUMAhKpNLAcU"
    );
    window.supabase = supabaseAdminClient;
  }

  async function supaTest() {
    initSupabase();

    supabase.auth.onAuthStateChange((event, session) => {
      console.log(event, session);
    });
    const { user: signUpUser, error: signUpError } = await supabase.auth.signUp(
      {
        email<EMAIL_ADDRESS>        password: "password",
      }
    );
    console.log("signUpUser", signUpUser);
    console.error("signUpError", signUpError);

    const { error: confirmEmailError, user: confirmEmailUser } =
      await supabase.auth.api.updateUserById(signUpUser.id, {
        email_confirm: true,
      });
    console.log("confirmEmailUser", confirmEmailUser);
    console.error("confirmEmailError", confirmEmailError);

    const { user, error: signInError } = await supabase.auth.signIn({
      email<EMAIL_ADDRESS>      password: "password",
    });
    console.error("signInError", signInError);
    console.log("signInUser", user);

    const session = supabase.auth.session();
    console.log("session", session);

    // Update user metadata
    await supabase.auth.api.updateUserById(supabase.auth.user().id, {
      app_metadata: { hello: "world" },
      user_metadata: { hello: "world" },
    });

    // Get user from API
    const { error: getUserError, user: getUserUser } =
      await supabase.auth.api.getUser(supabase.auth.session().access_token);
    console.error("getUserError", getUserError);
    console.log("getUserUser", getUserUser);

    const { data: newSession, error: refreshSessionError } =
      await supabase.auth.refreshSession();

    console.error("refreshSessionError", refreshSessionError);
    console.log("newSession", newSession);
  }
</script>

Hey @thorwebdev, thanks for the response.

Can you provide some recheck with the example I posted below and reopen with a sreenrecording?

Sure!

Can you provide a screenshot of the network response for the updateUserById, the getUser, and refreshSession calls please?

Thanks @naripok, so looks like getUser does give you the correct up to date user data, that's good.
refreshSession returning null however is not good. But at least this means it's not supabase-js related. Let's move it to https://github.com/supabase/cli and see if @soedirgo has any ideas, maybe it's a Linux related quirk?
@naripok can you confirm that your supabase cli is up to date?


Error when use "DefaultAzureCredential" in v1 Azure Function app

Hi,
I am getting the below error when use "DefaultAzureCredential" in v1 Azure Function app.
Error:
Could not load file or assembly 'Azure.Core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=92742159e12e44c8' or one of its dependencies. The system cannot find the file specified.

Thank you for your feedback.  Tagging and routing to the team members best able to assist.

Hi @jaganpt Could you provide some more details about how your function app is configured and its assembly and package dependencies?

Hi @christothes ,
Please refer the below details.
Function app run time version : ~1
Assembly and package ependencies:

Hi @christothes ,
It would be great if you could provide feedback on the above issue. Thanks.

related to #21361

Hi @jaganpt - Sorry for the slow response. I'll need to look further into this. To get unblocked, is it possible for you to use a later version of Azure Functions?

Does the solution mentioned in this issue resolve the issue for you?
https://github.com/Azure/azure-sdk-for-net/issues/21361#issuecomment-848895367

Hi @christothes ,
It is an existing v1 azure function app. Hence not possible to use later version for now.

Hi,
I have alredy provided the required information. Please let me know what additional information is needed.
Function App version : v1
Description : Error when use "DefaultAzureCredential" in v1 Azure Function app.
Error:
Could not load file or assembly 'Azure.Core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=92742159e12e44c8' or one of its dependencies. The system cannot find the file specified.
Note: It is working for v3 function app. But I could not upgrade my existing production v1 function app.

Hi @jaganpt - Did you try the suggestion in this comment? https://github.com/Azure/azure-sdk-for-net/issues/21262#issuecomment-848895760


version 1.5.2 Kotlin Conversion and API support for 24+   @oli107

Version 1.5.2 of my Fork is a kotlin conversion and includes bug fixes for this lib  @oli107
I would like to submit the work, I also added some new helpful functions to allow devs to control the RangBar programmatically better. As well as changing boundary colors without having to recreate a a new PinView when its not need if it is not null.

The "active colors" are still busted in 1.5.4.
If you set a new color on anything, then disable and re-enable the RangeBar, the colors will revert to the original color.
The "active colors" are set only in the constructor and are not updated with the respective color setters.


for failure details, remove everything after the first line break.

this will clean up the output on failing tests, currently its a wall of text.

fixed on 1.04


How to force regeneration of lua files

Is there an easy way to force fennel to regenerate the lua files?

I think nfnl has some ways of doing so:
Via nvim command :NfnlCompileAllFiles and via api (compile-all-files dir)


mobile: adjust bazel flags for Bazel 6

--host_java_toolchain and --java_toolchain are no-ops in Bazel 6
--auth_enabled is replaced with --google_default_credentials

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

/assign @keith


Case-insensitive claims

LBRY URLs ought to be case-insensitive, the way traditional domains are. Two claims with names with different case-sensitivity should be considered having the same name.

How would this affect already existing claims?

a) We need to have the claimtrie work with unicode, not bytes if we are attaching strict interpretations of what the bytes means, are non unicode bytes now invalid? Which unicode encoding are we using?
a) We need to define "case" here. English is of course easy, but it is kind of silly to deal with just English "cases" but ignore other languages. Is there a standard we can adopt or work off of here?
c) as aayani said, we need to determine what happens to the already existing claims when this change occurs. (Does claim dog and DOG get merged into the same name?) and implement this logic.
It is also possible to do this via soft fork by rejecting claims with cases in them after a certain block and letting already existing ones expire.

a) I'm not sure of the answer here, but if some claims get invalidated that would acceptable (though obviously not ideal). I don't have a problem with supporting non-Unicode or arbitrary byte claims, so long as when/if the claim is valid Unicode, we are considering case-sensitivity.
b) I believe Unicode already defines lower and upper-case characters, as you can use regular expressions categories like \p{Ll} and \p{Lu} to refer to upper and lower case characters.
c) Yes, these would get merged into the same name under this proposal.
I'd prefer the solution here to be that claims retain the original provided capitalization, rather than preventing capitalization in claims. That is, we want someone to be able to claim lbry://eXistenZ and retain the capitalization, but still have lbry://existenz resolve to it.

So if two people own the same channel/claim name with different capitalizations, the resolve will default to the one with the higher bid right?

Yes, that's the idea.

What is the status of this feature? I'd like to counter that by the argument of address space being tightly coupled to length of string and variance in characters. This added expense to hashing is equivalent to mandating everything be in lower-case and have a doubly long url.

This provides a standarized method of normalization for unicode characters: http://unicode.org/reports/tr15/#Notation , and is currently being proposed as the method for implementation case-insensitve claims along with other normalizations.

Here is a unicdoe vulnerability that was floating around as a phishing attack on Telegram recently:
https://krebsonsecurity.com/2011/09/right-to-left-override-aids-email-attacks/

Addressed by #159


import cycle not allowed

go build -o sample-controller .
import cycle not allowed
package k8s.io/sample-controller
        imports flag
        imports errors
        imports runtime
        imports internal/bytealg
        imports internal/cpu
        imports runtime

I have the same problem.

uninstall go as below common
sudo apt-get remove --auto-remove golang-go
and then use this  to install golang
https://golang.org/doc/install
its work for me

Closing since this currently works on master. You can also find instructions on using with go modules here: https://github.com/kubernetes/kubernetes/pull/79081.
Please reopen/comment if you still face the issue.


pack_int32_to_int4

In 'HAWQ-main/tvm_benchmark/hawq_utils_resnet50.py' ,we pack 8 'int4' number to 1 'int32' number, so we got int4 speedup.
Can we pack 16 'int2' to 1 'int32', to got int2 speedup?

Yes. The purpose of the packing is to handle with memory movement with a datatype that is supported in the target hardware (int8, int32 in cpu/gpu). If you want to further reduce the precision to int2, in cpu/gpu you also need to pack them into a byte-addressable data type (int8, int32) before the memory movement

Yes. The purpose of the packing is to handle with memory movement with a datatype that is supported in the target hardware (int8, int32 in cpu/gpu). If you want to further reduce the precision to int2, in cpu/gpu you also need to pack them into a byte-addressable data type (int8, int32) before the memory movement

Thank you for your reply!
Actually I am reproducing your great project, but sudo bash -c "$(wget -O - http://apt.llvm.org/llvm.sh)" mentioned in the readme does not download the corresponding version of LLVM , I tried to compile other versions of LLVM (including 10.01 and 11.0.0) and encountered a compilation error, because the llvm/Support/TargetRegistry.h file was missing, can you help me to complete the experiment successfully?


Create Pages “title”

Automatically generated by Netlify CMS

👷 Deploy Preview for sparkling-frangipane-896ca5 processing.

Name
Link

🔨 Latest commit
e20af46cd9e4893f961d16ec1f80b44805fca732

🔍 Latest deploy log
https://app.netlify.com/sites/sparkling-frangipane-896ca5/deploys/624a8b908f70250008c01006


New PWAs show Data must contain at least two columns

New PWAs show Data must contain at least two columns

fixed with #528


TC-DGGEN-1.1 Matter 1.3 TE#2

Please follow the below template to file issues/concerns/feature enhancement against Test Scripts:
Summary Title:
Assessment of PICS usage by the test script not identifying configuration issues
Description:
Test case does not seem to properly assess the PICS vs expectations during execution. What it looks to do is check that the DUT sends what is expected but it does not seem to be able to flag any inconsistencies between the PICS and the subject reports. Subsequently we are getting a PASS for somethings that should fail.
In this example, the uploader is stating no support for DGGEN.S.C01.Rsp or DGGEN.S.A0003, however both are in the lists when read back from the DUT.
Further to this, a mandatory events list item was not checked by the test script due to PICS_EVENT_LIST_ENABLED not been set. Expectation is that this should not need to be present.
Steps to reproduce:
Configure PICS and leave PICS_EVENT_LIST_ENABLED unset, discrepancies do not seem to be flagged.
Logs:
Not available, result was submitted in Matter 1.3 TE#2
Additional Info:
Please include any other additional or relevant information here. Example: Spec version, PICS version, Test plan version, PICS XML files used for testing PICS, Configuration of the project, related issues,  etc.

As with most 1.1 tests, this test does not properly verify the negative test cases. This is the reason we have added an explicit pics checker test.
1.1 tests are being slated for removal post TE2 in favour of the conformance checker and pics. Suggest low priority for this issue as the expectation is that this test case will be removed.

IMO this issue should be close. PICS validation is performed in IDM-10.4, this test, as with ALL tests in the test suite, assume the pics is correct. This test is slated for deletion in 1.4


Feature proposal: Live reload areas.json via API call

There should be a way to reload the areas file (whether its a static JSON or a Koji remote URL) without restarting the whole ReactMap backend.
The reload should be an API endpoint so that it can be triggered from Koji.

Resolved in #802


Configure venv

Hi, thanks for creating this super useful extension!
Is it possible to set up the python virtual environment at .venv instead of venv?

Hey,
I'm glad you found it useful.
Actually, as I remember, venv is the Charmcraft's preferred directory for virtual environment (for example, it's listed in .gitignore).
However, it's totally possible to keep the default behavior as is, but add some configuration parameter that allows overriding the virtual environment director
Actually, I hd this plan to provide some overrides via configuration, like this (in .vscode/settings.json):
{
    "charmcraft-ide.overrides": {
        "*": {
             // New default for all charms
        },
        "path/to/charm": {
             // Charm-specific overrides
        }
    }
}

When that is implemented, which will be soon, we can also add a parameter to override the default venv directory.
How's that sound?

An override for the default venv directory would be exactly what I was looking for!

@dashmage I just published the new version of the extension. It now supports some basic configurations and, of course, charm-specific overrides. You can find more details in the README.md (or the extension details page in VS Code).
In your case, if all charms in the workspace should use the same directory name (i.e., .venv) for the virtual environments, you can set it as a default in .vscode/settings.json, like this:
{
    "charmcraft-ide.defaultVirtualEnvDirectory": ".venv"
}

But, if you need to override the directory name just for a specific charm, then you'd need overrides:
{
   "charmcraft-ide.override": {
       "path/to/charm": {
            "virtualEnvDirectory": ".venv"
       }
   }
}

Hope this works for you. Thanks again for your good suggestion. :pray:
I close this issue now. If you found bugs in the new version, please comment here.


Remove two unused expected output files

They where part of the initial commit, but not changed since then.

@df7cb I would propose to merge this simple patch and #83. Later I propose to rebase and merge selectivity estimators PR.

Rebased both.


Add single sign on info to readme

@scytacki Is the language I used correct? Is it right to specify Google and Schoology as supported providers? Or could any OAuth provider be used without having to make significant changes to the code?

looks good to me. We would have to make changes to the code to support other OAuth providers. If we switched to OpenID (built ontop of OAuth), then we could probably support more providers out of the box.


CMake build/install dependency as a part of fips build?

Hi, I'm trying to write a fips wrapper for libwebsockets, but it's got a fairly involved CMake setup, and I'd rather not try to pull everything out into a custom fips CMakeFile.txt.  I would like to just have fips use CMake to build/install libwebsockets (install it locally, like into a subdirectory of fips-libwebsockets), and then I could use the output of that in other fips projects.  Is that possible?  Or am I approaching this the wrong way?  Any advice would be greatly appreciated!

Thanks, that worked!  Is there any way to get fips to add a subdirectory of the installed library as an include directory?  libwebsockets generates a lws_config.h that needs to be included for things to work.  For now, I just copied the generated one to include/libwebsockets: https://github.com/russpowers/fips-libwebsockets.

You can add a target_include_directories statement to your fips-wrapper CMakeLists.txt file with the directory where the config.h file is generated, something like this:
https://github.com/fips-libs/fips-soloud/blob/4cb8def50156a08e369d190f23acc23fbd297507/CMakeLists.txt#L119
You can experiment with the PUBLIC arg, if the header file is only included by libwebsockets implementation files, a PRIVATE would be better (see here: https://cmake.org/cmake/help/v3.0/command/target_include_directories.html).

I wasn't able to get target_include_directories working, CMake was complaining that I couldn't add them to another project (or, I suppose, a subproject?).  I did figure something out though, I added a fips-include.cmake: https://github.com/russpowers/fips-libwebsockets/blob/master/fips-include.cmake.  It seems to be working well, although I've only tested it on Windows builds.
Thanks again for your help!


 Example tweet is here: https://twitter.com/AussieParlEdits/status/1340830913412927489
I thought this might be solved by the change from phantom to puppeteer/chromium, but this bot is hosted on the Wikimedia instance, which I assume is running the latest version.
On a side note, I tried to take a look at the Wikimedia instance to make some ranges tweaks, but all the infra has changed since I last looked. From what I can tell, there's no running pod for anon on tools-dev.wmflabs.org or tools-login.wmflabs.org (according to anon status). Where do I need to go to edit the used config?

Hi @ineffyble. I'm sorry for the delay in responding. I ran into some trouble a few months ago running anon on the Wikimedia kubernetes infrastructure.
https://phabricator.wikimedia.org/T262734
This prompted me to start running the same configuration on my on VPS (just under a tmux session instead of Kubernetes). It seems to be running pretty well there, but I would have to do some work to make it available to you.
I'm trying again at Wikimedia to see if anything has changed recently.
Your point about the cutoff in the screenshots is well taken. You can see the same behavior in the congressedits bot running in the fediverse right? I think it may not be waiting long enough for the page to render?
It appears

Hi @ineffyble. I'm sorry for the delay in responding. I ran into some trouble a few months ago running anon on the Wikimedia kubernetes infrastructure.
https://phabricator.wikimedia.org/T262734
This prompted me to start running the same configuration on my on VPS (just under a tmux session instead of Kubernetes). It seems to be running pretty well there, but I would have to do some work to make it available to you.
I'm trying again at Wikimedia to see if anything has changed recently.
Your point about the cutoff in the screenshots is well taken. You can see the same behavior in the congressedits bot running in the fediverse right? I think it may not be waiting long enough for the page to render?
It appears

The saga continues:
https://phabricator.wikimedia.org/T271082
The bot at least appears to run now, but is missing a shared library when launching the headless browser....

The saga continues:
https://phabricator.wikimedia.org/T271082
The bot at least appears to run now, but is missing a shared library when launching the headless browser....

I'm still seeing it too. It's almost like it's not waiting long enough for the page to render before doing the screenshot?

I think that the problem is that the call to screenshot() is using the dimensions of the element containing the diff, but those dimensions often will exceed the height of the browser viewport (currently 768). I'm going to experiment with setting the viewport to a height that includes the full diff before the screenshot is taken.

I think this is finally fixed! e.g.
https://twitter.com/gccaedits/status/1440027946484338693

Thank you for being patient -- it took me an absurdly long time to fix this. Blame it on the A D D.


Release management

Hello, I'm relying on your awesome plugin for my vim based project http://webvim.org/,
I can see some tags and releases but they are pretty old.
Would you mind creating tags and releases when you hit some milestone, it's easier (from my point of view) to maintain plugins from tags/release than commit hashes.
Thank you very much. I really appreciate your plugin.

Just made 2.1 release


Update is_even.py

The absolute value of a number is not needed to check whether it is even.

Hi @Armaan-Goel-NTU,
Thank you for submitting this PR.
It seems I overlooked this very important point about the absolute value of a number not being required when checking whether it is even. I commend you for your attention to detail!
Warm Regards,
ardnep


Zephyr package modules

This is a complete Zephyr CMake package and CMake modules overhaul.
The intention with this major cleanup is to make the build system more modular in a way so that it can easier be reused through the Zephyr package mechanism.
Use-cases for this work is:

The sysbuild proposal: https://github.com/zephyrproject-rtos/zephyr/pull/40555
In #40555 it will remove the need for a boilerplate file like this: https://github.com/zephyrproject-rtos/zephyr/blob/641b86999750f1ed71fffda1da7c47387db22c23/sysbuild/cmake/boilerplate.cmake as this can now be handled as part of Zephyr package loading.
Possibility to run Zephyr CMake configure stages individually, for example dts or kconfig.
This will allow twister to generate the zephyr.dts tree and do filtering and not invoking a complete CMake configuration.
This opens for the use-case referenced by @mbolivar-nordic on Discord https://discord.com/channels/720317445772017664/733037738068148335/914952860192632852
See a PR for this work: #41302
Cleanup of find_package(ZephyrUnittest) as that can simply be loaded as specific module like: find_package(Zephyr COMPONENTS unittest)

With the improved Zephyr CMake package, it is now possible to do:
find_package(Zephyr) : Will load a standard build setup as always
find_package(Zephyr COMPONENTS unittest): Load specific unittest build setup
find_package(Zephyr COMPONENTS dts): Load only dts module (and its direct dependencies)
find_package(Zephyr COMPONENTS extensions west zephyr_module): Load specific modules(and their direct dependencies), for example sysbuild use-case/

Excellent initiative!

Thanks.

The only thing I would like to ask is that you fully qualify "Zephyr module" vs "CMake module" in every instance of each that could be prone to confusion with the other.

Good point, and done.
To everyone else, the reason for using the name CMake module here is because it's the official term used in CMake, ref:
https://cmake.org/cmake/help/latest/command/include.html#command:include

Load and run CMake code from a file or module.
include(<file|module> [OPTIONAL] [RESULT_VARIABLE <var>]
                      [NO_POLICY_SCOPE])

I'm currently reviewing. Please DNM until I have finished.

Codecov Report

Merging #41301 (7f5eb92) into main (fef2e30) will increase coverage by 0.16%.
The diff coverage is 61.11%.

:exclamation: Current head 7f5eb92 differs from pull request most recent head 1c57242. Consider uploading reports for the commit 1c57242 to get more accurate results

@@            Coverage Diff             @@
##             main   #41301      +/-   ##
==========================================
+ Coverage   51.24%   51.41%   +0.16%
==========================================
  Files         606      609       +3
  Lines       70896    71379     +483
  Branches    16335    16427      +92
==========================================
+ Hits        36334    36698     +364
- Misses      28572    28641      +69
- Partials     5990     6040      +50

Impacted Files
Coverage Δ

arch/x86/include/kernel_arch_func.h
100.00% <ø> (ø)

include/device.h
84.61% <ø> (ø)

include/drivers/can.h
100.00% <ø> (ø)

include/drivers/pinctrl.h
77.77% <ø> (ø)

include/logging/log.h
100.00% <ø> (ø)

include/logging/log_core.h
83.33% <ø> (ø)

include/net/ethernet.h
51.28% <ø> (ø)

subsys/bluetooth/controller/ll_sw/ull_conn.c
25.38% <0.00%> (-0.22%)
:arrow_down:

subsys/bluetooth/host/conn_internal.h
0.00% <ø> (ø)

kernel/userspace.c
87.50% <75.00%> (-0.27%)
:arrow_down:

... and 20 more

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 fef2e30...1c57242. Read the comment docs.

@sebo I considered a couple of you comments / recommendations in various places and decided to do an extra cleanup in this push: https://github.com/zephyrproject-rtos/zephyr/compare/af70e847808b0af1a92519e7f72ca65b54b903dd..387787f43be06eddeb11f30449502fb2af667c8d
Overview:

removed usage_boards / usage_shields and inlined the code instead.
Removed the possibility of calling those in script mode. (let's see if someone will request this feature to be re-added later)
It's good to get code cleaned up, now that we have the chance and if we need a usage feature for CMake modules later, let's make it a general feature following this outline in #41302.

Thanks for your suggestions.

@tejlmand I tested the latest update and this comment is unresolved:
https://github.com/zephyrproject-rtos/zephyr/pull/41301#discussion_r789768162

@tejlmand I tested the latest update and this comment is unresolved:

I know, had to be away.
Fixed now.

I am +1; I'm assuming here that this hasn't changed drastically since my last review.

it hasn't. I just waiting for merge gates to be opened again.

@mbolivar-nordic @SebastianBoe please re-approve, pure rebase.


3 ➡ 4

Forward-port ign-gazebo3 to ign-gazebo4.
Comparison:
https://github.com/ignitionrobotics/ign-gazebo/compare/ign-gazebo4...ign-gazebo3

Well, the PeerTracker test still looks flaky even with #452 :confused:
/Users/jenkins/workspace/ignition_gazebo-ci-pr_any-homebrew-amd64/ign-gazebo/src/network/PeerTracker_TEST.cc:177
Value of: stalePeers
  Actual: 2
Expected: 1


Input component has unremovable margin at the bottom

🐛 Bug Report
The Input component has some space at the bottom which I have not been able to remove by setting margin: 0

To Reproduce
Steps to reproduce the behavior:
// ...
<Layout>
  <Input style={{margin: 0}} />
</Layout>
//...

Expected behavior
I should see only the input element with no additional spacing
UI Kitten and Eva version

Package
Version

@eva-design/eva
1.4.0

@ui-kitten/components
4.4.1

Environment information
  System:
    OS: macOS 10.15.3
    CPU: (8) x64 Intel(R) Core(TM) i7-4850HQ CPU @ 2.30GHz
  Binaries:
    Node: 12.6.0 - /usr/local/bin/node
    Yarn: 1.17.3 - /usr/local/bin/yarn
    npm: 6.10.2 - /usr/local/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  SDKs:
    iOS SDK:
      Platforms: iOS 13.2, DriverKit 19.0, macOS 10.15, tvOS 13.2, watchOS 6.1
    Android SDK:
      API Levels: 21, 22, 23, 24, 25, 26, 27, 28, 29
      Build Tools: 28.0.3, 29.0.3
      System Images: android-25 | Google Play Intel x86 Atom, android-27 | Google Play Intel x86 Atom, android-29 | Intel x86 Atom, android-29 | Intel x86 Atom_64, android-29 | Google APIs Intel x86 Atom, android-29 | Google APIs Intel x86 Atom_64, android-29 | Google Play Intel x86 Atom, android-29 | Google Play Intel x86 Atom_64
  IDEs:
    Android Studio: 3.5 AI-191.8<IP_ADDRESS>91312
    Xcode: 11.3/11C29 - /usr/bin/xcodebuild
  npmPackages:
    react: 16.8.1 => 16.8.1
    react-native: ^0.61.5 => 0.61.5
  npmGlobalPackages:
    react-native-cli: 2.0.1

Thanks for reporting. I've got this fixed in the upcoming v5, so I'm going to close this.
As for workaround, you may use the negative margin marginBottom: -4

@artyorsh I'm running latest 5.0.0 version but margin:0/marginBottom:0 doesn't work and I still have to use your workaround. Any thoughts?


Add some fixes for the FunctionalTests and fix typing on a UIAccessibilityTextFieldElement

Noticing lots of failures with this PR: https://travis-ci.org/google/EarlGrey/builds/205961731
Please investigate.

Looks better now.

Not really. The gesture tests are consistently failing.

Good to go now.

@sid-github This is good to go. Cancelled the last CI run since the test was only a formatting one.


[YUNIKORN-2541] Upgrade golang.org/x/net for CVEs

What is this PR for?
This PR upgrades golang.org/x/net to 0.23.0 to address CVEs.
Additionally, it updates the versions of the scheduler-interface and core, as they also use this module and have been updated accordingly.
What type of PR is it?

[ ] - Bug Fix
[x] - Improvement
[ ] - Feature
[ ] - Documentation
[ ] - Hot Fix
[ ] - Refactoring

Todos
N/A
What is the Jira issue?
YUNIKORN-2541
How should this be tested?
It's tested by make test locally and all tests passed.
Screenshots (if appropriate)
N/A
Questions:
N/A

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

Project coverage is 66.13%. Comparing base (a59fd1c) to head (eab821d).

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #822      +/-   ##
==========================================
- Coverage   66.15%   66.13%   -0.03%
==========================================
  Files          69       69
  Lines        7517     7517
==========================================
- Hits         4973     4971       -2
- Misses       2333     2334       +1
- Partials      211      212       +1

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

The crypto dependency has been updated in both the SI and core projects, and k8shim has also been updated accordingly.


Group Buttons and Stacking Issue

Wouldn't it be nice to have something like ui stackable buttons? Currently, buttons do not display correctly on mobile devices. For example, please view this simple fiddle and resize your browser and the fiddle page frame.
http://jsfiddle.net/5dwxg5f3/1/

+1 too.
I referenced the .ui.stackable.menu, and I get stackable buttons as below:
@media only screen and (max-width: 767px) {
  .ui.stackable.buttons{
    display: -webkit-box;
    display: -webkit-flex;
    display: -ms-flexbox;
    display: flex;
    flex-direction: column;
    width: 100%;
  }
  .ui.stackable.buttons > .button{
    width: 100%;
  }
}

<div class="ui stackable two buttons">
     <button class="ui button">1</button>
     <button class="ui button">2</button>
</div>

I didn't test this very much, but hope it gives you some ideas.

@a8568730 Thank you, it works fine!


Fail stream on internal Google PubSub library error (CE2)

This addresses the issue described in #19.

In case of internal Google PubSub library error (i.e. io.grpc.StatusRuntimeException: NOT_FOUND) stream does not fail and it's impossible to track/handle/log the error.

The listener gets called only on unrecoverable failures in the underlying PubSub subscriber, so it makes sense to terminate the fs2 stream with a failure.

Hi @istreeter, sorry we've been really slow here! The change makes a lot of sense, thank you. What would you think about wrapping the raised error up into something like an InternalPubSubError just to make matching on it easier applications if people need to?
Once this change is merged would you need a release on cats-effect 2 or are you OK with the release only being available for CE3 ultimately?

I've opened my suggestion as a PR: https://github.com/istreeter/fs2-google-pubsub/pull/1. LMK what you think!

Hi @bastewart -- I just sat down to implement your suggested change.... and you had done it already!  I like it a lot, it's a good improvement.
It would be very helpful for me if you could do a release on cats-effect-2 branch, because the project I am working on is stuck on CE2 for a little while longer.
I would be interested to help you out by implementing the same thing on your CE3/master branch, but I have not done this yet.

Closed in favour of #282 as this can be backported to CE2 via a merge. As I mentioned in the other PR I have updated (so far locally) the CE2 branch to be up to date with master.
@istreeter I'll cut a CE2 release next week. I think we'll have to be a bit silly with version numbers (e.g. odd versions like 0.21.x are for CE3 and even versions like 0.22.y` are for CE3) but it should be able to work.
Internally we are looking to move away from CE2 ASAP; we're only blocked on a couple of other OSS projects which have milestone but no full CE3 release. As a result we'd want to drop support for CE2 in this repo at some point vaguely soon. If you happen to move over to CE3 and no-longer need CE2 let us know and we'd be more than happy to drop the CE2 branch!

Sorry for the total radio silence here @istreeter (and not releasing this). We had a service running in production on the release milestone and it was showing a memory leak so didn't want to do a full release (in case this library was at fault). I finally managed to track the rough cause (probably fs2) at the same time as someone else reported the issue to fs2, which has now been fixed: https://github.com/typelevel/fs2/issues/2598


Teddy kentor solution

completeness: 4
comfort: 4
more comfortable than previously assignments...jQuery has made it easier.

:+1: Nice job! Let me know if you have any questions


Feature request: Create PR template

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Need a template for PRs to make PR descriptions more streamlined
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Closed by #14


Update config.json

Codecov Report

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

@@            Coverage Diff            @@
##             master     #193   +/-   ##
=========================================
  Coverage     28.66%   28.66%
  Complexity      338      338
=========================================
  Files            46       46
  Lines          2990     2990
  Branches        379      379
=========================================
  Hits            857      857
  Misses         2049     2049
  Partials         84       84

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 eaa87b8...6afd9be. Read the comment docs.


npm run test hangs

Describe the bug
npm run test hangs after running HeadlessChrome tests
To Reproduce
Steps to reproduce the behavior:

Clone the repo
npm i
npm run test

Expected behavior
The test suite should run and exit when done.
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: macOS
Browser n/a
Version 3.9.0

@murtyjones Because, npm run test script runs karma with --no-single-run option.

If you want single run karma testing, exec npx karma start command in your project root directory.


Manual not formatting correctly

The markdown in the manual (docs/manual.md) isn't formatting correctly.  It used to, so I'm assuming the github formatter implementation changed to match the CommonMark spec.

Now fixed.


"directories_to_watch" documentation

@rastala @nibaccam Are there any documentation about "directories_to_watch". I looked around but couldn't find any.
Another question regarding ./outputs and ./logs. By default, they need to be on the root of the source_directory? It doesn't seem to work if they're in the same directory with the main entry script.

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

ID: 62a2a1c7-808f-e969-932c-9edc399071ee
Version Independent ID: 16f092af-03f0-852f-6b30-9b8267cdf1de
Content: Where to save & write experiment files - Azure Machine Learning
Content Source: articles/machine-learning/how-to-save-write-experiment-files.md
Service: machine-learning
Sub-service: core
GitHub Login: @rastala
Microsoft Alias: roastala

Does "directories_to_watch" apply to both "outputs" and "logs" folders?

#assign:@nibaccam

@rastala can you help here please?

Files in these folders are automatically uploaded to run history.

@usuyama does this answer your questions? If not, please provide additional clarity and @rastala  and myself (@nibaccam)

@usuyama
We will now proceed to close this thread. If there are further questions regarding this matter, please respond here and @YutongTie-MSFT and we will gladly continue the discussion.


Skip script analysis without PSSA installed.

Fixes the issue where the build fails if PSSA is not installed.
Also adds a warning/verify to the NewModule manifest for PSSA.

Aah, ok, good point. I always seem to default to doing things the hard way :P

That makes sense, I'll do that.
OK, yeah, I was in two minds about how it read, but more relevant info is better.

BTW this build script is really taking shape. Starting to feel pretty solid. Thanks for your efforts on this puppy!!

That should be good. I renamed the PSSA control options at the same time.
Yeah, it's definitely looking good. Thank you for your input!

Nice work Dave, this is great!


Fix OneWay binding warnings

Summary of the Pull Request
Fix those OneWay binding warnings by reverting to default behavior for the elements that don't have code updating it.

References

PR Checklist

[x] CLA signed. If not, go over here and sign the CLA
[x] Tests added/passed
[x] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

This is the most I could do for now. Sorry @carlos-zamora. Maybe you could help me find where those viewmodels are?

AddProfile page

the AddProfileViewModel doesn't exist. If you check AddProfile.h, you can see that we have a navigation state that holds any data we need on this page (i.e. CascadiaSettings Settings). This needs to be changed similar to how @PankajBhojwani changed the other pages (i.e. Launch page view model). So an AddProfileViewModel needs to be introduced to hold the state (list of profile view models), and bind to that instead of the navigation state. In fact, we should remove NavigationState entirely here. We should also ensure that the list of profile view models can be reused with Launch.xaml's (the other place you made the change to IconSource).

Launch page

the LaunchViewModel already exists, so this one is slightly easier. We don't need to worry about introducing another class. Instead, we just need to create that list of profile view models that can be shared with the AddProfile page/view model.
Honestly, I'd start here....

Start by creating and binding to the ProfileViewModel objects.
Then move that list to a shared space (probably MainPage?).
Then add the AddProfileViewModel class.
Then pass the list of profile view models into AddProfileViewModel
last, bind to the profile view models instead of the settings model profiles.

`Appearances.xaml

This one is pretty different. Here we're bound to the settings model color scheme. We actually want to bind to the view model color scheme introduced in https://github.com/microsoft/terminal/pull/13179. Similar idea, we want to store the color schemes list from the color schemes page in a shared location (probably main page) and reuse that.


Doc samples on how to update metadata based features on workbench instances.

Description
Fixes #
Note: If you are not associated with Google, open an issue for discussion before submitting a pull request.
Checklist
Readiness

[*] Yes, merge this PR after it is approved
[ ] No, don't merge this PR after it is approved

Style

[x] My sample follows the rules described for Terraform in the Effective Samples style guide
[x] My sample follows the other requirements and best practices in the Contributing
guide

Testing

[*] I have performed tests described in the Contributing guide:

[*] Tests pass: terraform apply
[*] Lint pass: terraform fmt check

Intended location

[*] Yes, this sample will be (or already is) included on cloud.google.com
Location(s):

[ ] No, this sample won't be included on cloud.google.com
Reason:

API enablement

[*] If the sample needs an API enabled to pass testing, I have added the service to the Test setup file

Review

[*] If this sample adds a new directory, I have added codeowners to the CODEOWNERS file

/gcbrun

snippet-bot:force-run

/gcbrun

/gcbrun

/gcbrun

requires review from @terraform-google-modules/dee-data-ai

/gcbrun

/gcbrun


env: node\r: No such file or directory

Describe the bug
After npm install and run "chatdown -v" it return error "env: node\r: No such file or directory"
To Reproduce
Steps to reproduce the behavior:

run command "npm install -g chatdown"
run command "chatdown -v"
err message shows "env: node\r: No such file or directory"

Expected behavior
Chatdown work as expect.
**Desktop

OS: macOS 10.13.3

I'm not able to repro on 10.13.5, but will try it on a clean macOS installation

The way we have our .gitattributes file configured we configure our line endings based on the OS checking out a given file.  so if we build a release on a Windows machine our js files will use CRLF, which cause the above failure on MacOS.


Fixes orchestrator x64 compilation and updates projects to use XUnit

Changes include:

Includes fixe made by @vishwacsena in #4434
Removed x64 configuration from solution.
Updated Orchestrator project to only target x64 and added TODO notes in the csproj so we remember to remove these settings once we support Any CPU.
Removed redundant settings and packages from Orchestrator.csproj
Migrated orchestrator tests to XUnit.

I tested an E2E sample in this branch and everything functions as expected. Thanks @gabog


when an item is dropped, it jumps instead of animate

To Reproduce
Steps to reproduce the behavior:

Open https://brainhubeu.github.io/react-carousel/docs/examples/defaultArrows/
Drag an item
Move the dragged item by a short distance so it's not replaced by another item
Drop the item
The item immediately goes back to its default position.

Expected behavior
5) The item should be animated until it goes back to its default position
Vidoes
https://recordit.co/yLABIaPOxo
Desktop (please complete the following information):

OS: Mac OS X 10.14.6
Browser: Chrome 80.0.3987.100
Version: 1.10.64

I close as I see it's now fixed in https://brainhubeu.github.io/


TextField is slow with long text with CanvasKit

Details
With CanvasKit, setting a TextEditingController.text to a string with 270k characters takes 90 seconds including the render.
With HTML renderer, it is takes less than a second although still noticeably slow.
This test app progressively sets the text to 1000, 2000, 3000, 4000, 5000, 6000, 7000, 8000, 9000, and 10000 lines (each 27 characters long) and measures the time before interactive again after each change.
With different renderers, it shows the following time in seconds (ran with --release):
Lines	1000	2000	3000	4000	5000	6000	7000	8000	9000	10000
HTML	0.111	0.154	0.188	0.211	0.252	0.394	0.447	0.423	0.44	0.491
Canvas	0.806	2.448	5.308	9.267	23.366	32.874	46.939	59.016	73.45	90.266

To produce this series, click "1-10k" button.
To measure a single change, click any of the other buttons.
In this video, the text field is set to 3000 lines, which takes almost 6 seconds:
https://github.com/flutter/flutter/assets/44893228/9a5b5dc3-8d9e-451c-b7bb-50579f4067eb
Subsequent edits of such long texts are also slow.
Nearly identical results are produced with expands: true and without it (wrapping in a SingleChildScrollView).

main.dart
import 'dart:js' as js;
import 'package:flutter/material.dart';

const maxKLines = 10;
const text = 'abcdefghijklmnopqrstuvwxyz\n';
final texts = [
  for (int i = 0; i <= maxKLines; i++) text * i * 1000,
];

final controller = TextEditingController();
final renderer = js.context['flutterCanvasKit'] == null ? 'HTML' : 'Canvas';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      debugShowCheckedModeBanner: false,
      home: Scaffold(
        appBar: AppBar(
          actions: [
            for (int i = 0; i <= maxKLines; i++)
              ElevatedButton(
                child: Text('${i}k'),
                onPressed: () => _setText(i),
              ),
            ElevatedButton(
              child: Text('1-${maxKLines}k'),
              onPressed: _setTexts,
            ),
          ],
        ),
        body: TextField(
          controller: controller,
          maxLines: null,
          expands: true,
        ),
      ),
    );
  }

  Future<Duration> _setText(int klines) async {
    final sw = Stopwatch();
    sw.start();

    controller.text = texts[klines];

    // Give up the control to render.
    await Future.delayed(const Duration(milliseconds: 100));

    final result = sw.elapsed;
    print('Set ${klines}k lines in ${result}');
    return result;
  }

  Future<void> _setTexts() async {
    final durations = <Duration>[];

    for (int i = 1; i <= maxKLines; i++) {
      durations.add(await _setText(i));
    }

    print('$renderer\t' + durations.map((d) => d.inMilliseconds / 1000).join('\t'));
  }
}

Target Platform: Web
Target OS version/browser: Ubuntu, Chrome 114
Logs

flutter doctor -v
[✓] Flutter (Channel master, 3.12.0-3.0.pre.35, on Ubuntu 20.04.5 LTS 5.14.0-1059-oem, locale en_US.UTF-8)
    • Flutter version 3.12.0-3.0.pre.35 on channel master at /home/alexey/snap/flutter/common/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 66cda5917d (2 hours ago), 2023-06-09 04:28:18 -0700
    • Engine revision cb93477008
    • Dart version 3.1.0 (build 3.1.0-180.0.dev)
    • DevTools version 2.24.0

[!] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /home/alexey/Android/Sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/linux#android-setup for more details.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Linux toolchain - develop for Linux desktop
    • clang version 10.0.0-4ubuntu1
    • cmake version 3.16.3
    • ninja version 1.10.0
    • pkg-config version 0.29.1

[✓] Android Studio (version 2022.1)
    • Android Studio at /home/alexey/bin/android-studio
    • Flutter plugin version 72.1.2
    • Dart plugin version 221.6096
    • Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-8887301)

[✓] VS Code (version 1.75.1)
    • VS Code at /usr/share/code
    • Flutter extension can be installed from:
      🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-x64      • Ubuntu 20.04.5 LTS 5.14.0-1059-oem
    • Chrome (web)    • chrome • web-javascript • Google Chrome 114.0.5735.90

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

Thanks for the detailed report. I was able to replicate the reported behavior in release and profile modes using canvaskit. With html renderer, it works as expected.

 stable, master flutter doctor -v
[!] Flutter (Channel stable, 3.10.3, on macOS 12.2.1 21D62 darwin-x64, locale
    en-GB)
    • Flutter version 3.10.3 on channel stable at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision f92f44110e (4 days ago), 2023-06-01 18:17:33 -0500
    • Engine revision 2a3401c9bb
    • Dart version 3.0.3
    • DevTools version 2.23.1
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

[!] Flutter (Channel master, 3.12.0-3.0.pre.66, on macOS 12.2.1 21D62
    darwin-x64, locale en-GB)
    • Flutter version 3.12.0-3.0.pre.66 on channel master at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 2b1f882e20 (5 hours ago), 2023-06-11 19:40:28 -0400
    • Engine revision 9bb0a59074
    • Dart version 3.1.0 (build 3.1.0-196.0.dev)
    • DevTools version 2.24.0
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/dhs/Library/Android/sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for
      more details.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 13C100
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] IntelliJ IDEA Ultimate Edition (version 2021.3.2)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 65.1.4
    • Dart plugin version 213.7228

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.29.0

[✓] Connected device (3 available)
    • Darshan's iphone (mobile) • 21150b119064aecc249dfcfe05e259197461ce23 • ios
      • iOS 15.3.1 19D52
    • macOS (desktop)           • macos                                    •
      darwin-x64     • macOS 12.2.1 21D62 darwin-x64
    • Chrome (web)              • chrome                                   •
      web-javascript • Google Chrome 109.0.5414.119

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

This is only slow with CanvasKit Chrome. Using Firefox, for example, the issue disappears.

It looks like the culprit might be Intl.Segmenter. Running 3k lines through it takes roughly 1.4 seconds on my current setup.
console.time();

const lines = new Array(3000).fill('abcdefghijklmnopqrstuvwxyz').join('\n');

const segmenter = new Intl.Segmenter(undefined, { granularity: 'word' }).segment(lines);

for (const segment of segmenter) {}

console.timeEnd();

This is by no means an accurate (or reproducible) form of measurement, but the magnitude of it is what really matters.

Can we have a priority label on this?

I've been looking into whether native text inputs helps resolve this (https://github.com/flutter/flutter/issues/120613)--shifting the text rendering to the browser entirely-- but the POC showed the same sluggishness.  @mdebbar do you have any insight here? Seems to only affect Canvaskit on Chrome

If the culprit is Intl.Segmenter, maybe we should open a Chrome issue and track from here?


Closing leaderboard crashes

django      | Closing leaderboard zxcv
django      | <IP_ADDRESS> - - [15/Apr/2021 15:57:44] "GET /app/leaderboard/28/close_if_ended HTTP/1.1" 500 -
django      | Traceback (most recent call last):
django      |   File "/usr/local/lib/python3.8/site-packages/django/core/handlers/wsgi.py", line 133, in __call__
django      |     response = self.get_response(request)
django      |   File "/usr/local/lib/python3.8/site-packages/django/core/handlers/base.py", line 130, in get_response
django      |     response = self._middleware_chain(request)
django      |   File "/usr/local/lib/python3.8/site-packages/django/core/handlers/exception.py", line 49, in inner
django      |     response = response_for_exception(request, exc)
django      |   File "/usr/local/lib/python3.8/site-packages/django/core/handlers/exception.py", line 103, in response_for_exception
django      |     response = handle_uncaught_exception(request, get_resolver(get_urlconf()), sys.exc_info())
django      |   File "/usr/local/lib/python3.8/site-packages/django/core/handlers/exception.py", line 138, in handle_uncaught_exception
django      |     return debug.technical_500_response(request, *exc_info)
django      |   File "/usr/local/lib/python3.8/site-packages/django_extensions/management/technical_response.py", line 40, in null_technical_500_response
django      |     raise exc_value.with_traceback(tb)
django      |   File "/usr/local/lib/python3.8/site-packages/django/core/handlers/exception.py", line 47, in inner
django      |     response = get_response(request)
django      |   File "/usr/local/lib/python3.8/site-packages/django/core/handlers/base.py", line 181, in _get_response
django      |     response = wrapped_callback(request, *callback_args, **callback_kwargs)
django      |   File "/usr/local/lib/python3.8/contextlib.py", line 75, in inner
django      |     return func(*args, **kwds)
django      |   File "/usr/local/lib/python3.8/site-packages/django/views/generic/base.py", line 70, in view
django      |     return self.dispatch(request, *args, **kwargs)
django      |   File "/usr/local/lib/python3.8/site-packages/django/utils/decorators.py", line 43, in _wrapper
django      |     return bound_method(*args, **kwargs)
django      |   File "/usr/local/lib/python3.8/site-packages/django/contrib/auth/decorators.py", line 21, in _wrapped_view
django      |     return view_func(request, *args, **kwargs)
django      |   File "/usr/local/lib/python3.8/site-packages/django/views/generic/base.py", line 98, in dispatch
django      |     return handler(request, *args, **kwargs)
django      |   File "/app/github_leaderboard/app/views.py", line 61, in get
django      |     leaderboard.close_if_ended()
django      |   File "/app/github_leaderboard/app/models.py", line 92, in close_if_ended
django      |     self.refresh()  # fetch latest commit data from repo before closing
django      | AttributeError: 'Leaderboard' object has no attribute 'refresh'

I removed that piece of code and currently the piece of code doesn't do anything. Look at app's model.py close_if_ended.py function.


Added EcoBytes struct

Add EcoBytes, which is the same as EcoString but holds a raw byte vector rather than a UTF-8 string; this is implemented by making DynamicVec public.

Codecov Report
Patch coverage: 74.42% and project coverage change: -6.06 :warning:

Comparison is base (c3fd036) 92.21% compared to head (55fd58b) 86.16%.

:exclamation: Current head 55fd58b differs from pull request most recent head a4fea81. Consider uploading reports for the commit a4fea81 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #23      +/-   ##
==========================================
- Coverage   92.21%   86.16%   -6.06%
==========================================
  Files           4        4
  Lines        1041     1200     +159
==========================================
+ Hits          960     1034      +74
- Misses         81      166      +85

Impacted Files
Coverage Δ

src/lib.rs
100.00% <ø> (ø)

src/string.rs
83.58% <46.15%> (-3.58%)
:arrow_down:

src/bytes.rs
75.52% <75.52%> (ø)

... and 1 file with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

@reknih any updates on whether I can expect this to be merged? Thanks!

Could you also adjust the README and crate-level docs and briefly show EcoBytes?

Closing this due to inactivity. Feel free to reopen it if you want to continue working on this.


license

should we throw a license file in this repo?

That sounds good.. MIT? GPL?

i vote MIT

MIT sounds great to me
On Thu, Oct 1, 2015 at 1:12 PM, Michael Kirschner<EMAIL_ADDRESS>wrote:

i vote MIT
—
Reply to this email directly or view it on GitHub
https://github.com/oderby/VVD/issues/10#issuecomment-144789339.

Sorry, github never notified me of this. MIT is fine by me.


GraphQLMiddleware - error when processing queries with an empty request body

When I send query {{domain}}/graphql?query={whatever} using GET http method to GraphQLite server I get the following error

InvalidArgumentException: Syntax error in body: "" in file /home/dockeruser/app/vendor/thecodingmachine/graphqlite/src/Http/WebonyxGraphqlMiddleware.php on line 87
Stack trace:
  1. InvalidArgumentException->() /home/dockeruser/app/vendor/thecodingmachine/graphqlite/src/Http/WebonyxGraphqlMiddleware.php:87
  2. TheCodingMachine\GraphQLite\Http\WebonyxGraphqlMiddleware->process() /home/dockeruser/app/src/Core/MiddlewareStack.php:54
  3. App\Core\MiddlewareStack->handle() /home/dockeruser/app/vendor/tuupola/cors-middleware/src/CorsMiddleware.php:124
  4. Tuupola\Middleware\CorsMiddleware->process() /home/dockeruser/app/src/Core/MiddlewareStack.php:54
  5. App\Core\MiddlewareStack->handle() /home/dockeruser/app/src/Core/Kernel.php:69
  6. App\Core\Kernel->processRequest() /home/dockeruser/app/src/Core/Kernel.php:42
  7. App\Core\Kernel->__construct() /home/dockeruser/app/vendor/php-di/php-di/src/Definition/Resolver/ObjectCreator.php:143
  8. DI\Definition\Resolver\ObjectCreator->createInstance() /home/dockeruser/app/vendor/php-di/php-di/src/Definition/Resolver/ObjectCreator.php:71
  9. DI\Definition\Resolver\ObjectCreator->resolve() /home/dockeruser/app/vendor/php-di/php-di/src/Definition/Resolver/ResolverDispatcher.php:71
 10. DI\Definition\Resolver\ResolverDispatcher->resolve() /home/dockeruser/app/vendor/php-di/php-di/src/Container.php:390
 11. DI\Container->resolveDefinition() /home/dockeruser/app/vendor/php-di/php-di/src/Container.php:199
 12. DI\Container->make() /home/dockeruser/app/src/bootstrap.php:42
 13. require() /home/dockeruser/app/public/index.php:3

However, when I add a body to the request, it works

It seems to work as it should after I added condition $request->getBody()->getContents() to check if request body is not empty before it is decoded from json
https://github.com/thecodingmachine/graphqlite/blob/8a081f63112768a82eb9a6533ebacdd27090bb36/src/Http/WebonyxGraphqlMiddleware.php#L81

I'm not exactly certain what issue you're running into here, but there are a couple of things that should be pointed out:

GraphQL requests should be over HTTP POST and not GET.
When getting the contents from the body, the body is cleared out from the request object IIRC.  I find this behavior strange, personally.  However, I'm sure there is a valid reason for that design decision.  Regardless, that's something to be aware of.  I'd assume from your example that there isn't any body contents being parsed and therefore no resulting error.

Please try dumping the full contents of the body and inspect what's being passed before the exception is being thrown.

@oojacoboo

According to https://graphql.org/learn/serving-over-http/#get-request it is possible to send GraphQL requests using GET method.

Problem is that when I send request using GET method with an empty body, InvalidArgumentException (line 86) is thrown. Variable $content contains an empty string, variable $data contains null and json_last_error() returns 4, which translates to syntax error.

https://github.com/thecodingmachine/graphqlite/blob/8a081f63112768a82eb9a6533ebacdd27090bb36/src/Http/WebonyxGraphqlMiddleware.php#L81-L90
It is same as if you did this
json_decode("");
var_dump(json_last_error()); // returns 4
var_dump(json_last_error_msg()); // returns Syntax error

@ironexdev Sorry, I overlooked that you were including the actual query in the request string.  I wasn't aware that was supported as part of the spec.  In general, GraphQL APIs are sent over POST requests with a body.  We don't use the lib with GET requests, and I'm assuming this is the same for many others.  So, it is possible this isn't being properly supported ATM.
When you added that check for the body content, does the query get executed properly with actual query string parameters (types and fields)?  AFAIK, the body content is required to process the request.

@oojacoboo

When you added that check for the body content, does the query get executed properly with actual query string parameters (types and fields)?

Yes, for example this query works as expected
{{domain}}/graphql?query={authenticatedUser{authenticated,securelyAuthenticated,user{email}}}
I think it's because query parameters are parsed when the request gets from WebonyxGraphqlMiddleware to GraphQL\Server\Helper->parsePsrRequest which does the following in order to parse parameters from the query parse_str(html_entity_decode($request->getUri()->getQuery()), $queryParams). And it also ignores request body if request method is GET.
Flow

WebonyxGraphqlMiddleware->process

https://github.com/thecodingmachine/graphqlite/blob/8a081f63112768a82eb9a6533ebacdd27090bb36/src/Http/WebonyxGraphqlMiddleware.php#L74-L100

GraphQL\Server\StandardServer->executePsrRequest

https://github.com/webonyx/graphql-php/blob/c13e5df784015f773e46b0afa518320f2ed3071b/src/Server/StandardServer.php#L165-L170

GraphQL\Server\Helper->parsePsrRequest

https://github.com/webonyx/graphql-php/blob/c13e5df784015f773e46b0afa518320f2ed3071b/src/Server/Helper.php#L505-L540

@ironexdev Can you submit a PR for this with tests?
We probably just want to add a request method check, if the query string is being properly parsed already.
if ($request->getMethod() === 'POST' && empty($request->getParsedBody())) {

I looked briefly for some HTTP GET style API requests in the tests, but didn't see anything.  I think a couple tests in tests/Integration/EndToEndTest.php should be good.

@ironexdev did this get resolved?


Configuration options?

I am seeing this error in my App Engine project:
(Locals and arguments are only displayed for the top `config.capture.maxExpandFrames` stack frames.

Is this expected? How do I configure these options?

For now you can look at the instructions here: https://github.com/GoogleCloudPlatform/cloud-debug-nodejs/pull/164. We are working on a better solution however wherein we will accept configuration at module start.

With the latest release, configuration can be provided when the agent is started (https://github.com/GoogleCloudPlatform/cloud-debug-nodejs/pull/167). Configuration is covered in our readme here: https://github.com/GoogleCloudPlatform/cloud-debug-nodejs#configuration. I'm going to close this for now but feel free to reopen if you have additional questions.